Passed
Branch master (901174)
by Aimeos
03:14
created
controller/frontend/src/Controller/Frontend/Basket/Decorator/Select.php 1 patch
Braces   +6 added lines, -4 removed lines patch added patch discarded remove patch
@@ -121,10 +121,11 @@  discard block
 block discarded – undo
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
 block discarded – undo
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
 
Please login to merge, or discard this patch.
controller/frontend/src/Controller/Frontend/Basket/Standard.php 1 patch
Braces   +3 added lines, -6 removed lines patch added patch discarded remove patch
@@ -410,16 +410,13 @@
 block discarded – undo
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 ) );
Please login to merge, or discard this patch.
controller/frontend/src/Controller/Frontend/Catalog/Standard.php 1 patch
Indentation   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -208,7 +208,7 @@
 block discarded – undo
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
 		}
Please login to merge, or discard this patch.
controller/frontend/src/Controller/Frontend/Product/Standard.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -398,7 +398,7 @@
 block discarded – undo
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 ) );
Please login to merge, or discard this patch.
controller/frontend/src/Controller/Frontend/Basket/Base.php 2 patches
Spacing   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -50,7 +50,7 @@  discard block
 block discarded – undo
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
 block discarded – undo
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
 block discarded – undo
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
 
Please login to merge, or discard this patch.
Braces   +3 added lines, -1 removed lines patch added patch discarded remove patch
@@ -305,7 +305,9 @@
 block discarded – undo
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
 
Please login to merge, or discard this patch.