aimeos / ai-controller-jobs

controller/common/tests/Controller/Common/Supplier/Import/Csv/Processor/Address/StandardTest.php

Spacing +1 added lines, -1 removed lines

@@ -260,7 +260,7 @@
 
 	public function reloadSupplier( &$supplier )
 	{
-		$supplier = self::$supplierManager->getItem( $supplier->getId(), [ 'supplier/address' ] );
+		$supplier = self::$supplierManager->getItem( $supplier->getId(), ['supplier/address'] );
 	}
 
 }

controller/common/tests/Controller/Common/Product/Import/Csv/Processor/Supplier/StandardTest.php

Spacing +3 added lines, -3 removed lines

@@ -75,7 +75,7 @@ 
 			3 => 'job_csv_test2',
 		);
 
-		$suppliersCodes = [ 'job_csv_test', 'job_csv_test2' ];
+		$suppliersCodes = ['job_csv_test', 'job_csv_test2'];
 
 		foreach( $suppliersCodes as $code )
 		{
@@ -111,7 +111,7 @@ 
 			1 => "job_csv_test\njob_csv_test2",
 		);
 
-		$suppliersCodes = [ 'job_csv_test', 'job_csv_test2' ];
+		$suppliersCodes = ['job_csv_test', 'job_csv_test2'];
 
 		foreach( $suppliersCodes as $code )
 		{
@@ -325,7 +325,7 @@ 
 		$search = $manager->createSearch();
 		$search->setConditions( $search->compare( '==', 'supplier.code', $code ) );
 
-		if( ($item = $manager->searchItems( $search, [ 'product' ] )->first()) === null )
+		if( ( $item = $manager->searchItems( $search, ['product'] )->first() ) === null )
 		{
 			throw new \RuntimeException( sprintf( 'No supplier item for code "%1$s"', $code ) );
 		}

controller/common/src/Controller/Common/Supplier/Import/Csv/Processor/Media/Standard.php

Spacing +3 added lines, -3 removed lines

@@ -120,7 +120,7 @@ 
 
 		foreach( $listItems as $listItem )
 		{
-			if( ($refItem = $listItem->getRefItem()) !== null )
+			if( ( $refItem = $listItem->getRefItem() ) !== null )
 			{
 				$listMap[$refItem->getUrl()][$refItem->getType()][$listItem->getType()] = $listItem;
 			}
@@ -176,13 +176,13 @@ 
 			return false;
 		}
 
-		if( ($type = $this->getValue( $list, 'supplier.lists.type' )) && !isset( $this->listTypes[$type] ) )
+		if( ( $type = $this->getValue( $list, 'supplier.lists.type' ) ) && !isset( $this->listTypes[$type] ) )
 		{
 			$msg = sprintf( 'Invalid type "%1$s" (%2$s)', $type, 'supplier list' );
 			throw new \Aimeos\Controller\Common\Exception( $msg );
 		}
 
-		if( ($type = $this->getValue( $list, 'media.type' )) && !isset( $this->types[$type] ) )
+		if( ( $type = $this->getValue( $list, 'media.type' ) ) && !isset( $this->types[$type] ) )
 		{
 			$msg = sprintf( 'Invalid type "%1$s" (%2$s)', $type, 'media' );
 			throw new \Aimeos\Controller\Common\Exception( $msg );

controller/common/src/Controller/Common/Supplier/Import/Csv/Processor/Text/Standard.php

Spacing +3 added lines, -3 removed lines

@@ -118,7 +118,7 @@ 
 
 		foreach( $listItems as $listItem )
 		{
-			if( ($refItem = $listItem->getRefItem()) !== null )
+			if( ( $refItem = $listItem->getRefItem() ) !== null )
 			{
 				$listMap[$refItem->getContent()][$refItem->getType()][$listItem->getType()] = $listItem;
 			}
@@ -173,13 +173,13 @@ 
 			return false;
 		}
 
-		if( ($type = $this->getValue( $list, 'supplier.lists.type' )) && !isset( $this->listTypes[$type] ) )
+		if( ( $type = $this->getValue( $list, 'supplier.lists.type' ) ) && !isset( $this->listTypes[$type] ) )
 		{
 			$msg = sprintf( 'Invalid type "%1$s" (%2$s)', $type, 'supplier list' );
 			throw new \Aimeos\Controller\Common\Exception( $msg );
 		}
 
-		if( ($type = $this->getValue( $list, 'text.type' )) && !isset( $this->types[$type] ) )
+		if( ( $type = $this->getValue( $list, 'text.type' ) ) && !isset( $this->types[$type] ) )
 		{
 			$msg = sprintf( 'Invalid type "%1$s" (%2$s)', $type, 'text' );
 			throw new \Aimeos\Controller\Common\Exception( $msg );

controller/common/src/Controller/Common/Supplier/Import/Csv/Processor/Address/Standard.php

Spacing +1 added lines, -1 removed lines

@@ -70,7 +70,7 @@
 					continue;
 				}
 
-				if( ($item = $items->pop()) === null )
+				if( ( $item = $items->pop() ) === null )
 				{
 					$item = $manager->createItem();
 				}

Braces +2 added lines, -1 removed lines

@@ -82,7 +82,8 @@
 			$data = $this->getObject()->process( $supplier, $data );
 
 			$manager->commit();
-		} catch( \Exception $e )
+		}
+		catch( \Exception $e )
 		{
 			$manager->rollback();
 			throw $e;

controller/common/src/Controller/Common/Product/Import/Csv/Cache/Supplier/Standard.php

Spacing +1 added lines, -1 removed lines

@@ -74,7 +74,7 @@
 		$search->setConditions( $search->compare( '==', 'supplier.code', $code ) );
 
 
-		if( ($item = $manager->searchItems( $search )->first()) !== null )
+		if( ( $item = $manager->searchItems( $search )->first() ) !== null )
 		{
 			$this->suppliers[$code] = $item->getId();
 			return $item->getId();

controller/common/src/Controller/Common/Product/Import/Csv/Processor/Supplier/Standard.php

Spacing +2 added lines, -2 removed lines

@@ -153,7 +153,7 @@ 
 				{
 					$code = trim( $code );
 
-					if( ($catid = $this->cache->get( $code )) === null )
+					if( ( $catid = $this->cache->get( $code ) ) === null )
 					{
 						$msg = 'No supplier for code "%1$s" available when importing product with code "%2$s"';
 						throw new \Aimeos\Controller\Jobs\Exception( sprintf( $msg, $code, $product->getCode() ) );
@@ -227,7 +227,7 @@ 
 			return false;
 		}
 
-		if( ($type = $this->getValue( $list, 'supplier.lists.type' )) && !isset( $this->listTypes[$type] ) )
+		if( ( $type = $this->getValue( $list, 'supplier.lists.type' ) ) && !isset( $this->listTypes[$type] ) )
 		{
 			$msg = sprintf( 'Invalid type "%1$s" (%2$s)', $type, 'supplier list' );
 			throw new \Aimeos\Controller\Common\Exception( $msg );

Braces +2 added lines, -1 removed lines

@@ -181,7 +181,8 @@
 			$data = $this->getObject()->process( $product, $data );
 
 			$manager->commit();
-		} catch( \Exception $e )
+		}
+		catch( \Exception $e )
 		{
 			$manager->rollback();
 			throw $e;

controller/jobs/src/Controller/Jobs/Supplier/Import/Csv/Standard.php

Spacing +4 added lines, -4 removed lines

@@ -227,7 +227,7 @@ 
 		 * @see controller/common/supplier/import/csv/mapping
 		 * @see controller/common/supplier/import/csv/converter
 		 */
-		$maxcnt = (int)$config->get( 'controller/common/supplier/import/csv/max-size', 1000 );
+		$maxcnt = (int) $config->get( 'controller/common/supplier/import/csv/max-size', 1000 );
 
 
 		/** controller/jobs/supplier/import/csv/skip-lines
@@ -249,7 +249,7 @@ 
 		 * @see controller/jobs/supplier/import/csv/backup
 		 * @see controller/common/supplier/import/csv/max-size
 		 */
-		$skiplines = (int)$config->get( 'controller/jobs/supplier/import/csv/skip-lines', 0 );
+		$skiplines = (int) $config->get( 'controller/jobs/supplier/import/csv/skip-lines', 0 );
 
 
 		/** controller/jobs/supplier/import/csv/strict
@@ -273,7 +273,7 @@ 
 		 * @see controller/jobs/supplier/import/csv/backup
 		 * @see controller/common/supplier/import/csv/max-size
 		 */
-		$strict = (bool)$config->get( 'controller/jobs/supplier/import/csv/strict', true );
+		$strict = (bool) $config->get( 'controller/jobs/supplier/import/csv/strict', true );
 
 
 		/** controller/jobs/supplier/import/csv/backup
@@ -340,7 +340,7 @@ 
 					$content->next();
 				}
 
-				while( ($data = $this->getData( $content, $maxcnt, $codePos )) !== [] )
+				while( ( $data = $this->getData( $content, $maxcnt, $codePos ) ) !== [] )
 				{
 					$data = $this->convertData( $convlist, $data );
 					$errcnt = $this->import( $supplierMap, $data, $mappings['item'], $processor, $strict );

Braces +4 added lines, -2 removed lines

@@ -356,7 +356,8 @@ 
 			}
 
 			$container->close();
-		} catch( \Exception $e )
+		}
+		catch( \Exception $e )
 		{
 			$logger->log( 'Supplier import error: ' . $e->getMessage() . "\n" . $e->getTraceAsString() );
 			$this->mail( 'Supplier CSV import error', $e->getMessage() . "\n" . $e->getTraceAsString() );
@@ -576,7 +577,8 @@ 
 				}
 
 				$manager->commit();
-			} catch( \Exception $e )
+			}
+			catch( \Exception $e )
 			{
 				$manager->rollback();