No image is displayed when switching base layer

[openlayers]

Check the Spherical Mercator example and try chosing a different base layer. Although the image tiles seem to be loading, they are not displayed.

SVN: 6569

[bartvde]

This is because the Google Layer does not have a minResolution and maxResolution, they are both null, so the calculateInRange won't work.

[crschmidt]

Is this new? I don't remember anything changing here recently... Perhaps this is the result of my recent display() changes, I suppose?

[crschmidt]

I've confirmed this is due to my recent layer changes. I think that the fact that the fixedzoomlevels subclasses don't have a min/max resolution set (and therefore no valid calculateInRange()) is a bug we should fix, rather than reverting my fix and changing it some other way.

I'll own this one, and fix it soon.

[crschmidt]

Patch attached. Seems to fix the problem for me. Anyone else want to try it anad confirm that it works for their application?

[crschmidt]

Serious regression. Moving to 2.6.

[crschmidt]

1457.patch adds a test. This test:

• Succeeds before change to Layer.js
• Fails on trunk
• Succeeds with patch.

This is a test of functionality, rather than a unit test: This is okay with me, but I can write more tests for calculateInRange if people care.

WIth test, marking for Review.

[crschmidt]

Move the change inside the 'if' loop: if this.RESOLUTIONS == null, then we would have gotten an error. So, fix that.

[tschaub]

looks good

[crschmidt]

(In [6572]) FixedZoomLevels subclassese don't propertly set min/max resolution, so calculateInRange always returns false, so the layers can never be displayed (due to recent code): Fix calculateInRange by setting min/max res. r=tschaub, (Closes #1457)

[openlayers]

Still not OK.

Check the VE example: the base layer sets "minZoomLevel: 4, maxZoomLevel: 6", but instead of displaying a 3 unit zoom scale, it shows a 14 units bar (from minZoomLevel 4 to MAX_ZOOM_LEVEL 17).

Actually, I think that the MAX_ZOOM_LEVEL should be 16 for VirtualEarth, as there are only 16 entries in the RESOLUTIONS array; furthermore, after FixedZoomLevels::initResolutions() minResolution is undefined, as being RESOLUTIONS[17].

[crschmidt]

This is a totally different issue, unrelated to this ticket.

It has also been the case since 2.2:  http://dev.openlayers.org/releases/OpenLayers-2.2/examples/ve.html

Please create a new ticket for that issue.