aimeos / aimeos-core

lib/mwlib/tests/MW/Process/NoneTest.php

Braces +3 added lines, -1 removed lines

@@ -16,7 +16,9 @@
 	{
 		$object = new \Aimeos\MW\Process\None();
 
-		$this->assertInstanceOf( '\Aimeos\MW\Process\Iface', $object->start( function() {}, [] ) );
+		$this->assertInstanceOf( '\Aimeos\MW\Process\Iface', $object->start( function()
+		{
+}, [] ) );
 	}
 
 

lib/mshoplib/src/MShop/Catalog/Manager/Standard.php

Braces +12 added lines, -8 removed lines

@@ -647,9 +647,12 @@ 
 
 		foreach( $sitePath as $siteId )
 		{
-			try {
+			try
+			{
 				$path = $this->createTreeManager( $siteId )->getPath( $id );
-			} catch( \Exception $e ) {
+			}
+			catch( \Exception $e )
+			{
 				continue;
 			}
 
@@ -684,9 +687,12 @@ 
 
 		foreach( $sitePath as $siteId )
 		{
-			try {
+			try
+			{
 				$node = $this->createTreeManager( $siteId )->getNode( $id, $level, $criteria );
-			} catch( \Exception $e ) {
+			}
+			catch( \Exception $e )
+			{
 				continue;
 			}
 
@@ -807,8 +813,7 @@ 
 				 * @see mshop/catalog/manager/standard/insert-usage/ansi
 				 */
 				$path = 'mshop/catalog/manager/standard/update-usage';
-			}
-			else
+			} else
 			{
 				/** mshop/catalog/manager/standard/insert-usage/mysql
 				 * Updates the config, editor, ctime and mtime value of an inserted record
@@ -864,8 +869,7 @@ 
 			{
 				$stmt->bind( 5, $siteid, \Aimeos\MW\DB\Statement\Base::PARAM_INT );
 				$stmt->bind( 6, $id, \Aimeos\MW\DB\Statement\Base::PARAM_INT );
-			}
-			else
+			} else
 			{
 				$stmt->bind( 5, $date ); // ctime
 				$stmt->bind( 6, $siteid, \Aimeos\MW\DB\Statement\Base::PARAM_INT );

Spacing +1 added lines, -1 removed lines

@@ -612,7 +612,7 @@
 			$cfgPathCount = 'mshop/catalog/manager/standard/count';
 
 			if( $search->getSortations() === [] ) {
-				$search->setSortations( [$search->sort( '+', 'catalog.left')] );
+				$search->setSortations( [$search->sort( '+', 'catalog.left' )] );
 			}
 
 			$results = $this->searchItemsBase( $conn, $search, $cfgPathSearch, $cfgPathCount, $required, $total, $level );

Doc Comments +3 added lines, -2 removed lines

@@ -271,7 +271,7 @@ 
 	/**
 	 * Deletes the item specified by its ID.
 	 *
-	 * @param mixed $id ID of the item object
+	 * @param string $id ID of the item object
 	 */
 	public function deleteItem( $id )
 	{
@@ -294,7 +294,7 @@ 
 	/**
 	 * Removes multiple items specified by ids in the array.
 	 *
-	 * @param array $ids List of IDs
+	 * @param integer[] $ids List of IDs
 	 */
 	public function deleteItems( array $ids )
 	{
@@ -478,6 +478,7 @@ 
 	 * @param \Aimeos\MW\Criteria\Iface $search Search criteria object
 	 * @param string[] $ref List of domains to fetch list items and referenced items for
 	 * @param integer|null &$total Number of items that are available in total
+	 * @param integer $total
 	 * @return array List of items implementing \Aimeos\MShop\Common\Item\Iface
 	 */
 	public function searchItems( \Aimeos\MW\Criteria\Iface $search, array $ref = [], &$total = null )

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

Spacing +14 added lines, -14 removed lines

@@ -19,13 +19,13 @@ 
 	{
 		$this->editor = \TestHelperMShop::getContext()->getEditor();
 		$manager = \Aimeos\MShop\Product\Manager\Factory::createManager( \TestHelperMShop::getContext() );
-		$this->object = $manager->getSubManager( 'property' )->getSubManager('type');
+		$this->object = $manager->getSubManager( 'property' )->getSubManager( 'type' );
 	}
 
 
 	protected function tearDown()
 	{
-		unset($this->object);
+		unset( $this->object );
 	}
 
 
@@ -68,7 +68,7 @@ 
 
 		$results = $this->object->searchItems( $search );
 
-		if( ($expected = reset($results) ) === false )
+		if( ( $expected = reset( $results ) ) === false )
 		{
 			throw new \RuntimeException( 'No property type item found.' );
 		}
@@ -90,13 +90,13 @@ 
 	{
 		$search = $this->object->createSearch();
 		$search->setConditions( $search->compare( '==', 'product.property.type.editor', $this->editor ) );
-		$results = $this->object->searchItems($search);
+		$results = $this->object->searchItems( $search );
 
-		if( ( $item = reset($results) ) === false ) {
+		if( ( $item = reset( $results ) ) === false ) {
 			throw new \RuntimeException( 'No type item found' );
 		}
 
-		$item->setId(null);
+		$item->setId( null );
 		$item->setCode( 'unitTestSave' );
 		$resultSaved = $this->object->saveItem( $item );
 		$itemSaved = $this->object->getItem( $item->getId() );
@@ -156,24 +156,24 @@ 
 		$expr[] = $search->compare( '>=', 'product.property.type.ctime', '1970-01-01 00:00:00' );
 		$expr[] = $search->compare( '==', 'product.property.type.editor', $this->editor );
 
-		$search->setConditions( $search->combine('&&', $expr) );
+		$search->setConditions( $search->combine( '&&', $expr ) );
 		$results = $this->object->searchItems( $search, [], $total );
 		$this->assertEquals( 1, count( $results ) );
 
 
 		$search = $this->object->createSearch();
 		$conditions = array(
-			$search->compare( '=~', 'product.property.type.code', 'package-'),
+			$search->compare( '=~', 'product.property.type.code', 'package-' ),
 			$search->compare( '==', 'product.property.type.editor', $this->editor )
 		);
-		$search->setConditions( $search->combine('&&', $conditions ) );
-		$search->setSlice(0, 1);
-		$items = $this->object->searchItems( $search, [], $total);
+		$search->setConditions( $search->combine( '&&', $conditions ) );
+		$search->setSlice( 0, 1 );
+		$items = $this->object->searchItems( $search, [], $total );
 
 		$this->assertEquals( 1, count( $items ) );
 		$this->assertEquals( 4, $total );
 
-		foreach($items as $itemId => $item) {
+		foreach( $items as $itemId => $item ) {
 			$this->assertEquals( $itemId, $item->getId() );
 		}
 	}
@@ -181,7 +181,7 @@ 
 
 	public function testGetSubManager()
 	{
-		$this->setExpectedException('\\Aimeos\\MShop\\Exception');
-		$this->object->getSubManager('unknown');
+		$this->setExpectedException( '\\Aimeos\\MShop\\Exception' );
+		$this->object->getSubManager( 'unknown' );
 	}
 }

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

Spacing +13 added lines, -13 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();
 		$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' ) );
 		}
 
@@ -166,7 +166,7 @@ 
 		$expr[] = $search->compare( '>=', 'product.property.type.ctime', '1970-01-01 00:00:00' );
 		$expr[] = $search->compare( '==', 'product.property.type.editor', $this->editor );
 
-		$search->setConditions( $search->combine('&&', $expr) );
+		$search->setConditions( $search->combine( '&&', $expr ) );
 		$results = $this->object->searchItems( $search, [], $total );
 		$this->assertEquals( 1, count( $results ) );
 
@@ -177,13 +177,13 @@ 
 			$search->compare( '==', 'product.property.editor', $this->editor )
 		);
 		$search->setConditions( $search->combine( '&&', $conditions ) );
-		$search->setSlice(0, 1);
+		$search->setSlice( 0, 1 );
 		$items = $this->object->searchItems( $search, [], $total );
 
 		$this->assertEquals( 1, count( $items ) );
 		$this->assertEquals( 8, $total );
 
-		foreach($items as $itemId => $item) {
+		foreach( $items as $itemId => $item ) {
 			$this->assertEquals( $itemId, $item->getId() );
 		}
 	}
@@ -191,17 +191,17 @@ 
 
 	public function testGetSubManager()
 	{
-		$this->assertInstanceOf( '\\Aimeos\\MShop\\Common\\Manager\\Iface', $this->object->getSubManager('type') );
-		$this->assertInstanceOf( '\\Aimeos\\MShop\\Common\\Manager\\Iface', $this->object->getSubManager('type', 'Standard') );
+		$this->assertInstanceOf( '\\Aimeos\\MShop\\Common\\Manager\\Iface', $this->object->getSubManager( 'type' ) );
+		$this->assertInstanceOf( '\\Aimeos\\MShop\\Common\\Manager\\Iface', $this->object->getSubManager( 'type', 'Standard' ) );
 
-		$this->setExpectedException('\\Aimeos\\MShop\\Exception');
-		$this->object->getSubManager('unknown');
+		$this->setExpectedException( '\\Aimeos\\MShop\\Exception' );
+		$this->object->getSubManager( 'unknown' );
 	}
 
 
 	public function testGetSubManagerInvalidName()
 	{
-		$this->setExpectedException('\\Aimeos\\MShop\\Exception');
-		$this->object->getSubManager('type', 'unknown');
+		$this->setExpectedException( '\\Aimeos\\MShop\\Exception' );
+		$this->object->getSubManager( 'type', 'unknown' );
 	}
 }

lib/mshoplib/tests/MShop/Supplier/Manager/Lists/Type/StandardTest.php

Spacing +12 added lines, -12 removed lines

@@ -53,10 +53,10 @@ 
 	public function testGetItem()
 	{
 		$search = $this->object->createSearch();
-		$search->setConditions( $search->compare( '==', 'supplier.lists.type.editor', $this->editor) );
-		$results = $this->object->searchItems($search);
+		$search->setConditions( $search->compare( '==', 'supplier.lists.type.editor', $this->editor ) );
+		$results = $this->object->searchItems( $search );
 
-		if( ( $expected = reset($results) ) === false ) {
+		if( ( $expected = reset( $results ) ) === false ) {
 			throw new \RuntimeException( 'No attribute list type item found' );
 		}
 
@@ -74,14 +74,14 @@ 
 	public function testSaveUpdateDeleteItem()
 	{
 		$search = $this->object->createSearch();
-		$search->setConditions( $search->compare( '==', 'supplier.lists.type.editor', $this->editor) );
-		$results = $this->object->searchItems($search);
+		$search->setConditions( $search->compare( '==', 'supplier.lists.type.editor', $this->editor ) );
+		$results = $this->object->searchItems( $search );
 
-		if( ( $item = reset($results) ) === false ) {
+		if( ( $item = reset( $results ) ) === false ) {
 			throw new \RuntimeException( 'No type item found' );
 		}
 
-		$item->setId(null);
+		$item->setId( null );
 		$item->setCode( 'unitTestSave' );
 		$resultSaved = $this->object->saveItem( $item );
 		$itemSaved = $this->object->getItem( $item->getId() );
@@ -140,13 +140,13 @@ 
 		$expr[] = $search->compare( '>=', 'supplier.lists.type.ctime', '1970-01-01 00:00:00' );
 		$expr[] = $search->compare( '==', 'supplier.lists.type.editor', $this->editor );
 
-		$search->setConditions( $search->combine('&&', $expr) );
-		$search->setSlice(0, 1);
+		$search->setConditions( $search->combine( '&&', $expr ) );
+		$search->setSlice( 0, 1 );
 		$results = $this->object->searchItems( $search, [], $total );
 		$this->assertEquals( 1, count( $results ) );
 		$this->assertEquals( 1, $total );
 
-		foreach($results as $itemId => $item) {
+		foreach( $results as $itemId => $item ) {
 			$this->assertEquals( $itemId, $item->getId() );
 		}
 	}
@@ -154,7 +154,7 @@ 
 
 	public function testGetSubManager()
 	{
-		$this->setExpectedException('\\Aimeos\\MShop\\Exception');
-		$this->object->getSubManager('unknown');
+		$this->setExpectedException( '\\Aimeos\\MShop\\Exception' );
+		$this->object->getSubManager( 'unknown' );
 	}
 }

lib/mshoplib/tests/MShop/Supplier/Manager/Lists/StandardTest.php

Spacing +11 added lines, -11 removed lines

@@ -80,7 +80,7 @@ 
 		$search->setConditions( $search->combine( '&&', $conditions ) );
 		$results = $this->object->searchItems( $search );
 
-		if( ( $item = reset($results) ) === false ) {
+		if( ( $item = reset( $results ) ) === false ) {
 			throw new \RuntimeException( 'No list item found' );
 		}
 
@@ -91,11 +91,11 @@ 
 
 	public function testGetSubManager()
 	{
-		$this->assertInstanceOf( '\\Aimeos\\MShop\\Common\\Manager\\Iface', $this->object->getSubManager('type') );
-		$this->assertInstanceOf( '\\Aimeos\\MShop\\Common\\Manager\\Iface', $this->object->getSubManager('type', 'Standard') );
+		$this->assertInstanceOf( '\\Aimeos\\MShop\\Common\\Manager\\Iface', $this->object->getSubManager( 'type' ) );
+		$this->assertInstanceOf( '\\Aimeos\\MShop\\Common\\Manager\\Iface', $this->object->getSubManager( 'type', 'Standard' ) );
 
-		$this->setExpectedException('\\Aimeos\\MShop\\Exception');
-		$this->object->getSubManager('unknown');
+		$this->setExpectedException( '\\Aimeos\\MShop\\Exception' );
+		$this->object->getSubManager( 'unknown' );
 	}
 
 
@@ -122,7 +122,7 @@ 
 			throw new \RuntimeException( 'No item found' );
 		}
 
-		$item->setId(null);
+		$item->setId( null );
 		$item->setDomain( 'unittest' );
 		$resultSaved = $this->object->saveItem( $item );
 		$itemSaved = $this->object->getItem( $item->getId() );
@@ -301,25 +301,25 @@ 
 		$expr[] = $search->compare( '>=', 'supplier.lists.ctime', '1970-01-01 00:00:00' );
 		$expr[] = $search->compare( '==', 'supplier.lists.editor', $this->editor );
 
-		$search->setConditions( $search->combine('&&', $expr) );
+		$search->setConditions( $search->combine( '&&', $expr ) );
 		$results = $this->object->searchItems( $search, [], $total );
 		$this->assertEquals( 1, count( $results ) );
 
 
 		//search with base criteria
-		$search = $this->object->createSearch(true);
+		$search = $this->object->createSearch( true );
 		$expr = array(
 			$search->compare( '==', 'supplier.lists.domain', 'text' ),
 			$search->compare( '==', 'supplier.lists.editor', $this->editor ),
 			$search->getConditions(),
 		);
 		$search->setConditions( $search->combine( '&&', $expr ) );
-		$search->setSlice(0, 1);
-		$results = $this->object->searchItems($search, [], $total);
+		$search->setSlice( 0, 1 );
+		$results = $this->object->searchItems( $search, [], $total );
 		$this->assertEquals( 1, count( $results ) );
 		$this->assertEquals( 3, $total );
 
-		foreach($results as $itemId => $item) {
+		foreach( $results as $itemId => $item ) {
 			$this->assertEquals( $itemId, $item->getId() );
 		}
 	}

lib/mwlib/src/MW/Criteria/MySQL.php

Spacing +2 added lines, -2 removed lines

@@ -44,9 +44,9 @@
 						$len = strlen( $part );
 
 						if( $len > 3 ) {
-							$str .= ' +' . $part . '*';
+							$str .= ' +'.$part.'*';
 						} else if( $len > 0 ) {
-							$str .= ' ' . $part . '*';
+							$str .= ' '.$part.'*';
 						}
 					}
 

lib/mwlib/src/MW/Criteria/PgSQL.php

Spacing +1 added lines, -1 removed lines

@@ -38,7 +38,7 @@
 					$regex = '/(\s|\&|\%|\?|\#|\=|\{|\}|\||\\\\|\~|\[|\]|\`|\^|\/|\-|\+|\>|\<|\(|\)|\*|\:|\"|\!|\§|\$|\'|\;|\.|\,|\@)+/';
 					$search = trim( preg_replace( $regex, ' ', $params[2] ) );
 
-					$params[2] = implode( ':* & ', explode( ' ', $search ) ) . ':*';
+					$params[2] = implode( ':* & ', explode( ' ', $search ) ).':*';
 				}
 				break;
 		}

lib/mshoplib/src/MShop/Service/Provider/Payment/DirectDebit.php

Spacing +3 added lines, -3 removed lines

@@ -78,10 +78,10 @@ 
 			$address = $basket->getAddress( \Aimeos\MShop\Order\Item\Base\Address\Base::TYPE_PAYMENT );
 
 			if( ( $fn = $address->getFirstname() ) !== '' && ( $ln = $address->getLastname() ) !== '' ) {
-				$feconfig['directdebit.accountowner']['default'] = $fn . ' ' . $ln;
+				$feconfig['directdebit.accountowner']['default'] = $fn.' '.$ln;
 			}
 		}
-		catch( \Aimeos\MShop\Order\Exception $e ) { ; } // If address isn't available
+		catch( \Aimeos\MShop\Order\Exception $e ) {; } // If address isn't available
 
 		return $this->getConfigItems( $feconfig );
 	}
@@ -119,7 +119,7 @@ 
 			$len = strlen( $value );
 			$xstr = ( $len > 3 ? str_repeat( 'X', $len - 3 ) : '' );
 
-			$attrItem->setValue( $xstr . substr( $value, -3 ) );
+			$attrItem->setValue( $xstr.substr( $value, -3 ) );
 			$orderServiceItem->setAttributeItem( $attrItem );
 		}
 	}

Braces +3 added lines, -1 removed lines

@@ -81,7 +81,9 @@
 				$feconfig['directdebit.accountowner']['default'] = $fn . ' ' . $ln;
 			}
 		}
-		catch( \Aimeos\MShop\Order\Exception $e ) { ; } // If address isn't available
+		catch( \Aimeos\MShop\Order\Exception $e )
+		{
+; } // If address isn't available
 
 		return $this->getConfigItems( $feconfig );
 	}