Skip to content
This repository has been archived by the owner on Sep 27, 2023. It is now read-only.

Windows 7 Settings dialog is memoryless #72

Closed
ms31415a16 opened this issue Oct 19, 2015 · 7 comments
Closed

Windows 7 Settings dialog is memoryless #72

ms31415a16 opened this issue Oct 19, 2015 · 7 comments
Assignees

Comments

@ms31415a16
Copy link

Hello,

When I open settings dialog to check / update settings it deletes previously entered info.
Also I can't close dialog without re entering everything.

@mbos
Copy link
Owner

mbos commented Oct 19, 2015

Hiya,
It needs write access to your profile in order to save the information. You could check the errorlog in windows to check what went wrong.

Cheers,

Mike

@ms31415a16
Copy link
Author

Hi Mike, thanks for your reply!
It seems I don't have rights to write into this directory.
Can't this info be written into the same directory as where passwords database is stored?

@mbos mbos self-assigned this Oct 20, 2015
@mbos
Copy link
Owner

mbos commented Oct 20, 2015

Not in this version :-)
I like the idea though so will change it.

@ms31415a16
Copy link
Author

Thanks!)

@mbos
Copy link
Owner

mbos commented Oct 20, 2015

I just realized, in order to read from the password directory the program needs to know where that is.....
Basically the problem is that you can either store in the profile or in the program's directory. The programs's directory is usually protected (needs admin rights) and in some cases the profile directory is protected.

So what I think will do is:

  • First look into profile
  • Look into application directory
  • Look in the directory given with a command line argument or fail when no argument is given

@mapitman
Copy link
Collaborator

You could also use Isolated Storage. It is dedicated per app/user and the application will have access. If I get some time, I may add this and send you a pull request.

@mbos
Copy link
Owner

mbos commented Nov 11, 2015

This is actually a very good idea thanks!

@mbos mbos closed this as completed in db9a8fd Nov 13, 2015
mbos added a commit that referenced this issue Nov 13, 2015
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

3 participants