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

[G. Allegri]

This pull request shoud fix it, but please review it in case it would break
somethign else https://github.com/qgis/QGIS/pull/1462


2014-06-18 17:02 GMT+02:00 G. Allegri <giohappy at gmail.com>:

> 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
>



-- 
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/1dab1f98/attachment.html>