tonyair/silverstripe-framework
1
0
Fork 0
mirror of https://github.com/silverstripe/silverstripe-framework synced 2024-10-01 05:39:10 +02:00
Code Issues Projects Releases Wiki Activity

Merge pull request #911 from chillu/pulls/formfield-extraclass

Browse Source 
BUG FormField->removeExtraClass() works on indexed arrays
This commit is contained in:
Ingo Schommer 2012-10-31 15:34:29 -07:00
commit 15a092a3e9
2 changed files with 19 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
forms/FormField.php
View File

@ -321,9 +321,9 @@ class FormField extends RequestHandler {
* @param $class String * @param $class String
*/ */
public function removeExtraClass($class) { public function removeExtraClass($class) {
if(isset($this->extraClasses) && array_key_exists($class, $this->extraClasses)) { $pos = array_search($class, $this->extraClasses);
unset($this->extraClasses[$class]); if($pos !== false) unset($this->extraClasses[$pos]);
}
return $this; return $this;
} }

16
tests/forms/FormFieldTest.php
View File

@ -5,6 +5,22 @@
*/ */
class FormFieldTest extends SapphireTest { class FormFieldTest extends SapphireTest {
public function testAddExtraClass() {
$field = new FormField('MyField');
$field->addExtraClass('class1');
$field->addExtraClass('class2');
$this->assertStringEndsWith('class1 class2', $field->extraClass());
}
public function testRemoveExtraClass() {
$field = new FormField('MyField');
$field->addExtraClass('class1');
$field->addExtraClass('class2');
$this->assertStringEndsWith('class1 class2', $field->extraClass());
$field->removeExtraClass('class1');
$this->assertStringEndsWith('class2', $field->extraClass());
}
public function testAttributes() { public function testAttributes() {
$field = new FormField('MyField'); $field = new FormField('MyField');
$field->setAttribute('foo', 'bar'); $field->setAttribute('foo', 'bar');