[FOLLOWUP][BUGFIX] EXT:form - Cover empty strings for "skipIfValueIsEmpty" 72/54972/4
authorMathias Brodala <mbrodala@pagemachine.de>
Thu, 7 Dec 2017 17:10:03 +0000 (18:10 +0100)
committerSusanne Moog <susanne.moog@typo3.org>
Thu, 7 Dec 2017 21:07:29 +0000 (22:07 +0100)
This ensures that e.g. unchecked checkboxes are not tried to be
stored in the database as empty strings causing an error on int
fields.

Resolves: #82938
Releases: 8.7
Change-Id: I9f7032fa9a296be1dc94fc9e77cf2a8917cac580
Reviewed-on: https://review.typo3.org/54972
Reviewed-by: Bjoern Jacob <bjoern.jacob@tritum.de>
Tested-by: Bjoern Jacob <bjoern.jacob@tritum.de>
Tested-by: TYPO3com <no-reply@typo3.com>
Reviewed-by: Susanne Moog <susanne.moog@typo3.org>
Tested-by: Susanne Moog <susanne.moog@typo3.org>
typo3/sysext/form/Documentation/ApiReference/Index.rst
typo3/sysext/form/Documentation/Config/prototypes/finishersDefinition/finishers/SaveToDatabase.rst

index 747ff76..e8cd07f 100644 (file)
@@ -2814,7 +2814,7 @@ elements.<formElementIdentifier>.skipIfValueIsEmpty
 
 :aspect:`Description`
       Set this to true if the database column should not be written if the value from the submitted form element with the identifier
-      ``<formElementIdentifier>`` is empty (think about password fields etc.)
+      ``<formElementIdentifier>`` is empty (think about password fields etc.). Empty means strings without content, whitespace is valid content.
 
 
 .. _apireference-finisheroptions-savetodatabasefinisher-options-elements-<formelementidentifier>-savefileidentifierinsteadofuid:
@@ -2903,7 +2903,7 @@ databaseColumnMappings.<databaseColumnName>.skipIfValueIsEmpty
       false
 
 :aspect:`Description`
-      Set this to true if the database column should not be written if the value from ``options.databaseColumnMappings.<databaseColumnName>.value`` is empty.
+      Set this to true if the database column should not be written if the value from ``options.databaseColumnMappings.<databaseColumnName>.value`` is empty. Empty means strings without content, whitespace is valid content.
 
 
 
@@ -5650,4 +5650,4 @@ original ``FormElement model``.
 
 .. code-block:: javascript
 
-    var dolly = formElement.clone();
\ No newline at end of file
+    var dolly = formElement.clone();
index d1163c7..33a304e 100644 (file)
@@ -251,7 +251,8 @@ options.elements.<formElementIdentifier>.mapOnDatabaseColumn.skipIfValueIsEmpty
 
 :aspect:`Description`
       Set this to true if the database column should not be written if the value from the submitted form element with the identifier
-      ``<formElementIdentifier>`` is empty (think about password fields etc.)
+      ``<formElementIdentifier>`` is empty (think about password fields etc.). Empty means strings without content, whitespace
+      is valid content.
 
 
 .. _typo3.cms.form.prototypes.<prototypeIdentifier>.finishersdefinition.savetodatabase.options.databasecolumnmappings:
@@ -347,7 +348,7 @@ options.databaseColumnMappings.<databaseColumnName>.skipIfValueIsEmpty
       - :ref:`"Accessing form runtime values"<concepts-frontendrendering-codecomponents-customfinisherimplementations-accessingoptions-formruntimeaccessor>`
 
 :aspect:`Description`
-      Set this to true if the database column should not be written if the value from ``options.databaseColumnMappings.<databaseColumnName>.value`` is empty.
+      Set this to true if the database column should not be written if the value from ``options.databaseColumnMappings.<databaseColumnName>.value`` is empty. Empty means strings without content, whitespace is valid content.
 
 
 .. _typo3.cms.form.prototypes.<prototypeIdentifier>.finishersdefinition.savetodatabase.options.translation.translationfile: