Merge lp:~popey/music-app/fix-1392587 into lp:music-app/remix
Proposed by
Alan Pope πΊπ§π± π¦
Status: | Merged | ||||
---|---|---|---|---|---|
Approved by: | Alan Pope πΊπ§π± π¦ | ||||
Approved revision: | 739 | ||||
Merged at revision: | 739 | ||||
Proposed branch: | lp:~popey/music-app/fix-1392587 | ||||
Merge into: | lp:music-app/remix | ||||
Diff against target: |
11 lines (+1/-0) 1 file modified
meta-database.js (+1/-0) |
||||
To merge this branch: | bzr merge lp:~popey/music-app/fix-1392587 | ||||
Related bugs: |
|
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Oliver Grawert (community) | Approve | ||
Ubuntu Phone Apps Jenkins Bot | continuous-integration | Approve | |
Music App Developers | Pending | ||
Review via email:
|
Commit message
Fix 1392587
Description of the change
In testing we found the queue doesn't get populated when launched from the music scope.
It seems we aren't checking for the existence of the queue table before attempting to clear it, before populating with songs sent from the scope.
This one line fix adds the createQueue(); to ensure the table exists.
To post a comment you must log in.
FAILED: Continuous integration, rev:739 /code.launchpad .net/~popey/ music-app/ fix-1392587/ +merge/ 241770/ +edit-commit- message
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:/
http:// 91.189. 93.70:8080/ job/music- app-remix- ci/265/ 91.189. 93.70:8080/ job/generic- mediumtests- vivid/214 91.189. 93.70:8080/ job/generic- mediumtests- vivid/214/ artifact/ work/output/ *zip*/output. zip 91.189. 93.70:8080/ job/music- app-remix- vivid-amd64- ci/40
Executed test runs:
SUCCESS: http://
deb: http://
SUCCESS: http://
Click here to trigger a rebuild: 91.189. 93.70:8080/ job/music- app-remix- ci/265/ rebuild
http://