Don't overwrite original field classes in frameworks

Author: Anahkiasen

src/Former/Framework/TwitterBootstrap.php

@@ -130,12 +130,7 @@ public function getFieldClasses(Field $field, $classes)
       $classes = $this->filterFieldClasses($classes);
     }
 
-    // If we found any class, add them
-    if ($classes) {
-      $field->class(implode(' ', $classes));
-    }
-
-    return $field;
+    return $this->addClassesToField($field, $classes);
   }
 
   /**

src/Former/Framework/TwitterBootstrap3.php

@@ -202,12 +202,7 @@ public function getFieldClasses(Field $field, $classes)
       $classes[] = 'form-control';
     }
 
-    // If we found any class, add them
-    if ($classes) {
-      $field->class(implode(' ', $classes));
-    }
-
-    return $field;
+    return $this->addClassesToField($field, $classes);
   }
 
   /**

src/Former/Framework/ZurbFoundation.php

@@ -108,12 +108,7 @@ public function getFieldClasses(Field $field, $classes = array())
   {
     $classes = $this->filterFieldClasses($classes);
 
-    // If we found any class, add them
-    if ($classes) {
-      $field->class(implode(' ', $classes));
-    }
-
-    return $field;
+    return $this->addClassesToField($field, $classes);
   }
 
   public function getGroupClasses()

src/Former/Framework/ZurbFoundation4.php

@@ -110,12 +110,7 @@ public function getFieldClasses(Field $field, $classes = array())
       $classes = $this->filterFieldClasses($classes);
     }
 
-    // If we found any class, add them
-    if ($classes) {
-      $field->class(implode(' ', $classes));
-    }
-
-    return $field;
+    return $this->addClassesToField($field, $classes);
   }
 
   public function getGroupClasses()

src/Former/Traits/Framework.php

@@ -196,6 +196,22 @@ public function createIcon($iconType, $attributes = array(), $iconSettings = arr
   ///////////////////////////// HELPERS //////////////////////////////
   ////////////////////////////////////////////////////////////////////
 
+  /**
+   * Add classes to a field
+   *
+   * @param Field $field
+   * @param array $classes
+   */
+  protected function addClassesToField($field, $classes)
+  {
+    // If we found any class, add them
+    if ($classes) {
+      $field->addClass(implode(' ', $classes));
+    }
+
+    return $field;
+  }
+
   /**
    * Prepend an array of classes with a string
    *
@@ -249,6 +265,13 @@ public function createLabelOf(Field $field, Element $label = null)
     return $label;
   }
 
+  /**
+   * Get an option for the current framework
+   *
+   * @param string $option
+   *
+   * @return string
+   */
   protected function getFrameworkOption($option)
   {
     return $this->app['config']->get("former::{$this->current()}.$option");
@@ -269,5 +292,4 @@ public function wrapLabel($label)
   {
     return $label;
   }
-
 }