aimeos / aimeos-core

lib/mshoplib/setup/default/schema/stock.php

Spacing +2 added lines, -2 removed lines

@@ -8,7 +8,7 @@ 
 
 return array(
 	'table' => array(
-		'mshop_stock_type' => function ( \Doctrine\DBAL\Schema\Schema $schema ) {
+		'mshop_stock_type' => function( \Doctrine\DBAL\Schema\Schema $schema ) {
 
 			$table = $schema->createTable( 'mshop_stock_type' );
 
@@ -31,7 +31,7 @@ 
 			return $schema;
 		},
 
-		'mshop_stock' => function ( \Doctrine\DBAL\Schema\Schema $schema ) {
+		'mshop_stock' => function( \Doctrine\DBAL\Schema\Schema $schema ) {
 
 			$table = $schema->createTable( 'mshop_stock' );
 

Braces +4 added lines, -2 removed lines

@@ -8,7 +8,8 @@ 
 
 return array(
 	'table' => array(
-		'mshop_stock_type' => function ( \Doctrine\DBAL\Schema\Schema $schema ) {
+		'mshop_stock_type' => function ( \Doctrine\DBAL\Schema\Schema $schema )
+		{
 
 			$table = $schema->createTable( 'mshop_stock_type' );
 
@@ -31,7 +32,8 @@ 
 			return $schema;
 		},
 
-		'mshop_stock' => function ( \Doctrine\DBAL\Schema\Schema $schema ) {
+		'mshop_stock' => function ( \Doctrine\DBAL\Schema\Schema $schema )
+		{
 
 			$table = $schema->createTable( 'mshop_stock' );
 

lib/mshoplib/setup/default/schema/product.php

Braces +12 added lines, -6 removed lines

@@ -8,7 +8,8 @@ 
 
 return array(
 	'table' => array(
-		'mshop_product_type' => function ( \Doctrine\DBAL\Schema\Schema $schema ) {
+		'mshop_product_type' => function ( \Doctrine\DBAL\Schema\Schema $schema )
+		{
 
 			$table = $schema->createTable( 'mshop_product_type' );
 
@@ -31,7 +32,8 @@ 
 			return $schema;
 		},
 
-		'mshop_product' => function ( \Doctrine\DBAL\Schema\Schema $schema ) {
+		'mshop_product' => function ( \Doctrine\DBAL\Schema\Schema $schema )
+		{
 
 			$table = $schema->createTable( 'mshop_product' );
 
@@ -63,7 +65,8 @@ 
 			return $schema;
 		},
 
-		'mshop_product_list_type' => function ( \Doctrine\DBAL\Schema\Schema $schema ) {
+		'mshop_product_list_type' => function ( \Doctrine\DBAL\Schema\Schema $schema )
+		{
 
 			$table = $schema->createTable( 'mshop_product_list_type' );
 
@@ -86,7 +89,8 @@ 
 			return $schema;
 		},
 
-		'mshop_product_list' => function ( \Doctrine\DBAL\Schema\Schema $schema ) {
+		'mshop_product_list' => function ( \Doctrine\DBAL\Schema\Schema $schema )
+		{
 
 			$table = $schema->createTable( 'mshop_product_list' );
 
@@ -124,7 +128,8 @@ 
 			return $schema;
 		},
 
-		'mshop_product_property_type' => function ( \Doctrine\DBAL\Schema\Schema $schema ) {
+		'mshop_product_property_type' => function ( \Doctrine\DBAL\Schema\Schema $schema )
+		{
 
 			$table = $schema->createTable( 'mshop_product_property_type' );
 
@@ -147,7 +152,8 @@ 
 			return $schema;
 		},
 
-		'mshop_product_property' => function ( \Doctrine\DBAL\Schema\Schema $schema ) {
+		'mshop_product_property' => function ( \Doctrine\DBAL\Schema\Schema $schema )
+		{
 
 			$table = $schema->createTable( 'mshop_product_property' );
 

Spacing +8 added lines, -8 removed lines

@@ -8,7 +8,7 @@ 
 
 return array(
 	'table' => array(
-		'mshop_product_type' => function ( \Doctrine\DBAL\Schema\Schema $schema ) {
+		'mshop_product_type' => function( \Doctrine\DBAL\Schema\Schema $schema ) {
 
 			$table = $schema->createTable( 'mshop_product_type' );
 
@@ -31,7 +31,7 @@ 
 			return $schema;
 		},
 
-		'mshop_product' => function ( \Doctrine\DBAL\Schema\Schema $schema ) {
+		'mshop_product' => function( \Doctrine\DBAL\Schema\Schema $schema ) {
 
 			$table = $schema->createTable( 'mshop_product' );
 
@@ -46,8 +46,8 @@ 
 			$table->addColumn( 'status', 'smallint', [] );
 			$table->addColumn( 'mtime', 'datetime', [] );
 			$table->addColumn( 'ctime', 'datetime', [] );
-			$table->addColumn( 'editor', 'string', array('length' => 255 ) );
-			$table->addColumn( 'target', 'string', array('length' => 255 ) );
+			$table->addColumn( 'editor', 'string', array( 'length' => 255 ) );
+			$table->addColumn( 'target', 'string', array( 'length' => 255 ) );
 
 			$table->setPrimaryKey( array( 'id' ), 'pk_mspro_id' );
 			$table->addUniqueIndex( array( 'siteid', 'code' ), 'unq_mspro_siteid_code' );
@@ -64,7 +64,7 @@ 
 			return $schema;
 		},
 
-		'mshop_product_list_type' => function ( \Doctrine\DBAL\Schema\Schema $schema ) {
+		'mshop_product_list_type' => function( \Doctrine\DBAL\Schema\Schema $schema ) {
 
 			$table = $schema->createTable( 'mshop_product_list_type' );
 
@@ -87,7 +87,7 @@ 
 			return $schema;
 		},
 
-		'mshop_product_list' => function ( \Doctrine\DBAL\Schema\Schema $schema ) {
+		'mshop_product_list' => function( \Doctrine\DBAL\Schema\Schema $schema ) {
 
 			$table = $schema->createTable( 'mshop_product_list' );
 
@@ -125,7 +125,7 @@ 
 			return $schema;
 		},
 
-		'mshop_product_property_type' => function ( \Doctrine\DBAL\Schema\Schema $schema ) {
+		'mshop_product_property_type' => function( \Doctrine\DBAL\Schema\Schema $schema ) {
 
 			$table = $schema->createTable( 'mshop_product_property_type' );
 
@@ -148,7 +148,7 @@ 
 			return $schema;
 		},
 
-		'mshop_product_property' => function ( \Doctrine\DBAL\Schema\Schema $schema ) {
+		'mshop_product_property' => function( \Doctrine\DBAL\Schema\Schema $schema ) {
 
 			$table = $schema->createTable( 'mshop_product_property' );
 

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

Spacing +4 added lines, -4 removed lines

@@ -552,8 +552,8 @@ 
 
 			while( ( $row = $results->fetch() ) !== false )
 			{
-				$map[ $row['stock.id'] ] = $row;
-				$typeIds[ $row['stock.typeid'] ] = null;
+				$map[$row['stock.id']] = $row;
+				$typeIds[$row['stock.typeid']] = null;
 			}
 
 			$dbm->release( $conn, $dbname );
@@ -576,9 +576,9 @@ 
 
 			foreach( $map as $id => $row )
 			{
-				if( isset( $typeItems[ $row['stock.typeid'] ] ) )
+				if( isset( $typeItems[$row['stock.typeid']] ) )
 				{
-					$row['stock.type'] = $typeItems[ $row['stock.typeid'] ]->getCode();
+					$row['stock.type'] = $typeItems[$row['stock.typeid']]->getCode();
 					$row['stock.typename'] = $typeItems[$row['stock.typeid']]->getName();
 				}
 

Braces +1 added lines, -2 removed lines

@@ -200,8 +200,7 @@
 				 * @see mshop/stock/manager/standard/stocklevel
 				 */
 				$path = 'mshop/stock/manager/standard/insert';
-			}
-			else
+			} else
 			{
 				/** mshop/stock/manager/standard/update/mysql
 				 * Updates an existing product stock record in the database

Doc Comments +1 added lines

@@ -435,6 +435,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 stock items implementing \Aimeos\MShop\Stock\Item\Iface
 	 */
 	public function searchItems( \Aimeos\MW\Criteria\Iface $search, array $ref = [], &$total = null )

lib/mshoplib/src/MShop/Stock/Manager/Type/Standard.php

Spacing +1 added lines, -1 removed lines

@@ -283,7 +283,7 @@
 		 * @see mshop/stock/manager/type/decorators/global
 		 */
 
-		return $this->getSubManagerBase( 'stock', 'type/' . $manager, $name );
+		return $this->getSubManagerBase( 'stock', 'type/'.$manager, $name );
 	}
 
 

lib/mshoplib/src/MShop/Stock/Manager/Factory.php

Spacing +2 added lines, -2 removed lines

@@ -71,12 +71,12 @@
 
 		if( ctype_alnum( $name ) === false )
 		{
-			$classname = is_string( $name ) ? '\\Aimeos\\MShop\\Stock\\Manager\\' . $name : '<not a string>';
+			$classname = is_string( $name ) ? '\\Aimeos\\MShop\\Stock\\Manager\\'.$name : '<not a string>';
 			throw new \Aimeos\MShop\Stock\Exception( sprintf( 'Invalid characters in class name "%1$s"', $classname ) );
 		}
 
 		$iface = '\\Aimeos\\MShop\\Stock\\Manager\\Iface';
-		$classname = '\\Aimeos\\MShop\\Stock\\Manager\\' . $name;
+		$classname = '\\Aimeos\\MShop\\Stock\\Manager\\'.$name;
 
 		$manager = self::createManagerBase( $context, $classname, $iface );
 

lib/mshoplib/setup/StockMigrateProductcode.php

Braces +4 added lines, -8 removed lines

@@ -72,8 +72,7 @@ 
 				{
 					$this->execute( $stmt );
 					$this->status( 'done' );
-				}
-				else
+				} else
 				{
 					$this->status( 'OK' );
 				}
@@ -86,8 +85,7 @@ 
 			{
 				$this->executeList( $this->updates );
 				$this->status( 'done' );
-			}
-			else
+			} else
 			{
 				$this->status( 'OK' );
 			}
@@ -101,8 +99,7 @@ 
 				{
 					$this->execute( $stmt );
 					$this->status( 'done' );
-				}
-				else
+				} else
 				{
 					$this->status( 'OK' );
 				}
@@ -114,8 +111,7 @@ 
 			{
 				$this->execute( $this->column );
 				$this->status( 'done' );
-			}
-			else
+			} else
 			{
 				$this->status( 'OK' );
 			}

lib/mshoplib/tests/MShop/Service/Provider/Decorator/CategoryTest.php

Doc Comments +3 added lines

@@ -197,6 +197,9 @@
 	}
 
 
+	/**
+	 * @param string $code
+	 */
 	protected function getOrderProduct( $code )
 	{
 		$productManager = \Aimeos\MShop\Factory::createManager( $this->context, 'product' );

lib/mshoplib/src/MShop/Factory.php

Spacing +6 added lines, -6 removed lines

@@ -91,7 +91,7 @@ 
 
 			if( self::$cache === false || !isset( self::$managers[$id][$domain] ) )
 			{
-				$factory = '\\Aimeos\\MShop\\' . ucwords( $domain ) . '\\Manager\\Factory';
+				$factory = '\\Aimeos\\MShop\\'.ucwords( $domain ).'\\Manager\\Factory';
 
 				if( class_exists( $factory ) === false ) {
 					throw new \Aimeos\MShop\Exception( sprintf( 'Class "%1$s" not available', $factory ) );
@@ -111,14 +111,14 @@ 
 
 			foreach( $parts as $part )
 			{
-				$subpath .= $part . '/';
-				$classname = $context->getConfig()->get( 'mshop/' . $domain . '/manager/' . $subpath . 'name' );
+				$subpath .= $part.'/';
+				$classname = $context->getConfig()->get( 'mshop/'.$domain.'/manager/'.$subpath.'name' );
 
-				if( self::$cache === false || !isset( self::$managers[$id][$tmppath . '/' . $part] ) ) {
-					self::$managers[$id][$tmppath . '/' . $part] = self::$managers[$id][$tmppath]->getSubManager( $part, $classname );
+				if( self::$cache === false || !isset( self::$managers[$id][$tmppath.'/'.$part] ) ) {
+					self::$managers[$id][$tmppath.'/'.$part] = self::$managers[$id][$tmppath]->getSubManager( $part, $classname );
 				}
 
-				$tmppath .= '/' . $part;
+				$tmppath .= '/'.$part;
 			}
 		}
 

lib/mwlib/tests/MW/Media/FactoryTest.php

Spacing +4 added lines, -4 removed lines

@@ -15,7 +15,7 @@ 
 	public function testGetImage()
 	{
 		$ds = DIRECTORY_SEPARATOR;
-		$object = \Aimeos\MW\Media\Factory::get( __DIR__ . $ds .'_testfiles' . $ds . 'image.png' );
+		$object = \Aimeos\MW\Media\Factory::get( __DIR__.$ds.'_testfiles'.$ds.'image.png' );
 
 		$this->assertInstanceOf( '\\Aimeos\\MW\\Media\\Iface', $object );
 		$this->assertInstanceOf( '\\Aimeos\\MW\\Media\\Image\\Iface', $object );
@@ -25,7 +25,7 @@ 
 	public function testGetImageAsResource()
 	{
 		$ds = DIRECTORY_SEPARATOR;
-		$resource = fopen( __DIR__ . $ds .'_testfiles' . $ds . 'image.png', 'rw' );
+		$resource = fopen( __DIR__.$ds.'_testfiles'.$ds.'image.png', 'rw' );
 		$object = \Aimeos\MW\Media\Factory::get( $resource );
 
 		$this->assertInstanceOf( '\\Aimeos\\MW\\Media\\Iface', $object );
@@ -36,7 +36,7 @@ 
 	public function testGetImageAsString()
 	{
 		$ds = DIRECTORY_SEPARATOR;
-		$content = file_get_contents( __DIR__ . $ds .'_testfiles' . $ds . 'image.png' );
+		$content = file_get_contents( __DIR__.$ds.'_testfiles'.$ds.'image.png' );
 		$object = \Aimeos\MW\Media\Factory::get( $content );
 
 		$this->assertInstanceOf( '\\Aimeos\\MW\\Media\\Iface', $object );
@@ -48,7 +48,7 @@ 
 	public function testGetBinary()
 	{
 		$ds = DIRECTORY_SEPARATOR;
-		$object = \Aimeos\MW\Media\Factory::get( __DIR__ . $ds . '_testfiles' . $ds . 'application.txt' );
+		$object = \Aimeos\MW\Media\Factory::get( __DIR__.$ds.'_testfiles'.$ds.'application.txt' );
 
 		$this->assertInstanceOf( '\\Aimeos\\MW\\Media\\Iface', $object );
 		$this->assertInstanceOf( '\\Aimeos\\MW\\Media\\Application\\Iface', $object );