aimeos / ai-controller-jobs

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

Spacing +10 added lines, -10 removed lines

@@ -40,16 +40,16 @@ 
 	 *
 	 * @param \Aimeos\MShop\Context\Item\Iface $context Context object
 	 */
-	public function __construct( \Aimeos\MShop\Context\Item\Iface $context )
+	public function __construct(\Aimeos\MShop\Context\Item\Iface $context)
 	{
-		parent::__construct( $context );
+		parent::__construct($context);
 
-		$manager = \Aimeos\MShop\Factory::createManager( $context, 'product/stock/type' );
+		$manager = \Aimeos\MShop\Factory::createManager($context, 'product/stock/type');
 		$search = $manager->createSearch();
-		$search->setSlice( 0, 1000 );
+		$search->setSlice(0, 1000);
 
-		foreach( $manager->searchItems( $search ) as $id => $item ) {
-			$this->types[ $item->getCode() ] = $id;
+		foreach ($manager->searchItems($search) as $id => $item) {
+			$this->types[$item->getCode()] = $id;
 		}
 	}
 
@@ -61,9 +61,9 @@ 
 	 * @param string|null $type Attribute type
 	 * @return string|null Stock type ID or null if not found
 	 */
-	public function get( $code, $type = null )
+	public function get($code, $type = null)
 	{
-		if( isset( $this->types[$code] ) ) {
+		if (isset($this->types[$code])) {
 			return $this->types[$code];
 		}
 	}
@@ -74,8 +74,8 @@ 
 	 *
 	 * @param \Aimeos\MShop\Common\Item\Iface $item Stock type object
 	 */
-	public function set( \Aimeos\MShop\Common\Item\Iface $item )
+	public function set(\Aimeos\MShop\Common\Item\Iface $item)
 	{
-		$this->types[ $item->getCode() ] = $item->getId();
+		$this->types[$item->getCode()] = $item->getId();
 	}
 }
\ No newline at end of file

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

Spacing +25 added lines, -25 removed lines

@@ -42,12 +42,12 @@ 
 	 * @param array $mapping Associative list of field position in CSV as key and domain item key as value
 	 * @param \Aimeos\Controller\Common\Product\Import\Csv\Processor\Iface $object Decorated processor
 	 */
-	public function __construct( \Aimeos\MShop\Context\Item\Iface $context, array $mapping,
-		\Aimeos\Controller\Common\Product\Import\Csv\Processor\Iface $object = null )
+	public function __construct(\Aimeos\MShop\Context\Item\Iface $context, array $mapping,
+		\Aimeos\Controller\Common\Product\Import\Csv\Processor\Iface $object = null)
 	{
-		parent::__construct( $context, $mapping, $object );
+		parent::__construct($context, $mapping, $object);
 
-		$this->cache = $this->getCache( 'stocktype' );
+		$this->cache = $this->getCache('stocktype');
 	}
 
 
@@ -58,53 +58,53 @@ 
 	 * @param array $data List of CSV fields with position as key and data as value
 	 * @return array List of data which hasn't been imported
 	 */
-	public function process( \Aimeos\MShop\Product\Item\Iface $product, array $data )
+	public function process(\Aimeos\MShop\Product\Item\Iface $product, array $data)
 	{
-		$manager = \Aimeos\MShop\Factory::createManager( $this->getContext(), 'product/stock' );
+		$manager = \Aimeos\MShop\Factory::createManager($this->getContext(), 'product/stock');
 		$manager->begin();
 
 		try
 		{
-			$map = $this->getMappedChunk( $data );
-			$items = $this->getStockItems( $product->getId() );
+			$map = $this->getMappedChunk($data);
+			$items = $this->getStockItems($product->getId());
 
-			foreach( $map as $pos => $list )
+			foreach ($map as $pos => $list)
 			{
-				if( !array_key_exists( 'product.stock.stocklevel', $list ) ) {
+				if (!array_key_exists('product.stock.stocklevel', $list)) {
 					continue;
 				}
 
-				$stockType = ( isset( $list['product.stock.type'] ) ? $list['product.stock.type'] : 'default' );
+				$stockType = (isset($list['product.stock.type']) ? $list['product.stock.type'] : 'default');
 
-				if( !isset( $list['product.stock.typeid'] ) ) {
-					$list['product.stock.typeid'] = $this->cache->get( $stockType );
+				if (!isset($list['product.stock.typeid'])) {
+					$list['product.stock.typeid'] = $this->cache->get($stockType);
 				}
 
-				if( isset( $list['product.stock.dateback'] ) && $list['product.stock.dateback'] === '' ) {
+				if (isset($list['product.stock.dateback']) && $list['product.stock.dateback'] === '') {
 					$list['product.stock.dateback'] = null;
 				}
 
-				if( $list['product.stock.stocklevel'] === '' ) {
+				if ($list['product.stock.stocklevel'] === '') {
 					$list['product.stock.stocklevel'] = null;
 				}
 
 				$list['product.stock.parentid'] = $product->getId();
 
-				if( ( $item = array_pop( $items ) ) === null ) {
+				if (($item = array_pop($items)) === null) {
 					$item = $manager->createItem();
 				}
 
-				$item->fromArray( $list );
-				$manager->saveItem( $item );
+				$item->fromArray($list);
+				$manager->saveItem($item);
 			}
 
-			$manager->deleteItems( array_keys( $items ) );
+			$manager->deleteItems(array_keys($items));
 
-			$remaining = $this->getObject()->process( $product, $data );
+			$remaining = $this->getObject()->process($product, $data);
 
 			$manager->commit();
 		}
-		catch( \Exception $e )
+		catch (\Exception $e)
 		{
 			$manager->rollback();
 			throw $e;
@@ -120,13 +120,13 @@ 
 	 * @param string $prodid Unique product ID
 	 * @return array Associative list of product stock items
 	 */
-	protected function getStockItems( $prodid )
+	protected function getStockItems($prodid)
 	{
-		$manager = \Aimeos\MShop\Factory::createManager( $this->getContext(), 'product/stock' );
+		$manager = \Aimeos\MShop\Factory::createManager($this->getContext(), 'product/stock');
 
 		$search = $manager->createSearch();
-		$search->setConditions( $search->compare( '==', 'product.stock.parentid', $prodid ) );
+		$search->setConditions($search->compare('==', 'product.stock.parentid', $prodid));
 
-		return $manager->searchItems( $search );
+		return $manager->searchItems($search);
 	}
 }

controller/common/tests/Controller/Common/Product/Import/Csv/Cache/Warehouse/StandardTest.php

Spacing +12 added lines, -12 removed lines

@@ -14,43 +14,43 @@
 
 	protected function setUp()
 	{
-		\Aimeos\MShop\Factory::setCache( true );
+		\Aimeos\MShop\Factory::setCache(true);
 
 		$context = \TestHelperCntl::getContext();
-		$this->object = new \Aimeos\Controller\Common\Product\Import\Csv\Cache\Stocktype\Standard( $context );
+		$this->object = new \Aimeos\Controller\Common\Product\Import\Csv\Cache\Stocktype\Standard($context);
 	}
 
 
 	protected function tearDown()
 	{
-		\Aimeos\MShop\Factory::setCache( false );
+		\Aimeos\MShop\Factory::setCache(false);
 		\Aimeos\MShop\Factory::clear();
 	}
 
 
 	public function testGet()
 	{
-		$result = $this->object->get( 'default' );
+		$result = $this->object->get('default');
 
-		$this->assertNotEquals( null, $result );
+		$this->assertNotEquals(null, $result);
 	}
 
 
 	public function testGetUnknown()
 	{
-		$this->assertEquals( null, $this->object->get( 'cache-test' ) );
+		$this->assertEquals(null, $this->object->get('cache-test'));
 	}
 
 
 	public function testSet()
 	{
-		$item = \Aimeos\MShop\Factory::createManager( \TestHelperCntl::getContext(), 'product/stock/type' )->createItem();
-		$item->setCode( 'cache-test' );
-		$item->setId( 1 );
+		$item = \Aimeos\MShop\Factory::createManager(\TestHelperCntl::getContext(), 'product/stock/type')->createItem();
+		$item->setCode('cache-test');
+		$item->setId(1);
 
-		$this->object->set( $item );
-		$id = $this->object->get( 'cache-test' );
+		$this->object->set($item);
+		$id = $this->object->get('cache-test');
 
-		$this->assertEquals( $item->getId(), $id );
+		$this->assertEquals($item->getId(), $id);
 	}
 }
\ No newline at end of file