aimeos / aimeos-core

lib/mshoplib/setup/unittest/ProductListAddTestData.php

Spacing +8 added lines, -8 removed lines

@@ -52,7 +52,7 @@ 
 		$this->additional->setEditor( 'core:unittest' );
 
 		$ds = DIRECTORY_SEPARATOR;
-		$path = __DIR__ . $ds . 'data' . $ds . 'product-list.php';
+		$path = __DIR__.$ds.'data'.$ds.'product-list.php';
 
 		if( ( $testdata = include( $path ) ) == false ) {
 			throw new \Aimeos\MShop\Exception( sprintf( 'No file "%1$s" found for product domain', $path ) );
@@ -125,7 +125,7 @@ 
 
 		$refIds = [];
 		foreach( $attributeManager->searchItems( $search ) as $item ) {
-			$refIds['attribute/' . $item->getDomain() . '/' . $item->getType() . '/' . $item->getCode()] = $item->getId();
+			$refIds['attribute/'.$item->getDomain().'/'.$item->getType().'/'.$item->getCode()] = $item->getId();
 		}
 
 		return $refIds;
@@ -158,7 +158,7 @@ 
 
 		$refIds = [];
 		foreach( $mediaManager->searchItems( $search ) as $item ) {
-			$refIds['media/' . $item->getUrl()] = $item->getId();
+			$refIds['media/'.$item->getUrl()] = $item->getId();
 		}
 
 		return $refIds;
@@ -214,7 +214,7 @@ 
 
 		$refIds = [];
 		foreach( $priceManager->searchItems( $search ) as $item ) {
-			$refIds['price/' . $item->getDomain() . '/' . $item->getType() . '/' . $item->getValue() . '/' . $item->getCosts()] = $item->getId();
+			$refIds['price/'.$item->getDomain().'/'.$item->getType().'/'.$item->getValue().'/'.$item->getCosts()] = $item->getId();
 		}
 
 		return $refIds;
@@ -246,7 +246,7 @@ 
 
 		$refIds = [];
 		foreach( $textManager->searchItems( $search ) as $item ) {
-			$refIds['text/' . $item->getLabel()] = $item->getId();
+			$refIds['text/'.$item->getLabel()] = $item->getId();
 		}
 
 		return $refIds;
@@ -281,7 +281,7 @@ 
 
 		$refIds = [];
 		foreach( $tagManager->searchItems( $search ) as $item ) {
-			$refIds['tag/' . $item->getLabel()] = $item->getId();
+			$refIds['tag/'.$item->getLabel()] = $item->getId();
 		}
 
 		return $refIds;
@@ -396,7 +396,7 @@ 
 		$search->setConditions( $search->compare( '==', 'product.code', array_unique( $parentCodes ) ) );
 
 		foreach( $manager->searchItems( $search ) as $item ) {
-			$parentIds['product/' . $item->getCode()] = $item->getId();
+			$parentIds['product/'.$item->getCode()] = $item->getId();
 		}
 
 		return $parentIds;
@@ -427,7 +427,7 @@ 
 		$search->setConditions( $search->compare( '==', 'product.code', $codes ) );
 
 		foreach( $manager->searchItems( $search ) as $item ) {
-			$refIds['product/' . $item->getCode()] = $item->getId();
+			$refIds['product/'.$item->getCode()] = $item->getId();
 		}
 
 		return $refIds;

lib/mshoplib/setup/unittest/SupplierListAddTestData.php

Spacing +19 added lines, -19 removed lines

@@ -50,15 +50,15 @@ 
 		$this->additional->setEditor( 'core:unittest' );
 
 		$ds = DIRECTORY_SEPARATOR;
-		$path = __DIR__ . $ds . 'data' . $ds . 'supplier-list.php';
+		$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 = [];
@@ -87,7 +87,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 ) );
 			}
 
@@ -99,7 +99,7 @@ 
 
 		$refIds = [];
 		foreach( $manager->searchItems( $search ) as $item ) {
-			$refIds[ 'product/' . $item->getCode() ] = $item->getId();
+			$refIds['product/'.$item->getCode()] = $item->getId();
 		}
 
 		return $refIds;
@@ -119,7 +119,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 ) );
 			}
 
@@ -130,8 +130,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;
@@ -155,7 +155,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'] ) );
 			}
 
@@ -163,11 +163,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();
 		}
 
 		$listItemTypeIds = [];
@@ -184,28 +184,28 @@ 
 			$listItemType->setStatus( $dataset['status'] );
 
 			$supplierListTypeManager->saveItem( $listItemType );
-			$listItemTypeIds[ $key ] = $listItemType->getId();
+			$listItemTypeIds[$key] = $listItemType->getId();
 		}
 
 		$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'] ) );
 			}
 
-			if( !isset( $listItemTypeIds[ $dataset['typeid'] ] ) ) {
+			if( !isset( $listItemTypeIds[$dataset['typeid']] ) ) {
 				throw new \Aimeos\MW\Setup\Exception( sprintf( 'No supplier list type ID found for "%1$s"', $dataset['typeid'] ) );
 			}
 
 			$listItem->setId( null );
-			$listItem->setParentId( $parentIds[ $dataset['parentid'] ] );
-			$listItem->setTypeId( $listItemTypeIds[ $dataset['typeid'] ] );
-			$listItem->setRefId( $refIds[ $dataset['domain'] ] [ $dataset['refid'] ] );
+			$listItem->setParentId( $parentIds[$dataset['parentid']] );
+			$listItem->setTypeId( $listItemTypeIds[$dataset['typeid']] );
+			$listItem->setRefId( $refIds[$dataset['domain']] [$dataset['refid']] );
 			$listItem->setDomain( $dataset['domain'] );
 			$listItem->setDateStart( $dataset['start'] );
 			$listItem->setDateEnd( $dataset['end'] );

lib/mshoplib/setup/unittest/MediaListAddTestData.php

Spacing +4 added lines, -4 removed lines

@@ -51,7 +51,7 @@ 
 		$this->additional->setEditor( 'core:unittest' );
 
 		$ds = DIRECTORY_SEPARATOR;
-		$path = __DIR__ . $ds . 'data' . $ds . 'media-list.php';
+		$path = __DIR__.$ds.'data'.$ds.'media-list.php';
 
 		if( ( $testdata = include( $path ) ) == false ) {
 			throw new \Aimeos\MShop\Exception( sprintf( 'No file "%1$s" found for media list domain', $path ) );
@@ -120,7 +120,7 @@ 
 
 		$refIds = [];
 		foreach( $attributeManager->searchItems( $search ) as $item ) {
-			$refIds['attribute/' . $item->getDomain() . '/' . $item->getType() . '/' . $item->getCode()] = $item->getId();
+			$refIds['attribute/'.$item->getDomain().'/'.$item->getType().'/'.$item->getCode()] = $item->getId();
 		}
 
 		return $refIds;
@@ -152,7 +152,7 @@ 
 
 		$refIds = [];
 		foreach( $textManager->searchItems( $search ) as $item ) {
-			$refIds['text/' . $item->getLabel()] = $item->getId();
+			$refIds['text/'.$item->getLabel()] = $item->getId();
 		}
 
 		return $refIds;
@@ -189,7 +189,7 @@ 
 
 		$parentIds = [];
 		foreach( $result as $item ) {
-			$parentIds['media/' . $item->getUrl()] = $item->getId();
+			$parentIds['media/'.$item->getUrl()] = $item->getId();
 		}
 
 		$medListTypes = [];

lib/mshoplib/setup/unittest/CatalogListAddTestData.php

Spacing +5 added lines, -5 removed lines

@@ -51,7 +51,7 @@ 
 		$this->additional->setEditor( 'core:unittest' );
 
 		$ds = DIRECTORY_SEPARATOR;
-		$path = __DIR__ . $ds . 'data' . $ds . 'catalog-list.php';
+		$path = __DIR__.$ds.'data'.$ds.'catalog-list.php';
 
 		if( ( $testdata = include( $path ) ) == false ) {
 			throw new \Aimeos\MShop\Exception( sprintf( 'No file "%1$s" found for catalog list domain', $path ) );
@@ -98,7 +98,7 @@ 
 
 		$refIds = [];
 		foreach( $textManager->searchItems( $search ) as $item ) {
-			$refIds['text/' . $item->getLabel()] = $item->getId();
+			$refIds['text/'.$item->getLabel()] = $item->getId();
 		}
 
 		return $refIds;
@@ -130,7 +130,7 @@ 
 
 		$refIds = [];
 		foreach( $mediaManager->searchItems( $search ) as $item ) {
-			$refIds['media/' . $item->getUrl()] = $item->getId();
+			$refIds['media/'.$item->getUrl()] = $item->getId();
 		}
 
 		return $refIds;
@@ -162,7 +162,7 @@ 
 
 		$refIds = [];
 		foreach( $productManager->searchItems( $search ) as $item ) {
-			$refIds['product/' . $item->getCode()] = $item->getId();
+			$refIds['product/'.$item->getCode()] = $item->getId();
 		}
 
 		return $refIds;
@@ -197,7 +197,7 @@ 
 
 		$parentIds = [];
 		foreach( $catalogManager->searchItems( $search ) as $item ) {
-			$parentIds['catalog/' . $item->getCode()] = $item->getId();
+			$parentIds['catalog/'.$item->getCode()] = $item->getId();
 		}
 
 		$listItemTypeIds = [];

lib/mshoplib/setup/unittest/TextListAddTestData.php

Spacing +3 added lines, -3 removed lines

@@ -51,7 +51,7 @@ 
 		$this->additional->setEditor( 'core:unittest' );
 
 		$ds = DIRECTORY_SEPARATOR;
-		$path = __DIR__ . $ds . 'data' . $ds . 'text-list.php';
+		$path = __DIR__.$ds.'data'.$ds.'text-list.php';
 
 		if( ( $testdata = include( $path ) ) == false ) {
 			throw new \Aimeos\MShop\Exception( sprintf( 'No file "%1$s" found for text list domain', $path ) );
@@ -98,7 +98,7 @@ 
 
 		$refIds = [];
 		foreach( $result as $item ) {
-			$refIds['media/' . $item->getUrl()] = $item->getId();
+			$refIds['media/'.$item->getUrl()] = $item->getId();
 		}
 
 		return $refIds;
@@ -133,7 +133,7 @@ 
 
 		$parentIds = [];
 		foreach( $textManager->searchItems( $search ) as $item ) {
-			$parentIds['text/' . $item->getLabel()] = $item->getId();
+			$parentIds['text/'.$item->getLabel()] = $item->getId();
 		}
 
 		$tListTypeIds = [];

lib/mshoplib/setup/unittest/PriceListAddTestData.php

Spacing +3 added lines, -3 removed lines

@@ -51,7 +51,7 @@ 
 		$this->additional->setEditor( 'core:unittest' );
 
 		$ds = DIRECTORY_SEPARATOR;
-		$path = __DIR__ . $ds . 'data' . $ds . 'price-list.php';
+		$path = __DIR__.$ds.'data'.$ds.'price-list.php';
 
 		if( ( $testdata = include( $path ) ) == false ) {
 			throw new \Aimeos\MShop\Exception( sprintf( 'No file "%1$s" found for price domain', $path ) );
@@ -97,7 +97,7 @@ 
 
 		$refIds = [];
 		foreach( $customerManager->searchItems( $search ) as $item ) {
-			$refIds['customer/' . $item->getCode()] = $item->getId();
+			$refIds['customer/'.$item->getCode()] = $item->getId();
 		}
 
 		return $refIds;
@@ -193,7 +193,7 @@ 
 
 		$parentIds = [];
 		foreach( $manager->searchItems( $search ) as $item ) {
-			$parentIds['price/' . $item->getDomain() . '/' . $item->getType() . '/' . $item->getValue() . '/' . $item->getCosts()] = $item->getId();
+			$parentIds['price/'.$item->getDomain().'/'.$item->getType().'/'.$item->getValue().'/'.$item->getCosts()] = $item->getId();
 		}
 
 		return $parentIds;

lib/mshoplib/setup/unittest/CustomerListAddTestData.php

Spacing +5 added lines, -5 removed lines

@@ -51,7 +51,7 @@ 
 		$this->additional->setEditor( 'core:unittest' );
 
 		$ds = DIRECTORY_SEPARATOR;
-		$path = __DIR__ . $ds . 'data' . $ds . 'customer-list.php';
+		$path = __DIR__.$ds.'data'.$ds.'customer-list.php';
 
 		if( ( $testdata = include( $path ) ) == false ) {
 			throw new \Aimeos\MShop\Exception( sprintf( 'No file "%1$s" found for customer list domain', $path ) );
@@ -98,7 +98,7 @@ 
 
 		$refIds = [];
 		foreach( $groupManager->searchItems( $search ) as $item ) {
-			$refIds['customer/group/' . $item->getCode()] = $item->getId();
+			$refIds['customer/group/'.$item->getCode()] = $item->getId();
 		}
 
 		return $refIds;
@@ -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;
@@ -162,7 +162,7 @@ 
 
 		$refIds = [];
 		foreach( $textManager->searchItems( $search ) as $item ) {
-			$refIds['text/' . $item->getLabel()] = $item->getId();
+			$refIds['text/'.$item->getLabel()] = $item->getId();
 		}
 
 		return $refIds;
@@ -198,7 +198,7 @@ 
 
 		$parentIds = [];
 		foreach( $customerManager->searchItems( $search ) as $item ) {
-			$parentIds['customer/' . $item->getCode()] = $item->getId();
+			$parentIds['customer/'.$item->getCode()] = $item->getId();
 		}
 
 		$listItemTypeIds = [];

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

Spacing +8 added lines, -8 removed lines

@@ -67,7 +67,7 @@ 
 
 	public function testSetLanguageId()
 	{
-		$return = $this->object->setLanguageId('fr');
+		$return = $this->object->setLanguageId( 'fr' );
 
 		$this->assertInstanceOf( '\Aimeos\MShop\Product\Item\Property\Iface', $return );
 		$this->assertEquals( 'fr', $this->object->getLanguageId() );
@@ -95,7 +95,7 @@ 
 
 	public function testSetTypeId()
 	{
-		$return = $this->object->setTypeId(33);
+		$return = $this->object->setTypeId( 33 );
 
 		$this->assertInstanceOf( '\Aimeos\MShop\Product\Item\Property\Iface', $return );
 		$this->assertEquals( 33, $this->object->getTypeId() );
@@ -161,14 +161,14 @@ 
 			'product.property.value' => 'value',
 		);
 
-		$unknown = $item->fromArray($list);
+		$unknown = $item->fromArray( $list );
 
-		$this->assertEquals([], $unknown);
+		$this->assertEquals( [], $unknown );
 
-		$this->assertEquals($list['product.property.parentid'], $item->getParentId());
-		$this->assertEquals($list['product.property.typeid'], $item->getTypeId());
-		$this->assertEquals($list['product.property.languageid'], $item->getLanguageId());
-		$this->assertEquals($list['product.property.value'], $item->getValue());
+		$this->assertEquals( $list['product.property.parentid'], $item->getParentId() );
+		$this->assertEquals( $list['product.property.typeid'], $item->getTypeId() );
+		$this->assertEquals( $list['product.property.languageid'], $item->getLanguageId() );
+		$this->assertEquals( $list['product.property.value'], $item->getValue() );
 		$this->assertNull( $item->getSiteId() );
 		$this->assertNull( $item->getTypeName() );
 		$this->assertNull( $item->getType() );

lib/mshoplib/tests/MShop/Plugin/Provider/ExceptionTest.php

Braces +6 added lines, -4 removed lines

@@ -27,7 +27,8 @@ 
 
 	public function test()
 	{
-		try {
+		try
+		{
 			throw new \Aimeos\MShop\Plugin\Provider\Exception( 'msg', 13, null, $this->codes );
 		}
 		catch( \Aimeos\MShop\Plugin\Provider\Exception $mppe )
@@ -38,11 +39,12 @@ 
 
 		}
 
-		try {
+		try
+		{
 			throw new \Aimeos\MShop\Plugin\Provider\Exception( 'msg2', 11 );

 		}
-		catch( \Aimeos\MShop\Plugin\Provider\Exception $e )

-		{

+		catch( \Aimeos\MShop\Plugin\Provider\Exception $e )
+		{
 			$this->assertEquals( [], $e->getErrorCodes() );

 		}

 	}