adamjakab / SuiteCRM

include/language/en_us.lang.php

Spacing +135 added lines, -135 removed lines

@@ -1,5 +1,5 @@ 
 <?php
-if(!defined('sugarEntry') || !sugarEntry) die('Not A Valid Entry Point');
+if (!defined('sugarEntry') || !sugarEntry) die('Not A Valid Entry Point');
 /*********************************************************************************
  * SugarCRM Community Edition is a customer relationship management program developed by
  * SugarCRM, Inc. Copyright (C) 2004-2013 SugarCRM Inc.
@@ -48,11 +48,11 @@ 
 
 //the left value is the key stored in the db and the right value is ie display value
 //to translate, only modify the right value in each key/value pair
-$app_list_strings = array (
+$app_list_strings = array(
 //e.g. auf Deutsch 'Contacts'=>'Contakten',
   'language_pack_name' => 'US English',
   'moduleList' =>
-  array (
+  array(
     'Home' => 'Home',
     'Contacts' => 'Contacts',
     'Accounts' => 'Accounts',
@@ -113,7 +113,7 @@ 
   ),
 
   'moduleListSingular' =>
-  array (
+  array(
     'Home' => 'Home',
     'Dashboard' => 'Dashboard',
     'Contacts' => 'Contact',
@@ -153,7 +153,7 @@ 
 
   //e.g. en fran�ais 'Analyst'=>'Analyste',
   'account_type_dom' =>
-  array (
+  array(
     '' => '',
     'Analyst' => 'Analyst',
     'Competitor' => 'Competitor',
@@ -168,7 +168,7 @@ 
   ),
   //e.g. en espa�ol 'Apparel'=>'Ropa',
   'industry_dom' =>
-  array (
+  array(
     '' => '',
     'Apparel' => 'Apparel',
     'Banking' => 'Banking',
@@ -203,7 +203,7 @@ 
   ),
   'lead_source_default_key' => 'Self Generated',
   'lead_source_dom' =>
-  array (
+  array(
     '' => '',
     'Cold Call' => 'Cold Call',
     'Existing Customer' => 'Existing Customer',
@@ -221,13 +221,13 @@ 
     'Other' => 'Other',
   ),
   'opportunity_type_dom' =>
-  array (
+  array(
     '' => '',
     'Existing Business' => 'Existing Business',
     'New Business' => 'New Business',
   ),
   'roi_type_dom' =>
-    array (
+    array(
     'Revenue' => 'Revenue',
     'Investment'=>'Investment',
     'Expected_Revenue'=>'Expected Revenue',
@@ -238,7 +238,7 @@ 
 //       it is the key for the default opportunity_relationship_type_dom value
   'opportunity_relationship_type_default_key' => 'Primary Decision Maker',
   'opportunity_relationship_type_dom' =>
-  array (
+  array(
     '' => '',
     'Primary Decision Maker' => 'Primary Decision Maker',
     'Business Decision Maker' => 'Business Decision Maker',
@@ -253,24 +253,24 @@ 
 //       it is the key for the default case_relationship_type_dom value
   'case_relationship_type_default_key' => 'Primary Contact',
   'case_relationship_type_dom' =>
-  array (
+  array(
     '' => '',
     'Primary Contact' => 'Primary Contact',
     'Alternate Contact' => 'Alternate Contact',
   ),
   'payment_terms' =>
-  array (
+  array(
     '' => '',
     'Net 15' => 'Net 15',
     'Net 30' => 'Net 30',
   ),
   'sales_stage_default_key' => 'Prospecting',
-  'fts_type' => array (
+  'fts_type' => array(
       '' => '',
       'Elastic' => 'elasticsearch'
   ),
   'sales_stage_dom' =>
-  array (
+  array(
     'Prospecting' => 'Prospecting',
     'Qualification' => 'Qualification',
     'Needs Analysis' => 'Needs Analysis',
@@ -282,7 +282,7 @@ 
     'Closed Won' => 'Closed Won',
     'Closed Lost' => 'Closed Lost',
   ),
-  'in_total_group_stages' => array (
+  'in_total_group_stages' => array(
     'Draft' => 'Draft',
     'Negotiation' => 'Negotiation',
     'Delivered' => 'Delivered',
@@ -293,7 +293,7 @@ 
     'Closed Dead' => 'Closed Dead',
   ),
   'sales_probability_dom' => // keys must be the same as sales_stage_dom
-  array (
+  array(
     'Prospecting' => '10',
     'Qualification' => '20',
     'Needs Analysis' => '25',
@@ -306,7 +306,7 @@ 
     'Closed Lost' => '0',
   ),
   'activity_dom' =>
-  array (
+  array(
     'Call' => 'Call',
     'Meeting' => 'Meeting',
     'Task' => 'Task',
@@ -314,7 +314,7 @@ 
     'Note' => 'Note',
   ),
   'salutation_dom' =>
-      array (
+      array(
         '' => '',
         'Mr.' => 'Mr.',
         'Ms.' => 'Ms.',
@@ -324,7 +324,7 @@ 
       ),
   //time is in seconds; the greater the time the longer it takes;
   'reminder_max_time' => 90000,
-  'reminder_time_options' => array( 60=> '1 minute prior',
+  'reminder_time_options' => array(60=> '1 minute prior',
                                   300=> '5 minutes prior',
                                   600=> '10 minutes prior',
                                   900=> '15 minutes prior',
@@ -338,14 +338,14 @@ 
 
   'task_priority_default' => 'Medium',
   'task_priority_dom' =>
-  array (
+  array(
     'High' => 'High',
     'Medium' => 'Medium',
     'Low' => 'Low',
   ),
   'task_status_default' => 'Not Started',
   'task_status_dom' =>
-  array (
+  array(
     'Not Started' => 'Not Started',
     'In Progress' => 'In Progress',
     'Completed' => 'Completed',
@@ -354,35 +354,35 @@ 
   ),
   'meeting_status_default' => 'Planned',
   'meeting_status_dom' =>
-  array (
+  array(
     'Planned' => 'Planned',
     'Held' => 'Held',
     'Not Held' => 'Not Held',
   ),
   'extapi_meeting_password' =>
-  array (
+  array(
       'WebEx' => 'WebEx',
   ),
   'meeting_type_dom' =>
-   array (
+   array(
       'Other' => 'Other',
       'Sugar' => 'SuiteCRM',
    ),
   'call_status_default' => 'Planned',
   'call_status_dom' =>
-  array (
+  array(
     'Planned' => 'Planned',
     'Held' => 'Held',
     'Not Held' => 'Not Held',
   ),
   'call_direction_default' => 'Outbound',
   'call_direction_dom' =>
-  array (
+  array(
     'Inbound' => 'Inbound',
     'Outbound' => 'Outbound',
   ),
   'lead_status_dom' =>
-  array (
+  array(
     '' => '',
     'New' => 'New',
     'Assigned' => 'Assigned',
@@ -392,7 +392,7 @@ 
     'Dead' => 'Dead',
   ),
    'gender_list' =>
-  array (
+  array(
     'male' => 'Male',
     'female' => 'Female',
   ),
@@ -400,7 +400,7 @@ 
 //       it is the key for the default case_status_dom value
   'case_status_default_key' => 'New',
   'case_status_dom' =>
-  array (
+  array(
     'New' => 'New',
     'Assigned' => 'Assigned',
     'Closed' => 'Closed',
@@ -410,49 +410,49 @@ 
   ),
   'case_priority_default_key' => 'P2',
   'case_priority_dom' =>
-  array (
+  array(
     'P1' => 'High',
     'P2' => 'Medium',
     'P3' => 'Low',
   ),
   'user_type_dom' =>
-  array (
+  array(
     'RegularUser' => 'Regular User',
     'Administrator' => 'Administrator',
   ),
   'user_status_dom' =>
-  array (
+  array(
     'Active' => 'Active',
     'Inactive' => 'Inactive',
   ),
   'employee_status_dom' =>
-  array (
+  array(
     'Active' => 'Active',
     'Terminated' => 'Terminated',
     'Leave of Absence' => 'Leave of Absence',
   ),
   'messenger_type_dom' =>
-  array (
+  array(
     '' => '',
     'MSN' => 'MSN',
     'Yahoo!' => 'Yahoo!',
     'AOL' => 'AOL',
   ),
-    'project_task_priority_options' => array (
+    'project_task_priority_options' => array(
         'High' => 'High',
         'Medium' => 'Medium',
         'Low' => 'Low',
     ),
     'project_task_priority_default' => 'Medium',
 
-    'project_task_status_options' => array (
+    'project_task_status_options' => array(
         'Not Started' => 'Not Started',
         'In Progress' => 'In Progress',
         'Completed' => 'Completed',
         'Pending Input' => 'Pending Input',
         'Deferred' => 'Deferred',
     ),
-    'project_task_utilization_options' => array (
+    'project_task_utilization_options' => array(
         '0' => 'none',
         '25' => '25',
         '50' => '50',
@@ -460,7 +460,7 @@ 
         '100' => '100',
     ),
 
-    'project_status_dom' => array (
+    'project_status_dom' => array(
         'Draft' => 'Draft',
         'In Review' => 'In Review',
         'Underway' => 'Underway',
@@ -469,12 +469,12 @@ 
     ),
     'project_status_default' => 'Draft',
 
-    'project_duration_units_dom' => array (
+    'project_duration_units_dom' => array(
         'Days' => 'Days',
         'Hours' => 'Hours',
     ),
 
-    'project_priority_options' => array (
+    'project_priority_options' => array(
         'High' => 'High',
         'Medium' => 'Medium',
         'Low' => 'Low',
@@ -484,7 +484,7 @@ 
 //       it is the key for the default record_type_module value
   'record_type_default_key' => 'Accounts',
   'record_type_display' =>
-  array (
+  array(
     '' => '',
     'Accounts' => 'Account',
     'Opportunities' => 'Opportunity',
@@ -505,7 +505,7 @@ 
   ),
 
   'record_type_display_notes' =>
-  array (
+  array(
     'Accounts' => 'Account',
     'Contacts' => 'Contact',
     'Opportunities' => 'Opportunity',
@@ -524,7 +524,7 @@ 
   ),
 
   'parent_type_display' =>
-  array (
+  array(
     'Accounts' => 'Account',
     'Contacts' => 'Contact',
     'Tasks' => 'Task',
@@ -545,7 +545,7 @@ 
 
   'issue_priority_default_key' => 'Medium',
   'issue_priority_dom' =>
-  array (
+  array(
     'Urgent' => 'Urgent',
     'High' => 'High',
     'Medium' => 'Medium',
@@ -553,7 +553,7 @@ 
   ),
   'issue_resolution_default_key' => '',
   'issue_resolution_dom' =>
-  array (
+  array(
     '' => '',
     'Accepted' => 'Accepted',
     'Duplicate' => 'Duplicate',
@@ -564,7 +564,7 @@ 
 
   'issue_status_default_key' => 'New',
   'issue_status_dom' =>
-  array (
+  array(
     'New' => 'New',
     'Assigned' => 'Assigned',
     'Closed' => 'Closed',
@@ -574,7 +574,7 @@ 
 
   'bug_priority_default_key' => 'Medium',
   'bug_priority_dom' =>
-  array (
+  array(
     'Urgent' => 'Urgent',
     'High' => 'High',
     'Medium' => 'Medium',
@@ -582,7 +582,7 @@ 
   ),
    'bug_resolution_default_key' => '',
   'bug_resolution_dom' =>
-  array (
+  array(
     '' => '',
     'Accepted' => 'Accepted',
     'Duplicate' => 'Duplicate',
@@ -593,7 +593,7 @@ 
   ),
   'bug_status_default_key' => 'New',
   'bug_status_dom' =>
-  array (
+  array(
     'New' => 'New',
     'Assigned' => 'Assigned',
     'Closed' => 'Closed',
@@ -602,12 +602,12 @@ 
   ),
    'bug_type_default_key' => 'Bug',
   'bug_type_dom' =>
-  array (
+  array(
     'Defect' => 'Defect',
     'Feature' => 'Feature',
   ),
  'case_type_dom' =>
-  array (
+  array(
     'Administration' => 'Administration',
     'Product' => 'Product',
     'User' => 'User',
@@ -615,7 +615,7 @@ 
 
   'source_default_key' => '',
   'source_dom' =>
-  array (
+  array(
     '' => '',
     'Internal' => 'Internal',
     'Forum' => 'Forum',
@@ -625,7 +625,7 @@ 
 
   'product_category_default_key' => '',
   'product_category_dom' =>
-  array (
+  array(
     '' => '',
     'Accounts' => 'Accounts',
     'Activities' => 'Activities',
@@ -658,7 +658,7 @@ 
   ),
   /*Added entries 'Queued' and 'Sending' for 4.0 release..*/
   'campaign_status_dom' =>
-  array (
+  array(
         '' => '',
         'Planning' => 'Planning',
         'Active' => 'Active',
@@ -668,7 +668,7 @@ 
         'Sending'=> 'Sending',
   ),
   'campaign_type_dom' =>
-  array (
+  array(
         '' => '',
         'Telesales' => 'Telesales',
         'Mail' => 'Mail',
@@ -681,7 +681,7 @@ 
         ),
 
   'newsletter_frequency_dom' =>
-  array (
+  array(
         '' => '',
         'Weekly' => 'Weekly',
         'Monthly' => 'Monthly',
@@ -690,7 +690,7 @@ 
         ),
 
   'notifymail_sendtype' =>
-  array (
+  array(
     'SMTP' => 'SMTP',
   ),
       'dom_cal_month_long'=>array(
@@ -767,7 +767,7 @@ 
         'inbound'   => 'Inbound',
         'campaign'  => 'Campaign'
     ),
-    'dom_email_status' => array (
+    'dom_email_status' => array(
         'archived'  => 'Archived',
         'closed'    => 'Closed',
         'draft'     => 'In Draft',
@@ -777,11 +777,11 @@ 
         'send_error'=> 'Send Error',
         'unread'    => 'Unread',
     ),
-    'dom_email_archived_status' => array (
+    'dom_email_archived_status' => array(
         'archived'  => 'Archived',
     ),
 
-    'dom_email_server_type' => array(   ''          => '--None--',
+    'dom_email_server_type' => array(''          => '--None--',
                                         'imap'      => 'IMAP',
     ),
     'dom_mailbox_type'      => array(/*''           => '--None Specified--',*/
@@ -806,19 +806,19 @@ 
     'dom_int_bool'          => array(1 => 'Yes',
                                      0 => 'No',
     ),
-    'dom_switch_bool'       => array ('on' => 'Yes',
+    'dom_switch_bool'       => array('on' => 'Yes',
                                         'off' => 'No',
-                                        '' => 'No', ),
+                                        '' => 'No',),
 
-    'dom_email_link_type'   => array(   'sugar'     => 'SuiteCRM Email Client',
+    'dom_email_link_type'   => array('sugar'     => 'SuiteCRM Email Client',
                                         'mailto'    => 'External Email Client'),
 
 
-    'dom_email_editor_option'=> array(  ''          => 'Default Email Format',
+    'dom_email_editor_option'=> array(''          => 'Default Email Format',
                                         'html'      => 'HTML Email',
                                         'plain'     => 'Plain Text Email'),
 
-    'schedulers_times_dom'  => array(   'not run'       => 'Past Run Time, Not Executed',
+    'schedulers_times_dom'  => array('not run'       => 'Past Run Time, Not Executed',
                                         'ready'         => 'Ready',
                                         'in progress'   => 'In Progress',
                                         'failed'        => 'Failed',
@@ -827,28 +827,28 @@ 
     ),
 
     'scheduler_status_dom' =>
-        array (
+        array(
         'Active' => 'Active',
         'Inactive' => 'Inactive',
         ),
 
     'scheduler_period_dom' =>
-        array (
+        array(
         'min' => 'Minutes',
         'hour' => 'Hours',
         ),
     'forecast_schedule_status_dom' =>
-    array (
+    array(
     'Active' => 'Active',
     'Inactive' => 'Inactive',
   ),
     'forecast_type_dom' =>
-    array (
+    array(
     'Direct' => 'Direct',
     'Rollup' => 'Rollup',
   ),
     'document_category_dom' =>
-    array (
+    array(
     '' => '',
     'Marketing' => 'Marketing',
     'Knowledege Base' => 'Knowledge Base',
@@ -856,7 +856,7 @@ 
   ),
 
     'document_subcategory_dom' =>
-    array (
+    array(
     '' => '',
     'Marketing Collateral' => 'Marketing Collateral',
     'Product Brochures' => 'Product Brochures',
@@ -864,7 +864,7 @@ 
   ),
 
     'document_status_dom' =>
-    array (
+    array(
     'Active' => 'Active',
     'Draft' => 'Draft',
     'FAQ' => 'FAQ',
@@ -881,13 +881,13 @@ 
     'license'=>'License Agreement',
   ),
     'dom_meeting_accept_options' =>
-    array (
+    array(
     'accept' => 'Accept',
     'decline' => 'Decline',
     'tentative' => 'Tentative',
   ),
     'dom_meeting_accept_status' =>
-    array (
+    array(
     'accept' => 'Accepted',
     'decline' => 'Declined',
     'tentative' => 'Tentative',
@@ -939,7 +939,7 @@ 
 */
 //prospect list type dom
   'prospect_list_type_dom' =>
-  array (
+  array(
     'default' => 'Default',
     'seed' => 'Seed',
     'exempt_domain' => 'Suppression List - By Domain',
@@ -955,14 +955,14 @@ 
         '50'    => '50'
     ),
   'email_marketing_status_dom' =>
-  array (
+  array(
     '' => '',
     'active'=>'Active',
     'inactive'=>'Inactive'
   ),
 
   'campainglog_activity_type_dom' =>
-  array (
+  array(
     ''=>'',
     'targeted' => 'Message Sent/Attempted',
     'send error'=>'Bounced Messages,Other',
@@ -976,33 +976,33 @@ 
   ),
 
   'campainglog_target_type_dom' =>
-  array (
+  array(
     'Contacts' => 'Contacts',
     'Users'=>'Users',
     'Prospects'=>'Targets',
     'Leads'=>'Leads',
     'Accounts'=>'Accounts',
   ),
-  'merge_operators_dom' => array (
+  'merge_operators_dom' => array(
     'like'=>'Contains',
     'exact'=>'Exactly',
     'start'=>'Starts With',
   ),
 
-  'custom_fields_importable_dom' => array (
+  'custom_fields_importable_dom' => array(
     'true'=>'Yes',
     'false'=>'No',
     'required'=>'Required',
   ),
 
-    'Elastic_boost_options' => array (
+    'Elastic_boost_options' => array(
         '0' =>'Disabled',
         '1'=>'Low Boost',
         '2'=>'Medium Boost',
         '3'=>'High Boost',
     ),
 
-  'custom_fields_merge_dup_dom'=> array (
+  'custom_fields_merge_dup_dom'=> array(
         0=>'Disabled',
         1=>'Enabled',
   ),
@@ -1013,19 +1013,19 @@ 
   ),
 
 
-    'projects_priority_options' => array (
+    'projects_priority_options' => array(
         'high'      => 'High',
         'medium'    => 'Medium',
         'low'       => 'Low',
     ),
 
-    'projects_status_options' => array (
+    'projects_status_options' => array(
         'notstarted'    => 'Not Started',
         'inprogress'    => 'In Progress',
         'completed'     => 'Completed',
     ),
     // strings to pass to Flash charts
-    'chart_strings' => array (
+    'chart_strings' => array(
         'expandlegend'      => 'Expand Legend',
         'collapselegend'    => 'Collapse Legend',
         'clickfordrilldown' => 'Click for Drilldown',
@@ -1042,25 +1042,25 @@ 
         'pieWedgeName'      => 'sections',
     ),
     'release_status_dom' =>
-    array (
+    array(
         'Active' => 'Active',
         'Inactive' => 'Inactive',
     ),
     'email_settings_for_ssl' =>
-    array (
+    array(
         '0' => '',
         '1' => 'SSL',
         '2' => 'TLS',
     ),
     'import_enclosure_options' =>
-    array (
+    array(
         '\'' => 'Single Quote (\')',
         '"' => 'Double Quote (")',
         '' => 'None',
         'other' => 'Other:',
     ),
     'import_delimeter_options' =>
-    array (
+    array(
         ',' => ',',
         ';' => ';',
         '\t' => '\t',
@@ -1070,12 +1070,12 @@ 
         'other' => 'Other:',
     ),
     'link_target_dom' =>
-    array (
+    array(
         '_blank' => 'New Window',
         '_self' => 'Same Window',
     ),
     'dashlet_auto_refresh_options' =>
-    array (
+    array(
         '-1'  => 'Do not auto-refresh',
         '30'  => 'Every 30 seconds',
         '60'  => 'Every 1 minute',
@@ -1084,7 +1084,7 @@ 
         '600'   => 'Every 10 minutes',
     ),
   'dashlet_auto_refresh_options_admin' =>
-    array (
+    array(
         '-1'  => 'Never',
         '30'  => 'Every 30 seconds',
         '60'  => 'Every 1 minute',
@@ -1128,7 +1128,7 @@ 
   ),
 );
 
-$app_strings = array (
+$app_strings = array(
   'LBL_TOUR_NEXT' => 'Next',
   'LBL_TOUR_SKIP' => 'Skip',
   'LBL_TOUR_BACK' => 'Back',
@@ -2255,7 +2255,7 @@ 
     'LBL_COPY_ADDRESS_FROM_LEFT' => 'Copy address from left:',
     'LBL_SAVE_AND_CONTINUE' => 'Save and Continue',
 
-    'LBL_SEARCH_HELP_TEXT' => '<p><br /><strong>Multiselect controls</strong></p><ul><li>Click on the values to select an attribute.</li><li>Ctrl-click&nbsp;to&nbsp;select multiple. Mac users use CMD-click.</li><li>To select all values between two attributes,&nbsp; click first value&nbsp;and then shift-click last value.</li></ul><p><strong>Advanced Search & Layout Options</strong><br><br>Using the <b>Saved Search & Layout</b> option, you can save a set of search parameters and/or a custom List View layout in order to quickly obtain the desired search results in the future. You can save an unlimited number of custom searches and layouts. All saved searches appear by name in the Saved Searches list, with the last loaded saved search appearing at the top of the list.<br><br>To customize the List View layout, use the Hide Columns and Display Columns boxes to select which fields to display in the search results. For example, you can view or hide details such as the record name, and assigned user, and assigned team in the search results. To add a column to List View, select the field from the Hide Columns list and use the left arrow to move it to the Display Columns list. To remove a column from List View, select it from the Display Columns list and use the right arrow to move it to the Hide Columns list.<br><br>If you save layout settings, you will be able to load them at any time to view the search results in the custom layout.<br><br>To save and update a search and/or layout:<ol><li>Enter a name for the search results in the <b>Save this search as</b> field and click <b>Save</b>.The name now displays in the Saved Searches list adjacent to the <b>Clear</b> button.</li><li>To view a saved search, select it from the Saved Searches list. The search results are displayed in the List View.</li><li>To update the properties of a saved search, select the saved search from the list, enter the new search criteria and/or layout options in the Advanced Search area, and click <b>Update</b> next to <b>Modify Current Search</b>.</li><li>To delete a saved search, select it in the Saved Searches list, click <b>Delete</b> next to <b>Modify Current Search</b>, and then click <b>OK</b> to confirm the deletion.</li></ol><p><strong>Tips</strong><br><br>By using the % as a wildcard operator you can make your search more broad.  For example instead of just searching for results that equal "Apples" you could change your search to "Apples%" which would match all results that start with the word Apples but could contain other characters as well.</p>' ,
+    'LBL_SEARCH_HELP_TEXT' => '<p><br /><strong>Multiselect controls</strong></p><ul><li>Click on the values to select an attribute.</li><li>Ctrl-click&nbsp;to&nbsp;select multiple. Mac users use CMD-click.</li><li>To select all values between two attributes,&nbsp; click first value&nbsp;and then shift-click last value.</li></ul><p><strong>Advanced Search & Layout Options</strong><br><br>Using the <b>Saved Search & Layout</b> option, you can save a set of search parameters and/or a custom List View layout in order to quickly obtain the desired search results in the future. You can save an unlimited number of custom searches and layouts. All saved searches appear by name in the Saved Searches list, with the last loaded saved search appearing at the top of the list.<br><br>To customize the List View layout, use the Hide Columns and Display Columns boxes to select which fields to display in the search results. For example, you can view or hide details such as the record name, and assigned user, and assigned team in the search results. To add a column to List View, select the field from the Hide Columns list and use the left arrow to move it to the Display Columns list. To remove a column from List View, select it from the Display Columns list and use the right arrow to move it to the Hide Columns list.<br><br>If you save layout settings, you will be able to load them at any time to view the search results in the custom layout.<br><br>To save and update a search and/or layout:<ol><li>Enter a name for the search results in the <b>Save this search as</b> field and click <b>Save</b>.The name now displays in the Saved Searches list adjacent to the <b>Clear</b> button.</li><li>To view a saved search, select it from the Saved Searches list. The search results are displayed in the List View.</li><li>To update the properties of a saved search, select the saved search from the list, enter the new search criteria and/or layout options in the Advanced Search area, and click <b>Update</b> next to <b>Modify Current Search</b>.</li><li>To delete a saved search, select it in the Saved Searches list, click <b>Delete</b> next to <b>Modify Current Search</b>, and then click <b>OK</b> to confirm the deletion.</li></ol><p><strong>Tips</strong><br><br>By using the % as a wildcard operator you can make your search more broad.  For example instead of just searching for results that equal "Apples" you could change your search to "Apples%" which would match all results that start with the word Apples but could contain other characters as well.</p>',
 
     //resource management
     'ERR_QUERY_LIMIT' => 'Error: Query limit of $limit reached for $module module.',
@@ -2496,7 +2496,7 @@ 
     'LBL_FIRST_INPUT_SEARCH_KEY' => '9',
     'LBL_GLOBAL_SEARCH_LNK_KEY' => '0',
     'LBL_KEYBOARD_SHORTCUTS_HELP_TITLE' => 'Keyboard Shortcuts',
-    'LBL_KEYBOARD_SHORTCUTS_HELP' => '<p><strong>Form Functionality - Alt+</strong><br/> I = ed<b>I</b>t (detailview)<br/> U = d<b>U</b>plicate (detailview)<br/> D = <b>D</b>elete (detailview)<br/> A = s<b>A</b>ve (editview)<br/> L = cance<b>L</b> (editview) <br/><br/></p><p><strong>Search and Navigation  - Alt+</strong><br/> 7 = first input on Edit form<br/> 8 = Advanced Search link<br/> 9 = First Search Form input<br/> 0 = Unified search input<br></p>' ,
+    'LBL_KEYBOARD_SHORTCUTS_HELP' => '<p><strong>Form Functionality - Alt+</strong><br/> I = ed<b>I</b>t (detailview)<br/> U = d<b>U</b>plicate (detailview)<br/> D = <b>D</b>elete (detailview)<br/> A = s<b>A</b>ve (editview)<br/> L = cance<b>L</b> (editview) <br/><br/></p><p><strong>Search and Navigation  - Alt+</strong><br/> 7 = first input on Edit form<br/> 8 = Advanced Search link<br/> 9 = First Search Form input<br/> 0 = Unified search input<br></p>',
 
     'ERR_CONNECTOR_NOT_ARRAY' => 'connector array in {0} been defined incorrectly or is empty and could not be used.',
     'ERR_SUHOSIN' => 'Upload stream is blocked by Suhosin, please add &quot;upload&quot; to suhosin.executor.include.whitelist (See suitecrm.log for more information)',
@@ -2521,13 +2521,13 @@ 
 $app_list_strings['library_type'] = array('Books'=>'Book', 'Music'=>'Music', 'DVD'=>'DVD', 'Magazines'=>'Magazines');
 $app_list_strings['moduleList']['EmailAddresses'] = 'Email Address';
 $app_list_strings['project_priority_default'] = 'Medium';
-$app_list_strings['project_priority_options'] = array (
+$app_list_strings['project_priority_options'] = array(
     'High' => 'High',
     'Medium' => 'Medium',
     'Low' => 'Low',
 );
 
-$app_list_strings['kbdocument_status_dom'] = array (
+$app_list_strings['kbdocument_status_dom'] = array(
     'Draft' => 'Draft',
     'Expired' => 'Expired',
     'In Review' => 'In Review',
@@ -2535,7 +2535,7 @@ 
   );
 
    $app_list_strings['kbadmin_actions_dom'] =
-    array (
+    array(
     ''          => '--Admin Actions--',
     'Create New Tag' => 'Create New Tag',
     'Delete Tag'=>'Delete Tag',
@@ -3106,26 +3106,26 @@ 
   'Asia/Magadan' => 'Asia/Magadan',
   'Asia/Kamchatka' => 'Asia/Kamchatka',
   'Asia/Anadyr' => 'Asia/Anadyr',
-  'Europe/Belgrade' => 'Europe/Belgrade' ,
-  'Europe/Madrid' =>'Europe/Madrid' ,
+  'Europe/Belgrade' => 'Europe/Belgrade',
+  'Europe/Madrid' =>'Europe/Madrid',
   'Africa/Ceuta' => 'Africa/Ceuta',
   'Atlantic/Canary' => 'Atlantic/Canary',
   'Europe/Stockholm' => 'Europe/Stockholm',
-  'Europe/Zurich' => 'Europe/Zurich' ,
+  'Europe/Zurich' => 'Europe/Zurich',
   'Europe/Istanbul' => 'Europe/Istanbul',
   'Europe/Kiev' => 'Europe/Kiev',
   'Europe/Uzhgorod' => 'Europe/Uzhgorod',
   'Europe/Zaporozhye' => 'Europe/Zaporozhye',
   'Europe/Simferopol' => 'Europe/Simferopol',
   'America/New_York' => 'America/New York',
-  'America/Chicago' =>'America/Chicago' ,
+  'America/Chicago' =>'America/Chicago',
   'America/North_Dakota/Center' => 'America/North Dakota/Center',
   'America/Denver' => 'America/Denver',
   'America/Los_Angeles' => 'America/Los Angeles',
   'America/Juneau' => 'America/Juneau',
   'America/Yakutat' => 'America/Yakutat',
   'America/Anchorage' => 'America/Anchorage',
-  'America/Nome' =>'America/Nome' ,
+  'America/Nome' =>'America/Nome',
   'America/Adak' => 'America/Adak',
   'Pacific/Honolulu' => 'Pacific/Honolulu',
   'America/Phoenix' => 'America/Phoenix',
@@ -3134,17 +3134,17 @@ 
   'America/Indiana/Marengo' => 'America/Indiana/Marengo',
   'America/Indiana/Knox' =>  'America/Indiana/Knox',
   'America/Indiana/Vevay' => 'America/Indiana/Vevay',
-  'America/Kentucky/Louisville' =>'America/Kentucky/Louisville'  ,
-  'America/Kentucky/Monticello' =>  'America/Kentucky/Monticello' ,
+  'America/Kentucky/Louisville' =>'America/Kentucky/Louisville',
+  'America/Kentucky/Monticello' =>  'America/Kentucky/Monticello',
   'America/Detroit' => 'America/Detroit',
   'America/Menominee' => 'America/Menominee',
   'America/St_Johns' => 'America/St. Johns',
-  'America/Goose_Bay' => 'America/Goose_Bay' ,
+  'America/Goose_Bay' => 'America/Goose_Bay',
   'America/Halifax' => 'America/Halifax',
-  'America/Glace_Bay' =>'America/Glace Bay' ,
+  'America/Glace_Bay' =>'America/Glace Bay',
   'America/Montreal' => 'America/Montreal',
   'America/Toronto' => 'America/Toronto',
-  'America/Thunder_Bay' => 'America/Thunder Bay' ,
+  'America/Thunder_Bay' => 'America/Thunder Bay',
   'America/Nipigon' => 'America/Nipigon',
   'America/Rainy_River' => 'America/Rainy River',
   'America/Winnipeg' => 'America/Winnipeg',
@@ -3153,14 +3153,14 @@ 
   'America/Edmonton' =>  'America/Edmonton',
   'America/Vancouver' => 'America/Vancouver',
   'America/Dawson_Creek' => 'America/Dawson Creek',
-  'America/Pangnirtung' => 'America/Pangnirtung'  ,
-  'America/Iqaluit' => 'America/Iqaluit' ,
-  'America/Coral_Harbour' => 'America/Coral Harbour' ,
+  'America/Pangnirtung' => 'America/Pangnirtung',
+  'America/Iqaluit' => 'America/Iqaluit',
+  'America/Coral_Harbour' => 'America/Coral Harbour',
   'America/Rankin_Inlet' => 'America/Rankin Inlet',
   'America/Cambridge_Bay' => 'America/Cambridge Bay',
   'America/Yellowknife' => 'America/Yellowknife',
-  'America/Inuvik' =>'America/Inuvik' ,
-  'America/Whitehorse' => 'America/Whitehorse' ,
+  'America/Inuvik' =>'America/Inuvik',
+  'America/Whitehorse' => 'America/Whitehorse',
   'America/Dawson' => 'America/Dawson',
   'America/Cancun' => 'America/Cancun',
   'America/Merida' => 'America/Merida',
@@ -3172,7 +3172,7 @@ 
   'America/Tijuana' => 'America/Tijuana',
   'America/Anguilla' => 'America/Anguilla',
   'America/Antigua' => 'America/Antigua',
-  'America/Nassau' =>'America/Nassau' ,
+  'America/Nassau' =>'America/Nassau',
   'America/Barbados' => 'America/Barbados',
   'America/Belize' => 'America/Belize',
   'Atlantic/Bermuda' => 'Atlantic/Bermuda',
@@ -3192,7 +3192,7 @@ 
   'America/Montserrat' => 'America/Montserrat',
   'America/Managua' => 'America/Managua',
   'America/Panama' => 'America/Panama',
-  'America/Puerto_Rico' =>'America/Puerto_Rico' ,
+  'America/Puerto_Rico' =>'America/Puerto_Rico',
   'America/St_Kitts' => 'America/St_Kitts',
   'America/St_Lucia' => 'America/St_Lucia',
   'America/Miquelon' => 'America/Miquelon',
@@ -3228,11 +3228,11 @@ 
   'America/Eirunepe' => 'America/Eirunepe',
   'America/Rio_Branco' => 'America/Rio Branco',
   'America/Santiago' => 'America/Santiago',
-  'Pacific/Easter' => 'Pacific/Easter' ,
+  'Pacific/Easter' => 'Pacific/Easter',
   'America/Bogota' => 'America/Bogota',
   'America/Curacao' => 'America/Curacao',
   'America/Guayaquil' => 'America/Guayaquil',
-  'Pacific/Galapagos' => 'Pacific/Galapagos' ,
+  'Pacific/Galapagos' => 'Pacific/Galapagos',
   'Atlantic/Stanley' => 'Atlantic/Stanley',
   'America/Cayenne' => 'America/Cayenne',
   'America/Guyana' => 'America/Guyana',
@@ -3246,7 +3246,7 @@ 
   );
 
   $app_list_strings['moduleList']['Sugar_Favorites'] = 'Favorites';
-  $app_list_strings['eapm_list']= array(
+  $app_list_strings['eapm_list'] = array(
     'Sugar'=>'SuiteCRM',
     'WebEx'=>'WebEx',
     'GoToMeeting'=>'GoToMeeting',
@@ -3256,10 +3256,10 @@ 
     'Facebook'=>'Facebook',
     'Twitter'=>'Twitter',
   );
-  $app_list_strings['eapm_list_import']= array(
+  $app_list_strings['eapm_list_import'] = array(
   	'Google' => 'Google Contacts',
   );
-$app_list_strings['eapm_list_documents']= array(
+$app_list_strings['eapm_list_documents'] = array(
   	'Google' => 'Google Docs',
   );
 	$app_list_strings['token_status'] = array(
@@ -3268,23 +3268,23 @@ 
         3 => 'Invalid',
     );
 
-$app_list_strings ['emailTemplates_type_list'] = array (
-    '' => '' ,
-    'campaign' => 'Campaign' ,
+$app_list_strings ['emailTemplates_type_list'] = array(
+    '' => '',
+    'campaign' => 'Campaign',
     'email' => 'Email',
   );
 
-$app_list_strings ['emailTemplates_type_list_campaigns'] = array (
-    '' => '' ,
-    'campaign' => 'Campaign' ,
+$app_list_strings ['emailTemplates_type_list_campaigns'] = array(
+    '' => '',
+    'campaign' => 'Campaign',
   );
 
-$app_list_strings ['emailTemplates_type_list_no_workflow'] = array (
-    '' => '' ,
-    'campaign' => 'Campaign' ,
+$app_list_strings ['emailTemplates_type_list_no_workflow'] = array(
+    '' => '',
+    'campaign' => 'Campaign',
     'email' => 'Email',
   );
-$app_strings ['documentation'] = array (
+$app_strings ['documentation'] = array(
     'LBL_DOCS' => 'Documentation',
     'ULT' => '02_Sugar_Ultimate',
 	'ENT' => '02_Sugar_Enterprise',
@@ -3342,14 +3342,14 @@ 
 
 //aop
 $app_list_strings['case_state_dom'] =
-    array (
+    array(
         'Open' => 'Open',
         'Closed' => 'Closed',
     );
 $app_list_strings['case_state_default_key'] = 'Open';
 $app_list_strings['case_status_default_key'] = 'Open_New';
 $app_list_strings['case_status_dom'] =
-    array (
+    array(
         'Open_New' => 'New',
         'Open_Assigned' => 'Assigned',
         'Closed_Closed' => 'Closed',
@@ -3358,11 +3358,11 @@ 
         'Closed_Duplicate' => 'Duplicate',
     );
 $app_list_strings['contact_portal_user_type_dom'] =
-    array (
+    array(
         'Single' => 'Single user',
         'Account' => 'Account user',
     );
-$app_list_strings['dom_email_distribution_for_auto_create']=array (
+$app_list_strings['dom_email_distribution_for_auto_create'] = array(
     'AOPDefault' => 'Use AOP Default',
     'singleUser' => 'Single User',
     'roundRobin' => 'Round-Robin',
@@ -3990,6 +3990,6 @@ 
 
 $app_strings['LBL_DISCOVER_SUITECRM'] = 'Discover SuiteCRM';
 
-$app_list_strings['collection_temp_list'] = array ( 'Tasks' => 'Tasks', 'Meetings' => 'Meetings', 'Calls' => 'Calls', 'Notes' => 'Notes', 'Emails' => 'Emails' );
+$app_list_strings['collection_temp_list'] = array('Tasks' => 'Tasks', 'Meetings' => 'Meetings', 'Calls' => 'Calls', 'Notes' => 'Notes', 'Emails' => 'Emails');
 
 ?>

include/SugarObjects/VardefManager.php

Spacing +58 added lines, -58 removed lines

@@ -42,7 +42,7 @@ 
  * Vardefs management
  * @api
  */
-class VardefManager{
+class VardefManager {
     static $custom_disabled_modules = array();
     static $linkFields;
 
@@ -67,8 +67,8 @@ 
         if (isset(VardefManager::$custom_disabled_modules[$module]))
         {
             $vardef_paths = array(
-                'custom/modules/' . $module . '/Ext/Vardefs/vardefs.ext.php',
-                'custom/Extension/modules/' . $module . '/Ext/Vardefs/vardefs.php'
+                'custom/modules/'.$module.'/Ext/Vardefs/vardefs.ext.php',
+                'custom/Extension/modules/'.$module.'/Ext/Vardefs/vardefs.php'
             );
 
             //search a predefined set of locations for the vardef files
@@ -95,41 +95,41 @@ 
         }
     }
 
-    static function addTemplate($module, $object, $template, $object_name=false){
-        if($template == 'default')$template = 'basic';
+    static function addTemplate($module, $object, $template, $object_name = false) {
+        if ($template == 'default')$template = 'basic';
         $templates = array();
         $fields = array();
-        if(empty($object_name))$object_name = $object;
+        if (empty($object_name))$object_name = $object;
         $_object_name = strtolower($object_name);
-        if(!empty($GLOBALS['dictionary'][$object]['table'])){
+        if (!empty($GLOBALS['dictionary'][$object]['table'])) {
             $table_name = $GLOBALS['dictionary'][$object]['table'];
-        }else{
+        } else {
             $table_name = strtolower($module);
         }
 
-        if(empty($templates[$template])){
-            $path = 'include/SugarObjects/templates/' . $template . '/vardefs.php';
-            if(file_exists($path)){
+        if (empty($templates[$template])) {
+            $path = 'include/SugarObjects/templates/'.$template.'/vardefs.php';
+            if (file_exists($path)) {
                 require($path);
                 $templates[$template] = $vardefs;
-            }else{
-                $path = 'include/SugarObjects/implements/' . $template . '/vardefs.php';
-                if(file_exists($path)){
+            } else {
+                $path = 'include/SugarObjects/implements/'.$template.'/vardefs.php';
+                if (file_exists($path)) {
                     require($path);
                     $templates[$template] = $vardefs;
                 }
             }
         }
        
-        if(!empty($templates[$template])){
-            if(empty($GLOBALS['dictionary'][$object]['fields']))$GLOBALS['dictionary'][$object]['fields'] = array();
-            if(empty($GLOBALS['dictionary'][$object]['relationships']))$GLOBALS['dictionary'][$object]['relationships'] = array();
-            if(empty($GLOBALS['dictionary'][$object]['indices']))$GLOBALS['dictionary'][$object]['indices'] = array();
+        if (!empty($templates[$template])) {
+            if (empty($GLOBALS['dictionary'][$object]['fields']))$GLOBALS['dictionary'][$object]['fields'] = array();
+            if (empty($GLOBALS['dictionary'][$object]['relationships']))$GLOBALS['dictionary'][$object]['relationships'] = array();
+            if (empty($GLOBALS['dictionary'][$object]['indices']))$GLOBALS['dictionary'][$object]['indices'] = array();
             $GLOBALS['dictionary'][$object]['fields'] = array_merge($templates[$template]['fields'], $GLOBALS['dictionary'][$object]['fields']);
-            if(!empty($templates[$template]['relationships']))$GLOBALS['dictionary'][$object]['relationships'] = array_merge($templates[$template]['relationships'], $GLOBALS['dictionary'][$object]['relationships']);
-            if(!empty($templates[$template]['indices']))$GLOBALS['dictionary'][$object]['indices'] = array_merge($templates[$template]['indices'], $GLOBALS['dictionary'][$object]['indices']);
+            if (!empty($templates[$template]['relationships']))$GLOBALS['dictionary'][$object]['relationships'] = array_merge($templates[$template]['relationships'], $GLOBALS['dictionary'][$object]['relationships']);
+            if (!empty($templates[$template]['indices']))$GLOBALS['dictionary'][$object]['indices'] = array_merge($templates[$template]['indices'], $GLOBALS['dictionary'][$object]['indices']);
             // maintain a record of this objects inheritance from the SugarObject templates...
-            $GLOBALS['dictionary'][$object]['templates'][ $template ] = $template ;
+            $GLOBALS['dictionary'][$object]['templates'][$template] = $template;
         }
     }
 
@@ -142,7 +142,7 @@ 
      */
     static function cleanVardefs($fieldDefs)
     {
-        if(isset($fieldDefs['fields'])) {
+        if (isset($fieldDefs['fields'])) {
             foreach ($fieldDefs['fields'] as $field => $defs) {
                 if (empty($defs['name']) || empty($defs['type'])) {
                     unset($fieldDefs['fields'][$field]);
@@ -158,7 +158,7 @@ 
      * @param string $module the name of the module
      * @param string $object the name of the object
      */
-    static function saveCache($module,$object, $additonal_objects= array()){
+    static function saveCache($module, $object, $additonal_objects = array()) {
 
         if (empty($GLOBALS['dictionary'][$object]))
             $object = BeanFactory::getObjectName($module);
@@ -166,17 +166,17 @@ 
         //Sometimes bad definitions can get in from left over extensions or file system lag(caching). We need to clean those.
         $data = self::cleanVardefs($GLOBALS['dictionary'][$object]);
 
-        $file = create_cache_directory('modules/' . $module . '/' . $object . 'vardefs.php');
+        $file = create_cache_directory('modules/'.$module.'/'.$object.'vardefs.php');
 
-        $out="<?php \n \$GLOBALS[\"dictionary\"][\"". $object . "\"]=" . var_export($data, true) .";";
+        $out = "<?php \n \$GLOBALS[\"dictionary\"][\"".$object."\"]=".var_export($data, true).";";
         sugar_file_put_contents_atomic($file, $out);
-        if ( sugar_is_file($file) && is_readable($file)) {
+        if (sugar_is_file($file) && is_readable($file)) {
             include($file);
         }
 
         // put the item in the sugar cache.
         $key = "VardefManager.$module.$object";
-        sugar_cache_put($key,$data);
+        sugar_cache_put($key, $data);
     }
 
     /**
@@ -186,14 +186,14 @@ 
      *                      clear vardef cache for all modules.
      * @param string object_name the name of the object we are clearing this is for sugar_cache
      */
-    static function clearVardef($module_dir = '', $object_name = ''){
+    static function clearVardef($module_dir = '', $object_name = '') {
         //if we have a module name specified then just remove that vardef file
         //otherwise go through each module and remove the vardefs.php
-        if(!empty($module_dir) && !empty($object_name)){
+        if (!empty($module_dir) && !empty($object_name)) {
             VardefManager::_clearCache($module_dir, $object_name);
-        }else{
+        } else {
             global $beanList;
-            foreach($beanList as $module_dir => $object_name){
+            foreach ($beanList as $module_dir => $object_name) {
                 VardefManager::_clearCache($module_dir, $object_name);
             }
         }
@@ -204,8 +204,8 @@ 
      * @param string module_dir the module_dir to clear
      * @param string object_name the name of the object we are clearing this is for sugar_cache
      */
-    static function _clearCache($module_dir = '', $object_name = ''){
-        if(!empty($module_dir) && !empty($object_name)){
+    static function _clearCache($module_dir = '', $object_name = '') {
+        if (!empty($module_dir) && !empty($object_name)) {
 
             //Some modules like cases have a bean name that doesn't match the object name
             if (empty($GLOBALS['dictionary'][$object_name])) {
@@ -213,9 +213,9 @@ 
                 $object_name = $newName != false ? $newName : $object_name;
             }
 
-            $file = sugar_cached('modules/').$module_dir.'/' . $object_name . 'vardefs.php';
+            $file = sugar_cached('modules/').$module_dir.'/'.$object_name.'vardefs.php';
 
-            if(file_exists($file)){
+            if (file_exists($file)) {
                 unlink($file);
                 $key = "VardefManager.$module_dir.$object_name";
                 sugar_cache_clear($key);
@@ -231,7 +231,7 @@ 
      * @param string $object the given object we wish to load the vardefs for
      * @param array $additional_search_paths an array which allows a consumer to pass in additional vardef locations to search
      */
-    static function refreshVardefs($module, $object, $additional_search_paths = null, $cacheCustom = true, $params = array()){
+    static function refreshVardefs($module, $object, $additional_search_paths = null, $cacheCustom = true, $params = array()) {
         // Some of the vardefs do not correctly define dictionary as global.  Declare it first.
         global $dictionary, $beanList;
         $vardef_paths = array(
@@ -241,20 +241,20 @@ 
                  );
 
         // Add in additional search paths if they were provided.
-        if(!empty($additional_search_paths) && is_array($additional_search_paths))
+        if (!empty($additional_search_paths) && is_array($additional_search_paths))
         {
             $vardef_paths = array_merge($vardef_paths, $additional_search_paths);
         }
         $found = false;
         //search a predefined set of locations for the vardef files
-        foreach($vardef_paths as $path){
-            if(file_exists($path)){
+        foreach ($vardef_paths as $path) {
+            if (file_exists($path)) {
                 require($path);
                 $found = true;
             }
         }
         //Some modules have multiple beans, we need to see if this object has a module_dir that is different from its module_name
-        if(!$found){
+        if (!$found) {
             $temp = BeanFactory::newBean($module);
             if ($temp)
             {
@@ -273,15 +273,15 @@ 
         }
 
         //load custom fields into the vardef cache
-        if($cacheCustom){
+        if ($cacheCustom) {
             require_once("modules/DynamicFields/DynamicField.php");
-            $df = new DynamicField ($module) ;
+            $df = new DynamicField($module);
             $df->buildCache($module, false);
         }
 
         //great! now that we have loaded all of our vardefs.
         //let's go save them to the cache file.
-        if(!empty($dictionary[$object])) {
+        if (!empty($dictionary[$object])) {
             VardefManager::saveCache($module, $object);
         }
     }
@@ -318,10 +318,10 @@ 
 
         $vardef = $dictionary[$object];
         $links = array();
-        foreach($vardef['fields'] as $name => $def)
+        foreach ($vardef['fields'] as $name => $def)
         {
             //Look through all link fields for related modules that have calculated fields that use that relationship
-            if(!empty($def['type']) && $def['type'] == 'link' && !empty($def['relationship']))
+            if (!empty($def['type']) && $def['type'] == 'link' && !empty($def['relationship']))
             {
                 $links[$name] = $def;
             }
@@ -337,14 +337,14 @@ 
     {
         $cacheKey = "LFR{$module}{$object}{$relName}";
         $cacheValue = sugar_cache_retrieve($cacheKey);
-        if(!empty($cacheValue))
+        if (!empty($cacheValue))
             return $cacheValue;
 
         $relLinkFields = self::getLinkFieldsForModule($module, $object);
         $matches = array();
         if (!empty($relLinkFields))
         {
-            foreach($relLinkFields as $rfName => $rfDef)
+            foreach ($relLinkFields as $rfName => $rfDef)
             {
                 if ($rfDef['relationship'] == $relName)
                 {
@@ -361,7 +361,7 @@ 
             $results = $matches;
 
         sugar_cache_put($cacheKey, $results);
-        return $results ;
+        return $results;
     }
 
 
@@ -400,22 +400,22 @@ 
      * @param string $object the given object we wish to load the vardefs for
      * @param bool   $refresh whether or not we wish to refresh the cache file.
      */
-    static function loadVardef($module, $object, $refresh=false, $params = array()){
+    static function loadVardef($module, $object, $refresh = false, $params = array()) {
         //here check if the cache file exists, if it does then load it, if it doesn't
         //then call refreshVardef
         //if either our session or the system is set to developerMode then refresh is set to true
-        if(inDeveloperMode() || !empty($_SESSION['developerMode'])){
+        if (inDeveloperMode() || !empty($_SESSION['developerMode'])) {
             $refresh = true;
         }
         // Retrieve the vardefs from cache.
         $key = "VardefManager.$module.$object";
 
-        if(!$refresh)
+        if (!$refresh)
         {
             $return_result = sugar_cache_retrieve($key);
             $return_result = self::applyGlobalAccountRequirements($return_result);
 
-            if(!empty($return_result))
+            if (!empty($return_result))
             {
                 $GLOBALS['dictionary'][$object] = $return_result;
                 return;
@@ -424,28 +424,28 @@ 
 
         // Some of the vardefs do not correctly define dictionary as global.  Declare it first.
         global $dictionary;
-        if(empty($GLOBALS['dictionary'][$object]) || $refresh){
+        if (empty($GLOBALS['dictionary'][$object]) || $refresh) {
             //if the consumer has demanded a refresh or the cache/modules... file
             //does not exist, then we should do out and try to reload things
 
-			$cachedfile = sugar_cached('modules/'). $module . '/' . $object . 'vardefs.php';
-			if($refresh || !file_exists($cachedfile)){
+			$cachedfile = sugar_cached('modules/').$module.'/'.$object.'vardefs.php';
+			if ($refresh || !file_exists($cachedfile)) {
 				VardefManager::refreshVardefs($module, $object, null, true, $params);
 			}
 
             //at this point we should have the cache/modules/... file
             //which was created from the refreshVardefs so let's try to load it.
-            if(file_exists($cachedfile))
+            if (file_exists($cachedfile))
             {
                 if (is_readable($cachedfile))
                 {
                     include($cachedfile);
                 }
                 // now that we hae loaded the data from disk, put it in the cache.
-                if(!empty($GLOBALS['dictionary'][$object]))
+                if (!empty($GLOBALS['dictionary'][$object]))
                 {
                     $GLOBALS['dictionary'][$object] = self::applyGlobalAccountRequirements($GLOBALS['dictionary'][$object]);
-                    sugar_cache_put($key,$GLOBALS['dictionary'][$object]);
+                    sugar_cache_put($key, $GLOBALS['dictionary'][$object]);
                 }
             }
         }

include/database/MssqlManager.php

Spacing +254 added lines, -254 removed lines

@@ -1,5 +1,5 @@ 
 <?php
-if(!defined('sugarEntry') || !sugarEntry) die('Not A Valid Entry Point');
+if (!defined('sugarEntry') || !sugarEntry) die('Not A Valid Entry Point');
 /*********************************************************************************
  * SugarCRM Community Edition is a customer relationship management program developed by
  * SugarCRM, Inc. Copyright (C) 2004-2013 SugarCRM Inc.
@@ -175,11 +175,11 @@ 
         ini_set('mssql.datetimeconvert', '0');
 
         //set the text size and textlimit to max number so that blob columns are not truncated
-        ini_set('mssql.textlimit','2147483647');
-        ini_set('mssql.textsize','2147483647');
-        ini_set('mssql.charset','UTF-8');
+        ini_set('mssql.textlimit', '2147483647');
+        ini_set('mssql.textsize', '2147483647');
+        ini_set('mssql.charset', 'UTF-8');
 
-        if(!empty($configOptions['db_host_instance'])) {
+        if (!empty($configOptions['db_host_instance'])) {
             $configOptions['db_host_instance'] = trim($configOptions['db_host_instance']);
         }
         //set the connections parameters
@@ -191,37 +191,37 @@ 
 
         //create persistent connection
         if ($this->getOption('persistent')) {
-            $this->database =@mssql_pconnect(
-                $connect_param ,
+            $this->database = @mssql_pconnect(
+                $connect_param,
                 $configOptions['db_user_name'],
                 $configOptions['db_password']
                 );
         }
         //if no persistent connection created, then create regular connection
-        if(!$this->database){
+        if (!$this->database) {
             $this->database = mssql_connect(
-                    $connect_param ,
+                    $connect_param,
                     $configOptions['db_user_name'],
                     $configOptions['db_password']
                     );
-            if(!$this->database){
+            if (!$this->database) {
                 $GLOBALS['log']->fatal("Could not connect to server ".$configOptions['db_host_name'].
                     " as ".$configOptions['db_user_name'].".");
-                if($dieOnError) {
+                if ($dieOnError) {
                     sugar_die($GLOBALS['app_strings']['ERR_NO_DB']);
                 } else {
                     return false;
                 }
             }
-            if($this->database && $this->getOption('persistent')){
+            if ($this->database && $this->getOption('persistent')) {
                 $_SESSION['administrator_error'] = "<B>Severe Performance Degradation: Persistent Database Connections "
                     . "not working.  Please set \$sugar_config['dbconfigoption']['persistent'] to false in your "
                     . "config.php file</B>";
             }
         }
         //make sure connection exists
-        if(!$this->database) {
-                if($dieOnError) {
+        if (!$this->database) {
+                if ($dieOnError) {
                     sugar_die($GLOBALS['app_strings']['ERR_NO_DB']);
                 } else {
                     return false;
@@ -235,19 +235,19 @@ 
         //mssql db maximum number of 5 times at the interval of .2 second. If can not connect
         //it will throw an Unable to select database message.
 
-        if(!empty($configOptions['db_name']) && !@mssql_select_db($configOptions['db_name'], $this->database)){
+        if (!empty($configOptions['db_name']) && !@mssql_select_db($configOptions['db_name'], $this->database)) {
 			$connected = false;
-			for($i=0;$i<5;$i++){
+			for ($i = 0; $i < 5; $i++) {
 				usleep(200000);
-				if(@mssql_select_db($configOptions['db_name'], $this->database)){
+				if (@mssql_select_db($configOptions['db_name'], $this->database)) {
 					$connected = true;
 					break;
 				}
 			}
-			if(!$connected){
-			    $GLOBALS['log']->fatal( "Unable to select database {$configOptions['db_name']}");
-                if($dieOnError) {
-                    if(isset($GLOBALS['app_strings']['ERR_NO_DB'])) {
+			if (!$connected) {
+			    $GLOBALS['log']->fatal("Unable to select database {$configOptions['db_name']}");
+                if ($dieOnError) {
+                    if (isset($GLOBALS['app_strings']['ERR_NO_DB'])) {
                         sugar_die($GLOBALS['app_strings']['ERR_NO_DB']);
                     } else {
                         sugar_die("Could not connect to the database. Please refer to suitecrm.log for details.");
@@ -258,7 +258,7 @@ 
 			}
          }
 
-        if(!$this->checkError('Could Not Connect', $dieOnError))
+        if (!$this->checkError('Could Not Connect', $dieOnError))
             $GLOBALS['log']->info("connected to db");
 
         $this->connectOptions = $configOptions;
@@ -280,16 +280,16 @@ 
 	 */
 	public function query($sql, $dieOnError = false, $msg = '', $suppress = false, $keepResult = false)
     {
-        if(is_array($sql)) {
+        if (is_array($sql)) {
             return $this->queryArray($sql, $dieOnError, $msg, $suppress);
         }
         // Flag if there are odd number of single quotes
         if ((substr_count($sql, "'") & 1))
-            $GLOBALS['log']->error("SQL statement[" . $sql . "] has odd number of single quotes.");
+            $GLOBALS['log']->error("SQL statement[".$sql."] has odd number of single quotes.");
 
 		$sql = $this->_appendN($sql);
 
-        $GLOBALS['log']->info('Query:' . $sql);
+        $GLOBALS['log']->info('Query:'.$sql);
         $this->checkConnection();
         $this->countQuery($sql);
         $this->query_time = microtime(true);
@@ -297,7 +297,7 @@ 
         // Bug 34892 - Clear out previous error message by checking the @@ERROR global variable
 		@mssql_query("SELECT @@ERROR", $this->database);
 
-        $result = $suppress?@mssql_query($sql, $this->database):mssql_query($sql, $this->database);
+        $result = $suppress ? @mssql_query($sql, $this->database) : mssql_query($sql, $this->database);
 
         if (!$result) {
             // awu Bug 10657: ignoring mssql error message 'Changed database context to' - an intermittent
@@ -309,13 +309,13 @@ 
 			$sqlpos3 = strpos($sqlmsg, 'Checking identity information:');
 
 			if ($sqlpos !== false || $sqlpos2 !== false || $sqlpos3 !== false)		// if sqlmsg has 'Changed database context to', just log it
-				$GLOBALS['log']->debug($sqlmsg . ": " . $sql );
+				$GLOBALS['log']->debug($sqlmsg.": ".$sql);
 			else {
-				$GLOBALS['log']->fatal($sqlmsg . ": " . $sql );
-				if($dieOnError)
-					sugar_die('SQL Error : ' . $sqlmsg);
+				$GLOBALS['log']->fatal($sqlmsg.": ".$sql);
+				if ($dieOnError)
+					sugar_die('SQL Error : '.$sqlmsg);
 				else
-					echo 'SQL Error : ' . $sqlmsg;
+					echo 'SQL Error : '.$sqlmsg;
 			}
         }
 
@@ -323,7 +323,7 @@ 
         $GLOBALS['log']->info('Query Execution Time:'.$this->query_time);
 
 
-        $this->checkError($msg.' Query Failed: ' . $sql, $dieOnError);
+        $this->checkError($msg.' Query Failed: '.$sql, $dieOnError);
 
         return $result;
     }
@@ -341,14 +341,14 @@ 
     {
         //set the start to 0, no negs
         if ($start < 0)
-            $start=0;
+            $start = 0;
 
-        $GLOBALS['log']->debug(print_r(func_get_args(),true));
+        $GLOBALS['log']->debug(print_r(func_get_args(), true));
 
         $this->lastsql = $sql;
 
         //change the casing to lower for easier string comparison, and trim whitespaces
-        $sql = strtolower(trim($sql)) ;
+        $sql = strtolower(trim($sql));
 
         //set default sql
         $limitUnionSQL = $sql;
@@ -364,16 +364,16 @@ 
         $arr_count = 0;
 
         //process if there are elements
-        if ($unionOrderByCount){
+        if ($unionOrderByCount) {
             //we really want the last order by, so reconstruct string
             //adding a 1 to count, as we dont wish to process the last element
             $unionsql = '';
-            while ($unionOrderByCount>$arr_count+1) {
+            while ($unionOrderByCount > $arr_count + 1) {
                 $unionsql .= $orderByArray[$arr_count];
-                $arr_count = $arr_count+1;
+                $arr_count = $arr_count + 1;
                 //add an "order by" string back if we are coming into loop again
                 //remember they were taken out when array was created
-                if ($unionOrderByCount>$arr_count+1) {
+                if ($unionOrderByCount > $arr_count + 1) {
                     $unionsql .= "order by";
                 }
             }
@@ -403,7 +403,7 @@ 
         //so we do not want to strip the alias like in other queries.  Just add the "order by" string and
         //pass column name as is
         if ($unionOrderBy != '') {
-            $unionOrderBy = ' order by ' . $unionOrderBy;
+            $unionOrderBy = ' order by '.$unionOrderBy;
         }
 
         //Bug 56560, use top query in conjunction with rownumber() function
@@ -412,13 +412,13 @@ 
         //If not for paging, no need to use rownumber() function
         if ($count == 1 && $start == 0)
         {
-            $limitUnionSQL = "SELECT TOP $count * FROM (" .$unionsql .") as top_count ".$unionOrderBy;
+            $limitUnionSQL = "SELECT TOP $count * FROM (".$unionsql.") as top_count ".$unionOrderBy;
         }
         else
         {
             $limitUnionSQL = "SELECT TOP $count * FROM( select ROW_NUMBER() OVER ( order by "
             .$rowNumOrderBy.") AS row_number, * FROM ("
-            .$unionsql .") As numbered) "
+            .$unionsql.") As numbered) "
             . "As top_count_limit WHERE row_number > $start "
             .$unionOrderBy;
         }
@@ -437,64 +437,64 @@ 
         $newSQL = $sql;
         $distinctSQLARRAY = array();
         if (strpos($sql, "UNION") && !preg_match("/(')(UNION).?(')/i", $sql))
-            $newSQL = $this->handleUnionLimitQuery($sql,$start,$count);
+            $newSQL = $this->handleUnionLimitQuery($sql, $start, $count);
         else {
             if ($start < 0)
                 $start = 0;
-            $GLOBALS['log']->debug(print_r(func_get_args(),true));
+            $GLOBALS['log']->debug(print_r(func_get_args(), true));
             $this->lastsql = $sql;
             $matches = array();
-            preg_match('/^(.*SELECT\b)(.*?\bFROM\b.*\bWHERE\b)(.*)$/isU',$sql, $matches);
+            preg_match('/^(.*SELECT\b)(.*?\bFROM\b.*\bWHERE\b)(.*)$/isU', $sql, $matches);
             if (!empty($matches[3])) {
                 if ($start == 0) {
                     $match_two = strtolower($matches[2]);
-                    if (!strpos($match_two, "distinct")> 0 && strpos($match_two, "distinct") !==0) {
+                    if (!strpos($match_two, "distinct") > 0 && strpos($match_two, "distinct") !== 0) {
                         $orderByMatch = array();
-                        preg_match('/^(.*)(\bORDER BY\b)(.*)$/is',$matches[3], $orderByMatch);
+                        preg_match('/^(.*)(\bORDER BY\b)(.*)$/is', $matches[3], $orderByMatch);
                         if (!empty($orderByMatch[3])) {
                             $selectPart = array();
                             preg_match('/^(.*)(\bFROM\b.*)$/isU', $matches[2], $selectPart);
                             $newSQL = "SELECT TOP $count * FROM
                                 (
-                                    " . $matches[1] . $selectPart[1] . ", ROW_NUMBER()
-                                    OVER (ORDER BY " . $this->returnOrderBy($sql, $orderByMatch[3]) . ") AS row_number
-                                    " . $selectPart[2] . $orderByMatch[1]. "
+                                    ".$matches[1].$selectPart[1].", ROW_NUMBER()
+                                    OVER (ORDER BY " . $this->returnOrderBy($sql, $orderByMatch[3]).") AS row_number
+                                    " . $selectPart[2].$orderByMatch[1]."
                                 ) AS a
                                 WHERE row_number > $start";
                         }
                         else {
-                            $newSQL = $matches[1] . " TOP $count " . $matches[2] . $matches[3];
+                            $newSQL = $matches[1]." TOP $count ".$matches[2].$matches[3];
                         }
                     }
                     else {
                         $distinct_o = strpos($match_two, "distinct");
                         $up_to_distinct_str = substr($match_two, 0, $distinct_o);
                         //check to see if the distinct is within a function, if so, then proceed as normal
-                        if (strpos($up_to_distinct_str,"(")) {
+                        if (strpos($up_to_distinct_str, "(")) {
                             //proceed as normal
-                            $newSQL = $matches[1] . " TOP $count " . $matches[2] . $matches[3];
+                            $newSQL = $matches[1]." TOP $count ".$matches[2].$matches[3];
                         }
                         else {
                             //if distinct is not within a function, then parse
                             //string contains distinct clause, "TOP needs to come after Distinct"
                             //get position of distinct
                             $match_zero = strtolower($matches[0]);
-                            $distinct_pos = strpos($match_zero , "distinct");
+                            $distinct_pos = strpos($match_zero, "distinct");
                             //get position of where
                             $where_pos = strpos($match_zero, "where");
                             //parse through string
-                            $beg = substr($matches[0], 0, $distinct_pos+9 );
-                            $mid = substr($matches[0], strlen($beg), ($where_pos+5) - (strlen($beg)));
-                            $end = substr($matches[0], strlen($beg) + strlen($mid) );
+                            $beg = substr($matches[0], 0, $distinct_pos + 9);
+                            $mid = substr($matches[0], strlen($beg), ($where_pos + 5) - (strlen($beg)));
+                            $end = substr($matches[0], strlen($beg) + strlen($mid));
                             //repopulate matches array
                             $matches[1] = $beg; $matches[2] = $mid; $matches[3] = $end;
 
-                            $newSQL = $matches[1] . " TOP $count " . $matches[2] . $matches[3];
+                            $newSQL = $matches[1]." TOP $count ".$matches[2].$matches[3];
                         }
                     }
                 } else {
                     $orderByMatch = array();
-                    preg_match('/^(.*)(\bORDER BY\b)(.*)$/is',$matches[3], $orderByMatch);
+                    preg_match('/^(.*)(\bORDER BY\b)(.*)$/is', $matches[3], $orderByMatch);
 
                     //if there is a distinct clause, parse sql string as we will have to insert the rownumber
                     //for paging, AFTER the distinct clause
@@ -512,8 +512,8 @@ 
 
                         //take out the select and distinct from string so we can reuse in group by
                         $dist_str = 'distinct';
-                        preg_match('/\b' . $dist_str . '\b/simU', $matches_sql, $matchesPartSQL, PREG_OFFSET_CAPTURE);
-                        $matches_sql = trim(substr($matches_sql,$matchesPartSQL[0][1] + strlen($dist_str)));
+                        preg_match('/\b'.$dist_str.'\b/simU', $matches_sql, $matchesPartSQL, PREG_OFFSET_CAPTURE);
+                        $matches_sql = trim(substr($matches_sql, $matchesPartSQL[0][1] + strlen($dist_str)));
                         //get the position of where and from for further processing
                         preg_match('/\bfrom\b/simU', $matches_sql, $matchesPartSQL, PREG_OFFSET_CAPTURE);
                         $from_pos = $matchesPartSQL[0][1];
@@ -521,17 +521,17 @@ 
                         $where_pos = $matchesPartSQL[0][1];
                         //split the sql into a string before and after the from clause
                         //we will use the columns being selected to construct the group by clause
-                        if ($from_pos>0 ) {
+                        if ($from_pos > 0) {
                             $distinctSQLARRAY[0] = substr($matches_sql, 0, $from_pos);
                             $distinctSQLARRAY[1] = substr($matches_sql, $from_pos);
                             //get position of order by (if it exists) so we can strip it from the string
                             $ob_pos = strpos($distinctSQLARRAY[1], "order by");
                             if ($ob_pos) {
-                                $distinctSQLARRAY[1] = substr($distinctSQLARRAY[1],0,$ob_pos);
+                                $distinctSQLARRAY[1] = substr($distinctSQLARRAY[1], 0, $ob_pos);
                             }
 
                             // strip off last closing parentheses from the where clause
-                            $distinctSQLARRAY[1] = preg_replace('/\)\s$/',' ',$distinctSQLARRAY[1]);
+                            $distinctSQLARRAY[1] = preg_replace('/\)\s$/', ' ', $distinctSQLARRAY[1]);
                         }
 
                         $grpByStr = array();
@@ -550,30 +550,30 @@ 
                             $newSQL = "SELECT TOP $count * FROM
                                         (
                                             SELECT ROW_NUMBER()
-                                                OVER (ORDER BY " . preg_replace('/^' . $dist_str . '\s+/', '', $this->returnOrderBy($sql, $orderByMatch[3])) . ") AS row_number,
-                                                count(*) counter, " . $distinctSQLARRAY[0] . "
-                                                " . $distinctSQLARRAY[1] . "
-                                                group by " . $grpByStr . "
+                                                OVER (ORDER BY ".preg_replace('/^'.$dist_str.'\s+/', '', $this->returnOrderBy($sql, $orderByMatch[3])).") AS row_number,
+                                                count(*) counter, " . $distinctSQLARRAY[0]."
+                                                " . $distinctSQLARRAY[1]."
+                                                group by " . $grpByStr."
                                         ) AS a
                                         WHERE row_number > $start";
                         }
                         else {
                         $newSQL = "SELECT TOP $count * FROM
                                     (
-                                        " . $matches[1] . " ROW_NUMBER()
-                                        OVER (ORDER BY " . $this->returnOrderBy($sql, $orderByMatch[3]) . ") AS row_number,
-                                        " . $matches[2] . $orderByMatch[1]. "
+                                        ".$matches[1]." ROW_NUMBER()
+                                        OVER (ORDER BY " . $this->returnOrderBy($sql, $orderByMatch[3]).") AS row_number,
+                                        " . $matches[2].$orderByMatch[1]."
                                     ) AS a
                                     WHERE row_number > $start";
                         }
-                    }else{
+                    } else {
                         //if there is a distinct clause, form query with rownumber after distinct
                         if ($hasDistinct) {
                              $newSQL = "SELECT TOP $count * FROM
                                             (
-                            SELECT ROW_NUMBER() OVER (ORDER BY ".$grpByStr.") AS row_number, count(*) counter, " . $distinctSQLARRAY[0] . "
-                                                        " . $distinctSQLARRAY[1] . "
-                                                    group by " . $grpByStr . "
+                            SELECT ROW_NUMBER() OVER (ORDER BY ".$grpByStr.") AS row_number, count(*) counter, ".$distinctSQLARRAY[0]."
+                                                        " . $distinctSQLARRAY[1]."
+                                                    group by " . $grpByStr."
                                             )
                                             AS a
                                             WHERE row_number > $start";
@@ -581,7 +581,7 @@ 
                         else {
                              $newSQL = "SELECT TOP $count * FROM
                                            (
-                                  " . $matches[1] . " ROW_NUMBER() OVER (ORDER BY " . $sqlArray['FROM'][0]['alias'] . ".id) AS row_number, " . $matches[2] . $matches[3]. "
+                                  ".$matches[1]." ROW_NUMBER() OVER (ORDER BY ".$sqlArray['FROM'][0]['alias'].".id) AS row_number, ".$matches[2].$matches[3]."
                                            )
                                            AS a
                                            WHERE row_number > $start";
@@ -591,9 +591,9 @@ 
             }
         }
 
-        $GLOBALS['log']->debug('Limit Query: ' . $newSQL);
-        if($execute) {
-            $result =  $this->query($newSQL, $dieOnError, $msg);
+        $GLOBALS['log']->debug('Limit Query: '.$newSQL);
+        if ($execute) {
+            $result = $this->query($newSQL, $dieOnError, $msg);
             $this->dump_slow_queries($newSQL);
             return $result;
         } else {
@@ -615,15 +615,15 @@ 
     private function removePatternFromSQL($p_sql, $strip_beg, $strip_end, $patt = 'patt')
     {
         //strip all single quotes out
-        $count = substr_count ( $p_sql, $strip_beg);
+        $count = substr_count($p_sql, $strip_beg);
         $increment = 1;
         if ($strip_beg != $strip_end)
             $increment = 2;
 
-        $i=0;
+        $i = 0;
         $offset = 0;
         $strip_array = array();
-        while ($i<$count && $offset<strlen($p_sql)) {
+        while ($i < $count && $offset < strlen($p_sql)) {
             if ($offset > strlen($p_sql))
             {
 				break;
@@ -634,23 +634,23 @@ 
             {
                 break;
             }
-            $sec_sin = strpos($p_sql, $strip_end, $beg_sin+1);
-            $strip_array[$patt.$i] = substr($p_sql, $beg_sin, $sec_sin - $beg_sin +1);
+            $sec_sin = strpos($p_sql, $strip_end, $beg_sin + 1);
+            $strip_array[$patt.$i] = substr($p_sql, $beg_sin, $sec_sin - $beg_sin + 1);
             if ($increment > 1) {
                 //we are in here because beginning and end patterns are not identical, so search for nesting
-                $exists = strpos($strip_array[$patt.$i], $strip_beg );
-                if ($exists>=0) {
-                    $nested_pos = (strrpos($strip_array[$patt.$i], $strip_beg ));
-                    $strip_array[$patt.$i] = substr($p_sql,$nested_pos+$beg_sin,$sec_sin - ($nested_pos+$beg_sin)+1);
-                    $p_sql = substr($p_sql, 0, $nested_pos+$beg_sin) . " ##". $patt.$i."## " . substr($p_sql, $sec_sin+1);
+                $exists = strpos($strip_array[$patt.$i], $strip_beg);
+                if ($exists >= 0) {
+                    $nested_pos = (strrpos($strip_array[$patt.$i], $strip_beg));
+                    $strip_array[$patt.$i] = substr($p_sql, $nested_pos + $beg_sin, $sec_sin - ($nested_pos + $beg_sin) + 1);
+                    $p_sql = substr($p_sql, 0, $nested_pos + $beg_sin)." ##".$patt.$i."## ".substr($p_sql, $sec_sin + 1);
                     $i = $i + 1;
                     continue;
                 }
             }
-            $p_len = strlen("##". $patt.$i."##");
-            $p_sql = substr($p_sql, 0, $beg_sin) . " ##". $patt.$i."## " . substr($p_sql, $sec_sin+1);
+            $p_len = strlen("##".$patt.$i."##");
+            $p_sql = substr($p_sql, 0, $beg_sin)." ##".$patt.$i."## ".substr($p_sql, $sec_sin + 1);
             //move the marker up
-            $offset = ($sec_sin-($sec_sin-$beg_sin))+$p_len+1; // Adjusting the starting point of the marker
+            $offset = ($sec_sin - ($sec_sin - $beg_sin)) + $p_len + 1; // Adjusting the starting point of the marker
 
             $i = $i + 1;
         }
@@ -672,7 +672,7 @@ 
         $pattern_array = array_reverse($pattern_array);
 
         foreach ($pattern_array as $key => $replace) {
-            $token = str_replace( " ##".$key."## ", $replace,$token);
+            $token = str_replace(" ##".$key."## ", $replace, $token);
         }
 
         return $token;
@@ -688,11 +688,11 @@ 
 	private function getAliasFromSQL($sql, $alias)
     {
         $matches = array();
-        preg_match('/^(.*SELECT)(.*?FROM.*WHERE)(.*)$/isU',$sql, $matches);
+        preg_match('/^(.*SELECT)(.*?FROM.*WHERE)(.*)$/isU', $sql, $matches);
         //parse all single and double  quotes out of array
-        $sin_array = $this->removePatternFromSQL($matches[2], "'", "'","sin_");
+        $sin_array = $this->removePatternFromSQL($matches[2], "'", "'", "sin_");
         $new_sql = array_pop($sin_array);
-        $dub_array = $this->removePatternFromSQL($new_sql, "\"", "\"","dub_");
+        $dub_array = $this->removePatternFromSQL($new_sql, "\"", "\"", "dub_");
         $new_sql = array_pop($dub_array);
 
         //search for parenthesis
@@ -701,7 +701,7 @@ 
 
         //all functions should be removed now, so split the array on commas
         $mstr_sql_array = explode(",", $new_sql);
-        foreach($mstr_sql_array as $token ) {
+        foreach ($mstr_sql_array as $token) {
             if (strpos($token, $alias)) {
                 //found token, add back comments
                 $token = $this->addPatternToSQL($token, $paren_array);
@@ -739,7 +739,7 @@ 
         $found = $found_in_sql;
 
         //if still no match found, then we need to parse through the string
-        if (!$found_in_sql){
+        if (!$found_in_sql) {
             //get count of how many times the match exists in string
             $found_count = substr_count($sql, $orderMatch);
             $i = 0;
@@ -748,21 +748,21 @@ 
             //loop through string as many times as there is a match
             while ($found_count > $i) {
                 //get the first match
-                $found_in_sql = strpos($sql, $orderMatch,$first_);
+                $found_in_sql = strpos($sql, $orderMatch, $first_);
                 //make sure there was a match
-                if($found_in_sql){
+                if ($found_in_sql) {
                     //grab the next 2 individual characters
-                    $str_plusone = substr($sql,$found_in_sql + $len,1);
-                    $str_plustwo = substr($sql,$found_in_sql + $len+1,1);
+                    $str_plusone = substr($sql, $found_in_sql + $len, 1);
+                    $str_plustwo = substr($sql, $found_in_sql + $len + 1, 1);
                     //if one of those characters is a comma, then we have our alias
-                    if ($str_plusone === "," || $str_plustwo === ","){
+                    if ($str_plusone === "," || $str_plustwo === ",") {
                         //keep track of this position
                         $found = $found_in_sql;
                     }
                 }
                 //set the offset and increase the iteration counter
-                $first_ = $found_in_sql+$len;
-                $i = $i+1;
+                $first_ = $found_in_sql + $len;
+                $i = $i + 1;
             }
         }
         //return $found, defaults have been set, so if no match was found it will be a negative number
@@ -794,8 +794,8 @@ 
         $firstSpace = strpos($orig_order_match, " ");
 
         //split order by into column name and ascending/descending
-        $orderMatch = " " . strtolower(substr($orig_order_match, 0, $firstSpace));
-        $asc_desc = trim(substr($orig_order_match,$firstSpace));
+        $orderMatch = " ".strtolower(substr($orig_order_match, 0, $firstSpace));
+        $asc_desc = trim(substr($orig_order_match, $firstSpace));
 
         //look for column name as an alias in sql string
         $found_in_sql = $this->findColumnByAlias($sql, $orderMatch);
@@ -806,18 +806,18 @@ 
             $colMatchPos = strpos($sql, $orderMatch);
             if ($colMatchPos !== false) {
                 //grab sub string up to column name
-                $containsColStr = substr($sql,0, $colMatchPos);
+                $containsColStr = substr($sql, 0, $colMatchPos);
                 //get position of first space, so we can grab table name
                 $lastSpacePos = strrpos($containsColStr, " ");
                 //use positions of column name, space before name, and length of column to find the correct column name
-                $col_name = substr($sql, $lastSpacePos, $colMatchPos-$lastSpacePos+strlen($orderMatch));
+                $col_name = substr($sql, $lastSpacePos, $colMatchPos - $lastSpacePos + strlen($orderMatch));
 				//bug 25485. When sorting by a custom field in Account List and then pressing NEXT >, system gives an error
 				$containsCommaPos = strpos($col_name, ",");
-				if($containsCommaPos !== false) {
-					$col_name = substr($col_name, $containsCommaPos+1);
+				if ($containsCommaPos !== false) {
+					$col_name = substr($col_name, $containsCommaPos + 1);
 				}
                 //add the "asc/desc" order back
-                $col_name = $col_name. " ". $asc_desc;
+                $col_name = $col_name." ".$asc_desc;
 
                 //return column name
                 return $col_name;
@@ -831,22 +831,22 @@ 
             //grab string up to the aliased column
             $GLOBALS['log']->debug("order by found, process sql string");
 
-            $psql = (trim($this->getAliasFromSQL($sql, $orderMatch )));
+            $psql = (trim($this->getAliasFromSQL($sql, $orderMatch)));
             if (empty($psql))
                 $psql = trim(substr($sql, 0, $found_in_sql));
 
             //grab the last comma before the alias
-            preg_match('/\s+' . trim($orderMatch). '/', $psql, $match, PREG_OFFSET_CAPTURE);
+            preg_match('/\s+'.trim($orderMatch).'/', $psql, $match, PREG_OFFSET_CAPTURE);
             $comma_pos = $match[0][1];
             //substring between the comma and the alias to find the joined_table alias and column name
-            $col_name = substr($psql,0, $comma_pos);
+            $col_name = substr($psql, 0, $comma_pos);
 
             //make sure the string does not have an end parenthesis
             //and is not part of a function (i.e. "ISNULL(leads.last_name,'') as name"  )
             //this is especially true for unified search from home screen
 
             $alias_beg_pos = 0;
-            if(strpos($psql, " as "))
+            if (strpos($psql, " as "))
                 $alias_beg_pos = strpos($psql, " as ");
 
             // Bug # 44923 - This breaks the query and does not properly filter isnull
@@ -855,13 +855,13 @@ 
                 $alias_beg_pos = strpos($psql, " "); */
 
             if ($alias_beg_pos > 0) {
-                $col_name = substr($psql,0, $alias_beg_pos );
+                $col_name = substr($psql, 0, $alias_beg_pos);
             }
             //add the "asc/desc" order back
-            $col_name = $col_name. " ". $asc_desc;
+            $col_name = $col_name." ".$asc_desc;
 
             //pass in new order by
-            $GLOBALS['log']->debug("order by being returned is " . $col_name);
+            $GLOBALS['log']->debug("order by being returned is ".$col_name);
             return $col_name;
         }
     }
@@ -877,12 +877,12 @@ 
     {
 
         global $beanList, $beanFiles;
-        $GLOBALS['log']->debug("Module being processed is " . $module_str);
+        $GLOBALS['log']->debug("Module being processed is ".$module_str);
         //get the right module files
         //the module string exists in bean list, then process bean for correct table name
         //note that we exempt the reports module from this, as queries from reporting module should be parsed for
         //correct table name.
-        if (($module_str != 'Reports' && $module_str != 'SavedReport') && isset($beanList[$module_str])  &&  isset($beanFiles[$beanList[$module_str]])){
+        if (($module_str != 'Reports' && $module_str != 'SavedReport') && isset($beanList[$module_str]) && isset($beanFiles[$beanList[$module_str]])) {
             //if the class is not already loaded, then load files
             if (!class_exists($beanList[$module_str]))
                 require_once($beanFiles[$beanList[$module_str]]);
@@ -894,7 +894,7 @@ 
             //make sure table name is not just a blank space, or empty
             $tbl_name = trim($tbl_name);
 
-            if(empty($tbl_name)){
+            if (empty($tbl_name)) {
                 $GLOBALS['log']->debug("Could not find table name for module $module_str. ");
                 $tbl_name = $module_str;
             }
@@ -911,20 +911,20 @@ 
             $sql = str_replace(array("\n", "\r"), " ", $sql);
 
             //look for the location of the "from" in sql string
-            $fromLoc = strpos($sql," from " );
-            if ($fromLoc>0){
+            $fromLoc = strpos($sql, " from ");
+            if ($fromLoc > 0) {
                 //found from, substring from the " FROM " string in sql to end
-                $tableEnd = substr($sql, $fromLoc+6);
+                $tableEnd = substr($sql, $fromLoc + 6);
                 //We know that tablename will be next parameter after from, so
                 //grab the next space after table name.
                 // MFH BUG #14009: Also check to see if there are any carriage returns before the next space so that we don't grab any arbitrary joins or other tables.
-                $carriage_ret = strpos($tableEnd,"\n");
-                $next_space = strpos($tableEnd," " );
+                $carriage_ret = strpos($tableEnd, "\n");
+                $next_space = strpos($tableEnd, " ");
                 if ($carriage_ret < $next_space)
                     $next_space = $carriage_ret;
                 if ($next_space > 0) {
-                    $tbl_name= substr($tableEnd,0, $next_space);
-                    if(empty($tbl_name)){
+                    $tbl_name = substr($tableEnd, 0, $next_space);
+                    if (empty($tbl_name)) {
                         $GLOBALS['log']->debug("Could not find table name sql either, return $module_str. ");
                         $tbl_name = $module_str;
                     }
@@ -932,11 +932,11 @@ 
 
                 //grab the table, to see if it is aliased
                 $aliasTableEnd = trim(substr($tableEnd, $next_space));
-                $alias_space = strpos ($aliasTableEnd, " " );
-                if ($alias_space > 0){
-                    $alias_tbl_name= substr($aliasTableEnd,0, $alias_space);
+                $alias_space = strpos($aliasTableEnd, " ");
+                if ($alias_space > 0) {
+                    $alias_tbl_name = substr($aliasTableEnd, 0, $alias_space);
                     strtolower($alias_tbl_name);
-                    if(empty($alias_tbl_name)
+                    if (empty($alias_tbl_name)
                         || $alias_tbl_name == "where"
                         || $alias_tbl_name == "inner"
                         || $alias_tbl_name == "left"
@@ -948,9 +948,9 @@ 
                     elseif ($alias_tbl_name == "as") {
                             //the next word is the table name
                             $aliasTableEnd = trim(substr($aliasTableEnd, $alias_space));
-                            $alias_space = strpos ($aliasTableEnd, " " );
+                            $alias_space = strpos($aliasTableEnd, " ");
                             if ($alias_space > 0) {
-                                $alias_tbl_name= trim(substr($aliasTableEnd,0, $alias_space));
+                                $alias_tbl_name = trim(substr($aliasTableEnd, 0, $alias_space));
                                 if (!empty($alias_tbl_name))
                                     $tbl_name = $alias_tbl_name;
                             }
@@ -975,7 +975,7 @@ 
 	{
 		$field_array = array();
 
-		if(! isset($result) || empty($result))
+		if (!isset($result) || empty($result))
             return 0;
 
         $i = 0;
@@ -983,7 +983,7 @@ 
             $meta = mssql_fetch_field($result, $i);
             if (!$meta)
                 return 0;
-            if($make_lower_case==true)
+            if ($make_lower_case == true)
                 $meta->name = strtolower($meta->name);
 
             $field_array[] = $meta->name;
@@ -1012,11 +1012,11 @@ 
         $row = mssql_fetch_assoc($result);
         //MSSQL returns a space " " when a varchar column is empty ("") and not null.
         //We need to iterate through the returned row array and strip empty spaces
-        if(!empty($row)){
-            foreach($row as $key => $column) {
+        if (!empty($row)) {
+            foreach ($row as $key => $column) {
                //notice we only strip if one space is returned.  we do not want to strip
                //strings with intentional spaces (" foo ")
-               if (!empty($column) && $column ==" ") {
+               if (!empty($column) && $column == " ") {
                    $row[$key] = '';
                }
             }
@@ -1029,10 +1029,10 @@ 
      */
     public function quote($string)
     {
-        if(is_array($string)) {
+        if (is_array($string)) {
             return $this->arrayQuote($string);
         }
-        return str_replace("'","''", $this->quoteInternal($string));
+        return str_replace("'", "''", $this->quoteInternal($string));
     }
 
     /**
@@ -1070,7 +1070,7 @@ 
             if (!empty($r)) {
                 while ($a = $this->fetchByAssoc($r)) {
                     $row = array_values($a);
-					$tables[]=$row[0];
+					$tables[] = $row[0];
                 }
                 return $tables;
             }
@@ -1085,7 +1085,7 @@ 
     {
         $GLOBALS['log']->debug('MSSQL fetching table list');
 
-        if($this->getDatabase()) {
+        if ($this->getDatabase()) {
             $tables = array();
             $r = $this->query('SELECT TABLE_NAME FROM INFORMATION_SCHEMA.TABLES');
             if (is_resource($r)) {
@@ -1109,7 +1109,7 @@ 
     {
         $GLOBALS['log']->debug('MSSQL about to wakeup FTS');
 
-        if($this->getDatabase()) {
+        if ($this->getDatabase()) {
                 //create wakeup catalog
                 $FTSqry[] = "if not exists(  select * from sys.fulltext_catalogs where name ='wakeup_catalog' )
                 CREATE FULLTEXT CATALOG wakeup_catalog
@@ -1149,7 +1149,7 @@ 
                 $FTSqry[] = 'ALTER FULLTEXT INDEX ON fts_wakeup START FULL POPULATION';
                 $FTSqry[] = 'ALTER FULLTEXT INDEX ON fts_wakeup SET CHANGE_TRACKING AUTO';
 
-                foreach($FTSqry as $q){
+                foreach ($FTSqry as $q) {
                     sleep(3);
                     $this->query($q);
                 }
@@ -1172,12 +1172,12 @@ 
     {
         // convert the parameters array into a comma delimited string
         if (!empty($additional_parameters)) {
-            $additional_parameters_string = ','.implode(',',$additional_parameters);
+            $additional_parameters_string = ','.implode(',', $additional_parameters);
         } else {
             $additional_parameters_string = '';
         }
         $all_parameters = $additional_parameters;
-        if(is_array($string)) {
+        if (is_array($string)) {
             $all_parameters = array_merge($string, $all_parameters);
         } elseif (!is_null($string)) {
             array_unshift($all_parameters, $string);
@@ -1189,22 +1189,22 @@ 
             case 'left':
                 return "LEFT($string$additional_parameters_string)";
             case 'date_format':
-                if(!empty($additional_parameters[0]) && $additional_parameters[0][0] == "'") {
+                if (!empty($additional_parameters[0]) && $additional_parameters[0][0] == "'") {
                     $additional_parameters[0] = trim($additional_parameters[0], "'");
                 }
-                if(!empty($additional_parameters) && isset($this->date_formats[$additional_parameters[0]])) {
+                if (!empty($additional_parameters) && isset($this->date_formats[$additional_parameters[0]])) {
                     $len = $this->date_formats[$additional_parameters[0]];
-                    return "LEFT(CONVERT(varchar($len),". $string . ",120),$len)";
+                    return "LEFT(CONVERT(varchar($len),".$string.",120),$len)";
                 } else {
-                   return "LEFT(CONVERT(varchar(10),". $string . ",120),10)";
+                   return "LEFT(CONVERT(varchar(10),".$string.",120),10)";
                 }
             case 'ifnull':
-                if(empty($additional_parameters_string)) {
+                if (empty($additional_parameters_string)) {
                     $additional_parameters_string = ",''";
                 }
                 return "ISNULL($string$additional_parameters_string)";
             case 'concat':
-                return implode("+",$all_parameters);
+                return implode("+", $all_parameters);
             case 'text2char':
                 return "CAST($string AS varchar(8000))";
             case 'quarter':
@@ -1220,7 +1220,7 @@ 
             case 'add_tz_offset' :
                 $getUserUTCOffset = $GLOBALS['timedate']->getUserUTCOffset();
                 $operation = $getUserUTCOffset < 0 ? '-' : '+';
-                return 'DATEADD(minute, ' . $operation . abs($getUserUTCOffset) . ', ' . $string. ')';
+                return 'DATEADD(minute, '.$operation.abs($getUserUTCOffset).', '.$string.')';
             case 'avg':
                 return "avg($string)";
         }
@@ -1233,9 +1233,9 @@ 
      */
     public function fromConvert($string, $type)
     {
-        switch($type) {
+        switch ($type) {
             case 'datetimecombo':
-            case 'datetime': return substr($string, 0,19);
+            case 'datetime': return substr($string, 0, 19);
             case 'date': return substr($string, 0, 10);
             case 'time': return substr($string, 11);
 		}
@@ -1264,8 +1264,8 @@ 
     public function isTextType($type)
     {
         $type = strtolower($type);
-        if(!isset($this->type_map[$type])) return false;
-        return in_array($this->type_map[$type], array('ntext','text','image', 'nvarchar(max)'));
+        if (!isset($this->type_map[$type])) return false;
+        return in_array($this->type_map[$type], array('ntext', 'text', 'image', 'nvarchar(max)'));
     }
 
     /**
@@ -1275,13 +1275,13 @@ 
     public function emptyValue($type)
     {
         $ctype = $this->getColumnType($type);
-        if($ctype == "datetime") {
+        if ($ctype == "datetime") {
             return $this->convert($this->quoted("1970-01-01 00:00:00"), "datetime");
         }
-        if($ctype == "date") {
+        if ($ctype == "date") {
             return $this->convert($this->quoted("1970-01-01"), "datetime");
         }
-        if($ctype == "time") {
+        if ($ctype == "time") {
             return $this->convert($this->quoted("00:00:00"), "time");
         }
         return parent::emptyValue($type);
@@ -1304,21 +1304,21 @@ 
      */
     protected function alterSQLRep($action, array $def, $ignoreRequired, $tablename)
     {
-        switch($action){
+        switch ($action) {
         case 'add':
-             $f_def=$this->oneColumnSQLRep($def, $ignoreRequired,$tablename,false);
-            return "ADD " . $f_def;
+             $f_def = $this->oneColumnSQLRep($def, $ignoreRequired, $tablename, false);
+            return "ADD ".$f_def;
             break;
         case 'drop':
-            return "DROP COLUMN " . $def['name'];
+            return "DROP COLUMN ".$def['name'];
             break;
         case 'modify':
             //You cannot specify a default value for a column for MSSQL
-            $f_def  = $this->oneColumnSQLRep($def, $ignoreRequired,$tablename, true);
+            $f_def  = $this->oneColumnSQLRep($def, $ignoreRequired, $tablename, true);
             $f_stmt = "ALTER COLUMN ".$f_def['name'].' '.$f_def['colType'].' '.
                         $f_def['required'].' '.$f_def['auto_increment']."\n";
-            if (!empty( $f_def['default']))
-                $f_stmt .= " ALTER TABLE " . $tablename .  " ADD  ". $f_def['default'] . " FOR " . $def['name'];
+            if (!empty($f_def['default']))
+                $f_stmt .= " ALTER TABLE ".$tablename." ADD  ".$f_def['default']." FOR ".$def['name'];
             return $f_stmt;
             break;
         default:
@@ -1334,7 +1334,7 @@ 
      */
     protected function changeColumnSQL($tablename, $fieldDefs, $action, $ignoreRequired = false)
     {
-        $sql=$sql2='';
+        $sql = $sql2 = '';
         $constraints = $this->get_field_default_constraint_name($tablename);
         $columns = array();
         if ($this->isFieldArray($fieldDefs)) {
@@ -1343,41 +1343,41 @@ 
           		//if the column is being modified drop the default value
           		//constraint if it exists. alterSQLRep will add the constraint back
           		if (!empty($constraints[$def['name']])) {
-          			$sql.=" ALTER TABLE " . $tablename . " DROP CONSTRAINT " . $constraints[$def['name']];
+          			$sql .= " ALTER TABLE ".$tablename." DROP CONSTRAINT ".$constraints[$def['name']];
           		}
           		//check to see if we need to drop related indexes before the alter
           		$indices = $this->get_indices($tablename);
-                foreach ( $indices as $index ) {
-                    if ( in_array($def['name'],$index['fields']) ) {
-                        $sql  .= ' ' . $this->add_drop_constraint($tablename,$index,true).' ';
-                        $sql2 .= ' ' . $this->add_drop_constraint($tablename,$index,false).' ';
+                foreach ($indices as $index) {
+                    if (in_array($def['name'], $index['fields'])) {
+                        $sql  .= ' '.$this->add_drop_constraint($tablename, $index, true).' ';
+                        $sql2 .= ' '.$this->add_drop_constraint($tablename, $index, false).' ';
                     }
                 }
 
-          		$columns[] = $this->alterSQLRep($action, $def, $ignoreRequired,$tablename);
+          		$columns[] = $this->alterSQLRep($action, $def, $ignoreRequired, $tablename);
       		}
         }
         else {
             //if the column is being modified drop the default value
       		//constraint if it exists. alterSQLRep will add the constraint back
       		if (!empty($constraints[$fieldDefs['name']])) {
-      			$sql.=" ALTER TABLE " . $tablename . " DROP CONSTRAINT " . $constraints[$fieldDefs['name']];
+      			$sql .= " ALTER TABLE ".$tablename." DROP CONSTRAINT ".$constraints[$fieldDefs['name']];
       		}
       		//check to see if we need to drop related indexes before the alter
             $indices = $this->get_indices($tablename);
-            foreach ( $indices as $index ) {
-                if ( in_array($fieldDefs['name'],$index['fields']) ) {
-                    $sql  .= ' ' . $this->add_drop_constraint($tablename,$index,true).' ';
-                    $sql2 .= ' ' . $this->add_drop_constraint($tablename,$index,false).' ';
+            foreach ($indices as $index) {
+                if (in_array($fieldDefs['name'], $index['fields'])) {
+                    $sql  .= ' '.$this->add_drop_constraint($tablename, $index, true).' ';
+                    $sql2 .= ' '.$this->add_drop_constraint($tablename, $index, false).' ';
                 }
             }
 
 
-          	$columns[] = $this->alterSQLRep($action, $fieldDefs, $ignoreRequired,$tablename);
+          	$columns[] = $this->alterSQLRep($action, $fieldDefs, $ignoreRequired, $tablename);
         }
 
         $columns = implode(", ", $columns);
-        $sql .= " ALTER TABLE $tablename $columns " . $sql2;
+        $sql .= " ALTER TABLE $tablename $columns ".$sql2;
 
         return $sql;
     }
@@ -1392,7 +1392,7 @@ 
      */
     public function setAutoIncrementStart($table, $field_name, $start_value)
     {
-        if($start_value > 1)
+        if ($start_value > 1)
             $start_value -= 1;
 		$this->query("DBCC CHECKIDENT ('$table', RESEED, $start_value) WITH NO_INFOMSGS");
         return true;
@@ -1427,11 +1427,11 @@ 
         $result = $this->query($query);
 
         $indices = array();
-        while (($row=$this->fetchByAssoc($result)) != null) {
+        while (($row = $this->fetchByAssoc($result)) != null) {
             $index_type = 'index';
             if ($row['is_primary_key'] == '1')
                 $index_type = 'primary';
-            elseif ($row['is_unique'] == 1 )
+            elseif ($row['is_unique'] == 1)
                 $index_type = 'unique';
             $name = strtolower($row['index_name']);
             $indices[$name]['name']     = $name;
@@ -1450,36 +1450,36 @@ 
         $result = $this->query("sp_columns $tablename");
 
         $columns = array();
-        while (($row=$this->fetchByAssoc($result)) !=null) {
+        while (($row = $this->fetchByAssoc($result)) != null) {
             $column_name = strtolower($row['COLUMN_NAME']);
-            $columns[$column_name]['name']=$column_name;
-            $columns[$column_name]['type']=strtolower($row['TYPE_NAME']);
-            if ( $row['TYPE_NAME'] == 'decimal' ) {
-                $columns[$column_name]['len']=strtolower($row['PRECISION']);
-                $columns[$column_name]['len'].=','.strtolower($row['SCALE']);
+            $columns[$column_name]['name'] = $column_name;
+            $columns[$column_name]['type'] = strtolower($row['TYPE_NAME']);
+            if ($row['TYPE_NAME'] == 'decimal') {
+                $columns[$column_name]['len'] = strtolower($row['PRECISION']);
+                $columns[$column_name]['len'] .= ','.strtolower($row['SCALE']);
             }
-			elseif ( in_array($row['TYPE_NAME'],array('nchar','nvarchar')) )
-				$columns[$column_name]['len']=strtolower($row['PRECISION']);
-            elseif ( !in_array($row['TYPE_NAME'],array('datetime','text')) )
-                $columns[$column_name]['len']=strtolower($row['LENGTH']);
-            if ( stristr($row['TYPE_NAME'],'identity') ) {
+			elseif (in_array($row['TYPE_NAME'], array('nchar', 'nvarchar')))
+				$columns[$column_name]['len'] = strtolower($row['PRECISION']);
+            elseif (!in_array($row['TYPE_NAME'], array('datetime', 'text')))
+                $columns[$column_name]['len'] = strtolower($row['LENGTH']);
+            if (stristr($row['TYPE_NAME'], 'identity')) {
                 $columns[$column_name]['auto_increment'] = '1';
-                $columns[$column_name]['type']=str_replace(' identity','',strtolower($row['TYPE_NAME']));
+                $columns[$column_name]['type'] = str_replace(' identity', '', strtolower($row['TYPE_NAME']));
             }
 
-            if (!empty($row['IS_NULLABLE']) && $row['IS_NULLABLE'] == 'NO' && (empty($row['KEY']) || !stristr($row['KEY'],'PRI')))
+            if (!empty($row['IS_NULLABLE']) && $row['IS_NULLABLE'] == 'NO' && (empty($row['KEY']) || !stristr($row['KEY'], 'PRI')))
                 $columns[strtolower($row['COLUMN_NAME'])]['required'] = 'true';
 
             $column_def = 1;
-            if ( strtolower($tablename) == 'relationships' ) {
+            if (strtolower($tablename) == 'relationships') {
                 $column_def = $this->getOne("select cdefault from syscolumns where id = object_id('relationships') and name = '$column_name'");
             }
-            if ( $column_def != 0 && ($row['COLUMN_DEF'] != null)) {	// NOTE Not using !empty as an empty string may be a viable default value.
+            if ($column_def != 0 && ($row['COLUMN_DEF'] != null)) {	// NOTE Not using !empty as an empty string may be a viable default value.
                 $matches = array();
-                $row['COLUMN_DEF'] = html_entity_decode($row['COLUMN_DEF'],ENT_QUOTES);
-                if ( preg_match('/\([\(|\'](.*)[\)|\']\)/i',$row['COLUMN_DEF'],$matches) )
+                $row['COLUMN_DEF'] = html_entity_decode($row['COLUMN_DEF'], ENT_QUOTES);
+                if (preg_match('/\([\(|\'](.*)[\)|\']\)/i', $row['COLUMN_DEF'], $matches))
                     $columns[$column_name]['default'] = $matches[1];
-                elseif ( preg_match('/\(N\'(.*)\'\)/i',$row['COLUMN_DEF'],$matches) )
+                elseif (preg_match('/\(N\'(.*)\'\)/i', $row['COLUMN_DEF'], $matches))
                     $columns[$column_name]['default'] = $matches[1];
                 else
                     $columns[$column_name]['default'] = $row['COLUMN_DEF'];
@@ -1494,7 +1494,7 @@ 
      */
     protected function ftsCatalogName()
     {
-        if(isset($this->connectOptions['db_name'])) {
+        if (isset($this->connectOptions['db_name'])) {
             return $this->connectOptions['db_name']."_fts_catalog";
         }
         return 'sugar_fts_catalog';
@@ -1506,11 +1506,11 @@ 
     public function add_drop_constraint($table, $definition, $drop = false)
     {
         $type         = $definition['type'];
-        $fields       = is_array($definition['fields'])?implode(',',$definition['fields']):$definition['fields'];
+        $fields       = is_array($definition['fields']) ? implode(',', $definition['fields']) : $definition['fields'];
         $name         = $definition['name'];
         $sql          = '';
 
-        switch ($type){
+        switch ($type) {
         // generic indices
         case 'index':
         case 'alternate_key':
@@ -1548,13 +1548,13 @@ 
             if ($this->full_text_indexing_enabled() && $drop) {
                 $sql = "DROP FULLTEXT INDEX ON {$table}";
             } elseif ($this->full_text_indexing_enabled()) {
-                $catalog_name=$this->ftsCatalogName();
-                if ( isset($definition['catalog_name']) && $definition['catalog_name'] != 'default')
+                $catalog_name = $this->ftsCatalogName();
+                if (isset($definition['catalog_name']) && $definition['catalog_name'] != 'default')
                     $catalog_name = $definition['catalog_name'];
 
                 $language = "Language 1033";
                 if (isset($definition['language']) && !empty($definition['language']))
-                    $language = "Language " . $definition['language'];
+                    $language = "Language ".$definition['language'];
 
                 $key_index = $definition['key_index'];
 
@@ -1562,7 +1562,7 @@ 
                 if (isset($definition['change_tracking']) && !empty($definition['change_tracking']))
                     $change_tracking = $definition['change_tracking'];
 
-                $sql = " CREATE FULLTEXT INDEX ON $table ($fields $language) KEY INDEX $key_index ON $catalog_name WITH CHANGE_TRACKING $change_tracking" ;
+                $sql = " CREATE FULLTEXT INDEX ON $table ($fields $language) KEY INDEX $key_index ON $catalog_name WITH CHANGE_TRACKING $change_tracking";
             }
             break;
         }
@@ -1586,11 +1586,11 @@ 
     protected function full_text_indexing_enabled($dbname = null)
     {
         // check to see if we already have install setting in session
-    	if(!isset($_SESSION['IsFulltextInstalled']))
+    	if (!isset($_SESSION['IsFulltextInstalled']))
             $_SESSION['IsFulltextInstalled'] = $this->full_text_indexing_installed();
 
         // check to see if FTS Indexing service is installed
-        if(empty($_SESSION['IsFulltextInstalled']))
+        if (empty($_SESSION['IsFulltextInstalled']))
             return false;
 
         // grab the dbname if it was not passed through
@@ -1640,7 +1640,7 @@ 
     {
         static $results = array();
 
-        if ( empty($column) && isset($results[$table]) )
+        if (empty($column) && isset($results[$table]))
             return $results[$table];
 
         $query = <<<EOQ
@@ -1654,17 +1654,17 @@ 
             on s.schema_id = o.schema_id
     where o.name = '$table'
 EOQ;
-        if ( !empty($column) )
+        if (!empty($column))
             $query .= " and c.name = '$column'";
         $res = $this->query($query);
-        if ( !empty($column) ) {
+        if (!empty($column)) {
             $row = $this->fetchByAssoc($res);
             if (!empty($row))
                 return $row['ctrt'];
         }
         else {
             $returnResult = array();
-            while ( $row = $this->fetchByAssoc($res) )
+            while ($row = $this->fetchByAssoc($res))
                 $returnResult[$row['dtrt']] = $row['ctrt'];
             $results[$table] = $returnResult;
             return $returnResult;
@@ -1678,14 +1678,14 @@ 
      */
     public function massageFieldDef(&$fieldDef, $tablename)
     {
-        parent::massageFieldDef($fieldDef,$tablename);
+        parent::massageFieldDef($fieldDef, $tablename);
 
         if ($fieldDef['type'] == 'int')
             $fieldDef['len'] = '4';
 
-        if(empty($fieldDef['len']))
+        if (empty($fieldDef['len']))
         {
-            switch($fieldDef['type']) {
+            switch ($fieldDef['type']) {
                 case 'bit'      :
                 case 'bool'     : $fieldDef['len'] = '1'; break;
                 case 'smallint' : $fieldDef['len'] = '2'; break;
@@ -1695,22 +1695,22 @@ 
                                   $fieldDef['len'] = $this->isTextType($fieldDef['dbType']) ? 'max' : '255';
                                   break;
                 case 'image'    : $fieldDef['len'] = '2147483647'; break;
-                case 'ntext'    : $fieldDef['len'] = '2147483646'; break;   // Note: this is from legacy code, don't know if this is correct
+                case 'ntext'    : $fieldDef['len'] = '2147483646'; break; // Note: this is from legacy code, don't know if this is correct
             }
         }
-        if($fieldDef['type'] == 'decimal'
+        if ($fieldDef['type'] == 'decimal'
            && empty($fieldDef['precision'])
            && !strpos($fieldDef['len'], ','))
         {
              $fieldDef['len'] .= ',0'; // Adding 0 precision if it is not specified
         }
 
-        if(empty($fieldDef['default'])
-            && in_array($fieldDef['type'],array('bit','bool')))
+        if (empty($fieldDef['default'])
+            && in_array($fieldDef['type'], array('bit', 'bool')))
         {
             $fieldDef['default'] = '0';
         }
-		if (isset($fieldDef['required']) && $fieldDef['required'] && !isset($fieldDef['default']) )
+		if (isset($fieldDef['required']) && $fieldDef['required'] && !isset($fieldDef['default']))
 			$fieldDef['default'] = '';
 //        if ($fieldDef['type'] == 'bit' && empty($fieldDef['len']) )
 //            $fieldDef['len'] = '1';
@@ -1741,13 +1741,13 @@ 
     protected function oneColumnSQLRep($fieldDef, $ignoreRequired = false, $table = '', $return_as_array = false)
     {
     	//Bug 25814
-		if(isset($fieldDef['name'])){
+		if (isset($fieldDef['name'])) {
 		    $colType = $this->getFieldType($fieldDef);
-        	if(stristr($this->getFieldType($fieldDef), 'decimal') && isset($fieldDef['len'])){
-				$fieldDef['len'] = min($fieldDef['len'],38);
+        	if (stristr($this->getFieldType($fieldDef), 'decimal') && isset($fieldDef['len'])) {
+				$fieldDef['len'] = min($fieldDef['len'], 38);
 			}
 		    //bug: 39690 float(8) is interpreted as real and this generates a diff when doing repair
-			if(stristr($colType, 'float') && isset($fieldDef['len']) && $fieldDef['len'] == 8){
+			if (stristr($colType, 'float') && isset($fieldDef['len']) && $fieldDef['len'] == 8) {
 				unset($fieldDef['len']);
 			}
 		}
@@ -1756,10 +1756,10 @@ 
 		$ref = parent::oneColumnSQLRep($fieldDef, $ignoreRequired, $table, true);
 
 		// Bug 24307 - Don't add precision for float fields.
-		if ( stristr($ref['colType'],'float') )
-			$ref['colType'] = preg_replace('/(,\d+)/','',$ref['colType']);
+		if (stristr($ref['colType'], 'float'))
+			$ref['colType'] = preg_replace('/(,\d+)/', '', $ref['colType']);
 
-        if ( $return_as_array )
+        if ($return_as_array)
             return $ref;
         else
             return "{$ref['name']} {$ref['colType']} {$ref['default']} {$ref['required']} {$ref['auto_increment']}";
@@ -1774,10 +1774,10 @@ 
     public function save_audit_records(SugarBean $bean, $changes)
 	{
 		//Bug 25078 fixed by Martin Hu: sqlserver haven't 'date' type, trim extra "00:00:00"
-		if($changes['data_type'] == 'date'){
-			$changes['before'] = str_replace(' 00:00:00','',$changes['before']);
+		if ($changes['data_type'] == 'date') {
+			$changes['before'] = str_replace(' 00:00:00', '', $changes['before']);
 		}
-		parent::save_audit_records($bean,$changes);
+		parent::save_audit_records($bean, $changes);
 	}
 
     /**
@@ -1788,7 +1788,7 @@ 
     public function disconnect()
     {
     	$GLOBALS['log']->debug('Calling Mssql::disconnect()');
-        if(!empty($this->database)){
+        if (!empty($this->database)) {
             $this->freeResult();
             mssql_close($this->database);
             $this->database = null;
@@ -1800,7 +1800,7 @@ 
      */
     protected function freeDbResult($dbResult)
     {
-        if(!empty($dbResult))
+        if (!empty($dbResult))
             mssql_free_result($dbResult);
     }
 
@@ -1811,11 +1811,11 @@ 
     public function lastDbError()
     {
         $sqlmsg = mssql_get_last_message();
-        if(empty($sqlmsg)) return false;
+        if (empty($sqlmsg)) return false;
         global $app_strings;
         if (empty($app_strings)
 		    or !isset($app_strings['ERR_MSSQL_DB_CONTEXT'])
-			or !isset($app_strings['ERR_MSSQL_WARNING']) ) {
+			or !isset($app_strings['ERR_MSSQL_WARNING'])) {
         //ignore the message from sql-server if $app_strings array is empty. This will happen
         //only if connection if made before language is set.
 		    return false;
@@ -1824,7 +1824,7 @@ 
         $sqlpos = strpos($sqlmsg, 'Changed database context to');
         $sqlpos2 = strpos($sqlmsg, 'Warning:');
         $sqlpos3 = strpos($sqlmsg, 'Checking identity information:');
-        if ( $sqlpos !== false || $sqlpos2 !== false || $sqlpos3 !== false ) {
+        if ($sqlpos !== false || $sqlpos2 !== false || $sqlpos3 !== false) {
             return false;
         } else {
         	global $app_strings;
@@ -1832,18 +1832,18 @@ 
             if (empty($app_strings) or !isset($app_strings['ERR_MSSQL_DB_CONTEXT'])) {
                 //ignore the message from sql-server if $app_strings array is empty. This will happen
                 //only if connection if made before languge is set.
-                $GLOBALS['log']->debug("Ignoring this database message: " . $sqlmsg);
+                $GLOBALS['log']->debug("Ignoring this database message: ".$sqlmsg);
                 return false;
             }
             else {
                 $sqlpos = strpos($sqlmsg, $app_strings['ERR_MSSQL_DB_CONTEXT']);
-                if ( $sqlpos !== false )
+                if ($sqlpos !== false)
                     return false;
             }
         }
 
-        if ( strlen($sqlmsg) > 2 ) {
-        	return "SQL Server error: " . $sqlmsg;
+        if (strlen($sqlmsg) > 2) {
+        	return "SQL Server error: ".$sqlmsg;
         }
 
         return false;
@@ -1864,7 +1864,7 @@ 
      */
     public function validateQuery($query)
     {
-        if(!$this->isSelect($query)) {
+        if (!$this->isSelect($query)) {
             return false;
         }
         $this->query("SET SHOWPLAN_TEXT ON");
@@ -1888,7 +1888,7 @@ 
 
         // Flag if there are odd number of single quotes, just continue without trying to append N
         if ((substr_count($sql, "'") & 1)) {
-            $GLOBALS['log']->error("SQL statement[" . $sql . "] has odd number of single quotes.");
+            $GLOBALS['log']->error("SQL statement[".$sql."] has odd number of single quotes.");
             return $sql;
         }
 
@@ -1918,7 +1918,7 @@ 
                 // We are assuming that all nvarchar columns are no more than 200 characters in length
                 // One problem we face is the image column type in reports which cannot accept nvarchar data
                 if (!empty($value) && !is_numeric(trim(str_replace(array("'", ","), "", $value))) && !preg_match('/^\'[\,]\'$/', $value)) {
-                    $replace[$value] = 'N' . trim($value, "N");
+                    $replace[$value] = 'N'.trim($value, "N");
                 }
             }
         }
@@ -1929,7 +1929,7 @@ 
         if (!empty($pairs))
             $sql = str_replace(array_keys($pairs), $pairs, $sql);
 
-        if(strpos($sql, "<@#@#@PAIR@#@#@>"))
+        if (strpos($sql, "<@#@#@PAIR@#@#@>"))
             $sql = str_replace(array('<@#@#@PAIR@#@#@>'), array("''"), $sql);
 
         return $sql;
@@ -1956,22 +1956,22 @@ 
     public function getFulltextQuery($field, $terms, $must_terms = array(), $exclude_terms = array())
     {
         $condition = $or_condition = array();
-        foreach($must_terms as $term) {
+        foreach ($must_terms as $term) {
             $condition[] = $this->quoteTerm($term);
         }
 
-        foreach($terms as $term) {
+        foreach ($terms as $term) {
             $or_condition[] = $this->quoteTerm($term);
         }
 
-        if(!empty($or_condition)) {
+        if (!empty($or_condition)) {
             $condition[] = "(".join(" | ", $or_condition).")";
         }
 
-        foreach($exclude_terms as $term) {
+        foreach ($exclude_terms as $term) {
             $condition[] = " NOT ".$this->quoteTerm($term);
         }
-        $condition = $this->quoted(join(" AND ",$condition));
+        $condition = $this->quoted(join(" AND ", $condition));
         return "CONTAINS($field, $condition)";
     }
 
@@ -2060,7 +2060,7 @@ 
     public function isDatabaseNameValid($name)
     {
         // No funny chars, does not begin with number
-        return preg_match('/^[0-9#@]+|[\"\'\*\/\\?\:\\<\>\-\ \&\!\(\)\[\]\{\}\;\,\.\`\~\|\\\\]+/', $name)==0;
+        return preg_match('/^[0-9#@]+|[\"\'\*\/\\?\:\\<\>\-\ \&\!\(\)\[\]\{\}\;\,\.\`\~\|\\\\]+/', $name) == 0;
     }
 
     public function installConfig()

include/database/DBManager.php

Spacing +314 added lines, -314 removed lines

@@ -1,5 +1,5 @@ 
 <?php
-if(!defined('sugarEntry') || !sugarEntry) die('Not A Valid Entry Point');
+if (!defined('sugarEntry') || !sugarEntry) die('Not A Valid Entry Point');
 /*********************************************************************************
  * SugarCRM Community Edition is a customer relationship management program developed by
  * SugarCRM, Inc. Copyright (C) 2004-2013 SugarCRM Inc.
@@ -314,7 +314,7 @@ 
 		}
 
 		$dberror = $this->lastDbError();
-		if($dberror === false) {
+		if ($dberror === false) {
     		$this->last_error = false;
 	    	return false;
 		}
@@ -332,16 +332,16 @@ 
 	 */
 	protected function registerError($userMessage, $message, $dieOnError = false)
 	{
-		if(!empty($message)) {
-			if(!empty($userMessage)) {
+		if (!empty($message)) {
+			if (!empty($userMessage)) {
 				$message = "$userMessage: $message";
 			}
-			if(empty($message)) {
+			if (empty($message)) {
 			    $message = "Database error";
 			}
 			$this->log->fatal($message);
 			if ($dieOnError || $this->dieOnError) {
-				if(isset($GLOBALS['app_strings']['ERR_DB_FAIL'])) {
+				if (isset($GLOBALS['app_strings']['ERR_DB_FAIL'])) {
 					sugar_die($GLOBALS['app_strings']['ERR_DB_FAIL']);
 				} else {
 					sugar_die("Database error. Please check suitecrm.log for details.");
@@ -379,8 +379,8 @@ 
 		$slow_query_time_msec = isset($sugar_config['slow_query_time_msec'])
 			? $sugar_config['slow_query_time_msec'] : 5000;
 
-		if($do_the_dump) {
-			if($slow_query_time_msec < ($this->query_time * 1000)) {
+		if ($do_the_dump) {
+			if ($slow_query_time_msec < ($this->query_time * 1000)) {
 				// Then log both the query and the query time
 				$this->log->fatal('Slow Query (time:'.$this->query_time."\n".$query);
 				return true;
@@ -412,20 +412,20 @@ 
 		$indices = $GLOBALS['dictionary'][$object_name]['indices'];
 
 	if (empty($indices)) {
-		foreach ( $GLOBALS['dictionary'] as $current ) {
-			if ($current['table'] == $table){
+		foreach ($GLOBALS['dictionary'] as $current) {
+			if ($current['table'] == $table) {
 				$indices = $current['indices'];
 				break;
 			}
 		}
 	}
 	if (empty($indices)) {
-		$this->log->warn('CHECK QUERY: Could not find index definitions for table ' . $table);
+		$this->log->warn('CHECK QUERY: Could not find index definitions for table '.$table);
 		return false;
 	}
 	if (!empty($match[2][0])) {
 		$orderBys = explode(' ', $match[2][0]);
-		foreach ($orderBys as $orderBy){
+		foreach ($orderBys as $orderBy) {
 			$orderBy = trim($orderBy);
 			if (empty($orderBy))
 				continue;
@@ -433,7 +433,7 @@ 
 			if ($orderBy == 'asc' || $orderBy == 'desc')
 				continue;
 
-			$orderBy = str_replace(array($table . '.', ','), '', $orderBy);
+			$orderBy = str_replace(array($table.'.', ','), '', $orderBy);
 
 			foreach ($indices as $index)
 				if (empty($index['db']) || $index['db'] == $this->dbType)
@@ -441,11 +441,11 @@ 
 						if ($field == $orderBy)
 							return true;
 
-			$warning = 'Missing Index For Order By Table: ' . $table . ' Order By:' . $orderBy ;
+			$warning = 'Missing Index For Order By Table: '.$table.' Order By:'.$orderBy;
 			if (!empty($GLOBALS['sugar_config']['dump_slow_queries']))
-				$this->log->fatal('CHECK QUERY:' .$warning);
+				$this->log->fatal('CHECK QUERY:'.$warning);
 			else
-				$this->log->warn('CHECK QUERY:' .$warning);
+				$this->log->warn('CHECK QUERY:'.$warning);
 		}
 	}
 	return false;
@@ -491,9 +491,9 @@ 
 	public function insert(SugarBean $bean)
 	{
 		$sql = $this->insertSQL($bean);
-		$tablename =  $bean->getTableName();
+		$tablename = $bean->getTableName();
 		$msg = "Error inserting into table: $tablename:";
-		return $this->query($sql,true,$msg);
+		return $this->query($sql, true, $msg);
 	}
 
 	/**
@@ -512,13 +512,13 @@ 
 		{
 			if (isset($fieldDef['source']) && $fieldDef['source'] != 'db')  continue;
 			//custom fields handle there save seperatley
-			if(!empty($field_map) && !empty($field_map[$field]['custom_type'])) continue;
+			if (!empty($field_map) && !empty($field_map[$field]['custom_type'])) continue;
 
-			if(isset($data[$field])) {
+			if (isset($data[$field])) {
 				// clean the incoming value..
 				$val = from_html($data[$field]);
 			} else {
-				if(isset($fieldDef['default']) && strlen($fieldDef['default']) > 0) {
+				if (isset($fieldDef['default']) && strlen($fieldDef['default']) > 0) {
 					$val = $fieldDef['default'];
 				} else {
 					$val = null;
@@ -528,26 +528,26 @@ 
 			//handle auto increment values here - we may have to do something like nextval for oracle
 			if (!empty($fieldDef['auto_increment'])) {
 				$auto = $this->getAutoIncrementSQL($table, $fieldDef['name']);
-				if(!empty($auto)) {
+				if (!empty($auto)) {
 					$values[$field] = $auto;
 				}
 			} elseif ($fieldDef['name'] == 'deleted') {
 				$values['deleted'] = (int)$val;
 			} else {
 				// need to do some thing about types of values
-				if(!is_null($val) || !empty($fieldDef['required'])) {
+				if (!is_null($val) || !empty($fieldDef['required'])) {
 					$values[$field] = $this->massageValue($val, $fieldDef);
 				}
 			}
 		}
 
 		if (empty($values))
-			return $execute?true:''; // no columns set
+			return $execute ? true : ''; // no columns set
 
 		// get the entire sql
 		$query = "INSERT INTO $table (".implode(",", array_keys($values)).")
 					VALUES (".implode(",", $values).")";
-		return $execute?$this->query($query):$query;
+		return $execute ? $this->query($query) : $query;
 	}
 
     /**
@@ -565,7 +565,7 @@ 
 		$sql = $this->updateSQL($bean, $where);
 		$tablename = $bean->getTableName();
 		$msg = "Error updating table: $tablename:";
-		return $this->query($sql,true,$msg);
+		return $this->query($sql, true, $msg);
 	}
 
     /**
@@ -581,8 +581,8 @@ 
 	{
 		$sql = $this->deleteSQL($bean, $where);
 		$tableName = $bean->getTableName();
-		$msg = "Error deleting from table: ".$tableName. ":";
-		return $this->query($sql,true,$msg);
+		$msg = "Error deleting from table: ".$tableName.":";
+		return $this->query($sql, true, $msg);
 	}
 
 	/**
@@ -599,8 +599,8 @@ 
 	{
 		$sql = $this->retrieveSQL($bean, $where);
 		$tableName = $bean->getTableName();
-		$msg = "Error retriving values from table:".$tableName. ":";
-		return $this->query($sql,true,$msg);
+		$msg = "Error retriving values from table:".$tableName.":";
+		return $this->query($sql, true, $msg);
 	}
 
 	/**
@@ -623,7 +623,7 @@ 
 	{
 		$sql = $this->retrieveViewSQL($beans, $cols, $where);
 		$msg = "Error retriving values from View Collection:";
-		return $this->query($sql,true,$msg);
+		return $this->query($sql, true, $msg);
 	}
 
 
@@ -638,8 +638,8 @@ 
 		$sql = $this->createTableSQL($bean);
 		$tablename = $bean->getTableName();
 		$msg = "Error creating table: $tablename:";
-		$this->query($sql,true,$msg);
-		if(!$this->supports("inline_keys")) {
+		$this->query($sql, true, $msg);
+		if (!$this->supports("inline_keys")) {
 		// handle constraints and indices
 			$indicesArr = $this->createConstraintSql($bean);
 			if (count($indicesArr) > 0)
@@ -676,9 +676,9 @@ 
 			$res = true;
 			if ($sql) {
 				$msg = "Error creating table: $tablename";
-				$res = ($res and $this->query($sql,true,$msg));
+				$res = ($res and $this->query($sql, true, $msg));
 			}
-			if(!$this->supports("inline_keys")) {
+			if (!$this->supports("inline_keys")) {
 				// handle constraints and indices
 				$indicesArr = $this->getConstraintSql($indices, $tablename);
 				if (count($indicesArr) > 0)
@@ -705,20 +705,20 @@ 
 
 		//Clean the indexes to prevent duplicate definitions
 		$new_index = array();
-		foreach($indices as $ind_def){
+		foreach ($indices as $ind_def) {
 			$new_index[$ind_def['name']] = $ind_def;
 		}
 		//jc: added this for beans that do not actually have a table, namely
 		//ForecastOpportunities
-		if($tablename == 'does_not_exist' || $tablename == '')
+		if ($tablename == 'does_not_exist' || $tablename == '')
 			return '';
 
 		global $dictionary;
-		$engine=null;
-		if (isset($dictionary[$bean->getObjectName()]['engine']) && !empty($dictionary[$bean->getObjectName()]['engine']) )
+		$engine = null;
+		if (isset($dictionary[$bean->getObjectName()]['engine']) && !empty($dictionary[$bean->getObjectName()]['engine']))
 			$engine = $dictionary[$bean->getObjectName()]['engine'];
 
-		return $this->repairTableParams($tablename, $fielddefs,$new_index,$execute,$engine);
+		return $this->repairTableParams($tablename, $fielddefs, $new_index, $execute, $engine);
 	}
 
 	/**
@@ -730,12 +730,12 @@ 
 	protected function isNullable($vardef)
 	{
 
-		if(isset($vardef['isnull']) && (strtolower($vardef['isnull']) == 'false' || $vardef['isnull'] === false)
+		if (isset($vardef['isnull']) && (strtolower($vardef['isnull']) == 'false' || $vardef['isnull'] === false)
 			&& !empty($vardef['required'])) {
 				/* required + is_null=false => not null */
 			return false;
 		}
-		if(empty($vardef['auto_increment']) && (empty($vardef['type']) || $vardef['type'] != 'id')
+		if (empty($vardef['auto_increment']) && (empty($vardef['type']) || $vardef['type'] != 'id')
 					&& (empty($vardef['dbType']) || $vardef['dbType'] != 'id')
 					&& (empty($vardef['name']) || ($vardef['name'] != 'id' && $vardef['name'] != 'deleted'))
 		) {
@@ -756,7 +756,7 @@ 
      * @todo: refactor engine param to be more generic
      * @return string
      */
-	public function repairTableParams($tablename, $fielddefs,  $indices, $execute = true, $engine = null)
+	public function repairTableParams($tablename, $fielddefs, $indices, $execute = true, $engine = null)
 	{
 		//jc: had a bug when running the repair if the tablename is blank the repair will
 		//fail when it tries to create a repair table
@@ -766,13 +766,13 @@ 
 		//if the table does not exist create it and we are done
 		$sql = "/* Table : $tablename */\n";
 		if (!$this->tableExists($tablename)) {
-			$createtablesql = $this->createTableSQLParams($tablename,$fielddefs,$indices,$engine);
-			if($execute && $createtablesql){
-				$this->createTableParams($tablename,$fielddefs,$indices,$engine);
+			$createtablesql = $this->createTableSQLParams($tablename, $fielddefs, $indices, $engine);
+			if ($execute && $createtablesql) {
+				$this->createTableParams($tablename, $fielddefs, $indices, $engine);
 			}
 
 			$sql .= "/* MISSING TABLE: {$tablename} */\n";
-			$sql .= $createtablesql . "\n";
+			$sql .= $createtablesql."\n";
 			return $sql;
 		}
 
@@ -782,7 +782,7 @@ 
 		$take_action = false;
 
 		// do column comparisons
-		$sql .=	"/*COLUMNS*/\n";
+		$sql .= "/*COLUMNS*/\n";
 		foreach ($fielddefs as $name => $value) {
 			if (isset($value['source']) && $value['source'] != 'db')
 				continue;
@@ -801,9 +801,9 @@ 
 
 			$name = strtolower($value['name']);
 			// add or fix the field defs per what the DB is expected to give us back
-			$this->massageFieldDef($value,$tablename);
+			$this->massageFieldDef($value, $tablename);
 
-			$ignorerequired=false;
+			$ignorerequired = false;
 
 			//Do not track requiredness in the DB, auto_increment, ID,
 			// and deleted fields are always required in the DB, so don't force those
@@ -819,32 +819,32 @@ 
 				$value['required'] = $compareFieldDefs[$name]['required'];
 			}
 
-			if ( !isset($compareFieldDefs[$name]) ) {
+			if (!isset($compareFieldDefs[$name])) {
 				// ok we need this field lets create it
-				$sql .=	"/*MISSING IN DATABASE - $name -  ROW*/\n";
-				$sql .= $this->addColumnSQL($tablename, $value) .  "\n";
+				$sql .= "/*MISSING IN DATABASE - $name -  ROW*/\n";
+				$sql .= $this->addColumnSQL($tablename, $value)."\n";
 				if ($execute)
 					$this->addColumn($tablename, $value);
 				$take_action = true;
-			} elseif ( !$this->compareVarDefs($compareFieldDefs[$name],$value)) {
+			} elseif (!$this->compareVarDefs($compareFieldDefs[$name], $value)) {
 				//fields are different lets alter it
-				$sql .=	"/*MISMATCH WITH DATABASE - $name -  ROW ";
-				foreach($compareFieldDefs[$name] as $rKey => $rValue) {
-					$sql .=	"[$rKey] => '$rValue'  ";
+				$sql .= "/*MISMATCH WITH DATABASE - $name -  ROW ";
+				foreach ($compareFieldDefs[$name] as $rKey => $rValue) {
+					$sql .= "[$rKey] => '$rValue'  ";
 				}
-				$sql .=	"*/\n";
-				$sql .=	"/* VARDEF - $name -  ROW";
-				foreach($value as $rKey => $rValue) {
-					$sql .=	"[$rKey] => '$rValue'  ";
+				$sql .= "*/\n";
+				$sql .= "/* VARDEF - $name -  ROW";
+				foreach ($value as $rKey => $rValue) {
+					$sql .= "[$rKey] => '$rValue'  ";
 				}
-				$sql .=	"*/\n";
+				$sql .= "*/\n";
 
 				//jc: oracle will complain if you try to execute a statement that sets a column to (not) null
 				//when it is already (not) null
-				if ( isset($value['isnull']) && isset($compareFieldDefs[$name]['isnull']) &&
+				if (isset($value['isnull']) && isset($compareFieldDefs[$name]['isnull']) &&
 					$value['isnull'] === $compareFieldDefs[$name]['isnull']) {
 					unset($value['required']);
-					$ignorerequired=true;
+					$ignorerequired = true;
 				}
 
 				//dwheeler: Once a column has been defined as null, we cannot try to force it back to !null
@@ -853,12 +853,12 @@ 
 				{
 					$ignorerequired = true;
 				}
-				$altersql = $this->alterColumnSQL($tablename, $value,$ignorerequired);
-				if(is_array($altersql)) {
+				$altersql = $this->alterColumnSQL($tablename, $value, $ignorerequired);
+				if (is_array($altersql)) {
 					$altersql = join("\n", $altersql);
 				}
-				$sql .= $altersql .  "\n";
-				if($execute){
+				$sql .= $altersql."\n";
+				if ($execute) {
 					$this->alterColumn($tablename, $value, $ignorerequired);
 				}
 				$take_action = true;
@@ -866,13 +866,13 @@ 
 		}
 
 		// do index comparisons
-		$sql .=	"/* INDEXES */\n";
+		$sql .= "/* INDEXES */\n";
 		$correctedIndexs = array();
 
         $compareIndices_case_insensitive = array();
 
 		// do indices comparisons case-insensitive
-		foreach($compareIndices as $k => $value){
+		foreach ($compareIndices as $k => $value) {
 			$value['name'] = strtolower($value['name']);
 			$compareIndices_case_insensitive[strtolower($k)] = $value;
 		}
@@ -899,59 +899,59 @@ 
 				continue;
 
 			//database helpers do not know how to handle full text indices
-			if ($value['type']=='fulltext')
+			if ($value['type'] == 'fulltext')
 				continue;
 
-			if ( in_array($value['type'],array('alternate_key','foreign')) )
+			if (in_array($value['type'], array('alternate_key', 'foreign')))
 				$value['type'] = 'index';
 
-			if ( !isset($compareIndices[$name]) ) {
+			if (!isset($compareIndices[$name])) {
 				//First check if an index exists that doesn't match our name, if so, try to rename it
 				$found = false;
 				foreach ($compareIndices as $ex_name => $ex_value) {
-					if($this->compareVarDefs($ex_value, $value, true)) {
+					if ($this->compareVarDefs($ex_value, $value, true)) {
 						$found = $ex_name;
 						break;
 					}
 				}
 				if ($found) {
-					$sql .=	 "/*MISSNAMED INDEX IN DATABASE - $name - $ex_name */\n";
+					$sql .= "/*MISSNAMED INDEX IN DATABASE - $name - $ex_name */\n";
 					$rename = $this->renameIndexDefs($ex_value, $value, $tablename);
-					if($execute) {
+					if ($execute) {
 						$this->query($rename, true, "Cannot rename index");
 					}
-					$sql .= is_array($rename)?join("\n", $rename). "\n":$rename."\n";
+					$sql .= is_array($rename) ? join("\n", $rename)."\n" : $rename."\n";
 
 				} else {
 					// ok we need this field lets create it
-					$sql .=	 "/*MISSING INDEX IN DATABASE - $name -{$value['type']}  ROW */\n";
-					$sql .= $this->addIndexes($tablename,array($value), $execute) .  "\n";
+					$sql .= "/*MISSING INDEX IN DATABASE - $name -{$value['type']}  ROW */\n";
+					$sql .= $this->addIndexes($tablename, array($value), $execute)."\n";
 				}
 				$take_action = true;
 				$correctedIndexs[$name] = true;
-			} elseif ( !$this->compareVarDefs($compareIndices[$name],$value) ) {
+			} elseif (!$this->compareVarDefs($compareIndices[$name], $value)) {
 				// fields are different lets alter it
-				$sql .=	"/*INDEX MISMATCH WITH DATABASE - $name -  ROW ";
+				$sql .= "/*INDEX MISMATCH WITH DATABASE - $name -  ROW ";
 				foreach ($compareIndices[$name] as $n1 => $t1) {
-					$sql .=	 "<$n1>";
-					if ( $n1 == 'fields' )
-						foreach($t1 as $rKey => $rValue)
+					$sql .= "<$n1>";
+					if ($n1 == 'fields')
+						foreach ($t1 as $rKey => $rValue)
 							$sql .= "[$rKey] => '$rValue'  ";
 					else
 						$sql .= " $t1 ";
 				}
-				$sql .=	"*/\n";
-				$sql .=	"/* VARDEF - $name -  ROW";
+				$sql .= "*/\n";
+				$sql .= "/* VARDEF - $name -  ROW";
 				foreach ($value as $n1 => $t1) {
-					$sql .=	"<$n1>";
-					if ( $n1 == 'fields' )
+					$sql .= "<$n1>";
+					if ($n1 == 'fields')
 						foreach ($t1 as $rKey => $rValue)
-							$sql .=	"[$rKey] => '$rValue'  ";
+							$sql .= "[$rKey] => '$rValue'  ";
 					else
 						$sql .= " $t1 ";
 				}
-				$sql .=	"*/\n";
-				$sql .= $this->modifyIndexes($tablename,array($value), $execute) .  "\n";
+				$sql .= "*/\n";
+				$sql .= $this->modifyIndexes($tablename, array($value), $execute)."\n";
 				$take_action = true;
 				$correctedIndexs[$name] = true;
 			}
@@ -970,7 +970,7 @@ 
      */
 	public function compareVarDefs($fielddef1, $fielddef2, $ignoreName = false)
 	{
-		foreach ( $fielddef1 as $key => $value ) {
+		foreach ($fielddef1 as $key => $value) {
 			if ($key == 'name' && $ignoreName)
 				continue;
             if (isset($fielddef2[$key]))
@@ -984,7 +984,7 @@ 
                 }
                 else
                 {
-                    if (array_map('strtolower', $fielddef1[$key]) == array_map('strtolower',$fielddef2[$key]))
+                    if (array_map('strtolower', $fielddef1[$key]) == array_map('strtolower', $fielddef2[$key]))
                     {
                         continue;
                     }
@@ -1033,9 +1033,9 @@ 
 				}
 				else {
 					$returnArray['msg'] = 'match';
-					foreach($row1 as $key => $value){
+					foreach ($row1 as $key => $value) {
 						//ignore keys when checking we will check them when we do the index check
-						if( !isset($ignore_filter[$key]) && (!isset($row2[$key]) || $row1[$key] !== $row2[$key])){
+						if (!isset($ignore_filter[$key]) && (!isset($row2[$key]) || $row1[$key] !== $row2[$key])) {
 							$returnArray['msg'] = 'no_match';
 						}
 					}
@@ -1111,7 +1111,7 @@ 
 		$sql = $this->createIndexSQL($bean, $fieldDefs, $name, $unique);
 		$tablename = $bean->getTableName();
 		$msg = "Error creating index $name on table: $tablename:";
-		return $this->query($sql,true,$msg);
+		return $this->query($sql, true, $msg);
 	}
 
 	/**
@@ -1128,14 +1128,14 @@ 
 		$columns = array();
 
 		foreach ($indices as $index) {
-			if(!empty($index['db']) && $index['db'] != $this->dbType)
+			if (!empty($index['db']) && $index['db'] != $this->dbType)
 				continue;
 			if (isset($index['source']) && $index['source'] != 'db')
 			continue;
 
 			$sql = $this->add_drop_constraint($table, $index);
 
-			if(!empty($sql)) {
+			if (!empty($sql)) {
 				$columns[] = $sql;
 			}
 		}
@@ -1155,11 +1155,11 @@ 
 	{
 		$alters = $this->getConstraintSql($indexes, $tablename);
 		if ($execute) {
-			foreach($alters as $sql) {
+			foreach ($alters as $sql) {
 				$this->query($sql, true, "Error adding index: ");
 			}
 		}
-		if(!empty($alters)) {
+		if (!empty($alters)) {
 			$sql = join(";\n", $alters).";\n";
 		} else {
 			$sql = '';
@@ -1179,17 +1179,17 @@ 
 	{
 		$sqls = array();
 		foreach ($indexes as $index) {
-			$name =$index['name'];
-			$sqls[$name] = $this->add_drop_constraint($tablename,$index,true);
+			$name = $index['name'];
+			$sqls[$name] = $this->add_drop_constraint($tablename, $index, true);
 		}
 		if (!empty($sqls) && $execute) {
-			foreach($sqls as $name => $sql) {
+			foreach ($sqls as $name => $sql) {
 				unset(self::$index_descriptions[$tablename][$name]);
 				$this->query($sql);
 			}
 		}
-		if(!empty($sqls)) {
-			return join(";\n",$sqls).";";
+		if (!empty($sqls)) {
+			return join(";\n", $sqls).";";
 		} else {
 			return '';
 		}
@@ -1219,7 +1219,7 @@ 
 	public function addColumn($tablename, $fieldDefs)
 	{
 		$sql = $this->addColumnSQL($tablename, $fieldDefs);
-		if ($this->isFieldArray($fieldDefs)){
+		if ($this->isFieldArray($fieldDefs)) {
 			$columns = array();
 			foreach ($fieldDefs as $fieldDef)
 				$columns[] = $fieldDef['name'];
@@ -1229,7 +1229,7 @@ 
 			$columns = $fieldDefs['name'];
 		}
 		$msg = "Error adding column(s) $columns on table: $tablename:";
-		return $this->query($sql,true,$msg);
+		return $this->query($sql, true, $msg);
 	}
 
 	/**
@@ -1242,8 +1242,8 @@ 
      */
 	public function alterColumn($tablename, $newFieldDef, $ignoreRequired = false)
 	{
-		$sql = $this->alterColumnSQL($tablename, $newFieldDef,$ignoreRequired);
-		if ($this->isFieldArray($newFieldDef)){
+		$sql = $this->alterColumnSQL($tablename, $newFieldDef, $ignoreRequired);
+		if ($this->isFieldArray($newFieldDef)) {
 			$columns = array();
 			foreach ($newFieldDef as $fieldDef) {
 				$columns[] = $fieldDef['name'];
@@ -1255,8 +1255,8 @@ 
 		}
 
 		$msg = "Error altering column(s) $columns on table: $tablename:";
-		$res = $this->query($sql,true,$msg);
-		if($res) {
+		$res = $this->query($sql, true, $msg);
+		if ($res) {
 			$this->getTableDescription($tablename, true); // reload table description after altering
 		}
 		return $res;
@@ -1282,7 +1282,7 @@ 
 	public function dropTableName($name)
 	{
 		$sql = $this->dropTableNameSQL($name);
-		return $this->query($sql,true,"Error dropping table $name:");
+		return $this->query($sql, true, "Error dropping table $name:");
 	}
 
     /**
@@ -1297,7 +1297,7 @@ 
 		$tablename = $bean->getTableName();
 		$sql = $this->dropColumnSQL($tablename, $fieldDefs);
 		$msg = "Error deleting column(s) on table: $tablename:";
-		return $this->query($sql,true,$msg);
+		return $this->query($sql, true, $msg);
 	}
 
     /**
@@ -1320,18 +1320,18 @@ 
 
 		$rows_found = 0;
 		$count_query = $bean->create_list_count_query($select_query);
-		if(!empty($count_query))
+		if (!empty($count_query))
 		{
 			// We have a count query.  Run it and get the results.
 			$result = $this->query($count_query, true, "Error running count query for $this->object_name List: ");
 			$assoc = $this->fetchByAssoc($result);
-			if(!empty($assoc['c']))
+			if (!empty($assoc['c']))
 			{
 				$rows_found = $assoc['c'];
 			}
 		}
-		if($count == -1){
-			$count 	= $sugar_config['list_max_entries_per_page'];
+		if ($count == -1) {
+			$count = $sugar_config['list_max_entries_per_page'];
 		}
 		$next_offset = $start + $count;
 
@@ -1344,13 +1344,13 @@ 
 		$fields = $bean->getFieldDefinitions();
 		$custom_fields = array();
 
-		if($bean->hasCustomFields()){
-			foreach ($fields as $fieldDef){
-				if($fieldDef['source'] == 'custom_fields'){
+		if ($bean->hasCustomFields()) {
+			foreach ($fields as $fieldDef) {
+				if ($fieldDef['source'] == 'custom_fields') {
 					$custom_fields[$fieldDef['name']] = $fieldDef['name'];
 				}
 			}
-			if(!empty($custom_fields)){
+			if (!empty($custom_fields)) {
 				$custom_fields['id_c'] = 'id_c';
 				$id_field = array('name' => 'id_c', 'custom_type' => 'id',);
 				$fields[] = $id_field;
@@ -1363,50 +1363,50 @@ 
 		$cstm_row_array = array();
 		$cstm_columns = array();
 		$built_columns = false;
-		while(($row = $this->fetchByAssoc($result)) != null)
+		while (($row = $this->fetchByAssoc($result)) != null)
 		{
 			$values = array();
 			$cstm_values = array();
-			if(!$is_related_query){
+			if (!$is_related_query) {
 				foreach ($fields as $fieldDef)
 				{
-					if(isset($fieldDef['source']) && $fieldDef['source'] != 'db' && $fieldDef['source'] != 'custom_fields') continue;
+					if (isset($fieldDef['source']) && $fieldDef['source'] != 'db' && $fieldDef['source'] != 'custom_fields') continue;
 					$val = $row[$fieldDef['name']];
 
 					//handle auto increment values here only need to do this on insert not create
-					if ($fieldDef['name'] == 'deleted'){
+					if ($fieldDef['name'] == 'deleted') {
 							$values['deleted'] = $val;
-							if(!$built_columns){
+							if (!$built_columns) {
 							$columns[] = 'deleted';
 						}
 					}
 					else
 					{
 						$type = $fieldDef['type'];
-						if(!empty($fieldDef['custom_type'])){
+						if (!empty($fieldDef['custom_type'])) {
 							$type = $fieldDef['custom_type'];
 						}
 						// need to do some thing about types of values
-						if($this->dbType == 'mysql' && $val == '' && ($type == 'datetime' ||  $type == 'date' || $type == 'int' || $type == 'currency' || $type == 'decimal')){
-							if(!empty($custom_fields[$fieldDef['name']]))
+						if ($this->dbType == 'mysql' && $val == '' && ($type == 'datetime' || $type == 'date' || $type == 'int' || $type == 'currency' || $type == 'decimal')) {
+							if (!empty($custom_fields[$fieldDef['name']]))
 								$cstm_values[$fieldDef['name']] = 'null';
 							else
 								$values[$fieldDef['name']] = 'null';
-						}else{
-							if(isset($type) && $type=='int') {
-								if(!empty($custom_fields[$fieldDef['name']]))
+						} else {
+							if (isset($type) && $type == 'int') {
+								if (!empty($custom_fields[$fieldDef['name']]))
 									$cstm_values[$fieldDef['name']] = $GLOBALS['db']->quote(from_html($val));
 								else
 									$values[$fieldDef['name']] = $GLOBALS['db']->quote(from_html($val));
 							} else {
-								if(!empty($custom_fields[$fieldDef['name']]))
+								if (!empty($custom_fields[$fieldDef['name']]))
 									$cstm_values[$fieldDef['name']] = "'".$GLOBALS['db']->quote(from_html($val))."'";
 								else
 									$values[$fieldDef['name']] = "'".$GLOBALS['db']->quote(from_html($val))."'";
 							}
 						}
-						if(!$built_columns){
-							if(!empty($custom_fields[$fieldDef['name']]))
+						if (!$built_columns) {
+							if (!empty($custom_fields[$fieldDef['name']]))
 								$cstm_columns[] = $fieldDef['name'];
 							else
 								$columns[] = $fieldDef['name'];
@@ -1417,19 +1417,19 @@ 
 			} else {
 			foreach ($row as $key=>$val)
 			{
-					if($key != 'orc_row'){
+					if ($key != 'orc_row') {
 						$values[$key] = "'$val'";
-						if(!$built_columns){
+						if (!$built_columns) {
 							$columns[] = $key;
 						}
 					}
 			}
 			}
 			$built_columns = true;
-			if(!empty($values)){
+			if (!empty($values)) {
 				$row_array[] = $values;
 			}
-			if(!empty($cstm_values) && !empty($cstm_values['id_c']) && (strlen($cstm_values['id_c']) > 7)){
+			if (!empty($cstm_values) && !empty($cstm_values['id_c']) && (strlen($cstm_values['id_c']) > 7)) {
 				$cstm_row_array[] = $cstm_values;
 			}
 		}
@@ -1439,9 +1439,9 @@ 
 		// get the entire sql
 		$sql .= "(".implode(",", $columns).") ";
 		$sql .= "VALUES";
-		for($i = 0; $i < count($row_array); $i++){
+		for ($i = 0; $i < count($row_array); $i++) {
 			$sql .= " (".implode(",", $row_array[$i]).")";
-			if($i < (count($row_array) - 1)){
+			if ($i < (count($row_array) - 1)) {
 				$sql .= ", ";
 			}
 		}
@@ -1450,9 +1450,9 @@ 
 		$custom_sql .= "(".implode(",", $cstm_columns).") ";
 		$custom_sql .= "VALUES";
 
-		for($i = 0; $i < count($cstm_row_array); $i++){
+		for ($i = 0; $i < count($cstm_row_array); $i++) {
 			$custom_sql .= " (".implode(",", $cstm_row_array[$i]).")";
-			if($i < (count($cstm_row_array) - 1)){
+			if ($i < (count($cstm_row_array) - 1)) {
 				$custom_sql .= ", ";
 			}
 		}
@@ -1545,17 +1545,17 @@ 
      */
     public function quoteType($type, $value)
 	{
-	    if($type == 'date') {
+	    if ($type == 'date') {
 	        return $this->convert($this->quoted($value), "date");
 	    }
-	    if($type == 'time') {
+	    if ($type == 'time') {
 	        return $this->convert($this->quoted($value), "time");
 	    }
-        if(isset($this->type_class[$type]) &&  $this->type_class[$type] == "date") {
+        if (isset($this->type_class[$type]) && $this->type_class[$type] == "date") {
             return $this->convert($this->quoted($value), "datetime");
         }
-        if($this->isNumericType($type)) {
-            return 0+$value; // ensure it's numeric
+        if ($this->isNumericType($type)) {
+            return 0 + $value; // ensure it's numeric
         }
 
         return $this->quoted($value);
@@ -1571,7 +1571,7 @@ 
      */
 	public function arrayQuote(array &$array)
 	{
-		foreach($array as &$val) {
+		foreach ($array as &$val) {
 			$val = $this->quote($val);
 		}
 		return $array;
@@ -1584,10 +1584,10 @@ 
      */
 	protected function freeResult($result = false)
 	{
-		if($result) {
+		if ($result) {
 			$this->freeDbResult($result);
 		}
-		if($this->lastResult) {
+		if ($this->lastResult) {
 			$this->freeDbResult($this->lastResult);
 			$this->lastResult = null;
 		}
@@ -1616,17 +1616,17 @@ 
 	public function getOne($sql, $dieOnError = false, $msg = '')
 	{
 		$this->log->info("Get One: |$sql|");
-		if(!$this->hasLimit($sql)) {
+		if (!$this->hasLimit($sql)) {
 		    $queryresult = $this->limitQuery($sql, 0, 1, $dieOnError, $msg);
 		} else {
 		    // support old code that passes LIMIT to sql
 		    // works only for mysql, so do not rely on this
 		    $queryresult = $this->query($sql, $dieOnError, $msg);
 		}
-		$this->checkError($msg.' Get One Failed:' . $sql, $dieOnError);
+		$this->checkError($msg.' Get One Failed:'.$sql, $dieOnError);
 		if (!$queryresult) return false;
 		$row = $this->fetchByAssoc($queryresult);
-		if(!empty($row)) {
+		if (!empty($row)) {
 			return array_shift($row);
 		}
 		return false;
@@ -1646,12 +1646,12 @@ 
 		$this->log->info("Fetch One: |$sql|");
 		$this->checkConnection();
 		$queryresult = $this->query($sql, $dieOnError, $msg);
-		$this->checkError($msg.' Fetch One Failed:' . $sql, $dieOnError);
+		$this->checkError($msg.' Fetch One Failed:'.$sql, $dieOnError);
 
 		if (!$queryresult) return false;
 
 		$row = $this->fetchByAssoc($queryresult);
-		if ( !$row ) return false;
+		if (!$row) return false;
 
 		$this->freeResult($queryresult);
 		return $row;
@@ -1692,7 +1692,7 @@ 
      */
 	public function getTableDescription($tablename, $reload = false)
 	{
-		if($reload || empty(self::$table_descriptions[$tablename])) {
+		if ($reload || empty(self::$table_descriptions[$tablename])) {
 			self::$table_descriptions[$tablename] = $this->get_columns($tablename);
 		}
 		return self::$table_descriptions[$tablename];
@@ -1708,12 +1708,12 @@ 
 	protected function describeField($name, $tablename)
 	{
 		$table = $this->getTableDescription($tablename);
-		if(!empty($table) && isset($table[$name]))
+		if (!empty($table) && isset($table[$name]))
 			return 	$table[$name];
 
 		$table = $this->getTableDescription($tablename, true);
 
-		if(isset($table[$name]))
+		if (isset($table[$name]))
 		return $table[$name];
 
 		return array();
@@ -1728,13 +1728,13 @@ 
 	 */
 	protected function describeIndex($name, $tablename)
 	{
-		if(isset(self::$index_descriptions[$tablename]) && isset(self::$index_descriptions[$tablename]) && isset(self::$index_descriptions[$tablename][$name])){
+		if (isset(self::$index_descriptions[$tablename]) && isset(self::$index_descriptions[$tablename]) && isset(self::$index_descriptions[$tablename][$name])) {
 			return 	self::$index_descriptions[$tablename][$name];
 		}
 
 		self::$index_descriptions[$tablename] = $this->get_indices($tablename);
 
-		if(isset(self::$index_descriptions[$tablename][$name])){
+		if (isset(self::$index_descriptions[$tablename][$name])) {
 			return 	self::$index_descriptions[$tablename][$name];
 		}
 
@@ -1751,9 +1751,9 @@ 
      */
 	public function truncate($string, $len)
 	{
-		if ( is_numeric($len) && $len > 0)
+		if (is_numeric($len) && $len > 0)
 		{
-			$string = mb_substr($string,0,(int) $len, "UTF-8");
+			$string = mb_substr($string, 0, (int)$len, "UTF-8");
 		}
 		return $string;
 	}
@@ -1768,11 +1768,11 @@ 
      */
 	public function concat($table, array $fields, $space = ' ')
 	{
-		if(empty($fields)) return '';
+		if (empty($fields)) return '';
 		$elems = array();
 		$space = $this->quoted($space);
-		foreach ( $fields as $field ) {
-			if(!empty($elems)) $elems[] = $space;
+		foreach ($fields as $field) {
+			if (!empty($elems)) $elems[] = $space;
 			$elems[] = $this->convert("$table.$field", 'IFNULL', array("''"));
 		}
 		$first = array_shift($elems);
@@ -1829,8 +1829,8 @@ 
 	 */
 	public function executePreparedQuery($stmt, $data = array())
 	{
-		if(!empty($this->preparedTokens[$stmt])){
-			if(!is_array($data)){
+		if (!empty($this->preparedTokens[$stmt])) {
+			if (!is_array($data)) {
 				$data = array($data);
 			}
 
@@ -1838,7 +1838,7 @@ 
 
 			//ensure that the number of data elements matches the number of replacement tokens
 			//we found in prepare().
-			if(count($data) != $pTokens['tokenCount']){
+			if (count($data) != $pTokens['tokenCount']) {
 				//error the data count did not match the token count
 				return false;
 			}
@@ -1864,7 +1864,7 @@ 
 				}//switch
 			}//foreach
 			return $this->query($query);
-		}else{
+		} else {
 			return false;
 		}
 	}
@@ -1908,7 +1908,7 @@ 
 	{
 		// get column names and values
 		$sql = $this->insertParams($bean->getTableName(), $bean->getFieldDefinitions(), get_object_vars($bean),
-		        isset($bean->field_name_map)?$bean->field_name_map:null, false);
+		        isset($bean->field_name_map) ? $bean->field_name_map : null, false);
 		return $sql;
 	}
 
@@ -1936,48 +1936,48 @@ 
     		if (!empty($bean->field_name_map[$field]['auto_increment'])) continue;
 
     		//custom fields handle their save separately
-    		if(isset($bean->field_name_map) && !empty($bean->field_name_map[$field]['custom_type']))  continue;
+    		if (isset($bean->field_name_map) && !empty($bean->field_name_map[$field]['custom_type']))  continue;
 
     		// no need to clear deleted since we only update not deleted records anyway
-    		if($fieldDef['name'] == 'deleted' && empty($bean->deleted)) continue;
+    		if ($fieldDef['name'] == 'deleted' && empty($bean->deleted)) continue;
 
-    		if(isset($bean->$field)) {
+    		if (isset($bean->$field)) {
     			$val = from_html($bean->$field);
     		} else {
     			continue;
     		}
 
-    		if(!empty($fieldDef['type']) && $fieldDef['type'] == 'bool'){
+    		if (!empty($fieldDef['type']) && $fieldDef['type'] == 'bool') {
     			$val = $bean->getFieldValue($field);
     		}
 
-    		if(strlen($val) == 0) {
-    			if(isset($fieldDef['default']) && strlen($fieldDef['default']) > 0) {
+    		if (strlen($val) == 0) {
+    			if (isset($fieldDef['default']) && strlen($fieldDef['default']) > 0) {
     				$val = $fieldDef['default'];
     			} else {
     				$val = null;
     			}
     		}
 
-    		if(!empty($val) && !empty($fieldDef['len']) && strlen($val) > $fieldDef['len']) {
+    		if (!empty($val) && !empty($fieldDef['len']) && strlen($val) > $fieldDef['len']) {
 			    $val = $this->truncate($val, $fieldDef['len']);
 			}
 		$columnName = $this->quoteIdentifier($fieldDef['name']);
-    		if(!is_null($val) || !empty($fieldDef['required'])) {
+    		if (!is_null($val) || !empty($fieldDef['required'])) {
     			$columns[] = "{$columnName}=".$this->massageValue($val, $fieldDef);
-    		} elseif($this->isNullable($fieldDef)) {
+    		} elseif ($this->isNullable($fieldDef)) {
     			$columns[] = "{$columnName}=NULL";
     		} else {
     		    $columns[] = "{$columnName}=".$this->emptyValue($fieldDef['type']);
     		}
 		}
 
-		if ( sizeof($columns) == 0 )
+		if (sizeof($columns) == 0)
 			return ""; // no columns set
 
 		// build where clause
 		$where = $this->getWhereClause($bean, $this->updateWhereArray($bean, $where));
-		if(isset($fields['deleted'])) {
+		if (isset($fields['deleted'])) {
 		    $where .= " AND deleted=0";
 		}
 
@@ -2001,7 +2001,7 @@ 
 			$primaryColumn = $fieldDef['name'];
 
 			$val = $bean->getFieldValue($fieldDef['name']);
-			if ($val != FALSE){
+			if ($val != FALSE) {
 				$where[$primaryColumn] = $val;
 			}
 		}
@@ -2027,7 +2027,7 @@ 
 			if (is_array($val)) {
 				$op = "IN";
 				$temp = array();
-				foreach ($val as $tval){
+				foreach ($val as $tval) {
 					$temp[] = $this->quoted($tval);
 				}
 				$val = implode(",", $temp);
@@ -2053,9 +2053,9 @@ 
 	 * @param  array  $whereArray Optional, where conditions in an array
 	 * @return string
 	 */
-	protected function getWhereClause(SugarBean $bean, array $whereArray=array())
+	protected function getWhereClause(SugarBean $bean, array $whereArray = array())
 	{
-	    return " WHERE " . $this->getColumnWhereClause($bean->getTableName(), $whereArray);
+	    return " WHERE ".$this->getColumnWhereClause($bean->getTableName(), $whereArray);
 	}
 
 	/**
@@ -2069,13 +2069,13 @@ 
 	{
 		$type = $this->getFieldType($fieldDef);
 
-		if(isset($this->type_class[$type])) {
+		if (isset($this->type_class[$type])) {
 			// handle some known types
-			switch($this->type_class[$type]) {
+			switch ($this->type_class[$type]) {
 				case 'bool':
 				case 'int':
-					if (!empty($fieldDef['required']) && $val == ''){
-						if (isset($fieldDef['default'])){
+					if (!empty($fieldDef['required']) && $val == '') {
+						if (isset($fieldDef['default'])) {
 							return $fieldDef['default'];
 						}
 						return 0;
@@ -2083,16 +2083,16 @@ 
 					return intval($val);
                 case 'bigint' :
                     $val = (float)$val;
-					if (!empty($fieldDef['required']) && $val == false){
-						if (isset($fieldDef['default'])){
+					if (!empty($fieldDef['required']) && $val == false) {
+						if (isset($fieldDef['default'])) {
 							return $fieldDef['default'];
 						}
 						return 0;
 					}
                     return $val;
 				case 'float':
-					if (!empty($fieldDef['required'])  && $val == ''){
-						if (isset($fieldDef['default'])){
+					if (!empty($fieldDef['required']) && $val == '') {
+						if (isset($fieldDef['default'])) {
 							return $fieldDef['default'];
 						}
 						return 0;
@@ -2101,7 +2101,7 @@ 
 				case 'time':
 				case 'date':
 					// empty date can't be '', so convert it to either NULL or empty date value
-					if($val == '') {
+					if ($val == '') {
 						if (!empty($fieldDef['required'])) {
 							if (isset($fieldDef['default'])) {
 								return $fieldDef['default'];
@@ -2113,14 +2113,14 @@ 
 					break;
 			}
 		} else {
-		    if(!empty($val) && !empty($fieldDef['len']) && strlen($val) > $fieldDef['len']) {
+		    if (!empty($val) && !empty($fieldDef['len']) && strlen($val) > $fieldDef['len']) {
 			    $val = $this->truncate($val, $fieldDef['len']);
 			}
 		}
 
-		if ( is_null($val) ) {
-			if(!empty($fieldDef['required'])) {
-				if (isset($fieldDef['default'])  && $fieldDef['default'] != ''){
+		if (is_null($val)) {
+			if (!empty($fieldDef['required'])) {
+				if (isset($fieldDef['default']) && $fieldDef['default'] != '') {
 					return $fieldDef['default'];
 				}
 				return $this->emptyValue($type);
@@ -2128,7 +2128,7 @@ 
 				return "NULL";
 			}
 		}
-        if($type == "datetimecombo") {
+        if ($type == "datetimecombo") {
             $type = "datetime";
         }
 		return $this->convert($this->quoted($val), $type);
@@ -2143,23 +2143,23 @@ 
 	 */
 	public function massageFieldDef(&$fieldDef, $tablename)
 	{
-		if ( !isset($fieldDef['dbType']) ) {
-			if ( isset($fieldDef['dbtype']) )
+		if (!isset($fieldDef['dbType'])) {
+			if (isset($fieldDef['dbtype']))
 				$fieldDef['dbType'] = $fieldDef['dbtype'];
 			else
 				$fieldDef['dbType'] = $fieldDef['type'];
 		}
-		$type = $this->getColumnType($fieldDef['dbType'],$fieldDef['name'],$tablename);
+		$type = $this->getColumnType($fieldDef['dbType'], $fieldDef['name'], $tablename);
 		$matches = array();
         // len can be a number or a string like 'max', for example, nvarchar(max)
         preg_match_all('/(\w+)(?:\(([0-9]+,?[0-9]*|\w+)\)|)/i', $type, $matches);
-		if ( isset($matches[1][0]) )
+		if (isset($matches[1][0]))
 			$fieldDef['type'] = $matches[1][0];
-		if ( isset($matches[2][0]) && empty($fieldDef['len']) )
+		if (isset($matches[2][0]) && empty($fieldDef['len']))
 			$fieldDef['len'] = $matches[2][0];
-		if ( !empty($fieldDef['precision']) && is_numeric($fieldDef['precision']) && !strstr($fieldDef['len'],',') )
+		if (!empty($fieldDef['precision']) && is_numeric($fieldDef['precision']) && !strstr($fieldDef['len'], ','))
 			$fieldDef['len'] .= ",{$fieldDef['precision']}";
-		if (!empty($fieldDef['required']) || ($fieldDef['name'] == 'id' && !isset($fieldDef['required'])) ) {
+		if (!empty($fieldDef['required']) || ($fieldDef['name'] == 'id' && !isset($fieldDef['required']))) {
 			$fieldDef['required'] = 'true';
 		}
 	}
@@ -2180,7 +2180,7 @@ 
 		$level = 0;
 		$selectField = "";
 		$strLen = strlen($selectStatement);
-		for($i = 0; $i < $strLen; $i++)
+		for ($i = 0; $i < $strLen; $i++)
 		{
 			$char = $selectStatement[$i];
 
@@ -2190,16 +2190,16 @@ 
 				$fields[$field] = $selectField;
 				$selectField = "";
 			}
-			else if ($char == "("){
+			else if ($char == "(") {
 				$level++;
 				$selectField .= $char;
 			}
-			else if($char == ")"){
+			else if ($char == ")") {
 				$level--;
 				$selectField .= $char;
 
 
-			}else{
+			} else {
 				$selectField .= $char;
 			}
 
@@ -2215,7 +2215,7 @@ 
      */
 	protected function getFieldNameFromSelect($string)
 	{
-		if(strncasecmp($string, "DISTINCT ", 9) == 0) {
+		if (strncasecmp($string, "DISTINCT ", 9) == 0) {
 			$string = substr($string, 9);
 		}
 		if (stripos($string, " as ") !== false)
@@ -2293,14 +2293,14 @@ 
 				foreach ($cols[$beanID] as $def) $select[] = $table.".".$def['name'];
 
 			// build part of where clause
-			if (is_array($whereClause[$beanID])){
+			if (is_array($whereClause[$beanID])) {
 				$where[] = $this->getColumnWhereClause($table, $whereClause[$beanID]);
 			}
 			// initialize so that it can be used properly in form clause generation
 			$table_used_in_from[$table] = false;
 
 			$indices = $bean->getIndices();
-			foreach ($indices as $index){
+			foreach ($indices as $index) {
 				if ($index['type'] == 'foreign') {
 					$relationship[$table][] = array('foreignTable'=> $index['foreignTable']
 												,'foreignColumn'=>$index['foreignField']
@@ -2321,12 +2321,12 @@ 
 		// table2 is assumed to joining through primary key called id
 		$separator = "";
 		$from = ''; $table_used_in_from = array();
-		foreach ($relations as $table1 => $rightsidearray){
+		foreach ($relations as $table1 => $rightsidearray) {
 			if ($table_used_in_from[$table1]) continue; // table has been joined
 
 			$from .= $separator." ".$table1;
 			$table_used_in_from[$table1] = true;
-			foreach ($rightsidearray as $tablearray){
+			foreach ($rightsidearray as $tablearray) {
 				$table2 = $tablearray['foreignTable']; // get foreign table
 				$tableAlias = $aliases[$table2]; // get a list of aliases for this table
 				foreach ($tableAlias as $table2) {
@@ -2386,9 +2386,9 @@ 
 		// we do not have change a lot of existing code
 		// and add dbtype where type is being used for some special
 		// purposes like referring to foreign table etc.
-		if(!empty($fieldDef['dbType']))
+		if (!empty($fieldDef['dbType']))
 			return  $fieldDef['dbType'];
-		if(!empty($fieldDef['dbtype']))
+		if (!empty($fieldDef['dbtype']))
 			return  $fieldDef['dbtype'];
 		if (!empty($fieldDef['type']))
 			return  $fieldDef['type'];
@@ -2408,17 +2408,17 @@ 
      */
     public function getTypeParts($type)
     {
-        if(preg_match("#(?P<type>\w+)\s*(?P<arg>\((?P<len>\w+)\s*(,\s*(?P<scale>\d+))*\))*#", $type, $matches))
+        if (preg_match("#(?P<type>\w+)\s*(?P<arg>\((?P<len>\w+)\s*(,\s*(?P<scale>\d+))*\))*#", $type, $matches))
         {
-            $return = array();  // Not returning matches array as such as we don't want to expose the regex make up on the interface
+            $return = array(); // Not returning matches array as such as we don't want to expose the regex make up on the interface
             $return['baseType'] = $matches['type'];
-            if( isset($matches['arg'])) {
+            if (isset($matches['arg'])) {
                 $return['arg'] = $matches['arg'];
             }
-            if( isset($matches['len'])) {
+            if (isset($matches['len'])) {
                 $return['len'] = $matches['len'];
             }
-            if( isset($matches['scale'])) {
+            if (isset($matches['scale'])) {
                 $return['scale'] = $matches['scale'];
             }
             return $return;
@@ -2442,28 +2442,28 @@ 
 		$type = $this->getFieldType($fieldDef);
         $colType = $this->getColumnType($type);
 
-        if($parts = $this->getTypeParts($colType))
+        if ($parts = $this->getTypeParts($colType))
         {
             $colBaseType = $parts['baseType'];
-            $defLen =  isset($parts['len']) ? $parts['len'] : '255'; // Use the mappings length (precision) as default if it exists
+            $defLen = isset($parts['len']) ? $parts['len'] : '255'; // Use the mappings length (precision) as default if it exists
         }
 
-        if(!empty($fieldDef['len'])) {
-            if (in_array($colBaseType, array( 'nvarchar', 'nchar', 'varchar', 'varchar2', 'char',
+        if (!empty($fieldDef['len'])) {
+            if (in_array($colBaseType, array('nvarchar', 'nchar', 'varchar', 'varchar2', 'char',
                                           'clob', 'blob', 'text'))) {
           	    $colType = "$colBaseType(${fieldDef['len']})";
-            } elseif(($colBaseType == 'decimal' || $colBaseType == 'float')){
-                  if(!empty($fieldDef['precision']) && is_numeric($fieldDef['precision']))
-                      if(strpos($fieldDef['len'],',') === false){
-                          $colType = $colBaseType . "(".$fieldDef['len'].",".$fieldDef['precision'].")";
-                      }else{
-                          $colType = $colBaseType . "(".$fieldDef['len'].")";
+            } elseif (($colBaseType == 'decimal' || $colBaseType == 'float')) {
+                  if (!empty($fieldDef['precision']) && is_numeric($fieldDef['precision']))
+                      if (strpos($fieldDef['len'], ',') === false) {
+                          $colType = $colBaseType."(".$fieldDef['len'].",".$fieldDef['precision'].")";
+                      } else {
+                          $colType = $colBaseType."(".$fieldDef['len'].")";
                       }
                   else
-                          $colType = $colBaseType . "(".$fieldDef['len'].")";
+                          $colType = $colBaseType."(".$fieldDef['len'].")";
               }
         } else {
-            if (in_array($colBaseType, array( 'nvarchar', 'nchar', 'varchar', 'varchar2', 'char'))) {
+            if (in_array($colBaseType, array('nvarchar', 'nchar', 'varchar', 'varchar2', 'char'))) {
                 $colType = "$colBaseType($defLen)";
             }
         }
@@ -2485,29 +2485,29 @@ 
         }
 
 		$auto_increment = '';
-		if(!empty($fieldDef['auto_increment']) && $fieldDef['auto_increment'])
-			$auto_increment = $this->setAutoIncrement($table , $fieldDef['name']);
+		if (!empty($fieldDef['auto_increment']) && $fieldDef['auto_increment'])
+			$auto_increment = $this->setAutoIncrement($table, $fieldDef['name']);
 
-		$required = 'NULL';  // MySQL defaults to NULL, SQL Server defaults to NOT NULL -- must specify
+		$required = 'NULL'; // MySQL defaults to NULL, SQL Server defaults to NOT NULL -- must specify
 		//Starting in 6.0, only ID and auto_increment fields will be NOT NULL in the DB.
 		if ((empty($fieldDef['isnull']) || strtolower($fieldDef['isnull']) == 'false') &&
 			(!empty($auto_increment) || $name == 'id' || ($fieldDef['type'] == 'id' && !empty($fieldDef['required'])))) {
-			$required =  "NOT NULL";
+			$required = "NOT NULL";
 		}
 		// If the field is marked both required & isnull=>false - alwqys make it not null
 		// Use this to ensure primary key fields never defined as null
-		if(isset($fieldDef['isnull']) && (strtolower($fieldDef['isnull']) == 'false' || $fieldDef['isnull'] === false)
+		if (isset($fieldDef['isnull']) && (strtolower($fieldDef['isnull']) == 'false' || $fieldDef['isnull'] === false)
 			&& !empty($fieldDef['required'])) {
-			$required =  "NOT NULL";
+			$required = "NOT NULL";
 		}
 		if ($ignoreRequired)
 			$required = "";
 
-		if ( $return_as_array ) {
+		if ($return_as_array) {
 			return array(
 				'name' => $name,
 				'colType' => $colType,
-                'colBaseType' => $colBaseType,  // Adding base type for easier processing in derived classes
+                'colBaseType' => $colBaseType, // Adding base type for easier processing in derived classes
 				'default' => $default,
 				'required' => $required,
 				'auto_increment' => $auto_increment,
@@ -2532,14 +2532,14 @@ 
 
 		if ($this->isFieldArray($fieldDefs)) {
 			foreach ($fieldDefs as $fieldDef) {
-				if(!isset($fieldDef['source']) || $fieldDef['source'] == 'db') {
-					$columns[] = $this->oneColumnSQLRep($fieldDef,false, $tablename);
+				if (!isset($fieldDef['source']) || $fieldDef['source'] == 'db') {
+					$columns[] = $this->oneColumnSQLRep($fieldDef, false, $tablename);
 				}
 			}
 			$columns = implode(",", $columns);
 		}
 		else {
-			$columns = $this->oneColumnSQLRep($fieldDefs,$ignoreRequired, $tablename);
+			$columns = $this->oneColumnSQLRep($fieldDefs, $ignoreRequired, $tablename);
 		}
 
 		return $columns;
@@ -2701,40 +2701,40 @@ 
      */
 	public function getValidDBName($name, $ensureUnique = false, $type = 'column', $force = false)
 	{
-		if(is_array($name)) {
+		if (is_array($name)) {
 			$result = array();
-			foreach($name as $field) {
+			foreach ($name as $field) {
 				$result[] = $this->getValidDBName($field, $ensureUnique, $type);
 			}
 			return $result;
 		} else {
-		    if(strchr($name, ".")) {
+		    if (strchr($name, ".")) {
 		        // this is a compound name with dots, handle separately
 		        $parts = explode(".", $name);
-		        if(count($parts) > 2) {
+		        if (count($parts) > 2) {
 		            // some weird name, cut to table.name
-		            array_splice($parts, 0, count($parts)-2);
+		            array_splice($parts, 0, count($parts) - 2);
 		        }
 		        $parts = $this->getValidDBName($parts, $ensureUnique, $type, $force);
                 return join(".", $parts);
 		    }
 			// first strip any invalid characters - all but word chars (which is alphanumeric and _)
-			$name = preg_replace( '/[^\w]+/i', '', $name ) ;
-			$len = strlen( $name ) ;
+			$name = preg_replace('/[^\w]+/i', '', $name);
+			$len = strlen($name);
 			$maxLen = empty($this->maxNameLengths[$type]) ? $this->maxNameLengths[$type]['column'] : $this->maxNameLengths[$type];
 			if ($len <= $maxLen && !$force) {
 				return strtolower($name);
 			}
 			if ($ensureUnique) {
 				$md5str = md5($name);
-				$tail = substr ( $name, -11) ;
-				$temp = substr($md5str , strlen($md5str)-4 );
-				$result = substr( $name, 0, 10) . $temp . $tail ;
+				$tail = substr($name, -11);
+				$temp = substr($md5str, strlen($md5str) - 4);
+				$result = substr($name, 0, 10).$temp.$tail;
 			} else {
-				$result = substr( $name, 0, 11) . substr( $name, 11 - $maxLen);
+				$result = substr($name, 0, 11).substr($name, 11 - $maxLen);
 			}
 
-			return strtolower( $result ) ;
+			return strtolower($result);
 		}
 	}
 
@@ -2746,7 +2746,7 @@ 
 	 */
 	public function getColumnType($type)
 	{
-		return isset($this->type_map[$type])?$this->type_map[$type]:$type;
+		return isset($this->type_map[$type]) ? $this->type_map[$type] : $type;
 	}
 
 	/**
@@ -2760,10 +2760,10 @@ 
 	 */
 	public function isFieldArray($defArray)
 	{
-		if ( !is_array($defArray) )
+		if (!is_array($defArray))
 			return false;
 
-		if ( isset($defArray['type']) ){
+		if (isset($defArray['type'])) {
 			// type key exists. May be an array of defs or a simple definition
 			return is_array($defArray['type']); // type is not an array => definition else array
 		}
@@ -2798,19 +2798,19 @@ 
 		require('metadata/audit_templateMetaData.php');
 		$fieldDefs = $dictionary['audit']['fields'];
 
-		$values=array();
+		$values = array();
 		$values['id'] = $this->massageValue(create_guid(), $fieldDefs['id']);
-		$values['parent_id']= $this->massageValue($bean->id, $fieldDefs['parent_id']);
-		$values['field_name']= $this->massageValue($changes['field_name'], $fieldDefs['field_name']);
+		$values['parent_id'] = $this->massageValue($bean->id, $fieldDefs['parent_id']);
+		$values['field_name'] = $this->massageValue($changes['field_name'], $fieldDefs['field_name']);
 		$values['data_type'] = $this->massageValue($changes['data_type'], $fieldDefs['data_type']);
-		if ($changes['data_type']=='text') {
+		if ($changes['data_type'] == 'text') {
 			$values['before_value_text'] = $this->massageValue($changes['before'], $fieldDefs['before_value_text']);
 			$values['after_value_text'] = $this->massageValue($changes['after'], $fieldDefs['after_value_text']);
 		} else {
 			$values['before_value_string'] = $this->massageValue($changes['before'], $fieldDefs['before_value_string']);
 			$values['after_value_string'] = $this->massageValue($changes['after'], $fieldDefs['after_value_string']);
 		}
-		$values['date_created'] = $this->massageValue(TimeDate::getInstance()->nowDb(), $fieldDefs['date_created'] );
+		$values['date_created'] = $this->massageValue(TimeDate::getInstance()->nowDb(), $fieldDefs['date_created']);
 		$values['created_by'] = $this->massageValue($current_user->id, $fieldDefs['created_by']);
 
 		$sql .= "(".implode(",", array_keys($values)).") ";
@@ -2840,9 +2840,9 @@ 
      * @param array|null $field_filter Array of filter names to be inspected (NULL means all fields)
      * @return array
      */
-    public function getDataChanges(SugarBean &$bean, array $field_filter = null)
+    public function getDataChanges(SugarBean & $bean, array $field_filter = null)
 	{
-        $changed_values=array();
+        $changed_values = array();
 
         $fetched_row = array();
         if (is_array($bean->fetched_row))
@@ -2862,40 +2862,40 @@ 
             $field_defs = array_intersect_key($field_defs, $fetched_row);
 
             // remove fields which do not exist as bean property
-            $field_defs = array_intersect_key($field_defs, (array) $bean);
+            $field_defs = array_intersect_key($field_defs, (array)$bean);
 
             foreach ($field_defs as $field => $properties) {
                 $before_value = $fetched_row[$field];
-                $after_value=$bean->$field;
+                $after_value = $bean->$field;
                 if (isset($properties['type'])) {
-                    $field_type=$properties['type'];
+                    $field_type = $properties['type'];
                 } else {
                     if (isset($properties['dbType'])) {
-                        $field_type=$properties['dbType'];
+                        $field_type = $properties['dbType'];
                     }
-                    else if(isset($properties['data_type'])) {
-                        $field_type=$properties['data_type'];
+                    else if (isset($properties['data_type'])) {
+                        $field_type = $properties['data_type'];
                     }
                     else {
-                        $field_type=$properties['dbtype'];
+                        $field_type = $properties['dbtype'];
                     }
                 }
 
                 //Because of bug #25078(sqlserver haven't 'date' type, trim extra "00:00:00" when insert into *_cstm table).
                 // so when we read the audit datetime field from sqlserver, we have to replace the extra "00:00:00" again.
-                if(!empty($field_type) && $field_type == 'date'){
-                    $before_value = $this->fromConvert($before_value , $field_type);
+                if (!empty($field_type) && $field_type == 'date') {
+                    $before_value = $this->fromConvert($before_value, $field_type);
                 }
                 //if the type and values match, do nothing.
-                if (!($this->_emptyValue($before_value,$field_type) && $this->_emptyValue($after_value,$field_type))) {
+                if (!($this->_emptyValue($before_value, $field_type) && $this->_emptyValue($after_value, $field_type))) {
                     $change = false;
                     if (trim($before_value) !== trim($after_value)) {
                         // Bug #42475: Don't directly compare numeric values, instead do the subtract and see if the comparison comes out to be "close enough", it is necessary for floating point numbers.
                         // Manual merge of fix 95727f2eed44852f1b6bce9a9eccbe065fe6249f from DBHelper
                         // This fix also fixes Bug #44624 in a more generic way and therefore eliminates the need for fix 0a55125b281c4bee87eb347709af462715f33d2d in DBHelper
                         if ($this->isNumericType($field_type)) {
-                            $numerator = abs(2*((trim($before_value)+0)-(trim($after_value)+0)));
-                            $denominator = abs(((trim($before_value)+0)+(trim($after_value)+0)));
+                            $numerator = abs(2 * ((trim($before_value) + 0) - (trim($after_value) + 0)));
+                            $denominator = abs(((trim($before_value) + 0) + (trim($after_value) + 0)));
                             // detect whether to use absolute or relative error. use absolute if denominator is zero to avoid division by zero
                             $error = ($denominator == 0) ? $numerator : $numerator / $denominator;
                             if ($error >= 0.0000000001) {    // Smaller than 10E-10
@@ -2911,7 +2911,7 @@ 
                             $change = true;
                         }
                         if ($change) {
-                            $changed_values[$field]=array('field_name'=>$field,
+                            $changed_values[$field] = array('field_name'=>$field,
                                 'data_type'=>$field_type,
                                 'before'=>$before_value,
                                 'after'=>$after_value);
@@ -2982,8 +2982,8 @@ 
      */
 	public function renameIndexDefs($old_definition, $new_definition, $table_name)
 	{
-		return array($this->add_drop_constraint($table_name,$old_definition,true),
-				$this->add_drop_constraint($table_name,$new_definition), false);
+		return array($this->add_drop_constraint($table_name, $old_definition, true),
+				$this->add_drop_constraint($table_name, $new_definition), false);
 	}
 
 	/**
@@ -3005,7 +3005,7 @@ 
 	protected function _getBooleanValue($val)
 	{
 		//need to put the === sign here otherwise true == 'non empty string'
-		if (empty($val) or $val==='off')
+		if (empty($val) or $val === 'off')
 			return false;
 
 		return true;
@@ -3018,7 +3018,7 @@ 
 	 */
 	public function isNumericType($type)
 	{
-	    if(isset($this->type_class[$type]) && ($this->type_class[$type] == 'int' || $this->type_class[$type] == 'float')) {
+	    if (isset($this->type_class[$type]) && ($this->type_class[$type] == 'int' || $this->type_class[$type] == 'float')) {
 	        return true;
 	    }
 		return false;
@@ -3035,7 +3035,7 @@ 
 		if (empty($val))
 			return true;
 
-		if($this->emptyValue($type) == $val) {
+		if ($this->emptyValue($type) == $val) {
 			return true;
 		}
 		switch ($type) {
@@ -3094,7 +3094,7 @@ 
 		$i = 0;
 		$order_by_arr = array();
 		foreach ($values as $key => $value) {
-			if($key == '') {
+			if ($key == '') {
 				$order_by_arr[] = "WHEN ($order_by='' OR $order_by IS NULL) THEN $i";
 			} else {
 				$order_by_arr[] = "WHEN $order_by=".$this->quoted($key)." THEN $i";
@@ -3112,7 +3112,7 @@ 
      */
 	public function emptyValue($type)
 	{
-		if(isset($this->type_class[$type]) && ($this->type_class[$type] == 'bool' || $this->type_class[$type] == 'int' || $this->type_class[$type] == 'float')) {
+		if (isset($this->type_class[$type]) && ($this->type_class[$type] == 'bool' || $this->type_class[$type] == 'int' || $this->type_class[$type] == 'float')) {
 			return 0;
 		}
 
@@ -3161,9 +3161,9 @@ 
      * @return resource|bool query result
      * @see DBManager::limitQuery()
 	 */
-	public function limitQuerySql($sql, $start, $count, $dieOnError=false, $msg='')
+	public function limitQuerySql($sql, $start, $count, $dieOnError = false, $msg = '')
 	{
-		return $this->limitQuery($sql,$start,$count,$dieOnError,$msg,false);
+		return $this->limitQuery($sql, $start, $count, $dieOnError, $msg, false);
 	}
 
 	/**
@@ -3182,7 +3182,7 @@ 
      */
 	public function checkPrivilege($privilege)
 	{
-		switch($privilege) {
+		switch ($privilege) {
 			case "CREATE TABLE":
 				$this->query("CREATE TABLE temp (id varchar(36))");
 				break;
@@ -3203,23 +3203,23 @@ 
 				break;
 			case "ADD COLUMN":
 				$test = array("test" => array("name" => "test", "type" => "varchar", "len" => 50));
-				$sql = 	$this->changeColumnSQL("temp", $test, "add");
+				$sql = $this->changeColumnSQL("temp", $test, "add");
 				$this->query($sql);
 				break;
 			case "CHANGE COLUMN":
 				$test = array("test" => array("name" => "test", "type" => "varchar", "len" => 100));
-				$sql = 	$this->changeColumnSQL("temp", $test, "modify");
+				$sql = $this->changeColumnSQL("temp", $test, "modify");
 				$this->query($sql);
 				break;
 			case "DROP COLUMN":
 				$test = array("test" => array("name" => "test", "type" => "varchar", "len" => 100));
-				$sql = 	$this->changeColumnSQL("temp", $test, "drop");
+				$sql = $this->changeColumnSQL("temp", $test, "drop");
 				$this->query($sql);
 				break;
 			default:
 				return false;
 		}
-		if($this->checkError("Checking privileges")) {
+		if ($this->checkError("Checking privileges")) {
 			return false;
 		}
 		return true;
@@ -3236,7 +3236,7 @@ 
 		$select_check = strpos(strtolower($query), strtolower("SELECT"));
 		//Checks to see if there is union select which is valid
 		$select_check2 = strpos(strtolower($query), strtolower("(SELECT"));
-		if($select_check==0 || $select_check2==0){
+		if ($select_check == 0 || $select_check2 == 0) {
 			//Returning false means query is ok!
 			return true;
 		}
@@ -3255,10 +3255,10 @@ 
 	public function parseFulltextQuery($query)
 	{
 		/* split on space or comma, double quotes with \ for escape */
-		if(strpbrk($query, " ,")) {
+		if (strpbrk($query, " ,")) {
 			// ("([^"]*?)"|[^" ,]+)((, )+)?
 			// '/([^" ,]+|".*?[^\\\\]")(,|\s)\s*/'
-			if(!preg_match_all('/("([^"]*?)"|[^"\s,]+)((,\s)+)?/', $query, $m)) {
+			if (!preg_match_all('/("([^"]*?)"|[^"\s,]+)((,\s)+)?/', $query, $m)) {
 				return false;
 			}
 			$qterms = $m[1];
@@ -3266,17 +3266,17 @@ 
 			$qterms = array($query);
 		}
 		$terms = $must_terms = $not_terms = array();
-		foreach($qterms as $item) {
-			if($item[0] == '"') {
+		foreach ($qterms as $item) {
+			if ($item[0] == '"') {
 				$item = trim($item, '"');
 			}
-			if($item[0] == '+') {
+			if ($item[0] == '+') {
                 if (strlen($item) > 1) {
                     $must_terms[] = substr($item, 1);
                 }
                 continue;
 			}
-			if($item[0] == '-') {
+			if ($item[0] == '-') {
                 if (strlen($item) > 1) {
 				    $not_terms[] = substr($item, 1);
                 }
@@ -3325,11 +3325,11 @@ 
 	public function verifySQLStatement($query, $skipTables)
 	{
 		$query = trim($query);
-		foreach($this->standardQueries as $qstart => $check) {
-			if(strncasecmp($qstart, $query, strlen($qstart)) == 0) {
-				if(is_callable(array($this, $check))) {
+		foreach ($this->standardQueries as $qstart => $check) {
+			if (strncasecmp($qstart, $query, strlen($qstart)) == 0) {
+				if (is_callable(array($this, $check))) {
 					$table = $this->extractTableName($query);
-					if(!in_array($table, $skipTables)) {
+					if (!in_array($table, $skipTables)) {
 						return call_user_func(array($this, $check), $table, $query);
 					} else {
 						$this->log->debug("Skipping table $table as blacklisted");
@@ -3358,20 +3358,20 @@ 
 		$tempname = $table."__uw_temp";
 		$tempTableQuery = str_replace("CREATE TABLE {$table}", "CREATE TABLE $tempname", $query);
 
-		if(strpos($tempTableQuery, '__uw_temp') === false) {
+		if (strpos($tempTableQuery, '__uw_temp') === false) {
 			return 'Could not use a temp table to test query!';
 		}
 
 		$this->query($tempTableQuery, false, "Preflight Failed for: {$query}");
 
 		$error = $this->lastError(); // empty on no-errors
-		if(!empty($error)) {
+		if (!empty($error)) {
 			return $error;
 		}
 
 		// check if table exists
 		$this->log->debug('testing for table: '.$table);
-		if(!$this->tableExists($tempname)) {
+		if (!$this->tableExists($tempname)) {
 			return "Failed to create temp table!";
 		}
 
@@ -3390,8 +3390,8 @@ 
 	public function queryArray(array $sqls, $dieOnError = false, $msg = '', $suppress = false)
 	{
 		$last = true;
-		foreach($sqls as $sql) {
-			if(!($last = $this->query($sql, $dieOnError, $msg, $suppress))) {
+		foreach ($sqls as $sql) {
+			if (!($last = $this->query($sql, $dieOnError, $msg, $suppress))) {
 				break;
 			}
 		}
@@ -3409,7 +3409,7 @@ 
 	{
 	    if (empty($result))	return false;
 
-	    if(is_int($encode) && func_num_args() == 3) {
+	    if (is_int($encode) && func_num_args() == 3) {
 	        // old API: $result, $rowNum, $encode
 	        $GLOBALS['log']->deprecated("Using row number in fetchByAssoc is not portable and no longer supported. Please fix your code.");
 	        $encode = func_get_arg(2);
@@ -3460,7 +3460,7 @@ 
 	 */
 	public function getOption($option)
 	{
-	    if(isset($this->options[$option])) {
+	    if (isset($this->options[$option])) {
 	        return $this->options[$option];
 	    }
 	    return null;
@@ -3500,7 +3500,7 @@ 
 	public function isDatabaseNameValid($name)
 	{
 		// Generic case - no slashes, no dots
-		return preg_match('#[/.\\\\]#', $name)==0;
+		return preg_match('#[/.\\\\]#', $name) == 0;
 	}
 
 	/**

include/database/MysqliManager.php

Spacing +36 added lines, -36 removed lines

@@ -1,5 +1,5 @@ 
 <?php
-if(!defined('sugarEntry') || !sugarEntry) die('Not A Valid Entry Point');
+if (!defined('sugarEntry') || !sugarEntry) die('Not A Valid Entry Point');
 /*********************************************************************************
  * SugarCRM Community Edition is a customer relationship management program developed by
  * SugarCRM, Inc. Copyright (C) 2004-2013 SugarCRM Inc.
@@ -121,18 +121,18 @@ 
 	 */
 	public function query($sql, $dieOnError = false, $msg = '', $suppress = false, $keepResult = false)
 	{
-		if(is_array($sql)) {
+		if (is_array($sql)) {
 			return $this->queryArray($sql, $dieOnError, $msg, $suppress);
         }
 
 		static $queryMD5 = array();
 
 		parent::countQuery($sql);
-		$GLOBALS['log']->info('Query:' . $sql);
+		$GLOBALS['log']->info('Query:'.$sql);
 		$this->checkConnection();
 		$this->query_time = microtime(true);
 		$this->lastsql = $sql;
-		$result = $suppress?@mysqli_query($this->database,$sql):mysqli_query($this->database,$sql);
+		$result = $suppress ? @mysqli_query($this->database, $sql) : mysqli_query($this->database, $sql);
 		$md5 = md5($sql);
 
 		if (empty($queryMD5[$md5]))
@@ -154,9 +154,9 @@ 
 		*/
 
 
-		if($keepResult)
+		if ($keepResult)
 			$this->lastResult = $result;
-		$this->checkError($msg.' Query Failed: ' . $sql, $dieOnError);
+		$this->checkError($msg.' Query Failed: '.$sql, $dieOnError);
 
 		return $result;
 	}
@@ -193,10 +193,10 @@ 
 	 */
 	public function disconnect()
 	{
-		if(isset($GLOBALS['log']) && !is_null($GLOBALS['log'])) {
+		if (isset($GLOBALS['log']) && !is_null($GLOBALS['log'])) {
 			$GLOBALS['log']->debug('Calling MySQLi::disconnect()');
 		}
-		if(!empty($this->database)){
+		if (!empty($this->database)) {
 			$this->freeResult();
 			mysqli_close($this->database);
 			$this->database = null;
@@ -208,7 +208,7 @@ 
 	 */
 	protected function freeDbResult($dbResult)
 	{
-		if(!empty($dbResult))
+		if (!empty($dbResult))
 			mysqli_free_result($dbResult);
 	}
 
@@ -228,7 +228,7 @@ 
 			if (!$meta)
 				return 0;
 
-			if($make_lower_case == true)
+			if ($make_lower_case == true)
 				$meta->name = strtolower($meta->name);
 
 			$field_array[] = $meta->name;
@@ -247,7 +247,7 @@ 
 		if (empty($result))	return false;
 
 		$row = mysqli_fetch_assoc($result);
-		if($row == null) $row = false; //Make sure MySQLi driver results are consistent with other database drivers
+		if ($row == null) $row = false; //Make sure MySQLi driver results are consistent with other database drivers
 		return $row;
 	}
 
@@ -256,7 +256,7 @@ 
 	 */
 	public function quote($string)
 	{
-		return mysqli_real_escape_string($this->getDatabase(),$this->quoteInternal($string));
+		return mysqli_real_escape_string($this->getDatabase(), $this->quoteInternal($string));
 	}
 
 	/**
@@ -269,23 +269,23 @@ 
 		if (is_null($configOptions))
 			$configOptions = $sugar_config['dbconfig'];
 
-		if(!isset($this->database)) {
+		if (!isset($this->database)) {
 
 			//mysqli connector has a separate parameter for port.. We need to separate it out from the host name
-			$dbhost=$configOptions['db_host_name'];
-            $dbport=isset($configOptions['db_port']) ? ($configOptions['db_port'] == '' ? null : $configOptions['db_port']) : null;
+			$dbhost = $configOptions['db_host_name'];
+            $dbport = isset($configOptions['db_port']) ? ($configOptions['db_port'] == '' ? null : $configOptions['db_port']) : null;
 			
-			$pos=strpos($configOptions['db_host_name'],':');
+			$pos = strpos($configOptions['db_host_name'], ':');
 			if ($pos !== false) {
-				$dbhost=substr($configOptions['db_host_name'],0,$pos);
-				$dbport=substr($configOptions['db_host_name'],$pos+1);
+				$dbhost = substr($configOptions['db_host_name'], 0, $pos);
+				$dbport = substr($configOptions['db_host_name'], $pos + 1);
 			}
 
-			$this->database = @mysqli_connect($dbhost,$configOptions['db_user_name'],$configOptions['db_password'],isset($configOptions['db_name'])?$configOptions['db_name']:'',$dbport);
-			if(empty($this->database)) {
-				$GLOBALS['log']->fatal("Could not connect to DB server ".$dbhost." as ".$configOptions['db_user_name'].". port " .$dbport . ": " . mysqli_connect_error());
-				if($dieOnError) {
-					if(isset($GLOBALS['app_strings']['ERR_NO_DB'])) {
+			$this->database = @mysqli_connect($dbhost, $configOptions['db_user_name'], $configOptions['db_password'], isset($configOptions['db_name']) ? $configOptions['db_name'] : '', $dbport);
+			if (empty($this->database)) {
+				$GLOBALS['log']->fatal("Could not connect to DB server ".$dbhost." as ".$configOptions['db_user_name'].". port ".$dbport.": ".mysqli_connect_error());
+				if ($dieOnError) {
+					if (isset($GLOBALS['app_strings']['ERR_NO_DB'])) {
 						sugar_die($GLOBALS['app_strings']['ERR_NO_DB']);
 					} else {
 						sugar_die("Could not connect to the database. Please refer to suitecrm.log for details.");
@@ -296,10 +296,10 @@ 
 			}
 		}
 
-		if(!empty($configOptions['db_name']) && !@mysqli_select_db($this->database,$configOptions['db_name'])) {
-			$GLOBALS['log']->fatal( "Unable to select database {$configOptions['db_name']}: " . mysqli_connect_error());
-			if($dieOnError) {
-					if(isset($GLOBALS['app_strings']['ERR_NO_DB'])) {
+		if (!empty($configOptions['db_name']) && !@mysqli_select_db($this->database, $configOptions['db_name'])) {
+			$GLOBALS['log']->fatal("Unable to select database {$configOptions['db_name']}: ".mysqli_connect_error());
+			if ($dieOnError) {
+					if (isset($GLOBALS['app_strings']['ERR_NO_DB'])) {
 						sugar_die($GLOBALS['app_strings']['ERR_NO_DB']);
 					} else {
 						sugar_die("Could not connect to the database. Please refer to suitecrm.log for details.");
@@ -312,13 +312,13 @@ 
 		// cn: using direct calls to prevent this from spamming the Logs
 	    
 	    $collation = $this->getOption('collation');
-	    if(!empty($collation)) {
+	    if (!empty($collation)) {
 	    	$names = "SET NAMES 'utf8' COLLATE '$collation'";
-	    	mysqli_query($this->database,$names);
+	    	mysqli_query($this->database, $names);
 		}
-	    mysqli_set_charset ($this->database , "utf8" );
+	    mysqli_set_charset($this->database, "utf8");
 
-		if($this->checkError('Could Not Connect', $dieOnError))
+		if ($this->checkError('Could Not Connect', $dieOnError))
 			$GLOBALS['log']->info("connected to db");
 
 		$this->connectOptions = $configOptions;
@@ -331,13 +331,13 @@ 
 	 */
 	public function lastDbError()
 	{
-		if($this->database) {
-		    if(mysqli_errno($this->database)) {
+		if ($this->database) {
+		    if (mysqli_errno($this->database)) {
 			    return "MySQL error ".mysqli_errno($this->database).": ".mysqli_error($this->database);
 		    }
 		} else {
-			$err =  mysqli_connect_error();
-			if($err) {
+			$err = mysqli_connect_error();
+			if ($err) {
 			    return $err;
 			}
 		}
@@ -349,7 +349,7 @@ 
 	{
 		$charsets = $this->getCharsetInfo();
 		$charset_str = array();
-		foreach($charsets as $name => $value) {
+		foreach ($charsets as $name => $value) {
 			$charset_str[] = "$name = $value";
 		}
 		return array(