adamjakab / SuiteCRM

modules/Project/ProjectJjwg_MapsLogicHook.php

Braces +2 added lines, -1 removed lines

@@ -1,6 +1,7 @@
 <?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
  * SugarCRM, Inc. Copyright (C) 2004-2013 SugarCRM Inc.

modules/Opportunities/OpportunitiesJjwg_MapsLogicHook.php

Braces +2 added lines, -1 removed lines

@@ -1,6 +1,7 @@
 <?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
  * SugarCRM, Inc. Copyright (C) 2004-2013 SugarCRM Inc.

modules/Cases/CasesJjwg_MapsLogicHook.php

Braces +2 added lines, -1 removed lines

@@ -1,6 +1,7 @@
 <?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
  * SugarCRM, Inc. Copyright (C) 2004-2013 SugarCRM Inc.

modules/Accounts/AccountsJjwg_MapsLogicHook.php

Braces +2 added lines, -1 removed lines

@@ -1,6 +1,7 @@
 <?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
  * SugarCRM, Inc. Copyright (C) 2004-2013 SugarCRM Inc.

modules/AOR_Charts/AOR_Chart.php

Indentation +21 added lines, -21 removed lines

@@ -25,25 +25,25 @@ 
 class AOR_Chart extends Basic {
 
     var $colours = "['#1f78b4','#a6cee3','#b2df8a','#33a02c','#fb9a99','#e31a1c','#fdbf6f','#ff7f00','#cab2d6','#6a3d9a','#ffff99','#b15928']";
-	var $new_schema = true;
-	var $module_dir = 'AOR_Charts';
-	var $object_name = 'AOR_Chart';
-	var $table_name = 'aor_charts';
-	var $importable = true;
-	var $disable_row_level_security = true ;
+    var $new_schema = true;
+    var $module_dir = 'AOR_Charts';
+    var $object_name = 'AOR_Chart';
+    var $table_name = 'aor_charts';
+    var $importable = true;
+    var $disable_row_level_security = true ;
 	
-	var $id;
-	var $name;
-	var $date_entered;
-	var $date_modified;
-	var $modified_user_id;
-	var $modified_by_name;
-	var $created_by;
-	var $created_by_name;
-	var $description;
-	var $deleted;
-	var $created_by_link;
-	var $modified_user_link;
+    var $id;
+    var $name;
+    var $date_entered;
+    var $date_modified;
+    var $modified_user_id;
+    var $modified_by_name;
+    var $created_by;
+    var $created_by_name;
+    var $description;
+    var $deleted;
+    var $created_by_link;
+    var $modified_user_link;
 
     var $type;
     var $x_field;
@@ -52,9 +52,9 @@ 
 
 
 
-	function AOR_Chart(){
-		parent::Basic();
-	}
+    function AOR_Chart(){
+        parent::Basic();
+    }
 
     function save_lines(array $post,AOR_Report $bean,$postKey){
         $seenIds = array();

modules/AOR_Reports/controller.php

Indentation +21 added lines, -21 removed lines

@@ -1,26 +1,26 @@
 <?php
 /**
- * Advanced OpenReports, SugarCRM Reporting.
- * @package Advanced OpenReports for SugarCRM
- * @copyright SalesAgility Ltd http://www.salesagility.com
- *
- * This program is free software; you can redistribute it and/or modify
- * it under the terms of the GNU AFFERO GENERAL PUBLIC LICENSE as published by
- * the Free Software Foundation; either version 3 of the License, or
- * (at your option) any later version.
- *
- * This program is distributed in the hope that it will be useful,
- * but WITHOUT ANY WARRANTY; without even the implied warranty of
- * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
- * GNU General Public License for more details.
- *
- * You should have received a copy of the GNU AFFERO GENERAL PUBLIC LICENSE
- * along with this program; if not, see http://www.gnu.org/licenses
- * or write to the Free Software Foundation,Inc., 51 Franklin Street,
- * Fifth Floor, Boston, MA 02110-1301  USA
- *
- * @author SalesAgility <info@salesagility.com>
- */
+     * Advanced OpenReports, SugarCRM Reporting.
+     * @package Advanced OpenReports for SugarCRM
+     * @copyright SalesAgility Ltd http://www.salesagility.com
+     *
+     * This program is free software; you can redistribute it and/or modify
+     * it under the terms of the GNU AFFERO GENERAL PUBLIC LICENSE as published by
+     * the Free Software Foundation; either version 3 of the License, or
+     * (at your option) any later version.
+     *
+     * This program is distributed in the hope that it will be useful,
+     * but WITHOUT ANY WARRANTY; without even the implied warranty of
+     * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+     * GNU General Public License for more details.
+     *
+     * You should have received a copy of the GNU AFFERO GENERAL PUBLIC LICENSE
+     * along with this program; if not, see http://www.gnu.org/licenses
+     * or write to the Free Software Foundation,Inc., 51 Franklin Street,
+     * Fifth Floor, Boston, MA 02110-1301  USA
+     *
+     * @author SalesAgility <info@salesagility.com>
+     */
 
 /**
  * This file adds support for studio

modules/AOR_Reports/language/en_us.lang.php

Indentation +58 added lines, -58 removed lines

@@ -23,66 +23,66 @@
  */
 
 $mod_strings = array (
-  'LBL_ASSIGNED_TO_ID' => 'Assigned User Id',
-  'LBL_ASSIGNED_TO_NAME' => 'Assigned to',
-  'LBL_ID' => 'ID',
-  'LBL_DATE_ENTERED' => 'Date Created',
-  'LBL_DATE_MODIFIED' => 'Date Modified',
-  'LBL_MODIFIED' => 'Modified By',
-  'LBL_MODIFIED_ID' => 'Modified By Id',
-  'LBL_MODIFIED_NAME' => 'Modified By Name',
-  'LBL_CREATED' => 'Created By',
-  'LBL_CREATED_ID' => 'Created By Id',
-  'LBL_DESCRIPTION' => 'Description',
-  'LBL_DELETED' => 'Deleted',
-  'LBL_NAME' => 'Name',
-  'LBL_CREATED_USER' => 'Created by User',
-  'LBL_MODIFIED_USER' => 'Modified by User',
-  'LBL_LIST_NAME' => 'Name',
-  'LBL_EDIT_BUTTON' => 'Edit',
-  'LBL_REMOVE' => 'Remove',
-  'LBL_LIST_FORM_TITLE' => 'Reports List',
-  'LBL_MODULE_NAME' => 'Reports',
-  'LBL_MODULE_TITLE' => 'Reports',
-  'LBL_HOMEPAGE_TITLE' => 'My Reports',
-  'LNK_NEW_RECORD' => 'Create Report',
-  'LNK_LIST' => 'View Reports',
-  'LNK_IMPORT_AOR_REPORTS' => 'Import Reports',
-  'LBL_SEARCH_FORM_TITLE' => 'Search Reports',
-  'LBL_HISTORY_SUBPANEL_TITLE' => 'View History',
-  'LBL_ACTIVITIES_SUBPANEL_TITLE' => 'Activities',
-  'LBL_AOR_REPORTS_SUBPANEL_TITLE' => 'Reports',
-  'LBL_NEW_FORM_TITLE' => 'New Reports',
-  'LBL_REPORT_MODULE' => 'Report Module',
+    'LBL_ASSIGNED_TO_ID' => 'Assigned User Id',
+    'LBL_ASSIGNED_TO_NAME' => 'Assigned to',
+    'LBL_ID' => 'ID',
+    'LBL_DATE_ENTERED' => 'Date Created',
+    'LBL_DATE_MODIFIED' => 'Date Modified',
+    'LBL_MODIFIED' => 'Modified By',
+    'LBL_MODIFIED_ID' => 'Modified By Id',
+    'LBL_MODIFIED_NAME' => 'Modified By Name',
+    'LBL_CREATED' => 'Created By',
+    'LBL_CREATED_ID' => 'Created By Id',
+    'LBL_DESCRIPTION' => 'Description',
+    'LBL_DELETED' => 'Deleted',
+    'LBL_NAME' => 'Name',
+    'LBL_CREATED_USER' => 'Created by User',
+    'LBL_MODIFIED_USER' => 'Modified by User',
+    'LBL_LIST_NAME' => 'Name',
+    'LBL_EDIT_BUTTON' => 'Edit',
+    'LBL_REMOVE' => 'Remove',
+    'LBL_LIST_FORM_TITLE' => 'Reports List',
+    'LBL_MODULE_NAME' => 'Reports',
+    'LBL_MODULE_TITLE' => 'Reports',
+    'LBL_HOMEPAGE_TITLE' => 'My Reports',
+    'LNK_NEW_RECORD' => 'Create Report',
+    'LNK_LIST' => 'View Reports',
+    'LNK_IMPORT_AOR_REPORTS' => 'Import Reports',
+    'LBL_SEARCH_FORM_TITLE' => 'Search Reports',
+    'LBL_HISTORY_SUBPANEL_TITLE' => 'View History',
+    'LBL_ACTIVITIES_SUBPANEL_TITLE' => 'Activities',
+    'LBL_AOR_REPORTS_SUBPANEL_TITLE' => 'Reports',
+    'LBL_NEW_FORM_TITLE' => 'New Reports',
+    'LBL_REPORT_MODULE' => 'Report Module',
     'LBL_GRAPHS_PER_ROW' => 'Charts per row',
-  'LBL_FIELD_LINES' => 'Display Fields',
-  'LBL_ADD_FIELD' => 'Add Field',
-  'LBL_CONDITION_LINES' => 'Conditions',
-  'LBL_ADD_CONDITION' => 'Add Condition',
-  'LBL_EXPORT' => 'Export',
-  'LBL_DOWNLOAD_PDF' => 'Download PDF',
-  'LBL_ADD_TO_PROSPECT_LIST' => 'Add To Target List',
-  'LBL_AOR_MODULETREE_SUBPANEL_TITLE' => 'Module tree',
-  'LBL_AOR_FIELDS_SUBPANEL_TITLE' => 'Fields',
-  'LBL_AOR_CONDITIONS_SUBPANEL_TITLE' => 'Conditions',
-  'LBL_TOTAL' => 'Total',
-  'LBL_AOR_CHARTS_SUBPANEL_TITLE' => 'Charts',
-  'LBL_ADD_CHART' => 'Add chart',
+    'LBL_FIELD_LINES' => 'Display Fields',
+    'LBL_ADD_FIELD' => 'Add Field',
+    'LBL_CONDITION_LINES' => 'Conditions',
+    'LBL_ADD_CONDITION' => 'Add Condition',
+    'LBL_EXPORT' => 'Export',
+    'LBL_DOWNLOAD_PDF' => 'Download PDF',
+    'LBL_ADD_TO_PROSPECT_LIST' => 'Add To Target List',
+    'LBL_AOR_MODULETREE_SUBPANEL_TITLE' => 'Module tree',
+    'LBL_AOR_FIELDS_SUBPANEL_TITLE' => 'Fields',
+    'LBL_AOR_CONDITIONS_SUBPANEL_TITLE' => 'Conditions',
+    'LBL_TOTAL' => 'Total',
+    'LBL_AOR_CHARTS_SUBPANEL_TITLE' => 'Charts',
+    'LBL_ADD_CHART' => 'Add chart',
     'LBL_ADD_PARENTHESIS' => 'Drop parenthesis',
-  'LBL_CHART_TITLE' => 'Title',
-  'LBL_CHART_TYPE' => 'Type',
-  'LBL_CHART_X_FIELD' => 'X Axis',
-  'LBL_CHART_Y_FIELD' => 'Y Axis',
-  'LBL_AOR_REPORTS_DASHLET' => 'Reports',
-  'LBL_DASHLET_TITLE' => 'Title',
-  'LBL_DASHLET_REPORT' => 'Report',
-  'LBL_DASHLET_CHOOSE_REPORT' => 'Please choose a report',
-  'LBL_DASHLET_SAVE' => 'Save',
-  'LBL_DASHLET_CHARTS' => 'Charts',
-  'LBL_DASHLET_ONLY_CHARTS' => 'Only show charts',
-  'LBL_AOR_SCHEDULED_REPORTS_AOR_REPORTS_FROM_AOR_SCHEDULED_REPORTS_TITLE' => 'Scheduled Reports',
-  'LBL_UPDATE_PARAMETERS' => 'Update',
-  'LBL_PARAMETERS' => 'Parameters',
+    'LBL_CHART_TITLE' => 'Title',
+    'LBL_CHART_TYPE' => 'Type',
+    'LBL_CHART_X_FIELD' => 'X Axis',
+    'LBL_CHART_Y_FIELD' => 'Y Axis',
+    'LBL_AOR_REPORTS_DASHLET' => 'Reports',
+    'LBL_DASHLET_TITLE' => 'Title',
+    'LBL_DASHLET_REPORT' => 'Report',
+    'LBL_DASHLET_CHOOSE_REPORT' => 'Please choose a report',
+    'LBL_DASHLET_SAVE' => 'Save',
+    'LBL_DASHLET_CHARTS' => 'Charts',
+    'LBL_DASHLET_ONLY_CHARTS' => 'Only show charts',
+    'LBL_AOR_SCHEDULED_REPORTS_AOR_REPORTS_FROM_AOR_SCHEDULED_REPORTS_TITLE' => 'Scheduled Reports',
+    'LBL_UPDATE_PARAMETERS' => 'Update',
+    'LBL_PARAMETERS' => 'Parameters',
     'LBL_TOOLTIP_DRAG_DROP_ELEMS' => 'Drag and drop elements into field or condition area',
     'LBL_MAIN_GROUPS' => 'Main Group:',
 );

data/SugarBean.php

Indentation +739 added lines, -739 removed lines

@@ -84,27 +84,27 @@ 
     public $id;
 
     /**
-	 * When createing a bean, you can specify a value in the id column as
-	 * long as that value is unique.  During save, if the system finds an
-	 * id, it assumes it is an update.  Setting new_with_id to true will
-	 * make sure the system performs an insert instead of an update.
-	 *
-	 * @var BOOL -- default false
-	 */
-	var $new_with_id = false;
-
-
-	/**
-	 * How deep logic hooks can go
-	 * @var int
-	 */
-	protected $max_logic_depth = 10;
-
-	/**
-	 * Disble vardefs.  This should be set to true only for beans that do not have varders.  Tracker is an example
-	 *
-	 * @var BOOL -- default false
-	 */
+     * When createing a bean, you can specify a value in the id column as
+     * long as that value is unique.  During save, if the system finds an
+     * id, it assumes it is an update.  Setting new_with_id to true will
+     * make sure the system performs an insert instead of an update.
+     *
+     * @var BOOL -- default false
+     */
+    var $new_with_id = false;
+
+
+    /**
+     * How deep logic hooks can go
+     * @var int
+     */
+    protected $max_logic_depth = 10;
+
+    /**
+     * Disble vardefs.  This should be set to true only for beans that do not have varders.  Tracker is an example
+     *
+     * @var BOOL -- default false
+     */
     var $disable_vardefs = false;
 
 
@@ -116,20 +116,20 @@ 
      */
     var $new_assigned_user_name;
 
-	/**
-	 * An array of booleans.  This array is cleared out when data is loaded.
-	 * As date/times are converted, a "1" is placed under the key, the field is converted.
-	 *
-	 * @var Array of booleans
-	 */
-	var $processed_dates_times = array();
+    /**
+     * An array of booleans.  This array is cleared out when data is loaded.
+     * As date/times are converted, a "1" is placed under the key, the field is converted.
+     *
+     * @var Array of booleans
+     */
+    var $processed_dates_times = array();
 
-	/**
-	 * Whether to process date/time fields for storage in the database in GMT
-	 *
-	 * @var BOOL
-	 */
-	var $process_save_dates =true;
+    /**
+     * Whether to process date/time fields for storage in the database in GMT
+     *
+     * @var BOOL
+     */
+    var $process_save_dates =true;
 
     /**
      * This signals to the bean that it is being saved in a mass mode.
@@ -140,20 +140,20 @@ 
      */
     var $save_from_post = true;
 
-	/**
-	 * When running a query on related items using the method: retrieve_by_string_fields
-	 * this value will be set to true if more than one item matches the search criteria.
-	 *
-	 * @var BOOL
-	 */
-	var $duplicates_found = false;
+    /**
+     * When running a query on related items using the method: retrieve_by_string_fields
+     * this value will be set to true if more than one item matches the search criteria.
+     *
+     * @var BOOL
+     */
+    var $duplicates_found = false;
 
-	/**
-	 * true if this bean has been deleted, false otherwise.
-	 *
-	 * @var BOOL
-	 */
-	var $deleted = 0;
+    /**
+     * true if this bean has been deleted, false otherwise.
+     *
+     * @var BOOL
+     */
+    var $deleted = 0;
 
     /**
      * Should the date modified column of the bean be updated during save?
@@ -201,22 +201,22 @@ 
     var $table_name = '';
 
     /**
-    * This is the singular name of the bean.  (i.e. Contact).
-    *
-    * @var String
-    */
+     * This is the singular name of the bean.  (i.e. Contact).
+     *
+     * @var String
+     */
     var $object_name = '';
 
     /** Set this to true if you query contains a sub-select and bean is converting both select statements
-    * into count queries.
-    */
+     * into count queries.
+     */
     var $ungreedy_count=false;
 
     /**
-    * The name of the module folder for this type of bean.
-    *
-    * @var String
-    */
+     * The name of the module folder for this type of bean.
+     *
+     * @var String
+     */
     var $module_dir = '';
     var $module_name = '';
     var $field_name_map;
@@ -249,8 +249,8 @@ 
     var $importable = false;
 
     /**
-    * Set to true in the child beans if the module use the special notification template
-    */
+     * Set to true in the child beans if the module use the special notification template
+     */
     var $special_notification = false;
 
     /**
@@ -282,7 +282,7 @@ 
      */
     protected $loaded_relationships = array();
 
-	/**
+    /**
      * set to true if dependent fields updated
      */
     protected $is_updated_dependent_fields = false;
@@ -661,7 +661,7 @@ 
             return;
 
         foreach ($this->field_defs as $field => $value) {
-		    if( !empty($this->$field)
+            if( !empty($this->$field)
                   && ((isset($value['default']) && $this->$field == $value['default']) || (!empty($value['display_default']) && $this->$field == $value['display_default']))
                     ) {
                 $this->$field = null;
@@ -710,7 +710,7 @@ 
                             $this->$field = $this->parseDateDefault($value['display_default']);
                         }
                         break;
-                   case 'datetime':
+                    case 'datetime':
                    case 'datetimecombo':
                         if(!empty($value['display_default'])){
                             $this->$field = $this->parseDateDefault($value['display_default'], true);
@@ -723,9 +723,9 @@ 
                             $this->$field = $value['default'];
                         break;
                     case 'bool':
-                    	if(isset($this->$field)){
-                    		break;
-                    	}
+                        if(isset($this->$field)){
+                            break;
+                        }
                     default:
                         if ( isset($value['default']) && $value['default'] !== '' ) {
                             $this->$field = htmlentities($value['default'], ENT_QUOTES, 'UTF-8');
@@ -788,10 +788,10 @@ 
     /**
      * This method has been deprecated.
      *
-    * @see removeRelationshipMeta()
+     * @see removeRelationshipMeta()
      * @deprecated 4.5.1 - Nov 14, 2006
      * @static
-    */
+     */
     static function remove_relationship_meta($key,$db,$log,$tablename,$dictionary,$module_dir)
     {
         SugarBean::removeRelationshipMeta($key,$db,$tablename,$dictionary,$module_dir);
@@ -927,10 +927,10 @@ 
 
     /**
      * This method has been deprecated.
-    * @see createRelationshipMeta()
+     * @see createRelationshipMeta()
      * @deprecated 4.5.1 - Nov 14, 2006
      * @static
-    */
+     */
     static function create_relationship_meta($key,&$db,&$log,$tablename,$dictionary,$module_dir)
     {
         SugarBean::createRelationshipMeta($key,$db,$tablename,$dictionary,$module_dir);
@@ -1084,7 +1084,7 @@ 
 
         $linked_fields=array();
 
- //   	require_once('data/Link.php');
+    //   	require_once('data/Link.php');
 
         $fieldDefs = $this->getFieldDefinitions();
 
@@ -1314,8 +1314,8 @@ 
     }
 
     /**
-    * Cleans char, varchar, text, etc. fields of XSS type materials
-    */
+     * Cleans char, varchar, text, etc. fields of XSS type materials
+     */
     function cleanBean() {
         foreach($this->field_defs as $key => $def) {
 
@@ -1338,14 +1338,14 @@ 
     }
 
     /**
-    * Implements a generic insert and update logic for any SugarBean
-    * This method only works for subclasses that implement the same variable names.
-    * This method uses the presence of an id field that is not null to signify and update.
-    * The id field should not be set otherwise.
-    *
-    * @param boolean $check_notify Optional, default false, if set to true assignee of the record is notified via email.
-    * @todo Add support for field type validation and encoding of parameters.
-    */
+     * Implements a generic insert and update logic for any SugarBean
+     * This method only works for subclasses that implement the same variable names.
+     * This method uses the presence of an id field that is not null to signify and update.
+     * The id field should not be set otherwise.
+     *
+     * @param boolean $check_notify Optional, default false, if set to true assignee of the record is notified via email.
+     * @todo Add support for field type validation and encoding of parameters.
+     */
     function save($check_notify = FALSE)
     {
         $this->in_save = true;
@@ -1362,14 +1362,14 @@ 
             $isUpdate = false;
         }
 
-		if ( $this->new_with_id == true )
-		{
-			$isUpdate = false;
-		}
-		if(empty($this->date_modified) || $this->update_date_modified)
-		{
-			$this->date_modified = $GLOBALS['timedate']->nowDb();
-		}
+        if ( $this->new_with_id == true )
+        {
+            $isUpdate = false;
+        }
+        if(empty($this->date_modified) || $this->update_date_modified)
+        {
+            $this->date_modified = $GLOBALS['timedate']->nowDb();
+        }
 
         $this->_checkOptimisticLocking($action, $isUpdate);
 
@@ -1483,10 +1483,10 @@ 
         }
 
 
-		/* BEGIN - SECURITY GROUPS - inheritance */ 
-		require_once('modules/SecurityGroups/SecurityGroup.php');
-		SecurityGroup::inherit($this,$isUpdate);
-		/* END - SECURITY GROUPS */ 
+        /* BEGIN - SECURITY GROUPS - inheritance */ 
+        require_once('modules/SecurityGroups/SecurityGroup.php');
+        SecurityGroup::inherit($this,$isUpdate);
+        /* END - SECURITY GROUPS */ 
         //If we aren't in setup mode and we have a current user and module, then we track
         if(isset($GLOBALS['current_user']) && isset($this->module_dir))
         {
@@ -1527,8 +1527,8 @@ 
     }
 
     /**
-    * Determines which users receive a notification
-    */
+     * Determines which users receive a notification
+     */
     function get_notification_recipients() {
         $notify_user = new User();
         $notify_user->retrieve($this->assigned_user_id);
@@ -1545,11 +1545,11 @@ 
     }
 
     /**
-    * Handles sending out email notifications when items are first assigned to users
-    *
-    * @param string $notify_user user to notify
-    * @param string $admin the admin user that sends out the notification
-    */
+     * Handles sending out email notifications when items are first assigned to users
+     *
+     * @param string $notify_user user to notify
+     * @param string $admin the admin user that sends out the notification
+     */
     function send_assignment_notifications($notify_user, $admin)
     {
         global $current_user;
@@ -1580,7 +1580,7 @@ 
                 $notify_mail->FromName = $from_name;
             }
 
-           $oe = new OutboundEmail();
+            $oe = new OutboundEmail();
             $oe = $oe->getUserMailerSettings($current_user);
             //only send if smtp server is defined
             if($sendEmail){
@@ -1616,9 +1616,9 @@ 
     }
 
     /**
-    * This function handles create the email notifications email.
-    * @param string $notify_user the user to send the notification email to
-    */
+     * This function handles create the email notifications email.
+     * @param string $notify_user the user to send the notification email to
+     */
     function create_notification_email($notify_user) {
         global $sugar_version;
         global $sugar_config;
@@ -2027,104 +2027,104 @@ 
     }
 
     /**
-    * This function retrieves a record of the appropriate type from the DB.
-    * It fills in all of the fields from the DB into the object it was called on.
-    *
-    * @param $id - If ID is specified, it overrides the current value of $this->id.  If not specified the current value of $this->id will be used.
-    * @return this - The object that it was called apon or null if exactly 1 record was not found.
-    *
-	*/
-
-	function check_date_relationships_load()
-	{
-		global $disable_date_format;
-		global $timedate;
-		if (empty($timedate))
-			$timedate=TimeDate::getInstance();
-
-		if(empty($this->field_defs))
-		{
-			return;
-		}
-		foreach($this->field_defs as $fieldDef)
-		{
-			$field = $fieldDef['name'];
-			if(!isset($this->processed_dates_times[$field]))
-			{
-				$this->processed_dates_times[$field] = '1';
-				if(empty($this->$field)) continue;
-				if($field == 'date_modified' || $field == 'date_entered')
-				{
-					$this->$field = $this->db->fromConvert($this->$field, 'datetime');
-					if(empty($disable_date_format)) {
-						$this->$field = $timedate->to_display_date_time($this->$field);
-					}
-				}
-				elseif(isset($this->field_name_map[$field]['type']))
-				{
-					$type = $this->field_name_map[$field]['type'];
-
-					if($type == 'relate'  && isset($this->field_name_map[$field]['custom_module']))
-					{
-						$type = $this->field_name_map[$field]['type'];
-					}
-
-					if($type == 'date')
-					{
-						if($this->$field == '0000-00-00')
-						{
-							$this->$field = '';
-						} elseif(!empty($this->field_name_map[$field]['rel_field']))
-						{
-							$rel_field = $this->field_name_map[$field]['rel_field'];
-
-							if(!empty($this->$rel_field))
-							{
-								if(empty($disable_date_format)) {
-									$mergetime = $timedate->merge_date_time($this->$field,$this->$rel_field);
-									$this->$field = $timedate->to_display_date($mergetime);
-									$this->$rel_field = $timedate->to_display_time($mergetime);
-								}
-							}
-						}
-						else
-						{
-							if(empty($disable_date_format)) {
-								$this->$field = $timedate->to_display_date($this->$field, false);
-							}
-						}
-					} elseif($type == 'datetime' || $type == 'datetimecombo')
-					{
-						if($this->$field == '0000-00-00 00:00:00')
-						{
-							$this->$field = '';
-						}
-						else
-						{
-							if(empty($disable_date_format)) {
-								$this->$field = $timedate->to_display_date_time($this->$field, true, true);
-							}
-						}
-					} elseif($type == 'time')
-					{
-						if($this->$field == '00:00:00')
-						{
-							$this->$field = '';
-						} else
-						{
-							//$this->$field = from_db_convert($this->$field, 'time');
-							if(empty($this->field_name_map[$field]['rel_field']) && empty($disable_date_format))
-							{
-								$this->$field = $timedate->to_display_time($this->$field,true, false);
-							}
-						}
-					} elseif($type == 'encrypt' && empty($disable_date_format)){
-						$this->$field = $this->decrypt_after_retrieve($this->$field);
-					}
-				}
-			}
-		}
-	}
+     * This function retrieves a record of the appropriate type from the DB.
+     * It fills in all of the fields from the DB into the object it was called on.
+     *
+     * @param $id - If ID is specified, it overrides the current value of $this->id.  If not specified the current value of $this->id will be used.
+     * @return this - The object that it was called apon or null if exactly 1 record was not found.
+     *
+     */
+
+    function check_date_relationships_load()
+    {
+        global $disable_date_format;
+        global $timedate;
+        if (empty($timedate))
+            $timedate=TimeDate::getInstance();
+
+        if(empty($this->field_defs))
+        {
+            return;
+        }
+        foreach($this->field_defs as $fieldDef)
+        {
+            $field = $fieldDef['name'];
+            if(!isset($this->processed_dates_times[$field]))
+            {
+                $this->processed_dates_times[$field] = '1';
+                if(empty($this->$field)) continue;
+                if($field == 'date_modified' || $field == 'date_entered')
+                {
+                    $this->$field = $this->db->fromConvert($this->$field, 'datetime');
+                    if(empty($disable_date_format)) {
+                        $this->$field = $timedate->to_display_date_time($this->$field);
+                    }
+                }
+                elseif(isset($this->field_name_map[$field]['type']))
+                {
+                    $type = $this->field_name_map[$field]['type'];
+
+                    if($type == 'relate'  && isset($this->field_name_map[$field]['custom_module']))
+                    {
+                        $type = $this->field_name_map[$field]['type'];
+                    }
+
+                    if($type == 'date')
+                    {
+                        if($this->$field == '0000-00-00')
+                        {
+                            $this->$field = '';
+                        } elseif(!empty($this->field_name_map[$field]['rel_field']))
+                        {
+                            $rel_field = $this->field_name_map[$field]['rel_field'];
+
+                            if(!empty($this->$rel_field))
+                            {
+                                if(empty($disable_date_format)) {
+                                    $mergetime = $timedate->merge_date_time($this->$field,$this->$rel_field);
+                                    $this->$field = $timedate->to_display_date($mergetime);
+                                    $this->$rel_field = $timedate->to_display_time($mergetime);
+                                }
+                            }
+                        }
+                        else
+                        {
+                            if(empty($disable_date_format)) {
+                                $this->$field = $timedate->to_display_date($this->$field, false);
+                            }
+                        }
+                    } elseif($type == 'datetime' || $type == 'datetimecombo')
+                    {
+                        if($this->$field == '0000-00-00 00:00:00')
+                        {
+                            $this->$field = '';
+                        }
+                        else
+                        {
+                            if(empty($disable_date_format)) {
+                                $this->$field = $timedate->to_display_date_time($this->$field, true, true);
+                            }
+                        }
+                    } elseif($type == 'time')
+                    {
+                        if($this->$field == '00:00:00')
+                        {
+                            $this->$field = '';
+                        } else
+                        {
+                            //$this->$field = from_db_convert($this->$field, 'time');
+                            if(empty($this->field_name_map[$field]['rel_field']) && empty($disable_date_format))
+                            {
+                                $this->$field = $timedate->to_display_time($this->$field,true, false);
+                            }
+                        }
+                    } elseif($type == 'encrypt' && empty($disable_date_format)){
+                        $this->$field = $this->decrypt_after_retrieve($this->$field);
+                    }
+                }
+            }
+        }
+    }
 
     /**
      * This function processes the fields before save.
@@ -2136,21 +2136,21 @@ 
     }
 
     /**
-    * Removes formatting from values posted from the user interface.
+     * Removes formatting from values posted from the user interface.
      * It only unformats numbers.  Function relies on user/system prefernce for format strings.
      *
      * Internal Function, do not override.
-    */
+     */
     function unformat_all_fields()
     {
         $GLOBALS['log']->deprecated('SugarBean.php: unformat_all_fields() is deprecated');
     }
 
     /**
-    * This functions adds formatting to all number fields before presenting them to user interface.
+     * This functions adds formatting to all number fields before presenting them to user interface.
      *
      * Internal function, do not override.
-    */
+     */
     function format_all_fields()
     {
         $GLOBALS['log']->deprecated('SugarBean.php: format_all_fields() is deprecated');
@@ -2190,8 +2190,8 @@ 
                 case 'datetimecombo':
                     if(empty($this->$field)) break;
                     if ($this->$field == 'NULL') {
-                    	$this->$field = '';
-                    	break;
+                        $this->$field = '';
+                        break;
                     }
                     if ( ! preg_match('/^[0-9]{4}-[0-9]{2}-[0-9]{2} [0-9]{2}:[0-9]{2}:[0-9]{2}$/',$this->$field) ) {
                         // This appears to be formatted in user date/time
@@ -2202,8 +2202,8 @@ 
                 case 'date':
                     if(empty($this->$field)) break;
                     if ($this->$field == 'NULL') {
-                    	$this->$field = '';
-                    	break;
+                        $this->$field = '';
+                        break;
                     }
                     if ( ! preg_match('/^[0-9]{4}-[0-9]{2}-[0-9]{2}$/',$this->$field) ) {
                         // This date appears to be formatted in the user's format
@@ -2214,8 +2214,8 @@ 
                 case 'time':
                     if(empty($this->$field)) break;
                     if ($this->$field == 'NULL') {
-                    	$this->$field = '';
-                    	break;
+                        $this->$field = '';
+                        break;
                     }
                     if ( preg_match('/(am|pm)/i',$this->$field) ) {
                         // This time appears to be formatted in the user's format
@@ -2235,7 +2235,7 @@ 
                         $reformatted = true;
                     }
                     break;
-               case 'uint':
+                case 'uint':
                case 'ulong':
                case 'long':
                case 'short':
@@ -2248,21 +2248,21 @@ 
                         $this->$field = (int)unformat_number($this->$field);
                         $reformatted = true;
                     }
-                   break;
-               case 'bool':
+                    break;
+                case 'bool':
                    if (empty($this->$field)) {
-                       $this->$field = false;
-                   } else if(true === $this->$field || 1 == $this->$field) {
-                       $this->$field = true;
-                   } else if(in_array(strval($this->$field), $boolean_false_values)) {
-                       $this->$field = false;
-                       $reformatted = true;
-                   } else {
-                       $this->$field = true;
-                       $reformatted = true;
-                   }
-                   break;
-               case 'encrypt':
+                        $this->$field = false;
+                    } else if(true === $this->$field || 1 == $this->$field) {
+                        $this->$field = true;
+                    } else if(in_array(strval($this->$field), $boolean_false_values)) {
+                        $this->$field = false;
+                        $reformatted = true;
+                    } else {
+                        $this->$field = true;
+                        $reformatted = true;
+                    }
+                    break;
+                case 'encrypt':
                     $this->$field = $this->encrpyt_before_save($this->$field);
                     break;
             }
@@ -2284,7 +2284,7 @@ 
      * @param boolean $deleted Optional, default true, if set to false deleted filter will not be added.
      *
      * Internal function, do not override.
-    */
+     */
     function retrieve($id = -1, $encode=true,$deleted=true)
     {
 
@@ -2340,18 +2340,18 @@ 
             $this->custom_fields->fill_relationships();
         }
 
-		$this->is_updated_dependent_fields = false;
+        $this->is_updated_dependent_fields = false;
         $this->fill_in_additional_detail_fields();
         $this->fill_in_relationship_fields();
 // save related fields values for audit
-         foreach ($this->get_related_fields() as $rel_field_name)
-         {
-             $field_name = $rel_field_name['name'];
-             if (! empty($this->$field_name))
-             {
-                 $this->fetched_rel_row[$rel_field_name['name']] = $this->$field_name;
-             }
-         }
+            foreach ($this->get_related_fields() as $rel_field_name)
+            {
+                $field_name = $rel_field_name['name'];
+                if (! empty($this->$field_name))
+                {
+                    $this->fetched_rel_row[$rel_field_name['name']] = $this->$field_name;
+                }
+            }
         //make a copy of fields in the relationship_fields array. These field values will be used to
         //clear relationship.
         foreach ( $this->field_defs as $key => $def )
@@ -2364,7 +2364,7 @@ 
                 }
                 else
                     $this->rel_fields_before_value[$key]=null;
-           }
+            }
         }
         if (isset($this->relationship_fields) && is_array($this->relationship_fields))
         {
@@ -2419,14 +2419,14 @@ 
 
 
     /**
-    * Add any required joins to the list count query.  The joins are required if there
-    * is a field in the $where clause that needs to be joined.
-    *
-    * @param string $query
-    * @param string $where
-    *
-    * Internal Function, do Not override.
-    */
+     * Add any required joins to the list count query.  The joins are required if there
+     * is a field in the $where clause that needs to be joined.
+     *
+     * @param string $query
+     * @param string $where
+     *
+     * Internal Function, do Not override.
+     */
     function add_list_count_joins(&$query, $where)
     {
         $custom_join = $this->getCustomJoin();
@@ -2435,15 +2435,15 @@ 
     }
 
     /**
-    * Changes the select expression of the given query to be 'count(*)' so you
-    * can get the number of items the query will return.  This is used to
-    * populate the upper limit on ListViews.
+     * Changes the select expression of the given query to be 'count(*)' so you
+     * can get the number of items the query will return.  This is used to
+     * populate the upper limit on ListViews.
      *
      * @param string $query Select query string
      * @return string count query
      *
      * Internal function, do not override.
-    */
+     */
     function create_list_count_query($query)
     {
         // remove the 'order by' clause which is expected to be at the end of the query
@@ -2496,21 +2496,21 @@ 
     }
 
     /**
-    * This function returns a paged list of the current object type.  It is intended to allow for
-    * hopping back and forth through pages of data.  It only retrieves what is on the current page.
-    *
-    * @internal This method must be called on a new instance.  It trashes the values of all the fields in the current one.
-    * @param string $order_by
-    * @param string $where Additional where clause
-    * @param int $row_offset Optaional,default 0, starting row number
-    * @param init $limit Optional, default -1
-    * @param int $max Optional, default -1
-    * @param boolean $show_deleted Optional, default 0, if set to 1 system will show deleted records.
-    * @return array Fetched data.
-    *
-    * Internal function, do not override.
-    *
-    */
+     * This function returns a paged list of the current object type.  It is intended to allow for
+     * hopping back and forth through pages of data.  It only retrieves what is on the current page.
+     *
+     * @internal This method must be called on a new instance.  It trashes the values of all the fields in the current one.
+     * @param string $order_by
+     * @param string $where Additional where clause
+     * @param int $row_offset Optaional,default 0, starting row number
+     * @param init $limit Optional, default -1
+     * @param int $max Optional, default -1
+     * @param boolean $show_deleted Optional, default 0, if set to 1 system will show deleted records.
+     * @return array Fetched data.
+     *
+     * Internal function, do not override.
+     *
+     */
     function get_list($order_by = "", $where = "", $row_offset = 0, $limit=-1, $max=-1, $show_deleted = 0, $singleSelect=false, $select_fields = array())
     {
         $GLOBALS['log']->debug("get_list:  order_by = '$order_by' and where = '$where' and limit = '$limit'");
@@ -2540,15 +2540,15 @@ 
     }
 
     /**
-    * Prefixes column names with this bean's table name.
-    *
-    * @param string $order_by  Order by clause to be processed
-    * @param SugarBean $submodule name of the module this order by clause is for
-    * @param boolean $suppress_table_name Whether table name should be suppressed
-    * @return string Processed order by clause
-    *
-    * Internal function, do not override.
-    */
+     * Prefixes column names with this bean's table name.
+     *
+     * @param string $order_by  Order by clause to be processed
+     * @param SugarBean $submodule name of the module this order by clause is for
+     * @param boolean $suppress_table_name Whether table name should be suppressed
+     * @return string Processed order by clause
+     *
+     * Internal function, do not override.
+     */
     public function process_order_by($order_by, $submodule = null, $suppress_table_name = false)
     {
         if (empty($order_by))
@@ -2622,21 +2622,21 @@ 
 
 
     /**
-    * Returns a detail object like retrieving of the current object type.
-    *
-    * It is intended for use in navigation buttons on the DetailView.  It will pass an offset and limit argument to the sql query.
-    * @internal This method must be called on a new instance.  It overrides the values of all the fields in the current one.
-    *
-    * @param string $order_by
-    * @param string $where Additional where clause
-    * @param int $row_offset Optaional,default 0, starting row number
-    * @param init $limit Optional, default -1
-    * @param int $max Optional, default -1
-    * @param boolean $show_deleted Optioanl, default 0, if set to 1 system will show deleted records.
-    * @return array Fetched data.
-    *
-    * Internal function, do not override.
-    */
+     * Returns a detail object like retrieving of the current object type.
+     *
+     * It is intended for use in navigation buttons on the DetailView.  It will pass an offset and limit argument to the sql query.
+     * @internal This method must be called on a new instance.  It overrides the values of all the fields in the current one.
+     *
+     * @param string $order_by
+     * @param string $where Additional where clause
+     * @param int $row_offset Optaional,default 0, starting row number
+     * @param init $limit Optional, default -1
+     * @param int $max Optional, default -1
+     * @param boolean $show_deleted Optioanl, default 0, if set to 1 system will show deleted records.
+     * @return array Fetched data.
+     *
+     * Internal function, do not override.
+     */
     function get_detail($order_by = "", $where = "",  $offset = 0, $row_offset = 0, $limit=-1, $max=-1, $show_deleted = 0)
     {
         $GLOBALS['log']->debug("get_detail:  order_by = '$order_by' and where = '$where' and limit = '$limit' and offset = '$offset'");
@@ -2660,25 +2660,25 @@ 
             }
         }
 
-		/* BEGIN - SECURITY GROUPS */
-    	if($this->bean_implements('ACL') && ACLController::requireSecurityGroup($this->module_dir, 'list') )
-    	{
-			require_once('modules/SecurityGroups/SecurityGroup.php');
-    		global $current_user;
-    		$owner_where = $this->getOwnerWhere($current_user->id);
-    		$group_where = SecurityGroup::getGroupWhere($this->table_name,$this->module_dir,$current_user->id);
-	    	if(!empty($owner_where)){
-				if(empty($where))
-	    		{
-	    			$where = " (".  $owner_where." or ".$group_where.") ";
-	    		} else {
-	    			$where .= " AND (".  $owner_where." or ".$group_where.") ";
-	    		}
-			} else {
-				$where .= ' AND '.  $group_where;
-			}
-    	}
-    	/* END - SECURITY GROUPS */
+        /* BEGIN - SECURITY GROUPS */
+        if($this->bean_implements('ACL') && ACLController::requireSecurityGroup($this->module_dir, 'list') )
+        {
+            require_once('modules/SecurityGroups/SecurityGroup.php');
+            global $current_user;
+            $owner_where = $this->getOwnerWhere($current_user->id);
+            $group_where = SecurityGroup::getGroupWhere($this->table_name,$this->module_dir,$current_user->id);
+            if(!empty($owner_where)){
+                if(empty($where))
+                {
+                    $where = " (".  $owner_where." or ".$group_where.") ";
+                } else {
+                    $where .= " AND (".  $owner_where." or ".$group_where.") ";
+                }
+            } else {
+                $where .= ' AND '.  $group_where;
+            }
+        }
+        /* END - SECURITY GROUPS */
         $query = $this->create_new_list_query($order_by, $where,array(),array(), $show_deleted, $offset);
 
         //Add Limit and Offset to query
@@ -2688,16 +2688,16 @@ 
     }
 
     /**
-    * Fetches data from all related tables.
-    *
-    * @param object $child_seed
-    * @param string $related_field_name relation to fetch data for
-    * @param string $order_by Optional, default empty
-    * @param string $where Optional, additional where clause
-    * @return array Fetched data.
-    *
-    * Internal function, do not override.
-    */
+     * Fetches data from all related tables.
+     *
+     * @param object $child_seed
+     * @param string $related_field_name relation to fetch data for
+     * @param string $order_by Optional, default empty
+     * @param string $where Optional, additional where clause
+     * @return array Fetched data.
+     *
+     * Internal function, do not override.
+     */
     function get_related_list($child_seed,$related_field_name, $order_by = "", $where = "",
     $row_offset = 0, $limit=-1, $max=-1, $show_deleted = 0)
     {
@@ -3073,13 +3073,13 @@ 
 
 
     /**
-    * Returns a full (ie non-paged) list of the current object type.
-    *
-    * @param string $order_by the order by SQL parameter. defaults to ""
-    * @param string $where where clause. defaults to ""
-    * @param boolean $check_dates. defaults to false
-    * @param int $show_deleted show deleted records. defaults to 0
-    */
+     * Returns a full (ie non-paged) list of the current object type.
+     *
+     * @param string $order_by the order by SQL parameter. defaults to ""
+     * @param string $where where clause. defaults to ""
+     * @param boolean $check_dates. defaults to false
+     * @param int $show_deleted show deleted records. defaults to 0
+     */
     function get_full_list($order_by = "", $where = "", $check_dates=false, $show_deleted = 0)
     {
         $GLOBALS['log']->debug("get_full_list:  order_by = '$order_by' and where = '$where'");
@@ -3107,7 +3107,7 @@ 
      * @param boolean $singleSelect Optional, default false.
      * @return String select query string, optionally an array value will be returned if $return_array= true.
      */
-	function create_new_list_query($order_by, $where,$filter=array(),$params=array(), $show_deleted = 0,$join_type='', $return_array = false,$parentbean=null, $singleSelect = false, $ifListForExport = false)
+    function create_new_list_query($order_by, $where,$filter=array(),$params=array(), $show_deleted = 0,$join_type='', $return_array = false,$parentbean=null, $singleSelect = false, $ifListForExport = false)
     {
         global $beanFiles, $beanList;
         $selectedFields = array();
@@ -3127,41 +3127,41 @@ 
                 $where .= ' AND '.  $owner_where;
             }
         }
-		/* BEGIN - SECURITY GROUPS */
-    	global $current_user, $sugar_config;
-    	if($this->module_dir == 'Users' && !is_admin($current_user)
-				&& isset($sugar_config['securitysuite_filter_user_list'])
-				&& $sugar_config['securitysuite_filter_user_list'] == true
-    	) {
-			require_once('modules/SecurityGroups/SecurityGroup.php');
-    		global $current_user;
-    		$group_where = SecurityGroup::getGroupUsersWhere($current_user->id);
-    		//$group_where = "user_name = 'admin'";
-    		if(empty($where))
-    		{
-    			$where = " (".$group_where.") ";
-    		} else {
-    			$where .= " AND (".$group_where.") ";
-    		}
-    	} else
-    	if($this->bean_implements('ACL') && ACLController::requireSecurityGroup($this->module_dir, 'list') )
-    	{
-			require_once('modules/SecurityGroups/SecurityGroup.php');
-    		global $current_user;
-    		$owner_where = $this->getOwnerWhere($current_user->id);
-    		$group_where = SecurityGroup::getGroupWhere($this->table_name,$this->module_dir,$current_user->id);
-	    	if(!empty($owner_where)){
-				if(empty($where))
-	    		{
-	    			$where = " (".  $owner_where." or ".$group_where.") ";
-	    		} else {
-	    			$where .= " AND (".  $owner_where." or ".$group_where.") ";
-	    		}
-			} else {
-				$where .= ' AND '.  $group_where;
-			}
-    	}
-    	/* END - SECURITY GROUPS */
+        /* BEGIN - SECURITY GROUPS */
+        global $current_user, $sugar_config;
+        if($this->module_dir == 'Users' && !is_admin($current_user)
+                && isset($sugar_config['securitysuite_filter_user_list'])
+                && $sugar_config['securitysuite_filter_user_list'] == true
+        ) {
+            require_once('modules/SecurityGroups/SecurityGroup.php');
+            global $current_user;
+            $group_where = SecurityGroup::getGroupUsersWhere($current_user->id);
+            //$group_where = "user_name = 'admin'";
+            if(empty($where))
+            {
+                $where = " (".$group_where.") ";
+            } else {
+                $where .= " AND (".$group_where.") ";
+            }
+        } else
+        if($this->bean_implements('ACL') && ACLController::requireSecurityGroup($this->module_dir, 'list') )
+        {
+            require_once('modules/SecurityGroups/SecurityGroup.php');
+            global $current_user;
+            $owner_where = $this->getOwnerWhere($current_user->id);
+            $group_where = SecurityGroup::getGroupWhere($this->table_name,$this->module_dir,$current_user->id);
+            if(!empty($owner_where)){
+                if(empty($where))
+                {
+                    $where = " (".  $owner_where." or ".$group_where.") ";
+                } else {
+                    $where .= " AND (".  $owner_where." or ".$group_where.") ";
+                }
+            } else {
+                $where .= ' AND '.  $group_where;
+            }
+        }
+        /* END - SECURITY GROUPS */
         if(!empty($params['distinct']))
         {
             $distinct = ' DISTINCT ';
@@ -3247,24 +3247,24 @@ 
 
         $used_join_key = array();
 
-	//walk through the fields and for every relationship field add their relationship_info field
-	//relationshipfield-aliases are resolved in SugarBean::create_new_list_query through their relationship_info field
-	$addrelate = array();
-	foreach($fields as $field=>$value)
-	{
-		if (isset($this->field_defs[$field]) && isset($this->field_defs[$field]['source']) && 
-			$this->field_defs[$field]['source'] == 'non-db')
-		{
-			$addrelatefield = $this->get_relationship_field($field);
-			if ($addrelatefield)
-				$addrelate[$addrelatefield] = true;
-		}
-		if(!empty($this->field_defs[$field]['id_name'])){
-			$addrelate[$this->field_defs[$field]['id_name']] = true;
-		}
-	}
-
-	$fields = array_merge($addrelate, $fields);
+    //walk through the fields and for every relationship field add their relationship_info field
+    //relationshipfield-aliases are resolved in SugarBean::create_new_list_query through their relationship_info field
+    $addrelate = array();
+    foreach($fields as $field=>$value)
+    {
+        if (isset($this->field_defs[$field]) && isset($this->field_defs[$field]['source']) && 
+            $this->field_defs[$field]['source'] == 'non-db')
+        {
+            $addrelatefield = $this->get_relationship_field($field);
+            if ($addrelatefield)
+                $addrelate[$addrelatefield] = true;
+        }
+        if(!empty($this->field_defs[$field]['id_name'])){
+            $addrelate[$this->field_defs[$field]['id_name']] = true;
+        }
+    }
+
+    $fields = array_merge($addrelate, $fields);
 
         foreach($fields as $field=>$value)
         {
@@ -3297,7 +3297,7 @@ 
             //in opportunities module remove the contact_role/opportunity_role field.
             if (isset($data['relationship_fields']) and !empty($data['relationship_fields']))
             {
-		$process_field = false;
+        $process_field = false;
                 foreach ($data['relationship_fields'] as $field_name)
                 {
                     if (isset($fields[$field_name]))
@@ -3307,8 +3307,8 @@ 
                     }
                 }
 		
-            	if (!$process_field)
-                	continue;
+                if (!$process_field)
+                    continue;
             }
 
             if(  (!isset($data['source']) || $data['source'] == 'db') && (!empty($alias) || !empty($filter) ))
@@ -3416,27 +3416,27 @@ 
                     $rel_module = $this->$linkField->getRelatedModuleName();
                     $table_joined = !empty($joined_tables[$params['join_table_alias']]) || (!empty($joined_tables[$params['join_table_link_alias']]) && isset($data['link_type']) && $data['link_type'] == 'relationship_info');
 
-					//if rname is set to 'name', and bean files exist, then check if field should be a concatenated name
-					global $beanFiles, $beanList;
+                    //if rname is set to 'name', and bean files exist, then check if field should be a concatenated name
+                    global $beanFiles, $beanList;
                     // °3/21/2014 FIX NS-TEAM - Relationship fields could not be displayed in subpanels
                     //if($data['rname'] && !empty($beanFiles[$beanList[$rel_module]])) {
                     if(isset($data['rname']) && $data['rname'] == 'name' && !empty($beanFiles[$beanList[$rel_module]])) {
 
-						//create an instance of the related bean
-						require_once($beanFiles[$beanList[$rel_module]]);
-						$rel_mod = new $beanList[$rel_module]();
-						//if bean has first and last name fields, then name should be concatenated
-						if(isset($rel_mod->field_name_map['first_name']) && isset($rel_mod->field_name_map['last_name'])){
-								$data['db_concat_fields'] = array(0=>'first_name', 1=>'last_name');
-						}
-					}
+                        //create an instance of the related bean
+                        require_once($beanFiles[$beanList[$rel_module]]);
+                        $rel_mod = new $beanList[$rel_module]();
+                        //if bean has first and last name fields, then name should be concatenated
+                        if(isset($rel_mod->field_name_map['first_name']) && isset($rel_mod->field_name_map['last_name'])){
+                                $data['db_concat_fields'] = array(0=>'first_name', 1=>'last_name');
+                        }
+                    }
 
 
-    				if($join['type'] == 'many-to-many')
-    				{
-    					if(empty($ret_array['secondary_select']))
-    					{
-    						$ret_array['secondary_select'] = " SELECT $this->table_name.id ref_id  ";
+                    if($join['type'] == 'many-to-many')
+                    {
+                        if(empty($ret_array['secondary_select']))
+                        {
+                            $ret_array['secondary_select'] = " SELECT $this->table_name.id ref_id  ";
 
                             if(!empty($beanFiles[$beanList[$rel_module]]) && $join_primary)
                             {
@@ -3548,42 +3548,42 @@ 
                     // and this code changes accounts to jt4 as there is a self join with the accounts table.
                     //Martin fix #27494
                     if(isset($data['db_concat_fields'])){
-                    	$buildWhere = false;
+                        $buildWhere = false;
                         if(in_array('first_name', $data['db_concat_fields']) && in_array('last_name', $data['db_concat_fields']))
-                    	{
-                     	   $exp = '/\(\s*?'.$data['name'].'.*?\%\'\s*?\)/';
-                    	   if(preg_match($exp, $where, $matches))
-                    	   {
-                    	   	  $search_expression = $matches[0];
-                    	   	  //Create three search conditions - first + last, first, last
-                    	   	  $first_name_search = str_replace($data['name'], $params['join_table_alias'] . '.first_name', $search_expression);
-                    	   	  $last_name_search = str_replace($data['name'], $params['join_table_alias'] . '.last_name', $search_expression);
-							  $full_name_search = str_replace($data['name'], $this->db->concat($params['join_table_alias'], $data['db_concat_fields']), $search_expression);
-							  $buildWhere = true;
-							  $where = str_replace($search_expression, '(' . $full_name_search . ' OR ' . $first_name_search . ' OR ' . $last_name_search . ')', $where);
-                    	   }
-                    	}
-
-                    	if(!$buildWhere)
-                    	{
-	                       $db_field = $this->db->concat($params['join_table_alias'], $data['db_concat_fields']);
-	                       $where = preg_replace('/'.$data['name'].'/', $db_field, $where);
-
-				// For relationship fields replace their alias by the corresponsding link table and r_name
-				if(isset($data['relationship_fields']))
-					foreach($data['relationship_fields'] as $r_name=>$alias_name)
-					{
-						$db_field = $this->db->concat($params['join_table_link_alias'], $r_name);
-						$where = preg_replace('/' . $alias_name . '/', $db_field, $where);
-					}
-                    	}
+                        {
+                            $exp = '/\(\s*?'.$data['name'].'.*?\%\'\s*?\)/';
+                            if(preg_match($exp, $where, $matches))
+                            {
+                                    $search_expression = $matches[0];
+                                    //Create three search conditions - first + last, first, last
+                                    $first_name_search = str_replace($data['name'], $params['join_table_alias'] . '.first_name', $search_expression);
+                                    $last_name_search = str_replace($data['name'], $params['join_table_alias'] . '.last_name', $search_expression);
+                                $full_name_search = str_replace($data['name'], $this->db->concat($params['join_table_alias'], $data['db_concat_fields']), $search_expression);
+                                $buildWhere = true;
+                                $where = str_replace($search_expression, '(' . $full_name_search . ' OR ' . $first_name_search . ' OR ' . $last_name_search . ')', $where);
+                            }
+                        }
+
+                        if(!$buildWhere)
+                        {
+                            $db_field = $this->db->concat($params['join_table_alias'], $data['db_concat_fields']);
+                            $where = preg_replace('/'.$data['name'].'/', $db_field, $where);
+
+                // For relationship fields replace their alias by the corresponsding link table and r_name
+                if(isset($data['relationship_fields']))
+                    foreach($data['relationship_fields'] as $r_name=>$alias_name)
+                    {
+                        $db_field = $this->db->concat($params['join_table_link_alias'], $r_name);
+                        $where = preg_replace('/' . $alias_name . '/', $db_field, $where);
+                    }
+                        }
                     }else{
                         $where = preg_replace('/(^|[\s(])' . $data['name'] . '/', '${1}' . $params['join_table_alias'] . '.'.$data['rname'], $where);
 
-			// For relationship fields replace their alias by the corresponsding link table and r_name
-			if(isset($data['relationship_fields']))
-				foreach($data['relationship_fields'] as $r_name=>$alias_name)
-					$where = preg_replace('/(^|[\s(])' . $alias_name . '/', '${1}' . $params['join_table_link_alias'] . '.'.$r_name, $where);
+            // For relationship fields replace their alias by the corresponsding link table and r_name
+            if(isset($data['relationship_fields']))
+                foreach($data['relationship_fields'] as $r_name=>$alias_name)
+                    $where = preg_replace('/(^|[\s(])' . $alias_name . '/', '${1}' . $params['join_table_link_alias'] . '.'.$r_name, $where);
                     }
                     if(!$table_joined)
                     {
@@ -3612,12 +3612,12 @@ 
 
         }
 
-	if ($ifListForExport) {
-		if(isset($this->field_defs['email1'])) {
-			$ret_array['select'].= " ,email_addresses.email_address email1";
-			$ret_array['from'].= " LEFT JOIN email_addr_bean_rel on {$this->table_name}.id = email_addr_bean_rel.bean_id and email_addr_bean_rel.bean_module='{$this->module_dir}' and email_addr_bean_rel.deleted=0 and email_addr_bean_rel.primary_address=1 LEFT JOIN email_addresses on email_addresses.id = email_addr_bean_rel.email_address_id ";
-		}
-	}
+    if ($ifListForExport) {
+        if(isset($this->field_defs['email1'])) {
+            $ret_array['select'].= " ,email_addresses.email_address email1";
+            $ret_array['from'].= " LEFT JOIN email_addr_bean_rel on {$this->table_name}.id = email_addr_bean_rel.bean_id and email_addr_bean_rel.bean_module='{$this->module_dir}' and email_addr_bean_rel.deleted=0 and email_addr_bean_rel.primary_address=1 LEFT JOIN email_addresses on email_addresses.id = email_addr_bean_rel.email_address_id ";
+        }
+    }
 
         $where_auto = '1=1';
         if($show_deleted == 0)
@@ -3652,20 +3652,20 @@ 
         return  $ret_array['select'] . $ret_array['from'] . $ret_array['where']. $ret_array['order_by'];
     }
 
-	// Check if field is defined through a relationship_info field, add this field when not present 
-	function get_relationship_field($field)
-	{
-		foreach ($this->field_defs as $field_def => $value)
-		{
-			if (isset($value['relationship_fields']) && 
-				in_array($field, $value['relationship_fields']) &&
+    // Check if field is defined through a relationship_info field, add this field when not present 
+    function get_relationship_field($field)
+    {
+        foreach ($this->field_defs as $field_def => $value)
+        {
+            if (isset($value['relationship_fields']) && 
+                in_array($field, $value['relationship_fields']) &&
                 (!isset($value['link_type']) || $value['link_type'] != 'relationship_info')
             )
-				return $field_def;
-		}
+                return $field_def;
+        }
 
-		return false;
-	}
+        return false;
+    }
 
     /**
      * Returns parent record data for objects that store relationship information
@@ -3772,8 +3772,8 @@ 
         global $sugar_config;
         $db = DBManagerFactory::getInstance('listviews');
         /**
-        * if the row_offset is set to 'end' go to the end of the list
-        */
+         * if the row_offset is set to 'end' go to the end of the list
+         */
         $toEnd = strval($row_offset) == 'end';
         $GLOBALS['log']->debug("process_list_query: ".$query);
         if($max_per_page == -1)
@@ -3901,13 +3901,13 @@ 
     }
 
     /**
-    * Returns the number of rows that the given SQL query should produce
+     * Returns the number of rows that the given SQL query should produce
      *
      * Internal function, do not override.
      * @param string $query valid select  query
      * @param boolean $is_count_query Optional, Default false, set to true if passed query is a count query.
      * @return int count of rows found
-    */
+     */
     function _get_num_rows_in_query($query, $is_count_query=false)
     {
         $num_rows_in_query = 0;
@@ -3949,8 +3949,8 @@ 
     {
         $db = DBManagerFactory::getInstance('listviews');
         /**
-        * if the row_offset is set to 'end' go to the end of the list
-        */
+         * if the row_offset is set to 'end' go to the end of the list
+         */
         $toEnd = strval($row_offset) == 'end';
         global $sugar_config;
         $use_count_query=false;
@@ -4188,7 +4188,7 @@ 
             }
             if(!empty($sugar_config['disable_count_query']) && !empty($limit))
             {
-            	//C.L. Bug 43535 - Use the $index value to set the $rows_found value here
+                //C.L. Bug 43535 - Use the $index value to set the $rows_found value here
                 $rows_found = isset($index) ? $index : $row_offset + count($list);
 
                 if(count($list) >= $limit)
@@ -4338,28 +4338,28 @@ 
     }
 
     /**
-    * Tracks the viewing of a detail record.
-    * This leverages get_summary_text() which is object specific.
-    *
-    * Internal function, do not override.
-    * @param string $user_id - String value of the user that is viewing the record.
-    * @param string $current_module - String value of the module being processed.
-    * @param string $current_view - String value of the current view
-	*/
-	function track_view($user_id, $current_module, $current_view='')
-	{
-	    $trackerManager = TrackerManager::getInstance();
-		if($monitor = $trackerManager->getMonitor('tracker')){
-	        $monitor->setValue('date_modified', $GLOBALS['timedate']->nowDb());
-	        $monitor->setValue('user_id', $user_id);
-	        $monitor->setValue('module_name', $current_module);
-	        $monitor->setValue('action', $current_view);
-	        $monitor->setValue('item_id', $this->id);
-	        $monitor->setValue('item_summary', $this->get_summary_text());
-	        $monitor->setValue('visible', $this->tracker_visibility);
-	        $trackerManager->saveMonitor($monitor);
-		}
-	}
+     * Tracks the viewing of a detail record.
+     * This leverages get_summary_text() which is object specific.
+     *
+     * Internal function, do not override.
+     * @param string $user_id - String value of the user that is viewing the record.
+     * @param string $current_module - String value of the module being processed.
+     * @param string $current_view - String value of the current view
+     */
+    function track_view($user_id, $current_module, $current_view='')
+    {
+        $trackerManager = TrackerManager::getInstance();
+        if($monitor = $trackerManager->getMonitor('tracker')){
+            $monitor->setValue('date_modified', $GLOBALS['timedate']->nowDb());
+            $monitor->setValue('user_id', $user_id);
+            $monitor->setValue('module_name', $current_module);
+            $monitor->setValue('action', $current_view);
+            $monitor->setValue('item_id', $this->id);
+            $monitor->setValue('item_summary', $this->get_summary_text());
+            $monitor->setValue('visible', $this->tracker_visibility);
+            $trackerManager->saveMonitor($monitor);
+        }
+    }
 
     /**
      * Returns the summary text that should show up in the recent history list for this object.
@@ -4372,12 +4372,12 @@ 
     }
 
     /**
-    * This is designed to be overridden and add specific fields to each record.
-    * This allows the generic query to fill in the major fields, and then targeted
-    * queries to get related fields and add them to the record.  The contact's
-    * account for instance.  This method is only used for populating extra fields
-    * in lists.
-    */
+     * This is designed to be overridden and add specific fields to each record.
+     * This allows the generic query to fill in the major fields, and then targeted
+     * queries to get related fields and add them to the record.  The contact's
+     * account for instance.  This method is only used for populating extra fields
+     * in lists.
+     */
     function fill_in_additional_list_fields(){
         if(!empty($this->field_defs['parent_name']) && empty($this->parent_name)){
             $this->fill_in_additional_parent_fields();
@@ -4385,12 +4385,12 @@ 
     }
 
     /**
-    * This is designed to be overridden and add specific fields to each record.
-    * This allows the generic query to fill in the major fields, and then targeted
-    * queries to get related fields and add them to the record.  The contact's
-    * account for instance.  This method is only used for populating extra fields
-    * in the detail form
-    */
+     * This is designed to be overridden and add specific fields to each record.
+     * This allows the generic query to fill in the major fields, and then targeted
+     * queries to get related fields and add them to the record.  The contact's
+     * account for instance.  This method is only used for populating extra fields
+     * in the detail form
+     */
     function fill_in_additional_detail_fields()
     {
         if(!empty($this->field_defs['assigned_user_name']) && !empty($this->assigned_user_id)){
@@ -4398,20 +4398,20 @@ 
             $this->assigned_user_name = get_assigned_user_name($this->assigned_user_id);
 
         }
-		if(!empty($this->field_defs['created_by']) && !empty($this->created_by))
-		$this->created_by_name = get_assigned_user_name($this->created_by);
-		if(!empty($this->field_defs['modified_user_id']) && !empty($this->modified_user_id))
-		$this->modified_by_name = get_assigned_user_name($this->modified_user_id);
+        if(!empty($this->field_defs['created_by']) && !empty($this->created_by))
+        $this->created_by_name = get_assigned_user_name($this->created_by);
+        if(!empty($this->field_defs['modified_user_id']) && !empty($this->modified_user_id))
+        $this->modified_by_name = get_assigned_user_name($this->modified_user_id);
 
-		if(!empty($this->field_defs['parent_name'])){
-			$this->fill_in_additional_parent_fields();
-		}
+        if(!empty($this->field_defs['parent_name'])){
+            $this->fill_in_additional_parent_fields();
+        }
     }
 
     /**
-    * This is desgined to be overridden or called from extending bean. This method
-    * will fill in any parent_name fields.
-    */
+     * This is desgined to be overridden or called from extending bean. This method
+     * will fill in any parent_name fields.
+     */
     function fill_in_additional_parent_fields() {
 
         if(!empty($this->parent_id) && !empty($this->last_parent_id) && $this->last_parent_id == $this->parent_id){
@@ -4452,8 +4452,8 @@ 
     }
 
     /**
-    * Fill in fields where type = relate
-    */
+     * Fill in fields where type = relate
+     */
     function fill_in_relationship_fields(){
         global $fill_in_rel_depth;
         if(empty($fill_in_rel_depth) || $fill_in_rel_depth < 0)
@@ -4477,7 +4477,7 @@ 
 
                     if (empty($this->$id_name))
                     {
-                       $this->fill_in_link_field($id_name, $field);
+                        $this->fill_in_link_field($id_name, $field);
                     }
                     if(!empty($this->$id_name) && ( $this->object_name != $related_module || ( $this->object_name == $related_module && $this->$id_name != $this->id ))){
                         if(isset($GLOBALS['beanList'][ $related_module])){
@@ -4503,7 +4503,7 @@ 
                     if(!empty($this->$id_name) && isset($this->$name))
                     {
                         if(!isset($field['additionalFields']))
-                           $field['additionalFields'] = array();
+                            $field['additionalFields'] = array();
                         if(!empty($field['rname']))
                         {
                             $field['additionalFields'][$field['rname']]= $name;
@@ -4521,8 +4521,8 @@ 
     }
 
     /**
-    * This is a helper function that is used to quickly created indexes when creating tables.
-    */
+     * This is a helper function that is used to quickly created indexes when creating tables.
+     */
     function create_index($query)
     {
         $GLOBALS['log']->info("create_index: $query");
@@ -4534,21 +4534,21 @@ 
      * This function should be overridden in each module.  It marks an item as deleted.
      *
      * If it is not overridden, then marking this type of item is not allowed
-	 */
-	function mark_deleted($id)
-	{
-		global $current_user;
-		$date_modified = $GLOBALS['timedate']->nowDb();
+     */
+    function mark_deleted($id)
+    {
+        global $current_user;
+        $date_modified = $GLOBALS['timedate']->nowDb();
         $id = $this->db->quote($id);
-		if(isset($_SESSION['show_deleted']))
-		{
-			$this->mark_undeleted($id);
-		}
-		else
-		{
-			// call the custom business logic
-			$custom_logic_arguments['id'] = $id;
-			$this->call_custom_logic("before_delete", $custom_logic_arguments);
+        if(isset($_SESSION['show_deleted']))
+        {
+            $this->mark_undeleted($id);
+        }
+        else
+        {
+            // call the custom business logic
+            $custom_logic_arguments['id'] = $id;
+            $this->call_custom_logic("before_delete", $custom_logic_arguments);
             $this->deleted = 1;
             $this->mark_relationships_deleted($id);
             if ( isset($this->field_defs['modified_user_id']) ) {
@@ -4581,16 +4581,16 @@ 
      * Restores data deleted by call to mark_deleted() function.
      *
      * Internal function, do not override.
-    */
+     */
     function mark_undeleted($id)
     {
         // call the custom business logic
         $custom_logic_arguments['id'] = $id;
         $this->call_custom_logic("before_restore", $custom_logic_arguments);
 
-		$date_modified = $GLOBALS['timedate']->nowDb();
-		$query = "UPDATE $this->table_name set deleted=0 , date_modified = '$date_modified' where id='" . $this->db->quote($id) ."'";
-		$this->db->query($query, true,"Error marking record undeleted: ");
+        $date_modified = $GLOBALS['timedate']->nowDb();
+        $query = "UPDATE $this->table_name set deleted=0 , date_modified = '$date_modified' where id='" . $this->db->quote($id) ."'";
+        $this->db->query($query, true,"Error marking record undeleted: ");
 
         $this->restoreFiles();
 
@@ -4598,17 +4598,17 @@ 
         $this->call_custom_logic("after_restore", $custom_logic_arguments);
     }
 
-   /**
-    * This function deletes relationships to this object.  It should be overridden
-    * to handle the relationships of the specific object.
-    * This function is called when the item itself is being deleted.
-    *
-    * @param int $id id of the relationship to delete
-    */
-   function mark_relationships_deleted($id)
-   {
+    /**
+     * This function deletes relationships to this object.  It should be overridden
+     * to handle the relationships of the specific object.
+     * This function is called when the item itself is being deleted.
+     *
+     * @param int $id id of the relationship to delete
+     */
+    function mark_relationships_deleted($id)
+    {
     $this->delete_linked($id);
-   }
+    }
 
     /**
      * Returns path for files of bean or false on error
@@ -4788,16 +4788,16 @@ 
     }
 
     /**
-    * This function is used to execute the query and create an array template objects
-    * from the resulting ids from the query.
-    * It is currently used for building sub-panel arrays.
-    *
-    * @param string $query - the query that should be executed to build the list
-    * @param object $template - The object that should be used to copy the records.
-    * @param int $row_offset Optional, default 0
-    * @param int $limit Optional, default -1
-    * @return array
-    */
+     * This function is used to execute the query and create an array template objects
+     * from the resulting ids from the query.
+     * It is currently used for building sub-panel arrays.
+     *
+     * @param string $query - the query that should be executed to build the list
+     * @param object $template - The object that should be used to copy the records.
+     * @param int $row_offset Optional, default 0
+     * @param int $limit Optional, default -1
+     * @return array
+     */
     function build_related_list($query, &$template, $row_offset = 0, $limit = -1)
     {
         $GLOBALS['log']->debug("Finding linked records $this->object_name: ".$query);
@@ -4833,17 +4833,17 @@ 
         return $list;
     }
 
-  /**
-    * This function is used to execute the query and create an array template objects
-    * from the resulting ids from the query.
-    * It is currently used for building sub-panel arrays. It supports an additional
-    * where clause that is executed as a filter on the results
-    *
-    * @param string $query - the query that should be executed to build the list
-    * @param object $template - The object that should be used to copy the records.
-    */
-  function build_related_list_where($query, &$template, $where='', $in='', $order_by, $limit='', $row_offset = 0)
-  {
+    /**
+     * This function is used to execute the query and create an array template objects
+     * from the resulting ids from the query.
+     * It is currently used for building sub-panel arrays. It supports an additional
+     * where clause that is executed as a filter on the results
+     *
+     * @param string $query - the query that should be executed to build the list
+     * @param object $template - The object that should be used to copy the records.
+     */
+    function build_related_list_where($query, &$template, $where='', $in='', $order_by, $limit='', $row_offset = 0)
+    {
     $db = DBManagerFactory::getInstance('listviews');
     // No need to do an additional query
     $GLOBALS['log']->debug("Finding linked records $this->object_name: ".$query);
@@ -4904,7 +4904,7 @@ 
     }
 
     return $list;
-  }
+    }
 
     /**
      * Constructs an comma separated list of ids from passed query results.
@@ -4940,15 +4940,15 @@ 
     }
 
     /**
-    * Optionally copies values from fetched row into the bean.
-    *
-    * Internal function, do not override.
-    *
-    * @param string $query - the query that should be executed to build the list
-    * @param object $template - The object that should be used to copy the records
-    * @param array $field_list List of  fields.
-    * @return array
-    */
+     * Optionally copies values from fetched row into the bean.
+     *
+     * Internal function, do not override.
+     *
+     * @param string $query - the query that should be executed to build the list
+     * @param object $template - The object that should be used to copy the records
+     * @param array $field_list List of  fields.
+     * @return array
+     */
     function build_related_list2($query, &$template, &$field_list)
     {
         $GLOBALS['log']->debug("Finding linked values $this->object_name: ".$query);
@@ -5121,19 +5121,19 @@ 
         $row = $this->convertRow($row);
         $this->fetched_row = $row;
         $this->fromArray($row);
-		$this->is_updated_dependent_fields = false;
+        $this->is_updated_dependent_fields = false;
         $this->fill_in_additional_detail_fields();
         return $this;
     }
 
     /**
-    * This method is called during an import before inserting a bean
-    * Define an associative array called $special_fields
-    * the keys are user defined, and don't directly map to the bean's fields
-    * the value is the method name within that bean that will do extra
-    * processing for that field. example: 'full_name'=>'get_names_from_full_name'
-    *
-    */
+     * This method is called during an import before inserting a bean
+     * Define an associative array called $special_fields
+     * the keys are user defined, and don't directly map to the bean's fields
+     * the value is the method name within that bean that will do extra
+     * processing for that field. example: 'full_name'=>'get_names_from_full_name'
+     *
+     */
     function process_special_fields()
     {
         foreach ($this->special_functions as $func_name)
@@ -5267,8 +5267,8 @@ 
     {
         $where = '';
 
-		// make sure there is a date modified
-		$date_modified = $this->db->convert("'".$GLOBALS['timedate']->nowDb()."'", 'datetime');
+        // make sure there is a date modified
+        $date_modified = $this->db->convert("'".$GLOBALS['timedate']->nowDb()."'", 'datetime');
 
         $row=null;
         if($check_duplicates)
@@ -5345,30 +5345,30 @@ 
     }
 
     /**
-    * Trigger custom logic for this module that is defined for the provided hook
-    * The custom logic file is located under custom/modules/[CURRENT_MODULE]/logic_hooks.php.
-    * That file should define the $hook_version that should be used.
-    * It should also define the $hook_array.  The $hook_array will be a two dimensional array
-    * the first dimension is the name of the event, the second dimension is the information needed
-    * to fire the hook.  Each entry in the top level array should be defined on a single line to make it
-    * easier to automatically replace this file.  There should be no contents of this file that are not replacable.
-    *
-    * $hook_array['before_save'][] = Array(1, testtype, 'custom/modules/Leads/test12.php', 'TestClass', 'lead_before_save_1');
-    * This sample line creates a before_save hook.  The hooks are procesed in the order in which they
-    * are added to the array.  The second dimension is an array of:
-    *		processing index (for sorting before exporting the array)
-    *		A logic type hook
-    *		label/type
-    *		php file to include
-    *		php class the method is in
-    *		php method to call
-    *
-    * The method signature for version 1 hooks is:
-    * function NAME(&$bean, $event, $arguments)
-    * 		$bean - $this bean passed in by reference.
-    *		$event - The string for the current event (i.e. before_save)
-    * 		$arguments - An array of arguments that are specific to the event.
-    */
+     * Trigger custom logic for this module that is defined for the provided hook
+     * The custom logic file is located under custom/modules/[CURRENT_MODULE]/logic_hooks.php.
+     * That file should define the $hook_version that should be used.
+     * It should also define the $hook_array.  The $hook_array will be a two dimensional array
+     * the first dimension is the name of the event, the second dimension is the information needed
+     * to fire the hook.  Each entry in the top level array should be defined on a single line to make it
+     * easier to automatically replace this file.  There should be no contents of this file that are not replacable.
+     *
+     * $hook_array['before_save'][] = Array(1, testtype, 'custom/modules/Leads/test12.php', 'TestClass', 'lead_before_save_1');
+     * This sample line creates a before_save hook.  The hooks are procesed in the order in which they
+     * are added to the array.  The second dimension is an array of:
+     *		processing index (for sorting before exporting the array)
+     *		A logic type hook
+     *		label/type
+     *		php file to include
+     *		php class the method is in
+     *		php method to call
+     *
+     * The method signature for version 1 hooks is:
+     * function NAME(&$bean, $event, $arguments)
+     * 		$bean - $this bean passed in by reference.
+     *		$event - The string for the current event (i.e. before_save)
+     * 		$arguments - An array of arguments that are specific to the event.
+     */
     function call_custom_logic($event, $arguments = null)
     {
         if(!isset($this->processed) || $this->processed == false){
@@ -5432,14 +5432,14 @@ 
         return false;
     }
     /**
-    * Check whether the user has access to a particular view for the current bean/module
-    * @param $view string required, the view to determine access for i.e. DetailView, ListView...
-    * @param $is_owner bool optional, this is part of the ACL check if the current user is an owner they will receive different access
-    */
-	/* BEGIN - SECURITY GROUPS - aclaccess */  
-	/**
+     * Check whether the user has access to a particular view for the current bean/module
+     * @param $view string required, the view to determine access for i.e. DetailView, ListView...
+     * @param $is_owner bool optional, this is part of the ACL check if the current user is an owner they will receive different access
+     */
+    /* BEGIN - SECURITY GROUPS - aclaccess */  
+    /**
     function ACLAccess($view,$is_owner='not_set')
-	*/
+     */
     function ACLAccess($view,$is_owner='not_set',$in_group='not_set')
     {
         global $current_user;
@@ -5447,61 +5447,61 @@ 
             return true;
         }
         $not_set = false;
-		/**
+        /**
         if($is_owner == 'not_set')
-		*/
-    	if($is_owner === 'not_set') //eggsurplus: should be ===
+         */
+        if($is_owner === 'not_set') //eggsurplus: should be ===
         {
             $not_set = true;
             $is_owner = $this->isOwner($current_user->id);
         }
-		// DJM - OBS Customizations - May 2009
-		// Moved this code to convert to lowercase from below.
-		// Added new action variable.
-		$view = strtolower($view);
-		$action = '';
-		// DJM - OBS Customizations - END CHANGE
-    	if($in_group === 'not_set')
-    	{
-			require_once("modules/SecurityGroups/SecurityGroup.php");
-			// DJM - OBS Customizations - May 2009
-			// Added the following switch statement to convert the view
-			// into an action value.  As per the switch below.
-			// Added the action parameter to the groupHasAccess call.
-    			switch ($view)
-    			{
-    				case 'list':
-    				case 'index':
-    				case 'listview':
-    					$action = "list";
-					break;
-    				case 'edit':
-    				case 'save':
-		    		case 'popupeditview':
- 		   		case 'editview':
-  		  			$action = "edit";
-					break;
- 		   		case 'view':
- 		   		case 'detail':
- 		   		case 'detailview':
- 		   			$action = "view";
-					break;
- 		   		case 'delete':
- 		   			$action = "delete" ;
-					break;
- 		   		case 'export':
- 		   			$action = "export";
-					break;
- 		   		case 'import':
-  		  			$action = "import";
-					break;
-				default:
-					$action = "";
-					break;
-    			}
-			$in_group = SecurityGroup::groupHasAccess($this->module_dir,$this->id, $action); 
-			// DJM - OBS Customizations - END CHANGE
-    	}
+        // DJM - OBS Customizations - May 2009
+        // Moved this code to convert to lowercase from below.
+        // Added new action variable.
+        $view = strtolower($view);
+        $action = '';
+        // DJM - OBS Customizations - END CHANGE
+        if($in_group === 'not_set')
+        {
+            require_once("modules/SecurityGroups/SecurityGroup.php");
+            // DJM - OBS Customizations - May 2009
+            // Added the following switch statement to convert the view
+            // into an action value.  As per the switch below.
+            // Added the action parameter to the groupHasAccess call.
+                switch ($view)
+                {
+                    case 'list':
+                    case 'index':
+                    case 'listview':
+                        $action = "list";
+                    break;
+                    case 'edit':
+                    case 'save':
+                    case 'popupeditview':
+                    case 'editview':
+                        $action = "edit";
+                    break;
+                    case 'view':
+                    case 'detail':
+                    case 'detailview':
+                        $action = "view";
+                    break;
+                    case 'delete':
+                        $action = "delete" ;
+                    break;
+                    case 'export':
+                        $action = "export";
+                    break;
+                    case 'import':
+                        $action = "import";
+                    break;
+                default:
+                    $action = "";
+                    break;
+                }
+            $in_group = SecurityGroup::groupHasAccess($this->module_dir,$this->id, $action); 
+            // DJM - OBS Customizations - END CHANGE
+        }
         //if we don't implent acls return true
         if(!$this->bean_implements('ACL'))
         return true;
@@ -5511,10 +5511,10 @@ 
             case 'list':
             case 'index':
             case 'listview':
-				/**
+                /**
                 return ACLController::checkAccess($this->module_dir,'list', true);
-				*/
-    			return ACLController::checkAccess($this->module_dir,'list', true, $this->acltype, $in_group);
+                 */
+                return ACLController::checkAccess($this->module_dir,'list', true, $this->acltype, $in_group);
             case 'edit':
             case 'save':
                 if( !$is_owner && $not_set && !empty($this->id)){
@@ -5529,32 +5529,32 @@ 
                 }
             case 'popupeditview':
             case 'editview':
-				/**
+                /**
                 return ACLController::checkAccess($this->module_dir,'edit', $is_owner, $this->acltype);
-				*/
-    			return ACLController::checkAccess($this->module_dir,'edit', $is_owner, $this->acltype, $in_group);
+                 */
+                return ACLController::checkAccess($this->module_dir,'edit', $is_owner, $this->acltype, $in_group);
             case 'view':
             case 'detail':
             case 'detailview':
-				/**
+                /**
                 return ACLController::checkAccess($this->module_dir,'view', $is_owner, $this->acltype);
-				*/
-    			return ACLController::checkAccess($this->module_dir,'view', $is_owner, $this->acltype, $in_group);
+                 */
+                return ACLController::checkAccess($this->module_dir,'view', $is_owner, $this->acltype, $in_group);
             case 'delete':
-				/**
+                /**
                 return ACLController::checkAccess($this->module_dir,'delete', $is_owner, $this->acltype);
-				*/
-    			return ACLController::checkAccess($this->module_dir,'delete', $is_owner, $this->acltype, $in_group);
+                 */
+                return ACLController::checkAccess($this->module_dir,'delete', $is_owner, $this->acltype, $in_group);
             case 'export':
-				/**
+                /**
                 return ACLController::checkAccess($this->module_dir,'export', $is_owner, $this->acltype);
-				*/
-    			return ACLController::checkAccess($this->module_dir,'export', $is_owner, $this->acltype, $in_group);
+                 */
+                return ACLController::checkAccess($this->module_dir,'export', $is_owner, $this->acltype, $in_group);
             case 'import':
-				/**
+                /**
                 return ACLController::checkAccess($this->module_dir,'import', true, $this->acltype);
-				*/
-    			return ACLController::checkAccess($this->module_dir,'import', true, $this->acltype, $in_group);
+                 */
+                return ACLController::checkAccess($this->module_dir,'import', true, $this->acltype, $in_group);
         }
         //if it is not one of the above views then it should be implemented on the page level
         return true;
@@ -5562,10 +5562,10 @@ 
     /* END - SECURITY GROUPS */
 
     /**
-    * Get owner field
-    *
-    * @return STRING
-    */
+     * Get owner field
+     *
+     * @return STRING
+     */
     function getOwnerField($returnFieldName = false)
     {
         if (isset($this->field_defs['assigned_user_id']))
@@ -5582,11 +5582,11 @@ 
     }
 
     /**
-    * Returns true of false if the user_id passed is the owner
-    *
-    * @param GUID $user_id
-    * @return boolean
-    */
+     * Returns true of false if the user_id passed is the owner
+     *
+     * @param GUID $user_id
+     * @return boolean
+     */
     function isOwner($user_id)
     {
         //if we don't have an id we must be the owner as we are creating it
@@ -5615,11 +5615,11 @@ 
         return false;
     }
     /**
-    * Gets there where statement for checking if a user is an owner
-    *
-    * @param GUID $user_id
-    * @return STRING
-    */
+     * Gets there where statement for checking if a user is an owner
+     *
+     * @param GUID $user_id
+     * @return STRING
+     */
     function getOwnerWhere($user_id)
     {
         if(isset($this->field_defs['assigned_user_id']))
@@ -5634,12 +5634,12 @@ 
     }
 
     /**
-    *
-    * Used in order to manage ListView links and if they should
-    * links or not based on the ACL permissions of the user
-    *
-    * @return ARRAY of STRINGS
-    */
+     *
+     * Used in order to manage ListView links and if they should
+     * links or not based on the ACL permissions of the user
+     *
+     * @return ARRAY of STRINGS
+     */
     function listviewACLHelper()
     {
         $array_assign = array();
@@ -5655,10 +5655,10 @@ 
     }
 
     /**
-    * returns this bean as an array
-    *
-    * @return array of fields with id, name, access and category
-    */
+     * returns this bean as an array
+     *
+     * @return array of fields with id, name, access and category
+     */
     function toArray($dbOnly = false, $stringOnly = false, $upperKeys=false)
     {
         static $cache = array();
@@ -5688,10 +5688,10 @@ 
     }
 
     /**
-    * Converts an array into an acl mapping name value pairs into files
-    *
-    * @param Array $arr
-    */
+     * Converts an array into an acl mapping name value pairs into files
+     *
+     * @param Array $arr
+     */
     function fromArray($arr)
     {
         foreach($arr as $name=>$value)
@@ -5709,13 +5709,13 @@ 
     public function convertRow($row)
     {
         foreach($this->field_defs as $name => $fieldDef)
-		{
-		    // skip empty fields and non-db fields
+        {
+            // skip empty fields and non-db fields
             if (isset($name) && !empty($row[$name])) {
                 $row[$name] = $this->convertField($row[$name], $fieldDef);
             }
         }
-		return $row;
+        return $row;
     }
 
     /**
@@ -5763,18 +5763,18 @@ 
         return !empty($GLOBALS['dictionary'][$this->object_name]['custom_fields']);
     }
 
-   /**
-    * Ensure that fields within order by clauses are properly qualified with
-    * their tablename.  This qualification is a requirement for sql server support.
-    *
-    * @param string $order_by original order by from the query
-    * @param string $qualify prefix for columns in the order by list.
-    * @return  prefixed
-    *
-    * Internal function do not override.
-    */
-   function create_qualified_order_by( $order_by, $qualify)
-   {	// if the column is empty, but the sort order is defined, the value will throw an error, so do not proceed if no order by is given
+    /**
+     * Ensure that fields within order by clauses are properly qualified with
+     * their tablename.  This qualification is a requirement for sql server support.
+     *
+     * @param string $order_by original order by from the query
+     * @param string $qualify prefix for columns in the order by list.
+     * @return  prefixed
+     *
+     * Internal function do not override.
+     */
+    function create_qualified_order_by( $order_by, $qualify)
+    {	// if the column is empty, but the sort order is defined, the value will throw an error, so do not proceed if no order by is given
     if (empty($order_by))
     {
         return $order_by;
@@ -5789,17 +5789,17 @@ 
         $comma = ", ";
     }
     return $order_by_clause;
-   }
+    }
 
-   /**
-    * Combined the contents of street field 2 thru 4 into the main field
-    *
-    * @param string $street_field
-    */
+    /**
+     * Combined the contents of street field 2 thru 4 into the main field
+     *
+     * @param string $street_field
+     */
 
-   function add_address_streets(
-       $street_field
-       )
+    function add_address_streets(
+        $street_field
+        )
     {
         if (isset($this->$street_field)) {
             $street_field_2 = $street_field.'_2';
@@ -5853,9 +5853,9 @@ 
     }
 
     /**
-    * Moved from save() method, functionality is the same, but this is intended to handle
-    * Optimistic locking functionality.
-    */
+     * Moved from save() method, functionality is the same, but this is intended to handle
+     * Optimistic locking functionality.
+     */
     private function _checkOptimisticLocking($action, $isUpdate){
         if($this->optimistic_lock && !isset($_SESSION['o_lock_fs'])){
             if(isset($_SESSION['o_lock_id']) && $_SESSION['o_lock_id'] == $this->id && $_SESSION['o_lock_on'] == $this->object_name)
@@ -5894,8 +5894,8 @@ 
     }
 
     /**
-    * Send assignment notifications and invites for meetings and calls
-    */
+     * Send assignment notifications and invites for meetings and calls
+     */
     private function _sendNotifications($check_notify){
         if($check_notify || (isset($this->notify_inworkflow) && $this->notify_inworkflow == true) // cn: bug 5795 - no invites sent to Contacts, and also bug 25995, in workflow, it will set the notify_on_save=true.
            && !$this->isOwner($this->created_by) )  // cn: bug 42727 no need to send email to owner (within workflow)
@@ -5983,11 +5983,11 @@ 
         $result = sugar_cache_retrieve($cache_key);
         if(!empty($result))
         {
-        	// Use SugarCache::EXTERNAL_CACHE_NULL_VALUE to store null values in the cache.
-        	if($result == SugarCache::EXTERNAL_CACHE_NULL_VALUE)
-        	{
-        		return null;
-        	}
+            // Use SugarCache::EXTERNAL_CACHE_NULL_VALUE to store null values in the cache.
+            if($result == SugarCache::EXTERNAL_CACHE_NULL_VALUE)
+            {
+                return null;
+            }
 
             return $result;
         }
@@ -6004,7 +6004,7 @@ 
             if(empty($moduleDefs[$module]) || empty($moduleDefs[$module][$module][$key]))
             {
                 // It was not loaded....  Fail.  Cache null to prevent future repeats of this calculation
-				sugar_cache_put($cache_key, SugarCache::EXTERNAL_CACHE_NULL_VALUE);
+                sugar_cache_put($cache_key, SugarCache::EXTERNAL_CACHE_NULL_VALUE);
                 return  null;
             }
 
@@ -6015,8 +6015,8 @@ 
 
         // It was not loaded....  Fail.  Cache null to prevent future repeats of this calculation
         sugar_cache_put($cache_key, SugarCache::EXTERNAL_CACHE_NULL_VALUE);
-		return null;
-	}
+        return null;
+    }
 
     /**
      * Returns the ACL category for this module; defaults to the SugarBean::$acl_category if defined
@@ -6037,10 +6037,10 @@ 
      * @param string $where
      * @return string SQL query
      */
-	public function create_export_query($order_by, $where)
-	{
-		return $this->create_new_list_query($order_by, $where, array(), array(), 0, '', false, $this, true, true);
-	}
+    public function create_export_query($order_by, $where)
+    {
+        return $this->create_new_list_query($order_by, $where, array(), array(), 0, '', false, $this, true, true);
+    }
 
     /**
      * Determine whether the given field is a relate field

Braces +338 added lines, -325 removed lines

@@ -1,5 +1,7 @@ 
 <?php
-if(!defined('sugarEntry') || !sugarEntry) die('Not A Valid Entry Point');
+if(!defined('sugarEntry') || !sugarEntry) {
+    die('Not A Valid Entry Point');
+}
 /*********************************************************************************
  * SugarCRM Community Edition is a customer relationship management program developed by
  * SugarCRM, Inc. Copyright (C) 2004-2013 SugarCRM Inc.
@@ -315,8 +317,9 @@ 
         global  $dictionary, $current_user;
         static $loaded_defs = array();
         $this->db = DBManagerFactory::getInstance();
-        if (empty($this->module_name))
-            $this->module_name = $this->module_dir;
+        if (empty($this->module_name)) {
+                    $this->module_name = $this->module_dir;
+        }
         if((false == $this->disable_vardefs && empty($loaded_defs[$this->object_name])) || !empty($GLOBALS['reload_vardefs']))
         {
             VardefManager::loadVardef($this->module_dir, $this->object_name);
@@ -339,12 +342,15 @@ 
                 $this->setupCustomFields($this->module_dir);
             }
             //load up field_arrays from CacheHandler;
-            if(empty($this->list_fields))
-                $this->list_fields = $this->_loadCachedArray($this->module_dir, $this->object_name, 'list_fields');
-            if(empty($this->column_fields))
-                $this->column_fields = $this->_loadCachedArray($this->module_dir, $this->object_name, 'column_fields');
-            if(empty($this->required_fields))
-                $this->required_fields = $this->_loadCachedArray($this->module_dir, $this->object_name, 'required_fields');
+            if(empty($this->list_fields)) {
+                            $this->list_fields = $this->_loadCachedArray($this->module_dir, $this->object_name, 'list_fields');
+            }
+            if(empty($this->column_fields)) {
+                            $this->column_fields = $this->_loadCachedArray($this->module_dir, $this->object_name, 'column_fields');
+            }
+            if(empty($this->required_fields)) {
+                            $this->required_fields = $this->_loadCachedArray($this->module_dir, $this->object_name, 'required_fields');
+            }
 
             if(isset($GLOBALS['dictionary'][$this->object_name]) && !$this->disable_vardefs)
             {
@@ -361,8 +367,7 @@ 
             $loaded_defs[$this->object_name]['required_fields'] =& $this->required_fields;
             $loaded_defs[$this->object_name]['field_name_map'] =& $this->field_name_map;
             $loaded_defs[$this->object_name]['field_defs'] =& $this->field_defs;
-        }
-        else
+        } else
         {
             $this->column_fields =& $loaded_defs[$this->object_name]['column_fields'] ;
             $this->list_fields =& $loaded_defs[$this->object_name]['list_fields'];
@@ -400,8 +405,9 @@ 
      */
     function getObjectName()
     {
-        if ($this->object_name)
-            return $this->object_name;
+        if ($this->object_name) {
+                    return $this->object_name;
+        }
 
         // This is a quick way out. The generated metadata files have the table name
         // as the key. The correct way to do this is to override this function
@@ -459,8 +465,7 @@ 
         if (isset($dictionary[$this->getObjectName()]['audited']))
         {
             return $dictionary[$this->getObjectName()]['audited'];
-        }
-        else
+        } else
         {
             return false;
         }
@@ -598,8 +603,9 @@ 
      */
     function getFieldDefinition($name)
     {
-        if ( !isset($this->field_defs[$name]) )
-            return false;
+        if ( !isset($this->field_defs[$name]) ) {
+                    return false;
+        }
 
         return $this->field_defs[$name];
     }
@@ -657,8 +663,9 @@ 
      */
     public function unPopulateDefaultValues()
     {
-        if ( !is_array($this->field_defs) )
-            return;
+        if ( !is_array($this->field_defs) ) {
+                    return;
+        }
 
         foreach ($this->field_defs as $field => $value) {
 		    if( !empty($this->$field)
@@ -698,8 +705,9 @@ 
     }
 
     function populateDefaultValues($force=false){
-        if ( !is_array($this->field_defs) )
-            return;
+        if ( !is_array($this->field_defs) ) {
+                    return;
+        }
         foreach($this->field_defs as $field=>$value){
             if((isset($value['default']) || !empty($value['display_default'])) && ($force || empty($this->$field))){
                 $type = $value['type'];
@@ -717,10 +725,11 @@ 
                         }
                         break;
                     case 'multienum':
-                        if(empty($value['default']) && !empty($value['display_default']))
-                            $this->$field = $value['display_default'];
-                        else
-                            $this->$field = $value['default'];
+                        if(empty($value['default']) && !empty($value['display_default'])) {
+                                                    $this->$field = $value['display_default'];
+                        } else {
+                                                    $this->$field = $value['default'];
+                        }
                         break;
                     case 'bool':
                     	if(isset($this->$field)){
@@ -770,8 +779,7 @@ 
         {
             $GLOBALS['log']->fatal("removeRelationshipMeta: Metadata for table ".$tablename. " does not exist");
             display_notice("meta data absent for table ".$tablename." keyed to $key ");
-        }
-        else
+        } else
         {
             if (isset($dictionary[$key]['relationships']))
             {
@@ -821,8 +829,7 @@ 
             if($iscustom)
             {
                 $filename='custom/modules/' . $module_dir . '/Ext/Vardefs/vardefs.ext.php';
-            }
-            else
+            } else
             {
                 if ($key == 'User')
                 {
@@ -830,8 +837,7 @@ 
                     // this must be done because the Employees/vardefs.php does an include_once on
                     // Users/vardefs.php
                     $filename='modules/Users/vardefs.php';
-                }
-                else
+                } else
                 {
                     $filename='modules/'. $module_dir . '/vardefs.php';
                 }
@@ -845,8 +851,7 @@ 
                 {
                     $dictionary = $GLOBALS['dictionary'];
                 }
-            }
-            else
+            } else
             {
                 $GLOBALS['log']->debug("createRelationshipMeta: no metadata file found" . $filename);
                 return;
@@ -857,8 +862,7 @@ 
         {
             $GLOBALS['log']->fatal("createRelationshipMeta: Metadata for table ".$tablename. " does not exist");
             display_notice("meta data absent for table ".$tablename." keyed to $key ");
-        }
-        else
+        } else
         {
             if (isset($dictionary[$key]['relationships']))
             {
@@ -884,8 +888,7 @@ 
                     if (Relationship::exists($rel_name,$db))
                     {
                         $GLOBALS['log']->debug('Skipping, reltionship already exists '.$rel_name);
-                    }
-                    else
+                    } else
                     {
                         $seed = BeanFactory::getBean("Relationships");
                         $keys = array_keys($seed->field_defs);
@@ -895,12 +898,10 @@ 
                             if ($key == "id")
                             {
                                 $toInsert[$key] = create_guid();
-                            }
-                            else if ($key == "relationship_name")
+                            } else if ($key == "relationship_name")
                             {
                                 $toInsert[$key] = $rel_name;
-                            }
-                            else if (isset($rel_def[$key]))
+                            } else if (isset($rel_def[$key]))
                             {
                                 $toInsert[$key] = $rel_def[$key];
                             }
@@ -916,8 +917,7 @@ 
                         $db->query($insert_string, true);
                     }
                 }
-            }
-            else
+            } else
             {
                 //todo
                 //log informational message stating no relationships meta was set for this bean.
@@ -986,10 +986,11 @@ 
             //if rel_name is provided, search the fieldef array keys by name.
             if (isset($fieldDefs[$rel_name]['type']) && $fieldDefs[$rel_name]['type'] == 'link')
             {
-                if ($class == "Link2")
-                    $this->$rel_name = new $class($rel_name, $this);
-                else
-                    $this->$rel_name = new $class($fieldDefs[$rel_name]['relationship'], $this, $fieldDefs[$rel_name]);
+                if ($class == "Link2") {
+                                    $this->$rel_name = new $class($rel_name, $this);
+                } else {
+                                    $this->$rel_name = new $class($fieldDefs[$rel_name]['relationship'], $this, $fieldDefs[$rel_name]);
+                }
 
                 if (empty($this->$rel_name) ||
                         (method_exists($this->$rel_name, "loadedSuccesfully") && !$this->$rel_name->loadedSuccesfully()))
@@ -1048,8 +1049,9 @@ 
     function get_linked_beans($field_name,$bean_name = '', $order_by = '', $begin_index = 0, $end_index = -1, $deleted=0, $optional_where="")
     {
         //if bean_name is Case then use aCase
-        if($bean_name=="Case")
-            $bean_name = "aCase";
+        if($bean_name=="Case") {
+                    $bean_name = "aCase";
+        }
 
         if($this->load_relationship($field_name)) {
             if ($this->$field_name instanceof Link) {
@@ -1057,19 +1059,20 @@ 
                 return array_values($this->$field_name->getBeans(new $bean_name(), $order_by, $begin_index, $end_index, $deleted, $optional_where));
             } else {
                 // Link2 style
-                if ($end_index != -1 || !empty($deleted) || !empty($optional_where) || !empty($order_by))
-                    return array_values($this->$field_name->getBeans(array(
+                if ($end_index != -1 || !empty($deleted) || !empty($optional_where) || !empty($order_by)) {
+                                    return array_values($this->$field_name->getBeans(array(
                         'where' => $optional_where,
                         'deleted' => $deleted,
                         'limit' => ($end_index - $begin_index),
                         'order_by' => $order_by
                     )));
-                else
-                    return array_values($this->$field_name->getBeans());
+                } else {
+                                    return array_values($this->$field_name->getBeans());
+                }
             }
+        } else {
+                    return array();
         }
-        else
-            return array();
     }
 
     /**
@@ -1131,8 +1134,7 @@ 
                            || is_bool($value_array['importable']) && $value_array['importable'] == true)) {
                         $importableFields[$key]=$value_array;
                     }
-                }
-                else {
+                } else {
 
                     //Expose the cooresponding id field of a relate field if it is only defined as a link so that users can relate records by id during import
                     if( isset($value_array['type']) && ($value_array['type'] == 'relate') && isset($value_array['id_name']) )
@@ -1217,8 +1219,7 @@ 
             if ($this->load_relationship($name))
             {
                 $this->$name->delete($id);
-            }
-            else
+            } else
             {
                 $GLOBALS['log']->fatal("error loading relationship $name");
             }
@@ -1239,8 +1240,7 @@ 
         {
             $GLOBALS['log']->fatal("create_tables: Metadata for table ".$this->table_name. " does not exist");
             display_notice("meta data absent for table ".$this->table_name." keyed to $key ");
-        }
-        else
+        } else
         {
             if(!$this->db->tableExists($this->table_name))
             {
@@ -1248,12 +1248,11 @@ 
                     if($this->bean_implements('ACL')){
                         if(!empty($this->acltype)){
                             ACLAction::addActions($this->getACLCategory(), $this->acltype);
-                        }else{
+                        } else{
                             ACLAction::addActions($this->getACLCategory());
                         }
                     }
-            }
-            else
+            } else
             {
                 echo "Table already exists : $this->table_name<br>";
             }
@@ -1281,10 +1280,13 @@ 
             $GLOBALS['log']->fatal("drop_tables: Metadata for table ".$this->table_name. " does not exist");
             echo "meta data absent for table ".$this->table_name."<br>\n";
         } else {
-            if(empty($this->table_name))return;
-            if ($this->db->tableExists($this->table_name))
-
+            if(empty($this->table_name)) {
+                return;
+            }
+            if ($this->db->tableExists($this->table_name)) {
+            
                 $this->db->dropTable($this);
+            }
             if ($this->db->tableExists($this->table_name. '_cstm'))
             {
                 $this->db->dropTableName($this->table_name. '_cstm');
@@ -1322,8 +1324,9 @@ 
             if (isset($def['type'])) {
                 $type=$def['type'];
             }
-            if(isset($def['dbType']))
-                $type .= $def['dbType'];
+            if(isset($def['dbType'])) {
+                            $type .= $def['dbType'];
+            }
 
             if($def['type'] == 'html' || $def['type'] == 'longhtml') {
                 $this->$key = SugarCleaner::cleanHtml($this->$key, true);
@@ -1373,7 +1376,9 @@ 
 
         $this->_checkOptimisticLocking($action, $isUpdate);
 
-        if(!empty($this->modified_by_name)) $this->old_modified_by_name = $this->modified_by_name;
+        if(!empty($this->modified_by_name)) {
+            $this->old_modified_by_name = $this->modified_by_name;
+        }
         if($this->update_modified_by)
         {
             $this->modified_user_id = 1;
@@ -1384,8 +1389,9 @@ 
                 $this->modified_by_name = $current_user->user_name;
             }
         }
-        if ($this->deleted != 1)
-            $this->deleted = 0;
+        if ($this->deleted != 1) {
+                    $this->deleted = 0;
+        }
         if(!$isUpdate)
         {
             if (empty($this->date_entered))
@@ -1607,7 +1613,7 @@ 
 
                 if(!$notify_mail->Send()) {
                     $GLOBALS['log']->fatal("Notifications: error sending e-mail (method: {$notify_mail->Mailer}), (error: {$notify_mail->ErrorInfo})");
-                }else{
+                } else{
                     $GLOBALS['log']->info("Notifications: e-mail successfully sent");
                 }
             }
@@ -1646,8 +1652,7 @@ 
         $xtpl = new XTemplate(get_notify_template_file($current_language));
         if($this->module_dir == "Cases") {
             $template_name = "Case"; //we should use Case, you can refer to the en_us.notify_template.html.
-        }
-        else {
+        } else {
             $template_name = $beanList[$this->module_dir]; //bug 20637, in workflow this->object_name = strange chars.
         }
 
@@ -1745,8 +1750,7 @@ 
             // if we should use relation data from properties (for REQUEST-independent calls)
             $rel_id = isset($this->new_rel_id) ? $this->new_rel_id : '';
             $rel_link = isset($this->new_rel_relname) ? $this->new_rel_relname : '';
-        }
-        else
+        } else
         {
             // if we should use relation data from REQUEST
             $rel_id = isset($_REQUEST['relate_id']) ? $_REQUEST['relate_id'] : '';
@@ -1765,8 +1769,7 @@ 
             if ($isRelationshipLoaded && !empty($this->$rel_link) && $this->$rel_link->getRelationshipObject() && $this->$rel_link->getRelationshipObject()->getLHSModule() == $this->$rel_link->getRelationshipObject()->getRHSModule() )
             {
                 $new_rel_link = $this->$rel_link->getRelationshipObject()->getLHSLink();
-            }
-            else
+            } else
             {
                 //Try to find the link in this bean based on the relationship
                 foreach ($this->field_defs as $key => $def)
@@ -1800,7 +1803,9 @@ 
             foreach ($this->relationship_fields as $id => $rel_name)
             {
 
-                if (in_array($id, $exclude)) continue;
+                if (in_array($id, $exclude)) {
+                    continue;
+                }
 
                 if(!empty($this->$id))
                 {
@@ -1853,8 +1858,10 @@ 
         foreach ($this->field_defs as $def)
         {
             if ($def ['type'] == 'relate' && isset ($def ['id_name']) && isset ($def ['link']) && isset ($def['save'])) {
-                if (in_array($def['id_name'], $exclude) || in_array($def['id_name'], $this->relationship_fields))
-                    continue; // continue to honor the exclude array and exclude any relationships that will be handled by the relationship_fields mechanism
+                if (in_array($def['id_name'], $exclude) || in_array($def['id_name'], $this->relationship_fields)) {
+                                    continue;
+                }
+                // continue to honor the exclude array and exclude any relationships that will be handled by the relationship_fields mechanism
 
                 $linkField = $def ['link'];
                 if (isset($this->field_defs[$linkField])) {
@@ -1911,12 +1918,14 @@ 
         {
             if (!empty($def['type']) && $def['type'] == "parent")
             {
-                if (empty($def['type_name']) || empty($def['id_name']))
-                    continue;
+                if (empty($def['type_name']) || empty($def['id_name'])) {
+                                    continue;
+                }
                 $typeField = $def['type_name'];
                 $idField = $def['id_name'];
-                if (in_array($idField, $exclude))
-                    continue;
+                if (in_array($idField, $exclude)) {
+                                    continue;
+                }
                 //Determine if the parent field has changed.
                 if (
                     //First check if the fetched row parent existed and now we no longer have one
@@ -2039,8 +2048,9 @@ 
 	{
 		global $disable_date_format;
 		global $timedate;
-		if (empty($timedate))
-			$timedate=TimeDate::getInstance();
+		if (empty($timedate)) {
+					$timedate=TimeDate::getInstance();
+		}
 
 		if(empty($this->field_defs))
 		{
@@ -2052,15 +2062,16 @@ 
 			if(!isset($this->processed_dates_times[$field]))
 			{
 				$this->processed_dates_times[$field] = '1';
-				if(empty($this->$field)) continue;
+				if(empty($this->$field)) {
+				    continue;
+				}
 				if($field == 'date_modified' || $field == 'date_entered')
 				{
 					$this->$field = $this->db->fromConvert($this->$field, 'datetime');
 					if(empty($disable_date_format)) {
 						$this->$field = $timedate->to_display_date_time($this->$field);
 					}
-				}
-				elseif(isset($this->field_name_map[$field]['type']))
+				} elseif(isset($this->field_name_map[$field]['type']))
 				{
 					$type = $this->field_name_map[$field]['type'];
 
@@ -2086,8 +2097,7 @@ 
 									$this->$rel_field = $timedate->to_display_time($mergetime);
 								}
 							}
-						}
-						else
+						} else
 						{
 							if(empty($disable_date_format)) {
 								$this->$field = $timedate->to_display_date($this->$field, false);
@@ -2098,8 +2108,7 @@ 
 						if($this->$field == '0000-00-00 00:00:00')
 						{
 							$this->$field = '';
-						}
-						else
+						} else
 						{
 							if(empty($disable_date_format)) {
 								$this->$field = $timedate->to_display_date_time($this->$field, true, true);
@@ -2188,7 +2197,9 @@ 
             switch($def['type']) {
                 case 'datetime':
                 case 'datetimecombo':
-                    if(empty($this->$field)) break;
+                    if(empty($this->$field)) {
+                        break;
+                    }
                     if ($this->$field == 'NULL') {
                     	$this->$field = '';
                     	break;
@@ -2200,7 +2211,9 @@ 
                     }
                     break;
                 case 'date':
-                    if(empty($this->$field)) break;
+                    if(empty($this->$field)) {
+                        break;
+                    }
                     if ($this->$field == 'NULL') {
                     	$this->$field = '';
                     	break;
@@ -2212,7 +2225,9 @@ 
                     }
                     break;
                 case 'time':
-                    if(empty($this->$field)) break;
+                    if(empty($this->$field)) {
+                        break;
+                    }
                     if ($this->$field == 'NULL') {
                     	$this->$field = '';
                     	break;
@@ -2301,7 +2316,9 @@ 
 
         $query .= $custom_join['join'];
         $query .= " WHERE $this->table_name.id = ".$this->db->quoted($id);
-        if ($deleted) $query .= " AND $this->table_name.deleted=0";
+        if ($deleted) {
+            $query .= " AND $this->table_name.deleted=0";
+        }
         $GLOBALS['log']->debug("Retrieve $this->object_name : ".$query);
         $result = $this->db->limitQuery($query,0,1,true, "Retrieving record by id $this->table_name:$id found ");
         if(empty($result))
@@ -2359,21 +2376,23 @@ 
             if (isset($def [ 'type' ]) && $def [ 'type' ] == 'relate' && isset ( $def [ 'id_name'] ) && isset ( $def [ 'link'] ) && isset ( $def[ 'save' ])) {
                 if (isset($this->$key)) {
                     $this->rel_fields_before_value[$key]=$this->$key;
-                    if (isset($this->$def [ 'id_name']))
-                        $this->rel_fields_before_value[$def [ 'id_name']]=$this->$def [ 'id_name'];
+                    if (isset($this->$def [ 'id_name'])) {
+                                            $this->rel_fields_before_value[$def [ 'id_name']]=$this->$def [ 'id_name'];
+                    }
+                } else {
+                                    $this->rel_fields_before_value[$key]=null;
                 }
-                else
-                    $this->rel_fields_before_value[$key]=null;
            }
         }
         if (isset($this->relationship_fields) && is_array($this->relationship_fields))
         {
             foreach ($this->relationship_fields as $rel_id=>$rel_name)
             {
-                if (isset($this->$rel_id))
-                    $this->rel_fields_before_value[$rel_id]=$this->$rel_id;
-                else
-                    $this->rel_fields_before_value[$rel_id]=null;
+                if (isset($this->$rel_id)) {
+                                    $this->rel_fields_before_value[$rel_id]=$this->$rel_id;
+                } else {
+                                    $this->rel_fields_before_value[$rel_id]=null;
+                }
             }
         }
 
@@ -2399,8 +2418,9 @@ 
         $nullvalue='';
         foreach($this->field_defs as $field=>$field_value)
         {
-            if(($field == 'user_preferences' && $this->module_dir == 'Users') || ($field == 'internal' && $this->module_dir == 'Cases') )
-                continue;
+            if(($field == 'user_preferences' && $this->module_dir == 'Users') || ($field == 'internal' && $this->module_dir == 'Cases') ) {
+                            continue;
+            }
             if(isset($row[$field]))
             {
                 $this->$field = $row[$field];
@@ -2408,8 +2428,7 @@ 
                 if(!empty($row[$owner])){
                     $this->$owner = $row[$owner];
                 }
-            }
-            else
+            } else
             {
                 $this->$field = $nullvalue;
             }
@@ -2453,10 +2472,11 @@ 
         //handle distinct clause
         $star = '*';
         if(substr_count(strtolower($query), 'distinct')){
-            if (!empty($this->seed) && !empty($this->seed->table_name ))
-                $star = 'DISTINCT ' . $this->seed->table_name . '.id';
-            else
-                $star = 'DISTINCT ' . $this->table_name . '.id';
+            if (!empty($this->seed) && !empty($this->seed->table_name )) {
+                            $star = 'DISTINCT ' . $this->seed->table_name . '.id';
+            } else {
+                            $star = 'DISTINCT ' . $this->table_name . '.id';
+            }
 
         }
 
@@ -2477,10 +2497,11 @@ 
                 preg_match($pattern, $union_query, $matches);
                 if (!empty($matches)) {
                     if (stristr($matches[0], "distinct")) {
-                        if (!empty($this->seed) && !empty($this->seed->table_name ))
-                            $star = 'DISTINCT ' . $this->seed->table_name . '.id';
-                        else
-                            $star = 'DISTINCT ' . $this->table_name . '.id';
+                        if (!empty($this->seed) && !empty($this->seed->table_name )) {
+                                                    $star = 'DISTINCT ' . $this->seed->table_name . '.id';
+                        } else {
+                                                    $star = 'DISTINCT ' . $this->table_name . '.id';
+                        }
                     }
                 } // if
                 $replacement = 'SELECT count(' . $star . ') c FROM ';
@@ -2530,7 +2551,7 @@ 
             if(!empty($owner_where)){
                 if(empty($where)){
                     $where = $owner_where;
-                }else{
+                } else{
                     $where .= ' AND '.  $owner_where;
                 }
             }
@@ -2551,14 +2572,14 @@ 
     */
     public function process_order_by($order_by, $submodule = null, $suppress_table_name = false)
     {
-        if (empty($order_by))
-            return $order_by;
+        if (empty($order_by)) {
+                    return $order_by;
+        }
         //submodule is empty,this is for list object in focus
         if (empty($submodule))
         {
             $bean_queried = $this;
-        }
-        else
+        } else
         {
             //submodule is set, so this is for subpanel, use submodule
             $bean_queried = $submodule;
@@ -2653,8 +2674,7 @@ 
             if(empty($where))
             {
                 $where = $owner_where;
-            }
-            else
+            } else
             {
                 $where .= ' AND '.  $owner_where;
             }
@@ -2740,8 +2760,7 @@ 
             if(empty($entire_where))
             {
                 $entire_where = ' WHERE ' . $where;
-            }
-            else
+            } else
             {
                 $entire_where .= ' AND ' . $where;
             }
@@ -2781,16 +2800,15 @@ 
                             }
                             //call function from required file
                             $query_array = $shortcut_function_name($parameters);
-                        }else{
+                        } else{
                             //call function from parent bean
                             $query_array = $parentbean->$shortcut_function_name($parameters);
                         }
-                    }
-                    else
+                    } else
                     {
                         $query_array = $parentbean->$shortcut_function_name();
                     }
-                }  else {
+                } else {
                     $related_field_name = $this_subpanel->get_data_source_name();
                     if (!$parentbean->load_relationship($related_field_name)){
                         unset ($parentbean->$related_field_name);
@@ -2806,8 +2824,7 @@ 
                     if(empty($where_definition))
                     {
                         $where_definition = $table_where;
-                    }
-                    else
+                    } else
                     {
                         $where_definition .= ' AND ' . $table_where;
                     }
@@ -2898,8 +2915,7 @@ 
         {
             $subpanel_def->load_sub_subpanels();
             $subpanel_list=$subpanel_def->sub_subpanels;
-        }
-        else
+        } else
         {
             $subpanel_list[]=$subpanel_def;
         }
@@ -2926,7 +2942,7 @@ 
                         }
                         //call function from required file
                         $tmp_final_query =  $shortcut_function_name($parameters);
-                    }else{
+                    } else{
                         //call function from parent bean
                         $tmp_final_query =  $parentbean->$shortcut_function_name($parameters);
                     }
@@ -2954,8 +2970,9 @@ 
                 $query_fields = $GLOBALS['db']->getSelectFieldsFromQuery($subquery['select']);
                 foreach($query_fields as $field => $select)
                 {
-                    if (!in_array($field, $all_fields))
-                        $all_fields[] = $field;
+                    if (!in_array($field, $all_fields)) {
+                                            $all_fields[] = $field;
+                    }
                 }
                 $subqueries[$i]['query_fields'] = $query_fields;
             }
@@ -2969,8 +2986,7 @@ 
                     if (!isset($subquery['query_fields'][$field]))
                     {
                         $subquery['select'] .= " NULL $field,";
-                    }
-                    else
+                    } else
                     {
                         $subquery['select'] .= " {$subquery['query_fields'][$field]},";
                     }
@@ -2992,12 +3008,10 @@ 
                 if (!empty($subquery['params']['distinct']) && !empty($subpanel_def->table_name))
                 {
                     $query_rows = "( SELECT count(DISTINCT ". $subpanel_def->table_name . ".id)".  $subquery['from_min'].$query_array['join']. $subquery['where'].' )';
-                }
-                elseif ($select_position !== false && $distinct_position!= false)
+                } elseif ($select_position !== false && $distinct_position!= false)
                 {
                     $query_rows = "( ".substr_replace($query_array['select'],"SELECT count(",$select_position,6). ")" .  $subquery['from_min'].$query_array['join']. $subquery['where'].' )';
-                }
-                else
+                } else
                 {
                     //resort to default behavior.
                     $query_rows = "( SELECT count(*)".  $subquery['from_min'].$query_array['join']. $subquery['where'].' )';
@@ -3011,8 +3025,7 @@ 
                         if (empty($subquery['where']))
                         {
                             $subquerystring.=" WHERE " .$subquery['secondary_where'];
-                        }
-                        else
+                        } else
                         {
                             $subquerystring.=" AND " .$subquery['secondary_where'];
                         }
@@ -3055,8 +3068,7 @@ 
             {
                 $submodule->assign_display_fields($submodule->module_dir);
                 $response['list'] = array($submodule);
-            }
-            else
+            } else
         {
                 $response['list'] = array();
             }
@@ -3121,8 +3133,7 @@ 
             if(empty($where))
             {
                 $where = $owner_where;
-            }
-            else
+            } else
             {
                 $where .= ' AND '.  $owner_where;
             }
@@ -3169,8 +3180,7 @@ 
         if(empty($filter))
         {
             $ret_array['select'] = " SELECT $distinct $this->table_name.* ";
-        }
-        else
+        } else
         {
             $ret_array['select'] = " SELECT $distinct $this->table_name.id ";
         }
@@ -3184,8 +3194,7 @@ 
         {
             $ret_array['secondary_select']=& $ret_array['select'];
             $ret_array['secondary_from'] = & $ret_array['from'];
-        }
-        else
+        } else
         {
             $ret_array['secondary_select'] = '';
         }
@@ -3230,17 +3239,15 @@ 
                     if(isset($this->field_defs[$field]))
                     {
                         $fields[$field]= $this->field_defs[$field];
-                    }
-                    else
+                    } else
                     {
                         $fields[$field] = array('force_exists'=>true);
                     }
                 }
-            }else{
+            } else{
                 $fields = 	$filter;
             }
-        }
-        else
+        } else
         {
             $fields = 	$this->field_defs;
         }
@@ -3256,8 +3263,9 @@ 
 			$this->field_defs[$field]['source'] == 'non-db')
 		{
 			$addrelatefield = $this->get_relationship_field($field);
-			if ($addrelatefield)
-				$addrelate[$addrelatefield] = true;
+			if ($addrelatefield) {
+							$addrelate[$addrelatefield] = true;
+			}
 		}
 		if(!empty($this->field_defs[$field]['id_name'])){
 			$addrelate[$this->field_defs[$field]['id_name']] = true;
@@ -3279,15 +3287,15 @@ 
             {
                 if(!empty($filter) && isset($filter[$field]['force_exists']) && $filter[$field]['force_exists'])
                 {
-                    if ( isset($filter[$field]['force_default']) )
-                        $ret_array['select'] .= ", {$filter[$field]['force_default']} $field ";
-                    else
-                    //spaces are a fix for length issue problem with unions.  The union only returns the maximum number of characters from the first select statement.
+                    if ( isset($filter[$field]['force_default']) ) {
+                                            $ret_array['select'] .= ", {$filter[$field]['force_default']} $field ";
+                    } else {
+                                        //spaces are a fix for length issue problem with unions.  The union only returns the maximum number of characters from the first select statement.
                         $ret_array['select'] .= ", '                                                                                                                                                                                                                                                              ' $field ";
+                    }
                 }
                 continue;
-            }
-            else
+            } else
             {
                 $data = $this->field_defs[$field];
             }
@@ -3307,8 +3315,9 @@ 
                     }
                 }
 		
-            	if (!$process_field)
-                	continue;
+            	if (!$process_field) {
+            	                	continue;
+            	}
             }
 
             if(  (!isset($data['source']) || $data['source'] == 'db') && (!empty($alias) || !empty($filter) ))
@@ -3387,16 +3396,14 @@ 
                     if(empty($join_type))
                     {
                         $params['join_type'] = ' LEFT JOIN ';
-                    }
-                    else
+                    } else
                     {
                         $params['join_type'] = $join_type;
                     }
                     if(isset($data['join_name']))
                     {
                         $params['join_table_alias'] = $data['join_name'];
-                    }
-                    else
+                    } else
                     {
                         $params['join_table_alias']	= 'jt' . $jtcount;
 
@@ -3404,8 +3411,7 @@ 
                     if(isset($data['join_link_name']))
                     {
                         $params['join_table_link_alias'] = $data['join_link_name'];
-                    }
-                    else
+                    } else
                     {
                         $params['join_table_link_alias'] = 'jtl' . $jtcount;
                     }
@@ -3445,8 +3451,7 @@ 
                                 if(isset($rel_mod->field_defs['assigned_user_id']))
                                 {
                                     $ret_array['secondary_select'].= " , ".	$params['join_table_alias'] . ".assigned_user_id {$field}_owner, '$rel_module' {$field}_mod";
-                                }
-                                else
+                                } else
                                 {
                                     if(isset($rel_mod->field_defs['created_by']))
                                     {
@@ -3459,8 +3464,7 @@ 
                         if(isset($data['db_concat_fields']))
                         {
                             $ret_array['secondary_select'] .= ' , ' . $this->db->concat($params['join_table_alias'], $data['db_concat_fields']) . ' ' . $field;
-                        }
-                        else
+                        } else
                         {
                             if(!isset($data['relationship_fields']))
                             {
@@ -3473,7 +3477,9 @@ 
                         }
                         $count_used =0;
                         foreach($used_join_key as $used_key) {
-                            if($used_key == $join['rel_key']) $count_used++;
+                            if($used_key == $join['rel_key']) {
+                                $count_used++;
+                            }
                         }
                         if($count_used <= 1) {//27416, the $ret_array['secondary_select'] should always generate, regardless the dbtype
                             // add rel_key only if it was not aready added
@@ -3487,7 +3493,9 @@ 
                         {
                             foreach($data['relationship_fields'] as $r_name=>$alias_name)
                             {
-                                if(!empty( $secondarySelectedFields[$alias_name]))continue;
+                                if(!empty( $secondarySelectedFields[$alias_name])) {
+                                    continue;
+                                }
                                 $ret_array['secondary_select'] .= ', ' . $params['join_table_link_alias'].'.'. $r_name .' ' . $alias_name;
                                 $secondarySelectedFields[$alias_name] = true;
                             }
@@ -3500,14 +3508,12 @@ 
                                 $ret_array['secondary_where'] = $params['join_table_link_alias'] . '.' . $join['rel_key']. "='" .$parentbean->id . "'";
                             }
                         }
-                    }
-                    else
+                    } else
                     {
                         if(isset($data['db_concat_fields']))
                         {
                             $ret_array['select'] .= ' , ' . $this->db->concat($params['join_table_alias'], $data['db_concat_fields']) . ' ' . $field;
-                        }
-                        else
+                        } else
                         {
                             $ret_array['select'] .= ' , ' . $params['join_table_alias'] . '.' . $data['rname'] . ' ' . $field;
                         }
@@ -3534,8 +3540,7 @@ 
                                 if(isset($rel_mod->field_defs['assigned_user_id']))
                                 {
                                     $ret_array['select'] .= ' , ' .$params['join_table_alias'] . '.assigned_user_id ' .  $field . '_owner';
-                                }
-                                else
+                                } else
                                 {
                                     $ret_array['select'] .= ' , ' .$params['join_table_alias'] . '.created_by ' .  $field . '_owner';
                                 }
@@ -3570,20 +3575,22 @@ 
 	                       $where = preg_replace('/'.$data['name'].'/', $db_field, $where);
 
 				// For relationship fields replace their alias by the corresponsding link table and r_name
-				if(isset($data['relationship_fields']))
-					foreach($data['relationship_fields'] as $r_name=>$alias_name)
+				if(isset($data['relationship_fields'])) {
+									foreach($data['relationship_fields'] as $r_name=>$alias_name)
 					{
 						$db_field = $this->db->concat($params['join_table_link_alias'], $r_name);
+				}
 						$where = preg_replace('/' . $alias_name . '/', $db_field, $where);
 					}
                     	}
-                    }else{
+                    } else{
                         $where = preg_replace('/(^|[\s(])' . $data['name'] . '/', '${1}' . $params['join_table_alias'] . '.'.$data['rname'], $where);
 
 			// For relationship fields replace their alias by the corresponsding link table and r_name
-			if(isset($data['relationship_fields']))
-				foreach($data['relationship_fields'] as $r_name=>$alias_name)
+			if(isset($data['relationship_fields'])) {
+							foreach($data['relationship_fields'] as $r_name=>$alias_name)
 					$where = preg_replace('/(^|[\s(])' . $alias_name . '/', '${1}' . $params['join_table_link_alias'] . '.'.$r_name, $where);
+			}
                     }
                     if(!$table_joined)
                     {
@@ -3600,8 +3607,7 @@ 
             if(isset($this->field_defs['assigned_user_id']) && empty($selectedFields[$this->table_name.'.assigned_user_id']))
             {
                 $ret_array['select'] .= ", $this->table_name.assigned_user_id ";
-            }
-            else if(isset($this->field_defs['created_by']) &&  empty($selectedFields[$this->table_name.'.created_by']))
+            } else if(isset($this->field_defs['created_by']) &&  empty($selectedFields[$this->table_name.'.created_by']))
             {
                 $ret_array['select'] .= ", $this->table_name.created_by ";
             }
@@ -3623,14 +3629,15 @@ 
         if($show_deleted == 0)
         {
             $where_auto = "$this->table_name.deleted=0";
-        }else if($show_deleted == 1)
+        } else if($show_deleted == 1)
         {
             $where_auto = "$this->table_name.deleted=1";
         }
-        if($where != "")
-            $ret_array['where'] = " where ($where) AND $where_auto";
-        else
-            $ret_array['where'] = " where $where_auto";
+        if($where != "") {
+                    $ret_array['where'] = " where ($where) AND $where_auto";
+        } else {
+                    $ret_array['where'] = " where $where_auto";
+        }
 
         //make call to process the order by clause
         $order_by = $this->process_order_by($order_by);
@@ -3660,8 +3667,9 @@ 
 			if (isset($value['relationship_fields']) && 
 				in_array($field, $value['relationship_fields']) &&
                 (!isset($value['link_type']) || $value['link_type'] != 'relationship_info')
-            )
-				return $field_def;
+            ) {
+							return $field_def;
+			}
 		}
 
 		return false;
@@ -3709,24 +3717,23 @@ 
                         if(isset($field_def['db_concat_fields']))
                         {
                             $queries[$child_info['parent_type']] .= ' , ' . $this->db->concat($templates[$child_info['parent_type']]->table_name, $field_def['db_concat_fields']) . ' parent_name';
-                        }
-                        else
+                        } else
                         {
                             $queries[$child_info['parent_type']] .= ' , name parent_name';
                         }
                         if(isset($templates[$child_info['parent_type']]->field_defs['assigned_user_id']))
                         {
                             $queries[$child_info['parent_type']] .= ", assigned_user_id parent_name_owner , '{$child_info['parent_type']}' parent_name_mod";;
-                        }else if(isset($templates[$child_info['parent_type']]->field_defs['created_by']))
+                        } else if(isset($templates[$child_info['parent_type']]->field_defs['created_by']))
                         {
                             $queries[$child_info['parent_type']] .= ", created_by parent_name_owner, '{$child_info['parent_type']}' parent_name_mod";
                         }
                         $queries[$child_info['parent_type']] .= " FROM " . $templates[$child_info['parent_type']]->table_name ." WHERE id IN ('{$child_info['parent_id']}'";
-                    }
-                    else
+                    } else
                     {
-                        if(empty($parent_child_map[$child_info['parent_id']]))
-                        $queries[$child_info['parent_type']] .= " ,'{$child_info['parent_id']}'";
+                        if(empty($parent_child_map[$child_info['parent_id']])) {
+                                                $queries[$child_info['parent_type']] .= " ,'{$child_info['parent_id']}'";
+                        }
                     }
                     $parent_child_map[$child_info['parent_id']][] = $child_info['child_id'];
                 }
@@ -3799,8 +3806,7 @@ 
                     $row_offset = (floor(($rows_found -1) / $limit)) * $limit;
                 }
             }
-        }
-        else
+        } else
         {
             if((empty($limit) || $limit == -1))
             {
@@ -3817,8 +3823,7 @@ 
         if(!empty($limit) && $limit != -1 && $limit != -99)
         {
             $result = $db->limitQuery($query, $row_offset, $limit,true,"Error retrieving $this->object_name list: ");
-        }
-        else
+        } else
         {
             $result = $db->query($query,true,"Error retrieving $this->object_name list: ");
         }
@@ -3835,7 +3840,9 @@ 
             while ($max_per_page == -99 || ($index < $row_offset + $max_per_page))
             {
                 $row = $db->fetchByAssoc($result);
-                if (empty($row)) break;
+                if (empty($row)) {
+                    break;
+                }
 
                 //instantiate a new class each time. This is because php5 passes
                 //by reference by default so if we continually update $this, we will
@@ -3855,11 +3862,10 @@ 
                         }
 
                         $GLOBALS['log']->debug("$temp->object_name({$row['id']}): ".$field." = ".$temp->$field);
-                    }else if (isset($row[$this->table_name .'.'.$field]))
+                    } else if (isset($row[$this->table_name .'.'.$field]))
                     {
                         $temp->$field = $row[$this->table_name .'.'.$field];
-                    }
-                    else
+                    } else
                     {
                         $temp->$field = "";
                     }
@@ -3867,7 +3873,9 @@ 
 
                 $temp->check_date_relationships_load();
                 $temp->fill_in_additional_list_fields();
-                if($temp->hasCustomFields()) $temp->custom_fields->fill_relationships();
+                if($temp->hasCustomFields()) {
+                    $temp->custom_fields->fill_relationships();
+                }
                 $temp->call_custom_logic("process_record");
 
                 // fix defect #44206. implement the same logic as sugar_currency_format
@@ -3914,8 +3922,9 @@ 
         if (!$is_count_query)
         {
             $count_query = SugarBean::create_list_count_query($query);
-        } else
-            $count_query=$query;
+        } else {
+                    $count_query=$query;
+        }
 
         $result = $this->db->query($count_query, true, "Error running count query for $this->object_name List: ");
         $row_num = 0;
@@ -3988,8 +3997,7 @@ 
                 $row_offset = (floor(($rows_found -1) / $limit)) * $limit;
 
             }
-        }
-        else
+        } else
         {
             if((empty($limit) || $limit == -1))
             {
@@ -4011,8 +4019,7 @@ 
             if(!empty($limit) && $limit != -1 && $limit != -99)
             {
                 $result = $db->limitQuery($query, $row_offset, $limit,true,"Error retrieving $parent_bean->object_name list: ");
-            }
-            else
+            } else
             {
                 $result = $db->query($query,true,"Error retrieving $this->object_name list: ");
             }
@@ -4058,13 +4065,11 @@ 
                             {
                                 $current_bean->$field = $this->convertField($row[$field], $value);
                                 unset($row[$field]);
-                            }
-                            else if (isset($row[$this->table_name .'.'.$field]))
+                            } else if (isset($row[$this->table_name .'.'.$field]))
                             {
                                 $current_bean->$field = $this->convertField($row[$this->table_name .'.'.$field], $value);
                                 unset($row[$this->table_name .'.'.$field]);
-                            }
-                            else
+                            } else
                             {
                                 $current_bean->$field = "";
                                 unset($row[$field]);
@@ -4088,8 +4093,7 @@ 
                             {
                                 $execute_function[] = 	$value['function_class'];
                                 $execute_function[] = 	$value['function_name'];
-                            }
-                            else
+                            } else
                             {
                                 $execute_function	= $value['function_name'];
                             }
@@ -4102,8 +4106,7 @@ 
                                         $can_execute = false;
                                     } else if($param == '$this') {
                                         $execute_params[] = $this;
-                                    }
-                                    else
+                                    } else
                                     {
                                         $execute_params[] = $this->$param;
                                     }
@@ -4114,13 +4117,11 @@ 
                                         $can_execute = false;
                                     } else if($param == '$this') {
                                         $execute_params[] = $current_bean;
-                                    }
-                                    else
+                                    } else
                                     {
                                         $execute_params[] = $current_bean->$param;
                                     }
-                                }
-                                else
+                                } else
                                 {
                                     $can_execute = false;
                                 }
@@ -4181,8 +4182,7 @@ 
             if(isset($post_retrieve))
             {
                 $parent_fields = $this->retrieve_parent_fields($post_retrieve);
-            }
-            else
+            } else
             {
                 $parent_fields = array();
             }
@@ -4201,8 +4201,7 @@ 
                     $previous_offset++;
                 }
             }
-        }
-        else
+        } else
         {
             $row_found 	= 0;
             $parent_fields = array();
@@ -4271,8 +4270,9 @@ 
 
         $response = Array();
         $response['bean'] = $this;
-        if (empty($total_rows))
-            $total_rows=0;
+        if (empty($total_rows)) {
+                    $total_rows=0;
+        }
         $response['row_count'] = $total_rows;
         $response['next_offset'] = $next_offset;
         $response['previous_offset'] = $previous_offset;
@@ -4315,8 +4315,7 @@ 
                 {
                     $bean->$field = $row[$field];
                     $GLOBALS['log']->debug("process_full_list: $bean->object_name({$row['id']}): ".$field." = ".$bean->$field);
-                }
-                else
+                } else
                 {
                     $bean->$field = '';
                 }
@@ -4333,8 +4332,11 @@ 
             $list[] = $bean;
         }
         //}
-        if (isset($list)) return $list;
-        else return null;
+        if (isset($list)) {
+            return $list;
+        } else {
+            return null;
+        }
     }
 
     /**
@@ -4398,10 +4400,12 @@ 
             $this->assigned_user_name = get_assigned_user_name($this->assigned_user_id);
 
         }
-		if(!empty($this->field_defs['created_by']) && !empty($this->created_by))
-		$this->created_by_name = get_assigned_user_name($this->created_by);
-		if(!empty($this->field_defs['modified_user_id']) && !empty($this->modified_user_id))
-		$this->modified_by_name = get_assigned_user_name($this->modified_user_id);
+		if(!empty($this->field_defs['created_by']) && !empty($this->created_by)) {
+				$this->created_by_name = get_assigned_user_name($this->created_by);
+		}
+		if(!empty($this->field_defs['modified_user_id']) && !empty($this->modified_user_id)) {
+				$this->modified_by_name = get_assigned_user_name($this->modified_user_id);
+		}
 
 		if(!empty($this->field_defs['parent_name'])){
 			$this->fill_in_additional_parent_fields();
@@ -4416,7 +4420,7 @@ 
 
         if(!empty($this->parent_id) && !empty($this->last_parent_id) && $this->last_parent_id == $this->parent_id){
             return false;
-        }else{
+        } else{
             $this->parent_name = '';
         }
         if(!empty($this->parent_type)) {
@@ -4446,8 +4450,9 @@ 
         {
             $list=$this->$linkFieldName->get();
             $this->$idField = '' ; // match up with null value in $this->populateFromRow()
-            if (!empty($list))
-                $this->$idField = $list[0];
+            if (!empty($list)) {
+                            $this->$idField = $list[0];
+            }
         }
     }
 
@@ -4456,11 +4461,13 @@ 
     */
     function fill_in_relationship_fields(){
         global $fill_in_rel_depth;
-        if(empty($fill_in_rel_depth) || $fill_in_rel_depth < 0)
-            $fill_in_rel_depth = 0;
+        if(empty($fill_in_rel_depth) || $fill_in_rel_depth < 0) {
+                    $fill_in_rel_depth = 0;
+        }
 
-        if($fill_in_rel_depth > 1)
-            return;
+        if($fill_in_rel_depth > 1) {
+                    return;
+        }
 
         $fill_in_rel_depth++;
 
@@ -4502,13 +4509,13 @@ 
                     }
                     if(!empty($this->$id_name) && isset($this->$name))
                     {
-                        if(!isset($field['additionalFields']))
-                           $field['additionalFields'] = array();
+                        if(!isset($field['additionalFields'])) {
+                                                   $field['additionalFields'] = array();
+                        }
                         if(!empty($field['rname']))
                         {
                             $field['additionalFields'][$field['rname']]= $name;
-                        }
-                        else
+                        } else
                         {
                             $field['additionalFields']['name']= $name;
                         }
@@ -4543,8 +4550,7 @@ 
 		if(isset($_SESSION['show_deleted']))
 		{
 			$this->mark_undeleted($id);
-		}
-		else
+		} else
 		{
 			// call the custom business logic
 			$custom_logic_arguments['id'] = $id;
@@ -4806,8 +4812,7 @@ 
         if(!empty($row_offset) && $row_offset != 0 && !empty($limit) && $limit != -1)
         {
             $result = $db->limitQuery($query, $row_offset, $limit,true,"Error retrieving $template->object_name list: ");
-        }
-        else
+        } else
         {
             $result = $db->query($query, true);
         }
@@ -4876,8 +4881,7 @@ 
     if (!empty($limit))
     {
         $result = $db->limitQuery($query, $row_offset, $limit,true,"Error retrieving $this->object_name list: ");
-    }
-    else
+    } else
     {
         $result = $db->query($query, true);
     }
@@ -4923,8 +4927,7 @@ 
             if(empty($ids))
             {
                 $ids = "('" . $row['id'] . "'";
-            }
-            else
+            } else
             {
                 $ids .= ",'" . $row['id'] . "'";
             }
@@ -4932,7 +4935,7 @@ 
         if(empty($ids))
         {
             $ids = "('')";
-        }else{
+        } else{
             $ids .= ')';
         }
 
@@ -5004,10 +5007,12 @@ 
             if(isset($this->$field)){
 
                 // cn: bug 12270 - sensitive fields being passed arbitrarily in listViews
-                if(isset($sensitiveFields[$field]))
-                    continue;
-                if(!isset($cache[$field]))
-                    $cache[$field] = strtoupper($field);
+                if(isset($sensitiveFields[$field])) {
+                                    continue;
+                }
+                if(!isset($cache[$field])) {
+                                    $cache[$field] = strtoupper($field);
+                }
 
                 //Fields hidden by Dependent Fields
                 if (isset($value['hidden']) && $value['hidden'] === true) {
@@ -5024,18 +5029,17 @@ 
                     elseif(!empty($value['options']) && !empty($mod_strings[$value['options']][$this->$field]))
                     {
                         $return_array[$cache[$field]] = $mod_strings[$value['options']][$this->$field];
-                    }
-                    else{
+                    } else{
                         $return_array[$cache[$field]] = $this->$field;
                     }
                     //end bug 21672
 // tjy: no need to do this str_replace as the changes in 29994 for ListViewGeneric.tpl for translation handle this now
 //				}elseif(!empty($value['type']) && $value['type'] == 'multienum'&& empty($value['function'])){
 //					$return_array[strtoupper($field)] = str_replace('^,^', ', ', $this->$field );
-                }elseif(!empty($value['custom_module']) && $value['type'] != 'currency'){
+                } elseif(!empty($value['custom_module']) && $value['type'] != 'currency'){
 //					$this->format_field($value);
                     $return_array[$cache[$field]] = $this->$field;
-                }else{
+                } else{
                     $return_array[$cache[$field]] = $this->$field;
                 }
                 // handle "Assigned User Name"
@@ -5154,23 +5158,29 @@ 
     }
 
     function getRelatedFields($module, $id, $fields, $return_array = false){
-        if(empty($GLOBALS['beanList'][$module]))return '';
+        if(empty($GLOBALS['beanList'][$module])) {
+            return '';
+        }
         $object = BeanFactory::getObjectName($module);
 
         VardefManager::loadVardef($module, $object);
-        if(empty($GLOBALS['dictionary'][$object]['table']))return '';
+        if(empty($GLOBALS['dictionary'][$object]['table'])) {
+            return '';
+        }
         $table = $GLOBALS['dictionary'][$object]['table'];
         $query  = 'SELECT id';
         foreach($fields as $field=>$alias){
             if(!empty($GLOBALS['dictionary'][$object]['fields'][$field]['db_concat_fields'])){
                 $query .= ' ,' .$this->db->concat($table, $GLOBALS['dictionary'][$object]['fields'][$field]['db_concat_fields']) .  ' as ' . $alias ;
-            }else if(!empty($GLOBALS['dictionary'][$object]['fields'][$field]) &&
+            } else if(!empty($GLOBALS['dictionary'][$object]['fields'][$field]) &&
                 (empty($GLOBALS['dictionary'][$object]['fields'][$field]['source']) ||
                 $GLOBALS['dictionary'][$object]['fields'][$field]['source'] != "non-db"))
             {
                 $query .= ' ,' .$table . '.' . $field . ' as ' . $alias;
             }
-            if(!$return_array)$this->$alias = '';
+            if(!$return_array) {
+                $this->$alias = '';
+            }
         }
         if($query == 'SELECT id' || empty($id)){
             return '';
@@ -5181,8 +5191,7 @@ 
         {
             $query .= " , ".	$table  . ".assigned_user_id owner";
 
-        }
-        else if(isset($GLOBALS['dictionary'][$object]['fields']['created_by']))
+        } else if(isset($GLOBALS['dictionary'][$object]['fields']['created_by']))
         {
             $query .= " , ".	$table . ".created_by owner";
 
@@ -5218,8 +5227,9 @@ 
         {
             if(isset($this->field_name_map[$field]) && empty($this->$field))
             {
-                if($this->field_name_map[$field]['type'] != 'date' && $this->field_name_map[$field]['type'] != 'enum')
-                $this->$field = $field;
+                if($this->field_name_map[$field]['type'] != 'date' && $this->field_name_map[$field]['type'] != 'enum') {
+                                $this->$field = $field;
+                }
                 if($this->field_name_map[$field]['type'] == 'date')
                 {
                     $this->$field = $timedate->to_display_date('1980-07-09');
@@ -5294,8 +5304,7 @@ 
             $query = "INSERT INTO $table (id, ". implode(',', array_keys($relate_values)) . ", date_modified) VALUES ('" . create_guid() . "', " . "'" . implode("', '", $relate_values) . "', ".$date_modified.")" ;
 
             $this->db->query($query, false, "Creating Relationship:" . $query);
-        }
-        else if ($do_update)
+        } else if ($do_update)
         {
             $conds = array();
             foreach($data_values as $key=>$value)
@@ -5374,10 +5383,12 @@ 
         if(!isset($this->processed) || $this->processed == false){
             //add some logic to ensure we do not get into an infinite loop
             if(!empty($this->logicHookDepth[$event])) {
-                if($this->logicHookDepth[$event] > $this->max_logic_depth)
-                    return;
-            }else
-                $this->logicHookDepth[$event] = 0;
+                if($this->logicHookDepth[$event] > $this->max_logic_depth) {
+                                    return;
+                }
+            } else {
+                            $this->logicHookDepth[$event] = 0;
+            }
 
             //we have to put the increment operator here
             //otherwise we may never increase the depth for that event in the case
@@ -5407,16 +5418,13 @@ 
         if ($this->custom_fields->avail_fields[$name]['ext1'])
         {
             $realKey = 'ext1';
-        }
-        elseif ($this->custom_fields->avail_fields[$name]['ext2'])
+        } elseif ($this->custom_fields->avail_fields[$name]['ext2'])
         {
             $realKey = 'ext2';
-        }
-        elseif ($this->custom_fields->avail_fields[$name]['ext3'])
+        } elseif ($this->custom_fields->avail_fields[$name]['ext3'])
         {
             $realKey = 'ext3';
-        }
-        else
+        } else
         {
             $GLOBALS['log']->fatal("SUGARBEAN: cannot find Real Key for custom field of type dropdown - cannot return Value.");
             return false;
@@ -5450,9 +5458,11 @@ 
 		/**
         if($is_owner == 'not_set')
 		*/
-    	if($is_owner === 'not_set') //eggsurplus: should be ===
+    	if($is_owner === 'not_set') {
+    	    //eggsurplus: should be ===
         {
             $not_set = true;
+    	}
             $is_owner = $this->isOwner($current_user->id);
         }
 		// DJM - OBS Customizations - May 2009
@@ -5503,8 +5513,9 @@ 
 			// DJM - OBS Customizations - END CHANGE
     	}
         //if we don't implent acls return true
-        if(!$this->bean_implements('ACL'))
-        return true;
+        if(!$this->bean_implements('ACL')) {
+                return true;
+        }
         $view = strtolower($view);
         switch ($view)
         {
@@ -5522,7 +5533,7 @@ 
                     $temp = new $class();
                     if(!empty($this->fetched_row) && !empty($this->fetched_row['id']) && !empty($this->fetched_row['assigned_user_id']) && !empty($this->fetched_row['created_by'])){
                         $temp->populateFromRow($this->fetched_row);
-                    }else{
+                    } else{
                         $temp->retrieve($this->id);
                     }
                     $is_owner = $temp->isOwner($current_user->id);
@@ -5601,10 +5612,11 @@ 
             }
             return false;
         } elseif (isset($this->assigned_user_id)) {
-            if($this->assigned_user_id == $user_id) return true;
+            if($this->assigned_user_id == $user_id) {
+                return true;
+            }
             return false;
-        }
-        else
+        } else
         {
             //other wise if there is a created_by that is the owner
             if(isset($this->created_by) && $this->created_by == $user_id)
@@ -5646,8 +5658,7 @@ 
         if($this->ACLAccess('DetailView'))
         {
             $array_assign['MAIN'] = 'a';
-        }
-        else
+        } else
         {
             $array_assign['MAIN'] = 'span';
         }
@@ -5666,20 +5677,20 @@ 
 
         foreach($this->field_defs as $field=>$data)
         {
-            if( !$dbOnly || !isset($data['source']) || $data['source'] == 'db')
-            if(!$stringOnly || is_string($this->$field))
+            if( !$dbOnly || !isset($data['source']) || $data['source'] == 'db') {
+                        if(!$stringOnly || is_string($this->$field))
             if($upperKeys)
             {
                                 if(!isset($cache[$field])){
                                     $cache[$field] = strtoupper($field);
+            }
                                 }
                 $arr[$cache[$field]] = $this->$field;
-            }
-            else
+            } else
             {
                 if(isset($this->$field)){
                     $arr[$field] = $this->$field;
-                }else{
+                } else{
                     $arr[$field] = '';
                 }
             }
@@ -5754,7 +5765,9 @@ 
 
         $this->fill_in_additional_list_fields();
 
-        if($this->hasCustomFields())$this->custom_fields->fill_relationships();
+        if($this->hasCustomFields()) {
+            $this->custom_fields->fill_relationships();
+        }
         $this->call_custom_logic("process_record");
     }
 
@@ -5847,7 +5860,10 @@ 
  */
     function decrypt_after_retrieve($value)
     {
-        if(empty($value)) return $value; // no need to decrypt empty
+        if(empty($value)) {
+            return $value;
+        }
+        // no need to decrypt empty
         require_once("include/utils/encryption_utils.php");
         return blowfishDecode($this->getEncryptKey(), $value);
     }
@@ -5874,16 +5890,14 @@ 
                     $_SESSION['o_lock_save'] = $saveform;
                     header('Location: index.php?module=OptimisticLock&action=LockResolve');
                     die();
-                }
-                else
+                } else
                 {
                     unset ($_SESSION['o_lock_object']);
                     unset ($_SESSION['o_lock_id']);
                     unset ($_SESSION['o_lock_dm']);
                 }
             }
-        }
-        else
+        } else
         {
             if(isset($_SESSION['o_lock_object']))	{ unset ($_SESSION['o_lock_object']); }
             if(isset($_SESSION['o_lock_id']))		{ unset ($_SESSION['o_lock_id']); }
@@ -5898,9 +5912,11 @@ 
     */
     private function _sendNotifications($check_notify){
         if($check_notify || (isset($this->notify_inworkflow) && $this->notify_inworkflow == true) // cn: bug 5795 - no invites sent to Contacts, and also bug 25995, in workflow, it will set the notify_on_save=true.
-           && !$this->isOwner($this->created_by) )  // cn: bug 42727 no need to send email to owner (within workflow)
+           && !$this->isOwner($this->created_by) ) {
+            // cn: bug 42727 no need to send email to owner (within workflow)
         {
             $admin = new Administration();
+        }
             $admin->retrieveSettings();
             $sendNotifications = false;
 
@@ -5908,13 +5924,11 @@ 
             {
                 $GLOBALS['log']->info("Notifications: user assignment has changed, checking if user receives notifications");
                 $sendNotifications = true;
-            }
-            elseif(isset($_REQUEST['send_invites']) && $_REQUEST['send_invites'] == 1)
+            } elseif(isset($_REQUEST['send_invites']) && $_REQUEST['send_invites'] == 1)
             {
                 // cn: bug 5795 Send Invites failing for Contacts
                 $sendNotifications = true;
-            }
-            else
+            } else
             {
                 $GLOBALS['log']->info("Notifications: not sending e-mail, notify_on is set to OFF");
             }
@@ -6120,8 +6134,7 @@ 
             !empty($this->field_defs['email_addresses_non_primary']) && $this->field_defs['email_addresses_non_primary']['type'] == 'email')
         {
             return true;
-        }
-        else
+        } else
         {
             return false;
         }

modules/Users/language/en_us.lang.php

Indentation +2 added lines, -2 removed lines

@@ -48,8 +48,8 @@
 $mod_strings = array (
     'LBL_DELETE_USER_CONFIRM'           => 'When the User record is deleted, the corresponding Employee record will also be deleted. After the user is deleted, any workflow definitions and reports involving the user might need to be updated.<br/><br/> Deleting a User record cannot be undone.',
 
-	'LBL_DELETE_GROUP_CONFIRM'          => 'Are you sure you want to delete this Group User? Click OK to delete the User record.<br/>After clicking OK, you will be given the ability to reassign records assigned to the Group User to another user.',
-	'LBL_DELETE_PORTAL_CONFIRM'         => 'Are you sure you want to delete this Portal API User? Click OK to delete the User record.',
+    'LBL_DELETE_GROUP_CONFIRM'          => 'Are you sure you want to delete this Group User? Click OK to delete the User record.<br/>After clicking OK, you will be given the ability to reassign records assigned to the Group User to another user.',
+    'LBL_DELETE_PORTAL_CONFIRM'         => 'Are you sure you want to delete this Portal API User? Click OK to delete the User record.',
 
 
     'LNK_IMPORT_USERS' => 'Import Users',