[Paraview-developers] New Settings/Custom plugin bug

[Cory Quammen]

Mathieu,

I have a proposed fix for this under review at
http://review.source.kitware.com/#/t/4253/

Feel free to try it out with git command

git fetch http://review.source.kitware.com/p/ParaView
refs/changes/96/15596/1 && git checkout FETCH_HEAD -b
14760_fix_settings_saving

This fix changes where the user settings file is saved. The new pattern for
the user settings JSON file is

~/.config/ORGANIZATION/APPLICATION-UserSettings.json

It's not 100% certain that this is how it will be, but I think it makes
sense to have the organization and application name in the case that one
organization has more than one branded ParaView application.

Best,
Cory


On Wed, May 28, 2014 at 4:06 PM, Cory Quammen <cory.quammen at kitware.com>
wrote:

> Mathieu,
>
> Thank you for the report. I've should have this fixed tonight or tomorrow.
>
> Cory
>
> On Wed, May 28, 2014 at 3:56 PM, Mathieu Westphal
> <mathieu.westphal at gmail.com> wrote:
> > Hello
> >
> > With my custom application, it is not working.
> >
> > Each time i close my application i have :
> > Saving settings file failed
> >
> > Of course there is no UserSettings.json
> > the .ini file is written without problem however.
> >
> >
> > I've debugged it and the ini file is written into :
> > ~/.config/ORGANIZATION/cnsGUI0.3.6869.ini
> >
> > and the json file try to be written into:
> > ~/.config/ApplicationName/UserSettings.json
> >
> > while ~/.config/ApplicationName does not exist.
> >
> > So it seems the "ORGANIZATION" cmake variable is not taken into account
> > when creating the user config file path.
> >
> > Mathieu
> >
> > On Wed, May 28, 2014 at 03:11:04PM +0200, Felipe Bordeu wrote:
> >> -----BEGIN PGP SIGNED MESSAGE-----
> >> Hash: SHA1
> >>
> >> Hi Cory,
> >>
> >> Everything work as expected. Thanks.
> >>
> >>
> >> Felipe
> >>
> >> Le 25/05/2014 20:49, Cory Quammen a écrit :
> >> > Felipe,
> >> >
> >> > I have pushed a fix to the git next branch, commit
> >> > b154c1c4f31cb9d518965fd742a4603602036744. It will be in master after
> >> > the next gatekeeper review (assuming it is approved).
> >> >
> >> > Thanks,
> >> > Cory
> >> >
> >> > On Fri, May 23, 2014 at 11:31 AM, Felipe Bordeu
> >> > <felipe.bordeu at ec-nantes.fr> wrote:
> >> > Bug reported
> >> >
> >> > http://www.paraview.org/Bug/view.php?id=14753
> >> >
> >> >
> >> > Felipe
> >> >> _______________________________________________
> >> >> Paraview-developers mailing list
> >> >> Paraview-developers at paraview.org
> >> >> http://public.kitware.com/mailman/listinfo/paraview-developers
> >>
> >> - --
> >> Felipe Bordeu Weldt
> >> Ingénieur de Recherche
> >> - -------------------------------------
> >> Tél. : 33 (0)2 40 37 16 57
> >> Fax. : 33 (0)2 40 74 74 06
> >> Felipe.Bordeu at ec-nantes.fr
> >> Institut GeM - UMR CNRS 6183
> >> École Centrale Nantes
> >> 1 Rue de La Noë, 44321 Nantes, FRANCE
> >> - -------------------------------------
> >> -----BEGIN PGP SIGNATURE-----
> >> Version: GnuPG v1.4.11 (GNU/Linux)
> >> Comment: Using GnuPG with Thunderbird - http://www.enigmail.net/
> >>
> >> iQEcBAEBAgAGBQJTheBoAAoJEE/fMfNgU9/DX98IAINzUEBd5ejSy8wETHLbBPj6
> >> 57LUNWEIiTHizjHmo7wopbK+1bM8Ce7yXyM/XEBvHf1xy5bjsXls912Pr4p7bWqY
> >> NKLs/34LKBZ9NfiNTjM2lCHEWz4SkXSHu7HAmxAELfw5PRjmGU77woT5yKF+V6ao
> >> 9jKtxkQdZGGdsyLvFbYZ4Uv6hqlRQtRvUf2YIJtcDSRsZhkq87sAfu1iRdxmdEiI
> >> s5Ie82i/hhGINQCEk7kg6VR7lgyJQ/nDzNDv7PvcJims2w+zORFhj4gBYNzNU+pb
> >> Sfp4Eg8kx87u0tVPrrdtpL3kxJRLm+b90L5+1w99/T7E0u+Ld2/R86+CQKZ/WQw=
> >> =gv2W
> >> -----END PGP SIGNATURE-----
> >> _______________________________________________
> >> Paraview-developers mailing list
> >> Paraview-developers at paraview.org
> >> http://public.kitware.com/mailman/listinfo/paraview-developers
> > _______________________________________________
> > Paraview-developers mailing list
> > Paraview-developers at paraview.org
> > http://public.kitware.com/mailman/listinfo/paraview-developers
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://public.kitware.com/pipermail/paraview-developers/attachments/20140531/dcaa4bae/attachment.html>