[QGIS-Developer] QgsSettings + ows key cleanup?

[Alessandro Pasotti]

On Thu, May 18, 2017 at 1:27 PM, Richard Duivenvoorde <rdmailings at duif.net>
wrote:

> On 18-05-17 12:11, Alessandro Pasotti wrote:
> > Hi Richard,
> >
> > Cleaning is always very welcome!
> >
> > I think that to bring some more order to the root level settings, we
> > should eventually make more use of the optional section argument:
> >
> > https://qgis.org/api/classQgsSettings.html#
> aba2f174ee000905cac8aff26665ed7c7
> >
> > This is not implemented for groups though, but if you feel it could be
> > helpful I can add it.
>
> Hi Alessandro,
>
> I do not understand: "This is not implemented for groups though..."
> Isn't a section the same concept as a group?
> Where is this 'section' element to be used?
> Or is the idea to use it in a (free chosen) group?
>

I mean that the optional section argument is not available for beginGroup
https://qgis.org/api/classQgsSettings.html#ae0ffd9ce99f3fa8a712befb9be99412b


>
> I already have > 40 groups/sections(?) (as counted in the 'Advanced
> Settings Editor').
> I do not think it is possible to bring that back to the 8 you propose,
> do you?
>

I don't know, maybe you can move some of them under one of the available
sections. The idea of the sections is to reduce (to 8, but can be expanded
if needed) the root level elements, of course you are not forced to do that
but I think that grouping GUI settings under the "gui" section, application
settings under the "app" section, plugins settings under "plugins" etc.
etc. is the first step towards a more logical tree classification.



>
> Looking into the 'Advanced Settings Editor' I also have double values in
> the following 'groups'?
> PythonPlugins
> and
> pythonplugins
> but this could be due to recent changes. Ik will remove my config
>
>
This is bad design: see rule n. 1 in
http://doc.qt.io/qt-5/qsettings.html#section-and-key-syntax


-- 
Alessandro Pasotti
w3:   www.itopen.it
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.osgeo.org/pipermail/qgis-developer/attachments/20170518/410245a2/attachment.html>