Merge lp:~markjtully/ubuntu-app-review-board/unity-scopes-music-extras-oneiric into lp:~app-review-board/ubuntu-app-review-board/unity-scopes-music-extras-oneiric
Proposed by
Allison Randal
Status: | Merged |
---|---|
Merged at revision: | 17 |
Proposed branch: | lp:~markjtully/ubuntu-app-review-board/unity-scopes-music-extras-oneiric |
Merge into: | lp:~app-review-board/ubuntu-app-review-board/unity-scopes-music-extras-oneiric |
Diff against target: |
21 lines (+2/-2) 1 file modified
scopes/guayadeque/unity-scope-guayadeque (+2/-2) |
To merge this branch: | bzr merge lp:~markjtully/ubuntu-app-review-board/unity-scopes-music-extras-oneiric |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Allison Randal (community) | Approve | ||
Review via email: mp+101211@code.launchpad.net |
Description of the change
[from markjtully]
The problem was that the decade and genre checks were checking the wrong fields of the track (and panicking when it found what it thought was the year was in fact not a number at all). I've corrected these and it now functions correctly on my machine. This should allow us to publish all 4 scopes, and I can move on to updating them for Precise!
To post a comment you must log in.
Fixes an off-by-one error in the array index.