Merge lp:~mzanetti/reminders-app/fix-sort-init into lp:reminders-app
Proposed by
Michael Zanetti
Status: | Merged |
---|---|
Approved by: | Riccardo Padovani |
Approved revision: | 374 |
Merged at revision: | 378 |
Proposed branch: | lp:~mzanetti/reminders-app/fix-sort-init |
Merge into: | lp:reminders-app |
Diff against target: |
12 lines (+1/-1) 1 file modified
src/libqtevernote/notes.cpp (+1/-1) |
To merge this branch: | bzr merge lp:~mzanetti/reminders-app/fix-sort-init |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Riccardo Padovani | Approve | ||
Alan Pope πΊπ§π± π¦ (community) | Approve | ||
Ubuntu Phone Apps Jenkins Bot | continuous-integration | Approve | |
Review via email: mp+251850@code.launchpad.net |
Commit message
fix initialisation of sortOrder properties
Description of the change
If setting sort order to "created, newest first" the sort order will be wrong at startup because the model's property is set to CreatedNewestFirst while the ctor actually sorts for "updated, newest first".
To post a comment you must log in.
PASSED: Continuous integration, rev:374 91.189. 93.70:8080/ job/reminders- app-ci/ 697/ 91.189. 93.70:8080/ job/generic- mediumtests- utopic/ 2245 91.189. 93.70:8080/ job/generic- mediumtests- utopic/ 2245/artifact/ work/output/ *zip*/output. zip 91.189. 93.70:8080/ job/reminders- app-utopic- amd64-ci/ 299 91.189. 93.70:8080/ job/reminders- app-vivid- amd64-ci/ 118
http://
Executed test runs:
SUCCESS: http://
deb: http://
SUCCESS: http://
SUCCESS: http://
Click here to trigger a rebuild: 91.189. 93.70:8080/ job/reminders- app-ci/ 697/rebuild
http://