adamjakab / SuiteCRM

modules/EmailTemplates/templateFields.php

Spacing +8 added lines, -8 removed lines

@@ -46,8 +46,8 @@ 
             require_once($beanFiles[$beanList[$key]]);
             $focus = new $beanList[$key];
             $loopControl[$key][$key] = $focus;
-            $prefixes[$key] = strtolower($focus->object_name) . '_';
-            if ($focus->object_name == 'Case') $prefixes[$key] = 'a' . strtolower($focus->object_name) . '_';
+            $prefixes[$key] = strtolower($focus->object_name).'_';
+            if ($focus->object_name == 'Case') $prefixes[$key] = 'a'.strtolower($focus->object_name).'_';
         }
     }
 
@@ -125,16 +125,16 @@ 
         if (isset($beanList[$key]) && isset($beanFiles[$beanList[$key]]) && !str_begin($key, 'AOW_') && !str_begin($key, 'zr2_')) {
 
             if ($key == 'Contacts') {
-                $dropdown .= "<option value='" . $key . "'>
-						" . $lblContactAndOthers . "
+                $dropdown .= "<option value='".$key."'>
+						" . $lblContactAndOthers."
 		  	       </option>";
             } else if (isset($app_list_strings['moduleListSingular'][$key])) {
-                $dropdown .= "<option value='" . $key . "'>
-						" . $app_list_strings['moduleListSingular'][$key] . "
+                $dropdown .= "<option value='".$key."'>
+						" . $app_list_strings['moduleListSingular'][$key]."
 		  	       </option>";
             } else {
-                $dropdown .= "<option value='" . $key . "'>
-						" . $app_list_strings['moduleList'][$key] . "
+                $dropdown .= "<option value='".$key."'>
+						" . $app_list_strings['moduleList'][$key]."
 		  	       </option>";
             }
         }

modules/EmailTemplates/EmailTemplate.php

Spacing +32 added lines, -32 removed lines

@@ -352,7 +352,7 @@ 
                             }
                             if (!empty($tracker_url) && !empty($template_text) && !empty($matches[0][$i][0]) && !empty($tracked_urls[$matches[0][$i][0]])) {
                                 $template_text = substr_replace($template_text, $tracker_url, $matches[0][$i][1], strlen($matches[0][$i][0]));
-                                $template_text = str_replace($sugar_config['site_url'] . '/' . $sugar_config['site_url'], $sugar_config['site_url'], $template_text);
+                                $template_text = str_replace($sugar_config['site_url'].'/'.$sugar_config['site_url'], $sugar_config['site_url'], $template_text);
                             }
                         }
                     }
@@ -406,10 +406,10 @@ 
         if ($beanList[$focus_name] == 'User') {
             $pattern_prefix = '$contact_user_';
         } else {
-            $pattern_prefix = '$' . strtolower($beanList[$focus_name]) . '_';
+            $pattern_prefix = '$'.strtolower($beanList[$focus_name]).'_';
         }
         $pattern_prefix_length = strlen($pattern_prefix);
-        $pattern = '/\\' . $pattern_prefix . '[A-Za-z_0-9]*/';
+        $pattern = '/\\'.$pattern_prefix.'[A-Za-z_0-9]*/';
 
         foreach ($template_text_array as $key => $template_text) {
             if (!isset($this->parsed_entities[$key])) {
@@ -505,16 +505,16 @@ 
                 $translated = translate($field_def['options'], 'Users', $user->$field_def['name']);
 
                 if (isset($translated) && !is_array($translated)) {
-                    $repl_arr["contact_user_" . $field_def['name']] = $translated;
+                    $repl_arr["contact_user_".$field_def['name']] = $translated;
                 } else { // unset enum field, make sure we have a match string to replace with ""
-                    $repl_arr["contact_user_" . $field_def['name']] = '';
+                    $repl_arr["contact_user_".$field_def['name']] = '';
                 }
             } else {
                 if (isset($user->$field_def['name'])) {
                     // bug 47647 - allow for fields to translate before adding to template
-                    $repl_arr["contact_user_" . $field_def['name']] = self::_convertToType($field_def['type'], $user->$field_def['name']);
+                    $repl_arr["contact_user_".$field_def['name']] = self::_convertToType($field_def['type'], $user->$field_def['name']);
                 } else {
-                    $repl_arr["contact_user_" . $field_def['name']] = "";
+                    $repl_arr["contact_user_".$field_def['name']] = "";
                 }
             }
         }
@@ -540,8 +540,8 @@ 
                 continue;
             }
             $repl_arr = EmailTemplate::add_replacement($repl_arr, $field_def, array(
-                'contact_' . $field_def['name'] => '',
-                'contact_account_' . $field_def['name'] => '',
+                'contact_'.$field_def['name'] => '',
+                'contact_account_'.$field_def['name'] => '',
             ));
         }
         foreach ($prospect->field_defs as $field_def) {
@@ -549,8 +549,8 @@ 
                 continue;
             }
             $repl_arr = EmailTemplate::add_replacement($repl_arr, $field_def, array(
-                'contact_' . $field_def['name'] => '',
-                'contact_account_' . $field_def['name'] => '',
+                'contact_'.$field_def['name'] => '',
+                'contact_account_'.$field_def['name'] => '',
             ));
         }
         foreach ($contact->field_defs as $field_def) {
@@ -558,8 +558,8 @@ 
                 continue;
             }
             $repl_arr = EmailTemplate::add_replacement($repl_arr, $field_def, array(
-                'contact_' . $field_def['name'] => '',
-                'contact_account_' . $field_def['name'] => '',
+                'contact_'.$field_def['name'] => '',
+                'contact_account_'.$field_def['name'] => '',
             ));
         }
         foreach ($acct->field_defs as $field_def) {
@@ -567,8 +567,8 @@ 
                 continue;
             }
             $repl_arr = EmailTemplate::add_replacement($repl_arr, $field_def, array(
-                'account_' . $field_def['name'] => '',
-                'account_contact_' . $field_def['name'] => '',
+                'account_'.$field_def['name'] => '',
+                'account_contact_'.$field_def['name'] => '',
             ));
         }
         // cn: end bug 9277 fix
@@ -592,21 +592,21 @@ 
 
                         if (isset($translated) && !is_array($translated)) {
                             $repl_arr = EmailTemplate::add_replacement($repl_arr, $field_def, array(
-                                'account_' . $field_def['name'] => $translated,
-                                'contact_account_' . $field_def['name'] => $translated,
+                                'account_'.$field_def['name'] => $translated,
+                                'contact_account_'.$field_def['name'] => $translated,
                             ));
                         } else { // unset enum field, make sure we have a match string to replace with ""
                             $repl_arr = EmailTemplate::add_replacement($repl_arr, $field_def, array(
-                                'account_' . $field_def['name'] => '',
-                                'contact_account_' . $field_def['name'] => '',
+                                'account_'.$field_def['name'] => '',
+                                'contact_account_'.$field_def['name'] => '',
                             ));
                         }
                     } else {
                         // bug 47647 - allow for fields to translate before adding to template
                         $translated = self::_convertToType($field_def['type'], $acct->$field_def['name']);
                         $repl_arr = EmailTemplate::add_replacement($repl_arr, $field_def, array(
-                            'account_' . $field_def['name'] => $translated,
-                            'contact_account_' . $field_def['name'] => $translated,
+                            'account_'.$field_def['name'] => $translated,
+                            'contact_account_'.$field_def['name'] => $translated,
                         ));
                     }
                 }
@@ -636,13 +636,13 @@ 
 
                     if (isset($translated) && !is_array($translated)) {
                         $repl_arr = EmailTemplate::add_replacement($repl_arr, $field_def, array(
-                            'contact_' . $field_def['name'] => $translated,
-                            'contact_account_' . $field_def['name'] => $translated,
+                            'contact_'.$field_def['name'] => $translated,
+                            'contact_account_'.$field_def['name'] => $translated,
                         ));
                     } else { // unset enum field, make sure we have a match string to replace with ""
                         $repl_arr = EmailTemplate::add_replacement($repl_arr, $field_def, array(
-                            'contact_' . $field_def['name'] => '',
-                            'contact_account_' . $field_def['name'] => '',
+                            'contact_'.$field_def['name'] => '',
+                            'contact_account_'.$field_def['name'] => '',
                         ));
                     }
                 } else {
@@ -650,8 +650,8 @@ 
                         // bug 47647 - allow for fields to translate before adding to template
                         $translated = self::_convertToType($field_def['type'], $contact->$field_def['name']);
                         $repl_arr = EmailTemplate::add_replacement($repl_arr, $field_def, array(
-                            'contact_' . $field_def['name'] => $translated,
-                            'contact_account_' . $field_def['name'] => $translated,
+                            'contact_'.$field_def['name'] => $translated,
+                            'contact_account_'.$field_def['name'] => $translated,
                         ));
                     } // if
                 }
@@ -671,27 +671,27 @@ 
 
                     if (isset($translated) && !is_array($translated)) {
                         $repl_arr = EmailTemplate::add_replacement($repl_arr, $field_def, array(
-                            strtolower($beanList[$bean_name]) . "_" . $field_def['name'] => $translated,
+                            strtolower($beanList[$bean_name])."_".$field_def['name'] => $translated,
                         ));
                     } else { // unset enum field, make sure we have a match string to replace with ""
                         $repl_arr = EmailTemplate::add_replacement($repl_arr, $field_def, array(
-                            strtolower($beanList[$bean_name]) . "_" . $field_def['name'] => '',
+                            strtolower($beanList[$bean_name])."_".$field_def['name'] => '',
                         ));
                     }
                 } else {
                     // bug 47647 - translate currencies to appropriate values
                     $repl_arr = EmailTemplate::add_replacement($repl_arr, $field_def, array(
-                        strtolower($beanList[$bean_name]) . "_" . $field_def['name'] => self::_convertToType($field_def['type'], $focus->$field_def['name']),
+                        strtolower($beanList[$bean_name])."_".$field_def['name'] => self::_convertToType($field_def['type'], $focus->$field_def['name']),
                     ));
                 }
             } else {
                 if ($field_def['name'] == 'full_name') {
                     $repl_arr = EmailTemplate::add_replacement($repl_arr, $field_def, array(
-                        strtolower($beanList[$bean_name]) . '_full_name' => $focus->get_summary_text(),
+                        strtolower($beanList[$bean_name]).'_full_name' => $focus->get_summary_text(),
                     ));
                 } else {
                     $repl_arr = EmailTemplate::add_replacement($repl_arr, $field_def, array(
-                        strtolower($beanList[$bean_name]) . "_" . $field_def['name'] => '',
+                        strtolower($beanList[$bean_name])."_".$field_def['name'] => '',
                     ));
                 }
             }

modules/Calendar/processScreenSize.php

Spacing +1 added lines, -1 removed lines

@@ -7,7 +7,7 @@
  */
 
 
-if(
+if (
     (isset($_SESSION['screen_height']) ? $_SESSION['screen_height'] : null) != (isset($_POST['height']) ? $_POST['height'] : null) ||
     (isset($_SESSION['screen_width']) ? $_SESSION['screen_width'] : null) != (isset($_POST['width']) ? $_POST['width'] : null)) {
     $_SESSION['screen_height'] = $_POST['height'];

modules/FP_events/responseEntryPoint.php

Spacing +36 added lines, -36 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');
 
 	global $db;
 	 
@@ -12,25 +12,25 @@ 
     $event = new FP_events();
     $event->retrieve($even_id);
     
-    if($type == 'c'){
+    if ($type == 'c') {
     	
     	$event->load_relationship('fp_events_contacts'); // get related contacts
 
-    	if($response == 'accept'){
+    	if ($response == 'accept') {
 
             //check to see if they have already responded to the email
             $check_q = 'SELECT email_responded FROM fp_events_contacts_c WHERE fp_events_contactsfp_events_ida="'.$event->id.'" AND fp_events_contactscontacts_idb="'.$delegate_id.'"';
              $check = $db->getOne($check_q);
     		//update contact to accepted
     		$query = 'UPDATE fp_events_contacts_c SET accept_status="Accepted", email_responded="1" WHERE fp_events_contactsfp_events_ida="'.$event->id.'" AND fp_events_contactscontacts_idb="'.$delegate_id.'" AND email_responded="0"';
-    		if($db->query($query) && $check != '1'){
+    		if ($db->query($query) && $check != '1') {
     			
-                if(!IsNullOrEmptyString($event->accept_redirect)){
+                if (!IsNullOrEmptyString($event->accept_redirect)) {
                     
                     $url = $event->accept_redirect;
-                    header('Location: ' . $url);    
+                    header('Location: '.$url);    
                 }
-                else{
+                else {
                     echo 'Thank you for accepting';
                 }   
     		}
@@ -38,22 +38,22 @@ 
     			echo 'You have already responded to the invitation or there was a problem with the link. Please contact the sender of the invite for help.';
     		}	
     	}
-    	else if($response == 'decline'){
+    	else if ($response == 'decline') {
     		//check to see if they have already responded to the email
             $check_q = 'SELECT email_responded FROM fp_events_contacts_c WHERE fp_events_contactsfp_events_ida="'.$event->id.'" AND fp_events_contactscontacts_idb="'.$delegate_id.'"';
              $check = $db->getOne($check_q);
             //update contact to accepted
     		$query = 'UPDATE fp_events_contacts_c SET accept_status="Declined", email_responded="1" WHERE fp_events_contactsfp_events_ida="'.$event->id.'" AND fp_events_contactscontacts_idb="'.$delegate_id.'" AND email_responded="0"';
     		
-            if($db->query($query) && $check != '1'){
+            if ($db->query($query) && $check != '1') {
     			
-                if(!IsNullOrEmptyString($event->decline_redirect)){
+                if (!IsNullOrEmptyString($event->decline_redirect)) {
 
                     $url = $event->decline_redirect;
-                    header('Location: ' . $url);    
+                    header('Location: '.$url);    
                     
                 }
-                else{
+                else {
                     echo 'Thank you for declining';
                 }
     		}
@@ -62,25 +62,25 @@ 
     		}
     	}
     }
-    if($type == 't'){
+    if ($type == 't') {
     	
     	$event->load_relationship('fp_events_prospects_1'); //get related targets
 
-    	if($response == 'accept'){
+    	if ($response == 'accept') {
             //check to see if they have already responded to the email
             $check_q = 'SELECT email_responded FROM fp_events_prospects_1_c WHERE fp_events_prospects_1fp_events_ida="'.$event->id.'" AND fp_events_prospects_1prospects_idb="'.$delegate_id.'"';
              $check = $db->getOne($check_q);
 
     		//update contact to accepted
     		$query = 'UPDATE fp_events_prospects_1_c SET accept_status="Accepted", email_responded="1" WHERE fp_events_prospects_1fp_events_ida="'.$event->id.'" AND fp_events_prospects_1prospects_idb="'.$delegate_id.'" AND email_responded="0"';
-    		if($db->query($query) && $check != '1'){
+    		if ($db->query($query) && $check != '1') {
                 
-                if(!IsNullOrEmptyString($event->accept_redirect)){
+                if (!IsNullOrEmptyString($event->accept_redirect)) {
                     
                     $url = $event->accept_redirect;
-                    header('Location: ' . $url);    
+                    header('Location: '.$url);    
                 }
-                else{
+                else {
                     echo 'Thank you for accepting';
                 }   
             }
@@ -88,21 +88,21 @@ 
                 echo 'You have already responded to the invitation or there was a problem with the link. Please contact the sender of the invite for help.';
             }   
     	}
-    	else if($response == 'decline'){
+    	else if ($response == 'decline') {
             //check to see if they have already responded to the email
             $check_q = 'SELECT email_responded FROM fp_events_prospects_1_c WHERE fp_events_prospects_1fp_events_ida="'.$event->id.'" AND fp_events_prospects_1prospects_idb="'.$delegate_id.'"';
              $check = $db->getOne($check_q);
     		//update contact to accepted
     		$query = 'UPDATE fp_events_prospects_1_c SET accept_status="Declined", email_responded="1" WHERE fp_events_prospects_1fp_events_ida="'.$event->id.'" AND fp_events_prospects_1prospects_idb="'.$delegate_id.'" AND email_responded="0"';
-    		if($db->query($query) && $check != '1'){
+    		if ($db->query($query) && $check != '1') {
                 
-                if(!IsNullOrEmptyString($event->decline_redirect)){
+                if (!IsNullOrEmptyString($event->decline_redirect)) {
 
                     $url = $event->decline_redirect;
-                    header('Location: ' . $url);    
+                    header('Location: '.$url);    
                     
                 }
-                else{
+                else {
                     echo 'Thank you for declining';
                 }
             }
@@ -111,24 +111,24 @@ 
             }
     	}
     }
-    if($type == 'l'){
+    if ($type == 'l') {
     	
     	$event->load_relationship('fp_events_leads_1'); //get related leads
 
-    	if($response == 'accept'){
+    	if ($response == 'accept') {
             //check to see if they have already responded to the email
             $check_q = 'SELECT email_responded FROM fp_events_leads_1_c WHERE ffp_events_leads_1fp_events_ida="'.$event->id.'" AND fp_events_leads_1leads_idb="'.$delegate_id.'"';
              $check = $db->getOne($check_q);
     		//update contact to accepted
     		$query = 'UPDATE fp_events_leads_1_c SET accept_status="Accepted", email_responded="1" WHERE fp_events_leads_1fp_events_ida="'.$event->id.'" AND fp_events_leads_1leads_idb="'.$delegate_id.'" AND email_responded="0"';
-    		if($db->query($query) && $check != '1'){
+    		if ($db->query($query) && $check != '1') {
                 
-                if(!IsNullOrEmptyString($event->accept_redirect)){
+                if (!IsNullOrEmptyString($event->accept_redirect)) {
                     
                     $url = $event->accept_redirect;
-                    header('Location: ' . $url);    
+                    header('Location: '.$url);    
                 }
-                else{
+                else {
                     echo 'Thank you for accepting';
                 }   
             }
@@ -136,22 +136,22 @@ 
                 echo 'There was a problem with the link please contact the sender of the invite';
             }   
     	}
-    	else if($response == 'decline'){
+    	else if ($response == 'decline') {
     		//check to see if they have already responded to the email
             $check_q = 'SELECT email_responded FROM fp_events_leads_1_c WHERE fp_events_leads_1fp_events_ida="'.$event->id.'" AND fp_events_leads_1leads_idb="'.$delegate_id.'"';
              $check = $db->getOne($check_q);
             //update contact to accepted
     		$query = 'UPDATE fp_events_leads_1_c SET accept_status="Declined", email_responded="1" WHERE fp_events_leads_1fp_events_ida="'.$event->id.'" AND fp_events_leads_1leads_idb="'.$delegate_id.'" AND email_responded="0"';
 
-    		if($db->query($query) && $check != '1'){
+    		if ($db->query($query) && $check != '1') {
                 
-                if(!IsNullOrEmptyString($event->decline_redirect)){
+                if (!IsNullOrEmptyString($event->decline_redirect)) {
 
                     $url = $event->decline_redirect;
-                    header('Location: ' . $url);    
+                    header('Location: '.$url);    
                     
                 }
-                else{
+                else {
                     echo 'Thank you for declining';
                 }
             }
@@ -161,8 +161,8 @@ 
     	}
     }
     // Function for basic field validation (present and neither empty nor only white space nor just 'http://')
-    function IsNullOrEmptyString($question){
-        return (!isset($question) || trim($question)==='' || $question =='http://');
+    function IsNullOrEmptyString($question) {
+        return (!isset($question) || trim($question) === '' || $question == 'http://');
     }
 
 ?>

modules/Meetings/JoinExternalMeeting.php

Spacing +14 added lines, -14 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.
@@ -47,40 +47,40 @@ 
 
 global $db, $current_user, $mod_strings, $app_strings, $app_list_strings;
 
-$ret = $db->query("SELECT id FROM meetings_users WHERE meeting_id = '".$db->quote($_REQUEST['meeting_id'])."' AND user_id = '".$current_user->id."' AND deleted = 0",true);
+$ret = $db->query("SELECT id FROM meetings_users WHERE meeting_id = '".$db->quote($_REQUEST['meeting_id'])."' AND user_id = '".$current_user->id."' AND deleted = 0", true);
 $row = $db->fetchByAssoc($ret);
 
 $meetingBean = loadBean('Meetings');
 $meetingBean->retrieve($_REQUEST['meeting_id']);
 
-if ( $_REQUEST['host_meeting'] == '1' ) {
-    if($meetingBean->assigned_user_id == $GLOBALS['current_user']->id || is_admin($GLOBALS['current_user']) || is_admin_for_module($GLOBALS['current_user'],'Meetings')){
+if ($_REQUEST['host_meeting'] == '1') {
+    if ($meetingBean->assigned_user_id == $GLOBALS['current_user']->id || is_admin($GLOBALS['current_user']) || is_admin_for_module($GLOBALS['current_user'], 'Meetings')) {
         SugarApplication::redirect($meetingBean->host_url);
-    }else{
+    } else {
         //since they are now the owner of the meeting nor an Admin they cannot start the meeting.
         $tplFile = 'modules/Meetings/tpls/extMeetingNoStart.tpl';
-        if ( file_exists('custom/'.$tplFile) ) {
+        if (file_exists('custom/'.$tplFile)) {
             $tplFile = 'custom/'.$tplFile;
         }
 
         $ss = new Sugar_Smarty();
-        $ss->assign('current_user',$current_user);
-        $ss->assign('bean',$meetingBean->toArray());
+        $ss->assign('current_user', $current_user);
+        $ss->assign('bean', $meetingBean->toArray());
         $ss->display($tplFile);
     }
-}else{
-    if(isset($row['id']) || $meetingBean->assigned_user_id == $GLOBALS['current_user']->id || is_admin($GLOBALS['current_user']) || is_admin_for_module($GLOBALS['current_user'],'Meetings')){
+} else {
+    if (isset($row['id']) || $meetingBean->assigned_user_id == $GLOBALS['current_user']->id || is_admin($GLOBALS['current_user']) || is_admin_for_module($GLOBALS['current_user'], 'Meetings')) {
       SugarApplication::redirect($meetingBean->join_url);
-    }else{
+    } else {
         //if the user is not invited or the owner of the meeting or an admin then they cannot join the meeting.
         $tplFile = 'modules/Meetings/tpls/extMeetingNotInvited.tpl';
-        if ( file_exists('custom/'.$tplFile) ) {
+        if (file_exists('custom/'.$tplFile)) {
             $tplFile = 'custom/'.$tplFile;
         }
 
         $ss = new Sugar_Smarty();
-        $ss->assign('current_user',$current_user);
-        $ss->assign('bean',$meetingBean->toArray());
+        $ss->assign('current_user', $current_user);
+        $ss->assign('bean', $meetingBean->toArray());
         $ss->display($tplFile);
     }
 }
\ No newline at end of file

modules/Meetings/MeetingsQuickCreate.php

Spacing +13 added lines, -13 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.
@@ -55,13 +55,13 @@ 
 
         $this->ss->assign("STATUS_OPTIONS", get_select_options_with_id($app_list_strings['meeting_status_dom'], $app_list_strings['meeting_status_default']));
 		$this->ss->assign("CALENDAR_DATEFORMAT", $timedate->get_cal_date_format());
-		$this->ss->assign("TIME_FORMAT", '('. $timedate->get_user_time_format().')');
-		$this->ss->assign("USER_DATEFORMAT", '('. $timedate->get_user_date_format().')');
+		$this->ss->assign("TIME_FORMAT", '('.$timedate->get_user_time_format().')');
+		$this->ss->assign("USER_DATEFORMAT", '('.$timedate->get_user_date_format().')');
 		
 
 
         
-        if($this->viaAJAX) { // override for ajax call
+        if ($this->viaAJAX) { // override for ajax call
             $this->ss->assign('saveOnclick', "onclick='if(check_form(\"meetingsQuickCreate\")) return SUGAR.subpanelUtils.inlineSave(this.form.id, \"activities\"); else return false;'");
             $this->ss->assign('cancelOnclick', "onclick='return SUGAR.subpanelUtils.cancelCreate(\"subpanel_activities\")';");
         }
@@ -83,14 +83,14 @@ 
 			$focus->duration_hours = "1";
 
         
-        $date_start_array=explode(" ",trim($focus->date_start));
-        if (count($date_start_array)==2) {
+        $date_start_array = explode(" ", trim($focus->date_start));
+        if (count($date_start_array) == 2) {
 			$focus->time_start = $timedate->to_db_time($date_start_array[1], false);
         	//$focus->date_start = $date_start_array[0];
         }
 
 		$this->ss->assign("DATE_START", $focus->date_start);
-		$this->ss->assign("TIME_START", substr($focus->time_start,0,5));
+		$this->ss->assign("TIME_START", substr($focus->time_start, 0, 5));
 		$time_start_hour = intval(substr($focus->time_start, 0, 2));
 		$time_start_minutes = substr($focus->time_start, 3, 5);
 		
@@ -110,12 +110,12 @@ 
 		
 		
 		// We default the to assume that the time preference is set to 11:00 (i.e. without meridiem)
-		$hours_arr = array ();
+		$hours_arr = array();
 		$num_of_hours = 24;
 		$start_at = 0;
 
 		$time_pref = $timedate->get_time_format();
-		if(strpos($time_pref, 'a') || strpos($time_pref, 'A')) {
+		if (strpos($time_pref, 'a') || strpos($time_pref, 'A')) {
 		   $num_of_hours = 13;
 		   $start_at = 1;	
 		} 
@@ -128,7 +128,7 @@ 
 		}
 		*/
 		
-		for ($i = $start_at; $i < $num_of_hours; $i ++) {
+		for ($i = $start_at; $i < $num_of_hours; $i++) {
 			$i = $i."";
 			if (strlen($i) == 1) {
 				$i = "0".$i;
@@ -141,18 +141,18 @@ 
 		$this->ss->assign("DURATION_HOURS", $focus->duration_hours);
 		$this->ss->assign("DURATION_MINUTES_OPTIONS", get_select_options_with_id($focus->minutes_values, $focus->duration_minutes));
         // Test to see if time format is 11:00am; otherwise it's 11:00AM 
-        if($num_of_hours == 13) {    
+        if ($num_of_hours == 13) {    
         		
 		   if (strpos($time_pref, 'a')) {
 		   	   
-               if(!isset($focus->meridiem_am_values)) {
+               if (!isset($focus->meridiem_am_values)) {
                   $focus->meridiem_am_values = array('am'=>'am', 'pm'=>'pm');
                } 		
                
                $this->ss->assign("TIME_MERIDIEM", get_select_options_with_id($focus->meridiem_am_values, $time_start_hour < 12 ? 'am' : 'pm'));
                
 		   } else {
-		       if(!isset($focus->meridiem_AM_values)) {
+		       if (!isset($focus->meridiem_AM_values)) {
 		          $focus->meridiem_AM_values = array('AM'=>'AM', 'PM'=>'PM');
 		       }
 		       

modules/Meetings/MeetingsListViewSmarty.php

Spacing +8 added lines, -8 removed lines

@@ -16,14 +16,14 @@
 
         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']}', " .
+        $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']}', ".
             "'{$app_strings['LBL_LISTVIEW_NO_SELECTED']}')\">{$app_strings['LBL_MAP']}</a>";
 
         return $script;

modules/Meetings/SubPanelViewInvitees.php

Spacing +6 added lines, -6 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.
@@ -79,14 +79,14 @@ 
 
 // Stick the form header out there.
 echo get_form_header($mod_strings['LBL_INVITEE'], $button, false);
-$xtpl=new XTemplate ('modules/Meetings/SubPanelViewInvitees.html');
+$xtpl = new XTemplate('modules/Meetings/SubPanelViewInvitees.html');
 $xtpl->assign("MOD", $mod_strings);
 $xtpl->assign("APP", $app_strings);
 $xtpl->assign("RETURN_URL", "&return_module=$currentModule&return_action=DetailView&return_id=$focus->id");
 $xtpl->assign("MEETING_ID", $focus->id);
 
 $oddRow = true;
-foreach($focus_users_list as $user)
+foreach ($focus_users_list as $user)
 {
 	$user_fields = array(
 		'USER_NAME' => $user->user_name,
@@ -98,7 +98,7 @@ 
 
 	$xtpl->assign("USER", $user_fields);
 
-	if($oddRow)
+	if ($oddRow)
     {
         //todo move to themes
 		$xtpl->assign("ROW_COLOR", 'oddListRow');
@@ -118,7 +118,7 @@ 
 $xtpl->out("users");
 
 $oddRow = true;
-foreach($focus_contacts_list as $contact)
+foreach ($focus_contacts_list as $contact)
 {
 	$contact_fields = array(
 		'FIRST_NAME' => $contact->first_name,
@@ -131,7 +131,7 @@ 
 
 	$xtpl->assign("CONTACT", $contact_fields);
 
-	if($oddRow)
+	if ($oddRow)
     {
         //todo move to themes
 		$xtpl->assign("ROW_COLOR", 'oddListRow');

modules/Meetings/Meeting.php

Spacing +123 added lines, -123 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.
@@ -93,7 +93,7 @@ 
 	var $meetings_arr;
 	// when assoc w/ a user/contact:
 	var $minutes_value_default = 15;
-	var $minutes_values = array('0'=>'00','15'=>'15','30'=>'30','45'=>'45');
+	var $minutes_values = array('0'=>'00', '15'=>'15', '30'=>'30', '45'=>'45');
 	var $table_name = "meetings";
 	var $rel_users_table = "meetings_users";
 	var $rel_contacts_table = "meetings_contacts";
@@ -104,8 +104,8 @@ 
 	var $importable = true;
 	// This is used to retrieve related fields from form posts.
 	var $additional_column_fields = array('assigned_user_name', 'assigned_user_id', 'contact_id', 'user_id', 'contact_name', 'accept_status');
-	var $relationship_fields = array('account_id'=>'accounts','opportunity_id'=>'opportunity','case_id'=>'case',
-									 'assigned_user_id'=>'users','contact_id'=>'contacts', 'user_id'=>'users', 'meeting_id'=>'meetings');
+	var $relationship_fields = array('account_id'=>'accounts', 'opportunity_id'=>'opportunity', 'case_id'=>'case',
+									 'assigned_user_id'=>'users', 'contact_id'=>'contacts', 'user_id'=>'users', 'meeting_id'=>'meetings');
 	// so you can run get_users() twice and run query only once
 	var $cached_get_users = null;
 	var $new_schema = true;
@@ -117,11 +117,11 @@ 
 	function Meeting() {
 		parent::SugarBean();
 		$this->setupCustomFields('Meetings');
-		foreach($this->field_defs as $field) {
+		foreach ($this->field_defs as $field) {
 			$this->field_name_map[$field['name']] = $field;
 		}
 //		$this->fill_in_additional_detail_fields();
-        if(!empty($GLOBALS['app_list_strings']['duration_intervals'])) {
+        if (!empty($GLOBALS['app_list_strings']['duration_intervals'])) {
             $this->minutes_values = $GLOBALS['app_list_strings']['duration_intervals'];
         }
 	}
@@ -131,21 +131,21 @@ 
 	 * @param $view string
 	 * @param $is_owner bool
 	 */
-	function ACLAccess($view,$is_owner='not_set',$in_group='not_set'){
+	function ACLAccess($view, $is_owner = 'not_set', $in_group = 'not_set') {
 		// don't check if meeting is being synced from Outlook
-		if($this->syncing == false){
+		if ($this->syncing == false) {
 			$view = strtolower($view);
-			switch($view){
+			switch ($view) {
 				case 'edit':
 				case 'save':
 				case 'editview':
 				case 'delete':
-					if(!empty($this->recurring_source) && $this->recurring_source != "Sugar"){
+					if (!empty($this->recurring_source) && $this->recurring_source != "Sugar") {
 						return false;
 					}
 			}
 		}
-		return parent::ACLAccess($view,$is_owner,$in_group);
+		return parent::ACLAccess($view, $is_owner, $in_group);
 	}
 
 	/**
@@ -164,14 +164,14 @@ 
 
 		global $disable_date_format;
 
-        if(isset($this->date_start))
+        if (isset($this->date_start))
         {
             $td = $timedate->fromDb($this->date_start);
-            if(!$td){
+            if (!$td) {
             		$this->date_start = $timedate->to_db($this->date_start);
             		$td = $timedate->fromDb($this->date_start);
             }
-            if($td)
+            if ($td)
             {
                 if (isset($this->duration_hours) && $this->duration_hours != '')
                 {
@@ -185,17 +185,17 @@ 
             }
         }
 
-		$check_notify =(!empty($_REQUEST['send_invites']) && $_REQUEST['send_invites'] == '1') ? true : false;
-		if(empty($_REQUEST['send_invites'])) {
-			if(!empty($this->id)) {
+		$check_notify = (!empty($_REQUEST['send_invites']) && $_REQUEST['send_invites'] == '1') ? true : false;
+		if (empty($_REQUEST['send_invites'])) {
+			if (!empty($this->id)) {
 				$old_record = new Meeting();
 				$old_record->retrieve($this->id);
 				$old_assigned_user_id = $old_record->assigned_user_id;
 			}
-			if((empty($this->id) && isset($_REQUEST['assigned_user_id']) && !empty($_REQUEST['assigned_user_id']) && $GLOBALS['current_user']->id != $_REQUEST['assigned_user_id']) || (isset($old_assigned_user_id) && !empty($old_assigned_user_id) && isset($_REQUEST['assigned_user_id']) && !empty($_REQUEST['assigned_user_id']) && $old_assigned_user_id != $_REQUEST['assigned_user_id']) ){
+			if ((empty($this->id) && isset($_REQUEST['assigned_user_id']) && !empty($_REQUEST['assigned_user_id']) && $GLOBALS['current_user']->id != $_REQUEST['assigned_user_id']) || (isset($old_assigned_user_id) && !empty($old_assigned_user_id) && isset($_REQUEST['assigned_user_id']) && !empty($_REQUEST['assigned_user_id']) && $old_assigned_user_id != $_REQUEST['assigned_user_id'])) {
 				$this->special_notification = true;
 				$check_notify = true;
-                if(isset($_REQUEST['assigned_user_name'])) {
+                if (isset($_REQUEST['assigned_user_name'])) {
                     $this->new_assigned_user_name = $_REQUEST['assigned_user_name'];
                 }
 			}
@@ -207,23 +207,23 @@ 
 		}*/
 
 		// prevent a mass mailing for recurring meetings created in Calendar module
-		if(empty($this->id) && !empty($_REQUEST['module']) && $_REQUEST['module'] == "Calendar" && !empty($_REQUEST['repeat_type']) && !empty($this->repeat_parent_id))
+		if (empty($this->id) && !empty($_REQUEST['module']) && $_REQUEST['module'] == "Calendar" && !empty($_REQUEST['repeat_type']) && !empty($this->repeat_parent_id))
 			$check_notify = false;
 
-        if (empty($this->status) ) {
+        if (empty($this->status)) {
             $this->status = $this->getDefaultStatus();
         }
 
         // Do any external API saving
         // Clear out the old external API stuff if we have changed types
-        if (isset($this->fetched_row) && $this->fetched_row['type'] != $this->type ) {
+        if (isset($this->fetched_row) && $this->fetched_row['type'] != $this->type) {
             $this->join_url = '';
             $this->host_url = '';
             $this->external_id = '';
             $this->creator = '';
         }
 
-        if (!empty($this->type) && $this->type != 'Sugar' ) {
+        if (!empty($this->type) && $this->type != 'Sugar') {
             require_once('include/externalAPI/ExternalAPIFactory.php');
             $api = ExternalAPIFactory::loadAPI($this->type);
         }
@@ -232,27 +232,27 @@ 
 			$this->type = 'Sugar';
 		}
 
-        if ( isset($api) && is_a($api,'WebMeeting') && empty($this->in_relationship_update) ) {
+        if (isset($api) && is_a($api, 'WebMeeting') && empty($this->in_relationship_update)) {
             // Make sure the API initialized and it supports Web Meetings
             // Also make suer we have an ID, the external site needs something to reference
-            if ( !isset($this->id) || empty($this->id) ) {
+            if (!isset($this->id) || empty($this->id)) {
                 $this->id = create_guid();
                 $this->new_with_id = true;
             }
             $response = $api->scheduleMeeting($this);
-            if ( $response['success'] == TRUE ) {
+            if ($response['success'] == TRUE) {
                 // Need to send out notifications
-                if ( $api->canInvite ) {
+                if ($api->canInvite) {
                     $notifyList = $this->get_notification_recipients();
-                    foreach($notifyList as $person) {
-                        $api->inviteAttendee($this,$person,$check_notify);
+                    foreach ($notifyList as $person) {
+                        $api->inviteAttendee($this, $person, $check_notify);
                     }
 
                 }
             } else {
                 // Generic Message Provides no value to End User - Log the issue with message detail and continue
                 // SugarApplication::appendErrorMessage($GLOBALS['app_strings']['ERR_EXTERNAL_API_SAVE_FAIL']);
-                $GLOBALS['log']->warn('ERR_EXTERNAL_API_SAVE_FAIL' . ": " . $this->type . " - " .  $response['errorMessage']);
+                $GLOBALS['log']->warn('ERR_EXTERNAL_API_SAVE_FAIL'.": ".$this->type." - ".$response['errorMessage']);
             }
 
             $api->logoff();
@@ -260,11 +260,11 @@ 
 
 		$return_id = parent::save($check_notify);
 
-		if($this->update_vcal) {
+		if ($this->update_vcal) {
 			vCal::cache_sugar_vcal($current_user);
 		}
 
-		if(isset($_REQUEST['reminders_data'])) {
+		if (isset($_REQUEST['reminders_data'])) {
 			Reminder::saveRemindersDataJson('Meetings', $return_id, html_entity_decode($_REQUEST['reminders_data']));
 		}
 
@@ -282,7 +282,7 @@ 
 
 		parent::mark_deleted($id);
 
-		if($this->update_vcal) {
+		if ($this->update_vcal) {
 			vCal::cache_sugar_vcal($current_user);
 		}
 	}
@@ -291,13 +291,13 @@ 
 		return "$this->name";
 	}
 
-    function create_export_query($order_by, $where, $relate_link_join='')
+    function create_export_query($order_by, $where, $relate_link_join = '')
     {
         $custom_join = $this->getCustomJoin(true, true, $where);
         $custom_join['join'] .= $relate_link_join;
 		$contact_required = stristr($where, "contacts");
 
-		if($contact_required) {
+		if ($contact_required) {
 			$query = "SELECT meetings.*, contacts.first_name, contacts.last_name, contacts.assigned_user_id contact_name_owner, users.user_name as assigned_user_name   ";
             $query .= $custom_join['select'];
 			$query .= " FROM contacts, meetings, meetings_contacts ";
@@ -312,14 +312,14 @@ 
 
         $query .= $custom_join['join'];
 
-		if($where != "")
+		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)) {
-            $query .= ' ORDER BY ' . $order_by;
+            $query .= ' ORDER BY '.$order_by;
         }
 
 		return $query;
@@ -335,12 +335,12 @@ 
 		if (!empty($this->contact_id)) {
 			$query  = "SELECT first_name, last_name FROM contacts ";
 			$query .= "WHERE id='$this->contact_id' AND deleted=0";
-			$result = $this->db->limitQuery($query,0,1,true," Error filling in additional detail fields: ");
+			$result = $this->db->limitQuery($query, 0, 1, true, " Error filling in additional detail fields: ");
 
 			// Get the contact name.
 			$row = $this->db->fetchByAssoc($result);
 			$GLOBALS['log']->info("additional call fields $query");
-			if($row != null)
+			if ($row != null)
 			{
 				$this->contact_name = $locale->getLocaleFormattedName($row['first_name'], $row['last_name'], '', '');
 				$GLOBALS['log']->debug("Call($this->id): contact_name = $this->contact_name");
@@ -383,7 +383,7 @@ 
 
 		global $timedate;
         $this->time_meridiem = $timedate->AMPMMenu('', $this->time_start, 'onchange="SugarWidgetScheduler.update_time();"');
-		$hours_arr = array ();
+		$hours_arr = array();
 		$num_of_hours = 13;
 		$start_at = 1;
 
@@ -392,7 +392,7 @@ 
 			$start_at = 0;
 		} //if
 
-		for ($i = $start_at; $i < $num_of_hours; $i ++) {
+		for ($i = $start_at; $i < $num_of_hours; $i++) {
 			$i = $i."";
 			if (strlen($i) == 1) {
 				$i = "0".$i;
@@ -415,12 +415,12 @@ 
 		if (is_null($this->duration_minutes))
 			$this->duration_minutes = "1";
 
-		if(empty($this->id) && !empty($_REQUEST['date_start'])){
+		if (empty($this->id) && !empty($_REQUEST['date_start'])) {
 			$this->date_start = $_REQUEST['date_start'];
 		}
-        if(!empty($this->date_start))
+        if (!empty($this->date_start))
         {
-            $td = SugarDateTime::createFromFormat($GLOBALS['timedate']->get_date_time_format(),$this->date_start);
+            $td = SugarDateTime::createFromFormat($GLOBALS['timedate']->get_date_time_format(), $this->date_start);
             if (!empty($td)) 
             {
     	        if (!empty($this->duration_hours) && $this->duration_hours != '')
@@ -441,9 +441,9 @@ 
 
 		global $app_list_strings;
 		$parent_types = $app_list_strings['record_type_display'];
-		$disabled_parent_types = ACLController::disabledModuleList($parent_types,false, 'list');
-		foreach($disabled_parent_types as $disabled_parent_type){
-			if($disabled_parent_type != $this->parent_type){
+		$disabled_parent_types = ACLController::disabledModuleList($parent_types, false, 'list');
+		foreach ($disabled_parent_types as $disabled_parent_type) {
+			if ($disabled_parent_type != $this->parent_type) {
 				unset($parent_types[$disabled_parent_type]);
 			}
 		}
@@ -453,9 +453,9 @@ 
 			$this->reminder_time = -1;
 		}
 
-		if ( empty($this->id) ) {
+		if (empty($this->id)) {
 		    $reminder_t = $GLOBALS['current_user']->getPreference('reminder_time');
-		    if ( isset($reminder_t) )
+		    if (isset($reminder_t))
 		        $this->reminder_time = $reminder_t;
 		}
 		$this->reminder_checked = $this->reminder_time == -1 ? false : true;
@@ -463,9 +463,9 @@ 
 		if (empty($this->email_reminder_time)) {
 			$this->email_reminder_time = -1;
 		}
-		if(empty($this->id)){ 
+		if (empty($this->id)) { 
 			$reminder_t = $GLOBALS['current_user']->getPreference('email_reminder_time');
-			if(isset($reminder_t))
+			if (isset($reminder_t))
 		    		$this->email_reminder_time = $reminder_t;
 		}
 		$this->email_reminder_checked = $this->email_reminder_time == -1 ? false : true;
@@ -482,15 +482,15 @@ 
 		$meeting_fields = $this->get_list_view_array();
 
 		global $app_list_strings, $focus, $action, $currentModule;
-		if(isset($this->parent_type))
+		if (isset($this->parent_type))
 			$meeting_fields['PARENT_MODULE'] = $this->parent_type;
-		if($this->status == "Planned") {
+		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))
+			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')) {
-                $meeting_fields['SET_COMPLETE'] = $setCompleteUrl . SugarThemeRegistry::current()->getImage("close_inline"," border='0'",null,null,'.gif',translate('LBL_CLOSEINLINE'))."</a>";
+                $meeting_fields['SET_COMPLETE'] = $setCompleteUrl.SugarThemeRegistry::current()->getImage("close_inline", " border='0'", null, null, '.gif', translate('LBL_CLOSEINLINE'))."</a>";
             } else {
                 $meeting_fields['SET_COMPLETE'] = '';
             }
@@ -500,9 +500,9 @@ 
 		$nextday = $timedate->asDbDate($timedate->getNow()->get("+1 day"));
 		$mergeTime = $meeting_fields['DATE_START']; //$timedate->merge_date_time($meeting_fields['DATE_START'], $meeting_fields['TIME_START']);
 		$date_db = $timedate->to_db($mergeTime);
-		if($date_db	< $today	) {
-			$meeting_fields['DATE_START']= "<font class='overdueTask'>".$meeting_fields['DATE_START']."</font>";
-		}else if($date_db	< $nextday) {
+		if ($date_db < $today) {
+			$meeting_fields['DATE_START'] = "<font class='overdueTask'>".$meeting_fields['DATE_START']."</font>";
+		} else if ($date_db < $nextday) {
 			$meeting_fields['DATE_START'] = "<font class='todaysTask'>".$meeting_fields['DATE_START']."</font>";
 		} else {
 			$meeting_fields['DATE_START'] = "<font class='futureTask'>".$meeting_fields['DATE_START']."</font>";
@@ -524,8 +524,8 @@ 
         $meeting_fields['CONTACT_ID'] = $this->contact_id;
         $meeting_fields['CONTACT_NAME'] = $this->contact_name;
 		$meeting_fields['PARENT_NAME'] = $this->parent_name;
-        $meeting_fields['REMINDER_CHECKED'] = $this->reminder_time==-1 ? false : true;
-        $meeting_fields['EMAIL_REMINDER_CHECKED'] = $this->email_reminder_time==-1 ? false : true;
+        $meeting_fields['REMINDER_CHECKED'] = $this->reminder_time == -1 ? false : true;
+        $meeting_fields['EMAIL_REMINDER_CHECKED'] = $this->email_reminder_time == -1 ? false : true;
 
 
 		return $meeting_fields;
@@ -539,12 +539,12 @@ 
 
 
 		// cn: bug 9494 - passing a contact breaks this call
-		$notifyUser =($meeting->current_notify_user->object_name == 'User') ? $meeting->current_notify_user : $current_user;
+		$notifyUser = ($meeting->current_notify_user->object_name == 'User') ? $meeting->current_notify_user : $current_user;
 		// cn: bug 8078 - fixed call to $timedate
-		if(strtolower(get_class($meeting->current_notify_user)) == 'contact') {
+		if (strtolower(get_class($meeting->current_notify_user)) == 'contact') {
 			$xtpl->assign("ACCEPT_URL", $sugar_config['site_url'].
 							'/index.php?entryPoint=acceptDecline&module=Meetings&contact_id='.$meeting->current_notify_user->id.'&record='.$meeting->id);
-		} elseif(strtolower(get_class($meeting->current_notify_user)) == 'lead') {
+		} elseif (strtolower(get_class($meeting->current_notify_user)) == 'lead') {
 			$xtpl->assign("ACCEPT_URL", $sugar_config['site_url'].
 							'/index.php?entryPoint=acceptDecline&module=Meetings&lead_id='.$meeting->current_notify_user->id.'&record='.$meeting->id);
 		} else {
@@ -553,16 +553,16 @@ 
 		}
 		$xtpl->assign("MEETING_TO", $meeting->current_notify_user->new_assigned_user_name);
 		$xtpl->assign("MEETING_SUBJECT", trim($meeting->name));
-		$xtpl->assign("MEETING_STATUS",(isset($meeting->status)? $app_list_strings['meeting_status_dom'][$meeting->status]:""));
+		$xtpl->assign("MEETING_STATUS", (isset($meeting->status) ? $app_list_strings['meeting_status_dom'][$meeting->status] : ""));
 		$typekey = strtolower($meeting->type);
-		if(isset($meeting->type)) {
-		    if(!empty($app_list_strings['eapm_list'][$typekey])) {
+		if (isset($meeting->type)) {
+		    if (!empty($app_list_strings['eapm_list'][$typekey])) {
     		    $typestring = $app_list_strings['eapm_list'][$typekey];
 	    	} else {
 		        $typestring = $app_list_strings['meeting_type_dom'][$meeting->type];
 		    }
 		}
-		$xtpl->assign("MEETING_TYPE", isset($meeting->type)? $typestring:"");
+		$xtpl->assign("MEETING_TYPE", isset($meeting->type) ? $typestring : "");
 		$startdate = $timedate->fromDb($meeting->date_start);
 		$xtpl->assign("MEETING_STARTDATE", $timedate->asUser($startdate, $notifyUser)." ".TimeDate::userTimezoneSuffix($startdate, $notifyUser));
 		$enddate = $timedate->fromDb($meeting->date_end);
@@ -570,7 +570,7 @@ 
 		$xtpl->assign("MEETING_HOURS", $meeting->duration_hours);
 		$xtpl->assign("MEETING_MINUTES", $meeting->duration_minutes);
 		$xtpl->assign("MEETING_DESCRIPTION", $meeting->description);
-        if ( !empty($meeting->join_url) ) {
+        if (!empty($meeting->join_url)) {
             $xtpl->assign('MEETING_URL', $meeting->join_url);
             $xtpl->parse('Meeting.Meeting_External_API');
         }
@@ -581,7 +581,7 @@ 
 	/**
 	 * Redefine method to attach ics file to notification email
 	 */
-	public function create_notification_email($notify_user){
+	public function create_notification_email($notify_user) {
         // reset acceptance status for non organizer if date is changed
         if (($notify_user->id != $GLOBALS['current_user']->id) && $this->date_changed) {
             $this->set_accept_status($notify_user, 'none');
@@ -589,12 +589,12 @@ 
 
 		$notify_mail = parent::create_notification_email($notify_user);
 						
-		$path = SugarConfig::getInstance()->get('upload_dir','upload/') . $this->id;
+		$path = SugarConfig::getInstance()->get('upload_dir', 'upload/').$this->id;
 
 		require_once("modules/vCals/vCal.php");
 		$content = vCal::get_ical_event($this, $GLOBALS['current_user']);
 				
-		if(file_put_contents($path,$content)){
+		if (file_put_contents($path, $content)) {
 			$notify_mail->AddAttachment($path, 'meeting.ics', 'base64', 'text/calendar');
 		}
 		return $notify_mail;		
@@ -603,10 +603,10 @@ 
 	/**
 	 * Redefine method to remove ics after email is sent
 	 */
-	public function send_assignment_notifications($notify_user, $admin){
+	public function send_assignment_notifications($notify_user, $admin) {
 		parent::send_assignment_notifications($notify_user, $admin);
 		
-		$path = SugarConfig::getInstance()->get('upload_dir','upload/') . $this->id;
+		$path = SugarConfig::getInstance()->get('upload_dir', 'upload/').$this->id;
 		unlink($path);
 	}
 
@@ -618,13 +618,13 @@ 
 		$result = $this->db->query($query, true);
 		$list = Array();
 
-		while($row = $this->db->fetchByAssoc($result)) {
+		while ($row = $this->db->fetchByAssoc($result)) {
 			$template = new User(); // PHP 5 will retrieve by reference, always over-writing the "old" one
 			$record = $template->retrieve($row['user_id']);
 			$template->required = $row['required'];
 			$template->accept_status = $row['accept_status'];
 
-			if($record != null) {
+			if ($record != null) {
 				// this copies the object into the array
 				$list[] = $template;
 			}
@@ -640,13 +640,13 @@ 
 		$result = $this->db->query($query, true);
 		$list = Array();
 
-		while($row = $this->db->fetchByAssoc($result)) {
+		while ($row = $this->db->fetchByAssoc($result)) {
 			$record = $template->retrieve($row['meeting_id']);
 			$template->required = $row['required'];
 			$template->accept_status = $row['accept_status'];
 
 
-			if($record != null)
+			if ($record != null)
 			{
 			// this copies the object into the array
 			$list[] = $template;
@@ -656,54 +656,54 @@ 
 	}
 
 
-	function set_accept_status(&$user,$status)
+	function set_accept_status(&$user, $status)
 	{
-		if($user->object_name == 'User')
+		if ($user->object_name == 'User')
 		{
-			$relate_values = array('user_id'=>$user->id,'meeting_id'=>$this->id);
+			$relate_values = array('user_id'=>$user->id, 'meeting_id'=>$this->id);
 			$data_values = array('accept_status'=>$status);
-			$this->set_relationship($this->rel_users_table, $relate_values, true, true,$data_values);
+			$this->set_relationship($this->rel_users_table, $relate_values, true, true, $data_values);
 			global $current_user;
 
-			if($this->update_vcal)
+			if ($this->update_vcal)
 			{
 				vCal::cache_sugar_vcal($user);
 			}
 		}
-		else if($user->object_name == 'Contact')
+		else if ($user->object_name == 'Contact')
 		{
-			$relate_values = array('contact_id'=>$user->id,'meeting_id'=>$this->id);
+			$relate_values = array('contact_id'=>$user->id, 'meeting_id'=>$this->id);
 			$data_values = array('accept_status'=>$status);
-			$this->set_relationship($this->rel_contacts_table, $relate_values, true, true,$data_values);
+			$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,'meeting_id'=>$this->id);
+			$relate_values = array('lead_id'=>$user->id, 'meeting_id'=>$this->id);
 			$data_values = array('accept_status'=>$status);
-			$this->set_relationship($this->rel_leads_table, $relate_values, true, true,$data_values);
+			$this->set_relationship($this->rel_leads_table, $relate_values, true, true, $data_values);
 		}
 	}
 
 
 	function get_notification_recipients() {
-		if($this->special_notification) {
+		if ($this->special_notification) {
 			return parent::get_notification_recipients();
 		}
 
 		$list = array();
-		if(!is_array($this->contacts_arr)) {
-			$this->contacts_arr =	array();
+		if (!is_array($this->contacts_arr)) {
+			$this->contacts_arr = array();
 		}
 
-		if(!is_array($this->users_arr)) {
-			$this->users_arr =	array();
+		if (!is_array($this->users_arr)) {
+			$this->users_arr = array();
 		}
 
-        if(!is_array($this->leads_arr)) {
-			$this->leads_arr =	array();
+        if (!is_array($this->leads_arr)) {
+			$this->leads_arr = array();
 		}
 
-		foreach($this->users_arr as $user_id) {
+		foreach ($this->users_arr as $user_id) {
 			$notify_user = new User();
 			$notify_user->retrieve($user_id);
 			$notify_user->new_assigned_user_name = $notify_user->full_name;
@@ -711,7 +711,7 @@ 
 			$list[$notify_user->id] = $notify_user;
 		}
 
-		foreach($this->contacts_arr as $contact_id) {
+		foreach ($this->contacts_arr as $contact_id) {
 			$notify_user = new Contact();
 			$notify_user->retrieve($contact_id);
 			$notify_user->new_assigned_user_name = $notify_user->full_name;
@@ -719,7 +719,7 @@ 
 			$list[$notify_user->id] = $notify_user;
 		}
 
-        foreach($this->leads_arr as $lead_id) {
+        foreach ($this->leads_arr as $lead_id) {
 			$notify_user = new Lead();
 			$notify_user->retrieve($lead_id);
 			$notify_user->new_assigned_user_name = $notify_user->full_name;
@@ -728,9 +728,9 @@ 
 		}
 
 		global $sugar_config;
-		if(isset($sugar_config['disable_notify_current_user']) && $sugar_config['disable_notify_current_user']) {
+		if (isset($sugar_config['disable_notify_current_user']) && $sugar_config['disable_notify_current_user']) {
 			global $current_user;
-			if(isset($list[$current_user->id]))
+			if (isset($list[$current_user->id]))
 				unset($list[$current_user->id]);
 		}
 		return $list;
@@ -738,7 +738,7 @@ 
 
 
 	function bean_implements($interface) {
-		switch($interface) {
+		switch ($interface) {
 			case 'ACL':return true;
 		}
 		return false;
@@ -748,18 +748,18 @@ 
 		$array_assign = parent::listviewACLHelper();
 		$is_owner = false;
 		$in_group = false; //SECURITY GROUPS
-		if(!empty($this->parent_name)) {
+		if (!empty($this->parent_name)) {
 
-			if(!empty($this->parent_name_owner)) {
+			if (!empty($this->parent_name_owner)) {
 				global $current_user;
 				$is_owner = $current_user->id == $this->parent_name_owner;
 			}
 			/* BEGIN - SECURITY GROUPS */
 			//parent_name_owner not being set for whatever reason so we need to figure this out
-			else if(!empty($this->parent_type) && !empty($this->parent_id)) {
+			else if (!empty($this->parent_type) && !empty($this->parent_id)) {
 				global $current_user;
-                $parent_bean = BeanFactory::getBean($this->parent_type,$this->parent_id);
-                if($parent_bean !== false) {
+                $parent_bean = BeanFactory::getBean($this->parent_type, $this->parent_id);
+                if ($parent_bean !== false) {
                 	$is_owner = $current_user->id == $parent_bean->assigned_user_id;
                 }
 			}
@@ -772,7 +772,7 @@ 
 		/**
 		if(!ACLController::moduleSupportsACL($this->parent_type) || ACLController::checkAccess($this->parent_type, 'view', $is_owner)) {
 		*/
-		if(!ACLController::moduleSupportsACL($this->parent_type) || ACLController::checkAccess($this->parent_type, 'view', $is_owner, 'module', $in_group)){
+		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 {
@@ -782,8 +782,8 @@ 
 		$is_owner = false;
 		$in_group = false; //SECURITY GROUPS
 
-		if(!empty($this->contact_name)) {
-			if(!empty($this->contact_name_owner)) {
+		if (!empty($this->contact_name)) {
+			if (!empty($this->contact_name_owner)) {
 				global $current_user;
 				$is_owner = $current_user->id == $this->contact_name_owner;
 			}
@@ -791,8 +791,8 @@ 
 			//contact_name_owner not being set for whatever reason so we need to figure this out
 			else {
 				global $current_user;
-                $parent_bean = BeanFactory::getBean('Contacts',$this->contact_id);
-                if($parent_bean !== false) {
+                $parent_bean = BeanFactory::getBean('Contacts', $this->contact_id);
+                if ($parent_bean !== false) {
                 	$is_owner = $current_user->id == $parent_bean->assigned_user_id;
                 }
 			}
@@ -805,7 +805,7 @@ 
 		/**
 		if(ACLController::checkAccess('Contacts', 'view', $is_owner)) {
 		*/
-		if(ACLController::checkAccess('Contacts', 'view', $is_owner, 'module', $in_group)) {
+		if (ACLController::checkAccess('Contacts', 'view', $is_owner, 'module', $in_group)) {
         /* END - SECURITY GROUPS */
 			$array_assign['CONTACT'] = 'a';
 		} else {
@@ -816,17 +816,17 @@ 
 
 
 	function save_relationship_changes($is_update, $exclude = array()) {
-	    if(empty($this->in_workflow)) {
-           if(empty($this->in_import)){//if a meeting is being imported then contact_id  should not be excluded
+	    if (empty($this->in_workflow)) {
+           if (empty($this->in_import)) {//if a meeting is being imported then contact_id  should not be excluded
            //if the global soap_server_object variable is not empty (as in from a soap/OPI call), then process the assigned_user_id relationship, otherwise
            //add assigned_user_id to exclude list and let the logic from MeetingFormBase determine whether assigned user id gets added to the relationship
-           	if(!empty($GLOBALS['soap_server_object'])){
+           	if (!empty($GLOBALS['soap_server_object'])) {
            		$exclude = array('contact_id', 'user_id');
-           	}else{
-	    		$exclude = array('contact_id', 'user_id','assigned_user_id');
+           	} else {
+	    		$exclude = array('contact_id', 'user_id', 'assigned_user_id');
            	}
            }
-           else{
+           else {
            	$exclude = array('user_id');
            }
         }
@@ -839,14 +839,14 @@ 
 	 */
 	public function afterImportSave()
 	{
-	    if ( $this->parent_type == 'Contacts' ) {
+	    if ($this->parent_type == 'Contacts') {
 	        $this->load_relationship('contacts');
-	        if ( !$this->contacts->relationship_exists('contacts',array('id'=>$this->parent_id)) )
+	        if (!$this->contacts->relationship_exists('contacts', array('id'=>$this->parent_id)))
 	            $this->contacts->add($this->parent_id);
 	    }
-	    elseif ( $this->parent_type == 'Leads' ) {
+	    elseif ($this->parent_type == 'Leads') {
 	        $this->load_relationship('leads');
-	        if ( !$this->leads->relationship_exists('leads',array('id'=>$this->parent_id)) )
+	        if (!$this->leads->relationship_exists('leads', array('id'=>$this->parent_id)))
 	            $this->leads->add($this->parent_id);
 	    }
 
@@ -888,7 +888,7 @@ 
         sugar_cache_put($cacheKeyName, $apiList);
     }
 
-	if(!empty($value) && empty($apiList[$value]))
+	if (!empty($value) && empty($apiList[$value]))
 	{
 		$apiList[$value] = $value;
     }