Merge pull request #180 from tommycox/7.x-3.x-fix-division-by-zero

soniktrooth · soniktrooth · commit f110f008558d · 2015-06-05T20:07:26.000-07:00
Added check to prevent division by zero when determining responsive grid size
diff --git a/includes/views.inc b/includes/views.inc
@@ -47,7 +47,9 @@ function kalatheme_preprocess_views_view_grid(&$variables) {
     if (!$variables['options']['responsive_toggle']) {
       $variables['options']['columns_' . $tier] = 1;
     }
-    if ($variables['gridsize'] % $variables['options']['columns_' . $tier] === 0) {
+    if (!empty($variables['options']['columns_' . $tier]) &&
+      $variables['gridsize'] % $variables['options']['columns_' . $tier] === 0
+    ) {
       $variables[$tier] = $variables['gridsize'] / $variables['options']['columns_' . $tier];
     }
   }

Original file line number	Diff line number	Diff line change
`@@ -47,7 +47,9 @@ function kalatheme_preprocess_views_view_grid(&$variables) {`
`47`	`47`	`if (!$variables['options']['responsive_toggle']) {`
`48`	`48`	`$variables['options']['columns_' . $tier] = 1;`
`49`	`49`	`}`
`50`		`- if ($variables['gridsize'] % $variables['options']['columns_' . $tier] === 0) {`
	`50`	`+ if (!empty($variables['options']['columns_' . $tier]) &&`
	`51`	`+ $variables['gridsize'] % $variables['options']['columns_' . $tier] === 0`
	`52`	`+ ) {`
`51`	`53`	`$variables[$tier] = $variables['gridsize'] / $variables['options']['columns_' . $tier];`
`52`	`54`	`}`
`53`	`55`	`}`