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