Inspection of "Adapt to simplified MShop factory" - aimeos/ai-controller-frontend - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( 8086a8...921503 )

by Aimeos

created 2019-01-07 11:33 UTC

Status

Spacing +45 added lines, -45 removed lines patch added patch discarded remove patch

@@ -20,54 +20,54 @@  discard block
 block discarded – undo
 	{
 		$this->context = \TestHelperFrontend::getContext();
 
-		$this->stub = $this->getMockBuilder( \Aimeos\Controller\Frontend\Basket\Standard::class )
+		$this->stub = $this->getMockBuilder(\Aimeos\Controller\Frontend\Basket\Standard::class)
 			->disableOriginalConstructor()
 			->getMock();
 
-		$this->object = $this->getMockBuilder( \Aimeos\Controller\Frontend\Basket\Decorator\Base::class )
-			->setConstructorArgs( [$this->stub, $this->context] )
+		$this->object = $this->getMockBuilder(\Aimeos\Controller\Frontend\Basket\Decorator\Base::class)
+			->setConstructorArgs([$this->stub, $this->context])
 			->getMockForAbstractClass();
 	}
 
 
 	protected function tearDown()
 	{
-		unset( $this->context, $this->object, $this->stub );
+		unset($this->context, $this->object, $this->stub);
 	}
 
 
 	public function testConstructException()
 	{
-		$stub = $this->getMockBuilder( \Aimeos\Controller\Frontend\Iface::class )->getMock();
+		$stub = $this->getMockBuilder(\Aimeos\Controller\Frontend\Iface::class)->getMock();
 
-		$this->setExpectedException( \Aimeos\MW\Common\Exception::class );
+		$this->setExpectedException(\Aimeos\MW\Common\Exception::class);
 
-		$this->getMockBuilder( \Aimeos\Controller\Frontend\Basket\Decorator\Base::class )
-			->setConstructorArgs( [$stub, $this->context] )
+		$this->getMockBuilder(\Aimeos\Controller\Frontend\Basket\Decorator\Base::class)
+			->setConstructorArgs([$stub, $this->context])
 			->getMockForAbstractClass();
 	}
 
 
 	public function testCall()
 	{
-		$stub = $this->getMockBuilder( \Aimeos\Controller\Frontend\Basket\Standard::class )
+		$stub = $this->getMockBuilder(\Aimeos\Controller\Frontend\Basket\Standard::class)
 			->disableOriginalConstructor()
-			->setMethods( ['invalid'] )
+			->setMethods(['invalid'])
 			->getMock();
 
-		$object = $this->getMockBuilder( \Aimeos\Controller\Frontend\Basket\Decorator\Base::class )
-			->setConstructorArgs( [$stub, $this->context] )
+		$object = $this->getMockBuilder(\Aimeos\Controller\Frontend\Basket\Decorator\Base::class)
+			->setConstructorArgs([$stub, $this->context])
 			->getMockForAbstractClass();
 
-		$stub->expects( $this->once() )->method( 'invalid' )->will( $this->returnValue( true ) );
+		$stub->expects($this->once())->method('invalid')->will($this->returnValue(true));
 
-		$this->assertTrue( $object->invalid() );
+		$this->assertTrue($object->invalid());
 	}
 
 
 	public function testClear()
 	{
-		$this->stub->expects( $this->once() )->method( 'clear' );
+		$this->stub->expects($this->once())->method('clear');
 
 		$this->object->clear();
 	}
@@ -76,18 +76,18 @@  discard block
 block discarded – undo
 	public function testGet()
 	{
 		$context = \TestHelperFrontend::getContext();
-		$order = \Aimeos\MShop::create( $context, 'order/base' )->createItem();
+		$order = \Aimeos\MShop::create($context, 'order/base')->createItem();
 
-		$this->stub->expects( $this->once() )->method( 'get' )
-			->will( $this->returnValue( $order ) );
+		$this->stub->expects($this->once())->method('get')
+			->will($this->returnValue($order));
 
-		$this->assertInstanceOf( \Aimeos\MShop\Order\Item\Base\Iface::class, $this->object->get() );
+		$this->assertInstanceOf(\Aimeos\MShop\Order\Item\Base\Iface::class, $this->object->get());
 	}
 
 
 	public function testSave()
 	{
-		$this->stub->expects( $this->once() )->method( 'save' );
+		$this->stub->expects($this->once())->method('save');
 
 		$this->object->save();
 	}
@@ -95,15 +95,15 @@  discard block
 block discarded – undo
 
 	public function testSetType()
 	{
-		$this->stub->expects( $this->once() )->method( 'setType' );
+		$this->stub->expects($this->once())->method('setType');
 
-		$this->object->setType( 'test' );
+		$this->object->setType('test');
 	}
 
 
 	public function testStore()
 	{
-		$this->stub->expects( $this->once() )->method( 'store' );
+		$this->stub->expects($this->once())->method('store');
 
 		$this->object->store();
 	}
@@ -111,7 +111,7 @@  discard block
 block discarded – undo
 
 	public function testLoad()
 	{
-		$this->stub->expects( $this->once() )->method( 'load' );
+		$this->stub->expects($this->once())->method('load');
 
 		$this->object->load( -1 );
 	}
@@ -119,7 +119,7 @@  discard block
 block discarded – undo
 
 	public function testAddProduct()
 	{
-		$this->stub->expects( $this->once() )->method( 'addProduct' );
+		$this->stub->expects($this->once())->method('addProduct');
 
 		$this->object->addProduct( -1 );
 	}
@@ -127,73 +127,73 @@  discard block
 block discarded – undo
 
 	public function testDeleteProduct()
 	{
-		$this->stub->expects( $this->once() )->method( 'deleteProduct' );
+		$this->stub->expects($this->once())->method('deleteProduct');
 
-		$this->object->deleteProduct( 0 );
+		$this->object->deleteProduct(0);
 	}
 
 
 	public function testEditProduct()
 	{
-		$this->stub->expects( $this->once() )->method( 'editProduct' );
+		$this->stub->expects($this->once())->method('editProduct');
 
-		$this->object->editProduct( 0, 1 );
+		$this->object->editProduct(0, 1);
 	}
 
 
 	public function testAddCoupon()
 	{
-		$this->stub->expects( $this->once() )->method( 'addCoupon' );
+		$this->stub->expects($this->once())->method('addCoupon');
 
-		$this->object->addCoupon( 'test' );
+		$this->object->addCoupon('test');
 	}
 
 
 	public function testDeleteCoupon()
 	{
-		$this->stub->expects( $this->once() )->method( 'deleteCoupon' );
+		$this->stub->expects($this->once())->method('deleteCoupon');
 
-		$this->object->deleteCoupon( 'test' );
+		$this->object->deleteCoupon('test');
 	}
 
 
 	public function testSetAddress()
 	{
-		$this->stub->expects( $this->once() )->method( 'setAddress' );
+		$this->stub->expects($this->once())->method('setAddress');
 
-		$this->object->setAddress( \Aimeos\MShop\Order\Item\Base\Address\Base::TYPE_PAYMENT, null );
+		$this->object->setAddress(\Aimeos\MShop\Order\Item\Base\Address\Base::TYPE_PAYMENT, null);
 	}
 
 
 	public function testAddService()
 	{
-		$this->stub->expects( $this->once() )->method( 'addService' );
+		$this->stub->expects($this->once())->method('addService');
 
-		$this->object->addService( \Aimeos\MShop\Order\Item\Base\Service\Base::TYPE_PAYMENT, -1 );
+		$this->object->addService(\Aimeos\MShop\Order\Item\Base\Service\Base::TYPE_PAYMENT, -1);
 	}
 
 
 	public function testDeleteService()
 	{
-		$this->stub->expects( $this->once() )->method( 'deleteService' );
+		$this->stub->expects($this->once())->method('deleteService');
 
-		$this->object->deleteService( \Aimeos\MShop\Order\Item\Base\Service\Base::TYPE_PAYMENT );
+		$this->object->deleteService(\Aimeos\MShop\Order\Item\Base\Service\Base::TYPE_PAYMENT);
 	}
 
 
 	public function testGetController()
 	{
-		$result = $this->access( 'getController' )->invokeArgs( $this->object, [] );
+		$result = $this->access('getController')->invokeArgs($this->object, []);
 
-		$this->assertSame( $this->stub, $result );
+		$this->assertSame($this->stub, $result);
 	}
 
 
-	protected function access( $name )
+	protected function access($name)
 	{
-		$class = new \ReflectionClass( \Aimeos\Controller\Frontend\Basket\Decorator\Base::class );
-		$method = $class->getMethod( $name );
-		$method->setAccessible( true );
+		$class = new \ReflectionClass(\Aimeos\Controller\Frontend\Basket\Decorator\Base::class);
+		$method = $class->getMethod($name);
+		$method->setAccessible(true);
 
 		return $method;
 	}

Please login to merge, or discard this patch.

frontend/tests/Controller/Frontend/Basket/Decorator/BundleTest.php 1 patch

Spacing +14 added lines, -14 removed lines patch added patch discarded remove patch

@@ -18,40 +18,40 @@
 block discarded – undo
 	protected function setUp()
 	{
 		$this->context = \TestHelperFrontend::getContext();
-		$object = new \Aimeos\Controller\Frontend\Basket\Standard( $this->context );
-		$this->object = new \Aimeos\Controller\Frontend\Basket\Decorator\Bundle( $object, $this->context );
+		$object = new \Aimeos\Controller\Frontend\Basket\Standard($this->context);
+		$this->object = new \Aimeos\Controller\Frontend\Basket\Decorator\Bundle($object, $this->context);
 	}
 
 
 	protected function tearDown()
 	{
 		$this->object->clear();
-		$this->context->getSession()->set( 'aimeos', [] );
+		$this->context->getSession()->set('aimeos', []);
 
-		unset( $this->object );
+		unset($this->object);
 	}
 
 
 	public function testAddProductBundle()
 	{
-		$item = \Aimeos\MShop::create( $this->context, 'product' )->findItem( 'U:BUNDLE' );
+		$item = \Aimeos\MShop::create($this->context, 'product')->findItem('U:BUNDLE');
 
-		$this->object->addProduct( $item->getId(), 1 );
+		$this->object->addProduct($item->getId(), 1);
 
-		$this->assertEquals( 1, count( $this->object->get()->getProducts() ) );
-		$this->assertEquals( 'U:BUNDLE', $this->object->get()->getProduct( 0 )->getProductCode() );
-		$this->assertEquals( 2, count( $this->object->get()->getProduct( 0 )->getProducts() ) );
+		$this->assertEquals(1, count($this->object->get()->getProducts()));
+		$this->assertEquals('U:BUNDLE', $this->object->get()->getProduct(0)->getProductCode());
+		$this->assertEquals(2, count($this->object->get()->getProduct(0)->getProducts()));
 	}
 
 
 	public function testAddProductNoBundle()
 	{
-		$item = \Aimeos\MShop::create( $this->context, 'product' )->findItem( 'CNC' );
+		$item = \Aimeos\MShop::create($this->context, 'product')->findItem('CNC');
 
-		$this->object->addProduct( $item->getId(), 1 );
+		$this->object->addProduct($item->getId(), 1);
 
-		$this->assertEquals( 1, count( $this->object->get()->getProducts() ) );
-		$this->assertEquals( 'CNC', $this->object->get()->getProduct( 0 )->getProductCode() );
-		$this->assertEquals( 0, count( $this->object->get()->getProduct( 0 )->getProducts() ) );
+		$this->assertEquals(1, count($this->object->get()->getProducts()));
+		$this->assertEquals('CNC', $this->object->get()->getProduct(0)->getProductCode());
+		$this->assertEquals(0, count($this->object->get()->getProduct(0)->getProducts()));
 	}
 }

Please login to merge, or discard this patch.

frontend/tests/Controller/Frontend/Basket/Decorator/CategoryTest.php 1 patch

Spacing +11 added lines, -11 removed lines patch added patch discarded remove patch

@@ -18,35 +18,35 @@
 block discarded – undo
 	protected function setUp()
 	{
 		$this->context = \TestHelperFrontend::getContext();
-		$object = new \Aimeos\Controller\Frontend\Basket\Standard( $this->context );
-		$this->object = new \Aimeos\Controller\Frontend\Basket\Decorator\Category( $object, $this->context );
+		$object = new \Aimeos\Controller\Frontend\Basket\Standard($this->context);
+		$this->object = new \Aimeos\Controller\Frontend\Basket\Decorator\Category($object, $this->context);
 	}
 
 
 	protected function tearDown()
 	{
 		$this->object->clear();
-		$this->context->getSession()->set( 'aimeos', [] );
+		$this->context->getSession()->set('aimeos', []);
 
-		unset( $this->object );
+		unset($this->object);
 	}
 
 
 	public function testAddProductWithCategory()
 	{
-		$manager = \Aimeos\MShop::create( $this->context, 'product' );
-		$item = $manager->findItem( 'CNE' );
+		$manager = \Aimeos\MShop::create($this->context, 'product');
+		$item = $manager->findItem('CNE');
 
-		$this->object->addProduct( $item->getId(), 5 );
+		$this->object->addProduct($item->getId(), 5);
 	}
 
 
 	public function testAddProductNoCategory()
 	{
-		$manager = \Aimeos\MShop::create( $this->context, 'product' );
-		$item = $manager->findItem( 'ABCD' );
+		$manager = \Aimeos\MShop::create($this->context, 'product');
+		$item = $manager->findItem('ABCD');
 
-		$this->setExpectedException( \Aimeos\Controller\Frontend\Basket\Exception::class );
-		$this->object->addProduct( $item->getId(), 5 );
+		$this->setExpectedException(\Aimeos\Controller\Frontend\Basket\Exception::class);
+		$this->object->addProduct($item->getId(), 5);
 	}
 }

Please login to merge, or discard this patch.

frontend/tests/Controller/Frontend/Catalog/Decorator/BaseTest.php 1 patch

Spacing +31 added lines, -31 removed lines patch added patch discarded remove patch

@@ -20,95 +20,95 @@
 block discarded – undo
 	{
 		$this->context = \TestHelperFrontend::getContext();
 
-		$this->stub = $this->getMockBuilder( \Aimeos\Controller\Frontend\Catalog\Standard::class )
+		$this->stub = $this->getMockBuilder(\Aimeos\Controller\Frontend\Catalog\Standard::class)
 			->disableOriginalConstructor()
 			->getMock();
 
-		$this->object = $this->getMockBuilder( \Aimeos\Controller\Frontend\Catalog\Decorator\Base::class )
-			->setConstructorArgs( [$this->stub, $this->context] )
+		$this->object = $this->getMockBuilder(\Aimeos\Controller\Frontend\Catalog\Decorator\Base::class)
+			->setConstructorArgs([$this->stub, $this->context])
 			->getMockForAbstractClass();
 	}
 
 
 	protected function tearDown()
 	{
-		unset( $this->context, $this->object, $this->stub );
+		unset($this->context, $this->object, $this->stub);
 	}
 
 
 	public function testConstructException()
 	{
-		$stub = $this->getMockBuilder( \Aimeos\Controller\Frontend\Iface::class )->getMock();
+		$stub = $this->getMockBuilder(\Aimeos\Controller\Frontend\Iface::class)->getMock();
 
-		$this->setExpectedException( \Aimeos\MW\Common\Exception::class );
+		$this->setExpectedException(\Aimeos\MW\Common\Exception::class);
 
-		$this->getMockBuilder( \Aimeos\Controller\Frontend\Catalog\Decorator\Base::class )
-			->setConstructorArgs( [$stub, $this->context] )
+		$this->getMockBuilder(\Aimeos\Controller\Frontend\Catalog\Decorator\Base::class)
+			->setConstructorArgs([$stub, $this->context])
 			->getMockForAbstractClass();
 	}
 
 
 	public function testCall()
 	{
-		$stub = $this->getMockBuilder( \Aimeos\Controller\Frontend\Catalog\Standard::class )
+		$stub = $this->getMockBuilder(\Aimeos\Controller\Frontend\Catalog\Standard::class)
 			->disableOriginalConstructor()
-			->setMethods( ['invalid'] )
+			->setMethods(['invalid'])
 			->getMock();
 
-		$object = $this->getMockBuilder( \Aimeos\Controller\Frontend\Catalog\Decorator\Base::class )
-			->setConstructorArgs( [$stub, $this->context] )
+		$object = $this->getMockBuilder(\Aimeos\Controller\Frontend\Catalog\Decorator\Base::class)
+			->setConstructorArgs([$stub, $this->context])
 			->getMockForAbstractClass();
 
-		$stub->expects( $this->once() )->method( 'invalid' )->will( $this->returnValue( true ) );
+		$stub->expects($this->once())->method('invalid')->will($this->returnValue(true));
 
-		$this->assertTrue( $object->invalid() );
+		$this->assertTrue($object->invalid());
 	}
 
 
 	public function testCreateFilter()
 	{
-		$search = \Aimeos\MShop::create( $this->context, 'catalog' )->createSearch();
+		$search = \Aimeos\MShop::create($this->context, 'catalog')->createSearch();
 
-		$this->stub->expects( $this->once() )->method( 'createFilter' )
-			->will( $this->returnValue( $search ) );
+		$this->stub->expects($this->once())->method('createFilter')
+			->will($this->returnValue($search));
 
-		$this->assertInstanceOf( \Aimeos\MW\Criteria\Iface::class, $this->object->createFilter() );
+		$this->assertInstanceOf(\Aimeos\MW\Criteria\Iface::class, $this->object->createFilter());
 	}
 
 
 	public function testGetPath()
 	{
-		$this->stub->expects( $this->once() )->method( 'getPath' )
-			->will( $this->returnValue( [] ) );
+		$this->stub->expects($this->once())->method('getPath')
+			->will($this->returnValue([]));
 
-		$this->assertEquals( [], $this->object->getPath( -1 ) );
+		$this->assertEquals([], $this->object->getPath( -1 ));
 	}
 
 
 	public function testGetTree()
 	{
-		$catItem = \Aimeos\MShop::create( $this->context, 'catalog' )->createItem();
+		$catItem = \Aimeos\MShop::create($this->context, 'catalog')->createItem();
 
-		$this->stub->expects( $this->once() )->method( 'getTree' )
-			->will( $this->returnValue( $catItem ) );
+		$this->stub->expects($this->once())->method('getTree')
+			->will($this->returnValue($catItem));
 
-		$this->assertInstanceOf( \Aimeos\MShop\Catalog\Item\Iface::class, $this->object->getTree() );
+		$this->assertInstanceOf(\Aimeos\MShop\Catalog\Item\Iface::class, $this->object->getTree());
 	}
 
 
 	public function testGetController()
 	{
-		$result = $this->access( 'getController' )->invokeArgs( $this->object, [] );
+		$result = $this->access('getController')->invokeArgs($this->object, []);
 
-		$this->assertSame( $this->stub, $result );
+		$this->assertSame($this->stub, $result);
 	}
 
 
-	protected function access( $name )
+	protected function access($name)
 	{
-		$class = new \ReflectionClass( \Aimeos\Controller\Frontend\Catalog\Decorator\Base::class );
-		$method = $class->getMethod( $name );
-		$method->setAccessible( true );
+		$class = new \ReflectionClass(\Aimeos\Controller\Frontend\Catalog\Decorator\Base::class);
+		$method = $class->getMethod($name);
+		$method->setAccessible(true);
 
 		return $method;
 	}

Please login to merge, or discard this patch.

controller/frontend/tests/Controller/Frontend/Catalog/StandardTest.php 1 patch

Spacing +11 added lines, -11 removed lines patch added patch discarded remove patch

@@ -19,13 +19,13 @@  discard block
 block discarded – undo
 	protected function setUp()
 	{
 		$this->context = \TestHelperFrontend::getContext();
-		$this->object = new \Aimeos\Controller\Frontend\Catalog\Standard( $this->context );
+		$this->object = new \Aimeos\Controller\Frontend\Catalog\Standard($this->context);
 	}
 
 
 	protected function tearDown()
 	{
-		unset( $this->object, $this->context );
+		unset($this->object, $this->context);
 	}
 
 
@@ -33,19 +33,19 @@  discard block
 block discarded – undo
 	{
 		$filter = $this->object->createFilter();
 
-		$this->assertInstanceOf( '\\Aimeos\\MW\\Criteria\\Iface', $filter );
+		$this->assertInstanceOf('\\Aimeos\\MW\\Criteria\\Iface', $filter);
 	}
 
 
 	public function testGetPath()
 	{
-		$manager = \Aimeos\MShop::create( $this->context, 'catalog' );
-		$items = $this->object->getPath( $manager->findItem( 'cafe' )->getId() );
+		$manager = \Aimeos\MShop::create($this->context, 'catalog');
+		$items = $this->object->getPath($manager->findItem('cafe')->getId());
 
-		$this->assertEquals( 3, count( $items ) );
+		$this->assertEquals(3, count($items));
 
-		foreach( $items as $item ) {
-			$this->assertInstanceOf( '\\Aimeos\\MShop\\Catalog\\Item\\Iface', $item );
+		foreach ($items as $item) {
+			$this->assertInstanceOf('\\Aimeos\\MShop\\Catalog\\Item\\Iface', $item);
 		}
 	}
 
@@ -54,10 +54,10 @@  discard block
 block discarded – undo
 	{
 		$tree = $this->object->getTree();
 
-		$this->assertEquals( 2, count( $tree->getChildren() ) );
+		$this->assertEquals(2, count($tree->getChildren()));
 
-		foreach( $tree->getChildren() as $item ) {
-			$this->assertInstanceOf( '\\Aimeos\\MShop\\Catalog\\Item\\Iface', $item );
+		foreach ($tree->getChildren() as $item) {
+			$this->assertInstanceOf('\\Aimeos\\MShop\\Catalog\\Item\\Iface', $item);
 		}
 	}
 }

Please login to merge, or discard this patch.

controller/frontend/tests/Controller/Frontend/Stock/Decorator/BaseTest.php 1 patch

Spacing +40 added lines, -40 removed lines patch added patch discarded remove patch

@@ -20,119 +20,119 @@
 block discarded – undo
 	{
 		$this->context = \TestHelperFrontend::getContext();
 
-		$this->stub = $this->getMockBuilder( \Aimeos\Controller\Frontend\Stock\Standard::class )
+		$this->stub = $this->getMockBuilder(\Aimeos\Controller\Frontend\Stock\Standard::class)
 			->disableOriginalConstructor()
 			->getMock();
 
-		$this->object = $this->getMockBuilder( \Aimeos\Controller\Frontend\Stock\Decorator\Base::class )
-			->setConstructorArgs( [$this->stub, $this->context] )
+		$this->object = $this->getMockBuilder(\Aimeos\Controller\Frontend\Stock\Decorator\Base::class)
+			->setConstructorArgs([$this->stub, $this->context])
 			->getMockForAbstractClass();
 	}
 
 
 	protected function tearDown()
 	{
-		unset( $this->context, $this->object, $this->stub );
+		unset($this->context, $this->object, $this->stub);
 	}
 
 
 	public function testConstructException()
 	{
-		$stub = $this->getMockBuilder( \Aimeos\Controller\Frontend\Iface::class )->getMock();
+		$stub = $this->getMockBuilder(\Aimeos\Controller\Frontend\Iface::class)->getMock();
 
-		$this->setExpectedException( \Aimeos\MW\Common\Exception::class );
+		$this->setExpectedException(\Aimeos\MW\Common\Exception::class);
 
-		$this->getMockBuilder( \Aimeos\Controller\Frontend\Stock\Decorator\Base::class )
-			->setConstructorArgs( [$stub, $this->context] )
+		$this->getMockBuilder(\Aimeos\Controller\Frontend\Stock\Decorator\Base::class)
+			->setConstructorArgs([$stub, $this->context])
 			->getMockForAbstractClass();
 	}
 
 
 	public function testCall()
 	{
-		$stub = $this->getMockBuilder( \Aimeos\Controller\Frontend\Stock\Standard::class )
+		$stub = $this->getMockBuilder(\Aimeos\Controller\Frontend\Stock\Standard::class)
 			->disableOriginalConstructor()
-			->setMethods( ['invalid'] )
+			->setMethods(['invalid'])
 			->getMock();
 
-		$object = $this->getMockBuilder( \Aimeos\Controller\Frontend\Stock\Decorator\Base::class )
-			->setConstructorArgs( [$stub, $this->context] )
+		$object = $this->getMockBuilder(\Aimeos\Controller\Frontend\Stock\Decorator\Base::class)
+			->setConstructorArgs([$stub, $this->context])
 			->getMockForAbstractClass();
 
-		$stub->expects( $this->once() )->method( 'invalid' )->will( $this->returnValue( true ) );
+		$stub->expects($this->once())->method('invalid')->will($this->returnValue(true));
 
-		$this->assertTrue( $object->invalid() );
+		$this->assertTrue($object->invalid());
 	}
 
 
 	public function testAddFilterCodes()
 	{
-		$search = \Aimeos\MShop::create( $this->context, 'stock' )->createSearch();
+		$search = \Aimeos\MShop::create($this->context, 'stock')->createSearch();
 
-		$this->stub->expects( $this->once() )->method( 'addFilterCodes' )
-			->will( $this->returnArgument( 0 ) );
+		$this->stub->expects($this->once())->method('addFilterCodes')
+			->will($this->returnArgument(0));
 
-		$this->assertInstanceOf( \Aimeos\MW\Criteria\Iface::class, $this->object->addFilterCodes( $search, [] ) );
+		$this->assertInstanceOf(\Aimeos\MW\Criteria\Iface::class, $this->object->addFilterCodes($search, []));
 	}
 
 
 	public function testAddFilterTypes()
 	{
-		$search = \Aimeos\MShop::create( $this->context, 'stock' )->createSearch();
+		$search = \Aimeos\MShop::create($this->context, 'stock')->createSearch();
 
-		$this->stub->expects( $this->once() )->method( 'addFilterTypes' )
-			->will( $this->returnArgument( 0 ) );
+		$this->stub->expects($this->once())->method('addFilterTypes')
+			->will($this->returnArgument(0));
 
-		$this->assertInstanceOf( \Aimeos\MW\Criteria\Iface::class, $this->object->addFilterTypes( $search, [] ) );
+		$this->assertInstanceOf(\Aimeos\MW\Criteria\Iface::class, $this->object->addFilterTypes($search, []));
 	}
 
 
 	public function testCreateFilter()
 	{
-		$search = \Aimeos\MShop::create( $this->context, 'stock' )->createSearch();
+		$search = \Aimeos\MShop::create($this->context, 'stock')->createSearch();
 
-		$this->stub->expects( $this->once() )->method( 'createFilter' )
-			->will( $this->returnValue( $search ) );
+		$this->stub->expects($this->once())->method('createFilter')
+			->will($this->returnValue($search));
 
-		$this->assertInstanceOf( \Aimeos\MW\Criteria\Iface::class, $this->object->createFilter() );
+		$this->assertInstanceOf(\Aimeos\MW\Criteria\Iface::class, $this->object->createFilter());
 	}
 
 
 	public function testGetItem()
 	{
-		$item = \Aimeos\MShop::create( $this->context, 'stock' )->createItem();
+		$item = \Aimeos\MShop::create($this->context, 'stock')->createItem();
 
-		$this->stub->expects( $this->once() )->method( 'getItem' )
-			->will( $this->returnValue( $item ) );
+		$this->stub->expects($this->once())->method('getItem')
+			->will($this->returnValue($item));
 
-		$this->assertInstanceOf( \Aimeos\MShop\Stock\Item\Iface::class, $this->object->getItem( -1 ) );
+		$this->assertInstanceOf(\Aimeos\MShop\Stock\Item\Iface::class, $this->object->getItem( -1 ));
 	}
 
 
 	public function testSearchItems()
 	{
-		$filter = \Aimeos\MShop::create( $this->context, 'stock' )->createSearch();
+		$filter = \Aimeos\MShop::create($this->context, 'stock')->createSearch();
 
-		$this->stub->expects( $this->once() )->method( 'searchItems' )
-			->will( $this->returnValue( [] ) );
+		$this->stub->expects($this->once())->method('searchItems')
+			->will($this->returnValue([]));
 
-		$this->assertEquals( [], $this->object->searchItems( $filter ) );
+		$this->assertEquals([], $this->object->searchItems($filter));
 	}
 
 
 	public function testGetController()
 	{
-		$result = $this->access( 'getController' )->invokeArgs( $this->object, [] );
+		$result = $this->access('getController')->invokeArgs($this->object, []);
 
-		$this->assertSame( $this->stub, $result );
+		$this->assertSame($this->stub, $result);
 	}
 
 
-	protected function access( $name )
+	protected function access($name)
 	{
-		$class = new \ReflectionClass( \Aimeos\Controller\Frontend\Stock\Decorator\Base::class );
-		$method = $class->getMethod( $name );
-		$method->setAccessible( true );
+		$class = new \ReflectionClass(\Aimeos\Controller\Frontend\Stock\Decorator\Base::class);
+		$method = $class->getMethod($name);
+		$method->setAccessible(true);
 
 		return $method;
 	}

Please login to merge, or discard this patch.

controller/frontend/tests/Controller/Frontend/Stock/StandardTest.php 1 patch

Spacing +23 added lines, -23 removed lines patch added patch discarded remove patch

@@ -16,13 +16,13 @@  discard block
 block discarded – undo
 
 	protected function setUp()
 	{
-		$this->object = new \Aimeos\Controller\Frontend\Stock\Standard( \TestHelperFrontend::getContext() );
+		$this->object = new \Aimeos\Controller\Frontend\Stock\Standard(\TestHelperFrontend::getContext());
 	}
 
 
 	protected function tearDown()
 	{
-		unset( $this->object );
+		unset($this->object);
 	}
 
 
@@ -30,65 +30,65 @@  discard block
 block discarded – undo
 	{
 		$filter = $this->object->createFilter();
 
-		$this->assertInstanceOf( '\\Aimeos\\MW\\Criteria\\Iface', $filter );
-		$this->assertEquals( 0, $filter->getSliceStart() );
-		$this->assertEquals( 100, $filter->getSliceSize() );
+		$this->assertInstanceOf('\\Aimeos\\MW\\Criteria\\Iface', $filter);
+		$this->assertEquals(0, $filter->getSliceStart());
+		$this->assertEquals(100, $filter->getSliceSize());
 	}
 
 
 	public function testAddFilterCodes()
 	{
 		$filter = $this->object->createFilter();
-		$filter = $this->object->addFilterCodes( $filter, ['CNC', 'CNE'] );
+		$filter = $this->object->addFilterCodes($filter, ['CNC', 'CNE']);
 
 		$list = $filter->getConditions()->getExpressions();
 
-		if( !isset( $list[0] ) || !( $list[0] instanceof \Aimeos\MW\Criteria\Expression\Compare\Iface ) ) {
-			throw new \RuntimeException( 'Wrong expression' );
+		if (!isset($list[0]) || !($list[0] instanceof \Aimeos\MW\Criteria\Expression\Compare\Iface)) {
+			throw new \RuntimeException('Wrong expression');
 		}
 
-		$this->assertEquals( 'stock.productcode', $list[0]->getName() );
-		$this->assertEquals( 2, count( $list[0]->getValue() ) );
+		$this->assertEquals('stock.productcode', $list[0]->getName());
+		$this->assertEquals(2, count($list[0]->getValue()));
 	}
 
 
 	public function testAddFilterTypes()
 	{
 		$filter = $this->object->createFilter();
-		$filter = $this->object->addFilterTypes( $filter, ['default'] );
+		$filter = $this->object->addFilterTypes($filter, ['default']);
 
 		$list = $filter->getConditions()->getExpressions();
 
-		if( !isset( $list[0] ) || !( $list[0] instanceof \Aimeos\MW\Criteria\Expression\Compare\Iface ) ) {
-			throw new \RuntimeException( 'Wrong expression' );
+		if (!isset($list[0]) || !($list[0] instanceof \Aimeos\MW\Criteria\Expression\Compare\Iface)) {
+			throw new \RuntimeException('Wrong expression');
 		}
 
-		$this->assertEquals( 'stock.type', $list[0]->getName() );
-		$this->assertEquals( 1, count( $list[0]->getValue() ) );
+		$this->assertEquals('stock.type', $list[0]->getName());
+		$this->assertEquals(1, count($list[0]->getValue()));
 	}
 
 
 	public function testGetItem()
 	{
 		$context = \TestHelperFrontend::getContext();
-		$manager = \Aimeos\MShop::create( $context, 'stock' );
-		$id = $manager->findItem( 'CNC', [], 'product', 'default' )->getId();
+		$manager = \Aimeos\MShop::create($context, 'stock');
+		$id = $manager->findItem('CNC', [], 'product', 'default')->getId();
 
-		$result = $this->object->getItem( $id );
+		$result = $this->object->getItem($id);
 
-		$this->assertInstanceOf( \Aimeos\MShop\Stock\Item\Iface::class, $result );
+		$this->assertInstanceOf(\Aimeos\MShop\Stock\Item\Iface::class, $result);
 	}
 
 
 	public function testSearchItems()
 	{
 		$filter = $this->object->createFilter();
-		$filter = $this->object->addFilterCodes( $filter, ['CNC', 'CNE'] );
+		$filter = $this->object->addFilterCodes($filter, ['CNC', 'CNE']);
 
 		$total = 0;
-		$results = $this->object->searchItems( $filter, $total );
+		$results = $this->object->searchItems($filter, $total);
 
-		$this->assertEquals( 2, $total );
-		$this->assertEquals( 2, count( $results ) );
+		$this->assertEquals(2, $total);
+		$this->assertEquals(2, count($results));
 	}
 }

Please login to merge, or discard this patch.

frontend/tests/Controller/Frontend/Supplier/Decorator/BaseTest.php 1 patch

Spacing +35 added lines, -35 removed lines patch added patch discarded remove patch

@@ -20,106 +20,106 @@
 block discarded – undo
 	{
 		$this->context = \TestHelperFrontend::getContext();
 
-		$this->stub = $this->getMockBuilder( \Aimeos\Controller\Frontend\Supplier\Standard::class )
+		$this->stub = $this->getMockBuilder(\Aimeos\Controller\Frontend\Supplier\Standard::class)
 			->disableOriginalConstructor()
 			->getMock();
 
-		$this->object = $this->getMockBuilder( \Aimeos\Controller\Frontend\Supplier\Decorator\Base::class )
-			->setConstructorArgs( [$this->stub, $this->context] )
+		$this->object = $this->getMockBuilder(\Aimeos\Controller\Frontend\Supplier\Decorator\Base::class)
+			->setConstructorArgs([$this->stub, $this->context])
 			->getMockForAbstractClass();
 	}
 
 
 	protected function tearDown()
 	{
-		unset( $this->context, $this->object, $this->stub );
+		unset($this->context, $this->object, $this->stub);
 	}
 
 
 	public function testConstructException()
 	{
-		$stub = $this->getMockBuilder( \Aimeos\Controller\Frontend\Iface::class )->getMock();
+		$stub = $this->getMockBuilder(\Aimeos\Controller\Frontend\Iface::class)->getMock();
 
-		$this->setExpectedException( \Aimeos\MW\Common\Exception::class );
+		$this->setExpectedException(\Aimeos\MW\Common\Exception::class);
 
-		$this->getMockBuilder( \Aimeos\Controller\Frontend\Supplier\Decorator\Base::class )
-			->setConstructorArgs( [$stub, $this->context] )
+		$this->getMockBuilder(\Aimeos\Controller\Frontend\Supplier\Decorator\Base::class)
+			->setConstructorArgs([$stub, $this->context])
 			->getMockForAbstractClass();
 	}
 
 
 	public function testCall()
 	{
-		$stub = $this->getMockBuilder( \Aimeos\Controller\Frontend\Supplier\Standard::class )
+		$stub = $this->getMockBuilder(\Aimeos\Controller\Frontend\Supplier\Standard::class)
 			->disableOriginalConstructor()
-			->setMethods( ['invalid'] )
+			->setMethods(['invalid'])
 			->getMock();
 
-		$object = $this->getMockBuilder( \Aimeos\Controller\Frontend\Supplier\Decorator\Base::class )
-			->setConstructorArgs( [$stub, $this->context] )
+		$object = $this->getMockBuilder(\Aimeos\Controller\Frontend\Supplier\Decorator\Base::class)
+			->setConstructorArgs([$stub, $this->context])
 			->getMockForAbstractClass();
 
-		$stub->expects( $this->once() )->method( 'invalid' )->will( $this->returnValue( true ) );
+		$stub->expects($this->once())->method('invalid')->will($this->returnValue(true));
 
-		$this->assertTrue( $object->invalid() );
+		$this->assertTrue($object->invalid());
 	}
 
 
 	public function testCreateFilter()
 	{
-		$search = \Aimeos\MShop::create( $this->context, 'supplier' )->createSearch();
+		$search = \Aimeos\MShop::create($this->context, 'supplier')->createSearch();
 
-		$this->stub->expects( $this->once() )->method( 'createFilter' )
-			->will( $this->returnValue( $search ) );
+		$this->stub->expects($this->once())->method('createFilter')
+			->will($this->returnValue($search));
 
-		$this->assertInstanceOf( \Aimeos\MW\Criteria\Iface::class, $this->object->createFilter() );
+		$this->assertInstanceOf(\Aimeos\MW\Criteria\Iface::class, $this->object->createFilter());
 	}
 
 
 	public function testGetItem()
 	{
-		$item = \Aimeos\MShop::create( $this->context, 'supplier' )->createItem();
+		$item = \Aimeos\MShop::create($this->context, 'supplier')->createItem();
 
-		$this->stub->expects( $this->once() )->method( 'getItem' )
-			->will( $this->returnValue( $item ) );
+		$this->stub->expects($this->once())->method('getItem')
+			->will($this->returnValue($item));
 
-		$this->assertInstanceOf( \Aimeos\MShop\Supplier\Item\Iface::class, $this->object->getItem( -1 ) );
+		$this->assertInstanceOf(\Aimeos\MShop\Supplier\Item\Iface::class, $this->object->getItem( -1 ));
 	}
 
 
 	public function testGetItems()
 	{
-		$this->stub->expects( $this->once() )->method( 'getItems' )
-			->will( $this->returnValue( [] ) );
+		$this->stub->expects($this->once())->method('getItems')
+			->will($this->returnValue([]));
 
-		$this->assertEquals( [], $this->object->getItems( [-1], ['media'] ) );
+		$this->assertEquals([], $this->object->getItems([-1], ['media']));
 	}
 
 
 	public function testSearchItems()
 	{
-		$filter = \Aimeos\MShop::create( $this->context, 'supplier' )->createSearch();
+		$filter = \Aimeos\MShop::create($this->context, 'supplier')->createSearch();
 
-		$this->stub->expects( $this->once() )->method( 'searchItems' )
-			->will( $this->returnValue( [] ) );
+		$this->stub->expects($this->once())->method('searchItems')
+			->will($this->returnValue([]));
 
-		$this->assertEquals( [], $this->object->searchItems( $filter, ['media'] ) );
+		$this->assertEquals([], $this->object->searchItems($filter, ['media']));
 	}
 
 
 	public function testGetController()
 	{
-		$result = $this->access( 'getController' )->invokeArgs( $this->object, [] );
+		$result = $this->access('getController')->invokeArgs($this->object, []);
 
-		$this->assertSame( $this->stub, $result );
+		$this->assertSame($this->stub, $result);
 	}
 
 
-	protected function access( $name )
+	protected function access($name)
 	{
-		$class = new \ReflectionClass( \Aimeos\Controller\Frontend\Supplier\Decorator\Base::class );
-		$method = $class->getMethod( $name );
-		$method->setAccessible( true );
+		$class = new \ReflectionClass(\Aimeos\Controller\Frontend\Supplier\Decorator\Base::class);
+		$method = $class->getMethod($name);
+		$method->setAccessible(true);
 
 		return $method;
 	}

Please login to merge, or discard this patch.

controller/frontend/tests/Controller/Frontend/Supplier/StandardTest.php 1 patch

Spacing +17 added lines, -17 removed lines patch added patch discarded remove patch

@@ -16,13 +16,13 @@  discard block
 block discarded – undo
 
 	protected function setUp()
 	{
-		$this->object = new \Aimeos\Controller\Frontend\Supplier\Standard( \TestHelperFrontend::getContext() );
+		$this->object = new \Aimeos\Controller\Frontend\Supplier\Standard(\TestHelperFrontend::getContext());
 	}
 
 
 	protected function tearDown()
 	{
-		unset( $this->object );
+		unset($this->object);
 	}
 
 
@@ -30,34 +30,34 @@  discard block
 block discarded – undo
 	{
 		$filter = $this->object->createFilter();
 
-		$this->assertInstanceOf( '\\Aimeos\\MW\\Criteria\\Iface', $filter );
-		$this->assertEquals( 0, $filter->getSliceStart() );
-		$this->assertEquals( 100, $filter->getSliceSize() );
+		$this->assertInstanceOf('\\Aimeos\\MW\\Criteria\\Iface', $filter);
+		$this->assertEquals(0, $filter->getSliceStart());
+		$this->assertEquals(100, $filter->getSliceSize());
 	}
 
 
 	public function testGetItem()
 	{
 		$context = \TestHelperFrontend::getContext();
-		$manager = \Aimeos\MShop::create( $context, 'supplier' );
-		$id = $manager->findItem( 'unitCode001' )->getId();
+		$manager = \Aimeos\MShop::create($context, 'supplier');
+		$id = $manager->findItem('unitCode001')->getId();
 
-		$result = $this->object->getItem( $id );
+		$result = $this->object->getItem($id);
 
-		$this->assertInstanceOf( \Aimeos\MShop\Supplier\Item\Iface::class, $result );
+		$this->assertInstanceOf(\Aimeos\MShop\Supplier\Item\Iface::class, $result);
 	}
 
 
 	public function testGetItems()
 	{
 		$context = \TestHelperFrontend::getContext();
-		$manager = \Aimeos\MShop::create( $context, 'supplier' );
-		$id = $manager->findItem( 'unitCode001' )->getId();
+		$manager = \Aimeos\MShop::create($context, 'supplier');
+		$id = $manager->findItem('unitCode001')->getId();
 
-		$result = $this->object->getItems( [$id] );
+		$result = $this->object->getItems([$id]);
 
-		$this->assertInternalType( 'array', $result );
-		$this->assertEquals( 1, count( $result ) );
+		$this->assertInternalType('array', $result);
+		$this->assertEquals(1, count($result));
 	}
 
 
@@ -66,9 +66,9 @@  discard block
 block discarded – undo
 		$filter = $this->object->createFilter();
 
 		$total = 0;
-		$results = $this->object->searchItems( $filter, [], $total );
+		$results = $this->object->searchItems($filter, [], $total);
 
-		$this->assertGreaterThanOrEqual( 2, $total );
-		$this->assertGreaterThanOrEqual( 2, count( $results ) );
+		$this->assertGreaterThanOrEqual(2, $total);
+		$this->assertGreaterThanOrEqual(2, count($results));
 	}
 }

Please login to merge, or discard this patch.

		@@ -20,95 +20,95 @@
		block discarded – undo
20	20	{
21	21	$this->context = \TestHelperFrontend::getContext();
22	22
23		- $this->stub = $this->getMockBuilder( \Aimeos\Controller\Frontend\Catalog\Standard::class )
	23	+ $this->stub = $this->getMockBuilder(\Aimeos\Controller\Frontend\Catalog\Standard::class)
24	24	->disableOriginalConstructor()
25	25	->getMock();
26	26
27		- $this->object = $this->getMockBuilder( \Aimeos\Controller\Frontend\Catalog\Decorator\Base::class )
28		- ->setConstructorArgs( [$this->stub, $this->context] )
	27	+ $this->object = $this->getMockBuilder(\Aimeos\Controller\Frontend\Catalog\Decorator\Base::class)
	28	+ ->setConstructorArgs([$this->stub, $this->context])
29	29	->getMockForAbstractClass();
30	30	}
31	31
32	32
33	33	protected function tearDown()
34	34	{
35		- unset( $this->context, $this->object, $this->stub );
	35	+ unset($this->context, $this->object, $this->stub);
36	36	}
37	37
38	38
39	39	public function testConstructException()
40	40	{
41		- $stub = $this->getMockBuilder( \Aimeos\Controller\Frontend\Iface::class )->getMock();
	41	+ $stub = $this->getMockBuilder(\Aimeos\Controller\Frontend\Iface::class)->getMock();
42	42
43		- $this->setExpectedException( \Aimeos\MW\Common\Exception::class );
	43	+ $this->setExpectedException(\Aimeos\MW\Common\Exception::class);
44	44
45		- $this->getMockBuilder( \Aimeos\Controller\Frontend\Catalog\Decorator\Base::class )
46		- ->setConstructorArgs( [$stub, $this->context] )
	45	+ $this->getMockBuilder(\Aimeos\Controller\Frontend\Catalog\Decorator\Base::class)
	46	+ ->setConstructorArgs([$stub, $this->context])
47	47	->getMockForAbstractClass();
48	48	}
49	49
50	50
51	51	public function testCall()
52	52	{
53		- $stub = $this->getMockBuilder( \Aimeos\Controller\Frontend\Catalog\Standard::class )
	53	+ $stub = $this->getMockBuilder(\Aimeos\Controller\Frontend\Catalog\Standard::class)
54	54	->disableOriginalConstructor()
55		- ->setMethods( ['invalid'] )
	55	+ ->setMethods(['invalid'])
56	56	->getMock();
57	57
58		- $object = $this->getMockBuilder( \Aimeos\Controller\Frontend\Catalog\Decorator\Base::class )
59		- ->setConstructorArgs( [$stub, $this->context] )
	58	+ $object = $this->getMockBuilder(\Aimeos\Controller\Frontend\Catalog\Decorator\Base::class)
	59	+ ->setConstructorArgs([$stub, $this->context])
60	60	->getMockForAbstractClass();
61	61
62		- $stub->expects( $this->once() )->method( 'invalid' )->will( $this->returnValue( true ) );
	62	+ $stub->expects($this->once())->method('invalid')->will($this->returnValue(true));
63	63
64		- $this->assertTrue( $object->invalid() );
	64	+ $this->assertTrue($object->invalid());
65	65	}
66	66
67	67
68	68	public function testCreateFilter()
69	69	{
70		- $search = \Aimeos\MShop::create( $this->context, 'catalog' )->createSearch();
	70	+ $search = \Aimeos\MShop::create($this->context, 'catalog')->createSearch();
71	71
72		- $this->stub->expects( $this->once() )->method( 'createFilter' )
73		- ->will( $this->returnValue( $search ) );
	72	+ $this->stub->expects($this->once())->method('createFilter')
	73	+ ->will($this->returnValue($search));
74	74
75		- $this->assertInstanceOf( \Aimeos\MW\Criteria\Iface::class, $this->object->createFilter() );
	75	+ $this->assertInstanceOf(\Aimeos\MW\Criteria\Iface::class, $this->object->createFilter());
76	76	}
77	77
78	78
79	79	public function testGetPath()
80	80	{
81		- $this->stub->expects( $this->once() )->method( 'getPath' )
82		- ->will( $this->returnValue( [] ) );
	81	+ $this->stub->expects($this->once())->method('getPath')
	82	+ ->will($this->returnValue([]));
83	83
84		- $this->assertEquals( [], $this->object->getPath( -1 ) );
	84	+ $this->assertEquals([], $this->object->getPath( -1 ));
85	85	}
86	86
87	87
88	88	public function testGetTree()
89	89	{
90		- $catItem = \Aimeos\MShop::create( $this->context, 'catalog' )->createItem();
	90	+ $catItem = \Aimeos\MShop::create($this->context, 'catalog')->createItem();
91	91
92		- $this->stub->expects( $this->once() )->method( 'getTree' )
93		- ->will( $this->returnValue( $catItem ) );
	92	+ $this->stub->expects($this->once())->method('getTree')
	93	+ ->will($this->returnValue($catItem));
94	94
95		- $this->assertInstanceOf( \Aimeos\MShop\Catalog\Item\Iface::class, $this->object->getTree() );
	95	+ $this->assertInstanceOf(\Aimeos\MShop\Catalog\Item\Iface::class, $this->object->getTree());
96	96	}
97	97
98	98
99	99	public function testGetController()
100	100	{
101		- $result = $this->access( 'getController' )->invokeArgs( $this->object, [] );
	101	+ $result = $this->access('getController')->invokeArgs($this->object, []);
102	102
103		- $this->assertSame( $this->stub, $result );
	103	+ $this->assertSame($this->stub, $result);
104	104	}
105	105
106	106
107		- protected function access( $name )
	107	+ protected function access($name)
108	108	{
109		- $class = new \ReflectionClass( \Aimeos\Controller\Frontend\Catalog\Decorator\Base::class );
110		- $method = $class->getMethod( $name );
111		- $method->setAccessible( true );
	109	+ $class = new \ReflectionClass(\Aimeos\Controller\Frontend\Catalog\Decorator\Base::class);
	110	+ $method = $class->getMethod($name);
	111	+ $method->setAccessible(true);
112	112
113	113	return $method;
114	114	}

		@@ -20,119 +20,119 @@
		block discarded – undo
20	20	{
21	21	$this->context = \TestHelperFrontend::getContext();
22	22
23		- $this->stub = $this->getMockBuilder( \Aimeos\Controller\Frontend\Stock\Standard::class )
	23	+ $this->stub = $this->getMockBuilder(\Aimeos\Controller\Frontend\Stock\Standard::class)
24	24	->disableOriginalConstructor()
25	25	->getMock();
26	26
27		- $this->object = $this->getMockBuilder( \Aimeos\Controller\Frontend\Stock\Decorator\Base::class )
28		- ->setConstructorArgs( [$this->stub, $this->context] )
	27	+ $this->object = $this->getMockBuilder(\Aimeos\Controller\Frontend\Stock\Decorator\Base::class)
	28	+ ->setConstructorArgs([$this->stub, $this->context])
29	29	->getMockForAbstractClass();
30	30	}
31	31
32	32
33	33	protected function tearDown()
34	34	{
35		- unset( $this->context, $this->object, $this->stub );
	35	+ unset($this->context, $this->object, $this->stub);
36	36	}
37	37
38	38
39	39	public function testConstructException()
40	40	{
41		- $stub = $this->getMockBuilder( \Aimeos\Controller\Frontend\Iface::class )->getMock();
	41	+ $stub = $this->getMockBuilder(\Aimeos\Controller\Frontend\Iface::class)->getMock();
42	42
43		- $this->setExpectedException( \Aimeos\MW\Common\Exception::class );
	43	+ $this->setExpectedException(\Aimeos\MW\Common\Exception::class);
44	44
45		- $this->getMockBuilder( \Aimeos\Controller\Frontend\Stock\Decorator\Base::class )
46		- ->setConstructorArgs( [$stub, $this->context] )
	45	+ $this->getMockBuilder(\Aimeos\Controller\Frontend\Stock\Decorator\Base::class)
	46	+ ->setConstructorArgs([$stub, $this->context])
47	47	->getMockForAbstractClass();
48	48	}
49	49
50	50
51	51	public function testCall()
52	52	{
53		- $stub = $this->getMockBuilder( \Aimeos\Controller\Frontend\Stock\Standard::class )
	53	+ $stub = $this->getMockBuilder(\Aimeos\Controller\Frontend\Stock\Standard::class)
54	54	->disableOriginalConstructor()
55		- ->setMethods( ['invalid'] )
	55	+ ->setMethods(['invalid'])
56	56	->getMock();
57	57
58		- $object = $this->getMockBuilder( \Aimeos\Controller\Frontend\Stock\Decorator\Base::class )
59		- ->setConstructorArgs( [$stub, $this->context] )
	58	+ $object = $this->getMockBuilder(\Aimeos\Controller\Frontend\Stock\Decorator\Base::class)
	59	+ ->setConstructorArgs([$stub, $this->context])
60	60	->getMockForAbstractClass();
61	61
62		- $stub->expects( $this->once() )->method( 'invalid' )->will( $this->returnValue( true ) );
	62	+ $stub->expects($this->once())->method('invalid')->will($this->returnValue(true));
63	63
64		- $this->assertTrue( $object->invalid() );
	64	+ $this->assertTrue($object->invalid());
65	65	}
66	66
67	67
68	68	public function testAddFilterCodes()
69	69	{
70		- $search = \Aimeos\MShop::create( $this->context, 'stock' )->createSearch();
	70	+ $search = \Aimeos\MShop::create($this->context, 'stock')->createSearch();
71	71
72		- $this->stub->expects( $this->once() )->method( 'addFilterCodes' )
73		- ->will( $this->returnArgument( 0 ) );
	72	+ $this->stub->expects($this->once())->method('addFilterCodes')
	73	+ ->will($this->returnArgument(0));
74	74
75		- $this->assertInstanceOf( \Aimeos\MW\Criteria\Iface::class, $this->object->addFilterCodes( $search, [] ) );
	75	+ $this->assertInstanceOf(\Aimeos\MW\Criteria\Iface::class, $this->object->addFilterCodes($search, []));
76	76	}
77	77
78	78
79	79	public function testAddFilterTypes()
80	80	{
81		- $search = \Aimeos\MShop::create( $this->context, 'stock' )->createSearch();
	81	+ $search = \Aimeos\MShop::create($this->context, 'stock')->createSearch();
82	82
83		- $this->stub->expects( $this->once() )->method( 'addFilterTypes' )
84		- ->will( $this->returnArgument( 0 ) );
	83	+ $this->stub->expects($this->once())->method('addFilterTypes')
	84	+ ->will($this->returnArgument(0));
85	85
86		- $this->assertInstanceOf( \Aimeos\MW\Criteria\Iface::class, $this->object->addFilterTypes( $search, [] ) );
	86	+ $this->assertInstanceOf(\Aimeos\MW\Criteria\Iface::class, $this->object->addFilterTypes($search, []));
87	87	}
88	88
89	89
90	90	public function testCreateFilter()
91	91	{
92		- $search = \Aimeos\MShop::create( $this->context, 'stock' )->createSearch();
	92	+ $search = \Aimeos\MShop::create($this->context, 'stock')->createSearch();
93	93
94		- $this->stub->expects( $this->once() )->method( 'createFilter' )
95		- ->will( $this->returnValue( $search ) );
	94	+ $this->stub->expects($this->once())->method('createFilter')
	95	+ ->will($this->returnValue($search));
96	96
97		- $this->assertInstanceOf( \Aimeos\MW\Criteria\Iface::class, $this->object->createFilter() );
	97	+ $this->assertInstanceOf(\Aimeos\MW\Criteria\Iface::class, $this->object->createFilter());
98	98	}
99	99
100	100
101	101	public function testGetItem()
102	102	{
103		- $item = \Aimeos\MShop::create( $this->context, 'stock' )->createItem();
	103	+ $item = \Aimeos\MShop::create($this->context, 'stock')->createItem();
104	104
105		- $this->stub->expects( $this->once() )->method( 'getItem' )
106		- ->will( $this->returnValue( $item ) );
	105	+ $this->stub->expects($this->once())->method('getItem')
	106	+ ->will($this->returnValue($item));
107	107
108		- $this->assertInstanceOf( \Aimeos\MShop\Stock\Item\Iface::class, $this->object->getItem( -1 ) );
	108	+ $this->assertInstanceOf(\Aimeos\MShop\Stock\Item\Iface::class, $this->object->getItem( -1 ));
109	109	}
110	110
111	111
112	112	public function testSearchItems()
113	113	{
114		- $filter = \Aimeos\MShop::create( $this->context, 'stock' )->createSearch();
	114	+ $filter = \Aimeos\MShop::create($this->context, 'stock')->createSearch();
115	115
116		- $this->stub->expects( $this->once() )->method( 'searchItems' )
117		- ->will( $this->returnValue( [] ) );
	116	+ $this->stub->expects($this->once())->method('searchItems')
	117	+ ->will($this->returnValue([]));
118	118
119		- $this->assertEquals( [], $this->object->searchItems( $filter ) );
	119	+ $this->assertEquals([], $this->object->searchItems($filter));
120	120	}
121	121
122	122
123	123	public function testGetController()
124	124	{
125		- $result = $this->access( 'getController' )->invokeArgs( $this->object, [] );
	125	+ $result = $this->access('getController')->invokeArgs($this->object, []);
126	126
127		- $this->assertSame( $this->stub, $result );
	127	+ $this->assertSame($this->stub, $result);
128	128	}
129	129
130	130
131		- protected function access( $name )
	131	+ protected function access($name)
132	132	{
133		- $class = new \ReflectionClass( \Aimeos\Controller\Frontend\Stock\Decorator\Base::class );
134		- $method = $class->getMethod( $name );
135		- $method->setAccessible( true );
	133	+ $class = new \ReflectionClass(\Aimeos\Controller\Frontend\Stock\Decorator\Base::class);
	134	+ $method = $class->getMethod($name);
	135	+ $method->setAccessible(true);
136	136
137	137	return $method;
138	138	}

		@@ -20,106 +20,106 @@
		block discarded – undo
20	20	{
21	21	$this->context = \TestHelperFrontend::getContext();
22	22
23		- $this->stub = $this->getMockBuilder( \Aimeos\Controller\Frontend\Supplier\Standard::class )
	23	+ $this->stub = $this->getMockBuilder(\Aimeos\Controller\Frontend\Supplier\Standard::class)
24	24	->disableOriginalConstructor()
25	25	->getMock();
26	26
27		- $this->object = $this->getMockBuilder( \Aimeos\Controller\Frontend\Supplier\Decorator\Base::class )
28		- ->setConstructorArgs( [$this->stub, $this->context] )
	27	+ $this->object = $this->getMockBuilder(\Aimeos\Controller\Frontend\Supplier\Decorator\Base::class)
	28	+ ->setConstructorArgs([$this->stub, $this->context])
29	29	->getMockForAbstractClass();
30	30	}
31	31
32	32
33	33	protected function tearDown()
34	34	{
35		- unset( $this->context, $this->object, $this->stub );
	35	+ unset($this->context, $this->object, $this->stub);
36	36	}
37	37
38	38
39	39	public function testConstructException()
40	40	{
41		- $stub = $this->getMockBuilder( \Aimeos\Controller\Frontend\Iface::class )->getMock();
	41	+ $stub = $this->getMockBuilder(\Aimeos\Controller\Frontend\Iface::class)->getMock();
42	42
43		- $this->setExpectedException( \Aimeos\MW\Common\Exception::class );
	43	+ $this->setExpectedException(\Aimeos\MW\Common\Exception::class);
44	44
45		- $this->getMockBuilder( \Aimeos\Controller\Frontend\Supplier\Decorator\Base::class )
46		- ->setConstructorArgs( [$stub, $this->context] )
	45	+ $this->getMockBuilder(\Aimeos\Controller\Frontend\Supplier\Decorator\Base::class)
	46	+ ->setConstructorArgs([$stub, $this->context])
47	47	->getMockForAbstractClass();
48	48	}
49	49
50	50
51	51	public function testCall()
52	52	{
53		- $stub = $this->getMockBuilder( \Aimeos\Controller\Frontend\Supplier\Standard::class )
	53	+ $stub = $this->getMockBuilder(\Aimeos\Controller\Frontend\Supplier\Standard::class)
54	54	->disableOriginalConstructor()
55		- ->setMethods( ['invalid'] )
	55	+ ->setMethods(['invalid'])
56	56	->getMock();
57	57
58		- $object = $this->getMockBuilder( \Aimeos\Controller\Frontend\Supplier\Decorator\Base::class )
59		- ->setConstructorArgs( [$stub, $this->context] )
	58	+ $object = $this->getMockBuilder(\Aimeos\Controller\Frontend\Supplier\Decorator\Base::class)
	59	+ ->setConstructorArgs([$stub, $this->context])
60	60	->getMockForAbstractClass();
61	61
62		- $stub->expects( $this->once() )->method( 'invalid' )->will( $this->returnValue( true ) );
	62	+ $stub->expects($this->once())->method('invalid')->will($this->returnValue(true));
63	63
64		- $this->assertTrue( $object->invalid() );
	64	+ $this->assertTrue($object->invalid());
65	65	}
66	66
67	67
68	68	public function testCreateFilter()
69	69	{
70		- $search = \Aimeos\MShop::create( $this->context, 'supplier' )->createSearch();
	70	+ $search = \Aimeos\MShop::create($this->context, 'supplier')->createSearch();
71	71
72		- $this->stub->expects( $this->once() )->method( 'createFilter' )
73		- ->will( $this->returnValue( $search ) );
	72	+ $this->stub->expects($this->once())->method('createFilter')
	73	+ ->will($this->returnValue($search));
74	74
75		- $this->assertInstanceOf( \Aimeos\MW\Criteria\Iface::class, $this->object->createFilter() );
	75	+ $this->assertInstanceOf(\Aimeos\MW\Criteria\Iface::class, $this->object->createFilter());
76	76	}
77	77
78	78
79	79	public function testGetItem()
80	80	{
81		- $item = \Aimeos\MShop::create( $this->context, 'supplier' )->createItem();
	81	+ $item = \Aimeos\MShop::create($this->context, 'supplier')->createItem();
82	82
83		- $this->stub->expects( $this->once() )->method( 'getItem' )
84		- ->will( $this->returnValue( $item ) );
	83	+ $this->stub->expects($this->once())->method('getItem')
	84	+ ->will($this->returnValue($item));
85	85
86		- $this->assertInstanceOf( \Aimeos\MShop\Supplier\Item\Iface::class, $this->object->getItem( -1 ) );
	86	+ $this->assertInstanceOf(\Aimeos\MShop\Supplier\Item\Iface::class, $this->object->getItem( -1 ));
87	87	}
88	88
89	89
90	90	public function testGetItems()
91	91	{
92		- $this->stub->expects( $this->once() )->method( 'getItems' )
93		- ->will( $this->returnValue( [] ) );
	92	+ $this->stub->expects($this->once())->method('getItems')
	93	+ ->will($this->returnValue([]));
94	94
95		- $this->assertEquals( [], $this->object->getItems( [-1], ['media'] ) );
	95	+ $this->assertEquals([], $this->object->getItems([-1], ['media']));
96	96	}
97	97
98	98
99	99	public function testSearchItems()
100	100	{
101		- $filter = \Aimeos\MShop::create( $this->context, 'supplier' )->createSearch();
	101	+ $filter = \Aimeos\MShop::create($this->context, 'supplier')->createSearch();
102	102
103		- $this->stub->expects( $this->once() )->method( 'searchItems' )
104		- ->will( $this->returnValue( [] ) );
	103	+ $this->stub->expects($this->once())->method('searchItems')
	104	+ ->will($this->returnValue([]));
105	105
106		- $this->assertEquals( [], $this->object->searchItems( $filter, ['media'] ) );
	106	+ $this->assertEquals([], $this->object->searchItems($filter, ['media']));
107	107	}
108	108
109	109
110	110	public function testGetController()
111	111	{
112		- $result = $this->access( 'getController' )->invokeArgs( $this->object, [] );
	112	+ $result = $this->access('getController')->invokeArgs($this->object, []);
113	113
114		- $this->assertSame( $this->stub, $result );
	114	+ $this->assertSame($this->stub, $result);
115	115	}
116	116
117	117
118		- protected function access( $name )
	118	+ protected function access($name)
119	119	{
120		- $class = new \ReflectionClass( \Aimeos\Controller\Frontend\Supplier\Decorator\Base::class );
121		- $method = $class->getMethod( $name );
122		- $method->setAccessible( true );
	120	+ $class = new \ReflectionClass(\Aimeos\Controller\Frontend\Supplier\Decorator\Base::class);
	121	+ $method = $class->getMethod($name);
	122	+ $method->setAccessible(true);
123	123
124	124	return $method;
125	125	}

		@@ -20,54 +20,54 @@ discard block
		block discarded – undo
20	20	{
21	21	$this->context = \TestHelperFrontend::getContext();
22	22
23		- $this->stub = $this->getMockBuilder( \Aimeos\Controller\Frontend\Basket\Standard::class )
	23	+ $this->stub = $this->getMockBuilder(\Aimeos\Controller\Frontend\Basket\Standard::class)
24	24	->disableOriginalConstructor()
25	25	->getMock();
26	26
27		- $this->object = $this->getMockBuilder( \Aimeos\Controller\Frontend\Basket\Decorator\Base::class )
28		- ->setConstructorArgs( [$this->stub, $this->context] )
	27	+ $this->object = $this->getMockBuilder(\Aimeos\Controller\Frontend\Basket\Decorator\Base::class)
	28	+ ->setConstructorArgs([$this->stub, $this->context])
29	29	->getMockForAbstractClass();
30	30	}
31	31
32	32
33	33	protected function tearDown()
34	34	{
35		- unset( $this->context, $this->object, $this->stub );
	35	+ unset($this->context, $this->object, $this->stub);
36	36	}
37	37
38	38
39	39	public function testConstructException()
40	40	{
41		- $stub = $this->getMockBuilder( \Aimeos\Controller\Frontend\Iface::class )->getMock();
	41	+ $stub = $this->getMockBuilder(\Aimeos\Controller\Frontend\Iface::class)->getMock();
42	42
43		- $this->setExpectedException( \Aimeos\MW\Common\Exception::class );
	43	+ $this->setExpectedException(\Aimeos\MW\Common\Exception::class);
44	44
45		- $this->getMockBuilder( \Aimeos\Controller\Frontend\Basket\Decorator\Base::class )
46		- ->setConstructorArgs( [$stub, $this->context] )
	45	+ $this->getMockBuilder(\Aimeos\Controller\Frontend\Basket\Decorator\Base::class)
	46	+ ->setConstructorArgs([$stub, $this->context])
47	47	->getMockForAbstractClass();
48	48	}
49	49
50	50
51	51	public function testCall()
52	52	{
53		- $stub = $this->getMockBuilder( \Aimeos\Controller\Frontend\Basket\Standard::class )
	53	+ $stub = $this->getMockBuilder(\Aimeos\Controller\Frontend\Basket\Standard::class)
54	54	->disableOriginalConstructor()
55		- ->setMethods( ['invalid'] )
	55	+ ->setMethods(['invalid'])
56	56	->getMock();
57	57
58		- $object = $this->getMockBuilder( \Aimeos\Controller\Frontend\Basket\Decorator\Base::class )
59		- ->setConstructorArgs( [$stub, $this->context] )
	58	+ $object = $this->getMockBuilder(\Aimeos\Controller\Frontend\Basket\Decorator\Base::class)
	59	+ ->setConstructorArgs([$stub, $this->context])
60	60	->getMockForAbstractClass();
61	61
62		- $stub->expects( $this->once() )->method( 'invalid' )->will( $this->returnValue( true ) );
	62	+ $stub->expects($this->once())->method('invalid')->will($this->returnValue(true));
63	63
64		- $this->assertTrue( $object->invalid() );
	64	+ $this->assertTrue($object->invalid());
65	65	}
66	66
67	67
68	68	public function testClear()
69	69	{
70		- $this->stub->expects( $this->once() )->method( 'clear' );
	70	+ $this->stub->expects($this->once())->method('clear');
71	71
72	72	$this->object->clear();
73	73	}
		@@ -76,18 +76,18 @@ discard block
		block discarded – undo
76	76	public function testGet()
77	77	{
78	78	$context = \TestHelperFrontend::getContext();
79		- $order = \Aimeos\MShop::create( $context, 'order/base' )->createItem();
	79	+ $order = \Aimeos\MShop::create($context, 'order/base')->createItem();
80	80
81		- $this->stub->expects( $this->once() )->method( 'get' )
82		- ->will( $this->returnValue( $order ) );
	81	+ $this->stub->expects($this->once())->method('get')
	82	+ ->will($this->returnValue($order));
83	83
84		- $this->assertInstanceOf( \Aimeos\MShop\Order\Item\Base\Iface::class, $this->object->get() );
	84	+ $this->assertInstanceOf(\Aimeos\MShop\Order\Item\Base\Iface::class, $this->object->get());
85	85	}
86	86
87	87
88	88	public function testSave()
89	89	{
90		- $this->stub->expects( $this->once() )->method( 'save' );
	90	+ $this->stub->expects($this->once())->method('save');
91	91
92	92	$this->object->save();
93	93	}
		@@ -95,15 +95,15 @@ discard block
		block discarded – undo
95	95
96	96	public function testSetType()
97	97	{
98		- $this->stub->expects( $this->once() )->method( 'setType' );
	98	+ $this->stub->expects($this->once())->method('setType');
99	99
100		- $this->object->setType( 'test' );
	100	+ $this->object->setType('test');
101	101	}
102	102
103	103
104	104	public function testStore()
105	105	{
106		- $this->stub->expects( $this->once() )->method( 'store' );
	106	+ $this->stub->expects($this->once())->method('store');
107	107
108	108	$this->object->store();
109	109	}
		@@ -111,7 +111,7 @@ discard block
		block discarded – undo
111	111
112	112	public function testLoad()
113	113	{
114		- $this->stub->expects( $this->once() )->method( 'load' );
	114	+ $this->stub->expects($this->once())->method('load');
115	115
116	116	$this->object->load( -1 );
117	117	}
		@@ -119,7 +119,7 @@ discard block
		block discarded – undo
119	119
120	120	public function testAddProduct()
121	121	{
122		- $this->stub->expects( $this->once() )->method( 'addProduct' );
	122	+ $this->stub->expects($this->once())->method('addProduct');
123	123
124	124	$this->object->addProduct( -1 );
125	125	}
		@@ -127,73 +127,73 @@ discard block
		block discarded – undo
127	127
128	128	public function testDeleteProduct()
129	129	{
130		- $this->stub->expects( $this->once() )->method( 'deleteProduct' );
	130	+ $this->stub->expects($this->once())->method('deleteProduct');
131	131
132		- $this->object->deleteProduct( 0 );
	132	+ $this->object->deleteProduct(0);
133	133	}
134	134
135	135
136	136	public function testEditProduct()
137	137	{
138		- $this->stub->expects( $this->once() )->method( 'editProduct' );
	138	+ $this->stub->expects($this->once())->method('editProduct');
139	139
140		- $this->object->editProduct( 0, 1 );
	140	+ $this->object->editProduct(0, 1);
141	141	}
142	142
143	143
144	144	public function testAddCoupon()
145	145	{
146		- $this->stub->expects( $this->once() )->method( 'addCoupon' );
	146	+ $this->stub->expects($this->once())->method('addCoupon');
147	147
148		- $this->object->addCoupon( 'test' );
	148	+ $this->object->addCoupon('test');
149	149	}
150	150
151	151
152	152	public function testDeleteCoupon()
153	153	{
154		- $this->stub->expects( $this->once() )->method( 'deleteCoupon' );
	154	+ $this->stub->expects($this->once())->method('deleteCoupon');
155	155
156		- $this->object->deleteCoupon( 'test' );
	156	+ $this->object->deleteCoupon('test');
157	157	}
158	158
159	159
160	160	public function testSetAddress()
161	161	{
162		- $this->stub->expects( $this->once() )->method( 'setAddress' );
	162	+ $this->stub->expects($this->once())->method('setAddress');
163	163
164		- $this->object->setAddress( \Aimeos\MShop\Order\Item\Base\Address\Base::TYPE_PAYMENT, null );
	164	+ $this->object->setAddress(\Aimeos\MShop\Order\Item\Base\Address\Base::TYPE_PAYMENT, null);
165	165	}
166	166
167	167
168	168	public function testAddService()
169	169	{
170		- $this->stub->expects( $this->once() )->method( 'addService' );
	170	+ $this->stub->expects($this->once())->method('addService');
171	171
172		- $this->object->addService( \Aimeos\MShop\Order\Item\Base\Service\Base::TYPE_PAYMENT, -1 );
	172	+ $this->object->addService(\Aimeos\MShop\Order\Item\Base\Service\Base::TYPE_PAYMENT, -1);
173	173	}
174	174
175	175
176	176	public function testDeleteService()
177	177	{
178		- $this->stub->expects( $this->once() )->method( 'deleteService' );
	178	+ $this->stub->expects($this->once())->method('deleteService');
179	179
180		- $this->object->deleteService( \Aimeos\MShop\Order\Item\Base\Service\Base::TYPE_PAYMENT );
	180	+ $this->object->deleteService(\Aimeos\MShop\Order\Item\Base\Service\Base::TYPE_PAYMENT);
181	181	}
182	182
183	183
184	184	public function testGetController()
185	185	{
186		- $result = $this->access( 'getController' )->invokeArgs( $this->object, [] );
	186	+ $result = $this->access('getController')->invokeArgs($this->object, []);
187	187
188		- $this->assertSame( $this->stub, $result );
	188	+ $this->assertSame($this->stub, $result);
189	189	}
190	190
191	191
192		- protected function access( $name )
	192	+ protected function access($name)
193	193	{
194		- $class = new \ReflectionClass( \Aimeos\Controller\Frontend\Basket\Decorator\Base::class );
195		- $method = $class->getMethod( $name );
196		- $method->setAccessible( true );
	194	+ $class = new \ReflectionClass(\Aimeos\Controller\Frontend\Basket\Decorator\Base::class);
	195	+ $method = $class->getMethod($name);
	196	+ $method->setAccessible(true);
197	197
198	198	return $method;
199	199	}

		@@ -18,40 +18,40 @@
		block discarded – undo
18	18	protected function setUp()
19	19	{
20	20	$this->context = \TestHelperFrontend::getContext();
21		- $object = new \Aimeos\Controller\Frontend\Basket\Standard( $this->context );
22		- $this->object = new \Aimeos\Controller\Frontend\Basket\Decorator\Bundle( $object, $this->context );
	21	+ $object = new \Aimeos\Controller\Frontend\Basket\Standard($this->context);
	22	+ $this->object = new \Aimeos\Controller\Frontend\Basket\Decorator\Bundle($object, $this->context);
23	23	}
24	24
25	25
26	26	protected function tearDown()
27	27	{
28	28	$this->object->clear();
29		- $this->context->getSession()->set( 'aimeos', [] );
	29	+ $this->context->getSession()->set('aimeos', []);
30	30
31		- unset( $this->object );
	31	+ unset($this->object);
32	32	}
33	33
34	34
35	35	public function testAddProductBundle()
36	36	{
37		- $item = \Aimeos\MShop::create( $this->context, 'product' )->findItem( 'U:BUNDLE' );
	37	+ $item = \Aimeos\MShop::create($this->context, 'product')->findItem('U:BUNDLE');
38	38
39		- $this->object->addProduct( $item->getId(), 1 );
	39	+ $this->object->addProduct($item->getId(), 1);
40	40
41		- $this->assertEquals( 1, count( $this->object->get()->getProducts() ) );
42		- $this->assertEquals( 'U:BUNDLE', $this->object->get()->getProduct( 0 )->getProductCode() );
43		- $this->assertEquals( 2, count( $this->object->get()->getProduct( 0 )->getProducts() ) );
	41	+ $this->assertEquals(1, count($this->object->get()->getProducts()));
	42	+ $this->assertEquals('U:BUNDLE', $this->object->get()->getProduct(0)->getProductCode());
	43	+ $this->assertEquals(2, count($this->object->get()->getProduct(0)->getProducts()));
44	44	}
45	45
46	46
47	47	public function testAddProductNoBundle()
48	48	{
49		- $item = \Aimeos\MShop::create( $this->context, 'product' )->findItem( 'CNC' );
	49	+ $item = \Aimeos\MShop::create($this->context, 'product')->findItem('CNC');
50	50
51		- $this->object->addProduct( $item->getId(), 1 );
	51	+ $this->object->addProduct($item->getId(), 1);
52	52
53		- $this->assertEquals( 1, count( $this->object->get()->getProducts() ) );
54		- $this->assertEquals( 'CNC', $this->object->get()->getProduct( 0 )->getProductCode() );
55		- $this->assertEquals( 0, count( $this->object->get()->getProduct( 0 )->getProducts() ) );
	53	+ $this->assertEquals(1, count($this->object->get()->getProducts()));
	54	+ $this->assertEquals('CNC', $this->object->get()->getProduct(0)->getProductCode());
	55	+ $this->assertEquals(0, count($this->object->get()->getProduct(0)->getProducts()));
56	56	}
57	57	}

		@@ -18,35 +18,35 @@
		block discarded – undo
18	18	protected function setUp()
19	19	{
20	20	$this->context = \TestHelperFrontend::getContext();
21		- $object = new \Aimeos\Controller\Frontend\Basket\Standard( $this->context );
22		- $this->object = new \Aimeos\Controller\Frontend\Basket\Decorator\Category( $object, $this->context );
	21	+ $object = new \Aimeos\Controller\Frontend\Basket\Standard($this->context);
	22	+ $this->object = new \Aimeos\Controller\Frontend\Basket\Decorator\Category($object, $this->context);
23	23	}
24	24
25	25
26	26	protected function tearDown()
27	27	{
28	28	$this->object->clear();
29		- $this->context->getSession()->set( 'aimeos', [] );
	29	+ $this->context->getSession()->set('aimeos', []);
30	30
31		- unset( $this->object );
	31	+ unset($this->object);
32	32	}
33	33
34	34
35	35	public function testAddProductWithCategory()
36	36	{
37		- $manager = \Aimeos\MShop::create( $this->context, 'product' );
38		- $item = $manager->findItem( 'CNE' );
	37	+ $manager = \Aimeos\MShop::create($this->context, 'product');
	38	+ $item = $manager->findItem('CNE');
39	39
40		- $this->object->addProduct( $item->getId(), 5 );
	40	+ $this->object->addProduct($item->getId(), 5);
41	41	}
42	42
43	43
44	44	public function testAddProductNoCategory()
45	45	{
46		- $manager = \Aimeos\MShop::create( $this->context, 'product' );
47		- $item = $manager->findItem( 'ABCD' );
	46	+ $manager = \Aimeos\MShop::create($this->context, 'product');
	47	+ $item = $manager->findItem('ABCD');
48	48
49		- $this->setExpectedException( \Aimeos\Controller\Frontend\Basket\Exception::class );
50		- $this->object->addProduct( $item->getId(), 5 );
	49	+ $this->setExpectedException(\Aimeos\Controller\Frontend\Basket\Exception::class);
	50	+ $this->object->addProduct($item->getId(), 5);
51	51	}
52	52	}

		@@ -19,13 +19,13 @@ discard block
		block discarded – undo
19	19	protected function setUp()
20	20	{
21	21	$this->context = \TestHelperFrontend::getContext();
22		- $this->object = new \Aimeos\Controller\Frontend\Catalog\Standard( $this->context );
	22	+ $this->object = new \Aimeos\Controller\Frontend\Catalog\Standard($this->context);
23	23	}
24	24
25	25
26	26	protected function tearDown()
27	27	{
28		- unset( $this->object, $this->context );
	28	+ unset($this->object, $this->context);
29	29	}
30	30
31	31
		@@ -33,19 +33,19 @@ discard block
		block discarded – undo
33	33	{
34	34	$filter = $this->object->createFilter();
35	35
36		- $this->assertInstanceOf( '\\Aimeos\\MW\\Criteria\\Iface', $filter );
	36	+ $this->assertInstanceOf('\\Aimeos\\MW\\Criteria\\Iface', $filter);
37	37	}
38	38
39	39
40	40	public function testGetPath()
41	41	{
42		- $manager = \Aimeos\MShop::create( $this->context, 'catalog' );
43		- $items = $this->object->getPath( $manager->findItem( 'cafe' )->getId() );
	42	+ $manager = \Aimeos\MShop::create($this->context, 'catalog');
	43	+ $items = $this->object->getPath($manager->findItem('cafe')->getId());
44	44
45		- $this->assertEquals( 3, count( $items ) );
	45	+ $this->assertEquals(3, count($items));
46	46
47		- foreach( $items as $item ) {
48		- $this->assertInstanceOf( '\\Aimeos\\MShop\\Catalog\\Item\\Iface', $item );
	47	+ foreach ($items as $item) {
	48	+ $this->assertInstanceOf('\\Aimeos\\MShop\\Catalog\\Item\\Iface', $item);
49	49	}
50	50	}
51	51
		@@ -54,10 +54,10 @@ discard block
		block discarded – undo
54	54	{
55	55	$tree = $this->object->getTree();
56	56
57		- $this->assertEquals( 2, count( $tree->getChildren() ) );
	57	+ $this->assertEquals(2, count($tree->getChildren()));
58	58
59		- foreach( $tree->getChildren() as $item ) {
60		- $this->assertInstanceOf( '\\Aimeos\\MShop\\Catalog\\Item\\Iface', $item );
	59	+ foreach ($tree->getChildren() as $item) {
	60	+ $this->assertInstanceOf('\\Aimeos\\MShop\\Catalog\\Item\\Iface', $item);
61	61	}
62	62	}
63	63	}

		@@ -16,13 +16,13 @@ discard block
		block discarded – undo
16	16
17	17	protected function setUp()
18	18	{
19		- $this->object = new \Aimeos\Controller\Frontend\Stock\Standard( \TestHelperFrontend::getContext() );
	19	+ $this->object = new \Aimeos\Controller\Frontend\Stock\Standard(\TestHelperFrontend::getContext());
20	20	}
21	21
22	22
23	23	protected function tearDown()
24	24	{
25		- unset( $this->object );
	25	+ unset($this->object);
26	26	}
27	27
28	28
		@@ -30,65 +30,65 @@ discard block
		block discarded – undo
30	30	{
31	31	$filter = $this->object->createFilter();
32	32
33		- $this->assertInstanceOf( '\\Aimeos\\MW\\Criteria\\Iface', $filter );
34		- $this->assertEquals( 0, $filter->getSliceStart() );
35		- $this->assertEquals( 100, $filter->getSliceSize() );
	33	+ $this->assertInstanceOf('\\Aimeos\\MW\\Criteria\\Iface', $filter);
	34	+ $this->assertEquals(0, $filter->getSliceStart());
	35	+ $this->assertEquals(100, $filter->getSliceSize());
36	36	}
37	37
38	38
39	39	public function testAddFilterCodes()
40	40	{
41	41	$filter = $this->object->createFilter();
42		- $filter = $this->object->addFilterCodes( $filter, ['CNC', 'CNE'] );
	42	+ $filter = $this->object->addFilterCodes($filter, ['CNC', 'CNE']);
43	43
44	44	$list = $filter->getConditions()->getExpressions();
45	45
46		- if( !isset( $list[0] ) \|\| !( $list[0] instanceof \Aimeos\MW\Criteria\Expression\Compare\Iface ) ) {
47		- throw new \RuntimeException( 'Wrong expression' );
	46	+ if (!isset($list[0]) \|\| !($list[0] instanceof \Aimeos\MW\Criteria\Expression\Compare\Iface)) {
	47	+ throw new \RuntimeException('Wrong expression');
48	48	}
49	49
50		- $this->assertEquals( 'stock.productcode', $list[0]->getName() );
51		- $this->assertEquals( 2, count( $list[0]->getValue() ) );
	50	+ $this->assertEquals('stock.productcode', $list[0]->getName());
	51	+ $this->assertEquals(2, count($list[0]->getValue()));
52	52	}
53	53
54	54
55	55	public function testAddFilterTypes()
56	56	{
57	57	$filter = $this->object->createFilter();
58		- $filter = $this->object->addFilterTypes( $filter, ['default'] );
	58	+ $filter = $this->object->addFilterTypes($filter, ['default']);
59	59
60	60	$list = $filter->getConditions()->getExpressions();
61	61
62		- if( !isset( $list[0] ) \|\| !( $list[0] instanceof \Aimeos\MW\Criteria\Expression\Compare\Iface ) ) {
63		- throw new \RuntimeException( 'Wrong expression' );
	62	+ if (!isset($list[0]) \|\| !($list[0] instanceof \Aimeos\MW\Criteria\Expression\Compare\Iface)) {
	63	+ throw new \RuntimeException('Wrong expression');
64	64	}
65	65
66		- $this->assertEquals( 'stock.type', $list[0]->getName() );
67		- $this->assertEquals( 1, count( $list[0]->getValue() ) );
	66	+ $this->assertEquals('stock.type', $list[0]->getName());
	67	+ $this->assertEquals(1, count($list[0]->getValue()));
68	68	}
69	69
70	70
71	71	public function testGetItem()
72	72	{
73	73	$context = \TestHelperFrontend::getContext();
74		- $manager = \Aimeos\MShop::create( $context, 'stock' );
75		- $id = $manager->findItem( 'CNC', [], 'product', 'default' )->getId();
	74	+ $manager = \Aimeos\MShop::create($context, 'stock');
	75	+ $id = $manager->findItem('CNC', [], 'product', 'default')->getId();
76	76
77		- $result = $this->object->getItem( $id );
	77	+ $result = $this->object->getItem($id);
78	78
79		- $this->assertInstanceOf( \Aimeos\MShop\Stock\Item\Iface::class, $result );
	79	+ $this->assertInstanceOf(\Aimeos\MShop\Stock\Item\Iface::class, $result);
80	80	}
81	81
82	82
83	83	public function testSearchItems()
84	84	{
85	85	$filter = $this->object->createFilter();
86		- $filter = $this->object->addFilterCodes( $filter, ['CNC', 'CNE'] );
	86	+ $filter = $this->object->addFilterCodes($filter, ['CNC', 'CNE']);
87	87
88	88	$total = 0;
89		- $results = $this->object->searchItems( $filter, $total );
	89	+ $results = $this->object->searchItems($filter, $total);
90	90
91		- $this->assertEquals( 2, $total );
92		- $this->assertEquals( 2, count( $results ) );
	91	+ $this->assertEquals(2, $total);
	92	+ $this->assertEquals(2, count($results));
93	93	}
94	94	}

aimeos / ai-controller-frontend

Push — master ( 8086a8...921503 )

Status

Category

Spacing +45 added lines, -45 removed lines patch added patch discarded remove patch

Spacing +14 added lines, -14 removed lines patch added patch discarded remove patch

Spacing +11 added lines, -11 removed lines patch added patch discarded remove patch

Spacing +31 added lines, -31 removed lines patch added patch discarded remove patch

Spacing +11 added lines, -11 removed lines patch added patch discarded remove patch

Spacing +40 added lines, -40 removed lines patch added patch discarded remove patch

Spacing +23 added lines, -23 removed lines patch added patch discarded remove patch

Spacing +35 added lines, -35 removed lines patch added patch discarded remove patch

Spacing +17 added lines, -17 removed lines patch added patch discarded remove patch