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

[BUGFIX] Correct Typehint #26

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

christophlehmann
Copy link

@christophlehmann christophlehmann commented Nov 9, 2020

The change prevents the Exception

Return value of FoT3\Jumpurl\JumpUrlProcessor::getTypoScriptFrontendController() must be an instance of TYPO3\CMS\Frontend\Controller\TypoScriptFrontendController, null returned

The change relates to: #23 and makes it possible to use the redirects module. The change does not add support for the url processor in the redirects module

christophlehmann and others added 2 commits November 9, 2020 13:58
The return value is null at this point when $GLOBALS['TSFE'] is not initialized.

The change relates to: FriendsOfTYPO3#23 and makes it possible to use the redirects module. The change may not resolve FriendsOfTYPO3#23 fully.
@bmack
Copy link
Member

bmack commented Jan 27, 2021

Hey,

can you check #27 ?

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.

2 participants