Task #85034

Log message when typolink fails

Added by Mathias Brodala over 1 year ago. Updated 11 months ago.

Status:
Closed
Priority:
Should have
Category:
-
Target version:
Start date:
2018-05-17
Due date:
% Done:

100%

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

Description

ContentObjectRenderer::typoLink() catches the UnableToLinkException from link builders and only returns the link text. Additionally the exception details should be logged to simplify debugging.

Associated revisions

Revision f41eb9ce (diff)
Added by Mathias Brodala about 1 year ago

[TASK] Log message when typolink fails

We log this as debug to not fill system logs by default.

Resolves: #85034
Releases: master, 8.7
Change-Id: Ica686e42fa7a2ef09f2f5c4bf9b91781f5c14dc9
Reviewed-on: https://review.typo3.org/56995
Tested-by: TYPO3com <>
Reviewed-by: Susanne Moog <>
Tested-by: Susanne Moog <>
Reviewed-by: Frank Naegler <>
Tested-by: Frank Naegler <>

Revision 5bfc01ec (diff)
Added by Mathias Brodala about 1 year ago

[TASK] Log message when typolink fails

We log this as debug to not fill system logs by default.

Resolves: #85034
Releases: master, 8.7
Change-Id: Ica686e42fa7a2ef09f2f5c4bf9b91781f5c14dc9
Reviewed-on: https://review.typo3.org/57006
Tested-by: TYPO3com <>
Reviewed-by: Mathias Brodala <>
Tested-by: Mathias Brodala <>

History

#1 Updated by Gerrit Code Review over 1 year 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/56995

#2 Updated by Gerrit Code Review about 1 year 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/57006

#3 Updated by Gerrit Code Review about 1 year ago

Patch set 2 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/57006

#4 Updated by Mathias Brodala about 1 year ago

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

#5 Updated by Benni Mack 11 months ago

  • Status changed from Resolved to Closed

Also available in: Atom PDF