#4159: Increase textfields maxlength - better place for injecting max…

…length
GetDKAN · Jul 2, 2024 · b810750 · b810750
1 parent aaf3ed1
commit b810750
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/modules/json_form_widget/src/SchemaUiHandler.php b/modules/json_form_widget/src/SchemaUiHandler.php
@@ -278,6 +278,9 @@ public function updateWidgets(mixed $spec, array $element) {
       return $this->widgetRouter->getConfiguredWidget($spec, $element);
     }
     else {
+      if ($element['#type'] === 'textfield' && empty($element['#maxlength'])) {
+        $element['#maxlength'] = 256;
+      }
       return $element;
     }
   }

diff --git a/modules/json_form_widget/src/WidgetRouter.php b/modules/json_form_widget/src/WidgetRouter.php
@@ -80,9 +80,6 @@ public function getConfiguredWidget(mixed $spec, array $element) {
       $method_name = $widgets[$spec->widget];
       $element = $this->$method_name($spec, $element);
     }
-    if ($element['#type'] === 'textfield' && empty($element['#maxlength'])) {
-      $element['#maxlength'] = 256;
-    }
     return $element;
   }