adamjakab / SuiteCRM

include/generic/SugarWidgets/SugarWidgetFieldint.php

Indentation +39 added lines, -39 removed lines

@@ -42,59 +42,59 @@
 
 class SugarWidgetFieldInt extends SugarWidgetReportField
 {
- function displayList($layout_def)
- {
+    function displayList($layout_def)
+    {
 
- 	return $this->displayListPlain($layout_def);
- }
+        return $this->displayListPlain($layout_def);
+    }
 
- function queryFilterEquals(&$layout_def)
- {
+    function queryFilterEquals(&$layout_def)
+    {
                 return $this->_get_column_select($layout_def)."= '".$GLOBALS['db']->quote($layout_def['input_name0'])."'\n";
- }
+    }
 
- function queryFilterNot_Equals(&$layout_def)
- {
+    function queryFilterNot_Equals(&$layout_def)
+    {
                 return $this->_get_column_select($layout_def)."!='".$GLOBALS['db']->quote($layout_def['input_name0'])."'\n";
- }
+    }
 
- function queryFilterGreater(&$layout_def)
- {
+    function queryFilterGreater(&$layout_def)
+    {
                 return $this->_get_column_select($layout_def)." > '".$GLOBALS['db']->quote($layout_def['input_name0'])."'\n";
- }
+    }
 
- function queryFilterLess(&$layout_def)
- {
+    function queryFilterLess(&$layout_def)
+    {
                 return $this->_get_column_select($layout_def)." < '".$GLOBALS['db']->quote($layout_def['input_name0'])."'\n";
- }
+    }
 
- function queryFilterBetween(&$layout_def)
- {
- 	             return $this->_get_column_select($layout_def)." BETWEEN '".$GLOBALS['db']->quote($layout_def['input_name0']). "' AND '" . $GLOBALS['db']->quote($layout_def['input_name1']) . "'\n";
- }
+    function queryFilterBetween(&$layout_def)
+    {
+                    return $this->_get_column_select($layout_def)." BETWEEN '".$GLOBALS['db']->quote($layout_def['input_name0']). "' AND '" . $GLOBALS['db']->quote($layout_def['input_name1']) . "'\n";
+    }
 
- function queryFilterStarts_With(&$layout_def)
- {
- 	return $this->queryFilterEquals($layout_def);
- }
+    function queryFilterStarts_With(&$layout_def)
+    {
+        return $this->queryFilterEquals($layout_def);
+    }
 
- function displayInput(&$layout_def)
- {
- 	 return '<input type="text" size="20" value="' . $layout_def['input_name0'] . '" name="' . $layout_def['name'] . '">';
+    function displayInput(&$layout_def)
+    {
+        return '<input type="text" size="20" value="' . $layout_def['input_name0'] . '" name="' . $layout_def['name'] . '">';
 
- }
+    }
  
- function display($layout_def)
- {
-	   //Bug40995
-	   if(isset($obj->layout_manager->defs['reporter']->focus->field_name_map[$layout_def['name']]['precision']))
-	   {
-		   $precision=$obj->layout_manager->defs['reporter']->focus->field_name_map[$layout_def['name']]['precision'];
-		   $layout_def['precision']=$precision;
-	   }
-	   //Bug40995
-       return parent::display($layout_def);
- } 
+    function display($layout_def)
+    {
+        //Bug40995
+        if(isset($obj->layout_manager->defs['reporter']->focus->field_name_map[$layout_def['name']]['precision']))
+        {
+            $precision=$obj->layout_manager->defs['reporter']->focus->field_name_map[$layout_def['name']]['precision'];
+            $layout_def['precision']=$precision;
+        }
+        //Bug40995
+        return parent::display($layout_def);
+    } 
 
 }
 

Spacing +6 added lines, -6 removed lines

@@ -1,5 +1,5 @@ 
 <?php
-if(!defined('sugarEntry') || !sugarEntry) die('Not A Valid Entry Point');
+if (!defined('sugarEntry') || !sugarEntry) die('Not A Valid Entry Point');
 /*********************************************************************************
  * SugarCRM Community Edition is a customer relationship management program developed by
  * SugarCRM, Inc. Copyright (C) 2004-2013 SugarCRM Inc.
@@ -70,7 +70,7 @@ 
 
  function queryFilterBetween(&$layout_def)
  {
- 	             return $this->_get_column_select($layout_def)." BETWEEN '".$GLOBALS['db']->quote($layout_def['input_name0']). "' AND '" . $GLOBALS['db']->quote($layout_def['input_name1']) . "'\n";
+ 	             return $this->_get_column_select($layout_def)." BETWEEN '".$GLOBALS['db']->quote($layout_def['input_name0'])."' AND '".$GLOBALS['db']->quote($layout_def['input_name1'])."'\n";
  }
 
  function queryFilterStarts_With(&$layout_def)
@@ -80,17 +80,17 @@ 
 
  function displayInput(&$layout_def)
  {
- 	 return '<input type="text" size="20" value="' . $layout_def['input_name0'] . '" name="' . $layout_def['name'] . '">';
+ 	 return '<input type="text" size="20" value="'.$layout_def['input_name0'].'" name="'.$layout_def['name'].'">';
 
  }
  
  function display($layout_def)
  {
 	   //Bug40995
-	   if(isset($obj->layout_manager->defs['reporter']->focus->field_name_map[$layout_def['name']]['precision']))
+	   if (isset($obj->layout_manager->defs['reporter']->focus->field_name_map[$layout_def['name']]['precision']))
 	   {
-		   $precision=$obj->layout_manager->defs['reporter']->focus->field_name_map[$layout_def['name']]['precision'];
-		   $layout_def['precision']=$precision;
+		   $precision = $obj->layout_manager->defs['reporter']->focus->field_name_map[$layout_def['name']]['precision'];
+		   $layout_def['precision'] = $precision;
 	   }
 	   //Bug40995
        return parent::display($layout_def);

Braces +3 added lines, -1 removed lines

@@ -1,5 +1,7 @@
 <?php
-if(!defined('sugarEntry') || !sugarEntry) die('Not A Valid Entry Point');
+if(!defined('sugarEntry') || !sugarEntry) {
+    die('Not A Valid Entry Point');
+}
 /*********************************************************************************
  * SugarCRM Community Edition is a customer relationship management program developed by
  * SugarCRM, Inc. Copyright (C) 2004-2013 SugarCRM Inc.

include/generic/SugarWidgets/SugarWidgetSubPanelTopSummaryButton.php

Indentation +29 added lines, -29 removed lines

@@ -45,38 +45,38 @@
 
 class SugarWidgetSubPanelTopSummaryButton extends SugarWidgetSubPanelTopButton
 {
-	function display($widget_data, $additionalFormFields = NULL, $nonbutton = false)
-	{
+    function display($widget_data, $additionalFormFields = NULL, $nonbutton = false)
+    {
 		
 		
-		global $app_strings;
-		global $currentModule;
+        global $app_strings;
+        global $currentModule;
 
-		$popup_request_data = array(
-			'call_back_function' => 'set_return',
-			'form_name' => 'EditView',
-			'field_to_name_array' => array(),
-		);
+        $popup_request_data = array(
+            'call_back_function' => 'set_return',
+            'form_name' => 'EditView',
+            'field_to_name_array' => array(),
+        );
 
-		$json_encoded_php_array = $this->_create_json_encoded_popup_request($popup_request_data);
-		$title = $app_strings['LBL_ACCUMULATED_HISTORY_BUTTON_TITLE'];
-		//$accesskey = $app_strings['LBL_ACCUMULATED_HISTORY_BUTTON_KEY'];
-		$value = $app_strings['LBL_ACCUMULATED_HISTORY_BUTTON_LABEL'];
-		$module_name = 'Activities';
-		$id = $widget_data['focus']->id;
-		$initial_filter = "&record=$id&module_name=$currentModule";
-		if(ACLController::moduleSupportsACL($widget_data['module']) && !ACLController::checkAccess($widget_data['module'], 'detail', true)){
-			$temp =  '<input disabled type="button" name="summary_button" id="summary_button"'
-			. ' class="button"'
-			. ' title="' . $title . '"'
-			. ' value="' . $value . '"';
-			return $temp;
-		}
-		return '<input type="button" name="summary_button" id="summary_button"'
-			. ' class="button"'
-			. ' title="' . $title . '"'
-			. ' value="' . $value . '"'
-			. " onclick='open_popup(\"$module_name\",600,400,\"$initial_filter\",false,false,$json_encoded_php_array);' />\n";
-	}
+        $json_encoded_php_array = $this->_create_json_encoded_popup_request($popup_request_data);
+        $title = $app_strings['LBL_ACCUMULATED_HISTORY_BUTTON_TITLE'];
+        //$accesskey = $app_strings['LBL_ACCUMULATED_HISTORY_BUTTON_KEY'];
+        $value = $app_strings['LBL_ACCUMULATED_HISTORY_BUTTON_LABEL'];
+        $module_name = 'Activities';
+        $id = $widget_data['focus']->id;
+        $initial_filter = "&record=$id&module_name=$currentModule";
+        if(ACLController::moduleSupportsACL($widget_data['module']) && !ACLController::checkAccess($widget_data['module'], 'detail', true)){
+            $temp =  '<input disabled type="button" name="summary_button" id="summary_button"'
+            . ' class="button"'
+            . ' title="' . $title . '"'
+            . ' value="' . $value . '"';
+            return $temp;
+        }
+        return '<input type="button" name="summary_button" id="summary_button"'
+            . ' class="button"'
+            . ' title="' . $title . '"'
+            . ' value="' . $value . '"'
+            . " onclick='open_popup(\"$module_name\",600,400,\"$initial_filter\",false,false,$json_encoded_php_array);' />\n";
+    }
 }
 ?>
\ No newline at end of file

Spacing +7 added lines, -7 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.
@@ -65,17 +65,17 @@ 
 		$module_name = 'Activities';
 		$id = $widget_data['focus']->id;
 		$initial_filter = "&record=$id&module_name=$currentModule";
-		if(ACLController::moduleSupportsACL($widget_data['module']) && !ACLController::checkAccess($widget_data['module'], 'detail', true)){
-			$temp =  '<input disabled type="button" name="summary_button" id="summary_button"'
+		if (ACLController::moduleSupportsACL($widget_data['module']) && !ACLController::checkAccess($widget_data['module'], 'detail', true)) {
+			$temp = '<input disabled type="button" name="summary_button" id="summary_button"'
 			. ' class="button"'
-			. ' title="' . $title . '"'
-			. ' value="' . $value . '"';
+			. ' title="'.$title.'"'
+			. ' value="'.$value.'"';
 			return $temp;
 		}
 		return '<input type="button" name="summary_button" id="summary_button"'
 			. ' class="button"'
-			. ' title="' . $title . '"'
-			. ' value="' . $value . '"'
+			. ' title="'.$title.'"'
+			. ' value="'.$value.'"'
 			. " onclick='open_popup(\"$module_name\",600,400,\"$initial_filter\",false,false,$json_encoded_php_array);' />\n";
 	}
 }

Braces +3 added lines, -1 removed lines

@@ -1,5 +1,7 @@
 <?php
-if(!defined('sugarEntry') || !sugarEntry) die('Not A Valid Entry Point');
+if(!defined('sugarEntry') || !sugarEntry) {
+    die('Not A Valid Entry Point');
+}
 /*********************************************************************************
  * SugarCRM Community Edition is a customer relationship management program developed by
  * SugarCRM, Inc. Copyright (C) 2004-2013 SugarCRM Inc.

include/generic/SugarWidgets/SugarWidgetField.php

Indentation +139 added lines, -139 removed lines

@@ -1,6 +1,6 @@ 
 <?php
 if(!defined('sugarEntry') || !sugarEntry)
-	die('Not A Valid Entry Point');
+    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.
@@ -46,166 +46,166 @@ 
 
 class SugarWidgetField extends SugarWidget {
 
-	function SugarWidgetField(&$layout_manager) {
+    function SugarWidgetField(&$layout_manager) {
         parent::SugarWidget($layout_manager);
     }
 
-	function display($layout_def) {
-		//print $layout_def['start_link_wrapper']."===";
-		$context = $this->layout_manager->getAttribute('context'); //_ppd($context);
-		$func_name = 'display'.$context;
+    function display($layout_def) {
+        //print $layout_def['start_link_wrapper']."===";
+        $context = $this->layout_manager->getAttribute('context'); //_ppd($context);
+        $func_name = 'display'.$context;
 
-		if (!empty ($context) && method_exists($this, $func_name)) {
-			return $this-> $func_name ($layout_def);
-		} else {
-			return 'display not found:'.$func_name;
-		}
-	}
+        if (!empty ($context) && method_exists($this, $func_name)) {
+            return $this-> $func_name ($layout_def);
+        } else {
+            return 'display not found:'.$func_name;
+        }
+    }
 
-	function _get_column_alias($layout_def) {
-		$alias_arr = array ();
+    function _get_column_alias($layout_def) {
+        $alias_arr = array ();
 
-		if (!empty ($layout_def['name']) && $layout_def['name'] == 'count') {
-			return 'count';
-		}
+        if (!empty ($layout_def['name']) && $layout_def['name'] == 'count') {
+            return 'count';
+        }
 
-		if (!empty ($layout_def['table_alias'])) {
-			array_push($alias_arr, $layout_def['table_alias']);
-		}
+        if (!empty ($layout_def['table_alias'])) {
+            array_push($alias_arr, $layout_def['table_alias']);
+        }
 
-		if (!empty ($layout_def['name'])) {
-			array_push($alias_arr, $layout_def['name']);
-		}
+        if (!empty ($layout_def['name'])) {
+            array_push($alias_arr, $layout_def['name']);
+        }
 
-		return $this->getTruncatedColumnAlias(implode("_", $alias_arr));
-	}
+        return $this->getTruncatedColumnAlias(implode("_", $alias_arr));
+    }
 
-	function & displayDetailLabel(& $layout_def) {
+    function & displayDetailLabel(& $layout_def) {
 
-		return '';
-	}
+        return '';
+    }
 
-	function & displayDetail($layout_def) {
+    function & displayDetail($layout_def) {
 
         $layout_def = '';
-		return $layout_def;
-	}
+        return $layout_def;
+    }
 	
-	function displayHeaderCellPlain($layout_def) 
-	{
-		if (!empty ($layout_def['label'])) {
-			return $layout_def['label'];
-		}
-		if (!empty ($layout_def['vname'])) {
-			return translate($layout_def['vname'], $this->layout_manager->getAttribute('module_name'));
-		}
-		return '';
-	}
-
-	function displayHeaderCell($layout_def) {
-		$module_name = $this->layout_manager->getAttribute('module_name');
-
-		$this->local_current_module = $_REQUEST['module'];
-		$this->is_dynamic = true;
-		// don't show sort links if name isn't defined
-		if ((empty ($layout_def['name']) || (isset ($layout_def['sortable']) && !$layout_def['sortable']))
+    function displayHeaderCellPlain($layout_def) 
+    {
+        if (!empty ($layout_def['label'])) {
+            return $layout_def['label'];
+        }
+        if (!empty ($layout_def['vname'])) {
+            return translate($layout_def['vname'], $this->layout_manager->getAttribute('module_name'));
+        }
+        return '';
+    }
+
+    function displayHeaderCell($layout_def) {
+        $module_name = $this->layout_manager->getAttribute('module_name');
+
+        $this->local_current_module = $_REQUEST['module'];
+        $this->is_dynamic = true;
+        // don't show sort links if name isn't defined
+        if ((empty ($layout_def['name']) || (isset ($layout_def['sortable']) && !$layout_def['sortable']))
         && !empty ($layout_def['label'])) {
-			return $layout_def['label'];
-		}
-		if (isset ($layout_def['sortable']) && !$layout_def['sortable']) {
-			return $this->displayHeaderCellPlain($layout_def);
-		}
-
-		$header_cell_text = $this->displayHeaderCellPlain($layout_def);
-
-		$subpanel_module = $layout_def['subpanel_module'];
-		$html_var = $subpanel_module . "_CELL";
-		if (empty ($this->base_URL)) {
-			$objListView = new ListView();
-			$this->base_URL = $objListView -> getBaseURL($html_var);
-			$split_url = explode('&to_pdf=true&action=SubPanelViewer&subpanel=', $this->base_URL);
-			$this->base_URL = $split_url[0];
-			$this->base_URL .= '&inline=true&to_pdf=true&action=SubPanelViewer&subpanel=';
-		}
-		$sort_by_name = $layout_def['name'];
-		if (isset ($layout_def['sort_by'])) {
-			$sort_by_name = $layout_def['sort_by'];
-		}
-
-		$objListView = new ListView();
-		$sort_by = $objListView->getSessionVariableName($html_var, "ORDER_BY").'='.$sort_by_name;
-
-		$start = (empty ($layout_def['start_link_wrapper'])) ? '' : $layout_def['start_link_wrapper'];
-		$end = (empty ($layout_def['end_link_wrapper'])) ? '' : $layout_def['end_link_wrapper'];
-
-		$header_cell = "<a class=\"listViewThLinkS1\" href=\"".$start.$this->base_URL.$subpanel_module.'&'.$sort_by.$end."\">";
-		$header_cell .= $header_cell_text;
-
-		$imgArrow = '';
-
-		if (isset ($layout_def['sort'])) {
-			$imgArrow = $layout_def['sort'];
-		}
-		$arrow_start = $objListView->getArrowUpDownStart($imgArrow);
-		$arrow_end = $objListView->getArrowUpDownEnd($imgArrow);
-		$header_cell .= " ".$arrow_start.$arrow_end."</a>";
-
-		return $header_cell;
-
-	}
-
-	function displayList($layout_def) {
-		return $this->displayListPlain($layout_def);
-	}
-
-	function displayListPlain($layout_def) {
-		$value= $this->_get_list_value($layout_def);
-		if (isset($layout_def['widget_type']) && $layout_def['widget_type'] =='checkbox') {
-			if ($value != '' &&  ($value == 'on' || intval($value) == 1 || $value == 'yes'))
-			{
-				return "<input name='checkbox_display' class='checkbox' type='checkbox' disabled='true' checked>";
-			}
-			return "<input name='checkbox_display' class='checkbox' type='checkbox' disabled='true'>";
-		}
-		return $value;
-	}
-
-	function _get_list_value(& $layout_def) 
-	{
-		$key = '';
-		if ( isset($layout_def['varname']) ) {
-		    $key = strtoupper($layout_def['varname']);
-		} 
-		else {
-			$key = strtoupper($this->_get_column_alias($layout_def));
-		}
-
-		if ( isset($layout_def['fields'][$key]) ) {
-			return $layout_def['fields'][$key];
-		}
+            return $layout_def['label'];
+        }
+        if (isset ($layout_def['sortable']) && !$layout_def['sortable']) {
+            return $this->displayHeaderCellPlain($layout_def);
+        }
+
+        $header_cell_text = $this->displayHeaderCellPlain($layout_def);
+
+        $subpanel_module = $layout_def['subpanel_module'];
+        $html_var = $subpanel_module . "_CELL";
+        if (empty ($this->base_URL)) {
+            $objListView = new ListView();
+            $this->base_URL = $objListView -> getBaseURL($html_var);
+            $split_url = explode('&to_pdf=true&action=SubPanelViewer&subpanel=', $this->base_URL);
+            $this->base_URL = $split_url[0];
+            $this->base_URL .= '&inline=true&to_pdf=true&action=SubPanelViewer&subpanel=';
+        }
+        $sort_by_name = $layout_def['name'];
+        if (isset ($layout_def['sort_by'])) {
+            $sort_by_name = $layout_def['sort_by'];
+        }
+
+        $objListView = new ListView();
+        $sort_by = $objListView->getSessionVariableName($html_var, "ORDER_BY").'='.$sort_by_name;
+
+        $start = (empty ($layout_def['start_link_wrapper'])) ? '' : $layout_def['start_link_wrapper'];
+        $end = (empty ($layout_def['end_link_wrapper'])) ? '' : $layout_def['end_link_wrapper'];
+
+        $header_cell = "<a class=\"listViewThLinkS1\" href=\"".$start.$this->base_URL.$subpanel_module.'&'.$sort_by.$end."\">";
+        $header_cell .= $header_cell_text;
+
+        $imgArrow = '';
+
+        if (isset ($layout_def['sort'])) {
+            $imgArrow = $layout_def['sort'];
+        }
+        $arrow_start = $objListView->getArrowUpDownStart($imgArrow);
+        $arrow_end = $objListView->getArrowUpDownEnd($imgArrow);
+        $header_cell .= " ".$arrow_start.$arrow_end."</a>";
+
+        return $header_cell;
+
+    }
+
+    function displayList($layout_def) {
+        return $this->displayListPlain($layout_def);
+    }
+
+    function displayListPlain($layout_def) {
+        $value= $this->_get_list_value($layout_def);
+        if (isset($layout_def['widget_type']) && $layout_def['widget_type'] =='checkbox') {
+            if ($value != '' &&  ($value == 'on' || intval($value) == 1 || $value == 'yes'))
+            {
+                return "<input name='checkbox_display' class='checkbox' type='checkbox' disabled='true' checked>";
+            }
+            return "<input name='checkbox_display' class='checkbox' type='checkbox' disabled='true'>";
+        }
+        return $value;
+    }
+
+    function _get_list_value(& $layout_def) 
+    {
+        $key = '';
+        if ( isset($layout_def['varname']) ) {
+            $key = strtoupper($layout_def['varname']);
+        } 
+        else {
+            $key = strtoupper($this->_get_column_alias($layout_def));
+        }
+
+        if ( isset($layout_def['fields'][$key]) ) {
+            return $layout_def['fields'][$key];
+        }
 		
-		return '';
-	}
+        return '';
+    }
 
-	function & displayEditLabel($layout_def) {
-		return '';
-	}
+    function & displayEditLabel($layout_def) {
+        return '';
+    }
 
-	function & displayEdit($layout_def) {
-		return '';
-	}
+    function & displayEdit($layout_def) {
+        return '';
+    }
 
-	function & displaySearchLabel($layout_def) {
-		return '';
-	}
+    function & displaySearchLabel($layout_def) {
+        return '';
+    }
 
-	function & displaySearch($layout_def) {
-		return '';
-	}
+    function & displaySearch($layout_def) {
+        return '';
+    }
 
-	function displayInput($layout_def) {
-		return ' -- Not Implemented --';
-	}
+    function displayInput($layout_def) {
+        return ' -- Not Implemented --';
+    }
 
     function getVardef($layout_def) {
         $myName = $layout_def['column_key'];

Spacing +10 added lines, -10 removed lines

@@ -1,5 +1,5 @@ 
 <?php
-if(!defined('sugarEntry') || !sugarEntry)
+if (!defined('sugarEntry') || !sugarEntry)
 	die('Not A Valid Entry Point');
 /*********************************************************************************
  * SugarCRM Community Edition is a customer relationship management program developed by
@@ -56,14 +56,14 @@ 
 		$func_name = 'display'.$context;
 
 		if (!empty ($context) && method_exists($this, $func_name)) {
-			return $this-> $func_name ($layout_def);
+			return $this-> $func_name($layout_def);
 		} else {
 			return 'display not found:'.$func_name;
 		}
 	}
 
 	function _get_column_alias($layout_def) {
-		$alias_arr = array ();
+		$alias_arr = array();
 
 		if (!empty ($layout_def['name']) && $layout_def['name'] == 'count') {
 			return 'count';
@@ -119,7 +119,7 @@ 
 		$header_cell_text = $this->displayHeaderCellPlain($layout_def);
 
 		$subpanel_module = $layout_def['subpanel_module'];
-		$html_var = $subpanel_module . "_CELL";
+		$html_var = $subpanel_module."_CELL";
 		if (empty ($this->base_URL)) {
 			$objListView = new ListView();
 			$this->base_URL = $objListView -> getBaseURL($html_var);
@@ -159,9 +159,9 @@ 
 	}
 
 	function displayListPlain($layout_def) {
-		$value= $this->_get_list_value($layout_def);
-		if (isset($layout_def['widget_type']) && $layout_def['widget_type'] =='checkbox') {
-			if ($value != '' &&  ($value == 'on' || intval($value) == 1 || $value == 'yes'))
+		$value = $this->_get_list_value($layout_def);
+		if (isset($layout_def['widget_type']) && $layout_def['widget_type'] == 'checkbox') {
+			if ($value != '' && ($value == 'on' || intval($value) == 1 || $value == 'yes'))
 			{
 				return "<input name='checkbox_display' class='checkbox' type='checkbox' disabled='true' checked>";
 			}
@@ -173,14 +173,14 @@ 
 	function _get_list_value(& $layout_def) 
 	{
 		$key = '';
-		if ( isset($layout_def['varname']) ) {
+		if (isset($layout_def['varname'])) {
 		    $key = strtoupper($layout_def['varname']);
 		} 
 		else {
 			$key = strtoupper($this->_get_column_alias($layout_def));
 		}
 
-		if ( isset($layout_def['fields'][$key]) ) {
+		if (isset($layout_def['fields'][$key])) {
 			return $layout_def['fields'][$key];
 		}
 		
@@ -211,7 +211,7 @@ 
         $myName = $layout_def['column_key'];
         $vardef = $this->layout_manager->defs['reporter']->all_fields[$myName];
 
-        if ( !isset($vardef) ) {
+        if (!isset($vardef)) {
             // No vardef, return an empty array
             return array();
         } else {

Braces +3 added lines, -3 removed lines

@@ -1,6 +1,7 @@ 
 <?php
-if(!defined('sugarEntry') || !sugarEntry)
+if(!defined('sugarEntry') || !sugarEntry) {
 	die('Not A Valid Entry Point');
+}
 /*********************************************************************************
  * SugarCRM Community Edition is a customer relationship management program developed by
  * SugarCRM, Inc. Copyright (C) 2004-2013 SugarCRM Inc.
@@ -175,8 +176,7 @@ 
 		$key = '';
 		if ( isset($layout_def['varname']) ) {
 		    $key = strtoupper($layout_def['varname']);
-		} 
-		else {
+		} else {
 			$key = strtoupper($this->_get_column_alias($layout_def));
 		}
 

Doc Comments +3 added lines

@@ -62,6 +62,9 @@
 		}
 	}
 
+	/**
+	 * @return string
+	 */
 	function _get_column_alias($layout_def) {
 		$alias_arr = array ();
 

include/generic/SugarWidgets/SugarWidgetFielddatepicker.php

Indentation +1 added lines, -1 removed lines

@@ -43,7 +43,7 @@
 
 class SugarWidgetFieldDatePicker extends SugarWidgetFieldDateTime
 {
-	function displayInput($layout_def)
+    function displayInput($layout_def)
     {
         global $timedate;
 

Spacing +1 added lines, -1 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.

Braces +3 added lines, -1 removed lines

@@ -1,5 +1,7 @@
 <?php
-if(!defined('sugarEntry') || !sugarEntry) die('Not A Valid Entry Point');
+if(!defined('sugarEntry') || !sugarEntry) {
+    die('Not A Valid Entry Point');
+}
 /*********************************************************************************
  * SugarCRM Community Edition is a customer relationship management program developed by
  * SugarCRM, Inc. Copyright (C) 2004-2013 SugarCRM Inc.

include/generic/SugarWidgets/SugarWidgetSubPanelEmailLink.php

Indentation +58 added lines, -58 removed lines

@@ -45,62 +45,62 @@ 
 
 class SugarWidgetSubPanelEmailLink extends SugarWidgetField {
 
-	function displayList($layout_def) {
-		global $current_user;
-		global $beanList;
-		global $focus;
-		global $sugar_config;
-		global $locale;
-
-		if(isset($layout_def['varname'])) {
-			$key = strtoupper($layout_def['varname']);
-		} else {
-			$key = $this->_get_column_alias($layout_def);
-			$key = strtoupper($key);
-		}
-		$value = $layout_def['fields'][$key];
-
-
-
-			if(isset($_REQUEST['action'])) $action = $_REQUEST['action'];
-			else $action = '';
-
-			if(isset($_REQUEST['module'])) $module = $_REQUEST['module'];
-			else $module = '';
-
-			if(isset($_REQUEST['record'])) $record = $_REQUEST['record'];
-			else $record = '';
-
-			if (!empty($focus->name)) {
-				$name = $focus->name;
-			} else {
-				if( !empty($focus->first_name) && !empty($focus->last_name)) {
-					$name = $locale->getLocaleFormattedName($focus->first_name, $focus->last_name);
-					}
-				if(empty($name)) {
-					$name = '*';
-				}
-			}
-
-			$userPref = $current_user->getPreference('email_link_type');
-			$defaultPref = $sugar_config['email_default_client'];
-			if($userPref != '') {
-				$client = $userPref;
-			} else {
-				$client = $defaultPref;
-			}
-
-			if($client == 'sugar')
-			{
-			    $composeData = array(
-			        'load_id' => $layout_def['fields']['ID'],
+    function displayList($layout_def) {
+        global $current_user;
+        global $beanList;
+        global $focus;
+        global $sugar_config;
+        global $locale;
+
+        if(isset($layout_def['varname'])) {
+            $key = strtoupper($layout_def['varname']);
+        } else {
+            $key = $this->_get_column_alias($layout_def);
+            $key = strtoupper($key);
+        }
+        $value = $layout_def['fields'][$key];
+
+
+
+            if(isset($_REQUEST['action'])) $action = $_REQUEST['action'];
+            else $action = '';
+
+            if(isset($_REQUEST['module'])) $module = $_REQUEST['module'];
+            else $module = '';
+
+            if(isset($_REQUEST['record'])) $record = $_REQUEST['record'];
+            else $record = '';
+
+            if (!empty($focus->name)) {
+                $name = $focus->name;
+            } else {
+                if( !empty($focus->first_name) && !empty($focus->last_name)) {
+                    $name = $locale->getLocaleFormattedName($focus->first_name, $focus->last_name);
+                    }
+                if(empty($name)) {
+                    $name = '*';
+                }
+            }
+
+            $userPref = $current_user->getPreference('email_link_type');
+            $defaultPref = $sugar_config['email_default_client'];
+            if($userPref != '') {
+                $client = $userPref;
+            } else {
+                $client = $defaultPref;
+            }
+
+            if($client == 'sugar')
+            {
+                $composeData = array(
+                    'load_id' => $layout_def['fields']['ID'],
                     'load_module' => $this->layout_manager->defs['module_name'],
                     'parent_type' => $this->layout_manager->defs['module_name'],
                     'parent_id' => $layout_def['fields']['ID'],
-			        'return_module' => $module,
-			        'return_action' => $action,
-			        'return_id' => $record
-			    );
+                    'return_module' => $module,
+                    'return_action' => $action,
+                    'return_id' => $record
+                );
                 if(isset($layout_def['fields']['FULL_NAME'])){
                     $composeData['parent_name'] = $layout_def['fields']['FULL_NAME'];
                     $composeData['to_email_addrs'] = sprintf("%s <%s>", $layout_def['fields']['FULL_NAME'], $layout_def['fields']['EMAIL1']);
@@ -112,11 +112,11 @@ 
                 $j_quickComposeOptions = $eUi->generateComposePackageForQuickCreate($composeData, http_build_query($composeData), true);
 
                 $link = "<a href='javascript:void(0);' onclick='SUGAR.quickCompose.init($j_quickComposeOptions);'>";
-			} else {
-				$link = '<a href="mailto:' . $value .'" >';
-			}
+            } else {
+                $link = '<a href="mailto:' . $value .'" >';
+            }
 
-			return $link.$value.'</a>';
+            return $link.$value.'</a>';
 
-	}
+    }
 } // end class def

Spacing +11 added lines, -11 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.
@@ -52,7 +52,7 @@ 
 		global $sugar_config;
 		global $locale;
 
-		if(isset($layout_def['varname'])) {
+		if (isset($layout_def['varname'])) {
 			$key = strtoupper($layout_def['varname']);
 		} else {
 			$key = $this->_get_column_alias($layout_def);
@@ -62,35 +62,35 @@ 
 
 
 
-			if(isset($_REQUEST['action'])) $action = $_REQUEST['action'];
+			if (isset($_REQUEST['action'])) $action = $_REQUEST['action'];
 			else $action = '';
 
-			if(isset($_REQUEST['module'])) $module = $_REQUEST['module'];
+			if (isset($_REQUEST['module'])) $module = $_REQUEST['module'];
 			else $module = '';
 
-			if(isset($_REQUEST['record'])) $record = $_REQUEST['record'];
+			if (isset($_REQUEST['record'])) $record = $_REQUEST['record'];
 			else $record = '';
 
 			if (!empty($focus->name)) {
 				$name = $focus->name;
 			} else {
-				if( !empty($focus->first_name) && !empty($focus->last_name)) {
+				if (!empty($focus->first_name) && !empty($focus->last_name)) {
 					$name = $locale->getLocaleFormattedName($focus->first_name, $focus->last_name);
 					}
-				if(empty($name)) {
+				if (empty($name)) {
 					$name = '*';
 				}
 			}
 
 			$userPref = $current_user->getPreference('email_link_type');
 			$defaultPref = $sugar_config['email_default_client'];
-			if($userPref != '') {
+			if ($userPref != '') {
 				$client = $userPref;
 			} else {
 				$client = $defaultPref;
 			}
 
-			if($client == 'sugar')
+			if ($client == 'sugar')
 			{
 			    $composeData = array(
 			        'load_id' => $layout_def['fields']['ID'],
@@ -101,7 +101,7 @@ 
 			        'return_action' => $action,
 			        'return_id' => $record
 			    );
-                if(isset($layout_def['fields']['FULL_NAME'])){
+                if (isset($layout_def['fields']['FULL_NAME'])) {
                     $composeData['parent_name'] = $layout_def['fields']['FULL_NAME'];
                     $composeData['to_email_addrs'] = sprintf("%s <%s>", $layout_def['fields']['FULL_NAME'], $layout_def['fields']['EMAIL1']);
                 } else {
@@ -113,7 +113,7 @@ 
 
                 $link = "<a href='javascript:void(0);' onclick='SUGAR.quickCompose.init($j_quickComposeOptions);'>";
 			} else {
-				$link = '<a href="mailto:' . $value .'" >';
+				$link = '<a href="mailto:'.$value.'" >';
 			}
 
 			return $link.$value.'</a>';

Braces +18 added lines, -7 removed lines

@@ -1,5 +1,7 @@ 
 <?php
-if(!defined('sugarEntry') || !sugarEntry) die('Not A Valid Entry Point');
+if(!defined('sugarEntry') || !sugarEntry) {
+    die('Not A Valid Entry Point');
+}
 /*********************************************************************************
  * SugarCRM Community Edition is a customer relationship management program developed by
  * SugarCRM, Inc. Copyright (C) 2004-2013 SugarCRM Inc.
@@ -62,14 +64,23 @@ 
 
 
 
-			if(isset($_REQUEST['action'])) $action = $_REQUEST['action'];
-			else $action = '';
+			if(isset($_REQUEST['action'])) {
+			    $action = $_REQUEST['action'];
+			} else {
+			    $action = '';
+			}
 
-			if(isset($_REQUEST['module'])) $module = $_REQUEST['module'];
-			else $module = '';
+			if(isset($_REQUEST['module'])) {
+			    $module = $_REQUEST['module'];
+			} else {
+			    $module = '';
+			}
 
-			if(isset($_REQUEST['record'])) $record = $_REQUEST['record'];
-			else $record = '';
+			if(isset($_REQUEST['record'])) {
+			    $record = $_REQUEST['record'];
+			} else {
+			    $record = '';
+			}
 
 			if (!empty($focus->name)) {
 				$name = $focus->name;

include/generic/SugarWidgets/SugarWidgetSubPanelTopFilterButton.php

Spacing +3 added lines, -3 removed lines

@@ -1,5 +1,5 @@ 
 <?php
-if(!defined('sugarEntry') || !sugarEntry) die('Not A Valid Entry Point');
+if (!defined('sugarEntry') || !sugarEntry) die('Not A Valid Entry Point');
 /*********************************************************************************
  * SugarCRM Community Edition is a customer relationship management program developed by
  * SugarCRM, Inc. Copyright (C) 2004-2013 SugarCRM Inc.
@@ -38,7 +38,7 @@ 
  * display the words  "Powered by SugarCRM" and "Supercharged by SuiteCRM".
  ********************************************************************************/
 
-class SugarWidgetSubPanelTopFilterButton  extends SugarWidgetSubPanelTopButton{
+class SugarWidgetSubPanelTopFilterButton  extends SugarWidgetSubPanelTopButton {
 
     function display($defines, $additionalFormFields = NULL, $nonbutton = false)
     {
@@ -46,7 +46,7 @@ 
 
         $button = "<script src='include/SubPanel/SubPanel.js'></script>";
 
-        $button .= "<input class='button' type='button'  value='".$app_strings['LBL_SUBPANEL_FILTER_LABEL']."'  id='". $this->getWidgetId() ."'  name='".$app_strings['LBL_SUBPANEL_FILTER_LABEL']."'  title='".$app_strings['LBL_SUBPANEL_FILTER_LABEL']."' onclick=\"showSearchPanel('history');return false;\" />";
+        $button .= "<input class='button' type='button'  value='".$app_strings['LBL_SUBPANEL_FILTER_LABEL']."'  id='".$this->getWidgetId()."'  name='".$app_strings['LBL_SUBPANEL_FILTER_LABEL']."'  title='".$app_strings['LBL_SUBPANEL_FILTER_LABEL']."' onclick=\"showSearchPanel('history');return false;\" />";
 
         return $button;
     }

Braces +3 added lines, -1 removed lines

@@ -1,5 +1,7 @@
 <?php
-if(!defined('sugarEntry') || !sugarEntry) die('Not A Valid Entry Point');
+if(!defined('sugarEntry') || !sugarEntry) {
+    die('Not A Valid Entry Point');
+}
 /*********************************************************************************
  * SugarCRM Community Edition is a customer relationship management program developed by
  * SugarCRM, Inc. Copyright (C) 2004-2013 SugarCRM Inc.

include/generic/SugarWidgets/SugarWidgetSubPanelTopCreateTaskButton.php

Indentation +114 added lines, -114 removed lines

@@ -45,121 +45,121 @@
 
 class SugarWidgetSubPanelTopCreateTaskButton extends SugarWidgetSubPanelTopButtonQuickCreate
 {
-	function &_get_form($defines, $additionalFormFields = null, $asUrl = false)
-	{
-		global $app_strings;
-		global $currentModule;
-
-		$this->module="Tasks";
-		$this->subpanelDiv = "activities";
-
-		// Create the additional form fields with real values if they were not passed in
-		if(empty($additionalFormFields) && $this->additional_form_fields)
-		{
-			foreach($this->additional_form_fields as $key=>$value)
-			{
-				if(!empty($defines['focus']->$value))
-				{
-					$additionalFormFields[$key] = $defines['focus']->$value;
-				}
-				else
-				{
-					$additionalFormFields[$key] = '';
-				}
-			}
-		}
-
-		if(!empty($this->module))
-		{
-			$defines['child_module_name'] = $this->module;
-		}
-		else
-		{
-			$defines['child_module_name'] = $defines['module'];
-		}
-
-		if(empty($defines['subpanelDiv']))
-		{
-			$defines['subpanelDiv'] = $this->subpanelDiv;
-		}
-
-		$defines['parent_bean_name'] = get_class( $defines['focus']);
-
-		$form = 'form' . $defines['child_module_name'];
-		$button = '<form onsubmit="return SUGAR.subpanelUtils.sendAndRetrieve(this.id, \'subpanel_' . strtolower($defines['subpanelDiv']) . '\', \'' . addslashes($app_strings['LBL_LOADING']) . '\');" action="index.php" method="post" name="form" id="form' . $form . "\">\n";
-
-		//module_button is used to override the value of module name
-		$button .= "<input type='hidden' name='target_module' value='".$defines['child_module_name']."'>\n";
-		$button .= "<input type='hidden' name='".strtolower($defines['parent_bean_name'])."_id' value='".$defines['focus']->id."'>\n";
-
-		if(isset($defines['focus']->name))
-		{
-			$button .= "<input type='hidden' name='".strtolower($defines['parent_bean_name'])."_name' value='".$defines['focus']->name."'>";
-		}
-
-		$button .= '<input type="hidden" name="to_pdf" value="true" />';
+    function &_get_form($defines, $additionalFormFields = null, $asUrl = false)
+    {
+        global $app_strings;
+        global $currentModule;
+
+        $this->module="Tasks";
+        $this->subpanelDiv = "activities";
+
+        // Create the additional form fields with real values if they were not passed in
+        if(empty($additionalFormFields) && $this->additional_form_fields)
+        {
+            foreach($this->additional_form_fields as $key=>$value)
+            {
+                if(!empty($defines['focus']->$value))
+                {
+                    $additionalFormFields[$key] = $defines['focus']->$value;
+                }
+                else
+                {
+                    $additionalFormFields[$key] = '';
+                }
+            }
+        }
+
+        if(!empty($this->module))
+        {
+            $defines['child_module_name'] = $this->module;
+        }
+        else
+        {
+            $defines['child_module_name'] = $defines['module'];
+        }
+
+        if(empty($defines['subpanelDiv']))
+        {
+            $defines['subpanelDiv'] = $this->subpanelDiv;
+        }
+
+        $defines['parent_bean_name'] = get_class( $defines['focus']);
+
+        $form = 'form' . $defines['child_module_name'];
+        $button = '<form onsubmit="return SUGAR.subpanelUtils.sendAndRetrieve(this.id, \'subpanel_' . strtolower($defines['subpanelDiv']) . '\', \'' . addslashes($app_strings['LBL_LOADING']) . '\');" action="index.php" method="post" name="form" id="form' . $form . "\">\n";
+
+        //module_button is used to override the value of module name
+        $button .= "<input type='hidden' name='target_module' value='".$defines['child_module_name']."'>\n";
+        $button .= "<input type='hidden' name='".strtolower($defines['parent_bean_name'])."_id' value='".$defines['focus']->id."'>\n";
+
+        if(isset($defines['focus']->name))
+        {
+            $button .= "<input type='hidden' name='".strtolower($defines['parent_bean_name'])."_name' value='".$defines['focus']->name."'>";
+        }
+
+        $button .= '<input type="hidden" name="to_pdf" value="true" />';
         $button .= '<input type="hidden" name="tpl" value="QuickCreate.tpl" />';
-		$button .= '<input type="hidden" name="return_module" value="' . $currentModule . "\" />\n";
-		$button .= '<input type="hidden" name="return_action" value="' . $defines['action'] . "\" />\n";
-		$button .= '<input type="hidden" name="return_id" value="' . $defines['focus']->id . "\" />\n";
-		$button .= '<input type="hidden" name="record" value="" />';
-
-		// TODO: move this out and get $additionalFormFields working properly
-		if(empty($additionalFormFields['parent_type']))
-		{
-			if($defines['focus']->object_name=='Contact') {
-				$additionalFormFields['parent_type'] = 'Accounts';
-			}
-			else {
-				$additionalFormFields['parent_type'] = $defines['focus']->module_dir;
-			}
-		}
-		if(empty($additionalFormFields['parent_name']))
-		{
-			if($defines['focus']->object_name=='Contact') {
-				$additionalFormFields['parent_name'] = $defines['focus']->account_name;
-				$additionalFormFields['account_name'] = $defines['focus']->account_name;
-			}
-			else {
-				$additionalFormFields['parent_name'] = $defines['focus']->name;
-			}
-		}
-		if(empty($additionalFormFields['parent_id']))
-		{
-			if($defines['focus']->object_name=='Contact') {
-				$additionalFormFields['parent_id'] = $defines['focus']->account_id;
-				$additionalFormFields['account_id'] = $defines['focus']->account_id;
-			}
-			else {
-				$additionalFormFields['parent_id'] = $defines['focus']->id;
-			}
-		}
-
-		$button .= '<input type="hidden" name="action" value="SubpanelCreates" />' . "\n";
-		$button .= '<input type="hidden" name="module" value="Home" />' . "\n";
-		$button .= '<input type="hidden" name="target_action" value="QuickCreate" />' . "\n";
-
-		// fill in additional form fields for all but action
-		foreach($additionalFormFields as $key => $value)
-		{
-			if($key != 'action')
-			{
-				$button .= '<input type="hidden" name="' . $key . '" value="' . $value . '" />' . "\n";
-			}
-		}
-
-		return $button;
-	}
-
-
-	function display($defines, $additionalFormFields = null, $nonbutton = false)
-	{
-	    $focus = new Task;
-		if ( !$focus->ACLAccess('EditView') ) {
-		    return '';
-	    }
+        $button .= '<input type="hidden" name="return_module" value="' . $currentModule . "\" />\n";
+        $button .= '<input type="hidden" name="return_action" value="' . $defines['action'] . "\" />\n";
+        $button .= '<input type="hidden" name="return_id" value="' . $defines['focus']->id . "\" />\n";
+        $button .= '<input type="hidden" name="record" value="" />';
+
+        // TODO: move this out and get $additionalFormFields working properly
+        if(empty($additionalFormFields['parent_type']))
+        {
+            if($defines['focus']->object_name=='Contact') {
+                $additionalFormFields['parent_type'] = 'Accounts';
+            }
+            else {
+                $additionalFormFields['parent_type'] = $defines['focus']->module_dir;
+            }
+        }
+        if(empty($additionalFormFields['parent_name']))
+        {
+            if($defines['focus']->object_name=='Contact') {
+                $additionalFormFields['parent_name'] = $defines['focus']->account_name;
+                $additionalFormFields['account_name'] = $defines['focus']->account_name;
+            }
+            else {
+                $additionalFormFields['parent_name'] = $defines['focus']->name;
+            }
+        }
+        if(empty($additionalFormFields['parent_id']))
+        {
+            if($defines['focus']->object_name=='Contact') {
+                $additionalFormFields['parent_id'] = $defines['focus']->account_id;
+                $additionalFormFields['account_id'] = $defines['focus']->account_id;
+            }
+            else {
+                $additionalFormFields['parent_id'] = $defines['focus']->id;
+            }
+        }
+
+        $button .= '<input type="hidden" name="action" value="SubpanelCreates" />' . "\n";
+        $button .= '<input type="hidden" name="module" value="Home" />' . "\n";
+        $button .= '<input type="hidden" name="target_action" value="QuickCreate" />' . "\n";
+
+        // fill in additional form fields for all but action
+        foreach($additionalFormFields as $key => $value)
+        {
+            if($key != 'action')
+            {
+                $button .= '<input type="hidden" name="' . $key . '" value="' . $value . '" />' . "\n";
+            }
+        }
+
+        return $button;
+    }
+
+
+    function display($defines, $additionalFormFields = null, $nonbutton = false)
+    {
+        $focus = new Task;
+        if ( !$focus->ACLAccess('EditView') ) {
+            return '';
+        }
 		
-		return parent::display($defines, $additionalFormFields);
-	}
+        return parent::display($defines, $additionalFormFields);
+    }
 }
 ?>

Spacing +27 added lines, -27 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.
@@ -50,15 +50,15 @@ 
 		global $app_strings;
 		global $currentModule;
 
-		$this->module="Tasks";
+		$this->module = "Tasks";
 		$this->subpanelDiv = "activities";
 
 		// Create the additional form fields with real values if they were not passed in
-		if(empty($additionalFormFields) && $this->additional_form_fields)
+		if (empty($additionalFormFields) && $this->additional_form_fields)
 		{
-			foreach($this->additional_form_fields as $key=>$value)
+			foreach ($this->additional_form_fields as $key=>$value)
 			{
-				if(!empty($defines['focus']->$value))
+				if (!empty($defines['focus']->$value))
 				{
 					$additionalFormFields[$key] = $defines['focus']->$value;
 				}
@@ -69,7 +69,7 @@ 
 			}
 		}
 
-		if(!empty($this->module))
+		if (!empty($this->module))
 		{
 			$defines['child_module_name'] = $this->module;
 		}
@@ -78,45 +78,45 @@ 
 			$defines['child_module_name'] = $defines['module'];
 		}
 
-		if(empty($defines['subpanelDiv']))
+		if (empty($defines['subpanelDiv']))
 		{
 			$defines['subpanelDiv'] = $this->subpanelDiv;
 		}
 
-		$defines['parent_bean_name'] = get_class( $defines['focus']);
+		$defines['parent_bean_name'] = get_class($defines['focus']);
 
-		$form = 'form' . $defines['child_module_name'];
-		$button = '<form onsubmit="return SUGAR.subpanelUtils.sendAndRetrieve(this.id, \'subpanel_' . strtolower($defines['subpanelDiv']) . '\', \'' . addslashes($app_strings['LBL_LOADING']) . '\');" action="index.php" method="post" name="form" id="form' . $form . "\">\n";
+		$form = 'form'.$defines['child_module_name'];
+		$button = '<form onsubmit="return SUGAR.subpanelUtils.sendAndRetrieve(this.id, \'subpanel_'.strtolower($defines['subpanelDiv']).'\', \''.addslashes($app_strings['LBL_LOADING']).'\');" action="index.php" method="post" name="form" id="form'.$form."\">\n";
 
 		//module_button is used to override the value of module name
 		$button .= "<input type='hidden' name='target_module' value='".$defines['child_module_name']."'>\n";
 		$button .= "<input type='hidden' name='".strtolower($defines['parent_bean_name'])."_id' value='".$defines['focus']->id."'>\n";
 
-		if(isset($defines['focus']->name))
+		if (isset($defines['focus']->name))
 		{
 			$button .= "<input type='hidden' name='".strtolower($defines['parent_bean_name'])."_name' value='".$defines['focus']->name."'>";
 		}
 
 		$button .= '<input type="hidden" name="to_pdf" value="true" />';
         $button .= '<input type="hidden" name="tpl" value="QuickCreate.tpl" />';
-		$button .= '<input type="hidden" name="return_module" value="' . $currentModule . "\" />\n";
-		$button .= '<input type="hidden" name="return_action" value="' . $defines['action'] . "\" />\n";
-		$button .= '<input type="hidden" name="return_id" value="' . $defines['focus']->id . "\" />\n";
+		$button .= '<input type="hidden" name="return_module" value="'.$currentModule."\" />\n";
+		$button .= '<input type="hidden" name="return_action" value="'.$defines['action']."\" />\n";
+		$button .= '<input type="hidden" name="return_id" value="'.$defines['focus']->id."\" />\n";
 		$button .= '<input type="hidden" name="record" value="" />';
 
 		// TODO: move this out and get $additionalFormFields working properly
-		if(empty($additionalFormFields['parent_type']))
+		if (empty($additionalFormFields['parent_type']))
 		{
-			if($defines['focus']->object_name=='Contact') {
+			if ($defines['focus']->object_name == 'Contact') {
 				$additionalFormFields['parent_type'] = 'Accounts';
 			}
 			else {
 				$additionalFormFields['parent_type'] = $defines['focus']->module_dir;
 			}
 		}
-		if(empty($additionalFormFields['parent_name']))
+		if (empty($additionalFormFields['parent_name']))
 		{
-			if($defines['focus']->object_name=='Contact') {
+			if ($defines['focus']->object_name == 'Contact') {
 				$additionalFormFields['parent_name'] = $defines['focus']->account_name;
 				$additionalFormFields['account_name'] = $defines['focus']->account_name;
 			}
@@ -124,9 +124,9 @@ 
 				$additionalFormFields['parent_name'] = $defines['focus']->name;
 			}
 		}
-		if(empty($additionalFormFields['parent_id']))
+		if (empty($additionalFormFields['parent_id']))
 		{
-			if($defines['focus']->object_name=='Contact') {
+			if ($defines['focus']->object_name == 'Contact') {
 				$additionalFormFields['parent_id'] = $defines['focus']->account_id;
 				$additionalFormFields['account_id'] = $defines['focus']->account_id;
 			}
@@ -135,16 +135,16 @@ 
 			}
 		}
 
-		$button .= '<input type="hidden" name="action" value="SubpanelCreates" />' . "\n";
-		$button .= '<input type="hidden" name="module" value="Home" />' . "\n";
-		$button .= '<input type="hidden" name="target_action" value="QuickCreate" />' . "\n";
+		$button .= '<input type="hidden" name="action" value="SubpanelCreates" />'."\n";
+		$button .= '<input type="hidden" name="module" value="Home" />'."\n";
+		$button .= '<input type="hidden" name="target_action" value="QuickCreate" />'."\n";
 
 		// fill in additional form fields for all but action
-		foreach($additionalFormFields as $key => $value)
+		foreach ($additionalFormFields as $key => $value)
 		{
-			if($key != 'action')
+			if ($key != 'action')
 			{
-				$button .= '<input type="hidden" name="' . $key . '" value="' . $value . '" />' . "\n";
+				$button .= '<input type="hidden" name="'.$key.'" value="'.$value.'" />'."\n";
 			}
 		}
 
@@ -155,7 +155,7 @@ 
 	function display($defines, $additionalFormFields = null, $nonbutton = false)
 	{
 	    $focus = new Task;
-		if ( !$focus->ACLAccess('EditView') ) {
+		if (!$focus->ACLAccess('EditView')) {
 		    return '';
 	    }
 		

Braces +8 added lines, -11 removed lines

@@ -1,5 +1,7 @@ 
 <?php
-if(!defined('sugarEntry') || !sugarEntry) die('Not A Valid Entry Point');
+if(!defined('sugarEntry') || !sugarEntry) {
+    die('Not A Valid Entry Point');
+}
 /*********************************************************************************
  * SugarCRM Community Edition is a customer relationship management program developed by
  * SugarCRM, Inc. Copyright (C) 2004-2013 SugarCRM Inc.
@@ -61,8 +63,7 @@ 
 				if(!empty($defines['focus']->$value))
 				{
 					$additionalFormFields[$key] = $defines['focus']->$value;
-				}
-				else
+				} else
 				{
 					$additionalFormFields[$key] = '';
 				}
@@ -72,8 +73,7 @@ 
 		if(!empty($this->module))
 		{
 			$defines['child_module_name'] = $this->module;
-		}
-		else
+		} else
 		{
 			$defines['child_module_name'] = $defines['module'];
 		}
@@ -109,8 +109,7 @@ 
 		{
 			if($defines['focus']->object_name=='Contact') {
 				$additionalFormFields['parent_type'] = 'Accounts';
-			}
-			else {
+			} else {
 				$additionalFormFields['parent_type'] = $defines['focus']->module_dir;
 			}
 		}
@@ -119,8 +118,7 @@ 
 			if($defines['focus']->object_name=='Contact') {
 				$additionalFormFields['parent_name'] = $defines['focus']->account_name;
 				$additionalFormFields['account_name'] = $defines['focus']->account_name;
-			}
-			else {
+			} else {
 				$additionalFormFields['parent_name'] = $defines['focus']->name;
 			}
 		}
@@ -129,8 +127,7 @@ 
 			if($defines['focus']->object_name=='Contact') {
 				$additionalFormFields['parent_id'] = $defines['focus']->account_id;
 				$additionalFormFields['account_id'] = $defines['focus']->account_id;
-			}
-			else {
+			} else {
 				$additionalFormFields['parent_id'] = $defines['focus']->id;
 			}
 		}

include/generic/SugarWidgets/SugarWidgetSubPanelEditButton.php

Indentation +8 added lines, -8 removed lines

@@ -48,16 +48,16 @@
     protected static $defs = array();
     protected static $edit_icon_html;
 
-	function displayHeaderCell($layout_def)
-	{
-		return '';
-	}
+    function displayHeaderCell($layout_def)
+    {
+        return '';
+    }
 
-	function displayList($layout_def)
-	{
-		global $app_strings;
+    function displayList($layout_def)
+    {
+        global $app_strings;
         global $subpanel_item_count;
-		$unique_id = $layout_def['subpanel_id']."_edit_".$subpanel_item_count; //bug 51512
+        $unique_id = $layout_def['subpanel_id']."_edit_".$subpanel_item_count; //bug 51512
 
         if ($layout_def['EditView']) {
 

Spacing +5 added lines, -5 removed lines

@@ -1,5 +1,5 @@ 
 <?php
-if(!defined('sugarEntry') || !sugarEntry) die('Not A Valid Entry Point');
+if (!defined('sugarEntry') || !sugarEntry) die('Not A Valid Entry Point');
 /*********************************************************************************
  * SugarCRM Community Edition is a customer relationship management program developed by
  * SugarCRM, Inc. Copyright (C) 2004-2013 SugarCRM Inc.
@@ -71,14 +71,14 @@ 
                 }
             }
 
-            $handler = 'subp_nav(\'' . $layout_def['module'] . '\', \'' . $layout_def['fields']['ID'] . '\', \'e\', this';
+            $handler = 'subp_nav(\''.$layout_def['module'].'\', \''.$layout_def['fields']['ID'].'\', \'e\', this';
             if (!empty($relationship_name)) {
-                $handler .= ', \'' . $relationship_name . '\'';
+                $handler .= ', \''.$relationship_name.'\'';
             }
             $handler .= ');';
 
-            return '<a href="#" onmouseover="' . $handler . '" onfocus="' . $handler .
-                '" class="listViewTdToolsS1" id="' . $unique_id . '">' . $app_strings['LNK_EDIT'] . '</a>';
+            return '<a href="#" onmouseover="'.$handler.'" onfocus="'.$handler.
+                '" class="listViewTdToolsS1" id="'.$unique_id.'">'.$app_strings['LNK_EDIT'].'</a>';
         }
 
         return '';

Braces +3 added lines, -1 removed lines

@@ -1,5 +1,7 @@
 <?php
-if(!defined('sugarEntry') || !sugarEntry) die('Not A Valid Entry Point');
+if(!defined('sugarEntry') || !sugarEntry) {
+    die('Not A Valid Entry Point');
+}
 /*********************************************************************************
  * SugarCRM Community Edition is a customer relationship management program developed by
  * SugarCRM, Inc. Copyright (C) 2004-2013 SugarCRM Inc.

include/generic/SugarWidgets/SugarWidgetFieldtime.php

Indentation +4 added lines, -4 removed lines

@@ -46,7 +46,7 @@ 
                 global $timedate;
                 // i guess qualifier and column_function are the same..
                 if (! empty($layout_def['column_function']))
-                 {
+                    {
                         $func_name = 'displayList'.$layout_def['column_function'];
                         if ( method_exists($this,$func_name))
                         {
@@ -62,11 +62,11 @@ 
                 $content = $this->displayListPlain($layout_def);
                 
                 if(!empty($date)) { // able to get the date context of the time            	
-                	$td = explode(' ', $timedate->to_display_date_time($date . ' ' . $content));
-	                return $td[1];
+                    $td = explode(' ', $timedate->to_display_date_time($date . ' ' . $content));
+                    return $td[1];
                 }
                 else { // assume there is no time context
-                 	return $timedate->to_display_time($content);
+                        return $timedate->to_display_time($content);
                 }
         }
 }

Spacing +5 added lines, -5 removed lines

@@ -1,5 +1,5 @@ 
 <?php
-if(!defined('sugarEntry') || !sugarEntry) die('Not A Valid Entry Point');
+if (!defined('sugarEntry') || !sugarEntry) die('Not A Valid Entry Point');
 /*********************************************************************************
  * SugarCRM Community Edition is a customer relationship management program developed by
  * SugarCRM, Inc. Copyright (C) 2004-2013 SugarCRM Inc.
@@ -45,10 +45,10 @@ 
         {
                 global $timedate;
                 // i guess qualifier and column_function are the same..
-                if (! empty($layout_def['column_function']))
+                if (!empty($layout_def['column_function']))
                  {
                         $func_name = 'displayList'.$layout_def['column_function'];
-                        if ( method_exists($this,$func_name))
+                        if (method_exists($this, $func_name))
                         {
                                 return $this->$func_name($layout_def)." \n";
                         }
@@ -61,8 +61,8 @@ 
                 
                 $content = $this->displayListPlain($layout_def);
                 
-                if(!empty($date)) { // able to get the date context of the time            	
-                	$td = explode(' ', $timedate->to_display_date_time($date . ' ' . $content));
+                if (!empty($date)) { // able to get the date context of the time            	
+                	$td = explode(' ', $timedate->to_display_date_time($date.' '.$content));
 	                return $td[1];
                 }
                 else { // assume there is no time context

Braces +4 added lines, -3 removed lines

@@ -1,5 +1,7 @@ 
 <?php
-if(!defined('sugarEntry') || !sugarEntry) die('Not A Valid Entry Point');
+if(!defined('sugarEntry') || !sugarEntry) {
+    die('Not A Valid Entry Point');
+}
 /*********************************************************************************
  * SugarCRM Community Edition is a customer relationship management program developed by
  * SugarCRM, Inc. Copyright (C) 2004-2013 SugarCRM Inc.
@@ -64,8 +66,7 @@ 
                 if(!empty($date)) { // able to get the date context of the time            	
                 	$td = explode(' ', $timedate->to_display_date_time($date . ' ' . $content));
 	                return $td[1];
-                }
-                else { // assume there is no time context
+                } else { // assume there is no time context
                  	return $timedate->to_display_time($content);
                 }
         }