Project

General

Profile

Actions

Bug #90760

closed

Link validator throws exception with Page TSConfig configuration on sys_file_reference

Added by Riccardo De Contardi over 4 years ago. Updated over 4 years ago.

Status:
Closed
Priority:
Should have
Assignee:
-
Category:
Database API (Doctrine DBAL)
Target version:
-
Start date:
2020-03-14
Due date:
% Done:

100%

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

Description

I performed the following operation with the latest version TYPO3 10.4.0-dev (latest master)

Info > link validator > check links

It threw an exception:

(1/2) Doctrine\DBAL\Exception\InvalidFieldNameException
An exception occurred while executing 'SELECT `uid`, `pid`, `uid_local`, `link`, `sys_language_uid`, `uid_local:type` FROM `sys_file_reference` WHERE (`pid` IN (?)) AND ((`sys_file_reference`.`deleted` = 0) AND (`sys_file_reference`.`hidden` = 0))' with params [307]: Unknown column 'uid_local:type' in 'field list'

I tried to clear all caches from install tool, updated db schema and dump php autoload without success

[UPDATE]: I've found that params [307]: refers to the page I am when I start the link validator

[UPDATE 2]

the cause of the error was a page TSConfig configuration:

mod.linkvalidator.searchFields.sys_file_reference = link

Environment: MAMP 5.5 on MAC

PHP version: 7.2.21
MySQL version: 5.7.26

Let me know if you need further information or if you think that this issue should be closed


Related issues 1 (0 open1 closed)

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

Actions
Actions #1

Updated by Jonas Eberle over 4 years ago

I can't reproduce in master (from git, not composer)
Installed the introduction package, checked links, entered wrong links, checked again. For me it works as expected.

PHP 7.3.15
MariaDB 10.2.30

also tried:
PHP 7.2.28
MySQL 5.7.28

Actions #2

Updated by Jonas Eberle over 4 years ago

The `uid_local:type` comes from here ./typo3/sysext/core/Configuration/TCA/sys_file_reference.php line 16
It is set as [ctrl][type] of sys_file_reference.

IMHO it is not supposed to be part of a query like that.

Actions #3

Updated by Riccardo De Contardi over 4 years ago

  • File cattura3.png added
  • Description updated (diff)
Actions #4

Updated by Riccardo De Contardi over 4 years ago

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

Updated by Riccardo De Contardi over 4 years ago

  • Subject changed from Link validator throws exception to Link validator throws exception with Page TSConfig configuration on sys_file_reference
  • Description updated (diff)

[UPDATE]

I found the cause of the error; maybe I was testing #83835

Change title and description accordingly

Actions #6

Updated by Riccardo De Contardi over 4 years ago

[Update 3]

the "Check this level" dropdown problem is not related, therefore I remove it from here and open a new issue.
Actions #7

Updated by Riccardo De Contardi over 4 years ago

  • Description updated (diff)
Actions #8

Updated by Riccardo De Contardi over 4 years ago

  • File deleted (cattura3.png)
Actions #9

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/+/63727

Actions #10

Updated by Georg Ringer over 4 years ago

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

Updated by Benni Mack over 4 years ago

  • Status changed from Resolved to Closed
Actions

Also available in: Atom PDF