Bug 124952

I can confirm this bug and I found the problem, at least for the second part of the problem description above (not being able to use the "Media Browse" menu items.

When the QPopupMenu list is created, the parameters for "Accelerator" and "MenuID" are swapped. See patch extract below for a fix. Now the media browse works fine.

            mCameraMediaList->insertItem( name,  this,
-                              SLOT(slotDownloadImagesFromMedia( int )),i,0);
+                              SLOT(slotDownloadImagesFromMedia( int )),0,i);

Additional Hint: SuSE has the "subfs" automounter installed (at least from version 10.0 on, which I use, but I think earlier version do have it, too) which automatically mounts cdroms, dvd und usb storage devics under the "/media" directory. With this enabled, the scheme of accessing new media via "media:/" KURL doesn't work, but one has to go directly to "/media". 
I don't know whether the media access path would be better a configurable item in digikam, so that the user could change it from "media:/" to "/media" in such situations. I could provide a simple patch for supporting this if someone is interested.

Chris

SVN commit 540108 by cgilles:

digikam from stable : fix swapped parameters for "Accelerator" and "MenuID" when QPopupMenu list is created .
CCBUGS: 124952

 M  +12 -7     digikamapp.cpp  


--- branches/stable/extragear/graphics/digikam/digikam/digikamapp.cpp #540107:540108
@@ -1050,24 +1050,30 @@
 
 void DigikamApp::slotCameraMediaMenuEntries( Job *, const UDSEntryList & list )
 {
-    int i=0;
-    for(KIO::UDSEntryList::ConstIterator it = list.begin();
-                  it!=list.end(); ++it)
+    int i = 0;
+
+    for(KIO::UDSEntryList::ConstIterator it = list.begin() ; it != list.end(); ++it)
     {
         QString name;
         QString path;
-        bool unmounted=false;
-        for ( UDSEntry::const_iterator et = (*it).begin() ; et !=   (*it).end() ; ++ et ) {
+        bool unmounted = false;
+
+        for ( UDSEntry::const_iterator et = (*it).begin() ; et !=   (*it).end() ; ++ et ) 
+        {
             if ( (*et).m_uds == KIO::UDS_NAME)
                 name = ( *et ).m_str;
+
             if ( (*et).m_uds == KIO::UDS_URL)
                 path = ( *et ).m_str;
+
             if ( (*et).m_uds == KIO::UDS_MIME_TYPE &&
                    ( (*et).m_str == "media/removable_unmounted" || 
                      (*et).m_str == "media/camera_unmounted" ) )
                 unmounted=true;
+
             //kdDebug() << ( *et ).m_str << unmounted << endl;
        }
+
        if (!name.isEmpty() && !path.isEmpty())
        {
             if (i==0)
@@ -1075,8 +1081,7 @@
             
             mMediaItems[i] = qMakePair(path,unmounted);
             
-            mCameraMediaList->insertItem( name,  this, 
-                               SLOT(slotDownloadImagesFromMedia( int )),i,0);
+            mCameraMediaList->insertItem(name, this, SLOT(slotDownloadImagesFromMedia(int)), 0, i);
             mCameraMediaList->setItemParameter(i, i);
             i++;
        }

SVN commit 540109 by cgilles:

digikam from trunk : fix swapped parameters for "Accelerator" and "MenuID" when QPopupMenu list is created .
CCBUGS: 124952

 M  +8 -7      digikamapp.cpp  


--- trunk/extragear/graphics/digikam/digikam/digikamapp.cpp #540108:540109
@@ -1032,30 +1032,31 @@
 
 void DigikamApp::slotCameraMediaMenuEntries( Job *, const UDSEntryList & list )
 {
-    int i=0;
-    for(KIO::UDSEntryList::ConstIterator it = list.begin();
-                  it!=list.end(); ++it)
+    int i = 0;
+
+    for(KIO::UDSEntryList::ConstIterator it = list.begin() ; it!=list.end() ; ++it)
     {
         QString name;
         QString path;
-        for ( UDSEntry::const_iterator et = (*it).begin() ; et !=   (*it).end() ; ++ et ) 
+
+        for ( UDSEntry::const_iterator et = (*it).begin() ; et !=   (*it).end() ; ++et ) 
         {
             if ( (*et).m_uds == KIO::UDS_NAME)
                 name = ( *et ).m_str;
             if ( (*et).m_uds == KIO::UDS_URL)
                 path = ( *et ).m_str;
+
             kdDebug() << ( *et ).m_str << endl;
         }
        
         if (!name.isEmpty() && !path.isEmpty())
         {
-            if (i==0)
+            if (i == 0)
                 mCameraMediaList->clear();
             
             mMediaItems[i] = path;
             
-            mCameraMediaList->insertItem( name,  this, 
-                              SLOT(slotDownloadImagesFromMedia( int )),i,0);
+            mCameraMediaList->insertItem(name, this, SLOT(slotDownloadImagesFromMedia(int)), 0, i);
             mCameraMediaList->setItemParameter(i, i);
             i++;
         }

I can report the same first problem with the latest Dapper package (digikam 0.8.2-rc1)

For me the mount and download of photos works now using KDE 3.5.2 and suse10.0.