From a186a60c436b3ee3653ee8b55dbd2c5dd877f34f Mon Sep 17 00:00:00 2001 From: Samuel Georges Date: Wed, 9 Dec 2015 08:19:53 +1100 Subject: [PATCH] Fixes the datepicker saving an empty value when disabled Refs https://github.com/responsiv/campaign-plugin/issues/4 --- modules/backend/classes/FormWidgetBase.php | 3 ++- modules/backend/formwidgets/DatePicker.php | 5 +++++ 2 files changed, 7 insertions(+), 1 deletion(-) diff --git a/modules/backend/classes/FormWidgetBase.php b/modules/backend/classes/FormWidgetBase.php index 97182613b..168deedb8 100644 --- a/modules/backend/classes/FormWidgetBase.php +++ b/modules/backend/classes/FormWidgetBase.php @@ -97,7 +97,8 @@ abstract class FormWidgetBase extends WidgetBase } /** - * Process the postback value for this widget. + * Process the postback value for this widget. If the value is omitted from + * postback data, it will be NULL, otherwise it will be an empty string. * @param $value The existing value for this widget. * @return string The new value for this widget. */ diff --git a/modules/backend/formwidgets/DatePicker.php b/modules/backend/formwidgets/DatePicker.php index 0196f3072..49b40cdc7 100644 --- a/modules/backend/formwidgets/DatePicker.php +++ b/modules/backend/formwidgets/DatePicker.php @@ -1,6 +1,7 @@ formField->disabled) { + return FormField::NO_SAVE_DATA; + } + if (!strlen($value)) { return null; }