Task #81205

Improve error on link handler without class

Added by Mathias Brodala over 2 years ago. Updated about 1 year ago.

Status:
Closed
Priority:
Should have
Category:
Link Handling, Site Handling & Routing
Target version:
Start date:
2017-05-12
Due date:
% Done:

100%

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

Description

If a linkhandler is missing its handler in TCEMAIN.linkHandler, a generic exception is thrown by GeneralUtility::makeInstance which does not provide enough context to understand the error. This should be caught before trying to instantiate the missing handler class to simplify debugging.

Associated revisions

Revision 7f2c9573 (diff)
Added by Mathias Brodala over 2 years ago

[TASK] Throw explicit exception on missing link handler class

This adds the link handler identifier to the error message which
makes it easier to fix the missing class reference.

Resolves: #81205
Releases: master, 8.7, 7.6
Change-Id: I0ce610f7ec167704d5cbc5de20d43639c18e3fc1
Reviewed-on: https://review.typo3.org/52787
Tested-by: TYPO3com <>
Reviewed-by: Elmar Hinz <>
Reviewed-by: Henrik Elsner <>
Tested-by: Henrik Elsner <>
Reviewed-by: Wouter Wolters <>
Reviewed-by: Christian Kuhn <>
Tested-by: Christian Kuhn <>

Revision 17776e48 (diff)
Added by Mathias Brodala over 2 years ago

[TASK] Throw explicit exception on missing link handler class

This adds the link handler identifier to the error message which
makes it easier to fix the missing class reference.

Resolves: #81205
Releases: master, 8.7, 7.6
Change-Id: I0ce610f7ec167704d5cbc5de20d43639c18e3fc1
Reviewed-on: https://review.typo3.org/52792
Tested-by: TYPO3com <>
Reviewed-by: Christian Kuhn <>
Tested-by: Christian Kuhn <>

Revision 96a266ad (diff)
Added by Mathias Brodala over 2 years ago

[TASK] Throw explicit exception on missing link handler class

This adds the link handler identifier to the error message which
makes it easier to fix the missing class reference.

Resolves: #81205
Releases: master, 8.7, 7.6
Change-Id: I0ce610f7ec167704d5cbc5de20d43639c18e3fc1
Reviewed-on: https://review.typo3.org/52793
Tested-by: TYPO3com <>
Reviewed-by: Christian Kuhn <>
Tested-by: Christian Kuhn <>

History

#1 Updated by Gerrit Code Review over 2 years 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 https://review.typo3.org/52787

#2 Updated by Gerrit Code Review over 2 years ago

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

#3 Updated by Gerrit Code Review over 2 years ago

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

#4 Updated by Mathias Brodala over 2 years ago

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

#5 Updated by Benni Mack about 1 year ago

  • Status changed from Resolved to Closed

Also available in: Atom PDF