Improve error message

src/core/database.cpp

@@ -186,6 +186,8 @@ void Database::beginTransaction() { exec("BEGIN TRANSACTION"); }
 
 void Database::endTransaction() { exec("END TRANSACTION"); }
 
+void Database::rollbackTransaction() { exec("ROLLBACK TRANSACTION"); }
+
 unsigned int Database::storeSellers(std::vector<std::unique_ptr<Seller>>& sellers, bool onlyDelete)
 {
     int retCode{};
@@ -221,8 +223,9 @@ unsigned int Database::storeSellers(std::vector<std::unique_ptr<Seller>>& seller
             retCode = sqlite3_step(stmt);
 
             if (retCode != SQLITE_DONE) {
-
+                //rollbackTransaction();
                 std::string errMsg(sqlite3_errmsg(db_));
+                errMsg += "\nSellerNo: " + std::to_string(seller->getSellerNo());
                 sqlite3_finalize(stmt);
                 throw std::runtime_error(errMsg);
             }
@@ -720,4 +723,4 @@ void Database::updateCashPointNo(int oldCashPointNo, int newCashPointNo)
     sqlite3_finalize(stmt);
 
     endTransaction();
-}
+}

src/core/database.h

@@ -31,6 +31,7 @@ class Database
     void init();
     void beginTransaction();
     void endTransaction();
+    void rollbackTransaction();
     void createNew();
     int getVersion();
     unsigned int storeArticles(std::vector<Article*> articles);

src/core/jsonutil.cpp

@@ -31,18 +31,24 @@ void JsonUtil::exportSellers(const fs::path& filePath, Marketplace* market)
     writer->write(root, &file);
 }
 
-void JsonUtil::importSellers(const fs::path& filePath, Marketplace* market)
+void JsonUtil::importSellers(const fs::path& filePath, Marketplace* market, bool cleanup)
 {
-    for (auto& seller : market->getSellers()) {
-        seller->setState(Seller::State::DELETE);
+    if (cleanup) {
+        for (auto& seller : market->getSellers()) {
+            seller->setState(Seller::State::DELETE);
+        }
+        market->storeToDb(true);
     }
-    market->storeToDb(true);
 
     Json::Value jsonValues;
     std::ifstream file(filePath);
     file >> jsonValues;
 
     for (auto val : jsonValues["sellers"]) {
+        auto sellerExist = market->findSellerWithUuid(val["uuid"].asString());
+        if (sellerExist) {
+            continue;
+        }
         auto seller = std::make_unique<Seller>();
         seller->setUuidFromString(val["uuid"].asString());
         seller->setSellerNo(val["seller_no"].asInt());
@@ -80,6 +86,17 @@ void JsonUtil::exportSales(const fs::path& filePath, Marketplace* market, int ca
 
     std::unique_ptr<Json::StreamWriter> writer(builder.newStreamWriter());
 
+    // Do export sellers (maybe there are new sellers created during sale)
+    for (const auto& seller : market->getSellers()) {
+        Json::Value newEntry;
+        newEntry["uuid"] = seller->getUuidAsString();
+        newEntry["seller_no"] = seller->getSellerNo();
+        newEntry["last_name"] = seller->getLastName();
+        newEntry["first_name"] = seller->getFirstName();
+        newEntry["num_offered_articles"] = seller->numArticlesOffered();
+        root["sellers"].append(newEntry);
+    }
+
     root["source_no"] = cashPointNo;
 
     for (const auto& sale : market->getSales()) {
@@ -110,6 +127,9 @@ void JsonUtil::exportSales(const fs::path& filePath, Marketplace* market, int ca
 
 void JsonUtil::importSales(const fs::path& filePath, Marketplace* market, int cashPointNo)
 {
+    // First import newly created sellers during the sale phase
+    importSellers(filePath, market, false);
+
     Json::Value jsonValues;
     std::ifstream file(filePath);
     file >> jsonValues;

src/core/jsonutil.h

@@ -10,7 +10,7 @@ class JsonUtil
 {
   public:
     static void exportSellers(const std::filesystem::path& filePath, Marketplace* market);
-    static void importSellers(const std::filesystem::path& filePath, Marketplace* market);
+    static void importSellers(const std::filesystem::path& filePath, Marketplace* market, bool cleanup = true);
     static void exportSales(const std::filesystem::path& filePath, Marketplace* market, int cashPointNo);
     static void importSales(const std::filesystem::path& filePath, Marketplace* market, int cashPointNo);
 };

src/gui/mainwindow.cpp

@@ -503,6 +503,7 @@ void MainWindow::onImportSalesJsonActionTriggered()
         QMessageBox(QMessageBox::Icon::Warning, "Import nicht möglich", err.what(), QMessageBox::Ok,
                     this)
             .exec();
+        marketplace_->loadFromDb();
     }
     setSaleModel();
 }

src/gui/sellerdialog.cpp

@@ -25,14 +25,14 @@ SellerDialog::SellerDialog(QWidget* parent, Qt::WindowFlags f) : QDialog(parent,
 void SellerDialog::on_newButton_clicked()
 {
     // Don't allow new seller if market has already started
-    if (market_->getSales().size() > 0) {
+    /* 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 hinzufügen.",
                     QMessageBox::StandardButton::Ok, this)
             .exec();
         return;
-    }
+    } */
 
     ui_.tableView->reset();
     ui_.tableView->model()->insertRows(ui_.tableView->model()->rowCount(), 1);