Project

General

Profile

Actions

Bug #66430

closed

Epic #68397: Make TYPO3 work with MySQL strict mode

Avoid MySQL Warning in File Meta Index process

Added by Tim Lochmüller about 9 years ago. Updated over 5 years ago.

Status:
Closed
Priority:
Should have
Assignee:
-
Category:
File Abstraction Layer (FAL)
Target version:
-
Start date:
2015-04-15
Due date:
% Done:

100%

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

Description

Hey guys,

the current "typo3/sysext/core/Classes/Resource/Index/MetaDataRepository.php" create a index record with "file,pid,crdate,tstamp,cruser_id". But in the database definition the "l10n_diffsource" has no default value:
...
l10n_diffsource mediumblob NOT NULL,
...

So we get this MySQL Warning: Warning 1364 Field 'l10n_diffsource' doesn't have a default value

Should we add a "DEFAULT ''" to the definition or set a empty string for the field in MetaDataRepository?

Regards,
Tim


Related issues 3 (0 open3 closed)

Related to TYPO3 Core - Bug #69031: Use proper data sets in category collection testsClosedMorton Jonuschat2015-08-13

Actions
Related to TYPO3 Core - Bug #69000: Functional Tests: Field 'tx_irretutorial_1ncsv_hotels' doesn't have a default valueClosedMorton Jonuschat2015-08-12

Actions
Related to TYPO3 Core - Bug #85084: Language field in Alternative Page Language always defaults to 0Closed2018-05-25

Actions
Actions #1

Updated by Georg Ringer over 8 years ago

mysql strict is currently not supported by the core. but yes should be fixed. wanna create a patch?

Actions #2

Updated by Gerrit Code Review over 8 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/42397

Actions #3

Updated by Morton Jonuschat over 8 years ago

  • Parent task set to #68397
Actions #4

Updated by Gerrit Code Review over 8 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/42397

Actions #5

Updated by Gerrit Code Review over 8 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/42397

Actions #6

Updated by Gerrit Code Review over 8 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/42397

Actions #7

Updated by Gerrit Code Review over 8 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/42397

Actions #8

Updated by Oliver Hader over 8 years ago

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

Updated by Anja Leichsenring over 5 years ago

  • Related to Bug #85084: Language field in Alternative Page Language always defaults to 0 added
Actions #10

Updated by Benni Mack over 5 years ago

  • Status changed from Resolved to Closed
Actions

Also available in: Atom PDF