Passed
Push — master ( 6c88a8...382506 )
by Aimeos
21:30 queued 07:37
created
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.
src/Controller/Common/Product/Import/Csv/Processor/Media/Standard.php 1 patch
Braces   +2 added lines, -4 removed lines patch added patch discarded remove patch
@@ -81,8 +81,7 @@  discard block
 block discarded – undo
81 81
 			foreach( $manager->search( $search ) as $item ) {
82 82
 				$this->listTypes[$item->getCode()] = $item->getCode();
83 83
 			}
84
-		}
85
-		else
84
+		} else
86 85
 		{
87 86
 			$this->listTypes = array_combine( $this->listTypes, $this->listTypes );
88 87
 		}
@@ -151,8 +150,7 @@  discard block
 block discarded – undo
151 150
 					$listItem = $listMap[$url][$type][$langId][$listtype];
152 151
 					$refItem = $listItem->getRefItem();
153 152
 					unset( $listItems[$listItem->getId()] );
154
-				}
155
-				else
153
+				} else
156 154
 				{
157 155
 					$listItem = $manager->createListItem()->setType( $listtype );
158 156
 					$refItem = $refManager->create()->setType( $type );
Please login to merge, or discard this patch.
src/Controller/Common/Catalog/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
 		}
@@ -137,8 +136,7 @@  discard block
 block discarded – undo
137 136
 				$listItem = $listMap[$content][$type][$listtype];
138 137
 				$refItem = $listItem->getRefItem();
139 138
 				unset( $listItems[$listItem->getId()] );
140
-			}
141
-			else
139
+			} else
142 140
 			{
143 141
 				$listItem = $manager->createListItem()->setType( $listtype );
144 142
 				$refItem = $refManager->create()->setType( $type );
Please login to merge, or discard this patch.