Task #81600

Extbase has a lot of leftover unused Exception classes

Added by Benni Mack 4 months ago. Updated 4 months ago.

Status:
Resolved
Priority:
Should have
Assignee:
Category:
Extbase
Target version:
Start date:
2017-06-17
Due date:
% Done:

100%

TYPO3 Version:
9
PHP Version:
Tags:
Complexity:
Sprint Focus:

Associated revisions

Revision be3d473a (diff)
Added by Benni Mack 4 months ago

[TASK] Remove unused Extbase exception classes

There are a lot PHP exception classes which are not in use inside Extbase.

Some come from the fact that Fluid got more mature and
standalone, some from refactorings within ObjectManager,
PropertyManager, RequestHash and Persistence.

One highlight is the InvalidMarkerException which dates back
to the backport of Flow before Fluid was invented and Flow
was using the same marker-based approach as TYPO3 v4 had -
and was never used in TYPO3 v4 at all.

Class aliases are in place for all of them, so no breaking change.

Resolves: #81600
Releases: master
Change-Id: Ic1c3fc309954750f8c2e664ab94fa0e6f9336bea
Reviewed-on: https://review.typo3.org/53224
Tested-by: TYPO3com <>
Reviewed-by: Susanne Moog <>
Tested-by: Susanne Moog <>
Reviewed-by: Georg Ringer <>
Tested-by: Georg Ringer <>

Revision 5513a2b7 (diff)
Added by Benni Mack 4 months ago

[FOLLOWUP][TASK] Remove unused Extbase exception classes

Re-add existing class aliases introduced in TYPO3 v8

Resolves: #81600
Releases: master
Change-Id: I641347c084c1b671bfa97dab9765fc901a3e4031
Reviewed-on: https://review.typo3.org/53381
Reviewed-by: Susanne Moog <>
Tested-by: Susanne Moog <>
Tested-by: TYPO3com <>
Reviewed-by: Benni Mack <>
Tested-by: Benni Mack <>

History

#1 Updated by Gerrit Code Review 4 months ago

  • Status changed from New to Under Review

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/53224

#2 Updated by Gerrit Code Review 4 months 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/53224

#3 Updated by Gerrit Code Review 4 months 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/53224

#4 Updated by Benni Mack 4 months ago

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

#5 Updated by Gerrit Code Review 4 months ago

  • Status changed from Resolved 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/53381

#6 Updated by Benni Mack 4 months ago

  • Status changed from Under Review to Resolved

Also available in: Atom PDF