return response object on "loadend" event for Fixed strategy #624

Merged
merged 1 commit into from Aug 8, 2012

Projects

None yet

3 participants

@milkypostman

This is a fix as suggested by @elemoine in #85.

@milkypostman

Updated this commit to include fixes for Fixed and BBOX strategies. I also documented the change in the "Layers" documentation because that seemed to be the place where it fit best; it's where the event is mentioned. I am happy to update the BBOX and Fixed strategies but I'm not sure where I should put the text.

@ahocevar
OpenLayers member

Thanks @milkypostman for your contribution. The documentation works well were you put it.

@ahocevar ahocevar merged commit 4922796 into openlayers:master Aug 8, 2012
@tschaub
OpenLayers member

ICLA received. Thanks @milkypostman.

@tschaub tschaub added a commit that referenced this pull request Aug 8, 2012
@tschaub tschaub Making events easier to test.
This should make it more straightforward to test the new event properties being sent with loadend on BBOX and Fixed strategies (see #624).
4ae9f68
@tschaub tschaub added a commit that referenced this pull request Aug 8, 2012
@tschaub tschaub Tests for response property on loadend event.
This confirms that the loadend event includes a response property when it results from a call to the strategy merge method (see #624).
970448e
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment