Aude has uploaded a new change for review. https://gerrit.wikimedia.org/r/81192
Change subject: allow lat/lon values from -360 to 360, possible for non-earth globes ...................................................................... allow lat/lon values from -360 to 360, possible for non-earth globes Change-Id: I13d908df9d187f369429ebbc4cbcaa1c26f5db6a --- M DataValues/src/LatLongValue.php 1 file changed, 4 insertions(+), 4 deletions(-) git pull ssh://gerrit.wikimedia.org:29418/mediawiki/extensions/DataValues refs/changes/92/81192/1 diff --git a/DataValues/src/LatLongValue.php b/DataValues/src/LatLongValue.php index 68216c6..b1364db 100644 --- a/DataValues/src/LatLongValue.php +++ b/DataValues/src/LatLongValue.php @@ -68,8 +68,8 @@ throw new InvalidArgumentException( 'Can only construct LatLongValue with a numeric latitude' ); } - if ( $latitude < -90 || $latitude > 90 ) { - throw new OutOfRangeException( 'Latitude needs to be between -90 and 90' ); + if ( $latitude < -360 || $latitude > 360 ) { + throw new OutOfRangeException( 'Latitude needs to be between -360 and 360' ); } } @@ -78,8 +78,8 @@ throw new InvalidArgumentException( 'Can only construct LatLongValue with a numeric longitude' ); } - if ( $longitude < -180 || $longitude > 180 ) { - throw new OutOfRangeException( 'Longitude needs to be between -180 and 180' ); + if ( $longitude < -360 || $longitude > 360 ) { + throw new OutOfRangeException( 'Longitude needs to be between -360 and 360' ); } } -- To view, visit https://gerrit.wikimedia.org/r/81192 To unsubscribe, visit https://gerrit.wikimedia.org/r/settings Gerrit-MessageType: newchange Gerrit-Change-Id: I13d908df9d187f369429ebbc4cbcaa1c26f5db6a Gerrit-PatchSet: 1 Gerrit-Project: mediawiki/extensions/DataValues Gerrit-Branch: master Gerrit-Owner: Aude <aude.w...@gmail.com> _______________________________________________ MediaWiki-commits mailing list MediaWiki-commits@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits