aimeos / aimeos-core

lib/mwlib/src/MW/DB/Statement/Base.php

Spacing +2 added lines, -2 removed lines

@@ -132,9 +132,9 @@
 					&& ( $count += substr_count( $pr, '\'' ) ) % 2 !== 0 )
 				{
 					if( ( $part = next( $parts ) ) === false ) {
-						throw new \Aimeos\MW\DB\Exception( 'Number of apostrophes don\'t match: ' . $sql );
+						throw new \Aimeos\MW\DB\Exception( 'Number of apostrophes don\'t match: '.$sql );
 					}
-					$temp .= '?' . $part;
+					$temp .= '?'.$part;
 				}
 				$result[] = $temp;
 			}

Doc Comments +1 added lines, -1 removed lines

@@ -174,7 +174,7 @@
 	 * Reconnects to the database server if not in a transaction
 	 *
 	 * @param \Exception $e Exception to throw if a transaction is running
-	 * @return \Aimeos\MW\DB\Statement\Iface Object for method chaining
+	 * @return Base Object for method chaining
 	 * @throws \Exception If a transaction is running
 	 */
 	protected function reconnect( \Exception $e )

lib/mshoplib/tests/MShop/Order/Manager/Base/StandardTest.php

Spacing +1 added lines, -1 removed lines

@@ -335,7 +335,7 @@
 
 	public function testSearchItemsDefault()
 	{
-		$search = $this->object->createSearch(  true );
+		$search = $this->object->createSearch( true );
 		$items = $this->object->searchItems( $search );
 
 		$this->assertEquals( 0, count( $items ) );

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

Doc Comments +1 added lines

@@ -200,6 +200,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 type items implementing \Aimeos\MShop\Common\Item\Type\Iface
 	 */
 	public function searchItems( \Aimeos\MW\Criteria\Iface $search, array $ref = [], &$total = null )

Spacing +11 added lines, -11 removed lines

@@ -61,7 +61,7 @@ 
 	public function createItem()
 	{
 		$values = array(
-			$this->prefix . 'siteid' => $this->getContext()->getLocale()->getSiteId(),
+			$this->prefix.'siteid' => $this->getContext()->getLocale()->getSiteId(),
 		);
 		return $this->createItemBase( $values );
 	}
@@ -117,7 +117,7 @@ 
 				$type = 'update';
 			}
 
-			$stmt = $conn->create( $this->getSqlConfig( $this->getConfigPath() . $type ) );
+			$stmt = $conn->create( $this->getSqlConfig( $this->getConfigPath().$type ) );
 
 			$stmt->bind( 1, $item->getCode(), \Aimeos\MW\DB\Statement\Base::PARAM_STR );
 			$stmt->bind( 2, $item->getDomain(), \Aimeos\MW\DB\Statement\Base::PARAM_STR );
@@ -138,7 +138,7 @@ 
 			if( $fetch === true )
 			{
 				if( $id === null ) {
-					$item->setId( $this->newId( $conn, $this->getConfigPath() . 'newid' ) );
+					$item->setId( $this->newId( $conn, $this->getConfigPath().'newid' ) );
 				} else {
 					$item->setId( $id ); // modified false
 				}
@@ -163,7 +163,7 @@ 
 	 */
 	public function deleteItems( array $ids )
 	{
-		$this->deleteItemsBase( $ids, $this->getConfigPath() . 'delete' );
+		$this->deleteItemsBase( $ids, $this->getConfigPath().'delete' );
 	}
 
 
@@ -180,8 +180,8 @@ 
 	public function findItem( $code, array $ref = [], $domain = 'product', $type = null, $default = false )
 	{
 		$find = array(
-			$this->prefix . 'code' => $code,
-			$this->prefix . 'domain' => $domain,
+			$this->prefix.'code' => $code,
+			$this->prefix.'domain' => $domain,
 		);
 		return $this->findItemBase( $find, $ref, $default );
 	}
@@ -197,7 +197,7 @@ 
 	 */
 	public function getItem( $id, array $ref = [], $default = false )
 	{
-		return $this->getItemBase( $this->prefix . 'id', $id, $ref, $default );
+		return $this->getItemBase( $this->prefix.'id', $id, $ref, $default );
 	}
 
 
@@ -226,13 +226,13 @@ 
 			}
 
 			$level = \Aimeos\MShop\Locale\Manager\Base::SITE_ALL;
-			$cfgPathSearch = $this->getConfigPath() . 'search';
-			$cfgPathCount = $this->getConfigPath() . 'count';
+			$cfgPathSearch = $this->getConfigPath().'search';
+			$cfgPathCount = $this->getConfigPath().'count';
 			$required = array( trim( $this->prefix, '.' ) );
 
 			$results = $this->searchItemsBase( $conn, $search, $cfgPathSearch, $cfgPathCount, $required, $total, $level );
 			while( ( $row = $results->fetch() ) !== false ) {
-				$items[$row[$this->prefix . 'id']] = $this->createItemBase( $row );
+				$items[$row[$this->prefix.'id']] = $this->createItemBase( $row );
 			}
 
 			$dbm->release( $conn, $dbname );
@@ -256,7 +256,7 @@ 
 	 */
 	public function getSubManager( $manager, $name = null )
 	{
-		return $this->getSubManagerBase( 'common', 'type/' . $manager, $name );
+		return $this->getSubManagerBase( 'common', 'type/'.$manager, $name );
 	}
 
 

controller/common/tests/TestHelperCntl.php

Spacing +3 added lines, -3 removed lines

@@ -48,7 +48,7 @@ 
 	{
 		if( !isset( self::$aimeos ) )
 		{
-			require_once dirname( dirname( dirname( __DIR__ ) ) ) . DIRECTORY_SEPARATOR . 'Bootstrap.php';
+			require_once dirname( dirname( dirname( __DIR__ ) ) ).DIRECTORY_SEPARATOR.'Bootstrap.php';
 
 			self::$aimeos = new \Aimeos\Bootstrap( [], false );
 		}
@@ -70,8 +70,8 @@ 
 
 
 		$paths = $aimeos->getConfigPaths();
-		$paths[] = __DIR__ . DIRECTORY_SEPARATOR . 'config';
-		$file = __DIR__ . DIRECTORY_SEPARATOR . 'confdoc.ser';
+		$paths[] = __DIR__.DIRECTORY_SEPARATOR.'config';
+		$file = __DIR__.DIRECTORY_SEPARATOR.'confdoc.ser';
 
 		$conf = new \Aimeos\MW\Config\PHPArray( [], $paths );
 		$conf = new \Aimeos\MW\Config\Decorator\Memory( $conf );

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

Spacing +6 added lines, -6 removed lines

@@ -56,11 +56,11 @@ 
 		{
 			if( ctype_alnum( $name ) === false )
 			{
-				$classname = is_string( $name ) ? $classprefix . $name : '<not a string>';
+				$classname = is_string( $name ) ? $classprefix.$name : '<not a string>';
 				throw new \Aimeos\Controller\Jobs\Exception( sprintf( 'Invalid characters in class name "%1$s"', $classname ) );
 			}
 
-			$classname = $classprefix . $name;
+			$classname = $classprefix.$name;
 
 			if( class_exists( $classname ) === false ) {
 				throw new \Aimeos\Controller\Jobs\Exception( sprintf( 'Class "%1$s" not available', $classname ) );
@@ -119,7 +119,7 @@ 
 		 * @category Developer
 		 */
 		$decorators = $config->get( 'controller/jobs/common/decorators/default', [] );
-		$excludes = $config->get( 'controller/jobs/' . $domain . '/decorators/excludes', [] );
+		$excludes = $config->get( 'controller/jobs/'.$domain.'/decorators/excludes', [] );
 
 		foreach( $decorators as $key => $name )
 		{
@@ -132,11 +132,11 @@ 
 		$controller = self::addDecorators( $context, $aimeos, $controller, $decorators, $classprefix );
 
 		$classprefix = '\\Aimeos\\Controller\\Jobs\\Common\\Decorator\\';
-		$decorators = $config->get( 'controller/jobs/' . $domain . '/decorators/global', [] );
+		$decorators = $config->get( 'controller/jobs/'.$domain.'/decorators/global', [] );
 		$controller = self::addDecorators( $context, $aimeos, $controller, $decorators, $classprefix );
 
-		$classprefix = '\\Aimeos\\Controller\\Jobs\\' . ucfirst( $localClass ) . '\\Decorator\\';
-		$decorators = $config->get( 'controller/jobs/' . $domain . '/decorators/local', [] );
+		$classprefix = '\\Aimeos\\Controller\\Jobs\\'.ucfirst( $localClass ).'\\Decorator\\';
+		$decorators = $config->get( 'controller/jobs/'.$domain.'/decorators/local', [] );
 		$controller = self::addDecorators( $context, $aimeos, $controller, $decorators, $classprefix );
 
 		return $controller;

controller/jobs/tests/TestHelperJobs.php

Spacing +4 added lines, -4 removed lines

@@ -48,7 +48,7 @@ 
 	{
 		if( !isset( self::$aimeos ) )
 		{
-			require_once dirname( dirname( dirname( __DIR__ ) ) ) . DIRECTORY_SEPARATOR . 'Bootstrap.php';
+			require_once dirname( dirname( dirname( __DIR__ ) ) ).DIRECTORY_SEPARATOR.'Bootstrap.php';
 
 			self::$aimeos = new \Aimeos\Bootstrap( [], false );
 		}
@@ -81,8 +81,8 @@ 
 
 
 		$paths = $aimeos->getConfigPaths();
-		$paths[] = __DIR__ . DIRECTORY_SEPARATOR . 'config';
-		$file = __DIR__ . DIRECTORY_SEPARATOR . 'confdoc.ser';
+		$paths[] = __DIR__.DIRECTORY_SEPARATOR.'config';
+		$file = __DIR__.DIRECTORY_SEPARATOR.'confdoc.ser';
 
 		$conf = new \Aimeos\MW\Config\PHPArray( [], $paths );
 		$conf = new \Aimeos\MW\Config\Decorator\Memory( $conf );
@@ -90,7 +90,7 @@ 
 		$ctx->setConfig( $conf );
 
 
-		$logger = new \Aimeos\MW\Logger\File( $site . '.log', \Aimeos\MW\Logger\Base::DEBUG );
+		$logger = new \Aimeos\MW\Logger\File( $site.'.log', \Aimeos\MW\Logger\Base::DEBUG );
 		$ctx->setLogger( $logger );
 
 

lib/mwlib/src/MW/Container/Zip.php

Spacing +3 added lines, -3 removed lines

@@ -41,7 +41,7 @@ 
 	 */
 	public function __construct( $resourcepath, $format, array $options = [] )
 	{
-		$this->classname = '\\Aimeos\\MW\\Container\\Content\\' . $format;
+		$this->classname = '\\Aimeos\\MW\\Container\\Content\\'.$format;
 
 		if( class_exists( $this->classname ) === false ) {
 			throw new \Aimeos\MW\Container\Exception( sprintf( 'Unknown format "%1$s"', $format ) );
@@ -105,7 +105,7 @@ 
 		}
 
 		// $this->container->getStream( $name ) doesn't work correctly because the stream can't be rewinded
-		return new $this->classname( 'zip://' . $this->resourcepath . '#' . $name, $name, $this->getOptions() );
+		return new $this->classname( 'zip://'.$this->resourcepath.'#'.$name, $name, $this->getOptions() );
 	}
 
 
@@ -149,7 +149,7 @@ 
 		}
 
 		// $this->container->getStream( $name ) doesn't work correctly because the stream can't be rewinded
-		return new $this->classname( 'zip://' . $this->resourcepath . '#' . $name, $name, $this->getOptions() );
+		return new $this->classname( 'zip://'.$this->resourcepath.'#'.$name, $name, $this->getOptions() );
 	}
 
 

lib/mwlib/src/MW/Container/Directory.php

Spacing +3 added lines, -3 removed lines

@@ -38,7 +38,7 @@ 
 	 */
 	public function __construct( $resourcepath, $format, array $options = [] )
 	{
-		$this->classname = '\\Aimeos\\MW\\Container\\Content\\' . $format;
+		$this->classname = '\\Aimeos\\MW\\Container\\Content\\'.$format;
 
 		if( class_exists( $this->classname ) === false ) {
 			throw new \Aimeos\MW\Container\Exception( sprintf( 'Unknown format "%1$s"', $format ) );
@@ -64,7 +64,7 @@ 
 	 */
 	public function create( $name )
 	{
-		$resource = $this->resource->getPath() . DIRECTORY_SEPARATOR . $name;
+		$resource = $this->resource->getPath().DIRECTORY_SEPARATOR.$name;
 		return new $this->classname( $resource, $name, $this->getOptions() );
 	}
 
@@ -88,7 +88,7 @@ 
 	 */
 	public function get( $name )
 	{
-		return new $this->classname( $this->resource->getPath() . DIRECTORY_SEPARATOR . $name, $name, $this->getOptions() );
+		return new $this->classname( $this->resource->getPath().DIRECTORY_SEPARATOR.$name, $name, $this->getOptions() );
 	}
 
 

lib/mwlib/src/MW/Template/Base.php

Spacing +3 added lines, -3 removed lines

@@ -117,7 +117,7 @@ 
 	public function getMarkerNames()
 	{
 		$matches = [];
-		$regex = '/' . str_replace( '\$', '(.*)', preg_quote( $this->begin, '/' ) ) . '/U';
+		$regex = '/'.str_replace( '\$', '(.*)', preg_quote( $this->begin, '/' ) ).'/U';
 
 		if( preg_match_all( $regex, $this->text, $matches ) === false ) {
 			throw new \Aimeos\MW\Template\Exception( sprintf( 'Invalid regular expression: %1$s', $regex ) );
@@ -186,7 +186,7 @@ 
 		$matches = [];
 		$text = $this->text;
 
-		$regex = '/' . str_replace( '\$', '(.*)', preg_quote( $this->begin, '/' ) ) . '/U';
+		$regex = '/'.str_replace( '\$', '(.*)', preg_quote( $this->begin, '/' ) ).'/U';
 		if( preg_match_all( $regex, $text, $matches ) === false ) {
 			throw new \Aimeos\MW\Template\Exception( sprintf( 'Invalid regular expression: %1$s', $regex ) );
 		}
@@ -197,7 +197,7 @@ 
 			$begin = str_replace( '\$', $match, preg_quote( $this->begin, '/' ) );
 			$end = str_replace( '\$', $match, preg_quote( $this->end, '/' ) );
 
-			$regex = '/' . $begin . '.*' . $end . '/smU';
+			$regex = '/'.$begin.'.*'.$end.'/smU';
 			if( ( $text = preg_replace( $regex, '', $text ) ) === null ) {
 				throw new \Aimeos\MW\Template\Exception( sprintf( 'Invalid regular expression: %1$s', $regex ) );
 			}