Make error class of div in FormHelper::input configurable

This commit is contained in:
Mark van Driel 2017-03-08 13:15:00 +01:00
parent 8d0e1fadf7
commit 4728586365
2 changed files with 68 additions and 1 deletions

View file

@ -10734,4 +10734,70 @@ class FormHelperTest extends CakeTestCase {
$this->assertAttributeEquals($here, '_lastAction', $this->Form, "_lastAction shouldn't be empty.");
}
/**
* Tests the 'errorClass' option when error is returned.
*
* @return void
*/
public function testInputErrorClass() {
$result = $this->Form->input('Contact.field', array(
'type' => 'text',
'div' => array('errorClass' => 'has-error'),
'error' => true
));
$expected = array(
'div' => array('class' => 'input text has-error'),
'label' => array('for' => 'ContactField'),
'Field',
'/label',
'input' => array(
'type' => 'text', 'name' => 'data[Contact][field]',
'id' => 'ContactField', 'class' => 'form-error'
),
array('div' => array('class' => 'error')),
'Badness!',
'/div'
);
$this->assertTags($result, $expected);
$result = $this->Form->input('Contact.second', array(
'type' => 'text',
'error' => array('attributes' => array('class' => 'error'))
));
$expected = array(
'div' => array('class' => 'input text error'),
'label' => array('for' => 'ContactSecond'),
'Field',
'/label',
'input' => array(
'type' => 'text', 'name' => 'data[Contact][second]',
'id' => 'ContactSecond', 'class' => 'form-error'
),
array('div' => array('class' => 'error')),
'Badness!',
'/div'
);
$this->assertTags($result, $expected);
$result = $this->Form->input('Contact.third', array(
'type' => 'text',
'div' => array('errorClass' => 'has-error'),
'error' => array('attributes' => array('class' => 'form-control-feedback'))
));
$expected = array(
'div' => array('class' => 'input text has-error'),
'label' => array('for' => 'ContactThird'),
'Field',
'/label',
'input' => array(
'type' => 'text', 'name' => 'data[Contact][third]',
'id' => 'ContactThird', 'class' => 'form-error'
),
array('div' => array('class' => 'form-control-feedback')),
'Badness!',
'/div'
);
$this->assertTags($result, $expected);
}
}

View file

@ -1068,7 +1068,7 @@ class FormHelper extends AppHelper {
if ($type !== 'hidden' && $error !== false) {
$errMsg = $this->error($fieldName, $error);
if ($errMsg) {
$divOptions = $this->addClass($divOptions, 'error');
$divOptions = $this->addClass($divOptions, Hash::get($divOptions, 'errorClass', 'error'));
if ($errorMessage) {
$out['error'] = $errMsg;
}
@ -1089,6 +1089,7 @@ class FormHelper extends AppHelper {
if (!empty($divOptions['tag'])) {
$tag = $divOptions['tag'];
unset($divOptions['tag']);
unset($divOptions['errorClass']);
$output = $this->Html->tag($tag, $output, $divOptions);
}
return $output;