[Ubuntu-touch-coreapps-reviewers] [Merge] lp:~gary-wzl77/ubuntu-calendar-app/fix_1437305_and_1455377 into lp:ubuntu-calendar-app

2015-06-19 Thread Alan Pope 
The proposal to merge 
lp:~gary-wzl77/ubuntu-calendar-app/fix_1437305_and_1455377 into 
lp:ubuntu-calendar-app has been updated.

Status: Needs review = Approved

For more details, see:
https://code.launchpad.net/~gary-wzl77/ubuntu-calendar-app/fix_1437305_and_1455377/+merge/262059
-- 
Your team Ubuntu Calendar Developers is subscribed to branch 
lp:ubuntu-calendar-app.

-- 
Mailing list: https://launchpad.net/~ubuntu-touch-coreapps-reviewers
Post to : ubuntu-touch-coreapps-reviewers@lists.launchpad.net
Unsubscribe : https://launchpad.net/~ubuntu-touch-coreapps-reviewers
More help   : https://help.launchpad.net/ListHelp


[Ubuntu-touch-coreapps-reviewers] [Merge] lp:~gary-wzl77/ubuntu-calendar-app/fix_1437305_and_1455377 into lp:ubuntu-calendar-app

2015-06-19 Thread noreply
The proposal to merge 
lp:~gary-wzl77/ubuntu-calendar-app/fix_1437305_and_1455377 into 
lp:ubuntu-calendar-app has been updated.

Status: Approved = Merged

For more details, see:
https://code.launchpad.net/~gary-wzl77/ubuntu-calendar-app/fix_1437305_and_1455377/+merge/262059
-- 
Your team Ubuntu Calendar Developers is subscribed to branch 
lp:ubuntu-calendar-app.

-- 
Mailing list: https://launchpad.net/~ubuntu-touch-coreapps-reviewers
Post to : ubuntu-touch-coreapps-reviewers@lists.launchpad.net
Unsubscribe : https://launchpad.net/~ubuntu-touch-coreapps-reviewers
More help   : https://help.launchpad.net/ListHelp


Re: [Ubuntu-touch-coreapps-reviewers] [Merge] lp:~gary-wzl77/ubuntu-calendar-app/fix_1437305_and_1455377 into lp:ubuntu-calendar-app

2015-06-19 Thread Ubuntu Phone Apps Jenkins Bot
Review: Approve continuous-integration

PASSED: Continuous integration, rev:670
http://91.189.93.70:8080/job/ubuntu-calendar-app-ci/1197/
Executed test runs:
SUCCESS: http://91.189.93.70:8080/job/generic-mediumtests-utopic/2958
deb: 
http://91.189.93.70:8080/job/generic-mediumtests-utopic/2958/artifact/work/output/*zip*/output.zip
SUCCESS: 
http://91.189.93.70:8080/job/ubuntu-calendar-app-utopic-amd64-ci/668
SUCCESS: http://91.189.93.70:8080/job/ubuntu-calendar-app-vivid-amd64-ci/194

Click here to trigger a rebuild:
http://91.189.93.70:8080/job/ubuntu-calendar-app-ci/1197/rebuild

-- 
https://code.launchpad.net/~gary-wzl77/ubuntu-calendar-app/fix_1437305_and_1455377/+merge/262059
Your team Ubuntu Calendar Developers is subscribed to branch 
lp:ubuntu-calendar-app.

-- 
Mailing list: https://launchpad.net/~ubuntu-touch-coreapps-reviewers
Post to : ubuntu-touch-coreapps-reviewers@lists.launchpad.net
Unsubscribe : https://launchpad.net/~ubuntu-touch-coreapps-reviewers
More help   : https://help.launchpad.net/ListHelp


Re: [Ubuntu-touch-coreapps-reviewers] [Merge] lp:~gary-wzl77/ubuntu-calendar-app/fix_1437305_and_1455377 into lp:ubuntu-calendar-app

2015-06-17 Thread Ubuntu Phone Apps Jenkins Bot
Review: Needs Fixing continuous-integration

FAILED: Continuous integration, rev:670
No commit message was specified in the merge proposal. Click on the following 
link and set the commit message (if you want a jenkins rebuild you need to 
trigger it yourself):
https://code.launchpad.net/~gary-wzl77/ubuntu-calendar-app/fix_1437305_and_1455377/+merge/262059/+edit-commit-message

http://91.189.93.70:8080/job/ubuntu-calendar-app-ci/1194/
Executed test runs:
SUCCESS: http://91.189.93.70:8080/job/generic-mediumtests-utopic/2941
deb: 
http://91.189.93.70:8080/job/generic-mediumtests-utopic/2941/artifact/work/output/*zip*/output.zip
SUCCESS: 
http://91.189.93.70:8080/job/ubuntu-calendar-app-utopic-amd64-ci/665
SUCCESS: http://91.189.93.70:8080/job/ubuntu-calendar-app-vivid-amd64-ci/191

Click here to trigger a rebuild:
http://91.189.93.70:8080/job/ubuntu-calendar-app-ci/1194/rebuild

-- 
https://code.launchpad.net/~gary-wzl77/ubuntu-calendar-app/fix_1437305_and_1455377/+merge/262059
Your team Ubuntu Calendar Developers is subscribed to branch 
lp:ubuntu-calendar-app.

-- 
Mailing list: https://launchpad.net/~ubuntu-touch-coreapps-reviewers
Post to : ubuntu-touch-coreapps-reviewers@lists.launchpad.net
Unsubscribe : https://launchpad.net/~ubuntu-touch-coreapps-reviewers
More help   : https://help.launchpad.net/ListHelp


[Ubuntu-touch-coreapps-reviewers] [Merge] lp:~gary-wzl77/ubuntu-calendar-app/fix_1437305_and_1455377 into lp:ubuntu-calendar-app

2015-06-16 Thread Gary.Wang
Gary.Wang has proposed merging 
lp:~gary-wzl77/ubuntu-calendar-app/fix_1437305_and_1455377 into 
lp:ubuntu-calendar-app.

Requested reviews:
  Alan Pope  (popey)

For more details, see:
https://code.launchpad.net/~gary-wzl77/ubuntu-calendar-app/fix_1437305_and_1455377/+merge/262059

Confirmed by Renato
https://bugs.launchpad.net/ubuntu-calendar-app/+bug/1437305/comments/6

We save wrong end date for all day event(end date should be one day later than 
start date). This issue results in
1. Duplicated event display for all day event in weekly/daily view 
2. Wrong date day in monthly view

This patch fix these two issue.
-- 
Your team Ubuntu Calendar Developers is subscribed to branch 
lp:ubuntu-calendar-app.
=== modified file 'AllDayEventComponent.qml'
--- AllDayEventComponent.qml	2014-11-29 03:36:18 +
+++ AllDayEventComponent.qml	2015-06-16 09:28:53 +
@@ -18,6 +18,7 @@
 import QtQuick 2.3
 import Ubuntu.Components 1.1
 import Ubuntu.Components.Popups 1.0
+import QtOrganizer 5.0
 
 import dateExt.js as DateExt
 import ViewType.js as ViewType
@@ -39,7 +40,7 @@
 for(var i = 0 ; i  items.length ; ++i) {
 var event = items[(i)];
 if( event  event.allDay ) {
-for(var d = event.startDateTime; d = event.endDateTime; d = d.addDays(1)) {
+for(var d = event.startDateTime; d  event.endDateTime; d = d.addDays(1)) {
 var key = Qt.formatDateTime(d, dd-MMM-);
 if( !(key in map)) {
 map[key] = [];

=== modified file 'NewEvent.qml'
--- NewEvent.qml	2015-05-22 12:25:24 +
+++ NewEvent.qml	2015-06-16 09:28:53 +
@@ -179,14 +179,17 @@
 event = Qt.createQmlObject(import QtOrganizer 5.0; Event {}, Qt.application,NewEvent.qml);
 }
 
+event.allDay = allDayEventCheckbox.checked;
+
 event.startDateTime = startDate;
-event.endDateTime = endDate;
+if (event.allDay)
+event.endDateTime = startDate.addDays(1);
+else
+event.endDateTime = endDate;
 event.displayLabel = titleEdit.text;
 event.description = messageEdit.text;
 event.location = locationEdit.text
 
-event.allDay = allDayEventCheckbox.checked;
-
 if( event.itemType === Type.Event ) {
 event.attendees = []; // if Edit remove all attendes  add them again if any
 var contacts = [];

=== modified file 'po/com.ubuntu.calendar.pot'
--- po/com.ubuntu.calendar.pot	2015-06-09 10:45:01 +
+++ po/com.ubuntu.calendar.pot	2015-06-16 09:28:53 +
@@ -8,7 +8,7 @@
 msgstr 
 Project-Id-Version: \n
 Report-Msgid-Bugs-To: \n
-POT-Creation-Date: 2015-06-09 18:44+0800\n
+POT-Creation-Date: 2015-06-16 17:18+0800\n
 PO-Revision-Date: YEAR-MO-DA HO:MI+ZONE\n
 Last-Translator: FULL NAME EMAIL@ADDRESS\n
 Language-Team: LANGUAGE l...@li.org\n
@@ -46,13 +46,13 @@
 #. on a given day. Ev. is short form for Events.
 #. Please keep the translation of Ev. to 3 characters only, as the week view
 #. where it's shown has limited space
-#: ../AllDayEventComponent.qml:122
+#: ../AllDayEventComponent.qml:123
 #, qt-format
 msgid %1 ev.
 msgstr 
 
 #. TRANSLATORS: the argument refers to the number of all day events
-#: ../AllDayEventComponent.qml:126
+#: ../AllDayEventComponent.qml:127
 #, qt-format
 msgid %1 all day event
 msgid_plural %1 all day events
@@ -77,8 +77,8 @@
 msgid Syncing
 msgstr 
 
-#: ../CalendarChoicePopup.qml:71
-msgid Add new Calendar
+#: ../CalendarChoicePopup.qml:70
+msgid Add online Calendar
 msgstr 
 
 #: ../ColorPickerDialog.qml:25
@@ -136,7 +136,7 @@
 msgid Delete
 msgstr 
 
-#: ../EditEventConfirmationDialog.qml:29 ../NewEvent.qml:296
+#: ../EditEventConfirmationDialog.qml:29 ../NewEvent.qml:299
 msgid Edit Event
 msgstr 
 
@@ -154,7 +154,7 @@
 msgid Edit this
 msgstr 
 
-#: ../EventActions.qml:50 ../NewEvent.qml:296
+#: ../EventActions.qml:50 ../NewEvent.qml:299
 msgid New Event
 msgstr 
 
@@ -165,7 +165,7 @@
 msgid %1 b%2/b
 msgstr 
 
-#: ../EventDetails.qml:42 ../NewEvent.qml:407
+#: ../EventDetails.qml:42 ../NewEvent.qml:410
 msgid Event Details
 msgstr 
 
@@ -189,11 +189,11 @@
 msgid Edit
 msgstr 
 
-#: ../EventDetails.qml:347 ../NewEvent.qml:509
+#: ../EventDetails.qml:347 ../NewEvent.qml:512
 msgid Guests
 msgstr 
 
-#: ../EventDetails.qml:390 ../EventReminder.qml:35 ../NewEvent.qml:606
+#: ../EventDetails.qml:390 ../EventReminder.qml:35 ../NewEvent.qml:609
 msgid Reminder
 msgstr 
 
@@ -216,7 +216,7 @@
 #. TRANSLATORS: this refers to how often a recurrent event repeats
 #. and it is shown as the header of the option selector to choose
 #. its repetition
-#: ../EventRepetition.qml:237 ../NewEvent.qml:590
+#: ../EventRepetition.qml:237 ../NewEvent.qml:593
 msgid Repeats
 msgstr 
 
@@ -267,43 +267,43 @@
 msgid End time can't be before start time
 msgstr 
 
-#: ../NewEvent.qml:306
+#: ../NewEvent.qml:309
 msgid Error
 msgstr