Aude has uploaded a new change for review.

  https://gerrit.wikimedia.org/r/84140


Change subject: allow longitude coordinates up to 360 in JS globe coordinate
......................................................................

allow longitude coordinates up to 360 in JS globe coordinate

- up to 360 is required for Mars coordinates to render

Change-Id: Ifaaa4e3144971278c2faa4adfffbcfed8b935784
---
M DataValues/resources/globeCoordinate.js/src/globeCoordinate.GlobeCoordinate.js
M 
DataValues/resources/globeCoordinate.js/tests/globeCoordinate.GlobeCoordinate.tests.js
2 files changed, 2 insertions(+), 1 deletion(-)


  git pull ssh://gerrit.wikimedia.org:29418/mediawiki/extensions/DataValues 
refs/changes/40/84140/1

diff --git 
a/DataValues/resources/globeCoordinate.js/src/globeCoordinate.GlobeCoordinate.js
 
b/DataValues/resources/globeCoordinate.js/src/globeCoordinate.GlobeCoordinate.js
index ab6f358..1a432bb 100644
--- 
a/DataValues/resources/globeCoordinate.js/src/globeCoordinate.GlobeCoordinate.js
+++ 
b/DataValues/resources/globeCoordinate.js/src/globeCoordinate.GlobeCoordinate.js
@@ -47,7 +47,7 @@
                if( Math.abs( this._latitude ) > 90 ) {
                        throw new Error( 'Latitude (' + this._latitude + ') is 
out of bounds' );
                }
-               if( Math.abs( this._longitude ) > 180 ) {
+               if( Math.abs( this._longitude ) > 360 ) {
                        throw new Error( 'Longitude (' + this._longitude + ') 
is out of bounds' );
                }
 
diff --git 
a/DataValues/resources/globeCoordinate.js/tests/globeCoordinate.GlobeCoordinate.tests.js
 
b/DataValues/resources/globeCoordinate.js/tests/globeCoordinate.GlobeCoordinate.tests.js
index 1ddb754..8030d6f 100644
--- 
a/DataValues/resources/globeCoordinate.js/tests/globeCoordinate.GlobeCoordinate.tests.js
+++ 
b/DataValues/resources/globeCoordinate.js/tests/globeCoordinate.GlobeCoordinate.tests.js
@@ -75,6 +75,7 @@
                                { latitude: 0, longitude: 0, precision: 1 },
                                { latitude: -3, longitude: 2, precision: 1 },
                                { latitude: 1.1, longitude: 2, precision: 0.1 },
+                               { latitude: 11.92, longitude: 255.92, 
precision: 0.1 },
                                { latitude: 90, longitude: 30.1, precision: 
0.01 },
                                { latitude: 0.1, longitude: 0.0075, precision: 
1 / 3600 },
                                { latitude: -0.1, longitude: 0, precision: 1 / 
60 },

-- 
To view, visit https://gerrit.wikimedia.org/r/84140
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: Ifaaa4e3144971278c2faa4adfffbcfed8b935784
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/extensions/DataValues
Gerrit-Branch: master
Gerrit-Owner: Aude <[email protected]>

_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits

Reply via email to