Project

General

Profile

Actions

Bug #68602

closed

Epic #68397: Make TYPO3 work with MySQL strict mode

NULL values in records get transformed to FALSE on T3D import

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

Status:
Closed
Priority:
Should have
Category:
Import/Export (T3D)
Target version:
Start date:
2015-07-29
Due date:
% Done:

100%

Estimated time:
TYPO3 Version:
7
PHP Version:
5.6
Tags:
Complexity:
Is Regression:
No
Sprint Focus:

Description

Given a database record with a NULL value in a column the export file generated will have a XML node like <field index="link" type="NULL"></field> - properly representing the NULL value.
On Import the value gets transformed into a boolean somewhere during the import.


Related issues 3 (0 open3 closed)

Related to TYPO3 Core - Bug #68562: Bool values need to be cast to integer for MySQL strict modeClosedMorton Jonuschat2015-07-27

Actions
Related to TYPO3 Core - Task #55082: Impexp: for non active controls NULL has to be imported in xml modeClosedMarc Bastian Heinrichs2014-01-16

Actions
Related to TYPO3 Core - Bug #68810: Remove NULL on sys_file_reference link in impexp functional test dataClosed2015-08-06

Actions
Actions #1

Updated by Marc Bastian Heinrichs over 9 years ago

Wondering about this error. With #55082 this was normally fixed. And if you look at the assertion importPagesAndRelatedTtContentWithDifferentImageToExistingData.csv you see that link is asserted NULL.

Actions #2

Updated by Morton Jonuschat over 9 years ago

If I understand this correctly then the test should fail even when not running in MySQL strict mode. The link field in sys_file_reference is defined as "NOT NULL" with a default of '' (empty string). So the conversion of FALSE to '' makes this test pass?

Actions #3

Updated by Morton Jonuschat over 9 years ago

  • Status changed from New to In Progress
Actions #4

Updated by Gerrit Code Review over 9 years ago

  • Status changed from In Progress 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/42260

Actions #5

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

Actions #6

Updated by Morton Jonuschat over 9 years ago

  • Parent task set to #68397
Actions #7

Updated by Morton Jonuschat over 9 years ago

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

Updated by Marc Bastian Heinrichs over 9 years ago

Jep, sure, you are right. Pushed a fix for #68810 to adapt the impexp test data.

Actions #9

Updated by Anja Leichsenring almost 9 years ago

  • Sprint Focus set to On Location Sprint
Actions #10

Updated by Mathias Schreiber over 8 years ago

  • Sprint Focus deleted (On Location Sprint)
Actions #11

Updated by Riccardo De Contardi about 7 years ago

  • Status changed from Resolved to Closed
Actions

Also available in: Atom PDF