The visibility aspect must be taken into account to prevent (A) delivering hidden pages from cache or (B) enable admPanel preview of hidden pages (showHiddenPages).
Until then, b13/menus is (at least) broken/incompatible with Admin Panel's "showHiddenPages" feature:
If visiting a site with showHiddenPages=1, either
(A) b13/menus may deliver hidden pages without backend login (if b13/menus' data not yet cached) or
(B) "showHiddenPages" isn't respected (if b13/menus' data already cached)
Possible solutions:
own cache entries depending on visibility aspect (may be done in AbstractMenuCompiler::generateCacheIdentifierForMenu)
not caching b13/menus at all if (pseudo-code) TYPO3\CMS\Core\Context\Context::getAspect('visibility')->get('includeHiddenPages') returns true.
bmack
commented
2 years ago
The visibility aspect must be taken into account to prevent (A) delivering hidden pages from cache or (B) enable admPanel preview of hidden pages (
showHiddenPages).Until then, b13/menus is (at least) broken/incompatible with Admin Panel's "showHiddenPages" feature: If visiting a site with showHiddenPages=1, either (A) b13/menus may deliver hidden pages without backend login (if b13/menus' data not yet cached) or (B) "showHiddenPages" isn't respected (if b13/menus' data already cached)
Possible solutions:
AbstractMenuCompiler::generateCacheIdentifierForMenu)TYPO3\CMS\Core\Context\Context::getAspect('visibility')->get('includeHiddenPages')returns true.