aimeos / aimeos-core

lib/mwlib/tests/MW/Logger/FileTest.php

Spacing +3 added lines, -3 removed lines

@@ -58,13 +58,13 @@
 		$this->assertEquals( 'error', $msg[4] );
 
 
-		$this->setExpectedException('\\Aimeos\\MW\\Logger\\Exception');
-		$this->object->log( 'wrong log level', -1);
+		$this->setExpectedException( '\\Aimeos\\MW\\Logger\\Exception' );
+		$this->object->log( 'wrong log level', -1 );
 	}
 
 	public function testScalarLog()
 	{
-		$this->object->log( array ( 'scalar', 'errortest' ) );
+		$this->object->log( array( 'scalar', 'errortest' ) );
 
 		if( !file_exists( $this->filename ) ) {
 			throw new \RuntimeException( 'No test file found' );

lib/mwlib/tests/MW/DB/PDOTest.php

Braces +5 added lines, -2 removed lines

@@ -266,10 +266,13 @@
 
 		$conn = $this->object->acquire();
 
-		try {
+		try
+		{
 			$stmt = $conn->create( $sqlinsert );
 			$stmt->execute();
-		} catch ( \Aimeos\MW\DB\Exception $de ) {
+		}
+		catch ( \Aimeos\MW\DB\Exception $de )
+		{
 			$this->object->release( $conn );
 			return;
 		}

Spacing +22 added lines, -22 removed lines

@@ -207,7 +207,7 @@ 
 		$conn = $this->object->acquire();
 
 		$value = "(\\')";
-		$sqlinsert = str_replace( ':value', '\'' . $conn->escape( $value ) . '\'', $sqlinsert );
+		$sqlinsert = str_replace( ':value', '\''.$conn->escape( $value ).'\'', $sqlinsert );
 		$stmt = $conn->create( $sqlinsert );
 		$stmt->execute()->finish();
 
@@ -235,12 +235,12 @@ 
 
 	public function testStmtSimpleBindTwo()
 	{
-		$sqlinsert2 =  'INSERT INTO "mw_unit_test" ("id", "name") VALUES (?, ?)';
+		$sqlinsert2 = 'INSERT INTO "mw_unit_test" ("id", "name") VALUES (?, ?)';
 
 		$conn = $this->object->acquire();
 
 		$stmt2 = $conn->create( $sqlinsert2 );
-		$stmt2->bind( 1, 1, \Aimeos\MW\DB\Statement\Base::PARAM_INT);
+		$stmt2->bind( 1, 1, \Aimeos\MW\DB\Statement\Base::PARAM_INT );
 		$stmt2->bind( 2, 0.12 );
 		$stmt2->execute()->finish();
 
@@ -252,12 +252,12 @@ 
 
 	public function testStmtSimpleBindThree()
 	{
-		$sqlinsert3 =  'INSERT INTO "mw_unit_test" ("name", "id") VALUES (\'?te?st?\', ?)';
+		$sqlinsert3 = 'INSERT INTO "mw_unit_test" ("name", "id") VALUES (\'?te?st?\', ?)';
 
 		$conn = $this->object->acquire();
 
 		$stmt2 = $conn->create( $sqlinsert3 );
-		$stmt2->bind( 1, 1, \Aimeos\MW\DB\Statement\Base::PARAM_INT);
+		$stmt2->bind( 1, 1, \Aimeos\MW\DB\Statement\Base::PARAM_INT );
 		$stmt2->execute()->finish();
 
 		$this->object->release( $conn );
@@ -268,15 +268,15 @@ 
 
 	public function testStmtSimpleInvalidBindParamType()
 	{
-		$sqlinsert2 =  'INSERT INTO "mw_unit_test" ("id", "name") VALUES (?, ?)';
+		$sqlinsert2 = 'INSERT INTO "mw_unit_test" ("id", "name") VALUES (?, ?)';
 
 		$conn = $this->object->acquire();
 
 		try
 		{
 			$stmt2 = $conn->create( $sqlinsert2 );
-			$stmt2->bind( 1, 1, \Aimeos\MW\DB\Statement\Base::PARAM_INT);
-			$stmt2->bind( 2, 0.15, 123);
+			$stmt2->bind( 1, 1, \Aimeos\MW\DB\Statement\Base::PARAM_INT );
+			$stmt2->bind( 2, 0.15, 123 );
 			$stmt2->execute();
 		}
 		catch( \Aimeos\MW\DB\Exception $de )
@@ -286,7 +286,7 @@ 
 		}
 
 		$this->object->release( $conn );
-		$this->fail('An expected exception has not been raised');
+		$this->fail( 'An expected exception has not been raised' );
 	}
 
 
@@ -299,13 +299,13 @@ 
 		try {
 			$stmt = $conn->create( $sqlinsert );
 			$stmt->execute();
-		} catch ( \Aimeos\MW\DB\Exception $de ) {
+		} catch( \Aimeos\MW\DB\Exception $de ) {
 			$this->object->release( $conn );
 			return;
 		}
 
 		$this->object->release( $conn );
-		$this->fail('An expected exception has not been raised');
+		$this->fail( 'An expected exception has not been raised' );
 	}
 
 
@@ -408,7 +408,7 @@ 
 
 	public function testWrongFieldType()
 	{
-		$this->setExpectedException('\\Aimeos\\MW\\DB\\Exception');
+		$this->setExpectedException( '\\Aimeos\\MW\\DB\\Exception' );
 		$sqlinsert = 'INSERT INTO "mw_unit_test" ("id", "name") VALUES (?, ?)';
 
 		$conn = $this->object->acquire();
@@ -420,7 +420,7 @@ 
 			$stmt->bind( 2, 'test', 123 );
 			$stmt->execute();
 		}
-		catch ( \Aimeos\MW\DB\Exception $e )
+		catch( \Aimeos\MW\DB\Exception $e )
 		{
 			$this->object->release( $conn );
 			throw $e;
@@ -444,13 +444,13 @@ 
 
 		$conn = $this->object->acquire();
 
-		$this->setExpectedException('\\Aimeos\\MW\\DB\\Exception');
+		$this->setExpectedException( '\\Aimeos\\MW\\DB\\Exception' );
 
 		try
 		{
 			$conn->create( $sql )->execute()->finish();
 		}
-		catch ( \Aimeos\MW\DB\Exception $e )
+		catch( \Aimeos\MW\DB\Exception $e )
 		{
 			$this->object->release( $conn );
 			throw $e;
@@ -462,13 +462,13 @@ 
 	{
 		$conn = $this->object->acquire();
 
-		$this->setExpectedException('\\Aimeos\\MW\\DB\\Exception');
+		$this->setExpectedException( '\\Aimeos\\MW\\DB\\Exception' );
 
 		try
 		{
 			$conn->create( 'SELECT *' )->execute()->finish();
 		}
-		catch ( \Aimeos\MW\DB\Exception $e )
+		catch( \Aimeos\MW\DB\Exception $e )
 		{
 			$this->object->release( $conn );
 			throw $e;
@@ -482,13 +482,13 @@ 
 
 		$conn = $this->object->acquire();
 
-		$this->setExpectedException('\\Aimeos\\MW\\DB\\Exception');
+		$this->setExpectedException( '\\Aimeos\\MW\\DB\\Exception' );
 
 		try
 		{
 			$conn->create( $sql, 123 );
 		}
-		catch (\Aimeos\MW\DB\Exception $e)
+		catch( \Aimeos\MW\DB\Exception $e )
 		{
 			$this->object->release( $conn );
 			throw $e;
@@ -500,20 +500,20 @@ 
 	{
 		$mock = $this->getMockBuilder( '\Aimeos\MW\DB\Connection\Iface' )->getMock();
 
-		$this->setExpectedException('\\Aimeos\\MW\\DB\\Exception');
+		$this->setExpectedException( '\\Aimeos\\MW\\DB\\Exception' );
 		$this->object->release( $mock );
 	}
 
 
 	public function testDBFactory()
 	{
-		$this->assertInstanceOf('\\Aimeos\\MW\\DB\\Manager\\Iface', $this->object);
+		$this->assertInstanceOf( '\\Aimeos\\MW\\DB\\Manager\\Iface', $this->object );
 	}
 
 
 	public function testFactoryFail()
 	{
-		$this->setExpectedException('\\Aimeos\\MW\\DB\\Exception');
+		$this->setExpectedException( '\\Aimeos\\MW\\DB\\Exception' );
 		\Aimeos\MW\DB\Factory::createManager( \TestHelperMw::getConfig(), 'notDefined' );
 	}
 }

lib/mwlib/src/MW/Setup/Task/Base.php

Indentation +2 added lines, -2 removed lines

@@ -63,7 +63,7 @@ 
 	 * Undo all schema changes and migrate data back
 	 *
 	 * @return void
-	*/
+	 */
 	public function rollback()
 	{
 	}
@@ -73,7 +73,7 @@ 
 	 * Cleans up old data required for roll back
 	 *
 	 * @return void
-	*/
+	 */
 	public function clean()
 	{
 	}

Spacing +13 added lines, -13 removed lines

@@ -221,11 +221,11 @@ 
 	protected function msg( $msg, $level = 0 )
 	{
 		$pre = '';
-		for( $i = 0; $i < 2*$level; $i++ ) {
+		for( $i = 0; $i < 2 * $level; $i++ ) {
 			$pre .= ' ';
 		}
 
-		echo str_pad( $pre . $msg, 70 );
+		echo str_pad( $pre.$msg, 70 );
 	}
 
 
@@ -236,7 +236,7 @@ 
 	 */
 	protected function status( $status )
 	{
-		echo $status . PHP_EOL;
+		echo $status.PHP_EOL;
 	}
 
 
@@ -252,11 +252,11 @@ 
 		$matches = [];
 
 		$regex = '/CREATE TABLE \"?([a-zA-Z0-9_]+)\"? .*(\n\n|$)/sU';
-		if ( preg_match_all($regex, $content, $matches, PREG_SET_ORDER) === false ) {
-			throw new \Aimeos\MW\Setup\Exception('Unable to get table definitions');
+		if( preg_match_all( $regex, $content, $matches, PREG_SET_ORDER ) === false ) {
+			throw new \Aimeos\MW\Setup\Exception( 'Unable to get table definitions' );
 		}
 
-		foreach ( $matches as $match ) {
+		foreach( $matches as $match ) {
 			$defs[$match[1]] = $match[0];
 		}
 
@@ -275,12 +275,12 @@ 
 		$defs = [];
 		$matches = [];
 
-		if ( preg_match_all('/CREATE [a-zA-Z]* ?INDEX \"?([a-zA-Z0-9_]+)\"? ON \"?([a-zA-Z0-9_]+)\"? .+(\n\n|$)/sU', $content, $matches, PREG_SET_ORDER) === false ) {
-			throw new \Aimeos\MW\Setup\Exception('Unable to get index definitions');
+		if( preg_match_all( '/CREATE [a-zA-Z]* ?INDEX \"?([a-zA-Z0-9_]+)\"? ON \"?([a-zA-Z0-9_]+)\"? .+(\n\n|$)/sU', $content, $matches, PREG_SET_ORDER ) === false ) {
+			throw new \Aimeos\MW\Setup\Exception( 'Unable to get index definitions' );
 		}
 
-		foreach ( $matches as $match ) {
-			$name = $match[2] . '.' . $match[1];
+		foreach( $matches as $match ) {
+			$name = $match[2].'.'.$match[1];
 			$defs[$name] = $match[0];
 		}
 
@@ -300,11 +300,11 @@ 
 		$matches = [];
 
 		$regex = '/CREATE TRIGGER \"?([a-zA-Z0-9_]+)\"? .*(\n\n|$)/sU';
-		if ( preg_match_all($regex, $content, $matches, PREG_SET_ORDER) === false ) {
-			throw new \Aimeos\MW\Setup\Exception('Unable to get trigger definitions');
+		if( preg_match_all( $regex, $content, $matches, PREG_SET_ORDER ) === false ) {
+			throw new \Aimeos\MW\Setup\Exception( 'Unable to get trigger definitions' );
 		}
 
-		foreach ( $matches as $match ) {
+		foreach( $matches as $match ) {
 			$defs[$match[1]] = $match[0];
 		}
 

lib/mwlib/src/MW/Media/Image/Imagick.php

Braces +5 added lines, -2 removed lines

@@ -112,9 +112,12 @@
 			}
 		}
 
-		try {
+		try
+		{
 			$this->image->resizeImage( $width, $height, \Imagick::FILTER_CUBIC, 0.8 );
-		} catch( \Exception $e ) {
+		}
+		catch( \Exception $e )
+		{
 			throw new \Aimeos\MW\Media\Exception( $e->getMessage() );
 		}
 

lib/mwlib/src/MW/View/Standard.php

Spacing +4 added lines, -4 removed lines

@@ -65,12 +65,12 @@ 
 		{
 			if( ctype_alnum( $name ) === false )
 			{
-				$classname = is_string( $name ) ? '\\Aimeos\\MW\\View\\Helper\\' . $name : '<not a string>';
+				$classname = is_string( $name ) ? '\\Aimeos\\MW\\View\\Helper\\'.$name : '<not a string>';
 				throw new \Aimeos\MW\View\Exception( sprintf( 'Invalid characters in class name "%1$s"', $classname ) );
 			}
 
 			$iface = '\\Aimeos\\MW\\View\\Helper\\Iface';
-			$classname = '\\Aimeos\\MW\\View\\Helper\\' . ucfirst( $name ) . '\\Standard';
+			$classname = '\\Aimeos\\MW\\View\\Helper\\'.ucfirst( $name ).'\\Standard';
 
 			if( class_exists( $classname ) === false ) {
 				throw new \Aimeos\MW\View\Exception( sprintf( 'Class "%1$s" not available', $classname ) );
@@ -221,7 +221,7 @@ 
 
 		foreach( $this->engines as $fileext => $engine )
 		{
-			if( substr_compare( $filepath, $fileext, -strlen( $fileext ) ) ===0 ) {
+			if( substr_compare( $filepath, $fileext, -strlen( $fileext ) ) === 0 ) {
 				return $engine->render( $this, $filepath, $this->values );
 			}
 		}
@@ -273,7 +273,7 @@ 
 			{
 				foreach( $relPaths as $relPath )
 				{
-					$absPath = $path . $ds . $relPath . $ds . $file;
+					$absPath = $path.$ds.$relPath.$ds.$file;
 					if( $ds !== '/' ) {
 						$absPath = str_replace( '/', $ds, $absPath );
 					}

lib/mwlib/src/MW/View/Helper/Content/Standard.php

Spacing +1 added lines, -1 removed lines

@@ -54,7 +54,7 @@
 	public function transform( $url )
 	{
 		if( strncmp( $url, 'http', 4 ) !== 0 && strncmp( $url, 'data:', 5 ) !== 0 ) {
-			$url = $this->baseurl . ( $url && $url[0] === '/' ? $url : '/' . $url );
+			$url = $this->baseurl.( $url && $url[0] === '/' ? $url : '/'.$url );
 		}
 
 		return $this->enc->attr( $url );

lib/mwlib/src/MW/DB/Result/DBAL.php

Spacing +3 added lines, -3 removed lines

@@ -47,7 +47,7 @@ 
 	{
 		try {
 			return $this->statement->rowCount();
-		} catch ( \Doctrine\DBAL\DBALException $e ) {
+		} catch( \Doctrine\DBAL\DBALException $e ) {
 			throw new \Aimeos\MW\DB\Exception( $e->getMessage(), $e->getCode(), join( ', ', $this->statement->errorInfo() ) );
 		}
 	}
@@ -64,7 +64,7 @@ 
 	{
 		try {
 			return $this->statement->fetch( $this->style[$style] );
-		} catch ( \Doctrine\DBAL\DBALException $e ) {
+		} catch( \Doctrine\DBAL\DBALException $e ) {
 			throw new \Aimeos\MW\DB\Exception( $e->getMessage(), $e->getCode(), join( ', ', $this->statement->errorInfo() ) );
 		}
 	}
@@ -79,7 +79,7 @@ 
 	{
 		try {
 			$this->statement->closeCursor();
-		} catch ( \Doctrine\DBAL\DBALException $e ) {
+		} catch( \Doctrine\DBAL\DBALException $e ) {
 			throw new \Aimeos\MW\DB\Exception( $e->getMessage(), $e->getCode(), join( ', ', $this->statement->errorInfo() ) );
 		}
 	}

Braces +15 added lines, -6 removed lines

@@ -45,9 +45,12 @@ 
 	 */
 	public function affectedRows()
 	{
-		try {
+		try
+		{
 			return $this->statement->rowCount();
-		} catch ( \Doctrine\DBAL\DBALException $e ) {
+		}
+		catch ( \Doctrine\DBAL\DBALException $e )
+		{
 			throw new \Aimeos\MW\DB\Exception( $e->getMessage(), $e->getCode(), join( ', ', $this->statement->errorInfo() ) );
 		}
 	}
@@ -62,9 +65,12 @@ 
 	 */
 	public function fetch( $style = \Aimeos\MW\DB\Result\Base::FETCH_ASSOC )
 	{
-		try {
+		try
+		{
 			return $this->statement->fetch( $this->style[$style] );
-		} catch ( \Doctrine\DBAL\DBALException $e ) {
+		}
+		catch ( \Doctrine\DBAL\DBALException $e )
+		{
 			throw new \Aimeos\MW\DB\Exception( $e->getMessage(), $e->getCode(), join( ', ', $this->statement->errorInfo() ) );
 		}
 	}
@@ -77,9 +83,12 @@ 
 	 */
 	public function finish()
 	{
-		try {
+		try
+		{
 			$this->statement->closeCursor();
-		} catch ( \Doctrine\DBAL\DBALException $e ) {
+		}
+		catch ( \Doctrine\DBAL\DBALException $e )
+		{
 			throw new \Aimeos\MW\DB\Exception( $e->getMessage(), $e->getCode(), join( ', ', $this->statement->errorInfo() ) );
 		}
 	}

lib/mwlib/src/MW/DB/Connection/Base.php

Spacing +1 added lines, -1 removed lines

@@ -37,7 +37,7 @@
 	 * @param string $data Value to escape
 	 * @return string Escaped string
 	 */
-	public function escape($data)
+	public function escape( $data )
 	{
 		$quoted = $this->getRawObject()->quote( $data );
 

lib/mwlib/src/MW/Criteria/Expression/Combine/PHP.php

Spacing +3 added lines, -3 removed lines

@@ -92,7 +92,7 @@ 
 		}
 
 		if( $this->operator == '!' ) {
-			return ' ' . self::$operators[$this->operator] . ' ' . $item->toString( $types, $translations, $plugins );
+			return ' '.self::$operators[$this->operator].' '.$item->toString( $types, $translations, $plugins );
 		}
 
 		$string = $item->toString( $types, $translations, $plugins );
@@ -100,11 +100,11 @@ 
 		while( ( $item = next( $this->expressions ) ) !== false )
 		{
 			if( ( $itemstr = $item->toString( $types, $translations, $plugins ) ) !== '' ) {
-				$string .= ' ' . self::$operators[$this->operator] . ' ' . $itemstr;
+				$string .= ' '.self::$operators[$this->operator].' '.$itemstr;
 			}
 		}
 
-		return '( ' . $string . ' )';
+		return '( '.$string.' )';
 	}