YetiForceCompany / YetiForceCRM

modules/PaymentsOut/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/Products/models/ListView.php

Doc Comments -1 removed lines

@@ -332,7 +332,6 @@
 
 	/**
 	 * Function to get the list view entries
-	 * @param Vtiger_Paging_Model $pagingModel
 	 * @return <Array> - Associative array of record id mapped to Vtiger_Record_Model instance.
 	 */
 	public function getListViewCount()

modules/RecycleBin/actions/RecycleBinAjax.php

Doc Comments -3 removed lines

@@ -55,8 +55,6 @@ 
 
 	/**
 	 * Function to restore the deleted records.
-	 * @param type $sourceModule
-	 * @param type $recordIds
 	 */
 	public function restoreRecords(Vtiger_Request $request)
 	{
@@ -91,7 +89,6 @@ 
 
 	/**
 	 * Function to deleted the records permanently in CRM
-	 * @param type $reocrdIds
 	 */
 	public function deleteRecords(Vtiger_Request $request)
 	{

modules/RecycleBin/models/Module.php

Doc Comments +1 added lines, -2 removed lines

@@ -49,7 +49,6 @@ 
 
 	/**
 	 * Function to get the list of Mass actions for the module
-	 * @param <Array> $linkParams
 	 * @return <Array> - Associative array of Link type to List of  Vtiger_Link_Model instances for Mass Actions
 	 */
 	public function getListViewMassActions()
@@ -143,7 +142,7 @@ 
 
 	/**
 	 * Function to deleted the records perminently in CRM
-	 * @param type $reocrdIds
+	 * @param type $recordIds
 	 */
 	public function deleteRecords($recordIds)
 	{

modules/Reports/models/ListView.php

Doc Comments +2 added lines, -1 removed lines

@@ -16,6 +16,7 @@ 
 
 	/**
 	 * Function to get the list of listview links for the module
+	 * @param boolean $linkParams
 	 * @return <Array> - Associate array of Link Type to List of Vtiger_Link_Model instances
 	 */
 	public function getListViewLinks($linkParams)
@@ -70,7 +71,7 @@ 
 
 	/**
 	 * Function to get the list of Mass actions for the module
-	 * @param <Array> $linkParams
+	 * @param boolean $linkParams
 	 * @return <Array> - Associative array of Link type to List of  Vtiger_Link_Model instances for Mass Actions
 	 */
 	public function getListViewMassActions($linkParams)

modules/Settings/Github/models/Client.php

Doc Comments +5 added lines

@@ -106,6 +106,11 @@
 		return false;
 	}
 
+	/**
+	 * @param string $url
+	 * @param string $method
+	 * @param string $status
+	 */
 	private function doRequest($url, $method, $data, $status)
 	{
 		$url = self::url . $url;

Braces +6 added lines, -4 removed lines

@@ -122,14 +122,16 @@
 		switch ($method) {
 			case 'GET':
 				curl_setopt($curl, CURLOPT_HTTPGET, true);
-				if (count($data))
-					$url .= '?' . http_build_query($data);
+				if (count($data)) {
+									$url .= '?' . http_build_query($data);
+				}
 				break;
 
 			case 'POST':
 				curl_setopt($curl, CURLOPT_POST, true);
-				if (count($data))
-					curl_setopt($curl, CURLOPT_POSTFIELDS, $data);
+				if (count($data)) {
+									curl_setopt($curl, CURLOPT_POSTFIELDS, $data);
+				}
 				break;
 		}
 		curl_setopt($curl, CURLOPT_URL, $url);

Spacing +1 added lines, -1 removed lines

@@ -40,7 +40,7 @@
 		$path = '/search/issues';
 		$data['q'] = 'user:' . self::ownerRepository . ' repo:' . self::repository . " is:issue is:$state";
 		if ($author) {
-			$data['q'].=" author:$this->username";
+			$data['q'] .= " author:$this->username";
 		}
 		$issues = $this->doRequest($path, 'GET', $data, '200');
 		if ($issues === false) {

modules/Settings/GlobalPermission/models/Record.php

Doc Comments +3 added lines

@@ -42,6 +42,9 @@
 		return $globalPermissions;
 	}
 
+	/**
+	 * @param integer $checked
+	 */
 	public static function save($profileID, $globalactionid, $checked)
 	{
 		if ($globalactionid == 1) {

Spacing +2 added lines, -2 removed lines

@@ -31,12 +31,12 @@
 			->leftJoin('vtiger_profile', 'vtiger_profile.profileid = vtiger_profile2globalpermissions.profileid')
 			->createCommand()->query();
 		$globalPermissions = [];
-		while($row = $dataReader->read()) {
+		while ($row = $dataReader->read()) {
 			$profileid = $row['profileid'];
 			$actionId = $row['globalactionid'];
 			$permissionId = $row['globalactionpermission'];
 			$profilename = $row['profilename'];
-			$description =$row['description'];
+			$description = $row['description'];
 			$globalPermissions[$profileid]['gp_' . $actionId] = $permissionId;
 			$globalPermissions[$profileid]['profilename'] = $profilename;
 			$globalPermissions[$profileid]['description'] = $description;

modules/Settings/LangManagement/models/Module.php

Doc Comments +10 added lines

@@ -347,6 +347,9 @@ 
 		return $resp;
 	}
 
+	/**
+	 * @param string $dir
+	 */
 	public function DeleteDir($dir)
 	{
 		$fd = opendir($dir);
@@ -365,6 +368,10 @@ 
 		rmdir($dir);
 	}
 
+	/**
+	 * @param string $src
+	 * @param string $dst
+	 */
 	public function CopyDir($src, $dst)
 	{
 		$dir = opendir($src);
@@ -430,6 +437,9 @@ 
 		return $data;
 	}
 
+	/**
+	 * @param boolean $byModule
+	 */
 	public function getStats($data, $langBase, $byModule)
 	{
 		$differences = [];

Spacing +2 added lines, -2 removed lines

@@ -369,8 +369,8 @@
 	{
 		$dir = opendir($src);
 		@mkdir($dst);
-		while (false !== ( $file = readdir($dir))) {
-			if (( $file != '.' ) && ( $file != '..' )) {
+		while (false !== ($file = readdir($dir))) {
+			if (($file != '.') && ($file != '..')) {
 				if (is_dir($src . '/' . $file)) {
 					self::CopyDir($src . '/' . $file, $dst . '/' . $file);
 				} else {

Braces +14 added lines, -10 removed lines

@@ -349,11 +349,13 @@ 
 	public function DeleteDir($dir)
 	{
 		$fd = opendir($dir);
-		if (!$fd)
-			return false;
+		if (!$fd) {
+					return false;
+		}
 		while (($file = readdir($fd)) !== false) {
-			if ($file == "." || $file == "..")
-				continue;
+			if ($file == "." || $file == "..") {
+							continue;
+			}
 			if (is_dir($dir . "/" . $file)) {
 				self::DeleteDir($dir . "/" . $file);
 			} else {
@@ -405,10 +407,11 @@ 
 			$db->createCommand()->update('vtiger_language', ['isdefault' => 0], ['isdefault' => 1])->execute();
 		}
 		$status = $db->createCommand()->update('vtiger_language', ['isdefault' => 1], ['prefix' => $prefix])->execute();
-		if ($status)
-			$status = true;
-		else
-			$status = false;
+		if ($status) {
+					$status = true;
+		} else {
+					$status = false;
+		}
 		\App\Log::trace("Exiting Settings_LangManagement_Module_Model::setAsDefault() method ...");
 		return array('success' => $status, 'prefixOld' => $prefixOld);
 	}
@@ -435,8 +438,9 @@ 
 		$differences = [];
 		$i = 0;
 		foreach ($data as $id => $dataLang) {
-			if (!in_array($id, ['php', 'js']))
-				continue;
+			if (!in_array($id, ['php', 'js'])) {
+							continue;
+			}
 			foreach ($dataLang as $key => $langs) {
 				foreach ($langs as $lang => $value) {
 					if ($lang == $langBase) {

modules/Settings/LayoutEditor/models/Block.php

Doc Comments +1 added lines, -1 removed lines

@@ -99,7 +99,7 @@
 
 	/**
 	 * Function to retrieve block instances for a module
-	 * @param <type> $moduleModel - module instance
+	 * @param Settings_LayoutEditor_Module_Model $moduleModel - module instance
 	 * @return <array> - list of Vtiger_Block_Model
 	 */
 	public static function getAllForModule($moduleModel)