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] Have index updated when using frontend editing #648
Conversation
@@ -132,7 +132,8 @@ | |||
'solr::clearSolrConnectionCache', | |||
'ApacheSolrForTypo3\\Solr\\ConnectionManager->updateConnections' | |||
); | |||
|
|||
} | |||
if ((TYPO3_MODE == 'BE') || ((TYPO3_MODE == 'FE') && isset($_POST['TSFE_EDIT']))) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can you make two changes here:
- You don't need the braces around
TYPO3_MODE == 'FE'
- Please use triple equal signs for comparison
===
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Done.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ok, now obviously there's something up with the tests
cHash generation has changed with latest TYPO3 release for new installations and now takes id into calculation. See: https://typo3.org/teams/security/security-bulletins/typo3-core/typo3-core-sa-2016-022/ |
ah right, thanks @stmllr |
I've fixed the broken tests that are related to the cHash change. Tests are running through now and since everything else looks good, i'll merge this PR. |
DataHandler (TceMain) hooks are registered also in frontend mode when the TSFE_EDIT post parameter array is present.
Fixes: #647