Page MenuHomePhabricator

ImageFetcher doesn't fetch images for all monuments in list
Closed, ResolvedPublic

Description

ImageFetcher.prototype.send is constructing queries to request images of certain monuments however it is getting incomplete results due to exceeding the limit of titles (Too many values supplied for parameter 'titles': the limit is 50)

Here is an example url that we have been sending to the api:

https://commons.wikimedia.org/w/api.php?action=query&titles=File%3ASchw%C3%A4nberg+altes+Rathaus+aussen+von+S%C3%BCdwesten+nahe.JPG%7CFile%3AZug+Archiv+des+Amtes+f%C3%BCr+Denkmalpflege+und+Arch%C3%A4ologie%2C+Archiv+f%C3%BCr+Bauernhausforschung%2C+Museum+f%C3%BCr+Urgeschichte+1.JPG%7CFile%3ASilenen-Bauernhaus-49.jpg%7CFile%3ABauernhaus+Ins+(2).jpg%7CFile%3APohlern+Rohrmoos.jpg%7CFile%3ALangnau+Untere+D%C3%BCrsr%C3%BCti.jpg%7CFile%3ALenzburg+burghalde.jpg%7CFile%3ALangnau+Ch%C3%BCechlihus+mit+Dorfbrunnen.jpg%7CFile%3ALuetzelflueh+Kulturmuehle+4.jpg%7CFile%3ASilenen-Sust.jpg%7CFile%3AGuemmenen+Viadukt+mit+TGV+01+09.jpg%7CFile%3ASBB+Aarebruecke+Koblenz+(AG)+05+09.jpg%7CFile%3AZweibruggen+Eisenbr%C3%BCcke+West+2.JPG%7CFile%3ABaumallee+in+der+Elfenau.JPG%7CFile%3AOberdiessbach+Familienkapelle-1.jpg%7CFile%3ALa+Neuveville+Fontaines+des+Bannerets2.jpg%7CFile%3ALigerz+Gaberelhaus+frontal.jpg%7CFile%3AGarten+Schmidlin+2.jpg%7CFile%3AGarten+der+Villa+Bomonti+DSC05862.jpg%7CFile%3AReichenbach+BE%2C+Gasthaus+B%C3%A4ren+(1542).jpg%7CFile%3ABern-Zunfthaus-zu-Pfistern.jpg%7CFile%3AGoldhubel+Aegerten+Sud.jpg%7CFile%3AZofingen-Heiterenplatz.jpg%7CFile%3ALauperswil+Chalchmatt2.jpg%7CFile%3ALa+Neuveville+Hof+Ligerz.jpg%7CFile%3AGuemligen+Hofgut+Front.jpg%7CFile%3AInstitutsgeb%C3%A4ude+Hofwil+M%C3%BCnchenbuchsee1.jpg%7CFile%3AMh+JB+kleine+scheidegg.jpeg%7CFile%3AEndingen+J%C3%BCdischer+Friedhof.jpg%7CFile%3AMoutier+Kapelle+von+Chali%C3%A8res+mit+Friedhof.jpg%7CFile%3AFrick-Kath-Kirche.jpg%7CFile%3ADietwil-Preghejo+250.jpg%7CFile%3AFischbach-Goeslikon+peghejo+190.jpg%7CFile%3ALangenthal-Kaufhaus.jpg%7CFile%3ALauperswil+Kirche2.jpg%7CFile%3ALigerz+Kirche3.jpg%7CFile%3APZMTotale05557.jpg%7CFile%3AOberdiessbach+Diessenhof-2.jpg%7CFile%3AMontenach+Country+Estate+Balliswil+Feb+2011.jpg%7CFile%3AReichenbach+BE%2C+Letzi+M%C3%BClenen.jpg%7CFile%3AKGS-Wimmis-Letzi-(1).JPG%7CFile%3ALa+Neuveville+Bernerhaus.jpg%7CFile%3AMaison-Des-Dragons-La-Neuveville.jpg%7CFile%3AMeienberg+Stadtgraben.jpg%7CFile%3ASilenen-Meierturm-2.jpg%7CFile%3ALangenthal-Muehle.jpg%7CFile%3ABois+murat+(9).jpg%7CFile%3ALotzwil+Pfarrhaus2.jpg%7CFile%3ALuetzelflueh+Pfarrhaus.jpg%7CFile%3APfarrhaus00714.jpg%7CFile%3ALa+Neuveville+Rathaus.jpg%7CFile%3AKirche+Langnau+im+Emmental.jpg%7CFile%3AKirche+und+Friedhof+Limpach.JPG%7CFile%3AGr%C3%A4nichen+Kirche+innen.jpg%7CFile%3A6030+-+Meiringen+-+Evangelisch-Reformierten+Kirche.JPG%7CFile%3ABlanche-Eglise-La-Neuveville.jpg%7CFile%3AKGS+Densbueren-Rne-u-Hochwacht-Urgiz-(20).JPG%7CFile%3AGuemligen+Schloss+mit+Park.jpg%7CFile%3ASchloss+Hofwil+M%C3%BCnchenbuchsee1.jpg%7CFile%3ASchloss+Laupen+S%C3%BCdansicht.jpg%7CFile%3AM%C3%BCnchenwiler_Castle_Apr_2011.jpg%7CFile%3ASchloss+Nidau+Schlosspark.jpg%7CFile%3ASchlossOberhofen+6376.jpg%7CFile%3AAltenburg+Schloesschen.jpg%7CFile%3ABasel-Stadtmauer.jpg%7CFile%3APicswiss+ZG-04-01.jpg%7CFile%3AZug+Stallscheune+Nr.+424d+7.jpg%7CFile%3ACham+Stallscheune+Nr.+110b+2.jpg%7CFile%3AZug+Stallscheune+R%C3%BCschenhof+Nr.+424e+3.jpg%7CFile%3AFuniculaire+Fribourg+119.JPG%7CFile%3AEndingen+Synagoge.jpg%7CFile%3ABaden+Synagoge+Winter.jpg%7CFile%3AUntervogthaus+Gr%C3%A4nichen.jpg%7CFile%3AHolzbruecke+Urnaesch+St.+Gallen+01+09.jpg%7CFile%3AWichterheergut.jpeg%7CFile%3AHofstatt_im_D%C3%B6rfli_Wolfenschiessen_Jun_2012.jpg%7CFile%3ARinglihaus.jpg%7CFile%3ASilenen-Burghofstatt.jpg%7CFile%3ABaar+Wohnhaus+Nr.+196a+Sennweid+6.jpg%7CFile%3AZug+Wohnhaus+Nr.+354a+Oterswil+2.jpg%7CFile%3ACham+Wohnhaus+Nr.+49a+1.jpg%7CFile%3ABasel+032.jpg&prop=imageinfo&iiprop=url&format=json&iiurlwidth=64&iiurlheight=64


Version: unspecified
Severity: normal

Details

Reference
bz39182

Event Timeline

bzimport raised the priority of this task from to Needs Triage.Nov 22 2014, 1:06 AM
bzimport set Reference to bz39182.
bzimport added a subscriber: Unknown Object (MLST).

Limit of list is now 50 per above so that should match.