Clock app alarms ignores the day set by the user
Bug #1319401 reported by
Nekhelesh Ramananthan
This bug affects 3 people
| Affects | Status | Importance | Assigned to | Milestone | |
|---|---|---|---|---|---|
| Ubuntu Clock App |
Fix Released
|
Critical
|
Zsombor Egri | ||
| Ubuntu UI Toolkit |
Fix Released
|
Critical
|
Zsombor Egri | ||
| ubuntu-ui-toolkit (Ubuntu) |
Fix Released
|
Undecided
|
Unassigned | ||
| Vivid |
New
|
Undecided
|
Unassigned | ||
Bug Description
As of Utopic #26, the clock app does not take into account the day set by the user while creating a single type alarm. Instead it always creates the alarm for today. This does not happen with recurring alarms.
Steps to reproduce:
1. Create a single type alarm
2. Set the day to anything other day than today
3. Save alarm
Notice that the saved alarm is set to ring today and not on the day you chose. The alarm time is correct. Only the day is incorrect. There are no errors during the alarm save operation.
On using the SDK sample app which sets the alarm date object with the correct date and time, then the alarm is saved correctly. So it seems that the alarm.daysOfTheWeek property is being silently ignored by the SDK Alarms API.
Related branches
lp://qastaging/~zsombi/ubuntu-ui-toolkit/alarm-dow-fix
- PS Jenkins bot: Approve (continuous-integration)
- Cris Dywan: Approve
- Nekhelesh Ramananthan (community): Approve (device testing)
-
Diff: 105 lines (+61/-5)3 files modifiedmodules/Ubuntu/Components/plugin/ucalarm.cpp (+15/-4)
tests/resources/alarm/Alarms.qml (+1/-1)
tests/unit/tst_alarms/tst_alarms.cpp (+45/-0)
| Changed in ubuntu-ui-toolkit: | |
| assignee: | nobody → Zsombor Egri (zsombi) |
| milestone: | none → june-2014 |
| importance: | Undecided → Critical |
| description: | updated |
Revision history for this message
| Nicholas Skaggs (nskaggs) wrote | #1 |
Revision history for this message
| Nekhelesh Ramananthan (nik90) wrote | #2 |
| Changed in ubuntu-ui-toolkit: | |
| status: | New → Confirmed |
| Changed in ubuntu-ui-toolkit: | |
| status: | Confirmed → In Progress |
Revision history for this message
| PS Jenkins bot (ps-jenkins) wrote | #3 |
| Changed in ubuntu-ui-toolkit: | |
| status: | In Progress → Fix Committed |
| Changed in ubuntu-clock-app: | |
| status: | Confirmed → Fix Committed |
| Changed in ubuntu-ui-toolkit: | |
| status: | Fix Committed → Fix Released |
Revision history for this message
| Launchpad Janitor (janitor) wrote | #4 |
| Changed in ubuntu-ui-toolkit (Ubuntu): | |
| status: | New → Confirmed |
| Changed in ubuntu-ui-toolkit (Ubuntu): | |
| status: | Confirmed → Fix Released |
| Changed in ubuntu-clock-app: | |
| milestone: | alarm-blockers → rtm |
| Changed in ubuntu-clock-app: | |
| assignee: | nobody → Zsombor Egri (zsombi) |
| status: | Fix Committed → Fix Released |
To post a comment you must log in.
nik90, how was this discovered?