aimeos / aimeos-core

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( array(), $e->getErrorCodes() );

 		}

 	}

lib/mshoplib/tests/MShop/Service/Provider/Payment/BaseTest.php

Spacing +2 added lines, -2 removed lines

@@ -19,8 +19,8 @@
 
 		$servManager = \Aimeos\MShop\Service\Manager\Factory::createManager( $this->context );
 		$search = $servManager->createSearch();
-		$search->setConditions($search->compare('==', 'service.provider', 'Standard'));
-		$result = $servManager->searchItems($search, array('price'));
+		$search->setConditions( $search->compare( '==', 'service.provider', 'Standard' ) );
+		$result = $servManager->searchItems( $search, array( 'price' ) );
 
 		if( ( $item = reset( $result ) ) === false ) {
 			throw new \Exception( 'No order base item found' );

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

Spacing +11 added lines, -11 removed lines

@@ -95,7 +95,7 @@ 
 		$search->setConditions( $search->combine( '&&', $conditions ) );
 		$results = $this->object->searchItems( $search );
 
-		if( ( $item = reset($results) ) === false ) {
+		if( ( $item = reset( $results ) ) === false ) {
 			throw new \Exception( 'No list item found' );
 		}
 
@@ -105,11 +105,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' );
 	}
 
 
@@ -129,7 +129,7 @@ 
 			throw new \Exception( 'No item found' );
 		}
 
-		$item->setId(null);
+		$item->setId( null );
 		$item->setDomain( 'unittest' );
 		$this->object->saveItem( $item );
 		$itemSaved = $this->object->getItem( $item->getId() );
@@ -305,25 +305,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, array(), $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, array(), $total);
+		$search->setSlice( 0, 1 );
+		$results = $this->object->searchItems( $search, array(), $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/mshoplib/tests/MShop/Supplier/Manager/Lists/Type/StandardTest.php

Spacing +12 added lines, -12 removed lines

@@ -67,10 +67,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 \Exception( 'No attribute list type item found' );
 		}
 
@@ -80,14 +80,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 \Exception( 'No type item found' );
 		}
 
-		$item->setId(null);
+		$item->setId( null );
 		$item->setCode( 'unitTestSave' );
 		$this->object->saveItem( $item );
 		$itemSaved = $this->object->getItem( $item->getId() );
@@ -143,13 +143,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, array(), $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() );
 		}
 	}
@@ -157,7 +157,7 @@ 
 
 	public function testGetSubManager()
 	{
-		$this->setExpectedException('\\Aimeos\\MShop\\Exception');
-		$this->object->getSubManager('unknown');
+		$this->setExpectedException( '\\Aimeos\\MShop\\Exception' );
+		$this->object->getSubManager( 'unknown' );
 	}
 }

lib/mwlib/src/MW/Criteria/Attribute/Standard.php

Spacing +2 added lines, -2 removed lines

@@ -90,8 +90,8 @@
 
 		foreach( $required as $entry )
 		{
-			if ( !isset($params[$entry]) ) {
-				throw new \Aimeos\MW\Common\Exception( sprintf('Required parameter "%1$s" is missing', $entry) );
+			if( !isset( $params[$entry] ) ) {
+				throw new \Aimeos\MW\Common\Exception( sprintf( 'Required parameter "%1$s" is missing', $entry ) );
 			}
 		}
 

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

Braces +3 added lines, -6 removed lines

@@ -71,8 +71,7 @@ 
 
 		if( in_array( $op, $operators['combine'] ) ) {
 			return $this->createCombineExpression( $op, (array) $value );
-		}
-		else if( in_array( $op, $operators['compare'] ) ) {
+		} else if( in_array( $op, $operators['compare'] ) ) {
 			return $this->createCompareExpression( $op, (array) $value );
 		}
 
@@ -126,11 +125,9 @@ 
 
 			if( in_array( $op, $operators['combine'] ) ) {
 				$results[] = $this->createCombineExpression( $op, (array) $entry[$op] );
-			}
-			else if( in_array( $op, $operators['compare'] ) ) {
+			} else if( in_array( $op, $operators['compare'] ) ) {
 				$results[] = $this->createCompareExpression( $op, (array) $entry[$op] );
-			}
-			else {
+			} else {
 				throw new \Aimeos\MW\Common\Exception( sprintf( 'Invalid operator "%1$s"', $op ) );
 			}
 		}

lib/mwlib/src/MW/Criteria/Expression/Base.php

Braces +1 added lines, -2 removed lines

@@ -226,8 +226,7 @@
 
 					$params[] = implode( ',', $list );
 				}
-			}
-			else
+			} else
 			{
 				$params[] = $this->escape( '==', $this->getParamType( $string ), $string );
 			}

Spacing +7 added lines, -7 removed lines

@@ -32,7 +32,7 @@ 
 	 */
 	public static function createFunction( $name, array $params )
 	{
-		return $name . '(' . self::createSignature( $params ) . ')';
+		return $name.'('.self::createSignature( $params ).')';
 	}
 
 
@@ -55,9 +55,9 @@ 
 				case 'double':
 					$list[] = $param; break;
 				case 'array':
-					$list[] = '[' . self::createSignature( $param ) . ']'; break;
+					$list[] = '['.self::createSignature( $param ).']'; break;
 				default:
-					$list[] = '"' . $param . '"';
+					$list[] = '"'.$param.'"';
 			}
 		}
 
@@ -77,7 +77,7 @@ 
 	protected function isFunction( &$name, array &$params )
 	{
 		$len = strlen( $name );
-		if( $len === 0 || $name[$len-1] !== ')' ) { return false; }
+		if( $len === 0 || $name[$len - 1] !== ')' ) { return false; }
 
 		if( ( $pos = strpos( $name, '(' ) ) === false ) {
 			throw new \Aimeos\MW\Common\Exception( 'Missing opening bracket for function syntax' );
@@ -102,7 +102,7 @@ 
 			$params = $this->extractParams( $matches[1] );
 		}
 
-		$name = $namestr . '()';
+		$name = $namestr.'()';
 		return true;
 	}
 
@@ -130,7 +130,7 @@ 
 			$count = count( $params );
 
 			for( $i = 0; $i < $count; $i++ ) {
-				$find[$i] = '$' . ( $i + 1 );
+				$find[$i] = '$'.( $i + 1 );
 			}
 
 			return str_replace( $find, $params, $transname );
@@ -168,7 +168,7 @@ 
 	 */
 	protected function setPlugins( array $plugins )
 	{
-		\Aimeos\MW\Common\Base::checkClassList('\\Aimeos\\MW\\Criteria\\Plugin\\Iface', $plugins);
+		\Aimeos\MW\Common\Base::checkClassList( '\\Aimeos\\MW\\Criteria\\Plugin\\Iface', $plugins );
 
 		$this->plugins = $plugins;
 	}

lib/mwlib/src/MW/Criteria/Expression/Sort/SQL.php

Braces +2 added lines, -4 removed lines

@@ -154,12 +154,10 @@
 			}
 
 			return \Aimeos\MW\DB\Statement\Base::PARAM_STR;
-		}
-		else if( strpos( $item, '.' ) !== false )
+		} else if( strpos( $item, '.' ) !== false )
 		{
 			return \Aimeos\MW\DB\Statement\Base::PARAM_FLOAT;
-		}
-		else if( ctype_digit( $item ) !== false )
+		} else if( ctype_digit( $item ) !== false )
 		{
 			return \Aimeos\MW\DB\Statement\Base::PARAM_INT;
 		}

Spacing +3 added lines, -3 removed lines

@@ -102,7 +102,7 @@ 
 			throw new \Aimeos\MW\Common\Exception( sprintf( 'Invalid name "%1$s"', $name ) );
 		}
 
-		return $transname . ' ' . self::$operators[$this->operator];
+		return $transname.' '.self::$operators[$this->operator];
 	}
 
 
@@ -128,10 +128,10 @@ 
 				$value = (float) $value; break;
 			case \Aimeos\MW\DB\Statement\Base::PARAM_STR:
 				if( $operator == '~=' ) {
-					$value = '\'%' . $this->conn->escape( $value ) . '%\''; break;
+					$value = '\'%'.$this->conn->escape( $value ).'%\''; break;
 				}
 			default:
-				$value = '\'' . $this->conn->escape( $value ) . '\'';
+				$value = '\''.$this->conn->escape( $value ).'\'';
 		}
 
 		return (string) $value;

lib/mwlib/tests/MW/Criteria/Expression/Combine/PHPTest.php

Spacing +1 added lines, -1 removed lines

@@ -72,7 +72,7 @@
 		);
 
 		$expr1 = array();
-		$expr1[] = new \Aimeos\MW\Criteria\Expression\Compare\PHP( '==', 'listitem', array('a', 'b', 'c') );
+		$expr1[] = new \Aimeos\MW\Criteria\Expression\Compare\PHP( '==', 'listitem', array( 'a', 'b', 'c' ) );
 		$expr1[] = new \Aimeos\MW\Criteria\Expression\Compare\PHP( '==', 'stringvar', 'value' );