Bug 436347

Summary:	KRunner icons cache folder has confusing name
Product:	[Plasma] krunner	Reporter:	Allan Sandfeld <kde>
Component:	general	Assignee:	Alexander Lohnau <alexander.lohnau>
Status:	RESOLVED FIXED
Severity:	normal	CC:	plasma-bugs-null
Priority:	NOR
Version First Reported In:	5.20.90
Target Milestone:	---
Platform:	Other
OS:	Linux
Latest Commit:	https://invent.kde.org/plasma/plasma-workspace/commit/094100187d72771b4992a40a499cbff2b770e9de	Version Fixed/Implemented In:	5.22
Sentry Crash Report:

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