YetiForceCompany / YetiForceCRM

app/Db/Importer.php

Braces +1 added lines, -1 removed lines

@@ -680,7 +680,7 @@
 							$this->logs .= "done    ({$time}s)\n";
 							$tableSchema = $schema->getTableSchema($tableName, true);
 							$columnExists = isset($tableSchema->columns[$columnName]);
-						}elseif (!$columnExists && 1 !== $mode) {
+						} elseif (!$columnExists && 1 !== $mode) {
 							$this->logs .= "  > add column: $tableName:$columnName ... ";
 							$start = microtime(true);
 							$dbCommand->addColumn($tableName, $columnName, $column)->execute();

app/Installer/Developer.php

Spacing +2 added lines, -2 removed lines

@@ -29,7 +29,7 @@ 
 	 */
 	public static function generateSwagger(string $path = self::PATH): string
 	{
-		set_error_handler(function ($errNo, $errStr, $errFile, $errLine) {
+		set_error_handler(function($errNo, $errStr, $errFile, $errLine) {
 			$errorString = \App\ErrorHandler::error2string($errNo);
 			$msg = reset($errorString) . ": {$errStr}\nFile: {$errFile}\nLine: $errLine" . PHP_EOL;
 			echo "<pre>$msg</pre><hr>";
@@ -53,7 +53,7 @@ 
 	public static function generateSwaggerByType(string $type, string $path = self::PATH, $errorHandler = true): string
 	{
 		if ($errorHandler) {
-			set_error_handler(function ($errNo, $errStr, $errFile, $errLine) {
+			set_error_handler(function($errNo, $errStr, $errFile, $errLine) {
 				$errorString = \App\ErrorHandler::error2string($errNo);
 				$msg = reset($errorString) . ": {$errStr}\nFile: {$errFile}\nLine: $errLine" . PHP_EOL;
 				echo "<pre>$msg</pre><hr>";

modules/OSSMailScanner/models/Record.php

Spacing +1 added lines, -1 removed lines

@@ -339,7 +339,7 @@
 				$actions = $account['actions'];
 			}
 			if (!\in_array('CreatedEmail', $actions)) {
-				array_unshift($actions, 'CreatedEmail', );
+				array_unshift($actions, 'CreatedEmail',);
 			}
 			$params['actions'] = $actions;
 		}

modules/Calendar/ConfigTemplate.php

Spacing +4 added lines, -4 removed lines

@@ -33,7 +33,7 @@ 
 	'DASHBOARD_CALENDAR_WIDGET_FILTER_TYPE' => [
 		'default' => 'list',
 		'description' => 'Shows the switch button or filter list in the calendar widget: switch - Switch "To realize" and "History", list - filter list',
-		'validation' => function () {
+		'validation' => function() {
 			$arg = func_get_arg(0);
 			return 'list' === $arg || 'switch' === $arg;
 		},
@@ -41,7 +41,7 @@ 
 	'SHOW_QUICK_CREATE_BY_STATUS' => [
 		'default' => [],
 		'description' => 'Show the Calendar quick create window after changing the status: array - PLL_COMPLETED, PLL_CANCELLED',
-		'validation' => function () {
+		'validation' => function() {
 			$arg = func_get_arg(0);
 			return \is_array($arg) && empty(array_diff($arg, ['PLL_COMPLETED', 'PLL_CANCELLED']));
 		},
@@ -67,7 +67,7 @@ 
 	'HIDDEN_DAYS_IN_CALENDAR_VIEW' => [
 		'default' => [0, 6],
 		'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',
-		'validation' => function () {
+		'validation' => function() {
 			$arg = func_get_arg(0);
 			return 'Extended' === $arg || 'Standard' === $arg;
 		},
@@ -105,7 +105,7 @@ 
 	'CALENDAR_VIEW' => [
 		'default' => 'Extended',
 		'description' => 'Calendar view - allowed values: Extended, Standard, refresh menu files after you change this value',
-		'validation' => function () {
+		'validation' => function() {
 			$arg = func_get_arg(0);
 			return 'Extended' === $arg || 'Standard' === $arg;
 		},

api/webservice/Token/Auth/Basic.php

Spacing +1 added lines, -1 removed lines

@@ -21,7 +21,7 @@
 	{
 		$this->api->app = [];
 		$type = $this->api->request->getByType('_container', \App\Purifier::STANDARD);
-		$query = (new \App\Db\Query())->from('w_#__servers')->where(['type' => $type,  'status' => 1]);
+		$query = (new \App\Db\Query())->from('w_#__servers')->where(['type' => $type, 'status' => 1]);
 		if ($row = $query->one()) {
 			$row['id'] = (int) $row['id'];
 			$this->api->app = $row;

api/webservice/PBX/Auth/Basic.php

Spacing +1 added lines, -1 removed lines

@@ -21,7 +21,7 @@
 	{
 		$this->api->app = [];
 		$type = $this->api->request->getByType('_container', \App\Purifier::STANDARD);
-		$query = (new \App\Db\Query())->from('w_#__servers')->where(['type' => $type,  'status' => 1]);
+		$query = (new \App\Db\Query())->from('w_#__servers')->where(['type' => $type, 'status' => 1]);
 		if ($row = $query->one()) {
 			$row['id'] = (int) $row['id'];
 			$this->api->app = $row;

config/ConfigTemplates.php

Spacing +22 added lines, -22 removed lines

@@ -10,7 +10,7 @@ 
 		'enabledServices' => [
 			'default' => [],
 			'description' => 'List of active services. Available: dav, webservice',
-			'validation' => function () {
+			'validation' => function() {
 				$arg = func_get_arg(0);
 				return \is_array($arg) && empty(array_diff($arg, ['dav', 'webservice']));
 			},
@@ -100,7 +100,7 @@ 
 		'upload_maxsize' => [
 			'default' => 52428800,
 			'description' => 'Maximum file size for uploaded files in bytes also used when uploading import files: upload_maxsize default value = 52428800 (50MB)',
-			'validation' => function () {
+			'validation' => function() {
 				$arg = func_get_arg(0);
 				return $arg && \App\Validator::naturalNumber($arg) && ($arg * 1048576) <= \App\Config::getMaxUploadSize(false);
 			},
@@ -109,7 +109,7 @@ 
 		'allow_exports' => [
 			'default' => 'all',
 			'description' => 'Flag to allow export functionality: "all" - to allow anyone to use exports, "admin" - to only allow admins to export, "none" -  to block exports completely',
-			'validation' => function () {
+			'validation' => function() {
 				$arg = func_get_arg(0);
 				return \in_array($arg, ['all', 'admin', 'none']);
 			},
@@ -121,7 +121,7 @@ 
 		'list_max_entries_per_page' => [
 			'default' => 20,
 			'description' => 'List max entries per page: default value = 20',
-			'validation' => function () {
+			'validation' => function() {
 				$arg = func_get_arg(0);
 				return $arg && \App\Validator::naturalNumber($arg) && (100 >= $arg) && (0 < $arg);
 			},
@@ -130,7 +130,7 @@ 
 		'default_module' => [
 			'default' => 'Home',
 			'description' => 'Default module: default value = Home',
-			'validation' => function () {
+			'validation' => function() {
 				$arg = func_get_arg(0);
 				return true === \App\Module::isModuleActive($arg);
 			},
@@ -154,7 +154,7 @@ 
 		'listview_max_textlength' => [
 			'default' => 40,
 			'description' => 'Trim descriptions, titles in listviews to this value',
-			'validation' => function () {
+			'validation' => function() {
 				$arg = func_get_arg(0);
 				return $arg && \App\Validator::naturalNumber($arg) && (100 >= $arg) && (0 < $arg);
 			},
@@ -167,7 +167,7 @@ 
 		'default_timezone' => [
 			'default' => '_TIMEZONE_',
 			'description' => 'Set the default timezone as per your preference',
-			'validation' => function () {
+			'validation' => function() {
 				$arg = func_get_arg(0);
 				return \in_array($arg, timezone_identifiers_list());
 			},
@@ -175,7 +175,7 @@ 
 		'title_max_length' => [
 			'default' => 60,
 			'description' => 'Maximum length of characters for title',
-			'validation' => function () {
+			'validation' => function() {
 				$arg = func_get_arg(0);
 				return $arg && \App\Validator::naturalNumber($arg) && (100 >= $arg) && (0 < $arg);
 			},
@@ -184,7 +184,7 @@ 
 		'href_max_length' => [
 			'default' => 50,
 			'description' => 'Maximum length for href tag',
-			'validation' => function () {
+			'validation' => function() {
 				$arg = func_get_arg(0);
 				return $arg && \App\Validator::naturalNumber($arg) && (100 >= $arg) && (0 < $arg);
 			},
@@ -193,7 +193,7 @@ 
 		'MINIMUM_CRON_FREQUENCY' => [
 			'default' => 1,
 			'description' => 'Minimum cron frequency [min]',
-			'validation' => function () {
+			'validation' => function() {
 				$arg = func_get_arg(0);
 				return $arg && \App\Validator::naturalNumber($arg) && (100 >= $arg) && (0 < $arg);
 			},
@@ -211,7 +211,7 @@ 
 		'listMaxEntriesMassEdit' => [
 			'default' => 500,
 			'description' => 'Maximum number of records in a mass edition',
-			'validation' => function () {
+			'validation' => function() {
 				$arg = func_get_arg(0);
 				return $arg && \App\Validator::naturalNumber($arg) && (5000 >= $arg);
 			},
@@ -307,7 +307,7 @@ 
 		'LOG_LEVELS' => [
 			'default' => false,
 			'description' => 'Level of saved/displayed logs. Values: false = All / 3 = error and warning / ["error", "warning", "info", "trace", "profile"]',
-			'validation' => function () {
+			'validation' => function() {
 				$arg = func_get_arg(0);
 				return false === $arg || (\is_array($arg) && array_diff(['error', 'warning', 'info', 'trace', 'profile'], $arg));
 			},
@@ -388,7 +388,7 @@ 
 		'SMARTY_ERROR_REPORTING' => [
 			'default' => new \Nette\PhpGenerator\PhpLiteral('E_ALL & ~E_NOTICE'),
 			'description' => 'Do not show Smarty Notice in phpError.log',
-			'validation' => function () {
+			'validation' => function() {
 				$arg = (string) func_get_arg(0);
 				return \in_array($arg, ['E_ALL', 'E_ALL & ~E_NOTICE']);
 			},
@@ -396,7 +396,7 @@ 
 		'EXCEPTION_ERROR_LEVEL' => [
 			'default' => new \Nette\PhpGenerator\PhpLiteral('E_ALL & ~E_NOTICE'),
 			'description' => "Set the error reporting level. The parameter is either an integer representing a bit field, or named constants.\nhttps://secure.php.net/manual/en/errorfunc.configuration.php#ini.error-reporting\nAll errors - E_ALL & ~E_NOTICE & ~E_STRICT & ~E_DEPRECATED / Critical errors - E_ERROR | E_WARNING | E_CORE_ERROR | E_COMPILE_ERROR | E_USER_ERROR",
-			'validation' => function () {
+			'validation' => function() {
 				$arg = (string) func_get_arg(0);
 				return \in_array($arg, ['E_ALL', 'E_ALL & ~E_NOTICE']);
 			},
@@ -568,7 +568,7 @@ 
 		'CACHING_DRIVER' => [
 			'default' => 'Base',
 			'description' => 'Data caching is about storing some PHP variables in cache and retrieving it later from cache. Drivers: Base, Apcu',
-			'validation' => function () {
+			'validation' => function() {
 				$arg = func_get_arg(0);
 				return 'Basic' === $arg || 'Apcu' === $arg;
 			},
@@ -766,7 +766,7 @@ 
 		'MODULES_SPLITTED_EDIT_VIEW_LAYOUT' => [
 			'default' => [],
 			'description' => 'List of modules with splitted edit view layout',
-			'validation' => function () {
+			'validation' => function() {
 				$arg = func_get_arg(0);
 				return \is_array($arg) && array_diff($arg, App\Module::getAllModuleNames());
 			},
@@ -871,7 +871,7 @@ 
 		'GLOBAL_SEARCH_SORTING_RESULTS' => [
 			'default' => 0,
 			'description' => 'Global search - Should the results be sorted in MySQL or PHP while displaying (None = 0, PHP = 1, Mysql = 2). The parameter impacts system efficiency.',
-			'validation' => function () {
+			'validation' => function() {
 				$arg = func_get_arg(0);
 				return \is_int($arg) && \in_array($arg, [0, 1, 2]);
 			},
@@ -885,7 +885,7 @@ 
 		'GLOBAL_SEARCH_AUTOCOMPLETE' => [
 			'default' => 1,
 			'description' => 'Global search - Search for records while entering text  (1/0).',
-			'validation' => function () {
+			'validation' => function() {
 				$arg = func_get_arg(0);
 				return \is_int($arg) && \in_array($arg, [0, 1]);
 			},
@@ -924,7 +924,7 @@ 
 		'encryptionMethod' => [
 			'default' => 'aes-256-cbc',
 			'description' => 'Encryption method.',
-			'validation' => function () {
+			'validation' => function() {
 				$arg = func_get_arg(0);
 				return empty($arg) || ($arg && \in_array($arg, \App\Encryption::getMethods()));
 			},
@@ -992,7 +992,7 @@ 
 		'permittedModulesByCreatorField' => [
 			'default' => [],
 			'description' => 'List of modules to which access is based on the record creation.',
-			'validation' => function () {
+			'validation' => function() {
 				$arg = func_get_arg(0);
 				return \is_array($arg) && array_diff($arg, App\Module::getAllModuleNames());
 			},
@@ -1063,7 +1063,7 @@ 
 		'cookieForceHttpOnly' => [
 			'default' => true,
 			'description' => "Force the use of https only for cookie.\nValues: true, false, null",
-			'validation' => function () {
+			'validation' => function() {
 				$arg = func_get_arg(0);
 				return null === $arg ? $arg : \is_bool($arg);
 			},
@@ -1081,7 +1081,7 @@ 
 		'USER_AUTHY_MODE' => [
 			'default' => 'TOTP_OPTIONAL',
 			'description' => "User authentication mode.\n\n@see \\Users_Totp_Authmethod::ALLOWED_USER_AUTHY_MODE\nAvailable values:\nTOTP_OFF - 2FA TOTP is checking off\nTOTP_OPTIONAL - It is defined by the user\nTOTP_OBLIGATORY - It is obligatory.",
-			'validation' => function () {
+			'validation' => function() {
 				$arg = func_get_arg(0);
 				return \in_array($arg, \Users_Totp_Authmethod::ALLOWED_USER_AUTHY_MODE);
 			},

api/webservice/WebserviceStandard/Save.php

Spacing +1 added lines, -1 removed lines

@@ -49,7 +49,7 @@
 		$fieldModelList = $this->record->getModule()->getFields();
 		$viewName = $this->record->isNew() ? 'Create' : 'Edit';
 		$requestKeys = $request->getAllRaw();
-		unset($requestKeys['module'],$requestKeys['action'],$requestKeys['record']);
+		unset($requestKeys['module'], $requestKeys['action'], $requestKeys['record']);
 		if (empty($requestKeys)) {
 			throw new \Api\Core\Exception('No input data', 406);
 		}

tests/App/Purifier.php

Spacing +4 added lines, -4 removed lines

@@ -131,10 +131,10 @@
 			['Text', 'NotSame', 'ę€ółśążźćń23{}":?>><>?:"{}+_)(*&^%$#@!) &lt;svg/onabort=alert(3)//  <svg/onload=alert(1) onfocus=alert(2)//', 'ę€ółśążźćń23{}":?>><>?:"{}+_)(*&^%$#@!) &lt;svg/onabort=alert(3)//  <svg/onload=alert(1) onfocus=alert(2)//', 'Sample number should be purified', \App\Exceptions\IllegalValue::class],
 			['Default', 'Same', 'Test-text-string-for-purifier', 'Test-text-string-for-purifier', 'Sample number should be unchanged', null],
 			['Default', 'NotSame', 'ę€ółśążźćń23{}":?>><>?:"{}+_)(*&^%$#@!) &lt;svg/onabort=alert(3)//  <svg/onload=alert(1) onfocus=alert(2)//', 'ę€ółśążźćń23{}":?>><>?:"{}+_)(*&^%$#@!) &lt;svg/onabort=alert(3)//  <svg/onload=alert(1) onfocus=alert(2)//', 'Sample number should be purified', \App\Exceptions\IllegalValue::class],
-			['MailId', 'Same',  '5FB2B5EF@xx.cc.it> (added by postmaster@cc.it)', '<5FB2B5EF@xx.cc.it> (added by postmaster@cc.it)', 'Sample text should be unchanged', null],
-			['MailId', 'Same',  '30.123.12.JavaMail."admin.azure"@A-PROXY01', '<30.123.12.JavaMail."admin.azure"@A-PROXY01>', 'Sample text should be unchanged', null],
-			['MailId', 'Same',  'CAK01GN-UtTiM90_wQNB07OnE6aBm=w@mail.g.c', '<CAK01GN-UtTiM90_wQNB07OnE6aBm=w@mail.g.c>', 'Sample text should be unchanged', null],
-			['MailId', 'Same',  'AM9F9@AM9PR.eurprd01.prod.exchangelabs.com', ' <AM9F9@AM9PR.eurprd01.prod.exchangelabs.com', 'Sample text should be unchanged', null],
+			['MailId', 'Same', '5FB2B5EF@xx.cc.it> (added by postmaster@cc.it)', '<5FB2B5EF@xx.cc.it> (added by postmaster@cc.it)', 'Sample text should be unchanged', null],
+			['MailId', 'Same', '30.123.12.JavaMail."admin.azure"@A-PROXY01', '<30.123.12.JavaMail."admin.azure"@A-PROXY01>', 'Sample text should be unchanged', null],
+			['MailId', 'Same', 'CAK01GN-UtTiM90_wQNB07OnE6aBm=w@mail.g.c', '<CAK01GN-UtTiM90_wQNB07OnE6aBm=w@mail.g.c>', 'Sample text should be unchanged', null],
+			['MailId', 'Same', 'AM9F9@AM9PR.eurprd01.prod.exchangelabs.com', ' <AM9F9@AM9PR.eurprd01.prod.exchangelabs.com', 'Sample text should be unchanged', null],
 			[\App\Purifier::PATH, 'NotSame', '../Test', '../Test', 'Path should be discarded', \App\Exceptions\IllegalValue::class],
 			[\App\Purifier::PATH, 'Same', '/Test/test', '/Test/test', 'Path should be unchanged', null],
 		];