aimeos / aimeos-core

lib/mshoplib/setup/unittest/ProductAddStockTestData.php

Spacing +1 added lines, -1 removed lines

@@ -52,7 +52,7 @@
 		$this->additional->setEditor( 'core:unittest' );
 
 		$ds = DIRECTORY_SEPARATOR;
-		$path = __DIR__ . $ds . 'data' . $ds . 'stock.php';
+		$path = __DIR__.$ds.'data'.$ds.'stock.php';
 
 		if( ( $testdata = include( $path ) ) == false ) {
 			throw new \Aimeos\MShop\Exception( sprintf( 'No file "%1$s" found for stock domain', $path ) );

lib/mshoplib/setup/TablesCreateMShop.php

Spacing +36 added lines, -36 removed lines

@@ -48,27 +48,27 @@ 
 		$ds = DIRECTORY_SEPARATOR;
 
 		$files = array(
-			'db-locale' => 'default' . $ds . 'schema' . $ds . 'locale.php',
-			'db-attribute' => 'default' . $ds . 'schema' . $ds . 'attribute.php',
-			'db-customer' => 'default' . $ds . 'schema' . $ds . 'customer.php',
-			'db-media' => 'default' . $ds . 'schema' . $ds . 'media.php',
-			'db-order' => 'default' . $ds . 'schema' . $ds . 'order.php',
-			'db-plugin' => 'default' . $ds . 'schema' . $ds . 'plugin.php',
-			'db-price' => 'default' . $ds . 'schema' . $ds . 'price.php',
-			'db-product' => 'default' . $ds . 'schema' . $ds . 'product.php',
-			'db-stock' => 'default' . $ds . 'schema' . $ds . 'stock.php',
-			'db-service' => 'default' . $ds . 'schema' . $ds . 'service.php',
-			'db-supplier' => 'default' . $ds . 'schema' . $ds . 'supplier.php',
-			'db-text' => 'default' . $ds . 'schema' . $ds . 'text.php',
-			'db-coupon' => 'default' . $ds . 'schema' . $ds . 'coupon.php',
-			'db-catalog' => 'default' . $ds . 'schema' . $ds . 'catalog.php',
-			'db-tag' => 'default' . $ds . 'schema' . $ds . 'tag.php',
+			'db-locale' => 'default'.$ds.'schema'.$ds.'locale.php',
+			'db-attribute' => 'default'.$ds.'schema'.$ds.'attribute.php',
+			'db-customer' => 'default'.$ds.'schema'.$ds.'customer.php',
+			'db-media' => 'default'.$ds.'schema'.$ds.'media.php',
+			'db-order' => 'default'.$ds.'schema'.$ds.'order.php',
+			'db-plugin' => 'default'.$ds.'schema'.$ds.'plugin.php',
+			'db-price' => 'default'.$ds.'schema'.$ds.'price.php',
+			'db-product' => 'default'.$ds.'schema'.$ds.'product.php',
+			'db-stock' => 'default'.$ds.'schema'.$ds.'stock.php',
+			'db-service' => 'default'.$ds.'schema'.$ds.'service.php',
+			'db-supplier' => 'default'.$ds.'schema'.$ds.'supplier.php',
+			'db-text' => 'default'.$ds.'schema'.$ds.'text.php',
+			'db-coupon' => 'default'.$ds.'schema'.$ds.'coupon.php',
+			'db-catalog' => 'default'.$ds.'schema'.$ds.'catalog.php',
+			'db-tag' => 'default'.$ds.'schema'.$ds.'tag.php',
 		);
 
 		$this->setupSchema( $files, true );
 
 		$files = array(
-			'db-product' => 'default' . $ds . 'schema' . $ds . 'index.php',
+			'db-product' => 'default'.$ds.'schema'.$ds.'index.php',
 		);
 
 		$this->setupSchema( $files, true );
@@ -86,27 +86,27 @@ 
 		$ds = DIRECTORY_SEPARATOR;
 
 		$files = array(
-			'db-locale' => 'default' . $ds . 'schema' . $ds . 'locale.php',
-			'db-attribute' => 'default' . $ds . 'schema' . $ds . 'attribute.php',
-			'db-customer' => 'default' . $ds . 'schema' . $ds . 'customer.php',
-			'db-media' => 'default' . $ds . 'schema' . $ds . 'media.php',
-			'db-order' => 'default' . $ds . 'schema' . $ds . 'order.php',
-			'db-plugin' => 'default' . $ds . 'schema' . $ds . 'plugin.php',
-			'db-price' => 'default' . $ds . 'schema' . $ds . 'price.php',
-			'db-product' => 'default' . $ds . 'schema' . $ds . 'product.php',
-			'db-stock' => 'default' . $ds . 'schema' . $ds . 'stock.php',
-			'db-service' => 'default' . $ds . 'schema' . $ds . 'service.php',
-			'db-supplier' => 'default' . $ds . 'schema' . $ds . 'supplier.php',
-			'db-text' => 'default' . $ds . 'schema' . $ds . 'text.php',
-			'db-coupon' => 'default' . $ds . 'schema' . $ds . 'coupon.php',
-			'db-catalog' => 'default' . $ds . 'schema' . $ds . 'catalog.php',
-			'db-tag' => 'default' . $ds . 'schema' . $ds . 'tag.php',
+			'db-locale' => 'default'.$ds.'schema'.$ds.'locale.php',
+			'db-attribute' => 'default'.$ds.'schema'.$ds.'attribute.php',
+			'db-customer' => 'default'.$ds.'schema'.$ds.'customer.php',
+			'db-media' => 'default'.$ds.'schema'.$ds.'media.php',
+			'db-order' => 'default'.$ds.'schema'.$ds.'order.php',
+			'db-plugin' => 'default'.$ds.'schema'.$ds.'plugin.php',
+			'db-price' => 'default'.$ds.'schema'.$ds.'price.php',
+			'db-product' => 'default'.$ds.'schema'.$ds.'product.php',
+			'db-stock' => 'default'.$ds.'schema'.$ds.'stock.php',
+			'db-service' => 'default'.$ds.'schema'.$ds.'service.php',
+			'db-supplier' => 'default'.$ds.'schema'.$ds.'supplier.php',
+			'db-text' => 'default'.$ds.'schema'.$ds.'text.php',
+			'db-coupon' => 'default'.$ds.'schema'.$ds.'coupon.php',
+			'db-catalog' => 'default'.$ds.'schema'.$ds.'catalog.php',
+			'db-tag' => 'default'.$ds.'schema'.$ds.'tag.php',
 		);
 
 		$this->setupSchema( $files );
 
 		$files = array(
-			'db-product' => 'default' . $ds . 'schema' . $ds . 'index.php',
+			'db-product' => 'default'.$ds.'schema'.$ds.'index.php',
 		);
 
 		$this->setupSchema( $files );
@@ -127,7 +127,7 @@ 
 
 		foreach( $this->getSetupPaths() as $abspath )
 		{
-			$filepath = $abspath . DIRECTORY_SEPARATOR . $relpath;
+			$filepath = $abspath.DIRECTORY_SEPARATOR.$relpath;
 
 			if( !file_exists( $filepath ) ) {
 				continue;
@@ -165,7 +165,7 @@ 
 	{
 		foreach( $files as $rname => $relpath )
 		{
-			$this->msg( 'Using schema from ' . basename( $relpath ), 1 ); $this->status( '' );
+			$this->msg( 'Using schema from '.basename( $relpath ), 1 ); $this->status( '' );
 
 			$dbal = $this->getConnection( $rname )->getRawObject();
 
@@ -223,7 +223,7 @@ 
 	{
 		foreach( $files as $rname => $filepath )
 		{
-			$this->msg( 'Using tables from ' . basename( $filepath ), 1 ); $this->status( '' );
+			$this->msg( 'Using tables from '.basename( $filepath ), 1 ); $this->status( '' );
 
 			if( ( $content = file_get_contents( $filepath ) ) === false ) {
 				throw new \Aimeos\MW\Setup\Exception( sprintf( 'Unable to get content from file "%1$s"', $filepath ) );
@@ -270,7 +270,7 @@ 
 	{
 		foreach( $this->getSetupPaths() as $abspath )
 		{
-			$filepath = $abspath . DIRECTORY_SEPARATOR . $relpath;
+			$filepath = $abspath.DIRECTORY_SEPARATOR.$relpath;
 
 			if( !file_exists( $filepath ) ) {
 				continue;

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/Plugin/Provider/Order/ProductStock.php

Spacing +4 added lines, -4 removed lines

@@ -81,7 +81,7 @@ 
 		}
 
 		foreach( $this->getStockItems( $productCodes, $stockTypes ) as $stockItem ) {
-			$stockMap[ $stockItem->getProductCode() ][ $stockItem->getType() ] = $stockItem->getStocklevel();
+			$stockMap[$stockItem->getProductCode()][$stockItem->getType()] = $stockItem->getStocklevel();
 		}
 
 		return $this->checkStockLevels( $order, $stockMap );
@@ -106,10 +106,10 @@ 
 		{
 			$stocklevel = 0;
 
-			if( isset( $stockMap[ $orderProductItem->getProductCode() ] )
-				&& array_key_exists( $orderProductItem->getStockType(), $stockMap[ $orderProductItem->getProductCode() ] )
+			if( isset( $stockMap[$orderProductItem->getProductCode()] )
+				&& array_key_exists( $orderProductItem->getStockType(), $stockMap[$orderProductItem->getProductCode()] )
 			) {
-				$stocklevel = $stockMap[ $orderProductItem->getProductCode() ][ $orderProductItem->getStockType() ];
+				$stocklevel = $stockMap[$orderProductItem->getProductCode()][$orderProductItem->getStockType()];
 			}
 
 			if( $stocklevel === null || $stocklevel >= $orderProductItem->getQuantity() ) {

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;