Skip to content
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

Fixed imagery providers whose rectangle crosses the IDL. #2313

Merged
merged 12 commits into from
Dec 9, 2014
Merged

Conversation

bagnell
Copy link
Contributor

@bagnell bagnell commented Dec 9, 2014

For #2195.

@@ -460,7 +521,7 @@ define([
* @param {Rectangle} rectangle On rectangle to find an intersection
* @param {Rectangle} otherRectangle Another rectangle to find an intersection
* @param {Rectangle} [result] The object onto which to store the result.
* @returns {Rectangle} The modified result parameter or a new Rectangle instance if none was provided.
* @returns {Rectangle|undefined} The modified result parameter, a new Rectangle instance if none was provided or undefined if there is no intersection.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Technically a breaking change, but I don't think it is an issue.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hmm, actually we should deprecate this (one release before removal is fine) and replace it with a function with a new name.

@pjcozzi
Copy link
Contributor

pjcozzi commented Dec 9, 2014

Code in #2195 looks good in Sandcastle now. There's one test failure:

image

@pjcozzi
Copy link
Contributor

pjcozzi commented Dec 9, 2014

Looks good otherwise.

@bagnell
Copy link
Contributor Author

bagnell commented Dec 9, 2014

@pjcozzi This is ready for another look.

@pjcozzi
Copy link
Contributor

pjcozzi commented Dec 9, 2014

Looks good.

pjcozzi added a commit that referenced this pull request Dec 9, 2014
Fixed imagery providers whose rectangle crosses the IDL.
@pjcozzi pjcozzi merged commit 8f87a85 into master Dec 9, 2014
@pjcozzi pjcozzi deleted the extent branch December 9, 2014 18:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants