Passed
Pull Request — master (#47)
by
unknown
03:38
created
templates/order/email/payment/pdf.php 1 patch
Spacing   +8 added lines, -8 removed lines patch added patch discarded remove patch
@@ -20,7 +20,7 @@  discard block
 block discarded – undo
20 20
 $this->pdf->setFont( 'dejavusans', '', 10 );
21 21
 
22 22
 $vmargin = [
23
-	'h1' => [ // HTML tag
23
+	'h1' => [// HTML tag
24 24
 		0 => ['h' => 1.5, 'n' => 0], // space before = h * n
25 25
 		1 => ['h' => 1.5, 'n' => 3] // space after = h * n
26 26
 	],
@@ -93,14 +93,14 @@  discard block
 block discarded – undo
93 93
 $data = [
94 94
 	'BCD', // required
95 95
 	'002', // Version 2 (required, 1=UTF-8, 2=ISO 8859-1, 3=ISO 8859-2, 4=ISO 8859-4, 5=ISO 8859-5, 6=ISO 8859-7, 7=ISO 8859-10, 8=ISO 8859-15)
96
-	1,     // UTF-8 (required)
96
+	1, // UTF-8 (required)
97 97
 	'SCT', // SEPA Credit Transfer (required)
98
-	'',    // BIC (optional)
99
-	'',    // Name of recipient (required, name of your company)
100
-	'',    // IBAN (required)
101
-	$this->summaryBasket->getPrice()->getCurrencyId() . $total,    // Currency and value (required)
102
-	'',    // Purpose (optional, 4 char code, https://wiki.windata.de/index.php?title=Purpose-SEPA-Codes)
103
-	'',    // ISO 11649 RF Creditor Reference (optional, 35 characters structured code)
98
+	'', // BIC (optional)
99
+	'', // Name of recipient (required, name of your company)
100
+	'', // IBAN (required)
101
+	$this->summaryBasket->getPrice()->getCurrencyId() . $total, // Currency and value (required)
102
+	'', // Purpose (optional, 4 char code, https://wiki.windata.de/index.php?title=Purpose-SEPA-Codes)
103
+	'', // ISO 11649 RF Creditor Reference (optional, 35 characters structured code)
104 104
 	$this->translate( 'controller/jobs', 'Invoice' ) . ' ' . $this->orderItem->getInvoiceNumber(), // Reference of order and other data (optional, max. 140 characters)
105 105
 	$this->summaryBasket->getCustomerReference(), // Notice to the customer (optional, max. 70 characters)
106 106
 ];
Please login to merge, or discard this patch.
templates/order/email/voucher/pdf.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -19,7 +19,7 @@
 block discarded – undo
19 19
 $this->pdf->setFont( 'dejavusans', '', 10 );
20 20
 
21 21
 $vmargin = [
22
-	'h1' => [ // HTML tag
22
+	'h1' => [// HTML tag
23 23
 		0 => ['h' => 1.5, 'n' => 0], // space before = h * n
24 24
 		1 => ['h' => 1.5, 'n' => 3] // space after = h * n
25 25
 	],
Please login to merge, or discard this patch.
tests/Controller/Jobs/Admin/Log/StandardTest.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -63,7 +63,7 @@
 block discarded – undo
63 63
 
64 64
 		$this->object->run();
65 65
 
66
-		$expected = dirname( __DIR__, 4) . '/tmp/logs/aimeos_' . date( 'Y-m-d' ) . '.log';
66
+		$expected = dirname( __DIR__, 4 ) . '/tmp/logs/aimeos_' . date( 'Y-m-d' ) . '.log';
67 67
 		$this->assertFileExists( $expected );
68 68
 
69 69
 		unlink( $expected );
Please login to merge, or discard this patch.
src/Controller/Jobs/Subscription/Process/Renew/Standard.php 1 patch
Braces   +11 added lines, -4 removed lines patch added patch discarded remove patch
@@ -265,9 +265,12 @@  discard block
 block discarded – undo
265 265
 		{
266 266
 			foreach( $codes as $code )
267 267
 			{
268
-				try {
268
+				try
269
+				{
269 270
 					$newOrder->addCoupon( $code );
270
-				} catch( \Aimeos\MShop\Plugin\Provider\Exception | \Aimeos\MShop\Coupon\Exception $e ) {
271
+				}
272
+				catch( \Aimeos\MShop\Plugin\Provider\Exception | \Aimeos\MShop\Coupon\Exception $e )
273
+				{
271 274
 					$newOrder->deleteCoupon( $code );
272 275
 				}
273 276
 			}
@@ -383,7 +386,9 @@  discard block
 block discarded – undo
383 386
 			$context->setUserId( $order->getCustomerId() );
384 387
 			$context->setGroupIds( $customerItem->getGroups() );
385 388
 		}
386
-		catch( \Exception $e ) {} // Subscription without account
389
+		catch( \Exception $e )
390
+		{
391
+} // Subscription without account
387 392
 
388 393
 		return $context;
389 394
 	}
@@ -569,9 +574,11 @@  discard block
 block discarded – undo
569 574
 		}
570 575
 		catch( \Exception $e )
571 576
 		{
572
-			if( $e->getCode() < 1 ) // not a soft error
577
+			if( $e->getCode() < 1 ) {
578
+				// not a soft error
573 579
 			{
574 580
 				$item->setReason( \Aimeos\MShop\Subscription\Item\Iface::REASON_PAYMENT );
581
+			}
575 582
 
576 583
 				if( $this->ends() ) {
577 584
 					$item->setDateEnd( date_create()->format( 'Y-m-d' ) );
Please login to merge, or discard this patch.
src/Controller/Jobs/Catalog/Import/Xml/Standard.php 1 patch
Braces   +5 added lines, -5 removed lines patch added patch discarded remove patch
@@ -327,7 +327,9 @@  discard block
 block discarded – undo
327 327
 
328 328
 				return $currentid;
329 329
 			}
330
-			catch( \Aimeos\MShop\Exception $e ) {} // not found, create new
330
+			catch( \Aimeos\MShop\Exception $e )
331
+			{
332
+} // not found, create new
331 333
 		}
332 334
 
333 335
 		$item = $this->process( $manager->create(), $node );
@@ -365,13 +367,11 @@  discard block
 block discarded – undo
365 367
 
366 368
 				$currentid = $this->importNode( $node, $domains, $parentid, $childMap );
367 369
 				$total++;
368
-			}
369
-			elseif( $xml->nodeType === \XMLReader::ELEMENT && $xml->name === 'catalog' )
370
+			} elseif( $xml->nodeType === \XMLReader::ELEMENT && $xml->name === 'catalog' )
370 371
 			{
371 372
 				$this->importTree( $xml, $domains, $currentid, $childMap );
372 373
 				$childMap = [];
373
-			}
374
-			elseif( $xml->nodeType === \XMLReader::END_ELEMENT && $xml->name === 'catalog' )
374
+			} elseif( $xml->nodeType === \XMLReader::END_ELEMENT && $xml->name === 'catalog' )
375 375
 			{
376 376
 				\Aimeos\MShop::create( $this->context(), 'catalog' )->delete( $map );
377 377
 				break;
Please login to merge, or discard this patch.
src/Controller/Jobs/Order/Email/Payment/Standard.php 1 patch
Braces   +12 added lines, -4 removed lines patch added patch discarded remove patch
@@ -362,10 +362,18 @@
 block discarded – undo
362 362
 			private ?\Closure $headerFcn = null;
363 363
 			private ?\Closure $footerFcn = null;
364 364
 
365
-			public function Footer() { return ( $fcn = $this->footerFcn ) ? $fcn( $this ) : null; }
366
-			public function Header() { return ( $fcn = $this->headerFcn ) ? $fcn( $this ) : null; }
367
-			public function setFooterFunction( \Closure $fcn ) { $this->footerFcn = $fcn; }
368
-			public function setHeaderFunction( \Closure $fcn ) { $this->headerFcn = $fcn; }
365
+			public function Footer()
366
+			{
367
+return ( $fcn = $this->footerFcn ) ? $fcn( $this ) : null; }
368
+			public function Header()
369
+			{
370
+return ( $fcn = $this->headerFcn ) ? $fcn( $this ) : null; }
371
+			public function setFooterFunction( \Closure $fcn )
372
+			{
373
+$this->footerFcn = $fcn; }
374
+			public function setHeaderFunction( \Closure $fcn )
375
+			{
376
+$this->headerFcn = $fcn; }
369 377
 		};
370 378
 		$pdf->setCreator( PDF_CREATOR );
371 379
 		$pdf->setAuthor( 'Aimeos' );
Please login to merge, or discard this patch.
src/Controller/Jobs/Order/Email/Voucher/Standard.php 1 patch
Braces   +12 added lines, -4 removed lines patch added patch discarded remove patch
@@ -362,10 +362,18 @@
 block discarded – undo
362 362
 			private ?\Closure $headerFcn = null;
363 363
 			private ?\Closure $footerFcn = null;
364 364
 
365
-			public function Footer() { return ( $fcn = $this->footerFcn ) ? $fcn( $this ) : null; }
366
-			public function Header() { return ( $fcn = $this->headerFcn ) ? $fcn( $this ) : null; }
367
-			public function setFooterFunction( \Closure $fcn ) { $this->footerFcn = $fcn; }
368
-			public function setHeaderFunction( \Closure $fcn ) { $this->headerFcn = $fcn; }
365
+			public function Footer()
366
+			{
367
+return ( $fcn = $this->footerFcn ) ? $fcn( $this ) : null; }
368
+			public function Header()
369
+			{
370
+return ( $fcn = $this->headerFcn ) ? $fcn( $this ) : null; }
371
+			public function setFooterFunction( \Closure $fcn )
372
+			{
373
+$this->footerFcn = $fcn; }
374
+			public function setHeaderFunction( \Closure $fcn )
375
+			{
376
+$this->headerFcn = $fcn; }
369 377
 		};
370 378
 		$pdf->setCreator( PDF_CREATOR );
371 379
 		$pdf->setAuthor( 'Aimeos' );
Please login to merge, or discard this patch.
src/Controller/Common/Product/Import/Csv/Processor/Text/Standard.php 1 patch
Braces   +2 added lines, -4 removed lines patch added patch discarded remove patch
@@ -78,8 +78,7 @@  discard block
 block discarded – undo
78 78
 			foreach( $manager->search( $search ) as $item ) {
79 79
 				$this->listTypes[$item->getCode()] = $item->getCode();
80 80
 			}
81
-		}
82
-		else
81
+		} else
83 82
 		{
84 83
 			$this->listTypes = array_combine( $this->listTypes, $this->listTypes );
85 84
 		}
@@ -141,8 +140,7 @@  discard block
 block discarded – undo
141 140
 				$listItem = $listMap[$content][$language][$type][$listtype];
142 141
 				$refItem = $listItem->getRefItem();
143 142
 				unset( $listItems[$listItem->getId()] );
144
-			}
145
-			else
143
+			} else
146 144
 			{
147 145
 				$listItem = $manager->createListItem()->setType( $listtype );
148 146
 				$refItem = $refManager->create()->setType( $type );
Please login to merge, or discard this patch.
src/Controller/Common/Product/Import/Csv/Processor/Price/Standard.php 1 patch
Braces   +1 added lines, -2 removed lines patch added patch discarded remove patch
@@ -78,8 +78,7 @@
 block discarded – undo
78 78
 			foreach( $manager->search( $search ) as $item ) {
79 79
 				$this->listTypes[$item->getCode()] = $item->getCode();
80 80
 			}
81
-		}
82
-		else
81
+		} else
83 82
 		{
84 83
 			$this->listTypes = array_combine( $this->listTypes, $this->listTypes );
85 84
 		}
Please login to merge, or discard this patch.