Passed
Pull Request — developer (#15456)
by Arkadiusz
20:21
created
modules/OSSMail/ConfigTemplate.php 1 patch
Spacing   +15 added lines, -15 removed lines patch added patch discarded remove patch
@@ -9,7 +9,7 @@  discard block
 block discarded – undo
9 9
 	'default_host' => [
10 10
 		'default' => ['ssl://imap.gmail.com' => 'ssl://imap.gmail.com'],
11 11
 		'description' => 'Default host.',
12
-		'validation' => function () {
12
+		'validation' => function() {
13 13
 			$arg = func_get_arg(0);
14 14
 			if (!$arg) {
15 15
 				return false;
@@ -22,7 +22,7 @@  discard block
 block discarded – undo
22 22
 			}
23 23
 			return true;
24 24
 		},
25
-		'sanitization' => function () {
25
+		'sanitization' => function() {
26 26
 			$values = func_get_arg(0);
27 27
 			if (!\is_array($values)) {
28 28
 				$values = [$values];
@@ -39,14 +39,14 @@  discard block
 block discarded – undo
39 39
 		'default' => 993,
40 40
 		'description' => 'Port used to connect to IMAP.',
41 41
 		'validation' => '\App\Validator::port',
42
-		'sanitization' => function () {
42
+		'sanitization' => function() {
43 43
 			return (int) func_get_arg(0);
44 44
 		}
45 45
 	],
46 46
 	'smtp_server' => [
47 47
 		'default' => 'ssl://smtp.gmail.com',
48 48
 		'description' => 'Name of SMTP server',
49
-		'validation' => function () {
49
+		'validation' => function() {
50 50
 			$arg = func_get_arg(0);
51 51
 			return $arg && \App\Purifier::purify($arg);
52 52
 		},
@@ -56,14 +56,14 @@  discard block
 block discarded – undo
56 56
 		'default' => 465,
57 57
 		'description' => 'Default smtp port',
58 58
 		'validation' => '\App\Validator::port',
59
-		'sanitization' => function () {
59
+		'sanitization' => function() {
60 60
 			return (int) func_get_arg(0);
61 61
 		}
62 62
 	],
63 63
 	'username_domain' => [
64 64
 		'default' => 'gmail.com',
65 65
 		'description' => 'User name domain',
66
-		'validation' => function () {
66
+		'validation' => function() {
67 67
 			$arg = func_get_arg(0);
68 68
 			return '' === $arg || \App\Validator::domain($arg);
69 69
 		}
@@ -87,7 +87,7 @@  discard block
 block discarded – undo
87 87
 	'smtp_user' => [
88 88
 		'default' => '%u',
89 89
 		'description' => 'Login to SMTP server',
90
-		'validation' => function () {
90
+		'validation' => function() {
91 91
 			$arg = func_get_arg(0);
92 92
 			return $arg && !is_numeric($arg) && \is_string($arg) && $arg === strip_tags($arg) && 256 > \App\TextParser::getTextLength($arg);
93 93
 		},
@@ -96,7 +96,7 @@  discard block
 block discarded – undo
96 96
 	'smtp_pass' => [
97 97
 		'default' => '%p',
98 98
 		'description' => "SMTP password (if required) if you use %p as the password Roundcube will use the current user's password for login",
99
-		'validation' => function () {
99
+		'validation' => function() {
100 100
 			$arg = func_get_arg(0);
101 101
 			return !empty($arg) && 256 > \App\TextParser::getTextLength($arg);
102 102
 		}
@@ -104,7 +104,7 @@  discard block
 block discarded – undo
104 104
 	'language' => [
105 105
 		'default' => 'en_US',
106 106
 		'description' => 'Set default language',
107
-		'validation' => function () {
107
+		'validation' => function() {
108 108
 			$arg = func_get_arg(0);
109 109
 			return $arg && \in_array($arg, \Settings_OSSMail_Config_Model::LANGUAGES);
110 110
 		}
@@ -112,11 +112,11 @@  discard block
 block discarded – undo
112 112
 	'skin_logo' => [
113 113
 		'default' => '/images/null.png',
114 114
 		'description' => 'Skin logo',
115
-		'validation' => function () {
115
+		'validation' => function() {
116 116
 			$arg = func_get_arg(0);
117 117
 			return !empty($arg) && \App\Purifier::purify($arg);
118 118
 		},
119
-		'sanitization' => function () {
119
+		'sanitization' => function() {
120 120
 			$arg = func_get_arg(0);
121 121
 			return ['*' => \App\Purifier::encodeHtml($arg)];
122 122
 		}
@@ -136,11 +136,11 @@  discard block
 block discarded – undo
136 136
 	'identities_level' => [
137 137
 		'default' => 0,
138 138
 		'description' => 'Identities level.',
139
-		'validation' => function () {
139
+		'validation' => function() {
140 140
 			$arg = func_get_arg(0);
141 141
 			return is_numeric($arg) && \in_array($arg, [0, 1, 2, 3, 4]);
142 142
 		},
143
-		'sanitization' => function () {
143
+		'sanitization' => function() {
144 144
 			return (int) func_get_arg(0);
145 145
 		}
146 146
 	],
@@ -148,7 +148,7 @@  discard block
 block discarded – undo
148 148
 		'default' => 30,
149 149
 		'description' => 'Session lifetime in minutes',
150 150
 		'validation' => '\App\Validator::naturalNumber',
151
-		'sanitization' => function () {
151
+		'sanitization' => function() {
152 152
 			return (int) func_get_arg(0);
153 153
 		}
154 154
 	],
@@ -353,7 +353,7 @@  discard block
 block discarded – undo
353 353
 	'useragent' => [
354 354
 		'default' => 'YetiForce Webmail',
355 355
 		'description' => 'Add this user-agent to message headers when sending',
356
-		'validation' => function () {
356
+		'validation' => function() {
357 357
 			$arg = func_get_arg(0);
358 358
 			return $arg && \App\Purifier::purify($arg);
359 359
 		},
Please login to merge, or discard this patch.
app/Mail/ScannerAction/CreatedHelpDesk.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -75,7 +75,7 @@
 block discarded – undo
75 75
 				$dbCommand->insert('vtiger_senotesrel', ['crmid' => $id, 'notesid' => $documentId])->execute();
76 76
 			}
77 77
 			$dataReader->close();
78
-			unset($dataReader,$query, $dbCommand, $recordModel);
78
+			unset($dataReader, $query, $dbCommand, $recordModel);
79 79
 		}
80 80
 	}
81 81
 
Please login to merge, or discard this patch.
app/Layout.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -186,7 +186,7 @@
 block discarded – undo
186 186
 			return '';
187 187
 		}
188 188
 		$teaser = $css = $btn = '';
189
-		$btnTemplate = function (string $popoverText = '', ?string $btnClass = ''): string {
189
+		$btnTemplate = function(string $popoverText = '', ?string $btnClass = ''): string {
190 190
 			$popoverText = \App\Language::translate($popoverText);
191 191
 			return "<a href=\"#\" class=\"js-more noLinkBtn font-weight-lighter js-popover-tooltip {$btnClass}\" data-iframe=\"true\" data-content=\"{$popoverText}\"><span class=\"mdi mdi-overscan\"></span></a>";
192 192
 		};
Please login to merge, or discard this patch.
modules/Settings/Widgets/views/Widget.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -60,7 +60,7 @@
 block discarded – undo
60 60
 		$viewer->assign('WID', '');
61 61
 		$viewer->assign('WIDGETINFO', [
62 62
 			'data' => [
63
-				'limit' => 5, 'relatedmodule' => '',  'columns' => '',
63
+				'limit' => 5, 'relatedmodule' => '', 'columns' => '',
64 64
 				'action' => '', 'switchHeader' => '', 'filter' => '',
65 65
 				'checkbox' => '', 'customView' => '[]',
66 66
 			],
Please login to merge, or discard this patch.
app/Utils/ConfReport.php 1 patch
Spacing   +5 added lines, -5 removed lines patch added patch discarded remove patch
@@ -336,13 +336,13 @@  discard block
 block discarded – undo
336 336
 		'spaceBackup' => ['container' => 'env', 'type' => 'Space', 'testCli' => false, 'label' => 'SPACE_BACKUP'],
337 337
 		'lastCronStart' => ['container' => 'env', 'testCli' => false, 'label' => 'LAST_CRON_START', 'isHtml' => true],
338 338
 		'crmProvider' => ['container' => 'env', 'testCli' => true, 'label' => 'CRM_PROVIDER'],
339
-		'open_basedir' => ['container' => 'php',  'type' => 'OpenBasedir', 'testCli' => true, 'mode' => 'showWarnings'],
339
+		'open_basedir' => ['container' => 'php', 'type' => 'OpenBasedir', 'testCli' => true, 'mode' => 'showWarnings'],
340 340
 		'caCertBundle' => ['recommended' => 'On', 'container' => 'env', 'type' => 'OnOff', 'testCli' => true, 'label' => 'CACERTBUNDLE'],
341 341
 		'caCertBundlePath' => ['recommended' => 'On', 'container' => 'env', 'testCli' => true, 'label' => 'CACERTBUNDLE_PATH'],
342 342
 		'SSL_CERT_FILE' => ['container' => 'env', 'testCli' => true, 'label' => 'SSL_CERT_FILE'],
343 343
 		'SSL_CERT_DIR' => ['container' => 'env', 'testCli' => true, 'label' => 'SSL_CERT_DIR'],
344
-		'openssl.cafile' => ['container' => 'php',  'type' => 'NotEmpty', 'testCli' => true, 'mode' => 'showWarnings'],
345
-		'openssl.capath' => ['container' => 'php',  'type' => 'NotEmpty', 'testCli' => true, 'mode' => 'showWarnings'],
344
+		'openssl.cafile' => ['container' => 'php', 'type' => 'NotEmpty', 'testCli' => true, 'mode' => 'showWarnings'],
345
+		'openssl.capath' => ['container' => 'php', 'type' => 'NotEmpty', 'testCli' => true, 'mode' => 'showWarnings'],
346 346
 	];
347 347
 
348 348
 	/**
@@ -426,8 +426,8 @@  discard block
 block discarded – undo
426 426
 	 * @var array
427 427
 	 */
428 428
 	public static $functionalVerification = [
429
-		'branding' => ['type' => 'Branding',  'testCli' => false, 'label' => 'FOOTER', 'mode' => 'onlyText'],
430
-		'shop' => ['type' => 'ShopProducts',  'testCli' => false, 'label' => 'PREMIUM_MODULES', 'mode' => 'onlyText'],
429
+		'branding' => ['type' => 'Branding', 'testCli' => false, 'label' => 'FOOTER', 'mode' => 'onlyText'],
430
+		'shop' => ['type' => 'ShopProducts', 'testCli' => false, 'label' => 'PREMIUM_MODULES', 'mode' => 'onlyText'],
431 431
 	];
432 432
 	/**
433 433
 	 * Php variables.
Please login to merge, or discard this patch.
app/RecordCollectors/Helper/GusClient.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -195,7 +195,7 @@
 block discarded – undo
195 195
 		stream_context_set_option($this->streamContext, ['http' => ['header' => 'sid: ' . $this->sessionId]]);
196 196
 		$this->__setSoapHeaders();
197 197
 		$this->__setSoapHeaders($header);
198
-		$result = $this->DaneSzukajPodmioty(['pParametryWyszukiwania' => ['Nip' => $vatId,  'Krs' => $ncr, 'Regon' => $taxNumber]]);
198
+		$result = $this->DaneSzukajPodmioty(['pParametryWyszukiwania' => ['Nip' => $vatId, 'Krs' => $ncr, 'Regon' => $taxNumber]]);
199 199
 		$response = $this->parseResponse($result->DaneSzukajPodmiotyResult);
200 200
 		foreach ($response as &$info) {
201 201
 			$this->getAdvanceData($info);
Please login to merge, or discard this patch.
modules/Calendar/ConfigTemplate.php 1 patch
Spacing   +5 added lines, -5 removed lines patch added patch discarded remove patch
@@ -33,7 +33,7 @@  discard block
 block discarded – undo
33 33
 	'DASHBOARD_CALENDAR_WIDGET_FILTER_TYPE' => [
34 34
 		'default' => 'list',
35 35
 		'description' => 'Shows the switch button or filter list in the calendar widget: switch - Switch "To realize" and "History", list - filter list',
36
-		'validation' => function () {
36
+		'validation' => function() {
37 37
 			$arg = func_get_arg(0);
38 38
 			return 'list' === $arg || 'switch' === $arg;
39 39
 		}
@@ -41,7 +41,7 @@  discard block
 block discarded – undo
41 41
 	'SHOW_QUICK_CREATE_BY_STATUS' => [
42 42
 		'default' => [],
43 43
 		'description' => 'Show the Calendar quick create window after changing the status: array - PLL_COMPLETED, PLL_CANCELLED',
44
-		'validation' => function () {
44
+		'validation' => function() {
45 45
 			$arg = func_get_arg(0);
46 46
 			return \is_array($arg) && empty(array_diff($arg, ['PLL_COMPLETED', 'PLL_CANCELLED']));
47 47
 		}
@@ -67,7 +67,7 @@  discard block
 block discarded – undo
67 67
 	'HIDDEN_DAYS_IN_CALENDAR_VIEW' => [
68 68
 		'default' => [0, 6],
69 69
 		'description' => 'Exclude certain days-of-the-week from being displayed. The value is an array of day-of-week indices to hide. Each index is zero-base (Sunday=0) and ranges from 0-6. By default, no days are hidden',
70
-		'validation' => function () {
70
+		'validation' => function() {
71 71
 			$arg = func_get_arg(0);
72 72
 			return 'Extended' === $arg || 'Standard' === $arg;
73 73
 		}
@@ -94,7 +94,7 @@  discard block
 block discarded – undo
94 94
 		'default' => 5000,
95 95
 		'description' => 'Max number of records to update status in cron',
96 96
 		'validation' => '\App\Validator::naturalNumber',
97
-		'sanitization' => function () {
97
+		'sanitization' => function() {
98 98
 			return (int) func_get_arg(0);
99 99
 		}
100 100
 	],
@@ -107,7 +107,7 @@  discard block
 block discarded – undo
107 107
 	'CALENDAR_VIEW' => [
108 108
 		'default' => 'Extended',
109 109
 		'description' => 'Calendar view - allowed values: Extended, Standard, refresh menu files after you change this value',
110
-		'validation' => function () {
110
+		'validation' => function() {
111 111
 			$arg = func_get_arg(0);
112 112
 			return 'Extended' === $arg || 'Standard' === $arg;
113 113
 		}
Please login to merge, or discard this patch.
modules/com_vtiger_workflow/tasks/VTEmailTemplateTask.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -46,7 +46,7 @@
 block discarded – undo
46 46
 				$mailerContent['to'][] = $this->address_emails;
47 47
 			}
48 48
 			if ($this->relations_email && '-' !== $this->relations_email) {
49
-				[$relatedModule,$relatedFieldName,$onlyFirst] = array_pad(explode('::', $this->relations_email), 3, false);
49
+				[$relatedModule, $relatedFieldName, $onlyFirst] = array_pad(explode('::', $this->relations_email), 3, false);
50 50
 				$relationListView = Vtiger_RelationListView_Model::getInstance($recordModel, $relatedModule);
51 51
 				$relationListView->setFields(['id', $relatedFieldName]);
52 52
 				$relationListView->set('search_key', $relatedFieldName);
Please login to merge, or discard this patch.
tests/Settings/Calendar.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -46,7 +46,7 @@
 block discarded – undo
46 46
 	 */
47 47
 	public function testGetPicklistValue()
48 48
 	{
49
-		\App\Db::getInstance()->createCommand()->insert('vtiger_activitytype', ['activitytype' => 'UnitTestCalendar', 'presence' => 1,  'sortorderid' => 99, 'color' => 'A0B584'])->execute();
49
+		\App\Db::getInstance()->createCommand()->insert('vtiger_activitytype', ['activitytype' => 'UnitTestCalendar', 'presence' => 1, 'sortorderid' => 99, 'color' => 'A0B584'])->execute();
50 50
 		\App\Cache::clear();
51 51
 		$this->assertGreaterThan(0, (\count(\Settings_Calendar_Module_Model::getPicklistValue())), 'Calendar activity type picklist is empty');
52 52
 	}
Please login to merge, or discard this patch.