[QGIS-Developer] pylupdate5 does not understand classes in classes

Ari Jolma ari.jolma at gmail.com
Fri Mar 9 04:55:18 PST 2018


On 09.03.2018 14:51, Luigi Pirelli wrote:
> I was answering you to prepare a PR... and you just did it
> https://github.com/qgis/QGIS/pull/6566
> This is the best way to proceed, tnx

No, that PR is not about this issue. I did not create a PR for this 
because I don't know how to fix this. I assume buildvrt.py (and others 
with same issue) should be modified so that the class within the class 
is removed -- or maybe fix pylupdate5 since it is doing a bad job.

Ari

>
> "don't ask what the QGIS dev community can do for you, but what you
> can do for all"
> Luigi Pirelli
>
> **************************************************************************************************
> * LinkedIn: https://www.linkedin.com/in/luigipirelli
> * Stackexchange: http://gis.stackexchange.com/users/19667/luigi-pirelli
> * GitHub: https://github.com/luipir
> * Mastering QGIS 2nd Edition:
> * https://www.packtpub.com/big-data-and-business-intelligence/mastering-qgis-second-edition
> * Hire me: http://goo.gl/BYRQKg
> **************************************************************************************************
>
>
> On 9 March 2018 at 12:32, Ari Jolma <ari.jolma at gmail.com> wrote:
>> Hi,
>>
>> Two strings in python/plugins/processing/algs/gdal/buildvrt.py do not
>> translate because pylupdate5 puts them into wrong context
>> (ParameterVrtDestination should be in buildvrt).
>>
>> That's because in buildvrt.py ParameterVrtDestination class is defined
>> within buildvrt. Perhaps pylupdate5 simply does not support classes in
>> classes
>>
>> I suspect this kind of error may be also in other places.
>>
>> Ari
>>
>>
>> _______________________________________________
>> 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



More information about the QGIS-Developer mailing list