Chaning directories for logs and state files

[brainrom]

Fix for #3406 and #7654.

And cookie jar now also stores in StateLocation, because it's app state and not configuration. Added corresponding migration function.

Shouldn't negatively affect Windows and Mac OS, because StateLocation on these platforms are subdirectories of AppConfigLocation.

From Qt doc:

Path type	macOS	Windows
AppConfigLocation	"~/Library/Preferences/<APPNAME>"	"C:/Users/<USER>/AppData/Local/<APPNAME>", "C:/ProgramData/<APPNAME>"
StateLocation	"~/Library/Preferences/<APPNAME>/State"	"C:/Users/<USER>/AppData/Local/<APPNAME>/State", "C:/ProgramData/<APPNAME>/State"

[camilasan]

Shouldn't negatively affect Windows and Mac OS, because StateLocation on these platforms are subdirectories of AppConfigLocation.

Do you have access to Mac and/or Windows to check it?
Could you try adding testing for this change?

[brainrom]

Do you have access to Mac and/or Windows to check it?

I can try build and check on Windows, but have absolutely no idea how to build and check on Mac.

Could you try adding testing for this change?

What kind of testing?

[camilasan]

I can try build and check on Windows, but have absolutely no idea how to build and check on Mac.

On Windows it would already help.

What kind of testing?

Sorry, I mean tests. Unit/code tests.

[brainrom]

On Windows it would already help.

Ok, then I'll manually test it on Windows.

Sorry, I mean tests. Unit/code tests.

Nextcloud currently don't have tests for similar things, not configs, nor migration. What exactly should I test? I can make test for tryMigrateCookieJar() method. Create file in old location, test that it moves to the new location, etc.
Btw, should migration overwrite new file in case where old file and new file presents?

[camilasan]

Sorry I didn't reply before...

I can make test for tryMigrateCookieJar() method. Create file in old location, test that it moves to the new location, etc.

That makes sense.

Btw, should migration overwrite new file in case where old file and new file presents?

You should check if there is a file present before migrating. That is what we do with the config file nextcloud.cfg.