summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorArno <am@disconnect.de>2011-05-13 16:12:23 +0200
committerArno <am@disconnect.de>2011-05-13 16:12:23 +0200
commit5ca7f74c5b66cb9496f8a6aa4ecbb1f687247995 (patch)
tree31ca52a1cd70d14e4149f6f3eecebe6e43e68574
parenta4fa8d60eb78d410e3c51cb801a0537c890257ba (diff)
downloadSheMov-5ca7f74c5b66cb9496f8a6aa4ecbb1f687247995.tar.gz
SheMov-5ca7f74c5b66cb9496f8a6aa4ecbb1f687247995.tar.bz2
SheMov-5ca7f74c5b66cb9496f8a6aa4ecbb1f687247995.zip
Fix NewMovieWizard - subtitle dupe checking
Don't exit when there's already a SeriesPart zero. Since introducing subtitle zero is an indicator for a subtitle.
-rw-r--r--newmoviewizard.cpp18
1 files changed, 14 insertions, 4 deletions
diff --git a/newmoviewizard.cpp b/newmoviewizard.cpp
index 88f0aab..44897b4 100644
--- a/newmoviewizard.cpp
+++ b/newmoviewizard.cpp
@@ -75,13 +75,23 @@ void NewMovieWizard::accept(){
if(!hasPartNo){
seriesno = 0;
}
+ QString subTitle = field("subtitle").toString();
QModelIndex seriesPartIdx = seriesModel->find(seriesno, SeriesTreeModel::SeriesPart, seriesIdx);
if(seriesPartIdx.isValid()){
- QString error = QString(tr("Already have part %1 of %2")).arg(seriesno).arg(series);
- QMessageBox::critical(this, tr("Error"), error);
- return;
+ if(hasPartNo){
+ QString error = QString(tr("Already have part %1 of %2")).arg(seriesno).arg(series);
+ QMessageBox::critical(this, tr("Error"), error);
+ return;
+ }else{
+ QString curSubTitle = seriesPartIdx.data(SeriesTreeModel::SubtitleRole).toString();
+ if(curSubTitle == subTitle){
+ QString error = QString(tr("Already have a part with subtitle %1")).arg(curSubTitle);
+ QMessageBox::critical(this, tr("Error"), error);
+ return;
+ }
+ }
}
- QString subTitle = field("subtitle").toString();
+
if(!seriesModel->addSeriesPart(seriesno, seriesIdx, subTitle)){
QMessageBox::critical(this, tr("Error"), tr("Failed to create series part!"));
return;