aimeos / ai-controller-jobs

controller/common/src/Controller/Common/Order/Export/Csv/Processor/Address/Standard.php

Spacing +6 added lines, -6 removed lines

@@ -40,28 +40,28 @@
 	 * @param \Aimeos\MShop\Order\Item\Base\Iface $order Full order with associated items
 	 * @return array Two dimensional associative list of order data representing the lines in CSV
 	 */
-	public function process( \Aimeos\MShop\Order\Item\Iface $invoice, \Aimeos\MShop\Order\Item\Base\Iface $order )
+	public function process(\Aimeos\MShop\Order\Item\Iface $invoice, \Aimeos\MShop\Order\Item\Base\Iface $order)
 	{
 		$result = [];
 		$addresses = $order->getAddresses();
 
-		krsort( $addresses );
+		krsort($addresses);
 
-		foreach( $addresses as $item )
+		foreach ($addresses as $item)
 		{
 			$data = [];
 			$list = $item->toArray();
 
-			foreach( $this->getMapping() as $pos => $key )
+			foreach ($this->getMapping() as $pos => $key)
 			{
-				if( array_key_exists( $key, $list ) ) {
+				if (array_key_exists($key, $list)) {
 					$data[$pos] = $list[$key];
 				} else {
 					$data[$pos] = '';
 				}
 			}
 
-			ksort( $data );
+			ksort($data);
 			$result[] = $data;
 		}
 

controller/common/src/Controller/Common/Order/Export/Csv/Processor/Invoice/Standard.php

Spacing +4 added lines, -4 removed lines

@@ -40,21 +40,21 @@
 	 * @param \Aimeos\MShop\Order\Item\Base\Iface $order Full order with associated items
 	 * @return array Two dimensional associative list of order data representing the lines in CSV
 	 */
-	public function process( \Aimeos\MShop\Order\Item\Iface $invoice, \Aimeos\MShop\Order\Item\Base\Iface $order )
+	public function process(\Aimeos\MShop\Order\Item\Iface $invoice, \Aimeos\MShop\Order\Item\Base\Iface $order)
 	{
 		$result = [];
 		$list = $invoice->toArray() + $order->toArray();
 
-		foreach( $this->getMapping() as $pos => $key )
+		foreach ($this->getMapping() as $pos => $key)
 		{
-			if( array_key_exists( $key, $list ) ) {
+			if (array_key_exists($key, $list)) {
 				$result[$pos] = $list[$key];
 			} else {
 				$result[$pos] = '';
 			}
 		}
 
-		ksort( $result );
+		ksort($result);
 
 		return [$result];
 	}

controller/common/src/Controller/Common/Order/Export/Csv/Processor/Iface.php

Spacing +2 added lines, -2 removed lines

@@ -25,7 +25,7 @@ 
 	 * @param \Aimeos\MShop\Context\Item\Iface $context Context object
 	 * @param array $mapping Associative list of field position in CSV as key and domain item key as value
 	 */
-	public function __construct( \Aimeos\MShop\Context\Item\Iface $context, array $mapping );
+	public function __construct(\Aimeos\MShop\Context\Item\Iface $context, array $mapping);
 
 
 	/**
@@ -35,5 +35,5 @@ 
 	 * @param \Aimeos\MShop\Order\Item\Base\Iface $order Full order with associated items
 	 * @return array Two dimensional associative list of order data representing the lines in CSV
 	 */
-	public function process( \Aimeos\MShop\Order\Item\Iface $invoice, \Aimeos\MShop\Order\Item\Base\Iface $order );
+	public function process(\Aimeos\MShop\Order\Item\Iface $invoice, \Aimeos\MShop\Order\Item\Base\Iface $order);
 }
\ No newline at end of file

controller/common/src/Controller/Common/Coupon/Import/Csv/Processor/Base.php

Spacing +4 added lines, -4 removed lines

@@ -32,8 +32,8 @@ 
 	 * @param array $mapping Associative list of field position in CSV as key and domain item key as value
 	 * @param \Aimeos\Controller\Common\Coupon\Import\Csv\Processor\Iface $object Decorated processor
 	 */
-	public function __construct( \Aimeos\MShop\Context\Item\Iface $context, array $mapping,
-		\Aimeos\Controller\Common\Coupon\Import\Csv\Processor\Iface $object = null )
+	public function __construct(\Aimeos\MShop\Context\Item\Iface $context, array $mapping,
+		\Aimeos\Controller\Common\Coupon\Import\Csv\Processor\Iface $object = null)
 	{
 		$this->context = $context;
 		$this->mapping = $mapping;
@@ -71,8 +71,8 @@ 
 	 */
 	protected function getObject()
 	{
-		if( $this->object === null ) {
-			throw new \Aimeos\Controller\Jobs\Exception( 'No processor object available' );
+		if ($this->object === null) {
+			throw new \Aimeos\Controller\Jobs\Exception('No processor object available');
 		}
 
 		return $this->object;

controller/common/src/Controller/Common/Coupon/Import/Csv/Processor/Done.php

Spacing +3 added lines, -3 removed lines

@@ -27,8 +27,8 @@ 
 	 * @param array $mapping Associative list of field position in CSV as key and domain item key as value
 	 * @param \Aimeos\Controller\Common\Coupon\Import\Csv\Processor\Iface $object Decorated processor
 	 */
-	public function __construct( \Aimeos\MShop\Context\Item\Iface $context, array $mapping,
-		\Aimeos\Controller\Common\Coupon\Import\Csv\Processor\Iface $object = null )
+	public function __construct(\Aimeos\MShop\Context\Item\Iface $context, array $mapping,
+		\Aimeos\Controller\Common\Coupon\Import\Csv\Processor\Iface $object = null)
 	{
 	}
 
@@ -40,7 +40,7 @@ 
 	 * @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\Coupon\Item\Code\Iface $item, array $data )
+	public function process(\Aimeos\MShop\Coupon\Item\Code\Iface $item, array $data)
 	{
 		return $data;
 	}

controller/common/src/Controller/Common/Coupon/Import/Csv/Processor/Iface.php

Spacing +3 added lines, -3 removed lines

@@ -26,8 +26,8 @@ 
 	 * @param array $mapping Associative list of field position in CSV as key and domain item key as value
 	 * @param \Aimeos\Controller\Common\Coupon\Import\Csv\Processor\Iface $object Decorated processor
 	 */
-	public function __construct( \Aimeos\MShop\Context\Item\Iface $context, array $mapping,
-		\Aimeos\Controller\Common\Coupon\Import\Csv\Processor\Iface $object = null );
+	public function __construct(\Aimeos\MShop\Context\Item\Iface $context, array $mapping,
+		\Aimeos\Controller\Common\Coupon\Import\Csv\Processor\Iface $object = null);
 
 
 	/**
@@ -37,5 +37,5 @@ 
 	 * @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\Coupon\Item\Code\Iface $item, array $data );
+	public function process(\Aimeos\MShop\Coupon\Item\Code\Iface $item, array $data);
 }
\ No newline at end of file

controller/common/tests/Controller/Common/Coupon/Import/Csv/Processor/Code/StandardTest.php

Spacing +35 added lines, -35 removed lines

@@ -17,16 +17,16 @@ 
 
 	protected function setUp()
 	{
-		\Aimeos\MShop\Factory::setCache( true );
+		\Aimeos\MShop\Factory::setCache(true);
 
 		$this->context = \TestHelperCntl::getContext();
-		$this->endpoint = new \Aimeos\Controller\Common\Coupon\Import\Csv\Processor\Done( $this->context, [] );
+		$this->endpoint = new \Aimeos\Controller\Common\Coupon\Import\Csv\Processor\Done($this->context, []);
 	}
 
 
 	protected function tearDown()
 	{
-		\Aimeos\MShop\Factory::setCache( false );
+		\Aimeos\MShop\Factory::setCache(false);
 		\Aimeos\MShop\Factory::clear();
 	}
 
@@ -47,24 +47,24 @@ 
 			3 => '',
 		);
 
-		$manager = \Aimeos\MShop\Factory::createManager( $this->context, 'coupon' );
-		$codeManager = \Aimeos\MShop\Factory::createManager( $this->context, 'coupon/code' );
+		$manager = \Aimeos\MShop\Factory::createManager($this->context, 'coupon');
+		$codeManager = \Aimeos\MShop\Factory::createManager($this->context, 'coupon/code');
 
-		$coupon = $manager->saveItem( $manager->createItem()->setProvider( 'test' ) );
+		$coupon = $manager->saveItem($manager->createItem()->setProvider('test'));
 		$couponCode = $codeManager->createItem();
-		$couponCode->setParentId( $coupon->getId() );
+		$couponCode->setParentId($coupon->getId());
 
-		$object = new \Aimeos\Controller\Common\Coupon\Import\Csv\Processor\Code\Standard( $this->context, $mapping, $this->endpoint );
-		$object->process( $couponCode, $data );
+		$object = new \Aimeos\Controller\Common\Coupon\Import\Csv\Processor\Code\Standard($this->context, $mapping, $this->endpoint);
+		$object->process($couponCode, $data);
 
-		$codeManager->deleteItem( $couponCode->getId() );
-		$manager->deleteItem( $coupon->getId() );
+		$codeManager->deleteItem($couponCode->getId());
+		$manager->deleteItem($coupon->getId());
 
 
-		$this->assertEquals( 10, $couponCode->getCount() );
-		$this->assertEquals( 'jobimporttest', $couponCode->getCode() );
-		$this->assertEquals( '2000-01-01 00:00:00', $couponCode->getDateStart() );
-		$this->assertEquals( null, $couponCode->getDateEnd() );
+		$this->assertEquals(10, $couponCode->getCount());
+		$this->assertEquals('jobimporttest', $couponCode->getCode());
+		$this->assertEquals('2000-01-01 00:00:00', $couponCode->getDateStart());
+		$this->assertEquals(null, $couponCode->getDateEnd());
 	}
 
 
@@ -86,23 +86,23 @@ 
 		);
 
 
-		$manager = \Aimeos\MShop\Factory::createManager( $this->context, 'coupon' );
-		$codeManager = \Aimeos\MShop\Factory::createManager( $this->context, 'coupon/code' );
+		$manager = \Aimeos\MShop\Factory::createManager($this->context, 'coupon');
+		$codeManager = \Aimeos\MShop\Factory::createManager($this->context, 'coupon/code');
 
-		$coupon = $manager->saveItem( $manager->createItem()->setProvider( 'test' ) );
+		$coupon = $manager->saveItem($manager->createItem()->setProvider('test'));
 		$couponCode = $codeManager->createItem();
-		$couponCode->setParentId( $coupon->getId() );
+		$couponCode->setParentId($coupon->getId());
 
-		$object = new \Aimeos\Controller\Common\Coupon\Import\Csv\Processor\Code\Standard( $this->context, $mapping, $this->endpoint );
-		$object->process( $couponCode, $data );
-		$object->process( $couponCode, $dataUpdate );
+		$object = new \Aimeos\Controller\Common\Coupon\Import\Csv\Processor\Code\Standard($this->context, $mapping, $this->endpoint);
+		$object->process($couponCode, $data);
+		$object->process($couponCode, $dataUpdate);
 
-		$codeManager->deleteItem( $couponCode->getId() );
-		$manager->deleteItem( $coupon->getId() );
+		$codeManager->deleteItem($couponCode->getId());
+		$manager->deleteItem($coupon->getId());
 
 
-		$this->assertEquals( 5, $couponCode->getCount() );
-		$this->assertEquals( 'jobimporttest', $couponCode->getCode() );
+		$this->assertEquals(5, $couponCode->getCount());
+		$this->assertEquals('jobimporttest', $couponCode->getCode());
 	}
 
 
@@ -119,20 +119,20 @@ 
 		);
 
 
-		$manager = \Aimeos\MShop\Factory::createManager( $this->context, 'coupon' );
-		$codeManager = \Aimeos\MShop\Factory::createManager( $this->context, 'coupon/code' );
+		$manager = \Aimeos\MShop\Factory::createManager($this->context, 'coupon');
+		$codeManager = \Aimeos\MShop\Factory::createManager($this->context, 'coupon/code');
 
-		$coupon = $manager->saveItem( $manager->createItem()->setProvider( 'test' ) );
+		$coupon = $manager->saveItem($manager->createItem()->setProvider('test'));
 		$couponCode = $codeManager->createItem();
-		$couponCode->setParentId( $coupon->getId() );
+		$couponCode->setParentId($coupon->getId());
 
-		$object = new \Aimeos\Controller\Common\Coupon\Import\Csv\Processor\Code\Standard( $this->context, $mapping, $this->endpoint );
-		$object->process( $couponCode, $data );
+		$object = new \Aimeos\Controller\Common\Coupon\Import\Csv\Processor\Code\Standard($this->context, $mapping, $this->endpoint);
+		$object->process($couponCode, $data);
 
-		$codeManager->deleteItem( $couponCode->getId() );
-		$manager->deleteItem( $coupon->getId() );
+		$codeManager->deleteItem($couponCode->getId());
+		$manager->deleteItem($coupon->getId());
 
 
-		$this->assertEquals( 0, $couponCode->getCount() );
+		$this->assertEquals(0, $couponCode->getCount());
 	}
 }
\ No newline at end of file

controller/jobs/src/Controller/Jobs/Coupon/Import/Csv/Code/Factory.php

Spacing +6 added lines, -6 removed lines

@@ -29,7 +29,7 @@ 
 	 * @param string|null $name Name of the controller or "Standard" if null
 	 * @return \Aimeos\Controller\Jobs\Iface New controller object
 	 */
-	public static function createController( \Aimeos\MShop\Context\Item\Iface $context, \Aimeos\Bootstrap $aimeos, $name = null )
+	public static function createController(\Aimeos\MShop\Context\Item\Iface $context, \Aimeos\Bootstrap $aimeos, $name = null)
 	{
 		/** controller/jobs/coupon/import/csv/code/name
 		 * Class name of the used coupon code import job controller implementation
@@ -64,20 +64,20 @@ 
 		 * @since 2017.10
 		 * @category Developer
 		 */
-		if ( $name === null ) {
+		if ($name === null) {
 			$name = $context->getConfig()->get('controller/jobs/coupon/import/csv/code/name', 'Standard');
 		}
 
-		if ( ctype_alnum($name) === false )
+		if (ctype_alnum($name) === false)
 		{
 			$classname = is_string($name) ? '\\Aimeos\\Controller\\Jobs\\Coupon\\Import\\Csv\\Code\\' . $name : '<not a string>';
-			throw new \Aimeos\Controller\Jobs\Exception( sprintf( 'Invalid characters in class name "%1$s"', $classname ) );
+			throw new \Aimeos\Controller\Jobs\Exception(sprintf('Invalid characters in class name "%1$s"', $classname));
 		}
 
 		$iface = '\\Aimeos\\Controller\\Jobs\\Iface';
 		$classname = '\\Aimeos\\Controller\\Jobs\\Coupon\\Import\\Csv\\Code\\' . $name;
 
-		$controller = self::createControllerBase( $context, $aimeos, $classname, $iface );
+		$controller = self::createControllerBase($context, $aimeos, $classname, $iface);
 
 		/** controller/jobs/coupon/import/csv/code/decorators/excludes
 		 * Excludes decorators added by the "common" option from the coupon code import CSV job controller
@@ -154,6 +154,6 @@ 
 		 * @see controller/jobs/coupon/import/csv/code/decorators/excludes
 		 * @see controller/jobs/coupon/import/csv/code/decorators/global
 		 */
-		return self::addControllerDecorators( $context, $aimeos, $controller, 'coupon/import/csv/code' );
+		return self::addControllerDecorators($context, $aimeos, $controller, 'coupon/import/csv/code');
 	}
 }
\ No newline at end of file

controller/jobs/tests/Controller/Jobs/Product/Bought/StandardTest.php

Spacing +11 added lines, -11 removed lines

@@ -19,18 +19,18 @@ 
 
 	protected function setUp()
 	{
-		\Aimeos\MShop\Factory::setCache( true );
+		\Aimeos\MShop\Factory::setCache(true);
 
 		$this->context = \TestHelperJobs::getContext();
 		$this->aimeos = \TestHelperJobs::getAimeos();
 
-		$this->object = new \Aimeos\Controller\Jobs\Product\Bought\Standard( $this->context, $this->aimeos );
+		$this->object = new \Aimeos\Controller\Jobs\Product\Bought\Standard($this->context, $this->aimeos);
 	}
 
 
 	protected function tearDown()
 	{
-		\Aimeos\MShop\Factory::setCache( false );
+		\Aimeos\MShop\Factory::setCache(false);
 		\Aimeos\MShop\Factory::clear();
 
 		$this->object = null;
@@ -39,28 +39,28 @@ 
 
 	public function testGetName()
 	{
-		$this->assertEquals( 'Products bought together', $this->object->getName() );
+		$this->assertEquals('Products bought together', $this->object->getName());
 	}
 
 
 	public function testGetDescription()
 	{
 		$text = 'Creates bought together product suggestions';
-		$this->assertEquals( $text, $this->object->getDescription() );
+		$this->assertEquals($text, $this->object->getDescription());
 	}
 
 
 	public function testRun()
 	{
-		$stub = $this->getMockBuilder( '\\Aimeos\\MShop\\Product\\Manager\\Lists\\Standard' )
-			->setConstructorArgs( array( $this->context ) )
-			->setMethods( array( 'deleteItems', 'saveItem' ) )
+		$stub = $this->getMockBuilder('\\Aimeos\\MShop\\Product\\Manager\\Lists\\Standard')
+			->setConstructorArgs(array($this->context))
+			->setMethods(array('deleteItems', 'saveItem'))
 			->getMock();
 
-		\Aimeos\MShop\Factory::injectManager( $this->context, 'product/lists', $stub );
+		\Aimeos\MShop\Factory::injectManager($this->context, 'product/lists', $stub);
 
-		$stub->expects( $this->atLeastOnce() )->method( 'deleteItems' );
-		$stub->expects( $this->atLeastOnce() )->method( 'saveItem' );
+		$stub->expects($this->atLeastOnce())->method('deleteItems');
+		$stub->expects($this->atLeastOnce())->method('saveItem');
 
 		$this->object->run();
 	}