adamjakab / SuiteCRM

modules/SecurityGroups/SecurityGroup.php

Braces +8 added lines, -4 removed lines

@@ -15,8 +15,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();
@@ -178,7 +177,9 @@ 
         $GLOBALS['log']->debug("SecuritySuite: groupHasAccess $query");
         $result = $db->query($query);
         $row = $db->fetchByAssoc($result);
-        if(isset($row) && $row['results']>0) return true;
+        if(isset($row) && $row['results']>0) {
+            return true;
+        }
 
         return false;
     }
@@ -416,7 +417,10 @@ 
     }
 
     static function inherit_parentQuery(&$focus, $parent_type, $parent_id, $focus_id, $focus_module_dir) {
-        if(empty($parent_type) || empty($parent_id)) return; //no info passed
+        if(empty($parent_type) || empty($parent_id)) {
+            return;
+        }
+        //no info passed
 
         /** can speed this up by doing one query */
         //should be just one query but need a unique guid for each insert

modules/SecurityGroups/SecurityGroupMessage.php

Braces +12 added lines, -8 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');
+}
 
 class SecurityGroupMessage extends Basic {
 	var $new_schema = true;
@@ -47,8 +49,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();
@@ -106,7 +107,10 @@ 
 			$securitygroup_id = null; //6.4.0
 		}
 		$message = new SecurityGroupMessage();
-		if(empty($text)) return; // || !$feed->ACLAccess('save', true) )return;
+		if(empty($text)) {
+		    return;
+		}
+		// || !$feed->ACLAccess('save', true) )return;
 
 		$text = strip_tags($text);
 		$message->name = '';
@@ -135,18 +139,18 @@ 
 		if($weeks == 1){
 			$result = translate('LBL_TIME_LAST_WEEK','SugarFeed').' ';
 			return $result;
-		}else if($weeks > 1){
+		} else if($weeks > 1){
 			$result .= $weeks . ' '.translate('LBL_TIME_WEEKS','SugarFeed').' ';
 			if($days > 0) {
                 $result .= $days . ' '.translate('LBL_TIME_DAYS','SugarFeed').' ';
             }
-		}else{
+		} else{
 			if($days == 1){
 				$result = translate('LBL_TIME_YESTERDAY','SugarFeed').' ';
 				return $result;
-			}else if($days > 1){
+			} else if($days > 1){
 				$result .= $days . ' '. translate('LBL_TIME_DAYS','SugarFeed').' ';
-			}else{
+			} else{
 				if($hours == 1) {
                     $result .= $hours . ' '.translate('LBL_TIME_HOUR','SugarFeed').' ';
                 } else {

modules/SecurityGroups/SecurityGroupUserRelationship.php

Braces +9 added lines, -7 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');
+}
 
 
 require_once('data/SugarBean.php');
@@ -55,8 +57,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();
@@ -98,10 +99,11 @@ 
         $query = "SELECT id, first_name, last_name, user_name FROM users ";
         $where_auto = "deleted=0";
 
-        if($where != "")
-            $query .= "where $where AND ".$where_auto;
-        else
-            $query .= "where ".$where_auto;
+        if($where != "") {
+                    $query .= "where $where AND ".$where_auto;
+        } else {
+                    $query .= "where ".$where_auto;
+        }
 
         $query .= " ORDER BY last_name, first_name";
 

modules/AOR_Charts/AOR_Chart.php

Braces +7 added lines, -7 removed lines

@@ -63,8 +63,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();
@@ -226,7 +225,7 @@ 
         $img = ob_get_clean();
         if($asDataURI){
             return 'data:image/png;base64,'.base64_encode($img);
-        }else{
+        } else{
             return $img;
         }
     }
@@ -359,8 +358,9 @@ 
     {
         $dataArray = json_decode($chartDataValues);
         $grouping = 'grouped'; //$mainGroupField->label; //'grouped';
-        if(!$grouped)
-            $grouping='stacked';
+        if(!$grouped) {
+                    $grouping='stacked';
+        }
         if(!is_array($dataArray)||count($dataArray) < 1)
         {
             return "<h3>$this->noDataMessage</h3>";
@@ -639,9 +639,9 @@ 
         if(strlen($label) > $maxLabelSize)
         {
             return substr($label,0,$maxLabelSize).'...';
+        } else {
+                    return $label;
         }
-        else
-            return $label;
     }
 
 

modules/Calls/Call.php

Braces +50 added lines, -42 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.
@@ -134,8 +136,9 @@ 
 
 
 
-         if(!empty($GLOBALS['app_list_strings']['duration_intervals']))
-        	$this->minutes_values = $GLOBALS['app_list_strings']['duration_intervals'];
+         if(!empty($GLOBALS['app_list_strings']['duration_intervals'])) {
+                 	$this->minutes_values = $GLOBALS['app_list_strings']['duration_intervals'];
+         }
 	}
 
 	/**
@@ -145,8 +148,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();
@@ -298,16 +300,17 @@ 
 			$where_auto = '1=1';
        		 if($show_deleted == 0){
             	$where_auto = " $this->table_name.deleted=0  ";
-			}else if($show_deleted == 1){
+			} else if($show_deleted == 1){
 				$where_auto = " $this->table_name.deleted=1 ";
 			}
 
 			//$where_auto .= " GROUP BY calls.id";
 
-		if($where != "")
-			$query .= "where $where AND ".$where_auto;
-		else
-			$query .= "where ".$where_auto;
+		if($where != "") {
+					$query .= "where $where AND ".$where_auto;
+		} else {
+					$query .= "where ".$where_auto;
+		}
 
         $order_by = $this->process_order_by($order_by);
         if (empty($order_by)) {
@@ -329,8 +332,7 @@ 
                     $query .= $custom_join['select'];
                     $query .= " FROM contacts, calls, calls_contacts ";
                     $where_auto = "calls_contacts.contact_id = contacts.id AND calls_contacts.call_id = calls.id AND calls.deleted=0 AND contacts.deleted=0";
-            }
-            else
+            } else
             {
                     $query = 'SELECT calls.*, users.user_name as assigned_user_name ';
                     $query .= $custom_join['select'];
@@ -343,10 +345,11 @@ 
 
             $query .= $custom_join['join'];
 
-			if($where != "")
-                    $query .= "where $where AND ".$where_auto;
-            else
-                    $query .= "where ".$where_auto;
+			if($where != "") {
+			                    $query .= "where $where AND ".$where_auto;
+			} else {
+                                $query .= "where ".$where_auto;
+            }
 
         $order_by = $this->process_order_by($order_by);
         if (empty($order_by)) {
@@ -390,10 +393,12 @@ 
 			$this->date_start = $timedate->now();
 		}
 
-		if (is_null($this->duration_hours))
-			$this->duration_hours = "0";
-		if (is_null($this->duration_minutes))
-			$this->duration_minutes = "1";
+		if (is_null($this->duration_hours)) {
+					$this->duration_hours = "0";
+		}
+		if (is_null($this->duration_minutes)) {
+					$this->duration_minutes = "1";
+		}
 
 		$this->fill_in_additional_parent_fields();
 
@@ -414,8 +419,9 @@ 
 
 		if ( empty($this->id) ) {
 		    $reminder_t = $GLOBALS['current_user']->getPreference('reminder_time');
-		    if ( isset($reminder_t) )
-		        $this->reminder_time = $reminder_t;
+		    if ( isset($reminder_t) ) {
+		    		        $this->reminder_time = $reminder_t;
+		    }
 		}
 		$this->reminder_checked = $this->reminder_time == -1 ? false : true;
 
@@ -424,8 +430,9 @@ 
 		}
 		if(empty($this->id)){
 			$reminder_t = $GLOBALS['current_user']->getPreference('email_reminder_time');
-			if(isset($reminder_t))
-		    		$this->email_reminder_time = $reminder_t;
+			if(isset($reminder_t)) {
+					    		$this->email_reminder_time = $reminder_t;
+			}
 		}
 		$this->email_reminder_checked = $this->email_reminder_time == -1 ? false : true;
 
@@ -440,16 +447,20 @@ 
 	function get_list_view_data(){
 		$call_fields = $this->get_list_view_array();
 		global $app_list_strings, $focus, $action, $currentModule;
-		if (isset($focus->id)) $id = $focus->id;
-		else $id = '';
+		if (isset($focus->id)) {
+		    $id = $focus->id;
+		} else {
+		    $id = '';
+		}
 		if (isset($this->parent_type) && $this->parent_type != null)
 		{
 			$call_fields['PARENT_MODULE'] = $this->parent_type;
 		}
 		if ($this->status == "Planned") {
 			//cn: added this if() to deal with sequential Closes in Meetings.  this is a hack to a hack (formbase.php->handleRedirect)
-			if(empty($action))
-			    $action = "index";
+			if(empty($action)) {
+						    $action = "index";
+			}
 
             $setCompleteUrl = "<a id='{$this->id}' onclick='SUGAR.util.closeActivityPanel.show(\"{$this->module_dir}\",\"{$this->id}\",\"Held\",\"listview\",\"1\");'>";
 			if ($this->ACLAccess('edit')) {
@@ -465,9 +476,9 @@ 
 		$date_db = $timedate->to_db($mergeTime);
 		if( $date_db	< $today){
 			$call_fields['DATE_START']= "<font class='overdueTask'>".$call_fields['DATE_START']."</font>";
-		}else if($date_db < $nextday){
+		} else if($date_db < $nextday){
 			$call_fields['DATE_START'] = "<font class='todaysTask'>".$call_fields['DATE_START']."</font>";
-		}else{
+		} else{
 			$call_fields['DATE_START'] = "<font class='futureTask'>".$call_fields['DATE_START']."</font>";
 		}
 		$this->fill_in_additional_detail_fields();
@@ -597,14 +608,12 @@ 
       {
         vCal::cache_sugar_vcal($user);
       }
-    }
-    else if ( $user->object_name == 'Contact')
+    } else if ( $user->object_name == 'Contact')
     {
       $relate_values = array('contact_id'=>$user->id,'call_id'=>$this->id);
       $data_values = array('accept_status'=>$status);
       $this->set_relationship($this->rel_contacts_table, $relate_values, true, true,$data_values);
-    }
-    else if ( $user->object_name == 'Lead')
+    } else if ( $user->object_name == 'Lead')
     {
       $relate_values = array('lead_id'=>$user->id,'call_id'=>$this->id);
       $data_values = array('accept_status'=>$status);
@@ -659,8 +668,9 @@ 
 		global $sugar_config;
 		if(isset($sugar_config['disable_notify_current_user']) && $sugar_config['disable_notify_current_user']) {
 			global $current_user;
-			if(isset($list[$current_user->id]))
-				unset($list[$current_user->id]);
+			if(isset($list[$current_user->id])) {
+							unset($list[$current_user->id]);
+			}
 		}
 //		$GLOBALS['log']->debug('Call.php->get_notification_recipients():'.print_r($list,true));
 		return $list;
@@ -704,7 +714,7 @@ 
 			if(!ACLController::moduleSupportsACL($this->parent_type) || ACLController::checkAccess($this->parent_type, 'view', $is_owner, 'module', $in_group)){
         	/* END - SECURITY GROUPS */
 				$array_assign['PARENT'] = 'a';
-			}else{
+			} else{
 				$array_assign['PARENT'] = 'span';
 			}
 		$is_owner = false;
@@ -735,7 +745,7 @@ 
 			if( ACLController::checkAccess('Contacts', 'view', $is_owner, 'module', $in_group)){
         	/* END - SECURITY GROUPS */
 				$array_assign['CONTACT'] = 'a';
-			}else{
+			} else{
 				$array_assign['CONTACT'] = 'span';
 			}
 
@@ -752,13 +762,11 @@ 
                 if(!empty($GLOBALS['soap_server_object']))
                 {
            		    $exclude = array('lead_id', 'contact_id', 'user_id');
-           	    }
-                else
+           	    } else
                 {
 	                $exclude = array('lead_id', 'contact_id', 'user_id', 'assigned_user_id');
            	    }
-            }
-            else
+            } else
             {
                 $exclude = array('user_id');
             }

modules/AOP_Case_Updates/AOP_Case_Updates.php

Braces +4 added lines, -5 removed lines

@@ -41,8 +41,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();
@@ -55,12 +54,12 @@ 
         parent::save($check_notify);
         if(file_exists('custom/modules/AOP_Case_Updates/CaseUpdatesHook.php')){
             require_once 'custom/modules/AOP_Case_Updates/CaseUpdatesHook.php';
-        }else{
+        } else{
             require_once 'modules/AOP_Case_Updates/CaseUpdatesHook.php';
         }
         if(class_exists('CustomCaseUpdatesHook')){
             $hook = new CustomCaseUpdatesHook();
-        }else{
+        } else{
             $hook = new CaseUpdatesHook();
         }
         $hook->sendCaseUpdate($this);
@@ -176,7 +175,7 @@ 
             $emailObj->created_by = '1';
             $emailObj->status = 'sent';
             $emailObj->save();
-        }else{
+        } else{
             $GLOBALS['log']->info("AOPCaseUpdates: Could not send email:  " . $mailer->ErrorInfo);
             return false;
         }

modules/AOR_Reports/AOR_Report.php

Braces +40 added lines, -35 removed lines

@@ -61,8 +61,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();
@@ -232,8 +231,10 @@ 
                 $html = '<script src="modules/AOR_Reports/js/Chart.js"></script>';
                 break;
             case self::CHART_TYPE_RGRAPH:
-                if($_REQUEST['module']!= 'Home')//Need the require_once for the rgraphincludes as they are only loaded when the home page is hit
+                if($_REQUEST['module']!= 'Home') {
+                    //Need the require_once for the rgraphincludes as they are only loaded when the home page is hit
                     require_once('include/SuiteGraphs/RGraphIncludes.php');
+                }
 
                 break;
         }
@@ -256,15 +257,13 @@ 
 
         if(count($rows) > 1) {
             $GLOBALS['log']->fatal('ambiguous group display for report ' . $this->id);
-        }
-        else if(count($rows) == 1){
+        } else if(count($rows) == 1){
             $rows[0]['module_path'] = unserialize(base64_decode($rows[0]['module_path']));
             if(!$rows[0]['module_path'][0]) {
                 $module = new $beanList[$this->report_module]();
                 $rows[0]['field_id_name'] = $module->field_defs[$rows[0]['field']]['id_name'] ? $module->field_defs[$rows[0]['field']]['id_name'] : $module->field_defs[$rows[0]['field']]['name'];
                 $rows[0]['module_path'][0] = $module->table_name;
-            }
-            else {
+            } else {
                 $rows[0]['field_id_name'] = $rows[0]['field'];
             }
             $path[] = $rows[0];
@@ -276,8 +275,7 @@ 
             }
 
             return $this->buildMultiGroupReport($offset, $links, $level+1, $path);
-        }
-        else if(!$rows) {
+        } else if(!$rows) {
             if($path) {
                 $html = '';
                 foreach ($path as $pth) {
@@ -295,12 +293,10 @@ 
                     }
                 }
                 return $html;
-            }
-            else {
+            } else {
                 return $this->build_group_report($offset, $links);
             }
-        }
-        else {
+        } else {
             throw new Exception('incorrect results');
         }
         throw new Exception('incorrect state');
@@ -436,8 +432,7 @@ 
                 foreach($extra['select'] as $selectField => $selectAlias) {
                     if($selectAlias) {
                         $query_array['select'][] = $selectField . " AS " . $selectAlias;
-                    }
-                    else {
+                    } else {
                         $query_array['select'][] = $selectField;
                     }
                 }
@@ -492,14 +487,18 @@ 
             $result = $this->db->query($query);
 
             while ($row = $this->db->fetchByAssoc($result)) {
-                if($html != '') $html .= '<br />';
+                if($html != '') {
+                    $html .= '<br />';
+                }
 
                $html .= $this->build_report_html($offset, $links, $row[$field_label], '', $extra);
 
             }
         }
 
-        if($html == '') $html = $this->build_report_html($offset, $links);
+        if($html == '') {
+            $html = $this->build_report_html($offset, $links);
+        }
         return $html;
 
     }
@@ -693,7 +692,9 @@ 
                     if($att['total']){
                         $totals[$name][] = $row[$name];
                     }
-                    if($att['link'] && $links) $html .= "</a>";
+                    if($att['link'] && $links) {
+                        $html .= "</a>";
+                    }
                     $html .= "</td>";
                 }
             }
@@ -777,7 +778,7 @@ 
             if($field['total']){
                 $totalLabel = $field['label'] ." ".$app_list_strings['aor_total_options'][$field['total']];
                 $html .= "<th>{$totalLabel}</th>";
-            }else{
+            } else{
                 $html .= "<th></th>";
             }
         }
@@ -789,7 +790,7 @@ 
             }
             if($field['total'] && isset($totals[$label])){
                 $html .= "<td>".$this->calculateTotal($field['total'],$totals[$label])."</td>";
-            }else{
+            } else{
                 $html .= "<td></td>";
             }
         }
@@ -865,10 +866,11 @@ 
             $csv .= "\r\n";
             foreach($fields as $name => $att){
                 if($att['display']){
-                    if($att['function'] != '' )
-                        $csv .= $this->encloseForCSV($row[$name]);
-                    else
-                        $csv .= $this->encloseForCSV(trim(strip_tags(getModuleField($att['module'], $att['field'], $att['field'], 'DetailView',$row[$name]))));
+                    if($att['function'] != '' ) {
+                                            $csv .= $this->encloseForCSV($row[$name]);
+                    } else {
+                                            $csv .= $this->encloseForCSV(trim(strip_tags(getModuleField($att['module'], $att['field'], $att['field'], 'DetailView',$row[$name]))));
+                    }
                     $csv .= $delimiter;
                 }
             }
@@ -1052,7 +1054,9 @@ 
 
                 $query['select'][] = $select_field ." AS '".$field->label."'";
 
-                if($field->group_display == 1 && $group_value) $query['where'][] = $select_field." = '".$group_value."' AND ";
+                if($field->group_display == 1 && $group_value) {
+                    $query['where'][] = $select_field." = '".$group_value."' AND ";
+                }
                     ++$i;
             }
         }
@@ -1077,13 +1081,13 @@ 
                                 $params['right_join_table_alias'] = $this->db->quoteIdentifier($alias);
                                 $params['join_table_alias'] = $this->db->quoteIdentifier($alias);
                                 $params['left_join_table_alias'] = $this->db->quoteIdentifier($parentAlias);
-                            }else{
+                            } else{
                                 $params['right_join_table_alias'] = $this->db->quoteIdentifier($parentAlias);
                                 $params['join_table_alias'] = $this->db->quoteIdentifier($alias);
                                 $params['left_join_table_alias'] = $this->db->quoteIdentifier($alias);
                             }
 
-                        }else{
+                        } else{
                             $params['right_join_table_alias'] = $this->db->quoteIdentifier($parentAlias);
                             $params['join_table_alias'] = $this->db->quoteIdentifier($alias);
                             $params['left_join_table_alias'] = $this->db->quoteIdentifier($parentAlias);
@@ -1298,12 +1302,16 @@ 
 
                         case 'Multi':
                             $sep = ' AND ';
-                            if ($condition->operator == 'Equal_To') $sep = ' OR ';
+                            if ($condition->operator == 'Equal_To') {
+                                $sep = ' OR ';
+                            }
                             $multi_values = unencodeMultienum($condition->value);
                             if (!empty($multi_values)) {
                                 $value = '(';
                                 foreach ($multi_values as $multi_value) {
-                                    if ($value != '(') $value .= $sep;
+                                    if ($value != '(') {
+                                        $value .= $sep;
+                                    }
                                     $value .= $field . ' ' . $app_list_strings['aor_sql_operator_list'][$condition->operator] . " '" . $multi_value . "'";
                                 }
                                 $value .= ')';
@@ -1376,18 +1384,15 @@ 
                     }
 
                     $tiltLogicOp = false;
-                }
-                else if($condition->parenthesis) {
+                } else if($condition->parenthesis) {
                     if($condition->parenthesis == 'START') {
                         $query['where'][] = ($tiltLogicOp ? '' : ($condition->logic_op ? $condition->logic_op . ' ' : 'AND ')) .  '(';
                         $tiltLogicOp = true;
-                    }
-                    else {
+                    } else {
                         $query['where'][] = ')';
                         $tiltLogicOp = false;
                     }
-                }
-                else {
+                } else {
                     $GLOBALS['log']->debug('illegal condition');
                 }
 

modules/AOR_Reports/Dashlets/AORReportsDashlet/AORReportsDashlet.php

Braces +5 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');
+}
 
 
 require_once('include/Dashlets/Dashlet.php');
@@ -48,8 +50,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($id, $def);
@@ -77,7 +78,7 @@ 
         if(!empty($this->report->id)) {
             //return $this->report->build_report_chart($this->charts, AOR_Report::CHART_TYPE_CHARTJS);
             return $this->report->build_report_chart($this->charts, AOR_Report::CHART_TYPE_RGRAPH);
-        }else{
+        } else{
             return '';
         }
     }

modules/ACLRoles/Popup_picker.php

Braces +4 added lines, -3 removed lines

@@ -43,7 +43,9 @@ 
  * Date: 06/03/15
  * Comments
  */
-if(!defined('sugarEntry') || !sugarEntry) die('Not A Valid Entry Point');
+if(!defined('sugarEntry') || !sugarEntry) {
+    die('Not A Valid Entry Point');
+}
 
 class CasesController extends SugarController {
 
@@ -90,8 +92,7 @@ 
                 $count++;
             }
             echo '</table>';
-        }
-        else {
+        } else {
             echo $mod_strings['LBL_NO_SUGGESTIONS'];
         }
         die();