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

List:       kde-commits
Subject:    extragear/multimedia/amarok/src
From:       Casey Link <unnamedrambler () gmail ! com>
Date:       2008-05-31 23:10:32
Message-ID: 1212275432.225645.21686.nullmailer () svn ! kde ! org
[Download RAW message or body]

SVN commit 815021 by link:

Expose the local collection path via dbus in a more elegant fashion. The \
CollectionLocation changes should prove useful in the future for other  \
implementations. 


 M  +6 -0      collection/CollectionLocation.cpp  
 M  +12 -0     collection/CollectionLocation.h  
 M  +5 -0      collection/sqlcollection/SqlCollectionLocation.cpp  
 M  +1 -0      collection/sqlcollection/SqlCollectionLocation.h  
 M  +3 -11     dbus/amarokdbushandler.cpp  
 M  +1 -1      dbus/amarokdbushandler.h  
 M  +2 -2      dbus/org.kde.amarok.collection.xml  


--- trunk/extragear/multimedia/amarok/src/collection/CollectionLocation.cpp \
#815020:815021 @@ -59,6 +59,12 @@
     return QString();
 }
 
+QStringList
+CollectionLocation::actualLocation() const
+{
+    return QStringList();
+}
+
 bool
 CollectionLocation::isWriteable() const
 {
--- trunk/extragear/multimedia/amarok/src/collection/CollectionLocation.h \
#815020:815021 @@ -54,6 +54,9 @@
 
     Implementations which are only readable can reimplement getKIOCopyableUrls( \
                Meta::TrackList )
     if it is necessary, but can use the default implementations if possible.
+
+    Implementations that have a string expressable location(s), such as a URL or \
path on disk +    should reimplement actualLocation(). 
  
     Implementations that need additional information provided by the user have to \
                implement
     showSourceDialog() and showDestinationDialog(), depending on whether the \
information is required @@ -96,6 +99,15 @@
         virtual QString prettyLocation() const;
 
         /**
+            Returns a list of machine usable strings representingthe collection \
location. For example, +            a local collection would return a list of paths \
where tracks are stored, while an Ampache  +            collection would return a \
list with one string containing the URL of an ampache server. +            An iPod \
collection and a MTP device collection are examples of collections that do +          \
not need to reimplement this method. +        */
+        virtual QStringList actualLocation() const;
+
+        /**
             Returns whether the collection location is writeable or not. For \
                example, a local collection or an ipod
             collection would return true, a daap collection or a service collection \
                false. The value returned by this
             method indicates if it is possible to copy tracks to the collection, and \
                if it is generally possible to
--- trunk/extragear/multimedia/amarok/src/collection/sqlcollection/SqlCollectionLocation.cpp \
#815020:815021 @@ -62,6 +62,11 @@
     return i18n( "Local Collection" );
 }
 
+QStringList
+SqlCollectionLocation::actualLocation() const
+{
+    return MountPointManager::instance()->collectionFolders();
+}
 bool
 SqlCollectionLocation::isWriteable() const
 {
--- trunk/extragear/multimedia/amarok/src/collection/sqlcollection/SqlCollectionLocation.h \
#815020:815021 @@ -36,6 +36,7 @@
         virtual ~SqlCollectionLocation();
 
         virtual QString prettyLocation() const;
+        virtual QStringList actualLocation() const;
         virtual bool isWriteable() const;
         virtual bool isOrganizable() const;
         virtual bool remove( const Meta::TrackPtr &track );
--- trunk/extragear/multimedia/amarok/src/dbus/amarokdbushandler.cpp #815020:815021
@@ -29,6 +29,7 @@
 #include "collection/CollectionManager.h"
 #include "collection/SqlStorage.h"
 #include "collection/sqlcollection/SqlCollection.h"
+#include "collection/sqlcollection/SqlCollectionLocation.h"
 #include "context/LyricsManager.h"
 #include "EngineController.h"
 #include "equalizersetup.h"
@@ -848,18 +849,9 @@
         Q_UNUSED( path );
         return false;
     }
-    QString DbusCollectionHandler::mainCollectionPath()
+    QStringList DbusCollectionHandler::collectionLocation()
     {
-        /*The main collection path is defined as the first path encountered*/
-        QStringList folders = MountPointManager::instance()->collectionFolders();
-
-        QString firstPath = "NONE";
-
-        if(!folders.isEmpty()) {
-            firstPath = folders[0];
-        }
-
-        return firstPath;
+        return CollectionManager::instance()->primaryCollection()->location()->actualLocation();
  }
     bool DbusCollectionHandler::moveFile( const QString &oldURL, const QString \
&newURL, bool overwrite )  {
--- trunk/extragear/multimedia/amarok/src/dbus/amarokdbushandler.h #815020:815021
@@ -183,7 +183,7 @@
       virtual int totalGenres();
       virtual int totalTracks();
       virtual bool isDirInCollection( const QString &path );
-      virtual QString mainCollectionPath();
+      virtual QStringList collectionLocation();
       virtual bool moveFile( const QString &oldURL, const QString &newURL, bool \
overwrite );  virtual QStringList query(const QString& sql);
       virtual QStringList similarArtists( int artists );
--- trunk/extragear/multimedia/amarok/src/dbus/org.kde.amarok.collection.xml \
#815020:815021 @@ -56,8 +56,8 @@
       <arg name="url" type="s" direction="in"/>
       <arg name="oldLabel" type="as" direction="in"/>
     </method>
-    <method name="mainCollectionPath" >
-        <arg type="s" direction="out"/>
+    <method name="collectionLocation" >
+        <arg type="as" direction="out"/>
     </method>
   </interface>
 </node>


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