feedc0de/QtZeiterfassung
1
0
Fork 0
Code Issues 28 Pull Requests Actions Packages Projects Releases 6 Wiki Activity

Fixed strip rendering and added strip error messages

Browse Source
This commit is contained in:
0xFEEDC0DE64
2017-12-02 12:41:49 +01:00
parent d41e8c3099
commit e5413c00a7
4 changed files with 114 additions and 74 deletions
Download Patch File Download Diff File Expand all files Collapse all files

169
mainwindow.cpp
View File

@@ -8,6 +8,7 @@
#include <QStandardItem> #include <QStandardItem>
#include <QStringBuilder> #include <QStringBuilder>
#include <QMenu> #include <QMenu>
#include <QLabel>
#include <QDebug> #include <QDebug>
#include "eventloopwithstatus.h" #include "eventloopwithstatus.h"
@@ -586,7 +587,6 @@ void MainWindow::pushButtonStartPressed()
connect(&m_erfassung, &Zeiterfassung::updateKontierungFinished, &eventLoop, &EventLoopWithStatus::quitWithStatus); connect(&m_erfassung, &Zeiterfassung::updateKontierungFinished, &eventLoop, &EventLoopWithStatus::quitWithStatus);
auto timespan = timeBetween(m_lastKontierungStart, ui->timeEditTime->time()); auto timespan = timeBetween(m_lastKontierungStart, ui->timeEditTime->time());
qDebug() << "timespan" << timespan;
m_erfassung.doUpdateKontierung(kontierung.id, m_userInfo.userId, kontierung.date, m_erfassung.doUpdateKontierung(kontierung.id, m_userInfo.userId, kontierung.date,
kontierung.time, timespan, kontierung.time, timespan,
@@ -684,7 +684,6 @@ void MainWindow::pushButtonEndPressed()
connect(&m_erfassung, &Zeiterfassung::updateKontierungFinished, &eventLoop, &EventLoopWithStatus::quitWithStatus); connect(&m_erfassung, &Zeiterfassung::updateKontierungFinished, &eventLoop, &EventLoopWithStatus::quitWithStatus);
auto timespan = timeBetween(m_lastKontierungStart, ui->timeEditTime->time()); auto timespan = timeBetween(m_lastKontierungStart, ui->timeEditTime->time());
qDebug() << "timespan" << timespan;
m_erfassung.doUpdateKontierung(kontierung.id, m_userInfo.userId, kontierung.date, m_erfassung.doUpdateKontierung(kontierung.id, m_userInfo.userId, kontierung.date,
kontierung.time, timespan, kontierung.time, timespan,
@@ -724,6 +723,8 @@ void MainWindow::validateEntries()
m_kontierungTime = QTime(0, 0); m_kontierungTime = QTime(0, 0);
auto buchungTimespan = QTime(0, 0); auto buchungTimespan = QTime(0, 0);
QString errorMessage;
while(true) while(true)
{ {
if(buchungenIter == m_buchungenModel->constEnd() && if(buchungenIter == m_buchungenModel->constEnd() &&
@@ -741,18 +742,17 @@ void MainWindow::validateEntries()
if(buchungenIter == m_buchungenModel->constEnd()) if(buchungenIter == m_buchungenModel->constEnd())
{ {
QMessageBox::warning(this, tr("Illegal state!"), tr("Your Buchungen and Kontierungen for this day are in an invalid state:\n\n%0") errorMessage = tr("Missing Buchung.");
.arg(tr("Missing Buchung."))); goto after;
return;
} }
auto startBuchung = *buchungenIter++; auto startBuchung = *buchungenIter++;
qDebug() << "startBuchung" << startBuchung.time;
if(startBuchung.type != QStringLiteral("K")) if(startBuchung.type != QStringLiteral("K"))
{ {
QMessageBox::warning(this, tr("Illegal state!"), tr("Your Buchungen and Kontierungen for this day are in an invalid state:\n\n%0") errorMessage = tr("Expected Buchung for Kommen, instead got type %0\nBuchung ID: %1")
.arg(tr("Expected Buchung for Kommen, instead got type %0\nBuchung ID: %1").arg(startBuchung.type).arg(startBuchung.id))); .arg(startBuchung.type)
return; .arg(startBuchung.id);
goto after;
} }
m_lastKontierungStart = startBuchung.time; m_lastKontierungStart = startBuchung.time;
@@ -760,20 +760,18 @@ void MainWindow::validateEntries()
if(kontierungenIter == m_kontierungenModel->constEnd()) if(kontierungenIter == m_kontierungenModel->constEnd())
{ {
QMessageBox::warning(this, tr("Illegal state!"), tr("Your Buchungen and Kontierungen for this day are in an invalid state:\n\n%0") errorMessage = tr("Missing Kontierung.");
.arg(tr("Missing Kontierung."))); goto after;
return;
} }
auto kontierung = *kontierungenIter++; auto kontierung = *kontierungenIter++;
if(kontierung.time != m_kontierungTime) if(kontierung.time != m_kontierungTime)
{ {
QMessageBox::warning(this, tr("Illegal state!"), tr("Your Buchungen and Kontierungen for this day are in an invalid state:\n\n%0") errorMessage = tr("Expected time %0 but got %1 Kontierung.\nKontierung ID: %2")
.arg(tr("Expected time %0 but got %1 Kontierung.\nKontierung ID: %2") .arg(m_kontierungTime.toString("HH:mm:ss"))
.arg(m_kontierungTime.toString("HH:mm:ss")) .arg(kontierung.time.toString("HH:mm:ss"))
.arg(kontierung.time.toString("HH:mm:ss")) .arg(kontierung.id);
.arg(kontierung.id))); goto after;
return;
} }
ui->verticalLayout2->addWidget(new KontierungStrip(kontierung.id, kontierung.timespan, kontierung.projekt, kontierung.subprojekt, ui->verticalLayout2->addWidget(new KontierungStrip(kontierung.id, kontierung.timespan, kontierung.projekt, kontierung.subprojekt,
@@ -783,20 +781,18 @@ void MainWindow::validateEntries()
{ {
if(buchungenIter != m_buchungenModel->constEnd()) if(buchungenIter != m_buchungenModel->constEnd())
{ {
QMessageBox::warning(this, tr("Illegal state!"), tr("Your Buchungen and Kontierungen for this day are in an invalid state:\n\n%0") errorMessage = tr("There is another Buchung after an unfinished Kontierung.\nBuchung ID: %0\nKontierung ID: %1")
.arg(tr("There is another Buchung after an unfinished Kontierung.\nBuchung ID: %0\nKontierung ID: %1") .arg(buchungenIter->id)
.arg(buchungenIter->id) .arg(kontierung.id);
.arg(kontierung.id))); goto after;
return;
} }
if(kontierungenIter != m_kontierungenModel->constEnd()) if(kontierungenIter != m_kontierungenModel->constEnd())
{ {
QMessageBox::warning(this, tr("Illegal state!"), tr("Your Buchungen and Kontierungen for this day are in an invalid state:\n\n%0") errorMessage = tr("There is another Kontierung after an unfinished Kontierung.\nKontierung ID: %0\nKontierung ID: %1")
.arg(tr("There is another Kontierung after an unfinished Kontierung.\nKontierung ID: %0\nKontierung ID: %1") .arg(kontierungenIter->id)
.arg(kontierungenIter->id) .arg(kontierung.id);
.arg(kontierung.id))); goto after;
return;
} }
ui->pushButtonStart->setText(tr("Switch")); ui->pushButtonStart->setText(tr("Switch"));
@@ -814,32 +810,33 @@ void MainWindow::validateEntries()
{ {
if(kontierungenIter == m_kontierungenModel->constEnd()) if(kontierungenIter == m_kontierungenModel->constEnd())
{ {
QMessageBox::warning(this, tr("Illegal state!"), tr("Your Buchungen and Kontierungen for this day are in an invalid state:\n\n%0") errorMessage = tr("The last Kontierung is finished without Gehen-Buchung\nKontierung ID: %0")
.arg(tr("The last Kontierung is finished without Gehen-Buchung\nKontierung ID: %0") .arg(kontierung.id);
.arg(kontierung.id))); goto after;
return;
} }
kontierung = *kontierungenIter++; kontierung = *kontierungenIter++;
if(kontierung.time != m_kontierungTime) if(kontierung.time != m_kontierungTime)
{ {
QMessageBox::warning(this, tr("Illegal state!"), tr("Your Buchungen and Kontierungen for this day are in an invalid state:\n\n%0") errorMessage = tr("Expected time %0 but got %1 Kontierung.\nKontierung ID: %2")
.arg(tr("Expected time %0 but got %1 Kontierung.\nKontierung ID: %2") .arg(m_kontierungTime.toString("HH:mm:ss"))
.arg(m_kontierungTime.toString("HH:mm:ss")) .arg(kontierung.time.toString("HH:mm:ss"))
.arg(kontierung.time.toString("HH:mm:ss")) .arg(kontierung.id);
.arg(kontierung.id))); goto after;
return;
} }
ui->verticalLayout2->addWidget(new KontierungStrip(kontierung.id, kontierung.timespan, kontierung.projekt, kontierung.subprojekt,
kontierung.workpackage, kontierung.text, ui->scrollAreaWidgetContents));
if(kontierung.timespan == QTime(0, 0)) if(kontierung.timespan == QTime(0, 0))
{ {
if(kontierungenIter != m_kontierungenModel->constEnd()) if(kontierungenIter != m_kontierungenModel->constEnd())
{ {
QMessageBox::warning(this, tr("Illegal state!"), tr("Your Buchungen and Kontierungen for this day are in an invalid state:\n\n%0") errorMessage = tr("There is another Kontierung after an unfinished Kontierung.\n"
.arg(tr("There is another Kontierung after an unfinished Kontierung.\nKontierung ID: %0\nKontierung ID: %1") "Kontierung ID: %0\nKontierung ID: %1")
.arg(kontierung.id) .arg(kontierung.id)
.arg(kontierungenIter->id))); .arg(kontierungenIter->id);
return; goto after;
} }
ui->pushButtonStart->setText(tr("Switch")); ui->pushButtonStart->setText(tr("Switch"));
@@ -856,57 +853,56 @@ void MainWindow::validateEntries()
else else
{ {
auto endBuchung = *buchungenIter++; auto endBuchung = *buchungenIter++;
qDebug() << "endBuchung" << endBuchung.time;
if(endBuchung.type != QStringLiteral("G")) if(endBuchung.type != QStringLiteral("G"))
{ {
QMessageBox::warning(this, tr("Illegal state!"), tr("Your Buchungen and Kontierungen for this day are in an invalid state:\n\n%0") errorMessage = tr("Expected Buchung for Gehen, instead got type %0\nBuchung ID: %1")
.arg(tr("Expected Buchung for Gehen, instead got type %0\nBuchung ID: %1").arg(endBuchung.type).arg(endBuchung.id))); .arg(endBuchung.type)
return; .arg(endBuchung.id);
goto after;
} }
buchungTimespan = timeAdd(buchungTimespan, timeBetween(startBuchung.time, endBuchung.time)); buchungTimespan = timeAdd(buchungTimespan, timeBetween(startBuchung.time, endBuchung.time));
qDebug() << "buchungTimespan" << buchungTimespan;
while(m_kontierungTime < buchungTimespan) while(m_kontierungTime < buchungTimespan)
{ {
if(kontierungenIter == m_kontierungenModel->constEnd()) if(kontierungenIter == m_kontierungenModel->constEnd())
{ {
QMessageBox::warning(this, tr("Illegal state!"), tr("Your Buchungen and Kontierungen for this day are in an invalid state:\n\n%0") errorMessage = tr("Missing Kontierung! Time not filled: %0 - %1")
.arg(tr("Missing Kontierung! Time not filled: %0 - %1") .arg(m_kontierungTime.toString("HH:mm:ss"))
.arg(m_kontierungTime.toString("HH:mm:ss")) .arg(buchungTimespan.toString("HH:mm:ss"));
.arg(buchungTimespan.toString("HH:mm:ss")))); goto after;
return;
} }
kontierung = *kontierungenIter++; kontierung = *kontierungenIter++;
if(kontierung.time != m_kontierungTime) if(kontierung.time != m_kontierungTime)
{ {
QMessageBox::warning(this, tr("Illegal state!"), tr("Your Buchungen and Kontierungen for this day are in an invalid state:\n\n%0") errorMessage = tr("Expected time %0 but got %1 Kontierung.\nKontierung ID: %2")
.arg(tr("Expected time %0 but got %1 Kontierung.\nKontierung ID: %2") .arg(m_kontierungTime.toString("HH:mm:ss"))
.arg(m_kontierungTime.toString("HH:mm:ss")) .arg(kontierung.time.toString("HH:mm:ss"))
.arg(kontierung.time.toString("HH:mm:ss")) .arg(kontierung.id);
.arg(kontierung.id))); goto after;
return;
} }
ui->verticalLayout2->addWidget(new KontierungStrip(kontierung.id, kontierung.timespan, kontierung.projekt, kontierung.subprojekt,
kontierung.workpackage, kontierung.text, ui->scrollAreaWidgetContents));
if(kontierung.timespan == QTime(0, 0)) if(kontierung.timespan == QTime(0, 0))
{ {
if(buchungenIter != m_buchungenModel->constEnd()) if(buchungenIter != m_buchungenModel->constEnd())
{ {
QMessageBox::warning(this, tr("Illegal state!"), tr("Your Buchungen and Kontierungen for this day are in an invalid state:\n\n%0") errorMessage = tr("There is another Buchung after an unfinished Kontierung.\n"
.arg(tr("There is another Buchung after an unfinished Kontierung.\nBuchung ID: %0\nKontierung ID: %1") "Buchung ID: %0\nKontierung ID: %1")
.arg(buchungenIter->id) .arg(buchungenIter->id)
.arg(kontierung.id))); .arg(kontierung.id);
return; goto after;
} }
if(kontierungenIter != m_kontierungenModel->constEnd()) if(kontierungenIter != m_kontierungenModel->constEnd())
{ {
QMessageBox::warning(this, tr("Illegal state!"), tr("Your Buchungen and Kontierungen for this day are in an invalid state:\n\n%0") errorMessage = tr("There is another Kontierung after an unfinished Kontierung.\nKontierung ID: %0\nKontierung ID: %1")
.arg(tr("There is another Kontierung after an unfinished Kontierung.\nKontierung ID: %0\nKontierung ID: %1") .arg(kontierungenIter->id)
.arg(kontierungenIter->id) .arg(kontierung.id);
.arg(kontierung.id))); goto after;
return;
} }
ui->pushButtonStart->setText(tr("Switch")); ui->pushButtonStart->setText(tr("Switch"));
@@ -919,14 +915,45 @@ void MainWindow::validateEntries()
} }
} }
if(m_kontierungTime > buchungTimespan)
{
auto label = new QLabel(tr("Kontierung timespan too long!"), ui->scrollAreaWidgetContents);
label->setMinimumHeight(20);
label->setMaximumHeight(20);
ui->verticalLayout2->addWidget(label);
}
ui->verticalLayout2->addWidget(new BuchungStrip(endBuchung.id, endBuchung.time, endBuchung.type, ui->scrollAreaWidgetContents)); ui->verticalLayout2->addWidget(new BuchungStrip(endBuchung.id, endBuchung.time, endBuchung.type, ui->scrollAreaWidgetContents));
if(m_kontierungTime > buchungTimespan)
{
errorMessage = tr("The Kontierung timespan is longer than the Kommen-Gehen timespan. "
"The tool does not support this yet!\nKontierung: %0\nBuchung: %1")
.arg(m_kontierungTime.toString("HH:mm:ss"))
.arg(buchungTimespan.toString("HH:mm:ss"));
goto after;
}
} }
} }
} }
after: if(!errorMessage.isEmpty())
{
auto label = new QLabel(tr("Strip rendering aborted due error. May not be complete!"), ui->scrollAreaWidgetContents);
label->setMinimumHeight(20);
label->setMaximumHeight(20);
ui->verticalLayout2->addWidget(label);
}
qDebug() << "m_kontierTime" << m_kontierungTime; after:
ui->verticalLayout2->addStretch(1);
if(!errorMessage.isEmpty())
{
QMessageBox::warning(this, tr("Illegal state!"), tr("Your Buchungen and Kontierungen for this day are in an invalid state:\n\n%0")
.arg(errorMessage));
return;
}
ui->timeEditTime->setEnabled(true); ui->timeEditTime->setEnabled(true);
ui->comboBoxProjekt->setEnabled(true); ui->comboBoxProjekt->setEnabled(true);

11
mainwindow.ui
View File

@@ -184,8 +184,15 @@
<string>Optimized view</string> <string>Optimized view</string>
</attribute> </attribute>
<widget class="QWidget" name="scrollAreaWidgetContents"> <widget class="QWidget" name="scrollAreaWidgetContents">
<layout class="QVBoxLayout" name="verticalLayout2" stretch="0,0"> <property name="geometry">
</layout> <rect>
<x>0</x>
<y>0</y>
<width>1218</width>
<height>452</height>
</rect>
</property>
<layout class="QVBoxLayout" name="verticalLayout2"/>
</widget> </widget>
</widget> </widget>
<widget class="QSplitter" name="splitter"> <widget class="QSplitter" name="splitter">

3
strips/buchungstrip.cpp
View File

@@ -9,6 +9,9 @@ BuchungStrip::BuchungStrip(int id, const QTime &time, const QString &type, QWidg
{ {
ui->setupUi(this); ui->setupUi(this);
setMinimumHeight(minimumSizeHint().height());
setMaximumHeight(minimumSizeHint().height());
ui->labelTime->setText(time.toString("HH:mm")); ui->labelTime->setText(time.toString("HH:mm"));
if(type == QStringLiteral("K")) if(type == QStringLiteral("K"))
{ {

5
strips/kontierungstrip.cpp
View File

@@ -13,9 +13,12 @@ KontierungStrip::KontierungStrip(int id, const QTime &duration, const QString &p
{ {
ui->setupUi(this); ui->setupUi(this);
setMinimumHeight(minimumSizeHint().height());
setMaximumHeight(minimumSizeHint().height());
setStyleSheet("background-color: #7FFFFF;"); setStyleSheet("background-color: #7FFFFF;");
ui->labelTime->setText(duration == QTime(0, 0) ? QStringLiteral("???") : duration.toString("HH:mm")); ui->labelTime->setText(duration == QTime(0, 0) ? tr("Open") : duration.toString("HH:mm"));
ui->labelProjekt->setText(projekt); ui->labelProjekt->setText(projekt);
ui->labelId->setText(QString::number(id)); ui->labelId->setText(QString::number(id));