Bug 407835 - Log to file option does not work for login and logout notifications
Summary: Log to file option does not work for login and logout notifications
Status: CONFIRMED
Alias: None
Product: frameworks-knotifications
Classification: Frameworks and Libraries
Component: general (show other bugs)
Version: unspecified
Platform: Neon Linux
: VLO minor
Target Milestone: ---
Assignee: kdelibs bugs
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2019-05-22 17:17 UTC by blue_bullet
Modified: 2021-08-09 16:53 UTC (History)
3 users (show)

See Also:
Latest Commit:
Version Fixed In:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description blue_bullet 2019-05-22 17:17:44 UTC
SUMMARY
Log to file in login and logout produces no output

STEPS TO REPRODUCE
1. system-settings>notifications>Plasma Workspace>Configure Events
2. login checkbox log to a file ~/log/login.log do same for logout

OBSERVED RESULT
3. logout and login>no output to files.  Files exist with 1 line manually entered by me.  


EXPECTED RESULT
logged entry to each file upon login and logout


SOFTWARE/OS VERSIONS
Windows: 
macOS: 
Linux/KDE Plasma: 5.15.90
(available in About System)
KDE Plasma Version: 5.15.90
KDE Frameworks Version: 5.59.0
Qt Version: 5.12.0

ADDITIONAL INFORMATION
Comment 1 blue_bullet 2019-05-22 20:26:22 UTC
Linux: KDE neon Testing Edition
Comment 2 blue_bullet 2019-05-22 20:54:35 UTC
correction: logout>log to a file and play a sound do work.  login>log to a file does not.  If last comments to bug # 352976 still apply I suggest you remove the log to a file option on login, otherwise please make it work.
Comment 3 blue_bullet 2019-05-22 21:01:34 UTC
correction: logout>log to a file and play a sound do work.  login>log to a file does not.  If last comments to bug # 352976 still apply I suggest you remove the log to a file option on login, otherwise please make it work.
Comment 4 Kai Uwe Broulik 2019-05-23 07:31:00 UTC
The login sound isn't played through KNotification but using Phonon in a thread to avoid blocking startup on loading multimedia plugins, which is why the other actions don't work.
Comment 5 blue_bullet 2019-05-23 15:51:20 UTC
Kai,

Why offer options when it is known in advance they will not work?  That defies logic and lessens product quality.  I thought this testing period is being done to correct errors.  Seems like it would be a simple matter to remove the options that do not work.  Leaving the options in may lead others to file bug reports.
Comment 6 Kai Uwe Broulik 2019-05-23 17:05:55 UTC
I was merely stating what is going on for anyone who wants to pick up this bug report. Having the startup sound not block the login was apparently deemed more important than logging something to disk when that happens, I'm afraid. This bug can probably be fixed but is very low priority.
Comment 7 blue_bullet 2019-05-23 17:13:20 UTC
(In reply to Kai Uwe Broulik from comment #6)
> I was merely stating what is going on for anyone who wants to pick up this
> bug report. Having the startup sound not block the login was apparently
> deemed more important than logging something to disk when that happens, I'm
> afraid. This bug can probably be fixed but is very low priority.

Thank you for the clarification.  I understand the low priority.  Just think the
option should be removed until it works or at least have a note posted saying it may not work.