@@ -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 ) ); |
@@ -47,7 +47,7 @@ |
||
| 47 | 47 | public function testGet() |
| 48 | 48 | { |
| 49 | 49 | $manager = \Aimeos\MShop::create( $this->context, 'order' ); |
| 50 | - $items = $manager->searchItems( $manager->createSearch()->setSlice( 0, 1) ); |
|
| 50 | + $items = $manager->searchItems( $manager->createSearch()->setSlice( 0, 1 ) ); |
|
| 51 | 51 | |
| 52 | 52 | if( ( $item = reset( $items ) ) === null ) { |
| 53 | 53 | throw new \RuntimeException( 'No order item found' ); |
@@ -142,9 +142,11 @@ |
||
| 142 | 142 | throw new \Aimeos\Controller\Frontend\Basket\Exception( sprintf( $msg, $productItem->getId() ) ); |
| 143 | 143 | } |
| 144 | 144 | |
| 145 | - if( empty( $items ) && $requireVariant != false ) // count == 0 |
|
| 145 | + if( empty( $items ) && $requireVariant != false ) { |
|
| 146 | + // count == 0 |
|
| 146 | 147 | { |
| 147 | 148 | $msg = $context->getI18n()->dt( 'controller/frontend', 'No article found for selected attributes and product ID "%1$s"' ); |
| 149 | + } |
|
| 148 | 150 | throw new \Aimeos\Controller\Frontend\Basket\Exception( sprintf( $msg, $productItem->getId() ) ); |
| 149 | 151 | } |
| 150 | 152 | |
@@ -376,11 +376,11 @@ |
||
| 376 | 376 | public function testDeleteAddress() |
| 377 | 377 | { |
| 378 | 378 | $this->object->addAddress( 'payment', [] ); |
| 379 | - $this->assertEquals( 1, count( $this->object->get()->getAddress( 'payment') ) ); |
|
| 379 | + $this->assertEquals( 1, count( $this->object->get()->getAddress( 'payment' ) ) ); |
|
| 380 | 380 | |
| 381 | 381 | $result = $this->object->deleteAddress( 'payment' ); |
| 382 | 382 | |
| 383 | - $this->assertEquals( 0, count( $this->object->get()->getAddress( 'payment') ) ); |
|
| 383 | + $this->assertEquals( 0, count( $this->object->get()->getAddress( 'payment' ) ) ); |
|
| 384 | 384 | $this->assertInstanceOf( \Aimeos\Controller\Frontend\Basket\Iface::class, $result ); |
| 385 | 385 | } |
| 386 | 386 | |
@@ -47,7 +47,7 @@ discard block |
||
| 47 | 47 | { |
| 48 | 48 | $amount = $attr->getValue(); |
| 49 | 49 | |
| 50 | - if( preg_match( '/^[0-9]*(\.[0-9]+)?$/', $amount ) !== 1 || ((double) $amount) < 0.01 ) |
|
| 50 | + if( preg_match( '/^[0-9]*(\.[0-9]+)?$/', $amount ) !== 1 || ( (double) $amount ) < 0.01 ) |
|
| 51 | 51 | { |
| 52 | 52 | $msg = $context->getI18n()->dt( 'controller/frontend', 'Invalid price value "%1$s"' ); |
| 53 | 53 | throw new \Aimeos\Controller\Frontend\Basket\Exception( sprintf( $msg, $amount ) ); |
@@ -310,7 +310,7 @@ discard block |
||
| 310 | 310 | $this->getObject()->addService( $service, $attributes ); |
| 311 | 311 | $basket->deleteService( $type ); |
| 312 | 312 | } |
| 313 | - catch( \Exception $e ) { ; } // Don't notify the user as appropriate services can be added automatically |
|
| 313 | + catch( \Exception $e ) {; } // Don't notify the user as appropriate services can be added automatically |
|
| 314 | 314 | } |
| 315 | 315 | } |
| 316 | 316 | |
@@ -310,7 +310,9 @@ |
||
| 310 | 310 | $this->getObject()->addService( $service, $attributes ); |
| 311 | 311 | $basket->deleteService( $type ); |
| 312 | 312 | } |
| 313 | - catch( \Exception $e ) { ; } // Don't notify the user as appropriate services can be added automatically |
|
| 313 | + catch( \Exception $e ) |
|
| 314 | + { |
|
| 315 | +; } // Don't notify the user as appropriate services can be added automatically |
|
| 314 | 316 | } |
| 315 | 317 | } |
| 316 | 318 | |