jenkins-bot has submitted this change and it was merged.
Change subject: Always output even default globe
......................................................................
Always output even default globe
Per IRC, doesn't look logical
Change-Id: I28baeca7dd13c013e55254c4989bee21a721ccc5
---
M api/ApiQueryCoordinates.php
1 file changed, 1 insertion(+), 4 deletions(-)
Approvals:
Legoktm: Looks good to me, but someone else must approve
awjrichards: Verified; Looks good to me, approved
jenkins-bot: Verified
diff --git a/api/ApiQueryCoordinates.php b/api/ApiQueryCoordinates.php
index 7f8da15..807befd 100644
--- a/api/ApiQueryCoordinates.php
+++ b/api/ApiQueryCoordinates.php
@@ -61,10 +61,7 @@
foreach( $params['prop'] as $prop ) {
if ( isset( Coord::$fieldMapping[$prop] ) &&
isset( $row->{Coord::$fieldMapping[$prop]} ) ) {
$field = Coord::$fieldMapping[$prop];
- // Don't output default globe
- if ( !( $prop === 'globe' &&
$row->$field === $wgDefaultGlobe ) ) {
- $vals[$prop] = $row->$field;
- }
+ $vals[$prop] = $row->$field;
}
}
$fit = $this->addPageSubItem( $row->gt_page_id, $vals );
--
To view, visit https://gerrit.wikimedia.org/r/67886
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I28baeca7dd13c013e55254c4989bee21a721ccc5
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/extensions/GeoData
Gerrit-Branch: master
Gerrit-Owner: MaxSem <[email protected]>
Gerrit-Reviewer: Legoktm <[email protected]>
Gerrit-Reviewer: awjrichards <[email protected]>
Gerrit-Reviewer: jenkins-bot
_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
