diff options
author | Arno <am@disconnect.de> | 2011-05-13 16:12:23 +0200 |
---|---|---|
committer | Arno <am@disconnect.de> | 2011-05-13 16:12:23 +0200 |
commit | 5ca7f74c5b66cb9496f8a6aa4ecbb1f687247995 (patch) | |
tree | 31ca52a1cd70d14e4149f6f3eecebe6e43e68574 /newmoviewizard.cpp | |
parent | a4fa8d60eb78d410e3c51cb801a0537c890257ba (diff) | |
download | SheMov-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.
Diffstat (limited to 'newmoviewizard.cpp')
-rw-r--r-- | newmoviewizard.cpp | 18 |
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; |