aimeos / aimeos-core

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 +19 added lines, -19 removed lines

@@ -273,7 +273,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 );
 
@@ -376,9 +376,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 ) );
 	}
 
 
@@ -394,7 +394,7 @@ 
 	{
 		$domain = strtolower( $domain );
 		$manager = strtolower( $manager );
-		$key = $domain . $manager . $name;
+		$key = $domain.$manager.$name;
 
 		if( !isset( $this->subManagers[$key] ) )
 		{
@@ -407,7 +407,7 @@ 
 			}
 
 			if( $name === null ) {
-				$path = 'mshop/' . $domain . '/manager/' . $manager . '/name';
+				$path = 'mshop/'.$domain.'/manager/'.$manager.'/name';
 				$name = $this->context->getConfig()->get( $path, 'Standard' );
 			}
 
@@ -418,8 +418,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 ) );
@@ -458,7 +458,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 ) );
@@ -487,7 +487,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 )
 		{
@@ -500,12 +500,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 );
 	}
@@ -543,7 +543,7 @@ 
 	 * @return \Aimeos\MShop\Common\Item\Iface Requested item
 	 * @throws \Aimeos\MShop\Exception if no item with the given ID found
 	 */
-	protected function findItemBase( array $pairs, array $ref, $default  )
+	protected function findItemBase( array $pairs, array $ref, $default )
 	{
 		$expr = [];
 		$criteria = $this->getObject()->createSearch( $default );
@@ -632,7 +632,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();
@@ -690,7 +690,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;
@@ -760,7 +760,7 @@ 
 
 		foreach( $keys as $key )
 		{
-			$name = $key . $sep . 'siteid';
+			$name = $key.$sep.'siteid';
 
 			if( isset( $attributes[$name] ) ) {
 				$cond[] = $search->compare( '==', $name, $siteIds );
@@ -848,7 +848,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 ) {
@@ -866,7 +866,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;
@@ -892,7 +892,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/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' );
 		}

lib/mshoplib/setup/ListsAddStatus.php

Braces +1 added lines, -2 removed lines

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

lib/mshoplib/setup/MShopAddPluginData.php

Braces +6 added lines, -2 removed lines

@@ -115,10 +115,14 @@
 				$item->setPosition( $dataset['position'] );
 			}
 
-			try {
+			try
+			{
 				$pluginManager->saveItem( $item );
 				$num++;
-			} catch( \Exception $e ) {; } // if plugin configuration was already available
+			}
+			catch( \Exception $e )
+			{
+; } // if plugin configuration was already available
 		}
 
 		$this->status( $num > 0 ? $num . '/' . $total : 'OK' );

Spacing +3 added lines, -3 removed lines

@@ -64,7 +64,7 @@ 
 		$pluginManager = \Aimeos\MShop\Plugin\Manager\Factory::createManager( $this->additional, 'Standard' );
 
 
-		$filename = __DIR__ . $ds . 'default' . $ds . 'data' . $ds . 'plugin.php';
+		$filename = __DIR__.$ds.'default'.$ds.'data'.$ds.'plugin.php';
 
 		if( ( $data = include( $filename ) ) == false ) {
 			throw new \Aimeos\MW\Setup\Exception( sprintf( 'No data file "%1$s" found', $filename ) );
@@ -90,7 +90,7 @@ 
 		$manager = $pluginManager->getSubManager( 'type' );
 
 		foreach( $manager->searchItems( $manager->createSearch() ) as $item ) {
-			$types['plugin/' . $item->getCode()] = $item;
+			$types['plugin/'.$item->getCode()] = $item;
 		}
 
 		$num = $total = 0;
@@ -121,6 +121,6 @@ 
 			} catch( \Exception $e ) {; } // if plugin configuration was already available
 		}
 
-		$this->status( $num > 0 ? $num . '/' . $total : 'OK' );
+		$this->status( $num > 0 ? $num.'/'.$total : 'OK' );
 	}
 }
\ No newline at end of file