[Remix] Renaming playlists causes app to crash
Bug #1386712 reported by
Victor Thompson
This bug affects 1 person
| Affects | Status | Importance | Assigned to | Milestone | |
|---|---|---|---|---|---|
| Ubuntu Music App |
Fix Released
|
Critical
|
Andrew Hayzen | ||
Bug Description
Currently, when running with the most recent revision from the remix branch (lp:music-app/remix r712), renaming a playlist causes the following output which then causes the app to freeze:
QQmlExpression: Attempted to evaluate an expression in an invalid context
To reproduce:
1. Go to the Albums tab
2. Swipe to the left and click the "Add to playlist icon"
3. Create a new playlist if one does not exist by tapping the plus icon
4. Go to the Playlists tab and select the playlist
5. Tap the edit icon in the header (the left-most one)
6. Change the name in the text field and hit OK
Actual results: App becomes unresponsive.
Expected results: App renames playlist as expected.
Related branches
lp:~ahayzen/music-app/remix-fix-1386712-rename-playlist-crash
- Victor Thompson: Approve
- Ubuntu Phone Apps Jenkins Bot: Approve (continuous-integration)
-
Diff: 14 lines (+3/-1)1 file modifiedcommon/SongsPage.qml (+3/-1)
| summary: |
- [Remix] Renaming playlists causes error + [Remix] Renaming playlists causes app to crash |
| Changed in music-app: | |
| status: | New → Triaged |
| importance: | Undecided → Critical |
Revision history for this message
| Andrew Hayzen (ahayzen) wrote | #1 |
| Changed in music-app: | |
| assignee: | nobody → Andrew Hayzen (andrew-hayzen) |
| status: | Triaged → In Progress |
Revision history for this message
| Ubuntu Phone Apps Jenkins Bot (ubuntu-phone-apps-jenkins-bot) wrote | #2 |
| Changed in music-app: | |
| status: | In Progress → Fix Committed |
| Changed in music-app: | |
| status: | Fix Committed → Fix Released |
To post a comment you must log in.
The error is the same as we've seen before "QQmlExpression: Attempted to evaluate an expression in an invalid context", guess this needs to set the changed flag and refresh the playlists list *after* closing the songsPage as well.