-
Notifications
You must be signed in to change notification settings - Fork 1
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
GlobeCoordinate: support other globes #94
Conversation
|
@@ -40,7 +40,7 @@ | |||
throw new Error( 'Longitude (' + this._longitude + ') is out of bounds' ); | |||
} | |||
|
|||
this._globe = 'http://www.wikidata.org/entity/Q2'; // TODO: Support other globes | |||
this._globe = gcDef.globe; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
maybe gcDef.globe || 'http://www.wikidata.org/entity/Q2' just in case?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Done
Thanks a lot for changes. I just wanted to merge this but found that Fomafix is right: Notes:
|
Right. Thank you. |
{ latitude: 5, longitude: -0.00292, precision: 1 / 36000 } | ||
{ latitude: 5, longitude: -0.00292, precision: 1 / 36000 }, | ||
{ latitude: 5, longitude: 2, globe: 'http://www.wikidata.org/entity/Q2' }, | ||
{ latitude: 5, longitude: 2, globe: 'http://www.wikidata.org/entity/Q313' } |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The test error is caused by a bug in equals
. I will fix it in an other patch. To make your patch pass please add precision: 1
to these lines.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thx, it puzzled me.
Prefer the injected value instead of the hard-coded 'http://www.wikidata.org/entity/Q2'. Add tests. Bug: T105321
Prefer the injected value instead of the hard-coded 'http://www.wikidata.org/entity/Q2'.
Add tests.
Bug: T105321