aimeos / aimeos-core

lib/mwlib/src/MW/Cache/DB.php

Spacing +1 added lines, -1 removed lines

@@ -254,7 +254,7 @@
 			$result = $stmt->execute();
 
 			while( ( $row = $result->fetch() ) !== false ) {
-				$list[ $row['id'] ] = (string) $row['value'];
+				$list[$row['id']] = (string) $row['value'];
 			}
 
 			$this->dbm->release( $conn, $this->dbname );

lib/mshoplib/src/MShop/Common/Manager/ListRef/Traits.php

Braces +2 added lines, -4 removed lines

@@ -101,8 +101,7 @@ 
 						$search->compare( '==', $prefix . '.lists.type.domain', $key ),
 						$search->compare( '==', $prefix . '.lists.type.code', $domain ),
 					] );
-				}
-				else
+				} else
 				{
 					$list[] = $search->compare( '==', $prefix . '.lists.domain', $domain );
 				}
@@ -110,8 +109,7 @@ 
 
 			$expr[] = $search->combine( '||', $list );
 			$search->setConditions( $search->combine( '&&', $expr ) );
-		}
-		else
+		} else
 		{
 			$search->setConditions( $search->compare( '==', $prefix . '.lists.parentid', $ids ) );
 		}

lib/mwlib/tests/MW/Criteria/Expression/Compare/SQLTest.php

Spacing +5 added lines, -5 removed lines

@@ -85,10 +85,10 @@ 
 			'bool' => 't.bool',
 		);
 
-		$expr = new \Aimeos\MW\Criteria\Expression\Compare\SQL( $this->conn, '==', 'list', array('a', 'b', 'c') );
+		$expr = new \Aimeos\MW\Criteria\Expression\Compare\SQL( $this->conn, '==', 'list', array( 'a', 'b', 'c' ) );
 		$this->assertEquals( "t.list IN ('a','b','c')", $expr->toSource( $types, $translations ) );
 
-		$expr = new \Aimeos\MW\Criteria\Expression\Compare\SQL( $this->conn, '!=', 'list', array('a', 'b', 'c') );
+		$expr = new \Aimeos\MW\Criteria\Expression\Compare\SQL( $this->conn, '!=', 'list', array( 'a', 'b', 'c' ) );
 		$this->assertEquals( "t.list NOT IN ('a','b','c')", $expr->toSource( $types, $translations ) );
 
 		$expr = new \Aimeos\MW\Criteria\Expression\Compare\SQL( $this->conn, '~=', 'string', 'value' );
@@ -97,13 +97,13 @@ 
 		$expr = new \Aimeos\MW\Criteria\Expression\Compare\SQL( $this->conn, '<', 'float', 0.1 );
 		$this->assertEquals( "t.float < 0.1", $expr->toSource( $types, $translations ) );
 
-		$expr= new \Aimeos\MW\Criteria\Expression\Compare\SQL( $this->conn, '>', 'int', 10 );
+		$expr = new \Aimeos\MW\Criteria\Expression\Compare\SQL( $this->conn, '>', 'int', 10 );
 		$this->assertEquals( "t.int > 10", $expr->toSource( $types, $translations ) );
 
-		$expr= new \Aimeos\MW\Criteria\Expression\Compare\SQL( $this->conn, '!=', 'undefined', null );
+		$expr = new \Aimeos\MW\Criteria\Expression\Compare\SQL( $this->conn, '!=', 'undefined', null );
 		$this->assertEquals( "t.undefined IS NOT NULL", $expr->toSource( $types, $translations ) );
 
-		$expr= new \Aimeos\MW\Criteria\Expression\Compare\SQL( $this->conn, '==', 'bool', true );
+		$expr = new \Aimeos\MW\Criteria\Expression\Compare\SQL( $this->conn, '==', 'bool', true );
 		$this->assertEquals( "t.bool = 1", $expr->toSource( $types, $translations ) );
 	}
 

lib/mshoplib/setup/unitperf/CatalogAddPerfData.php

Spacing +26 added lines, -26 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
@@ -116,7 +116,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();
@@ -156,7 +156,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 );
 				}
 			}
@@ -191,13 +191,13 @@ 
 		$listItem = $productListManager->createItem( 'default', 'attribute' );
 
 		foreach( $attrIds as $attrId ) {
-			$prodItem->addListItem( 'attribute', (clone $listItem)->setRefId( $attrId ) );
+			$prodItem->addListItem( 'attribute', ( clone $listItem )->setRefId( $attrId ) );
 		}
 
 		$listItem = $productListManager->createItem( 'config', 'attribute' );
 
 		foreach( $this->attributes['sticker'] as $attrId => $label ) {
-			$prodItem->addListItem( 'attribute', (clone $listItem)->setRefId( $attrId ) );
+			$prodItem->addListItem( 'attribute', ( clone $listItem )->setRefId( $attrId ) );
 		}
 
 		return $prodItem;
@@ -236,7 +236,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 );
@@ -300,18 +300,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() )
 				->setPreview( $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 )
 			->setPreview( $prefix . 'unitperf/download-preview.jpg' )
 			->setUrl( $prefix . 'unitperf/download.pdf' )
 			->setMimeType( 'application/pdf' )
@@ -320,7 +320,7 @@ 
 
 		$litem = $productListManager->createItem( 'download', 'media' );
 
-		return $prodItem->addListItem( 'media', (clone $litem), $mediaItem );
+		return $prodItem->addListItem( 'media', ( clone $litem ), $mediaItem );
 	}
 
 
@@ -335,15 +335,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;
@@ -364,7 +364,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--;
 				}
@@ -388,7 +388,7 @@ 
 			->setLanguageId( 'en' )
 			->setStatus( 1 );
 
-		$prodItem->addListItem( 'text', (clone $listItem), $textItem );
+		$prodItem->addListItem( 'text', ( clone $listItem ), $textItem );
 
 		$textItem = $textManager->createItem( 'name', 'product' )
 			->setLanguageId( 'en' )
@@ -396,7 +396,7 @@ 
 			->setLabel( $label )
 			->setStatus( 1 );
 
-		$prodItem->addListItem( 'text', (clone $listItem)->setPosition( 0 ), $textItem );
+		$prodItem->addListItem( 'text', ( clone $listItem )->setPosition( 0 ), $textItem );
 
 		$textItem = $textManager->createItem( 'short', 'product' )
 			->setContent( 'Short description for ' . $label )
@@ -404,7 +404,7 @@ 
 			->setLanguageId( 'en' )
 			->setStatus( 1 );
 
-		$prodItem->addListItem( 'text', (clone $listItem)->setPosition( 1 ), $textItem );
+		$prodItem->addListItem( 'text', ( clone $listItem )->setPosition( 1 ), $textItem );
 
 		$textItem = $textManager->createItem( 'long', 'product' )
 			->setContent( 'Long description for ' . $label . '. This may include some "lorem ipsum" text' )
@@ -412,7 +412,7 @@ 
 			->setLanguageId( 'en' )
 			->setStatus( 1 );
 
-		$prodItem->addListItem( 'text', (clone $listItem)->setPosition( 2 ), $textItem );
+		$prodItem->addListItem( 'text', ( clone $listItem )->setPosition( 2 ), $textItem );
 
 		return $prodItem;
 	}
@@ -435,14 +435,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 );
 
@@ -470,7 +470,7 @@ 
 	{
 		$stockManager = \Aimeos\MShop\Factory::createManager( $this->additional, 'stock' );
 
-		$stockItem = $stockManager->createItem( 'default', 'product');
+		$stockItem = $stockManager->createItem( 'default', 'product' );
 		$stocklevels = $this->shuffle( [null, 100, 80, 60, 40, 20, 10, 5, 2, 0] );
 		$list = [];
 

Braces +10 added lines, -6 removed lines

@@ -69,7 +69,8 @@ 
 		$this->msg( 'Adding catalog performance data', 0 );
 
 
-		$treeFcn = function( array $parents, $catParentId, $numCatPerLevel, $level, $catLabel, $catIdx ) use ( &$treeFcn ) {
+		$treeFcn = function( array $parents, $catParentId, $numCatPerLevel, $level, $catLabel, $catIdx ) use ( &$treeFcn )
+		{
 
 			$catItem = $this->addCatalogItem( $catParentId, $catLabel, $catIdx );
 			array_unshift( $parents, $catItem );
@@ -79,10 +80,10 @@ 
 				for( $i = 0; $i < $numCatPerLevel; $i++ ){
 					$treeFcn( $parents, $catItem->getId(), $numCatPerLevel, $level - 1, $catLabel . '-' . ($i+1), $i );
 				}
-			}
-			else
+			} else
 			{
-				$fcn = function( array $parents, $catLabel ) {
+				$fcn = function( array $parents, $catLabel )
+				{
 
 					srand( getmypid() ); mt_srand( getmypid() );
 					$this->addProductItems( $parents, $catLabel );
@@ -139,9 +140,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
 			}
 		}

lib/mshoplib/src/MShop/Index/Manager/Text/MySQL.php

Braces +2 added lines, -1 removed lines

@@ -59,7 +59,8 @@
 
 		$site = $context->getLocale()->getSitePath();
 
-		$func = function( $source, array $params ) {
+		$func = function( $source, array $params )
+		{
 
 			if( isset( $params[1] ) )
 			{

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

Spacing +2 added lines, -2 removed lines

@@ -337,10 +337,10 @@
 		$expr[] = $search->compare( '==', 'product.editor', $this->editor );
 		$expr[] = $search->compare( '>=', 'product.target', '' );
 
-		$param = array( 'product','suggestion', $listItem->getRefId() );
+		$param = array( 'product', 'suggestion', $listItem->getRefId() );
 		$expr[] = $search->compare( '!=', $search->createFunction( 'product:has', $param ), null );
 
-		$param = array( 'product','default', $listItem->getRefId() );
+		$param = array( 'product', 'default', $listItem->getRefId() );
 		$expr[] = $search->compare( '==', $search->createFunction( 'product:has', $param ), null );
 
 		$expr[] = $search->compare( '!=', 'product.type.id', null );

lib/mshoplib/src/MShop/Index/Manager/Text/PgSQL.php

Braces +2 added lines, -1 removed lines

@@ -59,7 +59,8 @@
 
 		$site = $context->getLocale()->getSitePath();
 
-		$func = function( $source, array $params ) {
+		$func = function( $source, array $params )
+		{
 
 			if( isset( $params[1] ) )
 			{

lib/mwlib/tests/MW/Criteria/Expression/Compare/PgSQLTest.php

Spacing +6 added lines, -6 removed lines

@@ -52,10 +52,10 @@ 
 			'bool' => 't.bool',
 		);
 
-		$expr = new \Aimeos\MW\Criteria\Expression\Compare\PgSQL( $this->conn, '==', 'list', array('a', 'b', 'c') );
+		$expr = new \Aimeos\MW\Criteria\Expression\Compare\PgSQL( $this->conn, '==', 'list', array( 'a', 'b', 'c' ) );
 		$this->assertEquals( "t.list IN ('a','b','c')", $expr->toSource( $types, $translations ) );
 
-		$expr = new \Aimeos\MW\Criteria\Expression\Compare\PgSQL( $this->conn, '!=', 'list', array('a', 'b', 'c') );
+		$expr = new \Aimeos\MW\Criteria\Expression\Compare\PgSQL( $this->conn, '!=', 'list', array( 'a', 'b', 'c' ) );
 		$this->assertEquals( "t.list NOT IN ('a','b','c')", $expr->toSource( $types, $translations ) );
 
 		$expr = new \Aimeos\MW\Criteria\Expression\Compare\PgSQL( $this->conn, '~=', 'string', 'value' );
@@ -64,16 +64,16 @@ 
 		$expr = new \Aimeos\MW\Criteria\Expression\Compare\PgSQL( $this->conn, '<', 'float', 0.1 );
 		$this->assertEquals( "t.float < 0.1", $expr->toSource( $types, $translations ) );
 
-		$expr= new \Aimeos\MW\Criteria\Expression\Compare\PgSQL( $this->conn, '>', 'int', 10 );
+		$expr = new \Aimeos\MW\Criteria\Expression\Compare\PgSQL( $this->conn, '>', 'int', 10 );
 		$this->assertEquals( "t.int > 10", $expr->toSource( $types, $translations ) );
 
-		$expr= new \Aimeos\MW\Criteria\Expression\Compare\PgSQL( $this->conn, '!=', 'undefined', null );
+		$expr = new \Aimeos\MW\Criteria\Expression\Compare\PgSQL( $this->conn, '!=', 'undefined', null );
 		$this->assertEquals( "t.undefined IS NOT NULL", $expr->toSource( $types, $translations ) );
 
-		$expr= new \Aimeos\MW\Criteria\Expression\Compare\PgSQL( $this->conn, '==', 'bool', true );
+		$expr = new \Aimeos\MW\Criteria\Expression\Compare\PgSQL( $this->conn, '==', 'bool', true );
 		$this->assertEquals( "t.bool = 't'", $expr->toSource( $types, $translations ) );
 
-		$expr= new \Aimeos\MW\Criteria\Expression\Compare\PgSQL( $this->conn, '==', 'bool', false );
+		$expr = new \Aimeos\MW\Criteria\Expression\Compare\PgSQL( $this->conn, '==', 'bool', false );
 		$this->assertEquals( "t.bool = 'f'", $expr->toSource( $types, $translations ) );
 	}
 }

lib/mwlib/tests/MW/Criteria/PgSQLTest.php

Spacing +2 added lines, -2 removed lines

@@ -43,11 +43,11 @@
 
 	public function testGetConditionSource()
 	{
-		$types = array( 'column' => \Aimeos\MW\DB\Statement\Base::PARAM_BOOL);
+		$types = array( 'column' => \Aimeos\MW\DB\Statement\Base::PARAM_BOOL );
 		$this->object->setConditions( $this->object->compare( '==', 'column', 0 ) );
 		$this->assertEquals( "column = 'f'", $this->object->getConditionSource( $types ) );
 
-		$types = array( 'column' => \Aimeos\MW\DB\Statement\Base::PARAM_BOOL);
+		$types = array( 'column' => \Aimeos\MW\DB\Statement\Base::PARAM_BOOL );
 		$this->object->setConditions( $this->object->compare( '==', 'column', 1 ) );
 		$this->assertEquals( "column = 't'", $this->object->getConditionSource( $types ) );
 	}