composer/iTop
6
0
Fork 0
mirror of https://github.com/Combodo/iTop.git synced 2026-02-13 07:24:13 +01:00
Code Issues Packages Projects Releases Wiki Activity

Merge remote-tracking branch 'origin/support/3.1' into support/3.2

Browse Source
This commit is contained in:
Molkobain
2024-03-22 14:23:00 +01:00
parent 5d8a0f8e9f 90458f6048
commit babdd9f163
3 changed files with 201 additions and 52 deletions
Download Patch File Download Diff File Expand all files Collapse all files

20
core/designdocument.class.inc.php
View File

@@ -343,6 +343,26 @@ class DesignElement extends \DOMElement
return false;
}
/**
* True if the node is contained in a _delta="merge" tree
* @return bool
*/
public function IsInSpecifiedMerge(): bool
{
// Iterate through the parents: reset the flag if any of them has a flag set
for ($oParent = $this; $oParent instanceof MFElement; $oParent = $oParent->parentNode) {
$sDeltaSpec = $oParent->getAttribute('_delta');
if ($sDeltaSpec === 'merge') {
return true;
}
if (in_array($sDeltaSpec, ['define', 'define_if_not_exists', 'force', 'redefine'])) {
return false;
}
}
return false;
}
/**
* Find the child node matching the given node.
* UNSAFE: may return nodes marked as _alteration="removed"

10
setup/modelfactory.class.inc.php
View File

@@ -832,6 +832,7 @@ class ModelFactory
case '':
$bMustExist = ($sDeltaSpec === 'must_exist');
$bIfExists = ($sDeltaSpec === 'if_exists');
$bSpecifiedMerge = $oSourceNode->IsInSpecifiedMerge();
/** @var MFElement $oTargetNode */
$oTargetNode = $oTargetParentNode->_FindChildNode($oSourceNode, $sSearchId);
@@ -847,7 +848,7 @@ class ModelFactory
// Do not continue deeper
$oTargetNode = null;
} else {
if ($sMode === self::LOAD_DELTA_MODE_STRICT && ($sSearchId !== '' || is_null($oSourceNode->firstElementChild))) {
if (!$bSpecifiedMerge && $sMode === self::LOAD_DELTA_MODE_STRICT && ($sSearchId !== '' || is_null($oSourceNode->firstElementChild))) {
$iLine = ModelFactory::GetXMLLineNumber($oSourceNode);
$sItopNodePath = DesignDocument::GetItopNodePath($oSourceNode);
throw new MFException($sItopNodePath.' at line '.$iLine.': could not be found or marked as removed (strict mode)',
@@ -867,12 +868,15 @@ class ModelFactory
// Do not continue deeper everything is already copied
$oTargetNode = null;
} else {
// copy the node with attributes and continue deeper
// copy the node with attributes (except _delta) and continue deeper
$oTargetNode = $oTargetDocument->importNode($oSourceNode, false);
foreach ($oSourceNode->attributes as $oAttributeNode) {
$oTargetNode->setAttribute($oAttributeNode->name, $oAttributeNode->value);
}
if ($sSearchId !== '') {
if ($oTargetNode->hasAttribute('_delta')) {
$oTargetNode->removeAttribute('_delta');
}
if ($sSearchId !== '' || $bSpecifiedMerge) {
// Add the node by default
$oTargetParentNode->AddChildNode($oTargetNode);
} else {

223
tests/php-unit-tests/unitary-tests/setup/ModelFactoryTest.php
View File

@@ -75,10 +75,12 @@ class ModelFactoryTest extends ItopTestCase
// Canonicalize the expected XML (to cope with indentation)
$oExpectedDocument = new DOMDocument();
$oExpectedDocument->preserveWhiteSpace = false;
$oExpectedDocument->loadXML($sXML);
$oExpectedDocument->formatOutput = true;
$oExpectedDocument->loadXML($sXML);
return $oExpectedDocument->C14N(false, true);
$sSavedXML = $oExpectedDocument->SaveXML();
return str_replace(' encoding="UTF-8"', '', $sSavedXML);
}
/**
@@ -665,7 +667,7 @@ class ModelFactoryTest extends ItopTestCase
$oFactory = $this->MakeVanillaModelFactory($sInitialXML);
$oFactoryDocument = $this->GetNonPublicProperty($oFactory, 'oDOMDocument');
$sExpectedXML = null;
if (\utils::StartsWith($sExpectedXMLOrErrorMessage, '<')) {
if (\utils::StartsWith(trim($sExpectedXMLOrErrorMessage), '<')) {
$sExpectedXML = $sExpectedXMLOrErrorMessage;
}
@@ -3026,14 +3028,16 @@ XML
,
],
'Conditionally deleted class' => [
'sInitialXMLInternal' => '<itop_design>
'sInitialXMLInternal' => '
<itop_design>
<classes>
<class id="cmdbAbstractObject"/>
<class id="C_1_1" _alteration="removed"/>
<class id="C_1" _alteration="remove_needed"/>
</classes>
</itop_design>',
'sExpectedXMLDelta' => '<itop_design>
'sExpectedXMLDelta' => '
<itop_design>
<classes>
<class id="C_1_1" _delta="delete"/>
<class id="C_1" _delta="delete_if_exists"/>
@@ -3411,15 +3415,21 @@ XML
$oDocument->loadXML($sDeltaXML);
/* @var MFElement $oDeltaRoot */
$oDeltaRoot = $oDocument->firstChild;
$sExpectedXML = null;
if (\utils::StartsWith(trim($sExpectedXMLInLaxMode), '<')) {
$sExpectedXML = $sExpectedXMLInLaxMode;
}
try {
$oFactory->LoadDelta($oDeltaRoot, $oFactoryDocument, ModelFactory::LOAD_DELTA_MODE_LAX);
$this->AssertEqualModels($sExpectedXMLInLaxMode, $oFactory, 'LoadDelta(lax) did not produce the expected result');
$this->assertNotNull($sExpectedXML, "LoadDelta(lax) should have failed with exception: $sExpectedXMLInLaxMode");
$this->AssertEqualModels($sExpectedXML, $oFactory, 'LoadDelta(lax) did not produce the expected result');
}
catch (ExpectationFailedException $e) {
throw $e;
}
catch (\Exception $e) {
$this->assertNull($sExpectedXMLInLaxMode, 'LoadDelta(lax) should not have failed with exception: '.$e->getMessage());
$this->assertNull($sExpectedXML, 'LoadDelta(lax) should not have failed with exception: '.$e->getMessage());
$this->assertEquals($sExpectedXMLInLaxMode, $e->getMessage());
}
// Load in Strict mode
@@ -3430,118 +3440,233 @@ XML
$oDocument->loadXML($sDeltaXML);
/* @var MFElement $oDeltaRoot */
$oDeltaRoot = $oDocument->firstChild;
$sExpectedXML = null;
if (\utils::StartsWith(trim($sExpectedXMLInStrictMode), '<')) {
$sExpectedXML = $sExpectedXMLInStrictMode;
}
try {
$oFactory->LoadDelta($oDeltaRoot, $oFactoryDocument, ModelFactory::LOAD_DELTA_MODE_STRICT);
$this->AssertEqualModels($sExpectedXMLInStrictMode, $oFactory, 'LoadDelta(strict) did not produce the expected result');
$this->assertNotNull($sExpectedXML, "LoadDelta(lax) should have failed with exception: $sExpectedXMLInStrictMode");
$this->AssertEqualModels($sExpectedXML, $oFactory, 'LoadDelta(strict) did not produce the expected result');
}
catch (ExpectationFailedException $e) {
throw $e;
}
catch (\Exception $e) {
$this->assertNull($sExpectedXMLInStrictMode, 'LoadDelta(strict) should not have failed with exception: '.$e->getMessage());
$this->assertNull($sExpectedXML, 'LoadDelta(strict) should not have failed with exception: '.$e->getMessage());
$this->assertEquals($sExpectedXMLInStrictMode, $e->getMessage());
}
}
public function LoadDeltaModeProvider()
{
return [
'merge delta have different behavior depending on the mode' => [
'default no _delta have different behavior depending on the mode' => [
'sInitialXML' => '
<itop_design>
<nodeA>
</nodeA>
<nodeA/>
</itop_design>',
'sDeltaXML' => '<itop_design>
'sDeltaXML' => '
<itop_design>
<nodeA>
<nodeB id="C_1">
<parent>cmdbAbstractObject</parent>
</nodeB>
</nodeA>
</itop_design>',
'sExpectedXMLInLaxMode' => '<itop_design>
'sExpectedXMLInLaxMode' => '
<itop_design>
<nodeA>
<nodeB id="C_1" _alteration="added">
<parent>cmdbAbstractObject</parent>
</nodeB>
</nodeA>
</itop_design>',
'sExpectedXMLInStrictMode' => null,
'sExpectedXMLInStrictMode' => '/itop_design/nodeA/nodeB[C_1] at line 4: could not be found or marked as removed (strict mode)',
],
'mode specified in delta takes precedence' => [
'sInitialXML' => '
<itop_design>
<nodeA>
</nodeA>
<nodeA/>
</itop_design>',
'sDeltaXML' => '<itop_design load="strict">
'sDeltaXML' => '
<itop_design load="strict">
<nodeA>
<nodeB id="C_1">
<parent>cmdbAbstractObject</parent>
</nodeB>
</nodeA>
</itop_design>',
'sExpectedXMLInLaxMode' => null,
'sExpectedXMLInStrictMode' => null,
'sExpectedXMLInLaxMode' => '/itop_design/nodeA/nodeB[C_1] at line 4: could not be found or marked as removed (strict mode)',
'sExpectedXMLInStrictMode' => '/itop_design/nodeA/nodeB[C_1] at line 4: could not be found or marked as removed (strict mode)',
],
'merge leaf nodes have different behavior depending on the mode' => [
'default no _delta leaf nodes have different behavior depending on the mode' => [
'sInitialXML' => '
<itop_design>
<nodeA>Test</nodeA>
<nodeA>Test</nodeA>
</itop_design>',
'sDeltaXML' => '<itop_design>
'sDeltaXML' => '
<itop_design>
<nodeA>Taste</nodeA>
</itop_design>',
'sExpectedXMLInLaxMode' => '<itop_design>
<nodeA _alteration="replaced">Taste</nodeA>
'sExpectedXMLInLaxMode' => '
<itop_design>
<nodeA _alteration="replaced">Taste</nodeA>
</itop_design>',
'sExpectedXMLInStrictMode' => null,
'sExpectedXMLInStrictMode' => '/itop_design/nodeA at line 3: cannot be modified without _delta flag (strict mode)',
],
'merge existing leaf nodes without text have same behavior' => [
'default no _delta on existing leaf nodes without text have same behavior' => [
'sInitialXML' => '
<itop_design>
<nodeA/>
</itop_design>',
'sDeltaXML' => '<itop_design>
<nodeA/>
</itop_design>',
'sExpectedXMLInLaxMode' => '<itop_design>
<nodeA/>
</itop_design>',
'sExpectedXMLInStrictMode' => '<itop_design>
<nodeA/>
</itop_design>',
],
'merge non-existing leaf nodes without text have different behavior' => [
'sInitialXML' => '
'sDeltaXML' => '
<itop_design>
</itop_design>',
'sDeltaXML' => '<itop_design>
<nodeA/>
</itop_design>',
'sExpectedXMLInLaxMode' => '<itop_design>
<nodeA/>
'sExpectedXMLInLaxMode' => '
<itop_design>
<nodeA/>
</itop_design>',
'sExpectedXMLInStrictMode' => '
<itop_design>
<nodeA/>
</itop_design>',
'sExpectedXMLInStrictMode' => null,
],
'merge non-existing nodes with sub-nodes defined' => [
'default no _delta on non-existing leaf nodes without text have different behavior' => [
'sInitialXML' => '
<itop_design>
</itop_design>',
'sDeltaXML' => '<itop_design>
'sDeltaXML' => '
<itop_design>
<nodeA/>
</itop_design>',
'sExpectedXMLInLaxMode' => '
<itop_design>
<nodeA/>
</itop_design>',
'sExpectedXMLInStrictMode' => '/itop_design/nodeA at line 3: could not be found or marked as removed (strict mode)',
],
'default no _delta on non-existing nodes with sub-nodes defined' => [
'sInitialXML' => '
<itop_design>
</itop_design>',
'sDeltaXML' => '
<itop_design>
<nodeA>
<nodeB _delta="define"/>
</nodeA>
</itop_design>',
'sExpectedXMLInLaxMode' => '<itop_design>
'sExpectedXMLInLaxMode' => '
<itop_design>
<nodeA>
<nodeB _alteration="added"/>
</nodeA>
</itop_design>',
'sExpectedXMLInStrictMode' => '<itop_design>
'sExpectedXMLInStrictMode' => '
<itop_design>
<nodeA>
<nodeB _alteration="added"/>
</nodeA>
</itop_design>',
],
'merge _delta on non-existing node must create node' => [
'sInitialXML' => '
<itop_design>
<nodeA>
<nodeB/>
</nodeA>
</itop_design>',
'sDeltaXML' => '
<itop_design load="strict">
<nodeA>
<nodeB>
<nodeC id="C_1" _delta="merge">
<items>
<item id="I_1" _delta="define">Test</item>
</items>
</nodeC>
</nodeB>
</nodeA>
</itop_design>',
'sExpectedXMLInLaxMode' => '
<itop_design>
<nodeA>
<nodeB>
<nodeC id="C_1" _alteration="added">
<items>
<item id="I_1">Test</item>
</items>
</nodeC>
</nodeB>
</nodeA>
</itop_design>',
'sExpectedXMLInStrictMode' => '
<itop_design>
<nodeA>
<nodeB>
<nodeC id="C_1" _alteration="added">
<items>
<item id="I_1">Test</item>
</items>
</nodeC>
</nodeB>
</nodeA>
</itop_design>',
],
'merge _delta on existing tree must merge...' => [
'sInitialXML' => '
<itop_design>
<nodeA>
<nodeB>
<nodeC id="C_1">
<items/>
</nodeC>
</nodeB>
</nodeA>
</itop_design>',
'sDeltaXML' => '
<itop_design load="strict">
<nodeA>
<nodeB>
<nodeC id="C_1" _delta="merge">
<items>
<item id="I_1" _delta="define">Test</item>
</items>
</nodeC>
</nodeB>
</nodeA>
</itop_design>',
'sExpectedXMLInLaxMode' => '
<itop_design>
<nodeA>
<nodeB>
<nodeC id="C_1">
<items>
<item id="I_1" _alteration="added">Test</item>
</items>
</nodeC>
</nodeB>
</nodeA>
</itop_design>',
'sExpectedXMLInStrictMode' => '
<itop_design>
<nodeA>
<nodeB>
<nodeC id="C_1">
<items>
<item id="I_1" _alteration="added">Test</item>
</items>
</nodeC>
</nodeB>
</nodeA>
</itop_design>',
],
];