enabling FI from TreeGDE does not work

[rwoita]

look up following discussion on nabble.com :

http://www.nabble.com/MB-2.5-RC2---treeGDE-featureinfo1---2nd-to16874453.html

[mschulz]

setting the version to 2.5rc2

[mschulz]

Just going through the code (realizing there have been profound changes ;-)) i found, that when changing only the querylayer status of a layer, mb_restateLayers is not called, thus the querylayers array is not updated. Adding a call the handleSelection as when changing visibility does the trick, but i'm not sure if this is correct or the intended usage of this function.

btw. also changed the component to treeGDE

[christoph]

apparently a function handleLayer of the WMS object is called...however, this function does nothing. The obscure code makes it impossible to debug.

[nimix]

I did some investigation today and the problem seems to be that the call of mb_restateLayers is missing. If you change the FI-Checkboxes and change the visibility of a layer afterwards the FI-Selection works

[nimix]

unfortunately I don't have my test evironement now, but as far as I can see the treefolder2.php from nimix_dev branch does it the right way.

[nimix]

fixed in 2.5 branch and trunk

[mschulz]

there still seems to be a problem left: when enabling a complete wms with the switchwms-checkbox, the query checkboxes of the layers are checked, but the layers are not added to querylayers: click with FI-tool on the map raises a JS-popup "Please select a layer!".

[christoph]

​http://trac.osgeo.org/mapbender/changeset/2768

[christoph]

now layers that are non queryable are added to the request....

[christoph]

this code is really bad.

we definitely need to replace this module by another one

[nimix]

another try to really fix it ​http://trac.osgeo.org/mapbender/changeset/2902

[mschulz]

Thx, Nimix. Tests with these changes were successfull.