aimeos / aimeos-core

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

Doc Comments +3 added lines, -2 removed lines

@@ -259,7 +259,7 @@ 
 	/**
 	 * Deletes the item specified by its ID.
 	 *
-	 * @param mixed $id ID of the item object
+	 * @param integer $id ID of the item object
 	 */
 	public function deleteItem( $id )
 	{
@@ -282,7 +282,7 @@ 
 	/**
 	 * Removes multiple items specified by ids in the array.
 	 *
-	 * @param array $ids List of IDs
+	 * @param integer[] $ids List of IDs
 	 */
 	public function deleteItems( array $ids )
 	{
@@ -460,6 +460,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 items implementing \Aimeos\MShop\Common\Item\Iface
 	 */
 	public function searchItems( \Aimeos\MW\Criteria\Iface $search, array $ref = [], &$total = null )

lib/mshoplib/src/MShop/Plugin/Provider/Base.php

Doc Comments +2 added lines, -2 removed lines

@@ -42,7 +42,7 @@ 
 	 * Injects the outer object into the decorator stack
 	 *
 	 * @param \Aimeos\MShop\Plugin\Provider\Iface $object First object of the decorator stack
-	 * @return \Aimeos\MShop\Plugin\Provider\Iface Plugin object for chaining method calls
+	 * @return Base Plugin object for chaining method calls
 	 */
 	public function setObject( \Aimeos\MShop\Plugin\Provider\Iface $object )
 	{
@@ -81,7 +81,7 @@ 
 	 * Returns the configuration value from the service item specified by its key.
 	 *
 	 * @param string $key Configuration key
-	 * @param mixed $default Default value if configuration key isn't available
+	 * @param boolean $default Default value if configuration key isn't available
 	 * @return string|null Value from service item configuration
 	 */
 	protected function getConfigValue( $key, $default = null )

lib/mshoplib/src/MShop/Plugin/Provider/Decorator/Base.php

Doc Comments +1 added lines, -1 removed lines

@@ -69,7 +69,7 @@
 	 * Injects the outmost object into the decorator stack
 	 *
 	 * @param \Aimeos\MShop\Plugin\Provider\Iface $object First object of the decorator stack
-	 * @return \Aimeos\MShop\Plugin\Provider\Iface Plugin object for chaining method calls
+	 * @return Base Plugin object for chaining method calls
 	 */
 	public function setObject( \Aimeos\MShop\Plugin\Provider\Iface $object )
 	{

lib/mshoplib/src/MShop/Plugin/Provider/Decorator/Log.php

Spacing +3 added lines, -3 removed lines

@@ -29,7 +29,7 @@ 
 	public function register( \Aimeos\MW\Observer\Publisher\Iface $p )
 	{
 		$class = get_class( $this->getProvider() );
-		$this->getContext()->getLogger()->log( 'Plugin::register: ' . $class, \Aimeos\MW\Logger\Base::DEBUG );
+		$this->getContext()->getLogger()->log( 'Plugin::register: '.$class, \Aimeos\MW\Logger\Base::DEBUG );
 
 		$this->getProvider()->register( $p );
 	}
@@ -47,12 +47,12 @@ 
 		$class = get_class( $this->getProvider() );
 		$payload = ( is_object( $value ) ? get_class( $value ) : ( is_scalar( $value ) ? $value : '' ) );
 
-		$msg = 'Plugin::update:before: ' . $class . ', action: ' . $action . ', value: ' . $payload;
+		$msg = 'Plugin::update:before: '.$class.', action: '.$action.', value: '.$payload;
 		$this->getContext()->getLogger()->log( $msg, \Aimeos\MW\Logger\Base::DEBUG );
 
 		$result = $this->getProvider()->update( $order, $action, $value );
 
-		$msg = 'Plugin::update:after: ' . $class . ', action: ' . $action . ', value: ' . $payload;
+		$msg = 'Plugin::update:after: '.$class.', action: '.$action.', value: '.$payload;
 		$this->getContext()->getLogger()->log( $msg, \Aimeos\MW\Logger\Base::DEBUG );
 
 		return $result;

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

Spacing +3 added lines, -3 removed lines

@@ -215,7 +215,7 @@ 
 	{
 		$path = 'mshop/plugin/manager/submanagers';
 
-		return $this->getResourceTypeBase( 'plugin', $path, array( 'type'), $withsub );
+		return $this->getResourceTypeBase( 'plugin', $path, array( 'type' ), $withsub );
 	}
 
 
@@ -303,7 +303,7 @@ 
 		}
 
 		$interface = '\\Aimeos\\MShop\\Plugin\\Provider\\Factory\\Iface';
-		$classname = '\\Aimeos\\MShop\\Plugin\\Provider\\' . $type . '\\' . $provider;
+		$classname = '\\Aimeos\\MShop\\Plugin\\Provider\\'.$type.'\\'.$provider;
 
 		if( class_exists( $classname ) === false ) {
 			throw new \Aimeos\MShop\Plugin\Exception( sprintf( 'Class "%1$s" not available', $classname ) );
@@ -341,7 +341,7 @@ 
 		 * @category Developer
 		 * @see mshop/plugin/provider/order/decorators
 		 */
-		$decorators = $config->get( 'mshop/plugin/provider/' . $item->getType() . '/decorators', [] );
+		$decorators = $config->get( 'mshop/plugin/provider/'.$item->getType().'/decorators', [] );
 
 		$provider = $this->addPluginDecorators( $item, $provider, $names );
 		$provider = $this->addPluginDecorators( $item, $provider, $decorators );

lib/mshoplib/src/MShop/Common/Manager/ListRef/Base.php

Spacing +4 added lines, -4 removed lines

@@ -34,7 +34,7 @@ 
 	 */
 	public function updateListItems( \Aimeos\MShop\Common\Item\ListRef\Iface $item, array $map, $domain, $type )
 	{
-		$listManager =  $this->getObject()->getSubManager( 'lists' );
+		$listManager = $this->getObject()->getSubManager( 'lists' );
 
 		if( !isset( $this->typeIds[$domain][$type] ) )
 		{
@@ -159,8 +159,8 @@ 
 		$search = $manager->createSearch( true );
 
 		$expr = array(
-			$search->compare( '==', $prefix . '.lists.parentid', $ids ),
-			$search->compare( '==', $prefix . '.lists.domain', $domains ),
+			$search->compare( '==', $prefix.'.lists.parentid', $ids ),
+			$search->compare( '==', $prefix.'.lists.domain', $domains ),
 			$search->getConditions(),
 		);
 
@@ -190,7 +190,7 @@ 
 
 				$search = $manager->createSearch( true );
 				$expr = array(
-					$search->compare( '==', str_replace( '/', '.', $domain ) . '.id', array_keys( $list ) ),
+					$search->compare( '==', str_replace( '/', '.', $domain ).'.id', array_keys( $list ) ),
 					$search->getConditions(),
 				);
 				$search->setConditions( $search->combine( '&&', $expr ) );

lib/mshoplib/src/MShop/Common/Manager/Lists/Base.php

Spacing +29 added lines, -29 removed lines

@@ -63,7 +63,7 @@ 
 	public function aggregate( \Aimeos\MW\Criteria\Iface $search, $key )
 	{
 		$required = array( trim( $this->prefix, '.' ) );
-		return $this->aggregateBase( $search, $key, $this->getConfigPath() . 'aggregate', $required );
+		return $this->aggregateBase( $search, $key, $this->getConfigPath().'aggregate', $required );
 	}
 
 
@@ -75,7 +75,7 @@ 
 	public function createItem()
 	{
 		$values = array(
-			$this->prefix . 'siteid' => $this->getContext()->getLocale()->getSiteId()
+			$this->prefix.'siteid' => $this->getContext()->getLocale()->getSiteId()
 		);
 		return $this->createItemBase( $values );
 
@@ -112,7 +112,7 @@ 
 			}
 
 			$time = date( 'Y-m-d H:i:s' );
-			$statement = $this->getCachedStatement( $conn, $this->getConfigPath() . $type );
+			$statement = $this->getCachedStatement( $conn, $this->getConfigPath().$type );
 
 			$statement->bind( 1, $item->getParentId(), \Aimeos\MW\DB\Statement\Base::PARAM_INT );
 			$statement->bind( 2, $context->getLocale()->getSiteId(), \Aimeos\MW\DB\Statement\Base::PARAM_INT );
@@ -140,7 +140,7 @@ 
 			if( $fetch === true )
 			{
 				if( $id === null ) {
-					$path = $this->getConfigPath() . 'newid';
+					$path = $this->getConfigPath().'newid';
 					$item->setId( $this->newId( $conn, $path ) );
 				} else {
 					$item->setId( $id ); // modified false
@@ -164,7 +164,7 @@ 
 	 */
 	public function deleteItems( array $ids )
 	{
-		$this->deleteItemsBase( $ids, $this->getConfigPath() . 'delete' );
+		$this->deleteItemsBase( $ids, $this->getConfigPath().'delete' );
 	}
 
 
@@ -236,7 +236,7 @@ 
 			{
 				$newpos = $pos;
 
-				$sql = $this->getSqlConfig( $cfgPath . 'move' );
+				$sql = $this->getSqlConfig( $cfgPath.'move' );
 
 				$stmt = $conn->create( $sql );
 				$stmt->bind( 1, +1, \Aimeos\MW\DB\Statement\Base::PARAM_INT );
@@ -252,7 +252,7 @@ 
 			}
 			else
 			{
-				$sql = $this->getSqlConfig( $cfgPath . 'getposmax' );
+				$sql = $this->getSqlConfig( $cfgPath.'getposmax' );
 
 				$stmt = $conn->create( $sql );
 
@@ -270,7 +270,7 @@ 
 				}
 			}
 
-			$sql = $this->getSqlConfig( $cfgPath . 'updatepos' );
+			$sql = $this->getSqlConfig( $cfgPath.'updatepos' );
 
 			$stmt = $conn->create( $sql );
 			$stmt->bind( 1, $newpos, \Aimeos\MW\DB\Statement\Base::PARAM_INT );
@@ -286,7 +286,7 @@ 
 
 			if( $oldpos > 0 )
 			{
-				$sql = $this->getSqlConfig( $cfgPath . 'move' );
+				$sql = $this->getSqlConfig( $cfgPath.'move' );
 
 				$stmt = $conn->create( $sql );
 				$stmt->bind( 1, -1, \Aimeos\MW\DB\Statement\Base::PARAM_INT );
@@ -336,8 +336,8 @@ 
 			}
 
 			$level = \Aimeos\MShop\Locale\Manager\Base::SITE_ALL;
-			$cfgPathSearch = $this->getConfigPath() . 'search';
-			$cfgPathCount = $this->getConfigPath() . 'count';
+			$cfgPathSearch = $this->getConfigPath().'search';
+			$cfgPathCount = $this->getConfigPath().'count';
 
 			$name = trim( $this->prefix, '.' );
 			$required = array( $name );
@@ -346,12 +346,12 @@ 
 
 			while( ( $row = $results->fetch() ) !== false )
 			{
-				if( ( $row[$this->prefix . 'config'] = json_decode( $row[$this->prefix . 'config'], true ) ) === null ) {
-					$row[$this->prefix . 'config'] = [];
+				if( ( $row[$this->prefix.'config'] = json_decode( $row[$this->prefix.'config'], true ) ) === null ) {
+					$row[$this->prefix.'config'] = [];
 				}
 
-				$map[$row[$this->prefix . 'id']] = $row;
-				$typeIds[$row[$this->prefix . 'typeid']] = null;
+				$map[$row[$this->prefix.'id']] = $row;
+				$typeIds[$row[$this->prefix.'typeid']] = null;
 			}
 
 			$dbm->release( $conn, $dbname );
@@ -366,19 +366,19 @@ 
 		{
 			$typeManager = $this->getObject()->getSubManager( 'type' );
 			$typeSearch = $typeManager->createSearch();
-			$typeSearch->setConditions( $typeSearch->compare( '==', $name . '.type.id', array_keys( $typeIds ) ) );
+			$typeSearch->setConditions( $typeSearch->compare( '==', $name.'.type.id', array_keys( $typeIds ) ) );
 			$typeSearch->setSlice( 0, $search->getSliceSize() );
 			$typeItems = $typeManager->searchItems( $typeSearch );
 
 			foreach( $map as $id => $row )
 			{
-				if( isset( $typeItems[$row[$this->prefix . 'typeid']] ) )
+				if( isset( $typeItems[$row[$this->prefix.'typeid']] ) )
 				{
-					$row[$this->prefix . 'type'] = $typeItems[$row[$this->prefix . 'typeid']]->getCode();
-					$row[$this->prefix . 'typename'] = $typeItems[$row[$this->prefix . 'typeid']]->getName();
+					$row[$this->prefix.'type'] = $typeItems[$row[$this->prefix.'typeid']]->getCode();
+					$row[$this->prefix.'typename'] = $typeItems[$row[$this->prefix.'typeid']]->getName();
 				}
 
-				$items[$row[$this->prefix . 'id']] = $this->createItemBase( $row );
+				$items[$row[$this->prefix.'id']] = $this->createItemBase( $row );
 			}
 		}
 
@@ -417,8 +417,8 @@ 
 			}
 
 			$level = \Aimeos\MShop\Locale\Manager\Base::SITE_ALL;
-			$cfgPathSearch = $this->getConfigPath() . 'search';
-			$cfgPathCount = $this->getConfigPath() . 'count';
+			$cfgPathSearch = $this->getConfigPath().'search';
+			$cfgPathCount = $this->getConfigPath().'count';
 
 			$name = trim( $this->prefix, '.' );
 			$required = array( $name );
@@ -426,7 +426,7 @@ 
 			$results = $this->searchItemsBase( $conn, $search, $cfgPathSearch, $cfgPathCount, $required, $total, $level );
 
 			while( ( $row = $results->fetch() ) !== false ) {
-				$map[$row[$this->prefix . 'domain']][] = $row[$this->prefix . 'refid'];
+				$map[$row[$this->prefix.'domain']][] = $row[$this->prefix.'refid'];
 			}
 
 			$dbm->release( $conn, $dbname );
@@ -444,7 +444,7 @@ 
 
 			$search = $manager->createSearch( true );
 			$expr = array(
-				$search->compare( '==', str_replace( '/', '.', $domain ) . '.id', $list ),
+				$search->compare( '==', str_replace( '/', '.', $domain ).'.id', $list ),
 				$search->getConditions(),
 			);
 			$search->setConditions( $search->combine( '&&', $expr ) );
@@ -476,13 +476,13 @@ 
 			$expr[] = $object->getConditions();
 
 			$exprTwo = [];
-			$exprTwo[] = $object->compare( '<=', $prefix . '.datestart', $curDate );
-			$exprTwo[] = $object->compare( '==', $prefix . '.datestart', null );
+			$exprTwo[] = $object->compare( '<=', $prefix.'.datestart', $curDate );
+			$exprTwo[] = $object->compare( '==', $prefix.'.datestart', null );
 			$expr[] = $object->combine( '||', $exprTwo );
 
 			$exprTwo = [];
-			$exprTwo[] = $object->compare( '>=', $prefix . '.dateend', $curDate );
-			$exprTwo[] = $object->compare( '==', $prefix . '.dateend', null );
+			$exprTwo[] = $object->compare( '>=', $prefix.'.dateend', $curDate );
+			$exprTwo[] = $object->compare( '==', $prefix.'.dateend', null );
 			$expr[] = $object->combine( '||', $exprTwo );
 
 			$object->setConditions( $object->combine( '&&', $expr ) );
@@ -503,7 +503,7 @@ 
 	 */
 	public function getSubManager( $manager, $name = null )
 	{
-		return $this->getSubManagerBase( 'common', 'lists/' . $manager, $name );
+		return $this->getSubManagerBase( 'common', 'lists/'.$manager, $name );
 	}
 
 

lib/mshoplib/src/MShop/Common/Factory/Base.php

Spacing +5 added lines, -5 removed lines

@@ -57,7 +57,7 @@ 
 				throw new \Aimeos\MShop\Exception( sprintf( 'Invalid characters in class name "%1$s"', $name ) );
 			}
 
-			$classname = $classprefix . $name;
+			$classname = $classprefix.$name;
 
 			if( class_exists( $classname ) === false ) {
 				throw new \Aimeos\MShop\Exception( sprintf( 'Class "%1$s" not available', $classname ) );
@@ -110,7 +110,7 @@ 
 		 * @category Developer
 		 */
 		$decorators = $config->get( 'mshop/common/manager/decorators/default', [] );
-		$excludes = $config->get( 'mshop/' . $domain . '/manager/decorators/excludes', [] );
+		$excludes = $config->get( 'mshop/'.$domain.'/manager/decorators/excludes', [] );
 
 		foreach( $decorators as $key => $name )
 		{
@@ -123,11 +123,11 @@ 
 		$manager = self::addDecorators( $context, $manager, $decorators, $classprefix );
 
 		$classprefix = '\\Aimeos\\MShop\\Common\\Manager\\Decorator\\';
-		$decorators = $config->get( 'mshop/' . $domain . '/manager/decorators/global', [] );
+		$decorators = $config->get( 'mshop/'.$domain.'/manager/decorators/global', [] );
 		$manager = self::addDecorators( $context, $manager, $decorators, $classprefix );
 
-		$classprefix = '\\Aimeos\\MShop\\' . ucfirst( $domain ) . '\\Manager\\Decorator\\';
-		$decorators = $config->get( 'mshop/' . $domain . '/manager/decorators/local', [] );
+		$classprefix = '\\Aimeos\\MShop\\'.ucfirst( $domain ).'\\Manager\\Decorator\\';
+		$decorators = $config->get( 'mshop/'.$domain.'/manager/decorators/local', [] );
 		$manager = self::addDecorators( $context, $manager, $decorators, $classprefix );
 
 		return $manager->setObject( $manager );

lib/mshoplib/src/MShop/Order/Manager/Base/Base.php

Spacing +4 added lines, -4 removed lines

@@ -97,7 +97,7 @@ 
 		$currency = $locale->getCurrencyId();
 		$language = $locale->getLanguageId();
 		$sitecode = $locale->getSite()->getCode();
-		$key = 'aimeos/basket/content-' . $sitecode . '-' . $language . '-' . $currency . '-' . strval( $type );
+		$key = 'aimeos/basket/content-'.$sitecode.'-'.$language.'-'.$currency.'-'.strval( $type );
 
 		if( ( $serorder = $session->get( $key ) ) === null ) {
 			return $this->getObject()->createItem();
@@ -133,7 +133,7 @@ 
 		$currency = $locale->getCurrencyId();
 		$language = $locale->getLanguageId();
 		$sitecode = $locale->getSite()->getCode();
-		$key = 'aimeos/basket/lock-' . $sitecode . '-' . $language . '-' . $currency . '-' . strval( $type );
+		$key = 'aimeos/basket/lock-'.$sitecode.'-'.$language.'-'.$currency.'-'.strval( $type );
 
 		if( ( $value = $session->get( $key ) ) !== null ) {
 			return (int) $value;
@@ -157,7 +157,7 @@ 
 		$currency = $locale->getCurrencyId();
 		$language = $locale->getLanguageId();
 		$sitecode = $locale->getSite()->getCode();
-		$key = 'aimeos/basket/content-' . $sitecode . '-' . $language . '-' . $currency . '-' . strval( $type );
+		$key = 'aimeos/basket/content-'.$sitecode.'-'.$language.'-'.$currency.'-'.strval( $type );
 
 		$session->set( $key, serialize( clone $order ) );
 	}
@@ -181,7 +181,7 @@ 
 		$currency = $locale->getCurrencyId();
 		$language = $locale->getLanguageId();
 		$sitecode = $locale->getSite()->getCode();
-		$key = 'aimeos/basket/lock-' . $sitecode . '-' . $language . '-' . $currency . '-' . strval( $type );
+		$key = 'aimeos/basket/lock-'.$sitecode.'-'.$language.'-'.$currency.'-'.strval( $type );
 
 		$session->set( $key, strval( $lock ) );
 	}