Bug #18866
closed
Epic #68397: Make TYPO3 work with MySQL strict mode
Cannot create page with Mysl in strict mode (STRICT_TRANS_TABLES)
0%
Description
When I want to create a page I get an Mysql error saying that it has an inavlid value for an integer field. The problem is that "storage_pid" is when not set initially an empty string. It seems that I get only default values in strict mode for NULL, but no type conversion from empty string to 0 occurs.
A possible solution would be to use 0 as TCA default, but I think it is more proper to make the conversion in TCEmain if necessary.
(issue imported from #M8542)
Files
Updated by Johannes over 16 years ago
confirming this bug. seems to be related with 0004693
Updated by Martin Kutschker over 16 years ago
No, the definition of "storage_pid" has not been changed.
Updated by Johannes over 16 years ago
yes of course. my comment was missleading.I mean: both bug come from mysql-5 strictmode and could alternatively (at least for me) be solved by changing sql-mode to MYSQL40 in my.ini.
Updated by Rene van Asperen about 16 years ago
The attached diff file seems to fix the problem. Can you give me some feedback about it?
Updated by Marcos Fadul almost 16 years ago
The diff file fix the problem for me too.
Good work.
Updated by Gerrit Code Review over 11 years ago
- Status changed from New to Under Review
Patch set 1 for branch master has been pushed to the review server.
It is available at https://review.typo3.org/19671
Updated by Gerrit Code Review over 11 years ago
Patch set 2 for branch master has been pushed to the review server.
It is available at https://review.typo3.org/19671
Updated by Gerrit Code Review over 11 years ago
Patch set 3 for branch master has been pushed to the review server.
It is available at https://review.typo3.org/19671
Updated by Alexander Opitz over 11 years ago
- Assignee set to Alexander Opitz
- Target version deleted (
0)
Updated by Gerrit Code Review over 11 years ago
Patch set 4 for branch master has been pushed to the review server.
It is available at https://review.typo3.org/19671
Updated by Gerrit Code Review over 11 years ago
Patch set 5 for branch master has been pushed to the review server.
It is available at https://review.typo3.org/19671
Updated by Gerrit Code Review about 11 years ago
Patch set 6 for branch master has been pushed to the review server.
It is available at https://review.typo3.org/19671
Updated by Gerrit Code Review about 11 years ago
Patch set 7 for branch master has been pushed to the review server.
It is available at https://review.typo3.org/19671
Updated by Gerrit Code Review about 11 years ago
Patch set 8 for branch master has been pushed to the review server.
It is available at https://review.typo3.org/19671
Updated by Gerrit Code Review about 11 years ago
Patch set 9 for branch master has been pushed to the review server.
It is available at https://review.typo3.org/19671
Updated by Gerrit Code Review about 11 years ago
Patch set 10 for branch master has been pushed to the review server.
It is available at https://review.typo3.org/19671
Updated by Gerrit Code Review about 11 years ago
Patch set 11 for branch master has been pushed to the review server.
It is available at https://review.typo3.org/19671
Updated by Gerrit Code Review about 11 years ago
Patch set 12 for branch master has been pushed to the review server.
It is available at https://review.typo3.org/19671
Updated by Anonymous about 11 years ago
- Status changed from Under Review to Resolved
Applied in changeset bad3f24c1d106ce4d2254c1a8e5710edb769151e.
Updated by Gerrit Code Review about 11 years ago
- Status changed from Resolved to Under Review
Patch set 1 for branch TYPO3_6-1 has been pushed to the review server.
It is available at https://review.typo3.org/23237
Updated by Stefan Neufeind about 11 years ago
Patch was reverted again since, after some discussion, this was not the right place to add the additional check. Will need a follow-up. The general idea itself however was okay.
Updated by Anonymous about 11 years ago
- Status changed from Under Review to Resolved
Applied in changeset e727d5ff0b8ee93de38983555164b19f6bf87377.
Updated by
Updated by Mathias Schreiber over 9 years ago
- Status changed from New to Rejected
Making TYPO3 work with script mode is at least a year of work.
We have this on our agenda, but since this touches a wider aspect of the code I'm closing these tickets down.
Updated by Morton Jonuschat about 9 years ago
- Parent task set to #68397
- Is Regression set to No