fixed: repeat group with 2 hidden elements and 1 non-hidden. Applying…

… $groupModel->setColumnCss twice resulted in mal-formed html when the form was rendered
Fabrik · Sep 16, 2013 · b53a6de · b53a6de
1 parent fad3a4c
commit b53a6de
Showing 1 changed file with 7 additions and 2 deletions.
diff --git a/components/com_fabrik/models/form.php b/components/com_fabrik/models/form.php
@@ -4374,9 +4374,14 @@ public function getGroupView($tmpl = '')
 
 			// Style attribute for group columns (need to occur after randomisation of the elements otherwise clear's are not ordered correctly)
 			$rowix = -1;
-			foreach ($aElements as $elKey => $element)
+
+			// Don't double setColumnCss otherwise wierdness ensues
+			if (!$groupModel->canRepeat())
 			{
-				$rowix = $groupModel->setColumnCss($element, $rowix);
+				foreach ($aElements as $elKey => $element)
+				{
+					$rowix = $groupModel->setColumnCss($element, $rowix);
+				}
 			}
 			$group->elements = $aElements;
 			$group->subgroups = $aSubGroups;