aimeos / ai-controller-jobs

controller/common/tests/TestHelperCntl.php

Spacing +23 added lines, -23 removed lines

@@ -15,14 +15,14 @@ 
 	public static function bootstrap()
 	{
 		self::getAimeos();
-		\Aimeos\MShop\Factory::setCache( false );
+		\Aimeos\MShop\Factory::setCache(false);
 	}
 
 
-	public static function getContext( $site = 'unittest' )
+	public static function getContext($site = 'unittest')
 	{
-		if( !isset( self::$context[$site] ) ) {
-			self::$context[$site] = self::createContext( $site );
+		if (!isset(self::$context[$site])) {
+			self::$context[$site] = self::createContext($site);
 		}
 
 		return clone self::$context[$site];
@@ -31,13 +31,13 @@ 
 
 	public static function getAimeos()
 	{
-		if( !isset( self::$aimeos ) )
+		if (!isset(self::$aimeos))
 		{
 			require_once 'Bootstrap.php';
-			spl_autoload_register( 'Aimeos\\Bootstrap::autoload' );
+			spl_autoload_register('Aimeos\\Bootstrap::autoload');
 
-			$extdir = dirname( dirname( dirname( dirname( dirname( __FILE__ ) ) ) ) );
-			self::$aimeos = new \Aimeos\Bootstrap( array( $extdir ), true );
+			$extdir = dirname(dirname(dirname(dirname(dirname(__FILE__)))));
+			self::$aimeos = new \Aimeos\Bootstrap(array($extdir), true);
 		}
 
 		return self::$aimeos;
@@ -47,40 +47,40 @@ 
 	/**
 	 * @param string $site
 	 */
-	private static function createContext( $site )
+	private static function createContext($site)
 	{
 		$ctx = new \Aimeos\MShop\Context\Item\Standard();
 		$aimeos = self::getAimeos();
 
 
-		$paths = $aimeos->getConfigPaths( 'mysql' );
+		$paths = $aimeos->getConfigPaths('mysql');
 		$paths[] = __DIR__ . DIRECTORY_SEPARATOR . 'config';
 		$file = __DIR__ . DIRECTORY_SEPARATOR . 'confdoc.ser';
 
-		$conf = new \Aimeos\MW\Config\PHPArray( array(), $paths );
-		$conf = new \Aimeos\MW\Config\Decorator\Memory( $conf );
-		$conf = new \Aimeos\MW\Config\Decorator\Documentor( $conf, $file );
-		$ctx->setConfig( $conf );
+		$conf = new \Aimeos\MW\Config\PHPArray(array(), $paths);
+		$conf = new \Aimeos\MW\Config\Decorator\Memory($conf);
+		$conf = new \Aimeos\MW\Config\Decorator\Documentor($conf, $file);
+		$ctx->setConfig($conf);
 
 
-		$dbm = new \Aimeos\MW\DB\Manager\PDO( $conf );
-		$ctx->setDatabaseManager( $dbm );
+		$dbm = new \Aimeos\MW\DB\Manager\PDO($conf);
+		$ctx->setDatabaseManager($dbm);
 
 
-		$logger = new \Aimeos\MW\Logger\File( 'unittest.log', \Aimeos\MW\Logger\Base::DEBUG );
-		$ctx->setLogger( $logger );
+		$logger = new \Aimeos\MW\Logger\File('unittest.log', \Aimeos\MW\Logger\Base::DEBUG);
+		$ctx->setLogger($logger);
 
 
 		$session = new \Aimeos\MW\Session\None();
-		$ctx->setSession( $session );
+		$ctx->setSession($session);
 
 
-		$localeManager = \Aimeos\MShop\Locale\Manager\Factory::createManager( $ctx );
-		$locale = $localeManager->bootstrap( $site, '', '', false );
-		$ctx->setLocale( $locale );
+		$localeManager = \Aimeos\MShop\Locale\Manager\Factory::createManager($ctx);
+		$locale = $localeManager->bootstrap($site, '', '', false);
+		$ctx->setLocale($locale);
 
 
-		$ctx->setEditor( 'core:controller/common' );
+		$ctx->setEditor('core:controller/common');
 
 		return $ctx;
 	}

controller/common/tests/Controller/Common/Product/Import/Csv/Processor/DoneTest.php

Spacing +6 added lines, -6 removed lines

@@ -21,10 +21,10 @@ 
 	 */
 	protected function setUp()
 	{
-		\Aimeos\MShop\Factory::setCache( true );
+		\Aimeos\MShop\Factory::setCache(true);
 
 		$this->context = \TestHelperCntl::getContext();
-		$this->object = new \Aimeos\Controller\Common\Product\Import\Csv\Processor\Done( $this->context, array() );
+		$this->object = new \Aimeos\Controller\Common\Product\Import\Csv\Processor\Done($this->context, array());
 	}
 
 
@@ -36,7 +36,7 @@ 
 	 */
 	protected function tearDown()
 	{
-		\Aimeos\MShop\Factory::setCache( false );
+		\Aimeos\MShop\Factory::setCache(false);
 		\Aimeos\MShop\Factory::clear();
 
 		$this->object = null;
@@ -45,10 +45,10 @@ 
 
 	public function testProcess()
 	{
-		$product = \Aimeos\MShop\Factory::createManager( $this->context, 'product' )->createItem();
+		$product = \Aimeos\MShop\Factory::createManager($this->context, 'product')->createItem();
 
-		$result = $this->object->process( $product, array( 'test' ) );
+		$result = $this->object->process($product, array('test'));
 
-		$this->assertEquals( array( 'test' ), $result );
+		$this->assertEquals(array('test'), $result);
 	}
 }
\ No newline at end of file

controller/common/tests/Controller/Common/Product/Import/Csv/Cache/Attribute/StandardTest.php

Spacing +15 added lines, -15 removed lines

@@ -14,46 +14,46 @@
 
 	protected function setUp()
 	{
-		\Aimeos\MShop\Factory::setCache( true );
+		\Aimeos\MShop\Factory::setCache(true);
 
 		$context = \TestHelperCntl::getContext();
-		$this->object = new \Aimeos\Controller\Common\Product\Import\Csv\Cache\Attribute\Standard( $context );
+		$this->object = new \Aimeos\Controller\Common\Product\Import\Csv\Cache\Attribute\Standard($context);
 	}
 
 
 	protected function tearDown()
 	{
-		\Aimeos\MShop\Factory::setCache( false );
+		\Aimeos\MShop\Factory::setCache(false);
 		\Aimeos\MShop\Factory::clear();
 	}
 
 
 	public function testGet()
 	{
-		$item = $this->object->get( 'black', 'color' );
+		$item = $this->object->get('black', 'color');
 
-		$this->assertInstanceOf( '\\Aimeos\\MShop\\Attribute\\Item\\Iface', $item );
-		$this->assertEquals( 'black', $item->getCode() );
-		$this->assertEquals( 'color', $item->getType() );
+		$this->assertInstanceOf('\\Aimeos\\MShop\\Attribute\\Item\\Iface', $item);
+		$this->assertEquals('black', $item->getCode());
+		$this->assertEquals('color', $item->getType());
 	}
 
 
 	public function testGetUnknown()
 	{
-		$this->assertEquals( null, $this->object->get( 'cache-test', 'color' ) );
+		$this->assertEquals(null, $this->object->get('cache-test', 'color'));
 	}
 
 
 	public function testSet()
 	{
-		$item = $this->object->get( 'black', 'color' );
-		$item->setCode( 'cache-test' );
+		$item = $this->object->get('black', 'color');
+		$item->setCode('cache-test');
 
-		$this->object->set( $item );
-		$item = $this->object->get( 'cache-test', 'color' );
+		$this->object->set($item);
+		$item = $this->object->get('cache-test', 'color');
 
-		$this->assertInstanceOf( '\\Aimeos\\MShop\\Attribute\\Item\\Iface', $item );
-		$this->assertEquals( 'cache-test', $item->getCode() );
-		$this->assertEquals( 'color', $item->getType() );
+		$this->assertInstanceOf('\\Aimeos\\MShop\\Attribute\\Item\\Iface', $item);
+		$this->assertEquals('cache-test', $item->getCode());
+		$this->assertEquals('color', $item->getType());
 	}
 }
\ No newline at end of file

controller/common/tests/Controller/Common/Product/Import/Csv/Cache/Catalog/StandardTest.php

Spacing +13 added lines, -13 removed lines

@@ -14,44 +14,44 @@
 
 	protected function setUp()
 	{
-		\Aimeos\MShop\Factory::setCache( true );
+		\Aimeos\MShop\Factory::setCache(true);
 
 		$context = \TestHelperCntl::getContext();
-		$this->object = new \Aimeos\Controller\Common\Product\Import\Csv\Cache\Catalog\Standard( $context );
+		$this->object = new \Aimeos\Controller\Common\Product\Import\Csv\Cache\Catalog\Standard($context);
 	}
 
 
 	protected function tearDown()
 	{
-		\Aimeos\MShop\Factory::setCache( false );
+		\Aimeos\MShop\Factory::setCache(false);
 		\Aimeos\MShop\Factory::clear();
 	}
 
 
 	public function testGet()
 	{
-		$result = $this->object->get( 'root' );
+		$result = $this->object->get('root');
 
-		$this->assertNotEquals( null, $result );
-		$this->assertInternalType( 'integer', $result );
+		$this->assertNotEquals(null, $result);
+		$this->assertInternalType('integer', $result);
 	}
 
 
 	public function testGetUnknown()
 	{
-		$this->assertEquals( null, $this->object->get( 'cache-test' ) );
+		$this->assertEquals(null, $this->object->get('cache-test'));
 	}
 
 
 	public function testSet()
 	{
-		$item = \Aimeos\MShop\Factory::createManager( \TestHelperCntl::getContext(), 'catalog' )->createItem();
-		$item->setCode( 'cache-test' );
-		$item->setId( 1 );
+		$item = \Aimeos\MShop\Factory::createManager(\TestHelperCntl::getContext(), 'catalog')->createItem();
+		$item->setCode('cache-test');
+		$item->setId(1);
 
-		$this->object->set( $item );
-		$id = $this->object->get( 'cache-test' );
+		$this->object->set($item);
+		$id = $this->object->get('cache-test');
 
-		$this->assertEquals( $item->getId(), $id );
+		$this->assertEquals($item->getId(), $id);
 	}
 }
\ No newline at end of file

controller/common/tests/Controller/Common/Product/Import/Csv/Cache/Warehouse/StandardTest.php

Spacing +12 added lines, -12 removed lines

@@ -14,43 +14,43 @@
 
 	protected function setUp()
 	{
-		\Aimeos\MShop\Factory::setCache( true );
+		\Aimeos\MShop\Factory::setCache(true);
 
 		$context = \TestHelperCntl::getContext();
-		$this->object = new \Aimeos\Controller\Common\Product\Import\Csv\Cache\Warehouse\Standard( $context );
+		$this->object = new \Aimeos\Controller\Common\Product\Import\Csv\Cache\Warehouse\Standard($context);
 	}
 
 
 	protected function tearDown()
 	{
-		\Aimeos\MShop\Factory::setCache( false );
+		\Aimeos\MShop\Factory::setCache(false);
 		\Aimeos\MShop\Factory::clear();
 	}
 
 
 	public function testGet()
 	{
-		$result = $this->object->get( 'default' );
+		$result = $this->object->get('default');
 
-		$this->assertNotEquals( null, $result );
+		$this->assertNotEquals(null, $result);
 	}
 
 
 	public function testGetUnknown()
 	{
-		$this->assertEquals( null, $this->object->get( 'cache-test' ) );
+		$this->assertEquals(null, $this->object->get('cache-test'));
 	}
 
 
 	public function testSet()
 	{
-		$item = \Aimeos\MShop\Factory::createManager( \TestHelperCntl::getContext(), 'product/stock/warehouse' )->createItem();
-		$item->setCode( 'cache-test' );
-		$item->setId( 1 );
+		$item = \Aimeos\MShop\Factory::createManager(\TestHelperCntl::getContext(), 'product/stock/warehouse')->createItem();
+		$item->setCode('cache-test');
+		$item->setId(1);
 
-		$this->object->set( $item );
-		$id = $this->object->get( 'cache-test' );
+		$this->object->set($item);
+		$id = $this->object->get('cache-test');
 
-		$this->assertEquals( $item->getId(), $id );
+		$this->assertEquals($item->getId(), $id);
 	}
 }
\ No newline at end of file

controller/common/tests/Controller/Common/Product/Import/Csv/Cache/Product/StandardTest.php

Spacing +11 added lines, -11 removed lines

@@ -14,41 +14,41 @@
 
 	protected function setUp()
 	{
-		\Aimeos\MShop\Factory::setCache( true );
+		\Aimeos\MShop\Factory::setCache(true);
 
 		$context = \TestHelperCntl::getContext();
-		$this->object = new \Aimeos\Controller\Common\Product\Import\Csv\Cache\Product\Standard( $context );
+		$this->object = new \Aimeos\Controller\Common\Product\Import\Csv\Cache\Product\Standard($context);
 	}
 
 
 	protected function tearDown()
 	{
-		\Aimeos\MShop\Factory::setCache( false );
+		\Aimeos\MShop\Factory::setCache(false);
 		\Aimeos\MShop\Factory::clear();
 	}
 
 
 	public function testGet()
 	{
-		$this->assertNotEquals( null, $this->object->get( 'CNC' ) );
+		$this->assertNotEquals(null, $this->object->get('CNC'));
 	}
 
 
 	public function testGetUnknown()
 	{
-		$this->assertEquals( null, $this->object->get( 'cache-test' ) );
+		$this->assertEquals(null, $this->object->get('cache-test'));
 	}
 
 
 	public function testSet()
 	{
-		$item = \Aimeos\MShop\Factory::createManager( \TestHelperCntl::getContext(), 'product' )->createItem();
-		$item->setCode( 'cache-test' );
-		$item->setId( 1 );
+		$item = \Aimeos\MShop\Factory::createManager(\TestHelperCntl::getContext(), 'product')->createItem();
+		$item->setCode('cache-test');
+		$item->setId(1);
 
-		$this->object->set( $item );
-		$id = $this->object->get( 'cache-test' );
+		$this->object->set($item);
+		$id = $this->object->get('cache-test');
 
-		$this->assertEquals( $item->getId(), $id );
+		$this->assertEquals($item->getId(), $id);
 	}
 }
\ No newline at end of file

controller/common/tests/Controller/Common/Product/Import/Csv/BaseTest.php

Spacing +29 added lines, -29 removed lines

@@ -14,92 +14,92 @@ 
 
 	protected function setUp()
 	{
-		\Aimeos\MShop\Factory::setCache( true );
+		\Aimeos\MShop\Factory::setCache(true);
 
 		$context = \TestHelperCntl::getContext();
 		$aimeos = \TestHelperCntl::getAimeos();
 
-		$this->object = new TestAbstract( $context, $aimeos );
+		$this->object = new TestAbstract($context, $aimeos);
 	}
 
 
 	protected function tearDown()
 	{
-		\Aimeos\MShop\Factory::setCache( false );
+		\Aimeos\MShop\Factory::setCache(false);
 		\Aimeos\MShop\Factory::clear();
 	}
 
 
 	public function testGetCache()
 	{
-		$cache = $this->object->getCachePublic( 'attribute' );
+		$cache = $this->object->getCachePublic('attribute');
 
-		$this->assertInstanceOf( '\\Aimeos\\Controller\\Common\\Product\\Import\\Csv\\Cache\\Iface', $cache );
+		$this->assertInstanceOf('\\Aimeos\\Controller\\Common\\Product\\Import\\Csv\\Cache\\Iface', $cache);
 	}
 
 
 	public function testGetCacheInvalidType()
 	{
-		$this->setExpectedException( '\\Aimeos\\Controller\\Jobs\\Exception' );
-		$this->object->getCachePublic( '$' );
+		$this->setExpectedException('\\Aimeos\\Controller\\Jobs\\Exception');
+		$this->object->getCachePublic('$');
 	}
 
 
 	public function testGetCacheInvalidClass()
 	{
-		$this->setExpectedException( '\\Aimeos\\Controller\\Jobs\\Exception' );
-		$this->object->getCachePublic( 'unknown' );
+		$this->setExpectedException('\\Aimeos\\Controller\\Jobs\\Exception');
+		$this->object->getCachePublic('unknown');
 	}
 
 
 	public function testGetCacheInvalidInterface()
 	{
-		$this->setExpectedException( '\\Aimeos\\Controller\\Jobs\\Exception' );
-		$this->object->getCachePublic( 'attribute', 'unknown' );
+		$this->setExpectedException('\\Aimeos\\Controller\\Jobs\\Exception');
+		$this->object->getCachePublic('attribute', 'unknown');
 	}
 
 
 	public function testGetProcessors()
 	{
-		$processor = $this->object->getProcessorsPublic( array( 'attribute' => array() ) );
+		$processor = $this->object->getProcessorsPublic(array('attribute' => array()));
 
-		$this->assertInstanceOf( '\\Aimeos\\Controller\\Common\\Product\\Import\\Csv\\Processor\\Iface', $processor );
+		$this->assertInstanceOf('\\Aimeos\\Controller\\Common\\Product\\Import\\Csv\\Processor\\Iface', $processor);
 	}
 
 
 	public function testGetProcessorsInvalidType()
 	{
-		$this->setExpectedException( '\\Aimeos\\Controller\\Jobs\\Exception' );
-		$this->object->getProcessorsPublic( array( '$' => array() ) );
+		$this->setExpectedException('\\Aimeos\\Controller\\Jobs\\Exception');
+		$this->object->getProcessorsPublic(array('$' => array()));
 	}
 
 
 	public function testGetProcessorsInvalidClass()
 	{
-		$this->setExpectedException( '\\Aimeos\\Controller\\Jobs\\Exception' );
-		$this->object->getProcessorsPublic( array( 'unknown' => array() ) );
+		$this->setExpectedException('\\Aimeos\\Controller\\Jobs\\Exception');
+		$this->object->getProcessorsPublic(array('unknown' => array()));
 	}
 
 
 	public function testGetProcessorsInvalidInterface()
 	{
-		$this->setExpectedException( '\\Aimeos\\Controller\\Jobs\\Exception' );
-		$this->object->getProcessorsPublic( array( 'unknown' => array() ) );
+		$this->setExpectedException('\\Aimeos\\Controller\\Jobs\\Exception');
+		$this->object->getProcessorsPublic(array('unknown' => array()));
 	}
 
 
 	public function testGetTypeId()
 	{
-		$typeid = $this->object->getTypeIdPublic( 'attribute/type', 'product', 'color' );
+		$typeid = $this->object->getTypeIdPublic('attribute/type', 'product', 'color');
 
-		$this->assertNotEquals( null, $typeid );
+		$this->assertNotEquals(null, $typeid);
 	}
 
 
 	public function testGetTypeIdUnknown()
 	{
-		$this->setExpectedException( '\\Aimeos\\Controller\\Jobs\\Exception' );
-		$this->object->getTypeIdPublic( 'attribute/type', 'product', 'unknown' );
+		$this->setExpectedException('\\Aimeos\\Controller\\Jobs\\Exception');
+		$this->object->getTypeIdPublic('attribute/type', 'product', 'unknown');
 	}
 }
 
@@ -107,21 +107,21 @@ 
 class TestAbstract
 	extends \Aimeos\Controller\Common\Product\Import\Csv\Base
 {
-	public function getCachePublic( $type, $name = null )
+	public function getCachePublic($type, $name = null)
 	{
-		return $this->getCache( $type, $name );
+		return $this->getCache($type, $name);
 	}
 
 
-	public function getProcessorsPublic( array $mappings )
+	public function getProcessorsPublic(array $mappings)
 	{
-		return $this->getProcessors( $mappings );
+		return $this->getProcessors($mappings);
 	}
 
 
-	public function getTypeIdPublic( $path, $domain, $code )
+	public function getTypeIdPublic($path, $domain, $code)
 	{
-		return $this->getTypeId( $path, $domain, $code );
+		return $this->getTypeId($path, $domain, $code);
 	}
 }
 

controller/jobs/src/Controller/Jobs/Order/Email/Payment/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/order/email/payment/name
 		 * Class name of the used order email payment scheduler controller implementation
@@ -65,20 +65,20 @@ 
 		 * @since 2014.03
 		 * @category Developer
 		 */
-		if( $name === null ) {
-			$name = $context->getConfig()->get( 'controller/jobs/order/email/payment/name', 'Standard' );
+		if ($name === null) {
+			$name = $context->getConfig()->get('controller/jobs/order/email/payment/name', 'Standard');
 		}
 
-		if( ctype_alnum( $name ) === false )
+		if (ctype_alnum($name) === false)
 		{
-			$classname = is_string( $name ) ? '\\Aimeos\\Controller\\Jobs\\Order\\Email\\Payment\\' . $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\\Order\\Email\\Payment\\' . $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\\Order\\Email\\Payment\\' . $name;
 
-		$controller = self::createControllerBase( $context, $aimeos, $classname, $iface );
+		$controller = self::createControllerBase($context, $aimeos, $classname, $iface);
 
 		/** controller/jobs/order/email/payment/decorators/excludes
 		 * Excludes decorators added by the "common" option from the order email payment controllers
@@ -154,6 +154,6 @@ 
 		 * @see controller/jobs/order/email/payment/decorators/excludes
 		 * @see controller/jobs/order/email/payment/decorators/global
 		 */
-		return self::addControllerDecorators( $context, $aimeos, $controller, 'order/email/payment' );
+		return self::addControllerDecorators($context, $aimeos, $controller, 'order/email/payment');
 	}
 }
\ No newline at end of file

controller/jobs/src/Controller/Jobs/Order/Email/Payment/Standard.php

Spacing +38 added lines, -38 removed lines

@@ -29,7 +29,7 @@ 
 	 */
 	public function getName()
 	{
-		return $this->getContext()->getI18n()->dt( 'controller/jobs', 'Order payment related e-mails' );
+		return $this->getContext()->getI18n()->dt('controller/jobs', 'Order payment related e-mails');
 	}
 
 
@@ -40,7 +40,7 @@ 
 	 */
 	public function getDescription()
 	{
-		return $this->getContext()->getI18n()->dt( 'controller/jobs', 'Sends order confirmation or payment status update e-mails' );
+		return $this->getContext()->getI18n()->dt('controller/jobs', 'Sends order confirmation or payment status update e-mails');
 	}
 
 
@@ -57,16 +57,16 @@ 
 		$mailer = $context->getMail();
 		$view = $context->getView();
 
-		$templatePaths = $aimeos->getCustomPaths( 'client/html/templates' );
+		$templatePaths = $aimeos->getCustomPaths('client/html/templates');
 
-		$helper = new \Aimeos\MW\View\Helper\Config\Standard( $view, $config );
-		$view->addHelper( 'config', $helper );
+		$helper = new \Aimeos\MW\View\Helper\Config\Standard($view, $config);
+		$view->addHelper('config', $helper);
 
-		$client = \Aimeos\Client\Html\Email\Payment\Factory::createClient( $context, $templatePaths );
+		$client = \Aimeos\Client\Html\Email\Payment\Factory::createClient($context, $templatePaths);
 
-		$orderManager = \Aimeos\MShop\Order\Manager\Factory::createManager( $context );
-		$orderStatusManager = $orderManager->getSubManager( 'status' );
-		$orderBaseManager = $orderManager->getSubManager( 'base' );
+		$orderManager = \Aimeos\MShop\Order\Manager\Factory::createManager($context);
+		$orderStatusManager = $orderManager->getSubManager('status');
+		$orderBaseManager = $orderManager->getSubManager('base');
 
 		/** controller/jobs/order/email/payment/standard/limit-days
 		 * Only send payment e-mails of orders that were created in the past within the configured number of days
@@ -83,8 +83,8 @@ 
 		 * @see controller/jobs/order/email/delivery/standard/limit-days
 		 * @see controller/jobs/service/delivery/process/limit-days
 		 */
-		$limit = $config->get( 'controller/jobs/order/email/payment/standard/limit-days', 30 );
-		$limitDate = date( 'Y-m-d H:i:s', time() - $limit * 86400 );
+		$limit = $config->get('controller/jobs/order/email/payment/standard/limit-days', 30);
+		$limitDate = date('Y-m-d H:i:s', time() - $limit * 86400);
 
 		$default = array(
 			\Aimeos\MShop\Order\Item\Base::PAY_REFUND,
@@ -116,71 +116,71 @@ 
 		 * @see controller/jobs/order/email/delivery/standard/status
 		 * @see controller/jobs/order/email/payment/standard/limit-days
 		 */
-		foreach( (array) $config->get( 'controller/jobs/order/email/payment/standard/status', $default ) as $status )
+		foreach ((array) $config->get('controller/jobs/order/email/payment/standard/status', $default) as $status)
 		{
 			$orderSearch = $orderManager->createSearch();
 
-			$param = array( \Aimeos\MShop\Order\Item\Status\Base::EMAIL_PAYMENT, $status );
-			$orderFunc = $orderSearch->createFunction( 'order.containsStatus', $param );
+			$param = array(\Aimeos\MShop\Order\Item\Status\Base::EMAIL_PAYMENT, $status);
+			$orderFunc = $orderSearch->createFunction('order.containsStatus', $param);
 
 			$expr = array(
-				$orderSearch->compare( '>=', 'order.mtime', $limitDate ),
-				$orderSearch->compare( '==', 'order.statuspayment', $status ),
-				$orderSearch->compare( '==', $orderFunc, 0 ),
+				$orderSearch->compare('>=', 'order.mtime', $limitDate),
+				$orderSearch->compare('==', 'order.statuspayment', $status),
+				$orderSearch->compare('==', $orderFunc, 0),
 			);
-			$orderSearch->setConditions( $orderSearch->combine( '&&', $expr ) );
+			$orderSearch->setConditions($orderSearch->combine('&&', $expr));
 
 			$start = 0;
 
 			do
 			{
-				$items = $orderManager->searchItems( $orderSearch );
+				$items = $orderManager->searchItems($orderSearch);
 
-				foreach( $items as $id => $item )
+				foreach ($items as $id => $item)
 				{
 					try
 					{
-						$orderBaseItem = $orderBaseManager->load( $item->getBaseId() );
+						$orderBaseItem = $orderBaseManager->load($item->getBaseId());
 
-						$addr = $orderBaseItem->getAddress( \Aimeos\MShop\Order\Item\Base\Address\Base::TYPE_PAYMENT );
+						$addr = $orderBaseItem->getAddress(\Aimeos\MShop\Order\Item\Base\Address\Base::TYPE_PAYMENT);
 
 						$view->extAddressItem = $addr;
 						$view->extOrderBaseItem = $orderBaseItem;
 						$view->extOrderItem = $item;
 
-						$helper = new \Aimeos\MW\View\Helper\Translate\Standard( $view, $context->getI18n( $addr->getLanguageId() ) );
-						$view->addHelper( 'translate', $helper );
+						$helper = new \Aimeos\MW\View\Helper\Translate\Standard($view, $context->getI18n($addr->getLanguageId()));
+						$view->addHelper('translate', $helper);
 
 						$message = $mailer->createMessage();
-						$helper = new \Aimeos\MW\View\Helper\Mail\Standard( $view, $message );
-						$view->addHelper( 'mail', $helper );
+						$helper = new \Aimeos\MW\View\Helper\Mail\Standard($view, $message);
+						$view->addHelper('mail', $helper);
 
-						$client->setView( $view );
+						$client->setView($view);
 						$client->getHeader();
 						$client->getBody();
 
-						$mailer->send( $message );
+						$mailer->send($message);
 
 						$statusItem = $orderStatusManager->createItem();
-						$statusItem->setParentId( $id );
-						$statusItem->setType( \Aimeos\MShop\Order\Item\Status\Base::EMAIL_PAYMENT );
-						$statusItem->setValue( $status );
+						$statusItem->setParentId($id);
+						$statusItem->setType(\Aimeos\MShop\Order\Item\Status\Base::EMAIL_PAYMENT);
+						$statusItem->setValue($status);
 
-						$orderStatusManager->saveItem( $statusItem );
+						$orderStatusManager->saveItem($statusItem);
 					}
-					catch( \Exception $e )
+					catch (\Exception $e)
 					{
 						$str = 'Error while trying to send payment e-mail for order ID "%1$s" and status "%2$s": %3$s';
-						$msg = sprintf( $str, $item->getId(), $item->getPaymentStatus(), $e->getMessage() );
-						$context->getLogger()->log( $msg );
+						$msg = sprintf($str, $item->getId(), $item->getPaymentStatus(), $e->getMessage());
+						$context->getLogger()->log($msg);
 					}
 				}
 
-				$count = count( $items );
+				$count = count($items);
 				$start += $count;
-				$orderSearch->setSlice( $start );
+				$orderSearch->setSlice($start);
 			}
-			while( $count >= $orderSearch->getSliceSize() );
+			while ($count >= $orderSearch->getSliceSize());
 		}
 	}
 }

Braces +1 added lines, -2 removed lines

@@ -173,8 +173,7 @@
 						$statusItem->setValue( $status );
 
 						$orderStatusManager->saveItem( $statusItem );
-					}
-					catch( \Exception $e )
+					} catch( \Exception $e )
 					{
 						$str = 'Error while trying to send delivery e-mail for order ID "%1$s" and status "%2$s": %3$s';
 						$msg = sprintf( $str, $item->getId(), $item->getDeliveryStatus(), $e->getMessage() );