Hi Andreas and devs,

My fixe does not affect the layer order fixed by the WMS LAYERS parameters.

For exemple QGIS Web Client manage the layer order by building the WMS LAYERS parameters with all the layers and without groups. QGSI Web Client is not concerned by this fixe.

In Lizmap, user can specify Group as layer, which means that the group name is used in the WMS LAYERS parameter. We used it to reduce the number of names in the WMS LAYERS parameter, to have 3 rendering based on scale in one web client layer, to have based layer, etc. It is not a so old issue, probably introduce with the new layer tree.

I just fix it because the rendering was not in the same order as in the Desktop QGIS.

René-Luc


Le 15/09/2015 17:14, Andreas Neumann a écrit :
Hi René-Luc,

Thank you for the patch and the notice - do you know since when this bug was present. Probably since ages?

This change will basically mean that all QGIS Web Client users will have to patch their client at the same time they upgrade their server.

I am not against this change - I am just thinking about the consequences. I think we reversed rendering order on the client. If we don't communicate this properly - a lot of QGIS server/client users will run into this issue and will complain.

Andreas

On 15.09.2015 16:50, René-Luc Dhont wrote:
Hi devs and QGIS Server user,

I just fixed an issue in QGIS server and backport it to release-2_10 and release-2_8. https://github.com/qgis/QGIS/commit/8948721991bf321fb7eb2da259790624183f619d https://github.com/qgis/QGIS/commit/ae906b5eca9f99d7e4bd4e29e3b1bc4b49122be2 https://github.com/qgis/QGIS/commit/977d17751b9c05da1f09ed40d1cf27a97db5f534

If custom order is not enabled and a group is requested, layers was added in the DOM element order. The default rendering order is the reverse. To fix this issue, QgsWMSProjectParser::addLayersFromGroup has to read group element children from end (bottom) to start (top).

If you use QGIS server, I invite you to update your build.

Regards,
René-Luc
_______________________________________________
Qgis-developer mailing list
[email protected]
http://lists.osgeo.org/mailman/listinfo/qgis-developer

_______________________________________________
Qgis-developer mailing list
[email protected]
http://lists.osgeo.org/mailman/listinfo/qgis-developer

_______________________________________________
Qgis-developer mailing list
[email protected]
http://lists.osgeo.org/mailman/listinfo/qgis-developer

Reply via email to