Bug 433119

Summary: New Kickoff doesn't refresh the list of applications to account for removals/additions when 'Applications Updated.'
Product: [Plasma] plasmashell Reporter: The Feren OS Dev <ferenosdev>
Component: Application Launcher (Kickoff) widgetAssignee: David Edmundson <kde>
Status: RESOLVED NOT A BUG    
Severity: normal CC: ddrs, mikel5764, nate, plasma-bugs-null, this.is.reid
Priority: VHI Keywords: regression
Version First Reported In: 5.21.0   
Target Milestone: 1.0   
Platform: Other   
OS: Linux   
Latest Commit: Version Fixed/Implemented In:
Sentry Crash Report:

Description The Feren OS Dev 2021-02-17 21:56:30 UTC 
SUMMARY
Basically, I was working on a fork of Kickoff's new redesign when I noticed it didn't remove the "Facebook" icon I had just removed via a file manager, and then later didn't add it back in when I restored said icon via a file manager.

Confused, I tried doing it in official Kickoff and yielded the exact same result there.


STEPS TO REPRODUCE
1. Get a .desktop file in ~/.local/share/applications or something that Kickoff registers as being an application in a category
2. Open up Kickoff (in my testing I used plasmoidwindowed to keep the Plasmoid visible)
3. Delete the .desktop file

alternative 3. Restore the .desktop file after having deleted it
3. 

OBSERVED RESULT
The menu doesn't automatically add or remove the icon depending on the context - a category change and back is necessary

EXPECTED RESULT
The menu automatically adds or removes the icon depending on the context immediately upon timing out the "Applications Updated." label (aka when it normally refreshes)

SOFTWARE/OS VERSIONS
Windows: 
macOS: 
Linux/KDE Plasma: KDE neon 5.21.0
(available in About System)
KDE Plasma Version: 5.21.0

ADDITIONAL INFORMATION
Comment 1 Nate Graham 2021-02-17 21:57:42 UTC 
Works for me, I just did this yesterday and it worked.
Comment 2 The Feren OS Dev 2021-02-17 22:55:32 UTC 
Managed to produce it again but this time using a Terminal command to remove the icon for me while I had Kickoff open from a Plasma panel, and same with the Terminal adding the icon back for me as well.
Comment 3 The Feren OS Dev 2021-02-17 23:27:10 UTC 
...and even when trying to reinstall plasma-desktop plasma-desktop-data plasma-workspace plasma-widgets-addons it's still occuring, so I don't think it's something I did since that would've been overridden by the reinstalls.
Comment 4 Nate Graham 2021-02-18 04:41:10 UTC 
*** Bug 433130 has been marked as a duplicate of this bug. ***
Comment 5 Reiddragon 2021-02-18 10:47:50 UTC 
I have been able to reproduce this, also finding Bug 433151 in the process
Comment 6 The Feren OS Dev 2021-02-18 15:19:39 UTC 
Demonstrated the bug in this video:
https://www.youtube.com/watch?v=Wl5gzc2wVd4
Comment 7 Nate Graham 2021-02-18 15:39:40 UTC 
*** Bug 433151 has been marked as a duplicate of this bug. ***
Comment 8 David Edmundson 2021-04-18 13:59:28 UTC 
Investigated, and our code seems ok

One needs to call update-desktop-database after changing the layout.
or use xdg-desktop-menu install / uninstall for convenient installation

I could not reproduce the bug with those done correctly.
Comment 9 Nate Graham 2021-04-18 15:22:41 UTC 
It's working fine for me too FWIW.
Comment 10 David Edmundson 2021-04-18 15:31:34 UTC 
In the original video the author didn't make this menu update call, hence the bug being invalid.