Fixup Fix for MergeActivityWizard crash

.. TRUE/FALSE are case sensitive on Mac
.. ride2Loaded renamed ridesLoaded
.. ridesLoaded initialisation moved to constructor

Thanks to Jorn for the fix to a P0 bug !
This commit is contained in:
Mark Liversedge
2014-05-30 10:24:24 +01:00
parent 1f67774c53
commit 3607f1833b
2 changed files with 4 additions and 6 deletions

View File

@@ -72,7 +72,7 @@ MergeWelcome::MergeWelcome(MergeActivityWizard *parent) : QWizardPage(parent), w
}
// Upload
MergeUpload::MergeUpload(MergeActivityWizard *parent) : QWizardPage(parent), wizard(parent)
MergeUpload::MergeUpload(MergeActivityWizard *parent) : QWizardPage(parent), wizard(parent), ridesLoaded(false)
{
setTitle(tr("Upload"));
setSubTitle(tr("Select file to merge"));
@@ -132,8 +132,6 @@ MergeUpload::importFile()
void
MergeUpload::importFile(QList<QString> files)
{
// get fullpath name for processing
QFileInfo filename = QFileInfo(files[0]).absoluteFilePath();
@@ -159,7 +157,7 @@ MergeUpload::importFile(QList<QString> files)
ride2Label->setText(tr("Second ride")+" "+ride->startTime().toString(tr("MMM d, yyyy - hh:mm:ss")));
wizard->ride2 = ride;
ride2Loaded = TRUE;
ridesLoaded = true;
emit completeChanged();
}
@@ -174,7 +172,7 @@ MergeUpload::importFile(QList<QString> files)
bool MergeUpload::isComplete() const
{
return ride2Loaded;
return ridesLoaded;
}

View File

@@ -147,7 +147,7 @@ class MergeUpload : public QWizardPage
QPushButton *uploadButton;
QLabel *labelSuccess, *ride2Label;
bool isComplete() const;
bool ride2Loaded = FALSE;
bool ridesLoaded;
private slots:
void importFile();