aimeos / aimeos-core

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 );

lib/mwlib/src/MW/Media/Factory.php

Spacing +3 added lines, -3 removed lines

@@ -49,17 +49,17 @@
 		$mime = explode( '/', $mimetype );
 
 		$type = ( $mime[0] === 'image' ? 'Image' : 'Application' );
-		$name = ( isset( $options[ $mime[0] ]['name'] ) ? ucfirst( $options[ $mime[0] ]['name'] ) : 'Standard' );
+		$name = ( isset( $options[$mime[0]]['name'] ) ? ucfirst( $options[$mime[0]]['name'] ) : 'Standard' );
 
 
 		if( ctype_alnum( $name ) === false )
 		{
-			$classname = is_string( $name ) ? '\\Aimeos\\MW\\Media\\' . $name : '<not a string>';
+			$classname = is_string( $name ) ? '\\Aimeos\\MW\\Media\\'.$name : '<not a string>';
 			throw new \Aimeos\MW\Container\Exception( sprintf( 'Invalid characters in class name "%1$s"', $classname ) );
 		}
 
 		$iface = '\\Aimeos\\MW\\Media\\Iface';
-		$classname = '\\Aimeos\\MW\\Media\\' . $type . '\\' . $name;
+		$classname = '\\Aimeos\\MW\\Media\\'.$type.'\\'.$name;
 
 		if( class_exists( $classname ) === false ) {
 			throw new \Aimeos\MW\Media\Exception( sprintf( 'Class "%1$s" not available', $classname ) );

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

Spacing +6 added lines, -6 removed lines

@@ -41,7 +41,7 @@ 
 		parent::__construct( $mimetype );
 
 		if( ( $this->image = @imagecreatefromstring( $content ) ) === false ) {
-			throw new \Aimeos\MW\Media\Exception( sprintf( 'The image type isn\'t supported by GDlib.') );
+			throw new \Aimeos\MW\Media\Exception( sprintf( 'The image type isn\'t supported by GDlib.' ) );
 		}
 
 		if( ( $this->info = getimagesizefromstring( $content ) ) === false ) {
@@ -49,7 +49,7 @@ 
 		}
 
 		if( imagealphablending( $this->image, false ) === false ) {
-			throw new \Aimeos\MW\Media\Exception( sprintf( 'GD library failed (imagealphablending)') );
+			throw new \Aimeos\MW\Media\Exception( sprintf( 'GD library failed (imagealphablending)' ) );
 		}
 
 		$this->options = $options;
@@ -115,7 +115,7 @@ 
 				case 'image/png':
 
 					if( imagesavealpha( $this->image, true ) === false ) {
-						throw new \Aimeos\MW\Media\Exception( sprintf( 'GD library failed (imagesavealpha)') );
+						throw new \Aimeos\MW\Media\Exception( sprintf( 'GD library failed (imagesavealpha)' ) );
 					}
 
 					if( @imagepng( $this->image, $filename, (int) 10 - $quality / 10 ) === false ) {
@@ -198,15 +198,15 @@ 
 		try
 		{
 			if( imagealphablending( $newImage, false ) === false ) {
-				throw new \Aimeos\MW\Media\Exception( sprintf( 'GD library failed (imagealphablending)') );
+				throw new \Aimeos\MW\Media\Exception( sprintf( 'GD library failed (imagealphablending)' ) );
 			}
 
 			if( ( $transparent = imagecolorallocatealpha( $newImage, 255, 255, 255, 127 ) ) === false ) {
-				throw new \Aimeos\MW\Media\Exception( sprintf( 'GD library failed (imagecolorallocatealpha)') );
+				throw new \Aimeos\MW\Media\Exception( sprintf( 'GD library failed (imagecolorallocatealpha)' ) );
 			}
 
 			if( imagefilledrectangle( $newImage, 0, 0, $destWidth, $destHeight, $transparent ) === false ) {
-				throw new \Aimeos\MW\Media\Exception( sprintf( 'GD library failed (imagefilledrectangle)') );
+				throw new \Aimeos\MW\Media\Exception( sprintf( 'GD library failed (imagefilledrectangle)' ) );
 			}
 
 			if( imagecopyresampled( $newImage, $image, 0, 0, 0, 0, $destWidth, $destHeight, $srcWidth, $srcHeight ) === false ) {

Braces +1 added lines, -2 removed lines

@@ -166,8 +166,7 @@
 			}
 
 			$this->image = $result;
-		}
-		else
+		} else
 		{
 			$this->image = $this->resample( $this->image, $this->info[0], $this->info[1], $width, $height );
 		}