Wow, giving fileItem.url().url() to KFileMetaInfo is a bad idea -- it takes that...

QFile("file://fish://hostname")... I guess the only solution is to skip the whole KFileMetaInfo stuff for non-local-files.
But then we shouldn't see size 0 modified 12:00 am...

svn path=/trunk/KDE/kdebase/apps/; revision=813953

diff --git a/src/infosidebarpage.cpp b/src/infosidebarpage.cpp
index 002b1ffdddea3f6e6143921f043f2ad7a3eb41db..d2201eb2b48a205f69a665bc9f23d5e474eafc51 100644 (file)
--- a/src/infosidebarpage.cpp
+++ b/src/infosidebarpage.cpp
@@ -314,26 +314,28 @@ void InfoSidebarPage::showMetaInfo()
             m_metaTextLabel->add(i18nc("@label", "Size:"), KIO::convertSize(fileItem.size()));
             m_metaTextLabel->add(i18nc("@label", "Modified:"), fileItem.timeString());
 
-            // TODO: See convertMetaInfo below, find a way to display only interesting information
-            // in a readable way
-            const KFileMetaInfo::WhatFlags flags = KFileMetaInfo::Fastest |
-                                                   KFileMetaInfo::TechnicalInfo |
-                                                   KFileMetaInfo::ContentInfo |
-                                                   KFileMetaInfo::Thumbnail;
-            const QString path = fileItem.url().url();
-            const KFileMetaInfo fileMetaInfo(path, QString(), flags);
-            if (fileMetaInfo.isValid()) {
-                const QHash<QString, KFileMetaInfoItem>& items = fileMetaInfo.items();
-                QHash<QString, KFileMetaInfoItem>::const_iterator it = items.constBegin();
-                const QHash<QString, KFileMetaInfoItem>::const_iterator end = items.constEnd();
-                QString labelText;
-                while (it != end) {
-                    const KFileMetaInfoItem& metaInfoItem = it.value();
-                    const QVariant& value = metaInfoItem.value();
-                    if (value.isValid() && convertMetaInfo(metaInfoItem.name(), labelText)) {
-                        m_metaTextLabel->add(labelText, value.toString());
+            if (fileItem.isLocalFile()) {
+                // TODO: See convertMetaInfo below, find a way to display only interesting information
+                // in a readable way
+                const KFileMetaInfo::WhatFlags flags = KFileMetaInfo::Fastest |
+                                                       KFileMetaInfo::TechnicalInfo |
+                                                       KFileMetaInfo::ContentInfo |
+                                                       KFileMetaInfo::Thumbnail;
+                const QString path = fileItem.url().path();
+                const KFileMetaInfo fileMetaInfo(path, QString(), flags);
+                if (fileMetaInfo.isValid()) {
+                    const QHash<QString, KFileMetaInfoItem>& items = fileMetaInfo.items();
+                    QHash<QString, KFileMetaInfoItem>::const_iterator it = items.constBegin();
+                    const QHash<QString, KFileMetaInfoItem>::const_iterator end = items.constEnd();
+                    QString labelText;
+                    while (it != end) {
+                        const KFileMetaInfoItem& metaInfoItem = it.value();
+                        const QVariant& value = metaInfoItem.value();
+                        if (value.isValid() && convertMetaInfo(metaInfoItem.name(), labelText)) {
+                            m_metaTextLabel->add(labelText, value.toString());
+                        }
+                        ++it;
                     }
-                    ++it;
                 }
             }
         }