[QGIS-Developer] stronger minor version parsing is required inside GeoNode request

[G. Allegri]

Sure!

Il giorno ven 25 gen 2019 alle ore 11:41 Tim Sutton <tim at kartoza.com> ha
scritto:

> Can you assign the issue to me and I will ask Rohmat to look at it.
>
> Tim Sutton
> Co-founder of Kartoza
> Ex-QGIS project chairman
>
> On 25 Jan 2019, at 12:22, G. Allegri <giohappy at gmail.com> wrote:
>
> Eheh, right, it's a long time I didn't use it and I forgot it was changed
> ;)
> Thanks, I've opened the issue: https://issues.qgis.org/issues/21093
>
> Giovanni
>
> Il giorno ven 25 gen 2019 alle ore 11:17 Alessandro Pasotti <
> apasotti at gmail.com> ha scritto:
>
>> We are not using  trac anymore, here is the current issues website:
>> https://issues.qgis.org/
>>
>> You need an OSGEO id to log in.
>>
>>
>>
>> On Fri, Jan 25, 2019 at 11:13 AM G. Allegri <giohappy at gmail.com> wrote:
>>
>>> I would like but my trac account is not allowed to open tickets anymore.
>>> Don't know why and I don't know who I should ask :)
>>>
>>> Giovanni
>>>
>>> Il giorno ven 25 gen 2019 alle ore 11:10 Alessandro Pasotti <
>>> apasotti at gmail.com> ha scritto:
>>>
>>>> Giovanni,
>>>>
>>>> can you please file a ticket?
>>>>
>>>> On Fri, Jan 25, 2019 at 11:06 AM G. Allegri <giohappy at gmail.com> wrote:
>>>>
>>>>> Currently QGIS parses GeoNode's version with a simple .toInt():
>>>>> https://github.com/qgis/QGIS/blob/master/src/core/geocms/geonode/qgsgeonoderequest.cpp#L275
>>>>> but GeoNode adds more informations to the minor version, for instance
>>>>> "rc" for a release candidate. This makes the parsing fail and thus layers
>>>>> are not fetched.
>>>>>
>>>>> As an example look at the "geonode_version" field on the master demo
>>>>> instance: http://master.demo.geonode.org/api/layers/
>>>>>
>>>>> Probably a regex should be used to extract only the first numbers from
>>>>> the minor version part.
>>>>>
>>>>> Best,
>>>>> Giovanni
>>>>> _______________________________________________
>>>>> QGIS-Developer mailing list
>>>>> QGIS-Developer at lists.osgeo.org
>>>>> List info: https://lists.osgeo.org/mailman/listinfo/qgis-developer
>>>>> Unsubscribe: https://lists.osgeo.org/mailman/listinfo/qgis-developer
>>>>
>>>>
>>>>
>>>> --
>>>> Alessandro Pasotti
>>>> w3:   www.itopen.it
>>>>
>>>
>>
>> --
>> Alessandro Pasotti
>> w3:   www.itopen.it
>>
> _______________________________________________
> QGIS-Developer mailing list
> QGIS-Developer at lists.osgeo.org
> List info: https://lists.osgeo.org/mailman/listinfo/qgis-developer
> Unsubscribe: https://lists.osgeo.org/mailman/listinfo/qgis-developer
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.osgeo.org/pipermail/qgis-developer/attachments/20190125/67d4f95a/attachment.html>