| Summary: | [PATCH] Add "remove duplicates" button | ||
|---|---|---|---|
| Product: | [Applications] amarok | Reporter: | Kuba Serafinowski <kuba.serafinowski> |
| Component: | Playlist | Assignee: | Amarok Developers <amarok-bugs-dist> |
| Status: | RESOLVED FIXED | ||
| Severity: | wishlist | CC: | john, teo |
| Priority: | NOR | ||
| Version: | 2.3-GIT | ||
| Target Milestone: | 2.3.1 | ||
| Platform: | openSUSE | ||
| OS: | Linux | ||
| Latest Commit: | Version Fixed In: | ||
| Sentry Crash Report: | |||
|
Description
Kuba Serafinowski
2009-07-16 15:08:47 UTC
changing component to The Playlist. We could add this functionality to the playlistbrowser as well though. (In reply to comment #1) > changing component to The Playlist. We could add this functionality to the > playlistbrowser as well though. Right, it was me who thought in wrong way, that PlaylistBrowser was Playlist :) I have proposed a patch to add this functionality. Please look here for further information: http://gitorious.org/amarok/amarok/merge_requests/1436 Thanks for the patch, John, but it will have to wait as we are currently in feature freeze for 2.2. I've merged John Atkinson's patch into git master. Feature will be included in Amarok 2.2.1. Thanks John :) Hi John, there's an issue with your patch: !. Add n times a song (n>2) 2. Remove duplicates 3. You'll end-up with n-1 given songs instead of 1 If you could try to fix this it would be awesome! I'm sure someone will merge fix with pleasure :-) Thanks again for this patch! Oh dear! Yes indeed. Here's a fixed version: http://gitorious.org/amarok/amarok/merge_requests/1659 Thanks, John Pushed to master :) Thanks again John! |