Bug 449583

Summary: Discover becomes unresponsive when selecting some apps - Terminal outputs ...ScrollView.qml:98:27: QML ScrollBar: Binding loop detected for property "visible"
Product: [Applications] Discover Reporter: guimarcalsilva
Component: discoverAssignee: Dan Leinir Turthra Jensen <leinir>
Status: RESOLVED FIXED    
Severity: crash CC: aleixpol, c0d3g33k, nate
Priority: HI    
Version: unspecified   
Target Milestone: ---   
Platform: Other   
OS: Linux   
Latest Commit: Version Fixed In: 5.24.1
Attachments: Terminal Output

Description guimarcalsilva 2022-02-04 02:14:51 UTC
SUMMARY

STEPS TO REPRODUCE
1.  Select the Ardour Flatpak app or Albion Online Flatpak (I'm on KDE Neon Unstable, results might differ with other distros. If you can't reproduce, click on many entries and check if Discover locks up.)

PS: I'm not sure if it only happens with Flatpaks or not. For now, I could only reproduce the issue with Flatpaks.

OBSERVED RESULT

Discover becomes unresponsive. It doesn't crash, and you can still see the gear animation loading images, but it doesn't allow for any interaction with the app anymore.

When it freezes, it spams the terminal log with: 

file:///usr/lib/x86_64-linux-gnu/qt5/qml/QtQuick/Controls.2/org.kde.desktop/ScrollView.qml:98:27: QML ScrollBar: Binding loop detected for property "visible"

I attached the output.

SOFTWARE/OS VERSIONS
Operating System: KDE neon Unstable Edition
KDE Plasma Version: 5.24.80
KDE Frameworks Version: 5.91.0
Qt Version: 5.15.3
Kernel Version: 5.13.0-28-generic (64-bit)
Graphics Platform: Wayland
Processors: 6 × Intel® Core™ i5-9400F CPU @ 2.90GHz
Memory: 7,6 GiB of RAM
Graphics Processor: Radeon RX 570 Series
Comment 1 guimarcalsilva 2022-02-04 02:17:49 UTC
Created attachment 146243 [details]
Terminal Output
Comment 2 Code Geek 2022-02-04 14:23:15 UTC
Can confirm.  Attempting to load the detail page for some flatpaks Discover becomes completely unresponsive.  Checking the logs shows the exact same error message repeating multiple times per second.

OS:  KDE neon User edition
KDE Plasma Version: 5.23.5
KDE Frameworks Version: 5.90.0
QT Version: 5.13.3
Graphics Platform: X11

Forcibly closing Discover by clicking the X at upper right corner of the window and terminating it causes the error messages to cease.

I can load Discover immediately afterward and it behaves normally *until* I select certain Flatpak detail pages as before.  Interestingly, a few other flatpaks I have installed do not cause the bug - Discover remains responsive.

I guess this means that something could be wrong with the detail page for some flatpaks that results in a parsing error or something.

I did some random clicking around selecting packages of various types (deb packages, snaps, flatpaks).  So far, no debs or snaps have triggered the bug.  Some, but not all, flatpaks can cause the bug.  Flatpak does not have to be installed, and not all installed flatpaks trigger this bug.

My (very short) list of flatpaks that trigger the bug:

Calibre
Flatseal
Turtleblocks

Flatpaks that do not trigger bug:  

Bottles
Minigalaxy

Finally, for many apps of any type, the 'binding loop' error message appears, but just once, and Discover remains responsive.  So I'm not sure the error message has anything to do with the bug.  The repeating message may just be caused by Discover trying to unsuccessfully load the page and retrying forever.
Comment 3 Nate Graham 2022-02-04 16:38:38 UTC

*** This bug has been marked as a duplicate of bug 448784 ***
Comment 4 Nate Graham 2022-02-05 03:24:15 UTC
Git commit 14c9ef14f21b163df12fb77e704694c883851915 by Nate Graham, on behalf of Aleix Pol.
Committed on 05/02/2022 at 03:20.
Pushed by ngraham into branch 'master'.

ApplicationPage: Fix infinite resize loop on screenshoots

We were fitting it in height and it was freaking out whenever the
horizontal scrollbar appeared.

Now it depends on the ScrollView size instead of the ListView and so the
loop is broken.

M  +1    -0    discover/qml/ApplicationPage.qml
M  +4    -5    discover/qml/ApplicationScreenshots.qml

https://invent.kde.org/plasma/discover/commit/14c9ef14f21b163df12fb77e704694c883851915
Comment 5 Nate Graham 2022-02-05 03:31:28 UTC
Git commit 413ef017a7fb0b5836c7fb20c99dd9104c0dbde0 by Nate Graham, on behalf of Aleix Pol.
Committed on 05/02/2022 at 03:31.
Pushed by ngraham into branch 'Plasma/5.24'.

ApplicationPage: Fix infinite resize loop on screenshoots

We were fitting it in height and it was freaking out whenever the
horizontal scrollbar appeared.

Now it depends on the ScrollView size instead of the ListView and so the
loop is broken.
(cherry picked from commit 14c9ef14f21b163df12fb77e704694c883851915)

M  +1    -0    discover/qml/ApplicationPage.qml
M  +5    -5    discover/qml/ApplicationScreenshots.qml

https://invent.kde.org/plasma/discover/commit/413ef017a7fb0b5836c7fb20c99dd9104c0dbde0