[regression][clock] alarm still rings once it's been disabled or deleted
Bug #1387231 reported by
Paty Davila
This bug affects 5 people
| Affects | Status | Importance | Assigned to | Milestone | |
|---|---|---|---|---|---|
| Canonical System Image |
Fix Released
|
High
|
Canonical Devices Products | ||
| Ubuntu Clock App |
Fix Released
|
Critical
|
Charles Kerr | ||
| indicator-datetime (Ubuntu) |
Fix Released
|
Critical
|
Charles Kerr | ||
Bug Description
(r135)
Steps to reproduce:
1. Create a single type alarm to ring in the next 5 minutes.
2. Wait for the alarm to ring.
3. After dismissing the alarm, delete it from the clock app.
Actual behaviour:
Once the alarm's been deleted, it still rings!
Expected behaviour:
Once you delete the alarm, this shouldn't ring anymore.
-------
Mako, Image #5 ubuntu rtm,
Steps to reproduce:
1. Create an alarm to ring in the next 5 minutes.
2. Disable the alarm after it is created.
Actual Behaviour:
Alarm still rings!
Expected Behaviour:
Once you disable an alarm, it shouldn't ring anymore.
Related branches
lp://staging/~charlesk/indicator-datetime/lp-1387231-honor-x-canonical-disabled-tag
- Ted Gould (community): Approve
-
Diff: 373 lines (+65/-40)14 files modifieddebian/changelog (+7/-0)
include/datetime/appointment.h (+4/-1)
src/actions-live.cpp (+1/-1)
src/appointment.cpp (+3/-3)
src/engine-eds.cpp (+19/-3)
src/menu.cpp (+4/-4)
src/snap.cpp (+0/-6)
tests/manual (+15/-10)
tests/manual-test-snap.cpp (+1/-1)
tests/test-actions.cpp (+2/-2)
tests/test-alarm-queue.cpp (+2/-2)
tests/test-live-actions.cpp (+2/-2)
tests/test-menus.cpp (+4/-4)
tests/test-snap.cpp (+1/-1)
- Ted Gould (community): Approve
- PS Jenkins bot (community): Approve (continuous-integration)
-
Diff: 312 lines (+53/-30)13 files modifiedinclude/datetime/appointment.h (+4/-1)
src/actions-live.cpp (+1/-1)
src/appointment.cpp (+3/-3)
src/engine-eds.cpp (+19/-3)
src/menu.cpp (+4/-4)
src/snap.cpp (+0/-6)
tests/manual (+10/-0)
tests/manual-test-snap.cpp (+1/-1)
tests/test-actions.cpp (+2/-2)
tests/test-alarm-queue.cpp (+2/-2)
tests/test-live-actions.cpp (+2/-2)
tests/test-menus.cpp (+4/-4)
tests/test-snap.cpp (+1/-1)
Revision history for this message
| Nekhelesh Ramananthan (nik90) wrote | #1 |
| Changed in ubuntu-clock-app: | |
| status: | New → Incomplete |
Revision history for this message
| Michał Sawicz (saviq) wrote | #2 |
| affects: | unity8 (Ubuntu) → indicator-datetime (Ubuntu) |
Revision history for this message
| Paty Davila (dizzypaty) wrote | #3 |
Revision history for this message
| Alexander Sack (asac) wrote | #4 |
Revision history for this message
| Nekhelesh Ramananthan (nik90) wrote | #5 |
| summary: |
- [clock] alarm still rings once it's been deleted + [clock] alarm still rings once it's been disabled |
| Changed in ubuntu-clock-app: | |
| status: | Incomplete → Confirmed |
| Changed in indicator-datetime (Ubuntu): | |
| status: | New → Confirmed |
| Changed in ubuntu-clock-app: | |
| milestone: | none → rtm |
| importance: | Undecided → Critical |
| Changed in ubuntu-clock-app: | |
| assignee: | nobody → Charles Kerr (charlesk) |
| summary: |
- [clock] alarm still rings once it's been disabled + [clock] alarm still rings once it's been disabled or deleted |
| description: | updated |
| summary: |
- [clock] alarm still rings once it's been disabled or deleted + [regression][clock] alarm still rings once it's been disabled or deleted |
| tags: | added: ota-1 rtm14 |
| Changed in indicator-datetime (Ubuntu): | |
| importance: | Undecided → Critical |
| Changed in ubuntu-clock-app: | |
| milestone: | rtm → ota-1 |
| Changed in indicator-datetime (Ubuntu): | |
| assignee: | nobody → Charles Kerr (charlesk) |
| Changed in indicator-datetime (Ubuntu): | |
| status: | Confirmed → In Progress |
| Changed in canonical-devices-system-image: | |
| assignee: | nobody → Canonical Devices Products (canonical-devices-products-team) |
| importance: | Undecided → High |
| milestone: | none → r1 |
| status: | New → Confirmed |
Revision history for this message
| Launchpad Janitor (janitor) wrote | #6 |
| Changed in indicator-datetime (Ubuntu): | |
| status: | In Progress → Fix Released |
Revision history for this message
| Nekhelesh Ramananthan (nik90) wrote | #7 |
| Changed in ubuntu-clock-app: | |
| status: | Confirmed → Fix Released |
| Changed in canonical-devices-system-image: | |
| status: | Confirmed → Fix Released |
| milestone: | ww51-2014 → ww03-2015 |
To post a comment you must log in.
Thank you for taking the time to report the bug. I need some additional information to understand the bug report correctly.
- Which device and phone image do you use? Is this the ubuntu-rtm channel?
- When the alarm rings, do you dismiss the alarm by pressing "ok"? If so, then I am finding it hard to believe that the alarm rings again considering that it is a single type alarm. When does it ring again (in the next minute)?