aimeos / aimeos-core

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

Doc Comments +1 added lines, -1 removed lines

@@ -116,7 +116,7 @@
 	 *
 	 * @param string|null $domain Name of the domain (e.g. product, text, etc.) or null for all
 	 * @param array|string|null $type Name/Names of the item type or null for all
-	 * @param array|string|null $listtype Name/Names of the list item type or null for all
+	 * @param string $listtype Name/Names of the list item type or null for all
 	 * @return array List of items implementing \Aimeos\MShop\Common\Item\Iface
 	 */
 	public function getRefItems( $domain = null, $type = null, $listtype = null )

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

Doc Comments +1 added lines

@@ -695,6 +695,7 @@
 	 * @param string $path Configuration path to the sub-domains
 	 * @param array $default List of sub-domains if no others are configured
 	 * @param boolean $withsub Return also the resource type of sub-managers if true
+	 * @param string $type
 	 * @return array Type of the manager and submanagers, subtypes are separated by slashes
 	 */
 	protected function getResourceTypeBase( $type, $path, array $default, $withsub )

Braces +1 added lines, -2 removed lines

@@ -668,8 +668,7 @@
 
 		if( isset( $attributes[$name] ) && $attributes[$name] instanceof $iface ) {
 			return $attributes[$name]->getInternalDeps();
-		}
-		else if( isset( $attributes['id'] ) && $attributes['id'] instanceof $iface ) {
+		} else if( isset( $attributes['id'] ) && $attributes['id'] instanceof $iface ) {
 			return $attributes['id']->getInternalDeps();
 		}
 

Spacing +18 added lines, -18 removed lines

@@ -256,7 +256,7 @@ 
 		$conn = $dbm->acquire( $dbname );
 
 		$object = new \Aimeos\MW\Criteria\SQL( $conn );
-		$object->setConditions( $object->compare( '==', $domain . '.status', 1 ) );
+		$object->setConditions( $object->compare( '==', $domain.'.status', 1 ) );
 
 		$dbm->release( $conn, $dbname );
 
@@ -359,9 +359,9 @@ 
 	protected function getSqlConfig( $path )
 	{
 		$config = $this->getContext()->getConfig();
-		$adapter = $config->get( 'resource/' . $this->getResourceName() . '/adapter' );
+		$adapter = $config->get( 'resource/'.$this->getResourceName().'/adapter' );
 
-		return $config->get( $path . '/' . $adapter, $config->get( $path . '/ansi', $path ) );
+		return $config->get( $path.'/'.$adapter, $config->get( $path.'/ansi', $path ) );
 	}
 
 
@@ -377,7 +377,7 @@ 
 	{
 		$domain = strtolower( $domain );
 		$manager = strtolower( $manager );
-		$key = $domain . $manager . $name;
+		$key = $domain.$manager.$name;
 
 		if( !isset( $this->subManagers[$key] ) )
 		{
@@ -390,7 +390,7 @@ 
 			}
 
 			if( $name === null ) {
-				$path = 'mshop/' . $domain . '/manager/' . $manager . '/name';
+				$path = 'mshop/'.$domain.'/manager/'.$manager.'/name';
 				$name = $this->context->getConfig()->get( $path, 'Standard' );
 			}
 
@@ -401,8 +401,8 @@ 
 			$domainname = ucfirst( $domain );
 			$subnames = $this->createSubNames( $manager );
 
-			$classname = '\\Aimeos\\MShop\\' . $domainname . '\\Manager\\' . $subnames . '\\' . $name;
-			$interface = '\\Aimeos\\MShop\\' . $domainname . '\\Manager\\' . $subnames . '\\Iface';
+			$classname = '\\Aimeos\\MShop\\'.$domainname.'\\Manager\\'.$subnames.'\\'.$name;
+			$interface = '\\Aimeos\\MShop\\'.$domainname.'\\Manager\\'.$subnames.'\\Iface';
 
 			if( class_exists( $classname ) === false ) {
 				throw new \Aimeos\MShop\Exception( sprintf( 'Class "%1$s" not available', $classname ) );
@@ -441,7 +441,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 ) );
@@ -470,7 +470,7 @@ 
 		$config = $this->context->getConfig();
 
 		$decorators = $config->get( 'mshop/common/manager/decorators/default', [] );
-		$excludes = $config->get( 'mshop/' . $domain . '/manager/' . $managerpath . '/decorators/excludes', [] );
+		$excludes = $config->get( 'mshop/'.$domain.'/manager/'.$managerpath.'/decorators/excludes', [] );
 
 		foreach( $decorators as $key => $name )
 		{
@@ -483,12 +483,12 @@ 
 		$manager = $this->addDecorators( $this->context, $manager, $decorators, $classprefix );
 
 		$classprefix = '\\Aimeos\\MShop\\Common\\Manager\\Decorator\\';
-		$decorators = $config->get( 'mshop/' . $domain . '/manager/' . $managerpath . '/decorators/global', [] );
+		$decorators = $config->get( 'mshop/'.$domain.'/manager/'.$managerpath.'/decorators/global', [] );
 		$manager = $this->addDecorators( $this->context, $manager, $decorators, $classprefix );
 
 		$subpath = $this->createSubNames( $managerpath );
-		$classprefix = 'MShop_' . ucfirst( $domain ) . '_Manager_' . $subpath . '_Decorator_';
-		$decorators = $config->get( 'mshop/' . $domain . '/manager/' . $managerpath . '/decorators/local', [] );
+		$classprefix = 'MShop_'.ucfirst( $domain ).'_Manager_'.$subpath.'_Decorator_';
+		$decorators = $config->get( 'mshop/'.$domain.'/manager/'.$managerpath.'/decorators/local', [] );
 
 		return $this->addDecorators( $this->context, $manager, $decorators, $classprefix );
 	}
@@ -614,7 +614,7 @@ 
 	{
 		$iface = '\\Aimeos\\MW\\Criteria\\Attribute\\Iface';
 		$sep = $this->getKeySeparator();
-		$name = $prefix . $sep . 'id';
+		$name = $prefix.$sep.'id';
 
 		if( isset( $attributes[$name] ) && $attributes[$name] instanceof $iface ) {
 			return $attributes[$name]->getInternalDeps();
@@ -672,7 +672,7 @@ 
 	{
 		$config = $this->context->getConfig();
 
-		if( $config->get( 'resource/' . $name ) === null ) {
+		if( $config->get( 'resource/'.$name ) === null ) {
 			$this->resourceName = $config->get( 'resource/default', 'db' );
 		} else {
 			$this->resourceName = $name;
@@ -742,7 +742,7 @@ 
 
 		foreach( $keys as $key )
 		{
-			$name = $key . $sep . 'siteid';
+			$name = $key.$sep.'siteid';
 
 			if( isset( $attributes[$name] ) ) {
 				$cond[] = $search->compare( '==', $name, $siteIds );
@@ -830,7 +830,7 @@ 
 			$row = $results->fetch();
 			$results->finish();
 
-			$msg = get_class( $this ) . ' (' . ( ( microtime( true ) - $time ) * 1000 ) . 'ms): SQL = ' . $stmt;
+			$msg = get_class( $this ).' ('.( ( microtime( true ) - $time ) * 1000 ).'ms): SQL = '.$stmt;
 			$this->context->getLogger()->log( $msg, \Aimeos\MW\Logger\Base::DEBUG, 'core/sql' );
 
 			if( $row === false ) {
@@ -848,7 +848,7 @@ 
 		$stmt = $conn->create( $sql->str() );
 		$results = $stmt->execute();
 
-		$msg = get_class( $this ) . ' (' . ( ( microtime( true ) - $time ) * 1000 ) . 'ms): SQL = ' . $stmt;
+		$msg = get_class( $this ).' ('.( ( microtime( true ) - $time ) * 1000 ).'ms): SQL = '.$stmt;
 		$this->context->getLogger()->log( $msg, \Aimeos\MW\Logger\Base::DEBUG, 'core/sql' );
 
 		return $results;
@@ -874,7 +874,7 @@ 
 		$search->setConditions( $search->compare( '==', $name, $ids ) );
 
 		$types = array( $name => \Aimeos\MW\DB\Statement\Base::PARAM_STR );
-		$translations = array( $name => '"' . $name . '"' );
+		$translations = array( $name => '"'.$name.'"' );
 
 		$cond = $search->getConditionString( $types, $translations );
 		$sql = str_replace( ':cond', $cond, $this->getSqlConfig( $cfgpath ) );

controller/jobs/src/Controller/Jobs/Factory.php

Braces +1 added lines, -2 removed lines

@@ -136,8 +136,7 @@
 				$subList = self::createControllers( $it, $context, $aimeos, $pref );
 
 				$list = array_merge( $list, $subList );
-			}
-			else if( $prefix !== '' && $entry->getType() === 'file'
+			} else if( $prefix !== '' && $entry->getType() === 'file'
 				&& ( $name = $entry->getBaseName( '.php' ) ) === 'Factory' )
 			{
 				$list[$prefix] = self::createController( $context, $aimeos, $prefix );

Spacing +4 added lines, -4 removed lines

@@ -56,7 +56,7 @@ 
 			$parts[$key] = ucwords( $part );
 		}
 
-		$factory = '\\Aimeos\\Controller\\Jobs\\' . join( '\\', $parts ) . '\\Factory';
+		$factory = '\\Aimeos\\Controller\\Jobs\\'.join( '\\', $parts ).'\\Factory';
 
 		if( class_exists( $factory ) === false ) {
 			throw new \Aimeos\Controller\Jobs\Exception( sprintf( 'Class "%1$s" not found', $factory ) );
@@ -86,7 +86,7 @@ 
 		$cntlList = [];
 		$subFolder = trim( str_replace( '\\', '/', self::$prefix ), '/' );
 
-		if( strncmp( $subFolder, 'Aimeos' . '/', 7 ) === 0 ) {
+		if( strncmp( $subFolder, 'Aimeos'.'/', 7 ) === 0 ) {
 			$subFolder = substr( $subFolder, 7 );
 		}
 
@@ -94,7 +94,7 @@ 
 		{
 			foreach( $list as $relpath )
 			{
-				$path .= DIRECTORY_SEPARATOR . str_replace( '/', DIRECTORY_SEPARATOR, $relpath . '/' . $subFolder );
+				$path .= DIRECTORY_SEPARATOR.str_replace( '/', DIRECTORY_SEPARATOR, $relpath.'/'.$subFolder );
 
 				if( is_dir( $path ) )
 				{
@@ -132,7 +132,7 @@ 
 			{
 				$name = strtolower( $entry->getBaseName() );
 				$it = new \DirectoryIterator( $entry->getPathName() );
-				$pref = ( $prefix !== '' ? $prefix . '/' : '' ) . $name;
+				$pref = ( $prefix !== '' ? $prefix.'/' : '' ).$name;
 				$subList = self::createControllers( $it, $context, $aimeos, $pref );
 
 				$list = array_merge( $list, $subList );

lib/mshoplib/setup/AttributeChangeTypeidNotNull.php

Braces +1 added lines, -2 removed lines

@@ -72,8 +72,7 @@
 		{
 			$this->executeList( $stmts );
 			$this->status( 'migrated' );
-		}
-		else
+		} else
 		{
 			$this->status( 'OK' );
 		}

lib/mshoplib/setup/CatalogAddCode.php

Braces +1 added lines, -2 removed lines

@@ -65,8 +65,7 @@
 		{
 			$this->executeList( $stmts );
 			$this->status( 'added' );
-		}
-		else
+		} else
 		{
 			$this->status( 'OK' );
 		}

lib/mshoplib/setup/CatalogIndexRenameDomainTables.php

Braces +1 added lines, -2 removed lines

@@ -71,8 +71,7 @@
 			{
 				$this->execute( $stmt );
 				$this->status( 'done' );
-			}
-			else
+			} else
 			{
 				$this->status( 'OK' );
 			}

lib/mshoplib/setup/CatalogRemoveSuggest.php

Braces +1 added lines, -2 removed lines

@@ -90,8 +90,7 @@
 			$this->executeList( $stmts['drop'] );
 
 			$this->status( sprintf( 'migrated (%1$d)', $count ) );
-		}
-		else
+		} else
 		{
 			$this->status( 'OK' );
 		}

lib/mshoplib/setup/ColumnDomainChangeLength.php

Braces +1 added lines, -2 removed lines

@@ -172,8 +172,7 @@
 			{
 				$this->executeList( $stmtList );
 				$this->status( 'migrated' );
-			}
-			else
+			} else
 			{
 				$this->status( 'OK' );
 			}

lib/mshoplib/setup/CouponMigrateConfigKeys.php

Braces +1 added lines, -2 removed lines

@@ -90,8 +90,7 @@
 		{
 			$this->executeList( $stmts, 'db-coupon' );
 			$this->status( 'done' );
-		}
-		else
+		} else
 		{
 			$this->status( 'OK' );
 		}