BUGFIX: Do not add form-control to wrapping holder div, only field it…

…self

code/BootstrapFieldList.php

@@ -1,27 +1,32 @@
 <?php
 
 
-class BootstrapFieldList extends Extension {
 
+class BootstrapFieldList extends Extension {
 
+	/**
+	 * A list of ignored fields that should not take on Bootstrap transforms
+	 * @var array
+	 */
 	protected $ignores = array ();
 
-
-	public function bootstrapify() {
-		$inline_fields = Config::inst()->get('BootstrapForm','inline_fields');
-
+	/**
+	 * Transforms all fields in the FieldList to use Bootstrap templates
+	 * @return FieldList
+	 */
+	public function bootstrapify() {		
 		foreach($this->owner as $f) {
 
-
 			if(isset($this->ignores[$f->getName()])) continue;
 
-			if($f instanceof CompositeField) {
-				$f->getChildren()->bootstrapify();
-				continue;
+			// If we have a Tabset, bootstrapify all Tabs
+			if($f instanceof TabSet) {
+				$f->Tabs()->bootstrapify();
 			}
 
-			if(!in_array($f->class, $inline_fields )) {
-				$f->addExtraClass('form-control');
+			// If we have a Tab, bootstrapify all its Fields
+			if($f instanceof Tab) {
+				$f->Fields()->bootstrapify();
 			}
 
 			$template = "Bootstrap{$f->class}_holder";			
@@ -49,14 +54,17 @@ public function bootstrapify() {
 		}
 
 		return $this->owner;		
-
 	}
 
-
-
+	/**
+	 * Adds this field as ignored. Should not take on boostrap transformation
+	 * 
+	 * @param  string $field The name of the form field
+	 * @return FieldList
+	 */
 	public function bootstrapIgnore($field) {
 		$this->ignores[$field] = true;
 
 		return $this->owner;
 	}
-}
+}

code/BootstrapFormField.php

@@ -178,5 +178,22 @@ private function loadErrorMessage() {
 			$this->addHelpText($this->owner->message);
 		}
 	}
+
+	/**
+	 * Adds the form-control class to *just* the formfield, not the holder.
+	 * This seems a bit of a hack, but addExtraClass() affects both the holder
+	 * and the field, so that's not a realistic option. We can't have form-control
+	 * on the wrapping div.
+	 * 
+	 * @param  FormField $field	 
+	 */
+	public function onBeforeRender (FormField $field) {		
+		$inline_fields = Config::inst()->get('BootstrapForm','inline_fields');
+
+		if(!in_array($field->class, $inline_fields )) {
+			$field->addExtraClass('form-control');
+		}
+	}
+
 
 }