Project

General

Profile

Actions

Task #88991

closed

Unify handling of errorParams in linkvalidator

Added by Sybille Peters over 4 years ago. Updated about 2 years ago.

Status:
Closed
Priority:
Should have
Category:
Linkvalidator
Target version:
-
Start date:
2019-08-21
Due date:
% Done:

0%

Estimated time:
TYPO3 Version:
10
PHP Version:
Tags:
Complexity:
Sprint Focus:

Description

  • protected parameter errorParams is unnecessarily redefined in ExternalLinktype class (already introduced in parent class AbstractLinkType)
  • initializing of errorParams done differently in each class, can be done in one way in AbstractLinkType
  • usage of errorParams is done differently in various Linktype checker classes
  • the error message is constructed twice, once when it is written into tx_linkvalidator_link.url_reponse and again when it is read from the database for the report
  • only the errorParams['errorMessage'] needs to be stored in the database, not the rest This is actually not a good idea because error message should be localized when report is generated

Related issues 1 (0 open1 closed)

Related to TYPO3 Core - Task #84717: Reconsider serialized array for LinkvalidatorClosedSybille Peters2019-12-13

Actions
Actions #1

Updated by Gerrit Code Review over 4 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/c/Packages/TYPO3.CMS/+/61521

Actions #2

Updated by Gerrit Code Review over 4 years 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/c/Packages/TYPO3.CMS/+/61521

Actions #3

Updated by Sybille Peters over 4 years ago

  • Related to Task #84717: Reconsider serialized array for Linkvalidator added
Actions #4

Updated by Sybille Peters over 4 years ago

  • Description updated (diff)
Actions #5

Updated by Gerrit Code Review over 4 years 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/c/Packages/TYPO3.CMS/+/61521

Actions #6

Updated by Gerrit Code Review over 4 years 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/c/Packages/TYPO3.CMS/+/61521

Actions #7

Updated by Gerrit Code Review over 4 years 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/c/Packages/TYPO3.CMS/+/61521

Actions #8

Updated by Gerrit Code Review over 4 years 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/c/Packages/TYPO3.CMS/+/61521

Actions #9

Updated by Sybille Peters about 4 years ago

  • Status changed from Under Review to New
Actions #10

Updated by Sybille Peters almost 3 years ago

  • Category deleted (Linkvalidator)
Actions #11

Updated by Sybille Peters about 2 years ago

  • Category set to Linkvalidator
  • Status changed from New to Closed
Actions

Also available in: Atom PDF