Bug #69004: RelationHandler->readList() removes items with value of 0 - TYPO3 Core - TYPO3 Forge

Actions

Copy link

Bug #69004

closed

Epic #68397: Make TYPO3 work with MySQL strict mode

RelationHandler->readList() removes items with value of 0

Added by Morton Jonuschat over 9 years ago. Updated about 7 years ago.

Status:

Closed

Priority:

Should have

Assignee:

Category:

Database API (Doctrine DBAL)

Target version:

7.5

Start date:

2015-08-12

Due date:

% Done:

100%

Estimated time:

TYPO3 Version:

PHP Version:

5.6

Tags:

Complexity:

Is Regression:

Sprint Focus:

Description

When debugging MySQL strict mode problems in regard to handling sys_language_uid (Incorrect integer value: '' for column 'sys_language_uid') I traced the source to the function RelationHandler->readList(). The function is removing items from the list which have a value of 0. The condition is in line 398 of RelationHandler.php:

if (((string)$theID != '' && $theID) && $theTable && isset($this->tableArray[$theTable])) {

Actions

Copy link

Updated by Gerrit Code Review over 9 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 http://review.typo3.org/42550

Actions

Copy link

Updated by Morton Jonuschat over 9 years ago

Parent task set to #68397

Actions

Copy link

Updated by Oliver Hader over 9 years ago

I was able to reproduce this behavior by activating the MySQL strict mode
mysql -e "SET GLOBAL sql_mode='STRICT_ALL_TABLES';"

https://travis-ci.org/ohader/TYPO3.CMS/builds/75433440

Actions

Copy link

Updated by Gerrit Code Review over 9 years ago

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

Actions

Copy link

Updated by Gerrit Code Review over 9 years ago

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

Actions

Copy link

Updated by Gerrit Code Review over 9 years ago

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

Actions

Copy link

Updated by Gerrit Code Review over 9 years ago

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

Actions

Copy link

Updated by Gerrit Code Review over 9 years ago

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

Actions

Copy link

Updated by Gerrit Code Review over 9 years ago

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

Actions

Copy link

#10