[prev in list] [next in list] [prev in thread] [next in thread] 

List:       kde-bugs-dist
Subject:    [Bug 130359] editing tags for a track marked as played unmarks it
From:       Jeff Mitchell <kde-dev () emailgoeshere ! com>
Date:       2006-09-30 19:40:36
Message-ID: 20060930194036.3330.qmail () ktown ! kde ! org
[Download RAW message or body]

------- You are receiving this mail because: -------
You are on the CC list for the bug, or are watching someone who is.
         
http://bugs.kde.org/show_bug.cgi?id=130359         
kde-dev emailgoeshere com changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|ASSIGNED                    |RESOLVED
         Resolution|                            |FIXED



------- Additional Comments From kde-dev emailgoeshere com  2006-09-30 21:40 -------
SVN commit 590767 by mitchell:

Fix files added back to collection not always getting enabled in the Playlist.
BUG: 130359


 M  +2 -0      ChangeLog  
 M  +2 -1      src/playlistitem.cpp  


--- trunk/extragear/multimedia/amarok/ChangeLog #590766:590767
 @ -39,6 +39,8  @
     * Show a proper tag dialog when viewing information for DAAP music shares.
 
   BUGFIXES:
+    * Files that were detected as being added back to the collection would not
+      always be re-enabled in the Playlist. (BR 130359)
     * Fix some spelling and layout issues. Part of a patch by Malcolm Parsons
       <malcolm.parsons gmail com> 
     * Correctly handle horizontal wheel events in position slider. (BR 119254)
--- trunk/extragear/multimedia/amarok/src/playlistitem.cpp #590766:590767
 @ -230,6 +230,7  @
 void PlaylistItem::setFilestatusEnabled( bool enabled )
 {
     m_filestatusEnabled = enabled;
+    checkExists();
     setEnabled();
 }
 
 @ -672,7 +673,7  @
         }
     else
     {
-        const QColorGroup _cg = ( !exists() || !isEnabled() )
+        const QColorGroup _cg = ( !isEnabled() )
                                 ? listView()->palette().disabled()
                                 : listView()->palette().active();
[prev in list] [next in list] [prev in thread] [next in thread]