adamjakab / SuiteCRM

include/database/DBManager.php

Spacing +314 added lines, -314 removed lines

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

include/SugarEmailAddress/SugarEmailAddress.php

Spacing +173 added lines, -173 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.
@@ -81,9 +81,9 @@ 
     /**
      * @deprecated deprecated since version 7.6, PHP4 Style Constructors are deprecated and will be remove in 7.8, please update your code, use __construct instead
      */
-    public function SugarEmailAddress(){
+    public function SugarEmailAddress() {
         $deprecatedMessage = 'PHP4 Style Constructors are deprecated and will be remove in 7.8, please update your code';
-        if(isset($GLOBALS['log'])) {
+        if (isset($GLOBALS['log'])) {
             $GLOBALS['log']->deprecated($deprecatedMessage);
         }
         else {
@@ -99,16 +99,16 @@ 
      * @param string $module
      */
     function handleLegacySave($bean, $prefix = "") {
-        if(!isset($_REQUEST) || !isset($_REQUEST['useEmailWidget'])) {
+        if (!isset($_REQUEST) || !isset($_REQUEST['useEmailWidget'])) {
             if (empty($this->addresses) || !isset($_REQUEST['massupdate'])) {
                 $this->addresses = array();
                 $optOut = (isset($bean->email_opt_out) && $bean->email_opt_out == "1") ? true : false;
                 $invalid = (isset($bean->invalid_email) && $bean->invalid_email == "1") ? true : false;
 
                 $isPrimary = true;
-                for($i = 1; $i <= 10; $i++){
+                for ($i = 1; $i <= 10; $i++) {
                     $email = 'email'.$i;
-                    if(isset($bean->$email) && !empty($bean->$email)){
+                    if (isset($bean->$email) && !empty($bean->$email)) {
                         $opt_out_field = $email.'_opt_out';
                         $invalid_field = $email.'_invalid';
                         $field_optOut = (isset($bean->$opt_out_field)) ? $bean->$opt_out_field : $optOut;
@@ -119,8 +119,8 @@ 
                 }
             }
         }
-        $this->populateAddresses($bean->id, $bean->module_dir, array(),'');
-        if(isset($_REQUEST) && isset($_REQUEST['useEmailWidget'])) {
+        $this->populateAddresses($bean->id, $bean->module_dir, array(), '');
+        if (isset($_REQUEST) && isset($_REQUEST['useEmailWidget'])) {
             $this->populateLegacyFields($bean);
         }
     }
@@ -142,11 +142,11 @@ 
         return;
     }
 
-    function populateLegacyFields(&$bean){
+    function populateLegacyFields(&$bean) {
         $primary_found = false;
         $alternate_found = false;
         $alternate2_found = false;
-        foreach($this->addresses as $k=>$address) {
+        foreach ($this->addresses as $k=>$address) {
             if ($primary_found && $alternate_found)
                 break;
             if ($address['primary_address'] == 1 && !$primary_found) {
@@ -155,7 +155,7 @@ 
             } elseif (!$alternate_found) {
                 $alternate_index = $k;
                 $alternate_found = true;
-            } elseif (!$alternate2_found){
+            } elseif (!$alternate2_found) {
                 $alternate2_index = $k;
                 $alternate2_found = true;
             }
@@ -192,27 +192,27 @@ 
         $args = func_get_args();
         return call_user_func_array(array($this, '_save'), $args);
     }
-    private function _save($id, $module, $new_addrs=array(), $primary='', $replyTo='', $invalid='', $optOut='', $in_workflow=false) {
-        if(empty($this->addresses) || $in_workflow){
-            $this->populateAddresses($id, $module, $new_addrs,$primary);
+    private function _save($id, $module, $new_addrs = array(), $primary = '', $replyTo = '', $invalid = '', $optOut = '', $in_workflow = false) {
+        if (empty($this->addresses) || $in_workflow) {
+            $this->populateAddresses($id, $module, $new_addrs, $primary);
         }
 
         //find all email addresses..
-        $current_links=array();
+        $current_links = array();
         // Need to correct this to handle the Employee/User split
         $module = $this->getCorrectedModule($module);
-        $q2="select *  from email_addr_bean_rel eabr WHERE eabr.bean_id = '".$this->db->quote($id)."' AND eabr.bean_module = '".$this->db->quote($module)."' and eabr.deleted=0";
+        $q2 = "select *  from email_addr_bean_rel eabr WHERE eabr.bean_id = '".$this->db->quote($id)."' AND eabr.bean_module = '".$this->db->quote($module)."' and eabr.deleted=0";
         $r2 = $this->db->query($q2);
-        while(($row2=$this->db->fetchByAssoc($r2)) != null ) {
-            $current_links[$row2['email_address_id']]=$row2;
+        while (($row2 = $this->db->fetchByAssoc($r2)) != null) {
+            $current_links[$row2['email_address_id']] = $row2;
         }
 
         $isConversion = (isset($_REQUEST) && isset($_REQUEST['action']) && $_REQUEST['action'] == 'ConvertLead') ? true : false;
 
         if (!empty($this->addresses)) {
             // insert new relationships and create email address record, if they don't exist
-            foreach($this->addresses as $address) {
-                if(!empty($address['email_address'])) {
+            foreach ($this->addresses as $address) {
+                if (!empty($address['email_address'])) {
                     $guid = create_guid();
                     $emailId = isset($address['email_address_id'])
                         && isset($current_links[$address['email_address_id']])
@@ -220,14 +220,14 @@ 
                     $emailId = $this->AddUpdateEmailAddress($address['email_address'],
                                                             $address['invalid_email'],
                                                             $address['opt_out'],
-                                                            $emailId);// this will save the email address if not found
+                                                            $emailId); // this will save the email address if not found
 
                     //verify linkage and flags.
-                    $upd_eabr="";
+                    $upd_eabr = "";
                     if (isset($current_links[$emailId])) {
                         if (!$isConversion) { // do not update anything if this is for lead conversion
-                        if ($address['primary_address'] != $current_links[$emailId]['primary_address'] or $address['reply_to_address'] != $current_links[$emailId]['reply_to_address'] ) {
-                            $upd_eabr="UPDATE email_addr_bean_rel SET primary_address='".$this->db->quote($address['primary_address'])."', reply_to_address='".$this->db->quote($address['reply_to_address'])."' WHERE id='".$this->db->quote($current_links[$emailId]['id'])."'";
+                        if ($address['primary_address'] != $current_links[$emailId]['primary_address'] or $address['reply_to_address'] != $current_links[$emailId]['reply_to_address']) {
+                            $upd_eabr = "UPDATE email_addr_bean_rel SET primary_address='".$this->db->quote($address['primary_address'])."', reply_to_address='".$this->db->quote($address['reply_to_address'])."' WHERE id='".$this->db->quote($current_links[$emailId]['id'])."'";
                         }
 
                         unset($current_links[$emailId]);
@@ -258,13 +258,13 @@ 
         // for lead conversion, do not delete email addresses
         if (!empty($current_links) && !$isConversion) {
 
-            $delete="";
+            $delete = "";
             foreach ($current_links as $eabr) {
 
-                $delete.=empty($delete) ? "'".$this->db->quote($eabr['id']) . "' " : ",'" . $this->db->quote($eabr['id']) . "'";
+                $delete .= empty($delete) ? "'".$this->db->quote($eabr['id'])."' " : ",'".$this->db->quote($eabr['id'])."'";
             }
 
-            $eabr_unlink="update email_addr_bean_rel set deleted=1 where id in ({$delete})";
+            $eabr_unlink = "update email_addr_bean_rel set deleted=1 where id in ({$delete})";
             $this->db->query($eabr_unlink);
         }
         $this->stateBeforeWorkflow = null;
@@ -286,7 +286,7 @@ 
         )
     {
         $emailCaps = strtoupper(trim($email));
-        if(empty($emailCaps))
+        if (empty($emailCaps))
             return 0;
 
         $q = "SELECT *
@@ -324,10 +324,10 @@ 
         $r = $this->db->query($q, true);
 
         $retArr = array();
-        while($a = $this->db->fetchByAssoc($r)) {
+        while ($a = $this->db->fetchByAssoc($r)) {
             $retArr[] = $a['bean_id'];
         }
-        if(count($retArr) > 0) {
+        if (count($retArr) > 0) {
             return $retArr;
         } else {
             return false;
@@ -347,7 +347,7 @@ 
 
         $email = trim($email);
 
-        if(empty($email)) {
+        if (empty($email)) {
             return array();
         }
 
@@ -356,12 +356,12 @@ 
                 WHERE ea.email_address_caps = $emailCaps and eabl.deleted=0 ";
         $r = $this->db->query($q);
 
-        while($a = $this->db->fetchByAssoc($r)) {
-            if(isset($beanList[$a['bean_module']]) && !empty($beanList[$a['bean_module']])) {
+        while ($a = $this->db->fetchByAssoc($r)) {
+            if (isset($beanList[$a['bean_module']]) && !empty($beanList[$a['bean_module']])) {
                 $className = $beanList[$a['bean_module']];
 
-                if(isset($beanFiles[$className]) && !empty($beanFiles[$className])) {
-                    if(!class_exists($className)) {
+                if (isset($beanFiles[$className]) && !empty($beanFiles[$className])) {
+                    if (!class_exists($className)) {
                         require_once($beanFiles[$className]);
                     }
 
@@ -389,7 +389,7 @@ 
      * @param string $replyTo GUID of reply-to address
      * @param string $invalid GUID of invalid address
      */
-    function populateAddresses($id, $module, $new_addrs=array(), $primary='', $replyTo='', $invalid='', $optOut='') {
+    function populateAddresses($id, $module, $new_addrs = array(), $primary = '', $replyTo = '', $invalid = '', $optOut = '') {
         $module = $this->getCorrectedModule($module);
         //One last check for the ConvertLead action in which case we need to change $module to 'Leads'
         $module = (isset($_REQUEST) && isset($_REQUEST['action']) && $_REQUEST['action'] == 'ConvertLead') ? 'Leads' : $module;
@@ -400,54 +400,54 @@ 
         $hasEmailValue = false;
         $email_ids = array();
 
-        if (isset($_REQUEST) && isset($_REQUEST[$module .'_email_widget_id'])) {
+        if (isset($_REQUEST) && isset($_REQUEST[$module.'_email_widget_id'])) {
 
             $fromRequest = false;
             // determine which array to process
-            foreach($_REQUEST as $k => $v) {
-                if(strpos($k, 'emailAddress') !== false) {
+            foreach ($_REQUEST as $k => $v) {
+                if (strpos($k, 'emailAddress') !== false) {
                    $fromRequest = true;
                    break;
                 }
-            $widget_id = $_REQUEST[$module .'_email_widget_id'];
+            $widget_id = $_REQUEST[$module.'_email_widget_id'];
     }
 
             //Iterate over the widgets for this module, in case there are multiple email widgets for this module
-            while(isset($_REQUEST[$module . $widget_id . "emailAddress" . $widgetCount]))
+            while (isset($_REQUEST[$module.$widget_id."emailAddress".$widgetCount]))
             {
-                if (empty($_REQUEST[$module . $widget_id . "emailAddress" . $widgetCount])) {
+                if (empty($_REQUEST[$module.$widget_id."emailAddress".$widgetCount])) {
                     $widgetCount++;
                     continue;
                 }
 
                 $hasEmailValue = true;
 
-                $eId = $module . $widget_id;
-                if(isset($_REQUEST[$eId . 'emailAddressPrimaryFlag'])) {
-                   $primaryValue = $_REQUEST[$eId . 'emailAddressPrimaryFlag'];
-                } else if(isset($_REQUEST[$module . 'emailAddressPrimaryFlag'])) {
-                   $primaryValue = $_REQUEST[$module . 'emailAddressPrimaryFlag'];
+                $eId = $module.$widget_id;
+                if (isset($_REQUEST[$eId.'emailAddressPrimaryFlag'])) {
+                   $primaryValue = $_REQUEST[$eId.'emailAddressPrimaryFlag'];
+                } else if (isset($_REQUEST[$module.'emailAddressPrimaryFlag'])) {
+                   $primaryValue = $_REQUEST[$module.'emailAddressPrimaryFlag'];
                 }
 
                 $optOutValues = array();
-                if(isset($_REQUEST[$eId .'emailAddressOptOutFlag'])) {
-                   $optOutValues = $_REQUEST[$eId .'emailAddressOptOutFlag'];
-                } else if(isset($_REQUEST[$module . 'emailAddressOptOutFlag'])) {
-                   $optOutValues = $_REQUEST[$module . 'emailAddressOptOutFlag'];
+                if (isset($_REQUEST[$eId.'emailAddressOptOutFlag'])) {
+                   $optOutValues = $_REQUEST[$eId.'emailAddressOptOutFlag'];
+                } else if (isset($_REQUEST[$module.'emailAddressOptOutFlag'])) {
+                   $optOutValues = $_REQUEST[$module.'emailAddressOptOutFlag'];
                 }
 
                 $invalidValues = array();
-                if(isset($_REQUEST[$eId .'emailAddressInvalidFlag'])) {
-                   $invalidValues = $_REQUEST[$eId .'emailAddressInvalidFlag'];
-                } else if(isset($_REQUEST[$module . 'emailAddressInvalidFlag'])) {
-                   $invalidValues = $_REQUEST[$module . 'emailAddressInvalidFlag'];
+                if (isset($_REQUEST[$eId.'emailAddressInvalidFlag'])) {
+                   $invalidValues = $_REQUEST[$eId.'emailAddressInvalidFlag'];
+                } else if (isset($_REQUEST[$module.'emailAddressInvalidFlag'])) {
+                   $invalidValues = $_REQUEST[$module.'emailAddressInvalidFlag'];
                 }
 
                 $deleteValues = array();
-                if(isset($_REQUEST[$eId .'emailAddressDeleteFlag'])) {
-                   $deleteValues = $_REQUEST[$eId .'emailAddressDeleteFlag'];
-                } else if(isset($_REQUEST[$module . 'emailAddressDeleteFlag'])) {
-                   $deleteValues = $_REQUEST[$module . 'emailAddressDeleteFlag'];
+                if (isset($_REQUEST[$eId.'emailAddressDeleteFlag'])) {
+                   $deleteValues = $_REQUEST[$eId.'emailAddressDeleteFlag'];
+                } else if (isset($_REQUEST[$module.'emailAddressDeleteFlag'])) {
+                   $deleteValues = $_REQUEST[$module.'emailAddressDeleteFlag'];
                 }
 
                 // prep from form save
@@ -455,36 +455,36 @@ 
                 $replyToField = '';
                 $invalidField = '';
                 $optOutField = '';
-                if($fromRequest && empty($primary) && isset($primaryValue)) {
+                if ($fromRequest && empty($primary) && isset($primaryValue)) {
                     $primaryField = $primaryValue;
                 }
 
-                if($fromRequest && empty($replyTo)) {
-                    if(isset($_REQUEST[$eId .'emailAddressReplyToFlag'])) {
-                       $replyToField = $_REQUEST[$eId .'emailAddressReplyToFlag'];
-                    } else if(isset($_REQUEST[$module . 'emailAddressReplyToFlag'])) {
-                       $replyToField = $_REQUEST[$module . 'emailAddressReplyToFlag'];
+                if ($fromRequest && empty($replyTo)) {
+                    if (isset($_REQUEST[$eId.'emailAddressReplyToFlag'])) {
+                       $replyToField = $_REQUEST[$eId.'emailAddressReplyToFlag'];
+                    } else if (isset($_REQUEST[$module.'emailAddressReplyToFlag'])) {
+                       $replyToField = $_REQUEST[$module.'emailAddressReplyToFlag'];
                     }
                 }
-                if($fromRequest && empty($new_addrs)) {
-                    foreach($_REQUEST as $k => $v) {
-                        if(preg_match('/'.$eId.'emailAddress[0-9]+$/i', $k) && !empty($v)) {
+                if ($fromRequest && empty($new_addrs)) {
+                    foreach ($_REQUEST as $k => $v) {
+                        if (preg_match('/'.$eId.'emailAddress[0-9]+$/i', $k) && !empty($v)) {
                             $new_addrs[$k] = $v;
                         }
                     }
                 }
-                if($fromRequest && empty($email_ids)) {
-                    foreach($_REQUEST as $k => $v) {
-                        if(preg_match('/'.$eId.'emailAddressId[0-9]+$/i', $k) && !empty($v)) {
+                if ($fromRequest && empty($email_ids)) {
+                    foreach ($_REQUEST as $k => $v) {
+                        if (preg_match('/'.$eId.'emailAddressId[0-9]+$/i', $k) && !empty($v)) {
                             $key = str_replace('emailAddressId', 'emailAddress', $k);
                             $email_ids[$key] = $v;
                         }
                     }
                 }
 
-                if($fromRequest && empty($new_addrs)) {
-                    foreach($_REQUEST as $k => $v) {
-                        if(preg_match('/'.$eId.'emailAddressVerifiedValue[0-9]+$/i', $k) && !empty($v)) {
+                if ($fromRequest && empty($new_addrs)) {
+                    foreach ($_REQUEST as $k => $v) {
+                        if (preg_match('/'.$eId.'emailAddressVerifiedValue[0-9]+$/i', $k) && !empty($v)) {
                             $validateFlag = str_replace("Value", "Flag", $k);
                             if (isset($_REQUEST[$validateFlag]) && $_REQUEST[$validateFlag] == "true")
                               $new_addrs[$k] = $v;
@@ -493,21 +493,21 @@ 
                 }
 
                 //empty the addresses array if the post happened from email address widget.
-                if($post_from_email_address_widget) {
-                    $this->addresses=array();  //this gets populated during retrieve of the contact bean.
+                if ($post_from_email_address_widget) {
+                    $this->addresses = array(); //this gets populated during retrieve of the contact bean.
                 } else {
                     $optOutValues = array();
                     $invalidValues = array();
-                    foreach($new_addrs as $k=>$email) {
+                    foreach ($new_addrs as $k=>$email) {
                        preg_match('/emailAddress([0-9])+$/', $k, $matches);
                        $count = $matches[1];
-                       $result = $this->db->query("SELECT opt_out, invalid_email from email_addresses where email_address_caps = '" . $this->db->quote(strtoupper($email)) . "'");
-                       if(!empty($result)) {
-                          $row=$this->db->fetchByAssoc($result);
-                          if(!empty($row['opt_out'])) {
+                       $result = $this->db->query("SELECT opt_out, invalid_email from email_addresses where email_address_caps = '".$this->db->quote(strtoupper($email))."'");
+                       if (!empty($result)) {
+                          $row = $this->db->fetchByAssoc($result);
+                          if (!empty($row['opt_out'])) {
                              $optOutValues[$k] = "emailAddress$count";
                           }
-                          if(!empty($row['invalid_email'])) {
+                          if (!empty($row['invalid_email'])) {
                              $invalidValues[$k] = "emailAddress$count";
                           }
                        }
@@ -515,11 +515,11 @@ 
                 }
                 // Re-populate the addresses class variable if we have new address(es).
                 if (!empty($new_addrs)) {
-                    foreach($new_addrs as $k => $reqVar) {
+                    foreach ($new_addrs as $k => $reqVar) {
                         //$key = preg_match("/^$eId/s", $k) ? substr($k, strlen($eId)) : $k;
                         $reqVar = trim($reqVar);
-                        if(strpos($k, 'emailAddress') !== false) {
-                            if(!empty($reqVar) && !in_array($k, $deleteValues)) {
+                        if (strpos($k, 'emailAddress') !== false) {
+                            if (!empty($reqVar) && !in_array($k, $deleteValues)) {
                                 $email_id   = (array_key_exists($k, $email_ids)) ? $email_ids[$k] : null;
                                 $primary    = ($k == $primaryValue) ? true : false;
                                 $replyTo    = ($k == $replyToField) ? true : false;
@@ -536,7 +536,7 @@ 
         }
 
         //If no widgets, set addresses array to empty
-        if($post_from_email_address_widget && !$hasEmailValue) {
+        if ($post_from_email_address_widget && !$hasEmailValue) {
            $this->addresses = array();
         }
     }
@@ -547,9 +547,9 @@ 
      * @param bool $primary Default false
      * @param bool $replyTo Default false
      */
-    function addAddress($addr, $primary=false, $replyTo=false, $invalid=false, $optOut=false, $email_id = null) {
+    function addAddress($addr, $primary = false, $replyTo = false, $invalid = false, $optOut = false, $email_id = null) {
         $addr = html_entity_decode($addr, ENT_QUOTES);
-        if(preg_match($this->regex, $addr)) {
+        if (preg_match($this->regex, $addr)) {
             $primaryFlag = ($primary) ? '1' : '0';
             $replyToFlag = ($replyTo) ? '1' : '0';
             $invalidFlag = ($invalid) ? '1' : '0';
@@ -585,17 +585,17 @@ 
      * Updates invalid_email and opt_out flags for each address
      */
     function updateFlags() {
-        if(!empty($this->addresses)) {
-            foreach($this->addresses as $addressMeta) {
-                if(isset($addressMeta['email_address']) && !empty($addressMeta['email_address'])) {
+        if (!empty($this->addresses)) {
+            foreach ($this->addresses as $addressMeta) {
+                if (isset($addressMeta['email_address']) && !empty($addressMeta['email_address'])) {
                     $address = $this->db->quote($this->_cleanAddress($addressMeta['email_address']));
 
                     $q = "SELECT * FROM email_addresses WHERE email_address = '{$address}'";
                     $r = $this->db->query($q);
                     $a = $this->db->fetchByAssoc($r);
 
-                    if(!empty($a)) {
-                        if(isset($a['invalid_email']) && isset($addressMeta['invalid_email']) && isset($addressMeta['opt_out']) && $a['invalid_email'] != $addressMeta['invalid_email'] || $a['opt_out'] != $addressMeta['opt_out']) {
+                    if (!empty($a)) {
+                        if (isset($a['invalid_email']) && isset($addressMeta['invalid_email']) && isset($addressMeta['opt_out']) && $a['invalid_email'] != $addressMeta['invalid_email'] || $a['opt_out'] != $addressMeta['opt_out']) {
                             $qUpdate = "UPDATE email_addresses SET invalid_email = ".intval($addressMeta['invalid_email']).", opt_out = ".intval($addressMeta['opt_out']).", date_modified = '".TimeDate::getInstance()->nowDb()."' WHERE id = '".$this->db->quote($a['id'])."'";
                             $rUpdate = $this->db->query($qUpdate);
                         }
@@ -608,7 +608,7 @@ 
     public function splitEmailAddress($addr)
     {
         $email = $this->_cleanAddress($addr);
-        if(!preg_match($this->regex, $email)) {
+        if (!preg_match($this->regex, $email)) {
             $email = ''; // remove bad email addr
         }
         $name = trim(str_replace(array($email, '<', '>', '"', "'"), '', $addr));
@@ -624,8 +624,8 @@ 
     function _cleanAddress($addr) {
         $addr = trim(from_html($addr));
 
-        if(strpos($addr, "<") !== false && strpos($addr, ">") !== false) {
-            $address = trim(substr($addr, strrpos($addr, "<") +1, strrpos($addr, ">") - strrpos($addr, "<") -1));
+        if (strpos($addr, "<") !== false && strpos($addr, ">") !== false) {
+            $address = trim(substr($addr, strrpos($addr, "<") + 1, strrpos($addr, ">") - strrpos($addr, "<") - 1));
         } else {
             $address = trim($addr);
         }
@@ -646,11 +646,11 @@ 
         $r = $this->db->query($q);
         $a = $this->db->fetchByAssoc($r);
 
-        if(!empty($a) && !empty($a['id'])) {
+        if (!empty($a) && !empty($a['id'])) {
             return $a['id'];
         } else {
             $guid = '';
-            if(!empty($address)){
+            if (!empty($address)) {
                 $guid = create_guid();
                 $now = TimeDate::getInstance()->nowDb();
                 $qa = "INSERT INTO email_addresses (id, email_address, email_address_caps, date_created, date_modified, deleted)
@@ -672,7 +672,7 @@ 
      *        to propagate to the new SugarEmailAddress - see bug 39188
      * @return String GUID of Email Address or '' if cleaned address was empty.
      */
-    public function AddUpdateEmailAddress($addr,$invalid=0,$opt_out=0,$id=null)
+    public function AddUpdateEmailAddress($addr, $invalid = 0, $opt_out = 0, $id = null)
     {
         // sanity checks to avoid SQL injection.
         $invalid = intval($invalid);
@@ -722,12 +722,12 @@ 
             if ($duplicate_email['invalid_email'] != $new_invalid ||
                 $duplicate_email['opt_out'] != $new_opt_out ||
                 (trim($duplicate_email['email_address']) != $address)) {
-                $upd_q = 'UPDATE ' . $this->table_name . ' ' .
-                    'SET email_address=\'' . $address . '\', ' .
-                    'invalid_email=' . $new_invalid . ', ' .
-                    'opt_out=' . $new_opt_out . ', ' .
-                    'date_modified=' . $this->db->now() . ' ' .
-                    'WHERE id=\'' . $this->db->quote($duplicate_email['id']) . '\'';
+                $upd_q = 'UPDATE '.$this->table_name.' '.
+                    'SET email_address=\''.$address.'\', '.
+                    'invalid_email='.$new_invalid.', '.
+                    'opt_out='.$new_opt_out.', '.
+                    'date_modified='.$this->db->now().' '.
+                    'WHERE id=\''.$this->db->quote($duplicate_email['id']).'\'';
                 $upd_r = $this->db->query($upd_q);
             }
             return $duplicate_email['id'];
@@ -735,7 +735,7 @@ 
         else {
             // no case-insensitive address match - it's new, or undeleted.
             $guid = '';
-            if(!empty($address)){
+            if (!empty($address)) {
                 $guid = create_guid();
                 $now = TimeDate::getInstance()->nowDb();
                 $qa = "INSERT INTO email_addresses (id, email_address, email_address_caps, date_created, date_modified, deleted, invalid_email, opt_out)
@@ -751,12 +751,12 @@ 
      * @param object $focus Object in focus
      * @return string email
      */
-    function getPrimaryAddress($focus,$parent_id=null,$parent_type=null) {
+    function getPrimaryAddress($focus, $parent_id = null, $parent_type = null) {
 
-        $parent_type=empty($parent_type) ? $focus->module_dir : $parent_type;
+        $parent_type = empty($parent_type) ? $focus->module_dir : $parent_type;
         // Bug63174: Email address is not shown in the list view for employees
         $parent_type = $this->getCorrectedModule($parent_type);
-        $parent_id=empty($parent_id) ? $focus->id : $parent_id;
+        $parent_id = empty($parent_id) ? $focus->id : $parent_id;
 
         $q = "SELECT ea.email_address FROM email_addresses ea
                 LEFT JOIN email_addr_bean_rel ear ON ea.id = ear.email_address_id
@@ -768,7 +768,7 @@ 
         $r = $this->db->limitQuery($q, 0, 1);
         $a = $this->db->fetchByAssoc($r);
 
-        if(isset($a['email_address'])) {
+        if (isset($a['email_address'])) {
             return $a['email_address'];
         }
         return '';
@@ -809,7 +809,7 @@ 
         $r = $this->db->query($q);
         $a = $this->db->fetchByAssoc($r);
 
-        if(isset($a['email_address'])) {
+        if (isset($a['email_address'])) {
             return $a['email_address'];
         }
         return '';
@@ -834,7 +834,7 @@ 
                 ORDER BY ear.reply_to_address, ear.primary_address DESC";
         $r = $this->db->query($q);
 
-        while($a = $this->db->fetchByAssoc($r, FALSE)) {
+        while ($a = $this->db->fetchByAssoc($r, FALSE)) {
             $return[] = $a;
         }
 
@@ -848,9 +848,9 @@ 
      * @param bool asMetadata Default false
      * @return string HTML/JS for widget
      */
-    function getEmailAddressWidgetEditView($id, $module, $asMetadata=false, $tpl='',$tabindex='0')
+    function getEmailAddressWidgetEditView($id, $module, $asMetadata = false, $tpl = '', $tabindex = '0')
     {
-        if ( !($this->smarty instanceOf Sugar_Smarty ) )
+        if (!($this->smarty instanceOf Sugar_Smarty))
             $this->smarty = new Sugar_Smarty();
 
         global $app_strings, $dictionary, $beanList;
@@ -861,33 +861,33 @@ 
         $passedModule = $module;
         $module = $this->getCorrectedModule($module);
         $saveModule = $module;
-        if(isset($_POST['is_converted']) && $_POST['is_converted']==true){
-            $id=$_POST['return_id'];
-            $module=$_POST['return_module'];
+        if (isset($_POST['is_converted']) && $_POST['is_converted'] == true) {
+            $id = $_POST['return_id'];
+            $module = $_POST['return_module'];
         }
         $prefillDataArr = array();
-        if(!empty($id)) {
+        if (!empty($id)) {
             $prefillDataArr = $this->getAddressesByGUID($id, $module);
             //When coming from convert leads, sometimes module is Contacts while the id is for a lead.
             if (empty($prefillDataArr) && $module == "Contacts")
                 $prefillDataArr = $this->getAddressesByGUID($id, "Leads");
-        } else if(isset($_REQUEST['full_form']) && !empty($_REQUEST['emailAddressWidget'])){
-            $widget_id = isset($_REQUEST[$module . '_email_widget_id']) ? $_REQUEST[$module . '_email_widget_id'] : '0';
+        } else if (isset($_REQUEST['full_form']) && !empty($_REQUEST['emailAddressWidget'])) {
+            $widget_id = isset($_REQUEST[$module.'_email_widget_id']) ? $_REQUEST[$module.'_email_widget_id'] : '0';
             $count = 0;
-            $key = $module . $widget_id . 'emailAddress'.$count;
-            while(isset($_REQUEST[$key])) {
+            $key = $module.$widget_id.'emailAddress'.$count;
+            while (isset($_REQUEST[$key])) {
                    $email = $_REQUEST[$key];
-                   $prefillDataArr[] =  array('email_address'=>$email,
+                   $prefillDataArr[] = array('email_address'=>$email,
                                              'primary_address'=>isset($_REQUEST['emailAddressPrimaryFlag']) && $_REQUEST['emailAddressPrimaryFlag'] == $key,
                                              'invalid_email'=>isset($_REQUEST['emailAddressInvalidFlag']) && in_array($key, $_REQUEST['emailAddressInvalidFlag']),
                                              'opt_out'=>isset($_REQUEST['emailAddressOptOutFlag']) && in_array($key, $_REQUEST['emailAddressOptOutFlag']),
                                              'reply_to_address'=>false
                                         );
-                   $key = $module . $widget_id . 'emailAddress' . ++$count;
+                   $key = $module.$widget_id.'emailAddress'.++$count;
             } //while
         }
 
-        if(!empty($prefillDataArr)) {
+        if (!empty($prefillDataArr)) {
             $json = new JSON(JSON_LOOSE_TYPE);
             $prefillData = $json->encode($prefillDataArr);
             $prefill = !empty($prefillDataArr) ? 'true' : 'false';
@@ -910,23 +910,23 @@ 
         $form = $this->view;
 
         //determine if this should be a quickcreate form, or a quick create form under subpanels
-        if ($this->view == "QuickCreate"){
+        if ($this->view == "QuickCreate") {
             // Fixed #1120 - fixed email validation for: Accounts -> Contacts subpanel -> Select -> Create Contact -> Save.
             // If email is required it should highlight this field and show an error message.
             // It didnt because the the form was named form_DCSubpanelQuickCreate_Contacts instead of expected form_SubpanelQuickCreate_Contacts
-            if($this->object_name = 'EmailAddress' && $saveModule == 'Contacts') {
-                $form = 'form_'.$this->view .'_'.$module;
+            if ($this->object_name = 'EmailAddress' && $saveModule == 'Contacts') {
+                $form = 'form_'.$this->view.'_'.$module;
             } else {
-                $form = 'form_DC'.$this->view .'_'.$module;
+                $form = 'form_DC'.$this->view.'_'.$module;
             }
-            if(isset($_REQUEST['action']) && $_REQUEST['action']=='SubpanelCreates' ||  $_REQUEST['action']=='SubpanelEdits'){
-                $form = 'form_Subpanel'.$this->view .'_'.$module;
+            if (isset($_REQUEST['action']) && $_REQUEST['action'] == 'SubpanelCreates' || $_REQUEST['action'] == 'SubpanelEdits') {
+                $form = 'form_Subpanel'.$this->view.'_'.$module;
             }
         }
 
         $this->smarty->assign('emailView', $form);
 
-        if($module == 'Users') {
+        if ($module == 'Users') {
             $this->smarty->assign('useReplyTo', true);
         } else {
             $this->smarty->assign('useOptOut', true);
@@ -937,7 +937,7 @@ 
         $newEmail = $this->smarty->fetch($template);
 
 
-        if($asMetadata) {
+        if ($asMetadata) {
             // used by Email 2.0
             $ret = array();
             $ret['prefillData'] = $prefillDataArr;
@@ -955,18 +955,18 @@ 
      * @param object $focus Bean in focus
      * @return string HTML/JS for widget
      */
-    function getEmailAddressWidgetDetailView($focus, $tpl='')
+    function getEmailAddressWidgetDetailView($focus, $tpl = '')
     {
-        if ( !($this->smarty instanceOf Sugar_Smarty ) )
+        if (!($this->smarty instanceOf Sugar_Smarty))
             $this->smarty = new Sugar_Smarty();
 
         global $app_strings;
         global $current_user;
         $assign = array();
-        if(empty($focus->id))return '';
+        if (empty($focus->id))return '';
         $prefillData = $this->getAddressesByGUID($focus->id, $focus->module_dir);
 
-        foreach($prefillData as $addressItem) {
+        foreach ($prefillData as $addressItem) {
             $key = ($addressItem['primary_address'] == 1) ? 'primary' : "";
             $key = ($addressItem['reply_to_address'] == 1) ? 'reply_to' : $key;
             $key = ($addressItem['opt_out'] == 1) ? 'opt_out' : $key;
@@ -992,7 +992,7 @@ 
      */
     function getEmailAddressWidgetDuplicatesView($focus)
     {
-        if ( !($this->smarty instanceOf Sugar_Smarty ) )
+        if (!($this->smarty instanceOf Sugar_Smarty))
             $this->smarty = new Sugar_Smarty();
 
         $count = 0;
@@ -1002,49 +1002,49 @@ 
         $invalid = array();
         $mod = isset($focus) ? $focus->module_dir : "";
 
-        $widget_id = $_POST[$mod .'_email_widget_id'];
-        $this->smarty->assign('email_widget_id',$widget_id);
-        $this->smarty->assign('emailAddressWidget',$_POST['emailAddressWidget']);
+        $widget_id = $_POST[$mod.'_email_widget_id'];
+        $this->smarty->assign('email_widget_id', $widget_id);
+        $this->smarty->assign('emailAddressWidget', $_POST['emailAddressWidget']);
 
-        if(isset($_POST[$mod . $widget_id . 'emailAddressPrimaryFlag'])) {
-           $primary = $_POST[$mod . $widget_id . 'emailAddressPrimaryFlag'];
+        if (isset($_POST[$mod.$widget_id.'emailAddressPrimaryFlag'])) {
+           $primary = $_POST[$mod.$widget_id.'emailAddressPrimaryFlag'];
         }
 
-        while(isset($_POST[$mod . $widget_id . "emailAddress" . $count])) {
-            $emails[] = $_POST[$mod . $widget_id . 'emailAddress' . $count];
+        while (isset($_POST[$mod.$widget_id."emailAddress".$count])) {
+            $emails[] = $_POST[$mod.$widget_id.'emailAddress'.$count];
             $count++;
         }
 
-        if($count == 0) {
+        if ($count == 0) {
            return "";
         }
 
-        if(isset($_POST[$mod . $widget_id . 'emailAddressOptOutFlag'])) {
-           foreach($_POST[$mod . $widget_id . 'emailAddressOptOutFlag'] as $v) {
+        if (isset($_POST[$mod.$widget_id.'emailAddressOptOutFlag'])) {
+           foreach ($_POST[$mod.$widget_id.'emailAddressOptOutFlag'] as $v) {
               $optOut[] = $v;
            }
         }
 
-        if(isset($_POST[$mod . $widget_id . 'emailAddressInvalidFlag'])) {
-           foreach($_POST[$mod . $widget_id . 'emailAddressInvalidFlag'] as $v) {
+        if (isset($_POST[$mod.$widget_id.'emailAddressInvalidFlag'])) {
+           foreach ($_POST[$mod.$widget_id.'emailAddressInvalidFlag'] as $v) {
               $invalid[] = $v;
            }
         }
 
-        if(isset($_POST[$mod . $widget_id . 'emailAddressReplyToFlag'])) {
-           foreach($_POST[$mod . $widget_id . 'emailAddressReplyToFlag'] as $v) {
+        if (isset($_POST[$mod.$widget_id.'emailAddressReplyToFlag'])) {
+           foreach ($_POST[$mod.$widget_id.'emailAddressReplyToFlag'] as $v) {
               $replyTo[] = $v;
            }
         }
 
-        if(isset($_POST[$mod . $widget_id . 'emailAddressDeleteFlag'])) {
-           foreach($_POST[$mod . $widget_id . 'emailAddressDeleteFlag'] as $v) {
+        if (isset($_POST[$mod.$widget_id.'emailAddressDeleteFlag'])) {
+           foreach ($_POST[$mod.$widget_id.'emailAddressDeleteFlag'] as $v) {
               $delete[] = $v;
            }
         }
 
-        while(isset($_POST[$mod . $widget_id . "emailAddressVerifiedValue" . $count])) {
-            $verified[] = $_POST[$mod . $widget_id . 'emailAddressVerifiedValue' . $count];
+        while (isset($_POST[$mod.$widget_id."emailAddressVerifiedValue".$count])) {
+            $verified[] = $_POST[$mod.$widget_id.'emailAddressVerifiedValue'.$count];
             $count++;
         }
 
@@ -1069,33 +1069,33 @@ 
         $count = 0;
         $mod = isset($focus) ? $focus->module_dir : "";
 
-        $widget_id = $_POST[$mod .'_email_widget_id'];
-        $get .= '&' . $mod . '_email_widget_id='. $widget_id;
+        $widget_id = $_POST[$mod.'_email_widget_id'];
+        $get .= '&'.$mod.'_email_widget_id='.$widget_id;
         $get .= '&emailAddressWidget='.$_POST['emailAddressWidget'];
 
-        while(isset($_REQUEST[$mod . $widget_id . 'emailAddress' . $count])) {
-              $get .= "&" . $mod . $widget_id . "emailAddress" . $count . "=" . urlencode($_REQUEST[$mod . $widget_id . 'emailAddress' . $count]);
+        while (isset($_REQUEST[$mod.$widget_id.'emailAddress'.$count])) {
+              $get .= "&".$mod.$widget_id."emailAddress".$count."=".urlencode($_REQUEST[$mod.$widget_id.'emailAddress'.$count]);
               $count++;
         } //while
 
-        while(isset($_REQUEST[$mod . $widget_id . 'emailAddressVerifiedValue' . $count])) {
-              $get .= "&" . $mod . $widget_id . "emailAddressVerifiedValue" . $count . "=" . urlencode($_REQUEST[$mod . $widget_id . 'emailAddressVerifiedValue' . $count]);
+        while (isset($_REQUEST[$mod.$widget_id.'emailAddressVerifiedValue'.$count])) {
+              $get .= "&".$mod.$widget_id."emailAddressVerifiedValue".$count."=".urlencode($_REQUEST[$mod.$widget_id.'emailAddressVerifiedValue'.$count]);
               $count++;
         } //while
 
         $options = array('emailAddressPrimaryFlag', 'emailAddressOptOutFlag', 'emailAddressInvalidFlag', 'emailAddressDeleteFlag', 'emailAddressReplyToFlag');
 
-        foreach($options as $option) {
+        foreach ($options as $option) {
             $count = 0;
             $optionIdentifier = $mod.$widget_id.$option;
-            if(isset($_REQUEST[$optionIdentifier])) {
-               if(is_array($_REQUEST[$optionIdentifier])) {
-                   foreach($_REQUEST[$optionIdentifier] as $optOut) {
-                      $get .= "&" . $optionIdentifier . "[" . $count . "]=" . $optOut;
+            if (isset($_REQUEST[$optionIdentifier])) {
+               if (is_array($_REQUEST[$optionIdentifier])) {
+                   foreach ($_REQUEST[$optionIdentifier] as $optOut) {
+                      $get .= "&".$optionIdentifier."[".$count."]=".$optOut;
                       $count++;
                    } //foreach
                } else {
-                   $get .= "&" . $optionIdentifier . "=" . $_REQUEST[$optionIdentifier];
+                   $get .= "&".$optionIdentifier."=".$_REQUEST[$optionIdentifier];
                }
             } //if
         } //foreach
@@ -1113,7 +1113,7 @@ 
  * @return string The value for the bean_module column in the email_addr_bean_rel table
  */
     function getCorrectedModule(&$module) {
-        return ($module == "Employees")? "Users" : $module;
+        return ($module == "Employees") ? "Users" : $module;
     }
 
     public function stash($parentBeanId, $moduleName)
@@ -1123,7 +1123,7 @@ 
         $ids = array();
         while ($row = $this->db->fetchByAssoc($result, false))
         {
-            $ids[] =$this->db->quote($row['email_address_id']); // avoid 2nd order SQL Injection
+            $ids[] = $this->db->quote($row['email_address_id']); // avoid 2nd order SQL Injection
         }
         if (!empty($ids))
         {
@@ -1147,15 +1147,15 @@ 
  * @param string $view DetailView or EditView
  * @return string
  */
-function getEmailAddressWidget($focus, $field, $value, $view, $tabindex='0') {
+function getEmailAddressWidget($focus, $field, $value, $view, $tabindex = '0') {
     $sea = new SugarEmailAddress();
     $sea->setView($view);
 
-        if($view == 'EditView' || $view == 'QuickCreate' || $view == 'ConvertLead') {
+        if ($view == 'EditView' || $view == 'QuickCreate' || $view == 'ConvertLead') {
             $module = $focus->module_dir;
             if ($view == 'ConvertLead' && $module == "Contacts")  $module = "Leads";
 
-            return $sea->getEmailAddressWidgetEditView($focus->id, $module, false,'',$tabindex);
+            return $sea->getEmailAddressWidgetEditView($focus->id, $module, false, '', $tabindex);
         }
 
     return $sea->getEmailAddressWidgetDetailView($focus);

include/utils/sugar_file_utils.php

Spacing +2 added lines, -2 removed lines

@@ -186,7 +186,7 @@ 
     $temp = tempnam($dir, 'temp');
 
     if (!($f = @fopen($temp, $mode))) {
-        $temp = $dir . DIRECTORY_SEPARATOR . uniqid('temp');
+        $temp = $dir.DIRECTORY_SEPARATOR.uniqid('temp');
         if (!($f = @fopen($temp, $mode))) {
             trigger_error("sugar_file_put_contents_atomic() : error writing temporary file '$temp'", E_USER_WARNING);
 
@@ -393,7 +393,7 @@ 
     }
     if (!is_windows()) {
         $conf_inst = SugarConfig::getInstance();
-        $mode = $conf_inst->get('default_permissions.' . $key, $mode);
+        $mode = $conf_inst->get('default_permissions.'.$key, $mode);
     }
 
     return $mode;

include/generic/SugarWidgets/SugarWidgetSubPanelTopCreateCampaignMarketingEmailButton.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.
@@ -48,11 +48,11 @@ 
         $id = $layout_def['focus']->id;
         $module = $layout_def['focus']->module_name;
 
-        $href = "index.php?module=$module&action=WizardMarketing&campaign_id=$id" . (!empty($layout_def['func']) ? '&func=' . $layout_def['func'] : '');
+        $href = "index.php?module=$module&action=WizardMarketing&campaign_id=$id".(!empty($layout_def['func']) ? '&func='.$layout_def['func'] : '');
 
         $label = $app_strings['LBL_CREATE_BUTTON_LABEL'];
 
-        return '<a onclick="location.href=\'' . $href . '\';">' . $label . '</a>';
+        return '<a onclick="location.href=\''.$href.'\';">'.$label.'</a>';
     }
 
 }
\ No newline at end of file

tests/tests/modules/Contacts/ContactTest.php

Spacing +32 added lines, -32 removed lines

@@ -6,9 +6,9 @@ 
 
 		//execute the contructor and check for the Object type and  attributes
 		$contact = new Contact();
-		$this->assertInstanceOf('Contact',$contact);
-		$this->assertInstanceOf('Person',$contact);
-		$this->assertInstanceOf('SugarBean',$contact);
+		$this->assertInstanceOf('Contact', $contact);
+		$this->assertInstanceOf('Person', $contact);
+		$this->assertInstanceOf('SugarBean', $contact);
 
 		$this->assertAttributeEquals('Contacts', 'module_dir', $contact);
 		$this->assertAttributeEquals('Contact', 'object_name', $contact);
@@ -29,7 +29,7 @@ 
 		//test with empty strings
 		$query = "";
 		$contact->add_list_count_joins($query, '');
-		$this->assertEquals(" LEFT JOIN contacts_cstm ON contacts.id = contacts_cstm.id_c ",$query);
+		$this->assertEquals(" LEFT JOIN contacts_cstm ON contacts.id = contacts_cstm.id_c ", $query);
 
 
 		//test with valid string
@@ -37,14 +37,14 @@ 
 		$expected = "\n	            LEFT JOIN accounts_contacts\n	            ON contacts.id=accounts_contacts.contact_id\n	            LEFT JOIN accounts\n	            ON accounts_contacts.account_id=accounts.id\n                    LEFT JOIN contacts_cstm ON contacts.id = contacts_cstm.id_c ";
 		$contact->add_list_count_joins($query, 'accounts.name');
 		$query = preg_replace('/\s+/', '', $query);
-		$expected =preg_replace('/\s+/', '', $expected);
-		$this->assertSame($expected,$query);
+		$expected = preg_replace('/\s+/', '', $expected);
+		$this->assertSame($expected, $query);
 		
 		//test with valid string
 		$query = "";
 		$expected = "\n	            LEFT JOIN accounts_contacts\n	            ON contacts.id=accounts_contacts.contact_id\n	            LEFT JOIN accounts\n	            ON accounts_contacts.account_id=accounts.id\n	                 LEFT JOIN contacts_cstm ON contacts.id = contacts_cstm.id_c ";
 		$contact->add_list_count_joins($query, 'contacts.name');
-		$this->assertSame(" LEFT JOIN contacts_cstm ON contacts.id = contacts_cstm.id_c ",$query);
+		$this->assertSame(" LEFT JOIN contacts_cstm ON contacts.id = contacts_cstm.id_c ", $query);
 
 
 	}
@@ -53,9 +53,9 @@ 
 	{
 		$contact = new Contact();
 
-		$expected = array( "MAIN"=>"a", "ACCOUNT"=>"a");
+		$expected = array("MAIN"=>"a", "ACCOUNT"=>"a");
 		$actual = $contact->listviewACLHelper();
-		$this->assertSame($expected,$actual);
+		$this->assertSame($expected, $actual);
 
 	}
 
@@ -89,14 +89,14 @@ 
 
 		//test with empty string params
 		$expected = "SELECT LTRIM(RTRIM(CONCAT(IFNULL(contacts.first_name,''),'',IFNULL(contacts.last_name,'')))) name, \n				contacts.*,\n                accounts.name as account_name,\n                accounts.id as account_id,\n                accounts.assigned_user_id account_id_owner,\n                users.user_name as assigned_user_name ,contacts_cstm.*\n                FROM contacts LEFT JOIN users\n	                    ON contacts.assigned_user_id=users.id\n	                    LEFT JOIN accounts_contacts\n	                    ON contacts.id=accounts_contacts.contact_id  and accounts_contacts.deleted = 0\n	                    LEFT JOIN accounts\n	                    ON accounts_contacts.account_id=accounts.id AND accounts.deleted=0 LEFT JOIN email_addr_bean_rel eabl  ON eabl.bean_id = contacts.id AND eabl.bean_module = 'Contacts' and eabl.primary_address = 1 and eabl.deleted=0 LEFT JOIN email_addresses ea ON (ea.id = eabl.email_address_id)  LEFT JOIN contacts_cstm ON contacts.id = contacts_cstm.id_c where  contacts.deleted=0 ";
-		$actual = $contact->address_popup_create_new_list_query('','');
-		$this->assertSame($expected,$actual);
+		$actual = $contact->address_popup_create_new_list_query('', '');
+		$this->assertSame($expected, $actual);
 
 
 		//test with valid string params
 		$expected = "SELECT LTRIM(RTRIM(CONCAT(IFNULL(contacts.first_name,''),'',IFNULL(contacts.last_name,'')))) name, \n				contacts.*,\n                accounts.name as account_name,\n                accounts.id as account_id,\n                accounts.assigned_user_id account_id_owner,\n                users.user_name as assigned_user_name ,contacts_cstm.*\n                FROM contacts LEFT JOIN users\n	                    ON contacts.assigned_user_id=users.id\n	                    LEFT JOIN accounts_contacts\n	                    ON contacts.id=accounts_contacts.contact_id  and accounts_contacts.deleted = 0\n	                    LEFT JOIN accounts\n	                    ON accounts_contacts.account_id=accounts.id AND accounts.deleted=0 LEFT JOIN email_addr_bean_rel eabl  ON eabl.bean_id = contacts.id AND eabl.bean_module = 'Contacts' and eabl.primary_address = 1 and eabl.deleted=0 LEFT JOIN email_addresses ea ON (ea.id = eabl.email_address_id)  LEFT JOIN contacts_cstm ON contacts.id = contacts_cstm.id_c where (contacts.name=\"\") AND  contacts.deleted=0 ";
-		$actual = $contact->address_popup_create_new_list_query('contacts.id','contacts.name=""');
-		$this->assertSame($expected,$actual);
+		$actual = $contact->address_popup_create_new_list_query('contacts.id', 'contacts.name=""');
+		$this->assertSame($expected, $actual);
 
 	}
 
@@ -106,14 +106,14 @@ 
 
 		//test with empty string params
 		$expected = "SELECT\n                                contacts.*,\n                                email_addresses.email_address email_address,\n                                '' email_addresses_non_primary, accounts.name as account_name,\n                                users.user_name as assigned_user_name ,contacts_cstm.jjwg_maps_lng_c,contacts_cstm.jjwg_maps_lat_c,contacts_cstm.jjwg_maps_geocode_status_c,contacts_cstm.jjwg_maps_address_c FROM contacts LEFT JOIN users\n	                                ON contacts.assigned_user_id=users.id LEFT JOIN accounts_contacts\n	                                ON ( contacts.id=accounts_contacts.contact_id and (accounts_contacts.deleted is null or accounts_contacts.deleted = 0))\n	                                LEFT JOIN accounts\n	                                ON accounts_contacts.account_id=accounts.id  LEFT JOIN  email_addr_bean_rel on contacts.id = email_addr_bean_rel.bean_id and email_addr_bean_rel.bean_module='Contacts' and email_addr_bean_rel.deleted=0 and email_addr_bean_rel.primary_address=1  LEFT JOIN email_addresses on email_addresses.id = email_addr_bean_rel.email_address_id  LEFT JOIN contacts_cstm ON contacts.id = contacts_cstm.id_c where ( accounts.deleted IS NULL OR accounts.deleted=0 )\n                      AND contacts.deleted=0 ";
-		$actual = $contact->create_export_query('','');
-		$this->assertSame($expected,$actual);
+		$actual = $contact->create_export_query('', '');
+		$this->assertSame($expected, $actual);
 
 
 		//test with valid string params
 		$expected = "SELECT\n                                contacts.*,\n                                email_addresses.email_address email_address,\n                                '' email_addresses_non_primary, accounts.name as account_name,\n                                users.user_name as assigned_user_name ,contacts_cstm.jjwg_maps_lng_c,contacts_cstm.jjwg_maps_lat_c,contacts_cstm.jjwg_maps_geocode_status_c,contacts_cstm.jjwg_maps_address_c FROM contacts LEFT JOIN users\n	                                ON contacts.assigned_user_id=users.id LEFT JOIN accounts_contacts\n	                                ON ( contacts.id=accounts_contacts.contact_id and (accounts_contacts.deleted is null or accounts_contacts.deleted = 0))\n	                                LEFT JOIN accounts\n	                                ON accounts_contacts.account_id=accounts.id  LEFT JOIN  email_addr_bean_rel on contacts.id = email_addr_bean_rel.bean_id and email_addr_bean_rel.bean_module='Contacts' and email_addr_bean_rel.deleted=0 and email_addr_bean_rel.primary_address=1  LEFT JOIN email_addresses on email_addresses.id = email_addr_bean_rel.email_address_id  LEFT JOIN contacts_cstm ON contacts.id = contacts_cstm.id_c where (contacts.name=\"\") AND ( accounts.deleted IS NULL OR accounts.deleted=0 )\n                      AND contacts.deleted=0 ";
-		$actual = $contact->create_export_query('contacts.id','contacts.name=""');
-		$this->assertSame($expected,$actual);
+		$actual = $contact->create_export_query('contacts.id', 'contacts.name=""');
+		$this->assertSame($expected, $actual);
 
 	}
 
@@ -130,9 +130,9 @@ 
 
 		$contact->fill_in_additional_list_fields();
 
-		$this->assertEquals("firstn lastn",$contact->full_name);
-		$this->assertEquals("firstn lastn <1@test.com>",$contact->email_and_name1);
-		$this->assertEquals("firstn lastn <2@test.com>",$contact->email_and_name2);
+		$this->assertEquals("firstn lastn", $contact->full_name);
+		$this->assertEquals("firstn lastn <1@test.com>", $contact->email_and_name1);
+		$this->assertEquals("firstn lastn <2@test.com>", $contact->email_and_name2);
 
 	}
 
@@ -145,14 +145,14 @@ 
 
 		$contact->fill_in_additional_detail_fields();
 
-		$this->assertEquals("",$contact->account_name);
-		$this->assertEquals("",$contact->account_id);
-		$this->assertEquals("",$contact->report_to_name);
+		$this->assertEquals("", $contact->account_name);
+		$this->assertEquals("", $contact->account_id);
+		$this->assertEquals("", $contact->report_to_name);
 
 	}
 
 
-	public function testload_contacts_users_relationship(){
+	public function testload_contacts_users_relationship() {
 
 		$contact = new Contact();
 
@@ -175,7 +175,7 @@ 
 		$contact->first_name = "first";
 		$contact->last_name = "last";
 
-		$expected = array (
+		$expected = array(
 					  'NAME' => 'first last',
 					  'DELETED' => 0,
 					  'FIRST_NAME' => 'first',
@@ -201,7 +201,7 @@ 
 	}
 
 
-	public function testbuild_generic_where_clause ()
+	public function testbuild_generic_where_clause()
 	{
 
 		$contact = new Contact();
@@ -209,13 +209,13 @@ 
 		//test with string
 		$expected = "contacts.last_name like 'test%' or contacts.first_name like 'test%' or accounts.name like 'test%' or contacts.assistant like 'test%' or ea.email_address like 'test%'";
 		$actual = $contact->build_generic_where_clause('test');
-		$this->assertSame($expected,$actual);
+		$this->assertSame($expected, $actual);
 
 
 		//test with number
 		$expected = "contacts.last_name like '1%' or contacts.first_name like '1%' or accounts.name like '1%' or contacts.assistant like '1%' or ea.email_address like '1%' or contacts.phone_home like '%1%' or contacts.phone_mobile like '%1%' or contacts.phone_work like '%1%' or contacts.phone_other like '%1%' or contacts.phone_fax like '%1%' or contacts.assistant_phone like '%1%'";
 		$actual = $contact->build_generic_where_clause(1);
-		$this->assertSame($expected,$actual);
+		$this->assertSame($expected, $actual);
 
 	}
 
@@ -231,8 +231,8 @@ 
 
 		$result = $contact->set_notification_body(new Sugar_Smarty(), $contact);
 
-		$this->assertEquals($contact->full_name ,$result->_tpl_vars['CONTACT_NAME']);
-		$this->assertEquals($contact->description ,$result->_tpl_vars['CONTACT_DESCRIPTION']);
+		$this->assertEquals($contact->full_name, $result->_tpl_vars['CONTACT_NAME']);
+		$this->assertEquals($contact->description, $result->_tpl_vars['CONTACT_DESCRIPTION']);
 
 	}
 
@@ -241,7 +241,7 @@ 
 		$contact = new Contact();
 
 		$result = $contact->get_contact_id_by_email("");
-		$this->assertEquals(null,$result);
+		$this->assertEquals(null, $result);
 
 
 		//$result = $contact->get_contact_id_by_email("test@test.com");
@@ -282,7 +282,7 @@ 
 		//execute the method and verify that it retunrs expected results
 		$expected = "SELECT emails.id FROM emails  JOIN (select DISTINCT email_id from emails_email_addr_rel eear\n\n	join email_addr_bean_rel eabr on eabr.bean_id ='' and eabr.bean_module = 'Contacts' and\n	eabr.email_address_id = eear.email_address_id and eabr.deleted=0\n	where eear.deleted=0 and eear.email_id not in\n	(select eb.email_id from emails_beans eb where eb.bean_module ='Contacts' and eb.bean_id = '')\n	) derivedemails on derivedemails.email_id = emails.id";
 		$actual = $contact->get_unlinked_email_query();
-		$this->assertSame($expected,$actual);
+		$this->assertSame($expected, $actual);
 
 	}
 

tests/tests/modules/AOR_Reports/AOR_ReportTest.php

Spacing +1 added lines, -1 removed lines

@@ -35,7 +35,7 @@
             $output = $me->jsParser();
 
             return $output;
-        } catch (Exception $e) {
+        }catch (Exception $e) {
             // Exception handling is left up to the implementer.
             throw $e;
         }

modules/Tasks/metadata/SearchFields.php

Spacing +17 added lines, -17 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.
@@ -40,10 +40,10 @@ 
 global $current_user;
 $module_name = "Tasks";
 $searchFields['Tasks'] = 
-	array (
-		'name' => array( 'query_type'=>'default'),
+	array(
+		'name' => array('query_type'=>'default'),
         'contact_name' => array('query_type' => 'default', 'db_field' => array('contacts.first_name', 'contacts.last_name'), 'force_unifiedsearch' => true),        
-        'current_user_only'=> array('query_type'=>'default','db_field'=>array('assigned_user_id'),'my_items'=>true, 'vname' => 'LBL_CURRENT_USER_FILTER', 'type' => 'bool'),
+        'current_user_only'=> array('query_type'=>'default', 'db_field'=>array('assigned_user_id'), 'my_items'=>true, 'vname' => 'LBL_CURRENT_USER_FILTER', 'type' => 'bool'),
         'assigned_user_id'=> array('query_type'=>'default'),
         'status'=> array('query_type'=>'default', 'options' => 'task_status_dom', 'template_var' => 'STATUS_FILTER'),
         
@@ -60,22 +60,22 @@ 
 			'subquery' => "SELECT favorites.parent_id FROM favorites
 			                    WHERE favorites.deleted = 0
 			                        and favorites.parent_type = '".$module_name."'
-			                        and favorites.assigned_user_id = '" .$current_user->id . "') OR NOT ({0}",
+			                        and favorites.assigned_user_id = '" .$current_user->id."') OR NOT ({0}",
             'db_field'=>array('id')),
 		//Range Search Support
-	   'range_date_entered' => array ('query_type' => 'default', 'enable_range_search' => true, 'is_date_field' => true),
-	   'start_range_date_entered' => array ('query_type' => 'default',  'enable_range_search' => true, 'is_date_field' => true),
-	   'end_range_date_entered' => array ('query_type' => 'default', 'enable_range_search' => true, 'is_date_field' => true),
-	   'range_date_modified' => array ('query_type' => 'default', 'enable_range_search' => true, 'is_date_field' => true),
-	   'start_range_date_modified' => array ('query_type' => 'default',  'enable_range_search' => true, 'is_date_field' => true),
-       'end_range_date_modified' => array ('query_type' => 'default', 'enable_range_search' => true, 'is_date_field' => true),	
+	   'range_date_entered' => array('query_type' => 'default', 'enable_range_search' => true, 'is_date_field' => true),
+	   'start_range_date_entered' => array('query_type' => 'default', 'enable_range_search' => true, 'is_date_field' => true),
+	   'end_range_date_entered' => array('query_type' => 'default', 'enable_range_search' => true, 'is_date_field' => true),
+	   'range_date_modified' => array('query_type' => 'default', 'enable_range_search' => true, 'is_date_field' => true),
+	   'start_range_date_modified' => array('query_type' => 'default', 'enable_range_search' => true, 'is_date_field' => true),
+       'end_range_date_modified' => array('query_type' => 'default', 'enable_range_search' => true, 'is_date_field' => true),	
 
-	   'range_date_start' => array ('query_type' => 'default', 'enable_range_search' => true, 'is_date_field' => true),
-	   'start_range_date_start' => array ('query_type' => 'default',  'enable_range_search' => true, 'is_date_field' => true),
-	   'end_range_date_start' => array ('query_type' => 'default', 'enable_range_search' => true, 'is_date_field' => true),
-	   'range_date_due' => array ('query_type' => 'default', 'enable_range_search' => true, 'is_date_field' => true),
-	   'start_range_date_due' => array ('query_type' => 'default',  'enable_range_search' => true, 'is_date_field' => true),
-       'end_range_date_due' => array ('query_type' => 'default', 'enable_range_search' => true, 'is_date_field' => true),	
+	   'range_date_start' => array('query_type' => 'default', 'enable_range_search' => true, 'is_date_field' => true),
+	   'start_range_date_start' => array('query_type' => 'default', 'enable_range_search' => true, 'is_date_field' => true),
+	   'end_range_date_start' => array('query_type' => 'default', 'enable_range_search' => true, 'is_date_field' => true),
+	   'range_date_due' => array('query_type' => 'default', 'enable_range_search' => true, 'is_date_field' => true),
+	   'start_range_date_due' => array('query_type' => 'default', 'enable_range_search' => true, 'is_date_field' => true),
+       'end_range_date_due' => array('query_type' => 'default', 'enable_range_search' => true, 'is_date_field' => true),	
 		//Range Search Support 				
 	);
 ?>

modules/Calls/metadata/SearchFields.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.
@@ -39,33 +39,33 @@ 
  ********************************************************************************/
 global $current_user;
 $module_name = "Calls";
-$searchFields['Calls'] = array (
+$searchFields['Calls'] = array(
   'name' =>
-  array (
+  array(
     'query_type' => 'default',
   ),
   'contact_name' => 
-  array (
+  array(
     'query_type' => 'default',
     'db_field' => 
-    array (
+    array(
       0 => 'contacts.first_name',
       1 => 'contacts.last_name',
     ),
   ),
   'date_start' => 
-  array (
+  array(
     'query_type' => 'default',
   ),
   'location' => 
-  array (
+  array(
     'query_type' => 'default',
   ),
   'current_user_only' => 
-  array (
+  array(
     'query_type' => 'default',
     'db_field' => 
-    array (
+    array(
       0 => 'assigned_user_id',
     ),
     'my_items' => true,
@@ -73,98 +73,98 @@ 
     'type' => 'bool',
   ),
   'assigned_user_id' => 
-  array (
+  array(
     'query_type' => 'default',
   ),
   'status' => 
-  array (
+  array(
     'query_type' => 'default',
     'options' => 'call_status_dom',
     'template_var' => 'STATUS_FILTER',
   ),
   'open_only' => 
-  array (
+  array(
     'query_type' => 'default',
     'db_field' => 
-    array (
+    array(
       0 => 'status',
     ),
     'operator' => 'not in',
     'closed_values' => 
-    array (
+    array(
       0 => 'Held',
       1 => 'Not Held',
     ),
     'type' => 'bool',
   ),
   'range_date_entered' => 
-  array (
+  array(
     'query_type' => 'default',
     'enable_range_search' => true,
     'is_date_field' => true,
   ),
   'start_range_date_entered' => 
-  array (
+  array(
     'query_type' => 'default',
     'enable_range_search' => true,
     'is_date_field' => true,
   ),
   'end_range_date_entered' => 
-  array (
+  array(
     'query_type' => 'default',
     'enable_range_search' => true,
     'is_date_field' => true,
   ),
   'range_date_modified' => 
-  array (
+  array(
     'query_type' => 'default',
     'enable_range_search' => true,
     'is_date_field' => true,
   ),
   'start_range_date_modified' => 
-  array (
+  array(
     'query_type' => 'default',
     'enable_range_search' => true,
     'is_date_field' => true,
   ),
   'end_range_date_modified' => 
-  array (
+  array(
     'query_type' => 'default',
     'enable_range_search' => true,
     'is_date_field' => true,
   ),
   'range_date_start' => 
-  array (
+  array(
     'query_type' => 'default',
     'enable_range_search' => true,
     'is_date_field' => true,
   ),
   'start_range_date_start' => 
-  array (
+  array(
     'query_type' => 'default',
     'enable_range_search' => true,
     'is_date_field' => true,
   ),
   'end_range_date_start' => 
-  array (
+  array(
     'query_type' => 'default',
     'enable_range_search' => true,
     'is_date_field' => true,
   ),
   'range_date_end' => 
-  array (
+  array(
     'query_type' => 'default',
     'enable_range_search' => true,
     'is_date_field' => true,
   ),
   'start_range_date_end' => 
-  array (
+  array(
     'query_type' => 'default',
     'enable_range_search' => true,
     'is_date_field' => true,
   ),
   'end_range_date_end' => 
-  array (
+  array(
     'query_type' => 'default',
     'enable_range_search' => true,
     'is_date_field' => true,
@@ -175,6 +175,6 @@ 
         'subquery' => "SELECT favorites.parent_id FROM favorites
 			                    WHERE favorites.deleted = 0
 			                        and favorites.parent_type = '".$module_name."'
-			                        and favorites.assigned_user_id = '" .$current_user->id . "') OR NOT ({0}",
+			                        and favorites.assigned_user_id = '" .$current_user->id."') OR NOT ({0}",
         'db_field'=>array('id')),
 );
\ No newline at end of file

modules/Cases/metadata/SearchFields.php

Spacing +22 added lines, -22 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.
@@ -39,42 +39,42 @@ 
  ********************************************************************************/
 global $current_user;
 $module_name = "Cases";
-$searchFields['Cases'] = array (
+$searchFields['Cases'] = array(
   'name' => 
-  array (
+  array(
     'query_type' => 'default',
   ),
   'account_name' => 
-  array (
+  array(
     'query_type' => 'default',
     'db_field' => 
-    array (
+    array(
       0 => 'accounts.name',
     ),
   ),
   'status' => 
-  array (
+  array(
     'query_type' => 'default',
     'options' => 'case_status_dom',
     'template_var' => 'STATUS_OPTIONS',
   ),
   'priority' => 
-  array (
+  array(
     'query_type' => 'default',
     'options' => 'case_priority_dom',
     'template_var' => 'PRIORITY_OPTIONS',
     'options_add_blank' => true,
   ),
   'case_number' => 
-  array (
+  array(
     'query_type' => 'default',
     'operator' => 'in',
   ),
   'current_user_only' => 
-  array (
+  array(
     'query_type' => 'default',
     'db_field' => 
-    array (
+    array(
       0 => 'assigned_user_id',
     ),
     'my_items' => true,
@@ -82,19 +82,19 @@ 
     'type' => 'bool',
   ),
   'assigned_user_id' => 
-  array (
+  array(
     'query_type' => 'default',
   ),
   'open_only' => 
-  array (
+  array(
     'query_type' => 'default',
     'db_field' => 
-    array (
+    array(
       0 => 'status',
     ),
     'operator' => 'not in',
     'closed_values' => 
-    array (
+    array(
       0 => 'Closed',
       1 => 'Rejected',
       2 => 'Duplicate',
@@ -105,43 +105,43 @@ 
     'type' => 'bool',
   ),
   'range_date_entered' => 
-  array (
+  array(
     'query_type' => 'default',
     'enable_range_search' => true,
     'is_date_field' => true,
   ),
   'start_range_date_entered' => 
-  array (
+  array(
     'query_type' => 'default',
     'enable_range_search' => true,
     'is_date_field' => true,
   ),
   'end_range_date_entered' => 
-  array (
+  array(
     'query_type' => 'default',
     'enable_range_search' => true,
     'is_date_field' => true,
   ),
   'range_date_modified' => 
-  array (
+  array(
     'query_type' => 'default',
     'enable_range_search' => true,
     'is_date_field' => true,
   ),
   'start_range_date_modified' => 
-  array (
+  array(
     'query_type' => 'default',
     'enable_range_search' => true,
     'is_date_field' => true,
   ),
   'end_range_date_modified' => 
-  array (
+  array(
     'query_type' => 'default',
     'enable_range_search' => true,
     'is_date_field' => true,
   ),
   'state' => 
-  array (
+  array(
     'query_type' => 'default',
   ),
     'favorites_only' => array(
@@ -150,6 +150,6 @@ 
         'subquery' => "SELECT favorites.parent_id FROM favorites
 			                    WHERE favorites.deleted = 0
 			                        and favorites.parent_type = '".$module_name."'
-			                        and favorites.assigned_user_id = '" .$current_user->id . "') OR NOT ({0}",
+			                        and favorites.assigned_user_id = '" .$current_user->id."') OR NOT ({0}",
         'db_field'=>array('id')),
 );
\ No newline at end of file