Merge lp:~mhall119/ubuntu-calendar-app/fix-newevent-month-1233442 into lp:ubuntu-calendar-app
Proposed by
Michael Hall
Status: | Merged |
---|---|
Approved by: | Kunal Parmar |
Approved revision: | 127 |
Merged at revision: | 135 |
Proposed branch: | lp:~mhall119/ubuntu-calendar-app/fix-newevent-month-1233442 |
Merge into: | lp:ubuntu-calendar-app |
Diff against target: |
16 lines (+5/-0) 1 file modified
NewEvent.qml (+5/-0) |
To merge this branch: | bzr merge lp:~mhall119/ubuntu-calendar-app/fix-newevent-month-1233442 |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Kunal Parmar | Approve | ||
Ubuntu Phone Apps Jenkins Bot | continuous-integration | Approve | |
Review via email: mp+188488@code.launchpad.net |
Commit message
Set month and year, as well as date, before saving a new event
Description of the change
Calling startDate.setDate() only sets the day of the month, not the full date (day, month, year), so it was only changing the day of month part of the startDate and endDate.
This patch also sets the month and year for both.
To post a comment you must log in.
PASSED: Continuous integration, rev:127 91.189. 93.70:8080/ job/ubuntu- calendar- app-ci/ 101/ 91.189. 93.70:8080/ job/generic- mediumtests/ 584 91.189. 93.70:8080/ job/ubuntu- calendar- app-precise- amd64-ci/ 101 91.189. 93.70:8080/ job/ubuntu- calendar- app-quantal- amd64-ci/ 101 91.189. 93.70:8080/ job/ubuntu- calendar- app-raring- amd64-ci/ 101 91.189. 93.70:8080/ job/ubuntu- calendar- app-saucy- amd64-ci/ 101
http://
Executed test runs:
SUCCESS: http://
SUCCESS: http://
SUCCESS: http://
SUCCESS: http://
SUCCESS: http://
Click here to trigger a rebuild: 91.189. 93.70:8080/ job/ubuntu- calendar- app-ci/ 101/rebuild
http://