Bug #54412

Improve default value handling for NULL value fields

Added by Alexander Stehlik over 7 years ago. Updated over 3 years ago.

Status:
Closed
Priority:
Should have
Assignee:
-
Category:
FormEngine aka TCEforms
Target version:
Start date:
2013-12-14
Due date:
% Done:

100%

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

Description

Currently the default value handling for input fields that can be set to NULL (eval is set to null) is quite inconsistent when creating new records:

  • if mode is set to useOrOverridePlaceholder the field is NULL by default
  • if this setting is not used the default value is a non empty string

Additionally it is not possible to explicitly set a NULL default value.

It would be better to if the mode does not have any influence on the default value but if it is possible to set NULL as default.

When this is handled together with #53662 backward compatibility should not be a problem because currently new records will have NULL values by default but empty strings are stored in the database.


Files

screenshot_tca_config.png (48.6 KB) screenshot_tca_config.png Alexander Stehlik, 2014-03-05 23:30

Related issues

Related to TYPO3 Core - Bug #53662: DatabaseConnection INSERTquery does not allow NULL valuesClosed2013-11-15

Actions
Has duplicate TYPO3 Core - Bug #60487: File References override meta data with empty string per defaultClosedOliver Hader2014-07-23

Actions
#1

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

#2

Updated by Gerrit Code Review over 7 years ago

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

#3

Updated by Gerrit Code Review over 7 years ago

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

#4

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

#5

Updated by Gerrit Code Review over 7 years ago

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

#6

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

#7

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

#8

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

#9

Updated by Alexander Stehlik over 7 years ago

This is how it should look like in the configuration module when default is set to NULL and the patch is applied.

#10

Updated by Alexander Stehlik over 7 years ago

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

Updated by Riccardo De Contardi over 3 years ago

  • Status changed from Resolved to Closed

Also available in: Atom PDF