adamjakab / SuiteCRM

modules/jjwg_Maps/views/view.quick_radius.php

Braces +13 added lines, -5 removed lines

@@ -1,6 +1,8 @@ 
 <?php
 
-if(!defined('sugarEntry') || !sugarEntry) die('Not A Valid Entry Point');
+if(!defined('sugarEntry') || !sugarEntry) {
+    die('Not A Valid Entry Point');
+}
 
 class Jjwg_MapsViewQuick_Radius extends SugarView {
 
@@ -10,8 +12,10 @@ 
 
 	function display() {
 	
-        if (!isset($_REQUEST['quick_address'])) $_REQUEST['quick_address'] = '';
-?>
+        if (!isset($_REQUEST['quick_address'])) {
+            $_REQUEST['quick_address'] = '';
+        }
+        ?>
 
 <div class="moduleTitle"><h2><?php echo $GLOBALS['mod_strings']['LBL_MAP_QUICK_RADIUS']; ?><div class="clear"></div></h2></div>
 <div class="clear"></div>
@@ -38,7 +42,9 @@ 
             <select id="display_module" tabindex="111" title="<?php echo $GLOBALS['mod_strings']['LBL_MODULE_TYPE']; ?>" name="display_module">
                 <?php foreach ($GLOBALS['jjwg_config']['valid_geocode_modules'] as $module) { ?>
                     <option value="<?php echo htmlspecialchars($module); ?>" <?php 
-                    if (!empty($_REQUEST['display_module']) && $module == $_REQUEST['display_module']) echo 'selected="selected"';
+                    if (!empty($_REQUEST['display_module']) && $module == $_REQUEST['display_module']) {
+                        echo 'selected="selected"';
+                    }
                     ?>><?php echo htmlspecialchars($GLOBALS['app_list_strings']['moduleList'][$module]); ?>
                 <?php } ?>
             </select>
@@ -54,7 +60,9 @@ 
             <select id="unit_type" tabindex="131" title="<?php echo $GLOBALS['mod_strings']['LBL_DISTANCE']; ?>" name="unit_type">
                 <?php foreach ($GLOBALS['app_list_strings']['map_unit_type_list'] as $key=>$value) { ?>
                     <option value="<?php echo htmlspecialchars($key); ?>" <?php 
-                    if ($key == $_REQUEST['unit_type']) echo 'selected="selected"';
+                    if ($key == $_REQUEST['unit_type']) {
+                        echo 'selected="selected"';
+                    }
                     ?>><?php echo htmlspecialchars($value); ?>
                 <?php } ?>
             </select>

modules/jjwg_Maps/views/view.config.php

Braces +45 added lines, -15 removed lines

@@ -1,6 +1,8 @@ 
 <?php
 
-if (!defined('sugarEntry') || !sugarEntry) die('Not A Valid Entry Point');
+if (!defined('sugarEntry') || !sugarEntry) {
+    die('Not A Valid Entry Point');
+}
 
 class Jjwg_MapsViewConfig extends SugarView {
 
@@ -113,7 +115,9 @@ 
                 name="address_type_Accounts" title="">
                 <?php foreach ($address_types_billing_or_shipping as $key=>$value) { ?>
                     <option value="<?php echo htmlspecialchars($key); ?>" <?php 
-                    if ($key == $GLOBALS['jjwg_config']['geocode_modules_to_address_type']['Accounts']) echo 'selected="selected"';
+                    if ($key == $GLOBALS['jjwg_config']['geocode_modules_to_address_type']['Accounts']) {
+                        echo 'selected="selected"';
+                    }
                     ?>><?php echo htmlspecialchars($value); ?></option>
                 <?php } ?>
             </select>
@@ -130,7 +134,9 @@ 
                 name="address_type_Contacts" title="">
                 <?php foreach ($address_types_primary_or_alt as $key=>$value) { ?>
                     <option value="<?php echo htmlspecialchars($key); ?>" <?php 
-                    if ($key == $GLOBALS['jjwg_config']['geocode_modules_to_address_type']['Contacts']) echo 'selected="selected"';
+                    if ($key == $GLOBALS['jjwg_config']['geocode_modules_to_address_type']['Contacts']) {
+                        echo 'selected="selected"';
+                    }
                     ?>><?php echo htmlspecialchars($value); ?></option>
                 <?php } ?>
             </select>
@@ -147,7 +153,9 @@ 
                 name="address_type_Leads" title="">
                 <?php foreach ($address_types_primary_or_alt as $key=>$value) { ?>
                     <option value="<?php echo htmlspecialchars($key); ?>" <?php 
-                    if ($key == $GLOBALS['jjwg_config']['geocode_modules_to_address_type']['Leads']) echo 'selected="selected"';
+                    if ($key == $GLOBALS['jjwg_config']['geocode_modules_to_address_type']['Leads']) {
+                        echo 'selected="selected"';
+                    }
                     ?>><?php echo htmlspecialchars($value); ?></option>
                 <?php } ?>
             </select>
@@ -164,7 +172,9 @@ 
                 name="address_type_Opportunities" title="">
                 <?php foreach ($address_types_billing_or_shipping as $key=>$value) { ?>
                     <option value="<?php echo htmlspecialchars($key); ?>" <?php 
-                    if ($key == $GLOBALS['jjwg_config']['geocode_modules_to_address_type']['Opportunities']) echo 'selected="selected"';
+                    if ($key == $GLOBALS['jjwg_config']['geocode_modules_to_address_type']['Opportunities']) {
+                        echo 'selected="selected"';
+                    }
                     ?>><?php echo htmlspecialchars($value); ?></option>
                 <?php } ?>
             </select>
@@ -182,7 +192,9 @@ 
                 name="address_type_Cases" title="">
                 <?php foreach ($address_types_billing_or_shipping as $key=>$value) { ?>
                     <option value="<?php echo htmlspecialchars($key); ?>" <?php 
-                    if ($key == $GLOBALS['jjwg_config']['geocode_modules_to_address_type']['Cases']) echo 'selected="selected"';
+                    if ($key == $GLOBALS['jjwg_config']['geocode_modules_to_address_type']['Cases']) {
+                        echo 'selected="selected"';
+                    }
                     ?>><?php echo htmlspecialchars($value); ?></option>
                 <?php } ?>
             </select>
@@ -200,7 +212,9 @@ 
                 name="address_type_Project" title="">
                 <?php foreach ($address_types_billing_or_shipping as $key=>$value) { ?>
                     <option value="<?php echo htmlspecialchars($key); ?>" <?php 
-                    if ($key == $GLOBALS['jjwg_config']['geocode_modules_to_address_type']['Project']) echo 'selected="selected"';
+                    if ($key == $GLOBALS['jjwg_config']['geocode_modules_to_address_type']['Project']) {
+                        echo 'selected="selected"';
+                    }
                     ?>><?php echo htmlspecialchars($value); ?></option>
                 <?php } ?>
             </select>
@@ -218,7 +232,9 @@ 
                 name="address_type_Meetings" title="">
                 <?php foreach ($address_types_flex_relate as $key=>$value) { ?>
                     <option value="<?php echo htmlspecialchars($key); ?>" <?php 
-                    if ($key == $GLOBALS['jjwg_config']['geocode_modules_to_address_type']['Meetings']) echo 'selected="selected"';
+                    if ($key == $GLOBALS['jjwg_config']['geocode_modules_to_address_type']['Meetings']) {
+                        echo 'selected="selected"';
+                    }
                     ?>><?php echo htmlspecialchars($value); ?></option>
                 <?php } ?>
             </select>
@@ -234,7 +250,9 @@ 
                 name="address_type_Prospects" title="">
                 <?php foreach ($address_types_primary_or_alt as $key=>$value) { ?>
                     <option value="<?php echo htmlspecialchars($key); ?>" <?php 
-                    if ($key == $GLOBALS['jjwg_config']['geocode_modules_to_address_type']['Prospects']) echo 'selected="selected"';
+                    if ($key == $GLOBALS['jjwg_config']['geocode_modules_to_address_type']['Prospects']) {
+                        echo 'selected="selected"';
+                    }
                     ?>><?php echo htmlspecialchars($value); ?></option>
                 <?php } ?>
             </select>
@@ -255,7 +273,9 @@ 
                 name="address_type_<?php echo $module; ?>" title="">
                 <?php foreach ($address_types_all as $key=>$value) { ?>
                     <option value="<?php echo htmlspecialchars($key); ?>" <?php 
-                    if ($key == $GLOBALS['jjwg_config']['geocode_modules_to_address_type'][$module]) echo 'selected="selected"';
+                    if ($key == $GLOBALS['jjwg_config']['geocode_modules_to_address_type'][$module]) {
+                        echo 'selected="selected"';
+                    }
                     ?>><?php echo htmlspecialchars($value); ?></option>
                 <?php } ?>
             </select>
@@ -449,7 +469,9 @@ 
                 name="allow_approximate_location_type" title="">
                 <?php foreach ($enabled_disabled as $key=>$value) { ?>
                     <option value="<?php echo htmlspecialchars($key); ?>" <?php 
-                    if ($key == $enabled) echo 'selected="selected"';
+                    if ($key == $enabled) {
+                        echo 'selected="selected"';
+                    }
                     ?>><?php echo htmlspecialchars($value); ?>
                 <?php } ?>
             </select>
@@ -493,7 +515,9 @@ 
                 name="address_cache_get_enabled" title="">
                 <?php foreach ($enabled_disabled as $key=>$value) { ?>
                     <option value="<?php echo htmlspecialchars($key); ?>" <?php 
-                    if ($key == $enabled) echo 'selected="selected"';
+                    if ($key == $enabled) {
+                        echo 'selected="selected"';
+                    }
                     ?>><?php echo htmlspecialchars($value); ?>
                 <?php } ?>
             </select>
@@ -511,7 +535,9 @@ 
                 name="address_cache_save_enabled" title="">
                 <?php foreach ($enabled_disabled as $key=>$value) { ?>
                     <option value="<?php echo htmlspecialchars($key); ?>" <?php 
-                    if ($key == $enabled) echo 'selected="selected"';
+                    if ($key == $enabled) {
+                        echo 'selected="selected"';
+                    }
                     ?>><?php echo htmlspecialchars($value); ?>
                 <?php } ?>
             </select>
@@ -541,7 +567,9 @@ 
                 name="logic_hooks_enabled" title="">
                 <?php foreach ($enabled_disabled as $key=>$value) { ?>
                     <option value="<?php echo htmlspecialchars($key); ?>" <?php 
-                    if ($key == $enabled) echo 'selected="selected"';
+                    if ($key == $enabled) {
+                        echo 'selected="selected"';
+                    }
                     ?>><?php echo htmlspecialchars($value); ?>
                 <?php } ?>
             </select>
@@ -609,7 +637,9 @@ 
                 name="map_default_unit_type" title="">
                 <?php foreach ($unit_types as $key=>$value) { ?>
                     <option value="<?php echo htmlspecialchars($key); ?>" <?php 
-                    if ($key == $GLOBALS['jjwg_config']['map_default_unit_type']) echo 'selected="selected"';
+                    if ($key == $GLOBALS['jjwg_config']['map_default_unit_type']) {
+                        echo 'selected="selected"';
+                    }
                     ?>><?php echo htmlspecialchars($value); ?>
                 <?php } ?>
             </select>

modules/SecurityGroups/SecurityGroup.php

Braces +7 added lines, -2 removed lines

@@ -163,7 +163,9 @@ 
         $GLOBALS['log']->debug("SecuritySuite: groupHasAccess $query");
         $result = $db->query($query);
         $row = $db->fetchByAssoc($result);
-        if(isset($row) && $row['results']>0) return true;
+        if(isset($row) && $row['results']>0) {
+            return true;
+        }
 
         return false;
     }
@@ -401,7 +403,10 @@ 
     }
 
     static function inherit_parentQuery(&$focus, $parent_type, $parent_id, $focus_id, $focus_module_dir) {
-        if(empty($parent_type) || empty($parent_id)) return; //no info passed
+        if(empty($parent_type) || empty($parent_id)) {
+            return;
+        }
+        //no info passed
 
         /** can speed this up by doing one query */
         //should be just one query but need a unique guid for each insert

modules/SecurityGroups/SecurityGroupUserRelationship.php

Braces +8 added lines, -5 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');
+}
 
 
 require_once('data/SugarBean.php');
@@ -83,10 +85,11 @@ 
         $query = "SELECT id, first_name, last_name, user_name FROM users ";
         $where_auto = "deleted=0";
 
-        if($where != "")
-            $query .= "where $where AND ".$where_auto;
-        else
-            $query .= "where ".$where_auto;
+        if($where != "") {
+                    $query .= "where $where AND ".$where_auto;
+        } else {
+                    $query .= "where ".$where_auto;
+        }
 
         $query .= " ORDER BY last_name, first_name";
 

modules/SecurityGroups/SecurityGroupMessage.php

Braces +11 added lines, -6 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');
+}
 
 class SecurityGroupMessage extends Basic {
 	var $new_schema = true;
@@ -91,7 +93,10 @@ 
 			$securitygroup_id = null; //6.4.0
 		}
 		$message = new SecurityGroupMessage();
-		if(empty($text)) return; // || !$feed->ACLAccess('save', true) )return;
+		if(empty($text)) {
+		    return;
+		}
+		// || !$feed->ACLAccess('save', true) )return;
 
 		$text = strip_tags($text);
 		$message->name = '';
@@ -120,18 +125,18 @@ 
 		if($weeks == 1){
 			$result = translate('LBL_TIME_LAST_WEEK','SugarFeed').' ';
 			return $result;
-		}else if($weeks > 1){
+		} else if($weeks > 1){
 			$result .= $weeks . ' '.translate('LBL_TIME_WEEKS','SugarFeed').' ';
 			if($days > 0) { 
                 $result .= $days . ' '.translate('LBL_TIME_DAYS','SugarFeed').' ';
             }
-		}else{
+		} else{
 			if($days == 1){
 				$result = translate('LBL_TIME_YESTERDAY','SugarFeed').' ';
 				return $result;
-			}else if($days > 1){
+			} else if($days > 1){
 				$result .= $days . ' '. translate('LBL_TIME_DAYS','SugarFeed').' ';
-			}else{
+			} else{
 				if($hours == 1) {
                     $result .= $hours . ' '.translate('LBL_TIME_HOUR','SugarFeed').' ';
                 } else {

modules/SecurityGroups/AssignGroups.php

Braces +13 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');
+}
 
 
 class AssignGroups {
@@ -14,7 +16,9 @@ 
 		&& empty($bean->fetched_row['id']) && $bean->module_dir != "Users" && $bean->module_dir != "SugarFeed") {		
 		//Upload an attachment to an Email Template and save. If user with multi groups - popup select option
 		//it will redirect to notes instead of EmailTemplate and relationship will fail...check below to avoid
-		if(!empty($_REQUEST['module']) && $_REQUEST['module'] != $bean->module_dir) return;
+		if(!empty($_REQUEST['module']) && $_REQUEST['module'] != $bean->module_dir) {
+		    return;
+		}
 
 		if(!empty($_REQUEST['securitygroup_list'])) {
 			require_once('modules/SecurityGroups/SecurityGroup.php');
@@ -38,9 +42,7 @@ 
 			unset($_SESSION['securitysuite_error']); //to be safe
 			$_SESSION['securitysuite_error'] = $ss_mod_strings['LBL_ERROR_DUPLICATE'];
 		}
-	}
-
-	else if(isset($sugar_config['securitysuite_user_popup']) && $sugar_config['securitysuite_user_popup'] == true
+	} else if(isset($sugar_config['securitysuite_user_popup']) && $sugar_config['securitysuite_user_popup'] == true
 		&& empty($bean->fetched_row['id']) && $bean->module_dir == "Users"
 		&& isset($_REQUEST['action']) && $_REQUEST['action'] != 'SaveSignature' ) { //Bug: 589
 
@@ -63,7 +65,9 @@ 
 
 function popup_onload($event, $arguments)
 {
-	if(!empty($_REQUEST['to_pdf']) || !empty($_REQUEST['sugar_body_only'])) return;
+	if(!empty($_REQUEST['to_pdf']) || !empty($_REQUEST['sugar_body_only'])) {
+	    return;
+	}
 
 /** //test user popup
 	//always have this loaded
@@ -74,7 +78,9 @@ 
 	$module = $_REQUEST['module'];
 	$action = $_REQUEST['action'];
 
-	if(isset($action) && ($action == "Save" || $action == "SetTimezone")) return;  
+	if(isset($action) && ($action == "Save" || $action == "SetTimezone")) {
+	    return;
+	}
 
 	if( (
 			//(isset($sugar_config['securitysuite_popup_select']) && $sugar_config['securitysuite_popup_select'] == true)

modules/Contacts/disablePortalUser.php

Braces +5 added lines, -3 removed lines

@@ -21,7 +21,9 @@ 
  *
  * @author Salesagility Ltd <support@salesagility.com>
  */
-if(!defined('sugarEntry'))define('sugarEntry', true);
+if(!defined('sugarEntry')) {
+    define('sugarEntry', true);
+}
 require_once 'modules/AOP_Case_Updates/util.php';
 if(!isAOPEnabled()){
     return;
@@ -40,12 +42,12 @@ 
     if(!$res->success){
         $msg = $res->error ? $res->error : $mod_strings['LBL_DISABLE_PORTAL_USER_FAILED'];
         SugarApplication::appendErrorMessage($msg);
-    }else{
+    } else{
         $bean->portal_account_disabled = 1;
         $bean->save(false);
         SugarApplication::appendErrorMessage($mod_strings['LBL_DISABLE_PORTAL_USER_SUCCESS']);
     }
-}else{
+} else{
     SugarApplication::appendErrorMessage($mod_strings['LBL_NO_JOOMLA_URL']);
 }
 

modules/Contacts/metadata/additionalDetails.php

Braces +45 added lines, -17 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.
@@ -56,26 +58,52 @@ 
 
     if(!empty($fields['PRIMARY_ADDRESS_STREET']) || !empty($fields['PRIMARY_ADDRESS_CITY']) ||
 		!empty($fields['PRIMARY_ADDRESS_STATE']) || !empty($fields['PRIMARY_ADDRESS_POSTALCODE']) ||
-		!empty($fields['PRIMARY_ADDRESS_COUNTRY']))
-			$overlib_string .= '<b>' . $mod_strings['LBL_PRIMARY_ADDRESS'] . '</b><br>';
-	if(!empty($fields['PRIMARY_ADDRESS_STREET'])) $overlib_string .= $fields['PRIMARY_ADDRESS_STREET'] . '<br>';
-	if(!empty($fields['PRIMARY_ADDRESS_STREET_2'])) $overlib_string .= $fields['PRIMARY_ADDRESS_STREET_2'] . '<br>';
-	if(!empty($fields['PRIMARY_ADDRESS_STREET_3'])) $overlib_string .= $fields['PRIMARY_ADDRESS_STREET_3'] . '<br>';
-	if(!empty($fields['PRIMARY_ADDRESS_CITY'])) $overlib_string .= $fields['PRIMARY_ADDRESS_CITY'] . ', ';
-	if(!empty($fields['PRIMARY_ADDRESS_STATE'])) $overlib_string .= $fields['PRIMARY_ADDRESS_STATE'] . ' ';
-	if(!empty($fields['PRIMARY_ADDRESS_POSTALCODE'])) $overlib_string .= $fields['PRIMARY_ADDRESS_POSTALCODE'] . ' ';
-	if(!empty($fields['PRIMARY_ADDRESS_COUNTRY'])) $overlib_string .= $fields['PRIMARY_ADDRESS_COUNTRY'] . '<br>';
-	if(strlen($overlib_string) > 0 && !(strrpos($overlib_string, '<br>') == strlen($overlib_string) - 4)) 
-		$overlib_string .= '<br>';  
-	if(!empty($fields['PHONE_MOBILE'])) $overlib_string .= '<b>'. $mod_strings['LBL_MOBILE_PHONE'] . '</b> <span class="phone">' . $fields['PHONE_MOBILE'] . '</span><br>';
-	if(!empty($fields['PHONE_HOME'])) $overlib_string .= '<b>'. $mod_strings['LBL_HOME_PHONE'] . '</b> <span class="phone">' . $fields['PHONE_HOME'] . '</span><br>';
-	if(!empty($fields['PHONE_OTHER'])) $overlib_string .= '<b>'. $mod_strings['LBL_OTHER_PHONE'] . '</b> <span class="phone">' . $fields['PHONE_OTHER'] . '</span><br>';
+		!empty($fields['PRIMARY_ADDRESS_COUNTRY'])) {
+    			$overlib_string .= '<b>' . $mod_strings['LBL_PRIMARY_ADDRESS'] . '</b><br>';
+    }
+	if(!empty($fields['PRIMARY_ADDRESS_STREET'])) {
+	    $overlib_string .= $fields['PRIMARY_ADDRESS_STREET'] . '<br>';
+	}
+	if(!empty($fields['PRIMARY_ADDRESS_STREET_2'])) {
+	    $overlib_string .= $fields['PRIMARY_ADDRESS_STREET_2'] . '<br>';
+	}
+	if(!empty($fields['PRIMARY_ADDRESS_STREET_3'])) {
+	    $overlib_string .= $fields['PRIMARY_ADDRESS_STREET_3'] . '<br>';
+	}
+	if(!empty($fields['PRIMARY_ADDRESS_CITY'])) {
+	    $overlib_string .= $fields['PRIMARY_ADDRESS_CITY'] . ', ';
+	}
+	if(!empty($fields['PRIMARY_ADDRESS_STATE'])) {
+	    $overlib_string .= $fields['PRIMARY_ADDRESS_STATE'] . ' ';
+	}
+	if(!empty($fields['PRIMARY_ADDRESS_POSTALCODE'])) {
+	    $overlib_string .= $fields['PRIMARY_ADDRESS_POSTALCODE'] . ' ';
+	}
+	if(!empty($fields['PRIMARY_ADDRESS_COUNTRY'])) {
+	    $overlib_string .= $fields['PRIMARY_ADDRESS_COUNTRY'] . '<br>';
+	}
+	if(strlen($overlib_string) > 0 && !(strrpos($overlib_string, '<br>') == strlen($overlib_string) - 4)) {
+			$overlib_string .= '<br>';
+	}
+	if(!empty($fields['PHONE_MOBILE'])) {
+	    $overlib_string .= '<b>'. $mod_strings['LBL_MOBILE_PHONE'] . '</b> <span class="phone">' . $fields['PHONE_MOBILE'] . '</span><br>';
+	}
+	if(!empty($fields['PHONE_HOME'])) {
+	    $overlib_string .= '<b>'. $mod_strings['LBL_HOME_PHONE'] . '</b> <span class="phone">' . $fields['PHONE_HOME'] . '</span><br>';
+	}
+	if(!empty($fields['PHONE_OTHER'])) {
+	    $overlib_string .= '<b>'. $mod_strings['LBL_OTHER_PHONE'] . '</b> <span class="phone">' . $fields['PHONE_OTHER'] . '</span><br>';
+	}
 
-	if(!empty($fields['DATE_MODIFIED'])) $overlib_string .= '<b>'. $mod_strings['LBL_DATE_MODIFIED'] . '</b> ' . $fields['DATE_MODIFIED'] . '<br>';
+	if(!empty($fields['DATE_MODIFIED'])) {
+	    $overlib_string .= '<b>'. $mod_strings['LBL_DATE_MODIFIED'] . '</b> ' . $fields['DATE_MODIFIED'] . '<br>';
+	}
 	
 	if(!empty($fields['DESCRIPTION'])) { 
 		$overlib_string .= '<b>'. $mod_strings['LBL_DESCRIPTION'] . '</b> ' . substr($fields['DESCRIPTION'], 0, 300);
-		if(strlen($fields['DESCRIPTION']) > 300) $overlib_string .= '...';
+		if(strlen($fields['DESCRIPTION']) > 300) {
+		    $overlib_string .= '...';
+		}
 	}	
 	
 	return array('fieldToAddTo' => 'NAME', 

modules/Contacts/views/view.quickcreate.php

Braces +21 added lines, -7 removed lines

@@ -1,6 +1,8 @@ 
 <?php
 //FILE SUGARCRM flav=pro || flav=sales
-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.
@@ -51,12 +53,24 @@ 
     		//XXX TODO 20110329 Frank Steegmans: Hack to make quick create fields populate when used through the DC menu
     		//          NOTE HOWEVER that sqs_objects form fields are not properly populated because of some other hacks
     		//          resulting in none of the fields properly populating when selecting an account
-    		if(!empty($this->bean->phone_office))$_REQUEST['phone_work'] = $this->bean->phone_office;
-    		if(!empty($this->bean->billing_address_street))$_REQUEST['primary_address_street'] = $this->bean->billing_address_street;
-    		if(!empty($this->bean->billing_address_city))$_REQUEST['primary_address_city'] = $this->bean->billing_address_city;
-    		if(!empty($this->bean->billing_address_state))$_REQUEST['primary_address_state'] = $this->bean->billing_address_state;
-    		if(!empty($this->bean->billing_address_country))$_REQUEST['primary_address_country'] = $this->bean->billing_address_country;
-    		if(!empty($this->bean->billing_address_postalcode))$_REQUEST['primary_address_postalcode'] = $this->bean->billing_address_postalcode;
+    		if(!empty($this->bean->phone_office)) {
+    		    $_REQUEST['phone_work'] = $this->bean->phone_office;
+    		}
+    		if(!empty($this->bean->billing_address_street)) {
+    		    $_REQUEST['primary_address_street'] = $this->bean->billing_address_street;
+    		}
+    		if(!empty($this->bean->billing_address_city)) {
+    		    $_REQUEST['primary_address_city'] = $this->bean->billing_address_city;
+    		}
+    		if(!empty($this->bean->billing_address_state)) {
+    		    $_REQUEST['primary_address_state'] = $this->bean->billing_address_state;
+    		}
+    		if(!empty($this->bean->billing_address_country)) {
+    		    $_REQUEST['primary_address_country'] = $this->bean->billing_address_country;
+    		}
+    		if(!empty($this->bean->billing_address_postalcode)) {
+    		    $_REQUEST['primary_address_postalcode'] = $this->bean->billing_address_postalcode;
+    		}
 	   	}
     }    
 }
\ No newline at end of file