Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add TYPO3v10 support #13

Merged
merged 26 commits into from
Sep 17, 2020
Merged

Add TYPO3v10 support #13

merged 26 commits into from
Sep 17, 2020

Conversation

mbrodala
Copy link
Member

No description provided.

@mbrodala mbrodala force-pushed the typo3v10 branch 15 times, most recently from 569f74d to 490ff68 Compare September 17, 2020 07:38
@mbrodala mbrodala marked this pull request as ready for review September 17, 2020 09:50
@mbrodala mbrodala force-pushed the typo3v10 branch 2 times, most recently from a06d274 to 7f38c04 Compare September 17, 2020 10:16
The ObjectManager requires native dependency injection now so make sure
we always use a mock in tests.
This is essential for the checks in
SystemEnvironmentBuilder::initializeEnvironment() where TYPO3_PATH_ROOT
and TYPO3_PATH_APP must be properly related to each other. These are
defined in AbstractTestSystem::setTypo3Context() and involve some
processing.

With this the config/var path is always the same both for the test
itself as well as the website accessed by the temporary webserver.
This adds a handler for "/-/searchable/urls" urls which expects a list
of Typolink configurations and returns resolved links.
Since we are actually building URIs and not links here, there is no
point in passing around an unused title here.
For TYPO3v8 and within an eID search we must make sure to use the
default language if nothing can be determined otherwise.
This controller is explicitly kept around for TYPO3v8 support.
@mbrodala mbrodala merged commit 6af539d into master Sep 17, 2020
@mbrodala mbrodala deleted the typo3v10 branch September 17, 2020 13:21
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.

1 participant