YetiForceCompany / YetiForceCRM

include/recaptcha/recaptchalib.php

Spacing +1 added lines, -1 removed lines

@@ -77,7 +77,7 @@
 	$http_request .= $req;
 
 	$response = '';
-	if (false == ( $fs = @fsockopen($host, $port, $errno, $errstr, 10) )) {
+	if (false == ($fs = @fsockopen($host, $port, $errno, $errstr, 10))) {
 		die('Could not open socket');
 	}
 

include/InventoryPDFController.php

Spacing +1 added lines, -1 removed lines

@@ -148,7 +148,7 @@
 			$subProducts = $productLineItem["subProductArray{$productLineItemIndex}"];
 			if ($subProducts != '') {
 				foreach ($subProducts as $subProduct) {
-					$productName .="\n" . " - " . decode_html($subProduct);
+					$productName .= "\n" . " - " . decode_html($subProduct);
 				}
 			}
 			$contentModel->set('Name', $productName);

include/CRMEntity.php

Spacing +4 added lines, -4 removed lines

@@ -1970,7 +1970,7 @@ 
 					$matrix->addDependency($tab_name, $crmentityRelModuleFieldTable);
 
 					if ($queryPlanner->requireTable($crmentityRelModuleFieldTable, $matrix)) {
-						$relquery.= " left join vtiger_crmentity as $crmentityRelModuleFieldTable on $crmentityRelModuleFieldTable.crmid = $tab_name.$field_name and vtiger_crmentityRel$module$field_id.deleted=0";
+						$relquery .= " left join vtiger_crmentity as $crmentityRelModuleFieldTable on $crmentityRelModuleFieldTable.crmid = $tab_name.$field_name and vtiger_crmentityRel$module$field_id.deleted=0";
 					}
 
 					for ($j = 0; $j < $adb->num_rows($ui10_modules_query); $j++) {
@@ -1984,7 +1984,7 @@ 
 						$rel_tab_name_rel_module_table_alias = $rel_tab_name . "Rel$module$field_id";
 
 						if ($queryPlanner->requireTable($rel_tab_name_rel_module_table_alias)) {
-							$relquery.= " left join $rel_tab_name as $rel_tab_name_rel_module_table_alias  on $rel_tab_name_rel_module_table_alias.$rel_tab_index = $crmentityRelModuleFieldTable.crmid";
+							$relquery .= " left join $rel_tab_name as $rel_tab_name_rel_module_table_alias  on $rel_tab_name_rel_module_table_alias.$rel_tab_index = $crmentityRelModuleFieldTable.crmid";
 						}
 					}
 				}
@@ -2376,7 +2376,7 @@ 
 		$entitycolumnnames = $entityfields['fieldname'];
 		$query = "select crmid as id, $querycolumnnames, $entitycolumnnames as name ";
 		$query .= " FROM $this->table_name ";
-		$query .=" INNER JOIN vtiger_crmentity ON $this->table_name.$this->table_index = vtiger_crmentity.crmid && deleted = 0 ";
+		$query .= " INNER JOIN vtiger_crmentity ON $this->table_name.$this->table_index = vtiger_crmentity.crmid && deleted = 0 ";
 
 		//remove the base table
 		$LookupTable = array_unique($lookuptables);
@@ -2391,7 +2391,7 @@ 
 						on $this->table_name.$this->table_index = $tablename." . $this->tab_name_index[$tablename];
 		}
 		if (!empty($lookupcolumns) && $value !== false) {
-			$query .=" WHERE ";
+			$query .= " WHERE ";
 			$i = 0;
 			$columnCount = count($lookupcolumns);
 			foreach ($lookupcolumns as $columnname) {

include/ChartUtils.php

Spacing +2 added lines, -2 removed lines

@@ -56,7 +56,7 @@ 
 				$x = "";
 				for ($j = 0; $j < count($val); $j++) {
 					if ($j != $n) {
-						$x .=" " . $val[$j];
+						$x .= " " . $val[$j];
 					} else {
 						$x .= "@#" . $val[$j];
 					}
@@ -240,7 +240,7 @@ 
 				$x = "";
 				for ($j = 0; $j < count($val); $j++) {
 					if ($j != $n) {
-						$x .=" " . $val[$j];
+						$x .= " " . $val[$j];
 					} else {
 						$x .= "@#" . $val[$j];
 					}

include/simplehtmldom/simple_html_dom.php

Spacing +3 added lines, -3 removed lines

@@ -224,7 +224,7 @@ 
 	{
 	// I am SURE that this doesn't work properly.
 	// It fails to unset the current node from it's current parents nodes or children list first.
-	if($parent !== null)
+	if ($parent !== null)
 	{
 	$this->parent = $parent;
 	$this->parent->nodes[] = $this;
@@ -1427,7 +1427,7 @@ 
 		$node->tag = 'unknown';
 	}
 	if ($this->char === '>')
-		$node->_[HDOM_INFO_TEXT].='>';
+		$node->_[HDOM_INFO_TEXT] .= '>';
 	$this->link_nodes($node, true);
 	$this->char = ( ++$this->pos < $this->size) ? $this->doc[$this->pos] : null; // next
 	return true;
@@ -1450,7 +1450,7 @@ 
 	}
 
 	if ($this->char === '>')
-		$node->_[HDOM_INFO_TEXT].='>';
+		$node->_[HDOM_INFO_TEXT] .= '>';
 	$this->link_nodes($node, false);
 	$this->char = ( ++$this->pos < $this->size) ? $this->doc[$this->pos] : null; // next
 	return true;

include/utils/EmailTemplate.php

Spacing +2 added lines, -2 removed lines

@@ -120,7 +120,7 @@ 
 				$moduleTableIndexList = $meta->getEntityTableIndexList();
 				foreach ($tableList as $index => $tableName) {
 					if ($tableName != $tableList[0]) {
-						$sql .=' INNER JOIN ' . $tableName . ' ON ' . $tableList[0] . '.' .
+						$sql .= ' INNER JOIN ' . $tableName . ' ON ' . $tableList[0] . '.' .
 							$moduleTableIndexList[$tableList[0]] . '=' . $tableName . '.' .
 							$moduleTableIndexList[$tableName];
 					}
@@ -128,7 +128,7 @@ 
 				//If module is Leads and if you are not selected any leads fields then query failure is happening.
 				//By default we are checking where condition on base table.
 				if ($module == 'Leads' && !in_array('vtiger_leaddetails', $tableList)) {
-					$sql .=' INNER JOIN vtiger_leaddetails ON vtiger_leaddetails.leadid = vtiger_crmentity.crmid';
+					$sql .= ' INNER JOIN vtiger_leaddetails ON vtiger_leaddetails.leadid = vtiger_crmentity.crmid';
 				}
 
 				$sql .= ' WHERE';

include/utils/InventoryUtils.php

Spacing +8 added lines, -8 removed lines

@@ -313,7 +313,7 @@ 
 	$moduleName = $entity->getModuleName();
 
 	$statusChanged = false;
-	$vtEntityDelta = new VTEntityDelta ();
+	$vtEntityDelta = new VTEntityDelta();
 	$oldEntity = $vtEntityDelta->getOldValue($moduleName, $entity_id, $statusFieldName);
 	$recordDetails = $entity->getData();
 	$statusChanged = $vtEntityDelta->hasChanged($moduleName, $entity_id, $statusFieldName);
@@ -941,19 +941,19 @@ 
 		for ($i = 0; $i < $noOfRows; ++$i) {
 			$status = $adb->query_result($result, $i, 'temp_status');
 			if ($obj->getImportRecordStatus('none') == $status) {
-				$statusCount['PENDING'] ++;
+				$statusCount['PENDING']++;
 			} elseif ($obj->getImportRecordStatus('failed') == $status) {
-				$statusCount['FAILED'] ++;
+				$statusCount['FAILED']++;
 			} else {
-				$statusCount['IMPORTED'] ++;
+				$statusCount['IMPORTED']++;
 				switch ($status) {
-					case $obj->getImportRecordStatus('created') : $statusCount['CREATED'] ++;
+					case $obj->getImportRecordStatus('created') : $statusCount['CREATED']++;
 						break;
-					case $obj->getImportRecordStatus('skipped') : $statusCount['SKIPPED'] ++;
+					case $obj->getImportRecordStatus('skipped') : $statusCount['SKIPPED']++;
 						break;
-					case $obj->getImportRecordStatus('updated') : $statusCount['UPDATED'] ++;
+					case $obj->getImportRecordStatus('updated') : $statusCount['UPDATED']++;
 						break;
-					case $obj->getImportRecordStatus('merged') : $statusCount['MERGED'] ++;
+					case $obj->getImportRecordStatus('merged') : $statusCount['MERGED']++;
 						break;
 				}
 			}

include/utils/UserInfoUtil.php

Spacing +3 added lines, -3 removed lines

@@ -405,7 +405,7 @@ 
 		if (\AppConfig::security('PERMITTED_BY_RECORD_HIERARCHY')) {
 			$userPrivilegesModel = Users_Privileges_Model::getCurrentUserPrivilegesModel();
 			$role = $userPrivilegesModel->getRoleDetail();
-			if ((($actionid == 3 || $actionid == 4) && $role->get('previewrelatedrecord') != 0 ) || (($actionid == 0 || $actionid == 1) && $role->get('editrelatedrecord') != 0 )) {
+			if ((($actionid == 3 || $actionid == 4) && $role->get('previewrelatedrecord') != 0) || (($actionid == 0 || $actionid == 1) && $role->get('editrelatedrecord') != 0)) {
 				$parentRecord = Users_Privileges_Model::getParentRecord($record_id, $module, $role->get('previewrelatedrecord'), $actionid);
 				if ($parentRecord) {
 					$recordMetaData = vtlib\Functions::getCRMRecordMetadata($parentRecord);
@@ -1084,7 +1084,7 @@ 
 		$query = sprintf("SELECT shareid FROM %s WHERE %s = ?", $tablename, $colNameArr[0]);
 		$params = array($grpId);
 		if (sizeof($colNameArr) > 1) {
-			$query .=" or " . $colNameArr[1] . "=?";
+			$query .= " or " . $colNameArr[1] . "=?";
 			array_push($params, $grpId);
 		}
 
@@ -1120,7 +1120,7 @@ 
 		$query = sprintf("SELECT shareid FROM %s WHERE %s = ?", $tablename, $colNameArr[0]);
 		$params = array($grpId);
 		if (sizeof($colNameArr) > 1) {
-			$query .=" or " . $colNameArr[1] . "=?";
+			$query .= " or " . $colNameArr[1] . "=?";
 			array_push($params, $grpId);
 		}
 

include/utils/ListViewUtils.php

Spacing +5 added lines, -5 removed lines

@@ -194,15 +194,15 @@ 
 
 			//added to fix #5135
 			if (AppRequest::get('from_homepage') == 'upcoming_activities' || AppRequest::get('from_homepage') == 'pending_activities') {
-				$query.=' LEFT OUTER JOIN vtiger_recurringevents ON vtiger_recurringevents.activityid=vtiger_activity.activityid';
+				$query .= ' LEFT OUTER JOIN vtiger_recurringevents ON vtiger_recurringevents.activityid=vtiger_activity.activityid';
 			}
 			//end
 			$instance = CRMEntity::getInstance($module);
-			$query.=" WHERE vtiger_crmentity.deleted = 0 && activitytype != 'Emails' ";
+			$query .= " WHERE vtiger_crmentity.deleted = 0 && activitytype != 'Emails' ";
 			$securityParameter = $instance->getUserAccessConditionsQuerySR($module, $current_user);
 			if ($securityParameter != '')
-				$query.= $securityParameter;
-			$query.= ' ' . $where;
+				$query .= $securityParameter;
+			$query .= ' ' . $where;
 			break;
 		Case "Emails":
 			$query = "SELECT DISTINCT vtiger_crmentity.crmid, vtiger_crmentity.smownerid,
@@ -616,7 +616,7 @@ 
 	$chars = preg_split('/(&[^;\s]+;)|/', $str, -1, PREG_SPLIT_NO_EMPTY | PREG_SPLIT_DELIM_CAPTURE | PREG_SPLIT_OFFSET_CAPTURE);
 	$html_length = count($chars);
 	// check if we can predict the return value and save some processing time
-	if (($html_length === 0) || ( $start >= $html_length) || ( isset($length) && ( $length <= -$html_length)))
+	if (($html_length === 0) || ($start >= $html_length) || (isset($length) && ($length <= -$html_length)))
 		return "";
 
 	//calculate start position