Usability: disable subtitle when partNo is selected

Found several parts in the database which had both a subtitle and a
partno -> wrong! Prevent it by disabling and clearing the subtitle field
when hasPartno is checked.

1 files changed, 14 insertions, 4 deletions

diff --git a/newmoviewizard.cpp b/newmoviewizard.cpp
index 470297e..acc6375 100644
--- a/newmoviewizard.cpp
+++ b/newmoviewizard.cpp
@@ -100,11 +100,12 @@ void NewMovieWizard::accept(){
 		QMessageBox::critical(this, tr("Error"), tr("Failed to create series part!"));
 		return;
 	}
-	if(hasPartNo){
+    //model has changed, so we need to reevaluate...
+    if(hasPartNo){
 		seriesPartIdx = seriesModel->find(seriesno, SeriesTreeModel::SeriesPart, seriesIdx);
 	}else{
 		seriesPartIdx = seriesModel->find(subTitle, SeriesTreeModel::Subtitle, seriesIdx);
-	}
+    }
 	if(!seriesPartIdx.isValid()){
 		QMessageBox::critical(this, tr("Error"), tr("Strange. Could not find series part! Cowardly bailing out."));
 		return;
@@ -221,6 +222,7 @@ void MovieInfoPage::setupGui(){
 	QFormLayout *movieTitleLayout = new QFormLayout;
 	mTitle = new QLineEdit;
 	mSubtitle = new QLineEdit;
+    mSubtitle->setEnabled(false);
 	movieTitleLayout->addRow(tr("Movie &title"), mTitle);
 	movieTitleLayout->addRow(tr("Movie &subtitle"), mSubtitle);
 	SeriesTreeModel *seriesModel = static_cast<SeriesTreeModel*>(SmGlobals::instance()->model("SeriesModel"));
@@ -266,8 +268,8 @@ void MovieInfoPage::setupGui(){
 	mOnDvd = new QCheckBox(tr("Movie is on DVD"));
 	dvdLayout->addWidget(mOnDvd);
 	mHasPartno = new QCheckBox(tr("Movie has part no."));
-	mHasPartno->setChecked(true);
 	connect(mHasPartno, SIGNAL(stateChanged(int)), this, SLOT(hasPartnoStateChanged(int)));
+    mHasPartno->setChecked(true);
 	dvdLayout->addWidget(mHasPartno);
 	dvdLayout->addStretch();
 	QLabel *l6 = new QLabel(tr("Dvd no."));
@@ -484,7 +486,15 @@ void MovieInfoPage::itemClicked(const QModelIndex &index){
 }
 
 void MovieInfoPage::hasPartnoStateChanged(int state){
-	mPartno->setEnabled(state == Qt::Checked);
+    if(state == Qt::Checked){
+        mPartno->setEnabled(true);
+        mSubtitle->setText(tr("<fuck off!>"));
+        mSubtitle->setEnabled(false);
+    }else{
+        mPartno->setEnabled(false);
+        mSubtitle->clear();
+        mSubtitle->setEnabled(true);
+    }
 }
 
 MovieMappingPage::MovieMappingPage(const QString &table, QWidget *parent) : QWizardPage(parent){