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

List:       kde-commits
Subject:    extragear/multimedia/amarok
From:       Alejandro Daniel Wainzinger <aikawarazuni () gmail ! com>
Date:       2009-06-27 3:23:54
Message-ID: 1246073034.218248.4521.nullmailer () svn ! kde ! org
[Download RAW message or body]

SVN commit 987950 by awainzinger:

Added check to see if user actually clicked "ok" on playlist rename, so playlists \
don't get renamed to blank.

 M  +1 -0      ChangeLog  
 M  +5 -2      src/playlistmanager/sql/SqlUserPlaylistProvider.cpp  


--- trunk/extragear/multimedia/amarok/ChangeLog #987949:987950
@@ -35,6 +35,7 @@
       highlight currently playing artist within compilation albums
 
   BUGFIXES:
+    * Fixed local playlists being renamed to blank when user hits cancel
     * Fixed giant Edit Filter dialog with some translations. (BR 197566)
     * Files browser remembers its view mode. (BR 197122)
 
--- trunk/extragear/multimedia/amarok/src/playlistmanager/sql/SqlUserPlaylistProvider.cpp \
#987949:987950 @@ -114,9 +114,12 @@
     if( playlist.isNull() )
         return;
     //TODO: inline rename
+    bool ok;
     const QString newName = KInputDialog::getText( i18n("Change playlist"),
-                i18n("Enter new name for playlist:"), playlist->name() );
-    playlist->setName( newName.trimmed() );
+                i18n("Enter new name for playlist:"), playlist->name(),
+                                                   &ok );
+    if ( ok )
+        playlist->setName( newName.trimmed() );
 }
 
 QList<PopupDropperAction *>


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