Bug 460922

Summary:	Alarm sound plays forever if followed by the second one and not dismissed in the dialog
Product:	[Applications] KClock	Reporter:	wojnilowicz <lukasz.wojnilowicz>
Component:	General	Assignee:	Devin Lin <espidev>
Status:	RESOLVED FIXED
Severity:	normal	CC:	hanyoung
Priority:	NOR
Version First Reported In:	unspecified
Target Milestone:	---
Platform:	Fedora RPMs
OS:	Linux
Latest Commit:		Version Fixed/Implemented In:
Sentry Crash Report:

Description wojnilowicz 2022-10-24 07:10:42 UTC

SUMMARY
***
NOTE: If you are reporting a crash, please try to attach a backtrace with debug symbols.
See https://community.kde.org/Guidelines_and_HOWTOs/Debugging/How_to_create_useful_crash_reports
***


STEPS TO REPRODUCE
1.  click on "Timers"
2.  click on "New Timer"
3.  set "Duration:" to e.g. 2 s
4. click on "Done"
5. click on the new timer
6. click on "Start"
7. wait till the timer finishes (first alarm sound can be heard)
8. press "Return" key
9. wait till the timer finishes (second alarm sound can be heard)
10. press "Discard" on a dialog named "Timer has finished"


OBSERVED RESULT
First alarm is still heard.

EXPECTED RESULT
First alarm should not be heard.

SOFTWARE/OS VERSIONS
Linux/KDE Plasma: 
(available in About System)
KDE Plasma Version: 5.25.5
KDE Frameworks Version: 5.98.0
Qt Version: 5.15.6

ADDITIONAL INFORMATION

Comment 1 Devin Lin 2022-10-24 20:57:46 UTC

> 8. press "Return" key

Nothing happens when I press the "Return" key, is something selected?

Comment 2 wojnilowicz 2022-10-25 09:05:31 UTC

(In reply to Devin Lin from comment #1)
> > 8. press "Return" key
> 
> Nothing happens when I press the "Return" key, is something selected?

Nothing is selected. 
The same issue can be reproduced if instead of pressing "Return" key, you select "Loop Timer" before starting it.

Afterwards I need to kill klockd to stop multiple alarm sounds.

Comment 3 Devin Lin 2022-10-28 01:21:05 UTC

A single timer was able to create multiple notifications, of which only one was closed on dismiss.

Should be fixed with https://invent.kde.org/plasma-mobile/kclock/-/commit/2abc148ec239e0a644e1737a9db43b20817e4040, which will make it to next release.