Bug #79992

Error in DB Check -> Database Relations

Added by Nicole Cordes over 3 years ago. Updated over 2 years ago.

Status:
Closed
Priority:
Should have
Assignee:
Category:
-
Target version:
Start date:
2017-02-24
Due date:
% Done:

100%

TYPO3 Version:
8
PHP Version:
Tags:
Complexity:
Is Regression:
No
Sprint Focus:
Remote Sprint

Description

Given a table with a field name that might be reflected as a keyword (like group), the DB Check > Database Relations tool throws an exception.

Uncaught TYPO3 Exception
Call to a member function getType() on null

Error thrown in file
D:\Version Control\Git\TYPO3\TYPO3.CMS\master\typo3\sysext\core\Classes\Integrity\DatabaseIntegrityCheck.php in line 442.

Related issues

Related to TYPO3 Core - Bug #86167: Having a field in TCA but not in database makes the DatabaseIntegrityCheck crash Closed 2018-09-06

Associated revisions

Revision 32cb6480 (diff)
Added by Thomas Hohn over 3 years ago

[BUGFIX] DB Check causes PHP warnings and errors

In DatabaseIntegrityCheck the method selectNonEmptyRecordsWithFkeys
also needs to take the BLOB into account or else it will fail with
a PHP warning. Furthermore the returned tableColumns list may contain
quoted fields names if the field name can be interpreted as a keyword
for the used connection platform. This patch ensures the correct
column information is received before further processing is started.

Additionally the "Record Statics" module is fixed by checking for an
empty TCA table field which invokes an exception.

Resolves: #79347
Resolves: #79991
Resolves: #79992
Releases: master
Change-Id: Ie455abd2da90d7dabc773ec345b5c8a87b0b5836
Reviewed-on: https://review.typo3.org/51813
Reviewed-by: Christer V <>
Tested-by: Christer V <>
Tested-by: TYPO3com <>
Reviewed-by: Claus Due <>
Reviewed-by: Nicole Cordes <>
Tested-by: Nicole Cordes <>
Reviewed-by: Markus Klein <>
Reviewed-by: Anders Kostending <>
Reviewed-by: Christian Kuhn <>
Tested-by: Christian Kuhn <>

History

#1 Updated by Gerrit Code Review over 3 years ago

  • Status changed from New to Under Review

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/51813

#2 Updated by Gerrit Code Review over 3 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/51813

#3 Updated by Gerrit Code Review over 3 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/51813

#4 Updated by Gerrit Code Review over 3 years ago

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

#5 Updated by Gerrit Code Review over 3 years ago

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

#6 Updated by Thomas Hohn over 3 years ago

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

#7 Updated by Riccardo De Contardi over 2 years ago

  • Status changed from Resolved to Closed

#8 Updated by Christian Kuhn over 1 year ago

  • Related to Bug #86167: Having a field in TCA but not in database makes the DatabaseIntegrityCheck crash added

Also available in: Atom PDF