@@ -33,11 +33,11 @@ discard block |
||
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 |
||
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 |
||
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 |
||
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 |
||
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 |
||
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 |
||
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 |
||
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 |
||
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 |
||
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 |
||
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 |
||
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 |
||
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 |
||
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 |
||
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 |
||
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 |
||
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 |
||
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 | } |
@@ -31,52 +31,52 @@ discard block |
||
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 |
||
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 |
||
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 |
||
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 |
||
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 |
||
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 |
||
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 |
||
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 |
||
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 |
||
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 |
||
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 |
||
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 |
||
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 |
||
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 |
||
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 |
||
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 |
@@ -16,53 +16,53 @@ discard block |
||
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 |
||
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 |
||
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 |
||
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 |
||
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 | } |