When I disable "Show filetransfers and jobs" in the notification configuration, upon plasma start there is a spinner that never ends in the notification widget. Reproducible: Always Steps to Reproduce: 1.disable filetransfers and jobs in the notifiaction widget configuration 2.kquitapp plasma-desktop 3.plasma-desktop Actual Results: Never ending spinner with no notification Expected Results: Nothing (except filetransfers not being shown) enabling and disabling filetransfers and jobs in the widget configuration stops the spinner. Might be relevant : in the console output, this appears while changing the configuration : "file:///usr/share/apps/plasma/plasmoids/org.kde.notifications/contents/ui/Jobs.qml:28:15: Unable to assign undefined value file:///usr/share/apps/plasma/plasmoids/org.kde.notifications/contents/ui/Jobs.qml:29:16: Unable to assign undefined value file:///usr/share/apps/plasma/plasmoids/org.kde.notifications/contents/ui/NotificationIcon.qml:47: TypeError: Result of expression 'jobs' [null] is not an object. file:///usr/share/apps/plasma/plasmoids/org.kde.notifications/contents/ui/NotificationIcon.qml:66: TypeError: Result of expression 'jobs' [null] is not an object. file:///usr/share/apps/plasma/plasmoids/org.kde.notifications/contents/ui/Notifications.qml:160: Unable to assign null to bool "
Happens in 4.10.1...
Still happens in 4.10.4...
I tried to debug this, but was not successful, needs someone understanding QML. The following patch disables the animation, so that it does not consume CPU cycles. Now it is no longer required to change the settings on each start. diff --git a/plasma/generic/applets/notifications/contents/ui/NotificationIcon.qml b/plasma/generic/applets/notifications/contents/ui/NotificationIcon.qml index 819d39d..93e70da 100644 --- a/plasma/generic/applets/notifications/contents/ui/NotificationIcon.qml +++ b/plasma/generic/applets/notifications/contents/ui/NotificationIcon.qml @@ -64,7 +64,7 @@ Item { PlasmaComponents.BusyIndicator { anchors.fill: parent visible: jobs.count > 0 - running: visible + running: false } Column {
Created attachment 81780 [details] This is a patch which removes the bug
Patch from comment #4 works good, no regressions seen. Could you please commit it to 4.11 branch? If you do not want to commit, please state your full name, and I will do it. Could you also look at related bug 315847?
Where can I commit this patch?
Błażej, you need a developer account to get commit access to the KDE repositories. Please see http://techbase.kde.org/Contribute/Get_a_Contributor_Account
OK, thanks Christoph. But you can commit the patch to 4.11 branch.
Git commit ed7c2e824303e0c971100c2e2a9c6362129bf409 by Christoph Feck, on behalf of Błażej Szczygieł. Committed on 27/08/2013 at 12:13. Pushed by cfeck into branch 'KDE/4.11'. Fix neverending spinner when jobs are disabled FIXED-IN: 4.11.1 Reviewed by Christoph Feck M +1 -1 plasma/generic/applets/notifications/contents/ui/NotificationIcon.qml http://commits.kde.org/kde-workspace/ed7c2e824303e0c971100c2e2a9c6362129bf409
*** Bug 324583 has been marked as a duplicate of this bug. ***