aimeos / aimeos-core

lib/mshoplib/setup/default/schema/index.php

Spacing +5 added lines, -5 removed lines

@@ -15,7 +15,7 @@ 
 
 	'table' => array(
 
-		'mshop_index_attribute' => function ( \Doctrine\DBAL\Schema\Schema $schema ) {
+		'mshop_index_attribute' => function( \Doctrine\DBAL\Schema\Schema $schema ) {
 
 			$table = $schema->createTable( 'mshop_index_attribute' );
 
@@ -34,7 +34,7 @@ 
 			return $schema;
 		},
 
-		'mshop_index_catalog' => function ( \Doctrine\DBAL\Schema\Schema $schema ) {
+		'mshop_index_catalog' => function( \Doctrine\DBAL\Schema\Schema $schema ) {
 
 			$table = $schema->createTable( 'mshop_index_catalog' );
 
@@ -51,7 +51,7 @@ 
 			return $schema;
 		},
 
-		'mshop_index_price' => function ( \Doctrine\DBAL\Schema\Schema $schema ) {
+		'mshop_index_price' => function( \Doctrine\DBAL\Schema\Schema $schema ) {
 
 			$table = $schema->createTable( 'mshop_index_price' );
 
@@ -75,7 +75,7 @@ 
 			return $schema;
 		},
 
-		'mshop_index_supplier' => function ( \Doctrine\DBAL\Schema\Schema $schema ) {
+		'mshop_index_supplier' => function( \Doctrine\DBAL\Schema\Schema $schema ) {
 
 			$table = $schema->createTable( 'mshop_index_supplier' );
 
@@ -92,7 +92,7 @@ 
 			return $schema;
 		},
 
-		'mshop_index_text' => function ( \Doctrine\DBAL\Schema\Schema $schema ) {
+		'mshop_index_text' => function( \Doctrine\DBAL\Schema\Schema $schema ) {
 
 			$table = $schema->createTable( 'mshop_index_text' );
 			$table->addOption( 'engine', 'MyISAM' );

lib/mshoplib/setup/IndexRemovePriceMeta.php

Braces +2 added lines, -4 removed lines

@@ -71,8 +71,7 @@ 
 				$this->execute( $this->clear );
 				$this->execute( $stmt );
 				$this->status( 'done' );
-			}
-			else
+			} else
 			{
 				$this->status( 'OK' );
 			}
@@ -87,8 +86,7 @@ 
 			{
 				$this->execute( $stmt );
 				$this->status( 'done' );
-			}
-			else
+			} else
 			{
 				$this->status( 'OK' );
 			}

lib/mshoplib/setup/IndexRemoveTextMeta.php

Braces +2 added lines, -4 removed lines

@@ -71,8 +71,7 @@ 
 				$this->execute( $this->clear );
 				$this->execute( $stmt );
 				$this->status( 'done' );
-			}
-			else
+			} else
 			{
 				$this->status( 'OK' );
 			}
@@ -87,8 +86,7 @@ 
 			{
 				$this->execute( $stmt );
 				$this->status( 'done' );
-			}
-			else
+			} else
 			{
 				$this->status( 'OK' );
 			}

lib/mshoplib/tests/MShop/Media/Manager/Property/StandardTest.php

Spacing +9 added lines, -9 removed lines

@@ -54,11 +54,11 @@ 
 		$search->setConditions( $search->compare( '==', 'media.property.editor', $this->editor ) );
 		$results = $this->object->searchItems( $search );
 
-		if( ( $item = reset($results) ) === false ) {
+		if( ( $item = reset( $results ) ) === false ) {
 			throw new \RuntimeException( 'No property item found' );
 		}
 
-		$item->setId(null);
+		$item->setId( null );
 		$item->setLanguageId( 'en' );
 		$resultSaved = $this->object->saveItem( $item );
 		$itemSaved = $this->object->getItem( $item->getId() );
@@ -109,13 +109,13 @@ 
 	{
 		$search = $this->object->createSearch()->setSlice( 0, 1 );
 		$conditions = array(
-			$search->compare( '~=', 'media.property.value', '1024'),
+			$search->compare( '~=', 'media.property.value', '1024' ),
 			$search->compare( '==', 'media.property.editor', $this->editor )
 		);
 		$search->setConditions( $search->combine( '&&', $conditions ) );
 		$results = $this->object->searchItems( $search );
 
-		if( ($expected = reset($results)) === false ) {
+		if( ( $expected = reset( $results ) ) === false ) {
 			throw new \RuntimeException( sprintf( 'No media property item found for value "%1$s".', '1024' ) );
 		}
 
@@ -154,7 +154,7 @@ 
 		$expr[] = $search->compare( '==', 'media.property.value', '1024' );
 		$expr[] = $search->compare( '==', 'media.property.editor', $this->editor );
 
-		$search->setConditions( $search->combine('&&', $expr) );
+		$search->setConditions( $search->combine( '&&', $expr ) );
 		$results = $this->object->searchItems( $search, [], $total );
 		$this->assertEquals( 1, count( $results ) );
 	}
@@ -162,17 +162,17 @@ 
 
 	public function testGetSubManager()
 	{
-		$this->assertInstanceOf( \Aimeos\MShop\Common\Manager\Iface::class, $this->object->getSubManager('type') );
-		$this->assertInstanceOf( \Aimeos\MShop\Common\Manager\Iface::class, $this->object->getSubManager('type', 'Standard') );
+		$this->assertInstanceOf( \Aimeos\MShop\Common\Manager\Iface::class, $this->object->getSubManager( 'type' ) );
+		$this->assertInstanceOf( \Aimeos\MShop\Common\Manager\Iface::class, $this->object->getSubManager( 'type', 'Standard' ) );
 
 		$this->setExpectedException( \Aimeos\MShop\Exception::class );
-		$this->object->getSubManager('unknown');
+		$this->object->getSubManager( 'unknown' );
 	}
 
 
 	public function testGetSubManagerInvalidName()
 	{
 		$this->setExpectedException( \Aimeos\MShop\Exception::class );
-		$this->object->getSubManager('type', 'unknown');
+		$this->object->getSubManager( 'type', 'unknown' );
 	}
 }

lib/mshoplib/tests/MShop/Attribute/Manager/Property/StandardTest.php

Spacing +9 added lines, -9 removed lines

@@ -61,11 +61,11 @@ 
 		$search->setConditions( $search->compare( '==', 'attribute.property.editor', $this->editor ) );
 		$results = $this->object->searchItems( $search );
 
-		if( ( $item = reset($results) ) === false ) {
+		if( ( $item = reset( $results ) ) === false ) {
 			throw new \RuntimeException( 'No property item found' );
 		}
 
-		$item->setId(null);
+		$item->setId( null );
 		$item->setLanguageId( 'en' );
 		$resultSaved = $this->object->saveItem( $item );
 		$itemSaved = $this->object->getItem( $item->getId() );
@@ -116,13 +116,13 @@ 
 	{
 		$search = $this->object->createSearch()->setSlice( 0, 1 );
 		$conditions = array(
-			$search->compare( '~=', 'attribute.property.value', '1024'),
+			$search->compare( '~=', 'attribute.property.value', '1024' ),
 			$search->compare( '==', 'attribute.property.editor', $this->editor )
 		);
 		$search->setConditions( $search->combine( '&&', $conditions ) );
 		$results = $this->object->searchItems( $search );
 
-		if( ($expected = reset($results)) === false ) {
+		if( ( $expected = reset( $results ) ) === false ) {
 			throw new \RuntimeException( sprintf( 'No attribute property item found for value "%1$s".', '1024' ) );
 		}
 
@@ -161,7 +161,7 @@ 
 		$expr[] = $search->compare( '==', 'attribute.property.value', '1024' );
 		$expr[] = $search->compare( '==', 'attribute.property.editor', $this->editor );
 
-		$search->setConditions( $search->combine('&&', $expr) );
+		$search->setConditions( $search->combine( '&&', $expr ) );
 		$results = $this->object->searchItems( $search, [], $total );
 		$this->assertEquals( 1, count( $results ) );
 	}
@@ -169,17 +169,17 @@ 
 
 	public function testGetSubManager()
 	{
-		$this->assertInstanceOf( \Aimeos\MShop\Common\Manager\Iface::class, $this->object->getSubManager('type') );
-		$this->assertInstanceOf( \Aimeos\MShop\Common\Manager\Iface::class, $this->object->getSubManager('type', 'Standard') );
+		$this->assertInstanceOf( \Aimeos\MShop\Common\Manager\Iface::class, $this->object->getSubManager( 'type' ) );
+		$this->assertInstanceOf( \Aimeos\MShop\Common\Manager\Iface::class, $this->object->getSubManager( 'type', 'Standard' ) );
 
 		$this->setExpectedException( \Aimeos\MShop\Exception::class );
-		$this->object->getSubManager('unknown');
+		$this->object->getSubManager( 'unknown' );
 	}
 
 
 	public function testGetSubManagerInvalidName()
 	{
 		$this->setExpectedException( \Aimeos\MShop\Exception::class );
-		$this->object->getSubManager('type', 'unknown');
+		$this->object->getSubManager( 'type', 'unknown' );
 	}
 }

lib/mshoplib/tests/MShop/Customer/Manager/Property/StandardTest.php

Spacing +9 added lines, -9 removed lines

@@ -54,11 +54,11 @@ 
 		$search->setConditions( $search->compare( '==', 'customer.property.editor', $this->editor ) );
 		$results = $this->object->searchItems( $search );
 
-		if( ( $item = reset($results) ) === false ) {
+		if( ( $item = reset( $results ) ) === false ) {
 			throw new \RuntimeException( 'No property item found' );
 		}
 
-		$item->setId(null);
+		$item->setId( null );
 		$item->setLanguageId( 'en' );
 		$resultSaved = $this->object->saveItem( $item );
 		$itemSaved = $this->object->getItem( $item->getId() );
@@ -109,13 +109,13 @@ 
 	{
 		$search = $this->object->createSearch()->setSlice( 0, 1 );
 		$conditions = array(
-			$search->compare( '~=', 'customer.property.value', '1'),
+			$search->compare( '~=', 'customer.property.value', '1' ),
 			$search->compare( '==', 'customer.property.editor', $this->editor )
 		);
 		$search->setConditions( $search->combine( '&&', $conditions ) );
 		$results = $this->object->searchItems( $search );
 
-		if( ($expected = reset($results)) === false ) {
+		if( ( $expected = reset( $results ) ) === false ) {
 			throw new \RuntimeException( sprintf( 'No customer property item found for value "%1$s".', '1' ) );
 		}
 
@@ -154,7 +154,7 @@ 
 		$expr[] = $search->compare( '==', 'customer.property.value', '1' );
 		$expr[] = $search->compare( '==', 'customer.property.editor', $this->editor );
 
-		$search->setConditions( $search->combine('&&', $expr) );
+		$search->setConditions( $search->combine( '&&', $expr ) );
 		$results = $this->object->searchItems( $search, [], $total );
 		$this->assertEquals( 1, count( $results ) );
 	}
@@ -162,17 +162,17 @@ 
 
 	public function testGetSubManager()
 	{
-		$this->assertInstanceOf( \Aimeos\MShop\Common\Manager\Iface::class, $this->object->getSubManager('type') );
-		$this->assertInstanceOf( \Aimeos\MShop\Common\Manager\Iface::class, $this->object->getSubManager('type', 'Standard') );
+		$this->assertInstanceOf( \Aimeos\MShop\Common\Manager\Iface::class, $this->object->getSubManager( 'type' ) );
+		$this->assertInstanceOf( \Aimeos\MShop\Common\Manager\Iface::class, $this->object->getSubManager( 'type', 'Standard' ) );
 
 		$this->setExpectedException( \Aimeos\MShop\Exception::class );
-		$this->object->getSubManager('unknown');
+		$this->object->getSubManager( 'unknown' );
 	}
 
 
 	public function testGetSubManagerInvalidName()
 	{
 		$this->setExpectedException( \Aimeos\MShop\Exception::class );
-		$this->object->getSubManager('type', 'unknown');
+		$this->object->getSubManager( 'type', 'unknown' );
 	}
 }

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

Spacing +9 added lines, -9 removed lines

@@ -54,11 +54,11 @@ 
 		$search->setConditions( $search->compare( '==', 'product.property.editor', $this->editor ) );
 		$results = $this->object->searchItems( $search );
 
-		if( ( $item = reset($results) ) === false ) {
+		if( ( $item = reset( $results ) ) === false ) {
 			throw new \RuntimeException( 'No property item found' );
 		}
 
-		$item->setId(null);
+		$item->setId( null );
 		$item->setLanguageId( 'en' );
 		$resultSaved = $this->object->saveItem( $item );
 		$itemSaved = $this->object->getItem( $item->getId() );
@@ -109,13 +109,13 @@ 
 	{
 		$search = $this->object->createSearch()->setSlice( 0, 1 );
 		$conditions = array(
-			$search->compare( '~=', 'product.property.value', '25.0'),
+			$search->compare( '~=', 'product.property.value', '25.0' ),
 			$search->compare( '==', 'product.property.editor', $this->editor )
 		);
 		$search->setConditions( $search->combine( '&&', $conditions ) );
 		$results = $this->object->searchItems( $search );
 
-		if( ($expected = reset($results)) === false ) {
+		if( ( $expected = reset( $results ) ) === false ) {
 			throw new \RuntimeException( sprintf( 'No product property item found for value "%1$s".', '25.0' ) );
 		}
 
@@ -154,7 +154,7 @@ 
 		$expr[] = $search->compare( '==', 'product.property.value', '25.00' );
 		$expr[] = $search->compare( '==', 'product.property.editor', $this->editor );
 
-		$search->setConditions( $search->combine('&&', $expr) );
+		$search->setConditions( $search->combine( '&&', $expr ) );
 		$results = $this->object->searchItems( $search, [], $total );
 		$this->assertEquals( 1, count( $results ) );
 	}
@@ -162,17 +162,17 @@ 
 
 	public function testGetSubManager()
 	{
-		$this->assertInstanceOf( \Aimeos\MShop\Common\Manager\Iface::class, $this->object->getSubManager('type') );
-		$this->assertInstanceOf( \Aimeos\MShop\Common\Manager\Iface::class, $this->object->getSubManager('type', 'Standard') );
+		$this->assertInstanceOf( \Aimeos\MShop\Common\Manager\Iface::class, $this->object->getSubManager( 'type' ) );
+		$this->assertInstanceOf( \Aimeos\MShop\Common\Manager\Iface::class, $this->object->getSubManager( 'type', 'Standard' ) );
 
 		$this->setExpectedException( \Aimeos\MShop\Exception::class );
-		$this->object->getSubManager('unknown');
+		$this->object->getSubManager( 'unknown' );
 	}
 
 
 	public function testGetSubManagerInvalidName()
 	{
 		$this->setExpectedException( \Aimeos\MShop\Exception::class );
-		$this->object->getSubManager('type', 'unknown');
+		$this->object->getSubManager( 'type', 'unknown' );
 	}
 }

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

Braces +2 added lines, -4 removed lines

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

lib/mshoplib/setup/unittest/MediaAddPropertyTestData.php

Spacing +2 added lines, -2 removed lines

@@ -91,7 +91,7 @@ 
 		foreach( $testdata['media/property'] as $key => $dataset )
 		{
 			$prodProperty->setId( null );
-			$prodProperty->setParentId( $prodIds[ $dataset['parentid'] ] );
+			$prodProperty->setParentId( $prodIds[$dataset['parentid']] );
 			$prodProperty->setLanguageId( $dataset['langid'] );
 			$prodProperty->setValue( $dataset['value'] );
 			$prodProperty->setType( $dataset['type'] );
@@ -115,7 +115,7 @@ 
 		$search = $mediaManager->createSearch();
 
 		foreach( $mediaManager->searchItems( $search ) as $id => $item ) {
-			$entry[ 'media/' . $item->getUrl() ] = $id;
+			$entry['media/' . $item->getUrl()] = $id;
 		}
 
 		return $entry;