Ubuntu
ubuntu-ui-toolkit package

Recurring alarm can't be set for sunday

  1. Vivid (15.04)
  2. Bug #1285958
Bug #1285958 reported by Nicholas Skaggs
20
This bug affects 3 people
Affects Status Importance Assigned to Milestone
Ubuntu Clock App
Invalid
High
Renato Araujo Oliveira Filho
Ubuntu Clock App 1.8 "14.04-month-4"
Ubuntu UI Toolkit
Fix Released
High
Zsombor Egri
qtorganizer5-eds (Ubuntu)
Fix Released
High
Renato Araujo Oliveira Filho
ubuntu-ui-toolkit (Ubuntu)
Fix Released
Undecided
Unassigned
Trusty
New
Undecided
Unassigned
Vivid
New
Undecided
Unassigned

Bug Description

Recurring alarm can be set on Sunday. It works on all other days properly.

Steps to reproduce the bug:
1. Open the alarms tab
2. Enable the repeat switch to set the alarm type as recurring
3. Choose Sunday and also a couple of other days in the add alarms day dialog
4. Save alarm

What happens:
In the alarm page, it does not not show Sunday. It shows every other day you may have chosen in the add alarms day dialog.

What should happen:
The alarm page lists all the chosen days of the recurring alarm.

Facts:
Currently the clock app is at rev 362. I branches the clock app at rev 243 to check if this was caused by any recent commits in the clock app. However this bug was reproducible even at rev 243. I also repeated the steps in the sample alarm app provided by the SDK at http://bazaar.launchpad.net/~ubuntu-sdk-team/ubuntu-ui-toolkit/trunk/files/head:/tests/resources/alarm/ and I can see reproduce the alarm indicating that either the SDK or EDS broke things.

Update: On further testing, it has been found that EDS (qtorganizer5-eds) is causing this bug.

See original description

Related branches

lp://staging/~renatofilho/qtorganizer5-eds/fix-1285958
Bill Filler (community): Approve
PS Jenkins bot: Approve (continuous-integration)
Diff: 127 lines (+97/-2)
2 files modified
qorganizer/qorganizer-eds-engine.cpp (+14/-2)
tests/unittest/recurrence-test.cpp (+83/-0)
lp://staging/~zsombi/ubuntu-ui-toolkit/alarm-fetch-fix
PS Jenkins bot: Approve (continuous-integration)
Cris Dywan: Approve
Diff: 630 lines (+271/-73)
7 files modified
modules/Ubuntu/Components/plugin/adapters/alarmsadapter_organizer.cpp (+56/-23)
modules/Ubuntu/Components/plugin/adapters/alarmsadapter_p.h (+2/-0)
modules/Ubuntu/Components/plugin/alarmmanager_p.h (+2/-0)
modules/Ubuntu/Components/plugin/ucalarm.cpp (+33/-27)
modules/Ubuntu/Components/plugin/ucalarm_p.h (+1/-1)
tests/resources/alarm/Alarms.qml (+6/-2)
tests/unit/tst_alarms/tst_alarms.cpp (+171/-20)
PS Jenkins bot: Approve (continuous-integration)
Cris Dywan: Needs Fixing
Diff: 906 lines (+452/-91)
12 files modified
components.api (+2/-0)
modules/Ubuntu/Components/plugin/adapters/alarmsadapter_organizer.cpp (+56/-23)
modules/Ubuntu/Components/plugin/adapters/alarmsadapter_p.h (+2/-0)
modules/Ubuntu/Components/plugin/alarmmanager_p.h (+2/-0)
modules/Ubuntu/Components/plugin/ucalarm.cpp (+33/-27)
modules/Ubuntu/Components/plugin/ucalarm_p.h (+1/-1)
modules/Ubuntu/Test/UbuntuTestCase.qml (+69/-1)
modules/Ubuntu/Test/deployment.pri (+6/-1)
tests/resources/alarm/Alarms.qml (+6/-2)
tests/unit/runtest.sh (+1/-1)
tests/unit/tst_alarms/tst_alarms.cpp (+171/-20)
tests/unit_x11/tst_test/tst_ubuntutestcase.qml (+103/-15)
Nicholas Skaggs (nskaggs)
summary: - Reccuring Alarm drops the last day
+ Reccuring Alarm can't be set for sunday
description: updated
Leo Arias (elopio)
tags: added: clock elopio qa-new
Changed in ubuntu-clock-app:
status: New → Confirmed
Revision history for this message
Nekhelesh Ramananthan (nik90) wrote : Re: Reccuring Alarm can't be set for sunday

elopio, did you just add your name as a tag :P

tags: added: alarms dogfooding-blocker
Changed in ubuntu-clock-app:
importance: Undecided → High
milestone: none → 1.8
Nekhelesh Ramananthan (nik90)
Changed in ubuntu-clock-app:
assignee: nobody → Zsombor Egri (zsombi)
Changed in ubuntu-ui-toolkit:
status: New → Confirmed
Nekhelesh Ramananthan (nik90)
description: updated
Revision history for this message
Leo Arias (elopio) wrote :

I did :)
That's how I track the bugs that are important for me. And qa-new is my team, so that's how I tell the others that there's something important to track.

Nekhelesh Ramananthan (nik90)
affects: ubuntu-ui-toolkit → qtorganizer5-eds (Ubuntu)
Changed in ubuntu-clock-app:
assignee: Zsombor Egri (zsombi) → Renato Araujo Oliveira Filho (renatofilho)
description: updated
Revision history for this message
Nicholas Skaggs (nskaggs) wrote :

@boiko, are you able to have a look at this issue? I'm not sure renato is around to fix.

Bill Filler (bfiller)
Changed in qtorganizer5-eds (Ubuntu):
assignee: nobody → Renato Araujo Oliveira Filho (renatofilho)
importance: Undecided → High
Revision history for this message
Renato Araujo Oliveira Filho (renatofilho) wrote :

Me and zsombi are working to fix this bug on the sdk (alarm API).

Zsombor Egri (zsombi)
Changed in ubuntu-ui-toolkit:
assignee: nobody → Zsombor Egri (zsombi)
Zsombor Egri (zsombi)
Changed in ubuntu-ui-toolkit:
importance: Undecided → High
Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package qtorganizer5-eds - 0.1.1+14.04.20140307-0ubuntu1

---------------
qtorganizer5-eds (0.1.1+14.04.20140307-0ubuntu1) trusty; urgency=low

  [ Renato Araujo Oliveira Filho ]
  * Fixed parse from icalrecurrencetype to
    QtOrganizer::QOrganizerRecurrenceRule for weekly recurrence. (LP:
    #1285958)
 -- Ubuntu daily release <email address hidden> Fri, 07 Mar 2014 04:49:19 +0000

Changed in qtorganizer5-eds (Ubuntu):
status: Confirmed → Fix Released
Matthew Paul Thomas (mpt)
summary: - Reccuring Alarm can't be set for sunday
+ Recurring alarm can't be set for sunday
Nekhelesh Ramananthan (nik90)
Changed in ubuntu-clock-app:
status: Confirmed → Invalid
Revision history for this message
Nekhelesh Ramananthan (nik90) wrote :

Marked bug as invalid for the clock app since the bug lies in the toolkit and EDS.

Nekhelesh Ramananthan (nik90)
Changed in ubuntu-ui-toolkit:
status: New → In Progress
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :

Fix committed into lp:~ubuntu-sdk-team/ubuntu-ui-toolkit/staging at revision 1004, scheduled for release in ubuntu-ui-toolkit, milestone Unknown

Changed in ubuntu-ui-toolkit:
status: In Progress → Fix Committed
Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package ubuntu-ui-toolkit - 0.1.46+14.10.20140502.6-0ubuntu1

---------------
ubuntu-ui-toolkit (0.1.46+14.10.20140502.6-0ubuntu1) utopic; urgency=low

  [ Tarmac ]
  * [ Zoltán Balogh]   * Skip tests only for powerpc   [ Christian Dywan
    ]   * Use locale format specifier in textfield_numbers. (LP:
    #1311907, #1283236, #1285958, #1124071, #1256424, #1276172,
    #1228042, #1183255, #1283212, #1168914, #1273187, #1276821,
    #1308086, #1288937, #1292601)

  [ Zoltán Balogh ]
  * [ Zoltán Balogh]   * Skip tests only for powerpc   [ Christian Dywan
    ]   * Use locale format specifier in textfield_numbers. (LP:
    #1311907, #1283236, #1285958, #1124071, #1256424, #1276172,
    #1228042, #1183255, #1283212, #1168914, #1273187, #1276821,
    #1308086, #1288937, #1292601)

  [ tpeeters ]
  * [ Zoltán Balogh]   * Skip tests only for powerpc   [ Christian Dywan
    ]   * Use locale format specifier in textfield_numbers. (LP:
    #1311907, #1283236, #1285958, #1124071, #1256424, #1276172,
    #1228042, #1183255, #1283212, #1168914, #1273187, #1276821,
    #1308086, #1288937, #1292601)
 -- Ubuntu daily release <email address hidden> Fri, 02 May 2014 16:31:41 +0000

Changed in ubuntu-ui-toolkit (Ubuntu):
status: New → Fix Released
Renato Araujo Oliveira Filho (renatofilho)
Changed in ubuntu-ui-toolkit:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.
Subscribing...

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.