Path info saved in ini file even if history is disabled

Bugs found in XnView Classic. Please report only one bug per topic!

Moderators: XnTriq, helmut, xnview

Dreamer
XnThusiast
Posts: 4608
Joined: Sun Jul 25, 2004 9:08 pm

Path info saved in ini file even if history is disabled

Post by Dreamer »

Some paths are saved in xnview.ini file, even if the option "Purge 'Most recently used files and dirs' on exit" is disabled. I'm not sure if it's a bug, but the paths shouldn't be saved in ini IMHO.

E.g.: PathBrowse=

There are also others, but I'm not sure which are just options and which are the history of browsed folders.
Dreamer
Dreamer
XnThusiast
Posts: 4608
Joined: Sun Jul 25, 2004 9:08 pm

Post by Dreamer »

Looks fixed in 1.95, but I found more paths that shouldn't be saved in ini file, or anywhere, if option "Purge 'Most recently used files and dirs' on exit" is enabled.

:bug: #1 Go to Options > Interface > Toolbar, click the "Append" button and then [...] next to "Command", or "Icon file". Path with images is used here. Another path should be used here - My documents, root path, StartupDirectory= (from ini), or another path, but definitely not the path with images viewed after xnview close.

:bug: #2 In the xnview.ini file there is ANOTHER path with images in [Search] section - "Directory=" - this path should be deleted after xnview close too.

Edit:

:bug: #3 The same path as #1 is used in Open picture dialog - press Ctrl+O to open this dialog.

If option "Purge 'Most recently used files and dirs' on exit" is enabled, ALL paths should be deleted on xnview close and then NO path should be used in any dialog next time xnview is started.

Edit2: I found more paths, I'm not 100% sure it's all just history that should be deleted though.

[MultEx]
Dir=

[Start]
PathScript=
PathSlide=
PathSlideScript=

PathScript= is perhaps OK, it's not the path for images.

If option "Purge 'Most recently used files and dirs' on exit" is enabled, the path of actual image(s) - selected in browser/viewer should be used IMO, at least at start, then perhaps the last used, but ALL these paths should be deleted on close.
Dreamer
klaus2
Posts: 98
Joined: Thu Nov 02, 2006 7:27 pm

Post by klaus2 »

Hello Dreamer,

sorry, this time i disagree because it's not user-friendly; see http://newsgroup.xnview.com/viewtopic.php?t=16132

What's the benefit for to restart xnview using xnview's directory as the current directory?
I vote strictly for the option to purge "all" the history entries AND to have the possibility to restart using the last used directory, like before. At least by an option.

Your proposal is straight-forward, theoretically; practically for me it is a loss of usability. I really hope that a purge combined with a "remember last dir" will be kept. Please let's stay flexible.

Best regards,
klaus2
Dreamer
XnThusiast
Posts: 4608
Joined: Sun Jul 25, 2004 9:08 pm

Post by Dreamer »

I found another path...

[Html]
OutputDirectory=

klaus2 - hello,
I vote strictly for the option to purge "all" the history entries AND to have the possibility to restart using the last used directory, like before. At least by an option.
OK, but this is already possible!! There is such an option in General > Startup. BUT! When using "Specified" here, the path (and not only this path) is saved in xnview ini and it's used for Open dialog and so on...

So I'm NOT against your opinion, I think some paths must be saved in ini even if history is deleted on close, e.g. path for cache, folder for other files...
Dreamer
Dreamer
XnThusiast
Posts: 4608
Joined: Sun Jul 25, 2004 9:08 pm

Post by Dreamer »

I found another one - not paths, just names in:

[Rename]
Template0=
Template1=
...

There should be at least another option to disable saving paths and names, or delete the paths and names on close IMO.

There are many nice options if someone wants to delete some paths, but wants the other, e.g. favourites, rating, category, colour label, tag.

Option to disable/delete all history on close is very important IMHO and it's not working good now, so I still think this is very important issue.
Dreamer
klaus2
Posts: 98
Joined: Thu Nov 02, 2006 7:27 pm

Post by klaus2 »

Hello Dreamer,

thanks for your attention!
this is already possible!! There is such an option in General > Startup. BUT! When using "Specified" here
No, General>>Operation>>Startup directory>>"specified" is no good functional equivalent for "last used". Because the entry is static, whereas my last used dir may vary.
Using "specified" for to remember the last used dir, we would be enforced to enter the option dialog each time before exiting ...

About the "general>>.." options: as mentioned in the thread above, the option ("last used") ist still available ... but now it is simply ignored when requesting a purge.
If i tell "A" (purge) and "B" (keep last dir), want i want to say is: do "A" but respect "B". And not: do A .... and do quite the opposite of B.
Obviously a "none" option is missing here for to fullfil the purpose that the 1.95-change intends.

IMO at least the following items should be kept protectable against any "loss of memory":
- the location of the ini file eg. xnview's directory
- the location of the databases, eg. cache
- the "last used dir". Here the user should be able to decide whether it should be remembered or not.

Greetings!
klaus2
Dreamer
XnThusiast
Posts: 4608
Joined: Sun Jul 25, 2004 9:08 pm

Post by Dreamer »

Hello Klaus2,
klaus2 wrote:No, General>>Operation>>Startup directory>>"specified" is no good functional equivalent for "last used". Because the entry is static, whereas my last used dir may vary.
Using "specified" for to remember the last used dir, we would be enforced to enter the option dialog each time before exiting ...

About the "general>>.." options: as mentioned in the thread above, the option ("last used") ist still available ... but now it is simply ignored when requesting a purge.
If i tell "A" (purge) and "B" (keep last dir), want i want to say is: do "A" but respect "B". And not: do A .... and do quite the opposite of B.
Obviously a "none" option is missing here for to fullfil the purpose that the 1.95-change intends.
OK, so it looks the last used path is always saved to ini - it's not deleted on close if using "purge" option, but it's not used if using "last used" option!? I have just tried it and it works really strange, if using "last used" option, path before I closed xnview was D:\Pictures\Nature\ , but on xnview start it was D:\Programs\FreeCommander\plugins\anytag\ !? It's a bug IMO. :bug: :?:
klaus2 wrote:IMO at least the following items should be kept protectable against any "loss of memory":
- the location of the ini file eg. xnview's directory
- the location of the databases, eg. cache
- the "last used dir". Here the user should be able to decide whether it should be remembered or not.
"last used dir" should be saved / used ONLY if option "Startup directory" is set to "last used" - this option should have higher priority than "purge", I agree, but - if it's set to "specified", path should be deleted on xnview close.

...So it looks you don't want anything else than me, right?
Dreamer
klaus2
Posts: 98
Joined: Thu Nov 02, 2006 7:27 pm

Post by klaus2 »

Hello Dreamer,

no, i want the same :-)
If option "Purge 'Most recently used files and dirs' on exit" is enabled,
AND General>>Operation>>Startup directory>>"last used" is enabled,
then the value of the ini key [Start] PathBrowse= should not be wiped out.

But: what, if someone wants to have that wiped out too?
Seems an additional option "none" for the startup directoy might be needed.
Or a "exclude last dir" checkbox may accomplish the purge-most-recently option
.. or any better approach as you did propose, of course.

Have a nice weekend!
klaus2
Dreamer
XnThusiast
Posts: 4608
Joined: Sun Jul 25, 2004 9:08 pm

Post by Dreamer »

klaus2 wrote:But: what, if someone wants to have that wiped out too?
Seems an additional option "none" for the startup directoy might be needed.
Or a "exclude last dir" checkbox may accomplish the purge-most-recently option
.. or any better approach as you did propose, of course.
Why "none"? I think "specified" is enough, just "C:" can be used, or path with the pictures, but all paths (I've reported above) (incl. last used dir) should be deleted on close if "purge..." option is enabled. Currently these paths are NOT deleted, this is the first bug. :bug:

BTW There is another option "current" - it works almost as "none" - the path where xnview.exe is started is used, but I'm not sure if this is correct. E.g. when I start xnview from FreeCommander toolbar, then this path is used (D:\Programs\FreeCommander\).

Another bug (IMHO) is that path of last used dir IS deleted on close if option "purge..." is enabled and startup is set to "last used". :bug:
Dreamer
klaus2
Posts: 98
Joined: Thu Nov 02, 2006 7:27 pm

Post by klaus2 »

Hello Dreamer,

that's fine with me (your remark about a "none" option is correct!)

Kind regards,
klaus2
Dreamer
XnThusiast
Posts: 4608
Joined: Sun Jul 25, 2004 9:08 pm

Post by Dreamer »

It's in the changelog as "fixed", but these paths are still not deleted from xnview.ini file if xnview is closed.

PathBrowse=

Template0=
Template1=
Template2=
...

[Search]
Directory=

[Start]
PathSlide=
PathSlideScript=

[MultEx]
Dir=

Bugs #1 and #3 look fixed.
Dreamer
klaus2
Posts: 98
Joined: Thu Nov 02, 2006 7:27 pm

Post by klaus2 »

Hi Dreamer,

Maybe you are right, but pls exlude the PathBrowse= from your request.
Else bug #3 will reappear.
The last used dir is stored within this key, and purging it will remove the information about the last used dir.
I'm just happy that now it is possible again to remember that information, just as before 1.95, when having set startup to "last used".

Kind regards,
klaus2
Dreamer
XnThusiast
Posts: 4608
Joined: Sun Jul 25, 2004 9:08 pm

Post by Dreamer »

klaus2 wrote:Maybe you are right, but pls exlude the PathBrowse= from your request.
Else bug #3 will reappear.
The last used dir is stored within this key, and purging it will remove the information about the last used dir.
I'm just happy that now it is possible again to remember that information, just as before 1.95, when having set startup to "last used".
Right, when having set startup to "last used" - BUT the path in PathBrowse= is saved (not deleted) also if that option is set to Specified!! This is a bug IMO.
Dreamer
klaus2
Posts: 98
Joined: Thu Nov 02, 2006 7:27 pm

Post by klaus2 »

Hm, but if i try that and tell for instance "specified" and set that to C:\Temp\,
then (for me) xnview behaves correctly ... and the PathBrowse= is consequently set to C:\Temp\ ...
.
Whereelse the information about the specified directory selected should be stored?
.
For me that is ok, but maybe i miss something??
Dreamer
XnThusiast
Posts: 4608
Joined: Sun Jul 25, 2004 9:08 pm

Post by Dreamer »

I'm using startup option "Specified - D:\Pictures\", but when I go to path D:\ABC and exit xnview, PathBrowse= is set to this path - D:\ABC - try to do the same, but then go to another folder, exit xnview and check that option.

Browser startup option is stored in [Browser] section - StartupDirectory=.
Dreamer