Changeset 11073

Timestamp:

Jul 31, 2010, 12:03:59 AM (12 years ago)

Author:

charles

Message:

(trunk qt) fix filterbar issue reported by Lacrocivious

File:

• trunk/qt/filterbar.cc (modified) (3 diffs)

trunk/qt/filterbar.cc

@@ -11 +11 @@
  */
 
+#include <iostream>
+
 #include <QString>
 #include <QtGui>
@@ -465 +467 @@
         case Prefs :: FILTER_TRACKERS: {
             const QString tracker = myPrefs.getString( key );
-            QList<QStandardItem*> rows = myTrackerModel->findItems( tracker );
-            if( !rows.isEmpty( )  )
-                myTrackerCombo->setCurrentIndex( rows.first()->row() );
-            else if( myTorrents.rowCount( ) > 0 ) // uh-oh... we don't have this tracker anymore.  best use "show all" as a fallback
-                myPrefs.set( key, "" );
+            QModelIndexList indices = myTrackerModel->match( myTrackerModel->index(0,0), TrackerRole, tracker, 1, Qt::MatchFixedString );
+            if( !indices.isEmpty( )  )
+                myTrackerCombo->setCurrentIndex( indices.first().row() );
+            else { // hm, we don't seem to have this tracker anymore...
+                const bool isBootstrapping = myTrackerModel->rowCount( ) <= 2;
+                if( !isBootstrapping )
+                    myPrefs.set( key, "" );
+            }
             break;
         }
@@ -490 +495 @@
 {
     if( !myIsBootstrapping )
-        myPrefs.set( Prefs::FILTER_TRACKERS, myTrackerCombo->itemData( i, Qt::DisplayRole ).toString( ) );
+    {
+        QString str;
+        const bool isTracker = !myTrackerCombo->itemData(i,TrackerRole).toString().isEmpty();
+        if( isTracker )
+            str = myTrackerCombo->itemData(i,TrackerRole).toString();
+        else // show all
+            str = "";
+        myPrefs.set( Prefs::FILTER_TRACKERS, str );
+    }
 }