@@ -121,10 +121,11 @@ discard block |
||
121 | 121 | { |
122 | 122 | $msg = $context->getI18n()->dt( 'controller/frontend', 'No unique article found for selected attributes and product ID "%1$s"' ); |
123 | 123 | throw new \Aimeos\Controller\Frontend\Basket\Exception( sprintf( $msg, $productItem->getId() ) ); |
124 | - } |
|
125 | - else if( ( $result = reset( $productItems ) ) !== false ) // count == 1 |
|
124 | + } else if( ( $result = reset( $productItems ) ) !== false ) { |
|
125 | + // count == 1 |
|
126 | 126 | { |
127 | 127 | $productItem = $result; |
128 | + } |
|
128 | 129 | $orderBaseProductItem->setProductCode( $productItem->getCode() ); |
129 | 130 | |
130 | 131 | $subprices = $productItem->getRefItems( 'price', 'default', 'default' ); |
@@ -150,10 +151,11 @@ discard block |
||
150 | 151 | |
151 | 152 | $attr[] = $orderAttributeItem; |
152 | 153 | } |
153 | - } |
|
154 | - else if( $requireVariant != false ) // count == 0 |
|
154 | + } else if( $requireVariant != false ) { |
|
155 | + // count == 0 |
|
155 | 156 | { |
156 | 157 | $msg = $context->getI18n()->dt( 'controller/frontend', 'No article found for selected attributes and product ID "%1$s"' ); |
158 | + } |
|
157 | 159 | throw new \Aimeos\Controller\Frontend\Basket\Exception( sprintf( $msg, $productItem->getId() ) ); |
158 | 160 | } |
159 | 161 |
@@ -410,16 +410,13 @@ |
||
410 | 410 | if( $value instanceof \Aimeos\MShop\Common\Item\Address\Iface ) |
411 | 411 | { |
412 | 412 | $this->get()->setAddress( $address->copyFrom( $value ), $type ); |
413 | - } |
|
414 | - else if( is_array( $value ) ) |
|
413 | + } else if( is_array( $value ) ) |
|
415 | 414 | { |
416 | 415 | $this->get()->setAddress( $this->setAddressFromArray( $address, $value ), $type ); |
417 | - } |
|
418 | - else if( $value === null ) |
|
416 | + } else if( $value === null ) |
|
419 | 417 | { |
420 | 418 | $this->get()->deleteAddress( $type ); |
421 | - } |
|
422 | - else |
|
419 | + } else |
|
423 | 420 | { |
424 | 421 | $msg = $context->getI18n()->dt( 'controller/frontend', 'Invalid value for address type "%1$s"' ); |
425 | 422 | throw new \Aimeos\Controller\Frontend\Basket\Exception( sprintf( $msg, $type ) ); |
@@ -208,7 +208,7 @@ |
||
208 | 208 | * @category User |
209 | 209 | * @category Developer |
210 | 210 | * @see controller/frontend/catalog/levels-only |
211 | - */ |
|
211 | + */ |
|
212 | 212 | if( ( $levels = $config->get( 'controller/frontend/catalog/levels-always' ) ) != null ) { |
213 | 213 | $expr[] = $this->filter->compare( '<=', 'catalog.level', $levels ); |
214 | 214 | } |
@@ -398,7 +398,7 @@ |
||
398 | 398 | return []; |
399 | 399 | } |
400 | 400 | |
401 | - $list = [ $item->getId() ]; |
|
401 | + $list = [$item->getId()]; |
|
402 | 402 | |
403 | 403 | foreach( $item->getChildren() as $child ) { |
404 | 404 | $list = array_merge( $list, $this->getCatalogIdsFromTree( $child ) ); |
@@ -50,7 +50,7 @@ discard block |
||
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 | 55 | $msg = $context->getI18n()->dt( 'controller/frontend', 'Invalid price value "%1$s"' ); |
56 | 56 | throw new \Aimeos\Controller\Frontend\Basket\Exception( sprintf( $msg, $amount ) ); |
@@ -305,7 +305,7 @@ discard block |
||
305 | 305 | $this->addService( $type, $item->getServiceId(), $attributes ); |
306 | 306 | $basket->deleteService( $type ); |
307 | 307 | } |
308 | - catch( \Exception $e ) { ; } // Don't notify the user as appropriate services can be added automatically |
|
308 | + catch( \Exception $e ) {; } // Don't notify the user as appropriate services can be added automatically |
|
309 | 309 | } |
310 | 310 | } |
311 | 311 | |
@@ -470,7 +470,7 @@ discard block |
||
470 | 470 | $manager = \Aimeos\MShop::create( $context, 'product/lists/type' ); |
471 | 471 | |
472 | 472 | foreach( $manager->searchItems( $manager->createSearch( true ) ) as $item ) { |
473 | - $this->listTypeItems[ $item->getDomain() ][ $item->getCode() ] = $item; |
|
473 | + $this->listTypeItems[$item->getDomain()][$item->getCode()] = $item; |
|
474 | 474 | } |
475 | 475 | } |
476 | 476 |
@@ -305,7 +305,9 @@ |
||
305 | 305 | $this->addService( $type, $item->getServiceId(), $attributes ); |
306 | 306 | $basket->deleteService( $type ); |
307 | 307 | } |
308 | - catch( \Exception $e ) { ; } // Don't notify the user as appropriate services can be added automatically |
|
308 | + catch( \Exception $e ) |
|
309 | + { |
|
310 | +; } // Don't notify the user as appropriate services can be added automatically |
|
309 | 311 | } |
310 | 312 | } |
311 | 313 |