adamjakab / SuiteCRM

modules/ACLRoles/DetailUserRole.php

Braces +6 added lines, -2 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.
@@ -72,7 +74,9 @@ 
     
     $names = array();
     $names = ACLAction::setupCategoriesMatrix($categories);
-    if(!empty($names))$tdwidth = 100 / sizeof($names);
+    if(!empty($names)) {
+        $tdwidth = 100 / sizeof($names);
+    }
     $sugar_smarty->assign('APP', $app_list_strings);
     $sugar_smarty->assign('CATEGORIES', $categories);
     $sugar_smarty->assign('TDWIDTH', $tdwidth);

modules/ACLRoles/EditView.php

Braces +9 added lines, -5 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.
@@ -69,12 +71,12 @@ 
 		$role->id = '';
 		$sugar_smarty->assign('ISDUPLICATE', $_REQUEST['record']);
 		$duplicateString=translate('LBL_DUPLICATE_OF', 'ACLRoles');
-	}else{
+	} else{
 		$return['record']= $role->id;
 		$return['action']='DetailView';
 	}
 
-}else{
+} else{
 	$categories = ACLRole::getRoleActions('');
 }
 $sugar_smarty->assign('ROLE', $role->toArray());
@@ -82,7 +84,9 @@ 
 
 if(isset($_REQUEST['return_module'])){
 	$return['module']=$_REQUEST['return_module'];
-	if(isset($_REQUEST['return_id']))$return['record']=$_REQUEST['return_id'];
+	if(isset($_REQUEST['return_id'])) {
+	    $return['record']=$_REQUEST['return_id'];
+	}
 	if(isset($_REQUEST['return_record'])){$return['record']=$_REQUEST['return_record'];}
     if(isset($_REQUEST['return_action'])){$return['action']=$_REQUEST['return_action'];}
     if ( !empty($return['record']) ) {
@@ -101,7 +105,7 @@ 
 $params[] = "<a href='index.php?module=ACLRoles&action=index'>{$mod_strings['LBL_MODULE_NAME']}</a>";
 if(empty($role->id)){
 	$params[] = $GLOBALS['app_strings']['LBL_CREATE_BUTTON_LABEL'];
-}else{
+} else{
 	$params[] = $role->get_summary_text();
 }
 echo getClassicModuleTitle("ACLRoles", $params, true);

modules/ACLRoles/ListUsers.php

Braces +6 added lines, -2 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.
@@ -42,7 +44,9 @@ 
 	sugar_die('No Access');
 }
 $record = '';
-if(isset($_REQUEST['record'])) $record = $_REQUEST['record'];
+if(isset($_REQUEST['record'])) {
+    $record = $_REQUEST['record'];
+}
 ?>
 <form action="index.php" method="post" name="DetailView" id="form">
 

modules/ACLRoles/DetailView.php

Braces +6 added lines, -2 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.
@@ -59,7 +61,9 @@ 
 $role->retrieve($_REQUEST['record']);
 $categories = ACLRole::getRoleActions($_REQUEST['record']);
 $names = ACLAction::setupCategoriesMatrix($categories);
-if(!empty($names))$tdwidth = 100 / sizeof($names);
+if(!empty($names)) {
+    $tdwidth = 100 / sizeof($names);
+}
 $sugar_smarty->assign('ROLE', $role->toArray());
 $sugar_smarty->assign('CATEGORIES', $categories);
 $sugar_smarty->assign('TDWIDTH', $tdwidth);

modules/ACLRoles/views/view.list.php

Braces +3 added lines, -2 removed lines

@@ -45,8 +45,9 @@
     public function preDisplay()
     {
         //bug #46690: Developer Access to Users/Teams/Roles
-        if (!$GLOBALS['current_user']->isAdminForModule('Users') && !$GLOBALS['current_user']->isDeveloperForModule('Users'))
-            sugar_die('No Access');
+        if (!$GLOBALS['current_user']->isAdminForModule('Users') && !$GLOBALS['current_user']->isDeveloperForModule('Users')) {
+                    sugar_die('No Access');
+        }
 
         $this->lv = new ListViewSmarty();
         $this->lv->export = false;

modules/ACLRoles/Save.php

Braces +7 added lines, -3 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.
@@ -42,7 +44,9 @@ 
 
 
 $role = new ACLRole();
-if(isset($_REQUEST['record']))$role->id = $_POST['record'];
+if(isset($_REQUEST['record'])) {
+    $role->id = $_POST['record'];
+}
 if(!empty($_REQUEST['name'])){
 	$role->name = $_POST['name'];
 	$role->description = $_POST['description'];
@@ -59,7 +63,7 @@ 
 	        }
 	    }
 	}
-}else{
+} else{
     ob_clean();	
     $flc_module = 'All';
     foreach($_POST as $name=>$value){

modules/Users/authentication/AuthenticationController.php

Braces +10 added lines, -5 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.
@@ -127,7 +129,9 @@ 
 		$_SESSION['loginAttempts'] = (isset($_SESSION['loginAttempts']))? $_SESSION['loginAttempts'] + 1: 1;
 		unset($GLOBALS['login_error']);
 
-		if($this->loggedIn)return $this->loginSuccess;
+		if($this->loggedIn) {
+		    return $this->loginSuccess;
+		}
 		LogicHook::initialize()->call_custom_logic('Users', 'before_login');
 
 		$this->loginSuccess = $this->authController->loginAuthenticate($username, $password, false, $PARAMS);
@@ -146,8 +150,9 @@ 
 			}
 
 			//call business logic hook
-			if(isset($GLOBALS['current_user']))
-				$GLOBALS['current_user']->call_custom_logic('after_login');
+			if(isset($GLOBALS['current_user'])) {
+							$GLOBALS['current_user']->call_custom_logic('after_login');
+			}
 
 			// Check for running Admin Wizard
 			$config = new Administration();
@@ -172,7 +177,7 @@ 
 				header("Location: index.php?module=Users&action=Wizard");
 				sugar_cleanup(true);
 			}
-		}else{
+		} else{
 			//kbrill bug #13225
 			LogicHook::initialize();
 			$GLOBALS['logic_hook']->call_custom_logic('Users', 'login_failed');

modules/Users/authentication/SugarAuthenticate/SugarAuthenticateUser.php

Braces +6 added lines, -2 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.
@@ -94,7 +96,9 @@ 
 		global $login_error;
 
 		$GLOBALS['log']->debug("Starting user load for ". $name);
-		if(empty($name) || empty($password)) return false;
+		if(empty($name) || empty($password)) {
+		    return false;
+		}
 		$input_hash = $password;
 		$passwordEncrypted = false;
 		if (!empty($PARAMS) && isset($PARAMS['passwordEncrypted']) && $PARAMS['passwordEncrypted']) {

modules/Users/UserViewHelper.php

Braces +14 added lines, -11 removed lines

@@ -158,7 +158,7 @@ 
 
         if (isset($GLOBALS['sugar_config']['show_download_tab'])) {
             $enable_download_tab = $GLOBALS['sugar_config']['show_download_tab'];
-        }else{
+        } else{
             $enable_download_tab = true;
         }
 
@@ -175,8 +175,7 @@ 
             if ($this->bean->id == $current_user->id) {
                 $reset_pref_warning = translate('LBL_RESET_PREFERENCES_WARNING','Users');
                 $reset_home_warning = translate('LBL_RESET_HOMEPAGE_WARNING','Users');
-            }
-            else {
+            } else {
                 $reset_pref_warning = translate('LBL_RESET_PREFERENCES_WARNING_USER','Users');
                 $reset_home_warning = translate('LBL_RESET_HOMEPAGE_WARNING_USER','Users');
             }
@@ -193,8 +192,12 @@ 
             $buttons_footer[]="<input type='button' class='button' id='reset_homepage_footer' onclick='if(confirm(\"{$reset_home_warning}\"))window.location=\"".$_SERVER['PHP_SELF'] .'?'.$the_query_string."&reset_homepage=true\";' value='".translate('LBL_RESET_HOMEPAGE','Users')."' />";
 
         }
-        if (isset($buttons_header)) $this->ss->assign("BUTTONS_HEADER", $buttons_header);
-        if (isset($buttons_footer)) $this->ss->assign("BUTTONS_FOOTER", $buttons_footer);
+        if (isset($buttons_header)) {
+            $this->ss->assign("BUTTONS_HEADER", $buttons_header);
+        }
+        if (isset($buttons_footer)) {
+            $this->ss->assign("BUTTONS_FOOTER", $buttons_footer);
+        }
 
 
 
@@ -388,7 +391,9 @@ 
 
         $this->ss->assign('EXPORT_DELIMITER', $this->bean->getPreference('export_delimiter'));
 
-        if($this->bean->receive_notifications ||(!isset($this->bean->id) && $admin->settings['notify_send_by_default'])) $this->ss->assign("RECEIVE_NOTIFICATIONS", "checked");
+        if($this->bean->receive_notifications ||(!isset($this->bean->id) && $admin->settings['notify_send_by_default'])) {
+            $this->ss->assign("RECEIVE_NOTIFICATIONS", "checked");
+        }
 
         //jc:12293 - modifying to use the accessor method which will translate the
         //available character sets using the translation files
@@ -479,7 +484,7 @@ 
             $this->ss->assign('EXTERNAL_AUTH_CLASS_1', $sugar_config['authenticationClass']);
             $this->ss->assign('EXTERNAL_AUTH_CLASS', $sugar_config['authenticationClass']);
             $authclass = $sugar_config['authenticationClass'];
-        }else{
+        } else{
             if(!empty($GLOBALS['system_config']->settings['system_ldap_enabled'])){
                 $this->ss->assign('EXTERNAL_AUTH_CLASS_1', translate('LBL_LDAP','Users'));
                 $this->ss
@@ -772,12 +777,10 @@ 
         if ($user->is_admin)
         {
             $user->user_type = 'Administrator';
-        }
-        else if ($user->is_group)
+        } else if ($user->is_group)
         {
             $user->user_type = 'GROUP';
-        }
-        else
+        } else
         {
             $user->user_type = 'RegularUser';
         }