aimeos / ai-controller-jobs

controller/common/src/Controller/Common/Product/Import/Csv/Cache/Product/Standard.php

Spacing +1 added lines, -1 removed lines

@@ -70,6 +70,6 @@
 	 */
 	public function set( \Aimeos\MShop\Common\Item\Iface $item )
 	{
-		$this->prodmap[ $item->getCode() ] = $item->getId();
+		$this->prodmap[$item->getCode()] = $item->getId();
 	}
 }
\ No newline at end of file

controller/common/src/Controller/Common/Product/Import/Csv/Base.php

Spacing +6 added lines, -6 removed lines

@@ -57,7 +57,7 @@ 
 
 		if( ctype_alnum( $type ) === false )
 		{
-			$classname = is_string($name) ? '\\Aimeos\\Controller\\Common\\Product\\Import\\Csv\\Cache\\' . $type : '<not a string>';
+			$classname = is_string( $name ) ? '\\Aimeos\\Controller\\Common\\Product\\Import\\Csv\\Cache\\' . $type : '<not a string>';
 			throw new \Aimeos\Controller\Jobs\Exception( sprintf( 'Invalid characters in class name "%1$s"', $classname ) );
 		}
 
@@ -67,7 +67,7 @@ 
 
 		if( ctype_alnum( $name ) === false )
 		{
-			$classname = is_string($name) ? '\\Aimeos\\Controller\\Common\\Product\\Import\\Csv\\Cache\\' . $type . '\\' . $name : '<not a string>';
+			$classname = is_string( $name ) ? '\\Aimeos\\Controller\\Common\\Product\\Import\\Csv\\Cache\\' . $type . '\\' . $name : '<not a string>';
 			throw new \Aimeos\Controller\Jobs\Exception( sprintf( 'Invalid characters in class name "%1$s"', $classname ) );
 		}
 
@@ -119,7 +119,7 @@ 
 		while( $content->valid() && $count++ < $maxcnt )
 		{
 			$row = $content->current();
-			$data[ $row[$codePos] ] = $row;
+			$data[$row[$codePos]] = $row;
 			$content->next();
 		}
 
@@ -256,7 +256,7 @@ 
 		{
 			if( ctype_alnum( $type ) === false )
 			{
-				$classname = is_string($type) ? '\\Aimeos\\Controller\\Common\\Product\\Import\\Csv\\Processor\\' . $type : '<not a string>';
+				$classname = is_string( $type ) ? '\\Aimeos\\Controller\\Common\\Product\\Import\\Csv\\Processor\\' . $type : '<not a string>';
 				throw new \Aimeos\Controller\Jobs\Exception( sprintf( 'Invalid characters in class name "%1$s"', $classname ) );
 			}
 
@@ -264,7 +264,7 @@ 
 
 			if( ctype_alnum( $name ) === false )
 			{
-				$classname = is_string($name) ? '\\Aimeos\\Controller\\Common\\Product\\Import\\Csv\\Processor\\' . $type . '\\' . $name : '<not a string>';
+				$classname = is_string( $name ) ? '\\Aimeos\\Controller\\Common\\Product\\Import\\Csv\\Processor\\' . $type . '\\' . $name : '<not a string>';
 				throw new \Aimeos\Controller\Jobs\Exception( sprintf( 'Invalid characters in class name "%1$s"', $classname ) );
 			}
 
@@ -300,7 +300,7 @@ 
 		$search->setSlice( 0, count( $codes ) );
 
 		foreach( $manager->searchItems( $search, $domains ) as $item ) {
-			$result[ $item->getCode() ] = $item;
+			$result[$item->getCode()] = $item;
 		}
 
 		return $result;

controller/jobs/src/Controller/Jobs/Subscription/Process/Renew/Standard.php

Braces +8 added lines, -3 removed lines

@@ -170,9 +170,12 @@ 
 		{
 			foreach( $codes as $code )
 			{
-				try {
+				try
+				{
 					$basket->addCoupon( $code );
-				} catch( \Aimeos\MShop\Plugin\Provider\Exception $e ) {
+				}
+				catch( \Aimeos\MShop\Plugin\Provider\Exception $e )
+				{
 					$basket->deleteCoupon( $code );
 				}
 			}
@@ -282,7 +285,9 @@ 
 			$context->setUserId( $baseItem->getCustomerId() );
 			$context->setGroupIds( $customerItem->getGroups() );
 		}
-		catch( \Exception $e ) {} // Subscription without account
+		catch( \Exception $e )
+		{
+} // Subscription without account
 
 		return $context;
 	}

controller/jobs/src/Controller/Jobs/Catalog/Export/Sitemap/Factory.php

Spacing +3 added lines, -3 removed lines

@@ -64,13 +64,13 @@
 		 * @since 2019.02
 		 * @category Developer
 		 */
-		if ( $name === null ) {
+		if( $name === null ) {
 			$name = $context->getConfig()->get( 'controller/jobs/catalog/export/sitemap/name', 'Standard' );
 		}
 
-		if ( ctype_alnum($name) === false )
+		if( ctype_alnum( $name ) === false )
 		{
-			$classname = is_string($name) ? '\\Aimeos\\Controller\\Jobs\\Catalog\\Export\\Sitemap\\' . $name : '<not a string>';
+			$classname = is_string( $name ) ? '\\Aimeos\\Controller\\Jobs\\Catalog\\Export\\Sitemap\\' . $name : '<not a string>';
 			throw new \Aimeos\Controller\Jobs\Exception( sprintf( 'Invalid characters in class name "%1$s"', $classname ) );
 		}
 

controller/jobs/src/Controller/Jobs/Catalog/Export/Sitemap/Standard.php

Spacing +5 added lines, -5 removed lines

@@ -196,10 +196,10 @@ 
 		 * @see controller/jobs/catalog/export/sitemap/max-query
 		 * @see controller/jobs/catalog/export/sitemap/changefreq
 		 */
-		$default = array('gzip-mode' => 'wb');
+		$default = array( 'gzip-mode' => 'wb' );
 		$options = $config->get( 'controller/jobs/catalog/export/sitemap/container/options', $default );
 
-		if ( $location === null || $location === '' ) {
+		if( $location === null || $location === '' ) {
 			$msg = sprintf( 'Required configuration for "%1$s" is missing', 'controller/jobs/catalog/export/sitemap/location' );
 			throw new \Aimeos\Controller\Jobs\Exception( $msg );
 		}
@@ -417,7 +417,7 @@ 
 		$manager = \Aimeos\MShop::create( $this->getContext(), 'catalog' );
 
 		$search = $manager->createSearch( $default );
-		$search->setSortations( array($search->sort( '+', 'catalog.id' )) );
+		$search->setSortations( array( $search->sort( '+', 'catalog.id' ) ) );
 		$search->setSlice( 0, $maxQuery );
 
 		$content = $this->createContent( $container, $filenum );
@@ -431,12 +431,12 @@ 
 			$start += $count;
 			$search->setSlice( $start, $maxQuery );
 
-			if ( $start + $maxQuery > $maxItems * $filenum ) {
+			if( $start + $maxQuery > $maxItems * $filenum ) {
 				$this->closeContent( $content );
 				$content = $this->createContent( $container, ++$filenum );
 				$names[] = $content->getResource();
 			}
-		} while ($count >= $search->getSliceSize());
+		} while( $count >= $search->getSliceSize() );
 
 		$this->closeContent( $content );
 

controller/jobs/tests/Controller/Jobs/Catalog/Export/Sitemap/StandardTest.php

Spacing +2 added lines, -2 removed lines

@@ -77,7 +77,7 @@ 
 	{
 		$this->context->getConfig()->set( 'controller/jobs/catalog/export/sitemap/location', '' );
 
-		$this->setExpectedException('\\Aimeos\\Controller\\Jobs\\Exception');
+		$this->setExpectedException( '\\Aimeos\\Controller\\Jobs\\Exception' );
 
 		$this->object->run();
 	}
@@ -86,7 +86,7 @@ 
 	{
 		$this->context->getConfig()->set( 'controller/jobs/catalog/export/sitemap/location', null );
 
-		$this->setExpectedException('\\Aimeos\\Controller\\Jobs\\Exception');
+		$this->setExpectedException( '\\Aimeos\\Controller\\Jobs\\Exception' );
 
 		$this->object->run();
 	}

controller/jobs/tests/Controller/Jobs/Catalog/Export/Sitemap/FactoryTest.php

Spacing +1 added lines, -1 removed lines

@@ -17,7 +17,7 @@
 		$aimeos = \TestHelperJobs::getAimeos();
 
 		$obj = \Aimeos\Controller\Jobs\Catalog\Export\Sitemap\Factory::create( $context, $aimeos );
-		$this->assertInstanceOf( '\\Aimeos\\Controller\\Jobs\\Iface', $obj);
+		$this->assertInstanceOf( '\\Aimeos\\Controller\\Jobs\\Iface', $obj );
 	}