Bug 352976 - Login notification plugins do not get triggered
Summary: Login notification plugins do not get triggered
Status: REOPENED
Alias: None
Product: frameworks-knotifications
Classification: Frameworks and Libraries
Component: general (show other bugs)
Version: unspecified
Platform: Kubuntu Linux
: NOR normal
Target Milestone: ---
Assignee: Martin Klapetek
URL:
Keywords:
: 404491 410617 (view as bug list)
Depends on:
Blocks:
 
Reported: 2015-09-21 11:26 UTC by Jan Wiele
Modified: 2019-08-06 16:49 UTC (History)
5 users (show)

See Also:
Latest Commit:
Version Fixed In:
Sentry Crash Report:


Attachments
attachment-24068-0.html (1.45 KB, text/html)
2019-08-06 16:49 UTC, Rob Burness
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Jan Wiele 2015-09-21 11:26:48 UTC
Hi,
adding a command to run or playing a sound on login or logout does not work (Notifications, Event source: Plasma Workspace). Nothing happens.

I'm on Kubuntu 15.04 with kubuntu-ppa/backports @ Kde-plasma 5.3.2
Comment 1 Martin Klapetek 2015-09-23 14:52:58 UTC
Thanks for the report

This is fixed in Plasma 5.4, please update.

*** This bug has been marked as a duplicate of bug 335948 ***
Comment 2 hamelg 2015-10-15 16:48:02 UTC
I use Plasma 5.4.2.
This bug is still here, please reopen this issue.

How to reproduce :
System Settings > Notification > Event source: Plasma Workspace > event login 
Tick Run command, put "/usr/bin/touch /tmp/xxx" in the field Run command
Close & reopen your session.
There is no /tmp/xxx file.
Comment 3 Martin Klapetek 2015-10-15 16:52:58 UTC
Ah, I understood the report is about sound.

But you're correct, for login notifications, only sound is considered, currently nothing else gets executed. On logout, everything should work.

The problem is that KNotification is not thread-safe. Until it is, this won't work.
Comment 4 Kai Uwe Broulik 2019-03-31 10:28:04 UTC
*** Bug 404491 has been marked as a duplicate of this bug. ***
Comment 5 Rob Burness 2019-03-31 19:18:26 UTC
Martin,
"The problem is that KNotification is not thread-safe. Until it is, this won't work."

If it will not work, I suggest you either remove the option to log to a file at login or provide a message that the option will not work until KNotification is thread safe.  That way people will understand and won't be reporting bugs or duplicates.
Comment 6 Kai Uwe Broulik 2019-08-06 06:53:08 UTC
*** Bug 410617 has been marked as a duplicate of this bug. ***
Comment 7 Rob Burness 2019-08-06 16:49:43 UTC
Created attachment 121975 [details]
attachment-24068-0.html

Provide bug report, wait 2 months to hear it is duplicate of another bug
which also has not received attention.  This is a reason why people don't
report bugs.  I know everyone is a volunteer.

On Tue, Aug 6, 2019 at 12:53 AM Kai Uwe Broulik <bugzilla_noreply@kde.org>
wrote:

> https://bugs.kde.org/show_bug.cgi?id=352976
>
> Kai Uwe Broulik <kde@privat.broulik.de> changed:
>
>            What    |Removed                     |Added
>
> ----------------------------------------------------------------------------
>                  CC|                            |andreastedile97@gmail.com
>
> --- Comment #6 from Kai Uwe Broulik <kde@privat.broulik.de> ---
> *** Bug 410617 has been marked as a duplicate of this bug. ***
>
> --
> You are receiving this mail because:
> You are on the CC list for the bug.