Completed
Push — master ( 19aca8...39b41c )
by Aimeos
02:32
created
controller/frontend/tests/Controller/Frontend/Service/StandardTest.php 1 patch
Spacing   +81 added lines, -81 removed lines patch added patch discarded remove patch
@@ -17,125 +17,125 @@  discard block
 block discarded – undo
17 17
 
18 18
 	protected function setUp()
19 19
 	{
20
-		\Aimeos\MShop\Factory::setCache( true );
20
+		\Aimeos\MShop\Factory::setCache(true);
21 21
 
22 22
 		$this->context = \TestHelperFrontend::getContext();
23
-		$this->object = new \Aimeos\Controller\Frontend\Service\Standard( $this->context );
23
+		$this->object = new \Aimeos\Controller\Frontend\Service\Standard($this->context);
24 24
 	}
25 25
 
26 26
 
27 27
 	public static function setUpBeforeClass()
28 28
 	{
29
-		$orderManager = \Aimeos\MShop\Order\Manager\Factory::createManager( \TestHelperFrontend::getContext() );
30
-		$orderBaseMgr = $orderManager->getSubManager( 'base' );
29
+		$orderManager = \Aimeos\MShop\Order\Manager\Factory::createManager(\TestHelperFrontend::getContext());
30
+		$orderBaseMgr = $orderManager->getSubManager('base');
31 31
 		self::$basket = $orderBaseMgr->createItem();
32 32
 	}
33 33
 
34 34
 
35 35
 	protected function tearDown()
36 36
 	{
37
-		unset( $this->object, $this->context );
37
+		unset($this->object, $this->context);
38 38
 
39
-		\Aimeos\MShop\Factory::setCache( false );
39
+		\Aimeos\MShop\Factory::setCache(false);
40 40
 		\Aimeos\MShop\Factory::clear();
41 41
 	}
42 42
 
43 43
 
44 44
 	public function testCheckAttributes()
45 45
 	{
46
-		$attributes = $this->object->checkAttributes( $this->getServiceItem()->getId(), [] );
47
-		$this->assertEquals( [], $attributes );
46
+		$attributes = $this->object->checkAttributes($this->getServiceItem()->getId(), []);
47
+		$this->assertEquals([], $attributes);
48 48
 	}
49 49
 
50 50
 
51 51
 	public function testGetProviders()
52 52
 	{
53
-		$providers = $this->object->getProviders( 'delivery' );
54
-		$this->assertGreaterThan( 0, count( $providers ) );
53
+		$providers = $this->object->getProviders('delivery');
54
+		$this->assertGreaterThan(0, count($providers));
55 55
 
56
-		foreach( $providers as $provider ) {
57
-			$this->assertInstanceOf( '\\Aimeos\\MShop\\Service\\Provider\\Iface', $provider );
56
+		foreach ($providers as $provider) {
57
+			$this->assertInstanceOf('\\Aimeos\\MShop\\Service\\Provider\\Iface', $provider);
58 58
 		}
59 59
 	}
60 60
 
61 61
 
62 62
 	public function testGetProvider()
63 63
 	{
64
-		$provider = $this->object->getProvider( $this->getServiceItem()->getId() );
65
-		$this->assertInstanceOf( '\\Aimeos\\MShop\\Service\\Provider\\Iface', $provider );
64
+		$provider = $this->object->getProvider($this->getServiceItem()->getId());
65
+		$this->assertInstanceOf('\\Aimeos\\MShop\\Service\\Provider\\Iface', $provider);
66 66
 	}
67 67
 
68 68
 
69 69
 	public function testProcess()
70 70
 	{
71 71
 		$form = new \Aimeos\MShop\Common\Item\Helper\Form\Standard();
72
-		$item = \Aimeos\MShop\Factory::createManager( $this->context, 'order' )->createItem();
73
-		$serviceId = \Aimeos\MShop\Factory::createManager( $this->context, 'service' )->findItem( 'unitcode' )->getId();
72
+		$item = \Aimeos\MShop\Factory::createManager($this->context, 'order')->createItem();
73
+		$serviceId = \Aimeos\MShop\Factory::createManager($this->context, 'service')->findItem('unitcode')->getId();
74 74
 
75
-		$provider = $this->getMockBuilder( '\\Aimeos\\MShop\\Service\\Provider\\Delivery\\Standard' )
75
+		$provider = $this->getMockBuilder('\\Aimeos\\MShop\\Service\\Provider\\Delivery\\Standard')
76 76
 			->disableOriginalConstructor()
77
-			->setMethods( ['process'] )
77
+			->setMethods(['process'])
78 78
 			->getMock();
79 79
 
80
-		$manager = $this->getMockBuilder( '\\Aimeos\\MShop\\Service\\Manager\\Standard' )
81
-			->setConstructorArgs( [$this->context] )
82
-			->setMethods( ['getProvider'] )
80
+		$manager = $this->getMockBuilder('\\Aimeos\\MShop\\Service\\Manager\\Standard')
81
+			->setConstructorArgs([$this->context])
82
+			->setMethods(['getProvider'])
83 83
 			->getMock();
84 84
 
85
-		\Aimeos\MShop\Factory::injectManager( $this->context, 'service', $manager );
85
+		\Aimeos\MShop\Factory::injectManager($this->context, 'service', $manager);
86 86
 
87
-		$provider->expects( $this->once() )->method( 'process' )->will( $this->returnValue( $form ) );
88
-		$manager->expects( $this->once() )->method( 'getProvider' )->will( $this->returnValue( $provider ) );
87
+		$provider->expects($this->once())->method('process')->will($this->returnValue($form));
88
+		$manager->expects($this->once())->method('getProvider')->will($this->returnValue($provider));
89 89
 
90 90
 
91
-		$result = $this->object->process( $item, $serviceId, [], [] );
92
-		$this->assertInstanceOf( '\Aimeos\MShop\Common\Item\Helper\Form\Iface', $result );
91
+		$result = $this->object->process($item, $serviceId, [], []);
92
+		$this->assertInstanceOf('\Aimeos\MShop\Common\Item\Helper\Form\Iface', $result);
93 93
 	}
94 94
 
95 95
 
96 96
 	public function testUpdateSync()
97 97
 	{
98
-		$item = \Aimeos\MShop\Factory::createManager( $this->context, 'order' )->createItem();
98
+		$item = \Aimeos\MShop\Factory::createManager($this->context, 'order')->createItem();
99 99
 
100
-		$stream = $this->getMockBuilder( 'Psr\Http\Message\StreamInterface' )->getMock();
101
-		$request = $this->getMockBuilder( '\Psr\Http\Message\ServerRequestInterface' )->getMock();
102
-		$response = $this->getMockBuilder( '\Aimeos\MW\View\Helper\Response\Iface' )
100
+		$stream = $this->getMockBuilder('Psr\Http\Message\StreamInterface')->getMock();
101
+		$request = $this->getMockBuilder('\Psr\Http\Message\ServerRequestInterface')->getMock();
102
+		$response = $this->getMockBuilder('\Aimeos\MW\View\Helper\Response\Iface')
103 103
 			->getMock();
104 104
 
105
-		$provider = $this->getMockBuilder( '\\Aimeos\\MShop\\Service\\Provider\\Delivery\\Standard' )
106
-			->setMethods( ['updateSync', 'query', 'isImplemented'] )
105
+		$provider = $this->getMockBuilder('\\Aimeos\\MShop\\Service\\Provider\\Delivery\\Standard')
106
+			->setMethods(['updateSync', 'query', 'isImplemented'])
107 107
 			->disableOriginalConstructor()
108 108
 			->getMock();
109 109
 
110
-		$manager = $this->getMockBuilder( '\\Aimeos\\MShop\\Service\\Manager\\Standard' )
111
-			->setConstructorArgs( array( $this->context ) )
112
-			->setMethods( ['getProvider'] )
110
+		$manager = $this->getMockBuilder('\\Aimeos\\MShop\\Service\\Manager\\Standard')
111
+			->setConstructorArgs(array($this->context))
112
+			->setMethods(['getProvider'])
113 113
 			->getMock();
114 114
 
115
-		\Aimeos\MShop\Factory::injectManager( $this->context, 'service', $manager );
115
+		\Aimeos\MShop\Factory::injectManager($this->context, 'service', $manager);
116 116
 
117 117
 
118
-		$request->expects( $this->once() )->method( 'getQueryParams' )->will( $this->returnValue( ['code' => 'unitcode'] ) );
119
-		$response->expects( $this->once() )->method( 'createStreamFromString' )->will( $this->returnValue( $stream ) );
120
-		$manager->expects( $this->once() )->method( 'getProvider' )->will( $this->returnValue( $provider ) );
121
-		$provider->expects( $this->once() )->method( 'updateSync' )->will( $this->returnValue( $item ) );
122
-		$provider->expects( $this->once() )->method( 'isImplemented' )->will( $this->returnValue( true ) );
123
-		$provider->expects( $this->once() )->method( 'query' );
118
+		$request->expects($this->once())->method('getQueryParams')->will($this->returnValue(['code' => 'unitcode']));
119
+		$response->expects($this->once())->method('createStreamFromString')->will($this->returnValue($stream));
120
+		$manager->expects($this->once())->method('getProvider')->will($this->returnValue($provider));
121
+		$provider->expects($this->once())->method('updateSync')->will($this->returnValue($item));
122
+		$provider->expects($this->once())->method('isImplemented')->will($this->returnValue(true));
123
+		$provider->expects($this->once())->method('query');
124 124
 
125
-		$this->object->updateSync( $request, $response, [], 'paypalexpress', -1 );
125
+		$this->object->updateSync($request, $response, [], 'paypalexpress', -1);
126 126
 	}
127 127
 
128 128
 
129 129
 	public function testGetServices()
130 130
 	{
131
-		$orderManager = \Aimeos\MShop\Order\Manager\Factory::createManager( \TestHelperFrontend::getContext() );
132
-		$basket = $orderManager->getSubManager( 'base' )->createItem();
131
+		$orderManager = \Aimeos\MShop\Order\Manager\Factory::createManager(\TestHelperFrontend::getContext());
132
+		$basket = $orderManager->getSubManager('base')->createItem();
133 133
 
134
-		$services = $this->object->getServices( 'delivery', $basket );
135
-		$this->assertGreaterThan( 0, count( $services ) );
134
+		$services = $this->object->getServices('delivery', $basket);
135
+		$this->assertGreaterThan(0, count($services));
136 136
 
137
-		foreach( $services as $service ) {
138
-			$this->assertInstanceOf( '\\Aimeos\\MShop\\Service\\Item\\Iface', $service );
137
+		foreach ($services as $service) {
138
+			$this->assertInstanceOf('\\Aimeos\\MShop\\Service\\Item\\Iface', $service);
139 139
 		}
140 140
 	}
141 141
 
@@ -143,83 +143,83 @@  discard block
 block discarded – undo
143 143
 	public function testGetServiceAttributes()
144 144
 	{
145 145
 		$service = $this->getServiceItem();
146
-		$attributes = $this->object->getServiceAttributes( 'delivery', $service->getId(), self::$basket );
146
+		$attributes = $this->object->getServiceAttributes('delivery', $service->getId(), self::$basket);
147 147
 
148
-		$this->assertEquals( 0, count( $attributes ) );
148
+		$this->assertEquals(0, count($attributes));
149 149
 	}
150 150
 
151 151
 
152 152
 	public function testGetServiceAttributesCache()
153 153
 	{
154
-		$orderManager = \Aimeos\MShop\Order\Manager\Factory::createManager( \TestHelperFrontend::getContext() );
155
-		$basket = $orderManager->getSubManager( 'base' )->createItem();
154
+		$orderManager = \Aimeos\MShop\Order\Manager\Factory::createManager(\TestHelperFrontend::getContext());
155
+		$basket = $orderManager->getSubManager('base')->createItem();
156 156
 
157
-		$services = $this->object->getServices( 'delivery', $basket );
157
+		$services = $this->object->getServices('delivery', $basket);
158 158
 
159
-		if( ( $service = reset( $services ) ) === false ) {
160
-			throw new \RuntimeException( 'No service item found' );
159
+		if (($service = reset($services)) === false) {
160
+			throw new \RuntimeException('No service item found');
161 161
 		}
162 162
 
163
-		$attributes = $this->object->getServiceAttributes( 'delivery', $service->getId(), self::$basket );
163
+		$attributes = $this->object->getServiceAttributes('delivery', $service->getId(), self::$basket);
164 164
 
165
-		$this->assertEquals( 0, count( $attributes ) );
165
+		$this->assertEquals(0, count($attributes));
166 166
 	}
167 167
 
168 168
 
169 169
 	public function testGetServiceAttributesNoItems()
170 170
 	{
171
-		$this->setExpectedException( '\\Aimeos\\MShop\\Exception' );
172
-		$this->object->getServiceAttributes( 'invalid', -1, self::$basket );
171
+		$this->setExpectedException('\\Aimeos\\MShop\\Exception');
172
+		$this->object->getServiceAttributes('invalid', -1, self::$basket);
173 173
 	}
174 174
 
175 175
 
176 176
 	public function testGetServicePrice()
177 177
 	{
178
-		$orderManager = \Aimeos\MShop\Order\Manager\Factory::createManager( \TestHelperFrontend::getContext() );
179
-		$basket = $orderManager->getSubManager( 'base' )->createItem();
178
+		$orderManager = \Aimeos\MShop\Order\Manager\Factory::createManager(\TestHelperFrontend::getContext());
179
+		$basket = $orderManager->getSubManager('base')->createItem();
180 180
 
181 181
 		$service = $this->getServiceItem();
182
-		$price = $this->object->getServicePrice( 'delivery', $service->getId(), $basket );
182
+		$price = $this->object->getServicePrice('delivery', $service->getId(), $basket);
183 183
 
184
-		$this->assertEquals( '12.95', $price->getValue() );
185
-		$this->assertEquals( '1.99', $price->getCosts() );
184
+		$this->assertEquals('12.95', $price->getValue());
185
+		$this->assertEquals('1.99', $price->getCosts());
186 186
 	}
187 187
 
188 188
 
189 189
 	public function testGetServicePriceCache()
190 190
 	{
191
-		$orderManager = \Aimeos\MShop\Order\Manager\Factory::createManager( \TestHelperFrontend::getContext() );
192
-		$basket = $orderManager->getSubManager( 'base' )->createItem();
191
+		$orderManager = \Aimeos\MShop\Order\Manager\Factory::createManager(\TestHelperFrontend::getContext());
192
+		$basket = $orderManager->getSubManager('base')->createItem();
193 193
 
194
-		$services = $this->object->getServices( 'delivery', $basket );
194
+		$services = $this->object->getServices('delivery', $basket);
195 195
 
196
-		if( ( $service = reset( $services ) ) === false ) {
197
-			throw new \RuntimeException( 'No service item found' );
196
+		if (($service = reset($services)) === false) {
197
+			throw new \RuntimeException('No service item found');
198 198
 		}
199 199
 
200
-		$price = $this->object->getServicePrice( 'delivery', $service->getId(), $basket );
200
+		$price = $this->object->getServicePrice('delivery', $service->getId(), $basket);
201 201
 
202
-		$this->assertEquals( '12.95', $price->getValue() );
203
-		$this->assertEquals( '1.99', $price->getCosts() );
202
+		$this->assertEquals('12.95', $price->getValue());
203
+		$this->assertEquals('1.99', $price->getCosts());
204 204
 	}
205 205
 
206 206
 
207 207
 	public function testGetServicePriceNoItems()
208 208
 	{
209
-		$orderManager = \Aimeos\MShop\Order\Manager\Factory::createManager( \TestHelperFrontend::getContext() );
210
-		$basket = $orderManager->getSubManager( 'base' )->createItem();
209
+		$orderManager = \Aimeos\MShop\Order\Manager\Factory::createManager(\TestHelperFrontend::getContext());
210
+		$basket = $orderManager->getSubManager('base')->createItem();
211 211
 
212
-		$this->setExpectedException( '\\Aimeos\\MShop\\Exception' );
213
-		$this->object->getServicePrice( 'invalid', -1, $basket );
212
+		$this->setExpectedException('\\Aimeos\\MShop\\Exception');
213
+		$this->object->getServicePrice('invalid', -1, $basket);
214 214
 	}
215 215
 
216 216
 
217 217
 	public function testCheckServiceAttributes()
218 218
 	{
219 219
 		$service = $this->getServiceItem();
220
-		$attributes = $this->object->checkServiceAttributes( 'delivery', $service->getId(), [] );
220
+		$attributes = $this->object->checkServiceAttributes('delivery', $service->getId(), []);
221 221
 
222
-		$this->assertEquals( [], $attributes );
222
+		$this->assertEquals([], $attributes);
223 223
 	}
224 224
 
225 225
 
@@ -228,7 +228,7 @@  discard block
 block discarded – undo
228 228
 	 */
229 229
 	protected function getServiceItem()
230 230
 	{
231
-		$manager = \Aimeos\MShop\Service\Manager\Factory::createManager( \TestHelperFrontend::getContext() );
232
-		return $manager->findItem( 'unitcode' );
231
+		$manager = \Aimeos\MShop\Service\Manager\Factory::createManager(\TestHelperFrontend::getContext());
232
+		return $manager->findItem('unitcode');
233 233
 	}
234 234
 }
Please login to merge, or discard this patch.
frontend/tests/Controller/Frontend/Service/Decorator/BaseTest.php 1 patch
Spacing   +57 added lines, -57 removed lines patch added patch discarded remove patch
@@ -20,161 +20,161 @@
 block discarded – undo
20 20
 	{
21 21
 		$this->context = \TestHelperFrontend::getContext();
22 22
 
23
-		$this->stub = $this->getMockBuilder( '\Aimeos\Controller\Frontend\Service\Standard' )
23
+		$this->stub = $this->getMockBuilder('\Aimeos\Controller\Frontend\Service\Standard')
24 24
 			->disableOriginalConstructor()
25 25
 			->getMock();
26 26
 
27
-		$this->object = $this->getMockBuilder( '\Aimeos\Controller\Frontend\Service\Decorator\Base' )
28
-			->setConstructorArgs( [$this->stub, $this->context] )
27
+		$this->object = $this->getMockBuilder('\Aimeos\Controller\Frontend\Service\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\Service\Decorator\Base' )
46
-			->setConstructorArgs( [$stub, $this->context] )
45
+		$this->getMockBuilder('\Aimeos\Controller\Frontend\Service\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\Service\Standard' )
53
+		$stub = $this->getMockBuilder('\Aimeos\Controller\Frontend\Service\Standard')
54 54
 			->disableOriginalConstructor()
55
-			->setMethods( ['invalid'] )
55
+			->setMethods(['invalid'])
56 56
 			->getMock();
57 57
 
58
-		$object = $this->getMockBuilder( '\Aimeos\Controller\Frontend\Service\Decorator\Base' )
59
-			->setConstructorArgs( [$stub, $this->context] )
58
+		$object = $this->getMockBuilder('\Aimeos\Controller\Frontend\Service\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 testCheckAttributes()
69 69
 	{
70
-		$this->stub->expects( $this->once() )->method( 'checkAttributes' )
71
-			->will( $this->returnValue( [] ) );
70
+		$this->stub->expects($this->once())->method('checkAttributes')
71
+			->will($this->returnValue([]));
72 72
 
73
-		$this->assertEquals( [], $this->object->checkAttributes( -1, [] ) );
73
+		$this->assertEquals([], $this->object->checkAttributes( -1, [] ));
74 74
 	}
75 75
 
76 76
 
77 77
 	public function testGetProvider()
78 78
 	{
79
-		$manager = \Aimeos\MShop\Factory::createManager( $this->context, 'service' );
80
-		$provider = $manager->getProvider( $manager->findItem( 'unitcode', [], 'service', 'delivery' ) );
79
+		$manager = \Aimeos\MShop\Factory::createManager($this->context, 'service');
80
+		$provider = $manager->getProvider($manager->findItem('unitcode', [], 'service', 'delivery'));
81 81
 
82
-		$this->stub->expects( $this->once() )->method( 'getProvider' )
83
-			->will( $this->returnValue( $provider ) );
82
+		$this->stub->expects($this->once())->method('getProvider')
83
+			->will($this->returnValue($provider));
84 84
 
85
-		$this->assertSame( $provider, $this->object->getProvider( -1 ) );
85
+		$this->assertSame($provider, $this->object->getProvider( -1 ));
86 86
 	}
87 87
 
88 88
 
89 89
 	public function testGetProviders()
90 90
 	{
91
-		$this->stub->expects( $this->once() )->method( 'getProviders' )
92
-			->will( $this->returnValue( [] ) );
91
+		$this->stub->expects($this->once())->method('getProviders')
92
+			->will($this->returnValue([]));
93 93
 
94
-		$this->assertEquals( [], $this->object->getProviders( 'payment' ) );
94
+		$this->assertEquals([], $this->object->getProviders('payment'));
95 95
 	}
96 96
 
97 97
 
98 98
 	public function testProcess()
99 99
 	{
100
-		$item = \Aimeos\MShop\Factory::createManager( $this->context, 'order' )->createItem();
100
+		$item = \Aimeos\MShop\Factory::createManager($this->context, 'order')->createItem();
101 101
 
102
-		$this->stub->expects( $this->once() )->method( 'process' )
103
-			->will( $this->returnValue( new \Aimeos\MShop\Common\Item\Helper\Form\Standard() ) );
102
+		$this->stub->expects($this->once())->method('process')
103
+			->will($this->returnValue(new \Aimeos\MShop\Common\Item\Helper\Form\Standard()));
104 104
 
105
-		$this->assertInstanceOf( 'Aimeos\MShop\Common\Item\Helper\Form\Iface', $this->object->process( $item, -1, [], [] ) );
105
+		$this->assertInstanceOf('Aimeos\MShop\Common\Item\Helper\Form\Iface', $this->object->process($item, -1, [], []));
106 106
 	}
107 107
 
108 108
 
109 109
 	public function testUpdateSync()
110 110
 	{
111
-		$response = $this->getMockBuilder( '\Psr\Http\Message\ResponseInterface' )->getMock();
112
-		$request = $this->getMockBuilder( '\Psr\Http\Message\ServerRequestInterface' )->getMock();
113
-		$item = \Aimeos\MShop\Factory::createManager( $this->context, 'order' )->createItem();
111
+		$response = $this->getMockBuilder('\Psr\Http\Message\ResponseInterface')->getMock();
112
+		$request = $this->getMockBuilder('\Psr\Http\Message\ServerRequestInterface')->getMock();
113
+		$item = \Aimeos\MShop\Factory::createManager($this->context, 'order')->createItem();
114 114
 
115
-		$this->stub->expects( $this->once() )->method( 'updateSync' )
116
-			->will( $this->returnValue( $item ) );
115
+		$this->stub->expects($this->once())->method('updateSync')
116
+			->will($this->returnValue($item));
117 117
 
118
-		$this->assertInstanceOf( 'Aimeos\MShop\Order\Item\Iface', $this->object->updateSync( $request, $response, [], 'test', -1 ) );
118
+		$this->assertInstanceOf('Aimeos\MShop\Order\Item\Iface', $this->object->updateSync($request, $response, [], 'test', -1));
119 119
 	}
120 120
 
121 121
 
122 122
 	public function testGetServices()
123 123
 	{
124
-		$basket = \Aimeos\MShop\Factory::createManager( $this->context, 'order/base' )->createItem();
124
+		$basket = \Aimeos\MShop\Factory::createManager($this->context, 'order/base')->createItem();
125 125
 
126
-		$this->stub->expects( $this->once() )->method( 'getServices' )
127
-			->will( $this->returnValue( [] ) );
126
+		$this->stub->expects($this->once())->method('getServices')
127
+			->will($this->returnValue([]));
128 128
 
129
-		$this->assertEquals( [], $this->object->getServices( 'payment', $basket ) );
129
+		$this->assertEquals([], $this->object->getServices('payment', $basket));
130 130
 	}
131 131
 
132 132
 
133 133
 	public function testGetServiceAttributes()
134 134
 	{
135
-		$basket = \Aimeos\MShop\Factory::createManager( $this->context, 'order/base' )->createItem();
135
+		$basket = \Aimeos\MShop\Factory::createManager($this->context, 'order/base')->createItem();
136 136
 
137
-		$this->stub->expects( $this->once() )->method( 'getServiceAttributes' )
138
-			->will( $this->returnValue( [] ) );
137
+		$this->stub->expects($this->once())->method('getServiceAttributes')
138
+			->will($this->returnValue([]));
139 139
 
140
-		$this->assertEquals( [], $this->object->getServiceAttributes( 'payment', -1, $basket ) );
140
+		$this->assertEquals([], $this->object->getServiceAttributes('payment', -1, $basket));
141 141
 	}
142 142
 
143 143
 
144 144
 	public function testGetServicePrice()
145 145
 	{
146
-		$priceItem = \Aimeos\MShop\Factory::createManager( $this->context, 'price' )->createItem();
147
-		$basket = \Aimeos\MShop\Factory::createManager( $this->context, 'order/base' )->createItem();
146
+		$priceItem = \Aimeos\MShop\Factory::createManager($this->context, 'price')->createItem();
147
+		$basket = \Aimeos\MShop\Factory::createManager($this->context, 'order/base')->createItem();
148 148
 
149
-		$this->stub->expects( $this->once() )->method( 'getServicePrice' )
150
-			->will( $this->returnValue( $priceItem ) );
149
+		$this->stub->expects($this->once())->method('getServicePrice')
150
+			->will($this->returnValue($priceItem));
151 151
 
152
-		$this->assertInstanceOf( '\Aimeos\MShop\Price\Item\Iface', $this->object->getServicePrice( 'payment', -1, $basket ) );
152
+		$this->assertInstanceOf('\Aimeos\MShop\Price\Item\Iface', $this->object->getServicePrice('payment', -1, $basket));
153 153
 	}
154 154
 
155 155
 
156 156
 	public function testCheckServiceAttributes()
157 157
 	{
158
-		$this->stub->expects( $this->once() )->method( 'checkServiceAttributes' )
159
-			->will( $this->returnValue( [] ) );
158
+		$this->stub->expects($this->once())->method('checkServiceAttributes')
159
+			->will($this->returnValue([]));
160 160
 
161
-		$this->assertEquals( [], $this->object->checkServiceAttributes( 'payment', -1, [] ) );
161
+		$this->assertEquals([], $this->object->checkServiceAttributes('payment', -1, []));
162 162
 	}
163 163
 
164 164
 
165 165
 	public function testGetController()
166 166
 	{
167
-		$result = $this->access( 'getController' )->invokeArgs( $this->object, [] );
167
+		$result = $this->access('getController')->invokeArgs($this->object, []);
168 168
 
169
-		$this->assertSame( $this->stub, $result );
169
+		$this->assertSame($this->stub, $result);
170 170
 	}
171 171
 
172 172
 
173
-	protected function access( $name )
173
+	protected function access($name)
174 174
 	{
175
-		$class = new \ReflectionClass( '\Aimeos\Controller\Frontend\Service\Decorator\Base' );
176
-		$method = $class->getMethod( $name );
177
-		$method->setAccessible( true );
175
+		$class = new \ReflectionClass('\Aimeos\Controller\Frontend\Service\Decorator\Base');
176
+		$method = $class->getMethod($name);
177
+		$method->setAccessible(true);
178 178
 
179 179
 		return $method;
180 180
 	}
Please login to merge, or discard this patch.
controller/frontend/src/Controller/Frontend/Service/Iface.php 1 patch
Spacing   +5 added lines, -5 removed lines patch added patch discarded remove patch
@@ -30,7 +30,7 @@  discard block
 block discarded – undo
30 30
 	 * @param string[] $attributes List of attribute codes as keys and strings entered by the customer as value
31 31
 	 * @return string[] List of attributes codes as keys and error messages as values for invalid or missing values
32 32
 	 */
33
-	public function checkAttributes( $serviceId, array $attributes );
33
+	public function checkAttributes($serviceId, array $attributes);
34 34
 
35 35
 	/**
36 36
 	 * Returns the service item for the given ID
@@ -39,7 +39,7 @@  discard block
 block discarded – undo
39 39
 	 * @param array $ref List of domains for which the items referenced by the services should be fetched too
40 40
 	 * @return \Aimeos\MShop\Service\Provider\Iface Service provider object
41 41
 	 */
42
-	public function getProvider( $serviceId, $ref = ['media', 'price', 'text'] );
42
+	public function getProvider($serviceId, $ref = ['media', 'price', 'text']);
43 43
 
44 44
 	/**
45 45
 	 * Returns the service providers for the given type
@@ -48,7 +48,7 @@  discard block
 block discarded – undo
48 48
 	 * @param array $ref List of domains for which the items referenced by the services should be fetched too
49 49
 	 * @return \Aimeos\MShop\Service\Provider\Iface[] List of service IDs as keys and service provider objects as values
50 50
 	 */
51
-	public function getProviders( $type = null, $ref = ['media', 'price', 'text'] );
51
+	public function getProviders($type = null, $ref = ['media', 'price', 'text']);
52 52
 
53 53
 	/**
54 54
 	 * Processes the service for the given order, e.g. payment and delivery services
@@ -61,7 +61,7 @@  discard block
 block discarded – undo
61 61
 	 * @return \Aimeos\MShop\Common\Item\Helper\Form\Iface|null Form object with URL, parameters, etc.
62 62
 	 * 	or null if no form data is required
63 63
 	 */
64
-	public function process( \Aimeos\MShop\Order\Item\Iface $orderItem, $serviceId, array $urls, array $params );
64
+	public function process(\Aimeos\MShop\Order\Item\Iface $orderItem, $serviceId, array $urls, array $params);
65 65
 
66 66
 	/**
67 67
 	 * Updates the payment or delivery status for the given request
@@ -74,5 +74,5 @@  discard block
 block discarded – undo
74 74
 	 * @param string $orderid Unique ID of the order whose payment status should be updated
75 75
 	 * @return \Aimeos\MShop\Order\Item\Iface $orderItem Order item that has been updated
76 76
 	 */
77
-	public function updateSync( ServerRequestInterface $request, ResponseInterface $response, array $urls, $code, $orderid );
77
+	public function updateSync(ServerRequestInterface $request, ResponseInterface $response, array $urls, $code, $orderid);
78 78
 }
Please login to merge, or discard this patch.
controller/frontend/src/Controller/Frontend/Service/Standard.php 1 patch
Spacing   +70 added lines, -70 removed lines patch added patch discarded remove patch
@@ -35,12 +35,12 @@  discard block
 block discarded – undo
35 35
 	 * @param string[] $attributes List of attribute codes as keys and strings entered by the customer as value
36 36
 	 * @return string[] List of attributes codes as keys and error messages as values for invalid or missing values
37 37
 	 */
38
-	public function checkAttributes( $serviceId, array $attributes )
38
+	public function checkAttributes($serviceId, array $attributes)
39 39
 	{
40
-		$manager = \Aimeos\MShop\Factory::createManager( $this->getContext(), 'service' );
41
-		$provider = $manager->getProvider( $manager->getItem( $serviceId, [], true ) );
40
+		$manager = \Aimeos\MShop\Factory::createManager($this->getContext(), 'service');
41
+		$provider = $manager->getProvider($manager->getItem($serviceId, [], true));
42 42
 
43
-		return array_filter( $provider->checkConfigFE( $attributes ) );
43
+		return array_filter($provider->checkConfigFE($attributes));
44 44
 	}
45 45
 
46 46
 
@@ -51,10 +51,10 @@  discard block
 block discarded – undo
51 51
 	 * @param string[] $ref List of domain names whose items should be fetched too
52 52
 	 * @return \Aimeos\MShop\Service\Provider\Iface Service provider object
53 53
 	 */
54
-	public function getProvider( $serviceId, $ref = ['media', 'price', 'text'] )
54
+	public function getProvider($serviceId, $ref = ['media', 'price', 'text'])
55 55
 	{
56
-		$manager = \Aimeos\MShop\Factory::createManager( $this->getContext(), 'service' );
57
-		return $manager->getProvider( $manager->getItem( $serviceId, $ref, true ) );
56
+		$manager = \Aimeos\MShop\Factory::createManager($this->getContext(), 'service');
57
+		return $manager->getProvider($manager->getItem($serviceId, $ref, true));
58 58
 	}
59 59
 
60 60
 
@@ -65,26 +65,26 @@  discard block
 block discarded – undo
65 65
 	 * @param string[] $ref List of domain names whose items should be fetched too
66 66
 	 * @return \Aimeos\MShop\Service\Provider\Iface[] List of service IDs as keys and service provider objects as values
67 67
 	 */
68
-	public function getProviders( $type = null, $ref = ['media', 'price', 'text'] )
68
+	public function getProviders($type = null, $ref = ['media', 'price', 'text'])
69 69
 	{
70 70
 		$list = [];
71
-		$manager = \Aimeos\MShop\Factory::createManager( $this->getContext(), 'service' );
71
+		$manager = \Aimeos\MShop\Factory::createManager($this->getContext(), 'service');
72 72
 
73
-		$search = $manager->createSearch( true );
74
-		$search->setSortations( array( $search->sort( '+', 'service.position' ) ) );
73
+		$search = $manager->createSearch(true);
74
+		$search->setSortations(array($search->sort('+', 'service.position')));
75 75
 
76
-		if( $type != null )
76
+		if ($type != null)
77 77
 		{
78 78
 			$expr = array(
79 79
 				$search->getConditions(),
80
-				$search->compare( '==', 'service.type.code', $type ),
81
-				$search->compare( '==', 'service.type.domain', 'service' ),
80
+				$search->compare('==', 'service.type.code', $type),
81
+				$search->compare('==', 'service.type.domain', 'service'),
82 82
 			);
83
-			$search->setConditions( $search->combine( '&&', $expr ) );
83
+			$search->setConditions($search->combine('&&', $expr));
84 84
 		}
85 85
 
86
-		foreach( $manager->searchItems( $search, $ref ) as $id => $item ) {
87
-			$list[$id] = $manager->getProvider( $item );
86
+		foreach ($manager->searchItems($search, $ref) as $id => $item) {
87
+			$list[$id] = $manager->getProvider($item);
88 88
 		}
89 89
 
90 90
 		return $list;
@@ -102,14 +102,14 @@  discard block
 block discarded – undo
102 102
 	 * @return \Aimeos\MShop\Common\Item\Helper\Form\Iface|null Form object with URL, parameters, etc.
103 103
 	 * 	or null if no form data is required
104 104
 	 */
105
-	public function process( \Aimeos\MShop\Order\Item\Iface $orderItem, $serviceId, array $urls, array $params )
105
+	public function process(\Aimeos\MShop\Order\Item\Iface $orderItem, $serviceId, array $urls, array $params)
106 106
 	{
107
-		$manager = \Aimeos\MShop\Factory::createManager( $this->getContext(), 'service' );
107
+		$manager = \Aimeos\MShop\Factory::createManager($this->getContext(), 'service');
108 108
 
109
-		$provider = $manager->getProvider( $manager->getItem( $serviceId, [], true ) );
110
-		$provider->injectGlobalConfigBE( $urls );
109
+		$provider = $manager->getProvider($manager->getItem($serviceId, [], true));
110
+		$provider->injectGlobalConfigBE($urls);
111 111
 
112
-		return $provider->process( $orderItem, $params );
112
+		return $provider->process($orderItem, $params);
113 113
 	}
114 114
 
115 115
 
@@ -124,38 +124,38 @@  discard block
 block discarded – undo
124 124
 	 * @param string $orderid Unique ID of the order whose payment status should be updated
125 125
 	 * @return \Aimeos\MShop\Order\Item\Iface $orderItem Order item that has been updated
126 126
 	 */
127
-	public function updateSync( ServerRequestInterface $request, ResponseInterface $response, array $urls, $code, $orderid )
127
+	public function updateSync(ServerRequestInterface $request, ResponseInterface $response, array $urls, $code, $orderid)
128 128
 	{
129 129
 		$params = (array) $request->getAttributes() + (array) $request->getParsedBody() + (array) $request->getQueryParams();
130 130
 		$params['orderid'] = $orderid;
131 131
 
132 132
 		$context = $this->getContext();
133
-		$manager = \Aimeos\MShop\Factory::createManager( $context, 'service' );
133
+		$manager = \Aimeos\MShop\Factory::createManager($context, 'service');
134 134
 
135
-		$provider = $manager->getProvider( $manager->findItem( $code ) );
136
-		$provider->injectGlobalConfigBE( $urls );
135
+		$provider = $manager->getProvider($manager->findItem($code));
136
+		$provider->injectGlobalConfigBE($urls);
137 137
 
138 138
 		$body = (string) $request->getBody();
139 139
 		$output = null;
140 140
 		$headers = [];
141 141
 
142
-		if( ( $orderItem = $provider->updateSync( $params, $body, $output, $headers ) ) !== null )
142
+		if (($orderItem = $provider->updateSync($params, $body, $output, $headers)) !== null)
143 143
 		{
144
-			if( $orderItem->getPaymentStatus() === \Aimeos\MShop\Order\Item\Base::PAY_UNFINISHED
145
-				&& $provider->isImplemented( \Aimeos\MShop\Service\Provider\Payment\Base::FEAT_QUERY )
144
+			if ($orderItem->getPaymentStatus() === \Aimeos\MShop\Order\Item\Base::PAY_UNFINISHED
145
+				&& $provider->isImplemented(\Aimeos\MShop\Service\Provider\Payment\Base::FEAT_QUERY)
146 146
 			) {
147
-				$provider->query( $orderItem );
147
+				$provider->query($orderItem);
148 148
 			}
149 149
 
150 150
 			// update stock, coupons, etc.
151
-			\Aimeos\Controller\Frontend\Factory::createController( $context, 'order' )->update( $orderItem );
151
+			\Aimeos\Controller\Frontend\Factory::createController($context, 'order')->update($orderItem);
152 152
 		}
153 153
 
154
-		foreach( $headers as $name => $header ) {
155
-			$response->withHeader( $name, $header );
154
+		foreach ($headers as $name => $header) {
155
+			$response->withHeader($name, $header);
156 156
 		}
157 157
 
158
-		$response->withBody( $response->createStreamFromString( $output ) );
158
+		$response->withBody($response->createStreamFromString($output));
159 159
 
160 160
 		return $orderItem;
161 161
 	}
@@ -171,38 +171,38 @@  discard block
 block discarded – undo
171 171
 	 * @throws \Exception If an error occurs
172 172
 	 * @deprecated Use getProviders() instead
173 173
 	 */
174
-	public function getServices( $type, \Aimeos\MShop\Order\Item\Base\Iface $basket, $ref = ['media', 'price', 'text'] )
174
+	public function getServices($type, \Aimeos\MShop\Order\Item\Base\Iface $basket, $ref = ['media', 'price', 'text'])
175 175
 	{
176
-		$serviceManager = \Aimeos\MShop\Factory::createManager( $this->getContext(), 'service' );
176
+		$serviceManager = \Aimeos\MShop\Factory::createManager($this->getContext(), 'service');
177 177
 
178
-		$search = $serviceManager->createSearch( true );
178
+		$search = $serviceManager->createSearch(true);
179 179
 		$expr = array(
180 180
 			$search->getConditions(),
181
-			$search->compare( '==', 'service.type.domain', 'service' ),
182
-			$search->compare( '==', 'service.type.code', $type ),
181
+			$search->compare('==', 'service.type.domain', 'service'),
182
+			$search->compare('==', 'service.type.code', $type),
183 183
 		);
184
-		$search->setConditions( $search->combine( '&&', $expr ) );
185
-		$search->setSortations( array( $search->sort( '+', 'service.position' ) ) );
184
+		$search->setConditions($search->combine('&&', $expr));
185
+		$search->setSortations(array($search->sort('+', 'service.position')));
186 186
 
187
-		$items = $serviceManager->searchItems( $search, $ref );
187
+		$items = $serviceManager->searchItems($search, $ref);
188 188
 
189 189
 
190
-		foreach( $items as $id => $service )
190
+		foreach ($items as $id => $service)
191 191
 		{
192 192
 			try
193 193
 			{
194
-				$provider = $serviceManager->getProvider( $service );
194
+				$provider = $serviceManager->getProvider($service);
195 195
 
196
-				if( $provider->isAvailable( $basket ) ) {
196
+				if ($provider->isAvailable($basket)) {
197 197
 					$this->providers[$type][$id] = $provider;
198 198
 				} else {
199
-					unset( $items[$id] );
199
+					unset($items[$id]);
200 200
 				}
201 201
 			}
202
-			catch( \Aimeos\MShop\Service\Exception $e )
202
+			catch (\Aimeos\MShop\Service\Exception $e)
203 203
 			{
204
-				$msg = sprintf( 'Unable to create provider "%1$s" for service with ID "%2$s"', $service->getCode(), $id );
205
-				$this->getContext()->getLogger()->log( $msg, \Aimeos\MW\Logger\Base::WARN );
204
+				$msg = sprintf('Unable to create provider "%1$s" for service with ID "%2$s"', $service->getCode(), $id);
205
+				$this->getContext()->getLogger()->log($msg, \Aimeos\MW\Logger\Base::WARN);
206 206
 			}
207 207
 		}
208 208
 
@@ -223,16 +223,16 @@  discard block
 block discarded – undo
223 223
 	 * @throws \Exception If an error occurs
224 224
 	 * @deprecated Use getProvider() instead
225 225
 	 */
226
-	public function getServiceAttributes( $type, $serviceId, \Aimeos\MShop\Order\Item\Base\Iface $basket )
226
+	public function getServiceAttributes($type, $serviceId, \Aimeos\MShop\Order\Item\Base\Iface $basket)
227 227
 	{
228
-		if( isset( $this->providers[$type][$serviceId] ) ) {
229
-			return $this->providers[$type][$serviceId]->getConfigFE( $basket );
228
+		if (isset($this->providers[$type][$serviceId])) {
229
+			return $this->providers[$type][$serviceId]->getConfigFE($basket);
230 230
 		}
231 231
 
232
-		$serviceManager = \Aimeos\MShop\Factory::createManager( $this->getContext(), 'service' );
233
-		$item = $serviceManager->getItem( $serviceId, ['price'], true );
232
+		$serviceManager = \Aimeos\MShop\Factory::createManager($this->getContext(), 'service');
233
+		$item = $serviceManager->getItem($serviceId, ['price'], true);
234 234
 
235
-		return $serviceManager->getProvider( $item )->getConfigFE( $basket );
235
+		return $serviceManager->getProvider($item)->getConfigFE($basket);
236 236
 	}
237 237
 
238 238
 
@@ -248,16 +248,16 @@  discard block
 block discarded – undo
248 248
 	 * @throws \Exception If an error occurs
249 249
 	 * @deprecated Use getProvider() instead
250 250
 	 */
251
-	public function getServicePrice( $type, $serviceId, \Aimeos\MShop\Order\Item\Base\Iface $basket )
251
+	public function getServicePrice($type, $serviceId, \Aimeos\MShop\Order\Item\Base\Iface $basket)
252 252
 	{
253
-		if( isset( $this->providers[$type][$serviceId] ) ) {
254
-			return $this->providers[$type][$serviceId]->calcPrice( $basket );
253
+		if (isset($this->providers[$type][$serviceId])) {
254
+			return $this->providers[$type][$serviceId]->calcPrice($basket);
255 255
 		}
256 256
 
257
-		$serviceManager = \Aimeos\MShop\Factory::createManager( $this->getContext(), 'service' );
258
-		$item = $serviceManager->getItem( $serviceId, ['price'], true );
257
+		$serviceManager = \Aimeos\MShop\Factory::createManager($this->getContext(), 'service');
258
+		$item = $serviceManager->getItem($serviceId, ['price'], true);
259 259
 
260
-		return $serviceManager->getProvider( $item )->calcPrice( $basket );
260
+		return $serviceManager->getProvider($item)->calcPrice($basket);
261 261
 	}
262 262
 
263 263
 
@@ -273,22 +273,22 @@  discard block
 block discarded – undo
273 273
 	 * @throws \Aimeos\Controller\Frontend\Service\Exception If no active service provider for this ID is available
274 274
 	 * @deprecated Use checkAttributes() instead
275 275
 	 */
276
-	public function checkServiceAttributes( $type, $serviceId, array $attributes )
276
+	public function checkServiceAttributes($type, $serviceId, array $attributes)
277 277
 	{
278
-		if( !isset( $this->providers[$type][$serviceId] ) )
278
+		if (!isset($this->providers[$type][$serviceId]))
279 279
 		{
280
-			$serviceManager = \Aimeos\MShop\Factory::createManager( $this->getContext(), 'service' );
281
-			$item = $serviceManager->getItem( $serviceId, ['price'], true );
280
+			$serviceManager = \Aimeos\MShop\Factory::createManager($this->getContext(), 'service');
281
+			$item = $serviceManager->getItem($serviceId, ['price'], true);
282 282
 
283
-			$this->providers[$type][$serviceId] = $serviceManager->getProvider( $item );
283
+			$this->providers[$type][$serviceId] = $serviceManager->getProvider($item);
284 284
 		}
285 285
 
286
-		$errors = $this->providers[$type][$serviceId]->checkConfigFE( $attributes );
286
+		$errors = $this->providers[$type][$serviceId]->checkConfigFE($attributes);
287 287
 
288
-		foreach( $errors as $key => $msg )
288
+		foreach ($errors as $key => $msg)
289 289
 		{
290
-			if( $msg === null ) {
291
-				unset( $errors[$key] );
290
+			if ($msg === null) {
291
+				unset($errors[$key]);
292 292
 			}
293 293
 		}
294 294
 
Please login to merge, or discard this patch.
controller/frontend/src/Controller/Frontend/Service/Decorator/Base.php 1 patch
Spacing   +26 added lines, -26 removed lines patch added patch discarded remove patch
@@ -33,18 +33,18 @@  discard block
 block discarded – undo
33 33
 	 * @param \Aimeos\Controller\Frontend\Iface $controller Controller object
34 34
 	 * @param \Aimeos\MShop\Context\Item\Iface $context Context object with required objects
35 35
 	 */
36
-	public function __construct( \Aimeos\Controller\Frontend\Iface $controller, \Aimeos\MShop\Context\Item\Iface $context )
36
+	public function __construct(\Aimeos\Controller\Frontend\Iface $controller, \Aimeos\MShop\Context\Item\Iface $context)
37 37
 	{
38 38
 		$iface = '\Aimeos\Controller\Frontend\Service\Iface';
39
-		if( !( $controller instanceof $iface ) )
39
+		if (!($controller instanceof $iface))
40 40
 		{
41
-			$msg = sprintf( 'Class "%1$s" does not implement interface "%2$s"', get_class( $controller ), $iface );
42
-			throw new \Aimeos\Controller\Frontend\Exception( $msg );
41
+			$msg = sprintf('Class "%1$s" does not implement interface "%2$s"', get_class($controller), $iface);
42
+			throw new \Aimeos\Controller\Frontend\Exception($msg);
43 43
 		}
44 44
 
45 45
 		$this->controller = $controller;
46 46
 
47
-		parent::__construct( $context );
47
+		parent::__construct($context);
48 48
 	}
49 49
 
50 50
 
@@ -56,9 +56,9 @@  discard block
 block discarded – undo
56 56
 	 * @return mixed Returns the value of the called method
57 57
 	 * @throws \Aimeos\Controller\Frontend\Exception If method call failed
58 58
 	 */
59
-	public function __call( $name, array $param )
59
+	public function __call($name, array $param)
60 60
 	{
61
-		return @call_user_func_array( array( $this->controller, $name ), $param );
61
+		return @call_user_func_array(array($this->controller, $name), $param);
62 62
 	}
63 63
 
64 64
 
@@ -69,9 +69,9 @@  discard block
 block discarded – undo
69 69
 	 * @param string[] $attributes List of attribute codes as keys and strings entered by the customer as value
70 70
 	 * @return string[] List of attributes codes as keys and error messages as values for invalid or missing values
71 71
 	 */
72
-	public function checkAttributes( $serviceId, array $attributes )
72
+	public function checkAttributes($serviceId, array $attributes)
73 73
 	{
74
-		return $this->controller->checkAttributes( $serviceId, $attributes );
74
+		return $this->controller->checkAttributes($serviceId, $attributes);
75 75
 	}
76 76
 
77 77
 
@@ -82,9 +82,9 @@  discard block
 block discarded – undo
82 82
 	 * @param array $ref List of domains for which the items referenced by the services should be fetched too
83 83
 	 * @return \Aimeos\MShop\Service\Provider\Iface Service provider object
84 84
 	 */
85
-	public function getProvider( $serviceId, $ref = ['media', 'price', 'text'] )
85
+	public function getProvider($serviceId, $ref = ['media', 'price', 'text'])
86 86
 	{
87
-		return $this->controller->getProvider( $serviceId, $ref );
87
+		return $this->controller->getProvider($serviceId, $ref);
88 88
 	}
89 89
 
90 90
 
@@ -95,9 +95,9 @@  discard block
 block discarded – undo
95 95
 	 * @param array $ref List of domains for which the items referenced by the services should be fetched too
96 96
 	 * @return \Aimeos\MShop\Service\Provider\Iface[] List of service IDs as keys and service provider objects as values
97 97
 	 */
98
-	public function getProviders( $type = null, $ref = ['media', 'price', 'text'] )
98
+	public function getProviders($type = null, $ref = ['media', 'price', 'text'])
99 99
 	{
100
-		return $this->controller->getProviders( $type, $ref );
100
+		return $this->controller->getProviders($type, $ref);
101 101
 	}
102 102
 
103 103
 
@@ -112,9 +112,9 @@  discard block
 block discarded – undo
112 112
 	 * @return \Aimeos\MShop\Common\Item\Helper\Form\Iface|null Form object with URL, parameters, etc.
113 113
 	 * 	or null if no form data is required
114 114
 	 */
115
-	public function process( \Aimeos\MShop\Order\Item\Iface $orderItem, $serviceId, array $urls, array $params )
115
+	public function process(\Aimeos\MShop\Order\Item\Iface $orderItem, $serviceId, array $urls, array $params)
116 116
 	{
117
-		return $this->controller->process( $orderItem, $serviceId, $urls, $params );
117
+		return $this->controller->process($orderItem, $serviceId, $urls, $params);
118 118
 	}
119 119
 
120 120
 
@@ -129,9 +129,9 @@  discard block
 block discarded – undo
129 129
 	 * @param string $orderid Unique ID of the order whose payment status should be updated
130 130
 	 * @return \Aimeos\MShop\Order\Item\Iface $orderItem Order item that has been updated
131 131
 	 */
132
-	public function updateSync( ServerRequestInterface $request, ResponseInterface $response, array $urls, $code, $orderid )
132
+	public function updateSync(ServerRequestInterface $request, ResponseInterface $response, array $urls, $code, $orderid)
133 133
 	{
134
-		return $this->controller->updateSync( $request, $response, $urls, $code, $orderid );
134
+		return $this->controller->updateSync($request, $response, $urls, $code, $orderid);
135 135
 	}
136 136
 
137 137
 
@@ -144,10 +144,10 @@  discard block
 block discarded – undo
144 144
 	 * @return array List of service items implementing \Aimeos\MShop\Service\Item\Iface with referenced items
145 145
 	 * @deprecated Use getProviders() instead
146 146
 	 */
147
-	public function getServices( $type, \Aimeos\MShop\Order\Item\Base\Iface $basket,
148
-		$ref = array( 'media', 'price', 'text' ) )
147
+	public function getServices($type, \Aimeos\MShop\Order\Item\Base\Iface $basket,
148
+		$ref = array('media', 'price', 'text'))
149 149
 	{
150
-		return $this->controller->getServices( $type, $basket, $ref );
150
+		return $this->controller->getServices($type, $basket, $ref);
151 151
 	}
152 152
 
153 153
 
@@ -161,9 +161,9 @@  discard block
 block discarded – undo
161 161
 	 * @return array List of attribute definitions implementing \Aimeos\MW\Criteria\Attribute\Iface
162 162
 	 * @deprecated Use getProvider() instead
163 163
 	 */
164
-	public function getServiceAttributes( $type, $serviceId, \Aimeos\MShop\Order\Item\Base\Iface $basket )
164
+	public function getServiceAttributes($type, $serviceId, \Aimeos\MShop\Order\Item\Base\Iface $basket)
165 165
 	{
166
-		return $this->controller->getServiceAttributes( $type, $serviceId, $basket );
166
+		return $this->controller->getServiceAttributes($type, $serviceId, $basket);
167 167
 	}
168 168
 
169 169
 
@@ -179,9 +179,9 @@  discard block
 block discarded – undo
179 179
 	 * @throws \Exception If an error occurs
180 180
 	 * @deprecated Use getProvider() instead
181 181
 	 */
182
-	public function getServicePrice( $type, $serviceId, \Aimeos\MShop\Order\Item\Base\Iface $basket )
182
+	public function getServicePrice($type, $serviceId, \Aimeos\MShop\Order\Item\Base\Iface $basket)
183 183
 	{
184
-		return $this->controller->getServicePrice( $type, $serviceId, $basket );
184
+		return $this->controller->getServicePrice($type, $serviceId, $basket);
185 185
 	}
186 186
 
187 187
 
@@ -196,9 +196,9 @@  discard block
 block discarded – undo
196 196
 	 * 	missing
197 197
 	 * @deprecated Use checkAttributes() instead
198 198
 	 */
199
-	public function checkServiceAttributes( $type, $serviceId, array $attributes )
199
+	public function checkServiceAttributes($type, $serviceId, array $attributes)
200 200
 	{
201
-		return $this->controller->checkServiceAttributes( $type, $serviceId, $attributes );
201
+		return $this->controller->checkServiceAttributes($type, $serviceId, $attributes);
202 202
 	}
203 203
 
204 204
 
Please login to merge, or discard this patch.
controller/frontend/src/Controller/Frontend/Basket/Decorator/Bundle.php 1 patch
Spacing   +33 added lines, -33 removed lines patch added patch discarded remove patch
@@ -41,14 +41,14 @@  discard block
 block discarded – undo
41 41
 	 * @param string $stocktype Unique code of the stock type to deliver the products from
42 42
 	 * @throws \Aimeos\Controller\Frontend\Basket\Exception If the product isn't available
43 43
 	 */
44
-	public function addProduct( $prodid, $quantity = 1, array $options = [], array $variantAttributeIds = [],
44
+	public function addProduct($prodid, $quantity = 1, array $options = [], array $variantAttributeIds = [],
45 45
 		array $configAttributeIds = [], array $hiddenAttributeIds = [], array $customAttributeValues = [],
46
-		$stocktype = 'default' )
46
+		$stocktype = 'default')
47 47
 	{
48 48
 		$context = $this->getContext();
49
-		$productManager = \Aimeos\MShop\Factory::createManager( $context, 'product' );
49
+		$productManager = \Aimeos\MShop\Factory::createManager($context, 'product');
50 50
 
51
-		if( $productManager->getItem( $prodid, [], true )->getType() !== 'bundle' )
51
+		if ($productManager->getItem($prodid, [], true)->getType() !== 'bundle')
52 52
 		{
53 53
 			return $this->getController()->addProduct(
54 54
 				$prodid, $quantity, $options, $variantAttributeIds, $configAttributeIds,
@@ -57,29 +57,29 @@  discard block
 block discarded – undo
57 57
 		}
58 58
 
59 59
 		$attributeMap = [
60
-			'custom' => array_keys( $customAttributeValues ),
60
+			'custom' => array_keys($customAttributeValues),
61 61
 			'config' => $configAttributeIds,
62 62
 			'hidden' => $hiddenAttributeIds,
63 63
 		];
64
-		$this->checkListRef( $prodid, 'attribute', $attributeMap );
64
+		$this->checkListRef($prodid, 'attribute', $attributeMap);
65 65
 
66 66
 
67
-		$productItem = $productManager->getItem( $prodid, array( 'media', 'supplier', 'price', 'product', 'text' ), true );
68
-		$prices = $productItem->getRefItems( 'price', 'default', 'default' );
67
+		$productItem = $productManager->getItem($prodid, array('media', 'supplier', 'price', 'product', 'text'), true);
68
+		$prices = $productItem->getRefItems('price', 'default', 'default');
69 69
 
70
-		$orderBaseProductItem = \Aimeos\MShop\Factory::createManager( $context, 'order/base/product' )->createItem();
71
-		$orderBaseProductItem->copyFrom( $productItem )->setQuantity( $quantity )->setStockType( $stocktype );
70
+		$orderBaseProductItem = \Aimeos\MShop\Factory::createManager($context, 'order/base/product')->createItem();
71
+		$orderBaseProductItem->copyFrom($productItem)->setQuantity($quantity)->setStockType($stocktype);
72 72
 
73
-		$this->addBundleProducts( $orderBaseProductItem, $productItem, $variantAttributeIds, $stocktype );
73
+		$this->addBundleProducts($orderBaseProductItem, $productItem, $variantAttributeIds, $stocktype);
74 74
 
75
-		$attr = $this->getOrderProductAttributes( 'custom', array_keys( $customAttributeValues ), $customAttributeValues );
76
-		$attr = array_merge( $attr, $this->getOrderProductAttributes( 'config', $configAttributeIds ) );
77
-		$attr = array_merge( $attr, $this->getOrderProductAttributes( 'hidden', $hiddenAttributeIds ) );
75
+		$attr = $this->getOrderProductAttributes('custom', array_keys($customAttributeValues), $customAttributeValues);
76
+		$attr = array_merge($attr, $this->getOrderProductAttributes('config', $configAttributeIds));
77
+		$attr = array_merge($attr, $this->getOrderProductAttributes('hidden', $hiddenAttributeIds));
78 78
 
79
-		$orderBaseProductItem->setAttributes( $attr );
80
-		$orderBaseProductItem->setPrice( $this->calcPrice( $orderBaseProductItem, $prices, $quantity ) );
79
+		$orderBaseProductItem->setAttributes($attr);
80
+		$orderBaseProductItem->setPrice($this->calcPrice($orderBaseProductItem, $prices, $quantity));
81 81
 
82
-		$this->getController()->get()->addProduct( $orderBaseProductItem );
82
+		$this->getController()->get()->addProduct($orderBaseProductItem);
83 83
 		$this->getController()->save();
84 84
 	}
85 85
 
@@ -92,43 +92,43 @@  discard block
 block discarded – undo
92 92
 	 * @param array $variantAttributeIds List of product variant attribute IDs
93 93
 	 * @param string $stocktype
94 94
 	 */
95
-	protected function addBundleProducts( \Aimeos\MShop\Order\Item\Base\Product\Iface $orderBaseProductItem,
96
-		\Aimeos\MShop\Product\Item\Iface $productItem, array $variantAttributeIds, $stocktype )
95
+	protected function addBundleProducts(\Aimeos\MShop\Order\Item\Base\Product\Iface $orderBaseProductItem,
96
+		\Aimeos\MShop\Product\Item\Iface $productItem, array $variantAttributeIds, $stocktype)
97 97
 	{
98 98
 		$quantity = $orderBaseProductItem->getQuantity();
99 99
 		$products = $subProductIds = $orderProducts = [];
100
-		$orderProductManager = \Aimeos\MShop\Factory::createManager( $this->getContext(), 'order/base/product' );
100
+		$orderProductManager = \Aimeos\MShop\Factory::createManager($this->getContext(), 'order/base/product');
101 101
 
102
-		foreach( $productItem->getRefItems( 'product', null, 'default' ) as $item ) {
102
+		foreach ($productItem->getRefItems('product', null, 'default') as $item) {
103 103
 			$subProductIds[] = $item->getId();
104 104
 		}
105 105
 
106
-		if( count( $subProductIds ) > 0 )
106
+		if (count($subProductIds) > 0)
107 107
 		{
108
-			$productManager = \Aimeos\MShop\Factory::createManager( $this->getContext(), 'product' );
108
+			$productManager = \Aimeos\MShop\Factory::createManager($this->getContext(), 'product');
109 109
 
110
-			$search = $productManager->createSearch( true );
110
+			$search = $productManager->createSearch(true);
111 111
 			$expr = array(
112
-				$search->compare( '==', 'product.id', $subProductIds ),
112
+				$search->compare('==', 'product.id', $subProductIds),
113 113
 				$search->getConditions(),
114 114
 			);
115
-			$search->setConditions( $search->combine( '&&', $expr ) );
115
+			$search->setConditions($search->combine('&&', $expr));
116 116
 
117
-			$products = $productManager->searchItems( $search, array( 'attribute', 'media', 'price', 'text' ) );
117
+			$products = $productManager->searchItems($search, array('attribute', 'media', 'price', 'text'));
118 118
 		}
119 119
 
120
-		foreach( $products as $product )
120
+		foreach ($products as $product)
121 121
 		{
122
-			$prices = $product->getRefItems( 'price', 'default', 'default' );
122
+			$prices = $product->getRefItems('price', 'default', 'default');
123 123
 
124 124
 			$orderProduct = $orderProductManager->createItem();
125
-			$orderProduct->copyFrom( $product );
126
-			$orderProduct->setStockType( $stocktype );
127
-			$orderProduct->setPrice( $this->calcPrice( $orderProduct, $prices, $quantity ) );
125
+			$orderProduct->copyFrom($product);
126
+			$orderProduct->setStockType($stocktype);
127
+			$orderProduct->setPrice($this->calcPrice($orderProduct, $prices, $quantity));
128 128
 
129 129
 			$orderProducts[] = $orderProduct;
130 130
 		}
131 131
 
132
-		$orderBaseProductItem->setProducts( $orderProducts );
132
+		$orderBaseProductItem->setProducts($orderProducts);
133 133
 	}
134 134
 }
Please login to merge, or discard this patch.
controller/frontend/src/Controller/Frontend/Basket/Decorator/Select.php 1 patch
Spacing   +39 added lines, -39 removed lines patch added patch discarded remove patch
@@ -41,15 +41,15 @@  discard block
 block discarded – undo
41 41
 	 * @param string $stocktype Unique code of the stock type to deliver the products from
42 42
 	 * @throws \Aimeos\Controller\Frontend\Basket\Exception If the product isn't available
43 43
 	 */
44
-	public function addProduct( $prodid, $quantity = 1, array $options = [], array $variantAttributeIds = [],
44
+	public function addProduct($prodid, $quantity = 1, array $options = [], array $variantAttributeIds = [],
45 45
 		array $configAttributeIds = [], array $hiddenAttributeIds = [], array $customAttributeValues = [],
46
-		$stocktype = 'default' )
46
+		$stocktype = 'default')
47 47
 	{
48 48
 		$context = $this->getContext();
49
-		$productManager = \Aimeos\MShop\Factory::createManager( $context, 'product' );
50
-		$productItem = $productManager->getItem( $prodid, [], true );
49
+		$productManager = \Aimeos\MShop\Factory::createManager($context, 'product');
50
+		$productItem = $productManager->getItem($prodid, [], true);
51 51
 
52
-		if( $productManager->getItem( $prodid, [], true )->getType() !== 'select' )
52
+		if ($productManager->getItem($prodid, [], true)->getType() !== 'select')
53 53
 		{
54 54
 			return $this->getController()->addProduct(
55 55
 				$prodid, $quantity, $options, $variantAttributeIds, $configAttributeIds,
@@ -57,29 +57,29 @@  discard block
 block discarded – undo
57 57
 			);
58 58
 		}
59 59
 
60
-		$productItem = $productManager->getItem( $prodid, array( 'media', 'supplier', 'price', 'product', 'text' ), true );
61
-		$prices = $productItem->getRefItems( 'price', 'default', 'default' );
60
+		$productItem = $productManager->getItem($prodid, array('media', 'supplier', 'price', 'product', 'text'), true);
61
+		$prices = $productItem->getRefItems('price', 'default', 'default');
62 62
 
63
-		$orderBaseProductItem = \Aimeos\MShop\Factory::createManager( $context, 'order/base/product' )->createItem();
64
-		$orderBaseProductItem->copyFrom( $productItem )->setQuantity( $quantity )->setStockType( $stocktype );
63
+		$orderBaseProductItem = \Aimeos\MShop\Factory::createManager($context, 'order/base/product')->createItem();
64
+		$orderBaseProductItem->copyFrom($productItem)->setQuantity($quantity)->setStockType($stocktype);
65 65
 
66
-		$attr = $this->getVariantDetails( $orderBaseProductItem, $productItem, $prices, $variantAttributeIds, $options );
66
+		$attr = $this->getVariantDetails($orderBaseProductItem, $productItem, $prices, $variantAttributeIds, $options);
67 67
 
68 68
 		$attributeMap = [
69
-			'custom' => array_keys( $customAttributeValues ),
69
+			'custom' => array_keys($customAttributeValues),
70 70
 			'config' => $configAttributeIds,
71 71
 			'hidden' => $hiddenAttributeIds,
72 72
 		];
73
-		$this->checkListRef( array( $prodid, $productItem->getId() ), 'attribute', $attributeMap );
73
+		$this->checkListRef(array($prodid, $productItem->getId()), 'attribute', $attributeMap);
74 74
 
75
-		$attr = array_merge( $attr, $this->getOrderProductAttributes( 'custom', array_keys( $customAttributeValues ), $customAttributeValues ) );
76
-		$attr = array_merge( $attr, $this->getOrderProductAttributes( 'config', $configAttributeIds ) );
77
-		$attr = array_merge( $attr, $this->getOrderProductAttributes( 'hidden', $hiddenAttributeIds ) );
75
+		$attr = array_merge($attr, $this->getOrderProductAttributes('custom', array_keys($customAttributeValues), $customAttributeValues));
76
+		$attr = array_merge($attr, $this->getOrderProductAttributes('config', $configAttributeIds));
77
+		$attr = array_merge($attr, $this->getOrderProductAttributes('hidden', $hiddenAttributeIds));
78 78
 
79
-		$orderBaseProductItem->setAttributes( $attr );
80
-		$orderBaseProductItem->setPrice( $this->calcPrice( $orderBaseProductItem, $prices, $quantity ) );
79
+		$orderBaseProductItem->setAttributes($attr);
80
+		$orderBaseProductItem->setPrice($this->calcPrice($orderBaseProductItem, $prices, $quantity));
81 81
 
82
-		$this->getController()->get()->addProduct( $orderBaseProductItem );
82
+		$this->getController()->get()->addProduct($orderBaseProductItem);
83 83
 		$this->getController()->save();
84 84
 	}
85 85
 
@@ -95,50 +95,50 @@  discard block
 block discarded – undo
95 95
 	 * @return \Aimeos\MShop\Order\Item\Base\Product\Attribute\Iface[] List of order product attributes
96 96
 	 * @throws \Aimeos\Controller\Frontend\Basket\Exception If no product variant is found
97 97
 	 */
98
-	protected function getVariantDetails( \Aimeos\MShop\Order\Item\Base\Product\Iface $orderBaseProductItem,
99
-		\Aimeos\MShop\Product\Item\Iface &$productItem, array &$prices, array $variantAttributeIds, array $options )
98
+	protected function getVariantDetails(\Aimeos\MShop\Order\Item\Base\Product\Iface $orderBaseProductItem,
99
+		\Aimeos\MShop\Product\Item\Iface & $productItem, array &$prices, array $variantAttributeIds, array $options)
100 100
 	{
101 101
 		$attr = [];
102
-		$productItems = $this->getProductVariants( $productItem, $variantAttributeIds );
102
+		$productItems = $this->getProductVariants($productItem, $variantAttributeIds);
103 103
 
104
-		if( count( $productItems ) > 1 )
104
+		if (count($productItems) > 1)
105 105
 		{
106
-			$msg = sprintf( 'No unique article found for selected attributes and product ID "%1$s"', $productItem->getId() );
107
-			throw new \Aimeos\Controller\Frontend\Basket\Exception( $msg );
106
+			$msg = sprintf('No unique article found for selected attributes and product ID "%1$s"', $productItem->getId());
107
+			throw new \Aimeos\Controller\Frontend\Basket\Exception($msg);
108 108
 		}
109
-		else if( ( $result = reset( $productItems ) ) !== false ) // count == 1
109
+		else if (($result = reset($productItems)) !== false) // count == 1
110 110
 		{
111 111
 			$productItem = $result;
112
-			$orderBaseProductItem->setProductCode( $productItem->getCode() );
112
+			$orderBaseProductItem->setProductCode($productItem->getCode());
113 113
 
114
-			$subprices = $productItem->getRefItems( 'price', 'default', 'default' );
114
+			$subprices = $productItem->getRefItems('price', 'default', 'default');
115 115
 
116
-			if( !empty( $subprices ) ) {
116
+			if (!empty($subprices)) {
117 117
 				$prices = $subprices;
118 118
 			}
119 119
 
120
-			$submedia = $productItem->getRefItems( 'media', 'default', 'default' );
120
+			$submedia = $productItem->getRefItems('media', 'default', 'default');
121 121
 
122
-			if( ( $mediaItem = reset( $submedia ) ) !== false ) {
123
-				$orderBaseProductItem->setMediaUrl( $mediaItem->getPreview() );
122
+			if (($mediaItem = reset($submedia)) !== false) {
123
+				$orderBaseProductItem->setMediaUrl($mediaItem->getPreview());
124 124
 			}
125 125
 
126
-			$orderProductAttrManager = \Aimeos\MShop\Factory::createManager( $this->getContext(), 'order/base/product/attribute' );
127
-			$variantAttributes = $productItem->getRefItems( 'attribute', null, 'variant' );
126
+			$orderProductAttrManager = \Aimeos\MShop\Factory::createManager($this->getContext(), 'order/base/product/attribute');
127
+			$variantAttributes = $productItem->getRefItems('attribute', null, 'variant');
128 128
 
129
-			foreach( $this->getAttributes( array_keys( $variantAttributes ), array( 'text' ) ) as $attrItem )
129
+			foreach ($this->getAttributes(array_keys($variantAttributes), array('text')) as $attrItem)
130 130
 			{
131 131
 				$orderAttributeItem = $orderProductAttrManager->createItem();
132
-				$orderAttributeItem->copyFrom( $attrItem );
133
-				$orderAttributeItem->setType( 'variant' );
132
+				$orderAttributeItem->copyFrom($attrItem);
133
+				$orderAttributeItem->setType('variant');
134 134
 
135 135
 				$attr[] = $orderAttributeItem;
136 136
 			}
137 137
 		}
138
-		else if( !isset( $options['variant'] ) || $options['variant'] != false ) // count == 0
138
+		else if (!isset($options['variant']) || $options['variant'] != false) // count == 0
139 139
 		{
140
-			$msg = sprintf( 'No article found for selected attributes and product ID "%1$s"', $productItem->getId() );
141
-			throw new \Aimeos\Controller\Frontend\Basket\Exception( $msg );
140
+			$msg = sprintf('No article found for selected attributes and product ID "%1$s"', $productItem->getId());
141
+			throw new \Aimeos\Controller\Frontend\Basket\Exception($msg);
142 142
 		}
143 143
 
144 144
 		return $attr;
Please login to merge, or discard this patch.
controller/frontend/src/Controller/Frontend/Basket/Base.php 1 patch
Spacing   +174 added lines, -174 removed lines patch added patch discarded remove patch
@@ -31,46 +31,46 @@  discard block
 block discarded – undo
31 31
 	 * @param integer $quantity New product quantity
32 32
 	 * @return \Aimeos\MShop\Price\Item\Iface Price item with calculated price
33 33
 	 */
34
-	protected function calcPrice( \Aimeos\MShop\Order\Item\Base\Product\Iface $product, array $prices, $quantity )
34
+	protected function calcPrice(\Aimeos\MShop\Order\Item\Base\Product\Iface $product, array $prices, $quantity)
35 35
 	{
36 36
 		$context = $this->getContext();
37 37
 
38
-		if( empty( $prices ) )
38
+		if (empty($prices))
39 39
 		{
40
-			$manager = \Aimeos\MShop\Factory::createManager( $this->getContext(), 'product' );
41
-			$prices = $manager->getItem( $product->getProductId(), array( 'price' ) )->getRefItems( 'price', 'default' );
40
+			$manager = \Aimeos\MShop\Factory::createManager($this->getContext(), 'product');
41
+			$prices = $manager->getItem($product->getProductId(), array('price'))->getRefItems('price', 'default');
42 42
 		}
43 43
 
44 44
 
45
-		$priceManager = \Aimeos\MShop\Factory::createManager( $context, 'price' );
46
-		$price = $priceManager->getLowestPrice( $prices, $quantity );
45
+		$priceManager = \Aimeos\MShop\Factory::createManager($context, 'price');
46
+		$price = $priceManager->getLowestPrice($prices, $quantity);
47 47
 
48 48
 		// customers can pay what they would like to pay
49
-		if( ( $attr = $product->getAttributeItem( 'price', 'custom' ) ) !== null )
49
+		if (($attr = $product->getAttributeItem('price', 'custom')) !== null)
50 50
 		{
51 51
 			$amount = $attr->getValue();
52 52
 
53
-			if( preg_match( '/^[0-9]*(\.[0-9]+)?$/', $amount ) !== 1 || ((double) $amount) < 0.01 ) {
54
-				throw new \Aimeos\Controller\Frontend\Basket\Exception( sprintf( 'Invalid price value "%1$s"', $amount ) );
53
+			if (preg_match('/^[0-9]*(\.[0-9]+)?$/', $amount) !== 1 || ((double) $amount) < 0.01) {
54
+				throw new \Aimeos\Controller\Frontend\Basket\Exception(sprintf('Invalid price value "%1$s"', $amount));
55 55
 			}
56 56
 
57
-			$price->setValue( $amount );
57
+			$price->setValue($amount);
58 58
 		}
59 59
 
60 60
 		// add prices of (optional) attributes
61
-		foreach( $this->getAttributeItems( $product->getAttributes() ) as $attrItem )
61
+		foreach ($this->getAttributeItems($product->getAttributes()) as $attrItem)
62 62
 		{
63
-			$prices = $attrItem->getRefItems( 'price', 'default' );
63
+			$prices = $attrItem->getRefItems('price', 'default');
64 64
 
65
-			if( count( $prices ) > 0 )
65
+			if (count($prices) > 0)
66 66
 			{
67
-				$attrPrice = $priceManager->getLowestPrice( $prices, $quantity );
68
-				$price->addItem( $attrPrice );
67
+				$attrPrice = $priceManager->getLowestPrice($prices, $quantity);
68
+				$price->addItem($attrPrice);
69 69
 			}
70 70
 		}
71 71
 
72 72
 		// remove product rebate of original price in favor to rebates granted for the order
73
-		$price->setRebate( '0.00' );
73
+		$price->setRebate('0.00');
74 74
 
75 75
 		return $price;
76 76
 	}
@@ -84,42 +84,42 @@  discard block
 block discarded – undo
84 84
 	 * @param array $refMap Associative list of list type codes as keys and lists of reference IDs as values
85 85
 	 * @throws \Aimeos\Controller\Frontend\Basket\Exception If one or more of the IDs are not associated
86 86
 	 */
87
-	protected function checkListRef( $prodId, $domain, array $refMap )
87
+	protected function checkListRef($prodId, $domain, array $refMap)
88 88
 	{
89
-		if( empty( $refMap ) ) {
89
+		if (empty($refMap)) {
90 90
 			return;
91 91
 		}
92 92
 
93
-		$productManager = \Aimeos\MShop\Factory::createManager( $this->getContext(), 'product' );
94
-		$search = $productManager->createSearch( true );
93
+		$productManager = \Aimeos\MShop\Factory::createManager($this->getContext(), 'product');
94
+		$search = $productManager->createSearch(true);
95 95
 
96 96
 		$expr = array(
97
-			$search->compare( '==', 'product.id', $prodId ),
97
+			$search->compare('==', 'product.id', $prodId),
98 98
 			$search->getConditions(),
99 99
 		);
100 100
 
101
-		foreach( $refMap as $listType => $refIds )
101
+		foreach ($refMap as $listType => $refIds)
102 102
 		{
103
-			if( empty( $refIds ) ) {
103
+			if (empty($refIds)) {
104 104
 				continue;
105 105
 			}
106 106
 
107
-			foreach( $refIds as $key => $refId ) {
107
+			foreach ($refIds as $key => $refId) {
108 108
 				$refIds[$key] = (string) $refId;
109 109
 			}
110 110
 
111
-			$param = array( $domain, $this->getProductListTypeItem( $domain, $listType )->getId(), $refIds );
112
-			$cmpfunc = $search->createFunction( 'product.contains', $param );
111
+			$param = array($domain, $this->getProductListTypeItem($domain, $listType)->getId(), $refIds);
112
+			$cmpfunc = $search->createFunction('product.contains', $param);
113 113
 
114
-			$expr[] = $search->compare( '==', $cmpfunc, count( $refIds ) );
114
+			$expr[] = $search->compare('==', $cmpfunc, count($refIds));
115 115
 		}
116 116
 
117
-		$search->setConditions( $search->combine( '&&', $expr ) );
117
+		$search->setConditions($search->combine('&&', $expr));
118 118
 
119
-		if( count( $productManager->searchItems( $search, [] ) ) === 0 )
119
+		if (count($productManager->searchItems($search, [])) === 0)
120 120
 		{
121
-			$msg = sprintf( 'Invalid "%1$s" references for product with ID %2$s', $domain, json_encode( $prodId ) );
122
-			throw new \Aimeos\Controller\Frontend\Basket\Exception( $msg );
121
+			$msg = sprintf('Invalid "%1$s" references for product with ID %2$s', $domain, json_encode($prodId));
122
+			throw new \Aimeos\Controller\Frontend\Basket\Exception($msg);
123 123
 		}
124 124
 	}
125 125
 
@@ -134,34 +134,34 @@  discard block
 block discarded – undo
134 134
 	 * @throws \Aimeos\Controller\Frontend\Basket\Exception If one or more of the IDs are not associated
135 135
 	 * @deprecated Use checkListRef() instead
136 136
 	 */
137
-	protected function checkReferences( $prodId, $domain, $listTypeId, array $refIds )
137
+	protected function checkReferences($prodId, $domain, $listTypeId, array $refIds)
138 138
 	{
139
-		$productManager = \Aimeos\MShop\Factory::createManager( $this->getContext(), 'product' );
140
-		$search = $productManager->createSearch( true );
139
+		$productManager = \Aimeos\MShop\Factory::createManager($this->getContext(), 'product');
140
+		$search = $productManager->createSearch(true);
141 141
 
142 142
 		$expr = array(
143
-			$search->compare( '==', 'product.id', $prodId ),
143
+			$search->compare('==', 'product.id', $prodId),
144 144
 			$search->getConditions(),
145 145
 		);
146 146
 
147
-		if( count( $refIds ) > 0 )
147
+		if (count($refIds) > 0)
148 148
 		{
149
-			foreach( $refIds as $key => $refId ) {
149
+			foreach ($refIds as $key => $refId) {
150 150
 				$refIds[$key] = (string) $refId;
151 151
 			}
152 152
 
153
-			$param = array( $domain, $listTypeId, $refIds );
154
-			$cmpfunc = $search->createFunction( 'product.contains', $param );
153
+			$param = array($domain, $listTypeId, $refIds);
154
+			$cmpfunc = $search->createFunction('product.contains', $param);
155 155
 
156
-			$expr[] = $search->compare( '==', $cmpfunc, count( $refIds ) );
156
+			$expr[] = $search->compare('==', $cmpfunc, count($refIds));
157 157
 		}
158 158
 
159
-		$search->setConditions( $search->combine( '&&', $expr ) );
159
+		$search->setConditions($search->combine('&&', $expr));
160 160
 
161
-		if( count( $productManager->searchItems( $search, [] ) ) === 0 )
161
+		if (count($productManager->searchItems($search, [])) === 0)
162 162
 		{
163
-			$msg = sprintf( 'Invalid "%1$s" references for product with ID %2$s', $domain, json_encode( $prodId ) );
164
-			throw new \Aimeos\Controller\Frontend\Basket\Exception( $msg );
163
+			$msg = sprintf('Invalid "%1$s" references for product with ID %2$s', $domain, json_encode($prodId));
164
+			throw new \Aimeos\Controller\Frontend\Basket\Exception($msg);
165 165
 		}
166 166
 	}
167 167
 
@@ -171,41 +171,41 @@  discard block
 block discarded – undo
171 171
 	 *
172 172
 	 * @param string $type Basket type
173 173
 	 */
174
-	protected function checkLocale( $type )
174
+	protected function checkLocale($type)
175 175
 	{
176 176
 		$errors = [];
177 177
 		$context = $this->getContext();
178 178
 		$session = $context->getSession();
179 179
 		$locale = $this->get()->getLocale();
180 180
 
181
-		$localeStr = $session->get( 'aimeos/basket/locale' );
181
+		$localeStr = $session->get('aimeos/basket/locale');
182 182
 		$localeKey = $locale->getSite()->getCode() . '|' . $locale->getLanguageId() . '|' . $locale->getCurrencyId();
183 183
 
184
-		if( $localeStr !== null && $localeStr !== $localeKey )
184
+		if ($localeStr !== null && $localeStr !== $localeKey)
185 185
 		{
186
-			$locParts = explode( '|', $localeStr );
187
-			$locSite = ( isset( $locParts[0] ) ? $locParts[0] : '' );
188
-			$locLanguage = ( isset( $locParts[1] ) ? $locParts[1] : '' );
189
-			$locCurrency = ( isset( $locParts[2] ) ? $locParts[2] : '' );
186
+			$locParts = explode('|', $localeStr);
187
+			$locSite = (isset($locParts[0]) ? $locParts[0] : '');
188
+			$locLanguage = (isset($locParts[1]) ? $locParts[1] : '');
189
+			$locCurrency = (isset($locParts[2]) ? $locParts[2] : '');
190 190
 
191
-			$localeManager = \Aimeos\MShop\Factory::createManager( $context, 'locale' );
192
-			$locale = $localeManager->bootstrap( $locSite, $locLanguage, $locCurrency, false );
191
+			$localeManager = \Aimeos\MShop\Factory::createManager($context, 'locale');
192
+			$locale = $localeManager->bootstrap($locSite, $locLanguage, $locCurrency, false);
193 193
 
194 194
 			$context = clone $context;
195
-			$context->setLocale( $locale );
195
+			$context->setLocale($locale);
196 196
 
197
-			$manager = \Aimeos\MShop\Order\Manager\Factory::createManager( $context )->getSubManager( 'base' );
198
-			$basket = $manager->getSession( $type );
197
+			$manager = \Aimeos\MShop\Order\Manager\Factory::createManager($context)->getSubManager('base');
198
+			$basket = $manager->getSession($type);
199 199
 
200
-			$this->copyAddresses( $basket, $errors, $localeKey );
201
-			$this->copyServices( $basket, $errors );
202
-			$this->copyProducts( $basket, $errors, $localeKey );
203
-			$this->copyCoupons( $basket, $errors, $localeKey );
200
+			$this->copyAddresses($basket, $errors, $localeKey);
201
+			$this->copyServices($basket, $errors);
202
+			$this->copyProducts($basket, $errors, $localeKey);
203
+			$this->copyCoupons($basket, $errors, $localeKey);
204 204
 
205
-			$manager->setSession( $basket, $type );
205
+			$manager->setSession($basket, $type);
206 206
 		}
207 207
 
208
-		$session->set( 'aimeos/basket/locale', $localeKey );
208
+		$session->set('aimeos/basket/locale', $localeKey);
209 209
 	}
210 210
 
211 211
 
@@ -217,20 +217,20 @@  discard block
 block discarded – undo
217 217
 	 * @param string $localeKey Unique identifier of the site, language and currency
218 218
 	 * @return array Associative list of errors occured
219 219
 	 */
220
-	protected function copyAddresses( \Aimeos\MShop\Order\Item\Base\Iface $basket, array $errors, $localeKey )
220
+	protected function copyAddresses(\Aimeos\MShop\Order\Item\Base\Iface $basket, array $errors, $localeKey)
221 221
 	{
222
-		foreach( $basket->getAddresses() as $type => $item )
222
+		foreach ($basket->getAddresses() as $type => $item)
223 223
 		{
224 224
 			try
225 225
 			{
226
-				$this->setAddress( $type, $item->toArray() );
227
-				$basket->deleteAddress( $type );
226
+				$this->setAddress($type, $item->toArray());
227
+				$basket->deleteAddress($type);
228 228
 			}
229
-			catch( \Exception $e )
229
+			catch (\Exception $e)
230 230
 			{
231 231
 				$logger = $this->getContext()->getLogger();
232 232
 				$str = 'Error migrating address with type "%1$s" in basket to locale "%2$s": %3$s';
233
-				$logger->log( sprintf( $str, $type, $localeKey, $e->getMessage() ), \Aimeos\MW\Logger\Base::INFO );
233
+				$logger->log(sprintf($str, $type, $localeKey, $e->getMessage()), \Aimeos\MW\Logger\Base::INFO);
234 234
 				$errors['address'][$type] = $e->getMessage();
235 235
 			}
236 236
 		}
@@ -247,20 +247,20 @@  discard block
 block discarded – undo
247 247
 	 * @param string $localeKey Unique identifier of the site, language and currency
248 248
 	 * @return array Associative list of errors occured
249 249
 	 */
250
-	protected function copyCoupons( \Aimeos\MShop\Order\Item\Base\Iface $basket, array $errors, $localeKey )
250
+	protected function copyCoupons(\Aimeos\MShop\Order\Item\Base\Iface $basket, array $errors, $localeKey)
251 251
 	{
252
-		foreach( $basket->getCoupons() as $code => $list )
252
+		foreach ($basket->getCoupons() as $code => $list)
253 253
 		{
254 254
 			try
255 255
 			{
256
-				$this->addCoupon( $code );
257
-				$basket->deleteCoupon( $code, true );
256
+				$this->addCoupon($code);
257
+				$basket->deleteCoupon($code, true);
258 258
 			}
259
-			catch( \Exception $e )
259
+			catch (\Exception $e)
260 260
 			{
261 261
 				$logger = $this->getContext()->getLogger();
262 262
 				$str = 'Error migrating coupon with code "%1$s" in basket to locale "%2$s": %3$s';
263
-				$logger->log( sprintf( $str, $code, $localeKey, $e->getMessage() ), \Aimeos\MW\Logger\Base::INFO );
263
+				$logger->log(sprintf($str, $code, $localeKey, $e->getMessage()), \Aimeos\MW\Logger\Base::INFO);
264 264
 				$errors['coupon'][$code] = $e->getMessage();
265 265
 			}
266 266
 		}
@@ -277,11 +277,11 @@  discard block
 block discarded – undo
277 277
 	 * @param string $localeKey Unique identifier of the site, language and currency
278 278
 	 * @return array Associative list of errors occured
279 279
 	 */
280
-	protected function copyProducts( \Aimeos\MShop\Order\Item\Base\Iface $basket, array $errors, $localeKey )
280
+	protected function copyProducts(\Aimeos\MShop\Order\Item\Base\Iface $basket, array $errors, $localeKey)
281 281
 	{
282
-		foreach( $basket->getProducts() as $pos => $product )
282
+		foreach ($basket->getProducts() as $pos => $product)
283 283
 		{
284
-			if( $product->getFlags() & \Aimeos\MShop\Order\Item\Base\Product\Base::FLAG_IMMUTABLE ) {
284
+			if ($product->getFlags() & \Aimeos\MShop\Order\Item\Base\Product\Base::FLAG_IMMUTABLE) {
285 285
 				continue;
286 286
 			}
287 287
 
@@ -289,7 +289,7 @@  discard block
 block discarded – undo
289 289
 			{
290 290
 				$attrIds = [];
291 291
 
292
-				foreach( $product->getAttributes() as $attrItem ) {
292
+				foreach ($product->getAttributes() as $attrItem) {
293 293
 					$attrIds[$attrItem->getType()][] = $attrItem->getAttributeId();
294 294
 				}
295 295
 
@@ -297,23 +297,23 @@  discard block
 block discarded – undo
297 297
 					$product->getProductId(),
298 298
 					$product->getQuantity(),
299 299
 					[],
300
-					$this->getValue( $attrIds, 'variant', [] ),
301
-					$this->getValue( $attrIds, 'config', [] ),
302
-					$this->getValue( $attrIds, 'hidden', [] ),
303
-					$this->getValue( $attrIds, 'custom', [] ),
300
+					$this->getValue($attrIds, 'variant', []),
301
+					$this->getValue($attrIds, 'config', []),
302
+					$this->getValue($attrIds, 'hidden', []),
303
+					$this->getValue($attrIds, 'custom', []),
304 304
 					$product->getStockType()
305 305
 				);
306 306
 
307
-				$basket->deleteProduct( $pos );
307
+				$basket->deleteProduct($pos);
308 308
 			}
309
-			catch( \Exception $e )
309
+			catch (\Exception $e)
310 310
 			{
311 311
 				$code = $product->getProductCode();
312 312
 				$logger = $this->getContext()->getLogger();
313 313
 				$errors['product'][$pos] = $e->getMessage();
314 314
 
315 315
 				$str = 'Error migrating product with code "%1$s" in basket to locale "%2$s": %3$s';
316
-				$logger->log( sprintf( $str, $code, $localeKey, $e->getMessage() ), \Aimeos\MW\Logger\Base::INFO );
316
+				$logger->log(sprintf($str, $code, $localeKey, $e->getMessage()), \Aimeos\MW\Logger\Base::INFO);
317 317
 			}
318 318
 		}
319 319
 
@@ -328,22 +328,22 @@  discard block
 block discarded – undo
328 328
 	 * @param array $errors Associative list of previous errors
329 329
 	 * @return array Associative list of errors occured
330 330
 	 */
331
-	protected function copyServices( \Aimeos\MShop\Order\Item\Base\Iface $basket, array $errors )
331
+	protected function copyServices(\Aimeos\MShop\Order\Item\Base\Iface $basket, array $errors)
332 332
 	{
333
-		foreach( $basket->getServices() as $type => $item )
333
+		foreach ($basket->getServices() as $type => $item)
334 334
 		{
335 335
 			try
336 336
 			{
337 337
 				$attributes = [];
338 338
 
339
-				foreach( $item->getAttributes() as $attrItem ) {
339
+				foreach ($item->getAttributes() as $attrItem) {
340 340
 					$attributes[$attrItem->getCode()] = $attrItem->getValue();
341 341
 				}
342 342
 
343
-				$this->setService( $type, $item->getServiceId(), $attributes );
344
-				$basket->deleteService( $type );
343
+				$this->setService($type, $item->getServiceId(), $attributes);
344
+				$basket->deleteService($type);
345 345
 			}
346
-			catch( \Exception $e ) { ; } // Don't notify the user as appropriate services can be added automatically
346
+			catch (\Exception $e) {; } // Don't notify the user as appropriate services can be added automatically
347 347
 		}
348 348
 
349 349
 		return $errors;
@@ -362,36 +362,36 @@  discard block
 block discarded – undo
362 362
 	 * @return array List of items implementing \Aimeos\MShop\Order\Item\Product\Attribute\Iface
363 363
 	 * @deprecated Use getOrderProductAttributes(), checkReferences() and calcPrice() instead
364 364
 	 */
365
-	protected function createOrderProductAttributes( \Aimeos\MShop\Price\Item\Iface $price, $prodid, $quantity,
366
-			array $attributeIds, $type, array $attributeValues = [] )
365
+	protected function createOrderProductAttributes(\Aimeos\MShop\Price\Item\Iface $price, $prodid, $quantity,
366
+			array $attributeIds, $type, array $attributeValues = [])
367 367
 	{
368
-		if( empty( $attributeIds ) ) {
368
+		if (empty($attributeIds)) {
369 369
 			return [];
370 370
 		}
371 371
 
372
-		$attrTypeId = $this->getProductListTypeItem( 'attribute', $type )->getId();
373
-		$this->checkReferences( $prodid, 'attribute', $attrTypeId, $attributeIds );
372
+		$attrTypeId = $this->getProductListTypeItem('attribute', $type)->getId();
373
+		$this->checkReferences($prodid, 'attribute', $attrTypeId, $attributeIds);
374 374
 
375 375
 		$list = [];
376 376
 		$context = $this->getContext();
377 377
 
378
-		$priceManager = \Aimeos\MShop\Factory::createManager( $context, 'price' );
379
-		$orderProductAttributeManager = \Aimeos\MShop\Factory::createManager( $context, 'order/base/product/attribute' );
378
+		$priceManager = \Aimeos\MShop\Factory::createManager($context, 'price');
379
+		$orderProductAttributeManager = \Aimeos\MShop\Factory::createManager($context, 'order/base/product/attribute');
380 380
 
381
-		foreach( $this->getAttributes( $attributeIds ) as $id => $attrItem )
381
+		foreach ($this->getAttributes($attributeIds) as $id => $attrItem)
382 382
 		{
383
-			$prices = $attrItem->getRefItems( 'price', 'default', 'default' );
383
+			$prices = $attrItem->getRefItems('price', 'default', 'default');
384 384
 
385
-			if( !empty( $prices ) ) {
386
-				$price->addItem( $priceManager->getLowestPrice( $prices, $quantity ) );
385
+			if (!empty($prices)) {
386
+				$price->addItem($priceManager->getLowestPrice($prices, $quantity));
387 387
 			}
388 388
 
389 389
 			$item = $orderProductAttributeManager->createItem();
390
-			$item->copyFrom( $attrItem );
391
-			$item->setType( $type );
390
+			$item->copyFrom($attrItem);
391
+			$item->setType($type);
392 392
 
393
-			if( isset( $attributeValues[$id] ) ) {
394
-				$item->setValue( $attributeValues[$id] );
393
+			if (isset($attributeValues[$id])) {
394
+				$item->setValue($attributeValues[$id]);
395 395
 			}
396 396
 
397 397
 			$list[] = $item;
@@ -409,31 +409,31 @@  discard block
 block discarded – undo
409 409
 	 * @return array List of items implementing \Aimeos\MShop\Attribute\Item\Iface
410 410
 	 * @throws \Aimeos\Controller\Frontend\Basket\Exception If the actual attribute number doesn't match the expected one
411 411
 	 */
412
-	protected function getAttributes( array $attributeIds, array $domains = array( 'price', 'text' ) )
412
+	protected function getAttributes(array $attributeIds, array $domains = array('price', 'text'))
413 413
 	{
414
-		if( empty( $attributeIds ) ) {
414
+		if (empty($attributeIds)) {
415 415
 			return [];
416 416
 		}
417 417
 
418
-		$attributeManager = \Aimeos\MShop\Factory::createManager( $this->getContext(), 'attribute' );
418
+		$attributeManager = \Aimeos\MShop\Factory::createManager($this->getContext(), 'attribute');
419 419
 
420
-		$search = $attributeManager->createSearch( true );
420
+		$search = $attributeManager->createSearch(true);
421 421
 		$expr = array(
422
-			$search->compare( '==', 'attribute.id', $attributeIds ),
422
+			$search->compare('==', 'attribute.id', $attributeIds),
423 423
 			$search->getConditions(),
424 424
 		);
425
-		$search->setConditions( $search->combine( '&&', $expr ) );
426
-		$search->setSlice( 0, 0x7fffffff );
425
+		$search->setConditions($search->combine('&&', $expr));
426
+		$search->setSlice(0, 0x7fffffff);
427 427
 
428
-		$attrItems = $attributeManager->searchItems( $search, $domains );
428
+		$attrItems = $attributeManager->searchItems($search, $domains);
429 429
 
430
-		if( count( $attrItems ) !== count( $attributeIds ) )
430
+		if (count($attrItems) !== count($attributeIds))
431 431
 		{
432
-			$expected = implode( ',', $attributeIds );
433
-			$actual = implode( ',', array_keys( $attrItems ) );
434
-			$msg = sprintf( 'Available attribute IDs "%1$s" do not match the given attribute IDs "%2$s"', $actual, $expected );
432
+			$expected = implode(',', $attributeIds);
433
+			$actual = implode(',', array_keys($attrItems));
434
+			$msg = sprintf('Available attribute IDs "%1$s" do not match the given attribute IDs "%2$s"', $actual, $expected);
435 435
 
436
-			throw new \Aimeos\Controller\Frontend\Basket\Exception( $msg );
436
+			throw new \Aimeos\Controller\Frontend\Basket\Exception($msg);
437 437
 		}
438 438
 
439 439
 		return $attrItems;
@@ -446,31 +446,31 @@  discard block
 block discarded – undo
446 446
 	 * @param \Aimeos\MShop\Order\Item\Base\Product\Attribute\Item[] $orderAttributes List of order product attribute items
447 447
 	 * @return \Aimeos\MShop\Attribute\Item\Iface[] Associative list of attribute IDs as key and attribute items as values
448 448
 	 */
449
-	protected function getAttributeItems( array $orderAttributes )
449
+	protected function getAttributeItems(array $orderAttributes)
450 450
 	{
451
-		if( empty( $orderAttributes ) ) {
451
+		if (empty($orderAttributes)) {
452 452
 			return [];
453 453
 		}
454 454
 
455
-		$attributeManager = \Aimeos\MShop\Factory::createManager( $this->getContext(), 'attribute' );
456
-		$search = $attributeManager->createSearch( true );
455
+		$attributeManager = \Aimeos\MShop\Factory::createManager($this->getContext(), 'attribute');
456
+		$search = $attributeManager->createSearch(true);
457 457
 		$expr = [];
458 458
 
459
-		foreach( $orderAttributes as $item )
459
+		foreach ($orderAttributes as $item)
460 460
 		{
461 461
 			$tmp = array(
462
-				$search->compare( '==', 'attribute.domain', 'product' ),
463
-				$search->compare( '==', 'attribute.code', $item->getValue() ),
464
-				$search->compare( '==', 'attribute.type.domain', 'product' ),
465
-				$search->compare( '==', 'attribute.type.code', $item->getCode() ),
466
-				$search->compare( '>', 'attribute.type.status', 0 ),
462
+				$search->compare('==', 'attribute.domain', 'product'),
463
+				$search->compare('==', 'attribute.code', $item->getValue()),
464
+				$search->compare('==', 'attribute.type.domain', 'product'),
465
+				$search->compare('==', 'attribute.type.code', $item->getCode()),
466
+				$search->compare('>', 'attribute.type.status', 0),
467 467
 				$search->getConditions(),
468 468
 			);
469
-			$expr[] = $search->combine( '&&', $tmp );
469
+			$expr[] = $search->combine('&&', $tmp);
470 470
 		}
471 471
 
472
-		$search->setConditions( $search->combine( '||', $expr ) );
473
-		return $attributeManager->searchItems( $search, array( 'price' ) );
472
+		$search->setConditions($search->combine('||', $expr));
473
+		return $attributeManager->searchItems($search, array('price'));
474 474
 	}
475 475
 
476 476
 
@@ -485,23 +485,23 @@  discard block
 block discarded – undo
485 485
 	 * @throws \Aimeos\Controller\Frontend\Basket\Exception
486 486
 	 * @deprecated Use getItem() or findItem() instead
487 487
 	 */
488
-	protected function getDomainItem( $domain, $key, $value, array $ref )
488
+	protected function getDomainItem($domain, $key, $value, array $ref)
489 489
 	{
490
-		$manager = \Aimeos\MShop\Factory::createManager( $this->getContext(), $domain );
490
+		$manager = \Aimeos\MShop\Factory::createManager($this->getContext(), $domain);
491 491
 
492
-		$search = $manager->createSearch( true );
492
+		$search = $manager->createSearch(true);
493 493
 		$expr = array(
494
-			$search->compare( '==', $key, $value ),
494
+			$search->compare('==', $key, $value),
495 495
 			$search->getConditions(),
496 496
 		);
497
-		$search->setConditions( $search->combine( '&&', $expr ) );
497
+		$search->setConditions($search->combine('&&', $expr));
498 498
 
499
-		$result = $manager->searchItems( $search, $ref );
499
+		$result = $manager->searchItems($search, $ref);
500 500
 
501
-		if( ( $item = reset( $result ) ) === false )
501
+		if (($item = reset($result)) === false)
502 502
 		{
503
-			$msg = sprintf( 'No item for "%1$s" (%2$s) found', $value, $key );
504
-			throw new \Aimeos\Controller\Frontend\Basket\Exception( $msg );
503
+			$msg = sprintf('No item for "%1$s" (%2$s) found', $value, $key);
504
+			throw new \Aimeos\Controller\Frontend\Basket\Exception($msg);
505 505
 		}
506 506
 
507 507
 		return $item;
@@ -516,23 +516,23 @@  discard block
 block discarded – undo
516 516
 	 * @param array $attributeValues Associative list of attribute IDs as keys and their codes as values
517 517
 	 * @return array List of items implementing \Aimeos\MShop\Order\Item\Product\Attribute\Iface
518 518
 	 */
519
-	protected function getOrderProductAttributes( $type, array $attributeIds, array $attributeValues = [] )
519
+	protected function getOrderProductAttributes($type, array $attributeIds, array $attributeValues = [])
520 520
 	{
521
-		if( empty( $attributeIds ) ) {
521
+		if (empty($attributeIds)) {
522 522
 			return [];
523 523
 		}
524 524
 
525 525
 		$list = [];
526
-		$manager = \Aimeos\MShop\Factory::createManager( $this->getContext(), 'order/base/product/attribute' );
526
+		$manager = \Aimeos\MShop\Factory::createManager($this->getContext(), 'order/base/product/attribute');
527 527
 
528
-		foreach( $this->getAttributes( $attributeIds ) as $id => $attrItem )
528
+		foreach ($this->getAttributes($attributeIds) as $id => $attrItem)
529 529
 		{
530 530
 			$item = $manager->createItem();
531
-			$item->copyFrom( $attrItem );
532
-			$item->setType( $type );
531
+			$item->copyFrom($attrItem);
532
+			$item->setType($type);
533 533
 
534
-			if( isset( $attributeValues[$id] ) ) {
535
-				$item->setValue( $attributeValues[$id] );
534
+			if (isset($attributeValues[$id])) {
535
+				$item->setValue($attributeValues[$id]);
536 536
 			}
537 537
 
538 538
 			$list[] = $item;
@@ -549,21 +549,21 @@  discard block
 block discarded – undo
549 549
 	 * @param string $code Code of the list type
550 550
 	 * @return \Aimeos\MShop\Common\Item\Type\Iface List type item
551 551
 	 */
552
-	protected function getProductListTypeItem( $domain, $code )
552
+	protected function getProductListTypeItem($domain, $code)
553 553
 	{
554
-		if( empty( $this->listTypeItems ) )
554
+		if (empty($this->listTypeItems))
555 555
 		{
556
-			$manager = \Aimeos\MShop\Factory::createManager( $this->getContext(), 'product/lists/type' );
556
+			$manager = \Aimeos\MShop\Factory::createManager($this->getContext(), 'product/lists/type');
557 557
 
558
-			foreach( $manager->searchItems( $manager->createSearch( true ) ) as $item ) {
559
-				$this->listTypeItems[ $item->getDomain() ][ $item->getCode() ] = $item;
558
+			foreach ($manager->searchItems($manager->createSearch(true)) as $item) {
559
+				$this->listTypeItems[$item->getDomain()][$item->getCode()] = $item;
560 560
 			}
561 561
 		}
562 562
 
563
-		if( !isset( $this->listTypeItems[$domain][$code] ) )
563
+		if (!isset($this->listTypeItems[$domain][$code]))
564 564
 		{
565
-			$msg = sprintf( 'List type for domain "%1$s" and code "%2$s" not found', $domain, $code );
566
-			throw new \Aimeos\Controller\Frontend\Basket\Exception( $msg );
565
+			$msg = sprintf('List type for domain "%1$s" and code "%2$s" not found', $domain, $code);
566
+			throw new \Aimeos\Controller\Frontend\Basket\Exception($msg);
567 567
 		}
568 568
 
569 569
 		return $this->listTypeItems[$domain][$code];
@@ -578,43 +578,43 @@  discard block
 block discarded – undo
578 578
 	 * @param array $domains Names of the domain items that should be fetched too
579 579
 	 * @return array List of products matching the given attributes
580 580
 	 */
581
-	protected function getProductVariants( \Aimeos\MShop\Product\Item\Iface $productItem, array $variantAttributeIds,
582
-			array $domains = array( 'attribute', 'media', 'price', 'text' ) )
581
+	protected function getProductVariants(\Aimeos\MShop\Product\Item\Iface $productItem, array $variantAttributeIds,
582
+			array $domains = array('attribute', 'media', 'price', 'text'))
583 583
 	{
584 584
 		$subProductIds = [];
585
-		foreach( $productItem->getRefItems( 'product', 'default', 'default' ) as $item ) {
585
+		foreach ($productItem->getRefItems('product', 'default', 'default') as $item) {
586 586
 			$subProductIds[] = $item->getId();
587 587
 		}
588 588
 
589
-		if( count( $subProductIds ) === 0 ) {
589
+		if (count($subProductIds) === 0) {
590 590
 			return [];
591 591
 		}
592 592
 
593
-		$productManager = \Aimeos\MShop\Factory::createManager( $this->getContext(), 'product' );
594
-		$search = $productManager->createSearch( true );
593
+		$productManager = \Aimeos\MShop\Factory::createManager($this->getContext(), 'product');
594
+		$search = $productManager->createSearch(true);
595 595
 
596 596
 		$expr = array(
597
-			$search->compare( '==', 'product.id', $subProductIds ),
597
+			$search->compare('==', 'product.id', $subProductIds),
598 598
 			$search->getConditions(),
599 599
 		);
600 600
 
601
-		if( count( $variantAttributeIds ) > 0 )
601
+		if (count($variantAttributeIds) > 0)
602 602
 		{
603
-			foreach( $variantAttributeIds as $key => $id ) {
603
+			foreach ($variantAttributeIds as $key => $id) {
604 604
 				$variantAttributeIds[$key] = (string) $id;
605 605
 			}
606 606
 
607
-			$listTypeItem = $this->getProductListTypeItem( 'attribute', 'variant' );
607
+			$listTypeItem = $this->getProductListTypeItem('attribute', 'variant');
608 608
 
609
-			$param = array( 'attribute', $listTypeItem->getId(), $variantAttributeIds );
610
-			$cmpfunc = $search->createFunction( 'product.contains', $param );
609
+			$param = array('attribute', $listTypeItem->getId(), $variantAttributeIds);
610
+			$cmpfunc = $search->createFunction('product.contains', $param);
611 611
 
612
-			$expr[] = $search->compare( '==', $cmpfunc, count( $variantAttributeIds ) );
612
+			$expr[] = $search->compare('==', $cmpfunc, count($variantAttributeIds));
613 613
 		}
614 614
 
615
-		$search->setConditions( $search->combine( '&&', $expr ) );
615
+		$search->setConditions($search->combine('&&', $expr));
616 616
 
617
-		return $productManager->searchItems( $search, $domains );
617
+		return $productManager->searchItems($search, $domains);
618 618
 	}
619 619
 
620 620
 
@@ -626,9 +626,9 @@  discard block
 block discarded – undo
626 626
 	 * @param mixed $default Default value if no value is available for the given name
627 627
 	 * @return mixed Value from the array or default value
628 628
 	 */
629
-	protected function getValue( array $values, $name, $default = null )
629
+	protected function getValue(array $values, $name, $default = null)
630 630
 	{
631
-		if( isset( $values[$name] ) ) {
631
+		if (isset($values[$name])) {
632 632
 			return $values[$name];
633 633
 		}
634 634
 
Please login to merge, or discard this patch.
controller/frontend/src/Controller/Frontend/Basket/Standard.php 1 patch
Spacing   +126 added lines, -126 removed lines patch added patch discarded remove patch
@@ -33,11 +33,11 @@  discard block
 block discarded – undo
33 33
 	 * @param \Aimeos\MShop\Context\Item\Iface $context Object storing the required instances for manaing databases
34 34
 	 *  connections, logger, session, etc.
35 35
 	 */
36
-	public function __construct( \Aimeos\MShop\Context\Item\Iface $context )
36
+	public function __construct(\Aimeos\MShop\Context\Item\Iface $context)
37 37
 	{
38
-		parent::__construct( $context );
38
+		parent::__construct($context);
39 39
 
40
-		$this->domainManager = \Aimeos\MShop\Factory::createManager( $context, 'order/base' );
40
+		$this->domainManager = \Aimeos\MShop\Factory::createManager($context, 'order/base');
41 41
 	}
42 42
 
43 43
 
@@ -49,7 +49,7 @@  discard block
 block discarded – undo
49 49
 	public function clear()
50 50
 	{
51 51
 		$this->baskets[$this->type] = $this->domainManager->createItem();
52
-		$this->domainManager->setSession( $this->baskets[$this->type], $this->type );
52
+		$this->domainManager->setSession($this->baskets[$this->type], $this->type);
53 53
 
54 54
 		return $this;
55 55
 	}
@@ -62,10 +62,10 @@  discard block
 block discarded – undo
62 62
 	 */
63 63
 	public function get()
64 64
 	{
65
-		if( !isset( $this->baskets[$this->type] ) )
65
+		if (!isset($this->baskets[$this->type]))
66 66
 		{
67
-			$this->baskets[$this->type] = $this->domainManager->getSession( $this->type );
68
-			$this->checkLocale( $this->type );
67
+			$this->baskets[$this->type] = $this->domainManager->getSession($this->type);
68
+			$this->checkLocale($this->type);
69 69
 		}
70 70
 
71 71
 		return $this->baskets[$this->type];
@@ -79,8 +79,8 @@  discard block
 block discarded – undo
79 79
 	 */
80 80
 	public function save()
81 81
 	{
82
-		if( isset( $this->baskets[$this->type] ) && $this->baskets[$this->type]->isModified() ) {
83
-			$this->domainManager->setSession( $this->baskets[$this->type], $this->type );
82
+		if (isset($this->baskets[$this->type]) && $this->baskets[$this->type]->isModified()) {
83
+			$this->domainManager->setSession($this->baskets[$this->type], $this->type);
84 84
 		}
85 85
 
86 86
 		return $this;
@@ -93,7 +93,7 @@  discard block
 block discarded – undo
93 93
 	 * @param string $type Basket type
94 94
 	 * @return \Aimeos\Controller\Frontend\Basket\Iface Basket frontend object
95 95
 	 */
96
-	public function setType( $type )
96
+	public function setType($type)
97 97
 	{
98 98
 		$this->type = $type;
99 99
 		return $this;
@@ -126,7 +126,7 @@  discard block
 block discarded – undo
126 126
 		 * @category Developer
127 127
 		 * @see controller/frontend/basket/limit-seconds
128 128
 		 */
129
-		$count = $config->get( 'controller/frontend/basket/limit-count', 5 );
129
+		$count = $config->get('controller/frontend/basket/limit-count', 5);
130 130
 
131 131
 		/** controller/frontend/basket/limit-seconds
132 132
 		 * Order limitation time frame in seconds
@@ -144,27 +144,27 @@  discard block
 block discarded – undo
144 144
 		 * @category Developer
145 145
 		 * @see controller/frontend/basket/limit-count
146 146
 		 */
147
-		$seconds = $config->get( 'controller/frontend/basket/limit-seconds', 300 );
147
+		$seconds = $config->get('controller/frontend/basket/limit-seconds', 300);
148 148
 
149 149
 		$search = $this->domainManager->createSearch();
150 150
 		$expr = [
151
-			$search->compare( '==', 'order.base.editor', $this->getContext()->getEditor() ),
152
-			$search->compare( '>=', 'order.base.ctime', date( 'Y-m-d H:i:s', time() - $seconds ) ),
151
+			$search->compare('==', 'order.base.editor', $this->getContext()->getEditor()),
152
+			$search->compare('>=', 'order.base.ctime', date('Y-m-d H:i:s', time() - $seconds)),
153 153
 		];
154
-		$search->setConditions( $search->combine( '&&', $expr ) );
155
-		$search->setSlice( 0, 0 );
154
+		$search->setConditions($search->combine('&&', $expr));
155
+		$search->setSlice(0, 0);
156 156
 
157
-		$this->domainManager->searchItems( $search, [], $total );
157
+		$this->domainManager->searchItems($search, [], $total);
158 158
 
159
-		if( $total > $count ) {
160
-			throw new \Aimeos\Controller\Frontend\Basket\Exception( sprintf( 'Temporary order limit reached' ) );
159
+		if ($total > $count) {
160
+			throw new \Aimeos\Controller\Frontend\Basket\Exception(sprintf('Temporary order limit reached'));
161 161
 		}
162 162
 
163 163
 
164 164
 		$basket = $this->get()->finish();
165 165
 
166 166
 		$this->domainManager->begin();
167
-		$this->domainManager->store( $basket );
167
+		$this->domainManager->store($basket);
168 168
 		$this->domainManager->commit();
169 169
 
170 170
 		return $basket;
@@ -179,9 +179,9 @@  discard block
 block discarded – undo
179 179
 	 * @param boolean $default True to add default criteria (user logged in), false if not
180 180
 	 * @return \Aimeos\MShop\Order\Item\Base\Iface Order base object including the given parts
181 181
 	 */
182
-	public function load( $id, $parts = \Aimeos\MShop\Order\Manager\Base\Base::PARTS_ALL, $default = true )
182
+	public function load($id, $parts = \Aimeos\MShop\Order\Manager\Base\Base::PARTS_ALL, $default = true)
183 183
 	{
184
-		return $this->domainManager->load( $id, $parts, false, $default );
184
+		return $this->domainManager->load($id, $parts, false, $default);
185 185
 	}
186 186
 
187 187
 
@@ -201,34 +201,34 @@  discard block
 block discarded – undo
201 201
 	 * @param string $stocktype Unique code of the stock type to deliver the products from
202 202
 	 * @throws \Aimeos\Controller\Frontend\Basket\Exception If the product isn't available
203 203
 	 */
204
-	public function addProduct( $prodid, $quantity = 1, array $options = [], array $variantAttributeIds = [],
204
+	public function addProduct($prodid, $quantity = 1, array $options = [], array $variantAttributeIds = [],
205 205
 		array $configAttributeIds = [], array $hiddenAttributeIds = [], array $customAttributeValues = [],
206
-		$stocktype = 'default' )
206
+		$stocktype = 'default')
207 207
 	{
208 208
 		$attributeMap = [
209
-			'custom' => array_keys( $customAttributeValues ),
209
+			'custom' => array_keys($customAttributeValues),
210 210
 			'config' => $configAttributeIds,
211 211
 			'hidden' => $hiddenAttributeIds,
212 212
 		];
213
-		$this->checkListRef( $prodid, 'attribute', $attributeMap );
213
+		$this->checkListRef($prodid, 'attribute', $attributeMap);
214 214
 
215 215
 
216 216
 		$context = $this->getContext();
217
-		$productManager = \Aimeos\MShop\Factory::createManager( $context, 'product' );
218
-		$productItem = $productManager->getItem( $prodid, array( 'media', 'supplier', 'price', 'product', 'text' ), true );
219
-		$prices = $productItem->getRefItems( 'price', 'default', 'default' );
217
+		$productManager = \Aimeos\MShop\Factory::createManager($context, 'product');
218
+		$productItem = $productManager->getItem($prodid, array('media', 'supplier', 'price', 'product', 'text'), true);
219
+		$prices = $productItem->getRefItems('price', 'default', 'default');
220 220
 
221
-		$orderBaseProductItem = \Aimeos\MShop\Factory::createManager( $context, 'order/base/product' )->createItem();
222
-		$orderBaseProductItem->copyFrom( $productItem )->setQuantity( $quantity )->setStockType( $stocktype );
221
+		$orderBaseProductItem = \Aimeos\MShop\Factory::createManager($context, 'order/base/product')->createItem();
222
+		$orderBaseProductItem->copyFrom($productItem)->setQuantity($quantity)->setStockType($stocktype);
223 223
 
224
-		$attr = $this->getOrderProductAttributes( 'custom', array_keys( $customAttributeValues ), $customAttributeValues );
225
-		$attr = array_merge( $attr, $this->getOrderProductAttributes( 'config', $configAttributeIds ) );
226
-		$attr = array_merge( $attr, $this->getOrderProductAttributes( 'hidden', $hiddenAttributeIds ) );
224
+		$attr = $this->getOrderProductAttributes('custom', array_keys($customAttributeValues), $customAttributeValues);
225
+		$attr = array_merge($attr, $this->getOrderProductAttributes('config', $configAttributeIds));
226
+		$attr = array_merge($attr, $this->getOrderProductAttributes('hidden', $hiddenAttributeIds));
227 227
 
228
-		$orderBaseProductItem->setAttributes( $attr );
229
-		$orderBaseProductItem->setPrice( $this->calcPrice( $orderBaseProductItem, $prices, $quantity ) );
228
+		$orderBaseProductItem->setAttributes($attr);
229
+		$orderBaseProductItem->setPrice($this->calcPrice($orderBaseProductItem, $prices, $quantity));
230 230
 
231
-		$this->get()->addProduct( $orderBaseProductItem );
231
+		$this->get()->addProduct($orderBaseProductItem);
232 232
 		$this->save();
233 233
 	}
234 234
 
@@ -238,17 +238,17 @@  discard block
 block discarded – undo
238 238
 	 *
239 239
 	 * @param integer $position Position number (key) of the order product item
240 240
 	 */
241
-	public function deleteProduct( $position )
241
+	public function deleteProduct($position)
242 242
 	{
243
-		$product = $this->get()->getProduct( $position );
243
+		$product = $this->get()->getProduct($position);
244 244
 
245
-		if( $product->getFlags() === \Aimeos\MShop\Order\Item\Base\Product\Base::FLAG_IMMUTABLE )
245
+		if ($product->getFlags() === \Aimeos\MShop\Order\Item\Base\Product\Base::FLAG_IMMUTABLE)
246 246
 		{
247
-			$msg = sprintf( 'Basket item at position "%1$d" cannot be deleted manually', $position );
248
-			throw new \Aimeos\Controller\Frontend\Basket\Exception( $msg );
247
+			$msg = sprintf('Basket item at position "%1$d" cannot be deleted manually', $position);
248
+			throw new \Aimeos\Controller\Frontend\Basket\Exception($msg);
249 249
 		}
250 250
 
251
-		$this->get()->deleteProduct( $position );
251
+		$this->get()->deleteProduct($position);
252 252
 		$this->save();
253 253
 	}
254 254
 
@@ -262,33 +262,33 @@  discard block
 block discarded – undo
262 262
 	 * 	The 'stock'=>false option allows adding products without being in stock.
263 263
 	 * @param string[] $configAttributeCodes Codes of the product config attributes that should be REMOVED
264 264
 	 */
265
-	public function editProduct( $position, $quantity, array $options = [],
266
-		array $configAttributeCodes = [] )
265
+	public function editProduct($position, $quantity, array $options = [],
266
+		array $configAttributeCodes = [])
267 267
 	{
268
-		$product = $this->get()->getProduct( $position );
268
+		$product = $this->get()->getProduct($position);
269 269
 
270
-		if( $product->getFlags() & \Aimeos\MShop\Order\Item\Base\Product\Base::FLAG_IMMUTABLE )
270
+		if ($product->getFlags() & \Aimeos\MShop\Order\Item\Base\Product\Base::FLAG_IMMUTABLE)
271 271
 		{
272
-			$msg = sprintf( 'Basket item at position "%1$d" cannot be changed', $position );
273
-			throw new \Aimeos\Controller\Frontend\Basket\Exception( $msg );
272
+			$msg = sprintf('Basket item at position "%1$d" cannot be changed', $position);
273
+			throw new \Aimeos\Controller\Frontend\Basket\Exception($msg);
274 274
 		}
275 275
 
276
-		$product->setQuantity( $quantity );
276
+		$product->setQuantity($quantity);
277 277
 
278 278
 		$attributes = $product->getAttributes();
279
-		foreach( $attributes as $key => $attribute )
279
+		foreach ($attributes as $key => $attribute)
280 280
 		{
281
-			if( in_array( $attribute->getCode(), $configAttributeCodes ) ) {
282
-				unset( $attributes[$key] );
281
+			if (in_array($attribute->getCode(), $configAttributeCodes)) {
282
+				unset($attributes[$key]);
283 283
 			}
284 284
 		}
285
-		$product->setAttributes( $attributes );
285
+		$product->setAttributes($attributes);
286 286
 
287
-		$manager = \Aimeos\MShop\Factory::createManager( $this->getContext(), 'product' );
288
-		$productItem = $manager->findItem( $product->getProductCode(), array( 'price', 'text' ) );
289
-		$product->setPrice( $this->calcPrice( $product, $productItem->getRefItems( 'price', 'default' ), $quantity ) );
287
+		$manager = \Aimeos\MShop\Factory::createManager($this->getContext(), 'product');
288
+		$productItem = $manager->findItem($product->getProductCode(), array('price', 'text'));
289
+		$product->setPrice($this->calcPrice($product, $productItem->getRefItems('price', 'default'), $quantity));
290 290
 
291
-		$this->get()->editProduct( $product, $position );
291
+		$this->get()->editProduct($product, $position);
292 292
 
293 293
 		$this->save();
294 294
 	}
@@ -300,49 +300,49 @@  discard block
 block discarded – undo
300 300
 	 * @param string $code Coupon code entered by the user
301 301
 	 * @throws \Aimeos\Controller\Frontend\Basket\Exception if the coupon code is invalid or not allowed
302 302
 	 */
303
-	public function addCoupon( $code )
303
+	public function addCoupon($code)
304 304
 	{
305
-		$manager = \Aimeos\MShop\Factory::createManager( $this->getContext(), 'coupon' );
306
-		$codeManager = \Aimeos\MShop\Factory::createManager( $this->getContext(), 'coupon/code' );
305
+		$manager = \Aimeos\MShop\Factory::createManager($this->getContext(), 'coupon');
306
+		$codeManager = \Aimeos\MShop\Factory::createManager($this->getContext(), 'coupon/code');
307 307
 
308 308
 
309
-		$search = $codeManager->createSearch( true );
309
+		$search = $codeManager->createSearch(true);
310 310
 		$expr = array(
311
-			$search->compare( '==', 'coupon.code.code', $code ),
311
+			$search->compare('==', 'coupon.code.code', $code),
312 312
 			$search->getConditions(),
313 313
 		);
314
-		$search->setConditions( $search->combine( '&&', $expr ) );
315
-		$search->setSlice( 0, 1 );
314
+		$search->setConditions($search->combine('&&', $expr));
315
+		$search->setSlice(0, 1);
316 316
 
317
-		$result = $codeManager->searchItems( $search );
317
+		$result = $codeManager->searchItems($search);
318 318
 
319
-		if( ( $codeItem = reset( $result ) ) === false ) {
320
-			throw new \Aimeos\Controller\Frontend\Basket\Exception( sprintf( 'Coupon code "%1$s" is invalid or not available any more', $code ) );
319
+		if (($codeItem = reset($result)) === false) {
320
+			throw new \Aimeos\Controller\Frontend\Basket\Exception(sprintf('Coupon code "%1$s" is invalid or not available any more', $code));
321 321
 		}
322 322
 
323 323
 
324
-		$search = $manager->createSearch( true );
324
+		$search = $manager->createSearch(true);
325 325
 		$expr = array(
326
-			$search->compare( '==', 'coupon.id', $codeItem->getParentId() ),
326
+			$search->compare('==', 'coupon.id', $codeItem->getParentId()),
327 327
 			$search->getConditions(),
328 328
 		);
329
-		$search->setConditions( $search->combine( '&&', $expr ) );
330
-		$search->setSlice( 0, 1 );
329
+		$search->setConditions($search->combine('&&', $expr));
330
+		$search->setSlice(0, 1);
331 331
 
332
-		$result = $manager->searchItems( $search );
332
+		$result = $manager->searchItems($search);
333 333
 
334
-		if( ( $item = reset( $result ) ) === false ) {
335
-			throw new \Aimeos\Controller\Frontend\Basket\Exception( sprintf( 'Coupon for code "%1$s" is not available any more', $code ) );
334
+		if (($item = reset($result)) === false) {
335
+			throw new \Aimeos\Controller\Frontend\Basket\Exception(sprintf('Coupon for code "%1$s" is not available any more', $code));
336 336
 		}
337 337
 
338 338
 
339
-		$provider = $manager->getProvider( $item, $code );
339
+		$provider = $manager->getProvider($item, $code);
340 340
 
341
-		if( $provider->isAvailable( $this->get() ) !== true ) {
342
-			throw new \Aimeos\Controller\Frontend\Basket\Exception( sprintf( 'Requirements for coupon code "%1$s" aren\'t met', $code ) );
341
+		if ($provider->isAvailable($this->get()) !== true) {
342
+			throw new \Aimeos\Controller\Frontend\Basket\Exception(sprintf('Requirements for coupon code "%1$s" aren\'t met', $code));
343 343
 		}
344 344
 
345
-		$provider->addCoupon( $this->get() );
345
+		$provider->addCoupon($this->get());
346 346
 		$this->save();
347 347
 	}
348 348
 
@@ -353,21 +353,21 @@  discard block
 block discarded – undo
353 353
 	 * @param string $code Coupon code entered by the user
354 354
 	 * @throws \Aimeos\Controller\Frontend\Basket\Exception if the coupon code is invalid
355 355
 	 */
356
-	public function deleteCoupon( $code )
356
+	public function deleteCoupon($code)
357 357
 	{
358
-		$manager = \Aimeos\MShop\Factory::createManager( $this->getContext(), 'coupon' );
358
+		$manager = \Aimeos\MShop\Factory::createManager($this->getContext(), 'coupon');
359 359
 
360 360
 		$search = $manager->createSearch();
361
-		$search->setConditions( $search->compare( '==', 'coupon.code.code', $code ) );
362
-		$search->setSlice( 0, 1 );
361
+		$search->setConditions($search->compare('==', 'coupon.code.code', $code));
362
+		$search->setSlice(0, 1);
363 363
 
364
-		$result = $manager->searchItems( $search );
364
+		$result = $manager->searchItems($search);
365 365
 
366
-		if( ( $item = reset( $result ) ) === false ) {
367
-			throw new \Aimeos\Controller\Frontend\Basket\Exception( sprintf( 'Coupon code "%1$s" is invalid', $code ) );
366
+		if (($item = reset($result)) === false) {
367
+			throw new \Aimeos\Controller\Frontend\Basket\Exception(sprintf('Coupon code "%1$s" is invalid', $code));
368 368
 		}
369 369
 
370
-		$manager->getProvider( $item, $code )->deleteCoupon( $this->get() );
370
+		$manager->getProvider($item, $code)->deleteCoupon($this->get());
371 371
 		$this->save();
372 372
 	}
373 373
 
@@ -380,28 +380,28 @@  discard block
 block discarded – undo
380 380
 	 * @throws \Aimeos\Controller\Frontend\Basket\Exception If the billing or delivery address is not of any required type of
381 381
 	 * 	if one of the keys is invalid when using an array with key/value pairs
382 382
 	 */
383
-	public function setAddress( $type, $value )
383
+	public function setAddress($type, $value)
384 384
 	{
385
-		$address = \Aimeos\MShop\Factory::createManager( $this->getContext(), 'order/base/address' )->createItem();
386
-		$address->setType( $type );
385
+		$address = \Aimeos\MShop\Factory::createManager($this->getContext(), 'order/base/address')->createItem();
386
+		$address->setType($type);
387 387
 
388
-		if( $value instanceof \Aimeos\MShop\Common\Item\Address\Iface )
388
+		if ($value instanceof \Aimeos\MShop\Common\Item\Address\Iface)
389 389
 		{
390
-			$address->copyFrom( $value );
391
-			$this->get()->setAddress( $address, $type );
390
+			$address->copyFrom($value);
391
+			$this->get()->setAddress($address, $type);
392 392
 		}
393
-		else if( is_array( $value ) )
393
+		else if (is_array($value))
394 394
 		{
395
-			$this->setAddressFromArray( $address, $value );
396
-			$this->get()->setAddress( $address, $type );
395
+			$this->setAddressFromArray($address, $value);
396
+			$this->get()->setAddress($address, $type);
397 397
 		}
398
-		else if( $value === null )
398
+		else if ($value === null)
399 399
 		{
400
-			$this->get()->deleteAddress( $type );
400
+			$this->get()->deleteAddress($type);
401 401
 		}
402 402
 		else
403 403
 		{
404
-			throw new \Aimeos\Controller\Frontend\Basket\Exception( sprintf( 'Invalid value for address type "%1$s"', $type ) );
404
+			throw new \Aimeos\Controller\Frontend\Basket\Exception(sprintf('Invalid value for address type "%1$s"', $type));
405 405
 		}
406 406
 
407 407
 		$this->save();
@@ -417,48 +417,48 @@  discard block
 block discarded – undo
417 417
 	 * 	entered by the customer when choosing one of the delivery or payment options
418 418
 	 * @throws \Aimeos\Controller\Frontend\Basket\Exception If there is no price to the service item attached
419 419
 	 */
420
-	public function setService( $type, $id, array $attributes = [] )
420
+	public function setService($type, $id, array $attributes = [])
421 421
 	{
422
-		if( $id === null )
422
+		if ($id === null)
423 423
 		{
424
-			$this->get()->deleteService( $type );
424
+			$this->get()->deleteService($type);
425 425
 			$this->save();
426 426
 			return;
427 427
 		}
428 428
 
429 429
 		$context = $this->getContext();
430 430
 
431
-		$serviceManager = \Aimeos\MShop\Factory::createManager( $context, 'service' );
432
-		$serviceItem = $serviceManager->getItem( $id, array( 'media', 'price', 'text' ) );
431
+		$serviceManager = \Aimeos\MShop\Factory::createManager($context, 'service');
432
+		$serviceItem = $serviceManager->getItem($id, array('media', 'price', 'text'));
433 433
 
434
-		$provider = $serviceManager->getProvider( $serviceItem );
435
-		$result = $provider->checkConfigFE( $attributes );
436
-		$unknown = array_diff_key( $attributes, $result );
434
+		$provider = $serviceManager->getProvider($serviceItem);
435
+		$result = $provider->checkConfigFE($attributes);
436
+		$unknown = array_diff_key($attributes, $result);
437 437
 
438
-		if( count( $unknown ) > 0 )
438
+		if (count($unknown) > 0)
439 439
 		{
440
-			$msg = sprintf( 'Unknown attributes "%1$s"', implode( '","', array_keys( $unknown ) ) );
441
-			throw new \Aimeos\Controller\Frontend\Basket\Exception( $msg );
440
+			$msg = sprintf('Unknown attributes "%1$s"', implode('","', array_keys($unknown)));
441
+			throw new \Aimeos\Controller\Frontend\Basket\Exception($msg);
442 442
 		}
443 443
 
444
-		foreach( $result as $key => $value )
444
+		foreach ($result as $key => $value)
445 445
 		{
446
-			if( $value !== null ) {
447
-				throw new \Aimeos\Controller\Frontend\Basket\Exception( $value );
446
+			if ($value !== null) {
447
+				throw new \Aimeos\Controller\Frontend\Basket\Exception($value);
448 448
 			}
449 449
 		}
450 450
 
451
-		$orderBaseServiceManager = \Aimeos\MShop\Factory::createManager( $context, 'order/base/service' );
451
+		$orderBaseServiceManager = \Aimeos\MShop\Factory::createManager($context, 'order/base/service');
452 452
 		$orderServiceItem = $orderBaseServiceManager->createItem();
453
-		$orderServiceItem->copyFrom( $serviceItem );
453
+		$orderServiceItem->copyFrom($serviceItem);
454 454
 
455 455
 		// remove service rebate of original price
456
-		$price = $provider->calcPrice( $this->get() )->setRebate( '0.00' );
457
-		$orderServiceItem->setPrice( $price );
456
+		$price = $provider->calcPrice($this->get())->setRebate('0.00');
457
+		$orderServiceItem->setPrice($price);
458 458
 
459
-		$provider->setConfigFE( $orderServiceItem, $attributes );
459
+		$provider->setConfigFE($orderServiceItem, $attributes);
460 460
 
461
-		$this->get()->setService( $orderServiceItem, $type );
461
+		$this->get()->setService($orderServiceItem, $type);
462 462
 		$this->save();
463 463
 	}
464 464
 
@@ -471,18 +471,18 @@  discard block
 block discarded – undo
471 471
 	 * 	an address item.
472 472
 	 * @throws \Aimeos\Controller\Frontend\Basket\Exception
473 473
 	 */
474
-	protected function setAddressFromArray( \Aimeos\MShop\Order\Item\Base\Address\Iface $address, array $map )
474
+	protected function setAddressFromArray(\Aimeos\MShop\Order\Item\Base\Address\Iface $address, array $map)
475 475
 	{
476
-		foreach( $map as $key => $value ) {
477
-			$map[$key] = strip_tags( $value ); // prevent XSS
476
+		foreach ($map as $key => $value) {
477
+			$map[$key] = strip_tags($value); // prevent XSS
478 478
 		}
479 479
 
480
-		$errors = $address->fromArray( $map );
480
+		$errors = $address->fromArray($map);
481 481
 
482
-		if( count( $errors ) > 0 )
482
+		if (count($errors) > 0)
483 483
 		{
484
-			$msg = sprintf( 'Invalid address properties, please check your input' );
485
-			throw new \Aimeos\Controller\Frontend\Basket\Exception( $msg, 0, null, $errors );
484
+			$msg = sprintf('Invalid address properties, please check your input');
485
+			throw new \Aimeos\Controller\Frontend\Basket\Exception($msg, 0, null, $errors);
486 486
 		}
487 487
 	}
488 488
 }
Please login to merge, or discard this patch.