Bug #65473

Remove closure in signalSlotDispatcher

Added by Nicole Cordes 5 months ago. Updated 5 months ago.

Status:Resolved Start date:2015-03-03
Priority:Must have Due date:
Assigned To:Nicole Cordes % Done:

100%

Category:- Spent time: -
Target version:7.2 (Frontend)
TYPO3 Version:6.2 Is Regression:No
PHP Version: Sprint Focus:
Complexity:

Description

For non-composer installation an anonymous function is assigned to the signalSlotDispatcher. This breaks serialization of different objects (e.g FAL references)


Related issues

related to Core - Bug #65535: Exception: Serialization of 'Closure' is not allowed Under Review 2015-03-05

Associated revisions

Revision 9687a429
Added by Nicole Cordes 5 months ago

[BUGFIX] Remove Closure in signalSlotDispatcher

As the classAliasMap is a singleton we don't need to initialize it on
every dispatch, but we can call the method directly on the instance.
This prevents a closure for signalSlotDispatcher which otherwise throws
an exception on serialization.

Releases: master
Resolves: #65473
Change-Id: I1280a4a054f3b63b4b5e71262d4d054b511f001b
Reviewed-on: http://review.typo3.org/37485
Reviewed-by: Helmut Hummel <>
Tested-by: Helmut Hummel <>
Reviewed-by: Markus Klein <>
Reviewed-by: Christian Kuhn <>
Tested-by: Christian Kuhn <>

History

#1 Updated by Gerrit Code Review 5 months ago

  • Status changed from New to Under Review

Patch set 1 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/37485

#2 Updated by Gerrit Code Review 5 months ago

Patch set 2 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/37485

#3 Updated by Gerrit Code Review 5 months ago

Patch set 3 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/37485

#4 Updated by Gerrit Code Review 5 months ago

Patch set 4 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/37485

#5 Updated by Gerrit Code Review 5 months ago

Patch set 5 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/37485

#6 Updated by Nicole Cordes 5 months ago

  • Status changed from Under Review to Resolved
  • % Done changed from 0 to 100

Also available in: Atom PDF