Project

General

Profile

Actions
Copy link

Bug #103494

open

Linkvalidator uses tstamp field directly without checking TCA in recheckLinks

Added by Sybille Peters 8 months ago. Updated about 1 month ago.

Status:
Under Review
Priority:
Must have
Assignee:
Sybille Peters
Category:
Linkvalidator
Target version:
-
Start date:
2024-03-27
Due date:
% Done:

0%

Estimated time:
TYPO3 Version:
13
PHP Version:
Tags:
Complexity:
Is Regression:
Sprint Focus:

Description

This should be merged before https://review.typo3.org/c/Packages/TYPO3.CMS/+/83612

TCA should be used to determine which field is relevant for tstamp (and if there is such a field) before using it for a DB query

$GLOBALS['TCA'][$table]['ctrl']['tstamp']

Reproduce

  1. Change configuration to mod.linkvalidator.searchFields.sys_redirect.target
  2. check links (with a broken redirect target)
  3. in the list of broken links, click pencil to edit redirect target field
  4. change value, save and close edit field

Now, exception is thrown.

Related issues 1 (0 open — 1 closed)

Related to TYPO3 Core - Bug #83835: Linkvalidator should check more fields by default, such as sys_file_reference.linkClosed2018-02-09

Actions
Actions
Copy link
 #1

Updated by Gerrit Code Review 8 months ago

  • Status changed from New to Under Review

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

Actions
Copy link
 #2

Updated by Sybille Peters 8 months ago

  • Description updated (diff)
  • Status changed from Under Review to New
Actions
Copy link
 #3

Updated by Sybille Peters 8 months ago

  • Related to Bug #83835: Linkvalidator should check more fields by default, such as sys_file_reference.link added
Actions
Copy link
 #4

Updated by Gerrit Code Review 8 months ago

  • Status changed from New to Under Review

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

Actions
Copy link
 #5

Updated by Gerrit Code Review 7 months ago

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

Actions
Copy link
 #6

Updated by Gerrit Code Review 7 months ago

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

Actions
Copy link
 #7

Updated by Sybille Peters 7 months ago

  • Description updated (diff)
Actions
Copy link
 #8

Updated by Gerrit Code Review 7 months ago

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

Actions
Copy link
 #9

Updated by Gerrit Code Review 7 months ago

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

Actions
Copy link
 #10

Updated by Gerrit Code Review 7 months ago

Patch set 7 for branch main of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/c/Packages/TYPO3.CMS/+/83613

Actions
Copy link
 #11

Updated by Riccardo De Contardi 6 months ago

  • Category set to Linkvalidator
Actions
Copy link
 #12

Updated by Gerrit Code Review about 1 month ago

Patch set 8 for branch main of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/c/Packages/TYPO3.CMS/+/83613

Actions
Copy link

Also available in: Atom PDF