Task #56797

Cleanup ClassLoader

Added by Markus Klein over 1 year ago. Updated over 1 year ago.

Status:Resolved Start date:2014-03-11
Priority:Must have Due date:
Assigned To:Markus Klein % Done:

100%

Category:Default Spent time: -
Target version:next-patchlevel
TYPO3 Version:6.2 Complexity:easy
PHP Version: Sprint Focus:

Related issues

related to Core - Bug #56993: Cached classloader fails with fluid and is_callable() Resolved 2014-03-17
related to Core - Bug #57094: TCA eval 'int+' fails with a fatal error Rejected 2014-03-20
related to Core - Bug #56822: InvalidArgumentException in ActionControllerTest Resolved 2014-03-12
precedes Core - Bug #54906: Fatal error: Interface 'Psr\Log\LoggerInterface' not found Resolved 2014-03-12 2014-03-12

Associated revisions

Revision 2eaa891a
Added by Markus Klein over 1 year ago

[CLEANUP] ClassLoader

Add methods to better structure class information loading/creation.
This is necessary as the follow up patches will need this code
more often.

Resolves: #56797
Releases: 6.2
Change-Id: I1f15fbd262036b6b2d0159cd0e9613ad7135a296
Reviewed-on: https://review.typo3.org/28279
Reviewed-by: Steffen Ritter
Tested-by: Steffen Ritter
Reviewed-by: Sebastian Fischer
Reviewed-by: Patrick Broens
Tested-by: Patrick Broens

Revision 855ca509
Added by Steffen Ritter over 1 year ago

[BUGFIX] FollowUp to #56797 in I1f15fbd262036b6b2d0159cd0e9613ad7135a296

Somehow a double $this->$this slipped into the cleanup on line 390
which now fatals. This needs to be fixed.

Resolves: #56797
Releases: 6.2
Change-Id: Ibb23b5f56c95106c4fea6e9a3ba09fbc9f9905dd
Reviewed-on: https://review.typo3.org/28296
Reviewed-by: Ernesto Baschny
Tested-by: Ernesto Baschny

History

#1 Updated by Gerrit Code Review over 1 year ago

  • Status changed from Accepted 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 https://review.typo3.org/28279

#2 Updated by Gerrit Code Review over 1 year ago

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

#3 Updated by Gerrit Code Review over 1 year ago

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

#4 Updated by Gerrit Code Review over 1 year ago

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

#5 Updated by Gerrit Code Review over 1 year ago

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

#6 Updated by Gerrit Code Review over 1 year ago

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

#7 Updated by Gerrit Code Review over 1 year ago

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

#8 Updated by Gerrit Code Review over 1 year ago

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

#9 Updated by Steffen Ritter over 1 year ago

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

Also available in: Atom PDF