Merge pull request #97 from b13/bugfix/issue-96

bmack · web-flow · commit 4049414be919 · 2023-06-14T08:59:48.000+02:00
[BUGFIX] short cache identifier by md5
diff --git a/Classes/Compiler/AbstractMenuCompiler.php b/Classes/Compiler/AbstractMenuCompiler.php
@@ -66,7 +66,7 @@ protected function generateCacheIdentifierForMenu(string $prefix, array $configu
         $visibilityAspect = $this->context->getAspect('visibility');
         $visibility = $visibilityAspect->includeHiddenPages() ? '-with-hidden' : '';
         $root = $this->getCurrentSite()->getRootPageId();
-        $identifier = $prefix . '-root-' . $root . '-language-' . $language . '-groups-' . implode('_', $groupIds) . '-' . $visibility . '-' . substr(md5(json_encode($configuration)), 0, 10);
+        $identifier = $prefix . '-root-' . $root . '-language-' . $language . '-groups-' . md5(implode('_', $groupIds)) . '-' . $visibility . '-' . substr(md5(json_encode($configuration)), 0, 10);
         return $identifier;
     }
 

Original file line number	Diff line number	Diff line change
`@@ -66,7 +66,7 @@ protected function generateCacheIdentifierForMenu(string $prefix, array $configu`
`66`	`66`	`$visibilityAspect = $this->context->getAspect('visibility');`
`67`	`67`	`$visibility = $visibilityAspect->includeHiddenPages() ? '-with-hidden' : '';`
`68`	`68`	`$root = $this->getCurrentSite()->getRootPageId();`
`69`		`- $identifier = $prefix . '-root-' . $root . '-language-' . $language . '-groups-' . implode('_', $groupIds) . '-' . $visibility . '-' . substr(md5(json_encode($configuration)), 0, 10);`
	`69`	`+ $identifier = $prefix . '-root-' . $root . '-language-' . $language . '-groups-' . md5(implode('_', $groupIds)) . '-' . $visibility . '-' . substr(md5(json_encode($configuration)), 0, 10);`
`70`	`70`	`return $identifier;`
`71`	`71`	`}`
`72`	`72`