fix crash when opening a lot of columns and going back very fast by clicking on each...

[dolphin.git] / src / statusbarspaceinfo.cpp

diff --git a/src/statusbarspaceinfo.cpp b/src/statusbarspaceinfo.cpp
index 3a2f77aef8c4ebf19541fc49c5fd8e31e0e1fb4d..79a1169e58dc610c6fb08ce77120e384165751a9 100644 (file)
--- a/src/statusbarspaceinfo.cpp
+++ b/src/statusbarspaceinfo.cpp
@@ -29,14 +29,13 @@
 #include <QKeyEvent>
 
 StatusBarSpaceInfo::StatusBarSpaceInfo(QWidget* parent) :
-    QProgressBar(parent),
+    KCapacityBar(KCapacityBar::DrawTextInline, parent),
     m_gettingSize(false),
-    m_text()
+    m_foundMountPoint(false),
+    m_kBSize(0)
 {
-    setMinimum(0);
-    setMaximum(0);
-
     setMaximumWidth(200);
+    setMinimumWidth(200); // something to fix on kcapacitybar (ereslibre)
 
     // Update the space information each 10 seconds. Polling is useful
     // here, as files can be deleted/added outside the scope of Dolphin.
@@ -55,34 +54,44 @@ void StatusBarSpaceInfo::setUrl(const KUrl& url)
     refresh();
 }
 
-QString StatusBarSpaceInfo::text() const
-{
-    return m_text;
-}
-
 void StatusBarSpaceInfo::slotFoundMountPoint(const QString& mountPoint,
                                              quint64 kBSize,
                                              quint64 kBUsed,
                                              quint64 kBAvailable)
 {
-    Q_UNUSED(kBSize);
     Q_UNUSED(mountPoint);
 
     m_gettingSize = false;
-    const bool valuesChanged = (kBUsed != static_cast<quint64>(value())) ||
-                               (kBAvailable != static_cast<quint64>(maximum()));
+    m_foundMountPoint = true;
+    const bool valuesChanged = (kBUsed != static_cast<quint64>(value())) || (kBSize != m_kBSize);
     if (valuesChanged) {
-        m_text = i18nc("@info:status", "%1 free", KIO::convertSizeFromKiB(kBAvailable));
-        setMaximum(kBSize);
-        setValue(kBUsed);
+        setText(i18nc("@info:status Free disk space", "%1 free", KIO::convertSize(kBAvailable * 1024)));
+        setUpdatesEnabled(false);
+        m_kBSize = kBSize;
+        setValue(kBSize > 0 ? (kBUsed * 100) / kBSize : 0);
+        setUpdatesEnabled(true);
+        update();
+    }
+}
+
+void StatusBarSpaceInfo::slotDiskFreeSpaceDone()
+{
+    if (m_foundMountPoint) {
+        return;
     }
+
+    m_gettingSize = false;
+    setText(i18nc("@info:status", "Unknown size"));
+    setValue(0);
+    update();
 }
 
 void StatusBarSpaceInfo::refresh()
 {
     // KDiskFreeSpace is for local paths only
     if (!m_url.isLocalFile()) {
-        m_text = i18nc("@info:status", "Unknown size");
+        setText(i18nc("@info:status", "Unknown size"));
+        setValue(0);
         update();
         return;
     }
@@ -93,6 +102,7 @@ void StatusBarSpaceInfo::refresh()
     }
 
     m_gettingSize = true;
+    m_foundMountPoint = false;
     KDiskFreeSpace* job = new KDiskFreeSpace(this);
     connect(job, SIGNAL(foundMountPoint(const QString&,
                                         quint64,
@@ -102,6 +112,7 @@ void StatusBarSpaceInfo::refresh()
                                            quint64,
                                            quint64,
                                            quint64)));
+    connect(job, SIGNAL(done()), this, SLOT(slotDiskFreeSpaceDone()));
 
     job->readDF(mp->mountPoint());
 
@@ -115,10 +126,9 @@ void StatusBarSpaceInfo::refresh()
 void StatusBarSpaceInfo::showGettingSizeInfo()
 {
     if (m_gettingSize) {
-        m_text = i18nc("@info:status", "Getting size...");
+        m_kBSize = 0;
+        setText(i18nc("@info:status", "Getting size..."));
         update();
-        setMinimum(0);
-        setMaximum(0);
     }
 }