YetiForceCompany / YetiForceCRM

modules/PaymentsIn/helpers/subclass/mt940_Milenium.php

Doc Comments +6 added lines

@@ -20,6 +20,9 @@ 
 			$this->parseLine($line);
 	}
 
+	/**
+	 * @param string $line
+	 */
 	protected function parseLine($line)
 	{
 		$tag = substr($line, 1, strpos($line, ':', 1) - 1);
@@ -65,6 +68,9 @@ 
 		$this->_lastTag = $tag;
 	}
 
+	/**
+	 * @param string $value
+	 */
 	protected function parseOperation($value)
 	{
 		$this->operations[] = array(

Braces +13 added lines, -10 removed lines

@@ -16,8 +16,9 @@ 
 	public function parse()
 	{
 		$tab = $this->prepareFile();
-		foreach ($tab as $line)
-			$this->parseLine($line);
+		foreach ($tab as $line) {
+					$this->parseLine($line);
+		}
 	}
 
 	protected function parseLine($line)
@@ -36,10 +37,11 @@ 
 				break;
 			case 'NS':
 				$code = substr($value, 0, 2);
-				if ($code == '22')
-					$this->ownerName = substr($value, 2);
-				else if ($code == '23')
-					$this->accountName = substr($value, 2);
+				if ($code == '22') {
+									$this->ownerName = substr($value, 2);
+				} else if ($code == '23') {
+									$this->accountName = substr($value, 2);
+				}
 				break;
 			case '60F':
 				$this->openBalance = $this->parseBalance($value);
@@ -54,10 +56,11 @@ 
 				self::parseOperation($value);
 				break;
 			case '86':
-				if ($this->_lastTag == '61')
-					$this->parseTransaction($value);
-				else
-					$this->info .= $value;
+				if ($this->_lastTag == '61') {
+									$this->parseTransaction($value);
+				} else {
+									$this->info .= $value;
+				}
 				break;
 			default:
 				break;

modules/Reports/Reports.php

Doc Comments +1 added lines

@@ -387,6 +387,7 @@
 	 *  This function accepts the folderid
 	 *  This Generates the Reports under each Reports module
 	 *  This Returns a HTML sring
+	 * @param boolean $rpt_fldr_id
 	 */
 	public function sgetRptsforFldr($rpt_fldr_id, $paramsList = false)
 	{

Spacing +7 added lines, -7 removed lines

@@ -188,9 +188,9 @@ 
 			if ($modulerows) {
 				foreach ($modulerows as $resultrow) {
 					if ($resultrow['presence'] == '1')
-						continue;   // skip disabled modules
+						continue; // skip disabled modules
 					if ($resultrow['isentitytype'] != '1')
-						continue;  // skip extension modules
+						continue; // skip extension modules
 					if (in_array($resultrow['name'], $restricted_modules)) { // skip restricted modules
 						continue;
 					}
@@ -307,7 +307,7 @@ 
 					$details = [];
 					$details['state'] = $reportfldrow["state"];
 					$details['id'] = $reportfldrow["folderid"];
-					$details['name'] = ($mod_strings[$reportfldrow["foldername"]] == '' ) ? $reportfldrow["foldername"] : $mod_strings[$reportfldrow["foldername"]];
+					$details['name'] = ($mod_strings[$reportfldrow["foldername"]] == '') ? $reportfldrow["foldername"] : $mod_strings[$reportfldrow["foldername"]];
 					$details['description'] = $reportfldrow["description"];
 					$details['fname'] = popup_decode_html($details['name']);
 					$details['fdescription'] = popup_decode_html($reportfldrow["description"]);
@@ -320,7 +320,7 @@ 
 				$details = [];
 				$details['state'] = $reportfldrow["state"];
 				$details['id'] = $reportfldrow["folderid"];
-				$details['name'] = ($mod_strings[$reportfldrow["foldername"]] == '' ) ? $reportfldrow["foldername"] : $mod_strings[$reportfldrow["foldername"]];
+				$details['name'] = ($mod_strings[$reportfldrow["foldername"]] == '') ? $reportfldrow["foldername"] : $mod_strings[$reportfldrow["foldername"]];
 				$details['description'] = $reportfldrow["description"];
 				$details['fname'] = popup_decode_html($details['name']);
 				$details['fdescription'] = popup_decode_html($reportfldrow["description"]);
@@ -375,7 +375,7 @@ 
 
 				if (isPermitted($report["primarymodule"], 'index') == "yes")
 					$returndata[] = $report_details;
-			}while ($report = $adb->fetch_array($result));
+			} while ($report = $adb->fetch_array($result));
 		}
 		\App\Log::trace("Reports :: ListView->Successfully returned vtiger_report details HTML");
 		return $returndata;
@@ -462,7 +462,7 @@ 
 
 				if (isPermitted($report["primarymodule"], 'index') == "yes")
 					$returndata [$report["folderid"]][] = $report_details;
-			}while ($report = $adb->fetch_array($result));
+			} while ($report = $adb->fetch_array($result));
 		}
 
 		if ($rpt_fldr_id !== false) {
@@ -1264,7 +1264,7 @@ 
 		}
 
 		$temp_val = explode(",", $adv_filter_value);
-		if (($column_info[4] == 'D' || ($column_info[4] == 'T' && $column_info[1] != 'time_start' && $column_info[1] != 'time_end') || ($column_info[4] == 'DT')) && ($column_info[4] != '' && $adv_filter_value != '' )) {
+		if (($column_info[4] == 'D' || ($column_info[4] == 'T' && $column_info[1] != 'time_start' && $column_info[1] != 'time_end') || ($column_info[4] == 'DT')) && ($column_info[4] != '' && $adv_filter_value != '')) {
 			$val = [];
 			$countTempVal = count($temp_val);
 			for ($x = 0; $x < $countTempVal; $x++) {

Braces +73 added lines, -50 removed lines

@@ -122,10 +122,11 @@ 
 				$this->reportname = decode_html($cachedInfo["reportname"]);
 				$this->reportdescription = decode_html($cachedInfo["description"]);
 				$this->folderid = $cachedInfo["folderid"];
-				if ($is_admin === true || in_array($cachedInfo["owner"], $subordinate_users) || $cachedInfo["owner"] == $current_user->id)
-					$this->is_editable = 'true';
-				else
-					$this->is_editable = 'false';
+				if ($is_admin === true || in_array($cachedInfo["owner"], $subordinate_users) || $cachedInfo["owner"] == $current_user->id) {
+									$this->is_editable = 'true';
+				} else {
+									$this->is_editable = 'false';
+				}
 			} else {
 				throw new \App\Exceptions\NoPermitted('LBL_PERMISSION_DENIED');
 			}
@@ -136,8 +137,9 @@ 
 	public function updateModuleList($module)
 	{
 		$adb = PearDatabase::getInstance();
-		if (!isset($module))
-			return;
+		if (!isset($module)) {
+					return;
+		}
 		require_once('include/utils/utils.php');
 		$tabid = \App\Module::getModuleId($module);
 		if ($module == 'Calendar') {
@@ -146,12 +148,14 @@ 
 		$sql = sprintf('SELECT blockid, blocklabel FROM vtiger_blocks WHERE tabid IN (%s)', generateQuestionMarks($tabid));
 		$res = $adb->pquery($sql, [$tabid]);
 		$noOfRows = $adb->num_rows($res);
-		if ($noOfRows <= 0)
-			return;
+		if ($noOfRows <= 0) {
+					return;
+		}
 		for ($index = 0; $index < $noOfRows; ++$index) {
 			$blockid = $adb->query_result($res, $index, 'blockid');
-			if (in_array($blockid, $this->module_list[$module]))
-				continue;
+			if (in_array($blockid, $this->module_list[$module])) {
+							continue;
+			}
 			$blocklabel = $adb->query_result($res, $index, 'blocklabel');
 			$this->module_list[$module][$blocklabel] = $blockid;
 		}
@@ -181,10 +185,14 @@ 
 
 			if ($modulerows) {
 				foreach ($modulerows as $resultrow) {
-					if ($resultrow['presence'] == '1')
-						continue;   // skip disabled modules
-					if ($resultrow['isentitytype'] != '1')
-						continue;  // skip extension modules
+					if ($resultrow['presence'] == '1') {
+											continue;
+					}
+					// skip disabled modules
+					if ($resultrow['isentitytype'] != '1') {
+											continue;
+					}
+					// skip extension modules
 					if (in_array($resultrow['name'], $restricted_modules)) { // skip restricted modules
 						continue;
 					}
@@ -215,10 +223,11 @@ 
 						}
 
 						if (!empty($blocklabel)) {
-							if ($module == 'Calendar' && $blocklabel == 'LBL_CUSTOM_INFORMATION')
-								$this->module_list[$module][$blockid] = \App\Language::translate($blocklabel, $module);
-							else
-								$this->module_list[$module][$blockid] = \App\Language::translate($blocklabel, $module);
+							if ($module == 'Calendar' && $blocklabel == 'LBL_CUSTOM_INFORMATION') {
+															$this->module_list[$module][$blockid] = \App\Language::translate($blocklabel, $module);
+							} else {
+															$this->module_list[$module][$blockid] = \App\Language::translate($blocklabel, $module);
+							}
 							$prev_block_label = $blocklabel;
 						} else {
 							$this->module_list[$module][$blockid] = \App\Language::translate($prev_block_label, $module);
@@ -357,13 +366,15 @@ 
 				$report_details ['reportname'] = $report["reportname"];
 				$report_details ['sharingtype'] = $report["sharingtype"];
 				$report_details['folderid'] = $report["folderid"];
-				if ($is_admin === true)
-					$report_details ['editable'] = 'true';
-				else
-					$report_details['editable'] = 'false';
+				if ($is_admin === true) {
+									$report_details ['editable'] = 'true';
+				} else {
+									$report_details['editable'] = 'false';
+				}
 
-				if (isPermitted($report["primarymodule"], 'index') == "yes")
-					$returndata[] = $report_details;
+				if (isPermitted($report["primarymodule"], 'index') == "yes") {
+									$returndata[] = $report_details;
+				}
 			}while ($report = $adb->fetch_array($result));
 		}
 		\App\Log::trace("Reports :: ListView->Successfully returned vtiger_report details HTML");
@@ -444,13 +455,15 @@ 
 				$report_details['reportname'] = $report["reportname"];
 				$report_details['reporttype'] = $report["reporttype"];
 				$report_details['sharingtype'] = $report["sharingtype"];
-				if ($is_admin === true || in_array($report["owner"], $subordinate_users) || $report["owner"] == $currentUser->getId())
-					$report_details['editable'] = 'true';
-				else
-					$report_details['editable'] = 'false';
+				if ($is_admin === true || in_array($report["owner"], $subordinate_users) || $report["owner"] == $currentUser->getId()) {
+									$report_details['editable'] = 'true';
+				} else {
+									$report_details['editable'] = 'false';
+				}
 
-				if (isPermitted($report["primarymodule"], 'index') == "yes")
-					$returndata [$report["folderid"]][] = $report_details;
+				if (isPermitted($report["primarymodule"], 'index') == "yes") {
+									$returndata [$report["folderid"]][] = $report_details;
+				}
 			}while ($report = $adb->fetch_array($result));
 		}
 
@@ -565,8 +578,9 @@ 
 		$adb = PearDatabase::getInstance();
 		$currentUser = Users_Privileges_Model::getCurrentUserPrivilegesModel();
 
-		if (is_string($block))
-			$block = explode(',', $block);
+		if (is_string($block)) {
+					$block = explode(',', $block);
+		}
 		$skipTalbes = array('vtiger_attachments');
 
 		$tabid = \App\Module::getModuleId($module);
@@ -584,10 +598,11 @@ 
 		$sql .= ' and tablename NOT IN (' . generateQuestionMarks($skipTalbes) . ') ';
 
 		//fix for Ticket #4016
-		if ($module == "Calendar")
-			$sql .= " group by vtiger_field.fieldlabel order by sequence";
-		else
-			$sql .= " group by vtiger_field.fieldid order by sequence";
+		if ($module == "Calendar") {
+					$sql .= " group by vtiger_field.fieldlabel order by sequence";
+		} else {
+					$sql .= " group by vtiger_field.fieldid order by sequence";
+		}
 		array_push($params, $skipTalbes);
 
 		$result = $adb->pquery($sql, $params);
@@ -629,8 +644,9 @@ 
 				$this->adv_rel_fields[$fieldtypeofdata][] = $adv_rel_field_tod_value;
 			}
 			//added to escape attachments fields in Reports as we have multiple attachments
-			if ($module == 'HelpDesk' && $fieldname == 'filename')
-				continue;
+			if ($module == 'HelpDesk' && $fieldname == 'filename') {
+							continue;
+			}
 
 			if (is_string($block) || $group_res_by_block === false) {
 				$module_columnlist[$optionvalue] = $fieldlabel;
@@ -868,8 +884,9 @@ 
 
 			$result = $adb->pquery($ssql, array($reportid, $groupId));
 			$noOfColumns = $adb->num_rows($result);
-			if ($noOfColumns <= 0)
-				continue;
+			if ($noOfColumns <= 0) {
+							continue;
+			}
 
 			while ($relcriteriarow = $adb->fetch_array($result)) {
 				$criteria = [];
@@ -925,8 +942,9 @@ 
 			$i++;
 		}
 		// Clear the condition (and/or) for last group, if any.
-		if (!empty($advft_criteria[$i - 1]['condition']))
-			$advft_criteria[$i - 1]['condition'] = '';
+		if (!empty($advft_criteria[$i - 1]['condition'])) {
+					$advft_criteria[$i - 1]['condition'] = '';
+		}
 		$this->advft_criteria = $advft_criteria;
 		\App\Log::trace("Reports :: Successfully returned getAdvancedFilterList");
 		return true;
@@ -1167,13 +1185,15 @@ 
 	$idelrelcriteriagroupsql = 'delete from vtiger_relcriteria_grouping where queryid=?';
 	$adb->pquery($idelrelcriteriagroupsql, array($reportid));
 
-	if (empty($advft_criteria))
-		return;
+	if (empty($advft_criteria)) {
+			return;
+	}
 
 	foreach ($advft_criteria as $column_index => $column_condition) {
 
-		if (empty($column_condition))
-			continue;
+		if (empty($column_condition)) {
+					continue;
+		}
 
 		$adv_filter_column = $column_condition["columnname"];
 		$adv_filter_comparator = $column_condition["comparator"];
@@ -1234,10 +1254,13 @@ 
 
 	foreach ($advft_criteria_groups as $group_index => $group_condition_info) {
 
-		if (empty($group_condition_info))
-			continue;
-		if (empty($group_condition_info["conditionexpression"]))
-			continue; // Case when the group doesn't have any column criteria
+		if (empty($group_condition_info)) {
+					continue;
+		}
+		if (empty($group_condition_info["conditionexpression"])) {
+					continue;
+		}
+		// Case when the group doesn't have any column criteria
 
 		$irelcriteriagroupsql = "insert into vtiger_relcriteria_grouping(GROUPID,QUERYID,GROUP_CONDITION,CONDITION_EXPRESSION) values (?,?,?,?)";
 		$adb->pquery($irelcriteriagroupsql, array($group_index, $reportid, $group_condition_info["groupcondition"], $group_condition_info["conditionexpression"]));

modules/Reports/ScheduledReports.php

Doc Comments +3 added lines

@@ -340,6 +340,9 @@
 		return $rolesAndSubHTML;
 	}
 
+	/**
+	 * @param Users $user
+	 */
 	public static function getScheduledReports($adb, $user)
 	{
 

Braces +15 added lines, -10 removed lines

@@ -258,8 +258,9 @@ 
 	public static function generateRecipientOption($type, $value, $name = '')
 	{
 		switch ($type) {
-			case 'users' : if (empty($name))
-					$name = \App\Fields\Owner::getUserLabel($value);
+			case 'users' : if (empty($name)) {
+								$name = \App\Fields\Owner::getUserLabel($value);
+			}
 				$optionName = 'User::' . addslashes(decode_html($name));
 				$optionValue = 'users::' . $value;
 				break;
@@ -269,13 +270,15 @@ 
 				$optionName = 'Group::' . addslashes(decode_html($name));
 				$optionValue = 'groups::' . $value;
 				break;
-			case 'roles' : if (empty($name))
-					$name = \App\PrivilegeUtil::getRoleName($value);
+			case 'roles' : if (empty($name)) {
+								$name = \App\PrivilegeUtil::getRoleName($value);
+			}
 				$optionName = 'Roles::' . addslashes(decode_html($name));
 				$optionValue = 'roles::' . $value;
 				break;
-			case 'rs' : if (empty($name))
-					$name = \App\PrivilegeUtil::getRoleName($value);
+			case 'rs' : if (empty($name)) {
+								$name = \App\PrivilegeUtil::getRoleName($value);
+			}
 				$optionName = 'RoleAndSubordinates::' . addslashes(decode_html($name));
 				$optionValue = 'rs::' . $value;
 				break;
@@ -375,10 +378,12 @@ 
 
 		$currentModule = vglobal('currentModule');
 		$current_language = vglobal('current_language');
-		if (empty($currentModule))
-			$currentModule = 'Reports';
-		if (empty($current_language))
-			vglobal('current_language', 'en_us');
+		if (empty($currentModule)) {
+					$currentModule = 'Reports';
+		}
+		if (empty($current_language)) {
+					vglobal('current_language', 'en_us');
+		}
 
 		$scheduledReports = self::getScheduledReports($adb, $adminUser);
 		foreach ($scheduledReports as $scheduledReport) {

modules/Users/CreateUserPrivilegeFile.php

Doc Comments +6 added lines, -6 removed lines

@@ -938,8 +938,8 @@ 
 
 /** Gives an array which contains the information for what all roles, groups and user's related module data that is to be shared  for the specified parent module and shared module 
 
- * @param $par_mod -- parent module name:: Type varchar
- * @param $share_mod -- shared module name:: Type varchar
+ * @param string $par_mod -- parent module name:: Type varchar
+ * @param string $share_mod -- shared module name:: Type varchar
  * @param $userid -- user id:: Type integer
  * @param $def_org_share -- default organization sharing permission array:: Type array
  * @param $mod_sharingrule_members -- Sharing Rule Members array:: Type array
@@ -1355,9 +1355,9 @@ 
 
 /** Function to populate the read/wirte Sharing permissions data for the specified user into the database 
  * @param $userid -- user id:: Type integer
- * @param $enttype -- can have the value of User or Group:: Type varchar
+ * @param string $enttype -- can have the value of User or Group:: Type varchar
  * @param $module -- module name:: Type varchar
- * @param $pertype -- can have the value of read or write:: Type varchar
+ * @param string $pertype -- can have the value of read or write:: Type varchar
  * @param $var_name_arr - Variable to use instead of including the sharing access again
  */
 function populateSharingPrivileges($enttype, $userid, $module, $pertype, $var_name_arr = false)
@@ -1432,10 +1432,10 @@ 
 
 /** Function to populate the read/wirte Sharing permissions related module data for the specified user into the database 
  * @param $userid -- user id:: Type integer
- * @param $enttype -- can have the value of User or Group:: Type varchar
+ * @param string $enttype -- can have the value of User or Group:: Type varchar
  * @param $module -- module name:: Type varchar
  * @param $relmodule -- related module name:: Type varchar
- * @param $pertype -- can have the value of read or write:: Type varchar
+ * @param string $pertype -- can have the value of read or write:: Type varchar
  * @param $var_name_arr - Variable to use instead of including the sharing access again
  */
 function populateRelatedSharingPrivileges($enttype, $userid, $module, $relmodule, $pertype, $var_name_arr = false)

Braces +12 added lines, -8 removed lines

@@ -592,8 +592,9 @@ 
 			$var_name = $module . '_share_write_permission';
 		}
 		// Lookup for the variable if not set through function argument		
-		if (!$var_name_arr)
-			$var_name_arr = $$var_name;
+		if (!$var_name_arr) {
+					$var_name_arr = $$var_name;
+		}
 		$user_arr = [];
 		if (sizeof($var_name_arr['ROLE']) > 0) {
 			foreach ($var_name_arr['ROLE'] as $roleid => $roleusers) {
@@ -627,8 +628,9 @@ 
 			$var_name = $module . '_share_write_permission';
 		}
 		// Lookup for the variable if not set through function argument
-		if (!$var_name_arr)
-			$var_name_arr = $$var_name;
+		if (!$var_name_arr) {
+					$var_name_arr = $$var_name;
+		}
 		$grp_arr = [];
 		if (sizeof($var_name_arr['GROUP']) > 0) {
 
@@ -671,8 +673,9 @@ 
 			$var_name = $module . '_' . $relmodule . '_share_write_permission';
 		}
 		// Lookup for the variable if not set through function argument
-		if (!$var_name_arr)
-			$var_name_arr = $$var_name;
+		if (!$var_name_arr) {
+					$var_name_arr = $$var_name;
+		}
 		$user_arr = [];
 		if (sizeof($var_name_arr['ROLE']) > 0) {
 			foreach ($var_name_arr['ROLE'] as $roleid => $roleusers) {
@@ -706,8 +709,9 @@ 
 			$var_name = $module . '_' . $relmodule . '_share_write_permission';
 		}
 		// Lookup for the variable if not set through function argument
-		if (!$var_name_arr)
-			$var_name_arr = $$var_name;
+		if (!$var_name_arr) {
+					$var_name_arr = $$var_name;
+		}
 		$grp_arr = [];
 		if (sizeof($var_name_arr['GROUP']) > 0) {
 

Indentation -10 removed lines

@@ -150,7 +150,6 @@ 
 }
 
 /** Gives an array which contains the information for what all roles, groups and user's related module data that is to be shared  for the specified parent module and shared module 
-
  * @param $par_mod -- parent module name:: Type varchar
  * @param $share_mod -- shared module name:: Type varchar
  * @param $userid -- user id:: Type integer
@@ -310,7 +309,6 @@ 
 }
 
 /** Converts the input array  to a single string to facilitate the writing of the input array in a flat file 
-
  * @param $var -- input array:: Type array
  * @returns $code -- contains the whole array in a single string:: Type array 
  */
@@ -327,7 +325,6 @@ 
 }
 
 /** Converts the input array  to a single string to facilitate the writing of the input array in a flat file 
-
  * @param $var -- input array:: Type array
  * @returns $code -- contains the whole array in a single string:: Type array 
  */
@@ -352,7 +349,6 @@ 
 }
 
 /** Converts the input array  to a single string to facilitate the writing of the input array in a flat file 
-
  * @param $var -- input array:: Type array
  * @returns $code -- contains the whole array in a single string:: Type array 
  */
@@ -377,7 +373,6 @@ 
 }
 
 /** Converts the input array  to a single string to facilitate the writing of the input array in a flat file 
-
  * @param $var -- input array:: Type array
  * @returns $code -- contains the whole array in a single string:: Type array 
  */
@@ -394,7 +389,6 @@ 
 }
 
 /** Converts the input array  to a single string to facilitate the writing of the input array in a flat file 
-
  * @param $var -- input array:: Type array
  * @returns $code -- contains the whole array in a single string:: Type array 
  */
@@ -411,7 +405,6 @@ 
 }
 
 /** Converts the input array  to a single string to facilitate the writing of the input array in a flat file 
-
  * @param $var -- input array:: Type array
  * @returns $code -- contains the whole array in a single string:: Type array 
  */
@@ -432,7 +425,6 @@ 
 }
 
 /** Converts the input array  to a single string to facilitate the writing of the input array in a flat file 
-
  * @param $var -- input array:: Type array
  * @returns $code -- contains the whole array in a single string:: Type array 
  */
@@ -453,7 +445,6 @@ 
 }
 
 /** Converts the input array  to a single string to facilitate the writing of the input array in a flat file 
-
  * @param $var -- input array:: Type array
  * @returns $code -- contains the whole array in a single string:: Type array 
  */
@@ -474,7 +465,6 @@ 
 }
 
 /** Converts the input array  to a single string to facilitate the writing of the input array in a flat file 
-
  * @param $var -- input array:: Type array
  * @returns $code -- contains the whole array in a single string:: Type array 
  */

modules/Vtiger/actions/GenerateRecords.php

Doc Comments +3 added lines

@@ -17,6 +17,9 @@
 		}
 	}
 
+	/**
+	 * @param Vtiger_Record_Model $recordModel
+	 */
 	public function checkMandatoryFields($recordModel)
 	{
 		$mandatoryFields = $recordModel->getModule()->getMandatoryFieldModels();

modules/Vtiger/models/ChartFilter.php

Doc Comments +6 added lines

@@ -113,6 +113,9 @@ 
 		return [];
 	}
 
+	/**
+	 * @param Vtiger_Widget_Model $widgetModel
+	 */
 	public function setWidgetModel($widgetModel)
 	{
 		$this->widgetModel = $widgetModel;
@@ -127,6 +130,9 @@ 
 		}
 	}
 
+	/**
+	 * @return string
+	 */
 	public function getType()
 	{
 		return $this->extraData['chartType'];

Spacing +2 added lines, -2 removed lines

@@ -110,7 +110,7 @@ 
 					if (!isset($groupData[$displayValue]['count'])) {
 						$groupData[$displayValue]['count'] = 1;
 					} else {
-						$groupData[$displayValue]['count'] ++;
+						$groupData[$displayValue]['count']++;
 					}
 					if (!isset($groupData[$displayValue]['link'])) {
 						$moduleModel = $this->getTargetModuleModel();
@@ -127,7 +127,7 @@ 
 					if (!isset($count[$sectorId])) {
 						$count[$sectorId] = 1;
 					} else {
-						$count[$sectorId] ++;
+						$count[$sectorId]++;
 					}
 				}
 			}

modules/Vtiger/models/InventoryField.php

Doc Comments -5 removed lines

@@ -16,8 +16,6 @@ 
 
 	/**
 	 * Create the name of the Inventory data table
-	 * @param string $module Module name
-	 * @param string $prefix Prefix table
 	 * @return string Table name
 	 */
 	public function getTableName($type = 'data')
@@ -118,8 +116,6 @@ 
 
 	/**
 	 * Get inventory columns
-	 * @param string $module Module name
-	 * @param boolean $returnInBlock Should the result be divided into blocks
 	 * @return array Inventory columns
 	 */
 	public function getColumns()
@@ -164,7 +160,6 @@ 
 
 	/**
 	 * Retrieve list of all fields
-	 * @param string $moduleName Module name
 	 * @return array Fields instance Vtiger_Basic_InventoryField
 	 */
 	public function getAllFields()

Braces +6 added lines, -4 removed lines

@@ -133,8 +133,9 @@ 
 		$columns = [];
 		foreach ($this->getFields() as $key => $field) {
 			$column = $field->getColumnName();
-			if (!empty($column) && $column != '-')
-				$columns[] = $column;
+			if (!empty($column) && $column != '-') {
+							$columns[] = $column;
+			}
 			foreach ($field->getCustomColumn() as $name => $field) {
 				$columns[] = $name;
 			}
@@ -187,8 +188,9 @@ 
 		}
 		$fields = [];
 		foreach ($fieldPaths as $fieldPath) {
-			if (!is_dir($fieldPath))
-				continue;
+			if (!is_dir($fieldPath)) {
+							continue;
+			}
 			foreach (new DirectoryIterator($fieldPath) as $fileinfo) {
 				if ($fileinfo->isFile() && $fileinfo->getFilename() != 'Basic.php') {
 					$fieldName = str_replace('.php', '', $fileinfo->getFilename());

modules/Vtiger/models/ModuleMeta.php

Doc Comments +1 added lines, -1 removed lines

@@ -42,7 +42,7 @@
 
 	/**
 	 * Functions returns webservices meta object
-	 * @return webservices meta
+	 * @return boolean meta
 	 */
 	public function getMeta()
 	{

vtlib/tools/console.php

Doc Comments +13 added lines

@@ -32,6 +32,9 @@ 
 		
 	}
 
+	/**
+	 * @param boolean $interactive
+	 */
 	public function setArguments($args, $interactive)
 	{
 		$this->arguments = $args;
@@ -156,11 +159,18 @@ 
 		} while (true);
 	}
 
+	/**
+	 * @return string
+	 */
 	protected function toAlphaNumeric($value)
 	{
 		return preg_replace("/[^a-zA-Z0-9_]/", "", $value);
 	}
 
+	/**
+	 * @param string $dir
+	 * @param string $file_pattern
+	 */
 	protected function findFiles($dir, $file_pattern, &$files)
 	{
 		$items = glob($dir . '/*', GLOB_NOSORT);
@@ -257,6 +267,9 @@ 
 		echo "DONE.\n";
 	}
 
+	/**
+	 * @param string $name
+	 */
 	public function find($name)
 	{
 		return vtlib\Module::getInstance($name);

Braces +6 added lines, -4 removed lines

@@ -116,8 +116,9 @@ 
 	protected function prompt($msg = '', $type = self::PROMPT_ANY)
 	{
 		do {
-			if ($msg)
-				echo $msg;
+			if ($msg) {
+							echo $msg;
+			}
 			$value = trim(fgets(STDIN));
 
 			if (!$value && $type == self::PROMPT_OPTIONAL) {
@@ -463,8 +464,9 @@ 
 			$filename = basename($file, true);
 			$dir = substr($file, 0, strpos($file, $filename));
 			$dir = str_replace('languages/' . self::BASE_LANG_PREFIX, 'languages/' . $languageInformation['prefix'], $dir);
-			if (!file_exists($dir))
-				mkdir($dir);
+			if (!file_exists($dir)) {
+							mkdir($dir);
+			}
 
 			if (isset($languageInformation['prefix_value'])) {
 				$contents = file_get_contents($file);