Bug 186573

Summary:	Wrong item deleted in a filtered playlist
Product:	[Applications] amarok	Reporter:	Huynh Huu Long <long>
Component:	Playlist	Assignee:	Amarok Developers <amarok-bugs-dist>
Status:	RESOLVED NOT A BUG
Severity:	normal	CC:	jithin1987, nhn, simon.esneault
Priority:	NOR
Version:	2.1-SVN
Target Milestone:	---
Platform:	Compiled Sources
OS:	Linux
Latest Commit:		Version Fixed In:

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.