Completed
Push — master ( ac72d9...36c1ed )
by Aimeos
01:44
created
controller/frontend/tests/Controller/Frontend/Order/FactoryTest.php 1 patch
Spacing   +10 added lines, -10 removed lines patch added patch discarded remove patch
@@ -16,31 +16,31 @@
 block discarded – undo
16 16
 	{
17 17
 		$target = '\\Aimeos\\Controller\\Frontend\\Order\\Iface';
18 18
 
19
-		$controller = \Aimeos\Controller\Frontend\Order\Factory::createController( \TestHelperFrontend::getContext() );
20
-		$this->assertInstanceOf( $target, $controller );
19
+		$controller = \Aimeos\Controller\Frontend\Order\Factory::createController(\TestHelperFrontend::getContext());
20
+		$this->assertInstanceOf($target, $controller);
21 21
 
22
-		$controller = \Aimeos\Controller\Frontend\Order\Factory::createController( \TestHelperFrontend::getContext(), 'Standard' );
23
-		$this->assertInstanceOf( $target, $controller );
22
+		$controller = \Aimeos\Controller\Frontend\Order\Factory::createController(\TestHelperFrontend::getContext(), 'Standard');
23
+		$this->assertInstanceOf($target, $controller);
24 24
 	}
25 25
 
26 26
 
27 27
 	public function testCreateControllerInvalidImplementation()
28 28
 	{
29
-		$this->setExpectedException( '\\Aimeos\\MW\\Common\\Exception' );
30
-		\Aimeos\Controller\Frontend\Order\Factory::createController( \TestHelperFrontend::getContext(), 'Invalid' );
29
+		$this->setExpectedException('\\Aimeos\\MW\\Common\\Exception');
30
+		\Aimeos\Controller\Frontend\Order\Factory::createController(\TestHelperFrontend::getContext(), 'Invalid');
31 31
 	}
32 32
 
33 33
 
34 34
 	public function testCreateControllerInvalidName()
35 35
 	{
36
-		$this->setExpectedException( '\\Aimeos\\Controller\\Frontend\\Exception' );
37
-		\Aimeos\Controller\Frontend\Order\Factory::createController( \TestHelperFrontend::getContext(), '%^' );
36
+		$this->setExpectedException('\\Aimeos\\Controller\\Frontend\\Exception');
37
+		\Aimeos\Controller\Frontend\Order\Factory::createController(\TestHelperFrontend::getContext(), '%^');
38 38
 	}
39 39
 
40 40
 
41 41
 	public function testCreateControllerNotExisting()
42 42
 	{
43
-		$this->setExpectedException( '\\Aimeos\\Controller\\Frontend\\Exception' );
44
-		\Aimeos\Controller\Frontend\Order\Factory::createController( \TestHelperFrontend::getContext(), 'notexist' );
43
+		$this->setExpectedException('\\Aimeos\\Controller\\Frontend\\Exception');
44
+		\Aimeos\Controller\Frontend\Order\Factory::createController(\TestHelperFrontend::getContext(), 'notexist');
45 45
 	}
46 46
 }
Please login to merge, or discard this patch.
controller/frontend/tests/Controller/Frontend/Customer/FactoryTest.php 1 patch
Spacing   +10 added lines, -10 removed lines patch added patch discarded remove patch
@@ -15,31 +15,31 @@
 block discarded – undo
15 15
 	{
16 16
 		$target = '\\Aimeos\\Controller\\Frontend\\Customer\\Iface';
17 17
 
18
-		$controller = \Aimeos\Controller\Frontend\Customer\Factory::createController( \TestHelperFrontend::getContext() );
19
-		$this->assertInstanceOf( $target, $controller );
18
+		$controller = \Aimeos\Controller\Frontend\Customer\Factory::createController(\TestHelperFrontend::getContext());
19
+		$this->assertInstanceOf($target, $controller);
20 20
 
21
-		$controller = \Aimeos\Controller\Frontend\Customer\Factory::createController( \TestHelperFrontend::getContext(), 'Standard' );
22
-		$this->assertInstanceOf( $target, $controller );
21
+		$controller = \Aimeos\Controller\Frontend\Customer\Factory::createController(\TestHelperFrontend::getContext(), 'Standard');
22
+		$this->assertInstanceOf($target, $controller);
23 23
 	}
24 24
 
25 25
 
26 26
 	public function testCreateControllerInvalidImplementation()
27 27
 	{
28
-		$this->setExpectedException( '\\Aimeos\\MW\\Common\\Exception' );
29
-		\Aimeos\Controller\Frontend\Customer\Factory::createController( \TestHelperFrontend::getContext(), 'Invalid' );
28
+		$this->setExpectedException('\\Aimeos\\MW\\Common\\Exception');
29
+		\Aimeos\Controller\Frontend\Customer\Factory::createController(\TestHelperFrontend::getContext(), 'Invalid');
30 30
 	}
31 31
 
32 32
 
33 33
 	public function testCreateControllerInvalidName()
34 34
 	{
35
-		$this->setExpectedException( '\\Aimeos\\Controller\\Frontend\\Exception' );
36
-		\Aimeos\Controller\Frontend\Customer\Factory::createController( \TestHelperFrontend::getContext(), '%^' );
35
+		$this->setExpectedException('\\Aimeos\\Controller\\Frontend\\Exception');
36
+		\Aimeos\Controller\Frontend\Customer\Factory::createController(\TestHelperFrontend::getContext(), '%^');
37 37
 	}
38 38
 
39 39
 
40 40
 	public function testCreateControllerNotExisting()
41 41
 	{
42
-		$this->setExpectedException( '\\Aimeos\\Controller\\Frontend\\Exception' );
43
-		\Aimeos\Controller\Frontend\Customer\Factory::createController( \TestHelperFrontend::getContext(), 'notexist' );
42
+		$this->setExpectedException('\\Aimeos\\Controller\\Frontend\\Exception');
43
+		\Aimeos\Controller\Frontend\Customer\Factory::createController(\TestHelperFrontend::getContext(), 'notexist');
44 44
 	}
45 45
 }
Please login to merge, or discard this patch.
controller/frontend/tests/Controller/Frontend/Stock/FactoryTest.php 1 patch
Spacing   +10 added lines, -10 removed lines patch added patch discarded remove patch
@@ -15,31 +15,31 @@
 block discarded – undo
15 15
 	{
16 16
 		$target = '\\Aimeos\\Controller\\Frontend\\Stock\\Iface';
17 17
 
18
-		$controller = \Aimeos\Controller\Frontend\Stock\Factory::createController( \TestHelperFrontend::getContext() );
19
-		$this->assertInstanceOf( $target, $controller );
18
+		$controller = \Aimeos\Controller\Frontend\Stock\Factory::createController(\TestHelperFrontend::getContext());
19
+		$this->assertInstanceOf($target, $controller);
20 20
 
21
-		$controller = \Aimeos\Controller\Frontend\Stock\Factory::createController( \TestHelperFrontend::getContext(), 'Standard' );
22
-		$this->assertInstanceOf( $target, $controller );
21
+		$controller = \Aimeos\Controller\Frontend\Stock\Factory::createController(\TestHelperFrontend::getContext(), 'Standard');
22
+		$this->assertInstanceOf($target, $controller);
23 23
 	}
24 24
 
25 25
 
26 26
 	public function testCreateControllerInvalidImplementation()
27 27
 	{
28
-		$this->setExpectedException( '\\Aimeos\\MW\\Common\\Exception' );
29
-		\Aimeos\Controller\Frontend\Stock\Factory::createController( \TestHelperFrontend::getContext(), 'Invalid' );
28
+		$this->setExpectedException('\\Aimeos\\MW\\Common\\Exception');
29
+		\Aimeos\Controller\Frontend\Stock\Factory::createController(\TestHelperFrontend::getContext(), 'Invalid');
30 30
 	}
31 31
 
32 32
 
33 33
 	public function testCreateControllerInvalidName()
34 34
 	{
35
-		$this->setExpectedException( '\\Aimeos\\Controller\\Frontend\\Exception' );
36
-		\Aimeos\Controller\Frontend\Stock\Factory::createController( \TestHelperFrontend::getContext(), '%^' );
35
+		$this->setExpectedException('\\Aimeos\\Controller\\Frontend\\Exception');
36
+		\Aimeos\Controller\Frontend\Stock\Factory::createController(\TestHelperFrontend::getContext(), '%^');
37 37
 	}
38 38
 
39 39
 
40 40
 	public function testCreateControllerNotExisting()
41 41
 	{
42
-		$this->setExpectedException( '\\Aimeos\\Controller\\Frontend\\Exception' );
43
-		\Aimeos\Controller\Frontend\Stock\Factory::createController( \TestHelperFrontend::getContext(), 'notexist' );
42
+		$this->setExpectedException('\\Aimeos\\Controller\\Frontend\\Exception');
43
+		\Aimeos\Controller\Frontend\Stock\Factory::createController(\TestHelperFrontend::getContext(), 'notexist');
44 44
 	}
45 45
 }
Please login to merge, or discard this patch.
controller/frontend/tests/Controller/Frontend/Supplier/FactoryTest.php 1 patch
Spacing   +10 added lines, -10 removed lines patch added patch discarded remove patch
@@ -15,31 +15,31 @@
 block discarded – undo
15 15
 	{
16 16
 		$target = '\\Aimeos\\Controller\\Frontend\\Supplier\\Iface';
17 17
 
18
-		$controller = \Aimeos\Controller\Frontend\Supplier\Factory::createController( \TestHelperFrontend::getContext() );
19
-		$this->assertInstanceOf( $target, $controller );
18
+		$controller = \Aimeos\Controller\Frontend\Supplier\Factory::createController(\TestHelperFrontend::getContext());
19
+		$this->assertInstanceOf($target, $controller);
20 20
 
21
-		$controller = \Aimeos\Controller\Frontend\Supplier\Factory::createController( \TestHelperFrontend::getContext(), 'Standard' );
22
-		$this->assertInstanceOf( $target, $controller );
21
+		$controller = \Aimeos\Controller\Frontend\Supplier\Factory::createController(\TestHelperFrontend::getContext(), 'Standard');
22
+		$this->assertInstanceOf($target, $controller);
23 23
 	}
24 24
 
25 25
 
26 26
 	public function testCreateControllerInvalidImplementation()
27 27
 	{
28
-		$this->setExpectedException( '\\Aimeos\\MW\\Common\\Exception' );
29
-		\Aimeos\Controller\Frontend\Supplier\Factory::createController( \TestHelperFrontend::getContext(), 'Invalid' );
28
+		$this->setExpectedException('\\Aimeos\\MW\\Common\\Exception');
29
+		\Aimeos\Controller\Frontend\Supplier\Factory::createController(\TestHelperFrontend::getContext(), 'Invalid');
30 30
 	}
31 31
 
32 32
 
33 33
 	public function testCreateControllerInvalidName()
34 34
 	{
35
-		$this->setExpectedException( '\\Aimeos\\Controller\\Frontend\\Exception' );
36
-		\Aimeos\Controller\Frontend\Supplier\Factory::createController( \TestHelperFrontend::getContext(), '%^' );
35
+		$this->setExpectedException('\\Aimeos\\Controller\\Frontend\\Exception');
36
+		\Aimeos\Controller\Frontend\Supplier\Factory::createController(\TestHelperFrontend::getContext(), '%^');
37 37
 	}
38 38
 
39 39
 
40 40
 	public function testCreateControllerNotExisting()
41 41
 	{
42
-		$this->setExpectedException( '\\Aimeos\\Controller\\Frontend\\Exception' );
43
-		\Aimeos\Controller\Frontend\Supplier\Factory::createController( \TestHelperFrontend::getContext(), 'notexist' );
42
+		$this->setExpectedException('\\Aimeos\\Controller\\Frontend\\Exception');
43
+		\Aimeos\Controller\Frontend\Supplier\Factory::createController(\TestHelperFrontend::getContext(), 'notexist');
44 44
 	}
45 45
 }
Please login to merge, or discard this patch.
controller/frontend/tests/Controller/Frontend/Catalog/FactoryTest.php 1 patch
Spacing   +10 added lines, -10 removed lines patch added patch discarded remove patch
@@ -14,31 +14,31 @@
 block discarded – undo
14 14
 	{
15 15
 		$target = '\\Aimeos\\Controller\\Frontend\\Catalog\\Iface';
16 16
 
17
-		$controller = \Aimeos\Controller\Frontend\Catalog\Factory::createController( \TestHelperFrontend::getContext() );
18
-		$this->assertInstanceOf( $target, $controller );
17
+		$controller = \Aimeos\Controller\Frontend\Catalog\Factory::createController(\TestHelperFrontend::getContext());
18
+		$this->assertInstanceOf($target, $controller);
19 19
 
20
-		$controller = \Aimeos\Controller\Frontend\Catalog\Factory::createController( \TestHelperFrontend::getContext(), 'Standard' );
21
-		$this->assertInstanceOf( $target, $controller );
20
+		$controller = \Aimeos\Controller\Frontend\Catalog\Factory::createController(\TestHelperFrontend::getContext(), 'Standard');
21
+		$this->assertInstanceOf($target, $controller);
22 22
 	}
23 23
 
24 24
 
25 25
 	public function testCreateControllerInvalidImplementation()
26 26
 	{
27
-		$this->setExpectedException( '\\Aimeos\\MW\\Common\\Exception' );
28
-		\Aimeos\Controller\Frontend\Catalog\Factory::createController( \TestHelperFrontend::getContext(), 'Invalid' );
27
+		$this->setExpectedException('\\Aimeos\\MW\\Common\\Exception');
28
+		\Aimeos\Controller\Frontend\Catalog\Factory::createController(\TestHelperFrontend::getContext(), 'Invalid');
29 29
 	}
30 30
 
31 31
 
32 32
 	public function testCreateControllerInvalidName()
33 33
 	{
34
-		$this->setExpectedException( '\\Aimeos\\Controller\\Frontend\\Exception' );
35
-		\Aimeos\Controller\Frontend\Catalog\Factory::createController( \TestHelperFrontend::getContext(), '%^' );
34
+		$this->setExpectedException('\\Aimeos\\Controller\\Frontend\\Exception');
35
+		\Aimeos\Controller\Frontend\Catalog\Factory::createController(\TestHelperFrontend::getContext(), '%^');
36 36
 	}
37 37
 
38 38
 
39 39
 	public function testCreateControllerNotExisting()
40 40
 	{
41
-		$this->setExpectedException( '\\Aimeos\\Controller\\Frontend\\Exception' );
42
-		\Aimeos\Controller\Frontend\Catalog\Factory::createController( \TestHelperFrontend::getContext(), 'notexist' );
41
+		$this->setExpectedException('\\Aimeos\\Controller\\Frontend\\Exception');
42
+		\Aimeos\Controller\Frontend\Catalog\Factory::createController(\TestHelperFrontend::getContext(), 'notexist');
43 43
 	}
44 44
 }
Please login to merge, or discard this patch.
controller/frontend/tests/Controller/Frontend/Attribute/FactoryTest.php 1 patch
Spacing   +10 added lines, -10 removed lines patch added patch discarded remove patch
@@ -15,31 +15,31 @@
 block discarded – undo
15 15
 	{
16 16
 		$target = '\\Aimeos\\Controller\\Frontend\\Attribute\\Iface';
17 17
 
18
-		$controller = \Aimeos\Controller\Frontend\Attribute\Factory::createController( \TestHelperFrontend::getContext() );
19
-		$this->assertInstanceOf( $target, $controller );
18
+		$controller = \Aimeos\Controller\Frontend\Attribute\Factory::createController(\TestHelperFrontend::getContext());
19
+		$this->assertInstanceOf($target, $controller);
20 20
 
21
-		$controller = \Aimeos\Controller\Frontend\Attribute\Factory::createController( \TestHelperFrontend::getContext(), 'Standard' );
22
-		$this->assertInstanceOf( $target, $controller );
21
+		$controller = \Aimeos\Controller\Frontend\Attribute\Factory::createController(\TestHelperFrontend::getContext(), 'Standard');
22
+		$this->assertInstanceOf($target, $controller);
23 23
 	}
24 24
 
25 25
 
26 26
 	public function testCreateControllerInvalidImplementation()
27 27
 	{
28
-		$this->setExpectedException( '\\Aimeos\\MW\\Common\\Exception' );
29
-		\Aimeos\Controller\Frontend\Attribute\Factory::createController( \TestHelperFrontend::getContext(), 'Invalid' );
28
+		$this->setExpectedException('\\Aimeos\\MW\\Common\\Exception');
29
+		\Aimeos\Controller\Frontend\Attribute\Factory::createController(\TestHelperFrontend::getContext(), 'Invalid');
30 30
 	}
31 31
 
32 32
 
33 33
 	public function testCreateControllerInvalidName()
34 34
 	{
35
-		$this->setExpectedException( '\\Aimeos\\Controller\\Frontend\\Exception' );
36
-		\Aimeos\Controller\Frontend\Attribute\Factory::createController( \TestHelperFrontend::getContext(), '%^' );
35
+		$this->setExpectedException('\\Aimeos\\Controller\\Frontend\\Exception');
36
+		\Aimeos\Controller\Frontend\Attribute\Factory::createController(\TestHelperFrontend::getContext(), '%^');
37 37
 	}
38 38
 
39 39
 
40 40
 	public function testCreateControllerNotExisting()
41 41
 	{
42
-		$this->setExpectedException( '\\Aimeos\\Controller\\Frontend\\Exception' );
43
-		\Aimeos\Controller\Frontend\Attribute\Factory::createController( \TestHelperFrontend::getContext(), 'notexist' );
42
+		$this->setExpectedException('\\Aimeos\\Controller\\Frontend\\Exception');
43
+		\Aimeos\Controller\Frontend\Attribute\Factory::createController(\TestHelperFrontend::getContext(), 'notexist');
44 44
 	}
45 45
 }
Please login to merge, or discard this patch.
controller/frontend/tests/Controller/Frontend/Basket/FactoryTest.php 1 patch
Spacing   +10 added lines, -10 removed lines patch added patch discarded remove patch
@@ -14,31 +14,31 @@
 block discarded – undo
14 14
 	{
15 15
 		$target = '\\Aimeos\\Controller\\Frontend\\Basket\\Iface';
16 16
 
17
-		$controller = \Aimeos\Controller\Frontend\Basket\Factory::createController( \TestHelperFrontend::getContext() );
18
-		$this->assertInstanceOf( $target, $controller );
17
+		$controller = \Aimeos\Controller\Frontend\Basket\Factory::createController(\TestHelperFrontend::getContext());
18
+		$this->assertInstanceOf($target, $controller);
19 19
 
20
-		$controller = \Aimeos\Controller\Frontend\Basket\Factory::createController( \TestHelperFrontend::getContext(), 'Standard' );
21
-		$this->assertInstanceOf( $target, $controller );
20
+		$controller = \Aimeos\Controller\Frontend\Basket\Factory::createController(\TestHelperFrontend::getContext(), 'Standard');
21
+		$this->assertInstanceOf($target, $controller);
22 22
 	}
23 23
 
24 24
 
25 25
 	public function testCreateControllerInvalidImplementation()
26 26
 	{
27
-		$this->setExpectedException( '\\Aimeos\\MW\\Common\\Exception' );
28
-		\Aimeos\Controller\Frontend\Basket\Factory::createController( \TestHelperFrontend::getContext(), 'Invalid' );
27
+		$this->setExpectedException('\\Aimeos\\MW\\Common\\Exception');
28
+		\Aimeos\Controller\Frontend\Basket\Factory::createController(\TestHelperFrontend::getContext(), 'Invalid');
29 29
 	}
30 30
 
31 31
 
32 32
 	public function testCreateControllerInvalidName()
33 33
 	{
34
-		$this->setExpectedException( '\\Aimeos\\Controller\\Frontend\\Exception' );
35
-		\Aimeos\Controller\Frontend\Basket\Factory::createController( \TestHelperFrontend::getContext(), '%^' );
34
+		$this->setExpectedException('\\Aimeos\\Controller\\Frontend\\Exception');
35
+		\Aimeos\Controller\Frontend\Basket\Factory::createController(\TestHelperFrontend::getContext(), '%^');
36 36
 	}
37 37
 
38 38
 
39 39
 	public function testCreateControllerNotExisting()
40 40
 	{
41
-		$this->setExpectedException( '\\Aimeos\\Controller\\Frontend\\Exception' );
42
-		\Aimeos\Controller\Frontend\Basket\Factory::createController( \TestHelperFrontend::getContext(), 'notexist' );
41
+		$this->setExpectedException('\\Aimeos\\Controller\\Frontend\\Exception');
42
+		\Aimeos\Controller\Frontend\Basket\Factory::createController(\TestHelperFrontend::getContext(), 'notexist');
43 43
 	}
44 44
 }
Please login to merge, or discard this patch.
controller/frontend/src/Controller/Frontend/Basket/Decorator/Bundle.php 1 patch
Spacing   +35 added lines, -35 removed lines patch added patch discarded remove patch
@@ -36,13 +36,13 @@  discard block
 block discarded – undo
36 36
 	 * @param string $stocktype Unique code of the stock type to deliver the products from
37 37
 	 * @throws \Aimeos\Controller\Frontend\Basket\Exception If the product isn't available
38 38
 	 */
39
-	public function addProduct( $prodid, $quantity = 1, $stocktype = 'default', array $variantAttributeIds = [],
40
-		array $configAttributeIds = [], array $hiddenAttributeIds = [], array $customAttributeValues = [] )
39
+	public function addProduct($prodid, $quantity = 1, $stocktype = 'default', array $variantAttributeIds = [],
40
+		array $configAttributeIds = [], array $hiddenAttributeIds = [], array $customAttributeValues = [])
41 41
 	{
42 42
 		$context = $this->getContext();
43
-		$productManager = \Aimeos\MShop\Factory::createManager( $context, 'product' );
43
+		$productManager = \Aimeos\MShop\Factory::createManager($context, 'product');
44 44
 
45
-		if( $productManager->getItem( $prodid, [], true )->getType() !== 'bundle' )
45
+		if ($productManager->getItem($prodid, [], true)->getType() !== 'bundle')
46 46
 		{
47 47
 			return $this->getController()->addProduct(
48 48
 				$prodid, $quantity, $stocktype, $variantAttributeIds,
@@ -51,29 +51,29 @@  discard block
 block discarded – undo
51 51
 		}
52 52
 
53 53
 		$attributeMap = [
54
-			'custom' => array_keys( $customAttributeValues ),
55
-			'config' => array_keys( $configAttributeIds ),
54
+			'custom' => array_keys($customAttributeValues),
55
+			'config' => array_keys($configAttributeIds),
56 56
 		];
57
-		$this->checkListRef( $prodid, 'attribute', $attributeMap );
57
+		$this->checkListRef($prodid, 'attribute', $attributeMap);
58 58
 
59 59
 
60
-		$productItem = $productManager->getItem( $prodid, ['attribute', 'media', 'supplier', 'price', 'product', 'text'], true );
61
-		$prices = $productItem->getRefItems( 'price', 'default', 'default' );
62
-		$hidden = $productItem->getRefItems( 'attribute', null, 'hidden' );
60
+		$productItem = $productManager->getItem($prodid, ['attribute', 'media', 'supplier', 'price', 'product', 'text'], true);
61
+		$prices = $productItem->getRefItems('price', 'default', 'default');
62
+		$hidden = $productItem->getRefItems('attribute', null, 'hidden');
63 63
 
64
-		$orderBaseProductItem = \Aimeos\MShop\Factory::createManager( $context, 'order/base/product' )->createItem();
65
-		$orderBaseProductItem->copyFrom( $productItem )->setQuantity( $quantity )->setStockType( $stocktype );
64
+		$orderBaseProductItem = \Aimeos\MShop\Factory::createManager($context, 'order/base/product')->createItem();
65
+		$orderBaseProductItem->copyFrom($productItem)->setQuantity($quantity)->setStockType($stocktype);
66 66
 
67
-		$this->addBundleProducts( $orderBaseProductItem, $productItem, $variantAttributeIds, $stocktype );
67
+		$this->addBundleProducts($orderBaseProductItem, $productItem, $variantAttributeIds, $stocktype);
68 68
 
69
-		$custAttr = $this->getOrderProductAttributes( 'custom', array_keys( $customAttributeValues ), $customAttributeValues );
70
-		$confAttr = $this->getOrderProductAttributes( 'config', array_keys( $configAttributeIds ), [], $configAttributeIds );
71
-		$attr = array_merge( $custAttr, $confAttr, $this->getOrderProductAttributes( 'hidden', array_keys( $hidden ) ) );
69
+		$custAttr = $this->getOrderProductAttributes('custom', array_keys($customAttributeValues), $customAttributeValues);
70
+		$confAttr = $this->getOrderProductAttributes('config', array_keys($configAttributeIds), [], $configAttributeIds);
71
+		$attr = array_merge($custAttr, $confAttr, $this->getOrderProductAttributes('hidden', array_keys($hidden)));
72 72
 
73
-		$orderBaseProductItem->setAttributeItems( $attr );
74
-		$orderBaseProductItem->setPrice( $this->calcPrice( $orderBaseProductItem, $prices, $quantity ) );
73
+		$orderBaseProductItem->setAttributeItems($attr);
74
+		$orderBaseProductItem->setPrice($this->calcPrice($orderBaseProductItem, $prices, $quantity));
75 75
 
76
-		$this->getController()->get()->addProduct( $orderBaseProductItem );
76
+		$this->getController()->get()->addProduct($orderBaseProductItem);
77 77
 		$this->getController()->save();
78 78
 	}
79 79
 
@@ -86,43 +86,43 @@  discard block
 block discarded – undo
86 86
 	 * @param array $variantAttributeIds List of product variant attribute IDs
87 87
 	 * @param string $stocktype
88 88
 	 */
89
-	protected function addBundleProducts( \Aimeos\MShop\Order\Item\Base\Product\Iface $orderBaseProductItem,
90
-		\Aimeos\MShop\Product\Item\Iface $productItem, array $variantAttributeIds, $stocktype )
89
+	protected function addBundleProducts(\Aimeos\MShop\Order\Item\Base\Product\Iface $orderBaseProductItem,
90
+		\Aimeos\MShop\Product\Item\Iface $productItem, array $variantAttributeIds, $stocktype)
91 91
 	{
92 92
 		$quantity = $orderBaseProductItem->getQuantity();
93 93
 		$products = $subProductIds = $orderProducts = [];
94
-		$orderProductManager = \Aimeos\MShop\Factory::createManager( $this->getContext(), 'order/base/product' );
94
+		$orderProductManager = \Aimeos\MShop\Factory::createManager($this->getContext(), 'order/base/product');
95 95
 
96
-		foreach( $productItem->getRefItems( 'product', null, 'default' ) as $item ) {
96
+		foreach ($productItem->getRefItems('product', null, 'default') as $item) {
97 97
 			$subProductIds[] = $item->getId();
98 98
 		}
99 99
 
100
-		if( count( $subProductIds ) > 0 )
100
+		if (count($subProductIds) > 0)
101 101
 		{
102
-			$productManager = \Aimeos\MShop\Factory::createManager( $this->getContext(), 'product' );
102
+			$productManager = \Aimeos\MShop\Factory::createManager($this->getContext(), 'product');
103 103
 
104
-			$search = $productManager->createSearch( true );
104
+			$search = $productManager->createSearch(true);
105 105
 			$expr = array(
106
-				$search->compare( '==', 'product.id', $subProductIds ),
106
+				$search->compare('==', 'product.id', $subProductIds),
107 107
 				$search->getConditions(),
108 108
 			);
109
-			$search->setConditions( $search->combine( '&&', $expr ) );
109
+			$search->setConditions($search->combine('&&', $expr));
110 110
 
111
-			$products = $productManager->searchItems( $search, array( 'attribute', 'media', 'price', 'text' ) );
111
+			$products = $productManager->searchItems($search, array('attribute', 'media', 'price', 'text'));
112 112
 		}
113 113
 
114
-		foreach( $products as $product )
114
+		foreach ($products as $product)
115 115
 		{
116
-			$prices = $product->getRefItems( 'price', 'default', 'default' );
116
+			$prices = $product->getRefItems('price', 'default', 'default');
117 117
 
118 118
 			$orderProduct = $orderProductManager->createItem();
119
-			$orderProduct->copyFrom( $product );
120
-			$orderProduct->setStockType( $stocktype );
121
-			$orderProduct->setPrice( $this->calcPrice( $orderProduct, $prices, $quantity ) );
119
+			$orderProduct->copyFrom($product);
120
+			$orderProduct->setStockType($stocktype);
121
+			$orderProduct->setPrice($this->calcPrice($orderProduct, $prices, $quantity));
122 122
 
123 123
 			$orderProducts[] = $orderProduct;
124 124
 		}
125 125
 
126
-		$orderBaseProductItem->setProducts( $orderProducts );
126
+		$orderBaseProductItem->setProducts($orderProducts);
127 127
 	}
128 128
 }
Please login to merge, or discard this patch.
controller/frontend/tests/Controller/Frontend/Basket/BaseTest.php 2 patches
Doc Comments   +3 added lines patch added patch discarded remove patch
@@ -329,6 +329,9 @@
 block discarded – undo
329 329
 	}
330 330
 
331 331
 
332
+	/**
333
+	 * @param string $name
334
+	 */
332 335
 	protected function access( $name )
333 336
 	{
334 337
 		$class = new \ReflectionClass( \Aimeos\Controller\Frontend\Basket\Base::class );
Please login to merge, or discard this patch.
Spacing   +138 added lines, -138 removed lines patch added patch discarded remove patch
@@ -16,53 +16,53 @@  discard block
 block discarded – undo
16 16
 	protected function setUp()
17 17
 	{
18 18
 		$this->context = \TestHelperFrontend::getContext();
19
-		\Aimeos\MShop::cache( true );
19
+		\Aimeos\MShop::cache(true);
20 20
 	}
21 21
 
22 22
 
23 23
 	protected function tearDown()
24 24
 	{
25
-		\Aimeos\MShop::cache( false );
26
-		$this->context->getSession()->set( 'aimeos', [] );
25
+		\Aimeos\MShop::cache(false);
26
+		$this->context->getSession()->set('aimeos', []);
27 27
 
28
-		unset( $this->context );
28
+		unset($this->context);
29 29
 	}
30 30
 
31 31
 
32 32
 	public function testCheckLocale()
33 33
 	{
34
-		$object = $this->getMockBuilder( \Aimeos\Controller\Frontend\Basket\Standard::class )
35
-			->setConstructorArgs( [$this->context] )
36
-			->setMethods( null )
34
+		$object = $this->getMockBuilder(\Aimeos\Controller\Frontend\Basket\Standard::class)
35
+			->setConstructorArgs([$this->context])
36
+			->setMethods(null)
37 37
 			->getMock();
38 38
 
39
-		$this->context->getSession()->set( 'aimeos/basket/locale', 'unittest|en|EUR' );
40
-		$this->access( 'checkLocale' )->invokeArgs( $object, [$this->context->getLocale(), 'unittest'] );
39
+		$this->context->getSession()->set('aimeos/basket/locale', 'unittest|en|EUR');
40
+		$this->access('checkLocale')->invokeArgs($object, [$this->context->getLocale(), 'unittest']);
41 41
 	}
42 42
 
43 43
 
44 44
 	public function testCopyAddresses()
45 45
 	{
46
-		$manager = \Aimeos\MShop\Factory::createManager( \TestHelperFrontend::getContext(), 'order/base' );
46
+		$manager = \Aimeos\MShop\Factory::createManager(\TestHelperFrontend::getContext(), 'order/base');
47 47
 		$ordBaseItem = $manager->createItem();
48 48
 
49
-		$address = $this->getAddress( 'Example company' );
50
-		$ordBaseItem->setAddress( $address, \Aimeos\MShop\Order\Item\Base\Address\Base::TYPE_PAYMENT );
49
+		$address = $this->getAddress('Example company');
50
+		$ordBaseItem->setAddress($address, \Aimeos\MShop\Order\Item\Base\Address\Base::TYPE_PAYMENT);
51 51
 
52 52
 
53
-		$object = $this->getMockBuilder( \Aimeos\Controller\Frontend\Basket\Standard::class )
54
-			->setConstructorArgs( [$this->context] )
55
-			->setMethods( null )
53
+		$object = $this->getMockBuilder(\Aimeos\Controller\Frontend\Basket\Standard::class)
54
+			->setConstructorArgs([$this->context])
55
+			->setMethods(null)
56 56
 			->getMock();
57 57
 
58
-		$result = $this->access( 'copyAddresses' )->invokeArgs( $object, [$ordBaseItem, ['test'], 'unittest|en|EUR'] );
58
+		$result = $this->access('copyAddresses')->invokeArgs($object, [$ordBaseItem, ['test'], 'unittest|en|EUR']);
59 59
 
60 60
 
61
-		$this->assertEquals( ['test'], $result );
62
-		$this->assertEquals( 1, count( $object->get()->getAddresses() ) );
61
+		$this->assertEquals(['test'], $result);
62
+		$this->assertEquals(1, count($object->get()->getAddresses()));
63 63
 
64
-		$addr = $object->get()->getAddress( \Aimeos\MShop\Order\Item\Base\Address\Base::TYPE_PAYMENT );
65
-		$this->assertInstanceOf( \Aimeos\MShop\Order\Item\Base\Address\Iface::class, $addr );
64
+		$addr = $object->get()->getAddress(\Aimeos\MShop\Order\Item\Base\Address\Base::TYPE_PAYMENT);
65
+		$this->assertInstanceOf(\Aimeos\MShop\Order\Item\Base\Address\Iface::class, $addr);
66 66
 
67 67
 		$object->clear();
68 68
 	}
@@ -70,54 +70,54 @@  discard block
 block discarded – undo
70 70
 
71 71
 	public function testCopyAddressesException()
72 72
 	{
73
-		$manager = \Aimeos\MShop\Factory::createManager( \TestHelperFrontend::getContext(), 'order/base' );
73
+		$manager = \Aimeos\MShop\Factory::createManager(\TestHelperFrontend::getContext(), 'order/base');
74 74
 		$ordBaseItem = $manager->createItem();
75 75
 
76
-		$address = $this->getAddress( 'Example company' );
77
-		$ordBaseItem->setAddress( $address, \Aimeos\MShop\Order\Item\Base\Address\Base::TYPE_PAYMENT );
76
+		$address = $this->getAddress('Example company');
77
+		$ordBaseItem->setAddress($address, \Aimeos\MShop\Order\Item\Base\Address\Base::TYPE_PAYMENT);
78 78
 
79 79
 
80
-		$object = $this->getMockBuilder( \Aimeos\Controller\Frontend\Basket\Standard::class )
81
-			->setConstructorArgs( [$this->context] )
82
-			->setMethods( ['setAddress'] )
80
+		$object = $this->getMockBuilder(\Aimeos\Controller\Frontend\Basket\Standard::class)
81
+			->setConstructorArgs([$this->context])
82
+			->setMethods(['setAddress'])
83 83
 			->getMock();
84 84
 
85
-		$object->expects( $this->once() )->method( 'setAddress' )->will( $this->throwException( new \Exception() ) );
85
+		$object->expects($this->once())->method('setAddress')->will($this->throwException(new \Exception()));
86 86
 
87
-		$result = $this->access( 'copyAddresses' )->invokeArgs( $object, [$ordBaseItem, [], 'unittest|en|EUR'] );
87
+		$result = $this->access('copyAddresses')->invokeArgs($object, [$ordBaseItem, [], 'unittest|en|EUR']);
88 88
 
89
-		$this->assertEquals( 1, count( $result ) );
90
-		$this->assertArrayHasKey( 'address', $result );
89
+		$this->assertEquals(1, count($result));
90
+		$this->assertArrayHasKey('address', $result);
91 91
 	}
92 92
 
93 93
 
94 94
 	public function testCopyCoupon()
95 95
 	{
96
-		$manager = \Aimeos\MShop\Factory::createManager( \TestHelperFrontend::getContext(), 'order/base' );
96
+		$manager = \Aimeos\MShop\Factory::createManager(\TestHelperFrontend::getContext(), 'order/base');
97 97
 		$ordBaseItem = $manager->createItem();
98 98
 
99
-		$product = \Aimeos\MShop\Factory::createManager( $this->context, 'product' )->findItem( 'CNC', ['price'] );
100
-		$ordProdManager = \Aimeos\MShop\Factory::createManager( $this->context, 'order/base/product' );
101
-		$ordProdItem = $ordProdManager->createItem()->copyFrom( $product );
99
+		$product = \Aimeos\MShop\Factory::createManager($this->context, 'product')->findItem('CNC', ['price']);
100
+		$ordProdManager = \Aimeos\MShop\Factory::createManager($this->context, 'order/base/product');
101
+		$ordProdItem = $ordProdManager->createItem()->copyFrom($product);
102 102
 
103
-		$priceItems = $product->getRefItems( 'price' );
104
-		$ordProdItem->setPrice( reset( $priceItems ) );
103
+		$priceItems = $product->getRefItems('price');
104
+		$ordProdItem->setPrice(reset($priceItems));
105 105
 
106
-		$ordBaseItem->addProduct( $ordProdItem );
107
-		$ordBaseItem->addCoupon( 'OPQR', [] );
106
+		$ordBaseItem->addProduct($ordProdItem);
107
+		$ordBaseItem->addCoupon('OPQR', []);
108 108
 
109 109
 
110
-		$object = $this->getMockBuilder( \Aimeos\Controller\Frontend\Basket\Standard::class )
111
-			->setConstructorArgs( [$this->context] )
112
-			->setMethods( null )
110
+		$object = $this->getMockBuilder(\Aimeos\Controller\Frontend\Basket\Standard::class)
111
+			->setConstructorArgs([$this->context])
112
+			->setMethods(null)
113 113
 			->getMock();
114 114
 
115
-		$object->addProduct( $product->getId() );
115
+		$object->addProduct($product->getId());
116 116
 
117
-		$result = $this->access( 'copyCoupons' )->invokeArgs( $object, [$ordBaseItem, ['test'], 'unittest|en|EUR'] );
117
+		$result = $this->access('copyCoupons')->invokeArgs($object, [$ordBaseItem, ['test'], 'unittest|en|EUR']);
118 118
 
119 119
 
120
-		$this->assertEquals( ['test'], $result );
120
+		$this->assertEquals(['test'], $result);
121 121
 
122 122
 		$object->clear();
123 123
 	}
@@ -125,48 +125,48 @@  discard block
 block discarded – undo
125 125
 
126 126
 	public function testCopyCouponException()
127 127
 	{
128
-		$manager = \Aimeos\MShop\Factory::createManager( \TestHelperFrontend::getContext(), 'order/base' );
128
+		$manager = \Aimeos\MShop\Factory::createManager(\TestHelperFrontend::getContext(), 'order/base');
129 129
 		$ordBaseItem = $manager->createItem();
130 130
 
131
-		$ordBaseItem->addCoupon( '90AB', [] );
131
+		$ordBaseItem->addCoupon('90AB', []);
132 132
 
133 133
 
134
-		$object = $this->getMockBuilder( \Aimeos\Controller\Frontend\Basket\Standard::class )
135
-			->setConstructorArgs( [$this->context] )
136
-			->setMethods( ['addCoupon'] )
134
+		$object = $this->getMockBuilder(\Aimeos\Controller\Frontend\Basket\Standard::class)
135
+			->setConstructorArgs([$this->context])
136
+			->setMethods(['addCoupon'])
137 137
 			->getMock();
138 138
 
139
-		$object->expects( $this->once() )->method( 'addCoupon' )->will( $this->throwException( new \Exception() ) );
139
+		$object->expects($this->once())->method('addCoupon')->will($this->throwException(new \Exception()));
140 140
 
141
-		$result = $this->access( 'copyCoupons' )->invokeArgs( $object, [$ordBaseItem, [], 'unittest|en|EUR'] );
141
+		$result = $this->access('copyCoupons')->invokeArgs($object, [$ordBaseItem, [], 'unittest|en|EUR']);
142 142
 
143
-		$this->assertEquals( 1, count( $result ) );
144
-		$this->assertArrayHasKey( 'coupon', $result );
143
+		$this->assertEquals(1, count($result));
144
+		$this->assertArrayHasKey('coupon', $result);
145 145
 	}
146 146
 
147 147
 
148 148
 	public function testCopyProduct()
149 149
 	{
150
-		$manager = \Aimeos\MShop\Factory::createManager( \TestHelperFrontend::getContext(), 'order/base' );
150
+		$manager = \Aimeos\MShop\Factory::createManager(\TestHelperFrontend::getContext(), 'order/base');
151 151
 		$ordBaseItem = $manager->createItem();
152 152
 
153
-		$product = \Aimeos\MShop\Factory::createManager( $this->context, 'product' )->findItem( 'CNC' );
154
-		$ordProdManager = \Aimeos\MShop\Factory::createManager( $this->context, 'order/base/product' );
155
-		$ordProdItem = $ordProdManager->createItem()->copyFrom( $product );
156
-		$ordBaseItem->addProduct( $ordProdItem );
153
+		$product = \Aimeos\MShop\Factory::createManager($this->context, 'product')->findItem('CNC');
154
+		$ordProdManager = \Aimeos\MShop\Factory::createManager($this->context, 'order/base/product');
155
+		$ordProdItem = $ordProdManager->createItem()->copyFrom($product);
156
+		$ordBaseItem->addProduct($ordProdItem);
157 157
 
158 158
 
159
-		$object = $this->getMockBuilder( \Aimeos\Controller\Frontend\Basket\Standard::class )
160
-			->setConstructorArgs( [$this->context] )
161
-			->setMethods( null )
159
+		$object = $this->getMockBuilder(\Aimeos\Controller\Frontend\Basket\Standard::class)
160
+			->setConstructorArgs([$this->context])
161
+			->setMethods(null)
162 162
 			->getMock();
163 163
 
164
-		$result = $this->access( 'copyProducts' )->invokeArgs( $object, [$ordBaseItem, ['test'], 'unittest|en|EUR'] );
164
+		$result = $this->access('copyProducts')->invokeArgs($object, [$ordBaseItem, ['test'], 'unittest|en|EUR']);
165 165
 
166 166
 
167
-		$this->assertEquals( ['test'], $result );
168
-		$this->assertEquals( 1, count( $object->get()->getProducts() ) );
169
-		$this->assertInstanceOf( \Aimeos\MShop\Order\Item\Base\Product\Iface::class, $object->get()->getProduct( 0 ) );
167
+		$this->assertEquals(['test'], $result);
168
+		$this->assertEquals(1, count($object->get()->getProducts()));
169
+		$this->assertInstanceOf(\Aimeos\MShop\Order\Item\Base\Product\Iface::class, $object->get()->getProduct(0));
170 170
 
171 171
 		$object->clear();
172 172
 	}
@@ -174,58 +174,58 @@  discard block
 block discarded – undo
174 174
 
175 175
 	public function testCopyProductException()
176 176
 	{
177
-		$manager = \Aimeos\MShop\Factory::createManager( \TestHelperFrontend::getContext(), 'order/base' );
177
+		$manager = \Aimeos\MShop\Factory::createManager(\TestHelperFrontend::getContext(), 'order/base');
178 178
 		$ordBaseItem = $manager->createItem();
179 179
 
180
-		$product = \Aimeos\MShop\Factory::createManager( $this->context, 'product' )->findItem( 'CNC' );
181
-		$ordProdManager = \Aimeos\MShop\Factory::createManager( $this->context, 'order/base/product' );
182
-		$ordProdItem = $ordProdManager->createItem()->copyFrom( $product );
183
-		$ordBaseItem->addProduct( $ordProdItem );
180
+		$product = \Aimeos\MShop\Factory::createManager($this->context, 'product')->findItem('CNC');
181
+		$ordProdManager = \Aimeos\MShop\Factory::createManager($this->context, 'order/base/product');
182
+		$ordProdItem = $ordProdManager->createItem()->copyFrom($product);
183
+		$ordBaseItem->addProduct($ordProdItem);
184 184
 
185 185
 
186
-		$object = $this->getMockBuilder( \Aimeos\Controller\Frontend\Basket\Standard::class )
187
-			->setConstructorArgs( [$this->context] )
188
-			->setMethods( ['addProduct'] )
186
+		$object = $this->getMockBuilder(\Aimeos\Controller\Frontend\Basket\Standard::class)
187
+			->setConstructorArgs([$this->context])
188
+			->setMethods(['addProduct'])
189 189
 			->getMock();
190 190
 
191
-		$object->expects( $this->once() )->method( 'addProduct' )->will( $this->throwException( new \Exception() ) );
191
+		$object->expects($this->once())->method('addProduct')->will($this->throwException(new \Exception()));
192 192
 
193
-		$result = $this->access( 'copyProducts' )->invokeArgs( $object, [$ordBaseItem, [], 'unittest|en|EUR'] );
193
+		$result = $this->access('copyProducts')->invokeArgs($object, [$ordBaseItem, [], 'unittest|en|EUR']);
194 194
 
195
-		$this->assertEquals( 1, count( $result ) );
196
-		$this->assertArrayHasKey( 'product', $result );
195
+		$this->assertEquals(1, count($result));
196
+		$this->assertArrayHasKey('product', $result);
197 197
 	}
198 198
 
199 199
 
200 200
 	public function testCopyServices()
201 201
 	{
202
-		$manager = \Aimeos\MShop\Factory::createManager( \TestHelperFrontend::getContext(), 'order/base' );
202
+		$manager = \Aimeos\MShop\Factory::createManager(\TestHelperFrontend::getContext(), 'order/base');
203 203
 		$ordBaseItem = $manager->createItem();
204 204
 
205
-		$serviceManager = \Aimeos\MShop\Factory::createManager( $this->context, 'service' );
206
-		$ordServManager = \Aimeos\MShop\Factory::createManager( $this->context, 'order/base/service' );
205
+		$serviceManager = \Aimeos\MShop\Factory::createManager($this->context, 'service');
206
+		$ordServManager = \Aimeos\MShop\Factory::createManager($this->context, 'order/base/service');
207 207
 
208
-		$serviceItem = $serviceManager->findItem( 'unitcode', [], 'service', 'delivery' );
209
-		$ordServItem = $ordServManager->createItem()->copyFrom( $serviceItem );
208
+		$serviceItem = $serviceManager->findItem('unitcode', [], 'service', 'delivery');
209
+		$ordServItem = $ordServManager->createItem()->copyFrom($serviceItem);
210 210
 
211
-		$ordBaseItem->addService( $ordServItem, \Aimeos\MShop\Order\Item\Base\Service\Base::TYPE_DELIVERY );
211
+		$ordBaseItem->addService($ordServItem, \Aimeos\MShop\Order\Item\Base\Service\Base::TYPE_DELIVERY);
212 212
 
213 213
 
214
-		$object = $this->getMockBuilder( \Aimeos\Controller\Frontend\Basket\Standard::class )
215
-			->setConstructorArgs( [$this->context] )
216
-			->setMethods( null )
214
+		$object = $this->getMockBuilder(\Aimeos\Controller\Frontend\Basket\Standard::class)
215
+			->setConstructorArgs([$this->context])
216
+			->setMethods(null)
217 217
 			->getMock();
218 218
 
219
-		$result = $this->access( 'copyServices' )->invokeArgs( $object, [$ordBaseItem, ['test'], 'unittest|en|EUR'] );
219
+		$result = $this->access('copyServices')->invokeArgs($object, [$ordBaseItem, ['test'], 'unittest|en|EUR']);
220 220
 
221 221
 
222
-		$this->assertEquals( ['test'], $result );
223
-		$this->assertEquals( 1, count( $object->get()->getServices() ) );
222
+		$this->assertEquals(['test'], $result);
223
+		$this->assertEquals(1, count($object->get()->getServices()));
224 224
 
225
-		$services = $object->get()->getService( \Aimeos\MShop\Order\Item\Base\Service\Base::TYPE_DELIVERY );
225
+		$services = $object->get()->getService(\Aimeos\MShop\Order\Item\Base\Service\Base::TYPE_DELIVERY);
226 226
 
227
-		foreach( $services as $service ) {
228
-			$this->assertInstanceOf( \Aimeos\MShop\Order\Item\Base\Service\Iface::class, $service );
227
+		foreach ($services as $service) {
228
+			$this->assertInstanceOf(\Aimeos\MShop\Order\Item\Base\Service\Iface::class, $service);
229 229
 		}
230 230
 
231 231
 		$object->clear();
@@ -234,106 +234,106 @@  discard block
 block discarded – undo
234 234
 
235 235
 	public function testCopyServicesException()
236 236
 	{
237
-		$manager = \Aimeos\MShop\Factory::createManager( \TestHelperFrontend::getContext(), 'order/base' );
237
+		$manager = \Aimeos\MShop\Factory::createManager(\TestHelperFrontend::getContext(), 'order/base');
238 238
 		$ordBaseItem = $manager->createItem();
239 239
 
240
-		$serviceManager = \Aimeos\MShop\Factory::createManager( $this->context, 'service' );
241
-		$ordServManager = \Aimeos\MShop\Factory::createManager( $this->context, 'order/base/service' );
240
+		$serviceManager = \Aimeos\MShop\Factory::createManager($this->context, 'service');
241
+		$ordServManager = \Aimeos\MShop\Factory::createManager($this->context, 'order/base/service');
242 242
 
243
-		$serviceItem = $serviceManager->findItem( 'unitcode', [], 'service', 'delivery' );
244
-		$ordServItem = $ordServManager->createItem()->copyFrom( $serviceItem );
243
+		$serviceItem = $serviceManager->findItem('unitcode', [], 'service', 'delivery');
244
+		$ordServItem = $ordServManager->createItem()->copyFrom($serviceItem);
245 245
 
246
-		$ordBaseItem->addService( $ordServItem, \Aimeos\MShop\Order\Item\Base\Service\Base::TYPE_DELIVERY );
246
+		$ordBaseItem->addService($ordServItem, \Aimeos\MShop\Order\Item\Base\Service\Base::TYPE_DELIVERY);
247 247
 
248 248
 
249
-		$object = $this->getMockBuilder( \Aimeos\Controller\Frontend\Basket\Standard::class )
250
-			->setConstructorArgs( [$this->context] )
251
-			->setMethods( ['addService'] )
249
+		$object = $this->getMockBuilder(\Aimeos\Controller\Frontend\Basket\Standard::class)
250
+			->setConstructorArgs([$this->context])
251
+			->setMethods(['addService'])
252 252
 			->getMock();
253 253
 
254
-		$object->expects( $this->once() )->method( 'addService' )->will( $this->throwException( new \Exception() ) );
254
+		$object->expects($this->once())->method('addService')->will($this->throwException(new \Exception()));
255 255
 
256
-		$result = $this->access( 'copyServices' )->invokeArgs( $object, [$ordBaseItem, [], 'unittest|en|EUR'] );
256
+		$result = $this->access('copyServices')->invokeArgs($object, [$ordBaseItem, [], 'unittest|en|EUR']);
257 257
 
258
-		$this->assertEquals( 0, count( $result ) );
258
+		$this->assertEquals(0, count($result));
259 259
 	}
260 260
 
261 261
 
262 262
 	public function testCreateSubscriptions()
263 263
 	{
264
-		$baseManager = \Aimeos\MShop\Factory::createManager( $this->context, 'order/base' );
264
+		$baseManager = \Aimeos\MShop\Factory::createManager($this->context, 'order/base');
265 265
 
266 266
 		$search = $baseManager->createSearch();
267
-		$search->setConditions( $search->compare( '==', 'order.base.price', '53.50' ) );
267
+		$search->setConditions($search->compare('==', 'order.base.price', '53.50'));
268 268
 
269
-		$items = $baseManager->searchItems( $search, ['order/base/product'] );
269
+		$items = $baseManager->searchItems($search, ['order/base/product']);
270 270
 
271
-		if( ( $basket = reset( $items ) ) === false ) {
272
-			throw new \Exception( sprintf( 'No order base item found for price "%1$s"', '53,50' ) );
271
+		if (($basket = reset($items)) === false) {
272
+			throw new \Exception(sprintf('No order base item found for price "%1$s"', '53,50'));
273 273
 		}
274 274
 
275
-		$object = $this->getMockBuilder( \Aimeos\Controller\Frontend\Basket\Standard::class )
276
-			->setConstructorArgs( [$this->context] )
277
-			->setMethods( ['getAttributes'] )
275
+		$object = $this->getMockBuilder(\Aimeos\Controller\Frontend\Basket\Standard::class)
276
+			->setConstructorArgs([$this->context])
277
+			->setMethods(['getAttributes'])
278 278
 			->getMock();
279 279
 
280
-		$stub = $this->getMockBuilder( \Aimeos\MShop\Subscription\Manager\Standard::class )
281
-			->setConstructorArgs( [$this->context] )
282
-			->setMethods( ['saveItem'] )
280
+		$stub = $this->getMockBuilder(\Aimeos\MShop\Subscription\Manager\Standard::class)
281
+			->setConstructorArgs([$this->context])
282
+			->setMethods(['saveItem'])
283 283
 			->getMock();
284 284
 
285
-		\Aimeos\MShop::inject( $this->context, 'subscription', $stub );
285
+		\Aimeos\MShop::inject($this->context, 'subscription', $stub);
286 286
 
287
-		$stub->expects( $this->exactly( 2 ) )->method( 'saveItem' );
287
+		$stub->expects($this->exactly(2))->method('saveItem');
288 288
 
289
-		$this->access( 'createSubscriptions' )->invokeArgs( $object, [$basket] );
289
+		$this->access('createSubscriptions')->invokeArgs($object, [$basket]);
290 290
 	}
291 291
 
292 292
 
293 293
 	public function testGetOrderProductAttributes()
294 294
 	{
295
-		$object = $this->getMockBuilder( \Aimeos\Controller\Frontend\Basket\Standard::class )
296
-			->setConstructorArgs( [$this->context] )
297
-			->setMethods( ['getAttributes'] )
295
+		$object = $this->getMockBuilder(\Aimeos\Controller\Frontend\Basket\Standard::class)
296
+			->setConstructorArgs([$this->context])
297
+			->setMethods(['getAttributes'])
298 298
 			->getMock();
299 299
 
300
-		$list = [1 => new \Aimeos\MShop\Attribute\Item\Standard( ['attribute.code' => 'special_instructions'] )];
301
-		$object->expects( $this->once() )->method( 'getAttributes' )->will( $this->returnValue( $list ) );
300
+		$list = [1 => new \Aimeos\MShop\Attribute\Item\Standard(['attribute.code' => 'special_instructions'])];
301
+		$object->expects($this->once())->method('getAttributes')->will($this->returnValue($list));
302 302
 
303
-		$result = $this->access( 'getOrderProductAttributes' )->invokeArgs( $object, ['test', ['1'], ['1' => 'test']] );
303
+		$result = $this->access('getOrderProductAttributes')->invokeArgs($object, ['test', ['1'], ['1' => 'test']]);
304 304
 
305
-		$this->assertEquals( 1, count( $result ) );
306
-		$this->assertEquals( 'test', $result[0]->getValue() );
305
+		$this->assertEquals(1, count($result));
306
+		$this->assertEquals('test', $result[0]->getValue());
307 307
 	}
308 308
 
309 309
 
310 310
 	/**
311 311
 	 * @param string $company
312 312
 	 */
313
-	protected function getAddress( $company )
313
+	protected function getAddress($company)
314 314
 	{
315
-		$manager = \Aimeos\MShop\Factory::createManager( \TestHelperFrontend::getContext(), 'customer/address' );
315
+		$manager = \Aimeos\MShop\Factory::createManager(\TestHelperFrontend::getContext(), 'customer/address');
316 316
 
317 317
 		$search = $manager->createSearch();
318
-		$search->setConditions( $search->compare( '==', 'customer.address.company', $company ) );
319
-		$items = $manager->searchItems( $search );
318
+		$search->setConditions($search->compare('==', 'customer.address.company', $company));
319
+		$items = $manager->searchItems($search);
320 320
 
321
-		if( ( $item = reset( $items ) ) === false ) {
322
-			throw new \RuntimeException( sprintf( 'No address item with company "%1$s" found', $company ) );
321
+		if (($item = reset($items)) === false) {
322
+			throw new \RuntimeException(sprintf('No address item with company "%1$s" found', $company));
323 323
 		}
324 324
 
325
-		$ordAddrManager = \Aimeos\MShop\Factory::createManager( \TestHelperFrontend::getContext(), 'order/base/address' );
326
-		$ordAddrItem = $ordAddrManager->createItem()->copyFrom( $item );
325
+		$ordAddrManager = \Aimeos\MShop\Factory::createManager(\TestHelperFrontend::getContext(), 'order/base/address');
326
+		$ordAddrItem = $ordAddrManager->createItem()->copyFrom($item);
327 327
 
328 328
 		return $ordAddrItem;
329 329
 	}
330 330
 
331 331
 
332
-	protected function access( $name )
332
+	protected function access($name)
333 333
 	{
334
-		$class = new \ReflectionClass( \Aimeos\Controller\Frontend\Basket\Base::class );
335
-		$method = $class->getMethod( $name );
336
-		$method->setAccessible( true );
334
+		$class = new \ReflectionClass(\Aimeos\Controller\Frontend\Basket\Base::class);
335
+		$method = $class->getMethod($name);
336
+		$method->setAccessible(true);
337 337
 
338 338
 		return $method;
339 339
 	}
Please login to merge, or discard this patch.