Passed
Pull Request — developer (#16008)
by Arkadiusz
21:29
created
api/webservice/Controller.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -71,7 +71,7 @@
 block discarded – undo
71 71
 	 */
72 72
 	public function preProcess(): bool
73 73
 	{
74
-		register_shutdown_function(function () {
74
+		register_shutdown_function(function() {
75 75
 			if ($error = error_get_last()) {
76 76
 				$this->errorHandler($error['type'], $error['message'], $error['file'], $error['line']);
77 77
 			}
Please login to merge, or discard this patch.
app/MeetingService.php 1 patch
Spacing   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -133,13 +133,13 @@
 block discarded – undo
133 133
 	public function generateRoomName(string $prefix = ''): string
134 134
 	{
135 135
 		$prefix = preg_replace_callback_array([
136
-			'/[^a-z0-9 ]/' => function () {
136
+			'/[^a-z0-9 ]/' => function() {
137 137
 				return '';
138 138
 			},
139
-			'/\b[a-z]/' => function ($matches) {
139
+			'/\b[a-z]/' => function($matches) {
140 140
 				return mb_strtoupper($matches[0]);
141 141
 			},
142
-			'/[\s]/' => function () {
142
+			'/[\s]/' => function() {
143 143
 				return '';
144 144
 			},
145 145
 		], strtolower(\App\Utils::sanitizeSpecialChars($prefix, ' ')));
Please login to merge, or discard this patch.
app/QueryGenerator.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -966,7 +966,7 @@  discard block
 block discarded – undo
966 966
 			$subQuery->andHaving((new \yii\db\Expression('COUNT(1) > 1')));
967 967
 			$this->joins['duplicates'] = ['INNER JOIN', ['duplicates' => $subQuery], implode(' AND ', $duplicateCheckClause)];
968 968
 		}
969
-		uksort($this->joins, function ($a, $b) use ($moduleTableIndexList) {
969
+		uksort($this->joins, function($a, $b) use ($moduleTableIndexList) {
970 970
 			return (int) (!isset($moduleTableIndexList[$a]) && isset($moduleTableIndexList[$b]));
971 971
 		});
972 972
 		foreach ($this->joins as $join) {
@@ -1362,7 +1362,7 @@  discard block
 block discarded – undo
1362 1362
 			$searchParamsConditions['condition'] = $glueOrder[$key];
1363 1363
 			$searchParamsConditions['rules'] = [];
1364 1364
 			foreach ($conditions as $condition) {
1365
-				[$fieldName, , $sourceFieldName] = array_pad(explode(':', $condition[0]), 3, false);
1365
+				[$fieldName,, $sourceFieldName] = array_pad(explode(':', $condition[0]), 3, false);
1366 1366
 				if (!$sourceFieldName) {
1367 1367
 					$condition[0] = "{$fieldName}:{$this->getModule()}";
1368 1368
 				}
Please login to merge, or discard this patch.
modules/Vtiger/models/Module.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -675,7 +675,7 @@
 block discarded – undo
675 675
 	{
676 676
 		$quickCreateFieldList = [];
677 677
 		foreach ($this->getFieldsByBlocks() as $blockFields) {
678
-			uksort($blockFields, function ($a, $b) use ($blockFields) {
678
+			uksort($blockFields, function($a, $b) use ($blockFields) {
679 679
 				if ($blockFields[$a]->get('quicksequence') === $blockFields[$b]->get('quicksequence')) {
680 680
 					return 0;
681 681
 				}
Please login to merge, or discard this patch.
vtlib/Vtiger/Link.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -185,7 +185,7 @@
 block discarded – undo
185 185
 				if (self::IGNORE_MODULE === $tabid) {
186 186
 					$permittedTabIdList = \vtlib\Deprecated::getPermittedModuleIdList();
187 187
 					if (!empty($permittedTabIdList)) {
188
-						$permittedTabIdList[] = 0;  // Added to support one link for all modules
188
+						$permittedTabIdList[] = 0; // Added to support one link for all modules
189 189
 						foreach ($permittedTabIdList as $moduleId) {
190 190
 							foreach ($type as $typ) {
191 191
 								if (isset($rows[$moduleId][$typ])) {
Please login to merge, or discard this patch.
modules/Vtiger/models/ChartFilter.php 1 patch
Spacing   +7 added lines, -7 removed lines patch added patch discarded remove patch
@@ -619,7 +619,7 @@  discard block
 block discarded – undo
619 619
 		$colors = \App\Fields\Picklist::getColors($fieldName);
620 620
 		$this->colorsFrom = 'picklist';
621 621
 		$this->colorsFromRow = 'picklist_id';
622
-		$this->iterateAllRows(function ($row, $groupValue, $dividingValue, $rowIndex) use ($colors) {
622
+		$this->iterateAllRows(function($row, $groupValue, $dividingValue, $rowIndex) use ($colors) {
623 623
 			if (isset($colors[$row['picklist_id']])) {
624 624
 				$this->colors[$row['picklist_id']] = $colors[$row['picklist_id']];
625 625
 			}
@@ -634,7 +634,7 @@  discard block
 block discarded – undo
634 634
 		$this->colorsFrom = 'filters';
635 635
 		$this->colorsFromRow = 'color';
636 636
 		$colors = \App\Colors::getAllFilterColors();
637
-		$this->iterateAllRows(function (&$row, $groupValue, $dividingValue, $rowIndex) use ($colors) {
637
+		$this->iterateAllRows(function(&$row, $groupValue, $dividingValue, $rowIndex) use ($colors) {
638 638
 			$this->colors[$dividingValue] = $colors[$this->filterIds[$dividingValue]] ?? null;
639 639
 		});
640 640
 	}
@@ -646,7 +646,7 @@  discard block
 block discarded – undo
646 646
 	{
647 647
 		$this->colorsFrom = 'assigned_user_id';
648 648
 		$this->colorsFromRow = 'assigned_user_id';
649
-		$this->iterateAllRows(function ($row, $groupValue, $dividingValue, $rowIndex) {
649
+		$this->iterateAllRows(function($row, $groupValue, $dividingValue, $rowIndex) {
650 650
 			$this->colors[$row['assigned_user_id']] = \App\Fields\Owner::getColor($row['assigned_user_id']);
651 651
 		});
652 652
 	}
@@ -658,7 +658,7 @@  discard block
 block discarded – undo
658 658
 	{
659 659
 		$this->colorsFrom = 'record_id';
660 660
 		$this->colorsFromRow = 'id';
661
-		$this->iterateAllRows(function ($row, $groupValue, $dividingValue, $rowIndex) {
661
+		$this->iterateAllRows(function($row, $groupValue, $dividingValue, $rowIndex) {
662 662
 			$this->colors[$row['id']] = \App\Colors::getRandomColor('from_id_' . $row['id']);
663 663
 		});
664 664
 	}
@@ -670,7 +670,7 @@  discard block
 block discarded – undo
670 670
 	{
671 671
 		$this->colorsFrom = 'record_number';
672 672
 		$this->colorsFromRow = 'record_number';
673
-		$this->iterateAllRows(function ($row, $groupValue, $dividingValue, $rowIndex) {
673
+		$this->iterateAllRows(function($row, $groupValue, $dividingValue, $rowIndex) {
674 674
 			$this->colors[$groupValue] = \App\Colors::getRandomColor('generated_' . $groupValue);
675 675
 		});
676 676
 	}
@@ -895,7 +895,7 @@  discard block
 block discarded – undo
895 895
 					$firstReturnValueForSort = 1;
896 896
 					$secondReturnValueForSort = -1;
897 897
 				}
898
-				uksort($dataForSort, function ($a, $b) use ($dataForSort, $firstReturnValueForSort, $secondReturnValueForSort) {
898
+				uksort($dataForSort, function($a, $b) use ($dataForSort, $firstReturnValueForSort, $secondReturnValueForSort) {
899 899
 					if ($dataForSort[$a]['avg'] === $dataForSort[$b]['avg']) {
900 900
 						return 0;
901 901
 					}
@@ -998,7 +998,7 @@  discard block
 block discarded – undo
998 998
 		$this->calculateAverage();
999 999
 		$this->normalizeData();
1000 1000
 		$this->setColorsFrom($this->findOutColorsFromRows());
1001
-		$this->iterateAllRows(function ($row, $groupValue, $dividingValue, $rowIndex) {
1001
+		$this->iterateAllRows(function($row, $groupValue, $dividingValue, $rowIndex) {
1002 1002
 			$this->setColorFromRow($row, $groupValue, $dividingValue);
1003 1003
 		});
1004 1004
 		return $this->data;
Please login to merge, or discard this patch.
modules/OSSMailScanner/actions/SaveCRMuser.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -65,7 +65,7 @@
 block discarded – undo
65 65
 		$moduleName = $request->getModule();
66 66
 		$userId = $request->getInteger('userid');
67 67
 		$status = $request->getInteger('status');
68
-		if (!\in_array($status, [OSSMail_Record_Model::MAIL_BOX_STATUS_ACTIVE,  OSSMail_Record_Model::MAIL_BOX_STATUS_DISABLED])) {
68
+		if (!\in_array($status, [OSSMail_Record_Model::MAIL_BOX_STATUS_ACTIVE, OSSMail_Record_Model::MAIL_BOX_STATUS_DISABLED])) {
69 69
 			throw new \App\Exceptions\IllegalValue('ERR_NOT_ALLOWED_VALUE||' . $status, 406);
70 70
 		}
71 71
 		if ($userId) {
Please login to merge, or discard this patch.
app/Encryptions/Module.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -92,7 +92,7 @@
 block discarded – undo
92 92
 			$dataReader->close();
93 93
 
94 94
 			if ($decryptInstance->getQueryGenerator((int) $pauser->getValue())->createQuery()->exists()) {
95
-				(new \App\BatchMethod(['method' => __CLASS__ . '::recalculatePasswords', 'params' => [$method,  $password,  $vector,  $target, microtime()]]))->save();
95
+				(new \App\BatchMethod(['method' => __CLASS__ . '::recalculatePasswords', 'params' => [$method, $password, $vector, $target, microtime()]]))->save();
96 96
 			} else {
97 97
 				(new \App\ConfigFile('module', \App\Module::getModuleName($target)))
98 98
 					->set('encryptionMethod', $method)
Please login to merge, or discard this patch.
modules/Passwords/ConfigTemplate.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -10,14 +10,14 @@
 block discarded – undo
10 10
 	'encryptionPass' => [
11 11
 		'default' => 'yeti',
12 12
 		'description' => 'Key to encrypt passwords, changing the key results in the loss of all encrypted data.',
13
-		'validation' => function () {
13
+		'validation' => function() {
14 14
 			return true;
15 15
 		}
16 16
 	],
17 17
 	'encryptionMethod' => [
18 18
 		'default' => 'aes-256-cbc',
19 19
 		'description' => 'Encryption method.',
20
-		'validation' => function () {
20
+		'validation' => function() {
21 21
 			$arg = func_get_arg(0);
22 22
 			return empty($arg) || ($arg && \in_array($arg, \App\Encryption::getMethods()));
23 23
 		}
Please login to merge, or discard this patch.