Merge pull request #1368 from rasstislav/fix-textarea-max-length

FIX Allow setMaxLength for all field types (TextField, TextareaField)

Author: GuySartorelli

code/Model/EditableFormField/EditableTextField.php

@@ -2,6 +2,7 @@
 
 namespace SilverStripe\UserForms\Model\EditableFormField;
 
+use SilverStripe\Core\ClassInfo;
 use SilverStripe\Forms\DropdownField;
 use SilverStripe\Forms\FieldGroup;
 use SilverStripe\Forms\FieldList;
@@ -185,7 +186,7 @@ protected function updateFormField($field)
         }
 
         if (is_numeric($this->MaxLength) && $this->MaxLength > 0) {
-            if ($field instanceof TextField) {
+            if (ClassInfo::hasMethod($field, 'setMaxLength')) {
                 $field->setMaxLength((int) $this->MaxLength);
             }
             $field->setAttribute('data-rule-maxlength', (int) $this->MaxLength);

tests/php/Model/EditableFormFieldTest.php

@@ -232,7 +232,7 @@ public function testLengthRange()
         // textarea
         $textField->Rows = 3;
         $attributes = $textField->getFormField()->getAttributes();
-        $this->assertFalse(isset($attributes['maxLength']));
+        $this->assertEquals(20, $attributes['maxlength']);
         $this->assertEquals(10, $attributes['data-rule-minlength']);
         $this->assertEquals(20, $attributes['data-rule-maxlength']);
     }