Skip to content

Merge from original repository - #2

Open
Kleisli wants to merge 1356 commits into
monosize:localizationfrom
FluidTYPO3:development
Open

Kleisli wants to merge 1356 commits into
monosize:localizationfrom
FluidTYPO3:development

Conversation

@Kleisli

@Kleisli Kleisli commented Oct 18, 2017

Copy link
Copy Markdown

Hi @monosize

Is it possible to merge the current development branch from FluidTYPO3 into your fork? I would like to test it in a current Typo3v8.7 instance.

Selects backend-non-preview as context when no request is found,
which happens in CLI context.

Close: #2172
To my knowledge, this feature has never been used except in
my own proof-of-concept environments. Given that it would need
a rather excessive workaround to function on v13 it's probably
best to simply remove the feature.
Rather than allowing TYPO3 to fail with a hard exception, we allow
Flux to operate in a limited-support mode where no TypoScript is
available. Although this isn't ideal, it's preferable to both alternatives:

1. Let TYPO3 throw an exception, making uncached operation completely
    impossible.
2. Recreating the 8-class-dependency, 260-lines-of-code, non-API
    TypoScript reading from PrepareTypoScriptFrontendRendering,
    which is both final and readonly and completely impossible to extend.

"Fun" fact: TYPOv13 no longer has any short-API way of reading TypoScript
unless it is put into ServerRequest->attributes['frontend.typoscript'] -
and this simply does not happen in uncached (INT) contexts. Great.

There's no service, no utility, no API. The only way is to manually fetch
and parse every TS template from the root line and use the significantly
complex AST-based parsing mechanism.
…#2223)

Adds a fallback handling which delivers template path setup
based on Flux and provider extensions' paths when TypoScript
can not be resolved. If TypoScript can be resolved, the path
setup is read from plugin.tx_yourext (where "yourext" is your
provider extension key) as usual.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

8 participants