Merge pull request #356 from colemanw/noMaxlength

ConvertEntity - Don't write maxlength, it's redundant
totten · Sep 16, 2024 · 4625e96 · 4625e96
2 parents a4af293 + 5d9bffa
commit 4625e96
Showing 1 changed file with 0 additions and 3 deletions.
diff --git a/src/CRM/CivixBundle/Command/ConvertEntityCommand.php b/src/CRM/CivixBundle/Command/ConvertEntityCommand.php
@@ -277,9 +277,6 @@ private static function getFieldsFromXml($xml, $thisTables): array {
       }
       $attributes = isset($fieldXml->html) ? self::snakeCaseKeys((array) $fieldXml->html) : [];
       unset($attributes['type']);
-      if (!empty($typeAttributes['length'])) {
-        $attributes['maxlength'] = (int) $typeAttributes['length'];
-      }
       // Fix if xml erroneously includes multiple labels
       if (!empty($attributes['label']) && !is_string($attributes['label'])) {
         $attributes['label'] = ((array) $attributes['label'])[0];