adamjakab / SuiteCRM

modules/Prospects/metadata/searchdefs.php

Spacing +2 added lines, -2 removed lines

@@ -46,7 +46,7 @@ 
                        ),
                 'layout' => array(
   					'basic_search' => array(
- 							array('name'=>'search_name','label' =>'LBL_NAME', 'type' => 'name'),
+ 							array('name'=>'search_name', 'label' =>'LBL_NAME', 'type' => 'name'),
                            array('name'=>'current_user_only', 'label'=>'LBL_CURRENT_USER_FILTER', 'type'=>'bool'),
                            
 						),
@@ -60,7 +60,7 @@ 
 							array('name' => 'address_street', 'label'=>'LBL_ANY_ADDRESS', 'type' => 'name'),
 							array('name' => 'address_state', 'label' =>'LBL_STATE', 'type' => 'name'),
 							array('name' => 'address_postalcode', 'label' =>'LBL_POSTAL_CODE', 'type' => 'name'),
-							array('name' => 'primary_address_country', 'label' =>'LBL_COUNTRY', 'type' => 'name', 'options' => 'countries_dom', ), 
+							array('name' => 'primary_address_country', 'label' =>'LBL_COUNTRY', 'type' => 'name', 'options' => 'countries_dom',), 
 							array('name' => 'assigned_user_id', 'type' => 'enum', 'label' => 'LBL_ASSIGNED_TO', 'function' => array('name' => 'get_user_array', 'params' => array(false))),
 							
 					),

modules/Prospects/metadata/SearchFields.php

Spacing +17 added lines, -17 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.
@@ -39,12 +39,12 @@ 
  ********************************************************************************/
 
 $searchFields['Prospects'] = 
-	array (
-		'first_name' => array( 'query_type'=>'default'),
+	array(
+		'first_name' => array('query_type'=>'default'),
 		'last_name'=> array('query_type'=>'default'),
-		'search_name'=> array('query_type'=>'default','db_field'=>array('first_name','last_name'),'force_unifiedsearch'=>true),
+		'search_name'=> array('query_type'=>'default', 'db_field'=>array('first_name', 'last_name'), 'force_unifiedsearch'=>true),
 		'do_not_call'=> array('query_type'=>'default', 'operator'=>'='),
-		'phone'=> array('query_type'=>'default','db_field'=>array('phone_mobile','phone_work','phone_other','phone_fax','phone_home')),
+		'phone'=> array('query_type'=>'default', 'db_field'=>array('phone_mobile', 'phone_work', 'phone_other', 'phone_fax', 'phone_home')),
 		'email'=> array(
 			'query_type' => 'default',
 			'operator' => 'subquery',
@@ -54,20 +54,20 @@ 
 			),
 		),
 		'assistant'=> array('query_type'=>'default'),
-		'address_street'=> array('query_type'=>'default','db_field'=>array('primary_address_street','alt_address_street')),
-		'address_city'=> array('query_type'=>'default','db_field'=>array('primary_address_city','alt_address_city')),
-		'address_state'=> array('query_type'=>'default','db_field'=>array('primary_address_state','alt_address_state')),
-		'address_postalcode'=> array('query_type'=>'default','db_field'=>array('primary_address_postalcode','alt_address_postalcode')),
-		'address_country'=> array('query_type'=>'default','db_field'=>array('primary_address_country','alt_address_country')),
-		'current_user_only'=> array('query_type'=>'default','db_field'=>array('assigned_user_id'),'my_items'=>true, 'vname' => 'LBL_CURRENT_USER_FILTER', 'type' => 'bool'),
+		'address_street'=> array('query_type'=>'default', 'db_field'=>array('primary_address_street', 'alt_address_street')),
+		'address_city'=> array('query_type'=>'default', 'db_field'=>array('primary_address_city', 'alt_address_city')),
+		'address_state'=> array('query_type'=>'default', 'db_field'=>array('primary_address_state', 'alt_address_state')),
+		'address_postalcode'=> array('query_type'=>'default', 'db_field'=>array('primary_address_postalcode', 'alt_address_postalcode')),
+		'address_country'=> array('query_type'=>'default', 'db_field'=>array('primary_address_country', 'alt_address_country')),
+		'current_user_only'=> array('query_type'=>'default', 'db_field'=>array('assigned_user_id'), 'my_items'=>true, 'vname' => 'LBL_CURRENT_USER_FILTER', 'type' => 'bool'),
 		'assigned_user_id'=> array('query_type'=>'default'),
 	   //Range Search Support 
-	   'range_date_entered' => array ('query_type' => 'default', 'enable_range_search' => true, 'is_date_field' => true),
-	   'start_range_date_entered' => array ('query_type' => 'default', 'enable_range_search' => true, 'is_date_field' => true),
-	   'end_range_date_entered' => array ('query_type' => 'default', 'enable_range_search' => true, 'is_date_field' => true),
-	   'range_date_modified' => array ('query_type' => 'default', 'enable_range_search' => true, 'is_date_field' => true),
-	   'start_range_date_modified' => array ('query_type' => 'default', 'enable_range_search' => true, 'is_date_field' => true),
-       'end_range_date_modified' => array ('query_type' => 'default', 'enable_range_search' => true, 'is_date_field' => true),	
+	   'range_date_entered' => array('query_type' => 'default', 'enable_range_search' => true, 'is_date_field' => true),
+	   'start_range_date_entered' => array('query_type' => 'default', 'enable_range_search' => true, 'is_date_field' => true),
+	   'end_range_date_entered' => array('query_type' => 'default', 'enable_range_search' => true, 'is_date_field' => true),
+	   'range_date_modified' => array('query_type' => 'default', 'enable_range_search' => true, 'is_date_field' => true),
+	   'start_range_date_modified' => array('query_type' => 'default', 'enable_range_search' => true, 'is_date_field' => true),
+       'end_range_date_modified' => array('query_type' => 'default', 'enable_range_search' => true, 'is_date_field' => true),	
 	   //Range Search Support
 	);
 ?>

modules/Prospects/metadata/detailviewdefs.php

Spacing +42 added lines, -42 removed lines

@@ -38,27 +38,27 @@ 
  ********************************************************************************/
 
 $viewdefs ['Prospects'] = 
-array (
+array(
   'DetailView' => 
-  array (
+  array(
     'templateMeta' => 
-    array (
+    array(
       'form' => 
-      array (
+      array(
         'buttons' => 
-        array (
+        array(
           0 => 'EDIT',
           1 => 'DUPLICATE',
           2 => 'DELETE',
           3 => 
-          array (
+          array(
             'customCode' => '<input title="{$MOD.LBL_CONVERT_BUTTON_TITLE}" class="button" onclick="this.form.return_module.value=\'Prospects\'; this.form.return_action.value=\'DetailView\'; this.form.return_id.value=\'{$fields.id.value}\';this.form.module.value=\'Leads\';this.form.action.value=\'EditView\';" type="submit" name="CONVERT_LEAD_BTN" value="{$MOD.LBL_CONVERT_BUTTON_LABEL}"/>',
             'sugar_html' => 
-            array (
+            array(
               'type' => 'submit',
               'value' => '{$MOD.LBL_CONVERT_BUTTON_LABEL}',
               'htmlOptions' => 
-              array (
+              array(
                 'class' => 'button',
                 'name' => 'CONVERT_LEAD_BTN',
                 'id' => 'convert_target_button',
@@ -68,14 +68,14 @@ 
             ),
           ),
           4 => 
-          array (
+          array(
             'customCode' => '<input title="{$APP.LBL_MANAGE_SUBSCRIPTIONS}" class="button" onclick="this.form.return_module.value=\'Prospects\'; this.form.return_action.value=\'DetailView\'; this.form.return_id.value=\'{$fields.id.value}\'; this.form.action.value=\'Subscriptions\'; this.form.module.value=\'Campaigns\';" type="submit" name="Manage Subscriptions" value="{$APP.LBL_MANAGE_SUBSCRIPTIONS}"/>',
             'sugar_html' => 
-            array (
+            array(
               'type' => 'submit',
               'value' => '{$APP.LBL_MANAGE_SUBSCRIPTIONS}',
               'htmlOptions' => 
-              array (
+              array(
                 'class' => 'button',
                 'id' => 'manage_subscriptions_button',
                 'name' => 'Manage Subscriptions',
@@ -86,134 +86,134 @@ 
           ),
         ),
         'hidden' => 
-        array (
+        array(
           0 => '<input type="hidden" name="prospect_id" value="{$fields.id.value}">',
         ),
         'headerTpl' => 'modules/Prospects/tpls/DetailViewHeader.tpl',
       ),
       'maxColumns' => '2',
       'widths' => 
-      array (
+      array(
         0 => 
-        array (
+        array(
           'label' => '10',
           'field' => '30',
         ),
         1 => 
-        array (
+        array(
           'label' => '10',
           'field' => '30',
         ),
       ),
       'useTabs' => true,
       'tabDefs' => 
-      array (
+      array(
         'LBL_PROSPECT_INFORMATION' => 
-        array (
+        array(
           'newTab' => true,
           'panelDefault' => 'expanded',
         ),
         'LBL_MORE_INFORMATION' => 
-        array (
+        array(
           'newTab' => true,
           'panelDefault' => 'expanded',
         ),
         'LBL_PANEL_ASSIGNMENT' => 
-        array (
+        array(
           'newTab' => true,
           'panelDefault' => 'expanded',
         ),
       ),
     ),
     'panels' => 
-    array (
+    array(
       'lbl_prospect_information' => 
-      array (
+      array(
         0 => 
-        array (
+        array(
           0 => 
-          array (
+          array(
             'name' => 'full_name',
           ),
         ),
         1 => 
-        array (
+        array(
           0 => 'title',
           1 => 
-          array (
+          array(
             'name' => 'phone_work',
             'label' => 'LBL_OFFICE_PHONE',
           ),
         ),
         2 => 
-        array (
+        array(
           0 => 'department',
           1 => 'phone_mobile',
         ),
         3 => 
-        array (
+        array(
           0 => 
-          array (
+          array(
             'name' => 'account_name',
           ),
           1 => 'phone_fax',
         ),
         4 => 
-        array (
+        array(
           0 => 
-          array (
+          array(
             'name' => 'primary_address_street',
             'label' => 'LBL_PRIMARY_ADDRESS',
             'type' => 'address',
             'displayParams' => 
-            array (
+            array(
               'key' => 'primary',
             ),
           ),
           1 => 
-          array (
+          array(
             'name' => 'alt_address_street',
             'label' => 'LBL_ALTERNATE_ADDRESS',
             'type' => 'address',
             'displayParams' => 
-            array (
+            array(
               'key' => 'alt',
             ),
           ),
         ),
         5 => 
-        array (
+        array(
           0 => 'email1',
         ),
         6 => 
-        array (
+        array(
           0 => 'description',
         ),
         7 => 
-        array (
+        array(
           0 => 'assigned_user_name',
         ),
       ),
       'LBL_MORE_INFORMATION' => 
-      array (
+      array(
         0 => 
-        array (
+        array(
           0 => 'email_opt_out',
           1 => 'do_not_call',
         ),
       ),
       'LBL_PANEL_ASSIGNMENT' => 
-      array (
+      array(
         0 => 
-        array (
+        array(
           0 => 
-          array (
+          array(
             'name' => 'modified_by_name',
             'customCode' => '{$fields.date_modified.value} {$APP.LBL_BY} {$fields.modified_by_name.value}&nbsp;',
             'label' => 'LBL_DATE_MODIFIED',
           ),
           1 =>
-          array (
+          array(
             'name' => 'created_by_name',
             'customCode' => '{$fields.date_entered.value} {$APP.LBL_BY} {$fields.created_by_name.value}&nbsp;',
             'label' => 'LBL_DATE_ENTERED',

modules/Prospects/metadata/quickcreatedefs.php

Spacing +34 added lines, -34 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.
@@ -45,116 +45,116 @@ 
  * Contributor(s): ______________________________________..
  ********************************************************************************/
 
-$viewdefs = array (
+$viewdefs = array(
   'Prospects' => 
-  array (
+  array(
     'QuickCreate' => 
-    array (
+    array(
       'templateMeta' => 
-      array (
+      array(
         'maxColumns' => '2',
         'widths' => 
-        array (
+        array(
           0 => 
-          array (
+          array(
             'label' => '10',
             'field' => '30',
           ),
           1 => 
-          array (
+          array(
             'label' => '10',
             'field' => '30',
           ),
         ),
       ),
       'panels' => 
-      array (
+      array(
         'LBL_PROSPECT_INFORMATION' => 
-        array (
+        array(
           0 => 
-          array (
+          array(
             0 => 
-            array (
+            array(
               'name' => 'first_name',
             ),
             1 => 
-            array (
+            array(
               'name' => 'phone_work',
             ),
           ),
           1 => 
-          array (
+          array(
             0 => 
-            array (
+            array(
               'name' => 'last_name',
               'displayParams'=>array('required'=>true)
             ),
             1 => 
-            array (
+            array(
               'name' => 'phone_mobile',
             ),
           ),
           2 => 
-          array (
+          array(
             0 => 
-            array (
+            array(
               'name' => 'account_name',
             ),
             1 => 
-            array (
+            array(
               'name' => 'phone_fax',
             ),
           ),
           3 => 
-          array (
+          array(
             0 => 
-            array (
+            array(
               'name' => 'title',
             ),
             1 => 
-            array (
+            array(
               'name' => 'department',
             ),
           ),
           4 => 
-          array (
+          array(
             0 => 
-            array (
+            array(
               'name' => 'team_name',
             ),
             1 => 
-            array (
+            array(
               'name' => 'do_not_call',
             ),
           ),
           5 => 
-          array (
+          array(
             0 => 
-            array (
+            array(
               'name' => 'assigned_user_name',
             ),
           ),
         ),
         'lbl_email_addresses' => 
-        array (
+        array(
           0 => 
-          array (
+          array(
             0 => 
-            array (
+            array(
               'name' => 'email1',
             ),
           ),
         ),
         'LBL_ADDRESS_INFORMATION' => 
-        array (
+        array(
           0 => 
-          array (
+          array(
             0 => 
-            array (
+            array(
               'name' => 'primary_address_street',
             ),
             1 => 
-            array (
+            array(
               'name' => 'alt_address_street',
             ),
           ),

modules/Reminders/vardefs.php

Spacing +5 added lines, -5 removed lines

@@ -37,9 +37,9 @@ 
  * display the words  "Powered by SugarCRM" and "Supercharged by SuiteCRM".
  ********************************************************************************/
 
-$dictionary['Reminder']['table']= 'reminders';
-$dictionary['Reminder']['audited']= false;
-$dictionary['Reminder']['fields']= array(
+$dictionary['Reminder']['table'] = 'reminders';
+$dictionary['Reminder']['audited'] = false;
+$dictionary['Reminder']['fields'] = array(
     'popup' => array(
         'name' => 'popup',
         'vname' => 'LBL_POPUP',
@@ -102,9 +102,9 @@ 
 );
 
 
-if (!class_exists('VardefManager')){
+if (!class_exists('VardefManager')) {
     require_once('include/SugarObjects/VardefManager.php');
 }
-VardefManager::createVardef('Reminders','Reminder', array('basic','assignable'));
+VardefManager::createVardef('Reminders', 'Reminder', array('basic', 'assignable'));
 
 ?>
\ No newline at end of file

modules/Reminders/Reminder.php

Spacing +64 added lines, -64 removed lines

@@ -69,8 +69,8 @@ 
         parent::Basic();
     }
 
-    public function bean_implements($interface){
-        switch($interface){
+    public function bean_implements($interface) {
+        switch ($interface) {
             case 'ACL': return true;
         }
         return false;
@@ -89,7 +89,7 @@ 
 	 */
     public static function saveRemindersDataJson($eventModule, $eventModuleId, $remindersDataJson) {
         $reminderData = json_decode($remindersDataJson);
-        if(!json_last_error()) {
+        if (!json_last_error()) {
             Reminder::saveRemindersData($eventModule, $eventModuleId, $reminderData);
         }
         else {
@@ -99,8 +99,8 @@ 
 	
     private static function saveRemindersData($eventModule, $eventModuleId, $remindersData) {
         $savedReminderIds = array();
-        foreach($remindersData as $reminderData) {
-			if(isset($_POST['isDuplicate']) && $_POST['isDuplicate']) $reminderData->id = '';
+        foreach ($remindersData as $reminderData) {
+			if (isset($_POST['isDuplicate']) && $_POST['isDuplicate']) $reminderData->id = '';
             $reminderBean = BeanFactory::getBean('Reminders', $reminderData->id);
             $reminderBean->popup = $reminderData->popup;
             $reminderBean->email = $reminderData->email;
@@ -114,7 +114,7 @@ 
             Reminder_Invitee::saveRemindersInviteesData($reminderId, $reminderData->invitees);
         }
         $reminders = BeanFactory::getBean('Reminders')->get_full_list("", "reminders.related_event_module = '$eventModule' AND reminders.related_event_module_id = '$eventModuleId'");
-        if($reminders) {
+        if ($reminders) {
             foreach ($reminders as $reminder) {
                 if (!in_array($reminder->id, $savedReminderIds)) {
                     Reminder_Invitee::deleteRemindersInviteesMultiple($reminder->id);
@@ -138,7 +138,7 @@ 
     public static function loadRemindersDataJson($eventModule, $eventModuleId, $isDuplicate = false) {
         $remindersData = self::loadRemindersData($eventModule, $eventModuleId, $isDuplicate);
         $remindersDataJson = json_encode($remindersData);
-        if(!$remindersDataJson && json_last_error()) {
+        if (!$remindersDataJson && json_last_error()) {
             throw new Exception(json_last_error_msg());
         }
         return $remindersDataJson;
@@ -154,7 +154,7 @@ 
 	 * @throws Exception
 	 */
 	public static function loadRemindersData($eventModule, $eventModuleId, $isDuplicate = false) {
-		if(!isset(self::$remindersData[$eventModule][$eventModuleId]) || !$eventModuleId || $isDuplicate) {
+		if (!isset(self::$remindersData[$eventModule][$eventModuleId]) || !$eventModuleId || $isDuplicate) {
 			$ret = array();
 			$reminders = BeanFactory::getBean('Reminders')->get_full_list("reminders.date_entered", "reminders.related_event_module = '$eventModule' AND reminders.related_event_module_id = '$eventModuleId'");
 			if ($reminders) {
@@ -185,11 +185,11 @@ 
 	 * @param boolean $checkDecline (optional) Send email if user accept status is not decline. Default is TRUE.
 	 */
 	public static function sendEmailReminders(EmailReminder $emailReminder, Administration $admin, $checkDecline = true) {
-        if($reminders = self::getUnsentEmailReminders()) {
-            foreach($reminders as $reminderId => $reminder) {
+        if ($reminders = self::getUnsentEmailReminders()) {
+            foreach ($reminders as $reminderId => $reminder) {
 				$recipients = self::getEmailReminderInviteesRecipients($reminderId, $checkDecline);
 				$eventBean = BeanFactory::getBean($reminder->related_event_module, $reminder->related_event_module_id);
-				if ( $emailReminder->sendReminders($eventBean, $admin, $recipients) ) {
+				if ($emailReminder->sendReminders($eventBean, $admin, $recipients)) {
 					$reminder->email_sent = 1;
 					$reminder->save();
 				}
@@ -203,15 +203,15 @@ 
 		$eventModule = $reminder->related_event_module;
 		$eventModuleId = $reminder->related_event_module_id;		
 		$event = BeanFactory::getBean($eventModule, $eventModuleId);
-		if(!isset($event->status) || $event->status != 'Held') {
+		if (!isset($event->status) || $event->status != 'Held') {
 			$invitees = BeanFactory::getBean('Reminders_Invitees')->get_full_list('', "reminders_invitees.reminder_id = '$reminderId'");
-			foreach($invitees as $invitee) {
+			foreach ($invitees as $invitee) {
 				$inviteeModule = $invitee->related_invitee_module;
 				$inviteeModuleId = $invitee->related_invitee_module_id;
 				$personBean = BeanFactory::getBean($inviteeModule, $inviteeModuleId);
 				// The original email reminders check the accept_status field in related users/leads/contacts etc. and filtered these users who not decline this event.
-				if($checkDecline && !self::isDecline($event, $personBean)) {
-					if ( !empty($personBean->email1) ) {
+				if ($checkDecline && !self::isDecline($event, $personBean)) {
+					if (!empty($personBean->email1)) {
 						$arr = array(
 							'type' => $inviteeModule,
 							'name' => $personBean->full_name,
@@ -229,14 +229,14 @@ 
         global $db;
 		$reminders = array();
 		$reminderBeans = BeanFactory::getBean('Reminders')->get_full_list('', "reminders.email = 1 AND reminders.email_sent = 0");
-		foreach($reminderBeans as $reminderBean) {
+		foreach ($reminderBeans as $reminderBean) {
 			$eventBean = BeanFactory::getBean($reminderBean->related_event_module, $reminderBean->related_event_module_id);
 			$dateStart = $eventBean->date_start;
-			$time = strtotime($db->fromConvert($dateStart,'datetime'));
+			$time = strtotime($db->fromConvert($dateStart, 'datetime'));
 			$dateStart = date(TimeDate::DB_DATETIME_FORMAT, $time);
-			$remind_ts = $GLOBALS['timedate']->fromDb($db->fromConvert($dateStart,'datetime'))->modify("-{$reminderBean->timer_email} seconds")->ts;
+			$remind_ts = $GLOBALS['timedate']->fromDb($db->fromConvert($dateStart, 'datetime'))->modify("-{$reminderBean->timer_email} seconds")->ts;
             $now_ts = $GLOBALS['timedate']->getNow()->ts;
-            if ( $now_ts >= $remind_ts ) {
+            if ($now_ts >= $remind_ts) {
                 $reminders[$reminderBean->id] = $reminderBean;
             }
 		}
@@ -281,8 +281,8 @@ 
 		///////////////////////////////////////////////////////////////////////
 		////	MEETING INTEGRATION
 		$meetingIntegration = null;
-		if(isset($sugar_config['meeting_integration']) && !empty($sugar_config['meeting_integration'])) {
-			if(!class_exists($sugar_config['meeting_integration'])) {
+		if (isset($sugar_config['meeting_integration']) && !empty($sugar_config['meeting_integration'])) {
+			if (!class_exists($sugar_config['meeting_integration'])) {
 				require_once("modules/{$sugar_config['meeting_integration']}/{$sugar_config['meeting_integration']}.php");
 			}
 			$meetingIntegration = new $sugar_config['meeting_integration']();
@@ -292,31 +292,31 @@ 
 		
 		$popupReminders = BeanFactory::getBean('Reminders')->get_full_list('', "reminders.popup = 1");
 
-		if($popupReminders) {
-			foreach($popupReminders as $popupReminder) {
+		if ($popupReminders) {
+			foreach ($popupReminders as $popupReminder) {
 				$relatedEvent = BeanFactory::getBean($popupReminder->related_event_module, $popupReminder->related_event_module_id);
-				if( $relatedEvent &&
+				if ($relatedEvent &&
 					(!isset($relatedEvent->status) || $relatedEvent->status == 'Planned') &&
-					(!isset($relatedEvent->date_start) || (strtotime($relatedEvent->date_start) >= strtotime(self::unQuoteTime($dateTimeNow)) && strtotime($relatedEvent->date_start) <= strtotime(self::unQuoteTime($dateTimeMax))) ) &&
+					(!isset($relatedEvent->date_start) || (strtotime($relatedEvent->date_start) >= strtotime(self::unQuoteTime($dateTimeNow)) && strtotime($relatedEvent->date_start) <= strtotime(self::unQuoteTime($dateTimeMax)))) &&
 					(!$checkDecline || ($checkDecline && !self::isDecline($relatedEvent, BeanFactory::getBean('Users', $current_user->id))))
 				) {
 					// The original popup/alert reminders check the accept_status field in related users/leads/contacts etc. and filtered these users who not decline this event.
 					$invitees = BeanFactory::getBean('Reminders_Invitees')->get_full_list('', "reminders_invitees.reminder_id = '{$popupReminder->id}' AND reminders_invitees.related_invitee_module_id = '{$current_user->id}'");
-					if($invitees) {
-						foreach($invitees as $invitee) {
+					if ($invitees) {
+						foreach ($invitees as $invitee) {
 							// need to concatenate since GMT times can bridge two local days
 							$timeStart = strtotime($db->fromConvert(isset($relatedEvent->date_start) ? $relatedEvent->date_start : date(TimeDate::DB_DATETIME_FORMAT), 'datetime'));
 							$timeRemind = $popupReminder->timer_popup;
 							$timeStart -= $timeRemind;
 
-							$url = 'index.php?action=DetailView&module=' . $popupReminder->related_event_module . '&record=' . $popupReminder->related_event_module_id;
+							$url = 'index.php?action=DetailView&module='.$popupReminder->related_event_module.'&record='.$popupReminder->related_event_module_id;
 							$instructions = $app_strings['MSG_JS_ALERT_MTG_REMINDER_MEETING_MSG'];
 
-							if($popupReminder->related_event_module == 'Meetings') {
+							if ($popupReminder->related_event_module == 'Meetings') {
 								///////////////////////////////////////////////////////////////////
 								////	MEETING INTEGRATION
-								if(!empty($meetingIntegration) && $meetingIntegration->isIntegratedMeeting($popupReminder->related_event_module_id)) {
-									$url = $meetingIntegration->miUrlGetJsAlert((array) $popupReminder);
+								if (!empty($meetingIntegration) && $meetingIntegration->isIntegratedMeeting($popupReminder->related_event_module_id)) {
+									$url = $meetingIntegration->miUrlGetJsAlert((array)$popupReminder);
 									$instructions = $meetingIntegration->miGetJsAlertInstructions();
 								}
 								////	END MEETING INTEGRATION
@@ -330,16 +330,16 @@ 
 							$relatedToMeeting = $alert->getRelatedName($popupReminder->related_event_module, $popupReminder->related_event_module_id);
 							
 							$description = empty($desc1) ? '' : $app_strings['MSG_JS_ALERT_MTG_REMINDER_AGENDA'].$desc1."\n";
-							$description = $description  ."\n" .$app_strings['MSG_JS_ALERT_MTG_REMINDER_STATUS'] . (isset($relatedEvent->status) ? $relatedEvent->status : '') ."\n". $app_strings['MSG_JS_ALERT_MTG_REMINDER_RELATED_TO']. $relatedToMeeting;
+							$description = $description."\n".$app_strings['MSG_JS_ALERT_MTG_REMINDER_STATUS'].(isset($relatedEvent->status) ? $relatedEvent->status : '')."\n".$app_strings['MSG_JS_ALERT_MTG_REMINDER_RELATED_TO'].$relatedToMeeting;
 
 
-							if(isset($relatedEvent->date_start)) {
+							if (isset($relatedEvent->date_start)) {
 								$time_dbFromConvert = $db->fromConvert($relatedEvent->date_start, 'datetime');
 								$time = $timedate->to_display_date_time($time_dbFromConvert);
-								if(!$time) {
+								if (!$time) {
 									$time = $relatedEvent->date_start;
 								}
-								if(!$time) {
+								if (!$time) {
 									$time = $app_strings['MSG_JS_ALERT_MTG_REMINDER_NO_START_DATE'];
 								}
 							}
@@ -365,8 +365,8 @@ 
 
 	private static function unQuoteTime($timestr) {
 		$ret = '';
-		for($i=0; $i<strlen($timestr); $i++) {
-			if($timestr[$i]!="'") $ret .= $timestr[$i];
+		for ($i = 0; $i < strlen($timestr); $i++) {
+			if ($timestr[$i] != "'") $ret .= $timestr[$i];
 		}
 		return $ret;
 	}
@@ -378,10 +378,10 @@ 
 	}
 	
 	private static function testEventPersonAcceptStatus(SugarBean $event, SugarBean $person, $acceptStatus = 'decline') {
-		if($acceptStats = self::getEventPersonAcceptStatus($event, $person)) {
+		if ($acceptStats = self::getEventPersonAcceptStatus($event, $person)) {
 			$acceptStatusLower = strtolower($acceptStatus);
-			foreach((array) $acceptStats as $acceptStat) {
-				if(strtolower($acceptStat) == $acceptStatusLower) {
+			foreach ((array)$acceptStats as $acceptStat) {
+				if (strtolower($acceptStat) == $acceptStatusLower) {
 					return true;
 				}
 			}
@@ -393,12 +393,12 @@ 
 		global $db;
 		$rel_person_table_Key = "rel_{$person->table_name}_table";
 		$rel_person_table_Value = "{$event->table_name}_{$person->table_name}";
-		if(isset($event->$rel_person_table_Key) && $event->$rel_person_table_Key == $rel_person_table_Value) {
+		if (isset($event->$rel_person_table_Key) && $event->$rel_person_table_Key == $rel_person_table_Value) {
 			$query = self::getEventPersonQuery($event, $person);
 			$re = $db->query($query);
 			$ret = array();
-			while($row = $db->fetchByAssoc($re) ) {
-				if(!isset($row['accept_status'])) {
+			while ($row = $db->fetchByAssoc($re)) {
+				if (!isset($row['accept_status'])) {
 					return null;
 				}
 				$ret[] = $row['accept_status'];
@@ -421,10 +421,10 @@ 
 
 	private static function getEventPersonQuery(SugarBean $event, SugarBean $person) {
 		$eventIdField = array_search($event->table_name, $event->relationship_fields);
-		if(!$eventIdField) {
-			$eventIdField = strtolower($event->object_name . '_id');
+		if (!$eventIdField) {
+			$eventIdField = strtolower($event->object_name.'_id');
 		}
-		$personIdField = strtolower($person->object_name) . '_id';
+		$personIdField = strtolower($person->object_name).'_id';
 		$query = "
 			SELECT * FROM {$event->table_name}_{$person->table_name}
 			WHERE
@@ -444,7 +444,7 @@ 
 	 */
 	public static function loadRemindersDefaultValuesDataJson() {
 		$ret = json_encode(self::loadRemindersDefaultValuesData());
-		if(!$ret && json_last_error()) {
+		if (!$ret && json_last_error()) {
             throw new Exception(json_last_error_msg());
         }
 		return $ret;
@@ -484,14 +484,14 @@ 
 	}
 
 	private static function upgradeRestoreReminders() {
-		if($reminders = BeanFactory::getBean('Reminders')->get_full_list('', 'reminders.deleted = 1')) {
-			foreach($reminders as $reminder) {
+		if ($reminders = BeanFactory::getBean('Reminders')->get_full_list('', 'reminders.deleted = 1')) {
+			foreach ($reminders as $reminder) {
 				$reminder->deleted = 0;
 				$reminder->save();
 			}
 		}
-		if($reminderInvitees = BeanFactory::getBean('Reminders_Invitees')->get_full_list('', 'reminders_invitees.deleted = 1')) {
-			foreach($reminderInvitees as $invitee) {
+		if ($reminderInvitees = BeanFactory::getBean('Reminders_Invitees')->get_full_list('', 'reminders_invitees.deleted = 1')) {
+			foreach ($reminderInvitees as $invitee) {
 				$invitee->deleted = 0;
 				$invitee->save();
 			}
@@ -505,7 +505,7 @@ 
 
 	private static function upgradeUserPreferences() {
 		$users = User::getActiveUsers();
-		foreach($users as $user_id => $user_name) {
+		foreach ($users as $user_id => $user_name) {
 			$user = new User();
 			$user->retrieve($user_id);
 
@@ -546,7 +546,7 @@ 
 
 				$oldReminderEmailSent = $event->email_reminder_sent;
 
-				if( ($oldInvitees = self::getOldEventInvitees($event)) && ($event->reminder_time != -1 || ($event->email_reminder_time != -1 && $event->email_reminder_sent != 1)) ) {
+				if (($oldInvitees = self::getOldEventInvitees($event)) && ($event->reminder_time != -1 || ($event->email_reminder_time != -1 && $event->email_reminder_sent != 1))) {
 
 					self::migrateReminder(
 							$eventModule,
@@ -569,11 +569,11 @@ 
 	private static function getOldEventInvitees(SugarBean $event) {
 		global $db;
 		$ret = array();
-		$persons = array('users','contacts','leads');
-		foreach($persons as $person){
+		$persons = array('users', 'contacts', 'leads');
+		foreach ($persons as $person) {
 			$query = self::upgradeEventPersonQuery($event, $person);
 			$re = $db->query($query);
-			while($row = $db->fetchByAssoc($re) ) {
+			while ($row = $db->fetchByAssoc($re)) {
 				$ret[] = $row;
 			}
 		}
@@ -608,7 +608,7 @@ 
 
 	private static function migrateReminderInvitees($reminderId, $invitees) {
 		$ret = array();
-		foreach((array) $invitees as $invitee) {
+		foreach ((array)$invitees as $invitee) {
 			$newInvitee = BeanFactory::getBean('Reminders_Invitees');
 			$newInvitee->reminder_id = $reminderId;
 			$newInvitee->related_invitee_module = self::getRelatedInviteeModuleFromInviteeArray($invitee);
@@ -619,13 +619,13 @@ 
 	}
 
 	private static function getRelatedInviteeModuleFromInviteeArray($invitee) {
-		if(array_key_exists('user_id', $invitee)) {
+		if (array_key_exists('user_id', $invitee)) {
 			return 'Users';
 		}
-		if(array_key_exists('lead_id', $invitee)) {
+		if (array_key_exists('lead_id', $invitee)) {
 			return 'Leads';
 		}
-		if(array_key_exists('contact_id', $invitee)) {
+		if (array_key_exists('contact_id', $invitee)) {
 			return 'Contacts';
 		}
 		// TODO:!!!!
@@ -634,13 +634,13 @@ 
 	}
 
 	private static function getRelatedInviteeModuleIdFromInviteeArray($invitee) {
-		if(array_key_exists('user_id', $invitee)) {
+		if (array_key_exists('user_id', $invitee)) {
 			return $invitee['user_id'];
 		}
-		if(array_key_exists('lead_id', $invitee)) {
+		if (array_key_exists('lead_id', $invitee)) {
 			return $invitee['lead_id'];
 		}
-		if(array_key_exists('contact_id', $invitee)) {
+		if (array_key_exists('contact_id', $invitee)) {
 			return $invitee['contact_id'];
 		}
 		// TODO:!!!!
@@ -685,7 +685,7 @@ 
 	 */
 	public static function getRemindersListInlineEditView(SugarBean $event) {
 		// TODO: getEditFieldHTML() function in InlineEditing.php:218 doesn't pass the Bean ID to this custom inline edit view function but we have to know which Bean are in the focus to editing.
-		if(!$event->id) {
+		if (!$event->id) {
 			throw new Exception("No GUID for edit.");
 		}
 	}

modules/Documents/GetLatestRevision.php

Spacing +3 added lines, -3 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,8 +47,8 @@ 
 
 	$document = new Document();
 	$document->retrieve($_REQUEST['record']);
-	if (!empty($document->document_revision_id) && !empty($_REQUEST['get_latest_for_id']))  {
-		$query="update linked_documents set document_revision_id='{$document->document_revision_id}', date_modified='".TimeDate::getInstance()->nowDb()."' where id ='{$_REQUEST['get_latest_for_id']}' ";
+	if (!empty($document->document_revision_id) && !empty($_REQUEST['get_latest_for_id'])) {
+		$query = "update linked_documents set document_revision_id='{$document->document_revision_id}', date_modified='".TimeDate::getInstance()->nowDb()."' where id ='{$_REQUEST['get_latest_for_id']}' ";
 		$document->db->query($query);
 	}	
 }

modules/Documents/DocumentExternalApiDropDown.php

Spacing +3 added lines, -3 removed lines

@@ -1,5 +1,5 @@ 
 <?php
-if(!defined('sugarEntry') || !sugarEntry)
+if (!defined('sugarEntry') || !sugarEntry)
 	die('Not A Valid Entry Point');
 /*********************************************************************************
  * SugarCRM Community Edition is a customer relationship management program developed by
@@ -44,8 +44,8 @@ 
 
     $apiList = ExternalAPIFactory::getModuleDropDown('Documents');
 
-    $apiList = array_merge(array('Sugar'=>$GLOBALS['app_list_strings']['eapm_list']['Sugar']),$apiList);
-    if(!empty($value) && empty($apiList[$value])){
+    $apiList = array_merge(array('Sugar'=>$GLOBALS['app_list_strings']['eapm_list']['Sugar']), $apiList);
+    if (!empty($value) && empty($apiList[$value])) {
         $apiList[$value] = $value;
     }
     return $apiList;

modules/Documents/Document.php

Spacing +35 added lines, -35 removed lines

@@ -1,5 +1,5 @@ 
 <?php
-if(!defined('sugarEntry') || !sugarEntry)
+if (!defined('sugarEntry') || !sugarEntry)
 	die('Not A Valid Entry Point');
 /*********************************************************************************
  * SugarCRM Community Edition is a customer relationship management program developed by
@@ -84,10 +84,10 @@ 
 	var $object_name = "Document";
 	var $user_preferences;
 
-	var $encodeFields = Array ();
+	var $encodeFields = Array();
 
 	// This is used to retrieve related fields from form posts.
-	var $additional_column_fields = Array ('revision');
+	var $additional_column_fields = Array('revision');
 
 	var $new_schema = true;
 	var $module_dir = 'Documents';
@@ -115,7 +115,7 @@ 
                 $this->new_with_id = true;
             }
 
-            if ( isset($_REQUEST) && isset($_REQUEST['duplicateSave']) && $_REQUEST['duplicateSave'] == true && isset($_REQUEST['filename_old_doctype']) ) {
+            if (isset($_REQUEST) && isset($_REQUEST['duplicateSave']) && $_REQUEST['duplicateSave'] == true && isset($_REQUEST['filename_old_doctype'])) {
                 $this->doc_type = $_REQUEST['filename_old_doctype'];
                 $isDuplicate = true;
             } else {
@@ -128,26 +128,26 @@ 
             $Revision->not_use_rel_in_req = true;
             $Revision->new_rel_id = $this->id;
             $Revision->new_rel_relname = 'Documents';
-            $Revision->change_log = translate('DEF_CREATE_LOG','Documents');
+            $Revision->change_log = translate('DEF_CREATE_LOG', 'Documents');
             $Revision->revision = $this->revision;
             $Revision->document_id = $this->id;
             $Revision->filename = $this->filename;
 
-            if(isset($this->file_ext))
+            if (isset($this->file_ext))
             {
             	$Revision->file_ext = $this->file_ext;
             }
 
-            if(isset($this->file_mime_type))
+            if (isset($this->file_mime_type))
             {
             	$Revision->file_mime_type = $this->file_mime_type;
             }
 
             $Revision->doc_type = $this->doc_type;
-            if ( isset($this->doc_id) ) {
+            if (isset($this->doc_id)) {
                 $Revision->doc_id = $this->doc_id;
             }
-            if ( isset($this->doc_url) ) {
+            if (isset($this->doc_url)) {
                 $Revision->doc_url = $this->doc_url;
             }
 
@@ -159,7 +159,7 @@ 
             if (!empty($_FILES['filename_file'])) {
                 rename("upload://{$this->id}", "upload://{$Revision->id}");
                 $createRevision = true;
-            } else if ( $isDuplicate && ( empty($this->doc_type) || $this->doc_type == 'Sugar' ) ) {
+            } else if ($isDuplicate && (empty($this->doc_type) || $this->doc_type == 'Sugar')) {
                 // Looks like we need to duplicate a file, this is tricky
                 $oldDocument = new Document();
                 $oldDocument->retrieve($_REQUEST['duplicateId']);
@@ -171,27 +171,27 @@ 
             }
 
             // For external documents, we just need to make sure we have a doc_id
-            if ( !empty($this->doc_id) && $this->doc_type != 'Sugar' ) {
+            if (!empty($this->doc_id) && $this->doc_type != 'Sugar') {
                 $createRevision = true;
             }
 
-            if ( $createRevision ) {
+            if ($createRevision) {
                 $Revision->save();
                 //update document with latest revision id
-                $this->process_save_dates=false; //make sure that conversion does not happen again.
+                $this->process_save_dates = false; //make sure that conversion does not happen again.
                 $this->document_revision_id = $Revision->id;
             }
 
 
             //set relationship field values if contract_id is passed (via subpanel create)
             if (!empty($_POST['contract_id'])) {
-                $save_revision['document_revision_id']=$this->document_revision_id;
+                $save_revision['document_revision_id'] = $this->document_revision_id;
                 $this->load_relationship('contracts');
-                $this->contracts->add($_POST['contract_id'],$save_revision);
+                $this->contracts->add($_POST['contract_id'], $save_revision);
             }
 
             if ((isset($_POST['load_signed_id']) and !empty($_POST['load_signed_id']))) {
-                $query="update linked_documents set deleted=1 where id='".$_POST['load_signed_id']."'";
+                $query = "update linked_documents set deleted=1 where id='".$_POST['load_signed_id']."'";
                 $this->db->query($query);
             }
         }
@@ -232,14 +232,14 @@ 
             $row = $this->db->fetchByAssoc($result);
 
             //populate name
-            if(isset($this->document_name))
+            if (isset($this->document_name))
             {
             	$this->name = $this->document_name;
             }
 
-            if(isset($row['filename']))$this->filename = $row['filename'];
+            if (isset($row['filename']))$this->filename = $row['filename'];
             //$this->latest_revision = $row['revision'];
-            if(isset($row['revision']))$this->revision = $row['revision'];
+            if (isset($row['revision']))$this->revision = $row['revision'];
 
             //image is selected based on the extension name <ext>_icon_inline, extension is stored in document_revisions.
             //if file is not found then default image file will be used.
@@ -257,16 +257,16 @@ 
 		} else {
 			$img_name = "def_image_inline"; //todo change the default image.
 		}
-		if($this->ACLAccess('DetailView')) {
-			if(!empty($this->doc_type) && $this->doc_type != 'Sugar' && !empty($this->doc_url)) {
-                $file_url= "<a href='".$this->doc_url."' target='_blank'>".SugarThemeRegistry::current()->getImage($this->doc_type.'_image_inline', 'border="0"',null,null,'.png',$mod_strings['LBL_LIST_VIEW_DOCUMENT'])."</a>";
+		if ($this->ACLAccess('DetailView')) {
+			if (!empty($this->doc_type) && $this->doc_type != 'Sugar' && !empty($this->doc_url)) {
+                $file_url = "<a href='".$this->doc_url."' target='_blank'>".SugarThemeRegistry::current()->getImage($this->doc_type.'_image_inline', 'border="0"', null, null, '.png', $mod_strings['LBL_LIST_VIEW_DOCUMENT'])."</a>";
 			} else {
-			    $file_url = "<a href='index.php?entryPoint=download&id={$this->document_revision_id}&type=Documents' target='_blank'>".SugarThemeRegistry::current()->getImage($img_name, 'border="0"', null,null,'.gif',$mod_strings['LBL_LIST_VIEW_DOCUMENT'])."</a>";
+			    $file_url = "<a href='index.php?entryPoint=download&id={$this->document_revision_id}&type=Documents' target='_blank'>".SugarThemeRegistry::current()->getImage($img_name, 'border="0"', null, null, '.gif', $mod_strings['LBL_LIST_VIEW_DOCUMENT'])."</a>";
 			}
 
     		$this->file_url = $file_url;
     		$this->file_url_noimage = "index.php?entryPoint=download&type=Documents&id={$this->document_revision_id}";
-		}else{
+		} else {
             $this->file_url = "";
             $this->file_url_noimage = "";
 		}
@@ -280,7 +280,7 @@ 
 		}
 
 		global $app_list_strings;
-	    if(!empty($this->status_id)) {
+	    if (!empty($this->status_id)) {
 	       //_pp($this->status_id);
 	       $this->status = $app_list_strings['document_status_dom'][$this->status_id];
 	    }
@@ -294,15 +294,15 @@ 
 		return $list_form;
 	}
 
-    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;
 		$query = "SELECT
 						documents.*";
-        $query .=  $custom_join['select'];
+        $query .= $custom_join['select'];
 		$query .= " FROM documents ";
-        $query .=  $custom_join['join'];
+        $query .= $custom_join['join'];
 
 		$where_auto = " documents.deleted = 0";
 
@@ -351,11 +351,11 @@ 
 	function mark_relationships_deleted($id)
     {
         $this->load_relationships('revisions');
-       	$revisions= $this->get_linked_beans('revisions','DocumentRevision');
+       	$revisions = $this->get_linked_beans('revisions', 'DocumentRevision');
 
        	if (!empty($revisions) && is_array($revisions)) {
-       		foreach($revisions as $key=>$version) {
-       			UploadFile::unlink_file($version->id,$version->filename);
+       		foreach ($revisions as $key=>$version) {
+       			UploadFile::unlink_file($version->id, $version->filename);
        			//mark the version deleted.
        			$version->mark_deleted($version->id);
        		}
@@ -373,14 +373,14 @@ 
 	}
 
 	//static function.
-	function get_document_name($doc_id){
+	function get_document_name($doc_id) {
 		if (empty($doc_id)) return null;
 
 		$db = DBManagerFactory::getInstance();
-		$query="select document_name from documents where id='$doc_id'  and deleted=0";
-		$result=$db->query($query);
+		$query = "select document_name from documents where id='$doc_id'  and deleted=0";
+		$result = $db->query($query);
 		if (!empty($result)) {
-			$row=$db->fetchByAssoc($result);
+			$row = $db->fetchByAssoc($result);
 			if (!empty($row)) {
 				return $row['document_name'];
 			}