adamjakab / SuiteCRM

modules/Campaigns/WebToLeadCreation.php

Braces +14 added lines, -9 removed lines

@@ -1,5 +1,7 @@ 
 <?php
-if(!defined('sugarEntry') || !sugarEntry) die('Not A Valid Entry Point');
+if(!defined('sugarEntry') || !sugarEntry) {
+    die('Not A Valid Entry Point');
+}
 /*********************************************************************************
  * SugarCRM Community Edition is a customer relationship management program developed by
  * SugarCRM, Inc. Copyright (C) 2004-2013 SugarCRM Inc.
@@ -97,8 +99,7 @@ 
         $personTypeList.="<option value='".$count."'>".$b->name."</option>";
         $count++;
     }
-}
-else
+} else
 {
     $personTypeList.="<option value='noPerson'>No matching types</option>";
 }
@@ -133,8 +134,12 @@ 
 
 $xtpl->assign("WEB_POST_URL",$web_post_url);
 
-if (empty($focus->assigned_user_id) && empty($focus->id))  $focus->assigned_user_id = $current_user->id;
-if (empty($focus->assigned_name) && empty($focus->id))  $focus->assigned_user_name = $current_user->user_name;
+if (empty($focus->assigned_user_id) && empty($focus->id)) {
+    $focus->assigned_user_id = $current_user->id;
+}
+if (empty($focus->assigned_name) && empty($focus->id)) {
+    $focus->assigned_user_name = $current_user->user_name;
+}
 $xtpl->assign("ASSIGNED_USER_NAME", $focus->assigned_user_name);
 $xtpl->assign("ASSIGNED_USER_ID", $focus->assigned_user_id );
 
@@ -197,8 +202,7 @@ 
                 $col_arr[0]=$cols_name;
                 $col_arr[1]=$field_def['name'];
                 $col_arr[2]=true;
-            }
-            else{
+            } else{
                 $cols_name=$field_def['vname'];
                 $col_arr[0]=$cols_name;
                 $col_arr[1]=$field_def['name'];
@@ -232,8 +236,9 @@ 
     foreach($GLOBALS['moduleList'] as $mod)
     {
         $item = BeanFactory::getBean($mod);
-        if($item && is_subclass_of($item,$superclass))
-            $subclasses[] = $item;
+        if($item && is_subclass_of($item,$superclass)) {
+                    $subclasses[] = $item;
+        }
     }
     return $subclasses;
 }

install/installConfig.php

Braces +42 added lines, -27 removed lines

@@ -1,5 +1,7 @@ 
 <?php
-if(!defined('sugarEntry') || !sugarEntry) die('Not A Valid Entry Point');
+if(!defined('sugarEntry') || !sugarEntry) {
+    die('Not A Valid Entry Point');
+}
 /*********************************************************************************
  * SugarCRM Community Edition is a customer relationship management program developed by
  * SugarCRM, Inc. Copyright (C) 2004-2013 SugarCRM Inc.
@@ -89,7 +91,9 @@ 
         $out = "<select name=\"$name\">";
         foreach($options as $key => $value) {
             $selected = '';
-            if($key==$default) $selected = ' selected="selected"';
+            if($key==$default) {
+                $selected = ' selected="selected"';
+            }
             $out .= "<option label=\"$value\" value=\"$key\"$selected>$value</option>";
         }
         $out .= "</select>";
@@ -275,8 +279,7 @@ 
                 if(!empty($value)) {
                     if(!empty($value['required'])) {
                         $form .= "<span class=\"required\">*</span>";
-                    }
-                    else {
+                    } else {
                     }
                     if(!empty($_SESSION[$name])) {
                         $sessval = $_SESSION[$name];
@@ -295,7 +298,7 @@ 
                     //if the type is password, set a hidden field to capture the value.  This is so that we can properly encode special characters, which is a limitation with password fields
                     if($type=='password'){
                         $form .= "</div><div class=\"install_block\"><label>{$mod_strings['LBL_DBCONF_TITLE_PSWD_INFO_LABEL']}</label><span>&nbsp;</span><input type='$type' name='{$name}_entry' id='{$name}_entry' value='".urldecode($sessval)."'><input type='hidden' name='$name' id='$name' value='".urldecode($sessval)."'></div><div class=\"install_block\">";
-                    }else{
+                    } else{
                         $form .= "<input type='$type' name='$name' id='$name' value='$sessval'>";
                     }
 
@@ -590,7 +593,9 @@ 
         $notify_allow_default_outbound_checked = empty($_SESSION['notify_allow_default_outbound']) ? '' : ' checked="checked" ';
 
         // set default smtp toggle buttons selected value
-        if(empty($_SESSION['smtp_tab_selected'])) $_SESSION['smtp_tab_selected'] = 'smtp_tab_other';
+        if(empty($_SESSION['smtp_tab_selected'])) {
+            $_SESSION['smtp_tab_selected'] = 'smtp_tab_other';
+        }
 
         $out .= <<<EOQ
         <div class="floatbox full" id="fb2">
@@ -1803,28 +1808,37 @@ 
 
 
 if( is_file("config.php") ){
-    if(!empty($sugar_config['default_theme']))
-        $_SESSION['site_default_theme'] = $sugar_config['default_theme'];
+    if(!empty($sugar_config['default_theme'])) {
+            $_SESSION['site_default_theme'] = $sugar_config['default_theme'];
+    }
 
-    if(!empty($sugar_config['disable_persistent_connections']))
-        $_SESSION['disable_persistent_connections'] =
+    if(!empty($sugar_config['disable_persistent_connections'])) {
+            $_SESSION['disable_persistent_connections'] =
             $sugar_config['disable_persistent_connections'];
-    if(!empty($sugar_config['default_language']))
-        $_SESSION['default_language'] = $sugar_config['default_language'];
-    if(!empty($sugar_config['translation_string_prefix']))
-        $_SESSION['translation_string_prefix'] = $sugar_config['translation_string_prefix'];
-    if(!empty($sugar_config['default_charset']))
-        $_SESSION['default_charset'] = $sugar_config['default_charset'];
-
-    if(!empty($sugar_config['default_currency_name']))
-        $_SESSION['default_currency_name'] = $sugar_config['default_currency_name'];
-    if(!empty($sugar_config['default_currency_symbol']))
-        $_SESSION['default_currency_symbol'] = $sugar_config['default_currency_symbol'];
-    if(!empty($sugar_config['default_currency_iso4217']))
-        $_SESSION['default_currency_iso4217'] = $sugar_config['default_currency_iso4217'];
-
-    if(!empty($sugar_config['rss_cache_time']))
-        $_SESSION['rss_cache_time'] = $sugar_config['rss_cache_time'];
+    }
+    if(!empty($sugar_config['default_language'])) {
+            $_SESSION['default_language'] = $sugar_config['default_language'];
+    }
+    if(!empty($sugar_config['translation_string_prefix'])) {
+            $_SESSION['translation_string_prefix'] = $sugar_config['translation_string_prefix'];
+    }
+    if(!empty($sugar_config['default_charset'])) {
+            $_SESSION['default_charset'] = $sugar_config['default_charset'];
+    }
+
+    if(!empty($sugar_config['default_currency_name'])) {
+            $_SESSION['default_currency_name'] = $sugar_config['default_currency_name'];
+    }
+    if(!empty($sugar_config['default_currency_symbol'])) {
+            $_SESSION['default_currency_symbol'] = $sugar_config['default_currency_symbol'];
+    }
+    if(!empty($sugar_config['default_currency_iso4217'])) {
+            $_SESSION['default_currency_iso4217'] = $sugar_config['default_currency_iso4217'];
+    }
+
+    if(!empty($sugar_config['rss_cache_time'])) {
+            $_SESSION['rss_cache_time'] = $sugar_config['rss_cache_time'];
+    }
     if(!empty($sugar_config['languages']))
     {
         // We need to encode the languages in a way that can be retrieved later.
@@ -1844,8 +1858,9 @@ 
 
 //Load in the array for the site scenarios
 require_once('install/suite_install/scenarios.php');
-if(isset($installation_scenarios))
+if(isset($installation_scenarios)) {
     $_SESSION['installation_scenarios'] = $installation_scenarios;
+}
 
 ////	errors
 $errors = '';

modules/Spots/Spots.php

Braces +3 added lines, -6 removed lines

@@ -176,8 +176,7 @@ 
                 $jsonConfig['exclusions'][$newKey] = $jsonConfig['exclusions'][$key];
                 unset($jsonConfig['exclusions'][$key]);
             }
-        }
-        elseif(isset($jsonConfig['exclusions']) && count($jsonConfig['exclusions']) == 0)
+        } elseif(isset($jsonConfig['exclusions']) && count($jsonConfig['exclusions']) == 0)
         {
             $jsonConfig['exclusions'] = new stdClass();
         }
@@ -188,8 +187,7 @@ 
                 $jsonConfig['inclusions'][$newKey] = $jsonConfig['inclusions'][$key];
                 unset($jsonConfig['inclusions'][$key]);
             }
-        }
-        elseif(isset($jsonConfig['inclusions']) && count($jsonConfig['inclusions']) == 0)
+        } elseif(isset($jsonConfig['inclusions']) && count($jsonConfig['inclusions']) == 0)
         {
             $jsonConfig['inclusions'] = new stdClass();
         }
@@ -200,8 +198,7 @@ 
                 $jsonConfig['inclusionsInfo'][$newKey] = $jsonConfig['inclusionsInfo'][$key];
                 unset($jsonConfig['inclusionsInfo'][$key]);
             }
-        }
-        elseif(isset($jsonConfig['inclusionsInfo']) && count($jsonConfig['inclusionsInfo']) == 0)
+        } elseif(isset($jsonConfig['inclusionsInfo']) && count($jsonConfig['inclusionsInfo']) == 0)
         {
             $jsonConfig['inclusionsInfo'] = new stdClass();
         }

modules/jjwg_Markers/Menu.php

Braces +9 added lines, -3 removed lines

@@ -44,7 +44,13 @@
 
     global $mod_strings, $app_strings, $sugar_config;
 
-    if(ACLController::checkAccess('jjwg_Markers', 'edit', true))$module_menu[]=Array("index.php?module=jjwg_Markers&action=EditView", $mod_strings['LNK_NEW_RECORD'],"Create", 'jjwg_Markers');
-    if(ACLController::checkAccess('jjwg_Markers', 'list', true))$module_menu[]=Array("index.php?module=jjwg_Markers&action=index", $mod_strings['LNK_LIST'],"List", 'jjwg_Markers');
-    if(ACLController::checkAccess('jjwg_Markers', 'import', true))$module_menu[]=Array("index.php?module=jjwg_Markers&action=index", $mod_strings['LNK_IMPORT_JJWG_MARKERS'],"Import", 'jjwg_Markers');
+    if(ACLController::checkAccess('jjwg_Markers', 'edit', true)) {
+        $module_menu[]=Array("index.php?module=jjwg_Markers&action=EditView", $mod_strings['LNK_NEW_RECORD'],"Create", 'jjwg_Markers');
+    }
+    if(ACLController::checkAccess('jjwg_Markers', 'list', true)) {
+        $module_menu[]=Array("index.php?module=jjwg_Markers&action=index", $mod_strings['LNK_LIST'],"List", 'jjwg_Markers');
+    }
+    if(ACLController::checkAccess('jjwg_Markers', 'import', true)) {
+        $module_menu[]=Array("index.php?module=jjwg_Markers&action=index", $mod_strings['LNK_IMPORT_JJWG_MARKERS'],"Import", 'jjwg_Markers');
+    }
 

modules/jjwg_Areas/Menu.php

Braces +12 added lines, -4 removed lines

@@ -1,6 +1,14 @@
 <?php
-if(!defined('sugarEntry') || !sugarEntry) die('Not A Valid Entry Point'); 
+if(!defined('sugarEntry') || !sugarEntry) {
+    die('Not A Valid Entry Point');
+}
 
-if(ACLController::checkAccess('jjwg_Areas', 'edit', true)) $module_menu[]=Array("index.php?module=jjwg_Areas&action=EditView&return_module=jjwg_Areas&return_action=index", $GLOBALS['mod_strings']['LNK_NEW_RECORD'], "Create", 'jjwg_Areas');
-if(ACLController::checkAccess('jjwg_Areas', 'list', true)) $module_menu[]=Array("index.php?module=jjwg_Areas&action=index&return_module=jjwg_Areas&return_action=DetailView", $GLOBALS['mod_strings']['LNK_LIST'], "List", 'jjwg_Areas');
-if(ACLController::checkAccess('jjwg_Areas', 'import', true))$module_menu[]=Array("index.php?module=jjwg_Areas&action=Step1&import_module=jjwg_Areas&return_module=jjwg_Areas&return_action=index", $GLOBALS['app_strings']['LBL_IMPORT'],"Import", 'jjwg_Areas');
+if(ACLController::checkAccess('jjwg_Areas', 'edit', true)) {
+    $module_menu[]=Array("index.php?module=jjwg_Areas&action=EditView&return_module=jjwg_Areas&return_action=index", $GLOBALS['mod_strings']['LNK_NEW_RECORD'], "Create", 'jjwg_Areas');
+}
+if(ACLController::checkAccess('jjwg_Areas', 'list', true)) {
+    $module_menu[]=Array("index.php?module=jjwg_Areas&action=index&return_module=jjwg_Areas&return_action=DetailView", $GLOBALS['mod_strings']['LNK_LIST'], "List", 'jjwg_Areas');
+}
+if(ACLController::checkAccess('jjwg_Areas', 'import', true)) {
+    $module_menu[]=Array("index.php?module=jjwg_Areas&action=Step1&import_module=jjwg_Areas&return_module=jjwg_Areas&return_action=index", $GLOBALS['app_strings']['LBL_IMPORT'],"Import", 'jjwg_Areas');
+}

modules/Spots/Menu.php

Braces +6 added lines, -2 removed lines

@@ -44,7 +44,11 @@
 
 global $mod_strings, $app_strings, $sugar_config;
 
-if(ACLController::checkAccess('Spots', 'edit', true))$module_menu[]=Array("index.php?module=Spots&action=EditView", $mod_strings['LNK_SPOT_CREATE'],"Create", 'Spots');
-if(ACLController::checkAccess('Spots', 'list', true))$module_menu[]=Array("index.php?module=Spots&action=index", $mod_strings['LNK_SPOT_LIST'],"List", 'Spots');
+if(ACLController::checkAccess('Spots', 'edit', true)) {
+    $module_menu[]=Array("index.php?module=Spots&action=EditView", $mod_strings['LNK_SPOT_CREATE'],"Create", 'Spots');
+}
+if(ACLController::checkAccess('Spots', 'list', true)) {
+    $module_menu[]=Array("index.php?module=Spots&action=index", $mod_strings['LNK_SPOT_LIST'],"List", 'Spots');
+}
 
 ;

modules/AOK_KnowledgeBase/Menu.php

Braces +6 added lines, -2 removed lines

@@ -44,6 +44,10 @@
 
     global $mod_strings, $app_strings, $sugar_config;
 
-    if(ACLController::checkAccess('AOK_KnowledgeBase', 'edit', true))$module_menu[]=Array("index.php?module=AOK_KnowledgeBase&action=EditView", $mod_strings['LNK_NEW_RECORD'],"Create", 'AOK_KnowledgeBase');
-    if(ACLController::checkAccess('AOK_KnowledgeBase', 'list', true))$module_menu[]=Array("index.php?module=AOK_KnowledgeBase&action=index", $mod_strings['LNK_LIST'],"List", 'AOK_KnowledgeBase');
+    if(ACLController::checkAccess('AOK_KnowledgeBase', 'edit', true)) {
+        $module_menu[]=Array("index.php?module=AOK_KnowledgeBase&action=EditView", $mod_strings['LNK_NEW_RECORD'],"Create", 'AOK_KnowledgeBase');
+    }
+    if(ACLController::checkAccess('AOK_KnowledgeBase', 'list', true)) {
+        $module_menu[]=Array("index.php?module=AOK_KnowledgeBase&action=index", $mod_strings['LNK_LIST'],"List", 'AOK_KnowledgeBase');
+    }
 

modules/AM_ProjectTemplates/Menu.php

Braces +12 added lines, -4 removed lines

@@ -1,5 +1,7 @@ 
 <?php
-if(!defined('sugarEntry') || !sugarEntry) die('Not A Valid Entry Point');
+if(!defined('sugarEntry') || !sugarEntry) {
+    die('Not A Valid Entry Point');
+}
 /**
  *
  * SugarCRM Community Edition is a customer relationship management program developed by
@@ -50,7 +52,13 @@ 
 global $mod_strings, $app_strings, $sugar_config;
 
 		
-if(ACLController::checkAccess('AM_ProjectTemplates', 'edit', true))$module_menu[]=Array("index.php?module=AM_ProjectTemplates&action=EditView&return_module=AM_ProjectTemplates&return_action=index", $mod_strings['LNK_NEW_RECORD'],"Create", 'AM_ProjectTemplates');
-if(ACLController::checkAccess('AM_ProjectTemplates', 'list', true))$module_menu[]=Array("index.php?module=AM_ProjectTemplates&action=index&return_module=AM_ProjectTemplates&return_action=DetailView", $mod_strings['LNK_LIST'],"List", 'AM_ProjectTemplates');
-if(ACLController::checkAccess('AM_ProjectTemplates', 'import', true))$module_menu[]=Array("index.php?module=Import&action=Step1&import_module=AM_ProjectTemplates&return_module=AM_ProjectTemplates&return_action=index", $mod_strings['LNK_IMPORT_AM_PROJECTTEMPLATES'],"Import", 'AM_ProjectTemplates');
+if(ACLController::checkAccess('AM_ProjectTemplates', 'edit', true)) {
+    $module_menu[]=Array("index.php?module=AM_ProjectTemplates&action=EditView&return_module=AM_ProjectTemplates&return_action=index", $mod_strings['LNK_NEW_RECORD'],"Create", 'AM_ProjectTemplates');
+}
+if(ACLController::checkAccess('AM_ProjectTemplates', 'list', true)) {
+    $module_menu[]=Array("index.php?module=AM_ProjectTemplates&action=index&return_module=AM_ProjectTemplates&return_action=DetailView", $mod_strings['LNK_LIST'],"List", 'AM_ProjectTemplates');
+}
+if(ACLController::checkAccess('AM_ProjectTemplates', 'import', true)) {
+    $module_menu[]=Array("index.php?module=Import&action=Step1&import_module=AM_ProjectTemplates&return_module=AM_ProjectTemplates&return_action=index", $mod_strings['LNK_IMPORT_AM_PROJECTTEMPLATES'],"Import", 'AM_ProjectTemplates');
+}
 ?>
\ No newline at end of file

modules/AM_ProjectTemplates/AM_ProjectTemplates.php

Braces +3 added lines, -4 removed lines

@@ -51,8 +51,7 @@ 
         $deprecatedMessage = 'PHP4 Style Constructors are deprecated and will be remove in 7.8, please update your code';
         if(isset($GLOBALS['log'])) {
             $GLOBALS['log']->deprecated($deprecatedMessage);
-        }
-        else {
+        } else {
             trigger_error($deprecatedMessage, E_USER_DEPRECATED);
         }
         self::__construct();
@@ -68,7 +67,7 @@ 
 		){
 			parent::save(true) ; 
 			$return_id = $focus->id;
-		}else{
+		} else{
 
 			if(!empty($_POST['user_invitees'])) {
 			   $userInvitees = explode(',', trim($_POST['user_invitees'], ','));
@@ -125,7 +124,7 @@ 
 				foreach($contacts as $a) {
 					  if(!in_array($a->id, $contactInvitees)) {
 						 $deleteContacts[$a->id] = $a->id;
-					  }	else {
+					  } else {
 						 $existingContacts[$a->id] = $a->id;
 					  }
 				}