[Qgis-developer] [QGIS Mapserver] addDrawingOrder changed logic in 2.4?

[G. Allegri]

Thanks Marco.
Done in http://hub.qgis.org/issues/10625

In case the changed logic is only a side effect of the parsing refactoring,
the solution seems quite straightforwars, rolling back to differentiating
between embedded/not embedded layers.

giovanni


2014-06-18 16:52 GMT+02:00 Marco Hugentobler <
marco.hugentobler at sourcepole.ch>:

> Hi Giovanni
>
> Please open a blocker ticket about the layer order and assign it to me.
> I'll try to look at it before the release.
>
> Regards, Marco
>
>
> Von Samsung Galaxy Note gesendet
>
>
>
> -------- Ursprüngliche Nachricht --------
> Von: "G. Allegri" <giohappy at gmail.com>
> Datum: 18.06.2014 16:34 (GMT+01:00)
> An: qgis-developer <qgis-developer at lists.osgeo.org>
> Betreff: [Qgis-developer] [QGIS Mapserver] addDrawingOrder changed logic
> in 2.4?
>
>
> I'm investingating why QWC is giving me problems with QGIS Server 2.4.
> I've spot a difference between QGIS 2.2 and the master code which seems to
> cause problems during the GetProjectSettings response parsing.
>
> In previous versions of QGIS Server addDrawingOrder worked for project
> layers [1] while for embedded layers we had addDrawingOrderEmbeddedGroup
> [2].
> Now we have only addDrawingOrder which seems to work only for embedded
> projects.
> The result is that, without emdedded layers, the <LayerDrawingOrder>
> element will always be empty.
>
> This seems to cause some problems on the QWC side. In fact now QWC works
> only with GetCapabilities, while it worked both with GetCapabilities and
> GetProjectSettings before.
>
> Is there a reason for this change?
>
> giovanni
>
> [1]
> https://github.com/qgis/QGIS/blob/release-2_2/src/mapserver/qgsprojectparser.cpp#L3644
> [2]
> https://github.com/qgis/QGIS/blob/release-2_2/src/mapserver/qgsprojectparser.cpp#L4067
>
> --
> Giovanni Allegri
> http://about.me/giovanniallegri
> Twitter: https://twitter.com/_giohappy_
> blog: http://blog.spaziogis.it
> GEO+ geomatica in Italia http://bit.ly/GEOplus
>



-- 
Giovanni Allegri
http://about.me/giovanniallegri
Twitter: https://twitter.com/_giohappy_
blog: http://blog.spaziogis.it
GEO+ geomatica in Italia http://bit.ly/GEOplus
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.osgeo.org/pipermail/qgis-developer/attachments/20140618/a79e6fc6/attachment.html>