aimeos / ai-controller-jobs

controller/jobs/src/Controller/Jobs/Index/Optimize/Standard.php

Spacing +3 added lines, -3 removed lines

@@ -29,7 +29,7 @@ 
 	 */
 	public function getName()
 	{
-		return $this->getContext()->getI18n()->dt( 'controller/jobs', 'Catalog index optimization' );
+		return $this->getContext()->getI18n()->dt('controller/jobs', 'Catalog index optimization');
 	}
 
 
@@ -40,7 +40,7 @@ 
 	 */
 	public function getDescription()
 	{
-		return $this->getContext()->getI18n()->dt( 'controller/jobs', 'Optimizes the catalog index for searching products' );
+		return $this->getContext()->getI18n()->dt('controller/jobs', 'Optimizes the catalog index for searching products');
 	}
 
 
@@ -51,6 +51,6 @@ 
 	 */
 	public function run()
 	{
-		\Aimeos\MShop\Index\Manager\Factory::createManager( $this->getContext() )->optimize();
+		\Aimeos\MShop\Index\Manager\Factory::createManager($this->getContext())->optimize();
 	}
 }

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

Spacing +8 added lines, -8 removed lines

@@ -30,7 +30,7 @@ 
 	 * @param string|null $name Name of the controller or "Standard" if null
 	 * @return \Aimeos\Controller\Jobs\Iface New controller object
 	 */
-	public static function createController( \Aimeos\MShop\Context\Item\Iface $context, \Aimeos\Bootstrap $aimeos, $name = null )
+	public static function createController(\Aimeos\MShop\Context\Item\Iface $context, \Aimeos\Bootstrap $aimeos, $name = null)
 	{
 		/** controller/jobs/index/rebuild/name
 		 * Class name of the used catalog index rebuild scheduler controller implementation
@@ -65,20 +65,20 @@ 
 		 * @since 2014.03
 		 * @category Developer
 		 */
-		if( $name === null ) {
-			$name = $context->getConfig()->get( 'controller/jobs/index/rebuild/name', 'Standard' );
+		if ($name === null) {
+			$name = $context->getConfig()->get('controller/jobs/index/rebuild/name', 'Standard');
 		}
 
-		if( ctype_alnum( $name ) === false )
+		if (ctype_alnum($name) === false)
 		{
-			$classname = is_string( $name ) ? '\\Aimeos\\Controller\\Jobs\\Index\\Rebuild\\' . $name : '<not a string>';
-			throw new \Aimeos\Controller\Jobs\Exception( sprintf( 'Invalid characters in class name "%1$s"', $classname ) );
+			$classname = is_string($name) ? '\\Aimeos\\Controller\\Jobs\\Index\\Rebuild\\' . $name : '<not a string>';
+			throw new \Aimeos\Controller\Jobs\Exception(sprintf('Invalid characters in class name "%1$s"', $classname));
 		}
 
 		$iface = '\\Aimeos\\Controller\\Jobs\\Iface';
 		$classname = '\\Aimeos\\Controller\\Jobs\\Index\\Rebuild\\' . $name;
 
-		$controller = self::createControllerBase( $context, $aimeos, $classname, $iface );
+		$controller = self::createControllerBase($context, $aimeos, $classname, $iface);
 
 		/** controller/jobs/index/rebuild/decorators/excludes
 		 * Excludes decorators added by the "common" option from the catalog index rebuild controllers
@@ -154,6 +154,6 @@ 
 		 * @see controller/jobs/index/rebuild/decorators/excludes
 		 * @see controller/jobs/index/rebuild/decorators/global
 		 */
-		return self::addControllerDecorators( $context, $aimeos, $controller, 'index/rebuild' );
+		return self::addControllerDecorators($context, $aimeos, $controller, 'index/rebuild');
 	}
 }
\ No newline at end of file

controller/jobs/src/Controller/Jobs/Index/Rebuild/Standard.php

Spacing +7 added lines, -7 removed lines

@@ -29,7 +29,7 @@ 
 	 */
 	public function getName()
 	{
-		return $this->getContext()->getI18n()->dt( 'controller/jobs', 'Catalog index rebuild' );
+		return $this->getContext()->getI18n()->dt('controller/jobs', 'Catalog index rebuild');
 	}
 
 
@@ -40,7 +40,7 @@ 
 	 */
 	public function getDescription()
 	{
-		return $this->getContext()->getI18n()->dt( 'controller/jobs', 'Rebuilds the catalog index for searching products' );
+		return $this->getContext()->getI18n()->dt('controller/jobs', 'Rebuilds the catalog index for searching products');
 	}
 
 
@@ -51,15 +51,15 @@ 
 	 */
 	public function run()
 	{
-		$timestamp = date( 'Y-m-d H:i:s' );
+		$timestamp = date('Y-m-d H:i:s');
 		$context = clone $this->getContext();
 
-		$context->getLocale()->setLanguageId( null );
-		$context->getLocale()->setCurrencyId( null );
+		$context->getLocale()->setLanguageId(null);
+		$context->getLocale()->setCurrencyId(null);
 
-		$manager = \Aimeos\MShop\Index\Manager\Factory::createManager( $context );
+		$manager = \Aimeos\MShop\Index\Manager\Factory::createManager($context);
 
 		$manager->rebuildIndex();
-		$manager->cleanupIndex( $timestamp );
+		$manager->cleanupIndex($timestamp);
 	}
 }

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

Spacing +38 added lines, -38 removed lines

@@ -37,35 +37,35 @@ 
 	 * @param string $path Name of the domain
 	 * @throws \Aimeos\Controller\Jobs\Exception If the given path is invalid or the controllers wasn't found
 	 */
-	static public function createController( \Aimeos\MShop\Context\Item\Iface $context, \Aimeos\Bootstrap $aimeos, $path )
+	static public function createController(\Aimeos\MShop\Context\Item\Iface $context, \Aimeos\Bootstrap $aimeos, $path)
 	{
-		$path = strtolower( trim( $path, "/ \n\t\r\0\x0B" ) );
+		$path = strtolower(trim($path, "/ \n\t\r\0\x0B"));
 
-		if( empty( $path ) ) {
-			throw new \Aimeos\Controller\Jobs\Exception( sprintf( 'Controller path is empty' ) );
+		if (empty($path)) {
+			throw new \Aimeos\Controller\Jobs\Exception(sprintf('Controller path is empty'));
 		}
 
-		$parts = explode( '/', $path );
+		$parts = explode('/', $path);
 
-		foreach( $parts as $key => $part )
+		foreach ($parts as $key => $part)
 		{
-			if( ctype_alnum( $part ) === false ) {
-				throw new \Aimeos\Controller\Jobs\Exception( sprintf( 'Invalid controller "%1$s" in "%2$s"', $part, $path ) );
+			if (ctype_alnum($part) === false) {
+				throw new \Aimeos\Controller\Jobs\Exception(sprintf('Invalid controller "%1$s" in "%2$s"', $part, $path));
 			}
 
-			$parts[$key] = ucwords( $part );
+			$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 ) );
+		if (class_exists($factory) === false) {
+			throw new \Aimeos\Controller\Jobs\Exception(sprintf('Class "%1$s" not found', $factory));
 		}
 
-		$controller = call_user_func_array( array( $factory, 'createController' ), array( $context, $aimeos ) );
+		$controller = call_user_func_array(array($factory, 'createController'), array($context, $aimeos));
 
-		if( $controller === false ) {
-			throw new \Aimeos\Controller\Jobs\Exception( sprintf( 'Invalid factory "%1$s"', $factory ) );
+		if ($controller === false) {
+			throw new \Aimeos\Controller\Jobs\Exception(sprintf('Invalid factory "%1$s"', $factory));
 		}
 
 		return $controller;
@@ -81,32 +81,32 @@ 
 	 * 	relative job controller paths (core and extensions)
 	 * @return array Associative list of controller names as key and the class instance as value
 	 */
-	static public function getControllers( \Aimeos\MShop\Context\Item\Iface $context, \Aimeos\Bootstrap $aimeos, array $cntlPaths )
+	static public function getControllers(\Aimeos\MShop\Context\Item\Iface $context, \Aimeos\Bootstrap $aimeos, array $cntlPaths)
 	{
 		$cntlList = array();
-		$subFolder = trim( str_replace( '\\', DIRECTORY_SEPARATOR, self::$prefix ), '/' );
+		$subFolder = trim(str_replace('\\', DIRECTORY_SEPARATOR, self::$prefix), '/');
 
-		if( strncmp( $subFolder, 'Aimeos' . DIRECTORY_SEPARATOR, 7 ) === 0 ) {
-			$subFolder = substr( $subFolder, 7 );
+		if (strncmp($subFolder, 'Aimeos' . DIRECTORY_SEPARATOR, 7) === 0) {
+			$subFolder = substr($subFolder, 7);
 		}
 
-		foreach( $cntlPaths as $path => $list )
+		foreach ($cntlPaths as $path => $list)
 		{
-			foreach( $list as $relpath )
+			foreach ($list as $relpath)
 			{
 				$path .= DIRECTORY_SEPARATOR . $relpath . DIRECTORY_SEPARATOR . $subFolder;
 
-				if( is_dir( $path ) )
+				if (is_dir($path))
 				{
-					$it = new \DirectoryIterator( $path );
-					$list = self::createControllers( $it, $context, $aimeos );
+					$it = new \DirectoryIterator($path);
+					$list = self::createControllers($it, $context, $aimeos);
 
-					$cntlList = array_merge( $cntlList, $list );
+					$cntlList = array_merge($cntlList, $list);
 				}
 			}
 		}
 
-		ksort( $cntlList );
+		ksort($cntlList);
 
 		return $cntlList;
 	}
@@ -121,26 +121,26 @@ 
 	 * @param string $prefix Part of the class name between "\Aimeos\Controller\Jobs" and "Factory"
 	 * @throws \Aimeos\Controller\Jobs\Exception If factory name is invalid or if the controller couldn't be instantiated
 	 */
-	static protected function createControllers( \DirectoryIterator $dir, \Aimeos\MShop\Context\Item\Iface $context,
-		\Aimeos\Bootstrap $aimeos, $prefix = '' )
+	static protected function createControllers(\DirectoryIterator $dir, \Aimeos\MShop\Context\Item\Iface $context,
+		\Aimeos\Bootstrap $aimeos, $prefix = '')
 	{
 		$list = array();
 
-		foreach( $dir as $entry )
+		foreach ($dir as $entry)
 		{
-			if( $entry->getType() === 'dir' && $entry->isDot() === false )
+			if ($entry->getType() === 'dir' && $entry->isDot() === false)
 			{
-				$name = strtolower( $entry->getBaseName() );
-				$it = new \DirectoryIterator( $entry->getPathName() );
-				$pref = ( $prefix !== '' ? $prefix . '/' : '' ) . $name;
-				$subList = self::createControllers( $it, $context, $aimeos, $pref );
+				$name = strtolower($entry->getBaseName());
+				$it = new \DirectoryIterator($entry->getPathName());
+				$pref = ($prefix !== '' ? $prefix . '/' : '') . $name;
+				$subList = self::createControllers($it, $context, $aimeos, $pref);
 
-				$list = array_merge( $list, $subList );
+				$list = array_merge($list, $subList);
 			}
-			else if( $prefix !== '' && $entry->getType() === 'file'
-				&& ( $name = $entry->getBaseName( '.php' ) ) === 'Factory' )
+			else if ($prefix !== '' && $entry->getType() === 'file'
+				&& ($name = $entry->getBaseName('.php')) === 'Factory')
 			{
-				$list[$prefix] = self::createController( $context, $aimeos, $prefix );
+				$list[$prefix] = self::createController($context, $aimeos, $prefix);
 			}
 		}
 

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 );

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

Spacing +6 added lines, -6 removed lines

@@ -29,7 +29,7 @@ 
 	 * @param string|null $name Name of the controller or "Standard" if null
 	 * @return \Aimeos\Controller\Jobs\Iface New controller object
 	 */
-	public static function createController( \Aimeos\MShop\Context\Item\Iface $context, \Aimeos\Bootstrap $aimeos, $name = null )
+	public static function createController(\Aimeos\MShop\Context\Item\Iface $context, \Aimeos\Bootstrap $aimeos, $name = null)
 	{
 		/** controller/jobs/product/export/name
 		 * Class name of the used product suggestions scheduler controller implementation
@@ -64,20 +64,20 @@ 
 		 * @since 2015.01
 		 * @category Developer
 		 */
-		if ( $name === null ) {
+		if ($name === null) {
 			$name = $context->getConfig()->get('controller/jobs/product/export/name', 'Standard');
 		}
 
-		if ( ctype_alnum($name) === false )
+		if (ctype_alnum($name) === false)
 		{
 			$classname = is_string($name) ? '\\Aimeos\\Controller\\Jobs\\Product\\Export\\' . $name : '<not a string>';
-			throw new \Aimeos\Controller\Jobs\Exception( sprintf( 'Invalid characters in class name "%1$s"', $classname ) );
+			throw new \Aimeos\Controller\Jobs\Exception(sprintf('Invalid characters in class name "%1$s"', $classname));
 		}
 
 		$iface = '\\Aimeos\\Controller\\Jobs\\Iface';
 		$classname = '\\Aimeos\\Controller\\Jobs\\Product\\Export\\' . $name;
 
-		$controller = self::createControllerBase( $context, $aimeos, $classname, $iface );
+		$controller = self::createControllerBase($context, $aimeos, $classname, $iface);
 
 		/** controller/jobs/product/export/decorators/excludes
 		 * Excludes decorators added by the "common" option from the product export job controller
@@ -154,6 +154,6 @@ 
 		 * @see controller/jobs/product/export/decorators/excludes
 		 * @see controller/jobs/product/export/decorators/global
 		 */
-		return self::addControllerDecorators( $context, $aimeos, $controller, 'product/export' );
+		return self::addControllerDecorators($context, $aimeos, $controller, 'product/export');
 	}
 }
\ No newline at end of file

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

Spacing +6 added lines, -6 removed lines

@@ -29,7 +29,7 @@ 
 	 * @param string|null $name Name of the controller or "Standard" if null
 	 * @return \Aimeos\Controller\Jobs\Iface New controller object
 	 */
-	public static function createController( \Aimeos\MShop\Context\Item\Iface $context, \Aimeos\Bootstrap $aimeos, $name = null )
+	public static function createController(\Aimeos\MShop\Context\Item\Iface $context, \Aimeos\Bootstrap $aimeos, $name = null)
 	{
 		/** controller/jobs/product/export/sitemap/name
 		 * Class name of the used product suggestions scheduler controller implementation
@@ -64,20 +64,20 @@ 
 		 * @since 2015.01
 		 * @category Developer
 		 */
-		if ( $name === null ) {
+		if ($name === null) {
 			$name = $context->getConfig()->get('controller/jobs/product/export/sitemap/name', 'Standard');
 		}
 
-		if ( ctype_alnum($name) === false )
+		if (ctype_alnum($name) === false)
 		{
 			$classname = is_string($name) ? '\\Aimeos\\Controller\\Jobs\\Product\\Export\\Sitemap\\' . $name : '<not a string>';
-			throw new \Aimeos\Controller\Jobs\Exception( sprintf( 'Invalid characters in class name "%1$s"', $classname ) );
+			throw new \Aimeos\Controller\Jobs\Exception(sprintf('Invalid characters in class name "%1$s"', $classname));
 		}
 
 		$iface = '\\Aimeos\\Controller\\Jobs\\Iface';
 		$classname = '\\Aimeos\\Controller\\Jobs\\Product\\Export\\Sitemap\\' . $name;
 
-		$controller = self::createControllerBase( $context, $aimeos, $classname, $iface );
+		$controller = self::createControllerBase($context, $aimeos, $classname, $iface);
 
 		/** controller/jobs/product/export/sitemap/decorators/excludes
 		 * Excludes decorators added by the "common" option from the product export sitemap job controller
@@ -154,6 +154,6 @@ 
 		 * @see controller/jobs/product/export/sitemap/export/sitemap/decorators/excludes
 		 * @see controller/jobs/product/export/sitemap/export/sitemap/decorators/global
 		 */
-		return self::addControllerDecorators( $context, $aimeos, $controller, 'product/export/sitemap' );
+		return self::addControllerDecorators($context, $aimeos, $controller, 'product/export/sitemap');
 	}
 }
\ No newline at end of file

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

Spacing +26 added lines, -26 removed lines

@@ -28,7 +28,7 @@ 
 	 */
 	public function getName()
 	{
-		return $this->getContext()->getI18n()->dt( 'controller/jobs', 'Product site map' );
+		return $this->getContext()->getI18n()->dt('controller/jobs', 'Product site map');
 	}
 
 
@@ -39,7 +39,7 @@ 
 	 */
 	public function getDescription()
 	{
-		return $this->getContext()->getI18n()->dt( 'controller/jobs', 'Creates a product site map for search engines' );
+		return $this->getContext()->getI18n()->dt('controller/jobs', 'Creates a product site map for search engines');
 	}
 
 
@@ -52,8 +52,8 @@ 
 	{
 		$container = $this->createContainer();
 
-		$files = $this->export( $container );
-		$this->createSitemapIndex( $container, $files );
+		$files = $this->export($container);
+		$this->createSitemapIndex($container, $files);
 
 		$container->close();
 	}
@@ -65,7 +65,7 @@ 
 	 * @param \Aimeos\MW\Container\Content\Iface $content File content object
 	 * @param \Aimeos\MShop\Product\Item\Iface[] $items List of product items
 	 */
-	protected function addItems( \Aimeos\MW\Container\Content\Iface $content, array $items )
+	protected function addItems(\Aimeos\MW\Container\Content\Iface $content, array $items)
 	{
 		$config = $this->getContext()->getConfig();
 
@@ -96,7 +96,7 @@ 
 		 * @see controller/jobs/product/export/sitemap/max-items
 		 * @see controller/jobs/product/export/sitemap/max-query
 		 */
-		$changefreq = $config->get( 'controller/jobs/product/export/sitemap/changefreq', 'daily' );
+		$changefreq = $config->get('controller/jobs/product/export/sitemap/changefreq', 'daily');
 
 		/** controller/jobs/product/export/sitemap/standard/template-items
 		 * Relative path to the XML items template of the product site map job controller.
@@ -128,7 +128,7 @@ 
 		$view->siteItems = $items;
 		$view->siteFreq = $changefreq;
 
-		$content->add( $view->render( $view->config( $tplconf, $default ) ) );
+		$content->add($view->render($view->config($tplconf, $default)));
 	}
 
 
@@ -174,7 +174,7 @@ 
 		 * @see controller/jobs/product/export/sitemap/max-query
 		 * @see controller/jobs/product/export/sitemap/changefreq
 		 */
-		$location = $config->get( 'controller/jobs/product/export/sitemap/location', sys_get_temp_dir() );
+		$location = $config->get('controller/jobs/product/export/sitemap/location', sys_get_temp_dir());
 
 		/** controller/jobs/product/export/sitemap/container/options
 		 * List of file container options for the site map files
@@ -194,9 +194,9 @@ 
 		 * @see controller/jobs/product/export/sitemap/max-query
 		 * @see controller/jobs/product/export/sitemap/changefreq
 		*/
-		$options = $config->get( 'controller/jobs/product/export/sitemap/container/options', array() );
+		$options = $config->get('controller/jobs/product/export/sitemap/container/options', array());
 
-		return \Aimeos\MW\Container\Factory::getContainer( $location, 'Directory', 'Gzip', $options );
+		return \Aimeos\MW\Container\Factory::getContainer($location, 'Directory', 'Gzip', $options);
 	}
 
 
@@ -207,7 +207,7 @@ 
 	 * @param integer $filenum New file number
 	 * @return \Aimeos\MW\Container\Content\Iface New content object
 	 */
-	protected function createContent( \Aimeos\MW\Container\Iface $container, $filenum )
+	protected function createContent(\Aimeos\MW\Container\Iface $container, $filenum)
 	{
 		$config = $this->getContext()->getConfig();
 
@@ -238,9 +238,9 @@ 
 
 		$view = $this->getContext()->getView();
 
-		$content = $container->create( $this->getFilename( $filenum ) );
-		$content->add( $view->render( $view->config( $tplconf, $default ) ) );
-		$container->add( $content );
+		$content = $container->create($this->getFilename($filenum));
+		$content->add($view->render($view->config($tplconf, $default)));
+		$container->add($content);
 
 		return $content;
 	}
@@ -251,7 +251,7 @@ 
 	 *
 	 * @param \Aimeos\MW\Container\Content\Iface $content
 	 */
-	protected function closeContent( \Aimeos\MW\Container\Content\Iface $content )
+	protected function closeContent(\Aimeos\MW\Container\Content\Iface $content)
 	{
 		$config = $this->getContext()->getConfig();
 
@@ -282,7 +282,7 @@ 
 
 		$view = $this->getContext()->getView();
 
-		$content->add( $view->render( $view->config( $tplconf, $default ) ) );
+		$content->add($view->render($view->config($tplconf, $default)));
 	}
 
 
@@ -292,7 +292,7 @@ 
 	 * @param \Aimeos\MW\Container\Iface $container File container object
 	 * @param array $files List of generated site map file names
 	 */
-	protected function createSitemapIndex( \Aimeos\MW\Container\Iface $container, array $files )
+	protected function createSitemapIndex(\Aimeos\MW\Container\Iface $container, array $files)
 	{
 		$config = $this->getContext()->getConfig();
 
@@ -324,9 +324,9 @@ 
 		$view = $this->getContext()->getView();
 		$view->siteFiles = $files;
 
-		$content = $container->create( 'aimeos-sitemap-index.xml' );
-		$content->add( $view->render( $view->config( $tplconf, $default ) ) );
-		$container->add( $content );
+		$content = $container->create('aimeos-sitemap-index.xml');
+		$content->add($view->render($view->config($tplconf, $default)));
+		$container->add($content);
 	}
 
 
@@ -337,11 +337,11 @@ 
 	 * @param mixed $default Default value if name is unknown
 	 * @return mixed Configuration value
 	 */
-	protected function getConfig( $name, $default = null )
+	protected function getConfig($name, $default = null)
 	{
 		$config = $this->getContext()->getConfig();
 
-		switch( $name )
+		switch ($name)
 		{
 			case 'domain':
 				return array();
@@ -368,7 +368,7 @@ 
 				 * @see controller/jobs/product/export/sitemap/max-query
 				 * @see controller/jobs/product/export/sitemap/changefreq
 				 */
-				return $config->get( 'controller/jobs/product/export/sitemap/max-items', 50000 );
+				return $config->get('controller/jobs/product/export/sitemap/max-items', 50000);
 
 			case 'max-query':
 				/** controller/jobs/product/export/sitemap/max-query
@@ -388,7 +388,7 @@ 
 				 * @see controller/jobs/product/export/sitemap/max-items
 				 * @see controller/jobs/product/export/sitemap/changefreq
 				 */
-				return $config->get( 'controller/jobs/product/export/sitemap/max-query', 1000 );
+				return $config->get('controller/jobs/product/export/sitemap/max-query', 1000);
 		}
 
 		return $default;
@@ -401,8 +401,8 @@ 
 	 * @param integer $number Current file number
 	 * @return string New file name
 	 */
-	protected function getFilename( $number )
+	protected function getFilename($number)
 	{
-		return sprintf( 'aimeos-sitemap-%d.xml', $number );
+		return sprintf('aimeos-sitemap-%d.xml', $number);
 	}
 }

controller/jobs/src/Controller/Jobs/Product/Export/Standard.php

Spacing +42 added lines, -42 removed lines

@@ -28,7 +28,7 @@ 
 	 */
 	public function getName()
 	{
-		return $this->getContext()->getI18n()->dt( 'controller/jobs', 'Product export' );
+		return $this->getContext()->getI18n()->dt('controller/jobs', 'Product export');
 	}
 
 
@@ -39,7 +39,7 @@ 
 	 */
 	public function getDescription()
 	{
-		return $this->getContext()->getI18n()->dt( 'controller/jobs', 'Exports all available products' );
+		return $this->getContext()->getI18n()->dt('controller/jobs', 'Exports all available products');
 	}
 
 
@@ -51,7 +51,7 @@ 
 	public function run()
 	{
 		$container = $this->createContainer();
-		$this->export( $container );
+		$this->export($container);
 		$container->close();
 	}
 
@@ -62,7 +62,7 @@ 
 	 * @param \Aimeos\MW\Container\Content\Iface $content File content object
 	 * @param \Aimeos\MShop\Product\Item\Iface[] $items List of product items
 	 */
-	protected function addItems( \Aimeos\MW\Container\Content\Iface $content, array $items )
+	protected function addItems(\Aimeos\MW\Container\Content\Iface $content, array $items)
 	{
 		$config = $this->getContext()->getConfig();
 
@@ -95,7 +95,7 @@ 
 
 		$view->exportItems = $items;
 
-		$content->add( $view->render( $view->config( $tplconf, $default ) ) );
+		$content->add($view->render($view->config($tplconf, $default)));
 	}
 
 
@@ -124,7 +124,7 @@ 
 		 * @see controller/jobs/product/export/max-items
 		 * @see controller/jobs/product/export/max-query
 		 */
-		$location = $config->get( 'controller/jobs/product/export/location', sys_get_temp_dir() );
+		$location = $config->get('controller/jobs/product/export/location', sys_get_temp_dir());
 
 		/** controller/jobs/product/export/standard/container/type
 		 * List of file container options for the export files
@@ -141,7 +141,7 @@ 
 		 * @see controller/jobs/product/export/max-items
 		 * @see controller/jobs/product/export/max-query
 		*/
-		$container = $config->get( 'controller/jobs/product/export/standard/container/type', 'Directory' );
+		$container = $config->get('controller/jobs/product/export/standard/container/type', 'Directory');
 
 		/** controller/jobs/product/export/standard/container/content
 		 * List of file container options for the export files
@@ -158,7 +158,7 @@ 
 		 * @see controller/jobs/product/export/max-items
 		 * @see controller/jobs/product/export/max-query
 		 */
-		$content = $config->get( 'controller/jobs/product/export/standard/container/content', 'Binary' );
+		$content = $config->get('controller/jobs/product/export/standard/container/content', 'Binary');
 
 		/** controller/jobs/product/export/standard/container/options
 		 * List of file container options for the export files
@@ -175,9 +175,9 @@ 
 		 * @see controller/jobs/product/export/max-items
 		 * @see controller/jobs/product/export/max-query
 		 */
-		$options = $config->get( 'controller/jobs/product/export/standard/container/options', array() );
+		$options = $config->get('controller/jobs/product/export/standard/container/options', array());
 
-		return \Aimeos\MW\Container\Factory::getContainer( $location, $container, $content, $options );
+		return \Aimeos\MW\Container\Factory::getContainer($location, $container, $content, $options);
 	}
 
 
@@ -188,7 +188,7 @@ 
 	 * @param integer $filenum New file number
 	 * @return \Aimeos\MW\Container\Content\Iface New content object
 	 */
-	protected function createContent( \Aimeos\MW\Container\Iface $container, $filenum )
+	protected function createContent(\Aimeos\MW\Container\Iface $container, $filenum)
 	{
 		$config = $this->getContext()->getConfig();
 
@@ -219,9 +219,9 @@ 
 
 		$view = $this->getContext()->getView();
 
-		$content = $container->create( $this->getFilename( $filenum ) );
-		$content->add( $view->render( $view->config( $tplconf, $default ) ) );
-		$container->add( $content );
+		$content = $container->create($this->getFilename($filenum));
+		$content->add($view->render($view->config($tplconf, $default)));
+		$container->add($content);
 
 		return $content;
 	}
@@ -232,7 +232,7 @@ 
 	 *
 	 * @param \Aimeos\MW\Container\Content\Iface $content
 	 */
-	protected function closeContent( \Aimeos\MW\Container\Content\Iface $content )
+	protected function closeContent(\Aimeos\MW\Container\Content\Iface $content)
 	{
 		$config = $this->getContext()->getConfig();
 
@@ -263,7 +263,7 @@ 
 
 		$view = $this->getContext()->getView();
 
-		$content->add( $view->render( $view->config( $tplconf, $default ) ) );
+		$content->add($view->render($view->config($tplconf, $default)));
 	}
 
 
@@ -273,45 +273,45 @@ 
 	 * @param \Aimeos\MW\Container\Iface $container Container object
 	 * @return array List of content (file) names
 	 */
-	protected function export( \Aimeos\MW\Container\Iface $container )
+	protected function export(\Aimeos\MW\Container\Iface $container)
 	{
-		$default = array( 'attribute', 'media', 'price', 'product', 'text' );
+		$default = array('attribute', 'media', 'price', 'product', 'text');
 
-		$domains = $this->getConfig( 'domains', $default );
-		$maxItems = $this->getConfig( 'max-items', 10000 );
-		$maxQuery = $this->getConfig( 'max-query', 1000 );
+		$domains = $this->getConfig('domains', $default);
+		$maxItems = $this->getConfig('max-items', 10000);
+		$maxQuery = $this->getConfig('max-query', 1000);
 
 		$start = 0; $filenum = 1;
 		$names = array();
 
-		$productManager = \Aimeos\MShop\Factory::createManager( $this->getContext(), 'product' );
+		$productManager = \Aimeos\MShop\Factory::createManager($this->getContext(), 'product');
 
-		$search = $productManager->createSearch( true );
-		$search->setSortations( array( $search->sort( '+', 'product.id' ) ) );
-		$search->setSlice( 0, $maxQuery );
+		$search = $productManager->createSearch(true);
+		$search->setSortations(array($search->sort('+', 'product.id')));
+		$search->setSlice(0, $maxQuery);
 
-		$content = $this->createContent( $container, $filenum );
+		$content = $this->createContent($container, $filenum);
 		$names[] = $content->getResource();
 
 		do
 		{
-			$items = $productManager->searchItems( $search, $domains );
-			$this->addItems( $content, $items );
+			$items = $productManager->searchItems($search, $domains);
+			$this->addItems($content, $items);
 
-			$count = count( $items );
+			$count = count($items);
 			$start += $count;
-			$search->setSlice( $start, $maxQuery );
+			$search->setSlice($start, $maxQuery);
 
-			if( $start + $maxQuery > $maxItems * $filenum )
+			if ($start + $maxQuery > $maxItems * $filenum)
 			{
-				$this->closeContent( $content );
-				$content = $this->createContent( $container, ++$filenum );
+				$this->closeContent($content);
+				$content = $this->createContent($container, ++$filenum);
 				$names[] = $content->getResource();
 			}
 		}
-		while( $count >= $search->getSliceSize() );
+		while ($count >= $search->getSliceSize());
 
-		$this->closeContent( $content );
+		$this->closeContent($content);
 
 		return $names;
 	}
@@ -324,11 +324,11 @@ 
 	 * @param mixed $default Default value if name is unknown
 	 * @return mixed Configuration value
 	 */
-	protected function getConfig( $name, $default = null )
+	protected function getConfig($name, $default = null)
 	{
 		$config = $this->getContext()->getConfig();
 
-		switch( $name )
+		switch ($name)
 		{
 			case 'domain':
 				/** controller/jobs/product/export/domains
@@ -350,7 +350,7 @@ 
 				 * @see controller/jobs/product/export/max-items
 				 * @see controller/jobs/product/export/max-query
 				 */
-				return $config->get( 'controller/jobs/product/export/domains', $default );
+				return $config->get('controller/jobs/product/export/domains', $default);
 
 			case 'max-items':
 				/** controller/jobs/product/export/max-items
@@ -372,7 +372,7 @@ 
 				 * @see controller/jobs/product/export/max-query
 				 * @see controller/jobs/product/export/domains
 				 */
-				return $config->get( 'controller/jobs/product/export/max-items', $default );
+				return $config->get('controller/jobs/product/export/max-items', $default);
 
 			case 'max-query':
 				/** controller/jobs/product/export/max-query
@@ -394,7 +394,7 @@ 
 				 * @see controller/jobs/product/export/max-items
 				 * @see controller/jobs/product/export/domains
 				 */
-				return $config->get( 'controller/jobs/product/export/max-query', $default );
+				return $config->get('controller/jobs/product/export/max-query', $default);
 		}
 
 		return $default;
@@ -407,8 +407,8 @@ 
 	 * @param integer $number Current file number
 	 * @return string New file name
 	 */
-	protected function getFilename( $number )
+	protected function getFilename($number)
 	{
-		return sprintf( 'aimeos-products-%d.xml', $number );
+		return sprintf('aimeos-products-%d.xml', $number);
 	}
 }

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

Spacing +8 added lines, -8 removed lines

@@ -30,7 +30,7 @@ 
 	 * @param string|null $name Name of the controller or "Standard" if null
 	 * @return \Aimeos\Controller\Jobs\Iface New controller object
 	 */
-	public static function createController( \Aimeos\MShop\Context\Item\Iface $context, \Aimeos\Bootstrap $aimeos, $name = null )
+	public static function createController(\Aimeos\MShop\Context\Item\Iface $context, \Aimeos\Bootstrap $aimeos, $name = null)
 	{
 		/** controller/jobs/product/bought/name
 		 * Class name of the used product suggestions scheduler controller implementation
@@ -65,20 +65,20 @@ 
 		 * @since 2014.03
 		 * @category Developer
 		 */
-		if( $name === null ) {
-			$name = $context->getConfig()->get( 'controller/jobs/product/bought/name', 'Standard' );
+		if ($name === null) {
+			$name = $context->getConfig()->get('controller/jobs/product/bought/name', 'Standard');
 		}
 
-		if( ctype_alnum( $name ) === false )
+		if (ctype_alnum($name) === false)
 		{
-			$classname = is_string( $name ) ? '\\Aimeos\\Controller\\Jobs\\Product\\Bought\\' . $name : '<not a string>';
-			throw new \Aimeos\Controller\Jobs\Exception( sprintf( 'Invalid characters in class name "%1$s"', $classname ) );
+			$classname = is_string($name) ? '\\Aimeos\\Controller\\Jobs\\Product\\Bought\\' . $name : '<not a string>';
+			throw new \Aimeos\Controller\Jobs\Exception(sprintf('Invalid characters in class name "%1$s"', $classname));
 		}
 
 		$iface = '\\Aimeos\\Controller\\Jobs\\Iface';
 		$classname = '\\Aimeos\\Controller\\Jobs\\Product\\Bought\\' . $name;
 
-		$controller = self::createControllerBase( $context, $aimeos, $classname, $iface );
+		$controller = self::createControllerBase($context, $aimeos, $classname, $iface);
 
 		/** controller/jobs/product/bought/decorators/excludes
 		 * Excludes decorators added by the "common" option from the product bought job controller
@@ -155,6 +155,6 @@ 
 		 * @see controller/jobs/product/bought/decorators/excludes
 		 * @see controller/jobs/product/bought/decorators/global
 		 */
-		return self::addControllerDecorators( $context, $aimeos, $controller, 'product/bought' );
+		return self::addControllerDecorators($context, $aimeos, $controller, 'product/bought');
 	}
 }
\ No newline at end of file