[LH4776] RC11 installs to wrong directory for standard user

User avatar
devinganger
Posts: 2438
Joined: Sat Nov 06, 2010 1:55 pm
Platform: Mac, Win + iOS
Location: Monroe, WA 98272
Contact:

Fri Nov 06, 2020 7:48 am Post

As an admin user, Scrivener properly installs to C:\Program Files or C:\Program Files (x86).

As a standard user, however, RC11 is suggesting C:\Users\<username>\Documents.

THIS IS WRONG. It should be C:\Users\<username>\AppData\Local in order to be compliant with Microsoft and Windows guidelines and expected behavior.

The user's Documents folder is meant to hold documents, not local installable applications. It can be redirected to the OneDrive folder or to other locations, including to networked drives. Storing application binaries here by default can lead to all sorts of unexpected behaviors.
--
Devin L. Ganger
Not a L&L employee; opinions are those of my cat
Life has a way of moving you past wants and hopes

User avatar
tiho_d
Posts: 1425
Joined: Tue Sep 13, 2011 1:14 pm
Platform: Linux + Windows

Fri Nov 06, 2020 6:58 pm Post

You are absolutely correct, Devin. Unfortunately we had issues with this using the previous versions of our installer. I will give it another try and hopefully it is improved.

User avatar
devinganger
Posts: 2438
Joined: Sat Nov 06, 2010 1:55 pm
Platform: Mac, Win + iOS
Location: Monroe, WA 98272
Contact:

Sat Nov 07, 2020 1:44 am Post

tiho_d wrote:You are absolutely correct, Devin. Unfortunately we had issues with this using the previous versions of our installer. I will give it another try and hopefully it is improved.


Thanks for the fast response. I'll be happy to provide an independent test of it when you're ready! :)
--
Devin L. Ganger
Not a L&L employee; opinions are those of my cat
Life has a way of moving you past wants and hopes

User avatar
narrsd
Posts: 733
Joined: Wed Sep 22, 2010 8:34 pm
Platform: Win + iOS

Sat Nov 07, 2020 9:02 pm Post

And good catch too, Devin - along with great response as ever from @tiho....

Best to both, and enjoynig watching this project accumulate its fullness, as it does,
Clive

User avatar
tiho_d
Posts: 1425
Joined: Tue Sep 13, 2011 1:14 pm
Platform: Linux + Windows

Wed Dec 02, 2020 12:07 am Post

This has been adjusted and can be tested in RC13. Thanks for pointing this out, Devin!

User avatar
devinganger
Posts: 2438
Joined: Sat Nov 06, 2010 1:55 pm
Platform: Mac, Win + iOS
Location: Monroe, WA 98272
Contact:

Wed Dec 02, 2020 12:34 am Post

tiho_d wrote:This has been adjusted and can be tested in RC13. Thanks for pointing this out, Devin!


If you want to toss me a test build I'd be happy to confirm it sooner than RC13...:)
--
Devin L. Ganger
Not a L&L employee; opinions are those of my cat
Life has a way of moving you past wants and hopes

User avatar
devinganger
Posts: 2438
Joined: Sat Nov 06, 2010 1:55 pm
Platform: Mac, Win + iOS
Location: Monroe, WA 98272
Contact:

Sun Dec 06, 2020 6:46 pm Post

tiho_d wrote:This has been adjusted and can be tested in RC13. Thanks for pointing this out, Devin!


Just to follow up, I installed RC13 on my test system as a non-admin user, and as expected, it now defaults to the C:\Users\<username>\AppData\Local folder.

However, there was one error thrown about trying to update a registry key in the root classes hive, which it should probably not be trying to update as a non-admin user:

NonAdminInstallError-Registry.PNG
NonAdminInstallError-Registry.PNG (14.77 KiB) Viewed 344 times
--
Devin L. Ganger
Not a L&L employee; opinions are those of my cat
Life has a way of moving you past wants and hopes

User avatar
tiho_d
Posts: 1425
Joined: Tue Sep 13, 2011 1:14 pm
Platform: Linux + Windows

Mon Dec 07, 2020 11:11 am Post

Thanks for following on this, Devin! We also spotted this problem, but could not get it fully fixed for RC13. We hope to get it fully fixed for the next update.

User avatar
devinganger
Posts: 2438
Joined: Sat Nov 06, 2010 1:55 pm
Platform: Mac, Win + iOS
Location: Monroe, WA 98272
Contact:

Mon Dec 07, 2020 4:56 pm Post

tiho_d wrote:Thanks for following on this, Devin! We also spotted this problem, but could not get it fully fixed for RC13. We hope to get it fully fixed for the next update.


Awesome! I will be ready to give it another shot come that day. :)
--
Devin L. Ganger
Not a L&L employee; opinions are those of my cat
Life has a way of moving you past wants and hopes

User avatar
tiho_d
Posts: 1425
Joined: Tue Sep 13, 2011 1:14 pm
Platform: Linux + Windows

Tue Dec 29, 2020 11:04 am Post

Hi Devin, when you have a chance please try these installers:

2.9.9.13 (64-bit) | 2.9.9.13 (32-bit)

They install and uninstall using only the user registry space. Obviously uninstalling your current version manually beforehand is recommended.

It is safe to install and use the new version until the new update is out.