aimeos / aimeos-core

controller/common/tests/Controller/Common/Media/StandardTest.php

Doc Comments +3 added lines

@@ -335,6 +335,9 @@
 	}
 
 
+	/**
+	 * @param string $name
+	 */
 	protected function access( $name )
 	{
 		$class = new \ReflectionClass( '\Aimeos\Controller\Common\Media\Standard' );

Spacing +15 added lines, -15 removed lines

@@ -40,7 +40,7 @@ 
 
 		$file = $this->getMockBuilder( '\Psr\Http\Message\UploadedFileInterface' )->getMock();
 		$file->expects( $this->once() )->method( 'getStream' )
-			->will( $this->returnValue( file_get_contents( __DIR__ . '/testfiles/test.gif' ) ) );
+			->will( $this->returnValue( file_get_contents( __DIR__.'/testfiles/test.gif' ) ) );
 
 		$item = \Aimeos\MShop\Factory::createManager( $this->context, 'media' )->createItem();
 
@@ -60,7 +60,7 @@ 
 
 		$file = $this->getMockBuilder( '\Psr\Http\Message\UploadedFileInterface' )->getMock();
 		$file->expects( $this->once() )->method( 'getStream' )
-			->will( $this->returnValue( file_get_contents( __DIR__ . '/testfiles/test.pdf' ) ) );
+			->will( $this->returnValue( file_get_contents( __DIR__.'/testfiles/test.pdf' ) ) );
 
 		$item = \Aimeos\MShop\Factory::createManager( $this->context, 'media' )->createItem();
 
@@ -108,7 +108,7 @@ 
 			->getMock();
 
 		$object->expects( $this->once() )->method( 'getFileContent' )
-			->will( $this->returnValue( file_get_contents( __DIR__ . '/testfiles/test.png' ) ) );
+			->will( $this->returnValue( file_get_contents( __DIR__.'/testfiles/test.png' ) ) );
 
 		$object->expects( $this->exactly( 2 ) )->method( 'storeFile' );
 
@@ -204,9 +204,9 @@ 
 
 	public function testGetFileContent()
 	{
-		$dest = dirname( dirname( dirname( __DIR__ ) ) ) . '/tmp/';
+		$dest = dirname( dirname( dirname( __DIR__ ) ) ).'/tmp/';
 		if( !is_dir( $dest ) ) { mkdir( $dest, 0755, true ); }
-		copy( __DIR__ . '/testfiles/test.gif', $dest . 'test.gif' );
+		copy( __DIR__.'/testfiles/test.gif', $dest.'test.gif' );
 
 		$result = $this->access( 'getFileContent' )->invokeArgs( $this->object, array( 'test.gif', 'fs-media' ) );
 
@@ -296,7 +296,7 @@ 
 
 	public function testGetMimeType()
 	{
-		$file = \Aimeos\MW\Media\Factory::get( __DIR__ . '/testfiles/test.png' );
+		$file = \Aimeos\MW\Media\Factory::get( __DIR__.'/testfiles/test.png' );
 
 		$result = $this->access( 'getMimeType' )->invokeArgs( $this->object, array( $file, 'files' ) );
 		$this->assertEquals( 'image/png', $result );
@@ -305,7 +305,7 @@ 
 
 	public function testGetMimeTypeNotAllowed()
 	{
-		$file = \Aimeos\MW\Media\Factory::get( __DIR__ . '/testfiles/test.gif' );
+		$file = \Aimeos\MW\Media\Factory::get( __DIR__.'/testfiles/test.gif' );
 		$this->context->getConfig()->set( 'controller/common/media/standard/files/allowedtypes', array( 'image/jpeg' ) );
 
 		$result = $this->access( 'getMimeType' )->invokeArgs( $this->object, array( $file, 'files' ) );
@@ -315,7 +315,7 @@ 
 
 	public function testGetMimeTypeNoTypes()
 	{
-		$file = \Aimeos\MW\Media\Factory::get( __DIR__ . '/testfiles/test.gif' );
+		$file = \Aimeos\MW\Media\Factory::get( __DIR__.'/testfiles/test.gif' );
 		$this->context->getConfig()->set( 'controller/common/media/standard/files/allowedtypes', [] );
 
 		$this->setExpectedException( '\Aimeos\Controller\Common\Exception' );
@@ -325,7 +325,7 @@ 
 
 	public function testScaleImage()
 	{
-		$file = \Aimeos\MW\Media\Factory::get( __DIR__ . '/testfiles/test.gif' );
+		$file = \Aimeos\MW\Media\Factory::get( __DIR__.'/testfiles/test.gif' );
 
 		$this->access( 'scaleImage' )->invokeArgs( $this->object, array( $file, 'files' ) );
 	}
@@ -333,18 +333,18 @@ 
 
 	public function testStoreFile()
 	{
-		$content = file_get_contents( __DIR__ . '/testfiles/test.gif' );
+		$content = file_get_contents( __DIR__.'/testfiles/test.gif' );
 
-		$dest = dirname( dirname( dirname( __DIR__ ) ) ) . '/tmp/';
+		$dest = dirname( dirname( dirname( __DIR__ ) ) ).'/tmp/';
 		if( !is_dir( $dest ) ) { mkdir( $dest, 0755, true ); }
-		copy( __DIR__ . '/testfiles/test.gif', $dest . 'test2.gif' );
+		copy( __DIR__.'/testfiles/test.gif', $dest.'test2.gif' );
 
 		$this->access( 'storeFile' )->invokeArgs( $this->object, array( $content, 'fs-media', 'test.gif', 'test2.gif' ) );
 
-		$this->assertFalse( file_exists( $dest . 'test2.gif' ) );
-		$this->assertTrue( file_exists( $dest . 'test.gif' ) );
+		$this->assertFalse( file_exists( $dest.'test2.gif' ) );
+		$this->assertTrue( file_exists( $dest.'test.gif' ) );
 
-		unlink( $dest . 'test.gif' );
+		unlink( $dest.'test.gif' );
 	}
 
 

lib/mshoplib/src/MShop/Locale/Manager/Currency/Standard.php

Doc Comments +2 added lines, -2 removed lines

@@ -605,7 +605,7 @@ 
 	 * Returns the search results for the given SQL statement.
 	 *
 	 * @param \Aimeos\MW\DB\Connection\Iface $conn Database connection
-	 * @param $sql SQL statement
+	 * @param string $sql SQL statement
 	 * @return \Aimeos\MW\DB\Result\Iface Search result object
 	 */
 	protected function getSearchResults( \Aimeos\MW\DB\Connection\Iface $conn, $sql )
@@ -635,7 +635,7 @@ 
 	 * Returns the total number of items found for the conditions
 	 *
 	 * @param \Aimeos\MW\DB\Connection\Iface $conn Database connection
-	 * @param array $find List of markers that should be replaced in the SQL statement
+	 * @param string[] $find List of markers that should be replaced in the SQL statement
 	 * @param array $replace List of replacements for the markers in the SQL statement
 	 * @throws \Aimeos\MShop\Locale\Exception If no total value was found
 	 * @return integer Total number of found items

Spacing +3 added lines, -3 removed lines

@@ -369,7 +369,7 @@ 
 			$replace = array(
 				$search->getConditionString( $types, $translations ),
 				$search->getSortationString( $types, $translations ),
-				( $columns ? ', ' . $columns : '' ),
+				( $columns ? ', '.$columns : '' ),
 				$search->getSliceStart(),
 				$search->getSliceSize(),
 			);
@@ -581,7 +581,7 @@ 
 		 * @see mshop/locale/manager/currency/decorators/global
 		 */
 
-		return $this->getSubManagerBase( 'locale', 'currency/' . $manager, $name );
+		return $this->getSubManagerBase( 'locale', 'currency/'.$manager, $name );
 	}
 
 
@@ -611,7 +611,7 @@ 
 	protected function getSearchResults( \Aimeos\MW\DB\Connection\Iface $conn, $sql )
 	{
 		$statement = $conn->create( $sql );
-		$this->getContext()->getLogger()->log( __METHOD__ . ': SQL statement: ' . $statement, \Aimeos\MW\Logger\Base::DEBUG );
+		$this->getContext()->getLogger()->log( __METHOD__.': SQL statement: '.$statement, \Aimeos\MW\Logger\Base::DEBUG );
 
 		$results = $statement->execute();
 

Braces +6 added lines, -4 removed lines

@@ -105,9 +105,12 @@ 
 	 */
 	public function createItem()
 	{
-		try {
+		try
+		{
 			$values = array( 'locale.currency.siteid' => $this->getContext()->getLocale()->getSiteId() );
-		} catch( \Exception $ex ) {
+		}
+		catch( \Exception $ex )
+		{
 			$values = array( 'locale.currency.siteid' => null );
 		}
 
@@ -177,8 +180,7 @@ 
 				 * @see mshop/locale/manager/currency/standard/count/ansi
 				 */
 				$path = 'mshop/locale/manager/currency/standard/insert';
-			}
-			else
+			} else
 			{
 				/** mshop/locale/manager/currency/standard/update/mysql
 				 * Updates an existing currency record in the database

lib/mshoplib/src/MShop/Plugin/Provider/Order/Shipping.php

Braces +6 added lines, -4 removed lines

@@ -55,9 +55,12 @@ 
 		$config = $this->getItemBase()->getConfig();
 		if( !isset( $config['threshold'] ) ) { return true; }
 
-		try {
+		try
+		{
 			$delivery = $order->getService( 'delivery' );
-		} catch( \Aimeos\MShop\Order\Exception $oe ) {
+		}
+		catch( \Aimeos\MShop\Order\Exception $oe )
+		{
 			// no delivery item available yet
 			return true;
 		}
@@ -94,8 +97,7 @@ 
 		{
 			$price->setRebate( $price->getCosts() );
 			$price->setCosts( '0.00' );
-		}
-		else if( $sum->getValue() + $sum->getRebate() < $threshold[$currency] && $price->getRebate() > '0.00' )
+		} else if( $sum->getValue() + $sum->getRebate() < $threshold[$currency] && $price->getRebate() > '0.00' )
 		{
 			$price->setCosts( $price->getRebate() );
 			$price->setRebate( '0.00' );

lib/mshoplib/src/MShop/Index/Manager/Price/Standard.php

Spacing +2 added lines, -2 removed lines

@@ -402,7 +402,7 @@ 
 		 * @see mshop/index/manager/price/decorators/global
 		 */
 
-		return $this->getSubManagerBase( 'index', 'price/' . $manager, $name );
+		return $this->getSubManagerBase( 'index', 'price/'.$manager, $name );
 	}
 
 
@@ -733,7 +733,7 @@ 
 
 				try {
 					$stmt->execute()->finish();
-				} catch( \Aimeos\MW\DB\Exception $e ) { ; } // Ignore duplicates
+				} catch( \Aimeos\MW\DB\Exception $e ) {; } // Ignore duplicates
 			}
 		}
 	}

Braces +6 added lines, -2 removed lines

@@ -726,9 +726,13 @@
 				$stmt->bind( 13, $siteid, \Aimeos\MW\DB\Statement\Base::PARAM_INT );
 				$stmt->bind( 14, $date ); //ctime
 
-				try {
+				try
+				{
 					$stmt->execute()->finish();
-				} catch( \Aimeos\MW\DB\Exception $e ) { ; } // Ignore duplicates
+				}
+				catch( \Aimeos\MW\DB\Exception $e )
+				{
+; } // Ignore duplicates
 			}
 		}
 	}

lib/mshoplib/src/MShop/Index/Manager/Attribute/Standard.php

Spacing +2 added lines, -2 removed lines

@@ -358,7 +358,7 @@ 
 		 * @see mshop/index/manager/attribute/decorators/global
 		 */
 
-		return $this->getSubManagerBase( 'index', 'attribute/' . $manager, $name );
+		return $this->getSubManagerBase( 'index', 'attribute/'.$manager, $name );
 	}
 
 
@@ -683,7 +683,7 @@ 
 
 				try {
 					$stmt->execute()->finish();
-				} catch( \Aimeos\MW\DB\Exception $e ) { ; } // Ignore duplicates
+				} catch( \Aimeos\MW\DB\Exception $e ) {; } // Ignore duplicates
 			}
 		}
 	}

Braces +6 added lines, -2 removed lines

@@ -676,9 +676,13 @@
 				$stmt->bind( 8, $siteid, \Aimeos\MW\DB\Statement\Base::PARAM_INT );
 				$stmt->bind( 9, $date ); // ctime
 
-				try {
+				try
+				{
 					$stmt->execute()->finish();
-				} catch( \Aimeos\MW\DB\Exception $e ) { ; } // Ignore duplicates
+				}
+				catch( \Aimeos\MW\DB\Exception $e )
+				{
+; } // Ignore duplicates
 			}
 		}
 	}

lib/mshoplib/src/MShop/Locale/Manager/Language/Standard.php

Spacing +3 added lines, -3 removed lines

@@ -454,7 +454,7 @@ 
 		 * @see mshop/locale/manager/language/decorators/global
 		 */
 
-		return $this->getSubManagerBase( 'locale', 'language/' . $manager, $name );
+		return $this->getSubManagerBase( 'locale', 'language/'.$manager, $name );
 	}
 
 
@@ -486,7 +486,7 @@ 
 			$replace = array(
 				$search->getConditionString( $types, $translations ),
 				$search->getSortationString( $types, $translations ),
-				( $columns ? ', ' . $columns : '' ),
+				( $columns ? ', '.$columns : '' ),
 				$search->getSliceStart(),
 				$search->getSliceSize(),
 			);
@@ -606,7 +606,7 @@ 
 	protected function getSearchResults( \Aimeos\MW\DB\Connection\Iface $conn, $sql )
 	{
 		$statement = $conn->create( $sql );
-		$this->getContext()->getLogger()->log( __METHOD__ . ': SQL statement: ' . $statement, \Aimeos\MW\Logger\Base::DEBUG );
+		$this->getContext()->getLogger()->log( __METHOD__.': SQL statement: '.$statement, \Aimeos\MW\Logger\Base::DEBUG );
 
 		$results = $statement->execute();
 

Doc Comments +2 added lines, -2 removed lines

@@ -605,7 +605,7 @@ 
 	 * Returns the search results for the given SQL statement.
 	 *
 	 * @param \Aimeos\MW\DB\Connection\Iface $conn Database connection
-	 * @param $sql SQL statement
+	 * @param string $sql SQL statement
 	 * @return \Aimeos\MW\DB\Result\Iface Search result object
 	 */
 	protected function getSearchResults( \Aimeos\MW\DB\Connection\Iface $conn, $sql )
@@ -635,7 +635,7 @@ 
 	 * Returns the total number of items found for the conditions
 	 *
 	 * @param \Aimeos\MW\DB\Connection\Iface $conn Database connection
-	 * @param array $find List of markers that should be replaced in the SQL statement
+	 * @param string[] $find List of markers that should be replaced in the SQL statement
 	 * @param array $replace List of replacements for the markers in the SQL statement
 	 * @throws \Aimeos\MShop\Locale\Exception If no total value was found
 	 * @return integer Total number of found items

Braces +6 added lines, -4 removed lines

@@ -104,9 +104,12 @@ 
 	 */
 	public function createItem()
 	{
-		try {
+		try
+		{
 			$values = array( 'locale.language.siteid' => $this->getContext()->getLocale()->getSiteId() );
-		} catch( \Exception $ex ) {
+		}
+		catch( \Exception $ex )
+		{
 			$values = array( 'locale.language.siteid' => null );
 		}
 
@@ -173,8 +176,7 @@ 
 				 * @see mshop/locale/manager/language/standard/count/ansi
 				 */
 				$path = 'mshop/locale/manager/language/standard/insert';
-			}
-			else
+			} else
 			{
 				/** mshop/locale/manager/language/standard/update/mysql
 				 * Updates an existing language record in the database

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

Spacing +2 added lines, -2 removed lines

@@ -366,7 +366,7 @@ 
 		 * @see mshop/index/manager/catalog/decorators/global
 		 */
 
-		return $this->getSubManagerBase( 'index', 'catalog/' . $manager, $name );
+		return $this->getSubManagerBase( 'index', 'catalog/'.$manager, $name );
 	}
 
 
@@ -485,7 +485,7 @@ 
 
 					try {
 						$stmt->execute()->finish();
-					} catch( \Aimeos\MW\DB\Exception $e ) { ; } // Ignore duplicates
+					} catch( \Aimeos\MW\DB\Exception $e ) {; } // Ignore duplicates
 				}
 			}
 

Braces +6 added lines, -2 removed lines

@@ -483,9 +483,13 @@
 					$stmt->bind( 7, $siteid, \Aimeos\MW\DB\Statement\Base::PARAM_INT );
 					$stmt->bind( 8, $date ); //ctime
 
-					try {
+					try
+					{
 						$stmt->execute()->finish();
-					} catch( \Aimeos\MW\DB\Exception $e ) { ; } // Ignore duplicates
+					}
+					catch( \Aimeos\MW\DB\Exception $e )
+					{
+; } // Ignore duplicates
 				}
 			}
 

lib/mshoplib/setup/unittest/CustomerAddTestData.php

Spacing +2 added lines, -2 removed lines

@@ -52,7 +52,7 @@ 
 		$ds = DIRECTORY_SEPARATOR;
 		$this->additional->setEditor( 'core:unittest' );
 
-		$this->process( __DIR__ . $ds . 'data' . $ds . 'customer.php' );
+		$this->process( __DIR__.$ds.'data'.$ds.'customer.php' );
 
 		$this->status( 'done' );
 
@@ -207,7 +207,7 @@ 
 
 			try {
 				$customerGroupManager->saveItem( $group, false );
-			} catch( \Exception $e ) { ; } // ignore duplicates
+			} catch( \Exception $e ) {; } // ignore duplicates
 		}
 	}
 }
\ No newline at end of file

Braces +6 added lines, -2 removed lines

@@ -205,9 +205,13 @@
 			$group->setCode( $dataset['code'] );
 			$group->setLabel( $dataset['label'] );
 
-			try {
+			try
+			{
 				$customerGroupManager->saveItem( $group, false );
-			} catch( \Exception $e ) { ; } // ignore duplicates
+			}
+			catch( \Exception $e )
+			{
+; } // ignore duplicates
 		}
 	}
 }
\ No newline at end of file

lib/mshoplib/setup/TablesCreatePlatform.php

Spacing +6 added lines, -6 removed lines

@@ -46,13 +46,13 @@
 
 		$ds = DIRECTORY_SEPARATOR;
 
-		$this->setupPlatform( 'db-index', 'mysql', realpath( __DIR__ ) . $ds . 'default' . $ds . 'schema' . $ds . 'index-mysql.sql' );
-		$this->setupPlatform( 'db-order', 'mysql', realpath( __DIR__ ) . $ds . 'default' . $ds . 'schema' . $ds . 'order-mysql.sql' );
-		$this->setupPlatform( 'db-text', 'mysql', realpath( __DIR__ ) . $ds . 'default' . $ds . 'schema' . $ds . 'text-mysql.sql' );
+		$this->setupPlatform( 'db-index', 'mysql', realpath( __DIR__ ).$ds.'default'.$ds.'schema'.$ds.'index-mysql.sql' );
+		$this->setupPlatform( 'db-order', 'mysql', realpath( __DIR__ ).$ds.'default'.$ds.'schema'.$ds.'order-mysql.sql' );
+		$this->setupPlatform( 'db-text', 'mysql', realpath( __DIR__ ).$ds.'default'.$ds.'schema'.$ds.'text-mysql.sql' );
 
-		$this->setupPlatform( 'db-index', 'pgsql', realpath( __DIR__ ) . $ds . 'default' . $ds . 'schema' . $ds . 'index-pgsql.sql' );
-		$this->setupPlatform( 'db-order', 'pgsql', realpath( __DIR__ ) . $ds . 'default' . $ds . 'schema' . $ds . 'order-pgsql.sql' );
-		$this->setupPlatform( 'db-text', 'pgsql', realpath( __DIR__ ) . $ds . 'default' . $ds . 'schema' . $ds . 'text-pgsql.sql' );
+		$this->setupPlatform( 'db-index', 'pgsql', realpath( __DIR__ ).$ds.'default'.$ds.'schema'.$ds.'index-pgsql.sql' );
+		$this->setupPlatform( 'db-order', 'pgsql', realpath( __DIR__ ).$ds.'default'.$ds.'schema'.$ds.'order-pgsql.sql' );
+		$this->setupPlatform( 'db-text', 'pgsql', realpath( __DIR__ ).$ds.'default'.$ds.'schema'.$ds.'text-pgsql.sql' );
 	}