composer/iTop
6
0
Fork 0
mirror of https://github.com/Combodo/iTop.git synced 2026-04-23 02:28:44 +02:00
Code Issues Packages Projects Releases Wiki Activity

N°3513 - Write object creation history within the same DB transaction for better consistency

Browse Source
This commit is contained in:
Eric
2021-06-28 16:59:05 +02:00
parent 0e55a30e5a
commit 0396914068
2 changed files with 26 additions and 26 deletions
Download Patch File Download Diff File Expand all files Collapse all files

5
core/dbobject.class.php
View File

@@ -2810,6 +2810,9 @@ abstract class DBObject implements iDisplay
$this->DBWriteLinks();
$this->WriteExternalAttributes();
// Write object creation history within the transaction
$this->RecordObjCreation();
if ($bIsTransactionEnabled) {
CMDBSource::Query('COMMIT');
}
@@ -2870,8 +2873,6 @@ abstract class DBObject implements iDisplay
}
}
$this->RecordObjCreation();
return $this->m_iKey;
}

47
test/core/CMDBSource/TransactionsTest.php
View File

@@ -123,30 +123,29 @@ class TransactionsTest extends ItopTestCase
"History 13" => ['iFailAt' => 15, 'bIsInDB' => false],
"History 14" => ['iFailAt' => 16, 'bIsInDB' => false],
"History 15" => ['iFailAt' => 17, 'bIsInDB' => false],
// For 3.0 when CRUD sequence is ok
// "History 16" => ['iFailAt' => 18, 'bIsInDB' => false],
// "History 17" => ['iFailAt' => 19, 'bIsInDB' => false],
// "History 18" => ['iFailAt' => 20, 'bIsInDB' => false],
// "History 19" => ['iFailAt' => 21, 'bIsInDB' => false],
// "History 20" => ['iFailAt' => 22, 'bIsInDB' => false],
// "History 21" => ['iFailAt' => 23, 'bIsInDB' => false],
// "History 22" => ['iFailAt' => 24, 'bIsInDB' => false],
// "History 23" => ['iFailAt' => 25, 'bIsInDB' => false],
// "History 24" => ['iFailAt' => 26, 'bIsInDB' => false],
// "History 25" => ['iFailAt' => 27, 'bIsInDB' => false],
// "History 26" => ['iFailAt' => 28, 'bIsInDB' => false],
// "History 27" => ['iFailAt' => 29, 'bIsInDB' => false],
// "History 28" => ['iFailAt' => 30, 'bIsInDB' => false],
// "History 29" => ['iFailAt' => 31, 'bIsInDB' => false],
// "History 30" => ['iFailAt' => 32, 'bIsInDB' => false],
// "History 31" => ['iFailAt' => 33, 'bIsInDB' => false],
// "History 32" => ['iFailAt' => 34, 'bIsInDB' => false],
// "History 33" => ['iFailAt' => 35, 'bIsInDB' => false],
// "History 34" => ['iFailAt' => 36, 'bIsInDB' => false],
// "History 35" => ['iFailAt' => 37, 'bIsInDB' => false],
// "History 36" => ['iFailAt' => 38, 'bIsInDB' => false],
// "History 37" => ['iFailAt' => 39, 'bIsInDB' => false],
// "History 38" => ['iFailAt' => 40, 'bIsInDB' => false],
"History 16" => ['iFailAt' => 18, 'bIsInDB' => false],
"History 17" => ['iFailAt' => 19, 'bIsInDB' => false],
"History 18" => ['iFailAt' => 20, 'bIsInDB' => false],
"History 19" => ['iFailAt' => 21, 'bIsInDB' => false],
"History 20" => ['iFailAt' => 22, 'bIsInDB' => false],
"History 21" => ['iFailAt' => 23, 'bIsInDB' => false],
"History 22" => ['iFailAt' => 24, 'bIsInDB' => false],
"History 23" => ['iFailAt' => 25, 'bIsInDB' => false],
"History 24" => ['iFailAt' => 26, 'bIsInDB' => false],
"History 25" => ['iFailAt' => 27, 'bIsInDB' => false],
"History 26" => ['iFailAt' => 28, 'bIsInDB' => false],
"History 27" => ['iFailAt' => 29, 'bIsInDB' => false],
"History 28" => ['iFailAt' => 30, 'bIsInDB' => false],
"History 29" => ['iFailAt' => 31, 'bIsInDB' => false],
"History 30" => ['iFailAt' => 32, 'bIsInDB' => false],
"History 31" => ['iFailAt' => 33, 'bIsInDB' => false],
"History 32" => ['iFailAt' => 34, 'bIsInDB' => false],
"History 33" => ['iFailAt' => 35, 'bIsInDB' => false],
"History 34" => ['iFailAt' => 36, 'bIsInDB' => false],
"History 35" => ['iFailAt' => 37, 'bIsInDB' => false],
"History 36" => ['iFailAt' => 38, 'bIsInDB' => false],
"History 37" => ['iFailAt' => 39, 'bIsInDB' => false],
"History 38" => ['iFailAt' => 40, 'bIsInDB' => false],
];
}