adamjakab / SuiteCRM

include/database/DBManager.php

Braces +276 added lines, -198 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.
@@ -403,13 +405,15 @@ 
 	$match = array();
 	preg_match_all("'.* FROM ([^ ]*).* ORDER BY (.*)'is", $sql, $match);
 	$indices = false;
-	if (!empty($match[1][0]))
-		$table = $match[1][0];
-	else
-		return false;
+	if (!empty($match[1][0])) {
+			$table = $match[1][0];
+	} else {
+			return false;
+	}
 
-	if (!empty($object_name) && !empty($GLOBALS['dictionary'][$object_name]))
-		$indices = $GLOBALS['dictionary'][$object_name]['indices'];
+	if (!empty($object_name) && !empty($GLOBALS['dictionary'][$object_name])) {
+			$indices = $GLOBALS['dictionary'][$object_name]['indices'];
+	}
 
 	if (empty($indices)) {
 		foreach ( $GLOBALS['dictionary'] as $current ) {
@@ -427,25 +431,29 @@ 
 		$orderBys = explode(' ', $match[2][0]);
 		foreach ($orderBys as $orderBy){
 			$orderBy = trim($orderBy);
-			if (empty($orderBy))
-				continue;
+			if (empty($orderBy)) {
+							continue;
+			}
 			$orderBy = strtolower($orderBy);
-			if ($orderBy == 'asc' || $orderBy == 'desc')
-				continue;
+			if ($orderBy == 'asc' || $orderBy == 'desc') {
+							continue;
+			}
 
 			$orderBy = str_replace(array($table . '.', ','), '', $orderBy);
 
-			foreach ($indices as $index)
-				if (empty($index['db']) || $index['db'] == $this->dbType)
+			foreach ($indices as $index) {
+							if (empty($index['db']) || $index['db'] == $this->dbType)
 					foreach ($index['fields'] as $field)
 						if ($field == $orderBy)
 							return true;
+			}
 
 			$warning = 'Missing Index For Order By Table: ' . $table . ' Order By:' . $orderBy ;
-			if (!empty($GLOBALS['sugar_config']['dump_slow_queries']))
-				$this->log->fatal('CHECK QUERY:' .$warning);
-			else
-				$this->log->warn('CHECK QUERY:' .$warning);
+			if (!empty($GLOBALS['sugar_config']['dump_slow_queries'])) {
+							$this->log->fatal('CHECK QUERY:' .$warning);
+			} else {
+							$this->log->warn('CHECK QUERY:' .$warning);
+			}
 		}
 	}
 	return false;
@@ -467,8 +475,9 @@ 
 	public function checkConnection()
 	{
 		$this->last_error = '';
-		if (!isset($this->database))
-			$this->connect();
+		if (!isset($this->database)) {
+					$this->connect();
+		}
 	}
 
 	/**
@@ -510,9 +519,13 @@ 
 		$values = array();
 		foreach ($field_defs as $field => $fieldDef)
 		{
-			if (isset($fieldDef['source']) && $fieldDef['source'] != 'db')  continue;
+			if (isset($fieldDef['source']) && $fieldDef['source'] != 'db') {
+			    continue;
+			}
 			//custom fields handle there save seperatley
-			if(!empty($field_map) && !empty($field_map[$field]['custom_type'])) continue;
+			if(!empty($field_map) && !empty($field_map[$field]['custom_type'])) {
+			    continue;
+			}
 
 			if(isset($data[$field])) {
 				// clean the incoming value..
@@ -541,8 +554,10 @@ 
 			}
 		}
 
-		if (empty($values))
-			return $execute?true:''; // no columns set
+		if (empty($values)) {
+					return $execute?true:'';
+		}
+		// no columns set
 
 		// get the entire sql
 		$query = "INSERT INTO $table (".implode(",", array_keys($values)).")
@@ -642,9 +657,10 @@ 
 		if(!$this->supports("inline_keys")) {
 		// handle constraints and indices
 			$indicesArr = $this->createConstraintSql($bean);
-			if (count($indicesArr) > 0)
-				foreach ($indicesArr as $indexSql)
+			if (count($indicesArr) > 0) {
+							foreach ($indicesArr as $indexSql)
 					$this->query($indexSql, true, $msg);
+			}
 		}
 	}
 
@@ -681,9 +697,10 @@ 
 			if(!$this->supports("inline_keys")) {
 				// handle constraints and indices
 				$indicesArr = $this->getConstraintSql($indices, $tablename);
-				if (count($indicesArr) > 0)
-					foreach ($indicesArr as $indexSql)
+				if (count($indicesArr) > 0) {
+									foreach ($indicesArr as $indexSql)
 						$res = ($res and $this->query($indexSql, true, "Error creating indexes"));
+				}
 			}
 			return $res;
 		}
@@ -710,13 +727,15 @@ 
 		}
 		//jc: added this for beans that do not actually have a table, namely
 		//ForecastOpportunities
-		if($tablename == 'does_not_exist' || $tablename == '')
-			return '';
+		if($tablename == 'does_not_exist' || $tablename == '') {
+					return '';
+		}
 
 		global $dictionary;
 		$engine=null;
-		if (isset($dictionary[$bean->getObjectName()]['engine']) && !empty($dictionary[$bean->getObjectName()]['engine']) )
-			$engine = $dictionary[$bean->getObjectName()]['engine'];
+		if (isset($dictionary[$bean->getObjectName()]['engine']) && !empty($dictionary[$bean->getObjectName()]['engine']) ) {
+					$engine = $dictionary[$bean->getObjectName()]['engine'];
+		}
 
 		return $this->repairTableParams($tablename, $fielddefs,$new_index,$execute,$engine);
 	}
@@ -760,8 +779,9 @@ 
 	{
 		//jc: had a bug when running the repair if the tablename is blank the repair will
 		//fail when it tries to create a repair table
-		if ($tablename == '' || empty($fielddefs))
-			return '';
+		if ($tablename == '' || empty($fielddefs)) {
+					return '';
+		}
 
 		//if the table does not exist create it and we are done
 		$sql = "/* Table : $tablename */\n";
@@ -784,16 +804,16 @@ 
 		// do column comparisons
 		$sql .=	"/*COLUMNS*/\n";
 		foreach ($fielddefs as $name => $value) {
-			if (isset($value['source']) && $value['source'] != 'db')
-				continue;
+			if (isset($value['source']) && $value['source'] != 'db') {
+							continue;
+			}
 
             // Bug #42406. Skipping breaked vardef without type or name
             if (isset($value['name']) == false || $value['name'] == false)
             {
                 $sql .= "/* NAME IS MISSING IN VARDEF $tablename::$name */\n";
                 continue;
-            }
-            else if (isset($value['type']) == false || $value['type'] == false)
+            } else if (isset($value['type']) == false || $value['type'] == false)
             {
                 $sql .= "/* TYPE IS MISSING IN VARDEF $tablename::$name */\n";
                 continue;
@@ -823,8 +843,9 @@ 
 				// ok we need this field lets create it
 				$sql .=	"/*MISSING IN DATABASE - $name -  ROW*/\n";
 				$sql .= $this->addColumnSQL($tablename, $value) .  "\n";
-				if ($execute)
-					$this->addColumn($tablename, $value);
+				if ($execute) {
+									$this->addColumn($tablename, $value);
+				}
 				$take_action = true;
 			} elseif ( !$this->compareVarDefs($compareFieldDefs[$name],$value)) {
 				//fields are different lets alter it
@@ -880,8 +901,9 @@ 
 		unset($compareIndices_case_insensitive);
 
 		foreach ($indices as $value) {
-			if (isset($value['source']) && $value['source'] != 'db')
-				continue;
+			if (isset($value['source']) && $value['source'] != 'db') {
+							continue;
+			}
 
 
 			$validDBName = $this->getValidDBName($value['name'], true, 'index', true);
@@ -891,19 +913,23 @@ 
 		    $name = strtolower($value['name']);
 
 			//Don't attempt to fix the same index twice in one pass;
-			if (isset($correctedIndexs[$name]))
-				continue;
+			if (isset($correctedIndexs[$name])) {
+							continue;
+			}
 
 			//don't bother checking primary nothing we can do about them
-			if (isset($value['type']) && $value['type'] == 'primary')
-				continue;
+			if (isset($value['type']) && $value['type'] == 'primary') {
+							continue;
+			}
 
 			//database helpers do not know how to handle full text indices
-			if ($value['type']=='fulltext')
-				continue;
+			if ($value['type']=='fulltext') {
+							continue;
+			}
 
-			if ( in_array($value['type'],array('alternate_key','foreign')) )
-				$value['type'] = 'index';
+			if ( in_array($value['type'],array('alternate_key','foreign')) ) {
+							$value['type'] = 'index';
+			}
 
 			if ( !isset($compareIndices[$name]) ) {
 				//First check if an index exists that doesn't match our name, if so, try to rename it
@@ -934,21 +960,23 @@ 
 				$sql .=	"/*INDEX MISMATCH WITH DATABASE - $name -  ROW ";
 				foreach ($compareIndices[$name] as $n1 => $t1) {
 					$sql .=	 "<$n1>";
-					if ( $n1 == 'fields' )
-						foreach($t1 as $rKey => $rValue)
+					if ( $n1 == 'fields' ) {
+											foreach($t1 as $rKey => $rValue)
 							$sql .= "[$rKey] => '$rValue'  ";
-					else
-						$sql .= " $t1 ";
+					} else {
+											$sql .= " $t1 ";
+					}
 				}
 				$sql .=	"*/\n";
 				$sql .=	"/* VARDEF - $name -  ROW";
 				foreach ($value as $n1 => $t1) {
 					$sql .=	"<$n1>";
-					if ( $n1 == 'fields' )
-						foreach ($t1 as $rKey => $rValue)
+					if ( $n1 == 'fields' ) {
+											foreach ($t1 as $rKey => $rValue)
 							$sql .=	"[$rKey] => '$rValue'  ";
-					else
-						$sql .= " $t1 ";
+					} else {
+											$sql .= " $t1 ";
+					}
 				}
 				$sql .=	"*/\n";
 				$sql .= $this->modifyIndexes($tablename,array($value), $execute) .  "\n";
@@ -971,8 +999,9 @@ 
 	public function compareVarDefs($fielddef1, $fielddef2, $ignoreName = false)
 	{
 		foreach ( $fielddef1 as $key => $value ) {
-			if ($key == 'name' && $ignoreName)
-				continue;
+			if ($key == 'name' && $ignoreName) {
+							continue;
+			}
             if (isset($fielddef2[$key]))
             {
                 if (!is_array($fielddef1[$key]) && !is_array($fielddef2[$key]))
@@ -981,8 +1010,7 @@ 
                     {
                         continue;
                     }
-                }
-                else
+                } else
                 {
                     if (array_map('strtolower', $fielddef1[$key]) == array_map('strtolower',$fielddef2[$key]))
                     {
@@ -991,8 +1019,9 @@ 
                 }
             }
 			//Ignore len if its not set in the vardef
-			if ($key == 'len' && empty($fielddef2[$key]))
-				continue;
+			if ($key == 'len' && empty($fielddef2[$key])) {
+							continue;
+			}
             // if the length in db is greather than the vardef, ignore it
             if ($key == 'len' && ($fielddef1[$key] >= $fielddef2[$key])) {
                 continue;
@@ -1026,12 +1055,10 @@ 
 			if (!$row2) {
 				// Exists on table1 but not table2
 				$returnArray['msg'] = 'not_exists_table2';
-			}
-			else {
+			} else {
 				if (sizeof($row1) != sizeof($row2)) {
 					$returnArray['msg'] = 'no_match';
-				}
-				else {
+				} else {
 					$returnArray['msg'] = 'match';
 					foreach($row1 as $key => $value){
 						//ignore keys when checking we will check them when we do the index check
@@ -1041,8 +1068,7 @@ 
 					}
 				}
 			}
-		}
-		else {
+		} else {
 			$returnArray['msg'] = 'not_exists_table1';
 		}
 
@@ -1122,16 +1148,19 @@ 
 	 */
 	public function getConstraintSql($indices, $table)
 	{
-		if (!$this->isFieldArray($indices))
-			$indices = array($indices);
+		if (!$this->isFieldArray($indices)) {
+					$indices = array($indices);
+		}
 
 		$columns = array();
 
 		foreach ($indices as $index) {
-			if(!empty($index['db']) && $index['db'] != $this->dbType)
-				continue;
-			if (isset($index['source']) && $index['source'] != 'db')
-			continue;
+			if(!empty($index['db']) && $index['db'] != $this->dbType) {
+							continue;
+			}
+			if (isset($index['source']) && $index['source'] != 'db') {
+						continue;
+			}
 
 			$sql = $this->add_drop_constraint($table, $index);
 
@@ -1221,11 +1250,11 @@ 
 		$sql = $this->addColumnSQL($tablename, $fieldDefs);
 		if ($this->isFieldArray($fieldDefs)){
 			$columns = array();
-			foreach ($fieldDefs as $fieldDef)
-				$columns[] = $fieldDef['name'];
+			foreach ($fieldDefs as $fieldDef) {
+							$columns[] = $fieldDef['name'];
+			}
 			$columns = implode(",", $columns);
-		}
-		else {
+		} else {
 			$columns = $fieldDefs['name'];
 		}
 		$msg = "Error adding column(s) $columns on table: $tablename:";
@@ -1249,8 +1278,7 @@ 
 				$columns[] = $fieldDef['name'];
 			}
 			$columns = implode(",", $columns);
-		}
-		else {
+		} else {
 			$columns = $newFieldDef['name'];
 		}
 
@@ -1370,7 +1398,9 @@ 
 			if(!$is_related_query){
 				foreach ($fields as $fieldDef)
 				{
-					if(isset($fieldDef['source']) && $fieldDef['source'] != 'db' && $fieldDef['source'] != 'custom_fields') continue;
+					if(isset($fieldDef['source']) && $fieldDef['source'] != 'db' && $fieldDef['source'] != 'custom_fields') {
+					    continue;
+					}
 					$val = $row[$fieldDef['name']];
 
 					//handle auto increment values here only need to do this on insert not create
@@ -1379,8 +1409,7 @@ 
 							if(!$built_columns){
 							$columns[] = 'deleted';
 						}
-					}
-					else
+					} else
 					{
 						$type = $fieldDef['type'];
 						if(!empty($fieldDef['custom_type'])){
@@ -1388,28 +1417,32 @@ 
 						}
 						// need to do some thing about types of values
 						if($this->dbType == 'mysql' && $val == '' && ($type == 'datetime' ||  $type == 'date' || $type == 'int' || $type == 'currency' || $type == 'decimal')){
-							if(!empty($custom_fields[$fieldDef['name']]))
-								$cstm_values[$fieldDef['name']] = 'null';
-							else
-								$values[$fieldDef['name']] = 'null';
-						}else{
+							if(!empty($custom_fields[$fieldDef['name']])) {
+															$cstm_values[$fieldDef['name']] = 'null';
+							} else {
+															$values[$fieldDef['name']] = 'null';
+							}
+						} else{
 							if(isset($type) && $type=='int') {
-								if(!empty($custom_fields[$fieldDef['name']]))
-									$cstm_values[$fieldDef['name']] = $GLOBALS['db']->quote(from_html($val));
-								else
-									$values[$fieldDef['name']] = $GLOBALS['db']->quote(from_html($val));
+								if(!empty($custom_fields[$fieldDef['name']])) {
+																	$cstm_values[$fieldDef['name']] = $GLOBALS['db']->quote(from_html($val));
+								} else {
+																	$values[$fieldDef['name']] = $GLOBALS['db']->quote(from_html($val));
+								}
 							} else {
-								if(!empty($custom_fields[$fieldDef['name']]))
-									$cstm_values[$fieldDef['name']] = "'".$GLOBALS['db']->quote(from_html($val))."'";
-								else
-									$values[$fieldDef['name']] = "'".$GLOBALS['db']->quote(from_html($val))."'";
+								if(!empty($custom_fields[$fieldDef['name']])) {
+																	$cstm_values[$fieldDef['name']] = "'".$GLOBALS['db']->quote(from_html($val))."'";
+								} else {
+																	$values[$fieldDef['name']] = "'".$GLOBALS['db']->quote(from_html($val))."'";
+								}
 							}
 						}
 						if(!$built_columns){
-							if(!empty($custom_fields[$fieldDef['name']]))
-								$cstm_columns[] = $fieldDef['name'];
-							else
-								$columns[] = $fieldDef['name'];
+							if(!empty($custom_fields[$fieldDef['name']])) {
+															$cstm_columns[] = $fieldDef['name'];
+							} else {
+															$columns[] = $fieldDef['name'];
+							}
 						}
 					}
 
@@ -1624,7 +1657,9 @@ 
 		    $queryresult = $this->query($sql, $dieOnError, $msg);
 		}
 		$this->checkError($msg.' Get One Failed:' . $sql, $dieOnError);
-		if (!$queryresult) return false;
+		if (!$queryresult) {
+		    return false;
+		}
 		$row = $this->fetchByAssoc($queryresult);
 		if(!empty($row)) {
 			return array_shift($row);
@@ -1648,10 +1683,14 @@ 
 		$queryresult = $this->query($sql, $dieOnError, $msg);
 		$this->checkError($msg.' Fetch One Failed:' . $sql, $dieOnError);
 
-		if (!$queryresult) return false;
+		if (!$queryresult) {
+		    return false;
+		}
 
 		$row = $this->fetchByAssoc($queryresult);
-		if ( !$row ) return false;
+		if ( !$row ) {
+		    return false;
+		}
 
 		$this->freeResult($queryresult);
 		return $row;
@@ -1708,13 +1747,15 @@ 
 	protected function describeField($name, $tablename)
 	{
 		$table = $this->getTableDescription($tablename);
-		if(!empty($table) && isset($table[$name]))
-			return 	$table[$name];
+		if(!empty($table) && isset($table[$name])) {
+					return 	$table[$name];
+		}
 
 		$table = $this->getTableDescription($tablename, true);
 
-		if(isset($table[$name]))
-		return $table[$name];
+		if(isset($table[$name])) {
+				return $table[$name];
+		}
 
 		return array();
 	}
@@ -1768,11 +1809,15 @@ 
      */
 	public function concat($table, array $fields, $space = ' ')
 	{
-		if(empty($fields)) return '';
+		if(empty($fields)) {
+		    return '';
+		}
 		$elems = array();
 		$space = $this->quoted($space);
 		foreach ( $fields as $field ) {
-			if(!empty($elems)) $elems[] = $space;
+			if(!empty($elems)) {
+			    $elems[] = $space;
+			}
 			$elems[] = $this->convert("$table.$field", 'IFNULL', array("''"));
 		}
 		$first = array_shift($elems);
@@ -1864,7 +1909,7 @@ 
 				}//switch
 			}//foreach
 			return $this->query($query);
-		}else{
+		} else{
 			return false;
 		}
 	}
@@ -1926,20 +1971,30 @@ 
         $fields = $bean->getFieldDefinitions();
 		// get column names and values
 		foreach ($fields as $field => $fieldDef) {
-			if (isset($fieldDef['source']) && $fieldDef['source'] != 'db')  continue;
+			if (isset($fieldDef['source']) && $fieldDef['source'] != 'db') {
+			    continue;
+			}
 			// Do not write out the id field on the update statement.
     		// We are not allowed to change ids.
-    		if (empty($fieldDef['name']) || $fieldDef['name'] == $primaryField['name']) continue;
+    		if (empty($fieldDef['name']) || $fieldDef['name'] == $primaryField['name']) {
+    		    continue;
+    		}
 
     		// If the field is an auto_increment field, then we shouldn't be setting it.  This was added
     		// specially for Bugs and Cases which have a number associated with them.
-    		if (!empty($bean->field_name_map[$field]['auto_increment'])) continue;
+    		if (!empty($bean->field_name_map[$field]['auto_increment'])) {
+    		    continue;
+    		}
 
     		//custom fields handle their save separately
-    		if(isset($bean->field_name_map) && !empty($bean->field_name_map[$field]['custom_type']))  continue;
+    		if(isset($bean->field_name_map) && !empty($bean->field_name_map[$field]['custom_type'])) {
+    		    continue;
+    		}
 
     		// no need to clear deleted since we only update not deleted records anyway
-    		if($fieldDef['name'] == 'deleted' && empty($bean->deleted)) continue;
+    		if($fieldDef['name'] == 'deleted' && empty($bean->deleted)) {
+    		    continue;
+    		}
 
     		if(isset($bean->$field)) {
     			$val = from_html($bean->$field);
@@ -1972,8 +2027,10 @@ 
     		}
 		}
 
-		if ( sizeof($columns) == 0 )
-			return ""; // no columns set
+		if ( sizeof($columns) == 0 ) {
+					return "";
+		}
+		// no columns set
 
 		// build where clause
 		$where = $this->getWhereClause($bean, $this->updateWhereArray($bean, $where));
@@ -2039,8 +2096,9 @@ 
 			$where[] = " $table.$name $op $val";
 		}
 
-		if (!empty($where))
-			return implode(" AND ", $where);
+		if (!empty($where)) {
+					return implode(" AND ", $where);
+		}
 
 		return '';
 	}
@@ -2144,21 +2202,25 @@ 
 	public function massageFieldDef(&$fieldDef, $tablename)
 	{
 		if ( !isset($fieldDef['dbType']) ) {
-			if ( isset($fieldDef['dbtype']) )
-				$fieldDef['dbType'] = $fieldDef['dbtype'];
-			else
-				$fieldDef['dbType'] = $fieldDef['type'];
+			if ( isset($fieldDef['dbtype']) ) {
+							$fieldDef['dbType'] = $fieldDef['dbtype'];
+			} else {
+							$fieldDef['dbType'] = $fieldDef['type'];
+			}
 		}
 		$type = $this->getColumnType($fieldDef['dbType'],$fieldDef['name'],$tablename);
 		$matches = array();
         // len can be a number or a string like 'max', for example, nvarchar(max)
         preg_match_all('/(\w+)(?:\(([0-9]+,?[0-9]*|\w+)\)|)/i', $type, $matches);
-		if ( isset($matches[1][0]) )
-			$fieldDef['type'] = $matches[1][0];
-		if ( isset($matches[2][0]) && empty($fieldDef['len']) )
-			$fieldDef['len'] = $matches[2][0];
-		if ( !empty($fieldDef['precision']) && is_numeric($fieldDef['precision']) && !strstr($fieldDef['len'],',') )
-			$fieldDef['len'] .= ",{$fieldDef['precision']}";
+		if ( isset($matches[1][0]) ) {
+					$fieldDef['type'] = $matches[1][0];
+		}
+		if ( isset($matches[2][0]) && empty($fieldDef['len']) ) {
+					$fieldDef['len'] = $matches[2][0];
+		}
+		if ( !empty($fieldDef['precision']) && is_numeric($fieldDef['precision']) && !strstr($fieldDef['len'],',') ) {
+					$fieldDef['len'] .= ",{$fieldDef['precision']}";
+		}
 		if (!empty($fieldDef['required']) || ($fieldDef['name'] == 'id' && !isset($fieldDef['required'])) ) {
 			$fieldDef['required'] = 'true';
 		}
@@ -2172,8 +2234,9 @@ 
 	public function getSelectFieldsFromQuery($selectStatement)
 	{
 		$selectStatement = trim($selectStatement);
-		if (strtoupper(substr($selectStatement, 0, 6)) == "SELECT")
-			$selectStatement = trim(substr($selectStatement, 6));
+		if (strtoupper(substr($selectStatement, 0, 6)) == "SELECT") {
+					$selectStatement = trim(substr($selectStatement, 6));
+		}
 
 		//Due to sql functions existing in many selects, we can't use php explode
 		$fields = array();
@@ -2189,17 +2252,15 @@ 
 				$field = $this->getFieldNameFromSelect(trim($selectField));
 				$fields[$field] = $selectField;
 				$selectField = "";
-			}
-			else if ($char == "("){
+			} else if ($char == "("){
 				$level++;
 				$selectField .= $char;
-			}
-			else if($char == ")"){
+			} else if($char == ")"){
 				$level--;
 				$selectField .= $char;
 
 
-			}else{
+			} else{
 				$selectField .= $char;
 			}
 
@@ -2218,18 +2279,19 @@ 
 		if(strncasecmp($string, "DISTINCT ", 9) == 0) {
 			$string = substr($string, 9);
 		}
-		if (stripos($string, " as ") !== false)
-			//"as" used for an alias
+		if (stripos($string, " as ") !== false) {
+					//"as" used for an alias
 			return trim(substr($string, strripos($string, " as ") + 4));
-		else if (strrpos($string, " ") != 0)
-			//Space used as a delimiter for an alias
+		} else if (strrpos($string, " ") != 0) {
+					//Space used as a delimiter for an alias
 			return trim(substr($string, strrpos($string, " ")));
-		else if (strpos($string, ".") !== false)
-			//No alias, but a table.field format was used
+		} else if (strpos($string, ".") !== false) {
+					//No alias, but a table.field format was used
 			return substr($string, strpos($string, ".") + 1);
-		else
-			//Give up and assume the whole thing is the field name
+		} else {
+					//Give up and assume the whole thing is the field name
 			return $string;
+		}
 	}
 
 	/**
@@ -2289,8 +2351,9 @@ 
 			$aliases[$tableName][] = $table;
 
 			// build part of select for this table
-			if (is_array($cols[$beanID]))
-				foreach ($cols[$beanID] as $def) $select[] = $table.".".$def['name'];
+			if (is_array($cols[$beanID])) {
+							foreach ($cols[$beanID] as $def) $select[] = $table.".".$def['name'];
+			}
 
 			// build part of where clause
 			if (is_array($whereClause[$beanID])){
@@ -2322,7 +2385,10 @@ 
 		$separator = "";
 		$from = ''; $table_used_in_from = array();
 		foreach ($relations as $table1 => $rightsidearray){
-			if ($table_used_in_from[$table1]) continue; // table has been joined
+			if ($table_used_in_from[$table1]) {
+			    continue;
+			}
+			// table has been joined
 
 			$from .= $separator." ".$table1;
 			$table_used_in_from[$table1] = true;
@@ -2333,7 +2399,9 @@ 
 					//choose first alias that does not match
 					// we are doing this because of self joins.
 					// in case of self joins, the same table will have many aliases.
-					if ($table2 != $table1) break;
+					if ($table2 != $table1) {
+					    break;
+					}
 				}
 
 				$col = $tablearray['foreingColumn'];
@@ -2362,11 +2430,13 @@ 
 		$tablename = $bean->getTableName();
 		$columns = array();
 		// get column names
-		foreach ($fields as $fieldDef)
-			$columns[] = $fieldDef['name'];
+		foreach ($fields as $fieldDef) {
+					$columns[] = $fieldDef['name'];
+		}
 
-		if (empty($columns))
-			return "";
+		if (empty($columns)) {
+					return "";
+		}
 
 		$columns = implode(",", $columns);
 
@@ -2386,16 +2456,21 @@ 
 		// we do not have change a lot of existing code
 		// and add dbtype where type is being used for some special
 		// purposes like referring to foreign table etc.
-		if(!empty($fieldDef['dbType']))
-			return  $fieldDef['dbType'];
-		if(!empty($fieldDef['dbtype']))
-			return  $fieldDef['dbtype'];
-		if (!empty($fieldDef['type']))
-			return  $fieldDef['type'];
-		if (!empty($fieldDef['Type']))
-			return  $fieldDef['Type'];
-		if (!empty($fieldDef['data_type']))
-			return  $fieldDef['data_type'];
+		if(!empty($fieldDef['dbType'])) {
+					return  $fieldDef['dbType'];
+		}
+		if(!empty($fieldDef['dbtype'])) {
+					return  $fieldDef['dbtype'];
+		}
+		if (!empty($fieldDef['type'])) {
+					return  $fieldDef['type'];
+		}
+		if (!empty($fieldDef['Type'])) {
+					return  $fieldDef['Type'];
+		}
+		if (!empty($fieldDef['data_type'])) {
+					return  $fieldDef['data_type'];
+		}
 
 		return null;
 	}
@@ -2453,14 +2528,15 @@ 
                                           'clob', 'blob', 'text'))) {
           	    $colType = "$colBaseType(${fieldDef['len']})";
             } elseif(($colBaseType == 'decimal' || $colBaseType == 'float')){
-                  if(!empty($fieldDef['precision']) && is_numeric($fieldDef['precision']))
-                      if(strpos($fieldDef['len'],',') === false){
+                  if(!empty($fieldDef['precision']) && is_numeric($fieldDef['precision'])) {
+                                        if(strpos($fieldDef['len'],',') === false){
                           $colType = $colBaseType . "(".$fieldDef['len'].",".$fieldDef['precision'].")";
-                      }else{
-                          $colType = $colBaseType . "(".$fieldDef['len'].")";
-                      }
-                  else
+                  }
+                      } else{
                           $colType = $colBaseType . "(".$fieldDef['len'].")";
+                      } else {
+                                            $colType = $colBaseType . "(".$fieldDef['len'].")";
+                  }
               }
         } else {
             if (in_array($colBaseType, array( 'nvarchar', 'nchar', 'varchar', 'varchar2', 'char'))) {
@@ -2474,19 +2550,18 @@ 
         if (!empty($fieldDef['no_default']))
         {
             // nothing to do
-        }
-        elseif (isset($fieldDef['default']) && strlen($fieldDef['default']) > 0)
+        } elseif (isset($fieldDef['default']) && strlen($fieldDef['default']) > 0)
         {
             $default = " DEFAULT ".$this->quoted($fieldDef['default']);
-        }
-        elseif (!isset($default) && $type == 'bool')
+        } elseif (!isset($default) && $type == 'bool')
         {
             $default = " DEFAULT 0 ";
         }
 
 		$auto_increment = '';
-		if(!empty($fieldDef['auto_increment']) && $fieldDef['auto_increment'])
-			$auto_increment = $this->setAutoIncrement($table , $fieldDef['name']);
+		if(!empty($fieldDef['auto_increment']) && $fieldDef['auto_increment']) {
+					$auto_increment = $this->setAutoIncrement($table , $fieldDef['name']);
+		}
 
 		$required = 'NULL';  // MySQL defaults to NULL, SQL Server defaults to NOT NULL -- must specify
 		//Starting in 6.0, only ID and auto_increment fields will be NOT NULL in the DB.
@@ -2500,8 +2575,9 @@ 
 			&& !empty($fieldDef['required'])) {
 			$required =  "NOT NULL";
 		}
-		if ($ignoreRequired)
-			$required = "";
+		if ($ignoreRequired) {
+					$required = "";
+		}
 
 		if ( $return_as_array ) {
 			return array(
@@ -2537,8 +2613,7 @@ 
 				}
 			}
 			$columns = implode(",", $columns);
-		}
-		else {
+		} else {
 			$columns = $this->oneColumnSQLRep($fieldDefs,$ignoreRequired, $tablename);
 		}
 
@@ -2760,8 +2835,9 @@ 
 	 */
 	public function isFieldArray($defArray)
 	{
-		if ( !is_array($defArray) )
-			return false;
+		if ( !is_array($defArray) ) {
+					return false;
+		}
 
 		if ( isset($defArray['type']) ){
 			// type key exists. May be an array of defs or a simple definition
@@ -2872,11 +2948,9 @@ 
                 } else {
                     if (isset($properties['dbType'])) {
                         $field_type=$properties['dbType'];
-                    }
-                    else if(isset($properties['data_type'])) {
+                    } else if(isset($properties['data_type'])) {
                         $field_type=$properties['data_type'];
-                    }
-                    else {
+                    } else {
                         $field_type=$properties['dbtype'];
                     }
                 }
@@ -2901,13 +2975,11 @@ 
                             if ($error >= 0.0000000001) {    // Smaller than 10E-10
                                 $change = true;
                             }
-                        }
-                        else if ($this->isBooleanType($field_type)) {
+                        } else if ($this->isBooleanType($field_type)) {
                             if ($this->_getBooleanValue($before_value) != $this->_getBooleanValue($after_value)) {
                                 $change = true;
                             }
-                        }
-                        else {
+                        } else {
                             $change = true;
                         }
                         if ($change) {
@@ -3005,8 +3077,9 @@ 
 	protected function _getBooleanValue($val)
 	{
 		//need to put the === sign here otherwise true == 'non empty string'
-		if (empty($val) or $val==='off')
-			return false;
+		if (empty($val) or $val==='off') {
+					return false;
+		}
 
 		return true;
 	}
@@ -3032,8 +3105,9 @@ 
      */
 	protected function _emptyValue($val, $type)
 	{
-		if (empty($val))
-			return true;
+		if (empty($val)) {
+					return true;
+		}
 
 		if($this->emptyValue($type) == $val) {
 			return true;
@@ -3050,10 +3124,12 @@ 
 			case 'short':
 				return ($val == 0);
 			case 'date':
-				if ($val == '0000-00-00')
-					return true;
-				if ($val == 'NULL')
-					return true;
+				if ($val == '0000-00-00') {
+									return true;
+				}
+				if ($val == 'NULL') {
+									return true;
+				}
 				return false;
 		}
 
@@ -3407,7 +3483,9 @@ 
 	 */
 	public function fetchByAssoc($result, $encode = true)
 	{
-	    if (empty($result))	return false;
+	    if (empty($result)) {
+	        return false;
+	    }
 
 	    if(is_int($encode) && func_num_args() == 3) {
 	        // old API: $result, $rowNum, $encode

modules/AOR_Reports/AOR_Report.php

Braces +43 added lines, -36 removed lines

@@ -61,8 +61,7 @@ 
         $deprecatedMessage = 'PHP4 Style Constructors are deprecated and will be remove in 7.8, please update your code';
         if(isset($GLOBALS['log'])) {
             $GLOBALS['log']->deprecated($deprecatedMessage);
-        }
-        else {
+        } else {
             trigger_error($deprecatedMessage, E_USER_DEPRECATED);
         }
         self::__construct();
@@ -232,8 +231,10 @@ 
                 $html = '<script src="modules/AOR_Reports/js/Chart.js"></script>';
                 break;
             case self::CHART_TYPE_RGRAPH:
-                if($_REQUEST['module']!= 'Home')//Need the require_once for the rgraphincludes as they are only loaded when the home page is hit
+                if($_REQUEST['module']!= 'Home') {
+                    //Need the require_once for the rgraphincludes as they are only loaded when the home page is hit
                     require_once('include/SuiteGraphs/RGraphIncludes.php');
+                }
 
                 break;
         }
@@ -256,15 +257,13 @@ 
 
         if(count($rows) > 1) {
             $GLOBALS['log']->fatal('ambiguous group display for report ' . $this->id);
-        }
-        else if(count($rows) == 1){
+        } else if(count($rows) == 1){
             $rows[0]['module_path'] = unserialize(base64_decode($rows[0]['module_path']));
             if(!$rows[0]['module_path'][0]) {
                 $module = new $beanList[$this->report_module]();
                 $rows[0]['field_id_name'] = $module->field_defs[$rows[0]['field']]['id_name'] ? $module->field_defs[$rows[0]['field']]['id_name'] : $module->field_defs[$rows[0]['field']]['name'];
                 $rows[0]['module_path'][0] = $module->table_name;
-            }
-            else {
+            } else {
                 $rows[0]['field_id_name'] = $rows[0]['field'];
             }
             $path[] = $rows[0];
@@ -276,8 +275,7 @@ 
             }
 
             return $this->buildMultiGroupReport($offset, $links, $level+1, $path);
-        }
-        else if(!$rows) {
+        } else if(!$rows) {
             if($path) {
                 $html = '';
                 foreach ($path as $pth) {
@@ -295,12 +293,10 @@ 
                     }
                 }
                 return $html;
-            }
-            else {
+            } else {
                 return $this->build_group_report($offset, $links);
             }
-        }
-        else {
+        } else {
             throw new Exception('incorrect results');
         }
         throw new Exception('incorrect state');
@@ -436,8 +432,7 @@ 
                 foreach($extra['select'] as $selectField => $selectAlias) {
                     if($selectAlias) {
                         $query_array['select'][] = $selectField . " AS " . $selectAlias;
-                    }
-                    else {
+                    } else {
                         $query_array['select'][] = $selectField;
                     }
                 }
@@ -492,14 +487,18 @@ 
             $result = $this->db->query($query);
 
             while ($row = $this->db->fetchByAssoc($result)) {
-                if($html != '') $html .= '<br />';
+                if($html != '') {
+                    $html .= '<br />';
+                }
 
                $html .= $this->build_report_html($offset, $links, $row[$field_label], '', $extra);
 
             }
         }
 
-        if($html == '') $html = $this->build_report_html($offset, $links);
+        if($html == '') {
+            $html = $this->build_report_html($offset, $links);
+        }
         return $html;
 
     }
@@ -693,7 +692,9 @@ 
                     if($att['total']){
                         $totals[$name][] = $row[$name];
                     }
-                    if($att['link'] && $links) $html .= "</a>";
+                    if($att['link'] && $links) {
+                        $html .= "</a>";
+                    }
                     $html .= "</td>";
                 }
             }
@@ -781,7 +782,7 @@ 
             if($field['total']){
                 $totalLabel = $field['label'] ." ".$app_list_strings['aor_total_options'][$field['total']];
                 $html .= "<th>{$totalLabel}</th>";
-            }else{
+            } else{
                 $html .= "<th></th>";
             }
         }
@@ -824,7 +825,7 @@ 
                         break;
                 }
                 $html .= "<td>".$total."</td>";
-            }else{
+            } else{
                 $html .= "<td></td>";
             }
         }
@@ -900,10 +901,11 @@ 
             $csv .= "\r\n";
             foreach($fields as $name => $att){
                 if($att['display']){
-                    if($att['function'] != '' )
-                        $csv .= $this->encloseForCSV($row[$name]);
-                    else
-                        $csv .= $this->encloseForCSV(trim(strip_tags(getModuleField($att['module'], $att['field'], $att['field'], 'DetailView',$row[$name]))));
+                    if($att['function'] != '' ) {
+                                            $csv .= $this->encloseForCSV($row[$name]);
+                    } else {
+                                            $csv .= $this->encloseForCSV(trim(strip_tags(getModuleField($att['module'], $att['field'], $att['field'], 'DetailView',$row[$name]))));
+                    }
                     $csv .= $delimiter;
                 }
             }
@@ -1087,7 +1089,9 @@ 
 
                 $query['select'][] = $select_field ." AS '".$field->label."'";
 
-                if($field->group_display == 1 && $group_value) $query['where'][] = $select_field." = '".$group_value."' AND ";
+                if($field->group_display == 1 && $group_value) {
+                    $query['where'][] = $select_field." = '".$group_value."' AND ";
+                }
                     ++$i;
             }
         }
@@ -1112,13 +1116,13 @@ 
                                 $params['right_join_table_alias'] = $this->db->quoteIdentifier($alias);
                                 $params['join_table_alias'] = $this->db->quoteIdentifier($alias);
                                 $params['left_join_table_alias'] = $this->db->quoteIdentifier($parentAlias);
-                            }else{
+                            } else{
                                 $params['right_join_table_alias'] = $this->db->quoteIdentifier($parentAlias);
                                 $params['join_table_alias'] = $this->db->quoteIdentifier($alias);
                                 $params['left_join_table_alias'] = $this->db->quoteIdentifier($alias);
                             }
 
-                        }else{
+                        } else{
                             $params['right_join_table_alias'] = $this->db->quoteIdentifier($parentAlias);
                             $params['join_table_alias'] = $this->db->quoteIdentifier($alias);
                             $params['left_join_table_alias'] = $this->db->quoteIdentifier($parentAlias);
@@ -1339,12 +1343,16 @@ 
 
                         case 'Multi':
                             $sep = ' AND ';
-                            if ($condition->operator == 'Equal_To') $sep = ' OR ';
+                            if ($condition->operator == 'Equal_To') {
+                                $sep = ' OR ';
+                            }
                             $multi_values = unencodeMultienum($condition->value);
                             if (!empty($multi_values)) {
                                 $value = '(';
                                 foreach ($multi_values as $multi_value) {
-                                    if ($value != '(') $value .= $sep;
+                                    if ($value != '(') {
+                                        $value .= $sep;
+                                    }
                                     $value .= $field . ' ' . $app_list_strings['aor_sql_operator_list'][$condition->operator] . " '" . $multi_value . "'";
                                 }
                                 $value .= ')';
@@ -1415,22 +1423,21 @@ 
                                     break;
                             }
                         } else {
-                            if (!$where_set) $query['where'][] = ($tiltLogicOp ? '' : ($condition->logic_op ? $condition->logic_op . ' ': 'AND ')) . $field . ' ' . $app_list_strings['aor_sql_operator_list'][$condition->operator] . ' ' . $value;
+                            if (!$where_set) {
+                                $query['where'][] = ($tiltLogicOp ? '' : ($condition->logic_op ? $condition->logic_op . ' ': 'AND ')) . $field . ' ' . $app_list_strings['aor_sql_operator_list'][$condition->operator] . ' ' . $value;
+                            }
                         }
                     }
                     $tiltLogicOp = false;
-                }
-                else if($condition->parenthesis) {
+                } else if($condition->parenthesis) {
                     if($condition->parenthesis == 'START') {
                         $query['where'][] = ($tiltLogicOp ? '' : ($condition->logic_op ? $condition->logic_op . ' ' : 'AND ')) .  '(';
                         $tiltLogicOp = true;
-                    }
-                    else {
+                    } else {
                         $query['where'][] = ')';
                         $tiltLogicOp = false;
                     }
-                }
-                else {
+                } else {
                     $GLOBALS['log']->debug('illegal condition');
                 }
 

modules/EmailTemplates/EmailTemplateData.php

Braces +7 added lines, -9 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');
+}
 
 function handleAttachmentForRemove() {
     if(!empty($_REQUEST['attachmentsRemove'])) {
@@ -60,8 +62,7 @@ 
                     $marketing = BeanFactory::getBean('EmailMarketing', $marketingId);
                     $marketing->template_id = $emailTemplateId;
                     $marketing->save();
-                }
-                else {
+                } else {
                     // TODO something is not OK, the selected campaign isn't related to this marketing!!
                     $GLOBALS['log']->debug('Selected marketing not found!');
                 }
@@ -75,8 +76,7 @@ 
             foreach($bean as $key => $value) {
                 if(in_array($key, $fields)) {
                     $newBean->$key = $_POST[$key];
-                }
-                else if(in_array($key, $fieldsForCopy)) {
+                } else if(in_array($key, $fieldsForCopy)) {
                     $newBean->$key = $bean->$key;
                 }
             }
@@ -125,16 +125,14 @@ 
                     }
                     $data['attachments'] = $attachments;
                 }
-            }
-            else {
+            } else {
                 $error = 'Email Template not found.';
             }
             break;
     }
 
 
-}
-else {
+} else {
     $error = 'Illegal GUID format.';
 }
 

modules/Campaigns/WizardNewsletterSave.php

Braces +8 added lines, -9 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.
@@ -53,8 +55,7 @@ 
     $campaign_focus = new Campaign();
     if(isset($_REQUEST['record']) && $_REQUEST['record'] && !(isset($_REQUEST['campaign_id']) && $_REQUEST['campaign_id'])) {
         $campaign_focus->retrieve($_REQUEST['record']);
-    }
-    else if(isset($_REQUEST['campaign_id']) && $_REQUEST['campaign_id']) {
+    } else if(isset($_REQUEST['campaign_id']) && $_REQUEST['campaign_id']) {
         $campaign_focus->retrieve($_REQUEST['campaign_id']);
     }
 
@@ -230,7 +231,7 @@ 
                          $action = 'WizardMarketing';
                         break;
                 }
-            }else{
+            } else{
                 $action = 'WizardHome&record='.$campaign_focus->id;
             }
             //require_once('modules/Campaigns/WizardMarketing.php');
@@ -240,8 +241,7 @@ 
             if(preg_match('/\s*Location:\s*(.*)$/', $header_URL, $matches)) {
                 $href = $matches[1];
                 SugarApplication::redirect($href);
-            }
-            else {
+            } else {
                 header($header_URL);
             }
 
@@ -402,7 +402,7 @@ 
                         $action = 'WizardHome';
                         break;
                 }
-            }else{
+            } else{
                 $action = 'WizardHome&record='.$campaign_focus->id;
             }
             //require_once('modules/Campaigns/WizardMarketing.php');
@@ -411,8 +411,7 @@ 
             if(preg_match('/\s*Location:\s*(.*)$/', $header_URL, $matches)) {
                 $href = $matches[1];
                 SugarApplication::redirect($href);
-            }
-            else {
+            } else {
                 header($header_URL);
             }
 

modules/Campaigns/WebToLeadCapture.php

Braces +15 added lines, -21 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.
@@ -157,8 +159,7 @@ 
 		        $lead->campaigns->add($camplog->id);
                 if(!empty($GLOBALS['check_notify'])) {
                     $lead->save($GLOBALS['check_notify']);
-                }
-                else {
+                } else {
                     $lead->save(FALSE);
                 }
             }
@@ -188,14 +189,14 @@ 
 				$get_and_post = array_merge($_GET, $_POST);
 				foreach($get_and_post as $param => $value) {
 
-					if($param == 'redirect_url' && $param == 'submit')
-						continue;
+					if($param == 'redirect_url' && $param == 'submit') {
+											continue;
+					}
 					
 					if($first_iteration){
 						$first_iteration = false;
 						$query_string .= $first_char;
-					}
-					else{
+					} else{
 						$query_string .= "&";
 					}
 					$query_string .= "{$param}=".urlencode($value);
@@ -203,8 +204,7 @@ 
 				if(empty($lead)) {
 					if($first_iteration){
 						$query_string .= $first_char;
-					}
-					else{
+					} else{
 						$query_string .= "&";
 					}
 					$query_string .= "error=1";
@@ -229,29 +229,25 @@ 
     				}
     				echo '</form><script language="javascript" type="text/javascript">document.redirect.submit();</script>';
     				echo '</body></html>';
-    			}
-				else{
+    			} else{
     				$header_URL = "Location: {$redirect_url}";
 
 					if(preg_match('/\s*Location:\s*(.*)$/', $header_URL, $matches)) {
 						$href = $matches[1];
 						SugarApplication::redirect($href);
-					}
-					else {
+					} else {
 						header($header_URL);
 					}
 
     				die();
 			    }
-			}
-			else{
+			} else{
 				echo $mod_strings['LBL_THANKS_FOR_SUBMITTING'];
 			}
 			sugar_cleanup();
 			// die to keep code from running into redirect case below
 			die();
-	    }
-	   else{
+	    } else{
 	  	  echo $mod_strings['LBL_SERVER_IS_CURRENTLY_UNAVAILABLE'];
 	  }
 }
@@ -262,15 +258,13 @@ 
     	echo '<form name="redirect" action="' .$_POST['redirect']. '" method="GET">';
     	echo '</form><script language="javascript" type="text/javascript">document.redirect.submit();</script>';
     	echo '</body></html>';
-    }
-    else{
+    } else{
     	$header_URL = "Location: {$_POST['redirect']}";
 
 		if(preg_match('/\s*Location:\s*(.*)$/', $header_URL, $matches)) {
 			$href = $matches[1];
 			SugarApplication::redirect($href);
-		}
-		else {
+		} else {
 			header($header_URL);
 		}
 

modules/Campaigns/WebToPersonCapture.php

Braces +2 added lines, -4 removed lines

@@ -240,8 +240,7 @@ 
                 if(preg_match('/\s*Location:\s*(.*)$/', $header_URL, $matches)) {
                     $href = $matches[1];
                     SugarApplication::redirect($href);
-                }
-                else {
+                } else {
                     header($header_URL);
                 }
 
@@ -276,8 +275,7 @@ 
         if(preg_match('/\s*Location:\s*(.*)$/', $header_URL, $matches)) {
             $href = $matches[1];
             SugarApplication::redirect($href);
-        }
-        else {
+        } else {
             header($header_URL);
         }
 

modules/Campaigns/WizardCampaignSave.php

Braces +6 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.
@@ -44,8 +46,7 @@ 
     $msgs = array();
     if(!$templateId) {
         $msgs[] = 'LBL_NO_SELECTED_TEMPLATE';
-    }
-    else {
+    } else {
         $template = new EmailTemplate();
         $template->retrieve($templateId);
         if (!$template->subject) {
@@ -67,8 +68,7 @@ 
 if($func == 'getTemplateValidation') {
     if (!empty($_POST['templateId'])) {
         $templateId = $db->quote($_POST['templateId']);
-    }
-    else {
+    } else {
         if (!$marketingId) {
             if (!empty($_SESSION['campaignWizard'][$campaignId]['defaultSelectedMarketingId']) && $func != 'createEmailMarketing') {
                 $marketingId = $_SESSION['campaignWizard'][$campaignId]['defaultSelectedMarketingId'];
@@ -83,8 +83,7 @@ 
     $return['marketingValidationMessages'] = $marketing->validate();
 
     echo json_encode($return);
-}
-else {
+} else {
     if (!$marketingId) {
         if (!empty($_SESSION['campaignWizard'][$campaignId]['defaultSelectedMarketingId']) && $func != 'createEmailMarketing') {
             $marketingId = $_SESSION['campaignWizard'][$campaignId]['defaultSelectedMarketingId'];

modules/Campaigns/WebToLeadCreation.php

Braces +14 added lines, -9 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.
@@ -97,8 +99,7 @@ 
         $personTypeList.="<option value='".$count."'>".$b->name."</option>";
         $count++;
     }
-}
-else
+} else
 {
     $personTypeList.="<option value='noPerson'>No matching types</option>";
 }
@@ -133,8 +134,12 @@ 
 
 $xtpl->assign("WEB_POST_URL",$web_post_url);
 
-if (empty($focus->assigned_user_id) && empty($focus->id))  $focus->assigned_user_id = $current_user->id;
-if (empty($focus->assigned_name) && empty($focus->id))  $focus->assigned_user_name = $current_user->user_name;
+if (empty($focus->assigned_user_id) && empty($focus->id)) {
+    $focus->assigned_user_id = $current_user->id;
+}
+if (empty($focus->assigned_name) && empty($focus->id)) {
+    $focus->assigned_user_name = $current_user->user_name;
+}
 $xtpl->assign("ASSIGNED_USER_NAME", $focus->assigned_user_name);
 $xtpl->assign("ASSIGNED_USER_ID", $focus->assigned_user_id );
 
@@ -197,8 +202,7 @@ 
                 $col_arr[0]=$cols_name;
                 $col_arr[1]=$field_def['name'];
                 $col_arr[2]=true;
-            }
-            else{
+            } else{
                 $cols_name=$field_def['vname'];
                 $col_arr[0]=$cols_name;
                 $col_arr[1]=$field_def['name'];
@@ -232,8 +236,9 @@ 
     foreach($GLOBALS['moduleList'] as $mod)
     {
         $item = BeanFactory::getBean($mod);
-        if($item && is_subclass_of($item,$superclass))
-            $subclasses[] = $item;
+        if($item && is_subclass_of($item,$superclass)) {
+                    $subclasses[] = $item;
+        }
     }
     return $subclasses;
 }

modules/Campaigns/WizardEmailSetupSave.php

Braces +6 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');
+}
 /*********************************************************************************
  * SugarCRM Community Edition is a customer relationship management program developed by
  * SugarCRM, Inc. Copyright (C) 2004-2013 SugarCRM Inc.
@@ -104,20 +106,18 @@ 
         if(preg_match('/\s*Location:\s*(.*)$/', $header_URL, $matches)) {
             $href = $matches[1];
             SugarApplication::redirect($href);
-        }
-        else {
+        } else {
             header($headers);
         }
 
-    }else{
+    } else{
         //set navigation details
         $header_URL = "Location: index.php?action=index&module=Campaigns";
 
         if(preg_match('/\s*Location:\s*(.*)$/', $header_URL, $matches)) {
             $href = $matches[1];
             SugarApplication::redirect($href);
-        }
-        else {
+        } else {
             header($header_URL);
         }