extragear/multimedia/amarok/src/playlistbrowser

Ian Monroe ian.monroe at gmail.com
Mon Jun 30 18:03:08 CEST 2008


SVN commit 826375 by ianmonroe:

this fixes a crash when adding the URL http://66.232.117.225:8020
So the question is... why does CollectionManager return a null ptr
for this URL?
CCMAIL: amarok-devel at kde.org


 M  +8 -1      PlaylistCategory.cpp  


--- trunk/extragear/multimedia/amarok/src/playlistbrowser/PlaylistCategory.cpp #826374:826375
@@ -190,7 +190,14 @@
     if( !dialog )
         return;
     Meta::TrackPtr track = CollectionManager::instance()->trackForUrl( dialog->streamUrl() );
-    PlaylistBrowserNS::UserModel::instance()->createNewStream(  dialog->streamName(), track );
+    if( !track.isNull() )
+    {
+        PlaylistBrowserNS::UserModel::instance()->createNewStream(  dialog->streamName(), track );
+    }
+    else
+    {
+        warning() << "No Meta::Track returned for " << dialog->streamUrl();
+    }
 }
 
 PlaylistBrowserNS::StreamEditor::StreamEditor( QWidget* parent )


More information about the Amarok-devel mailing list