[BUGFIX] provide some expression matcher variables

[achimfritz]

What this pr does

makes (more) TS Condition working when not in FE-Mode

currently: for most conditions the TYPO3.CMS.Core.TypoScript.IncludeTree.Visitor.IncludeTreeConditionMatcherVisitor
logs an Expression could not be parsed. error, e.g.

Expression could not be parsed. - {"expression":"request.getNormalizedParams() && request.getNormalizedParams().getHttpHost() == \"foo.bar.de\""}

Expression could not be parsed. - {"expression":"traverse(page, \"doktype\") == 24"}

Expression could not be parsed. - {"expression":"getTSFE() && page [\"uid\"] == 18"}

Expression could not be parsed. - {"expression":"208 in tree.rootLineIds || 209 in tree.rootLineIds || 217 in tree.rootLineIds"}

we should pass some important variables to the initializeExpressionMatcherWithVariables()

https://github.com/TYPO3/typo3/blob/65aa70e0e8c0bd64566d8726794c71525cbc4152/typo3/sysext/core/Classes/TypoScript/IncludeTree/Visitor/IncludeTreeConditionMatcherVisitor.php#L70

How to test

change Index Configuration

[site("identifier") == "my-site"]
  plugin.tx_solr.index.queue.pages.indexer.frontendDataHelper.host = my-site.ddev.site
[end]

my-site.ddev.site is not used as host

change index Configuration

[request && traverse(request.getQueryParams(), 'search/filter') > 0]
  # so log
[end]

gives an error log

Mon, 17 Feb 2025 08:02:47 +0100 [ERROR] request="4bea2d677e786" component="TYPO3.CMS.Core.TypoScript.IncludeTree.Visitor.IncludeTreeConditionMatcherVisitor": Expression could not be parsed. - {"expression":"request && traverse(request.getQueryParams(), 'search/filter') > 0"}

[dkd-kaehm]

@achimfritz
Thanks for fixing that issue.

Silences this patch all the log entries regarding TypoScript conditions?

PS: Using the conditions on EXT:solr indexing configs is a Pandora's box, avoid using it.

[achimfritz]

Hi @RafaelKa , i do not know if this "all" possible variable the expression matcher can handle. this PR works for our project, where we have about 15 error log entries per parsing TS.

[dkd-kaehm]

@achimfritz
Would you check the tests please. The mocks must be added to failed tests.

[achimfritz]

CI is happy now