Last modified: 2012-08-03 18:19:38 UTC

Wikimedia Bugzilla is closed!

Wikimedia migrated from Bugzilla to Phabricator. Bug reports are handled in Wikimedia Phabricator.
This static website is read-only and for historical purposes. It is not possible to log in and except for displaying bug reports and their history, links might be broken. See T40983, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 38983 - Use my current location hangs on areas with few/no monuments
Use my current location hangs on areas with few/no monuments
Status: RESOLVED FIXED
Product: WikiLoves Monuments Mobile
Classification: Unclassified
Maps (Other open bugs)
unspecified
All All
: Unprioritized normal
: ---
Assigned To: Nobody - You can work on this!
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2012-08-02 17:46 UTC by Jon
Modified: 2012-08-03 18:19 UTC (History)
2 users (show)

See Also:
Web browser: ---
Mobile Platform: ---
Assignee Huggle Beta Tester: ---


Attachments

Description Jon 2012-08-02 17:46:00 UTC
When clicking on use my current location in a location where there are few or no monuments (for example Sydney) the map is never loaded. This is due to the fix for bug 38741.

Essentially what happens is the map continues to zoom out from zoom level 18 until it finds monuments. When it gets to zoom <11 the api fails to respond possibly due to a throttling constraint the server imposes (as the client at this point would have made about 7 requests).

Possibly we want to give up looking at zoom level 10 to prevent this but I suspect the server needs to do something more useful such as refuse requests when such throttling occurs.

You can demonstrate this by running
WLMMobile.app.showMonumentsForPosition(-33.859972, 151.211111)
in your console (you will need to expose showMonumentsForPosition in app.js
Comment 1 Max Semenik 2012-08-02 18:25:19 UTC
Querying an arbitrarily large area is not supported. Because the API installation on TS is also used for stuff other than app, search area restriction is not enforced. However, the standalone API will simply return an error for areas larger than 0.2 x 0.2 degrees (discussable) - the app should enforce this restriction internally.

Note You need to log in before you can comment on or make changes to this bug.


Navigation
Links