diff --git a/src/gui/mainwindow.cpp b/src/gui/mainwindow.cpp index 8652de4..2631952 100644 --- a/src/gui/mainwindow.cpp +++ b/src/gui/mainwindow.cpp @@ -49,8 +49,7 @@ void MainWindow::on_actionEditSeller_triggered() auto dialog = std::make_unique(this); int retCode = dialog->exec(); - auto oldModel = ui_.salesView->model(); - ui_.salesView->setModel(nullptr); + delete ui_.salesView->model(); if (retCode == QDialog::Accepted) { marketplace_->sortSellers(); @@ -64,7 +63,6 @@ void MainWindow::on_actionEditSeller_triggered() } ui_.salesView->setModel(new SaleModel(getMarketplace(), ui_.salesView)); - delete oldModel; ui_.salesView->setColumnHidden(2, true); ui_.salesView->resizeColumnToContents(0); connect(static_cast(ui_.basketView->model()), &BasketModel::basketDataChanged, diff --git a/src/gui/sellerdialog.cpp b/src/gui/sellerdialog.cpp index 110244c..63f2461 100644 --- a/src/gui/sellerdialog.cpp +++ b/src/gui/sellerdialog.cpp @@ -9,9 +9,8 @@ SellerDialog::SellerDialog(QWidget* parent, Qt::WindowFlags f) : QDialog(parent, { ui_.setupUi(this); ui_.editButton->setVisible(false); - market_ = dynamic_cast(parent)->getMarketplace(); SellerModel* model = - new SellerModel(market_, ui_.tableView); + new SellerModel(dynamic_cast(parent)->getMarketplace(), ui_.tableView); ui_.tableView->setModel(model); ui_.tableView->setColumnHidden(0, true); // hide the uuid connect(ui_.newButton, &QPushButton::clicked, this, &SellerDialog::on_newButton_clicked); @@ -22,11 +21,6 @@ SellerDialog::SellerDialog(QWidget* parent, Qt::WindowFlags f) : QDialog(parent, &SellerDialog::onSellerViewSelectionChanged); } -SellerDialog::~SellerDialog() -{ - delete ui_.tableView->model(); -} - void SellerDialog::on_newButton_clicked() { ui_.tableView->reset(); @@ -44,14 +38,6 @@ void SellerDialog::on_deleteButton_clicked() if (selModel->hasSelection() == false) return; - if (market_->getSales().size() > 0) { - QMessageBox(QMessageBox::Icon::Warning, "Hinweis", - "Da die Verkaufsphase schon begonnen hat (Artikel wurden bereits verkauft) können Sie keine Verkäufer mehr löschen.", - QMessageBox::StandardButton::Ok, this) - .exec(); - return; - } - auto dlgResult = QMessageBox(QMessageBox::Icon::Warning, "Sind Sie sicher?", "Löschen wirkt sich direkt auf die Datenbank aus. Möchten Sie fortfahren?", diff --git a/src/gui/sellerdialog.h b/src/gui/sellerdialog.h index dbf6257..d48556b 100644 --- a/src/gui/sellerdialog.h +++ b/src/gui/sellerdialog.h @@ -14,7 +14,6 @@ class SellerDialog : public QDialog public: SellerDialog(QWidget* parent = nullptr, Qt::WindowFlags f = Qt::WindowTitleHint | Qt::WindowSystemMenuHint); - ~SellerDialog(); private slots: void onSellerViewSelectionChanged(const QItemSelection& selected, @@ -26,7 +25,6 @@ class SellerDialog : public QDialog void on_model_duplicateSellerNo(const QString& message); virtual void accept() override; Ui::SellerDialog ui_; - Marketplace* market_; }; #endif \ No newline at end of file