Passed
Pull Request — developer (#16159)
by Arkadiusz
16:31
created
modules/Settings/LayoutEditor/views/Index.php 1 patch
Spacing   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -49,13 +49,13 @@
 block discarded – undo
49 49
 			$fieldName = $fieldModel->getName();
50 50
 			$lastItem = strrchr($fieldName, '_');
51 51
 			$firstItem = '';
52
-			if($lastItem === '_extra'){
52
+			if ($lastItem === '_extra') {
53 53
 				$firstItem = str_replace($lastItem, '', $fieldName);
54 54
 			}
55
-			if((!empty($firstItem) && !empty($firstItemModuleModal = \Vtiger_Field_Model::getInstance($firstItem, \Vtiger_Module_Model::getInstance($sourceModule))) && $firstItemModuleModal->isActiveField() && $fieldModel->isActiveField()) && ($firstItemModuleModal->getUIType() == 11) && ($fieldModel->getUIType() == 1)){
55
+			if ((!empty($firstItem) && !empty($firstItemModuleModal = \Vtiger_Field_Model::getInstance($firstItem, \Vtiger_Module_Model::getInstance($sourceModule))) && $firstItemModuleModal->isActiveField() && $fieldModel->isActiveField()) && ($firstItemModuleModal->getUIType() == 11) && ($fieldModel->getUIType() == 1)) {
56 56
 				unset($fieldName);
57 57
 			}
58
-			if(isset($fieldName)){
58
+			if (isset($fieldName)) {
59 59
 				$blockIdFieldMap[$fieldModel->getBlockId()][$fieldName] = $fieldModel;
60 60
 			}
61 61
 			if (!$fieldModel->isActiveField()) {
Please login to merge, or discard this patch.
app/Mailer.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -38,7 +38,7 @@  discard block
 block discarded – undo
38 38
 		$this->mailer = new \PHPMailer\PHPMailer\PHPMailer(false);
39 39
 		if (\App\Config::debug('MAILER_DEBUG')) {
40 40
 			$this->mailer->SMTPDebug = 2;
41
-			$this->mailer->Debugoutput = function ($str, $level) {
41
+			$this->mailer->Debugoutput = function($str, $level) {
42 42
 				if (false !== stripos($str, 'error') || false !== stripos($str, 'failed')) {
43 43
 					static::$error[] = $str;
44 44
 					Log::error(trim($str), 'Mailer');
@@ -452,7 +452,7 @@  discard block
 block discarded – undo
452 452
 	{
453 453
 		$this->mailer->SMTPDebug = 2;
454 454
 		static::$error = [];
455
-		$this->mailer->Debugoutput = function ($str, $level) {
455
+		$this->mailer->Debugoutput = function($str, $level) {
456 456
 			if (false !== strpos(strtolower($str), 'error') || false !== strpos(strtolower($str), 'failed')) {
457 457
 				static::$error[] = trim($str);
458 458
 				Log::error(trim($str), 'Mailer');
Please login to merge, or discard this patch.
app/Integrations/Magento/Synchronizer/Order.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -79,7 +79,7 @@
 block discarded – undo
79 79
 								}
80 80
 								$dataCrm['accountid'] = $dataCrm['parent_id'];
81 81
 								$dataCrm['parent_id'] = $parentOrder;
82
-								unset($dataCrm['birthday'],$dataCrm['leadsource'],$dataCrm['mobile'],$dataCrm['mobile_extra'],$dataCrm['phone'],$dataCrm['phone_extra'],$dataCrm['salutationtype']);
82
+								unset($dataCrm['birthday'], $dataCrm['leadsource'], $dataCrm['mobile'], $dataCrm['mobile_extra'], $dataCrm['phone'], $dataCrm['phone_extra'], $dataCrm['salutationtype']);
83 83
 								$dataCrm['magento_id'] = $order['entity_id'];
84 84
 								$mapModel->setDataCrm($dataCrm);
85 85
 								$crmId = $this->createOrderInCrm($mapModel);
Please login to merge, or discard this patch.
modules/HelpDesk/ConfigTemplate.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -14,7 +14,7 @@
 block discarded – undo
14 14
 		'default' => 50,
15 15
 		'description' => 'Max depth of hierarchy',
16 16
 		'validation' => '\App\Validator::naturalNumber',
17
-		'sanitization' => function () {
17
+		'sanitization' => function() {
18 18
 			return (int) func_get_arg(0);
19 19
 		}
20 20
 	],
Please login to merge, or discard this patch.
modules/Settings/CurrencyUpdate/bankmodels/ECB.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -118,7 +118,7 @@
 block discarded – undo
118 118
 			if ($time['time'] == $dateParam) {
119 119
 				$num = \count($time->Cube);
120 120
 				for ($i = 0; $i < $num; ++$i) {
121
-					$currency = (string) $time->Cube[$i]['currency'];   // currency code
121
+					$currency = (string) $time->Cube[$i]['currency']; // currency code
122 122
 					foreach ($otherCurrencyCode as $key => $currId) {
123 123
 						if ($key == $currency && $currency != $mainCurrency) {
124 124
 							$exchange = $time->Cube[$i]['rate'];
Please login to merge, or discard this patch.
modules/Settings/CurrencyUpdate/bankmodels/NBR.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -116,7 +116,7 @@
 block discarded – undo
116 116
 			if ($time['date'] == $dateParam) {
117 117
 				$num = \count($time->Rate);
118 118
 				for ($i = 0; $i < $num; ++$i) {
119
-					$currency = (string) $time->Rate[$i]['currency'];   // currency code
119
+					$currency = (string) $time->Rate[$i]['currency']; // currency code
120 120
 					foreach ($otherCurrencyCode as $key => $currId) {
121 121
 						if ($key == $currency && $currency != $mainCurrency) {
122 122
 							$exchange = $time->Rate[$i];
Please login to merge, or discard this patch.
modules/Settings/Roles/views/DeleteAjax.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -20,7 +20,7 @@
 block discarded – undo
20 20
 		$baseParentRole = $recordModel->get('parentrole') . '::';
21 21
 		$allRoles = Settings_Roles_Record_Model::getAll();
22 22
 		unset($allRoles[$recordId]);
23
-		$allRoles = array_filter($allRoles, function (Settings_Roles_Record_Model $items) use ($baseParentRole) {
23
+		$allRoles = array_filter($allRoles, function(Settings_Roles_Record_Model $items) use ($baseParentRole) {
24 24
 			return false === strpos($items->get('parentrole'), $baseParentRole);
25 25
 		});
26 26
 		$viewer->assign('MODULE', $moduleName);
Please login to merge, or discard this patch.
app/Integrations/Magento/Synchronizer/Invoice.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -74,7 +74,7 @@
 block discarded – undo
74 74
 									$customerDataCrm = $customerMapModel->getDataCrm();
75 75
 									$dataCrm['accountid'] = $this->syncAccount($customerDataCrm);
76 76
 								}
77
-								unset($dataCrm['birthday'],$dataCrm['leadsource'],$dataCrm['mobile'],$dataCrm['mobile_extra'],$dataCrm['phone'],$dataCrm['phone_extra'],$dataCrm['salutationtype']);
77
+								unset($dataCrm['birthday'], $dataCrm['leadsource'], $dataCrm['mobile'], $dataCrm['mobile_extra'], $dataCrm['phone'], $dataCrm['phone_extra'], $dataCrm['salutationtype']);
78 78
 								$mapModel->setDataCrm($dataCrm);
79 79
 								$crmId = $this->createInvoiceInCrm($mapModel);
80 80
 								\App\Cache::staticSave('CrmIdByMagentoIdSSingleOrders', $invoice['entity_id'], $crmId);
Please login to merge, or discard this patch.
app/Installer/Developer.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -23,7 +23,7 @@  discard block
 block discarded – undo
23 23
 	 */
24 24
 	public static function generateSwagger(): string
25 25
 	{
26
-		set_error_handler(function ($errNo, $errStr, $errFile, $errLine) {
26
+		set_error_handler(function($errNo, $errStr, $errFile, $errLine) {
27 27
 			$errorString = \App\ErrorHandler::error2string($errNo);
28 28
 			$msg = reset($errorString) . ": {$errStr}\nFile: {$errFile}\nLine: $errLine" . PHP_EOL;
29 29
 			echo "<pre>$msg</pre><hr>";
@@ -46,7 +46,7 @@  discard block
 block discarded – undo
46 46
 	public static function generateSwaggerByType(string $type, $errorHandler = true): string
47 47
 	{
48 48
 		if ($errorHandler) {
49
-			set_error_handler(function ($errNo, $errStr, $errFile, $errLine) {
49
+			set_error_handler(function($errNo, $errStr, $errFile, $errLine) {
50 50
 				$errorString = \App\ErrorHandler::error2string($errNo);
51 51
 				$msg = reset($errorString) . ": {$errStr}\nFile: {$errFile}\nLine: $errLine" . PHP_EOL;
52 52
 				echo "<pre>$msg</pre><hr>";
Please login to merge, or discard this patch.