-
-
Notifications
You must be signed in to change notification settings - Fork 85
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
Fatal Error in combination with EXT:flux #585
Comments
Hi, thanks for the issue reporting. Fixed it like you suggested. Will release a new version soon. |
@rfuehricht Can you verify, that this fixes the issue for you? I'll wait with the release until I have a "go" from you, as I didn't setup the flux extension in my development environment to test it. |
Hi! I can confirm, that the fix solves the issue. Thank you very much. |
@rfuehricht Thank you for verifying! Mask 8.3.3 is released now. |
Thanks for the quick fix @nhovratov ;) |
Hi!
I asked in EXT:flux for help but it seems that this might rather be an issue in EXT:mask (see: FluidTYPO3/flux#2132).
When editing a content element, this error is shown:
Mask introduces two itemsProcFuncs and calls previously set itemsProcFuncs using this code:
GeneralUtility::callUserFunction( $params['config']['m_itemsProcFunc'], $params, $this );
As stated in the issue I created for EXT:flux, the third parameter should not be
$this
, but rather the parent object calling the itemsProcFunc.So, I propose to add a second parameter to the itemsProcFunc an pass it in the
callUserFunction
call:public function itemsProcFunc(&$params, $parentObj): void
...
GeneralUtility::callUserFunction( $params['config']['m_itemsProcFunc'], $params, $parentObject );
The text was updated successfully, but these errors were encountered: