- #604830 by mattyoung - 32 characters limit on field and group identifiers in "Manag...
authorMarc Ferran (aka Markus)
Mon, 2 Nov 2009 21:21:35 +0000 (21:21 +0000)
committerMarc Ferran (aka Markus)
Mon, 2 Nov 2009 21:21:35 +0000 (21:21 +0000)
CHANGELOG.txt
includes/content.admin.inc

index 45a5105..798726d 100644 (file)
@@ -67,6 +67,7 @@ Bugfixes:
 - #605152 by pokurek - Missing number formatter fr_2.
 - #618910 Multigroup filter fails when using relationship.
 - #538458 Do not allow to change the widget type for fields in multigroups when the change is not compatible.
+- #604830 by mattyoung - 32 characters limit on field and group identifiers in "Manage fields" screen.
 
 Changes:
 - #421116 Move content.js to js/content.admin.js for consistency with location of new js files.
index a54160e..bb32759 100644 (file)
@@ -258,6 +258,9 @@ function content_field_overview_form(&$form_state, $type_name) {
         '#field_suffix' => '</span>&lrm;',
         '#attributes' => array('dir'=>'ltr'),
         '#size' => 15,
+        // Field names are limited to 32 characters including the 'field_'
+        // prefix which is 6 characters long.
+        '#maxlength' => 26,
         '#description' => t('Field name (a-z, 0-9, _)'),
       ),
       'type' => array(
@@ -329,6 +332,9 @@ function content_field_overview_form(&$form_state, $type_name) {
         '#field_suffix' => '</span>&lrm;',
         '#attributes' => array('dir'=>'ltr'),
         '#size' => 15,
+        // Group names are limited to 32 characters including the 'group_'
+        // prefix which is 6 characters long.
+        '#maxlength' => 26,
         '#description' => t('Group name (a-z, 0-9, _)'),
       ),
       'group_option' => array(