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

List:       kde-commits
Subject:    KDE/kdemultimedia/juk
From:       Giorgos Kylafas <gekylafas () gmail ! com>
Date:       2011-06-06 1:52:11
Message-ID: 20110606015211.BED71AC783 () svn ! kde ! org
[Download RAW message or body]

SVN commit 1235482 by kylafas:

juk: load search playlists even when containing empty patterns.

When a new search playlist is created, we are allowed to not fill any
"search lines" with our criteria. This means that when a new search
playlist is created/saved, it may contain search components with empty
patterns. However, when loading a search playlist from disk, we signal
an error if we read an empty search pattern. As a result, the playlist
does not get loaded and JuK gives the impression that it does not save
the search playlists.

Since we are presented with two search lines when creating a new search
playlist (see AdvancedSearchDialog's ctor), the above has a rather high
chance of happening.

So, upon loading, we should not consider the presense of empty patterns
as an error.

 M  +0 -3      playlistsearch.cpp  


--- trunk/KDE/kdemultimedia/juk/playlistsearch.cpp #1235481:1235482
@@ -318,9 +318,6 @@
       >> columns
       >> mode;
 
-    if(pattern.isEmpty())
-        throw BICStreamException();
-
     if(patternSearch)
         c = PlaylistSearch::Component(QRegExp(pattern), columns);
     else
[prev in list] [next in list] [prev in thread] [next in thread]