adamjakab / SuiteCRM

include/Smarty/plugins/block.nocache.php

Indentation +1 added lines, -1 removed lines

@@ -2,6 +2,6 @@
 //implements Smarty 3 style {nocache}{/nocache} block to prevent caching of a section of a template.
 //remove this upon upgrade to Smarty 3
 function smarty_block_nocache($param, $content, &$smarty) {
-   return $content;
+    return $content;
 } 
 ?>
\ No newline at end of file

include/Smarty/plugins/function.html_radios.php

Spacing +17 added lines, -17 removed lines

@@ -40,7 +40,7 @@ 
  */
 function smarty_function_html_radios($params, &$smarty)
 {
-    require_once $smarty->_get_plugin_filepath('shared','escape_special_chars');
+    require_once $smarty->_get_plugin_filepath('shared', 'escape_special_chars');
    
     $name = 'radio';
     $values = null;
@@ -52,8 +52,8 @@ 
     $output = null;
     $extra = '';
 
-    foreach($params as $_key => $_val) {
-        switch($_key) {
+    foreach ($params as $_key => $_val) {
+        switch ($_key) {
             case 'name':
             case 'separator':
                 $$_key = (string)$_val;
@@ -61,8 +61,8 @@ 
 
             case 'checked':
             case 'selected':
-                if(is_array($_val)) {
-                    $smarty->trigger_error('html_radios: the "' . $_key . '" attribute cannot be an array', E_USER_WARNING);
+                if (is_array($_val)) {
+                    $smarty->trigger_error('html_radios: the "'.$_key.'" attribute cannot be an array', E_USER_WARNING);
                 } else {
                     $selected = (string)$_val;
                 }
@@ -91,7 +91,7 @@ 
                 break;
 
             default:
-                if(!is_array($_val)) {
+                if (!is_array($_val)) {
                     $extra .= ' '.$_key.'="'.smarty_function_escape_special_chars($_val).'"';
                 } else {
                     $smarty->trigger_error("html_radios: extra attribute '$_key' cannot be an array", E_USER_NOTICE);
@@ -119,10 +119,10 @@ 
 
     }
 
-    if(!empty($params['assign'])) {
+    if (!empty($params['assign'])) {
         $smarty->assign($params['assign'], $_html_result);
     } else {
-        return implode("\n",$_html_result);
+        return implode("\n", $_html_result);
     }
 
 }
@@ -130,25 +130,25 @@ 
 function smarty_function_html_radios_output($name, $value, $output, $selected, $extra, $separator, $labels, $label_ids) {
     $_output = '';
     if ($labels) {
-      if($label_ids) {
-          $_id = smarty_function_escape_special_chars(preg_replace('![^\w\-\.]!', '_', $name . '_' . $value));
-          $_output .= '<label for="' . $_id . '">';
+      if ($label_ids) {
+          $_id = smarty_function_escape_special_chars(preg_replace('![^\w\-\.]!', '_', $name.'_'.$value));
+          $_output .= '<label for="'.$_id.'">';
       } else {
           $_output .= '<label>';           
       }
    }
    $_output .= '<input type="radio" name="'
-        . smarty_function_escape_special_chars($name) . '" value="'
-        . smarty_function_escape_special_chars($value) . '"';
+        . smarty_function_escape_special_chars($name).'" value="'
+        . smarty_function_escape_special_chars($value).'"';
 
-   if ($labels && $label_ids) $_output .= ' id="' . $_id . '"';
+   if ($labels && $label_ids) $_output .= ' id="'.$_id.'"';
 
-    if ((string)$value==$selected) {
+    if ((string)$value == $selected) {
         $_output .= ' checked="checked"';
     }
-    $_output .= $extra . ' />' . $output;
+    $_output .= $extra.' />'.$output;
     if ($labels) $_output .= '</label>';
-    $_output .=  $separator;
+    $_output .= $separator;
 
     return $_output;
 }

Braces +13 added lines, -6 removed lines

@@ -100,15 +100,18 @@ 
         }
     }
 
-    if (!isset($options) && !isset($values))
-        return ''; /* raise error here? */
+    if (!isset($options) && !isset($values)) {
+            return '';
+    }
+    /* raise error here? */
 
     $_html_result = array();
 
     if (isset($options)) {
 
-        foreach ($options as $_key=>$_val)
-            $_html_result[] = smarty_function_html_radios_output($name, $_key, $_val, $selected, $extra, $separator, $labels, $label_ids);
+        foreach ($options as $_key=>$_val) {
+                    $_html_result[] = smarty_function_html_radios_output($name, $_key, $_val, $selected, $extra, $separator, $labels, $label_ids);
+        }
 
     } else {
 
@@ -141,13 +144,17 @@ 
         . smarty_function_escape_special_chars($name) . '" value="'
         . smarty_function_escape_special_chars($value) . '"';
 
-   if ($labels && $label_ids) $_output .= ' id="' . $_id . '"';
+   if ($labels && $label_ids) {
+       $_output .= ' id="' . $_id . '"';
+   }
 
     if ((string)$value==$selected) {
         $_output .= ' checked="checked"';
     }
     $_output .= $extra . ' />' . $output;
-    if ($labels) $_output .= '</label>';
+    if ($labels) {
+        $_output .= '</label>';
+    }
     $_output .=  $separator;
 
     return $_output;

Doc Comments +8 added lines

@@ -127,6 +127,14 @@
 
 }
 
+/**
+ * @param string $name
+ * @param null|string $selected
+ * @param string $extra
+ * @param string $separator
+ * @param boolean $labels
+ * @param boolean $label_ids
+ */
 function smarty_function_html_radios_output($name, $value, $output, $selected, $extra, $separator, $labels, $label_ids) {
     $_output = '';
     if ($labels) {

Indentation +9 added lines, -9 removed lines

@@ -130,18 +130,18 @@
 function smarty_function_html_radios_output($name, $value, $output, $selected, $extra, $separator, $labels, $label_ids) {
     $_output = '';
     if ($labels) {
-      if($label_ids) {
-          $_id = smarty_function_escape_special_chars(preg_replace('![^\w\-\.]!', '_', $name . '_' . $value));
-          $_output .= '<label for="' . $_id . '">';
-      } else {
-          $_output .= '<label>';           
-      }
-   }
-   $_output .= '<input type="radio" name="'
+        if($label_ids) {
+            $_id = smarty_function_escape_special_chars(preg_replace('![^\w\-\.]!', '_', $name . '_' . $value));
+            $_output .= '<label for="' . $_id . '">';
+        } else {
+            $_output .= '<label>';           
+        }
+    }
+    $_output .= '<input type="radio" name="'
         . smarty_function_escape_special_chars($name) . '" value="'
         . smarty_function_escape_special_chars($value) . '"';
 
-   if ($labels && $label_ids) $_output .= ' id="' . $_id . '"';
+    if ($labels && $label_ids) $_output .= ' id="' . $_id . '"';
 
     if ((string)$value==$selected) {
         $_output .= ' checked="checked"';

include/Smarty/plugins/function.sugar_getimage.php

Indentation +19 added lines, -19 removed lines

@@ -53,28 +53,28 @@
 
 function smarty_function_sugar_getimage($params, &$smarty) {
 
-	// error checking for required parameters
-	if(!isset($params['name'])) 
-		$smarty->trigger_error($GLOBALS['app_strings']['ERR_MISSING_REQUIRED_FIELDS'] . 'name');
+    // error checking for required parameters
+    if(!isset($params['name'])) 
+        $smarty->trigger_error($GLOBALS['app_strings']['ERR_MISSING_REQUIRED_FIELDS'] . 'name');
 
-	// temp hack to deprecate the use of other_attributes
-	if(isset($params['other_attributes']))
-		$params['attr'] = $params['other_attributes'];
+    // temp hack to deprecate the use of other_attributes
+    if(isset($params['other_attributes']))
+        $params['attr'] = $params['other_attributes'];
 
-	// set defaults
-	if(!isset($params['attr']))
-		$params['attr'] = '';
-	if(!isset($params['width']))
-		$params['width'] = null;
-	if(!isset($params['height']))
-		$params['height'] = null;
-	if(!isset($params['alt'])) 
-		$params['alt'] = '';
+    // set defaults
+    if(!isset($params['attr']))
+        $params['attr'] = '';
+    if(!isset($params['width']))
+        $params['width'] = null;
+    if(!isset($params['height']))
+        $params['height'] = null;
+    if(!isset($params['alt'])) 
+        $params['alt'] = '';
 
-	// deprecated ?
-	if(!isset($params['ext']))
-		$params['ext'] = null;
+    // deprecated ?
+    if(!isset($params['ext']))
+        $params['ext'] = null;
 
-	return SugarThemeRegistry::current()->getImage($params['name'], $params['attr'], $params['width'], $params['height'], $params['ext'], $params['alt']);	
+    return SugarThemeRegistry::current()->getImage($params['name'], $params['attr'], $params['width'], $params['height'], $params['ext'], $params['alt']);	
 }
 ?>

Spacing +8 added lines, -8 removed lines

@@ -54,25 +54,25 @@
 function smarty_function_sugar_getimage($params, &$smarty) {
 
 	// error checking for required parameters
-	if(!isset($params['name'])) 
-		$smarty->trigger_error($GLOBALS['app_strings']['ERR_MISSING_REQUIRED_FIELDS'] . 'name');
+	if (!isset($params['name'])) 
+		$smarty->trigger_error($GLOBALS['app_strings']['ERR_MISSING_REQUIRED_FIELDS'].'name');
 
 	// temp hack to deprecate the use of other_attributes
-	if(isset($params['other_attributes']))
+	if (isset($params['other_attributes']))
 		$params['attr'] = $params['other_attributes'];
 
 	// set defaults
-	if(!isset($params['attr']))
+	if (!isset($params['attr']))
 		$params['attr'] = '';
-	if(!isset($params['width']))
+	if (!isset($params['width']))
 		$params['width'] = null;
-	if(!isset($params['height']))
+	if (!isset($params['height']))
 		$params['height'] = null;
-	if(!isset($params['alt'])) 
+	if (!isset($params['alt'])) 
 		$params['alt'] = '';
 
 	// deprecated ?
-	if(!isset($params['ext']))
+	if (!isset($params['ext']))
 		$params['ext'] = null;
 
 	return SugarThemeRegistry::current()->getImage($params['name'], $params['attr'], $params['width'], $params['height'], $params['ext'], $params['alt']);	

Braces +21 added lines, -14 removed lines

@@ -54,26 +54,33 @@
 function smarty_function_sugar_getimage($params, &$smarty) {
 
 	// error checking for required parameters
-	if(!isset($params['name'])) 
-		$smarty->trigger_error($GLOBALS['app_strings']['ERR_MISSING_REQUIRED_FIELDS'] . 'name');
+	if(!isset($params['name'])) {
+			$smarty->trigger_error($GLOBALS['app_strings']['ERR_MISSING_REQUIRED_FIELDS'] . 'name');
+	}
 
 	// temp hack to deprecate the use of other_attributes
-	if(isset($params['other_attributes']))
-		$params['attr'] = $params['other_attributes'];
+	if(isset($params['other_attributes'])) {
+			$params['attr'] = $params['other_attributes'];
+	}
 
 	// set defaults
-	if(!isset($params['attr']))
-		$params['attr'] = '';
-	if(!isset($params['width']))
-		$params['width'] = null;
-	if(!isset($params['height']))
-		$params['height'] = null;
-	if(!isset($params['alt'])) 
-		$params['alt'] = '';
+	if(!isset($params['attr'])) {
+			$params['attr'] = '';
+	}
+	if(!isset($params['width'])) {
+			$params['width'] = null;
+	}
+	if(!isset($params['height'])) {
+			$params['height'] = null;
+	}
+	if(!isset($params['alt'])) {
+			$params['alt'] = '';
+	}
 
 	// deprecated ?
-	if(!isset($params['ext']))
-		$params['ext'] = null;
+	if(!isset($params['ext'])) {
+			$params['ext'] = null;
+	}
 
 	return SugarThemeRegistry::current()->getImage($params['name'], $params['attr'], $params['width'], $params['height'], $params['ext'], $params['alt']);	
 }

include/Smarty/plugins/function.config_load.php

Spacing +11 added lines, -11 removed lines

@@ -28,7 +28,7 @@ 
 {
         if ($smarty->debugging) {
             $_params = array();
-            require_once(SMARTY_CORE_DIR . 'core.get_microtime.php');
+            require_once(SMARTY_CORE_DIR.'core.get_microtime.php');
             $_debug_start_time = smarty_core_get_microtime($_params, $smarty);
         }
 
@@ -59,13 +59,13 @@ 
                          'resource_base_path' => $smarty->config_dir,
                          'get_source' => false);
         $smarty->_parse_resource_name($_params);
-        $_file_path = $_params['resource_type'] . ':' . $_params['resource_name'];
+        $_file_path = $_params['resource_type'].':'.$_params['resource_name'];
         if (isset($_section))
             $_compile_file = $smarty->_get_compile_path($_file_path.'|'.$_section);
         else
             $_compile_file = $smarty->_get_compile_path($_file_path);
 
-        if($smarty->force_compile || !file_exists($_compile_file)) {
+        if ($smarty->force_compile || !file_exists($_compile_file)) {
             $_compile = true;
         } elseif ($smarty->compile_check) {
             $_params = array('resource_name' => $_file,
@@ -77,10 +77,10 @@ 
             $_compile = false;
         }
 
-        if($_compile) {
+        if ($_compile) {
             // compile config file
-            if(!is_object($smarty->_conf_obj)) {
-                require_once SMARTY_DIR . $smarty->config_class . '.class.php';
+            if (!is_object($smarty->_conf_obj)) {
+                require_once SMARTY_DIR.$smarty->config_class.'.class.php';
                 $smarty->_conf_obj = new $smarty->config_class();
                 $smarty->_conf_obj->overwrite = $smarty->config_overwrite;
                 $smarty->_conf_obj->booleanize = $smarty->config_booleanize;
@@ -97,13 +97,13 @@ 
             $smarty->_conf_obj->set_file_contents($_file, $_params['source_content']);
             $_config_vars = array_merge($smarty->_conf_obj->get($_file),
                     $smarty->_conf_obj->get($_file, $_section));
-            if(function_exists('var_export')) {
-                $_output = '<?php $_config_vars = ' . var_export($_config_vars, true) . '; ?>';
+            if (function_exists('var_export')) {
+                $_output = '<?php $_config_vars = '.var_export($_config_vars, true).'; ?>';
             } else {
-                $_output = '<?php $_config_vars = unserialize(\'' . strtr(serialize($_config_vars),array('\''=>'\\\'', '\\'=>'\\\\')) . '\'); ?>';
+                $_output = '<?php $_config_vars = unserialize(\''.strtr(serialize($_config_vars), array('\''=>'\\\'', '\\'=>'\\\\')).'\'); ?>';
             }
             $_params = (array('compile_path' => $_compile_file, 'compiled_content' => $_output, 'resource_timestamp' => $_params['resource_timestamp']));
-            require_once(SMARTY_CORE_DIR . 'core.write_compiled_resource.php');
+            require_once(SMARTY_CORE_DIR.'core.write_compiled_resource.php');
             smarty_core_write_compiled_resource($_params, $smarty);
         } else {
             include($_compile_file);
@@ -128,7 +128,7 @@ 
 
         if ($smarty->debugging) {
             $_params = array();
-            require_once(SMARTY_CORE_DIR . 'core.get_microtime.php');
+            require_once(SMARTY_CORE_DIR.'core.get_microtime.php');
             $smarty->_smarty_debug_info[] = array('type'      => 'config',
                                                 'filename'  => $_file.' ['.$_section.'] '.$_scope,
                                                 'depth'     => $smarty->_inclusion_depth,

Braces +5 added lines, -4 removed lines

@@ -60,10 +60,11 @@
                          'get_source' => false);
         $smarty->_parse_resource_name($_params);
         $_file_path = $_params['resource_type'] . ':' . $_params['resource_name'];
-        if (isset($_section))
-            $_compile_file = $smarty->_get_compile_path($_file_path.'|'.$_section);
-        else
-            $_compile_file = $smarty->_get_compile_path($_file_path);
+        if (isset($_section)) {
+                    $_compile_file = $smarty->_get_compile_path($_file_path.'|'.$_section);
+        } else {
+                    $_compile_file = $smarty->_get_compile_path($_file_path);
+        }
 
         if($smarty->force_compile || !file_exists($_compile_file)) {
             $_compile = true;

Doc Comments +1 added lines

@@ -23,6 +23,7 @@
  *       'global' => overrides scope, setting to parent if true)
  * </pre>
  * @param Smarty
+ * @param Smarty $smarty
  */
 function smarty_function_config_load($params, &$smarty)
 {

Indentation +6 added lines, -6 removed lines

@@ -56,8 +56,8 @@ 
         }
 
         $_params = array('resource_name' => $_file,
-                         'resource_base_path' => $smarty->config_dir,
-                         'get_source' => false);
+                            'resource_base_path' => $smarty->config_dir,
+                            'get_source' => false);
         $smarty->_parse_resource_name($_params);
         $_file_path = $_params['resource_type'] . ':' . $_params['resource_name'];
         if (isset($_section))
@@ -69,8 +69,8 @@ 
             $_compile = true;
         } elseif ($smarty->compile_check) {
             $_params = array('resource_name' => $_file,
-                             'resource_base_path' => $smarty->config_dir,
-                             'get_source' => false);
+                                'resource_base_path' => $smarty->config_dir,
+                                'get_source' => false);
             $_compile = $smarty->_fetch_resource_info($_params) &&
                 $_params['resource_timestamp'] > filemtime($_compile_file);
         } else {
@@ -89,8 +89,8 @@ 
             }
 
             $_params = array('resource_name' => $_file,
-                             'resource_base_path' => $smarty->config_dir,
-                             $_params['get_source'] = true);
+                                'resource_base_path' => $smarty->config_dir,
+                                $_params['get_source'] = true);
             if (!$smarty->_fetch_resource_info($_params)) {
                 return;
             }

include/Smarty/plugins/function.sugar_evalcolumn_old.php

Spacing +8 added lines, -8 removed lines

@@ -47,19 +47,19 @@ 
 function smarty_function_sugar_evalcolumn_old($params, &$smarty)
 {
     if (!isset($params['var']) || !isset($params['rowData'])) {
-        if(!isset($params['var']))  
+        if (!isset($params['var']))  
             $smarty->trigger_error("evalcolumn: missing 'var' parameter");
-        if(!isset($params['rowData']))  
+        if (!isset($params['rowData']))  
             $smarty->trigger_error("evalcolumn: missing 'rowData' parameter");
         return;
     }
 
-    if($params['var'] == '') {
+    if ($params['var'] == '') {
         return;
     }
 
-    if(is_array($params['var'])) {
-        foreach($params['var'] as $key => $value) {
+    if (is_array($params['var'])) {
+        foreach ($params['var'] as $key => $value) {
             $params['var'][$key] = searchReplace($value, $params['rowData']);
         }
     }
@@ -67,7 +67,7 @@ 
         $params['var'] = searchReplace($params['var'], $params['rowData']);
     }
 
-    if(isset($params['toJSON'])) {
+    if (isset($params['toJSON'])) {
         $json = getJSONobj();
         $params['var'] = $json->encode($params['var']);
     }
@@ -82,8 +82,8 @@ 
 function searchReplace($value, &$rowData) {
     preg_match_all('/\{\$(.*)\}/U', $value, $matches);
 
-    for($wp = 0; $wp < count($matches[0]); $wp++) {
-        if(isset($rowData[$matches[1][$wp]])) 
+    for ($wp = 0; $wp < count($matches[0]); $wp++) {
+        if (isset($rowData[$matches[1][$wp]])) 
             $value = str_replace($matches[0][$wp], $rowData[$matches[1][$wp]], $value);
         else 
             $value = str_replace($matches[0][$wp], '', $value);

Braces +12 added lines, -10 removed lines

@@ -47,10 +47,12 @@ 
 function smarty_function_sugar_evalcolumn_old($params, &$smarty)
 {
     if (!isset($params['var']) || !isset($params['rowData'])) {
-        if(!isset($params['var']))  
-            $smarty->trigger_error("evalcolumn: missing 'var' parameter");
-        if(!isset($params['rowData']))  
-            $smarty->trigger_error("evalcolumn: missing 'rowData' parameter");
+        if(!isset($params['var'])) {
+                    $smarty->trigger_error("evalcolumn: missing 'var' parameter");
+        }
+        if(!isset($params['rowData'])) {
+                    $smarty->trigger_error("evalcolumn: missing 'rowData' parameter");
+        }
         return;
     }
 
@@ -62,8 +64,7 @@ 
         foreach($params['var'] as $key => $value) {
             $params['var'][$key] = searchReplace($value, $params['rowData']);
         }
-    }
-    else {
+    } else {
         $params['var'] = searchReplace($params['var'], $params['rowData']);
     }
 
@@ -83,10 +84,11 @@ 
     preg_match_all('/\{\$(.*)\}/U', $value, $matches);
 
     for($wp = 0; $wp < count($matches[0]); $wp++) {
-        if(isset($rowData[$matches[1][$wp]])) 
-            $value = str_replace($matches[0][$wp], $rowData[$matches[1][$wp]], $value);
-        else 
-            $value = str_replace($matches[0][$wp], '', $value);
+        if(isset($rowData[$matches[1][$wp]])) {
+                    $value = str_replace($matches[0][$wp], $rowData[$matches[1][$wp]], $value);
+        } else {
+                    $value = str_replace($matches[0][$wp], '', $value);
+        }
     }
     return $value;
 }

include/Smarty/plugins/function.sugar_replace_vars.php

Indentation +24 added lines, -24 removed lines

@@ -40,12 +40,12 @@ 
  */
 function smarty_function_sugar_replace_vars($params, &$smarty)
 {
-	if(empty($params['subject']))  {
-	    $smarty->trigger_error("sugarvar: missing 'subject' parameter");
-	    return;
-	} 
-	$fields = empty($params['fields']) ? $smarty->get_template_vars('fields') : $params['fields'];
-	$lDelim = "[";
+    if(empty($params['subject']))  {
+        $smarty->trigger_error("sugarvar: missing 'subject' parameter");
+        return;
+    } 
+    $fields = empty($params['fields']) ? $smarty->get_template_vars('fields') : $params['fields'];
+    $lDelim = "[";
     $rDelim = "]";
     if ($params['use_curly'])
     {
@@ -53,10 +53,10 @@ 
         $rDelim = "}";
     }
     $subject = $params['subject'];
-	$matches = array();
-	$count = preg_match_all('/\\' . $lDelim . '([^\\' . $rDelim . ']*)\\' . $rDelim . '/', $subject, $matches);
+    $matches = array();
+    $count = preg_match_all('/\\' . $lDelim . '([^\\' . $rDelim . ']*)\\' . $rDelim . '/', $subject, $matches);
     for($i = 0; $i < $count; $i++) {
-		$match = $matches[1][$i];
+        $match = $matches[1][$i];
         //List views will have fields be an array where all the keys are upper case and the values are jsut strings
         if (!isset($fields[$match]) && isset($fields[strtoupper($match)]))
             $match = strtoupper($match);
@@ -74,21 +74,21 @@ 
             }
 
             if (isset($fields[$match]['type']) && $fields[$match]['type']=='enum'
-				&& isset($fields[$match]['options']) && isset($fields[$match]['options'][$value]))
-			{
-				$subject = str_replace($matches[0][$i], $fields[$match]['options'][$value], $subject);
-			} else 
-			{
-				$subject = str_replace($matches[0][$i], $value, $subject);
-			}
-		}
-	}
+                && isset($fields[$match]['options']) && isset($fields[$match]['options'][$value]))
+            {
+                $subject = str_replace($matches[0][$i], $fields[$match]['options'][$value], $subject);
+            } else 
+            {
+                $subject = str_replace($matches[0][$i], $value, $subject);
+            }
+        }
+    }
 		
-	if (!empty($params['assign']))
-	{
-		$smarty->assign($params['assign'], $subject);
-		return '';
-	}
+    if (!empty($params['assign']))
+    {
+        $smarty->assign($params['assign'], $subject);
+        return '';
+    }
 	
-	return $subject;
+    return $subject;
 }
\ No newline at end of file

Spacing +4 added lines, -4 removed lines

@@ -40,7 +40,7 @@ 
  */
 function smarty_function_sugar_replace_vars($params, &$smarty)
 {
-	if(empty($params['subject']))  {
+	if (empty($params['subject'])) {
 	    $smarty->trigger_error("sugarvar: missing 'subject' parameter");
 	    return;
 	} 
@@ -54,8 +54,8 @@ 
     }
     $subject = $params['subject'];
 	$matches = array();
-	$count = preg_match_all('/\\' . $lDelim . '([^\\' . $rDelim . ']*)\\' . $rDelim . '/', $subject, $matches);
-    for($i = 0; $i < $count; $i++) {
+	$count = preg_match_all('/\\'.$lDelim.'([^\\'.$rDelim.']*)\\'.$rDelim.'/', $subject, $matches);
+    for ($i = 0; $i < $count; $i++) {
 		$match = $matches[1][$i];
         //List views will have fields be an array where all the keys are upper case and the values are jsut strings
         if (!isset($fields[$match]) && isset($fields[strtoupper($match)]))
@@ -73,7 +73,7 @@ 
                 $value = $value['value'];
             }
 
-            if (isset($fields[$match]['type']) && $fields[$match]['type']=='enum'
+            if (isset($fields[$match]['type']) && $fields[$match]['type'] == 'enum'
 				&& isset($fields[$match]['options']) && isset($fields[$match]['options'][$value]))
 			{
 				$subject = str_replace($matches[0][$i], $fields[$match]['options'][$value], $subject);

Braces +4 added lines, -4 removed lines

@@ -58,8 +58,9 @@ 
     for($i = 0; $i < $count; $i++) {
 		$match = $matches[1][$i];
         //List views will have fields be an array where all the keys are upper case and the values are jsut strings
-        if (!isset($fields[$match]) && isset($fields[strtoupper($match)]))
-            $match = strtoupper($match);
+        if (!isset($fields[$match]) && isset($fields[strtoupper($match)])) {
+                    $match = strtoupper($match);
+        }
 
         $value = isset($fields[$match]) ? $fields[$match] : null;
         if (!is_null($value)) {
@@ -67,8 +68,7 @@ 
             {
                 $bean  = $smarty->get_template_vars('bean');
                 $value = $bean->$match;
-            }
-            else if (is_array($value) && isset($value['value']))
+            } else if (is_array($value) && isset($value['value']))
             {
                 $value = $value['value'];
             }

include/Smarty/plugins/function.html_checkboxes.php

Spacing +10 added lines, -10 removed lines

@@ -40,7 +40,7 @@ 
  */
 function smarty_function_html_checkboxes($params, &$smarty)
 {
-    require_once $smarty->_get_plugin_filepath('shared','escape_special_chars');
+    require_once $smarty->_get_plugin_filepath('shared', 'escape_special_chars');
 
     $name = 'checkbox';
     $values = null;
@@ -52,8 +52,8 @@ 
 
     $extra = '';
 
-    foreach($params as $_key => $_val) {
-        switch($_key) {
+    foreach ($params as $_key => $_val) {
+        switch ($_key) {
             case 'name':
             case 'separator':
                 $$_key = $_val;
@@ -86,7 +86,7 @@ 
                 break;
 
             default:
-                if(!is_array($_val)) {
+                if (!is_array($_val)) {
                     $extra .= ' '.$_key.'="'.smarty_function_escape_special_chars($_val).'"';
                 } else {
                     $smarty->trigger_error("html_checkboxes: extra attribute '$_key' cannot be an array", E_USER_NOTICE);
@@ -115,10 +115,10 @@ 
 
     }
 
-    if(!empty($params['assign'])) {
+    if (!empty($params['assign'])) {
         $smarty->assign($params['assign'], $_html_result);
     } else {
-        return implode("\n",$_html_result);
+        return implode("\n", $_html_result);
     }
 
 }
@@ -127,15 +127,15 @@ 
     $_output = '';
     if ($labels) $_output .= '<label>';
     $_output .= '<input type="checkbox" name="'
-        . smarty_function_escape_special_chars($name) . '[]" value="'
-        . smarty_function_escape_special_chars($value) . '"';
+        . smarty_function_escape_special_chars($name).'[]" value="'
+        . smarty_function_escape_special_chars($value).'"';
 
     if (in_array((string)$value, $selected)) {
         $_output .= ' checked="checked"';
     }
-    $_output .= $extra . ' />' . $output;
+    $_output .= $extra.' />'.$output;
     if ($labels) $_output .= '</label>';
-    $_output .=  $separator;
+    $_output .= $separator;
 
     return $_output;
 }

Braces +13 added lines, -6 removed lines

@@ -95,16 +95,19 @@ 
         }
     }
 
-    if (!isset($options) && !isset($values))
-        return ''; /* raise error here? */
+    if (!isset($options) && !isset($values)) {
+            return '';
+    }
+    /* raise error here? */
 
     settype($selected, 'array');
     $_html_result = array();
 
     if (isset($options)) {
 
-        foreach ($options as $_key=>$_val)
-            $_html_result[] = smarty_function_html_checkboxes_output($name, $_key, $_val, $selected, $extra, $separator, $labels);
+        foreach ($options as $_key=>$_val) {
+                    $_html_result[] = smarty_function_html_checkboxes_output($name, $_key, $_val, $selected, $extra, $separator, $labels);
+        }
 
 
     } else {
@@ -125,7 +128,9 @@ 
 
 function smarty_function_html_checkboxes_output($name, $value, $output, $selected, $extra, $separator, $labels) {
     $_output = '';
-    if ($labels) $_output .= '<label>';
+    if ($labels) {
+        $_output .= '<label>';
+    }
     $_output .= '<input type="checkbox" name="'
         . smarty_function_escape_special_chars($name) . '[]" value="'
         . smarty_function_escape_special_chars($value) . '"';
@@ -134,7 +139,9 @@ 
         $_output .= ' checked="checked"';
     }
     $_output .= $extra . ' />' . $output;
-    if ($labels) $_output .= '</label>';
+    if ($labels) {
+        $_output .= '</label>';
+    }
     $_output .=  $separator;
 
     return $_output;

Doc Comments +6 added lines

@@ -123,6 +123,12 @@
 
 }
 
+/**
+ * @param string $name
+ * @param string $extra
+ * @param string $separator
+ * @param boolean $labels
+ */
 function smarty_function_html_checkboxes_output($name, $value, $output, $selected, $extra, $separator, $labels) {
     $_output = '';
     if ($labels) $_output .= '<label>';

include/Smarty/plugins/function.sugar_getscript.php

Indentation +4 added lines, -4 removed lines

@@ -56,9 +56,9 @@
  */
 function smarty_function_sugar_getscript($params, &$smarty)
 {
-	if(!isset($params['file'])) {
-		   $smarty->trigger_error($GLOBALS['app_strings']['ERR_MISSING_REQUIRED_FIELDS'] . 'file');
-	}
- 	return getVersionedScript($params['file']);
+    if(!isset($params['file'])) {
+            $smarty->trigger_error($GLOBALS['app_strings']['ERR_MISSING_REQUIRED_FIELDS'] . 'file');
+    }
+        return getVersionedScript($params['file']);
 }
 ?>
\ No newline at end of file

Spacing +2 added lines, -2 removed lines

@@ -43,8 +43,8 @@
  */
 function smarty_function_sugar_getjspath($params, &$smarty)
 {
-	if(!isset($params['file'])) {
-		   $smarty->trigger_error($GLOBALS['app_strings']['ERR_MISSING_REQUIRED_FIELDS'] . 'file');
+	if (!isset($params['file'])) {
+		   $smarty->trigger_error($GLOBALS['app_strings']['ERR_MISSING_REQUIRED_FIELDS'].'file');
 	}
  	return getJSPath($params['file']);
 }

include/Smarty/plugins/modifier.to_url.php

Indentation +2 added lines, -2 removed lines

@@ -77,9 +77,9 @@
 function smarty_modifier_to_url($string)
 {
     if (preg_match('/^[^:\/]*:\/\/.*/', $string)) {
-    	return $string;
+        return $string;
     } else {
-    	return 'http://' . $string;
+        return 'http://' . $string;
     }
 }
 

Spacing +1 added lines, -1 removed lines

@@ -79,7 +79,7 @@
     if (preg_match('/^[^:\/]*:\/\/.*/', $string)) {
     	return $string;
     } else {
-    	return 'http://' . $string;
+    	return 'http://'.$string;
     }
 }