Changeset 173

Timestamp:

Sep 9, 2009, 2:38:34 AM (16 years ago)

Author:

Dmitry A. Kuminov

Message:

gui/dialogs: Fixed: Duplicate file names were added to the file list in QFileDialog when an existing directory was typed directly in the filename entry field with a different case of letters than in the original (on platforms with case insensitive file systems), #65.

Location:

trunk/src/gui/dialogs

Files:

• qfilesystemmodel.cpp (modified) (3 diffs)
• qfilesystemmodel_p.h (modified) (5 diffs)

trunk/src/gui/dialogs/qfilesystemmodel.cpp

@@ -353 +353 @@
         absolutePath = QDir(longPath).absolutePath();
 
-    // ### TODO can we use bool QAbstractFileEngine::caseSensitive() const?
     QStringList pathElements = absolutePath.split(QLatin1Char('/'), QString::SkipEmptyParts);
     if ((pathElements.isEmpty())
@@ -1089 +1088 @@
 
     QList<QPair<QFileSystemModelPrivate::QFileSystemNode*, int> > values;
-    QHash<QString, QFileSystemNode *>::const_iterator iterator;
+    QHash<, QFileSystemNode *>::const_iterator iterator;
     int i = 0;
     for(iterator = indexNode->children.begin() ; iterator != indexNode->children.end() ; ++iterator) {
@@ -1588 +1587 @@
     QStringList newFiles = files;
     qSort(newFiles.begin(), newFiles.end());
-    QHash<QString, QFileSystemNode*>::const_iterator i = parentNode->children.constBegin();
+    QHash<, QFileSystemNode*>::const_iterator i = parentNode->children.constBegin();
     while (i != parentNode->children.constEnd()) {
         QStringList::iterator iterator;

trunk/src/gui/dialogs/qfilesystemmodel_p.h

@@ -81 +81 @@
 
 public:
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
     class QFileSystemNode
     {
@@ -87 +110 @@
             : fileName(filename), populatedChildren(false), isVisible(false), parent(p), info(0) {}
         ~QFileSystemNode() {
-            QHash<QString, QFileSystemNode*>::const_iterator i = children.constBegin();
+            QHash<, QFileSystemNode*>::const_iterator i = children.constBegin();
             while (i != children.constEnd()) {
                     delete i.value();
@@ -162 +185 @@
             if (info)
                 info->icon = iconProvider->icon(QFileInfo(path));
-            QHash<QString, QFileSystemNode *>::const_iterator iterator;
+            QHash<, QFileSystemNode *>::const_iterator iterator;
             for(iterator = children.constBegin() ; iterator != children.constEnd() ; ++iterator) {
                 iterator.value()->updateIcon(iconProvider, path + QLatin1Char('/') + iterator.value()->fileName);
@@ -171 +194 @@
             if (info)
                 info->displayType = iconProvider->type(QFileInfo(path));
-            QHash<QString, QFileSystemNode *>::const_iterator iterator;
+            QHash<, QFileSystemNode *>::const_iterator iterator;
             for(iterator = children.constBegin() ; iterator != children.constEnd() ; ++iterator) {
                  iterator.value()->retranslateStrings(iconProvider, path + QLatin1Char('/') + iterator.value()->fileName);
@@ -179 +202 @@
         bool populatedChildren;
         bool isVisible;
-        QHash<QString,QFileSystemNode *> children;
+        QHash<QFileSystemNode *> children;
         QList<QString> visibleChildren;
         QFileSystemNode *parent;