Task #35854

Streamline disposal of Signal Slot

Added by Oliver Hader over 7 years ago. Updated almost 2 years ago.

Status:
Closed
Priority:
Should have
Assignee:
-
Category:
Code Cleanup
Target version:
Start date:
2012-04-11
Due date:
% Done:

100%

TYPO3 Version:
4.7
PHP Version:
5.3
Tags:
Complexity:
Sprint Focus:

Description

The concept of Signal Slot has been introduced into Extbase of TYPO3 4.6 and then got copied to TYPO3 4.7 as well - basically to support that concept for the File Abstraction Layer as well. However, now this observer pattern is duplicated in the Core.

The solution:
  • remove SignalSlot from t3lib in TYPO3 4.7 and 6.0
  • adopt usage of SignalSlot in the Core (basically Workspaces in 4.7)
  • make Extbase a required extension in TYPO3 6.0 and modify FAL to use the Extbase Signal-Slot

Related issues

Related to TYPO3 Core - Task #35859: Streamline disposal of Signal Slot Closed 2012-04-11
Related to TYPO3 Core - Bug #35899: Signal-Slot Disposal fails in File Abstraction Layer Closed 2012-04-12

Associated revisions

Revision 9b1bdfc6 (diff)
Added by Oliver Hader over 7 years ago

[TASK] Streamline disposal of Signal Slot

The concept of Signal Slot has been introduced into Extbase of
TYPO3 4.6 and then got copied to TYPO3 4.7 as well - basically
to support that concept for the File Abstraction Layer as well.
However, now this observer pattern is duplicated in the Core,
thus is removed in favour of the available Extbase behaviour.

Change-Id: I406a2381771e91c119240347d0d0444844a2fc0a
Releases: 6.0, 4.7
Resolves: #35854
Reviewed-on: http://review.typo3.org/10252
Reviewed-by: Oliver Hader
Tested-by: Oliver Hader

Revision 936a8916 (diff)
Added by Oliver Hader over 7 years ago

[TASK] Streamline disposal of Signal Slot

The concept of Signal Slot has been introduced into Extbase of
TYPO3 4.6 and then got copied to TYPO3 4.7 as well - basically
to support that concept for the File Abstraction Layer as well.
However, now this observer pattern is duplicated in the Core,
thus is removed in favour of the available Extbase behaviour.

Change-Id: I1b66cab4dbb5ee18a9e72df346a4b64711dd268a
Releases: 6.0, 4.7
Resolves: #35854
Reviewed-on: http://review.typo3.org/10247
Reviewed-by: Oliver Hader
Tested-by: Oliver Hader

Revision cd2a2f7e (diff)
Added by Oliver Hader over 7 years ago

[TASK] Streamline disposal of Signal Slot

Make use of Extbase Signal-Slot dispatcher instead of using the clone in t3lib.

Change-Id: I2e4fa772eec60791fcd947b24f72d1bbf8579b48
Releases: 6.0, 4.7
Resolves: #35854
Related: #35854
Reviewed-on: http://review.typo3.org/10245
Reviewed-by: Oliver Hader
Tested-by: Oliver Hader

Revision 076bc126 (diff)
Added by Oliver Hader over 7 years ago

[TASK] Streamline disposal of Signal Slot

Make use of Extbase Signal-Slot dispatcher instead of using the clone in t3lib.

Change-Id: I9405ccdaab7ced005957e0e95cce52cc965f92ba
Releases: 6.0, 4.7
Resolves: #35854
Related: #35854
Reviewed-on: http://review.typo3.org/10253
Reviewed-by: Oliver Hader
Tested-by: Oliver Hader

History

#1 Updated by Gerrit Code Review over 7 years ago

  • Status changed from New to Under Review

Patch set 1 for branch workspaces_4-7 has been pushed to the review server.
It is available at http://review.typo3.org/10245

#2 Updated by Gerrit Code Review over 7 years ago

Patch set 1 for branch TYPO3_4-7 has been pushed to the review server.
It is available at http://review.typo3.org/10247

#3 Updated by Gerrit Code Review over 7 years ago

Patch set 2 for branch TYPO3_4-7 has been pushed to the review server.
It is available at http://review.typo3.org/10247

#4 Updated by Gerrit Code Review over 7 years ago

Patch set 1 for branch master has been pushed to the review server.
It is available at http://review.typo3.org/10252

#5 Updated by Gerrit Code Review over 7 years ago

Patch set 1 for branch master has been pushed to the review server.
It is available at http://review.typo3.org/10253

#6 Updated by Gerrit Code Review over 7 years ago

Patch set 1 for branch master has been pushed to the review server.
It is available at http://review.typo3.org/10305

#7 Updated by Oliver Hader over 7 years ago

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

#8 Updated by Riccardo De Contardi almost 2 years ago

  • Status changed from Resolved to Closed

Also available in: Atom PDF