Completed
Push — master ( 937f3b...5e44e2 )
by Aimeos
16:51 queued 07:27
created
controller/frontend/tests/Controller/Frontend/Basket/Decorator/BaseTest.php 1 patch
Spacing   +39 added lines, -39 removed lines patch added patch discarded remove patch
@@ -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' )
23
+		$this->stub = $this->getMockBuilder('\Aimeos\Controller\Frontend\Basket\Standard')
24 24
 			->disableOriginalConstructor()
25 25
 			->getMock();
26 26
 
27
-		$this->object = $this->getMockBuilder( '\Aimeos\Controller\Frontend\Basket\Decorator\Base' )
28
-			->setConstructorArgs( [$this->stub, $this->context] )
27
+		$this->object = $this->getMockBuilder('\Aimeos\Controller\Frontend\Basket\Decorator\Base')
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' )->getMock();
41
+		$stub = $this->getMockBuilder('\Aimeos\Controller\Frontend\Iface')->getMock();
42 42
 
43
-		$this->setExpectedException( '\Aimeos\Controller\Frontend\Exception' );
43
+		$this->setExpectedException('\Aimeos\Controller\Frontend\Exception');
44 44
 
45
-		$this->getMockBuilder( '\Aimeos\Controller\Frontend\Basket\Decorator\Base' )
46
-			->setConstructorArgs( [$stub, $this->context] )
45
+		$this->getMockBuilder('\Aimeos\Controller\Frontend\Basket\Decorator\Base')
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' )
53
+		$stub = $this->getMockBuilder('\Aimeos\Controller\Frontend\Basket\Standard')
54 54
 			->disableOriginalConstructor()
55
-			->setMethods( ['invalid'] )
55
+			->setMethods(['invalid'])
56 56
 			->getMock();
57 57
 
58
-		$object = $this->getMockBuilder( '\Aimeos\Controller\Frontend\Basket\Decorator\Base' )
59
-			->setConstructorArgs( [$stub, $this->context] )
58
+		$object = $this->getMockBuilder('\Aimeos\Controller\Frontend\Basket\Decorator\Base')
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\Factory::createManager( $context, 'order/base' )->createItem();
79
+		$order = \Aimeos\MShop\Factory::createManager($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', $this->object->get() );
84
+		$this->assertInstanceOf('\Aimeos\MShop\Order\Item\Base\Iface', $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,7 +95,7 @@  discard block
 block discarded – undo
95 95
 
96 96
 	public function testAddProduct()
97 97
 	{
98
-		$this->stub->expects( $this->once() )->method( 'addProduct' );
98
+		$this->stub->expects($this->once())->method('addProduct');
99 99
 
100 100
 		$this->object->addProduct( -1 );
101 101
 	}
@@ -103,65 +103,65 @@  discard block
 block discarded – undo
103 103
 
104 104
 	public function testDeleteProduct()
105 105
 	{
106
-		$this->stub->expects( $this->once() )->method( 'deleteProduct' );
106
+		$this->stub->expects($this->once())->method('deleteProduct');
107 107
 
108
-		$this->object->deleteProduct( 0 );
108
+		$this->object->deleteProduct(0);
109 109
 	}
110 110
 
111 111
 
112 112
 	public function testEditProduct()
113 113
 	{
114
-		$this->stub->expects( $this->once() )->method( 'editProduct' );
114
+		$this->stub->expects($this->once())->method('editProduct');
115 115
 
116
-		$this->object->editProduct( 0, 1 );
116
+		$this->object->editProduct(0, 1);
117 117
 	}
118 118
 
119 119
 
120 120
 	public function testAddCoupon()
121 121
 	{
122
-		$this->stub->expects( $this->once() )->method( 'addCoupon' );
122
+		$this->stub->expects($this->once())->method('addCoupon');
123 123
 
124
-		$this->object->addCoupon( 'test' );
124
+		$this->object->addCoupon('test');
125 125
 	}
126 126
 
127 127
 
128 128
 	public function testDeleteCoupon()
129 129
 	{
130
-		$this->stub->expects( $this->once() )->method( 'deleteCoupon' );
130
+		$this->stub->expects($this->once())->method('deleteCoupon');
131 131
 
132
-		$this->object->deleteCoupon( 'test' );
132
+		$this->object->deleteCoupon('test');
133 133
 	}
134 134
 
135 135
 
136 136
 	public function testSetAddress()
137 137
 	{
138
-		$this->stub->expects( $this->once() )->method( 'setAddress' );
138
+		$this->stub->expects($this->once())->method('setAddress');
139 139
 
140
-		$this->object->setAddress( \Aimeos\MShop\Order\Item\Base\Address\Base::TYPE_PAYMENT, null );
140
+		$this->object->setAddress(\Aimeos\MShop\Order\Item\Base\Address\Base::TYPE_PAYMENT, null);
141 141
 	}
142 142
 
143 143
 
144 144
 	public function testSetService()
145 145
 	{
146
-		$this->stub->expects( $this->once() )->method( 'setService' );
146
+		$this->stub->expects($this->once())->method('setService');
147 147
 
148
-		$this->object->setService( \Aimeos\MShop\Order\Item\Base\Service\Base::TYPE_PAYMENT, -1 );
148
+		$this->object->setService(\Aimeos\MShop\Order\Item\Base\Service\Base::TYPE_PAYMENT, -1);
149 149
 	}
150 150
 
151 151
 
152 152
 	public function testGetController()
153 153
 	{
154
-		$result = $this->access( 'getController' )->invokeArgs( $this->object, [] );
154
+		$result = $this->access('getController')->invokeArgs($this->object, []);
155 155
 
156
-		$this->assertSame( $this->stub, $result );
156
+		$this->assertSame($this->stub, $result);
157 157
 	}
158 158
 
159 159
 
160
-	protected function access( $name )
160
+	protected function access($name)
161 161
 	{
162
-		$class = new \ReflectionClass( '\Aimeos\Controller\Frontend\Basket\Decorator\Base' );
163
-		$method = $class->getMethod( $name );
164
-		$method->setAccessible( true );
162
+		$class = new \ReflectionClass('\Aimeos\Controller\Frontend\Basket\Decorator\Base');
163
+		$method = $class->getMethod($name);
164
+		$method->setAccessible(true);
165 165
 
166 166
 		return $method;
167 167
 	}
Please login to merge, or discard this patch.
frontend/tests/Controller/Frontend/Product/Decorator/BaseTest.php 1 patch
Spacing   +51 added lines, -51 removed lines patch added patch discarded remove patch
@@ -20,150 +20,150 @@
 block discarded – undo
20 20
 	{
21 21
 		$this->context = \TestHelperFrontend::getContext();
22 22
 
23
-		$this->stub = $this->getMockBuilder( '\Aimeos\Controller\Frontend\Product\Standard' )
23
+		$this->stub = $this->getMockBuilder('\Aimeos\Controller\Frontend\Product\Standard')
24 24
 			->disableOriginalConstructor()
25 25
 			->getMock();
26 26
 
27
-		$this->object = $this->getMockBuilder( '\Aimeos\Controller\Frontend\Product\Decorator\Base' )
28
-			->setConstructorArgs( [$this->stub, $this->context] )
27
+		$this->object = $this->getMockBuilder('\Aimeos\Controller\Frontend\Product\Decorator\Base')
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' )->getMock();
41
+		$stub = $this->getMockBuilder('\Aimeos\Controller\Frontend\Iface')->getMock();
42 42
 
43
-		$this->setExpectedException( '\Aimeos\Controller\Frontend\Exception' );
43
+		$this->setExpectedException('\Aimeos\Controller\Frontend\Exception');
44 44
 
45
-		$this->getMockBuilder( '\Aimeos\Controller\Frontend\Product\Decorator\Base' )
46
-			->setConstructorArgs( [$stub, $this->context] )
45
+		$this->getMockBuilder('\Aimeos\Controller\Frontend\Product\Decorator\Base')
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\Product\Standard' )
53
+		$stub = $this->getMockBuilder('\Aimeos\Controller\Frontend\Product\Standard')
54 54
 			->disableOriginalConstructor()
55
-			->setMethods( ['invalid'] )
55
+			->setMethods(['invalid'])
56 56
 			->getMock();
57 57
 
58
-		$object = $this->getMockBuilder( '\Aimeos\Controller\Frontend\Product\Decorator\Base' )
59
-			->setConstructorArgs( [$stub, $this->context] )
58
+		$object = $this->getMockBuilder('\Aimeos\Controller\Frontend\Product\Decorator\Base')
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 testAddFilterAttribute()
69 69
 	{
70
-		$search = \Aimeos\MShop\Factory::createManager( $this->context, 'index' )->createSearch();
70
+		$search = \Aimeos\MShop\Factory::createManager($this->context, 'index')->createSearch();
71 71
 
72
-		$this->stub->expects( $this->once() )->method( 'addFilterAttribute' )
73
-			->will( $this->returnArgument( 0 ) );
72
+		$this->stub->expects($this->once())->method('addFilterAttribute')
73
+			->will($this->returnArgument(0));
74 74
 
75
-		$this->assertInstanceOf( '\Aimeos\MW\Criteria\Iface', $this->object->addFilterAttribute( $search, [], [], [] ) );
75
+		$this->assertInstanceOf('\Aimeos\MW\Criteria\Iface', $this->object->addFilterAttribute($search, [], [], []));
76 76
 	}
77 77
 
78 78
 
79 79
 	public function testAddFilterCategory()
80 80
 	{
81
-		$search = \Aimeos\MShop\Factory::createManager( $this->context, 'index' )->createSearch();
81
+		$search = \Aimeos\MShop\Factory::createManager($this->context, 'index')->createSearch();
82 82
 
83
-		$this->stub->expects( $this->once() )->method( 'addFilterCategory' )
84
-			->will( $this->returnArgument( 0 ) );
83
+		$this->stub->expects($this->once())->method('addFilterCategory')
84
+			->will($this->returnArgument(0));
85 85
 
86
-		$this->assertInstanceOf( '\Aimeos\MW\Criteria\Iface', $this->object->addFilterCategory( $search, -1 ) );
86
+		$this->assertInstanceOf('\Aimeos\MW\Criteria\Iface', $this->object->addFilterCategory($search, -1));
87 87
 	}
88 88
 
89 89
 
90 90
 	public function testAddFilterText()
91 91
 	{
92
-		$search = \Aimeos\MShop\Factory::createManager( $this->context, 'index' )->createSearch();
92
+		$search = \Aimeos\MShop\Factory::createManager($this->context, 'index')->createSearch();
93 93
 
94
-		$this->stub->expects( $this->once() )->method( 'addFilterText' )
95
-			->will( $this->returnArgument( 0 ) );
94
+		$this->stub->expects($this->once())->method('addFilterText')
95
+			->will($this->returnArgument(0));
96 96
 
97
-		$this->assertInstanceOf( '\Aimeos\MW\Criteria\Iface', $this->object->addFilterText( $search, 'test' ) );
97
+		$this->assertInstanceOf('\Aimeos\MW\Criteria\Iface', $this->object->addFilterText($search, 'test'));
98 98
 	}
99 99
 
100 100
 
101 101
 	public function testAggregate()
102 102
 	{
103
-		$search = \Aimeos\MShop\Factory::createManager( $this->context, 'index' )->createSearch();
103
+		$search = \Aimeos\MShop\Factory::createManager($this->context, 'index')->createSearch();
104 104
 
105
-		$this->stub->expects( $this->once() )->method( 'aggregate' )
106
-			->will( $this->returnValue( [] ) );
105
+		$this->stub->expects($this->once())->method('aggregate')
106
+			->will($this->returnValue([]));
107 107
 
108
-		$this->assertEquals( [], $this->object->aggregate( $search, 'test' ) );
108
+		$this->assertEquals([], $this->object->aggregate($search, 'test'));
109 109
 	}
110 110
 
111 111
 
112 112
 	public function testCreateFilter()
113 113
 	{
114
-		$search = \Aimeos\MShop\Factory::createManager( $this->context, 'index' )->createSearch();
114
+		$search = \Aimeos\MShop\Factory::createManager($this->context, 'index')->createSearch();
115 115
 
116
-		$this->stub->expects( $this->once() )->method( 'createFilter' )
117
-			->will( $this->returnValue( $search ) );
116
+		$this->stub->expects($this->once())->method('createFilter')
117
+			->will($this->returnValue($search));
118 118
 
119
-		$this->assertInstanceOf( '\Aimeos\MW\Criteria\Iface', $this->object->createFilter() );
119
+		$this->assertInstanceOf('\Aimeos\MW\Criteria\Iface', $this->object->createFilter());
120 120
 	}
121 121
 
122 122
 
123 123
 	public function testGetItem()
124 124
 	{
125
-		$prodItem = \Aimeos\MShop\Factory::createManager( $this->context, 'index' )->createItem();
125
+		$prodItem = \Aimeos\MShop\Factory::createManager($this->context, 'index')->createItem();
126 126
 
127
-		$this->stub->expects( $this->once() )->method( 'getItem' )
128
-			->will( $this->returnValue( $prodItem ) );
127
+		$this->stub->expects($this->once())->method('getItem')
128
+			->will($this->returnValue($prodItem));
129 129
 
130
-		$this->assertInstanceOf( '\Aimeos\MShop\Product\Item\Iface', $this->object->getItem( -1 ) );
130
+		$this->assertInstanceOf('\Aimeos\MShop\Product\Item\Iface', $this->object->getItem( -1 ));
131 131
 	}
132 132
 
133 133
 
134 134
 	public function testGetItems()
135 135
 	{
136
-		$this->stub->expects( $this->once() )->method( 'getItems' )
137
-			->will( $this->returnValue( [] ) );
136
+		$this->stub->expects($this->once())->method('getItems')
137
+			->will($this->returnValue([]));
138 138
 
139
-		$this->assertEquals( [], $this->object->getItems( [-1] ) );
139
+		$this->assertEquals([], $this->object->getItems([-1]));
140 140
 	}
141 141
 
142 142
 
143 143
 	public function testSearchItems()
144 144
 	{
145
-		$search = \Aimeos\MShop\Factory::createManager( $this->context, 'index' )->createSearch();
145
+		$search = \Aimeos\MShop\Factory::createManager($this->context, 'index')->createSearch();
146 146
 
147
-		$this->stub->expects( $this->once() )->method( 'searchItems' )
148
-			->will( $this->returnValue( [] ) );
147
+		$this->stub->expects($this->once())->method('searchItems')
148
+			->will($this->returnValue([]));
149 149
 
150
-		$this->assertEquals( [], $this->object->searchItems( $search ) );
150
+		$this->assertEquals([], $this->object->searchItems($search));
151 151
 	}
152 152
 
153 153
 
154 154
 	public function testGetController()
155 155
 	{
156
-		$result = $this->access( 'getController' )->invokeArgs( $this->object, [] );
156
+		$result = $this->access('getController')->invokeArgs($this->object, []);
157 157
 
158
-		$this->assertSame( $this->stub, $result );
158
+		$this->assertSame($this->stub, $result);
159 159
 	}
160 160
 
161 161
 
162
-	protected function access( $name )
162
+	protected function access($name)
163 163
 	{
164
-		$class = new \ReflectionClass( '\Aimeos\Controller\Frontend\Product\Decorator\Base' );
165
-		$method = $class->getMethod( $name );
166
-		$method->setAccessible( true );
164
+		$class = new \ReflectionClass('\Aimeos\Controller\Frontend\Product\Decorator\Base');
165
+		$method = $class->getMethod($name);
166
+		$method->setAccessible(true);
167 167
 
168 168
 		return $method;
169 169
 	}
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
20 20
 	{
21 21
 		$this->context = \TestHelperFrontend::getContext();
22 22
 
23
-		$this->stub = $this->getMockBuilder( '\Aimeos\Controller\Frontend\Stock\Standard' )
23
+		$this->stub = $this->getMockBuilder('\Aimeos\Controller\Frontend\Stock\Standard')
24 24
 			->disableOriginalConstructor()
25 25
 			->getMock();
26 26
 
27
-		$this->object = $this->getMockBuilder( '\Aimeos\Controller\Frontend\Stock\Decorator\Base' )
28
-			->setConstructorArgs( [$this->stub, $this->context] )
27
+		$this->object = $this->getMockBuilder('\Aimeos\Controller\Frontend\Stock\Decorator\Base')
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' )->getMock();
41
+		$stub = $this->getMockBuilder('\Aimeos\Controller\Frontend\Iface')->getMock();
42 42
 
43
-		$this->setExpectedException( '\Aimeos\Controller\Frontend\Exception' );
43
+		$this->setExpectedException('\Aimeos\Controller\Frontend\Exception');
44 44
 
45
-		$this->getMockBuilder( '\Aimeos\Controller\Frontend\Stock\Decorator\Base' )
46
-			->setConstructorArgs( [$stub, $this->context] )
45
+		$this->getMockBuilder('\Aimeos\Controller\Frontend\Stock\Decorator\Base')
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' )
53
+		$stub = $this->getMockBuilder('\Aimeos\Controller\Frontend\Stock\Standard')
54 54
 			->disableOriginalConstructor()
55
-			->setMethods( ['invalid'] )
55
+			->setMethods(['invalid'])
56 56
 			->getMock();
57 57
 
58
-		$object = $this->getMockBuilder( '\Aimeos\Controller\Frontend\Stock\Decorator\Base' )
59
-			->setConstructorArgs( [$stub, $this->context] )
58
+		$object = $this->getMockBuilder('\Aimeos\Controller\Frontend\Stock\Decorator\Base')
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\Factory::createManager( $this->context, 'stock' )->createSearch();
70
+		$search = \Aimeos\MShop\Factory::createManager($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', $this->object->addFilterCodes( $search, [] ) );
75
+		$this->assertInstanceOf('\Aimeos\MW\Criteria\Iface', $this->object->addFilterCodes($search, []));
76 76
 	}
77 77
 
78 78
 
79 79
 	public function testAddFilterTypes()
80 80
 	{
81
-		$search = \Aimeos\MShop\Factory::createManager( $this->context, 'stock' )->createSearch();
81
+		$search = \Aimeos\MShop\Factory::createManager($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', $this->object->addFilterTypes( $search, [] ) );
86
+		$this->assertInstanceOf('\Aimeos\MW\Criteria\Iface', $this->object->addFilterTypes($search, []));
87 87
 	}
88 88
 
89 89
 
90 90
 	public function testCreateFilter()
91 91
 	{
92
-		$search = \Aimeos\MShop\Factory::createManager( $this->context, 'stock' )->createSearch();
92
+		$search = \Aimeos\MShop\Factory::createManager($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', $this->object->createFilter() );
97
+		$this->assertInstanceOf('\Aimeos\MW\Criteria\Iface', $this->object->createFilter());
98 98
 	}
99 99
 
100 100
 
101 101
 	public function testGetItem()
102 102
 	{
103
-		$item = \Aimeos\MShop\Factory::createManager( $this->context, 'stock' )->createItem();
103
+		$item = \Aimeos\MShop\Factory::createManager($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', $this->object->getItem( -1 ) );
108
+		$this->assertInstanceOf('\Aimeos\MShop\Stock\Item\Iface', $this->object->getItem( -1 ));
109 109
 	}
110 110
 
111 111
 
112 112
 	public function testSearchItems()
113 113
 	{
114
-		$filter = \Aimeos\MShop\Factory::createManager( $this->context, 'stock' )->createSearch();
114
+		$filter = \Aimeos\MShop\Factory::createManager($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' );
134
-		$method = $class->getMethod( $name );
135
-		$method->setAccessible( true );
133
+		$class = new \ReflectionClass('\Aimeos\Controller\Frontend\Stock\Decorator\Base');
134
+		$method = $class->getMethod($name);
135
+		$method->setAccessible(true);
136 136
 
137 137
 		return $method;
138 138
 	}
Please login to merge, or discard this patch.
controller/frontend/tests/Controller/Frontend/Product/StandardTest.php 1 patch
Spacing   +102 added lines, -102 removed lines patch added patch discarded remove patch
@@ -16,22 +16,22 @@  discard block
 block discarded – undo
16 16
 
17 17
 	protected function setUp()
18 18
 	{
19
-		$this->object = new \Aimeos\Controller\Frontend\Product\Standard( \TestHelperFrontend::getContext() );
19
+		$this->object = new \Aimeos\Controller\Frontend\Product\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
 
29 29
 	public function testAggregate()
30 30
 	{
31 31
 		$filter = $this->object->createFilter();
32
-		$list = $this->object->aggregate( $filter, 'index.attribute.id' );
32
+		$list = $this->object->aggregate($filter, 'index.attribute.id');
33 33
 
34
-		$this->assertGreaterThan( 0, count( $list ) );
34
+		$this->assertGreaterThan(0, count($list));
35 35
 	}
36 36
 
37 37
 
@@ -39,103 +39,103 @@  discard block
 block discarded – undo
39 39
 	{
40 40
 		$filter = $this->object->createFilter();
41 41
 
42
-		$this->assertInstanceOf( '\\Aimeos\\MW\\Criteria\\Iface', $filter );
43
-		$this->assertEquals( array(), $filter->getSortations() );
44
-		$this->assertEquals( 0, $filter->getSliceStart() );
45
-		$this->assertEquals( 100, $filter->getSliceSize() );
42
+		$this->assertInstanceOf('\\Aimeos\\MW\\Criteria\\Iface', $filter);
43
+		$this->assertEquals(array(), $filter->getSortations());
44
+		$this->assertEquals(0, $filter->getSliceStart());
45
+		$this->assertEquals(100, $filter->getSliceSize());
46 46
 	}
47 47
 
48 48
 
49 49
 	public function testAddFilterAttribute()
50 50
 	{
51 51
 		$filter = $this->object->createFilter();
52
-		$filter = $this->object->addFilterAttribute( $filter, array( 0, 1 ), array(), array() );
52
+		$filter = $this->object->addFilterAttribute($filter, array(0, 1), array(), array());
53 53
 
54 54
 		$list = $filter->getConditions()->getExpressions();
55 55
 
56
-		if( !isset( $list[0] ) || !( $list[0] instanceof \Aimeos\MW\Criteria\Expression\Compare\Iface ) ) {
57
-			throw new \RuntimeException( 'Wrong expression' );
56
+		if (!isset($list[0]) || !($list[0] instanceof \Aimeos\MW\Criteria\Expression\Compare\Iface)) {
57
+			throw new \RuntimeException('Wrong expression');
58 58
 		}
59 59
 
60
-		$this->assertEquals( 'index.attributeaggregate([0,1])', $list[0]->getName() );
61
-		$this->assertEquals( 2, $list[0]->getValue() );
60
+		$this->assertEquals('index.attributeaggregate([0,1])', $list[0]->getName());
61
+		$this->assertEquals(2, $list[0]->getValue());
62 62
 	}
63 63
 
64 64
 
65 65
 	public function testAddFilterAttributeOptions()
66 66
 	{
67 67
 		$filter = $this->object->createFilter();
68
-		$filter = $this->object->addFilterAttribute( $filter, array(), array( 1 ), array() );
68
+		$filter = $this->object->addFilterAttribute($filter, array(), array(1), array());
69 69
 
70 70
 		$list = $filter->getConditions()->getExpressions();
71 71
 
72
-		if( !isset( $list[0] ) || !( $list[0] instanceof \Aimeos\MW\Criteria\Expression\Compare\Iface ) ) {
73
-			throw new \RuntimeException( 'Wrong expression' );
72
+		if (!isset($list[0]) || !($list[0] instanceof \Aimeos\MW\Criteria\Expression\Compare\Iface)) {
73
+			throw new \RuntimeException('Wrong expression');
74 74
 		}
75 75
 
76
-		$this->assertEquals( 'index.attributeaggregate([1])', $list[0]->getName() );
77
-		$this->assertEquals( 0, $list[0]->getValue() );
76
+		$this->assertEquals('index.attributeaggregate([1])', $list[0]->getName());
77
+		$this->assertEquals(0, $list[0]->getValue());
78 78
 	}
79 79
 
80 80
 
81 81
 	public function testAddFilterAttributeOne()
82 82
 	{
83 83
 		$filter = $this->object->createFilter();
84
-		$filter = $this->object->addFilterAttribute( $filter, array(), array(), array( 'test' => array( 2 ) ) );
84
+		$filter = $this->object->addFilterAttribute($filter, array(), array(), array('test' => array(2)));
85 85
 
86 86
 		$list = $filter->getConditions()->getExpressions();
87 87
 
88
-		if( !isset( $list[0] ) || !( $list[0] instanceof \Aimeos\MW\Criteria\Expression\Compare\Iface ) ) {
89
-			throw new \RuntimeException( 'Wrong expression' );
88
+		if (!isset($list[0]) || !($list[0] instanceof \Aimeos\MW\Criteria\Expression\Compare\Iface)) {
89
+			throw new \RuntimeException('Wrong expression');
90 90
 		}
91 91
 
92
-		$this->assertEquals( 'index.attributeaggregate([2])', $list[0]->getName() );
93
-		$this->assertEquals( 0, $list[0]->getValue() );
92
+		$this->assertEquals('index.attributeaggregate([2])', $list[0]->getName());
93
+		$this->assertEquals(0, $list[0]->getValue());
94 94
 	}
95 95
 
96 96
 
97 97
 	public function testAddFilterCategory()
98 98
 	{
99 99
 		$context = \TestHelperFrontend::getContext();
100
-		$manager = \Aimeos\MShop\Factory::createManager( $context, 'catalog' );
100
+		$manager = \Aimeos\MShop\Factory::createManager($context, 'catalog');
101 101
 
102
-		$catId = $manager->findItem( 'root' )->getId();
102
+		$catId = $manager->findItem('root')->getId();
103 103
 		$level = \Aimeos\MW\Tree\Manager\Base::LEVEL_LIST;
104 104
 
105 105
 		$filter = $this->object->createFilter();
106
-		$filter = $this->object->addFilterCategory( $filter, $catId, $level );
106
+		$filter = $this->object->addFilterCategory($filter, $catId, $level);
107 107
 
108 108
 		$list = $filter->getConditions()->getExpressions();
109 109
 
110
-		if( !isset( $list[0] ) || !( $list[0] instanceof \Aimeos\MW\Criteria\Expression\Compare\Iface ) ) {
111
-			throw new \RuntimeException( 'Wrong expression' );
110
+		if (!isset($list[0]) || !($list[0] instanceof \Aimeos\MW\Criteria\Expression\Compare\Iface)) {
111
+			throw new \RuntimeException('Wrong expression');
112 112
 		}
113 113
 
114
-		$this->assertEquals( 'index.catalog.id', $list[0]->getName() );
115
-		$this->assertEquals( 3, count( $list[0]->getValue() ) );
116
-		$this->assertEquals( array(), $filter->getSortations() );
114
+		$this->assertEquals('index.catalog.id', $list[0]->getName());
115
+		$this->assertEquals(3, count($list[0]->getValue()));
116
+		$this->assertEquals(array(), $filter->getSortations());
117 117
 	}
118 118
 
119 119
 
120 120
 	public function testAddFilterText()
121 121
 	{
122 122
 		$filter = $this->object->createFilter();
123
-		$filter = $this->object->addFilterText( $filter, 'Espresso' );
123
+		$filter = $this->object->addFilterText($filter, 'Espresso');
124 124
 
125
-		$this->assertInstanceOf( '\\Aimeos\\MW\\Criteria\\Iface', $filter );
125
+		$this->assertInstanceOf('\\Aimeos\\MW\\Criteria\\Iface', $filter);
126 126
 
127 127
 		$list = $filter->getConditions()->getExpressions();
128 128
 
129 129
 
130
-		if( !isset( $list[0] ) || !( $list[0] instanceof \Aimeos\MW\Criteria\Expression\Compare\Iface ) ) {
131
-			throw new \RuntimeException( 'Wrong expression' );
130
+		if (!isset($list[0]) || !($list[0] instanceof \Aimeos\MW\Criteria\Expression\Compare\Iface)) {
131
+			throw new \RuntimeException('Wrong expression');
132 132
 		}
133
-		$this->assertEquals( 'index.text.relevance("default","de","Espresso")', $list[0]->getName() );
134
-		$this->assertEquals( 0, $list[0]->getValue() );
133
+		$this->assertEquals('index.text.relevance("default","de","Espresso")', $list[0]->getName());
134
+		$this->assertEquals(0, $list[0]->getValue());
135 135
 
136
-		$this->assertEquals( array(), $filter->getSortations() );
137
-		$this->assertEquals( 0, $filter->getSliceStart() );
138
-		$this->assertEquals( 100, $filter->getSliceSize() );
136
+		$this->assertEquals(array(), $filter->getSortations());
137
+		$this->assertEquals(0, $filter->getSliceStart());
138
+		$this->assertEquals(100, $filter->getSliceSize());
139 139
 	}
140 140
 
141 141
 
@@ -144,160 +144,160 @@  discard block
 block discarded – undo
144 144
 		$level = \Aimeos\MW\Tree\Manager\Base::LEVEL_ONE;
145 145
 
146 146
 		$filter = $this->object->createFilter();
147
-		$filter = $this->object->addFilterCategory( $filter, 0, $level, 'relevance', '-', 'test' );
147
+		$filter = $this->object->addFilterCategory($filter, 0, $level, 'relevance', '-', 'test');
148 148
 
149
-		$this->assertInstanceOf( '\\Aimeos\\MW\\Criteria\\Iface', $filter );
149
+		$this->assertInstanceOf('\\Aimeos\\MW\\Criteria\\Iface', $filter);
150 150
 
151 151
 		$sort = $filter->getSortations();
152
-		if( ( $item = reset( $sort ) ) === false ) {
153
-			throw new \RuntimeException( 'Sortation not set' );
152
+		if (($item = reset($sort)) === false) {
153
+			throw new \RuntimeException('Sortation not set');
154 154
 		}
155 155
 
156
-		$this->assertEquals( 'sort:index.catalog.position("test",["0"])', $item->getName() );
157
-		$this->assertEquals( '-', $item->getOperator() );
156
+		$this->assertEquals('sort:index.catalog.position("test",["0"])', $item->getName());
157
+		$this->assertEquals('-', $item->getOperator());
158 158
 	}
159 159
 
160 160
 
161 161
 	public function testCreateFilterSortRelevanceText()
162 162
 	{
163
-		$filter = $this->object->createFilter( 'relevance', '-', 1, 2, 'test' );
164
-		$filter = $this->object->addFilterText( $filter, 'Espresso' );
163
+		$filter = $this->object->createFilter('relevance', '-', 1, 2, 'test');
164
+		$filter = $this->object->addFilterText($filter, 'Espresso');
165 165
 
166
-		$this->assertInstanceOf( '\\Aimeos\\MW\\Criteria\\Iface', $filter );
167
-		$this->assertEquals( array(), $filter->getSortations() );
168
-		$this->assertEquals( 1, $filter->getSliceStart() );
169
-		$this->assertEquals( 2, $filter->getSliceSize() );
166
+		$this->assertInstanceOf('\\Aimeos\\MW\\Criteria\\Iface', $filter);
167
+		$this->assertEquals(array(), $filter->getSortations());
168
+		$this->assertEquals(1, $filter->getSliceStart());
169
+		$this->assertEquals(2, $filter->getSliceSize());
170 170
 	}
171 171
 
172 172
 
173 173
 	public function testCreateFilterSortCode()
174 174
 	{
175
-		$filter = $this->object->createFilter( 'code' );
175
+		$filter = $this->object->createFilter('code');
176 176
 
177
-		$this->assertInstanceOf( '\\Aimeos\\MW\\Criteria\\Iface', $filter );
177
+		$this->assertInstanceOf('\\Aimeos\\MW\\Criteria\\Iface', $filter);
178 178
 
179 179
 		$sort = $filter->getSortations();
180
-		if( ( $item = reset( $sort ) ) === false ) {
181
-			throw new \RuntimeException( 'Sortation not set' );
180
+		if (($item = reset($sort)) === false) {
181
+			throw new \RuntimeException('Sortation not set');
182 182
 		}
183 183
 
184
-		$this->assertEquals( 'product.code', $item->getName() );
184
+		$this->assertEquals('product.code', $item->getName());
185 185
 	}
186 186
 
187 187
 
188 188
 	public function testCreateFilterSortName()
189 189
 	{
190
-		$filter = $this->object->createFilter( 'name' );
190
+		$filter = $this->object->createFilter('name');
191 191
 
192
-		$this->assertInstanceOf( '\\Aimeos\\MW\\Criteria\\Iface', $filter );
192
+		$this->assertInstanceOf('\\Aimeos\\MW\\Criteria\\Iface', $filter);
193 193
 
194 194
 		$sort = $filter->getSortations();
195
-		if( ( $item = reset( $sort ) ) === false ) {
196
-			throw new \RuntimeException( 'Sortation not set' );
195
+		if (($item = reset($sort)) === false) {
196
+			throw new \RuntimeException('Sortation not set');
197 197
 		}
198 198
 
199
-		$this->assertEquals( 'sort:index.text.value("default","de","name")', $item->getName() );
199
+		$this->assertEquals('sort:index.text.value("default","de","name")', $item->getName());
200 200
 	}
201 201
 
202 202
 
203 203
 	public function testCreateFilterSortPrice()
204 204
 	{
205
-		$filter = $this->object->createFilter( 'price' );
205
+		$filter = $this->object->createFilter('price');
206 206
 
207
-		$this->assertInstanceOf( '\\Aimeos\\MW\\Criteria\\Iface', $filter );
207
+		$this->assertInstanceOf('\\Aimeos\\MW\\Criteria\\Iface', $filter);
208 208
 
209 209
 		$sort = $filter->getSortations();
210
-		if( ( $item = reset( $sort ) ) === false ) {
211
-			throw new \RuntimeException( 'Sortation not set' );
210
+		if (($item = reset($sort)) === false) {
211
+			throw new \RuntimeException('Sortation not set');
212 212
 		}
213 213
 
214
-		$this->assertStringStartsWith( 'sort:index.price.value("default","EUR","default")', $item->getName() );
214
+		$this->assertStringStartsWith('sort:index.price.value("default","EUR","default")', $item->getName());
215 215
 	}
216 216
 
217 217
 
218 218
 	public function testCreateFilterSortInvalid()
219 219
 	{
220
-		$filter = $this->object->createFilter( '', 'failure' );
220
+		$filter = $this->object->createFilter('', 'failure');
221 221
 
222
-		$this->assertInstanceOf( '\\Aimeos\\MW\\Criteria\\Iface', $filter );
223
-		$this->assertEquals( array(), $filter->getSortations() );
222
+		$this->assertInstanceOf('\\Aimeos\\MW\\Criteria\\Iface', $filter);
223
+		$this->assertEquals(array(), $filter->getSortations());
224 224
 	}
225 225
 
226 226
 
227 227
 	public function testGetItem()
228 228
 	{
229 229
 		$context = \TestHelperFrontend::getContext();
230
-		$id = \Aimeos\MShop\Factory::createManager( $context, 'product' )->findItem( 'CNC' )->getId();
230
+		$id = \Aimeos\MShop\Factory::createManager($context, 'product')->findItem('CNC')->getId();
231 231
 
232
-		$result = $this->object->getItem( $id );
232
+		$result = $this->object->getItem($id);
233 233
 
234
-		$this->assertInstanceOf( '\Aimeos\MShop\Product\Item\Iface', $result );
235
-		$this->assertGreaterThan( 0, $result->getPropertyItems() );
236
-		$this->assertGreaterThan( 0, $result->getRefItems( 'attribute' ) );
237
-		$this->assertGreaterThan( 0, $result->getRefItems( 'media' ) );
238
-		$this->assertGreaterThan( 0, $result->getRefItems( 'price' ) );
239
-		$this->assertGreaterThan( 0, $result->getRefItems( 'product' ) );
240
-		$this->assertGreaterThan( 0, $result->getRefItems( 'text' ) );
234
+		$this->assertInstanceOf('\Aimeos\MShop\Product\Item\Iface', $result);
235
+		$this->assertGreaterThan(0, $result->getPropertyItems());
236
+		$this->assertGreaterThan(0, $result->getRefItems('attribute'));
237
+		$this->assertGreaterThan(0, $result->getRefItems('media'));
238
+		$this->assertGreaterThan(0, $result->getRefItems('price'));
239
+		$this->assertGreaterThan(0, $result->getRefItems('product'));
240
+		$this->assertGreaterThan(0, $result->getRefItems('text'));
241 241
 	}
242 242
 
243 243
 
244 244
 	public function testGetItems()
245 245
 	{
246 246
 		$context = \TestHelperFrontend::getContext();
247
-		$manager = \Aimeos\MShop\Factory::createManager( $context, 'product' );
247
+		$manager = \Aimeos\MShop\Factory::createManager($context, 'product');
248 248
 
249 249
 		$search = $manager->createSearch();
250
-		$search->setConditions( $search->compare( '==', 'product.code', array( 'CNC', 'CNE' ) ) );
250
+		$search->setConditions($search->compare('==', 'product.code', array('CNC', 'CNE')));
251 251
 
252 252
 		$ids = array();
253
-		foreach( $manager->searchItems( $search ) as $productItem ) {
253
+		foreach ($manager->searchItems($search) as $productItem) {
254 254
 			$ids[] = $productItem->getId();
255 255
 		}
256 256
 
257 257
 
258
-		$result = $this->object->getItems( $ids );
258
+		$result = $this->object->getItems($ids);
259 259
 
260
-		$this->assertEquals( 2, count( $result ) );
260
+		$this->assertEquals(2, count($result));
261 261
 
262
-		foreach( $result as $productItem ) {
263
-			$this->assertInstanceOf( '\Aimeos\MShop\Product\Item\Iface', $productItem );
262
+		foreach ($result as $productItem) {
263
+			$this->assertInstanceOf('\Aimeos\MShop\Product\Item\Iface', $productItem);
264 264
 		}
265 265
 	}
266 266
 
267 267
 
268 268
 	public function testSearchItemsCategory()
269 269
 	{
270
-		$catalogManager = \Aimeos\MShop\Catalog\Manager\Factory::createManager( \TestHelperFrontend::getContext() );
270
+		$catalogManager = \Aimeos\MShop\Catalog\Manager\Factory::createManager(\TestHelperFrontend::getContext());
271 271
 		$search = $catalogManager->createSearch();
272 272
 
273
-		$search->setConditions( $search->compare( '==', 'catalog.code', 'new' ) );
274
-		$search->setSlice( 0, 1 );
275
-		$items = $catalogManager->searchItems( $search );
273
+		$search->setConditions($search->compare('==', 'catalog.code', 'new'));
274
+		$search->setSlice(0, 1);
275
+		$items = $catalogManager->searchItems($search);
276 276
 
277
-		if( ( $item = reset( $items ) ) === false ) {
278
-			throw new \RuntimeException( 'Product item not found' );
277
+		if (($item = reset($items)) === false) {
278
+			throw new \RuntimeException('Product item not found');
279 279
 		}
280 280
 
281
-		$filter = $this->object->createFilter( 'position', '+', 1, 1 );
282
-		$filter = $this->object->addFilterCategory( $filter, $item->getId() );
281
+		$filter = $this->object->createFilter('position', '+', 1, 1);
282
+		$filter = $this->object->addFilterCategory($filter, $item->getId());
283 283
 
284 284
 		$total = 0;
285
-		$results = $this->object->searchItems( $filter, array(), $total );
285
+		$results = $this->object->searchItems($filter, array(), $total);
286 286
 
287
-		$this->assertEquals( 3, $total );
288
-		$this->assertEquals( 1, count( $results ) );
287
+		$this->assertEquals(3, $total);
288
+		$this->assertEquals(1, count($results));
289 289
 	}
290 290
 
291 291
 
292 292
 	public function testSearchItemsText()
293 293
 	{
294
-		$filter = $this->object->createFilter( 'relevance', '+', 0, 1, 'unittype13' );
295
-		$filter = $this->object->addFilterText( $filter, 'Expresso', 'relevance', '+', 'unittype13' );
294
+		$filter = $this->object->createFilter('relevance', '+', 0, 1, 'unittype13');
295
+		$filter = $this->object->addFilterText($filter, 'Expresso', 'relevance', '+', 'unittype13');
296 296
 
297 297
 		$total = 0;
298
-		$results = $this->object->searchItems( $filter, array(), $total );
298
+		$results = $this->object->searchItems($filter, array(), $total);
299 299
 
300
-		$this->assertEquals( 2, $total );
301
-		$this->assertEquals( 1, count( $results ) );
300
+		$this->assertEquals(2, $total);
301
+		$this->assertEquals(1, count($results));
302 302
 	}
303 303
 }
Please login to merge, or discard this patch.
controller/frontend/src/Controller/Frontend/Attribute/Iface.php 1 patch
Spacing   +4 added lines, -4 removed lines patch added patch discarded remove patch
@@ -27,7 +27,7 @@  discard block
 block discarded – undo
27 27
 	 * @return \Aimeos\MW\Criteria\Iface Criteria object containing the conditions for searching
28 28
 	 * @since 2017.03
29 29
 	 */
30
-	public function addFilterTypes( \Aimeos\MW\Criteria\Iface $filter, array $codes );
30
+	public function addFilterTypes(\Aimeos\MW\Criteria\Iface $filter, array $codes);
31 31
 
32 32
 
33 33
 	/**
@@ -47,7 +47,7 @@  discard block
 block discarded – undo
47 47
 	 * @return \Aimeos\MShop\Attribute\Item\Iface Attribute item including the referenced domains items
48 48
 	 * @since 2017.03
49 49
 	 */
50
-	public function getItem( $id, array $domains = array( 'media', 'price', 'text' ) );
50
+	public function getItem($id, array $domains = array('media', 'price', 'text'));
51 51
 
52 52
 
53 53
 	/**
@@ -58,7 +58,7 @@  discard block
 block discarded – undo
58 58
 	 * @return \Aimeos\MShop\Attribute\Item\Iface[] Associative list of attribute item including the referenced domains items
59 59
 	 * @since 2017.03
60 60
 	 */
61
-	public function getItems( array $ids, array $domains = array( 'media', 'price', 'text' ) );
61
+	public function getItems(array $ids, array $domains = array('media', 'price', 'text'));
62 62
 
63 63
 
64 64
 	/**
@@ -70,5 +70,5 @@  discard block
 block discarded – undo
70 70
 	 * @return array Ordered list of attribute items implementing \Aimeos\MShop\Attribute\Item\Iface
71 71
 	 * @since 2017.03
72 72
 	 */
73
-	public function searchItems( \Aimeos\MW\Criteria\Iface $filter, array $domains = array( 'media', 'price', 'text' ), &$total = null );
73
+	public function searchItems(\Aimeos\MW\Criteria\Iface $filter, array $domains = array('media', 'price', 'text'), &$total = null);
74 74
 }
Please login to merge, or discard this patch.
controller/frontend/src/Controller/Frontend/Attribute/Standard.php 1 patch
Spacing   +16 added lines, -16 removed lines patch added patch discarded remove patch
@@ -29,15 +29,15 @@  discard block
 block discarded – undo
29 29
 	 * @return \Aimeos\MW\Criteria\Iface Criteria object containing the conditions for searching
30 30
 	 * @since 2017.03
31 31
 	 */
32
-	public function addFilterTypes( \Aimeos\MW\Criteria\Iface $filter, array $codes )
32
+	public function addFilterTypes(\Aimeos\MW\Criteria\Iface $filter, array $codes)
33 33
 	{
34
-		if( !empty( $codes ) )
34
+		if (!empty($codes))
35 35
 		{
36 36
 			$expr = [
37
-				$filter->compare( '==', 'attribute.type.code', $codes ),
37
+				$filter->compare('==', 'attribute.type.code', $codes),
38 38
 				$filter->getConditions(),
39 39
 			];
40
-			$filter->setConditions( $filter->combine( '&&', $expr ) );
40
+			$filter->setConditions($filter->combine('&&', $expr));
41 41
 		}
42 42
 
43 43
 		return $filter;
@@ -53,15 +53,15 @@  discard block
 block discarded – undo
53 53
 	 */
54 54
 	public function createFilter()
55 55
 	{
56
-		$filter = \Aimeos\MShop\Factory::createManager( $this->getContext(), 'attribute' )->createSearch( true );
56
+		$filter = \Aimeos\MShop\Factory::createManager($this->getContext(), 'attribute')->createSearch(true);
57 57
 
58 58
 		$expr = array(
59
-			$filter->compare( '==', 'attribute.domain', 'product' ),
59
+			$filter->compare('==', 'attribute.domain', 'product'),
60 60
 			$filter->getConditions(),
61 61
 		);
62 62
 
63
-		$filter->setConditions( $filter->combine( '&&', $expr ) );
64
-		$filter->setSortations( array( $filter->sort( '+', 'attribute.position' ) ) );
63
+		$filter->setConditions($filter->combine('&&', $expr));
64
+		$filter->setSortations(array($filter->sort('+', 'attribute.position')));
65 65
 
66 66
 		return $filter;
67 67
 	}
@@ -75,9 +75,9 @@  discard block
 block discarded – undo
75 75
 	 * @return \Aimeos\MShop\Attribute\Item\Iface Attribute item including the referenced domains items
76 76
 	 * @since 2017.03
77 77
 	 */
78
-	public function getItem( $id, array $domains = array( 'media', 'price', 'text' ) )
78
+	public function getItem($id, array $domains = array('media', 'price', 'text'))
79 79
 	{
80
-		return \Aimeos\MShop\Factory::createManager( $this->getContext(), 'attribute' )->getItem( $id, $domains );
80
+		return \Aimeos\MShop\Factory::createManager($this->getContext(), 'attribute')->getItem($id, $domains);
81 81
 	}
82 82
 
83 83
 
@@ -89,16 +89,16 @@  discard block
 block discarded – undo
89 89
 	 * @return \Aimeos\MShop\Attribute\Item\Iface[] Associative list of attribute item including the referenced domains items
90 90
 	 * @since 2017.03
91 91
 	 */
92
-	public function getItems( array $ids, array $domains = array( 'media', 'price', 'text' ) )
92
+	public function getItems(array $ids, array $domains = array('media', 'price', 'text'))
93 93
 	{
94 94
 		$filter = $this->createFilter();
95 95
 		$expr = [
96
-			$filter->compare( '==', 'attribute.id', $ids ),
96
+			$filter->compare('==', 'attribute.id', $ids),
97 97
 			$filter->getConditions(),
98 98
 		];
99
-		$filter->setConditions( $filter->combine( '&&', $expr ) );
99
+		$filter->setConditions($filter->combine('&&', $expr));
100 100
 
101
-		return $this->searchItems( $filter, $domains );
101
+		return $this->searchItems($filter, $domains);
102 102
 	}
103 103
 
104 104
 
@@ -111,8 +111,8 @@  discard block
 block discarded – undo
111 111
 	 * @return array Ordered list of attribute items implementing \Aimeos\MShop\Attribute\Item\Iface
112 112
 	 * @since 2017.03
113 113
 	 */
114
-	public function searchItems( \Aimeos\MW\Criteria\Iface $filter, array $domains = array( 'media', 'price', 'text' ), &$total = null )
114
+	public function searchItems(\Aimeos\MW\Criteria\Iface $filter, array $domains = array('media', 'price', 'text'), &$total = null)
115 115
 	{
116
-		return \Aimeos\MShop\Factory::createManager( $this->getContext(), 'attribute' )->searchItems( $filter, $domains, $total );
116
+		return \Aimeos\MShop\Factory::createManager($this->getContext(), 'attribute')->searchItems($filter, $domains, $total);
117 117
 	}
118 118
 }
Please login to merge, or discard this patch.
controller/frontend/tests/Controller/Frontend/Attribute/StandardTest.php 1 patch
Spacing   +28 added lines, -28 removed lines patch added patch discarded remove patch
@@ -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\Attribute\Standard( \TestHelperFrontend::getContext() );
19
+		$this->object = new \Aimeos\Controller\Frontend\Attribute\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,57 +30,57 @@  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( 1, count( $filter->getSortations() ) );
35
-		$this->assertEquals( 0, $filter->getSliceStart() );
36
-		$this->assertEquals( 100, $filter->getSliceSize() );
33
+		$this->assertInstanceOf('\\Aimeos\\MW\\Criteria\\Iface', $filter);
34
+		$this->assertEquals(1, count($filter->getSortations()));
35
+		$this->assertEquals(0, $filter->getSliceStart());
36
+		$this->assertEquals(100, $filter->getSliceSize());
37 37
 	}
38 38
 
39 39
 
40 40
 	public function testAddFilterTypes()
41 41
 	{
42 42
 		$filter = $this->object->createFilter();
43
-		$filter = $this->object->addFilterTypes( $filter, ['size'] );
43
+		$filter = $this->object->addFilterTypes($filter, ['size']);
44 44
 
45 45
 		$list = $filter->getConditions()->getExpressions();
46 46
 
47
-		if( !isset( $list[0] ) || !( $list[0] instanceof \Aimeos\MW\Criteria\Expression\Compare\Iface ) ) {
48
-			throw new \RuntimeException( 'Wrong expression' );
47
+		if (!isset($list[0]) || !($list[0] instanceof \Aimeos\MW\Criteria\Expression\Compare\Iface)) {
48
+			throw new \RuntimeException('Wrong expression');
49 49
 		}
50 50
 
51
-		$this->assertEquals( 'attribute.type.code', $list[0]->getName() );
52
-		$this->assertEquals( 1, count( $list[0]->getValue() ) );
51
+		$this->assertEquals('attribute.type.code', $list[0]->getName());
52
+		$this->assertEquals(1, count($list[0]->getValue()));
53 53
 	}
54 54
 
55 55
 
56 56
 	public function testGetItem()
57 57
 	{
58 58
 		$context = \TestHelperFrontend::getContext();
59
-		$manager = \Aimeos\MShop\Factory::createManager( $context, 'attribute' );
60
-		$id = $manager->findItem( 'xs', [], 'product', 'size' )->getId();
59
+		$manager = \Aimeos\MShop\Factory::createManager($context, 'attribute');
60
+		$id = $manager->findItem('xs', [], 'product', 'size')->getId();
61 61
 
62
-		$result = $this->object->getItem( $id, ['text'] );
62
+		$result = $this->object->getItem($id, ['text']);
63 63
 
64
-		$this->assertInstanceOf( '\Aimeos\MShop\Attribute\Item\Iface', $result );
65
-		$this->assertEquals( 3, count( $result->getRefItems( 'text' ) ) );
64
+		$this->assertInstanceOf('\Aimeos\MShop\Attribute\Item\Iface', $result);
65
+		$this->assertEquals(3, count($result->getRefItems('text')));
66 66
 	}
67 67
 
68 68
 
69 69
 	public function testGetItems()
70 70
 	{
71 71
 		$context = \TestHelperFrontend::getContext();
72
-		$manager = \Aimeos\MShop\Factory::createManager( $context, 'attribute' );
73
-		$id = $manager->findItem( 'xs', [], 'product', 'size' )->getId();
72
+		$manager = \Aimeos\MShop\Factory::createManager($context, 'attribute');
73
+		$id = $manager->findItem('xs', [], 'product', 'size')->getId();
74 74
 
75
-		$result = $this->object->getItems( [$id], ['text'] );
75
+		$result = $this->object->getItems([$id], ['text']);
76 76
 
77
-		$this->assertInternalType( 'array', $result );
78
-		$this->assertEquals( 1, count( $result ) );
77
+		$this->assertInternalType('array', $result);
78
+		$this->assertEquals(1, count($result));
79 79
 
80
-		foreach( $result as $attrItem )
80
+		foreach ($result as $attrItem)
81 81
 		{
82
-			$this->assertInstanceOf( '\Aimeos\MShop\Attribute\Item\Iface', $attrItem );
83
-			$this->assertEquals( 3, count( $attrItem->getRefItems( 'text' ) ) );
82
+			$this->assertInstanceOf('\Aimeos\MShop\Attribute\Item\Iface', $attrItem);
83
+			$this->assertEquals(3, count($attrItem->getRefItems('text')));
84 84
 		}
85 85
 	}
86 86
 
@@ -88,12 +88,12 @@  discard block
 block discarded – undo
88 88
 	public function testSearchItems()
89 89
 	{
90 90
 		$filter = $this->object->createFilter();
91
-		$filter = $this->object->addFilterTypes( $filter, ['size'] );
91
+		$filter = $this->object->addFilterTypes($filter, ['size']);
92 92
 
93 93
 		$total = 0;
94
-		$results = $this->object->searchItems( $filter, ['text'], $total );
94
+		$results = $this->object->searchItems($filter, ['text'], $total);
95 95
 
96
-		$this->assertEquals( 6, $total );
97
-		$this->assertEquals( 6, count( $results ) );
96
+		$this->assertEquals(6, $total);
97
+		$this->assertEquals(6, count($results));
98 98
 	}
99 99
 }
Please login to merge, or discard this patch.
frontend/tests/Controller/Frontend/Attribute/Decorator/BaseTest.php 1 patch
Spacing   +39 added lines, -39 removed lines patch added patch discarded remove patch
@@ -20,117 +20,117 @@
 block discarded – undo
20 20
 	{
21 21
 		$this->context = \TestHelperFrontend::getContext();
22 22
 
23
-		$this->stub = $this->getMockBuilder( '\Aimeos\Controller\Frontend\Attribute\Standard' )
23
+		$this->stub = $this->getMockBuilder('\Aimeos\Controller\Frontend\Attribute\Standard')
24 24
 			->disableOriginalConstructor()
25 25
 			->getMock();
26 26
 
27
-		$this->object = $this->getMockBuilder( '\Aimeos\Controller\Frontend\Attribute\Decorator\Base' )
28
-			->setConstructorArgs( [$this->stub, $this->context] )
27
+		$this->object = $this->getMockBuilder('\Aimeos\Controller\Frontend\Attribute\Decorator\Base')
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' )->getMock();
41
+		$stub = $this->getMockBuilder('\Aimeos\Controller\Frontend\Iface')->getMock();
42 42
 
43
-		$this->setExpectedException( '\Aimeos\Controller\Frontend\Exception' );
43
+		$this->setExpectedException('\Aimeos\Controller\Frontend\Exception');
44 44
 
45
-		$this->getMockBuilder( '\Aimeos\Controller\Frontend\Attribute\Decorator\Base' )
46
-			->setConstructorArgs( [$stub, $this->context] )
45
+		$this->getMockBuilder('\Aimeos\Controller\Frontend\Attribute\Decorator\Base')
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\Attribute\Standard' )
53
+		$stub = $this->getMockBuilder('\Aimeos\Controller\Frontend\Attribute\Standard')
54 54
 			->disableOriginalConstructor()
55
-			->setMethods( ['invalid'] )
55
+			->setMethods(['invalid'])
56 56
 			->getMock();
57 57
 
58
-		$object = $this->getMockBuilder( '\Aimeos\Controller\Frontend\Attribute\Decorator\Base' )
59
-			->setConstructorArgs( [$stub, $this->context] )
58
+		$object = $this->getMockBuilder('\Aimeos\Controller\Frontend\Attribute\Decorator\Base')
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 testAddFilterTypes()
69 69
 	{
70
-		$search = \Aimeos\MShop\Factory::createManager( $this->context, 'attribute' )->createSearch();
70
+		$search = \Aimeos\MShop\Factory::createManager($this->context, 'attribute')->createSearch();
71 71
 
72
-		$this->stub->expects( $this->once() )->method( 'addFilterTypes' )
73
-			->will( $this->returnArgument( 0 ) );
72
+		$this->stub->expects($this->once())->method('addFilterTypes')
73
+			->will($this->returnArgument(0));
74 74
 
75
-		$this->assertInstanceOf( '\Aimeos\MW\Criteria\Iface', $this->object->addFilterTypes( $search, [] ) );
75
+		$this->assertInstanceOf('\Aimeos\MW\Criteria\Iface', $this->object->addFilterTypes($search, []));
76 76
 	}
77 77
 
78 78
 
79 79
 	public function testCreateFilter()
80 80
 	{
81
-		$search = \Aimeos\MShop\Factory::createManager( $this->context, 'attribute' )->createSearch();
81
+		$search = \Aimeos\MShop\Factory::createManager($this->context, 'attribute')->createSearch();
82 82
 
83
-		$this->stub->expects( $this->once() )->method( 'createFilter' )
84
-			->will( $this->returnValue( $search ) );
83
+		$this->stub->expects($this->once())->method('createFilter')
84
+			->will($this->returnValue($search));
85 85
 
86
-		$this->assertInstanceOf( '\Aimeos\MW\Criteria\Iface', $this->object->createFilter() );
86
+		$this->assertInstanceOf('\Aimeos\MW\Criteria\Iface', $this->object->createFilter());
87 87
 	}
88 88
 
89 89
 
90 90
 	public function testGetItem()
91 91
 	{
92
-		$item = \Aimeos\MShop\Factory::createManager( $this->context, 'attribute' )->createItem();
92
+		$item = \Aimeos\MShop\Factory::createManager($this->context, 'attribute')->createItem();
93 93
 
94
-		$this->stub->expects( $this->once() )->method( 'getItem' )
95
-			->will( $this->returnValue( $item ) );
94
+		$this->stub->expects($this->once())->method('getItem')
95
+			->will($this->returnValue($item));
96 96
 
97
-		$this->assertInstanceOf( '\Aimeos\MShop\Attribute\Item\Iface', $this->object->getItem( -1 ) );
97
+		$this->assertInstanceOf('\Aimeos\MShop\Attribute\Item\Iface', $this->object->getItem( -1 ));
98 98
 	}
99 99
 
100 100
 
101 101
 	public function testGetItems()
102 102
 	{
103
-		$this->stub->expects( $this->once() )->method( 'getItems' )
104
-			->will( $this->returnValue( [] ) );
103
+		$this->stub->expects($this->once())->method('getItems')
104
+			->will($this->returnValue([]));
105 105
 
106
-		$this->assertEquals( [], $this->object->getItems( [-1], ['media'] ) );
106
+		$this->assertEquals([], $this->object->getItems([-1], ['media']));
107 107
 	}
108 108
 
109 109
 
110 110
 	public function testSearchItems()
111 111
 	{
112
-		$filter = \Aimeos\MShop\Factory::createManager( $this->context, 'attribute' )->createSearch();
112
+		$filter = \Aimeos\MShop\Factory::createManager($this->context, 'attribute')->createSearch();
113 113
 
114
-		$this->stub->expects( $this->once() )->method( 'searchItems' )
115
-			->will( $this->returnValue( [] ) );
114
+		$this->stub->expects($this->once())->method('searchItems')
115
+			->will($this->returnValue([]));
116 116
 
117
-		$this->assertEquals( [], $this->object->searchItems( $filter, ['media'] ) );
117
+		$this->assertEquals([], $this->object->searchItems($filter, ['media']));
118 118
 	}
119 119
 
120 120
 
121 121
 	public function testGetController()
122 122
 	{
123
-		$result = $this->access( 'getController' )->invokeArgs( $this->object, [] );
123
+		$result = $this->access('getController')->invokeArgs($this->object, []);
124 124
 
125
-		$this->assertSame( $this->stub, $result );
125
+		$this->assertSame($this->stub, $result);
126 126
 	}
127 127
 
128 128
 
129
-	protected function access( $name )
129
+	protected function access($name)
130 130
 	{
131
-		$class = new \ReflectionClass( '\Aimeos\Controller\Frontend\Attribute\Decorator\Base' );
132
-		$method = $class->getMethod( $name );
133
-		$method->setAccessible( true );
131
+		$class = new \ReflectionClass('\Aimeos\Controller\Frontend\Attribute\Decorator\Base');
132
+		$method = $class->getMethod($name);
133
+		$method->setAccessible(true);
134 134
 
135 135
 		return $method;
136 136
 	}
Please login to merge, or discard this patch.
frontend/tests/Controller/Frontend/Basket/Decorator/SelectTest.php 1 patch
Spacing   +86 added lines, -86 removed lines patch added patch discarded remove patch
@@ -19,217 +19,217 @@
 block discarded – undo
19 19
 	protected function setUp()
20 20
 	{
21 21
 		$this->context = \TestHelperFrontend::getContext();
22
-		$this->testItem = \Aimeos\MShop\Factory::createManager( $this->context, 'product' )->findItem( 'U:TESTP' );
22
+		$this->testItem = \Aimeos\MShop\Factory::createManager($this->context, 'product')->findItem('U:TESTP');
23 23
 
24
-		$object = new \Aimeos\Controller\Frontend\Basket\Standard( $this->context );
25
-		$this->object = new \Aimeos\Controller\Frontend\Basket\Decorator\Select( $object, $this->context );
24
+		$object = new \Aimeos\Controller\Frontend\Basket\Standard($this->context);
25
+		$this->object = new \Aimeos\Controller\Frontend\Basket\Decorator\Select($object, $this->context);
26 26
 	}
27 27
 
28 28
 
29 29
 	protected function tearDown()
30 30
 	{
31 31
 		$this->object->clear();
32
-		$this->context->getSession()->set( 'aimeos', array() );
32
+		$this->context->getSession()->set('aimeos', array());
33 33
 
34
-		unset( $this->object );
34
+		unset($this->object);
35 35
 	}
36 36
 
37 37
 
38 38
 	public function testAddDeleteProduct()
39 39
 	{
40 40
 		$basket = $this->object->get();
41
-		$this->object->addProduct( $this->testItem->getId(), 2 );
41
+		$this->object->addProduct($this->testItem->getId(), 2);
42 42
 
43
-		$this->assertEquals( 1, count( $basket->getProducts() ) );
44
-		$this->assertEquals( 2, $basket->getProduct( 0 )->getQuantity() );
45
-		$this->assertEquals( 'U:TESTPSUB01', $basket->getProduct( 0 )->getProductCode() );
43
+		$this->assertEquals(1, count($basket->getProducts()));
44
+		$this->assertEquals(2, $basket->getProduct(0)->getQuantity());
45
+		$this->assertEquals('U:TESTPSUB01', $basket->getProduct(0)->getProductCode());
46 46
 	}
47 47
 
48 48
 
49 49
 	public function testAddProductNoSelection()
50 50
 	{
51
-		$item = \Aimeos\MShop\Factory::createManager( $this->context, 'product' )->findItem( 'CNC' );
51
+		$item = \Aimeos\MShop\Factory::createManager($this->context, 'product')->findItem('CNC');
52 52
 
53
-		$this->object->addProduct( $item->getId(), 1 );
53
+		$this->object->addProduct($item->getId(), 1);
54 54
 
55
-		$this->assertEquals( 1, count( $this->object->get()->getProducts() ) );
56
-		$this->assertEquals( 'CNC', $this->object->get()->getProduct( 0 )->getProductCode() );
57
-		$this->assertEquals( 0, count( $this->object->get()->getProduct( 0 )->getProducts() ) );
55
+		$this->assertEquals(1, count($this->object->get()->getProducts()));
56
+		$this->assertEquals('CNC', $this->object->get()->getProduct(0)->getProductCode());
57
+		$this->assertEquals(0, count($this->object->get()->getProduct(0)->getProducts()));
58 58
 	}
59 59
 
60 60
 
61 61
 	public function testAddProductVariant()
62 62
 	{
63
-		$attributeManager = \Aimeos\MShop\Attribute\Manager\Factory::createManager( \TestHelperFrontend::getContext() );
63
+		$attributeManager = \Aimeos\MShop\Attribute\Manager\Factory::createManager(\TestHelperFrontend::getContext());
64 64
 
65 65
 		$search = $attributeManager->createSearch();
66
-		$search->setConditions( $search->compare( '==', 'attribute.code', array( 'xs', 'white' ) ) );
66
+		$search->setConditions($search->compare('==', 'attribute.code', array('xs', 'white')));
67 67
 
68
-		$attributes = $attributeManager->searchItems( $search );
68
+		$attributes = $attributeManager->searchItems($search);
69 69
 
70
-		if( count( $attributes ) === 0 ) {
71
-			throw new \RuntimeException( 'Attributes not found' );
70
+		if (count($attributes) === 0) {
71
+			throw new \RuntimeException('Attributes not found');
72 72
 		}
73 73
 
74 74
 
75
-		$item = \Aimeos\MShop\Factory::createManager( $this->context, 'product' )->findItem( 'CNC' );
75
+		$item = \Aimeos\MShop\Factory::createManager($this->context, 'product')->findItem('CNC');
76 76
 
77
-		$this->object->addProduct( $item->getId(), 1, array(), array_keys( $attributes ), array(), array(), array(), 'default' );
77
+		$this->object->addProduct($item->getId(), 1, array(), array_keys($attributes), array(), array(), array(), 'default');
78 78
 
79
-		$this->assertEquals( 1, count( $this->object->get()->getProducts() ) );
80
-		$this->assertEquals( 'CNC', $this->object->get()->getProduct( 0 )->getProductCode() );
79
+		$this->assertEquals(1, count($this->object->get()->getProducts()));
80
+		$this->assertEquals('CNC', $this->object->get()->getProduct(0)->getProductCode());
81 81
 	}
82 82
 
83 83
 
84 84
 	public function testAddProductVariantIncomplete()
85 85
 	{
86
-		$attributeManager = \Aimeos\MShop\Factory::createManager( $this->context, 'attribute' );
86
+		$attributeManager = \Aimeos\MShop\Factory::createManager($this->context, 'attribute');
87 87
 
88 88
 		$search = $attributeManager->createSearch();
89 89
 		$expr = array(
90
-			$search->compare( '==', 'attribute.domain', 'product' ),
91
-			$search->compare( '==', 'attribute.code', '30' ),
92
-			$search->compare( '==', 'attribute.type.code', 'length' ),
90
+			$search->compare('==', 'attribute.domain', 'product'),
91
+			$search->compare('==', 'attribute.code', '30'),
92
+			$search->compare('==', 'attribute.type.code', 'length'),
93 93
 		);
94
-		$search->setConditions( $search->combine( '&&', $expr ) );
94
+		$search->setConditions($search->combine('&&', $expr));
95 95
 
96
-		$attributes = $attributeManager->searchItems( $search );
96
+		$attributes = $attributeManager->searchItems($search);
97 97
 
98
-		if( count( $attributes ) === 0 ) {
99
-			throw new \RuntimeException( 'Attributes not found' );
98
+		if (count($attributes) === 0) {
99
+			throw new \RuntimeException('Attributes not found');
100 100
 		}
101 101
 
102 102
 
103
-		$item = \Aimeos\MShop\Factory::createManager( $this->context, 'product' )->findItem( 'U:TEST' );
103
+		$item = \Aimeos\MShop\Factory::createManager($this->context, 'product')->findItem('U:TEST');
104 104
 
105
-		$this->object->addProduct( $item->getId(), 1, array(), array_keys( $attributes ) );
105
+		$this->object->addProduct($item->getId(), 1, array(), array_keys($attributes));
106 106
 
107
-		$this->assertEquals( 1, count( $this->object->get()->getProducts() ) );
108
-		$this->assertEquals( 'U:TESTSUB02', $this->object->get()->getProduct( 0 )->getProductCode() );
109
-		$this->assertEquals( 2, count( $this->object->get()->getProduct( 0 )->getAttributes() ) );
107
+		$this->assertEquals(1, count($this->object->get()->getProducts()));
108
+		$this->assertEquals('U:TESTSUB02', $this->object->get()->getProduct(0)->getProductCode());
109
+		$this->assertEquals(2, count($this->object->get()->getProduct(0)->getAttributes()));
110 110
 	}
111 111
 
112 112
 
113 113
 	public function testAddProductVariantNonUnique()
114 114
 	{
115
-		$attributeManager = \Aimeos\MShop\Factory::createManager( $this->context, 'attribute' );
115
+		$attributeManager = \Aimeos\MShop\Factory::createManager($this->context, 'attribute');
116 116
 
117 117
 		$search = $attributeManager->createSearch();
118 118
 		$expr = array(
119
-			$search->compare( '==', 'attribute.domain', 'product' ),
120
-			$search->compare( '==', 'attribute.code', '30' ),
121
-			$search->compare( '==', 'attribute.type.code', 'width' ),
119
+			$search->compare('==', 'attribute.domain', 'product'),
120
+			$search->compare('==', 'attribute.code', '30'),
121
+			$search->compare('==', 'attribute.type.code', 'width'),
122 122
 		);
123
-		$search->setConditions( $search->combine( '&&', $expr ) );
123
+		$search->setConditions($search->combine('&&', $expr));
124 124
 
125
-		$attributes = $attributeManager->searchItems( $search );
125
+		$attributes = $attributeManager->searchItems($search);
126 126
 
127
-		if( count( $attributes ) === 0 ) {
128
-			throw new \RuntimeException( 'Attributes not found' );
127
+		if (count($attributes) === 0) {
128
+			throw new \RuntimeException('Attributes not found');
129 129
 		}
130 130
 
131 131
 
132
-		$item = \Aimeos\MShop\Factory::createManager( $this->context, 'product' )->findItem( 'U:TEST' );
132
+		$item = \Aimeos\MShop\Factory::createManager($this->context, 'product')->findItem('U:TEST');
133 133
 
134
-		$this->setExpectedException( '\\Aimeos\\Controller\\Frontend\\Basket\\Exception' );
135
-		$this->object->addProduct( $item->getId(), 1, array(), array_keys( $attributes ) );
134
+		$this->setExpectedException('\\Aimeos\\Controller\\Frontend\\Basket\\Exception');
135
+		$this->object->addProduct($item->getId(), 1, array(), array_keys($attributes));
136 136
 	}
137 137
 
138 138
 
139 139
 	public function testAddProductVariantNotRequired()
140 140
 	{
141
-		$attributeManager = \Aimeos\MShop\Factory::createManager( $this->context, 'attribute' );
141
+		$attributeManager = \Aimeos\MShop\Factory::createManager($this->context, 'attribute');
142 142
 
143 143
 		$search = $attributeManager->createSearch();
144
-		$search->setConditions( $search->compare( '==', 'attribute.code', 'xs' ) );
144
+		$search->setConditions($search->compare('==', 'attribute.code', 'xs'));
145 145
 
146
-		$attributes = $attributeManager->searchItems( $search );
146
+		$attributes = $attributeManager->searchItems($search);
147 147
 
148
-		if( count( $attributes ) === 0 ) {
149
-			throw new \RuntimeException( 'Attribute not found' );
148
+		if (count($attributes) === 0) {
149
+			throw new \RuntimeException('Attribute not found');
150 150
 		}
151 151
 
152
-		$options = array( 'variant' => false );
152
+		$options = array('variant' => false);
153 153
 
154
-		$this->object->addProduct( $this->testItem->getId(), 1, $options, array_keys( $attributes ) );
154
+		$this->object->addProduct($this->testItem->getId(), 1, $options, array_keys($attributes));
155 155
 
156
-		$this->assertEquals( 1, count( $this->object->get()->getProducts() ) );
157
-		$this->assertEquals( 'U:TESTP', $this->object->get()->getProduct( 0 )->getProductCode() );
156
+		$this->assertEquals(1, count($this->object->get()->getProducts()));
157
+		$this->assertEquals('U:TESTP', $this->object->get()->getProduct(0)->getProductCode());
158 158
 	}
159 159
 
160 160
 
161 161
 	public function testAddProductEmptySelectionException()
162 162
 	{
163
-		$item = \Aimeos\MShop\Factory::createManager( $this->context, 'product' )->findItem( 'U:noSel' );
163
+		$item = \Aimeos\MShop\Factory::createManager($this->context, 'product')->findItem('U:noSel');
164 164
 
165
-		$this->setExpectedException( '\\Aimeos\\Controller\\Frontend\\Basket\\Exception' );
166
-		$this->object->addProduct( $item->getId(), 1 );
165
+		$this->setExpectedException('\\Aimeos\\Controller\\Frontend\\Basket\\Exception');
166
+		$this->object->addProduct($item->getId(), 1);
167 167
 	}
168 168
 
169 169
 
170 170
 	public function testAddProductSelectionWithPricelessItem()
171 171
 	{
172
-		$this->object->addProduct( $this->testItem->getId(), 1 );
172
+		$this->object->addProduct($this->testItem->getId(), 1);
173 173
 
174
-		$this->assertEquals( 'U:TESTPSUB01', $this->object->get()->getProduct( 0 )->getProductCode() );
174
+		$this->assertEquals('U:TESTPSUB01', $this->object->get()->getProduct(0)->getProductCode());
175 175
 	}
176 176
 
177 177
 
178 178
 	public function testAddProductConfigAttribute()
179 179
 	{
180
-		$attributeManager = \Aimeos\MShop\Factory::createManager( $this->context, 'attribute' );
180
+		$attributeManager = \Aimeos\MShop\Factory::createManager($this->context, 'attribute');
181 181
 
182 182
 		$search = $attributeManager->createSearch();
183
-		$search->setConditions( $search->compare( '==', 'attribute.code', 'xs' ) );
183
+		$search->setConditions($search->compare('==', 'attribute.code', 'xs'));
184 184
 
185
-		$attributes = $attributeManager->searchItems( $search );
185
+		$attributes = $attributeManager->searchItems($search);
186 186
 
187
-		if( empty( $attributes ) ) {
188
-			throw new \RuntimeException( 'Attribute not found' );
187
+		if (empty($attributes)) {
188
+			throw new \RuntimeException('Attribute not found');
189 189
 		}
190 190
 
191
-		$this->object->addProduct( $this->testItem->getId(), 1, array(), array(), array_keys( $attributes ) );
191
+		$this->object->addProduct($this->testItem->getId(), 1, array(), array(), array_keys($attributes));
192 192
 		$basket = $this->object->get();
193 193
 
194
-		$this->assertEquals( 1, count( $basket->getProducts() ) );
195
-		$this->assertEquals( 'U:TESTPSUB01', $basket->getProduct( 0 )->getProductCode() );
196
-		$this->assertEquals( 'xs', $basket->getProduct( 0 )->getAttribute( 'size', 'config' ) );
194
+		$this->assertEquals(1, count($basket->getProducts()));
195
+		$this->assertEquals('U:TESTPSUB01', $basket->getProduct(0)->getProductCode());
196
+		$this->assertEquals('xs', $basket->getProduct(0)->getAttribute('size', 'config'));
197 197
 	}
198 198
 
199 199
 
200 200
 	public function testAddProductHiddenAttribute()
201 201
 	{
202
-		$attributeManager = \Aimeos\MShop\Factory::createManager( $this->context, 'attribute' );
202
+		$attributeManager = \Aimeos\MShop\Factory::createManager($this->context, 'attribute');
203 203
 
204 204
 		$search = $attributeManager->createSearch();
205 205
 		$expr = array(
206
-			$search->compare( '==', 'attribute.code', '29' ),
207
-			$search->compare( '==', 'attribute.type.code', 'width' ),
206
+			$search->compare('==', 'attribute.code', '29'),
207
+			$search->compare('==', 'attribute.type.code', 'width'),
208 208
 		);
209
-		$search->setConditions( $search->combine( '&&', $expr ) );
209
+		$search->setConditions($search->combine('&&', $expr));
210 210
 
211
-		$attributes = $attributeManager->searchItems( $search );
211
+		$attributes = $attributeManager->searchItems($search);
212 212
 
213
-		if( empty( $attributes ) ) {
214
-			throw new \RuntimeException( 'Attribute not found' );
213
+		if (empty($attributes)) {
214
+			throw new \RuntimeException('Attribute not found');
215 215
 		}
216 216
 
217
-		$this->object->addProduct( $this->testItem->getId(), 1, array(), array(), array(), array_keys( $attributes ) );
217
+		$this->object->addProduct($this->testItem->getId(), 1, array(), array(), array(), array_keys($attributes));
218 218
 
219 219
 		$basket = $this->object->get();
220
-		$this->assertEquals( 1, count( $basket->getProducts() ) );
220
+		$this->assertEquals(1, count($basket->getProducts()));
221 221
 
222
-		$product = $basket->getProduct( 0 );
223
-		$this->assertEquals( 'U:TESTPSUB01', $product->getProductCode() );
222
+		$product = $basket->getProduct(0);
223
+		$this->assertEquals('U:TESTPSUB01', $product->getProductCode());
224 224
 
225 225
 		$attributes = $product->getAttributes();
226
-		$this->assertEquals( 1, count( $attributes ) );
226
+		$this->assertEquals(1, count($attributes));
227 227
 
228
-		if( ( $attribute = reset( $attributes ) ) === false ) {
229
-			throw new \RuntimeException( 'No attribute' );
228
+		if (($attribute = reset($attributes)) === false) {
229
+			throw new \RuntimeException('No attribute');
230 230
 		}
231 231
 
232
-		$this->assertEquals( 'hidden', $attribute->getType() );
233
-		$this->assertEquals( '29', $product->getAttribute( 'width', 'hidden' ) );
232
+		$this->assertEquals('hidden', $attribute->getType());
233
+		$this->assertEquals('29', $product->getAttribute('width', 'hidden'));
234 234
 	}
235 235
 }
Please login to merge, or discard this patch.