adamjakab / SuiteCRM

modules/Cases/Case.php

Indentation +258 added lines, -258 removed lines

@@ -61,291 +61,291 @@
 // Case is used to store customer information.
 class aCase extends Basic {
         var $field_name_map = array();
-	// Stored fields
-	var $id;
-	var $date_entered;
-	var $date_modified;
-	var $modified_user_id;
-	var $assigned_user_id;
-	var $case_number;
-	var $resolution;
-	var $description;
-	var $name;
-	var $status;
-	var $priority;
-
-
-	var $created_by;
-	var $created_by_name;
-	var $modified_by_name;
-
-	// These are related
-	var $bug_id;
-	var $account_name;
-	var $account_id;
-	var $contact_id;
-	var $task_id;
-	var $note_id;
-	var $meeting_id;
-	var $call_id;
-	var $email_id;
-	var $assigned_user_name;
-	var $account_name1;
-
-	var $table_name = "cases";
-	var $rel_account_table = "accounts_cases";
-	var $rel_contact_table = "contacts_cases";
-	var $module_dir = 'Cases';
-	var $object_name = "Case";
-	var $importable = true;
-	/** "%1" is the case_number, for emails
-	 * leave the %1 in if you customize this
-	 * YOU MUST LEAVE THE BRACKETS AS WELL*/
-	var $emailSubjectMacro = '[CASE:%1]';
-
-	// This is used to retrieve related fields from form posts.
-	var $additional_column_fields = Array('bug_id', 'assigned_user_name', 'assigned_user_id', 'contact_id', 'task_id', 'note_id', 'meeting_id', 'call_id', 'email_id');
-
-	var $relationship_fields = Array('account_id'=>'accounts', 'bug_id' => 'bugs',
-									'task_id'=>'tasks', 'note_id'=>'notes',
-									'meeting_id'=>'meetings', 'call_id'=>'calls', 'email_id'=>'emails',
-									);
+    // Stored fields
+    var $id;
+    var $date_entered;
+    var $date_modified;
+    var $modified_user_id;
+    var $assigned_user_id;
+    var $case_number;
+    var $resolution;
+    var $description;
+    var $name;
+    var $status;
+    var $priority;
+
+
+    var $created_by;
+    var $created_by_name;
+    var $modified_by_name;
+
+    // These are related
+    var $bug_id;
+    var $account_name;
+    var $account_id;
+    var $contact_id;
+    var $task_id;
+    var $note_id;
+    var $meeting_id;
+    var $call_id;
+    var $email_id;
+    var $assigned_user_name;
+    var $account_name1;
+
+    var $table_name = "cases";
+    var $rel_account_table = "accounts_cases";
+    var $rel_contact_table = "contacts_cases";
+    var $module_dir = 'Cases';
+    var $object_name = "Case";
+    var $importable = true;
+    /** "%1" is the case_number, for emails
+     * leave the %1 in if you customize this
+     * YOU MUST LEAVE THE BRACKETS AS WELL*/
+    var $emailSubjectMacro = '[CASE:%1]';
+
+    // This is used to retrieve related fields from form posts.
+    var $additional_column_fields = Array('bug_id', 'assigned_user_name', 'assigned_user_id', 'contact_id', 'task_id', 'note_id', 'meeting_id', 'call_id', 'email_id');
+
+    var $relationship_fields = Array('account_id'=>'accounts', 'bug_id' => 'bugs',
+                                    'task_id'=>'tasks', 'note_id'=>'notes',
+                                    'meeting_id'=>'meetings', 'call_id'=>'calls', 'email_id'=>'emails',
+                                    );
 
     public function __construct() {
-		parent::__construct();
-		global $sugar_config;
-		if(!$sugar_config['require_accounts']){
-			unset($this->required_fields['account_name']);
-		}
+        parent::__construct();
+        global $sugar_config;
+        if(!$sugar_config['require_accounts']){
+            unset($this->required_fields['account_name']);
+        }
 
-		$this->setupCustomFields('Cases');
+        $this->setupCustomFields('Cases');
         foreach ($this->field_defs as $name => $field) {
             $this->field_name_map[$name] = $field;
         }
-	}
-
-	/**
-	 * @deprecated deprecated since version 7.6, PHP4 Style Constructors are deprecated and will be remove in 7.8, please update your code, use __construct instead
-	 */
-	function aCase(){
-		$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 {
-			trigger_error($deprecatedMessage, E_USER_DEPRECATED);
-		}
-		self::__construct();
-	}
-
-	var $new_schema = true;
-
-
-
-
-
-	function get_summary_text()
-	{
-		return "$this->name";
-	}
-
-	function listviewACLHelper(){
-		$array_assign = parent::listviewACLHelper();
-		$is_owner = false;
-		$in_group = false; //SECURITY GROUPS
-		if(!empty($this->account_id)){
-
-			if(!empty($this->account_id_owner)){
-				global $current_user;
-				$is_owner = $current_user->id == $this->account_id_owner;
-			}
-			/* BEGIN - SECURITY GROUPS */
-			else {
-				global $current_user;
+    }
+
+    /**
+     * @deprecated deprecated since version 7.6, PHP4 Style Constructors are deprecated and will be remove in 7.8, please update your code, use __construct instead
+     */
+    function aCase(){
+        $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 {
+            trigger_error($deprecatedMessage, E_USER_DEPRECATED);
+        }
+        self::__construct();
+    }
+
+    var $new_schema = true;
+
+
+
+
+
+    function get_summary_text()
+    {
+        return "$this->name";
+    }
+
+    function listviewACLHelper(){
+        $array_assign = parent::listviewACLHelper();
+        $is_owner = false;
+        $in_group = false; //SECURITY GROUPS
+        if(!empty($this->account_id)){
+
+            if(!empty($this->account_id_owner)){
+                global $current_user;
+                $is_owner = $current_user->id == $this->account_id_owner;
+            }
+            /* BEGIN - SECURITY GROUPS */
+            else {
+                global $current_user;
                 $parent_bean = BeanFactory::getBean('Accounts',$this->account_id);
                 if($parent_bean !== false) {
-                	$is_owner = $current_user->id == $parent_bean->assigned_user_id;
+                    $is_owner = $current_user->id == $parent_bean->assigned_user_id;
                 }
-			}
-			require_once("modules/SecurityGroups/SecurityGroup.php");
-			$in_group = SecurityGroup::groupHasAccess('Accounts', $this->account_id, 'view');
-        	/* END - SECURITY GROUPS */
-		}
-			/* BEGIN - SECURITY GROUPS */
-			/**
+            }
+            require_once("modules/SecurityGroups/SecurityGroup.php");
+            $in_group = SecurityGroup::groupHasAccess('Accounts', $this->account_id, 'view');
+            /* END - SECURITY GROUPS */
+        }
+            /* BEGIN - SECURITY GROUPS */
+            /**
 			if(!ACLController::moduleSupportsACL('Accounts') || ACLController::checkAccess('Accounts', 'view', $is_owner)){
-			*/
-			if(!ACLController::moduleSupportsACL('Accounts') || ACLController::checkAccess('Accounts', 'view', $is_owner, 'module', $in_group)){
-        	/* END - SECURITY GROUPS */
-				$array_assign['ACCOUNT'] = 'a';
-			}else{
-				$array_assign['ACCOUNT'] = 'span';
-			}
-
-		return $array_assign;
-	}
-
-	function save_relationship_changes($is_update, $exclude = array())
-	{
-		parent::save_relationship_changes($is_update, $exclude);
-
-		if (!empty($this->contact_id)) {
-			$this->set_case_contact_relationship($this->contact_id);
-		}
-	}
-
-	function set_case_contact_relationship($contact_id)
-	{
-		global $app_list_strings;
-		$default = $app_list_strings['case_relationship_type_default_key'];
-		$this->load_relationship('contacts');
-		$this->contacts->add($contact_id,array('contact_role'=>$default));
-	}
-
-	function fill_in_additional_list_fields()
-	{
-		parent::fill_in_additional_list_fields();
-		/*// Fill in the assigned_user_name
+             */
+            if(!ACLController::moduleSupportsACL('Accounts') || ACLController::checkAccess('Accounts', 'view', $is_owner, 'module', $in_group)){
+            /* END - SECURITY GROUPS */
+                $array_assign['ACCOUNT'] = 'a';
+            }else{
+                $array_assign['ACCOUNT'] = 'span';
+            }
+
+        return $array_assign;
+    }
+
+    function save_relationship_changes($is_update, $exclude = array())
+    {
+        parent::save_relationship_changes($is_update, $exclude);
+
+        if (!empty($this->contact_id)) {
+            $this->set_case_contact_relationship($this->contact_id);
+        }
+    }
+
+    function set_case_contact_relationship($contact_id)
+    {
+        global $app_list_strings;
+        $default = $app_list_strings['case_relationship_type_default_key'];
+        $this->load_relationship('contacts');
+        $this->contacts->add($contact_id,array('contact_role'=>$default));
+    }
+
+    function fill_in_additional_list_fields()
+    {
+        parent::fill_in_additional_list_fields();
+        /*// Fill in the assigned_user_name
 		//$this->assigned_user_name = get_assigned_user_name($this->assigned_user_id);
 
 		$account_info = $this->getAccount($this->id);
 		$this->account_name = $account_info['account_name'];
 		$this->account_id = $account_info['account_id'];*/
-	}
+    }
 
-	function fill_in_additional_detail_fields()
-	{
-		parent::fill_in_additional_detail_fields();
-		// Fill in the assigned_user_name
-		$this->assigned_user_name = get_assigned_user_name($this->assigned_user_id);
+    function fill_in_additional_detail_fields()
+    {
+        parent::fill_in_additional_detail_fields();
+        // Fill in the assigned_user_name
+        $this->assigned_user_name = get_assigned_user_name($this->assigned_user_id);
 
-		$this->created_by_name = get_assigned_user_name($this->created_by);
-		$this->modified_by_name = get_assigned_user_name($this->modified_user_id);
+        $this->created_by_name = get_assigned_user_name($this->created_by);
+        $this->modified_by_name = get_assigned_user_name($this->modified_user_id);
 
         if(!empty($this->id)) {
-		    $account_info = $this->getAccount($this->id);
+            $account_info = $this->getAccount($this->id);
             if(!empty($account_info)) {
                 $this->account_name = $account_info['account_name'];
                 $this->account_id = $account_info['account_id'];
             }
         }
-	}
-
-
-	/** Returns a list of the associated contacts
-	 * Portions created by SugarCRM are Copyright (C) SugarCRM, Inc..
-	 * All Rights Reserved..
-	 * Contributor(s): ______________________________________..
-	*/
-	function get_contacts()
-	{
-		$this->load_relationship('contacts');
-		$query_array=$this->contacts->getQuery(true);
-
-		//update the select clause in the retruned query.
-		$query_array['select']="SELECT contacts.id, contacts.first_name, contacts.last_name, contacts.title, contacts.email1, contacts.phone_work, contacts_cases.contact_role as case_role, contacts_cases.id as case_rel_id ";
-
-		$query='';
-		foreach ($query_array as $qstring) {
-			$query.=' '.$qstring;
-		}
-	    $temp = Array('id', 'first_name', 'last_name', 'title', 'email1', 'phone_work', 'case_role', 'case_rel_id');
-		return $this->build_related_list2($query, new Contact(), $temp);
-	}
-
-	function get_list_view_data(){
-		global $current_language;
-		$app_list_strings = return_app_list_strings_language($current_language);
-
-		$temp_array = $this->get_list_view_array();
-		$temp_array['NAME'] = (($this->name == "") ? "<em>blank</em>" : $this->name);
+    }
+
+
+    /** Returns a list of the associated contacts
+     * Portions created by SugarCRM are Copyright (C) SugarCRM, Inc..
+     * All Rights Reserved..
+     * Contributor(s): ______________________________________..
+     */
+    function get_contacts()
+    {
+        $this->load_relationship('contacts');
+        $query_array=$this->contacts->getQuery(true);
+
+        //update the select clause in the retruned query.
+        $query_array['select']="SELECT contacts.id, contacts.first_name, contacts.last_name, contacts.title, contacts.email1, contacts.phone_work, contacts_cases.contact_role as case_role, contacts_cases.id as case_rel_id ";
+
+        $query='';
+        foreach ($query_array as $qstring) {
+            $query.=' '.$qstring;
+        }
+        $temp = Array('id', 'first_name', 'last_name', 'title', 'email1', 'phone_work', 'case_role', 'case_rel_id');
+        return $this->build_related_list2($query, new Contact(), $temp);
+    }
+
+    function get_list_view_data(){
+        global $current_language;
+        $app_list_strings = return_app_list_strings_language($current_language);
+
+        $temp_array = $this->get_list_view_array();
+        $temp_array['NAME'] = (($this->name == "") ? "<em>blank</em>" : $this->name);
         $temp_array['PRIORITY'] = empty($this->priority)? "" : (!isset($app_list_strings[$this->field_name_map['priority']['options']][$this->priority]) ? $this->priority : $app_list_strings[$this->field_name_map['priority']['options']][$this->priority]);
         $temp_array['STATUS'] = empty($this->status)? "" : (!isset($app_list_strings[$this->field_name_map['status']['options']][$this->status]) ? $this->status : $app_list_strings[$this->field_name_map['status']['options']][$this->status]);
-		$temp_array['ENCODED_NAME'] = $this->name;
-		$temp_array['CASE_NUMBER'] = $this->case_number;
-		$temp_array['SET_COMPLETE'] =  "<a href='index.php?return_module=Home&return_action=index&action=EditView&module=Cases&record=$this->id&status=Closed'>".SugarThemeRegistry::current()->getImage("close_inline","title=".translate('LBL_LIST_CLOSE','Cases')." border='0'",null,null,'.gif',translate('LBL_LIST_CLOSE','Cases'))."</a>";
-		//$temp_array['ACCOUNT_NAME'] = $this->account_name; //overwrites the account_name value returned from the cases table.
-		return $temp_array;
-	}
-
-	/**
+        $temp_array['ENCODED_NAME'] = $this->name;
+        $temp_array['CASE_NUMBER'] = $this->case_number;
+        $temp_array['SET_COMPLETE'] =  "<a href='index.php?return_module=Home&return_action=index&action=EditView&module=Cases&record=$this->id&status=Closed'>".SugarThemeRegistry::current()->getImage("close_inline","title=".translate('LBL_LIST_CLOSE','Cases')." border='0'",null,null,'.gif',translate('LBL_LIST_CLOSE','Cases'))."</a>";
+        //$temp_array['ACCOUNT_NAME'] = $this->account_name; //overwrites the account_name value returned from the cases table.
+        return $temp_array;
+    }
+
+    /**
 		builds a generic search based on the query string using or
 		do not include any $this-> because this is called on without having the class instantiated
-	*/
-	function build_generic_where_clause ($the_query_string) {
-	$where_clauses = Array();
-	$the_query_string = $this->db->quote($the_query_string);
-	array_push($where_clauses, "cases.name like '$the_query_string%'");
-	array_push($where_clauses, "accounts.name like '$the_query_string%'");
-
-	if (is_numeric($the_query_string)) array_push($where_clauses, "cases.case_number like '$the_query_string%'");
-
-	$the_where = "";
-
-	foreach($where_clauses as $clause)
-	{
-		if($the_where != "") $the_where .= " or ";
-		$the_where .= $clause;
-	}
-
-	if($the_where != ""){
-		$the_where = "(".$the_where.")";
-	}
-
-	return $the_where;
-	}
-
-	function set_notification_body($xtpl, $case)
-	{
-		global $app_list_strings;
-
-		$xtpl->assign("CASE_SUBJECT", $case->name);
-		$xtpl->assign("CASE_PRIORITY", (isset($case->priority) ? $app_list_strings['case_priority_dom'][$case->priority]:""));
-		$xtpl->assign("CASE_STATUS", (isset($case->status) ? $app_list_strings['case_status_dom'][$case->status]:""));
-		$xtpl->assign("CASE_DESCRIPTION", $case->description);
-
-		return $xtpl;
-	}
-
-		function bean_implements($interface){
-		switch($interface){
-			case 'ACL':return true;
-		}
-		return false;
-	}
-
-	function save($check_notify = FALSE){
-		return parent::save($check_notify);
-	}
-
-	/**
-	 * retrieves the Subject line macro for InboundEmail parsing
-	 * @return string
-	 */
-	function getEmailSubjectMacro() {
-		global $sugar_config;
-		return (isset($sugar_config['inbound_email_case_subject_macro']) && !empty($sugar_config['inbound_email_case_subject_macro'])) ?
-			$sugar_config['inbound_email_case_subject_macro'] : $this->emailSubjectMacro;
-	}
-
-	function getAccount($case_id){
-		if(empty($case_id)) return array();
-	    $ret_array = array();
-		$query = "SELECT acc.id, acc.name from accounts  acc, cases  where acc.id = cases.account_id and cases.id = '" . $case_id . "' and cases.deleted=0 and acc.deleted=0";
-		$result = $this->db->query($query,true," Error filling in additional detail fields: ");
-
-		// Get the id and the name.
-		$row = $this->db->fetchByAssoc($result);
-
-		if($row != null){
-			$ret_array['account_name'] = stripslashes($row['name']);
-			$ret_array['account_id'] 	= $row['id'];
-		}
-		else{
-			$ret_array['account_name'] = '';
-			$ret_array['account_id'] 	= '';
-		}
-		return $ret_array;
-	}
+     */
+    function build_generic_where_clause ($the_query_string) {
+    $where_clauses = Array();
+    $the_query_string = $this->db->quote($the_query_string);
+    array_push($where_clauses, "cases.name like '$the_query_string%'");
+    array_push($where_clauses, "accounts.name like '$the_query_string%'");
+
+    if (is_numeric($the_query_string)) array_push($where_clauses, "cases.case_number like '$the_query_string%'");
+
+    $the_where = "";
+
+    foreach($where_clauses as $clause)
+    {
+        if($the_where != "") $the_where .= " or ";
+        $the_where .= $clause;
+    }
+
+    if($the_where != ""){
+        $the_where = "(".$the_where.")";
+    }
+
+    return $the_where;
+    }
+
+    function set_notification_body($xtpl, $case)
+    {
+        global $app_list_strings;
+
+        $xtpl->assign("CASE_SUBJECT", $case->name);
+        $xtpl->assign("CASE_PRIORITY", (isset($case->priority) ? $app_list_strings['case_priority_dom'][$case->priority]:""));
+        $xtpl->assign("CASE_STATUS", (isset($case->status) ? $app_list_strings['case_status_dom'][$case->status]:""));
+        $xtpl->assign("CASE_DESCRIPTION", $case->description);
+
+        return $xtpl;
+    }
+
+        function bean_implements($interface){
+        switch($interface){
+            case 'ACL':return true;
+        }
+        return false;
+    }
+
+    function save($check_notify = FALSE){
+        return parent::save($check_notify);
+    }
+
+    /**
+     * retrieves the Subject line macro for InboundEmail parsing
+     * @return string
+     */
+    function getEmailSubjectMacro() {
+        global $sugar_config;
+        return (isset($sugar_config['inbound_email_case_subject_macro']) && !empty($sugar_config['inbound_email_case_subject_macro'])) ?
+            $sugar_config['inbound_email_case_subject_macro'] : $this->emailSubjectMacro;
+    }
+
+    function getAccount($case_id){
+        if(empty($case_id)) return array();
+        $ret_array = array();
+        $query = "SELECT acc.id, acc.name from accounts  acc, cases  where acc.id = cases.account_id and cases.id = '" . $case_id . "' and cases.deleted=0 and acc.deleted=0";
+        $result = $this->db->query($query,true," Error filling in additional detail fields: ");
+
+        // Get the id and the name.
+        $row = $this->db->fetchByAssoc($result);
+
+        if($row != null){
+            $ret_array['account_name'] = stripslashes($row['name']);
+            $ret_array['account_id'] 	= $row['id'];
+        }
+        else{
+            $ret_array['account_name'] = '';
+            $ret_array['account_id'] 	= '';
+        }
+        return $ret_array;
+    }
 }
 ?>

Spacing +39 added lines, -39 removed lines

@@ -1,5 +1,5 @@ 
 <?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.
@@ -114,7 +114,7 @@ 
     public function __construct() {
 		parent::__construct();
 		global $sugar_config;
-		if(!$sugar_config['require_accounts']){
+		if (!$sugar_config['require_accounts']) {
 			unset($this->required_fields['account_name']);
 		}
 
@@ -127,9 +127,9 @@ 
 	/**
 	 * @deprecated deprecated since version 7.6, PHP4 Style Constructors are deprecated and will be remove in 7.8, please update your code, use __construct instead
 	 */
-	function aCase(){
+	function aCase() {
 		$deprecatedMessage = 'PHP4 Style Constructors are deprecated and will be remove in 7.8, please update your code';
-		if(isset($GLOBALS['log'])) {
+		if (isset($GLOBALS['log'])) {
 			$GLOBALS['log']->deprecated($deprecatedMessage);
 		}
 		else {
@@ -149,21 +149,21 @@ 
 		return "$this->name";
 	}
 
-	function listviewACLHelper(){
+	function listviewACLHelper() {
 		$array_assign = parent::listviewACLHelper();
 		$is_owner = false;
 		$in_group = false; //SECURITY GROUPS
-		if(!empty($this->account_id)){
+		if (!empty($this->account_id)) {
 
-			if(!empty($this->account_id_owner)){
+			if (!empty($this->account_id_owner)) {
 				global $current_user;
 				$is_owner = $current_user->id == $this->account_id_owner;
 			}
 			/* BEGIN - SECURITY GROUPS */
 			else {
 				global $current_user;
-                $parent_bean = BeanFactory::getBean('Accounts',$this->account_id);
-                if($parent_bean !== false) {
+                $parent_bean = BeanFactory::getBean('Accounts', $this->account_id);
+                if ($parent_bean !== false) {
                 	$is_owner = $current_user->id == $parent_bean->assigned_user_id;
                 }
 			}
@@ -175,10 +175,10 @@ 
 			/**
 			if(!ACLController::moduleSupportsACL('Accounts') || ACLController::checkAccess('Accounts', 'view', $is_owner)){
 			*/
-			if(!ACLController::moduleSupportsACL('Accounts') || ACLController::checkAccess('Accounts', 'view', $is_owner, 'module', $in_group)){
+			if (!ACLController::moduleSupportsACL('Accounts') || ACLController::checkAccess('Accounts', 'view', $is_owner, 'module', $in_group)) {
         	/* END - SECURITY GROUPS */
 				$array_assign['ACCOUNT'] = 'a';
-			}else{
+			} else {
 				$array_assign['ACCOUNT'] = 'span';
 			}
 
@@ -199,7 +199,7 @@ 
 		global $app_list_strings;
 		$default = $app_list_strings['case_relationship_type_default_key'];
 		$this->load_relationship('contacts');
-		$this->contacts->add($contact_id,array('contact_role'=>$default));
+		$this->contacts->add($contact_id, array('contact_role'=>$default));
 	}
 
 	function fill_in_additional_list_fields()
@@ -222,9 +222,9 @@ 
 		$this->created_by_name = get_assigned_user_name($this->created_by);
 		$this->modified_by_name = get_assigned_user_name($this->modified_user_id);
 
-        if(!empty($this->id)) {
+        if (!empty($this->id)) {
 		    $account_info = $this->getAccount($this->id);
-            if(!empty($account_info)) {
+            if (!empty($account_info)) {
                 $this->account_name = $account_info['account_name'];
                 $this->account_id = $account_info['account_id'];
             }
@@ -240,30 +240,30 @@ 
 	function get_contacts()
 	{
 		$this->load_relationship('contacts');
-		$query_array=$this->contacts->getQuery(true);
+		$query_array = $this->contacts->getQuery(true);
 
 		//update the select clause in the retruned query.
-		$query_array['select']="SELECT contacts.id, contacts.first_name, contacts.last_name, contacts.title, contacts.email1, contacts.phone_work, contacts_cases.contact_role as case_role, contacts_cases.id as case_rel_id ";
+		$query_array['select'] = "SELECT contacts.id, contacts.first_name, contacts.last_name, contacts.title, contacts.email1, contacts.phone_work, contacts_cases.contact_role as case_role, contacts_cases.id as case_rel_id ";
 
-		$query='';
+		$query = '';
 		foreach ($query_array as $qstring) {
-			$query.=' '.$qstring;
+			$query .= ' '.$qstring;
 		}
 	    $temp = Array('id', 'first_name', 'last_name', 'title', 'email1', 'phone_work', 'case_role', 'case_rel_id');
 		return $this->build_related_list2($query, new Contact(), $temp);
 	}
 
-	function get_list_view_data(){
+	function get_list_view_data() {
 		global $current_language;
 		$app_list_strings = return_app_list_strings_language($current_language);
 
 		$temp_array = $this->get_list_view_array();
 		$temp_array['NAME'] = (($this->name == "") ? "<em>blank</em>" : $this->name);
-        $temp_array['PRIORITY'] = empty($this->priority)? "" : (!isset($app_list_strings[$this->field_name_map['priority']['options']][$this->priority]) ? $this->priority : $app_list_strings[$this->field_name_map['priority']['options']][$this->priority]);
-        $temp_array['STATUS'] = empty($this->status)? "" : (!isset($app_list_strings[$this->field_name_map['status']['options']][$this->status]) ? $this->status : $app_list_strings[$this->field_name_map['status']['options']][$this->status]);
+        $temp_array['PRIORITY'] = empty($this->priority) ? "" : (!isset($app_list_strings[$this->field_name_map['priority']['options']][$this->priority]) ? $this->priority : $app_list_strings[$this->field_name_map['priority']['options']][$this->priority]);
+        $temp_array['STATUS'] = empty($this->status) ? "" : (!isset($app_list_strings[$this->field_name_map['status']['options']][$this->status]) ? $this->status : $app_list_strings[$this->field_name_map['status']['options']][$this->status]);
 		$temp_array['ENCODED_NAME'] = $this->name;
 		$temp_array['CASE_NUMBER'] = $this->case_number;
-		$temp_array['SET_COMPLETE'] =  "<a href='index.php?return_module=Home&return_action=index&action=EditView&module=Cases&record=$this->id&status=Closed'>".SugarThemeRegistry::current()->getImage("close_inline","title=".translate('LBL_LIST_CLOSE','Cases')." border='0'",null,null,'.gif',translate('LBL_LIST_CLOSE','Cases'))."</a>";
+		$temp_array['SET_COMPLETE'] = "<a href='index.php?return_module=Home&return_action=index&action=EditView&module=Cases&record=$this->id&status=Closed'>".SugarThemeRegistry::current()->getImage("close_inline", "title=".translate('LBL_LIST_CLOSE', 'Cases')." border='0'", null, null, '.gif', translate('LBL_LIST_CLOSE', 'Cases'))."</a>";
 		//$temp_array['ACCOUNT_NAME'] = $this->account_name; //overwrites the account_name value returned from the cases table.
 		return $temp_array;
 	}
@@ -272,7 +272,7 @@ 
 		builds a generic search based on the query string using or
 		do not include any $this-> because this is called on without having the class instantiated
 	*/
-	function build_generic_where_clause ($the_query_string) {
+	function build_generic_where_clause($the_query_string) {
 	$where_clauses = Array();
 	$the_query_string = $this->db->quote($the_query_string);
 	array_push($where_clauses, "cases.name like '$the_query_string%'");
@@ -282,13 +282,13 @@ 
 
 	$the_where = "";
 
-	foreach($where_clauses as $clause)
+	foreach ($where_clauses as $clause)
 	{
-		if($the_where != "") $the_where .= " or ";
+		if ($the_where != "") $the_where .= " or ";
 		$the_where .= $clause;
 	}
 
-	if($the_where != ""){
+	if ($the_where != "") {
 		$the_where = "(".$the_where.")";
 	}
 
@@ -300,21 +300,21 @@ 
 		global $app_list_strings;
 
 		$xtpl->assign("CASE_SUBJECT", $case->name);
-		$xtpl->assign("CASE_PRIORITY", (isset($case->priority) ? $app_list_strings['case_priority_dom'][$case->priority]:""));
-		$xtpl->assign("CASE_STATUS", (isset($case->status) ? $app_list_strings['case_status_dom'][$case->status]:""));
+		$xtpl->assign("CASE_PRIORITY", (isset($case->priority) ? $app_list_strings['case_priority_dom'][$case->priority] : ""));
+		$xtpl->assign("CASE_STATUS", (isset($case->status) ? $app_list_strings['case_status_dom'][$case->status] : ""));
 		$xtpl->assign("CASE_DESCRIPTION", $case->description);
 
 		return $xtpl;
 	}
 
-		function bean_implements($interface){
-		switch($interface){
+		function bean_implements($interface) {
+		switch ($interface) {
 			case 'ACL':return true;
 		}
 		return false;
 	}
 
-	function save($check_notify = FALSE){
+	function save($check_notify = FALSE) {
 		return parent::save($check_notify);
 	}
 
@@ -328,22 +328,22 @@ 
 			$sugar_config['inbound_email_case_subject_macro'] : $this->emailSubjectMacro;
 	}
 
-	function getAccount($case_id){
-		if(empty($case_id)) return array();
+	function getAccount($case_id) {
+		if (empty($case_id)) return array();
 	    $ret_array = array();
-		$query = "SELECT acc.id, acc.name from accounts  acc, cases  where acc.id = cases.account_id and cases.id = '" . $case_id . "' and cases.deleted=0 and acc.deleted=0";
-		$result = $this->db->query($query,true," Error filling in additional detail fields: ");
+		$query = "SELECT acc.id, acc.name from accounts  acc, cases  where acc.id = cases.account_id and cases.id = '".$case_id."' and cases.deleted=0 and acc.deleted=0";
+		$result = $this->db->query($query, true, " Error filling in additional detail fields: ");
 
 		// Get the id and the name.
 		$row = $this->db->fetchByAssoc($result);
 
-		if($row != null){
+		if ($row != null) {
 			$ret_array['account_name'] = stripslashes($row['name']);
-			$ret_array['account_id'] 	= $row['id'];
+			$ret_array['account_id'] = $row['id'];
 		}
-		else{
+		else {
 			$ret_array['account_name'] = '';
-			$ret_array['account_id'] 	= '';
+			$ret_array['account_id'] = '';
 		}
 		return $ret_array;
 	}

Braces +15 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.
@@ -131,8 +133,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 +179,7 @@ 
 			if(!ACLController::moduleSupportsACL('Accounts') || ACLController::checkAccess('Accounts', 'view', $is_owner, 'module', $in_group)){
         	/* END - SECURITY GROUPS */
 				$array_assign['ACCOUNT'] = 'a';
-			}else{
+			} else{
 				$array_assign['ACCOUNT'] = 'span';
 			}
 
@@ -278,13 +279,17 @@ 
 	array_push($where_clauses, "cases.name like '$the_query_string%'");
 	array_push($where_clauses, "accounts.name like '$the_query_string%'");
 
-	if (is_numeric($the_query_string)) array_push($where_clauses, "cases.case_number like '$the_query_string%'");
+	if (is_numeric($the_query_string)) {
+	    array_push($where_clauses, "cases.case_number like '$the_query_string%'");
+	}
 
 	$the_where = "";
 
 	foreach($where_clauses as $clause)
 	{
-		if($the_where != "") $the_where .= " or ";
+		if($the_where != "") {
+		    $the_where .= " or ";
+		}
 		$the_where .= $clause;
 	}
 
@@ -329,7 +334,9 @@ 
 	}
 
 	function getAccount($case_id){
-		if(empty($case_id)) return array();
+		if(empty($case_id)) {
+		    return array();
+		}
 	    $ret_array = array();
 		$query = "SELECT acc.id, acc.name from accounts  acc, cases  where acc.id = cases.account_id and cases.id = '" . $case_id . "' and cases.deleted=0 and acc.deleted=0";
 		$result = $this->db->query($query,true," Error filling in additional detail fields: ");
@@ -340,8 +347,7 @@ 
 		if($row != null){
 			$ret_array['account_name'] = stripslashes($row['name']);
 			$ret_array['account_id'] 	= $row['id'];
-		}
-		else{
+		} else{
 			$ret_array['account_name'] = '';
 			$ret_array['account_id'] 	= '';
 		}

modules/Cases/views/view.edit.php

Spacing +5 added lines, -5 removed lines

@@ -41,16 +41,16 @@ 
 
 class CasesViewEdit extends ViewEdit {
 
-    function __construct(){
+    function __construct() {
         parent::__construct();
     }
 
     /**
      * @deprecated deprecated since version 7.6, PHP4 Style Constructors are deprecated and will be remove in 7.8, please update your code, use __construct instead
      */
-    function CasesViewEdit(){
+    function CasesViewEdit() {
         $deprecatedMessage = 'PHP4 Style Constructors are deprecated and will be remove in 7.8, please update your code';
-        if(isset($GLOBALS['log'])) {
+        if (isset($GLOBALS['log'])) {
             $GLOBALS['log']->deprecated($deprecatedMessage);
         }
         else {
@@ -60,11 +60,11 @@ 
     }
 
 
-    function display(){
+    function display() {
         parent::display();
         global $sugar_config;
         $new = empty($this->bean->id);
-        if($new){
+        if ($new) {
             ?>
             <script>
                 $(document).ready(function(){

Braces +1 added lines, -2 removed lines

@@ -23,8 +23,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($seed, $module, $subPanel, $options);

modules/Cases/views/view.list.php

Spacing +4 added lines, -4 removed lines

@@ -5,16 +5,16 @@ 
 
 class CasesViewList extends ViewList {
 
-	function __construct(){
+	function __construct() {
 		parent::__construct();
 	}
 
     /**
      * @deprecated deprecated since version 7.6, PHP4 Style Constructors are deprecated and will be remove in 7.8, please update your code, use __construct instead
      */
-    function CasesViewList(){
+    function CasesViewList() {
         $deprecatedMessage = 'PHP4 Style Constructors are deprecated and will be remove in 7.8, please update your code';
-        if(isset($GLOBALS['log'])) {
+        if (isset($GLOBALS['log'])) {
             $GLOBALS['log']->deprecated($deprecatedMessage);
         }
         else {
@@ -24,7 +24,7 @@ 
     }
 
 
-	function preDisplay(){
+	function preDisplay() {
 		$this->lv = new CasesListViewSmarty();
 	}
 }

Braces +1 added lines, -2 removed lines

@@ -23,8 +23,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($seed, $module, $subPanel, $options);

modules/Cases/Dashlets/MyCasesDashlet/MyCasesDashlet.php

Spacing +4 added lines, -4 removed lines

@@ -1,5 +1,5 @@ 
 <?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.
@@ -51,7 +51,7 @@ 
 
         parent::__construct($id, $def);
 
-        if(empty($def['title'])) $this->title = translate('LBL_LIST_MY_CASES', 'Cases');
+        if (empty($def['title'])) $this->title = translate('LBL_LIST_MY_CASES', 'Cases');
         $this->searchFields = $dashletData['MyCasesDashlet']['searchFields'];
         $this->columns = $dashletData['MyCasesDashlet']['columns'];
         $this->seedBean = new aCase();
@@ -60,9 +60,9 @@ 
     /**
      * @deprecated deprecated since version 7.6, PHP4 Style Constructors are deprecated and will be remove in 7.8, please update your code, use __construct instead
      */
-    function MyCasesDashlet($id, $def = null){
+    function MyCasesDashlet($id, $def = null) {
         $deprecatedMessage = 'PHP4 Style Constructors are deprecated and will be remove in 7.8, please update your code';
-        if(isset($GLOBALS['log'])) {
+        if (isset($GLOBALS['log'])) {
             $GLOBALS['log']->deprecated($deprecatedMessage);
         }
         else {

Braces +7 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
  * SugarCRM, Inc. Copyright (C) 2004-2013 SugarCRM Inc.
@@ -51,7 +53,9 @@ 
 
         parent::__construct($id, $def);
 
-        if(empty($def['title'])) $this->title = translate('LBL_LIST_MY_CASES', 'Cases');
+        if(empty($def['title'])) {
+            $this->title = translate('LBL_LIST_MY_CASES', 'Cases');
+        }
         $this->searchFields = $dashletData['MyCasesDashlet']['searchFields'];
         $this->columns = $dashletData['MyCasesDashlet']['columns'];
         $this->seedBean = new aCase();
@@ -64,8 +68,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);

modules/Cases/CasesListViewSmarty.php

Spacing +9 added lines, -9 removed lines

@@ -12,9 +12,9 @@ 
     /**
      * @deprecated deprecated since version 7.6, PHP4 Style Constructors are deprecated and will be remove in 7.8, please update your code, use __construct instead
      */
-    function CasesListViewSmarty(){
+    function CasesListViewSmarty() {
         $deprecatedMessage = 'PHP4 Style Constructors are deprecated and will be remove in 7.8, please update your code';
-        if(isset($GLOBALS['log'])) {
+        if (isset($GLOBALS['log'])) {
             $GLOBALS['log']->deprecated($deprecatedMessage);
         }
         else {
@@ -29,13 +29,13 @@ 
         global $app_strings;
 
         $script = "<a href='javascript:void(0)' id='export_listview_top' ".
-                "onclick=\"return sListView.send_form(true, '{$_REQUEST['module']}', " .
-                "'index.php?entryPoint=export', " .
-                "'{$app_strings['LBL_LISTVIEW_NO_SELECTED']}')\">{$app_strings['LBL_EXPORT']}</a>" .
-                "</li><li>". // List item hack
-                "<a href='javascript:void(0)' id='map_listview_top' " .
-                " onclick=\"return sListView.send_form(true, 'jjwg_Maps', " .
-                "'index.php?entryPoint=jjwg_Maps&display_module={$_REQUEST['module']}', " .
+                "onclick=\"return sListView.send_form(true, '{$_REQUEST['module']}', ".
+                "'index.php?entryPoint=export', ".
+                "'{$app_strings['LBL_LISTVIEW_NO_SELECTED']}')\">{$app_strings['LBL_EXPORT']}</a>".
+                "</li><li>".// List item hack
+                "<a href='javascript:void(0)' id='map_listview_top' ".
+                " onclick=\"return sListView.send_form(true, 'jjwg_Maps', ".
+                "'index.php?entryPoint=jjwg_Maps&display_module={$_REQUEST['module']}', ".
                 "'{$app_strings['LBL_LISTVIEW_NO_SELECTED']}')\">{$app_strings['LBL_MAP']}</a>";
 
         return $script;

Braces +1 added lines, -2 removed lines

@@ -23,8 +23,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($seed, $module, $subPanel, $options);

modules/Prospects/views/view.detail.php

Indentation +12 added lines, -12 removed lines

@@ -44,9 +44,9 @@ 
 
 class ProspectsViewDetail extends ViewDetail {
 
- 	function __construct(){
- 		parent::__construct();
- 	}
+        function __construct(){
+            parent::__construct();
+        }
 
     /**
      * @deprecated deprecated since version 7.6, PHP4 Style Constructors are deprecated and will be remove in 7.8, please update your code, use __construct instead
@@ -63,15 +63,15 @@ 
     }
 
 
- 	function display() {
-		if(isset($this->bean->lead_id) && !empty($this->bean->lead_id)){
+        function display() {
+        if(isset($this->bean->lead_id) && !empty($this->bean->lead_id)){
 
-			//get lead name
-			$lead = new Lead();
-			$lead->retrieve($this->bean->lead_id);
-			$this->ss->assign('lead', $lead);
-		}
- 		parent::display();
- 	}
+            //get lead name
+            $lead = new Lead();
+            $lead->retrieve($this->bean->lead_id);
+            $this->ss->assign('lead', $lead);
+        }
+            parent::display();
+        }
 }
 ?>

Spacing +5 added lines, -5 removed lines

@@ -1,5 +1,5 @@ 
 <?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
@@ -44,16 +44,16 @@ 
 
 class ProspectsViewDetail extends ViewDetail {
 
- 	function __construct(){
+ 	function __construct() {
  		parent::__construct();
  	}
 
     /**
      * @deprecated deprecated since version 7.6, PHP4 Style Constructors are deprecated and will be remove in 7.8, please update your code, use __construct instead
      */
-    function ProspectsViewDetail(){
+    function ProspectsViewDetail() {
         $deprecatedMessage = 'PHP4 Style Constructors are deprecated and will be remove in 7.8, please update your code';
-        if(isset($GLOBALS['log'])) {
+        if (isset($GLOBALS['log'])) {
             $GLOBALS['log']->deprecated($deprecatedMessage);
         }
         else {
@@ -64,7 +64,7 @@ 
 
 
  	function display() {
-		if(isset($this->bean->lead_id) && !empty($this->bean->lead_id)){
+		if (isset($this->bean->lead_id) && !empty($this->bean->lead_id)) {
 
 			//get lead name
 			$lead = new Lead();

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();

modules/Prospects/Prospect.php

Spacing +23 added lines, -23 removed lines

@@ -1,5 +1,5 @@ 
 <?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.
@@ -114,9 +114,9 @@ 
     /**
      * @deprecated deprecated since version 7.6, PHP4 Style Constructors are deprecated and will be remove in 7.8, please update your code, use __construct instead
      */
-    function Prospect(){
+    function Prospect() {
         $deprecatedMessage = 'PHP4 Style Constructors are deprecated and will be remove in 7.8, please update your code';
-        if(isset($GLOBALS['log'])) {
+        if (isset($GLOBALS['log'])) {
             $GLOBALS['log']->deprecated($deprecatedMessage);
         }
         else {
@@ -143,7 +143,7 @@ 
 		builds a generic search based on the query string using or
 		do not include any $this-> because this is called on without having the class instantiated
 	*/
-	function build_generic_where_clause ($the_query_string)
+	function build_generic_where_clause($the_query_string)
 	{
 		$where_clauses = Array();
 		$the_query_string = $GLOBALS['db']->quote($the_query_string);
@@ -163,9 +163,9 @@ 
 		}
 
 		$the_where = "";
-		foreach($where_clauses as $clause)
+		foreach ($where_clauses as $clause)
 		{
-			if($the_where != "") $the_where .= " or ";
+			if ($the_where != "") $the_where .= " or ";
 			$the_where .= $clause;
 		}
 
@@ -173,13 +173,13 @@ 
 		return $the_where;
 	}
 
-    function converted_prospect($prospectid, $contactid, $accountid, $opportunityid){
+    function converted_prospect($prospectid, $contactid, $accountid, $opportunityid) {
     	$query = "UPDATE prospects set  contact_id=$contactid, account_id=$accountid, opportunity_id=$opportunityid where  id=$prospectid and deleted=0";
-		$this->db->query($query,true,"Error converting prospect: ");
+		$this->db->query($query, true, "Error converting prospect: ");
 		//todo--status='Converted', converted='1',
     }
-     function bean_implements($interface){
-		switch($interface){
+     function bean_implements($interface) {
+		switch ($interface) {
 			case 'ACL':return true;
 		}
 		return false;
@@ -189,15 +189,15 @@ 
      *  This method will be used by Mail Merge in order to retieve the targets as specified in the query
      *  @param query String - this is the query which contains the where clause for the query
      */
-    function retrieveTargetList($query, $fields, $offset = 0, $limit= -99, $max = -99, $deleted = 0, $module = ''){
+    function retrieveTargetList($query, $fields, $offset = 0, $limit = -99, $max = -99, $deleted = 0, $module = '') {
         global  $beanList, $beanFiles;
         $module_name = $this->module_dir;
 
-        if(empty($module))
+        if (empty($module))
         {
             //The call to retrieveTargetList contains a query that may contain a pound token
             $pattern = '/AND related_type = [\'#]([a-zA-Z]+)[\'#]/i';
-            if(preg_match($pattern, $query, $matches))
+            if (preg_match($pattern, $query, $matches))
             {
                 $module_name = $matches[1];
                 $query = preg_replace($pattern, "", $query);
@@ -207,15 +207,15 @@ 
         $count = count($fields);
         $index = 1;
         $sel_fields = "";
-        if(!empty($fields))
+        if (!empty($fields))
         {
-            foreach($fields as $field){
-                if($field == 'id'){
+            foreach ($fields as $field) {
+                if ($field == 'id') {
                 	$sel_fields .= 'prospect_lists_prospects.id id';
-                }else{
+                } else {
                 	$sel_fields .= strtolower($module_name).".".$field;
                 }
-                if($index < $count){
+                if ($index < $count) {
                     $sel_fields .= ",";
                 }
                 $index++;
@@ -226,7 +226,7 @@ 
         $class_name = $beanList[$module_name];
         require_once($beanFiles[$class_name]);
         $seed = new $class_name();
-        if(empty($sel_fields)){
+        if (empty($sel_fields)) {
             $sel_fields = $seed->table_name.'.*';
         }
         $select = "SELECT ".$sel_fields." FROM ".$seed->table_name;
@@ -248,23 +248,23 @@ 
      *  Given an id, looks up in the prospect_lists_prospects table
      *  and retrieve the correct type for this id
      */
-    function retrieveTarget($id){
+    function retrieveTarget($id) {
         $query = "SELECT related_id, related_type FROM prospect_lists_prospects WHERE id = '".$this->db->quote($id)."'";
         $result = $this->db->query($query);
-        if(($row = $this->db->fetchByAssoc($result))){
+        if (($row = $this->db->fetchByAssoc($result))) {
              global  $beanList, $beanFiles;
              $module_name = $row['related_type'];
              $class_name = $beanList[$module_name];
              require_once($beanFiles[$class_name]);
              $seed = new $class_name();
              return $seed->retrieve($row['related_id']);
-        }else{
+        } else {
             return null;
         }
     }
 
 
-	function get_unlinked_email_query($type=array()) {
+	function get_unlinked_email_query($type = array()) {
 
 		return get_unlinked_email_query($type, $this);
 	}

Braces +9 added lines, -6 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.
@@ -118,8 +120,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();
@@ -165,7 +166,9 @@ 
 		$the_where = "";
 		foreach($where_clauses as $clause)
 		{
-			if($the_where != "") $the_where .= " or ";
+			if($the_where != "") {
+			    $the_where .= " or ";
+			}
 			$the_where .= $clause;
 		}
 
@@ -212,7 +215,7 @@ 
             foreach($fields as $field){
                 if($field == 'id'){
                 	$sel_fields .= 'prospect_lists_prospects.id id';
-                }else{
+                } else{
                 	$sel_fields .= strtolower($module_name).".".$field;
                 }
                 if($index < $count){
@@ -258,7 +261,7 @@ 
              require_once($beanFiles[$class_name]);
              $seed = new $class_name();
              return $seed->retrieve($row['related_id']);
-        }else{
+        } else{
             return null;
         }
     }

modules/CampaignTrackers/CampaignTracker.php

Spacing +13 added lines, -13 removed lines

@@ -100,7 +100,7 @@ 
     var $additional_column_fields = Array('campaign_id');
     var $relationship_fields = Array('campaing_id'=>'campaign');
 
-    var $required_fields =  array('tracker_name'=>1,'tracker_url'=>1);
+    var $required_fields = array('tracker_name'=>1, 'tracker_url'=>1);
 
     /*This bean's constructor*/
     public function __construct() {
@@ -110,9 +110,9 @@ 
     /**
      * @deprecated deprecated since version 7.6, PHP4 Style Constructors are deprecated and will be remove in 7.8, please update your code, use __construct instead
      */
-    public function CampaignTracker(){
+    public function CampaignTracker() {
         $deprecatedMessage = 'PHP4 Style Constructors are deprecated and will be remove in 7.8, please update your code';
-        if(isset($GLOBALS['log'])) {
+        if (isset($GLOBALS['log'])) {
             $GLOBALS['log']->deprecated($deprecatedMessage);
         }
         else {
@@ -124,9 +124,9 @@ 
 
     function save($check_notify = false) {
         //make sure that the url has a scheme, if not then add http:// scheme
-        if ($this->is_optout!=1 ){
+        if ($this->is_optout != 1) {
             $url = strtolower(trim($this->tracker_url));
-            if(!preg_match('/^(http|https|ftp):\/\//i', $url)){
+            if (!preg_match('/^(http|https|ftp):\/\//i', $url)) {
                 $this->tracker_url = 'http://'.$url;
             }
         }
@@ -152,28 +152,28 @@ 
 
         //setup campaign name.
         $query = "SELECT name from campaigns where id = '$this->campaign_id'";
-        $result =$this->db->query($query,true," Error filling in additional detail fields: ");
+        $result = $this->db->query($query, true, " Error filling in additional detail fields: ");
 
         // Get the id and the name.
         $row = $this->db->fetchByAssoc($result);
-        if($row != null) {
-            $this->campaign_name=$row['name'];
+        if ($row != null) {
+            $this->campaign_name = $row['name'];
         }
 
         if (!class_exists('Administration')) {
 
         }
-        $admin=new Administration();
+        $admin = new Administration();
         $admin->retrieveSettings('massemailer'); //retrieve all admin settings.
-        if (isset($admin->settings['massemailer_tracking_entities_location_type']) and $admin->settings['massemailer_tracking_entities_location_type']=='2'  and isset($admin->settings['massemailer_tracking_entities_location']) ) {
-            $this->message_url=$admin->settings['massemailer_tracking_entities_location'];
+        if (isset($admin->settings['massemailer_tracking_entities_location_type']) and $admin->settings['massemailer_tracking_entities_location_type'] == '2' and isset($admin->settings['massemailer_tracking_entities_location'])) {
+            $this->message_url = $admin->settings['massemailer_tracking_entities_location'];
         } else {
-            $this->message_url=$sugar_config['site_url'];
+            $this->message_url = $sugar_config['site_url'];
         }
         if ($this->is_optout == 1) {
             $this->message_url .= '/index.php?entryPoint=removeme&identifier={MESSAGE_ID}';
         } else {
-            $this->message_url .= '/index.php?entryPoint=campaign_trackerv2&track=' . $this->id;
+            $this->message_url .= '/index.php?entryPoint=campaign_trackerv2&track='.$this->id;
         }
     }
 }

Braces +1 added lines, -2 removed lines

@@ -23,8 +23,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($seed, $module, $subPanel, $options);

modules/Calls_Reschedule/Calls_Reschedule_sugar.php

Spacing +6 added lines, -6 removed lines

@@ -51,7 +51,7 @@ 
 	var $table_name = 'calls_reschedule';
 	var $importable = true;
 	var $tracker_visibility = false;
-	var $disable_row_level_security = true ; // to ensure that modules created and deployed under CE will continue to function under team security if the instance is upgraded to PRO
+	var $disable_row_level_security = true; // to ensure that modules created and deployed under CE will continue to function under team security if the instance is upgraded to PRO
 		var $id;
 		var $name;
 		var $date_entered;
@@ -71,16 +71,16 @@ 
 		var $call_id_c;
 		var $calls;
 
-    function __construct(){
+    function __construct() {
 		parent::__construct();
 	}
 
     /**
      * @deprecated deprecated since version 7.6, PHP4 Style Constructors are deprecated and will be remove in 7.8, please update your code, use __construct instead
      */
-    function Calls_Reschedule_sugar(){
+    function Calls_Reschedule_sugar() {
         $deprecatedMessage = 'PHP4 Style Constructors are deprecated and will be remove in 7.8, please update your code';
-        if(isset($GLOBALS['log'])) {
+        if (isset($GLOBALS['log'])) {
             $GLOBALS['log']->deprecated($deprecatedMessage);
         }
         else {
@@ -90,8 +90,8 @@ 
     }
 
 
-	function bean_implements($interface){
-		switch($interface){
+	function bean_implements($interface) {
+		switch ($interface) {
 			case 'ACL': return true;
 		}
 		return false;

Braces +1 added lines, -2 removed lines

@@ -23,8 +23,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($seed, $module, $subPanel, $options);