aimeos / aimeos-core

lib/mshoplib/tests/MShop/Plugin/Provider/Order/ProductFreeOptionsTest.php

Spacing +1 added lines, -1 removed lines

@@ -46,7 +46,7 @@
 		$attribute = \Aimeos\MShop::create( $this->context, 'order/base/product/attribute' )->createItem();
 
 		$attribute = $attribute->setQuantity( 2 )->setCode( 'size' )->setType( 'config' )
-			->setAttributeId( $attrManager->findItem( 'xs', [], 'product', 'size')->getId() );
+			->setAttributeId( $attrManager->findItem( 'xs', [], 'product', 'size' )->getId() );
 
 		$product = $product->setAttributeItem( $attribute )->setProductId( $prodManager->findItem( 'CNE' )->getId() );
 

lib/mshoplib/src/MShop/Service/Provider/Decorator/Postal.php

Braces +3 added lines, -2 removed lines

@@ -119,12 +119,13 @@
 					return false;
 				}
 			}
-		}
-		elseif( ( $addresses = $basket->getAddress( $paymentType ) ) !== [] ) // use billing address if no delivery address is available
+		} elseif( ( $addresses = $basket->getAddress( $paymentType ) ) !== [] ) {
+			// use billing address if no delivery address is available
 		{
 			foreach( $addresses as $address )
 			{
 				$code = $address->getPostal();
+		}
 
 				if( $this->checkPostalCode( $code, 'postal.delivery-include' ) === false
 					|| $this->checkPostalCode( $code, 'postal.delivery-exclude' ) === true

lib/mshoplib/src/MShop/Service/Provider/Decorator/Country.php

Braces +3 added lines, -2 removed lines

@@ -119,12 +119,13 @@
 					return false;
 				}
 			}
-		}
-		elseif( ( $addresses = $basket->getAddress( $paymentType ) ) !== [] ) // use billing address if no delivery address is available
+		} elseif( ( $addresses = $basket->getAddress( $paymentType ) ) !== [] ) {
+			// use billing address if no delivery address is available
 		{
 			foreach( $addresses as $address )
 			{
 				$code = strtoupper( $address->getCountryId() );
+		}
 
 				if( $this->checkCountryCode( $code, 'country.delivery-include' ) === false
 					|| $this->checkCountryCode( $code, 'country.delivery-exclude' ) === true

lib/mshoplib/tests/MShop/Catalog/Manager/StandardTest.php

Braces +2 added lines, -4 removed lines

@@ -77,8 +77,7 @@ 
 
 	public function testRegisterItemFilter()
 	{
-		$callback = function( \Aimeos\MShop\Common\Item\ListRef\Iface $item, $index )
-		{
+		$callback = function( \Aimeos\MShop\Common\Item\ListRef\Iface $item, $index ) {
 			return true;
 		};
 
@@ -290,8 +289,7 @@ 
 	{
 		$this->assertEquals( 2, count( $this->object->getTree()->getChildren() ) );
 
-		$callback = function( \Aimeos\MShop\Common\Item\ListRef\Iface $item, $index )
-		{
+		$callback = function( \Aimeos\MShop\Common\Item\ListRef\Iface $item, $index ) {
 			return (bool) $index % 2;
 		};
 

lib/mshoplib/src/MShop/Plugin/Provider/Order/ProductFreeOptions.php

Braces +3 added lines, -6 removed lines

@@ -53,8 +53,7 @@ 
 			foreach( $value as $key => $product ) {
 				$value[$key] = $this->updatePrice( $product );
 			}
-		}
-		else
+		} else
 		{
 			$value = $this->updatePrice( $value );
 		}
@@ -130,8 +129,7 @@ 
 	{
 		$priceManager = \Aimeos\MShop::create( $this->getContext(), 'price' );
 
-		$sortFcn = function( $a, $b ) use( $priceManager, $attrQtys )
-		{
+		$sortFcn = function( $a, $b ) use( $priceManager, $attrQtys ) {
 			if( ( $pricesA = $a->getRefItems( 'price', 'default', 'default' ) ) === [] ) {
 				return 1;
 			}
@@ -194,8 +192,7 @@ 
 			{
 				$list = $this->sortByPrice( $list, $attrQtys );
 				$priceItem = $this->addPrices( $priceItem, $list, $attrQtys, (int) $prodConf[$type] );
-			}
-			else
+			} else
 			{
 				$priceItem = $this->addPrices( $priceItem, $list, $attrQtys, 0 );
 			}

lib/mshoplib/setup/unitperf/CatalogAddPerfData.php

Braces +6 added lines, -4 removed lines

@@ -79,8 +79,7 @@ 
 				for( $i = 0; $i < $numCatPerLevel; $i++ ){
 					$treeFcn( $parents, $catItem->getId(), $numCatPerLevel, $level - 1, $catLabel . '-' . ($i+1), $i );
 				}
-			}
-			else
+			} else
 			{
 				$fcn = function( array $parents, $catLabel ) {
 
@@ -139,9 +138,12 @@ 
 
 		while( true )
 		{
-			try {
+			try
+			{
 				return $catalogManager->insertItem( $item, $parentId );
-			} catch( \Aimeos\MW\DB\Exception $e ) {
+			}
+			catch( \Aimeos\MW\DB\Exception $e )
+			{
 				if( $e->getCode() !== 40001 ) { throw $e; } // transaction deadlock
 			}
 		}

Spacing +25 added lines, -25 removed lines

@@ -76,8 +76,8 @@ 
 
 			if( $level > 0 )
 			{
-				for( $i = 0; $i < $numCatPerLevel; $i++ ){
-					$treeFcn( $parents, $catItem->getId(), $numCatPerLevel, $level - 1, $catLabel . '-' . ($i+1), $i );
+				for( $i = 0; $i < $numCatPerLevel; $i++ ) {
+					$treeFcn( $parents, $catItem->getId(), $numCatPerLevel, $level - 1, $catLabel . '-' . ( $i + 1 ), $i );
 				}
 			}
 			else
@@ -117,7 +117,7 @@ 
 		}
 
 		for( $i = $begin; $i < $end; $i++ ) {
-			$treeFcn( [$catRootItem], $catRootItem->getId(), $numCatPerLevel, $this->numCatLevels - 1, $i+1, $i );
+			$treeFcn( [$catRootItem], $catRootItem->getId(), $numCatPerLevel, $this->numCatLevels - 1, $i + 1, $i );
 		}
 
 		$this->additional->getProcess()->wait();
@@ -163,7 +163,7 @@ 
 			{
 				if( $item->pos % $fraction === 0 )
 				{
-					$litem = (clone $defListItem)->setRefId( $item->getId() )->setPosition( $start + round( $item->pos / $fraction ) );
+					$litem = ( clone $defListItem )->setRefId( $item->getId() )->setPosition( $start + round( $item->pos / $fraction ) );
 					$catItem->addListItem( 'product', $litem );
 				}
 			}
@@ -199,13 +199,13 @@ 
 		$listItem = $productListManager->createItem()->setType( 'default' );
 
 		foreach( $attrIds as $attrId ) {
-			$prodItem->addListItem( 'attribute', (clone $listItem)->setRefId( $attrId ) );
+			$prodItem->addListItem( 'attribute', ( clone $listItem )->setRefId( $attrId ) );
 		}
 
 		$listItem = $productListManager->createItem()->setType( 'config' );
 
 		foreach( $this->attributes['sticker'] as $attrId => $label ) {
-			$prodItem->addListItem( 'attribute', (clone $listItem)->setRefId( $attrId ) );
+			$prodItem->addListItem( 'attribute', ( clone $listItem )->setRefId( $attrId ) );
 		}
 
 		return $prodItem;
@@ -244,7 +244,7 @@ 
 		{
 			$text = current( $color ) . ' ' . current( $property ) . ' ' . current( $material ) . ' ' . current( $articles );
 
-			$item = (clone $newItem)
+			$item = ( clone $newItem )
 				->setLabel( $text . ' (' . $catLabel . ')' )
 				->setCode( 'p-' . $i . ':' . $catLabel )
 				->setStatus( 1 );
@@ -308,18 +308,18 @@ 
 
 		foreach( array_values( $this->shuffle( range( 0, 3 ) ) ) as $pos => $i )
 		{
-			$num = (($idx + $i) % 4) + 1;
-			$mediaItem = (clone $newItem)
-				->setLabel( ($pos+1) . '. picture for ' . $prodItem->getLabel() )
+			$num = ( ( $idx + $i ) % 4 ) + 1;
+			$mediaItem = ( clone $newItem )
+				->setLabel( ( $pos + 1 ) . '. picture for ' . $prodItem->getLabel() )
 				->setPreviews( [1 => $prefix . 'unitperf/' . $num . '.jpg'] )
 				->setUrl( $prefix . 'unitperf/' . $num . '-big.jpg' )
 				->setMimeType( 'image/jpeg' )
 				->setStatus( 1 );
 
-			$prodItem->addListItem( 'media', (clone $litem)->setPosition( $pos ), $mediaItem );
+			$prodItem->addListItem( 'media', ( clone $litem )->setPosition( $pos ), $mediaItem );
 		}
 
-		$mediaItem = (clone $newItem)
+		$mediaItem = ( clone $newItem )
 			->setPreviews( [1 => $prefix . 'unitperf/download-preview.jpg'] )
 			->setUrl( $prefix . 'unitperf/download.pdf' )
 			->setMimeType( 'application/pdf' )
@@ -328,7 +328,7 @@ 
 
 		$litem = $productListManager->createItem()->setType( 'download' );
 
-		return $prodItem->addListItem( 'media', (clone $litem), $mediaItem );
+		return $prodItem->addListItem( 'media', ( clone $litem ), $mediaItem );
 	}
 
 
@@ -343,15 +343,15 @@ 
 
 		for( $i = 0; $i < 3; $i++ )
 		{
-			$priceItem = (clone $newItem)
+			$priceItem = ( clone $newItem )
 				->setLabel( $prodItem->getLabel() . ': from ' . ( 1 + $i * 5 ) )
-				->setValue( 100 + (( $base + $idx ) % 900) - $i * 10 )
+				->setValue( 100 + ( ( $base + $idx ) % 900 ) - $i * 10 )
 				->setQuantity( 1 + $i * 10 )
 				->setCurrencyId( 'EUR' )
 				->setRebate( $i * 10 )
 				->setStatus( 1 );
 
-			$prodItem->addListItem( 'price', (clone $litem)->setPosition( $i ), $priceItem );
+			$prodItem->addListItem( 'price', ( clone $litem )->setPosition( $i ), $priceItem );
 		}
 
 		return $prodItem;
@@ -372,7 +372,7 @@ 
 			{
 				if( !in_array( $litem->getRefId(), $ids ) )
 				{
-					$prodItem->addListItem( 'product', (clone $listItem)->setRefId( $litem->getRefId() ) );
+					$prodItem->addListItem( 'product', ( clone $listItem )->setRefId( $litem->getRefId() ) );
 					$ids[] = $litem->getRefId();
 					$num--;
 				}
@@ -397,7 +397,7 @@ 
 			->setType( 'url' )
 			->setStatus( 1 );
 
-		$prodItem->addListItem( 'text', (clone $listItem), $textItem );
+		$prodItem->addListItem( 'text', ( clone $listItem ), $textItem );
 
 		$textItem = $textManager->createItem()
 			->setLanguageId( 'en' )
@@ -406,7 +406,7 @@ 
 			->setType( 'name' )
 			->setStatus( 1 );
 
-		$prodItem->addListItem( 'text', (clone $listItem)->setPosition( 0 ), $textItem );
+		$prodItem->addListItem( 'text', ( clone $listItem )->setPosition( 0 ), $textItem );
 
 		$textItem = $textManager->createItem()
 			->setContent( 'Short description for ' . $label )
@@ -415,7 +415,7 @@ 
 			->setType( 'short' )
 			->setStatus( 1 );
 
-		$prodItem->addListItem( 'text', (clone $listItem)->setPosition( 1 ), $textItem );
+		$prodItem->addListItem( 'text', ( clone $listItem )->setPosition( 1 ), $textItem );
 
 		$textItem = $textManager->createItem()
 			->setContent( 'Long description for ' . $label . '. This may include some "lorem ipsum" text' )
@@ -424,7 +424,7 @@ 
 			->setType( 'long' )
 			->setStatus( 1 );
 
-		$prodItem->addListItem( 'text', (clone $listItem)->setPosition( 2 ), $textItem );
+		$prodItem->addListItem( 'text', ( clone $listItem )->setPosition( 2 ), $textItem );
 
 		return $prodItem;
 	}
@@ -447,14 +447,14 @@ 
 		{
 			$text = current( $length ) . ', ' . current( $width ) . ', ' . $prodItem->getLabel() . ' (' . current( $size ) . ')';
 
-			$item = (clone $newItem)
+			$item = ( clone $newItem )
 				->setCode( 'v-' . $idx . '/' . $i . ':' . $prodItem->getCode() )
 				->setLabel( $text )
 				->setStatus( 1 );
 
-			$item->addListItem( 'attribute', (clone $varListItem)->setRefId( key( $length ) ) );
-			$item->addListItem( 'attribute', (clone $varListItem)->setRefId( key( $width ) ) );
-			$item->addListItem( 'attribute', (clone $varListItem)->setRefId( key( $size ) ) );
+			$item->addListItem( 'attribute', ( clone $varListItem )->setRefId( key( $length ) ) );
+			$item->addListItem( 'attribute', ( clone $varListItem )->setRefId( key( $width ) ) );
+			$item->addListItem( 'attribute', ( clone $varListItem )->setRefId( key( $size ) ) );
 
 			$prodItem->addListItem( 'product', clone $defListItem, $item );
 

lib/mshoplib/setup/LocaleRemoveCharConstraints.php

Braces +2 added lines, -4 removed lines

@@ -53,8 +53,7 @@ 
 		) {
 			$this->execute( 'ALTER TABLE "mshop_locale" DROP FOREIGN KEY "fk_msloc_langid"', 'db-locale' );
 			$this->status( 'done' );
-		}
-		else
+		} else
 		{
 			$this->status( 'OK' );
 		}
@@ -66,8 +65,7 @@ 
 		) {
 			$this->execute( 'ALTER TABLE "mshop_locale" DROP FOREIGN KEY "fk_msloc_currid"', 'db-locale' );
 			$this->status( 'done' );
-		}
-		else
+		} else
 		{
 			$this->status( 'OK' );
 		}

lib/mshoplib/setup/unittest/data/text.php

Spacing +4 added lines, -4 removed lines

@@ -6,8 +6,8 @@ 
  * @copyright Aimeos (aimeos.org), 2015-2018
  */
 
-return array (
-	'text/type' => array (
+return array(
+	'text/type' => array(
 		'attribute/name' => array( 'domain' => 'attribute', 'code' => 'name', 'label' => 'Name', 'status' => 1 ),
 		'attribute/short' => array( 'domain' => 'attribute', 'code' => 'short', 'label' => 'Short description', 'status' => 1 ),
 		'attribute/long' => array( 'domain' => 'attribute', 'code' => 'long', 'label' => 'Long description', 'status' => 1 ),
@@ -48,10 +48,10 @@ 
 		'supplier/long' => array( 'domain' => 'supplier', 'code' => 'long', 'label' => 'Long description', 'status' => 1 ),
 	),
 
-	'text' => array (
+	'text' => array(
 
 //supplier
-		'text/supplier/description' => array( 'langid' => null, 'type' => 'description', 'domain' => 'supplier', 'label' => 'supplier/description', 'content' => 'Supplier description', 'status' => 1),
+		'text/supplier/description' => array( 'langid' => null, 'type' => 'description', 'domain' => 'supplier', 'label' => 'supplier/description', 'content' => 'Supplier description', 'status' => 1 ),
 //ATTR
 //size
 		'text/size/XS' => array( 'langid' => null, 'type' => 'name', 'domain' => 'attribute', 'label' => 'size/XS', 'content' => 'XS', 'status' => 1 ),

lib/mshoplib/setup/unittest/data/customer.php

Spacing +1 added lines, -1 removed lines

@@ -80,7 +80,7 @@
 				'customer.address.telefax' => '055544332212', 'customer.address.website' => 'www.example.com',
 				'customer.address.longitude' => '10.5', 'customer.address.latitude' => '51.0',
 				'customer.address.position' => '0',
-			],[
+			], [
 				'customer.address.company' => 'Example company LLC', 'customer.address.vatid' => 'DE999999999',
 				'customer.address.salutation' => 'mr', 'customer.address.title' => 'Dr.',
 				'customer.address.firstname' => 'Good', 'customer.address.lastname' => 'Unittest',