moving a file in playlist creates duplicate entry

Bug #545914 reported by Lars Volker
16
This bug affects 3 people
Affects Status Importance Assigned to Milestone
Exaile
Fix Released
High
Rocco Aliberti

Bug Description

In latest version 0.3.1.0+bzr2938 I get duplicate entries when moving a song inside a playlist. Instead of being moved, a copy appears at the destination.

Related branches

Revision history for this message
Lars Volker (lv) wrote :

I forgot to mention, that I also checked it for version 0.3.1.1-0ppakarmic1beta4, but there it does NOT happen.

reacocard (reacocard)
Changed in exaile:
importance: Undecided → High
status: New → Confirmed
assignee: nobody → Mathias Brodala (mathbr)
Revision history for this message
Mathias Brodala (mathbr) wrote :

Should be fixed with r2939.

Changed in exaile:
milestone: none → 0.3.2
Revision history for this message
Lars Volker (lv) wrote :

Yes. It does not occur in r2939 anymore. Thanks for the quick fix.

Changed in exaile:
status: Confirmed → Fix Committed
Revision history for this message
Lars Volker (lv) wrote :

This bug reappeared in the latest trunk version 0.3.1.99+bzr3024. Therefore i reopen the bug.

Changed in exaile:
status: Fix Committed → Confirmed
Revision history for this message
Ernst (ernst-blaauw) wrote :

Confirmed on bzr 3045

Revision history for this message
Rocco Aliberti (eri.trabiccolo) wrote :

I looked a bit the xlgui/playlist.py code, and I noticed that context.action is always

<flags GDK_ACTION_COPY GdkDragAction>

 so the istruction #736

if context.action == gtk.gdk.ACTION_MOVE:

is always false. I propose this "conservative" patch. Or do only if Playlist._is_drag_source.

Anyway the istruction #769 self.emit('track-count-changed', len(self.playlist)), should be performed only in case of a copy and not a move, i think.

Revision history for this message
Mathias Brodala (mathbr) wrote :

Fixed with r3046 in trunk and r2921 in 0.3.1.x.

Changed in exaile:
assignee: Mathias Brodala (mathbr) → nobody
status: Confirmed → Fix Committed
assignee: nobody → Rocco Aliberti (r0ck84)
reacocard (reacocard)
Changed in exaile:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.