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/src/MShop/Stock/Item/Standard.php

Spacing +2 added lines, -2 removed lines

@@ -91,7 +91,7 @@ 
 	 */
 	public function setTypeId( $id )
 	{
-		if ( $id == $this->getTypeId() ) { return $this; }
+		if( $id == $this->getTypeId() ) { return $this; }
 
 		$this->values['stock.typeid'] = (int) $id;
 		$this->setModified();
@@ -189,7 +189,7 @@ 
 	{
 		if( $backdate == $this->getDateBack() ) { return $this; }
 
-		$this->values['stock.backdate'] = $this->checkDateFormat( $backdate );;
+		$this->values['stock.backdate'] = $this->checkDateFormat( $backdate ); ;
 		$this->setModified();
 
 		return $this;

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

controller/common/src/Controller/Common/Media/Standard.php

Spacing +6 added lines, -6 removed lines

@@ -103,7 +103,7 @@ 
 				$fs->rm( $path );
 			}
 		}
-		catch( \Exception $e ) { ; } // Can be a mime icon with relative path
+		catch( \Exception $e ) {; } // Can be a mime icon with relative path
 
 		$item->setPreview( '' );
 	}
@@ -233,7 +233,7 @@ 
 			default: $ext = '';
 		}
 
-		$filename = md5( $filename . getmypid() . microtime( true ) );
+		$filename = md5( $filename.getmypid().microtime( true ) );
 
 		return "${type}/${filename[0]}/${filename[1]}/${filename}${ext}";
 	}
@@ -320,7 +320,7 @@ 
 		 */
 		$ext = $config->get( 'controller/common/media/standard/mimeicon/extension', '.png' );
 
-		return $mimedir . DIRECTORY_SEPARATOR . $mimetype . $ext;
+		return $mimedir.DIRECTORY_SEPARATOR.$mimetype.$ext;
 	}
 
 
@@ -364,7 +364,7 @@ 
 		 * @category User
 		 */
 		$default = array( 'image/jpeg', 'image/png', 'image/gif' );
-		$allowed = $config->get( 'controller/common/media/standard/' . $type . '/allowedtypes', $default );
+		$allowed = $config->get( 'controller/common/media/standard/'.$type.'/allowedtypes', $default );
 
 		if( in_array( $mimetype, $allowed ) === false )
 		{
@@ -425,7 +425,7 @@ 
 		 * @category Developer
 		 * @category User
 		 */
-		$maxwidth = $config->get( 'controller/common/media/standard/' . $type . '/maxwidth', null );
+		$maxwidth = $config->get( 'controller/common/media/standard/'.$type.'/maxwidth', null );
 
 		/** controller/common/media/standard/files/maxheight
 		 * Maximum height of the uploaded images
@@ -462,7 +462,7 @@ 
 		 * @category Developer
 		 * @category User
 		 */
-		$maxheight = $config->get( 'controller/common/media/standard/' . $type . '/maxheight', null );
+		$maxheight = $config->get( 'controller/common/media/standard/'.$type.'/maxheight', null );
 
 		return $media->scale( $maxwidth, $maxheight );
 	}

Braces +4 added lines, -3 removed lines

@@ -61,8 +61,7 @@ 
 			$filepath = $this->getFilePath( $file->getClientFilename(), 'preview', $mimeprev );
 			$this->storeFile( $media->save( null, $mimetype ), $fsname, $filepath, $item->getPreview() );
 			$item->setPreview( $filepath );
-		}
-		else
+		} else
 		{
 			$mimetype = $media->getMimeType();
 			$item->setPreview( $this->getMimeIcon( $mimetype ) );
@@ -103,7 +102,9 @@ 
 				$fs->rm( $path );
 			}
 		}
-		catch( \Exception $e ) { ; } // Can be a mime icon with relative path
+		catch( \Exception $e )
+		{
+; } // Can be a mime icon with relative path
 
 		$item->setPreview( '' );
 	}