Minor refactoring

Alkarex · Alkarex · commit 3f3e5d4bed80 · 2023-08-21T13:28:18.000+02:00
diff --git a/src/XlsxFastEditor.php b/src/XlsxFastEditor.php
@@ -604,6 +604,8 @@ private function makeNewSharedString(string $value): int
 
 		$uniqueCount = $dom->getElementsByTagName('si')->length;
 		$dom->firstElementChild->setAttribute('uniqueCount', (string)$uniqueCount);
+
+		$this->touchPath(self::SHARED_STRINGS_PATH);
 		return $uniqueCount - 1;	// Base 0
 	}
 
@@ -625,7 +627,6 @@ public function writeString(int $sheetNumber, string $cellName, string $value):
 		$sharedStringId = self::makeNewSharedString($value);
 		$v->nodeValue = (string)$sharedStringId;
 		$this->touchWorksheet($sheetNumber);
-		$this->touchPath(self::SHARED_STRINGS_PATH);
 	}
 
 	/**

Original file line number	Diff line number	Diff line change
`@@ -604,6 +604,8 @@ private function makeNewSharedString(string $value): int`
`604`	`604`
`605`	`605`	`$uniqueCount = $dom->getElementsByTagName('si')->length;`
`606`	`606`	`$dom->firstElementChild->setAttribute('uniqueCount', (string)$uniqueCount);`
	`607`	`+`
	`608`	`+ $this->touchPath(self::SHARED_STRINGS_PATH);`
`607`	`609`	`return $uniqueCount - 1; // Base 0`
`608`	`610`	`}`
`609`	`611`
`@@ -625,7 +627,6 @@ public function writeString(int $sheetNumber, string $cellName, string $value):`
`625`	`627`	`$sharedStringId = self::makeNewSharedString($value);`
`626`	`628`	`$v->nodeValue = (string)$sharedStringId;`
`627`	`629`	`$this->touchWorksheet($sheetNumber);`
`628`		`- $this->touchPath(self::SHARED_STRINGS_PATH);`
`629`	`630`	`}`
`630`	`631`
`631`	`632`	`/**`