adamjakab / SuiteCRM

include/generic/SugarWidgets/SugarWidgetSubPanelEditRoleButton.php

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.
@@ -56,19 +56,19 @@ 
         global $subpanel_item_count;
         $unique_id = $layout_def['subpanel_id']."_edit_".$subpanel_item_count; //bug 51512
 	
-		$href = 'index.php?module=' . $layout_def['module']
-			. '&action=' . 'ContactOpportunityRelationshipEdit'
-			. '&record=' . $layout_def['fields']['OPPORTUNITY_ROLE_ID']
-			. '&return_module=' . $_REQUEST['module']
-			. '&return_action=' . 'DetailView'
-			. '&return_id=' . $_REQUEST['record'];
+		$href = 'index.php?module='.$layout_def['module']
+			. '&action='.'ContactOpportunityRelationshipEdit'
+			. '&record='.$layout_def['fields']['OPPORTUNITY_ROLE_ID']
+			. '&return_module='.$_REQUEST['module']
+			. '&return_action='.'DetailView'
+			. '&return_id='.$_REQUEST['record'];
 			
 	//based on listview since that lets you select records
-	if($layout_def['ListView']){
-		return '<a href="' . $href . '"'
+	if ($layout_def['ListView']) {
+		return '<a href="'.$href.'"'
             . "id=\"$unique_id\""
-			. 'class="listViewTdToolsS1">' . $app_strings['LNK_EDIT'] .'</a>&nbsp;';
-	}else{
+			. 'class="listViewTdToolsS1">'.$app_strings['LNK_EDIT'].'</a>&nbsp;';
+	} else {
 		return '';
 	}
 	}

include/generic/SugarWidgets/SugarWidgetFieldparent_type.php

Spacing +4 added lines, -4 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,9 +48,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>";
             }

include/generic/SugarWidgets/SugarWidgetSubPanelTopCreateNoteButton.php

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="Notes";
+		$this->module = "Notes";
 		$this->subpanelDiv = "history";
 
 		// 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($this->subpanelDiv))
+		if (!empty($this->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 Note;
-		if ( !$focus->ACLAccess('EditView') ) {
+		if (!$focus->ACLAccess('EditView')) {
 		    return '';
 	    }
 

include/generic/SugarWidgets/SugarWidgetFieldtext.php

Spacing +2 added lines, -2 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.
@@ -56,7 +56,7 @@ 
     function queryFilterNot_Equals_Str($layout_def)
     {
         $column = $this->_get_column_select($layout_def);
-        return "($column IS NULL OR ". $this->reporter->db->convert($column, "text2char")." != ".
+        return "($column IS NULL OR ".$this->reporter->db->convert($column, "text2char")." != ".
             $this->reporter->db->quoted($layout_def['input_name0']).")";
     }
 

include/generic/SugarWidgets/SugarWidgetSubPanelTopSelectContactsButton.php

Spacing +33 added lines, -33 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.
@@ -46,14 +46,14 @@ 
 class SugarWidgetSubPanelTopSelectContactsButton extends SugarWidgetSubPanelTopSelectButton
 {
 	//button_properties is a collection of properties associated with the widget_class definition. layoutmanager
-	function SugarWidgetSubPanelTopSelectContactsButton($button_properties=array())
+	function SugarWidgetSubPanelTopSelectContactsButton($button_properties = array())
 	{
-		$this->button_properties=$button_properties;
+		$this->button_properties = $button_properties;
 	}
 
     public function getWidgetId($buttonSuffix = true)
     {
-        return parent::getWidgetId() . '_select_button';
+        return parent::getWidgetId().'_select_button';
     }
 
 	//widget_data is the collection of attributes associated with the button in the layout_defs file.
@@ -69,32 +69,32 @@ 
 		$this->module_name = 'Contacts';
 
 		if (is_array($this->button_properties)) {
-			if( isset($this->button_properties['title'])) {
+			if (isset($this->button_properties['title'])) {
 				$this->title = $app_strings[$this->button_properties['title']];
 			}
-			if( isset($this->button_properties['accesskey'])) {
+			if (isset($this->button_properties['accesskey'])) {
 				//$this->accesskey = $app_strings[$this->button_properties['accesskey']];
 			}
-			if( isset($this->button_properties['form_value'])) {
+			if (isset($this->button_properties['form_value'])) {
 				$this->value = $app_strings[$this->button_properties['form_value']];
 			}
-			if( isset($this->button_properties['module'])) {
+			if (isset($this->button_properties['module'])) {
 				$this->module_name = $this->button_properties['module'];
 			}
 		}
 
 		$focus = $widget_data['focus'];
-		if(ACLController::moduleSupportsACL($widget_data['module']) && !ACLController::checkAccess($widget_data['module'], 'list', true)){
-			$button = ' <input type="button" name="' .$this->getWidgetId() . '" id="' .$this->getWidgetId() . '" class="button"' . "\"\n"
-			. ' title="' . $this->title . '"'
-			. ' value="' . $this->value . "\"\n"
+		if (ACLController::moduleSupportsACL($widget_data['module']) && !ACLController::checkAccess($widget_data['module'], 'list', true)) {
+			$button = ' <input type="button" name="'.$this->getWidgetId().'" id="'.$this->getWidgetId().'" class="button"'."\"\n"
+			. ' title="'.$this->title.'"'
+			. ' value="'.$this->value."\"\n"
 			.' disabled />';
 			return $button;
 		}
 
 		//refresh the whole page after end of action?
 		$refresh_page = 0;
-		if(!empty($widget_data['subpanel_definition']->_instance_properties['refresh_page'])){
+		if (!empty($widget_data['subpanel_definition']->_instance_properties['refresh_page'])) {
 			$refresh_page = 1;
 		}
 
@@ -106,38 +106,38 @@ 
 			$this->module_name = $subpanel_name;
 		}
 
-		if ($subpanel_name == 'Project'){
+		if ($subpanel_name == 'Project') {
 			$link_field_name = 'project_contacts_1';
 		}
-		else{
+		else {
 			$link_field_name = $subpanel_definition->get_data_source_name(true);
 		}
 
-		$popup_mode='Single';
-		if(isset($widget_data['mode'])){
-			$popup_mode=$widget_data['mode'];
+		$popup_mode = 'Single';
+		if (isset($widget_data['mode'])) {
+			$popup_mode = $widget_data['mode'];
 		}
-		if(isset($widget_data['initial_filter_fields'])){
+		if (isset($widget_data['initial_filter_fields'])) {
 			if (is_array($widget_data['initial_filter_fields'])) {
 				foreach ($widget_data['initial_filter_fields'] as $value=>$alias) {
 					if (isset($focus->$value) and !empty($focus->$value)) {
-						$initial_filter.="&".$alias . '='.urlencode($focus->$value);
+						$initial_filter .= "&".$alias.'='.urlencode($focus->$value);
 					}
 				}
 			}
 		}
-		$create="true";
-		if(isset($widget_data['create'])){
-			$create=$widget_data['create'];
+		$create = "true";
+		if (isset($widget_data['create'])) {
+			$create = $widget_data['create'];
 		}
 		$return_module = $_REQUEST['module'];
 		$return_action = 'SubPanelViewer';
 		$return_id = $_REQUEST['record'];
 
 		//field_to_name_array
-		$fton_array= array('id' => 'subpanel_id');
-		if(isset($widget_data['field_to_name_array']) && is_array($widget_data['field_to_name_array'])){
-			$fton_array=array_merge($fton_array,$widget_data['field_to_name_array']);
+		$fton_array = array('id' => 'subpanel_id');
+		if (isset($widget_data['field_to_name_array']) && is_array($widget_data['field_to_name_array'])) {
+			$fton_array = array_merge($fton_array, $widget_data['field_to_name_array']);
 		}
 
 		$return_url = "index.php?module=$return_module&action=$return_action&subpanel=$subpanel_name&record=$return_id&sugar_body_only=1";
@@ -156,21 +156,21 @@ 
 		);
 
 		if (is_array($this->button_properties) && !empty($this->button_properties['add_to_passthru_data'])) {
-			$popup_request_data['passthru_data']= array_merge($popup_request_data['passthru_data'],$this->button_properties['add_to_passthru_data']);
+			$popup_request_data['passthru_data'] = array_merge($popup_request_data['passthru_data'], $this->button_properties['add_to_passthru_data']);
 		}
 
 		if (is_array($this->button_properties) && !empty($this->button_properties['add_to_passthru_data']['return_type'])) {
 
-			if ($this->button_properties['add_to_passthru_data']['return_type']=='report') {
-				$initial_filter = "&module_name=". urlencode($widget_data['module']);
+			if ($this->button_properties['add_to_passthru_data']['return_type'] == 'report') {
+				$initial_filter = "&module_name=".urlencode($widget_data['module']);
 			}
 		}
 		$json_encoded_php_array = $this->_create_json_encoded_popup_request($popup_request_data);
 
-		return '<form action="index.php">' . "\n"
-			. ' <input type="button" name="' .$this->getWidgetId() . '" id="' .$this->getWidgetId() . '" class="button"' . "\"\n"
-				. ' title="' . $this->title . '"'
-			. ' value="' . $this->value . "\"\n"
+		return '<form action="index.php">'."\n"
+			. ' <input type="button" name="'.$this->getWidgetId().'" id="'.$this->getWidgetId().'" class="button"'."\"\n"
+				. ' title="'.$this->title.'"'
+			. ' value="'.$this->value."\"\n"
 			. " onclick='open_popup(\"$this->module_name\",600,400,\"$initial_filter\",true,true,$json_encoded_php_array,\"$popup_mode\",$create);' /></form>\n";
 	}
 }

include/generic/SugarWidgets/SugarWidgetSubPanelTopSelectAccountButton.php

Spacing +2 added lines, -2 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.
@@ -53,7 +53,7 @@ 
 		* Bug25633 code change start
 		*/
 		if (!ACLController::checkAccess($widget_data["module"], "edit", true)) {
-			return ;
+			return;
 		}
 		/*
 		* Bug25633 code change end

include/generic/SugarWidgets/SugarWidgetSubPanelTopSelectButton.php

Spacing +31 added lines, -31 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.
@@ -46,14 +46,14 @@ 
 class SugarWidgetSubPanelTopSelectButton extends SugarWidgetSubPanelTopButton
 {
 	//button_properties is a collection of properties associated with the widget_class definition. layoutmanager
-	function SugarWidgetSubPanelTopSelectButton($button_properties=array())
+	function SugarWidgetSubPanelTopSelectButton($button_properties = array())
 	{
-		$this->button_properties=$button_properties;
+		$this->button_properties = $button_properties;
 	}
 
     public function getWidgetId($buttonSuffix = true)
     {
-        return parent::getWidgetId(false) . 'select_button';
+        return parent::getWidgetId(false).'select_button';
     }
 
     public function getDisplayName()
@@ -66,38 +66,38 @@ 
 		global $app_strings;
 		$initial_filter = '';
 
-	    $this->title     = $this->getTitle();
+	    $this->title = $this->getTitle();
         $this->accesskey = $this->getAccesskey();
         $this->value     = $this->getDisplayName();
 
 		if (is_array($this->button_properties)) {
-			if( isset($this->button_properties['title'])) {
+			if (isset($this->button_properties['title'])) {
 				$this->title = $app_strings[$this->button_properties['title']];
 			}
-			if( isset($this->button_properties['accesskey'])) {
+			if (isset($this->button_properties['accesskey'])) {
 				$this->accesskey = $app_strings[$this->button_properties['accesskey']];
 			}
-			if( isset($this->button_properties['form_value'])) {
+			if (isset($this->button_properties['form_value'])) {
 				$this->value = $app_strings[$this->button_properties['form_value']];
 			}
-			if( isset($this->button_properties['module'])) {
+			if (isset($this->button_properties['module'])) {
 				$this->module_name = $this->button_properties['module'];
 			}
 		}
 
 
 		$focus = $widget_data['focus'];
-		if(ACLController::moduleSupportsACL($widget_data['module']) && !ACLController::checkAccess($widget_data['module'], 'list', true)){
-			$button = ' <input type="button" name="' . $this->getWidgetId() . '" id="' . $this->getWidgetId() . '" class="button"' . "\n"
-			. ' title="' . $this->title . '"'
-			. ' value="' . $this->value . "\"\n"
+		if (ACLController::moduleSupportsACL($widget_data['module']) && !ACLController::checkAccess($widget_data['module'], 'list', true)) {
+			$button = ' <input type="button" name="'.$this->getWidgetId().'" id="'.$this->getWidgetId().'" class="button"'."\n"
+			. ' title="'.$this->title.'"'
+			. ' value="'.$this->value."\"\n"
 			.' disabled />';
 			return $button;
 		}
 
 		//refresh the whole page after end of action?
 		$refresh_page = 0;
-		if(!empty($widget_data['subpanel_definition']->_instance_properties['refresh_page'])){
+		if (!empty($widget_data['subpanel_definition']->_instance_properties['refresh_page'])) {
 			$refresh_page = 1;
 		}
 
@@ -109,31 +109,31 @@ 
 			$this->module_name = $subpanel_definition->get_module_name();
 		}
 		$link_field_name = $subpanel_definition->get_data_source_name(true);
-		$popup_mode='Single';
-		if(isset($widget_data['mode'])){
-			$popup_mode=$widget_data['mode'];
+		$popup_mode = 'Single';
+		if (isset($widget_data['mode'])) {
+			$popup_mode = $widget_data['mode'];
 		}
-		if(isset($widget_data['initial_filter_fields'])){
+		if (isset($widget_data['initial_filter_fields'])) {
 			if (is_array($widget_data['initial_filter_fields'])) {
 				foreach ($widget_data['initial_filter_fields'] as $value=>$alias) {
 					if (isset($focus->$value) and !empty($focus->$value)) {
-						$initial_filter.="&".$alias . '='.urlencode($focus->$value);
+						$initial_filter .= "&".$alias.'='.urlencode($focus->$value);
 					}
 				}
 			}
 		}
-		$create="true";
-		if(isset($widget_data['create'])){
-			$create=$widget_data['create'];
+		$create = "true";
+		if (isset($widget_data['create'])) {
+			$create = $widget_data['create'];
 		}
 		$return_module = $_REQUEST['module'];
 		$return_action = 'SubPanelViewer';
 		$return_id = $_REQUEST['record'];
 
 		//field_to_name_array
-		$fton_array= array('id' => 'subpanel_id');
-		if(isset($widget_data['field_to_name_array']) && is_array($widget_data['field_to_name_array'])){
-			$fton_array=array_merge($fton_array,$widget_data['field_to_name_array']);
+		$fton_array = array('id' => 'subpanel_id');
+		if (isset($widget_data['field_to_name_array']) && is_array($widget_data['field_to_name_array'])) {
+			$fton_array = array_merge($fton_array, $widget_data['field_to_name_array']);
 		}
 
 		$return_url = "index.php?module=$return_module&action=$return_action&subpanel=$subpanel_name&record=$return_id&sugar_body_only=1";
@@ -159,21 +159,21 @@ 
 
                 if (is_array($this->button_properties) && !empty($this->button_properties['add_to_passthru_data'])) 
                 {
-                    $popup_request_data['passthru_data']= array_merge($popup_request_data['passthru_data'],$this->button_properties['add_to_passthru_data']);
+                    $popup_request_data['passthru_data'] = array_merge($popup_request_data['passthru_data'], $this->button_properties['add_to_passthru_data']);
                 }
 
 		if (is_array($this->button_properties) && !empty($this->button_properties['add_to_passthru_data']['return_type'])) {
 
-			if ($this->button_properties['add_to_passthru_data']['return_type']=='report') {
-				$initial_filter = "&module_name=". urlencode($widget_data['module']);
+			if ($this->button_properties['add_to_passthru_data']['return_type'] == 'report') {
+				$initial_filter = "&module_name=".urlencode($widget_data['module']);
 			}
 		}
         //acl_roles_users_selectuser_button
 
         $json_encoded_php_array = $this->_create_json_encoded_popup_request($popup_request_data);
-		return ' <input type="button" name="' . $this->getWidgetId() . '" id="' . $this->getWidgetId() . '" class="button"' . "\n"
-				. ' title="' . $this->title . '"'
-			. ' value="' . $this->value . "\"\n"
+		return ' <input type="button" name="'.$this->getWidgetId().'" id="'.$this->getWidgetId().'" class="button"'."\n"
+				. ' title="'.$this->title.'"'
+			. ' value="'.$this->value."\"\n"
 			. " onclick='open_popup(\"$this->module_name\",600,400,\"$initial_filter\",true,true,$json_encoded_php_array,\"$popup_mode\",$create);' />\n";
 	}
 

include/generic/SugarWidgets/SugarWidgetFielddatetime.php

Spacing +54 added lines, -54 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.
@@ -44,7 +44,7 @@ 
 class SugarWidgetFieldDateTime extends SugarWidgetReportField
 {
 	var $reporter;
-	var $assigned_user=null;
+	var $assigned_user = null;
 
     function SugarWidgetFieldDateTime($layout_manager)
     {
@@ -66,7 +66,7 @@ 
 
 		$report_def_str = $json_obj->decode($this->reporter->report_def_str);
 
-		if(empty($report_def_str['assigned_user_id'])) return null;
+		if (empty($report_def_str['assigned_user_id'])) return null;
 
 		$this->assigned_user = new User();
 		$this->assigned_user->retrieve($report_def_str['assigned_user_id']);
@@ -111,12 +111,12 @@ 
 	protected function expandDate($date, $end = false)
 	{
 	    global $timedate;
-	    if($this->hasTime($date)) {
+	    if ($this->hasTime($date)) {
 	        return $date;
 	    }
 
         //C.L. Bug 48616 - If the $date is set to the Today macro, then adjust accordingly
-        if(strtolower($date) == 'today')
+        if (strtolower($date) == 'today')
         {
            $startEnd = $timedate->getDayStartEndGMT($timedate->getNow(true));
            return $end ? $startEnd['end'] : $startEnd['start'];
@@ -126,7 +126,7 @@ 
         $date = $timedate->tzUser(new SugarDateTime());
         $date->setDate($parsed->year, $parsed->month, $parsed->day);
 
-	    if($end) {
+	    if ($end) {
 	        return $date->setTime(23, 59, 59);
 	    } else {
 	        return $date->setTime(0, 0, 0);
@@ -161,13 +161,13 @@ 
         $column = $this->_get_column_select($layout_def);
         $begin = $layout_def['input_name0'];
         $hasTime = $this->hasTime($begin);
-        if(!$hasTime){
+        if (!$hasTime) {
      	    $end = $this->expandDate($begin, true);
             $begin = $this->expandDate($begin);
             $cond = $this->queryDateOp($column, $begin, "<", "datetime")." OR ".
                 $this->queryDateOp($column, $end, ">", "datetime");
         } else {
-            $cond =  $this->queryDateOp($column, $begin, "!=", "datetime");
+            $cond = $this->queryDateOp($column, $begin, "!=", "datetime");
         }
         return "($column IS NULL OR $cond)";
 	}
@@ -176,7 +176,7 @@ 
      * Get assigned or logged in user's current date and time value.
      * @param boolean $timestamp Format of return value, if set to true, return unix like timestamp , else a formatted date.
      */
-	function get_users_current_date_time($timestamp=false)
+	function get_users_current_date_time($timestamp = false)
 	{
 	 	global $current_user;
         global $timedate;
@@ -190,9 +190,9 @@ 
         	return $begin;
         } else {
         	$begin_parts = explode(' ', $begin);
-        	$date_parts=explode('-', $begin_parts[0]);
-        	$time_parts=explode(':', $begin_parts[1]);
-        	$curr_timestamp=mktime($time_parts[0],$time_parts[1],0,$date_parts[1], $date_parts[2],$date_parts[0]);
+        	$date_parts = explode('-', $begin_parts[0]);
+        	$time_parts = explode(':', $begin_parts[1]);
+        	$curr_timestamp = mktime($time_parts[0], $time_parts[1], 0, $date_parts[1], $date_parts[2], $date_parts[0]);
         	return $curr_timestamp;
         }
 
@@ -202,23 +202,23 @@ 
 	 * @param int $days Adjust date by this number of days, negative values are valid.
 	 * @param time string falg for desired time value, start: minimum time, end: maximum time, default: current time
 	 */
-	function get_db_date($days,$time) {
+	function get_db_date($days, $time) {
         global $timedate;
 
-        $begin = date($GLOBALS['timedate']->get_db_date_time_format(), time()+(86400 * $days));  //gmt date with day adjustment applied.
+        $begin = date($GLOBALS['timedate']->get_db_date_time_format(), time() + (86400 * $days)); //gmt date with day adjustment applied.
         //kbrill bug #13884
         //$begin = $timedate->to_display_date_time($begin,true,true,$this->assigned_user);
 		$begin = $timedate->handle_offset($begin, $timedate->get_db_date_time_format(), false, $this->assigned_user);
 
-        if ($time=='start') {
+        if ($time == 'start') {
             $begin_parts = explode(' ', $begin);
-            $be = $begin_parts[0] . ' 00:00:00';
+            $be = $begin_parts[0].' 00:00:00';
         }
-        else if ($time=='end') {
+        else if ($time == 'end') {
             $begin_parts = explode(' ', $begin);
-            $be = $begin_parts[0] . ' 23:59:59';
+            $be = $begin_parts[0].' 23:59:59';
         } else {
-            $be=$begin;
+            $be = $begin;
         }
 
         //convert date to db format without converting to GMT.
@@ -233,7 +233,7 @@ 
 	 * @param string $begin start date value (in DB format)
 	 * @param string $end End date value (in DB format)
 	 */
-	function get_start_end_date_filter(& $layout_def, $begin,$end)
+	function get_start_end_date_filter(& $layout_def, $begin, $end)
 	{
 	    if (isset ($layout_def['rel_field'])) {
 	        $field_name = $this->reporter->db->convert(
@@ -260,7 +260,7 @@ 
     protected function queryDateOp($arg1, $arg2, $op, $type)
     {
         global $timedate;
-        if($arg2 instanceof DateTime) {
+        if ($arg2 instanceof DateTime) {
             $arg2 = $timedate->asDbType($arg2, $type);
         }
         return "$arg1 $op ".$this->reporter->db->convert($this->reporter->db->quoted($arg2), $type)."\n";
@@ -285,7 +285,7 @@ 
     {
         $begin = $day->get_day_begin();
         $end = $day->get_day_end();
-        return $this->get_start_end_date_filter($layout_def,$begin->asDb(),$end->asDb());
+        return $this->get_start_end_date_filter($layout_def, $begin->asDb(), $end->asDb());
     }
 
 	function queryFilterTP_yesterday($layout_def)
@@ -313,7 +313,7 @@ 
 		$begin = $this->now()->get("-6 days")->get_day_begin();
 		$end = $this->now()->get_day_end();
 
-		return $this->get_start_end_date_filter($layout_def,$begin->asDb(),$end->asDb());
+		return $this->get_start_end_date_filter($layout_def, $begin->asDb(), $end->asDb());
 	}
 
 	function queryFilterTP_next_7_days($layout_def)
@@ -323,7 +323,7 @@ 
 		$begin = $this->now()->get_day_begin();
 		$end = $this->now()->get("+6 days")->get_day_end();
 
-		return $this->get_start_end_date_filter($layout_def,$begin->asDb(),$end->asDb());
+		return $this->get_start_end_date_filter($layout_def, $begin->asDb(), $end->asDb());
 	}
 
     /**
@@ -336,14 +336,14 @@ 
         $begin = $month->setTime(0, 0, 0);
         $end = clone($begin);
 		$end->setDate($begin->year, $begin->month, $begin->days_in_month)->setTime(23, 59, 59);
-        return $this->get_start_end_date_filter($layout_def,$begin->asDb(),$end->asDb());
+        return $this->get_start_end_date_filter($layout_def, $begin->asDb(), $end->asDb());
     }
 
     function queryFilterTP_last_month($layout_def)
 	{
 		global $timedate;
 		$month = $this->now();
-		return $this->queryMonth($layout_def, $month->setDate($month->year, $month->month-1, 1));
+		return $this->queryMonth($layout_def, $month->setDate($month->year, $month->month - 1, 1));
 	}
 
 	function queryFilterTP_this_month($layout_def)
@@ -363,7 +363,7 @@ 
 	{
 		global $timedate;
 		$month = $this->now();
-		return $this->queryMonth($layout_def, $month->setDate($month->year, $month->month+1, 1));
+		return $this->queryMonth($layout_def, $month->setDate($month->year, $month->month + 1, 1));
 	}
 
 	function queryFilterTP_last_30_days($layout_def)
@@ -371,7 +371,7 @@ 
 		global $timedate;
 		$begin = $this->now()->get("-29 days")->get_day_begin();
 		$end = $this->now()->get_day_end();
-		return $this->get_start_end_date_filter($layout_def,$begin->asDb(),$end->asDb());
+		return $this->get_start_end_date_filter($layout_def, $begin->asDb(), $end->asDb());
 	}
 
 	function queryFilterTP_next_30_days($layout_def)
@@ -379,7 +379,7 @@ 
 		global $timedate;
 		$begin = $this->now()->get_day_begin();
 		$end = $this->now()->get("+29 days")->get_day_end();
-		return $this->get_start_end_date_filter($layout_def,$begin->asDb(),$end->asDb());
+		return $this->get_start_end_date_filter($layout_def, $begin->asDb(), $end->asDb());
 	}
 
     /**
@@ -457,10 +457,10 @@ 
 	{
 		global $timedate;
 		$begin = $this->now();
-		$begin->setDate($begin->year-1, 1, 1)->setTime(0, 0);
+		$begin->setDate($begin->year - 1, 1, 1)->setTime(0, 0);
 		$end = clone $begin;
 		$end->setDate($end->year, 12, 31)->setTime(23, 59, 59);
-		return $this->get_start_end_date_filter($layout_def,$begin->asDb(),$end->asDb());
+		return $this->get_start_end_date_filter($layout_def, $begin->asDb(), $end->asDb());
 	}
 
 	function queryFilterTP_this_year($layout_def)
@@ -470,17 +470,17 @@ 
 		$begin->setDate($begin->year, 1, 1)->setTime(0, 0);
 		$end = clone $begin;
 		$end->setDate($end->year, 12, 31)->setTime(23, 59, 59);
-		return $this->get_start_end_date_filter($layout_def,$begin->asDb(),$end->asDb());
+		return $this->get_start_end_date_filter($layout_def, $begin->asDb(), $end->asDb());
 	}
 
 	function queryFilterTP_next_year(& $layout_def)
 	{
 		global $timedate;
 		$begin = $this->now();
-		$begin->setDate($begin->year+1, 1, 1)->setTime(0, 0);
+		$begin->setDate($begin->year + 1, 1, 1)->setTime(0, 0);
 		$end = clone $begin;
 		$end->setDate($end->year, 12, 31)->setTime(23, 59, 59);
-		return $this->get_start_end_date_filter($layout_def,$begin->asDb(),$end->asDb());
+		return $this->get_start_end_date_filter($layout_def, $begin->asDb(), $end->asDb());
 	}
 
 	function queryGroupBy($layout_def)
@@ -489,7 +489,7 @@ 
 		if (!empty ($layout_def['qualifier'])) {
 			$func_name = 'queryGroupBy'.$layout_def['qualifier'];
 			if (method_exists($this, $func_name)) {
-				return $this-> $func_name ($layout_def)." \n";
+				return $this-> $func_name($layout_def)." \n";
 			}
 		}
 		return parent :: queryGroupBy($layout_def)." \n";
@@ -498,9 +498,9 @@ 
 	function queryOrderBy($layout_def)
 	{
         if (!empty ($layout_def['qualifier'])) {
-			$func_name ='queryOrderBy'.$layout_def['qualifier'];
+			$func_name = 'queryOrderBy'.$layout_def['qualifier'];
 			if (method_exists($this, $func_name)) {
-				return $this-> $func_name ($layout_def)."\n";
+				return $this-> $func_name($layout_def)."\n";
 			}
 		}
 		$order_by = parent :: queryOrderBy($layout_def)."\n";
@@ -511,15 +511,15 @@ 
         global $timedate;
         $content = parent:: displayListPlain($layout_def);
         // awu: this if condition happens only in Reports where group by month comes back as YYYY-mm format
-        if (count(explode('-',$content)) == 2){
+        if (count(explode('-', $content)) == 2) {
             return $content;
         // if date field
-        }elseif(substr_count($layout_def['type'], 'date') > 0){
+        }elseif (substr_count($layout_def['type'], 'date') > 0) {
             // if date time field
-            if(substr_count($layout_def['type'], 'time') > 0 && $this->get_time_part($content)!= false){
+            if (substr_count($layout_def['type'], 'time') > 0 && $this->get_time_part($content) != false) {
                 $td = $timedate->to_display_date_time($content);
                 return $td;
-            }else{// if date only field
+            } else {// if date only field
                 $td = $timedate->to_display_date($content, false); // Avoid PHP notice of returning by reference.
                 return $td;
             }
@@ -530,7 +530,7 @@ 
     {
         global $timedate;
 
-        $date_parts=$timedate->split_date_time($date_time_value);
+        $date_parts = $timedate->split_date_time($date_time_value);
         if (count($date_parts) > 1) {
             return $date_parts[1];
         } else {
@@ -544,7 +544,7 @@ 
         if (!empty ($layout_def['column_function'])) {
             $func_name = 'displayList'.$layout_def['column_function'];
             if (method_exists($this, $func_name)) {
-                return $this-> $func_name ($layout_def);
+                return $this-> $func_name($layout_def);
             }
         }
         $content = parent :: displayListPlain($layout_def);
@@ -556,7 +556,7 @@ 
 		if (!empty ($layout_def['column_function'])) {
 			$func_name = 'querySelect'.$layout_def['column_function'];
 			if (method_exists($this, $func_name)) {
-				return $this-> $func_name ($layout_def)." \n";
+				return $this-> $func_name($layout_def)." \n";
 			}
 		}
 		return parent :: querySelect($layout_def)." \n";
@@ -598,7 +598,7 @@ 
         {
             $return = $this->reporter->db->convert($return, 'add_tz_offset');
         }
-        return $this->reporter->db->convert($return, "date_format", array('%Y-%m')) . ' ' . $this->_get_column_alias($layout_def) . "\n";
+        return $this->reporter->db->convert($return, "date_format", array('%Y-%m')).' '.$this->_get_column_alias($layout_def)."\n";
     }
 
     /**
@@ -614,7 +614,7 @@ 
         {
             $return = $this->reporter->db->convert($return, 'add_tz_offset');
         }
-        return $this->reporter->db->convert($return, "date_format", array('%Y-%m')) . "\n";
+        return $this->reporter->db->convert($return, "date_format", array('%Y-%m'))."\n";
     }
 
     /**
@@ -634,11 +634,11 @@ 
 
         if (empty($layout_def['sort_dir']) || $layout_def['sort_dir'] == 'a')
         {
-            return $orderBy . " ASC\n";
+            return $orderBy." ASC\n";
         }
         else
         {
-            return $orderBy . " DESC\n";
+            return $orderBy." DESC\n";
         }
     }
 
@@ -655,7 +655,7 @@ 
         {
             $return = $this->reporter->db->convert($return, 'add_tz_offset');
         }
-        return $this->reporter->db->convert($return, "date_format", array('%Y-%m-%d')) . ' ' . $this->_get_column_alias($layout_def) . "\n";
+        return $this->reporter->db->convert($return, "date_format", array('%Y-%m-%d')).' '.$this->_get_column_alias($layout_def)."\n";
     }
 
     /**
@@ -671,7 +671,7 @@ 
         {
             $return = $this->reporter->db->convert($return, 'add_tz_offset');
         }
-        return $this->reporter->db->convert($return, "date_format", array('%Y-%m-%d')) . "\n";
+        return $this->reporter->db->convert($return, "date_format", array('%Y-%m-%d'))."\n";
     }
 
     /**
@@ -687,7 +687,7 @@ 
         {
             $return = $this->reporter->db->convert($return, 'add_tz_offset');
         }
-        return $this->reporter->db->convert($return, "date_format", array('%Y')) . ' ' . $this->_get_column_alias($layout_def) . "\n";
+        return $this->reporter->db->convert($return, "date_format", array('%Y')).' '.$this->_get_column_alias($layout_def)."\n";
     }
 
     /**
@@ -703,7 +703,7 @@ 
         {
             $return = $this->reporter->db->convert($return, 'add_tz_offset');
         }
-        return $this->reporter->db->convert($return, "date_format", array('%Y')) . "\n";
+        return $this->reporter->db->convert($return, "date_format", array('%Y'))."\n";
     }
 
 	function querySelectquarter($layout_def)
@@ -751,11 +751,11 @@ 
 
         if (empty($layout_def['sort_dir']) || $layout_def['sort_dir'] == 'a')
         {
-            return $orderBy . " ASC\n";
+            return $orderBy." ASC\n";
         }
         else
         {
-            return $orderBy . " DESC\n";
+            return $orderBy." DESC\n";
         }
     }
 

include/generic/SugarWidgets/SugarWidgetFielddatetimecombo.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.
@@ -43,7 +43,7 @@ 
 
 class SugarWidgetFieldDateTimecombo extends SugarWidgetFieldDateTime {
 	var $reporter;
-	var $assigned_user=null;
+	var $assigned_user = null;
 
     function SugarWidgetFieldDateTimecombo(&$layout_manager) {
         parent::SugarWidgetFieldDateTime($layout_manager);
@@ -53,7 +53,7 @@ 
 	//If there is requirement, we can modify report.js::addFilterInputDatetimesBetween and this function
 	function queryFilterBetween_Datetimes(& $layout_def) {
 		global $timedate;
-		if($this->getAssignedUser()) {
+		if ($this->getAssignedUser()) {
 			$begin = $timedate->handle_offset($layout_def['input_name0'], $timedate->get_db_date_time_format(), false, $this->assigned_user);
 			$end = $timedate->handle_offset($layout_def['input_name2'], $timedate->get_db_date_time_format(), false, $this->assigned_user);
 		}