Bug 409273

Summary: Cache folder with account name ends up in home
Product: [Applications] Ruqola Reporter: Christian Dywan <cdywan>
Component: backendAssignee: Laurent Montel <montel>
Status: RESOLVED FIXED    
Severity: normal CC: timo.jyrinki
Priority: NOR    
Version: unspecified   
Target Milestone: ---   
Platform: Other   
OS: Linux   
Latest Commit: Version Fixed In:
Sentry Crash Report:

Description Christian Dywan 2019-06-28 09:40:07 UTC
A folder called ~/$ACCOUNTNAME/recordings is created after logging in. I wouldn't mind the empty folder so much, but this should probably go to $XDG_CACHE_HOME/Ruqola/$ACCOUNTNAME (~/.cache/Ruqola/$ACCOUNTNAME).
Comment 1 Christian Dywan 2019-06-28 10:12:00 UTC
Investigation revealed that $XDG_PICTURES_DIR and $XDG_VIDEOS_DIR are set to $HOME and this is where Ruqola wants to store files. And `Video` and `Picture` paths seem to be distinguished from cached files, so perhaps I need to reconsider my expectations here.

I would still expect Ruqola to use a parent folder called Ruqola, though. Otherwise a normal user has no way of associating this unexpected folder with the use of the client.
Comment 2 Laurent Montel 2019-06-28 10:17:02 UTC
it's a problem from $XDG_PICTURES_DIR and $XDG_VIDEOS_DIR for your distro as usual we use Picture and Video as folders.
You will have the same problem for each apps which use this two environments variable.

I think it's a problem as your distro doesn't define this variable.

After adding ruqola as toplevel why not but too bad that these variables are not defined.
Comment 3 Christian Dywan 2019-06-28 10:22:09 UTC
(In reply to Laurent Montel from comment #2)
> it's a problem from $XDG_PICTURES_DIR and $XDG_VIDEOS_DIR for your distro as
> usual we use Picture and Video as folders.
> You will have the same problem for each apps which use this two environments
> variable.
> 
> I think it's a problem as your distro doesn't define this variable.

Right. Investigating distro defaults is beyond Ruqola. This just caused my initial confusion. It wasn't previously available for SUSE, so teething problems are to be expected :-D

> After adding ruqola as toplevel why not but too bad that these variables are
> not defined.

Imho it would be best in all cases to make it clear what app saved these photos and videos. This is the first app I'm using that doesn't use its name at all here.
Comment 4 Timo Jyrinki 2019-06-28 10:44:51 UTC
(In reply to Christian Dywan from comment #1)
> Investigation revealed that $XDG_PICTURES_DIR and $XDG_VIDEOS_DIR are set to
> $HOME

This is likely just your specific settings, I'm on same distribution and the defaults are to my language specific Videos / Pictures folders.

If you have removed those directories while they were empty, they have been automatically modified to point to just $HOME in your .config/user-dirs.dirs - but the defaults are eg $HOME/Videos and $HOME/Pictures.
Comment 5 Laurent Montel 2019-06-28 12:00:29 UTC
Git commit 672de1657ee0b4f7b1e3caf9b688244b4ca14c41 by Laurent Montel.
Committed on 28/06/2019 at 11:17.
Pushed by mlaurent into branch 'master'.

Fix Bug 409273 - Cache folder with account name ends up in home

Add Ruqola as top level name as discussed on bug report

M  +1    -1    src/ruqolacore/managerdatapaths.cpp

https://commits.kde.org/ruqola/672de1657ee0b4f7b1e3caf9b688244b4ca14c41