- BUGFIX: RSS filters are now alphabetically sorted

- BUGFIX: Fix crash when renaming currently displayed RSS filter
    - BUGFIX: Remove overwriting confirmation when exporting RSS filters since Qt takes care of it
This commit is contained in:
Christophe Dumez
2010-01-11 14:47:01 +00:00
parent fa09f5f48a
commit 304c20d3d8
2 changed files with 10 additions and 2 deletions

View File

@@ -4,6 +4,9 @@
- BUGFIX: RSS Feed downloader ignores articles above maximum number of articles - BUGFIX: RSS Feed downloader ignores articles above maximum number of articles
- BUGFIX: Fix possible bug when deleting a RSS folder - BUGFIX: Fix possible bug when deleting a RSS folder
- BUGFIX: Remove persistant data when a RSS feed is deleted - BUGFIX: Remove persistant data when a RSS feed is deleted
- BUGFIX: RSS filters are now alphabetically sorted
- BUGFIX: Fix crash when renaming currently displayed RSS filter
- BUGFIX: Remove overwriting confirmation when exporting RSS filters since Qt takes care of it
* Tue Jan 5 2009 - Christophe Dumez <chris@qbittorrent.org> - v2.0.6 * Tue Jan 5 2009 - Christophe Dumez <chris@qbittorrent.org> - v2.0.6
- BUGFIX: Fix detection of invalid torrent files - BUGFIX: Fix detection of invalid torrent files

View File

@@ -257,6 +257,7 @@ public:
// Restore saved info // Restore saved info
enableDl_cb->setChecked(filters.isDownloadingEnabled()); enableDl_cb->setChecked(filters.isDownloadingEnabled());
fillFiltersList(); fillFiltersList();
filtersList->sortItems(Qt::AscendingOrder);
if(filters.size() > 0) { if(filters.size() > 0) {
// Select first filter // Select first filter
filtersList->setCurrentItem(filtersList->item(0)); filtersList->setCurrentItem(filtersList->item(0));
@@ -376,11 +377,14 @@ protected slots:
QMessageBox::warning(0, tr("Invalid filter name"), tr("This filter name is already in use.")); QMessageBox::warning(0, tr("Invalid filter name"), tr("This filter name is already in use."));
} }
}while(!validated); }while(!validated);
// Save the current filter
saveCurrentFilterSettings();
// Rename the filter // Rename the filter
filters.rename(current_name, new_name); filters.rename(current_name, new_name);
if(selected_filter == current_name) if(selected_filter == current_name)
selected_filter = new_name; selected_filter = new_name;
item->setText(new_name); item->setText(new_name);
filtersList->sortItems(Qt::AscendingOrder);
} }
} }
@@ -434,6 +438,7 @@ protected slots:
} }
}while(!validated); }while(!validated);
QListWidgetItem *it = new QListWidgetItem(filter_name, filtersList); QListWidgetItem *it = new QListWidgetItem(filter_name, filtersList);
filtersList->sortItems(Qt::AscendingOrder);
filtersList->setCurrentItem(it); filtersList->setCurrentItem(it);
//showFilterSettings(it); //showFilterSettings(it);
} }
@@ -489,10 +494,10 @@ protected slots:
// Append file extension // Append file extension
if(!destination.endsWith(".filters")) if(!destination.endsWith(".filters"))
destination += ".filters"; destination += ".filters";
if(QFile::exists(destination)) { /*if(QFile::exists(destination)) {
int ret = QMessageBox::question(0, tr("Overwriting confirmation"), tr("Are you sure you want to overwrite existing file?"), QMessageBox::Yes|QMessageBox::No); int ret = QMessageBox::question(0, tr("Overwriting confirmation"), tr("Are you sure you want to overwrite existing file?"), QMessageBox::Yes|QMessageBox::No);
if(ret != QMessageBox::Yes) return; if(ret != QMessageBox::Yes) return;
} }*/
if(filters.serialize(destination)) if(filters.serialize(destination))
QMessageBox::information(0, tr("Export successful"), tr("Filters export was successful.")); QMessageBox::information(0, tr("Export successful"), tr("Filters export was successful."));
else else