Bug 15341 - Button of mgaupdate notification didn't work (kde)
Summary: Button of mgaupdate notification didn't work (kde)
Status: RESOLVED OLD
Alias: None
Product: Mageia
Classification: Unclassified
Component: RPM Packages (show other bugs)
Version: 5
Hardware: x86_64 Linux
Priority: Normal normal
Target Milestone: ---
Assignee: Mageia tools maintainers
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2015-02-22 11:25 CET by Stephane
Modified: 2018-10-07 17:01 CEST (History)
5 users (show)

See Also:
Source RPM: kdebase4-runtime
CVE:
Status comment:


Attachments
mgaupdate notification in FR (56.63 KB, image/png)
2015-02-22 11:27 CET, Stephane
Details

Description Stephane 2015-02-22 11:25:57 CET
Description of problem:
When there are updates found by mgaupdate it popup a notification.
From this notification there is a button to install updates. This button doesn't call application to launch update like red icon do.

Mageia-5-beta3-x86_64-DVD on VM.

I also put a picture of 2 notifications :
- first notification is the default notification,
- second : button "install" disappear after click on it but not action is triggered.


Reproducible: 

Steps to Reproduce:
Comment 1 Stephane 2015-02-22 11:27:24 CET
Created attachment 5938 [details]
mgaupdate notification in FR

I forgot to mention : I use FR localization.
Comment 2 Manuel Hiebel 2015-02-22 22:40:39 CET
looks kde, thierry I remember it was supposed to be fixed no ?

CC: (none) => balcaen.john, lmenut, neoclust
Assignee: bugsquad => thierry.vignaud
Summary: Button of mgaupdate notification didn't work => Button of mgaupdate notification didn't work (kde)

Comment 3 Thierry Vignaud 2015-02-23 06:22:59 CET
That would be an issue with whatever notification daemon is used by KDE (knotify4), not an mgaonline bug

Assignee: thierry.vignaud => bugsquad
Source RPM: mgaonline => kdebase4-runtime

Comment 4 Rémi Verschelde 2015-04-03 12:36:27 CEST
Here is a patch that fixes the issue for me:


diff --git a/mgaapplet b/mgaapplet
index 4c80055..690a14f 100755
--- a/mgaapplet
+++ b/mgaapplet
@@ -685,7 +685,7 @@ sub setState {
             }
         } elsif ($state eq 'updates') {
             unshift @arr, 'upgrade_distro' if $new_distro;
-            $bubble->add_action('updates', N("Install updates"), sub { installUpdates() });
+            $bubble->add_action('updates', N("Install updates"), launch => \&installUpdates );
         } elsif (member($state, qw(no_enabled_medium no_update_medium))) {
             $bubble->add_action('add_med', N("Add media"), sub { add_media() });
         }


If the workaround is good enough, the same changes should probably be applied for the other "bubble" actions that are "upgrade", "no_more", "add_med".
WDYT Thierry?

CC: (none) => remi

Comment 5 Rémi Verschelde 2015-04-03 12:38:40 CEST
Nevermind my comment 4, I was testing my patch without having restarted mgaapplet, so maybe my patch is completely bogus. Will investigate more.
Comment 6 David Walser 2015-04-03 12:46:13 CEST
Note that the Install button *does* work when the notification first pops up.  It's once it gets saved into the notifications list (after a few seconds) that the button is still there but doesn't do anything.
Comment 7 Rémi Verschelde 2015-04-03 13:30:46 CEST
I've done a bit more debugging, and actually it looks like something times out after a couple of seconds and the button doesn't work anymore afterwards.

- If you click the first popup as soon as you are notified, the button works
- If you wait 3/4 seconds and then click the button (still in the same popup), the button does not work
- If you close or ignore the popup, but instead open the notifications list, it exhibits the same behaviour. If you open the list and click the button, all under less than 3/4 seconds after being first notified by the popup, the button works.
- Similarly, if you wait too long and click on the button in the notifications list, it doesn't work.
Samuel Verschelde 2015-06-06 01:15:14 CEST

Whiteboard: (none) => MGA5TOO FOR_ERRATA

Comment 8 Samuel Verschelde 2015-06-06 01:15:30 CEST
Is this bug likely to hit many users?
Comment 9 Marja Van Waes 2016-05-02 19:08:51 CEST
The same problem was reported in KDE4 with the popup in mgaapplet that says a repository needs to be added, see attachment 7732 [details]
After clicking on "Ajouter" (Add) it disappears and nothing happens.
https://bugs.mageia.org/show_bug.cgi?id=9386#c20

CC: (none) => marja11

Comment 10 Marja Van Waes 2016-05-02 22:11:01 CEST
don't know why this never got assigned to KDE team

Assignee: bugsquad => mageia

Marja Van Waes 2016-05-02 22:11:56 CEST

Version: Cauldron => 5
Whiteboard: MGA5TOO FOR_ERRATA => (none)

Samuel Verschelde 2016-08-25 16:25:11 CEST

Assignee: mageia => kde

Luc Menut 2016-08-25 16:42:53 CEST

CC: lmenut => (none)

Nicolas Lécureuil 2016-12-30 22:52:44 CET

CC: (none) => mageia
Assignee: kde => mageiatools

Comment 11 Marja Van Waes 2018-07-12 12:18:25 CEST
@ Stéphane,

We regret if this issue didn't get fixed in Mageia 5.

Mageia 5 has officially reached its End of Life on December 31st, 2017 https://blog.mageia.org/en/2017/11/07/mageia-5-eol-postponed/
It only continued to get important security updates since then, but non-security bugs have no chance of still getting fixed.

I assume this bug doesn't exist in Mageia 6 or later, because KDE and its tools are entirely different there.

Closing as OLD.

Please reopen this report and change its "Version:" at the top left to "6", if the same bug still exists in Mageia 6.

==> If you didn't reset your password after february 2018, then you'll need to reset it here https://identity.mageia.org/forgot_password to be able to log in and comment in this report.<==
Comment 12 Marja Van Waes 2018-10-07 17:01:42 CEST
(In reply to Marja Van Waes from comment #11)

> 
> Closing as OLD.
> 

Now really closing :-/

Status: NEW => RESOLVED
Resolution: (none) => OLD


Note You need to log in before you can comment on or make changes to this bug.