Integer field type does not allow null values for config #8932

Activity

1. duncanmcclean

   added

   fieldtypes

   on Nov 6, 2023
2. duncanmcclean commented on Nov 8, 2023

   

   duncanmcclean

   on Nov 8, 2023

   Member

   More actions

   Can you provide your blueprint?
3. maurice-ellis commented on Nov 14, 2023

   

   maurice-ellis

   on Nov 14, 2023

   ContributorAuthor

   More actions

   'display' => __('Behavior'),
   'fields' => [
       'min' => [
           'display' => __('Min'),
           'instructions' => __('jumpstart::fieldtypes.config.min'),
           'type' => 'integer',
           'default' => 0,
       ],
       'max' => [
           'display' => __('Max'),
           'instructions' => __('jumpstart::fieldtypes.config.max'),
           'type' => 'integer',
       ],
       'step' => [
           'display' => __('Step'),
           'instructions' => __('jumpstart::fieldtypes.config.step'),
           'type' => 'text',
           'default' => 1,
       ],
       'default' => [
           'display' => __('Default Value'),
           'instructions' => __('jumpstart::fieldtypes.config.default'),
           'type' => 'text',
           'input_type' => 'number',
           'default' => null,
       ],
   ]
4. duncanmcclean commented on Aug 13, 2026

   

   duncanmcclean

   on Aug 13, 2026

   Member

   More actions

   The original problem here — Integer::preProcessConfig() casting to (int) and turning null into 0 — no longer exists on 6.x. That override was removed in #12395, so config values now fall through to preProcess(), which returns null untouched. A config field of 'type' => 'integer' with no default stays null.

   While confirming that, I found the mirror-image bug and fixed it: Integer::rules() and Floatval::rules() used truthy checks (if ($min = $this->config('min'))), so a legitimate min: 0 or max: 0 was silently dropped and never validated. Now that Min/Max/Step are user-facing config fields on both fieldtypes, that was easy to hit. Both now use is_null() checks.
5. duncanmcclean

   closed this as completedon Aug 13, 2026