insert album tracks at the correct place instead of at the end
AbandonedPublic

Authored by mgallien on Sep 27 2019, 8:22 PM.

Details

Summary

inserting the tracks not at the end fix the bug

detected by QAbstractItemModelTester which is really a nice tool

BUG: 406466

Test Plan

new test is OK, no regressions

Diff Detail

Repository
R255 Elisa
Branch
fixPlayListInsertAlbum
Lint
No Linters Available
Unit
No Unit Test Coverage
Build Status
Buildable 17107
Build 17125: arc lint + arc unit
mgallien requested review of this revision.Sep 27 2019, 8:22 PM
mgallien created this revision.
ngraham requested changes to this revision.Sep 27 2019, 9:11 PM

This fixes the bug for me when navigating to an artist and clicking on the Add to Playlist or Replace and Play buttons in the toolbar. But it is still broken for the following other use cases:

  • Navigate to a genre and click on the Add to Playlist or Replace and Play buttons in the toolbar
  • Click on the Add to Playlist or Replace and Play buttons in the grid delegate for an artist or a genre

It appears that buttons in the grid delegates trigger different logic than the toolbar buttons, which is probably something that should be unified.

This revision now requires changes to proceed.Sep 27 2019, 9:11 PM
mgallien planned changes to this revision.Sep 28 2019, 8:00 AM

Oops, thanks for the review

mgallien abandoned this revision.Oct 8 2019, 7:29 PM

Moved to gitlab merge request