Jdlrobson has uploaded a new change for review.
https://gerrit.wikimedia.org/r/59171
Change subject: Address nearby FIXME using colimit
......................................................................
Address nearby FIXME using colimit
just removing if statement and adding colimit parameter
Change-Id: Iadf9db7d6cbf40e5703ebed45f7cabc1487724a6
---
M javascripts/specials/nearby.js
1 file changed, 13 insertions(+), 14 deletions(-)
git pull ssh://gerrit.wikimedia.org:29418/mediawiki/extensions/MobileFrontend
refs/changes/71/59171/1
diff --git a/javascripts/specials/nearby.js b/javascripts/specials/nearby.js
index ccb61be..2fb678d 100644
--- a/javascripts/specials/nearby.js
+++ b/javascripts/specials/nearby.js
@@ -66,21 +66,19 @@
pages = $.map( pages, function( page ) {
var coords, lngLat, thumb;
- if ( page.coordinates ) { // FIXME: protecting us
against an api bug 47133
- if ( page.thumbnail ) {
- thumb = page.thumbnail;
- page.pageimage = thumb.source;
- page.pageimageClass = thumb.width >
thumb.height ? 'listThumbH' : 'listThumbV';
- }
- page.url = M.history.getArticleUrl( page.title
);
-
- coords = page.coordinates[0],
- lngLat = { latitude: coords.lat, longitude:
coords.lon };
- page.dist = calculateDistance( curLocation,
lngLat );
- page.proximity = distanceMessage( page.dist );
- pages.push( page );
- return page;
+ if ( page.thumbnail ) {
+ thumb = page.thumbnail;
+ page.pageimage = thumb.source;
+ page.pageimageClass = thumb.width >
thumb.height ? 'listThumbH' : 'listThumbV';
}
+ page.url = M.history.getArticleUrl( page.title );
+
+ coords = page.coordinates[0],
+ lngLat = { latitude: coords.lat, longitude: coords.lon
};
+ page.dist = calculateDistance( curLocation, lngLat );
+ page.proximity = distanceMessage( page.dist );
+ pages.push( page );
+ return page;
} );
pages.sort( function( a, b ) {
return a.dist > b.dist ? 1 : -1;
@@ -101,6 +99,7 @@
url: endpoint || M.getApiUrl(),
data: {
action: 'query',
+ colimit: 'max',
prop: 'pageimages|coordinates',
pithumbsize: 180,
pilimit: limit,
--
To view, visit https://gerrit.wikimedia.org/r/59171
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: newchange
Gerrit-Change-Id: Iadf9db7d6cbf40e5703ebed45f7cabc1487724a6
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/extensions/MobileFrontend
Gerrit-Branch: master
Gerrit-Owner: Jdlrobson <[email protected]>
_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
