aimeos / aimeos-core

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;

lib/mshoplib/setup/unittest/AttributeAddPropertyTestData.php

Spacing +2 added lines, -2 removed lines

@@ -92,7 +92,7 @@ 
 		foreach( $testdata['attribute/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'] );
@@ -116,7 +116,7 @@ 
 		$search = $attributeManager->createSearch();
 
 		foreach( $attributeManager->searchItems( $search ) as $id => $item ) {
-			$entry[ 'attribute/' . $item->getDomain() . '/' . $item->getType() . '/' . $item->getCode() ] = $id;
+			$entry['attribute/' . $item->getDomain() . '/' . $item->getType() . '/' . $item->getCode()] = $id;
 		}
 
 		return $entry;

lib/mshoplib/setup/unittest/SupplierListAddTestData.php

Spacing +15 added lines, -15 removed lines

@@ -49,13 +49,13 @@ 
 		$ds = DIRECTORY_SEPARATOR;
 		$path = __DIR__ . $ds . 'data' . $ds . 'supplier-list.php';
 
-		if( ( $testdata = include( $path ) ) == false ){
+		if( ( $testdata = include( $path ) ) == false ) {
 			throw new \Aimeos\MShop\Exception( sprintf( 'No file "%1$s" found for supplier list domain', $path ) );
 		}
 
 		$refKeys = [];
 		foreach( $testdata['supplier/lists'] as $dataset ) {
-			$refKeys[ $dataset['domain'] ][] = $dataset['refid'];
+			$refKeys[$dataset['domain']][] = $dataset['refid'];
 		}
 
 		$refIds = [];
@@ -118,7 +118,7 @@ 
 		$codes = [];
 		foreach( $keys as $dataset )
 		{
-			if( ( $pos = strpos( $dataset, '/' ) ) === false || ( $str = substr( $dataset, $pos+1 ) ) === false ) {
+			if( ( $pos = strpos( $dataset, '/' ) ) === false || ( $str = substr( $dataset, $pos + 1 ) ) === false ) {
 				throw new \Aimeos\MW\Setup\Exception( sprintf( 'Some keys for ref products are set wrong "%1$s"', $dataset ) );
 			}
 
@@ -130,7 +130,7 @@ 
 
 		$refIds = [];
 		foreach( $manager->searchItems( $search ) as $item ) {
-			$refIds[ 'product/' . $item->getCode() ] = $item->getId();
+			$refIds['product/' . $item->getCode()] = $item->getId();
 		}
 
 		return $refIds;
@@ -150,7 +150,7 @@ 
 		$labels = [];
 		foreach( $keys as $dataset )
 		{
-			if( ( $pos = strpos( $dataset, '/' ) ) === false || ( $str = substr( $dataset, $pos+1 ) ) === false ) {
+			if( ( $pos = strpos( $dataset, '/' ) ) === false || ( $str = substr( $dataset, $pos + 1 ) ) === false ) {
 				throw new \Aimeos\MW\Setup\Exception( sprintf( 'Some keys for ref text are set wrong "%1$s"', $dataset ) );
 			}
 
@@ -161,8 +161,8 @@ 
 		$search->setConditions( $search->compare( '==', 'text.label', $labels ) );
 
 		$refIds = [];
-		foreach( $textManager->searchItems( $search ) as $item )	{
-			$refIds[ 'text/'.$item->getLabel() ] = $item->getId();
+		foreach( $textManager->searchItems( $search ) as $item ) {
+			$refIds['text/' . $item->getLabel()] = $item->getId();
 		}
 
 		return $refIds;
@@ -186,7 +186,7 @@ 
 		$itemCode = [];
 		foreach( $testdata['supplier/lists'] as $dataset )
 		{
-			if( ( $pos = strpos( $dataset['parentid'], '/' ) ) === false || ( $str = substr( $dataset['parentid'], $pos+1 ) ) === false ) {
+			if( ( $pos = strpos( $dataset['parentid'], '/' ) ) === false || ( $str = substr( $dataset['parentid'], $pos + 1 ) ) === false ) {
 				throw new \Aimeos\MW\Setup\Exception( sprintf( 'Some keys for parentid are set wrong "%1$s"', $dataset['parentid'] ) );
 			}
 
@@ -194,11 +194,11 @@ 
 		}
 
 		$search = $supplierManager->createSearch();
-		$search->setConditions( $search->compare( '==', 'supplier.code', $itemCode) );
+		$search->setConditions( $search->compare( '==', 'supplier.code', $itemCode ) );
 
 		$parentIds = [];
-		foreach( $supplierManager->searchItems( $search ) as $item )	{
-			$parentIds[ 'supplier/'.$item->getCode() ] = $item->getId();
+		foreach( $supplierManager->searchItems( $search ) as $item ) {
+			$parentIds['supplier/' . $item->getCode()] = $item->getId();
 		}
 
 		$listItemType = $supplierListTypeManager->createItem();
@@ -219,17 +219,17 @@ 
 		$listItem = $supplierListManager->createItem();
 		foreach( $testdata['supplier/lists'] as $dataset )
 		{
-			if( !isset( $parentIds[ $dataset['parentid'] ] ) ) {
+			if( !isset( $parentIds[$dataset['parentid']] ) ) {
 				throw new \Aimeos\MW\Setup\Exception( sprintf( 'No supplier ID found for "%1$s"', $dataset['parentid'] ) );
 			}
 
-			if( !isset( $refIds[ $dataset['domain'] ][ $dataset['refid'] ] ) ) {
+			if( !isset( $refIds[$dataset['domain']][$dataset['refid']] ) ) {
 				throw new \Aimeos\MW\Setup\Exception( sprintf( 'No "%2$s" ref ID found for "%1$s"', $dataset['refid'], $dataset['domain'] ) );
 			}
 
 			$listItem->setId( null );
-			$listItem->setParentId( $parentIds[ $dataset['parentid'] ] );
-			$listItem->setRefId( $refIds[ $dataset['domain'] ] [ $dataset['refid'] ] );
+			$listItem->setParentId( $parentIds[$dataset['parentid']] );
+			$listItem->setRefId( $refIds[$dataset['domain']] [$dataset['refid']] );
 			$listItem->setType( $dataset['type'] );
 			$listItem->setDomain( $dataset['domain'] );
 			$listItem->setDateStart( $dataset['start'] );

lib/mshoplib/setup/unittest/data/supplier-list.php

Spacing +3 added lines, -3 removed lines

@@ -5,15 +5,15 @@
  * @copyright Aimeos (aimeos.org), 2015-2018
  */
 
-return array (
-	'supplier/lists/type' => array (
+return array(
+	'supplier/lists/type' => array(
 		'attribute/default' => array( 'domain' => 'attribute', 'code' => 'default', 'label' => 'Standard', 'status' => 1 ),
 		'media/default' => array( 'domain' => 'media', 'code' => 'default', 'label' => 'Standard', 'status' => 1 ),
 		'product/default' => array( 'domain' => 'product', 'code' => 'default', 'label' => 'Standard', 'status' => 1 ),
 		'text/default' => array( 'domain' => 'text', 'code' => 'default', 'label' => 'Standard', 'status' => 1 ),
 	),
 
-	'supplier/lists' => array (
+	'supplier/lists' => array(
 		array( 'parentid' => 'supplier/unitCode001', 'type' => 'default', 'domain' => 'product', 'refid' => 'product/CNC', 'start' => '2010-01-01 00:00:00', 'end' => '2100-01-01 00:00:00', 'config' => [], 'pos' => 1, 'status' => 1 ),
 		array( 'parentid' => 'supplier/unitCode001', 'type' => 'default', 'domain' => 'product', 'refid' => 'product/CNE', 'start' => '2010-01-01 00:00:00', 'end' => null, 'config' => [], 'pos' => 2, 'status' => 1 ),
 

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

Spacing +3 added lines, -3 removed lines

@@ -8,7 +8,7 @@ 
 
 return array(
 	'table' => array(
-		'mshop_tag_type' => function ( \Doctrine\DBAL\Schema\Schema $schema ) {
+		'mshop_tag_type' => function( \Doctrine\DBAL\Schema\Schema $schema ) {
 
 			$table = $schema->createTable( 'mshop_tag_type' );
 
@@ -32,7 +32,7 @@ 
 			return $schema;
 		},
 
-		'mshop_tag' => function ( \Doctrine\DBAL\Schema\Schema $schema ) {
+		'mshop_tag' => function( \Doctrine\DBAL\Schema\Schema $schema ) {
 
 			$table = $schema->createTable( 'mshop_tag' );
 
@@ -44,7 +44,7 @@ 
 			$table->addColumn( 'label', 'string', array( 'length' => 255 ) );
 			$table->addColumn( 'mtime', 'datetime', [] );
 			$table->addColumn( 'ctime', 'datetime', [] );
-			$table->addColumn( 'editor', 'string', array('length' => 255 ) );
+			$table->addColumn( 'editor', 'string', array( 'length' => 255 ) );
 
 			$table->setPrimaryKey( array( 'id' ), 'pk_mstag_id' );
 			$table->addUniqueIndex( array( 'siteid', 'domain', 'type', 'langid', 'label' ), 'unq_mstag_sid_dom_ty_lid_lab' );