Bug 186573 - Wrong item deleted in a filtered playlist
Summary: Wrong item deleted in a filtered playlist
Status: RESOLVED NOT A BUG
Alias: None
Product: amarok
Classification: Applications
Component: Playlist (show other bugs)
Version: 2.1-SVN
Platform: Compiled Sources Linux
: NOR normal
Target Milestone: ---
Assignee: Amarok Developers
URL:
Keywords:
: 187915 (view as bug list)
Depends on:
Blocks:
 
Reported: 2009-03-08 18:27 UTC by Huynh Huu Long
Modified: 2009-07-30 17:56 UTC (History)
3 users (show)

See Also:
Latest Commit:
Version Fixed In:

Attachments
Add an attachment

Note You need to log in before you can comment on or make changes to this bug.
Description Huynh Huu Long 2009-03-08 18:27:20 UTC 
Version:            (using Devel)
OS:                Linux
Installed from:    Compiled sources

When one selects a song in a filtered playlist, apparently the song of the unfiltered playlist with the same position gets selected. After clicking on remove, the other song gets removed instead of the one I selected.

Example:
without filter:
Song1
Song2
Song3
Song4
Song5

with filter:
Song 3
Song 5 <- delete this

after removing the filter:
Song1
Song3
Song4
Song5 <- remains
Comment 1 Nikolaj Hald Nielsen 2009-03-08 20:10:43 UTC 
SVN commit 936977 by nhnielsen:

Fix actions choosen from context menu in a filtered playlist applying to wrong track. Also make the use of row numbers a bit more consistent. Whenever a view communicates a row number to another class, the view is responsible for 
translating this row number into the corrosponding row number in the source (PlaylistModel) model. No other calss should have to know what proxy models a particular view is using.
BUG: 186573



 M  +14 -18    GroupingProxy.cpp  
 M  +4 -5      PlaylistActions.cpp  
 M  +9 -2      view/listview/PrettyListView.cpp  


WebSVN link: http://websvn.kde.org/?view=rev&revision=936977
Comment 2 Simon ESNEAULT 2009-03-09 10:40:11 UTC 
Backtrace for this bug
http://pastebin.com/ddcb8724
Comment 3 Mark Kretschmann 2009-03-10 16:00:19 UTC 
Reopening per request of user Takahini (about to attach a BT).
Comment 4 Simon ESNEAULT 2009-03-10 16:02:52 UTC 
(In reply to comment #3)
> Reopening per request of user Takahini (about to attach a BT).

here it is 
http://pastebin.com/d26be1cdb
Comment 5 Mark Kretschmann 2009-03-10 16:19:04 UTC 
@Simon: Next time please paste the backtrace directly in here; makes it easier to read and to search for.
Comment 6 Nikolaj Hald Nielsen 2009-03-16 11:44:00 UTC 
Those pastebins have expired, and there is absolutely no description of how to reproduce the bug after my changes... You guys are not giving me a lot to go on here...
Comment 7 Mark Kretschmann 2009-03-16 12:00:46 UTC 
Please reopen if the problem can be reproduced, and then paste the backtrace here directly.
Comment 8 Nikolaj Hald Nielsen 2009-03-23 18:30:55 UTC 
*** Bug 187915 has been marked as a duplicate of this bug. ***
Comment 9 Myriam Schweingruber 2009-07-30 17:56:58 UTC 
Changed wrong status to resolved.