Bug 436347 - KRunner icons cache folder has confusing name
Summary: KRunner icons cache folder has confusing name
Status: RESOLVED FIXED
Alias: None
Product: krunner
Classification: Plasma
Component: general (show other bugs)
Version: 5.20.90
Platform: Other Linux
: NOR normal
Target Milestone: ---
Assignee: Alexander Lohnau
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2021-04-29 10:26 UTC by Allan Sandfeld
Modified: 2021-04-29 12:29 UTC (History)
1 user (show)

See Also:
Latest Commit:
Version Fixed In: 5.22
Sentry Crash Report:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Allan Sandfeld 2021-04-29 10:26:18 UTC
Found this directory:

carewolf@twilight% ls -l                                                                                                              ~/.cache/plasmashell
total 96
drwxrwxr-x 4 carewolf carewolf  4096 Mai 29  2017  attica
drwxrwxr-x 4 carewolf carewolf  4096 Mai 29  2017  knewstuff
drwxrwxr-x 2 carewolf carewolf  4096 Mai 29  2017  KRunner-Favicons-firefox-default
drwxrwxr-x 2 carewolf carewolf  4096 Feb  8 16:15 'KRunner-Favicons-SELECT icon FROM icons WHERE url = :url LIMIT 1;'
drwxrwxr-x 2 carewolf carewolf 77824 Apr 15 09:55  qmlcache

I don't think SQL queries were meant to be in the file-system names.

The creation date fits my 5.20.90 build, so it might already be fixed.
Comment 1 Allan Sandfeld 2021-04-29 10:29:26 UTC
It appears it is also present in more recent versions. It has just moved to .cache/krunner
Comment 2 Alexander Lohnau 2021-04-29 12:20:39 UTC
That is a compile time known string, but choosing the name of the browser will be better ;)
Comment 3 Alexander Lohnau 2021-04-29 12:29:36 UTC
Git commit 094100187d72771b4992a40a499cbff2b770e9de by Alexander Lohnau.
Committed on 29/04/2021 at 12:28.
Pushed by alex into branch 'master'.

bookmarksrunner: Choose less confusing folder name for cache
FIXED-IN: 5.22

M  +1    -1    runners/bookmarks/faviconfromblob.cpp

https://invent.kde.org/plasma/plasma-workspace/commit/094100187d72771b4992a40a499cbff2b770e9de