Improve mapper handling of new identifiers of the dabase objects

Author: dimkanovikov

src/corelib/data_layer/mapper/abstract_mapper.cpp

@@ -19,7 +19,6 @@ namespace DataMappingLayer {
 
 void AbstractMapper::clear()
 {
-    m_isLastIdentifierLoaded = false;
     for (auto& [key, value] : m_loadedObjectsMap) {
         delete value;
         value = nullptr;
@@ -55,16 +54,6 @@ QVector<Domain::DomainObject*> AbstractMapper::abstractFind(const QString& _filt
 
 bool AbstractMapper::abstractInsert(DomainObject* _object)
 {
-    //
-    // Установим идентификатор для нового объекта
-    //
-    _object->setId(findNextIdentifier());
-
-    //
-    // Добавим вновь созданный объект в список загруженных объектов
-    //
-    m_loadedObjectsMap.emplace(_object->id(), _object);
-
     //
     // Получим данные для формирования запроса на их добавление
     //
@@ -85,7 +74,15 @@ bool AbstractMapper::abstractInsert(DomainObject* _object)
     //
     const auto isInsertSuccesful = executeSql(insertQuery);
     if (isInsertSuccesful) {
+        //
+        // .. если добавление оказалось успешным, определим идентификатор объекта
+        //
+        _object->setId(Identifier(insertQuery.lastInsertId().toInt()));
         _object->markChangesStored();
+        //
+        // ... и добавим вновь созданный объект в список загруженных объектов
+        //
+        m_loadedObjectsMap.emplace(_object->id(), _object);
     }
     return isInsertSuccesful;
 }
@@ -191,31 +188,6 @@ DomainObject* AbstractMapper::loadObjectFromDatabase(const Identifier& _id)
     return result;
 }
 
-Identifier AbstractMapper::findNextIdentifier()
-{
-    if (!m_isLastIdentifierLoaded) {
-        //
-        // Если нет ещё последнего индекса по таблице, загрузим его
-        //
-        QSqlQuery query = Database::query();
-        query.prepare(findLastOneStatement());
-        query.exec();
-        query.next();
-        const QSqlRecord record = query.record();
-        load(record);
-
-        m_isLastIdentifierLoaded = true;
-    }
-
-    Identifier maxId(0);
-    if (!m_loadedObjectsMap.empty()) {
-        auto iter = m_loadedObjectsMap.cend();
-        --iter;
-        maxId = iter->first;
-    }
-    return maxId.next();
-}
-
 DomainObject* AbstractMapper::load(const QSqlRecord& _record)
 {
     const int idValue = _record.value("id").toInt();

src/corelib/data_layer/mapper/abstract_mapper.h

@@ -71,22 +71,12 @@ class AbstractMapper
      */
     Domain::DomainObject* loadObjectFromDatabase(const Domain::Identifier& _id);
 
-    /**
-     * @brief Получить идентификатор для нового объекта на сохранение в БД
-     */
-    Domain::Identifier findNextIdentifier();
-
     /**
      * @brief Загрузить или обновить объект из записи в БД
      */
     Domain::DomainObject* load(const QSqlRecord& _record);
 
 private:
-    /**
-     * @brief Был ли загружен идентификатор последнего элемента
-     */
-    bool m_isLastIdentifierLoaded = false;
-
     /**
      * @brief Загруженные объекты из базы данных
      */

src/corelib/data_layer/mapper/document_change_mapper.cpp

@@ -186,7 +186,7 @@ QString DocumentChangeMapper::insertStatement(Domain::DomainObject* _object,
 
     const auto documentChangeObject = static_cast<DocumentChangeObject*>(_object);
     _insertValues.clear();
-    _insertValues.append(documentChangeObject->id().value());
+    _insertValues.append(QVariant());
     _insertValues.append(documentChangeObject->documentUuid().toString());
     _insertValues.append(documentChangeObject->uuid().toString());
     _insertValues.append(qCompress(documentChangeObject->undoPatch()));

src/corelib/data_layer/mapper/document_mapper.cpp

@@ -126,7 +126,7 @@ QString DocumentMapper::insertStatement(DomainObject* _object, QVariantList& _in
 
     const auto documentObject = static_cast<DocumentObject*>(_object);
     _insertValues.clear();
-    _insertValues.append(documentObject->id().value());
+    _insertValues.append(QVariant());
     _insertValues.append(documentObject->uuid().toString());
     _insertValues.append(static_cast<int>(documentObject->type()));
     _insertValues.append(documentObject->content());