Bug 482349 - Empty "Pick Your Installation Option" popup for KNS items with multiple files
Summary: Empty "Pick Your Installation Option" popup for KNS items with multiple files
Status: RESOLVED FIXED
Alias: None
Product: frameworks-knewstuff
Classification: Frameworks and Libraries
Component: general (other bugs)
Version First Reported In: 6.0.0
Platform: Compiled Sources Linux
: HI major
Target Milestone: ---
Assignee: Dan Leinir Turthra Jensen
URL:
Keywords: qt6
: 482217 482532 482832 482879 486532 (view as bug list)
Depends on:
Blocks:
 
Reported: 2024-03-04 00:30 UTC by Luis Bocanegra
Modified: 2024-05-04 02:11 UTC (History)
13 users (show)

See Also:
Latest Commit: https://invent.kde.org/frameworks/knewstuff/-/commit/49f2037ac22fcb430fadd5d7b29bd8af234573a4
Version Fixed/Implemented In: Frameworks 6.3
Sentry Crash Report:

Attachments
Empty Pick install option (199.26 KB, image/png)
2024-03-04 00:30 UTC, Luis Bocanegra 		Details
Detailed view shows list but wont install (187.74 KB, image/png)
2024-03-04 00:33 UTC, Luis Bocanegra 		Details
after update to frameworks 6.1 (1.07 MB, image/png)
2024-04-14 11:41 UTC, Patrick Silva 		Details
View All Add an attachment

Note You need to log in before you can comment on or make changes to this bug.
Description Luis Bocanegra 2024-03-04 00:30:52 UTC 
Created attachment 166377 [details]
Empty Pick install option

SUMMARY
Installing widget from Panel > Add Widgets fails if it has multiple versions and Discover installs the first version (older?) instead
Most if not all of my widgets have multiple versions released, I have them correctly versioned so selecting the latest is always obvious to the user. This worked fine on Plasma 5 but doesn't in Plasma 6. 

Also, installing from Discover it installs the wrong version, for example https://store.kde.org/p/2130967 installs the first version instead of the latest (doesn't let user choose which) so I am getting bug reports of users unable to install or having bugs I already fixed in newer versions.

STEPS TO REPRODUCE
1. Right click on panel > Add Widgets
2. Search for "Panel Colorizer"
3. Try to install it from main widget grid
4. Try opening the widget description and installing from top right button

OBSERVED RESULT
Main widgets screen shows empty "Pick Your Installation Option"
Widget description view actually shows the options but can't install anything (no errors on screen)
Other widgets also show a empty "Pick Your Installation Option" but still install because there is only one version??

EXPECTED RESULT
Version list should appear while on widget compact view
Widget should install when picking a version

ERROR WHEN THIS HAPPENS
When trying to install any version from the widget description view journalctl shows this
Mar 03 18:06:15 archlinux plasmashell[1379513]: qrc:/qt/qml/org/kde/newstuff/EntryDetails.qml:87: ReferenceError: entryId is not defined

SOFTWARE/OS VERSIONS
Operating System: Arch Linux 
KDE Plasma Version: 6.0.80
KDE Frameworks Version: 6.0.0
Qt Version: 6.7.0
Graphics Platform: Wayland

ADDITIONAL INFORMATION

Due to this I have looked at other widgets and they all have only a single file published, should I do that instead for now? If so, just uploading the file without a version in the filename and replacing it with newer version and setting the version fields to the updated one is enough or should I delete the previous first?

Bugzilla Found those bug reports but both are very old 
https://bugs.kde.org/show_bug.cgi?id=293047  NOT A BUG
https://bugs.kde.org/show_bug.cgi?id=395720  FIXED
Comment 1 Luis Bocanegra 2024-03-04 00:33:33 UTC 
Created attachment 166378 [details]
Detailed view shows list but wont install
Comment 2 Doug 2024-03-04 05:21:59 UTC 
Can reproduce.

Operating System: KDE neon Testing Edition
KDE Plasma Version: 6.0.0
KDE Frameworks Version: 6.0.0
Qt Version: 6.6.2
Kernel Version: 6.5.0-21-generic (64-bit)
Graphics Platform: Wayland
Processors: 12 × Intel® Core™ i7-9750H CPU @ 2.60GHz
Memory: 15.4 GiB of RAM
Graphics Processor: AMD Radeon Pro WX 3200 Series
Comment 3 Luis Bocanegra 2024-03-04 05:36:36 UTC 
I have deleted older versions in one of my other widgets https://www.pling.com/p/2128047/ and the detailed view now correctly installs the widget without no "Pick Your Installation Option" dialog, but the grid view still shows the empty "Pick Your Installation Option"
Comment 4 Nate Graham 2024-03-05 22:30:22 UTC 
Unfortunately it looks like you're reporting multiple bugs in one bug report, which is a problem. Let's use this one for the empty "Pick Your Installation Option" bug, and can I ask you to submit new separate bug reports for other bugs?

Can reproduce that bug.
Comment 5 Luis Bocanegra 2024-03-06 05:39:59 UTC 
Should I also submit separate one for the widget not installing from Panel > Add Widgets > detailed view?

Will make one for Discover problem.
Comment 6 Nate Graham 2024-03-06 06:00:57 UTC 
Yes please.
Comment 7 Nate Graham 2024-03-06 19:29:46 UTC 
*** Bug 482532 has been marked as a duplicate of this bug. ***
Comment 8 Nate Graham 2024-03-07 16:01:47 UTC 
Git commit 49f2037ac22fcb430fadd5d7b29bd8af234573a4 by Nate Graham, on behalf of Akseli Lahtinen.
Committed on 07/03/2024 at 16:01.
Pushed by ngraham into branch 'master'.

Fix link list dialog for installation button

In gridview, installation buttons showed "Install..." for every item,
even if the item had only single download link. This fixes it
by showing correct button.

The problem was using `entry` directly instead of `model`. 
FIXED-IN: 6.1

M  +4    -4    src/qtquick/qml/private/entrygriddelegates/TileDelegate.qml

https://invent.kde.org/frameworks/knewstuff/-/commit/49f2037ac22fcb430fadd5d7b29bd8af234573a4
Comment 9 Nate Graham 2024-03-10 04:58:27 UTC 
*** Bug 482832 has been marked as a duplicate of this bug. ***
Comment 10 Nate Graham 2024-03-10 22:45:49 UTC 
*** Bug 482879 has been marked as a duplicate of this bug. ***
Comment 11 Nate Graham 2024-03-11 02:34:15 UTC 
*** Bug 482217 has been marked as a duplicate of this bug. ***
Comment 12 Patrick Silva 2024-04-14 11:41:04 UTC 
Created attachment 168509 [details]
after update to frameworks 6.1

This bug persists. Please see the attached screenshot after the provided steps on Arch Linux running frameworks 6.1.

Operating System: Arch Linux 
KDE Plasma Version: 6.0.3
KDE Frameworks Version: 6.1.0
Qt Version: 6.7.0
Graphics Platform: Wayland
Comment 13 Nate Graham 2024-04-14 13:58:50 UTC 
I can't reproduce that issue myself anymore, and judging from the screenshot, there are now additional glitches. I suspect that Qt 6.7 has regressed this, so please submit a new bug report.
Comment 14 Shayne Hartford 2024-04-17 03:26:00 UTC 
I have also encountered this exact same issue with the exact same widget (Panel Spacer Extended)  
I vote we don't make a new issue for the same problem, this one is already the one that shows up first when searching.

Operating System: CachyOS Linux 
KDE Plasma Version: 6.0.3
KDE Frameworks Version: 6.1.0
Qt Version: 6.7.0
Kernel Version: 6.8.6-2-cachyos (64-bit)
Graphics Platform: Wayland
Processors: 32 × AMD Ryzen Threadripper 1950X 16-Core Processor
Memory: 31.2 GiB of RAM
Graphics Processor: Mesa Intel® Arc
Manufacturer: Micro-Star International Co., Ltd.
Product Name: MS-7B09
System Version: 1.0
Comment 15 Nate Graham 2024-04-26 17:08:18 UTC 
There does appear to be a Qt 6.7 regression here, sigh.
Comment 16 Akseli Lahtinen 2024-04-29 13:28:25 UTC 
I am unable to reproduce this but I am on Qt 6.7 using plasma dev 

Operating System: Fedora Linux 40
KDE Plasma Version: 6.0.80
KDE Frameworks Version: 6.2.0
Qt Version: 6.7.0
Kernel Version: 6.8.7-300.fc40.x86_64 (64-bit)
Graphics Platform: Wayland
Processors: 12 × AMD Ryzen 5 3600 6-Core Processor
Memory: 15.5 GiB of RAM
Graphics Processor: AMD Radeon RX 6600
Comment 17 Akseli Lahtinen 2024-04-29 13:29:33 UTC 
Oh nevermind, I can, but the bug seems to be that the downloadItemsSheet is not aligned properly. Will investigate further.
Comment 18 Bug Janitor Service 2024-04-29 14:53:32 UTC 
A possibly relevant merge request was started @ https://invent.kde.org/frameworks/knewstuff/-/merge_requests/312
Comment 19 Nate Graham 2024-05-03 17:09:12 UTC 
Fixed with one of the commits in https://invent.kde.org/frameworks/knewstuff/-/merge_requests/314, made by Akseli Lahtinen and Ivan Tkachenko! The fix lands in Plasma 6.0.3 unfortunately, not the upcoming 6.0.2.
Comment 20 Justin Zobel 2024-05-04 02:11:00 UTC 
*** Bug 486532 has been marked as a duplicate of this bug. ***