Completed
Push — master ( afe8d8...095350 )
by Aimeos
02:02
created
controller/frontend/src/Controller/Frontend/Basket/Standard.php 1 patch
Spacing   +128 added lines, -128 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->baskets[$this->type]->getLocale(), $this->type );
67
+			$this->baskets[$this->type] = $this->domainManager->getSession($this->type);
68
+			$this->checkLocale($this->baskets[$this->type]->getLocale(), $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;
@@ -127,7 +127,7 @@  discard block
 block discarded – undo
127 127
 		 * @category Developer
128 128
 		 * @see controller/frontend/basket/limit-seconds
129 129
 		 */
130
-		$count = $config->get( 'controller/frontend/basket/limit-count', 5 );
130
+		$count = $config->get('controller/frontend/basket/limit-count', 5);
131 131
 
132 132
 		/** controller/frontend/basket/limit-seconds
133 133
 		 * Order limitation time frame in seconds
@@ -145,33 +145,33 @@  discard block
 block discarded – undo
145 145
 		 * @category Developer
146 146
 		 * @see controller/frontend/basket/limit-count
147 147
 		 */
148
-		$seconds = $config->get( 'controller/frontend/basket/limit-seconds', 300 );
148
+		$seconds = $config->get('controller/frontend/basket/limit-seconds', 300);
149 149
 
150 150
 		$search = $this->domainManager->createSearch();
151 151
 		$expr = [
152
-			$search->compare( '==', 'order.base.editor', $context->getEditor() ),
153
-			$search->compare( '>=', 'order.base.ctime', date( 'Y-m-d H:i:s', time() - $seconds ) ),
152
+			$search->compare('==', 'order.base.editor', $context->getEditor()),
153
+			$search->compare('>=', 'order.base.ctime', date('Y-m-d H:i:s', time() - $seconds)),
154 154
 		];
155
-		$search->setConditions( $search->combine( '&&', $expr ) );
156
-		$search->setSlice( 0, 0 );
155
+		$search->setConditions($search->combine('&&', $expr));
156
+		$search->setSlice(0, 0);
157 157
 
158
-		$this->domainManager->searchItems( $search, [], $total );
158
+		$this->domainManager->searchItems($search, [], $total);
159 159
 
160
-		if( $total > $count )
160
+		if ($total > $count)
161 161
 		{
162
-			$msg = $context->getI18n()->dt( 'controller/frontend', 'Temporary order limit reached' );
163
-			throw new \Aimeos\Controller\Frontend\Basket\Exception( $msg );
162
+			$msg = $context->getI18n()->dt('controller/frontend', 'Temporary order limit reached');
163
+			throw new \Aimeos\Controller\Frontend\Basket\Exception($msg);
164 164
 		}
165 165
 
166 166
 
167 167
 		$basket = $this->get()->finish();
168
-		$basket->setCustomerId( (string) $context->getUserId() );
168
+		$basket->setCustomerId((string) $context->getUserId());
169 169
 
170 170
 		$this->domainManager->begin();
171
-		$this->domainManager->store( $basket );
171
+		$this->domainManager->store($basket);
172 172
 		$this->domainManager->commit();
173 173
 
174
-		$this->createSubscriptions( $basket );
174
+		$this->createSubscriptions($basket);
175 175
 
176 176
 		return $basket;
177 177
 	}
@@ -185,9 +185,9 @@  discard block
 block discarded – undo
185 185
 	 * @param boolean $default True to add default criteria (user logged in), false if not
186 186
 	 * @return \Aimeos\MShop\Order\Item\Base\Iface Order base object including the given parts
187 187
 	 */
188
-	public function load( $id, $parts = \Aimeos\MShop\Order\Item\Base\Base::PARTS_ALL, $default = true )
188
+	public function load($id, $parts = \Aimeos\MShop\Order\Item\Base\Base::PARTS_ALL, $default = true)
189 189
 	{
190
-		return $this->domainManager->load( $id, $parts, false, $default );
190
+		return $this->domainManager->load($id, $parts, false, $default);
191 191
 	}
192 192
 
193 193
 
@@ -206,33 +206,33 @@  discard block
 block discarded – undo
206 206
 	 * @param string $stocktype Unique code of the stock type to deliver the products from
207 207
 	 * @throws \Aimeos\Controller\Frontend\Basket\Exception If the product isn't available
208 208
 	 */
209
-	public function addProduct( $prodid, $quantity = 1, $stocktype = 'default', array $variantAttributeIds = [],
210
-		array $configAttributeIds = [], array $hiddenAttributeIds = [], array $customAttributeValues = [] )
209
+	public function addProduct($prodid, $quantity = 1, $stocktype = 'default', array $variantAttributeIds = [],
210
+		array $configAttributeIds = [], array $hiddenAttributeIds = [], array $customAttributeValues = [])
211 211
 	{
212 212
 		$attributeMap = [
213
-			'custom' => array_keys( $customAttributeValues ),
214
-			'config' => array_keys( $configAttributeIds ),
213
+			'custom' => array_keys($customAttributeValues),
214
+			'config' => array_keys($configAttributeIds),
215 215
 			'hidden' => $hiddenAttributeIds,
216 216
 		];
217
-		$this->checkListRef( $prodid, 'attribute', $attributeMap );
217
+		$this->checkListRef($prodid, 'attribute', $attributeMap);
218 218
 
219 219
 
220 220
 		$context = $this->getContext();
221
-		$productManager = \Aimeos\MShop\Factory::createManager( $context, 'product' );
222
-		$productItem = $productManager->getItem( $prodid, array( 'media', 'supplier', 'price', 'product', 'text' ), true );
223
-		$prices = $productItem->getRefItems( 'price', 'default', 'default' );
221
+		$productManager = \Aimeos\MShop\Factory::createManager($context, 'product');
222
+		$productItem = $productManager->getItem($prodid, array('media', 'supplier', 'price', 'product', 'text'), true);
223
+		$prices = $productItem->getRefItems('price', 'default', 'default');
224 224
 
225
-		$orderBaseProductItem = \Aimeos\MShop\Factory::createManager( $context, 'order/base/product' )->createItem();
226
-		$orderBaseProductItem->copyFrom( $productItem )->setQuantity( $quantity )->setStockType( $stocktype );
225
+		$orderBaseProductItem = \Aimeos\MShop\Factory::createManager($context, 'order/base/product')->createItem();
226
+		$orderBaseProductItem->copyFrom($productItem)->setQuantity($quantity)->setStockType($stocktype);
227 227
 
228
-		$attr = $this->getOrderProductAttributes( 'custom', array_keys( $customAttributeValues ), $customAttributeValues );
229
-		$attr = array_merge( $attr, $this->getOrderProductAttributes( 'config', array_keys( $configAttributeIds ), [], $configAttributeIds ) );
230
-		$attr = array_merge( $attr, $this->getOrderProductAttributes( 'hidden', $hiddenAttributeIds ) );
228
+		$attr = $this->getOrderProductAttributes('custom', array_keys($customAttributeValues), $customAttributeValues);
229
+		$attr = array_merge($attr, $this->getOrderProductAttributes('config', array_keys($configAttributeIds), [], $configAttributeIds));
230
+		$attr = array_merge($attr, $this->getOrderProductAttributes('hidden', $hiddenAttributeIds));
231 231
 
232
-		$orderBaseProductItem->setAttributes( $attr );
233
-		$orderBaseProductItem->setPrice( $this->calcPrice( $orderBaseProductItem, $prices, $quantity ) );
232
+		$orderBaseProductItem->setAttributes($attr);
233
+		$orderBaseProductItem->setPrice($this->calcPrice($orderBaseProductItem, $prices, $quantity));
234 234
 
235
-		$this->get()->addProduct( $orderBaseProductItem );
235
+		$this->get()->addProduct($orderBaseProductItem);
236 236
 		$this->save();
237 237
 	}
238 238
 
@@ -242,17 +242,17 @@  discard block
 block discarded – undo
242 242
 	 *
243 243
 	 * @param integer $position Position number (key) of the order product item
244 244
 	 */
245
-	public function deleteProduct( $position )
245
+	public function deleteProduct($position)
246 246
 	{
247
-		$product = $this->get()->getProduct( $position );
247
+		$product = $this->get()->getProduct($position);
248 248
 
249
-		if( $product->getFlags() === \Aimeos\MShop\Order\Item\Base\Product\Base::FLAG_IMMUTABLE )
249
+		if ($product->getFlags() === \Aimeos\MShop\Order\Item\Base\Product\Base::FLAG_IMMUTABLE)
250 250
 		{
251
-			$msg = $this->getContext()->getI18n()->dt( 'controller/frontend', 'Basket item at position "%1$d" cannot be deleted manually' );
252
-			throw new \Aimeos\Controller\Frontend\Basket\Exception( sprintf( $msg, $position ) );
251
+			$msg = $this->getContext()->getI18n()->dt('controller/frontend', 'Basket item at position "%1$d" cannot be deleted manually');
252
+			throw new \Aimeos\Controller\Frontend\Basket\Exception(sprintf($msg, $position));
253 253
 		}
254 254
 
255
-		$this->get()->deleteProduct( $position );
255
+		$this->get()->deleteProduct($position);
256 256
 		$this->save();
257 257
 	}
258 258
 
@@ -264,32 +264,32 @@  discard block
 block discarded – undo
264 264
 	 * @param integer $quantity New quantiy of the product item
265 265
 	 * @param string[] $configAttributeCodes Codes of the product config attributes that should be REMOVED
266 266
 	 */
267
-	public function editProduct( $position, $quantity, array $configAttributeCodes = [] )
267
+	public function editProduct($position, $quantity, array $configAttributeCodes = [])
268 268
 	{
269
-		$product = $this->get()->getProduct( $position );
269
+		$product = $this->get()->getProduct($position);
270 270
 
271
-		if( $product->getFlags() & \Aimeos\MShop\Order\Item\Base\Product\Base::FLAG_IMMUTABLE )
271
+		if ($product->getFlags() & \Aimeos\MShop\Order\Item\Base\Product\Base::FLAG_IMMUTABLE)
272 272
 		{
273
-			$msg = $this->getContext()->getI18n()->dt( 'controller/frontend', 'Basket item at position "%1$d" cannot be changed' );
274
-			throw new \Aimeos\Controller\Frontend\Basket\Exception( sprintf( $msg, $position ) );
273
+			$msg = $this->getContext()->getI18n()->dt('controller/frontend', 'Basket item at position "%1$d" cannot be changed');
274
+			throw new \Aimeos\Controller\Frontend\Basket\Exception(sprintf($msg, $position));
275 275
 		}
276 276
 
277
-		$product->setQuantity( $quantity );
277
+		$product->setQuantity($quantity);
278 278
 
279 279
 		$attributes = $product->getAttributes();
280
-		foreach( $attributes as $key => $attribute )
280
+		foreach ($attributes as $key => $attribute)
281 281
 		{
282
-			if( in_array( $attribute->getCode(), $configAttributeCodes ) ) {
283
-				unset( $attributes[$key] );
282
+			if (in_array($attribute->getCode(), $configAttributeCodes)) {
283
+				unset($attributes[$key]);
284 284
 			}
285 285
 		}
286
-		$product->setAttributes( $attributes );
286
+		$product->setAttributes($attributes);
287 287
 
288
-		$manager = \Aimeos\MShop\Factory::createManager( $this->getContext(), 'product' );
289
-		$productItem = $manager->findItem( $product->getProductCode(), array( 'price', 'text' ), true );
290
-		$product->setPrice( $this->calcPrice( $product, $productItem->getRefItems( 'price', 'default' ), $quantity ) );
288
+		$manager = \Aimeos\MShop\Factory::createManager($this->getContext(), 'product');
289
+		$productItem = $manager->findItem($product->getProductCode(), array('price', 'text'), true);
290
+		$product->setPrice($this->calcPrice($product, $productItem->getRefItems('price', 'default'), $quantity));
291 291
 
292
-		$this->get()->editProduct( $product, $position );
292
+		$this->get()->editProduct($product, $position);
293 293
 
294 294
 		$this->save();
295 295
 	}
@@ -301,7 +301,7 @@  discard block
 block discarded – undo
301 301
 	 * @param string $code Coupon code entered by the user
302 302
 	 * @throws \Aimeos\Controller\Frontend\Basket\Exception if the coupon code is invalid or not allowed
303 303
 	 */
304
-	public function addCoupon( $code )
304
+	public function addCoupon($code)
305 305
 	{
306 306
 		$context = $this->getContext();
307 307
 
@@ -321,39 +321,39 @@  discard block
 block discarded – undo
321 321
 		 * @category User
322 322
 		 * @category Developer
323 323
 		 */
324
-		$allowed = $context->getConfig()->get( 'controller/frontend/basket/standard/coupon/allowed', 1 );
324
+		$allowed = $context->getConfig()->get('controller/frontend/basket/standard/coupon/allowed', 1);
325 325
 
326
-		if( $allowed <= count( $this->get()->getCoupons() ) )
326
+		if ($allowed <= count($this->get()->getCoupons()))
327 327
 		{
328
-			$msg = $context->getI18n()->dt( 'controller/frontend', 'Number of coupon codes exceeds the limit' );
329
-			throw new \Aimeos\Controller\Frontend\Basket\Exception( $msg );
328
+			$msg = $context->getI18n()->dt('controller/frontend', 'Number of coupon codes exceeds the limit');
329
+			throw new \Aimeos\Controller\Frontend\Basket\Exception($msg);
330 330
 		}
331 331
 
332 332
 
333
-		$manager = \Aimeos\MShop\Factory::createManager( $context, 'coupon' );
333
+		$manager = \Aimeos\MShop\Factory::createManager($context, 'coupon');
334 334
 
335 335
 		$search = $manager->createSearch();
336
-		$search->setConditions( $search->compare( '==', 'coupon.code.code', $code ) );
337
-		$search->setSlice( 0, 1 );
336
+		$search->setConditions($search->compare('==', 'coupon.code.code', $code));
337
+		$search->setSlice(0, 1);
338 338
 
339
-		$result = $manager->searchItems( $search );
339
+		$result = $manager->searchItems($search);
340 340
 
341
-		if( ( $item = reset( $result ) ) === false )
341
+		if (($item = reset($result)) === false)
342 342
 		{
343
-			$msg = sprintf( $context->getI18n()->dt( 'controller/frontend', 'Coupon code "%1$s" is invalid or not available any more' ), $code );
344
-			throw new \Aimeos\Controller\Frontend\Basket\Exception( $msg );
343
+			$msg = sprintf($context->getI18n()->dt('controller/frontend', 'Coupon code "%1$s" is invalid or not available any more'), $code);
344
+			throw new \Aimeos\Controller\Frontend\Basket\Exception($msg);
345 345
 		}
346 346
 
347 347
 
348
-		$provider = $manager->getProvider( $item, $code );
348
+		$provider = $manager->getProvider($item, $code);
349 349
 
350
-		if( $provider->isAvailable( $this->get() ) !== true )
350
+		if ($provider->isAvailable($this->get()) !== true)
351 351
 		{
352
-			$msg = sprintf( $context->getI18n()->dt( 'controller/frontend', 'Requirements for coupon code "%1$s" aren\'t met' ), $code );
353
-			throw new \Aimeos\Controller\Frontend\Basket\Exception( $msg );
352
+			$msg = sprintf($context->getI18n()->dt('controller/frontend', 'Requirements for coupon code "%1$s" aren\'t met'), $code);
353
+			throw new \Aimeos\Controller\Frontend\Basket\Exception($msg);
354 354
 		}
355 355
 
356
-		$provider->addCoupon( $this->get() );
356
+		$provider->addCoupon($this->get());
357 357
 		$this->save();
358 358
 	}
359 359
 
@@ -364,24 +364,24 @@  discard block
 block discarded – undo
364 364
 	 * @param string $code Coupon code entered by the user
365 365
 	 * @throws \Aimeos\Controller\Frontend\Basket\Exception if the coupon code is invalid
366 366
 	 */
367
-	public function deleteCoupon( $code )
367
+	public function deleteCoupon($code)
368 368
 	{
369 369
 		$context = $this->getContext();
370
-		$manager = \Aimeos\MShop\Factory::createManager( $context, 'coupon' );
370
+		$manager = \Aimeos\MShop\Factory::createManager($context, 'coupon');
371 371
 
372 372
 		$search = $manager->createSearch();
373
-		$search->setConditions( $search->compare( '==', 'coupon.code.code', $code ) );
374
-		$search->setSlice( 0, 1 );
373
+		$search->setConditions($search->compare('==', 'coupon.code.code', $code));
374
+		$search->setSlice(0, 1);
375 375
 
376
-		$result = $manager->searchItems( $search );
376
+		$result = $manager->searchItems($search);
377 377
 
378
-		if( ( $item = reset( $result ) ) === false )
378
+		if (($item = reset($result)) === false)
379 379
 		{
380
-			$msg = $context->getI18n()->dt( 'controller/frontend', 'Coupon code "%1$s" is invalid' );
381
-			throw new \Aimeos\Controller\Frontend\Basket\Exception( sprintf( $msg, $code ) );
380
+			$msg = $context->getI18n()->dt('controller/frontend', 'Coupon code "%1$s" is invalid');
381
+			throw new \Aimeos\Controller\Frontend\Basket\Exception(sprintf($msg, $code));
382 382
 		}
383 383
 
384
-		$manager->getProvider( $item, $code )->deleteCoupon( $this->get() );
384
+		$manager->getProvider($item, $code)->deleteCoupon($this->get());
385 385
 		$this->save();
386 386
 	}
387 387
 
@@ -394,30 +394,30 @@  discard block
 block discarded – undo
394 394
 	 * @throws \Aimeos\Controller\Frontend\Basket\Exception If the billing or delivery address is not of any required type of
395 395
 	 * 	if one of the keys is invalid when using an array with key/value pairs
396 396
 	 */
397
-	public function setAddress( $type, $value )
397
+	public function setAddress($type, $value)
398 398
 	{
399 399
 		$context = $this->getContext();
400
-		$address = \Aimeos\MShop\Factory::createManager( $context, 'order/base/address' )->createItem();
401
-		$address->setType( $type );
400
+		$address = \Aimeos\MShop\Factory::createManager($context, 'order/base/address')->createItem();
401
+		$address->setType($type);
402 402
 
403
-		if( $value instanceof \Aimeos\MShop\Common\Item\Address\Iface )
403
+		if ($value instanceof \Aimeos\MShop\Common\Item\Address\Iface)
404 404
 		{
405
-			$address->copyFrom( $value );
406
-			$this->get()->setAddress( $address, $type );
405
+			$address->copyFrom($value);
406
+			$this->get()->setAddress($address, $type);
407 407
 		}
408
-		else if( is_array( $value ) )
408
+		else if (is_array($value))
409 409
 		{
410
-			$this->setAddressFromArray( $address, $value );
411
-			$this->get()->setAddress( $address, $type );
410
+			$this->setAddressFromArray($address, $value);
411
+			$this->get()->setAddress($address, $type);
412 412
 		}
413
-		else if( $value === null )
413
+		else if ($value === null)
414 414
 		{
415
-			$this->get()->deleteAddress( $type );
415
+			$this->get()->deleteAddress($type);
416 416
 		}
417 417
 		else
418 418
 		{
419
-			$msg = $context->getI18n()->dt( 'controller/frontend', 'Invalid value for address type "%1$s"' );
420
-			throw new \Aimeos\Controller\Frontend\Basket\Exception( sprintf( $msg, $type ) );
419
+			$msg = $context->getI18n()->dt('controller/frontend', 'Invalid value for address type "%1$s"');
420
+			throw new \Aimeos\Controller\Frontend\Basket\Exception(sprintf($msg, $type));
421 421
 		}
422 422
 
423 423
 		$this->save();
@@ -433,42 +433,42 @@  discard block
 block discarded – undo
433 433
 	 * 	entered by the customer when choosing one of the delivery or payment options
434 434
 	 * @throws \Aimeos\Controller\Frontend\Basket\Exception If there is no price to the service item attached
435 435
 	 */
436
-	public function addService( $type, $id, array $attributes = [] )
436
+	public function addService($type, $id, array $attributes = [])
437 437
 	{
438 438
 		$context = $this->getContext();
439 439
 
440
-		$serviceManager = \Aimeos\MShop\Factory::createManager( $context, 'service' );
441
-		$serviceItem = $serviceManager->getItem( $id, array( 'media', 'price', 'text' ) );
440
+		$serviceManager = \Aimeos\MShop\Factory::createManager($context, 'service');
441
+		$serviceItem = $serviceManager->getItem($id, array('media', 'price', 'text'));
442 442
 
443
-		$provider = $serviceManager->getProvider( $serviceItem, $serviceItem->getType() );
444
-		$result = $provider->checkConfigFE( $attributes );
445
-		$unknown = array_diff_key( $attributes, $result );
443
+		$provider = $serviceManager->getProvider($serviceItem, $serviceItem->getType());
444
+		$result = $provider->checkConfigFE($attributes);
445
+		$unknown = array_diff_key($attributes, $result);
446 446
 
447
-		if( count( $unknown ) > 0 )
447
+		if (count($unknown) > 0)
448 448
 		{
449
-			$msg = $context->getI18n()->dt( 'controller/frontend', 'Unknown attributes "%1$s"' );
450
-			$msg = sprintf( $msg, implode( '","', array_keys( $unknown ) ) );
451
-			throw new \Aimeos\Controller\Frontend\Basket\Exception( $msg );
449
+			$msg = $context->getI18n()->dt('controller/frontend', 'Unknown attributes "%1$s"');
450
+			$msg = sprintf($msg, implode('","', array_keys($unknown)));
451
+			throw new \Aimeos\Controller\Frontend\Basket\Exception($msg);
452 452
 		}
453 453
 
454
-		foreach( $result as $key => $value )
454
+		foreach ($result as $key => $value)
455 455
 		{
456
-			if( $value !== null ) {
457
-				throw new \Aimeos\Controller\Frontend\Basket\Exception( $value );
456
+			if ($value !== null) {
457
+				throw new \Aimeos\Controller\Frontend\Basket\Exception($value);
458 458
 			}
459 459
 		}
460 460
 
461
-		$orderBaseServiceManager = \Aimeos\MShop\Factory::createManager( $context, 'order/base/service' );
461
+		$orderBaseServiceManager = \Aimeos\MShop\Factory::createManager($context, 'order/base/service');
462 462
 		$orderServiceItem = $orderBaseServiceManager->createItem();
463
-		$orderServiceItem->copyFrom( $serviceItem );
463
+		$orderServiceItem->copyFrom($serviceItem);
464 464
 
465 465
 		// remove service rebate of original price
466
-		$price = $provider->calcPrice( $this->get() )->setRebate( '0.00' );
467
-		$orderServiceItem->setPrice( $price );
466
+		$price = $provider->calcPrice($this->get())->setRebate('0.00');
467
+		$orderServiceItem->setPrice($price);
468 468
 
469
-		$provider->setConfigFE( $orderServiceItem, $attributes );
469
+		$provider->setConfigFE($orderServiceItem, $attributes);
470 470
 
471
-		$this->get()->addService( $orderServiceItem, $type );
471
+		$this->get()->addService($orderServiceItem, $type);
472 472
 		$this->save();
473 473
 	}
474 474
 
@@ -478,9 +478,9 @@  discard block
 block discarded – undo
478 478
 	 *
479 479
 	 * @param string $type Service type code like 'payment' or 'delivery'
480 480
 	 */
481
-	public function deleteService( $type )
481
+	public function deleteService($type)
482 482
 	{
483
-		$this->get()->deleteService( $type );
483
+		$this->get()->deleteService($type);
484 484
 		$this->save();
485 485
 	}
486 486
 
@@ -493,18 +493,18 @@  discard block
 block discarded – undo
493 493
 	 * 	an address item.
494 494
 	 * @throws \Aimeos\Controller\Frontend\Basket\Exception
495 495
 	 */
496
-	protected function setAddressFromArray( \Aimeos\MShop\Order\Item\Base\Address\Iface $address, array $map )
496
+	protected function setAddressFromArray(\Aimeos\MShop\Order\Item\Base\Address\Iface $address, array $map)
497 497
 	{
498
-		foreach( $map as $key => $value ) {
499
-			$map[$key] = strip_tags( $value ); // prevent XSS
498
+		foreach ($map as $key => $value) {
499
+			$map[$key] = strip_tags($value); // prevent XSS
500 500
 		}
501 501
 
502
-		$errors = $address->fromArray( $map );
502
+		$errors = $address->fromArray($map);
503 503
 
504
-		if( count( $errors ) > 0 )
504
+		if (count($errors) > 0)
505 505
 		{
506
-			$msg = $this->getContext()->getI18n()->dt( 'controller/frontend', 'Invalid address properties, please check your input' );
507
-			throw new \Aimeos\Controller\Frontend\Basket\Exception( $msg, 0, null, $errors );
506
+			$msg = $this->getContext()->getI18n()->dt('controller/frontend', 'Invalid address properties, please check your input');
507
+			throw new \Aimeos\Controller\Frontend\Basket\Exception($msg, 0, null, $errors);
508 508
 		}
509 509
 	}
510 510
 }
Please login to merge, or discard this patch.
controller/frontend/src/Controller/Frontend/Basket/Base.php 1 patch
Spacing   +150 added lines, -150 removed lines patch added patch discarded remove patch
@@ -31,52 +31,52 @@  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( $context, 'product' );
41
-			$prices = $manager->getItem( $product->getProductId(), array( 'price' ) )->getRefItems( 'price', 'default' );
40
+			$manager = \Aimeos\MShop\Factory::createManager($context, '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 )
53
+			if (preg_match('/^[0-9]*(\.[0-9]+)?$/', $amount) !== 1 || ((double) $amount) < 0.01)
54 54
 			{
55
-				$msg = $context->getI18n()->dt( 'controller/frontend', 'Invalid price value "%1$s"' );
56
-				throw new \Aimeos\Controller\Frontend\Basket\Exception( sprintf( $msg, $amount ) );
55
+				$msg = $context->getI18n()->dt('controller/frontend', 'Invalid price value "%1$s"');
56
+				throw new \Aimeos\Controller\Frontend\Basket\Exception(sprintf($msg, $amount));
57 57
 			}
58 58
 
59
-			$price->setValue( $amount );
59
+			$price->setValue($amount);
60 60
 		}
61 61
 
62 62
 		$orderAttributes = $product->getAttributes();
63
-		$attrItems = $this->getAttributeItems( $orderAttributes );
63
+		$attrItems = $this->getAttributeItems($orderAttributes);
64 64
 
65 65
 		// add prices of (optional) attributes
66
-		foreach( $orderAttributes as $orderAttrItem )
66
+		foreach ($orderAttributes as $orderAttrItem)
67 67
 		{
68 68
 			$attrId = $orderAttrItem->getAttributeId();
69 69
 
70
-			if( isset( $attrItems[$attrId] )
71
-				&& ( $prices = $attrItems[$attrId]->getRefItems( 'price', 'default' ) ) !== []
70
+			if (isset($attrItems[$attrId])
71
+				&& ($prices = $attrItems[$attrId]->getRefItems('price', 'default')) !== []
72 72
 			) {
73
-				$attrPrice = $priceManager->getLowestPrice( $prices, $orderAttrItem->getQuantity() );
74
-				$price->addItem( $attrPrice, $orderAttrItem->getQuantity() );
73
+				$attrPrice = $priceManager->getLowestPrice($prices, $orderAttrItem->getQuantity());
74
+				$price->addItem($attrPrice, $orderAttrItem->getQuantity());
75 75
 			}
76 76
 		}
77 77
 
78 78
 		// remove product rebate of original price in favor to rebates granted for the order
79
-		$price->setRebate( '0.00' );
79
+		$price->setRebate('0.00');
80 80
 
81 81
 		return $price;
82 82
 	}
@@ -90,43 +90,43 @@  discard block
 block discarded – undo
90 90
 	 * @param array $refMap Associative list of list type codes as keys and lists of reference IDs as values
91 91
 	 * @throws \Aimeos\Controller\Frontend\Basket\Exception If one or more of the IDs are not associated
92 92
 	 */
93
-	protected function checkListRef( $prodId, $domain, array $refMap )
93
+	protected function checkListRef($prodId, $domain, array $refMap)
94 94
 	{
95
-		if( empty( $refMap ) ) {
95
+		if (empty($refMap)) {
96 96
 			return;
97 97
 		}
98 98
 
99 99
 		$context = $this->getContext();
100
-		$productManager = \Aimeos\MShop\Factory::createManager( $context, 'product' );
101
-		$search = $productManager->createSearch( true );
100
+		$productManager = \Aimeos\MShop\Factory::createManager($context, 'product');
101
+		$search = $productManager->createSearch(true);
102 102
 
103 103
 		$expr = array(
104
-			$search->compare( '==', 'product.id', $prodId ),
104
+			$search->compare('==', 'product.id', $prodId),
105 105
 			$search->getConditions(),
106 106
 		);
107 107
 
108
-		foreach( $refMap as $listType => $refIds )
108
+		foreach ($refMap as $listType => $refIds)
109 109
 		{
110
-			if( empty( $refIds ) ) {
110
+			if (empty($refIds)) {
111 111
 				continue;
112 112
 			}
113 113
 
114
-			foreach( $refIds as $key => $refId ) {
114
+			foreach ($refIds as $key => $refId) {
115 115
 				$refIds[$key] = (string) $refId;
116 116
 			}
117 117
 
118
-			$param = array( $domain, $this->getProductListTypeItem( $domain, $listType )->getId(), $refIds );
119
-			$cmpfunc = $search->createFunction( 'product.contains', $param );
118
+			$param = array($domain, $this->getProductListTypeItem($domain, $listType)->getId(), $refIds);
119
+			$cmpfunc = $search->createFunction('product.contains', $param);
120 120
 
121
-			$expr[] = $search->compare( '==', $cmpfunc, count( $refIds ) );
121
+			$expr[] = $search->compare('==', $cmpfunc, count($refIds));
122 122
 		}
123 123
 
124
-		$search->setConditions( $search->combine( '&&', $expr ) );
124
+		$search->setConditions($search->combine('&&', $expr));
125 125
 
126
-		if( count( $productManager->searchItems( $search, [] ) ) === 0 )
126
+		if (count($productManager->searchItems($search, [])) === 0)
127 127
 		{
128
-			$msg = $context->getI18n()->dt( 'controller/frontend', 'Invalid "%1$s" references for product with ID %2$s' );
129
-			throw new \Aimeos\Controller\Frontend\Basket\Exception( sprintf( $msg, $domain, json_encode( $prodId ) ) );
128
+			$msg = $context->getI18n()->dt('controller/frontend', 'Invalid "%1$s" references for product with ID %2$s');
129
+			throw new \Aimeos\Controller\Frontend\Basket\Exception(sprintf($msg, $domain, json_encode($prodId)));
130 130
 		}
131 131
 	}
132 132
 
@@ -137,40 +137,40 @@  discard block
 block discarded – undo
137 137
 	 * @param \Aimeos\MShop\Locale\Item\Iface $locale Locale object from current basket
138 138
 	 * @param string $type Basket type
139 139
 	 */
140
-	protected function checkLocale( \Aimeos\MShop\Locale\Item\Iface $locale, $type )
140
+	protected function checkLocale(\Aimeos\MShop\Locale\Item\Iface $locale, $type)
141 141
 	{
142 142
 		$errors = [];
143 143
 		$context = $this->getContext();
144 144
 		$session = $context->getSession();
145 145
 
146
-		$localeStr = $session->get( 'aimeos/basket/locale' );
146
+		$localeStr = $session->get('aimeos/basket/locale');
147 147
 		$localeKey = $locale->getSite()->getCode() . '|' . $locale->getLanguageId() . '|' . $locale->getCurrencyId();
148 148
 
149
-		if( $localeStr !== null && $localeStr !== $localeKey )
149
+		if ($localeStr !== null && $localeStr !== $localeKey)
150 150
 		{
151
-			$locParts = explode( '|', $localeStr );
152
-			$locSite = ( isset( $locParts[0] ) ? $locParts[0] : '' );
153
-			$locLanguage = ( isset( $locParts[1] ) ? $locParts[1] : '' );
154
-			$locCurrency = ( isset( $locParts[2] ) ? $locParts[2] : '' );
151
+			$locParts = explode('|', $localeStr);
152
+			$locSite = (isset($locParts[0]) ? $locParts[0] : '');
153
+			$locLanguage = (isset($locParts[1]) ? $locParts[1] : '');
154
+			$locCurrency = (isset($locParts[2]) ? $locParts[2] : '');
155 155
 
156
-			$localeManager = \Aimeos\MShop\Factory::createManager( $context, 'locale' );
157
-			$locale = $localeManager->bootstrap( $locSite, $locLanguage, $locCurrency, false );
156
+			$localeManager = \Aimeos\MShop\Factory::createManager($context, 'locale');
157
+			$locale = $localeManager->bootstrap($locSite, $locLanguage, $locCurrency, false);
158 158
 
159 159
 			$context = clone $context;
160
-			$context->setLocale( $locale );
160
+			$context->setLocale($locale);
161 161
 
162
-			$manager = \Aimeos\MShop\Factory::createManager( $context, 'order/base' );
163
-			$basket = $manager->getSession( $type );
162
+			$manager = \Aimeos\MShop\Factory::createManager($context, 'order/base');
163
+			$basket = $manager->getSession($type);
164 164
 
165
-			$this->copyAddresses( $basket, $errors, $localeKey );
166
-			$this->copyServices( $basket, $errors );
167
-			$this->copyProducts( $basket, $errors, $localeKey );
168
-			$this->copyCoupons( $basket, $errors, $localeKey );
165
+			$this->copyAddresses($basket, $errors, $localeKey);
166
+			$this->copyServices($basket, $errors);
167
+			$this->copyProducts($basket, $errors, $localeKey);
168
+			$this->copyCoupons($basket, $errors, $localeKey);
169 169
 
170
-			$manager->setSession( $basket, $type );
170
+			$manager->setSession($basket, $type);
171 171
 		}
172 172
 
173
-		$session->set( 'aimeos/basket/locale', $localeKey );
173
+		$session->set('aimeos/basket/locale', $localeKey);
174 174
 	}
175 175
 
176 176
 
@@ -182,22 +182,22 @@  discard block
 block discarded – undo
182 182
 	 * @param string $localeKey Unique identifier of the site, language and currency
183 183
 	 * @return array Associative list of errors occured
184 184
 	 */
185
-	protected function copyAddresses( \Aimeos\MShop\Order\Item\Base\Iface $basket, array $errors, $localeKey )
185
+	protected function copyAddresses(\Aimeos\MShop\Order\Item\Base\Iface $basket, array $errors, $localeKey)
186 186
 	{
187
-		foreach( $basket->getAddresses() as $type => $item )
187
+		foreach ($basket->getAddresses() as $type => $item)
188 188
 		{
189 189
 			try
190 190
 			{
191
-				$this->setAddress( $type, $item->toArray() );
192
-				$basket->deleteAddress( $type );
191
+				$this->setAddress($type, $item->toArray());
192
+				$basket->deleteAddress($type);
193 193
 			}
194
-			catch( \Exception $e )
194
+			catch (\Exception $e)
195 195
 			{
196 196
 				$logger = $this->getContext()->getLogger();
197 197
 				$errors['address'][$type] = $e->getMessage();
198 198
 
199 199
 				$str = 'Error migrating address with type "%1$s" in basket to locale "%2$s": %3$s';
200
-				$logger->log( sprintf( $str, $type, $localeKey, $e->getMessage() ), \Aimeos\MW\Logger\Base::INFO );
200
+				$logger->log(sprintf($str, $type, $localeKey, $e->getMessage()), \Aimeos\MW\Logger\Base::INFO);
201 201
 			}
202 202
 		}
203 203
 
@@ -213,22 +213,22 @@  discard block
 block discarded – undo
213 213
 	 * @param string $localeKey Unique identifier of the site, language and currency
214 214
 	 * @return array Associative list of errors occured
215 215
 	 */
216
-	protected function copyCoupons( \Aimeos\MShop\Order\Item\Base\Iface $basket, array $errors, $localeKey )
216
+	protected function copyCoupons(\Aimeos\MShop\Order\Item\Base\Iface $basket, array $errors, $localeKey)
217 217
 	{
218
-		foreach( $basket->getCoupons() as $code => $list )
218
+		foreach ($basket->getCoupons() as $code => $list)
219 219
 		{
220 220
 			try
221 221
 			{
222
-				$this->addCoupon( $code );
223
-				$basket->deleteCoupon( $code, true );
222
+				$this->addCoupon($code);
223
+				$basket->deleteCoupon($code, true);
224 224
 			}
225
-			catch( \Exception $e )
225
+			catch (\Exception $e)
226 226
 			{
227 227
 				$logger = $this->getContext()->getLogger();
228 228
 				$errors['coupon'][$code] = $e->getMessage();
229 229
 
230 230
 				$str = 'Error migrating coupon with code "%1$s" in basket to locale "%2$s": %3$s';
231
-				$logger->log( sprintf( $str, $code, $localeKey, $e->getMessage() ), \Aimeos\MW\Logger\Base::INFO );
231
+				$logger->log(sprintf($str, $code, $localeKey, $e->getMessage()), \Aimeos\MW\Logger\Base::INFO);
232 232
 			}
233 233
 		}
234 234
 
@@ -244,11 +244,11 @@  discard block
 block discarded – undo
244 244
 	 * @param string $localeKey Unique identifier of the site, language and currency
245 245
 	 * @return array Associative list of errors occured
246 246
 	 */
247
-	protected function copyProducts( \Aimeos\MShop\Order\Item\Base\Iface $basket, array $errors, $localeKey )
247
+	protected function copyProducts(\Aimeos\MShop\Order\Item\Base\Iface $basket, array $errors, $localeKey)
248 248
 	{
249
-		foreach( $basket->getProducts() as $pos => $product )
249
+		foreach ($basket->getProducts() as $pos => $product)
250 250
 		{
251
-			if( $product->getFlags() & \Aimeos\MShop\Order\Item\Base\Product\Base::FLAG_IMMUTABLE ) {
251
+			if ($product->getFlags() & \Aimeos\MShop\Order\Item\Base\Product\Base::FLAG_IMMUTABLE) {
252 252
 				continue;
253 253
 			}
254 254
 
@@ -256,7 +256,7 @@  discard block
 block discarded – undo
256 256
 			{
257 257
 				$attrIds = [];
258 258
 
259
-				foreach( $product->getAttributes() as $attrItem ) {
259
+				foreach ($product->getAttributes() as $attrItem) {
260 260
 					$attrIds[$attrItem->getType()][] = $attrItem->getAttributeId();
261 261
 				}
262 262
 
@@ -264,22 +264,22 @@  discard block
 block discarded – undo
264 264
 					$product->getProductId(),
265 265
 					$product->getQuantity(),
266 266
 					$product->getStockType(),
267
-					$this->getValue( $attrIds, 'variant', [] ),
268
-					$this->getValue( $attrIds, 'config', [] ),
269
-					$this->getValue( $attrIds, 'hidden', [] ),
270
-					$this->getValue( $attrIds, 'custom', [] )
267
+					$this->getValue($attrIds, 'variant', []),
268
+					$this->getValue($attrIds, 'config', []),
269
+					$this->getValue($attrIds, 'hidden', []),
270
+					$this->getValue($attrIds, 'custom', [])
271 271
 				);
272 272
 
273
-				$basket->deleteProduct( $pos );
273
+				$basket->deleteProduct($pos);
274 274
 			}
275
-			catch( \Exception $e )
275
+			catch (\Exception $e)
276 276
 			{
277 277
 				$code = $product->getProductCode();
278 278
 				$logger = $this->getContext()->getLogger();
279 279
 				$errors['product'][$pos] = $e->getMessage();
280 280
 
281 281
 				$str = 'Error migrating product with code "%1$s" in basket to locale "%2$s": %3$s';
282
-				$logger->log( sprintf( $str, $code, $localeKey, $e->getMessage() ), \Aimeos\MW\Logger\Base::INFO );
282
+				$logger->log(sprintf($str, $code, $localeKey, $e->getMessage()), \Aimeos\MW\Logger\Base::INFO);
283 283
 			}
284 284
 		}
285 285
 
@@ -294,24 +294,24 @@  discard block
 block discarded – undo
294 294
 	 * @param array $errors Associative list of previous errors
295 295
 	 * @return array Associative list of errors occured
296 296
 	 */
297
-	protected function copyServices( \Aimeos\MShop\Order\Item\Base\Iface $basket, array $errors )
297
+	protected function copyServices(\Aimeos\MShop\Order\Item\Base\Iface $basket, array $errors)
298 298
 	{
299
-		foreach( $basket->getServices() as $type => $list )
299
+		foreach ($basket->getServices() as $type => $list)
300 300
 		{
301
-			foreach( $list as $item )
301
+			foreach ($list as $item)
302 302
 			{
303 303
 				try
304 304
 				{
305 305
 					$attributes = [];
306 306
 
307
-					foreach( $item->getAttributes() as $attrItem ) {
307
+					foreach ($item->getAttributes() as $attrItem) {
308 308
 						$attributes[$attrItem->getCode()] = $attrItem->getValue();
309 309
 					}
310 310
 
311
-					$this->addService( $type, $item->getServiceId(), $attributes );
312
-					$basket->deleteService( $type );
311
+					$this->addService($type, $item->getServiceId(), $attributes);
312
+					$basket->deleteService($type);
313 313
 				}
314
-				catch( \Exception $e ) { ; } // Don't notify the user as appropriate services can be added automatically
314
+				catch (\Exception $e) {; } // Don't notify the user as appropriate services can be added automatically
315 315
 			}
316 316
 		}
317 317
 
@@ -324,21 +324,21 @@  discard block
 block discarded – undo
324 324
 	 *
325 325
 	 * @param \Aimeos\MShop\Order\Item\Base\Iface $basket Basket object
326 326
 	 */
327
-	protected function createSubscriptions( \Aimeos\MShop\Order\Item\Base\Iface $basket )
327
+	protected function createSubscriptions(\Aimeos\MShop\Order\Item\Base\Iface $basket)
328 328
 	{
329
-		$manager = \Aimeos\MShop\Factory::createManager( $this->getContext(), 'subscription' );
329
+		$manager = \Aimeos\MShop\Factory::createManager($this->getContext(), 'subscription');
330 330
 
331
-		foreach( $basket->getProducts() as $orderProduct )
331
+		foreach ($basket->getProducts() as $orderProduct)
332 332
 		{
333
-			if( ( $interval = $orderProduct->getAttribute( 'interval', 'subscription' ) ) !== null )
333
+			if (($interval = $orderProduct->getAttribute('interval', 'subscription')) !== null)
334 334
 			{
335 335
 				$item = $manager->createItem();
336
-				$item->setOrderBaseId( $basket->getId() );
337
-				$item->setOrderProductId( $orderProduct->getId() );
338
-				$item->setInterval( $interval );
339
-				$item->setStatus( 1 );
336
+				$item->setOrderBaseId($basket->getId());
337
+				$item->setOrderProductId($orderProduct->getId());
338
+				$item->setInterval($interval);
339
+				$item->setStatus(1);
340 340
 
341
-				$manager->saveItem( $item, false );
341
+				$manager->saveItem($item, false);
342 342
 			}
343 343
 		}
344 344
 	}
@@ -352,32 +352,32 @@  discard block
 block discarded – undo
352 352
 	 * @return array List of items implementing \Aimeos\MShop\Attribute\Item\Iface
353 353
 	 * @throws \Aimeos\Controller\Frontend\Basket\Exception If the actual attribute number doesn't match the expected one
354 354
 	 */
355
-	protected function getAttributes( array $attributeIds, array $domains = array( 'price', 'text' ) )
355
+	protected function getAttributes(array $attributeIds, array $domains = array('price', 'text'))
356 356
 	{
357
-		if( empty( $attributeIds ) ) {
357
+		if (empty($attributeIds)) {
358 358
 			return [];
359 359
 		}
360 360
 
361
-		$attributeManager = \Aimeos\MShop\Factory::createManager( $this->getContext(), 'attribute' );
361
+		$attributeManager = \Aimeos\MShop\Factory::createManager($this->getContext(), 'attribute');
362 362
 
363
-		$search = $attributeManager->createSearch( true );
363
+		$search = $attributeManager->createSearch(true);
364 364
 		$expr = array(
365
-			$search->compare( '==', 'attribute.id', $attributeIds ),
365
+			$search->compare('==', 'attribute.id', $attributeIds),
366 366
 			$search->getConditions(),
367 367
 		);
368
-		$search->setConditions( $search->combine( '&&', $expr ) );
369
-		$search->setSlice( 0, 0x7fffffff );
368
+		$search->setConditions($search->combine('&&', $expr));
369
+		$search->setSlice(0, 0x7fffffff);
370 370
 
371
-		$attrItems = $attributeManager->searchItems( $search, $domains );
371
+		$attrItems = $attributeManager->searchItems($search, $domains);
372 372
 
373
-		if( count( $attrItems ) !== count( $attributeIds ) )
373
+		if (count($attrItems) !== count($attributeIds))
374 374
 		{
375 375
 			$i18n = $this->getContext()->getI18n();
376
-			$expected = implode( ',', $attributeIds );
377
-			$actual = implode( ',', array_keys( $attrItems ) );
378
-			$msg = $i18n->dt( 'controller/frontend', 'Available attribute IDs "%1$s" do not match the given attribute IDs "%2$s"' );
376
+			$expected = implode(',', $attributeIds);
377
+			$actual = implode(',', array_keys($attrItems));
378
+			$msg = $i18n->dt('controller/frontend', 'Available attribute IDs "%1$s" do not match the given attribute IDs "%2$s"');
379 379
 
380
-			throw new \Aimeos\Controller\Frontend\Basket\Exception( sprintf( $msg, $actual, $expected ) );
380
+			throw new \Aimeos\Controller\Frontend\Basket\Exception(sprintf($msg, $actual, $expected));
381 381
 		}
382 382
 
383 383
 		return $attrItems;
@@ -390,31 +390,31 @@  discard block
 block discarded – undo
390 390
 	 * @param \Aimeos\MShop\Order\Item\Base\Product\Attribute\Item[] $orderAttributes List of order product attribute items
391 391
 	 * @return \Aimeos\MShop\Attribute\Item\Iface[] Associative list of attribute IDs as key and attribute items as values
392 392
 	 */
393
-	protected function getAttributeItems( array $orderAttributes )
393
+	protected function getAttributeItems(array $orderAttributes)
394 394
 	{
395
-		if( empty( $orderAttributes ) ) {
395
+		if (empty($orderAttributes)) {
396 396
 			return [];
397 397
 		}
398 398
 
399
-		$attributeManager = \Aimeos\MShop\Factory::createManager( $this->getContext(), 'attribute' );
400
-		$search = $attributeManager->createSearch( true );
399
+		$attributeManager = \Aimeos\MShop\Factory::createManager($this->getContext(), 'attribute');
400
+		$search = $attributeManager->createSearch(true);
401 401
 		$expr = [];
402 402
 
403
-		foreach( $orderAttributes as $item )
403
+		foreach ($orderAttributes as $item)
404 404
 		{
405 405
 			$tmp = array(
406
-				$search->compare( '==', 'attribute.domain', 'product' ),
407
-				$search->compare( '==', 'attribute.code', $item->getValue() ),
408
-				$search->compare( '==', 'attribute.type.domain', 'product' ),
409
-				$search->compare( '==', 'attribute.type.code', $item->getCode() ),
410
-				$search->compare( '>', 'attribute.type.status', 0 ),
406
+				$search->compare('==', 'attribute.domain', 'product'),
407
+				$search->compare('==', 'attribute.code', $item->getValue()),
408
+				$search->compare('==', 'attribute.type.domain', 'product'),
409
+				$search->compare('==', 'attribute.type.code', $item->getCode()),
410
+				$search->compare('>', 'attribute.type.status', 0),
411 411
 				$search->getConditions(),
412 412
 			);
413
-			$expr[] = $search->combine( '&&', $tmp );
413
+			$expr[] = $search->combine('&&', $tmp);
414 414
 		}
415 415
 
416
-		$search->setConditions( $search->combine( '||', $expr ) );
417
-		return $attributeManager->searchItems( $search, array( 'price' ) );
416
+		$search->setConditions($search->combine('||', $expr));
417
+		return $attributeManager->searchItems($search, array('price'));
418 418
 	}
419 419
 
420 420
 
@@ -427,24 +427,24 @@  discard block
 block discarded – undo
427 427
 	 * @param array $quantities Associative list of attribute IDs as keys and their quantities as values
428 428
 	 * @return array List of items implementing \Aimeos\MShop\Order\Item\Product\Attribute\Iface
429 429
 	 */
430
-	protected function getOrderProductAttributes( $type, array $ids, array $values = [], array $quantities = [] )
430
+	protected function getOrderProductAttributes($type, array $ids, array $values = [], array $quantities = [])
431 431
 	{
432
-		if( empty( $ids ) ) {
432
+		if (empty($ids)) {
433 433
 			return [];
434 434
 		}
435 435
 
436 436
 		$list = [];
437
-		$manager = \Aimeos\MShop\Factory::createManager( $this->getContext(), 'order/base/product/attribute' );
437
+		$manager = \Aimeos\MShop\Factory::createManager($this->getContext(), 'order/base/product/attribute');
438 438
 
439
-		foreach( $this->getAttributes( $ids ) as $id => $attrItem )
439
+		foreach ($this->getAttributes($ids) as $id => $attrItem)
440 440
 		{
441 441
 			$item = $manager->createItem();
442
-			$item->copyFrom( $attrItem );
443
-			$item->setType( $type );
444
-			$item->setQuantity( isset( $quantities[$id] ) ? $quantities[$id] : 1 );
442
+			$item->copyFrom($attrItem);
443
+			$item->setType($type);
444
+			$item->setQuantity(isset($quantities[$id]) ? $quantities[$id] : 1);
445 445
 
446
-			if( isset( $values[$id] ) ) {
447
-				$item->setValue( $values[$id] );
446
+			if (isset($values[$id])) {
447
+				$item->setValue($values[$id]);
448 448
 			}
449 449
 
450 450
 			$list[] = $item;
@@ -461,23 +461,23 @@  discard block
 block discarded – undo
461 461
 	 * @param string $code Code of the list type
462 462
 	 * @return \Aimeos\MShop\Common\Item\Type\Iface List type item
463 463
 	 */
464
-	protected function getProductListTypeItem( $domain, $code )
464
+	protected function getProductListTypeItem($domain, $code)
465 465
 	{
466 466
 		$context = $this->getContext();
467 467
 
468
-		if( empty( $this->listTypeItems ) )
468
+		if (empty($this->listTypeItems))
469 469
 		{
470
-			$manager = \Aimeos\MShop\Factory::createManager( $context, 'product/lists/type' );
470
+			$manager = \Aimeos\MShop\Factory::createManager($context, 'product/lists/type');
471 471
 
472
-			foreach( $manager->searchItems( $manager->createSearch( true ) ) as $item ) {
473
-				$this->listTypeItems[ $item->getDomain() ][ $item->getCode() ] = $item;
472
+			foreach ($manager->searchItems($manager->createSearch(true)) as $item) {
473
+				$this->listTypeItems[$item->getDomain()][$item->getCode()] = $item;
474 474
 			}
475 475
 		}
476 476
 
477
-		if( !isset( $this->listTypeItems[$domain][$code] ) )
477
+		if (!isset($this->listTypeItems[$domain][$code]))
478 478
 		{
479
-			$msg = $context->getI18n()->dt( 'controller/frontend', 'List type for domain "%1$s" and code "%2$s" not found' );
480
-			throw new \Aimeos\Controller\Frontend\Basket\Exception( sprintf( $msg, $domain, $code ) );
479
+			$msg = $context->getI18n()->dt('controller/frontend', 'List type for domain "%1$s" and code "%2$s" not found');
480
+			throw new \Aimeos\Controller\Frontend\Basket\Exception(sprintf($msg, $domain, $code));
481 481
 		}
482 482
 
483 483
 		return $this->listTypeItems[$domain][$code];
@@ -492,43 +492,43 @@  discard block
 block discarded – undo
492 492
 	 * @param array $domains Names of the domain items that should be fetched too
493 493
 	 * @return array List of products matching the given attributes
494 494
 	 */
495
-	protected function getProductVariants( \Aimeos\MShop\Product\Item\Iface $productItem, array $variantAttributeIds,
496
-			array $domains = array( 'attribute', 'media', 'price', 'text' ) )
495
+	protected function getProductVariants(\Aimeos\MShop\Product\Item\Iface $productItem, array $variantAttributeIds,
496
+			array $domains = array('attribute', 'media', 'price', 'text'))
497 497
 	{
498 498
 		$subProductIds = [];
499
-		foreach( $productItem->getRefItems( 'product', 'default', 'default' ) as $item ) {
499
+		foreach ($productItem->getRefItems('product', 'default', 'default') as $item) {
500 500
 			$subProductIds[] = $item->getId();
501 501
 		}
502 502
 
503
-		if( count( $subProductIds ) === 0 ) {
503
+		if (count($subProductIds) === 0) {
504 504
 			return [];
505 505
 		}
506 506
 
507
-		$productManager = \Aimeos\MShop\Factory::createManager( $this->getContext(), 'product' );
508
-		$search = $productManager->createSearch( true );
507
+		$productManager = \Aimeos\MShop\Factory::createManager($this->getContext(), 'product');
508
+		$search = $productManager->createSearch(true);
509 509
 
510 510
 		$expr = array(
511
-			$search->compare( '==', 'product.id', $subProductIds ),
511
+			$search->compare('==', 'product.id', $subProductIds),
512 512
 			$search->getConditions(),
513 513
 		);
514 514
 
515
-		if( count( $variantAttributeIds ) > 0 )
515
+		if (count($variantAttributeIds) > 0)
516 516
 		{
517
-			foreach( $variantAttributeIds as $key => $id ) {
517
+			foreach ($variantAttributeIds as $key => $id) {
518 518
 				$variantAttributeIds[$key] = (string) $id;
519 519
 			}
520 520
 
521
-			$listTypeItem = $this->getProductListTypeItem( 'attribute', 'variant' );
521
+			$listTypeItem = $this->getProductListTypeItem('attribute', 'variant');
522 522
 
523
-			$param = array( 'attribute', $listTypeItem->getId(), $variantAttributeIds );
524
-			$cmpfunc = $search->createFunction( 'product.contains', $param );
523
+			$param = array('attribute', $listTypeItem->getId(), $variantAttributeIds);
524
+			$cmpfunc = $search->createFunction('product.contains', $param);
525 525
 
526
-			$expr[] = $search->compare( '==', $cmpfunc, count( $variantAttributeIds ) );
526
+			$expr[] = $search->compare('==', $cmpfunc, count($variantAttributeIds));
527 527
 		}
528 528
 
529
-		$search->setConditions( $search->combine( '&&', $expr ) );
529
+		$search->setConditions($search->combine('&&', $expr));
530 530
 
531
-		return $productManager->searchItems( $search, $domains );
531
+		return $productManager->searchItems($search, $domains);
532 532
 	}
533 533
 
534 534
 
@@ -540,9 +540,9 @@  discard block
 block discarded – undo
540 540
 	 * @param mixed $default Default value if no value is available for the given name
541 541
 	 * @return mixed Value from the array or default value
542 542
 	 */
543
-	protected function getValue( array $values, $name, $default = null )
543
+	protected function getValue(array $values, $name, $default = null)
544 544
 	{
545
-		if( isset( $values[$name] ) ) {
545
+		if (isset($values[$name])) {
546 546
 			return $values[$name];
547 547
 		}
548 548
 
Please login to merge, or discard this patch.
controller/frontend/tests/Controller/Frontend/Basket/BaseTest.php 1 patch
Spacing   +138 added lines, -138 removed lines patch added patch discarded remove patch
@@ -16,53 +16,53 @@  discard block
 block discarded – undo
16 16
 	protected function setUp()
17 17
 	{
18 18
 		$this->context = \TestHelperFrontend::getContext();
19
-		\Aimeos\MShop\Factory::setCache( true );
19
+		\Aimeos\MShop\Factory::setCache(true);
20 20
 	}
21 21
 
22 22
 
23 23
 	protected function tearDown()
24 24
 	{
25
-		\Aimeos\MShop\Factory::setCache( false );
26
-		$this->context->getSession()->set( 'aimeos', [] );
25
+		\Aimeos\MShop\Factory::setCache(false);
26
+		$this->context->getSession()->set('aimeos', []);
27 27
 
28
-		unset( $this->context );
28
+		unset($this->context);
29 29
 	}
30 30
 
31 31
 
32 32
 	public function testCheckLocale()
33 33
 	{
34
-		$object = $this->getMockBuilder( '\Aimeos\Controller\Frontend\Basket\Standard' )
35
-			->setConstructorArgs( [$this->context] )
36
-			->setMethods( null )
34
+		$object = $this->getMockBuilder('\Aimeos\Controller\Frontend\Basket\Standard')
35
+			->setConstructorArgs([$this->context])
36
+			->setMethods(null)
37 37
 			->getMock();
38 38
 
39
-		$this->context->getSession()->set( 'aimeos/basket/locale', 'unittest|en|EUR' );
40
-		$this->access( 'checkLocale' )->invokeArgs( $object, [$this->context->getLocale(), 'unittest'] );
39
+		$this->context->getSession()->set('aimeos/basket/locale', 'unittest|en|EUR');
40
+		$this->access('checkLocale')->invokeArgs($object, [$this->context->getLocale(), 'unittest']);
41 41
 	}
42 42
 
43 43
 
44 44
 	public function testCopyAddresses()
45 45
 	{
46
-		$manager = \Aimeos\MShop\Factory::createManager( \TestHelperFrontend::getContext(), 'order/base' );
46
+		$manager = \Aimeos\MShop\Factory::createManager(\TestHelperFrontend::getContext(), 'order/base');
47 47
 		$ordBaseItem = $manager->createItem();
48 48
 
49
-		$address = $this->getAddress( 'Example company' );
50
-		$ordBaseItem->setAddress( $address, \Aimeos\MShop\Order\Item\Base\Address\Base::TYPE_PAYMENT );
49
+		$address = $this->getAddress('Example company');
50
+		$ordBaseItem->setAddress($address, \Aimeos\MShop\Order\Item\Base\Address\Base::TYPE_PAYMENT);
51 51
 
52 52
 
53
-		$object = $this->getMockBuilder( '\Aimeos\Controller\Frontend\Basket\Standard' )
54
-			->setConstructorArgs( [$this->context] )
55
-			->setMethods( null )
53
+		$object = $this->getMockBuilder('\Aimeos\Controller\Frontend\Basket\Standard')
54
+			->setConstructorArgs([$this->context])
55
+			->setMethods(null)
56 56
 			->getMock();
57 57
 
58
-		$result = $this->access( 'copyAddresses' )->invokeArgs( $object, [$ordBaseItem, ['test'], 'unittest|en|EUR'] );
58
+		$result = $this->access('copyAddresses')->invokeArgs($object, [$ordBaseItem, ['test'], 'unittest|en|EUR']);
59 59
 
60 60
 
61
-		$this->assertEquals( ['test'], $result );
62
-		$this->assertEquals( 1, count( $object->get()->getAddresses() ) );
61
+		$this->assertEquals(['test'], $result);
62
+		$this->assertEquals(1, count($object->get()->getAddresses()));
63 63
 
64
-		$addr = $object->get()->getAddress( \Aimeos\MShop\Order\Item\Base\Address\Base::TYPE_PAYMENT );
65
-		$this->assertInstanceOf( '\Aimeos\MShop\Order\Item\Base\Address\Iface', $addr );
64
+		$addr = $object->get()->getAddress(\Aimeos\MShop\Order\Item\Base\Address\Base::TYPE_PAYMENT);
65
+		$this->assertInstanceOf('\Aimeos\MShop\Order\Item\Base\Address\Iface', $addr);
66 66
 
67 67
 		$object->clear();
68 68
 	}
@@ -70,54 +70,54 @@  discard block
 block discarded – undo
70 70
 
71 71
 	public function testCopyAddressesException()
72 72
 	{
73
-		$manager = \Aimeos\MShop\Factory::createManager( \TestHelperFrontend::getContext(), 'order/base' );
73
+		$manager = \Aimeos\MShop\Factory::createManager(\TestHelperFrontend::getContext(), 'order/base');
74 74
 		$ordBaseItem = $manager->createItem();
75 75
 
76
-		$address = $this->getAddress( 'Example company' );
77
-		$ordBaseItem->setAddress( $address, \Aimeos\MShop\Order\Item\Base\Address\Base::TYPE_PAYMENT );
76
+		$address = $this->getAddress('Example company');
77
+		$ordBaseItem->setAddress($address, \Aimeos\MShop\Order\Item\Base\Address\Base::TYPE_PAYMENT);
78 78
 
79 79
 
80
-		$object = $this->getMockBuilder( '\Aimeos\Controller\Frontend\Basket\Standard' )
81
-			->setConstructorArgs( [$this->context] )
82
-			->setMethods( ['setAddress'] )
80
+		$object = $this->getMockBuilder('\Aimeos\Controller\Frontend\Basket\Standard')
81
+			->setConstructorArgs([$this->context])
82
+			->setMethods(['setAddress'])
83 83
 			->getMock();
84 84
 
85
-		$object->expects( $this->once() )->method( 'setAddress' )->will( $this->throwException( new \Exception() ) );
85
+		$object->expects($this->once())->method('setAddress')->will($this->throwException(new \Exception()));
86 86
 
87
-		$result = $this->access( 'copyAddresses' )->invokeArgs( $object, [$ordBaseItem, [], 'unittest|en|EUR'] );
87
+		$result = $this->access('copyAddresses')->invokeArgs($object, [$ordBaseItem, [], 'unittest|en|EUR']);
88 88
 
89
-		$this->assertEquals( 1, count( $result ) );
90
-		$this->assertArrayHasKey( 'address', $result );
89
+		$this->assertEquals(1, count($result));
90
+		$this->assertArrayHasKey('address', $result);
91 91
 	}
92 92
 
93 93
 
94 94
 	public function testCopyCoupon()
95 95
 	{
96
-		$manager = \Aimeos\MShop\Factory::createManager( \TestHelperFrontend::getContext(), 'order/base' );
96
+		$manager = \Aimeos\MShop\Factory::createManager(\TestHelperFrontend::getContext(), 'order/base');
97 97
 		$ordBaseItem = $manager->createItem();
98 98
 
99
-		$product = \Aimeos\MShop\Factory::createManager( $this->context, 'product' )->findItem( 'CNC', ['price'] );
100
-		$ordProdManager = \Aimeos\MShop\Factory::createManager( $this->context, 'order/base/product' );
101
-		$ordProdItem = $ordProdManager->createItem()->copyFrom( $product );
99
+		$product = \Aimeos\MShop\Factory::createManager($this->context, 'product')->findItem('CNC', ['price']);
100
+		$ordProdManager = \Aimeos\MShop\Factory::createManager($this->context, 'order/base/product');
101
+		$ordProdItem = $ordProdManager->createItem()->copyFrom($product);
102 102
 
103
-		$priceItems = $product->getRefItems( 'price' );
104
-		$ordProdItem->setPrice( reset( $priceItems ) );
103
+		$priceItems = $product->getRefItems('price');
104
+		$ordProdItem->setPrice(reset($priceItems));
105 105
 
106
-		$ordBaseItem->addProduct( $ordProdItem );
107
-		$ordBaseItem->addCoupon( 'OPQR', [] );
106
+		$ordBaseItem->addProduct($ordProdItem);
107
+		$ordBaseItem->addCoupon('OPQR', []);
108 108
 
109 109
 
110
-		$object = $this->getMockBuilder( '\Aimeos\Controller\Frontend\Basket\Standard' )
111
-			->setConstructorArgs( [$this->context] )
112
-			->setMethods( null )
110
+		$object = $this->getMockBuilder('\Aimeos\Controller\Frontend\Basket\Standard')
111
+			->setConstructorArgs([$this->context])
112
+			->setMethods(null)
113 113
 			->getMock();
114 114
 
115
-		$object->addProduct( $product->getId() );
115
+		$object->addProduct($product->getId());
116 116
 
117
-		$result = $this->access( 'copyCoupons' )->invokeArgs( $object, [$ordBaseItem, ['test'], 'unittest|en|EUR'] );
117
+		$result = $this->access('copyCoupons')->invokeArgs($object, [$ordBaseItem, ['test'], 'unittest|en|EUR']);
118 118
 
119 119
 
120
-		$this->assertEquals( ['test'], $result );
120
+		$this->assertEquals(['test'], $result);
121 121
 
122 122
 		$object->clear();
123 123
 	}
@@ -125,48 +125,48 @@  discard block
 block discarded – undo
125 125
 
126 126
 	public function testCopyCouponException()
127 127
 	{
128
-		$manager = \Aimeos\MShop\Factory::createManager( \TestHelperFrontend::getContext(), 'order/base' );
128
+		$manager = \Aimeos\MShop\Factory::createManager(\TestHelperFrontend::getContext(), 'order/base');
129 129
 		$ordBaseItem = $manager->createItem();
130 130
 
131
-		$ordBaseItem->addCoupon( '90AB', [] );
131
+		$ordBaseItem->addCoupon('90AB', []);
132 132
 
133 133
 
134
-		$object = $this->getMockBuilder( '\Aimeos\Controller\Frontend\Basket\Standard' )
135
-			->setConstructorArgs( [$this->context] )
136
-			->setMethods( ['addCoupon'] )
134
+		$object = $this->getMockBuilder('\Aimeos\Controller\Frontend\Basket\Standard')
135
+			->setConstructorArgs([$this->context])
136
+			->setMethods(['addCoupon'])
137 137
 			->getMock();
138 138
 
139
-		$object->expects( $this->once() )->method( 'addCoupon' )->will( $this->throwException( new \Exception() ) );
139
+		$object->expects($this->once())->method('addCoupon')->will($this->throwException(new \Exception()));
140 140
 
141
-		$result = $this->access( 'copyCoupons' )->invokeArgs( $object, [$ordBaseItem, [], 'unittest|en|EUR'] );
141
+		$result = $this->access('copyCoupons')->invokeArgs($object, [$ordBaseItem, [], 'unittest|en|EUR']);
142 142
 
143
-		$this->assertEquals( 1, count( $result ) );
144
-		$this->assertArrayHasKey( 'coupon', $result );
143
+		$this->assertEquals(1, count($result));
144
+		$this->assertArrayHasKey('coupon', $result);
145 145
 	}
146 146
 
147 147
 
148 148
 	public function testCopyProduct()
149 149
 	{
150
-		$manager = \Aimeos\MShop\Factory::createManager( \TestHelperFrontend::getContext(), 'order/base' );
150
+		$manager = \Aimeos\MShop\Factory::createManager(\TestHelperFrontend::getContext(), 'order/base');
151 151
 		$ordBaseItem = $manager->createItem();
152 152
 
153
-		$product = \Aimeos\MShop\Factory::createManager( $this->context, 'product' )->findItem( 'CNC' );
154
-		$ordProdManager = \Aimeos\MShop\Factory::createManager( $this->context, 'order/base/product' );
155
-		$ordProdItem = $ordProdManager->createItem()->copyFrom( $product );
156
-		$ordBaseItem->addProduct( $ordProdItem );
153
+		$product = \Aimeos\MShop\Factory::createManager($this->context, 'product')->findItem('CNC');
154
+		$ordProdManager = \Aimeos\MShop\Factory::createManager($this->context, 'order/base/product');
155
+		$ordProdItem = $ordProdManager->createItem()->copyFrom($product);
156
+		$ordBaseItem->addProduct($ordProdItem);
157 157
 
158 158
 
159
-		$object = $this->getMockBuilder( '\Aimeos\Controller\Frontend\Basket\Standard' )
160
-			->setConstructorArgs( [$this->context] )
161
-			->setMethods( null )
159
+		$object = $this->getMockBuilder('\Aimeos\Controller\Frontend\Basket\Standard')
160
+			->setConstructorArgs([$this->context])
161
+			->setMethods(null)
162 162
 			->getMock();
163 163
 
164
-		$result = $this->access( 'copyProducts' )->invokeArgs( $object, [$ordBaseItem, ['test'], 'unittest|en|EUR'] );
164
+		$result = $this->access('copyProducts')->invokeArgs($object, [$ordBaseItem, ['test'], 'unittest|en|EUR']);
165 165
 
166 166
 
167
-		$this->assertEquals( ['test'], $result );
168
-		$this->assertEquals( 1, count( $object->get()->getProducts() ) );
169
-		$this->assertInstanceOf( '\Aimeos\MShop\Order\Item\Base\Product\Iface', $object->get()->getProduct( 0 ) );
167
+		$this->assertEquals(['test'], $result);
168
+		$this->assertEquals(1, count($object->get()->getProducts()));
169
+		$this->assertInstanceOf('\Aimeos\MShop\Order\Item\Base\Product\Iface', $object->get()->getProduct(0));
170 170
 
171 171
 		$object->clear();
172 172
 	}
@@ -174,58 +174,58 @@  discard block
 block discarded – undo
174 174
 
175 175
 	public function testCopyProductException()
176 176
 	{
177
-		$manager = \Aimeos\MShop\Factory::createManager( \TestHelperFrontend::getContext(), 'order/base' );
177
+		$manager = \Aimeos\MShop\Factory::createManager(\TestHelperFrontend::getContext(), 'order/base');
178 178
 		$ordBaseItem = $manager->createItem();
179 179
 
180
-		$product = \Aimeos\MShop\Factory::createManager( $this->context, 'product' )->findItem( 'CNC' );
181
-		$ordProdManager = \Aimeos\MShop\Factory::createManager( $this->context, 'order/base/product' );
182
-		$ordProdItem = $ordProdManager->createItem()->copyFrom( $product );
183
-		$ordBaseItem->addProduct( $ordProdItem );
180
+		$product = \Aimeos\MShop\Factory::createManager($this->context, 'product')->findItem('CNC');
181
+		$ordProdManager = \Aimeos\MShop\Factory::createManager($this->context, 'order/base/product');
182
+		$ordProdItem = $ordProdManager->createItem()->copyFrom($product);
183
+		$ordBaseItem->addProduct($ordProdItem);
184 184
 
185 185
 
186
-		$object = $this->getMockBuilder( '\Aimeos\Controller\Frontend\Basket\Standard' )
187
-			->setConstructorArgs( [$this->context] )
188
-			->setMethods( ['addProduct'] )
186
+		$object = $this->getMockBuilder('\Aimeos\Controller\Frontend\Basket\Standard')
187
+			->setConstructorArgs([$this->context])
188
+			->setMethods(['addProduct'])
189 189
 			->getMock();
190 190
 
191
-		$object->expects( $this->once() )->method( 'addProduct' )->will( $this->throwException( new \Exception() ) );
191
+		$object->expects($this->once())->method('addProduct')->will($this->throwException(new \Exception()));
192 192
 
193
-		$result = $this->access( 'copyProducts' )->invokeArgs( $object, [$ordBaseItem, [], 'unittest|en|EUR'] );
193
+		$result = $this->access('copyProducts')->invokeArgs($object, [$ordBaseItem, [], 'unittest|en|EUR']);
194 194
 
195
-		$this->assertEquals( 1, count( $result ) );
196
-		$this->assertArrayHasKey( 'product', $result );
195
+		$this->assertEquals(1, count($result));
196
+		$this->assertArrayHasKey('product', $result);
197 197
 	}
198 198
 
199 199
 
200 200
 	public function testCopyServices()
201 201
 	{
202
-		$manager = \Aimeos\MShop\Factory::createManager( \TestHelperFrontend::getContext(), 'order/base' );
202
+		$manager = \Aimeos\MShop\Factory::createManager(\TestHelperFrontend::getContext(), 'order/base');
203 203
 		$ordBaseItem = $manager->createItem();
204 204
 
205
-		$serviceManager = \Aimeos\MShop\Factory::createManager( $this->context, 'service' );
206
-		$ordServManager = \Aimeos\MShop\Factory::createManager( $this->context, 'order/base/service' );
205
+		$serviceManager = \Aimeos\MShop\Factory::createManager($this->context, 'service');
206
+		$ordServManager = \Aimeos\MShop\Factory::createManager($this->context, 'order/base/service');
207 207
 
208
-		$serviceItem = $serviceManager->findItem( 'unitcode', [], 'service', 'delivery' );
209
-		$ordServItem = $ordServManager->createItem()->copyFrom( $serviceItem );
208
+		$serviceItem = $serviceManager->findItem('unitcode', [], 'service', 'delivery');
209
+		$ordServItem = $ordServManager->createItem()->copyFrom($serviceItem);
210 210
 
211
-		$ordBaseItem->addService( $ordServItem, \Aimeos\MShop\Order\Item\Base\Service\Base::TYPE_DELIVERY );
211
+		$ordBaseItem->addService($ordServItem, \Aimeos\MShop\Order\Item\Base\Service\Base::TYPE_DELIVERY);
212 212
 
213 213
 
214
-		$object = $this->getMockBuilder( '\Aimeos\Controller\Frontend\Basket\Standard' )
215
-			->setConstructorArgs( [$this->context] )
216
-			->setMethods( null )
214
+		$object = $this->getMockBuilder('\Aimeos\Controller\Frontend\Basket\Standard')
215
+			->setConstructorArgs([$this->context])
216
+			->setMethods(null)
217 217
 			->getMock();
218 218
 
219
-		$result = $this->access( 'copyServices' )->invokeArgs( $object, [$ordBaseItem, ['test'], 'unittest|en|EUR'] );
219
+		$result = $this->access('copyServices')->invokeArgs($object, [$ordBaseItem, ['test'], 'unittest|en|EUR']);
220 220
 
221 221
 
222
-		$this->assertEquals( ['test'], $result );
223
-		$this->assertEquals( 1, count( $object->get()->getServices() ) );
222
+		$this->assertEquals(['test'], $result);
223
+		$this->assertEquals(1, count($object->get()->getServices()));
224 224
 
225
-		$services = $object->get()->getService( \Aimeos\MShop\Order\Item\Base\Service\Base::TYPE_DELIVERY );
225
+		$services = $object->get()->getService(\Aimeos\MShop\Order\Item\Base\Service\Base::TYPE_DELIVERY);
226 226
 
227
-		foreach( $services as $service ) {
228
-			$this->assertInstanceOf( '\Aimeos\MShop\Order\Item\Base\Service\Iface', $service );
227
+		foreach ($services as $service) {
228
+			$this->assertInstanceOf('\Aimeos\MShop\Order\Item\Base\Service\Iface', $service);
229 229
 		}
230 230
 
231 231
 		$object->clear();
@@ -234,106 +234,106 @@  discard block
 block discarded – undo
234 234
 
235 235
 	public function testCopyServicesException()
236 236
 	{
237
-		$manager = \Aimeos\MShop\Factory::createManager( \TestHelperFrontend::getContext(), 'order/base' );
237
+		$manager = \Aimeos\MShop\Factory::createManager(\TestHelperFrontend::getContext(), 'order/base');
238 238
 		$ordBaseItem = $manager->createItem();
239 239
 
240
-		$serviceManager = \Aimeos\MShop\Factory::createManager( $this->context, 'service' );
241
-		$ordServManager = \Aimeos\MShop\Factory::createManager( $this->context, 'order/base/service' );
240
+		$serviceManager = \Aimeos\MShop\Factory::createManager($this->context, 'service');
241
+		$ordServManager = \Aimeos\MShop\Factory::createManager($this->context, 'order/base/service');
242 242
 
243
-		$serviceItem = $serviceManager->findItem( 'unitcode', [], 'service', 'delivery' );
244
-		$ordServItem = $ordServManager->createItem()->copyFrom( $serviceItem );
243
+		$serviceItem = $serviceManager->findItem('unitcode', [], 'service', 'delivery');
244
+		$ordServItem = $ordServManager->createItem()->copyFrom($serviceItem);
245 245
 
246
-		$ordBaseItem->addService( $ordServItem, \Aimeos\MShop\Order\Item\Base\Service\Base::TYPE_DELIVERY );
246
+		$ordBaseItem->addService($ordServItem, \Aimeos\MShop\Order\Item\Base\Service\Base::TYPE_DELIVERY);
247 247
 
248 248
 
249
-		$object = $this->getMockBuilder( '\Aimeos\Controller\Frontend\Basket\Standard' )
250
-			->setConstructorArgs( [$this->context] )
251
-			->setMethods( ['addService'] )
249
+		$object = $this->getMockBuilder('\Aimeos\Controller\Frontend\Basket\Standard')
250
+			->setConstructorArgs([$this->context])
251
+			->setMethods(['addService'])
252 252
 			->getMock();
253 253
 
254
-		$object->expects( $this->once() )->method( 'addService' )->will( $this->throwException( new \Exception() ) );
254
+		$object->expects($this->once())->method('addService')->will($this->throwException(new \Exception()));
255 255
 
256
-		$result = $this->access( 'copyServices' )->invokeArgs( $object, [$ordBaseItem, [], 'unittest|en|EUR'] );
256
+		$result = $this->access('copyServices')->invokeArgs($object, [$ordBaseItem, [], 'unittest|en|EUR']);
257 257
 
258
-		$this->assertEquals( 0, count( $result ) );
258
+		$this->assertEquals(0, count($result));
259 259
 	}
260 260
 
261 261
 
262 262
 	public function testCreateSubscriptions()
263 263
 	{
264
-		$baseManager = \Aimeos\MShop\Factory::createManager( $this->context, 'order/base' );
264
+		$baseManager = \Aimeos\MShop\Factory::createManager($this->context, 'order/base');
265 265
 
266 266
 		$search = $baseManager->createSearch();
267
-		$search->setConditions( $search->compare( '==', 'order.base.price', '53.50' ) );
267
+		$search->setConditions($search->compare('==', 'order.base.price', '53.50'));
268 268
 
269
-		$items = $baseManager->searchItems( $search, ['order/base/product'] );
269
+		$items = $baseManager->searchItems($search, ['order/base/product']);
270 270
 
271
-		if( ( $basket = reset( $items ) ) === false ) {
272
-			throw new \Exception( sprintf( 'No order base item found for price "%1$s"', '53,50' ) );
271
+		if (($basket = reset($items)) === false) {
272
+			throw new \Exception(sprintf('No order base item found for price "%1$s"', '53,50'));
273 273
 		}
274 274
 
275
-		$object = $this->getMockBuilder( '\Aimeos\Controller\Frontend\Basket\Standard' )
276
-			->setConstructorArgs( [$this->context] )
277
-			->setMethods( ['getAttributes'] )
275
+		$object = $this->getMockBuilder('\Aimeos\Controller\Frontend\Basket\Standard')
276
+			->setConstructorArgs([$this->context])
277
+			->setMethods(['getAttributes'])
278 278
 			->getMock();
279 279
 
280
-		$stub = $this->getMockBuilder( '\Aimeos\MShop\Subscription\Manager\Standard' )
281
-			->setConstructorArgs( [$this->context] )
282
-			->setMethods( ['saveItem'] )
280
+		$stub = $this->getMockBuilder('\Aimeos\MShop\Subscription\Manager\Standard')
281
+			->setConstructorArgs([$this->context])
282
+			->setMethods(['saveItem'])
283 283
 			->getMock();
284 284
 
285
-		\Aimeos\MShop\Factory::injectManager( $this->context, 'subscription', $stub );
285
+		\Aimeos\MShop\Factory::injectManager($this->context, 'subscription', $stub);
286 286
 
287
-		$stub->expects( $this->exactly( 2 ) )->method( 'saveItem' );
287
+		$stub->expects($this->exactly(2))->method('saveItem');
288 288
 
289
-		$this->access( 'createSubscriptions' )->invokeArgs( $object, [$basket] );
289
+		$this->access('createSubscriptions')->invokeArgs($object, [$basket]);
290 290
 	}
291 291
 
292 292
 
293 293
 	public function testGetOrderProductAttributes()
294 294
 	{
295
-		$object = $this->getMockBuilder( '\Aimeos\Controller\Frontend\Basket\Standard' )
296
-			->setConstructorArgs( [$this->context] )
297
-			->setMethods( ['getAttributes'] )
295
+		$object = $this->getMockBuilder('\Aimeos\Controller\Frontend\Basket\Standard')
296
+			->setConstructorArgs([$this->context])
297
+			->setMethods(['getAttributes'])
298 298
 			->getMock();
299 299
 
300
-		$list = [1 => new \Aimeos\MShop\Attribute\Item\Standard( ['attribute.code' => 'special_instructions'] )];
301
-		$object->expects( $this->once() )->method( 'getAttributes' )->will( $this->returnValue( $list ) );
300
+		$list = [1 => new \Aimeos\MShop\Attribute\Item\Standard(['attribute.code' => 'special_instructions'])];
301
+		$object->expects($this->once())->method('getAttributes')->will($this->returnValue($list));
302 302
 
303
-		$result = $this->access( 'getOrderProductAttributes' )->invokeArgs( $object, ['test', ['1'], ['1' => 'test']] );
303
+		$result = $this->access('getOrderProductAttributes')->invokeArgs($object, ['test', ['1'], ['1' => 'test']]);
304 304
 
305
-		$this->assertEquals( 1, count( $result ) );
306
-		$this->assertEquals( 'test', $result[0]->getValue() );
305
+		$this->assertEquals(1, count($result));
306
+		$this->assertEquals('test', $result[0]->getValue());
307 307
 	}
308 308
 
309 309
 
310 310
 	/**
311 311
 	 * @param string $company
312 312
 	 */
313
-	protected function getAddress( $company )
313
+	protected function getAddress($company)
314 314
 	{
315
-		$manager = \Aimeos\MShop\Factory::createManager( \TestHelperFrontend::getContext(), 'customer/address' );
315
+		$manager = \Aimeos\MShop\Factory::createManager(\TestHelperFrontend::getContext(), 'customer/address');
316 316
 
317 317
 		$search = $manager->createSearch();
318
-		$search->setConditions( $search->compare( '==', 'customer.address.company', $company ) );
319
-		$items = $manager->searchItems( $search );
318
+		$search->setConditions($search->compare('==', 'customer.address.company', $company));
319
+		$items = $manager->searchItems($search);
320 320
 
321
-		if( ( $item = reset( $items ) ) === false ) {
322
-			throw new \RuntimeException( sprintf( 'No address item with company "%1$s" found', $company ) );
321
+		if (($item = reset($items)) === false) {
322
+			throw new \RuntimeException(sprintf('No address item with company "%1$s" found', $company));
323 323
 		}
324 324
 
325
-		$ordAddrManager = \Aimeos\MShop\Factory::createManager( \TestHelperFrontend::getContext(), 'order/base/address' );
326
-		$ordAddrItem = $ordAddrManager->createItem()->copyFrom( $item );
325
+		$ordAddrManager = \Aimeos\MShop\Factory::createManager(\TestHelperFrontend::getContext(), 'order/base/address');
326
+		$ordAddrItem = $ordAddrManager->createItem()->copyFrom($item);
327 327
 
328 328
 		return $ordAddrItem;
329 329
 	}
330 330
 
331 331
 
332
-	protected function access( $name )
332
+	protected function access($name)
333 333
 	{
334
-		$class = new \ReflectionClass( '\Aimeos\Controller\Frontend\Basket\Base' );
335
-		$method = $class->getMethod( $name );
336
-		$method->setAccessible( true );
334
+		$class = new \ReflectionClass('\Aimeos\Controller\Frontend\Basket\Base');
335
+		$method = $class->getMethod($name);
336
+		$method->setAccessible(true);
337 337
 
338 338
 		return $method;
339 339
 	}
Please login to merge, or discard this patch.