aimeos / ai-controller-jobs

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

Spacing +21 added lines, -21 removed lines

@@ -28,7 +28,7 @@ 
 	 */
 	public function getName()
 	{
-		return $this->getContext()->getI18n()->dt( 'controller/jobs', 'Subscription process start' );
+		return $this->getContext()->getI18n()->dt('controller/jobs', 'Subscription process start');
 	}
 
 
@@ -39,7 +39,7 @@ 
 	 */
 	public function getDescription()
 	{
-		return $this->getContext()->getI18n()->dt( 'controller/jobs', 'Process subscriptions initially' );
+		return $this->getContext()->getI18n()->dt('controller/jobs', 'Process subscriptions initially');
 	}
 
 
@@ -65,49 +65,49 @@ 
 		 * @since 2018.04
 		 * @category Developer
 		 */
-		$names = (array) $config->get( 'controller/common/subscription/process/processors', [] );
+		$names = (array) $config->get('controller/common/subscription/process/processors', []);
 
-		$processors = $this->getProcessors( $names );
-		$manager = \Aimeos\MShop\Factory::createManager( $context, 'subscription' );
+		$processors = $this->getProcessors($names);
+		$manager = \Aimeos\MShop\Factory::createManager($context, 'subscription');
 
-		$search = $manager->createSearch( true );
+		$search = $manager->createSearch(true);
 		$expr = [
-			$search->compare( '==', 'subscription.datenext', null ),
+			$search->compare('==', 'subscription.datenext', null),
 			$search->getConditions(),
 		];
-		$search->setConditions( $search->combine( '&&', $expr ) );
-		$search->setSortations( [$search->sort( '+', 'subscription.id' )] );
+		$search->setConditions($search->combine('&&', $expr));
+		$search->setSortations([$search->sort('+', 'subscription.id')]);
 
 		$start = 0;
 
 		do
 		{
-			$search->setSlice( $start, 100 );
-			$items = $manager->searchItems( $search );
+			$search->setSlice($start, 100);
+			$items = $manager->searchItems($search);
 
-			foreach( $items as $item )
+			foreach ($items as $item)
 			{
 				try
 				{
-					foreach( $processors as $processor ) {
-						$processor->begin( $item );
+					foreach ($processors as $processor) {
+						$processor->begin($item);
 					}
 
-					$interval = \DateInterval::createFromDateString( $item->getInterval() );
-					$item->setDateNext( date_create( $item->getTimeCreated() )->add( $interval )->format( 'Y-m-d' ) );
+					$interval = \DateInterval::createFromDateString($item->getInterval());
+					$item->setDateNext(date_create($item->getTimeCreated())->add($interval)->format('Y-m-d'));
 
-					$manager->saveItem( $item );
+					$manager->saveItem($item);
 				}
-				catch( \Exception $e )
+				catch (\Exception $e)
 				{
 					$msg = 'Unable to process subscription with ID "%1$S": %2$s';
-					$logger->log( sprintf( $msg, $item->getId(), $e->getMessage() ) );
+					$logger->log(sprintf($msg, $item->getId(), $e->getMessage()));
 				}
 			}
 
-			$count = count( $items );
+			$count = count($items);
 			$start += $count;
 		}
-		while( $count === $search->getSliceSize() );
+		while ($count === $search->getSliceSize());
 	}
 }

Braces +1 added lines, -2 removed lines

@@ -97,8 +97,7 @@
 					$item->setDateNext( date_create( $item->getTimeCreated() )->add( $interval )->format( 'Y-m-d' ) );
 
 					$manager->saveItem( $item );
-				}
-				catch( \Exception $e )
+				} catch( \Exception $e )
 				{
 					$msg = 'Unable to process subscription with ID "%1$S": %2$s';
 					$logger->log( sprintf( $msg, $item->getId(), $e->getMessage() ) );

controller/common/tests/Controller/Common/Subscription/Process/Processor/Cgroup/StandardTest.php

Spacing +33 added lines, -33 removed lines

@@ -12,13 +12,13 @@ 
 {
 	protected function setUp()
 	{
-		\Aimeos\MShop\Factory::setCache( true );
+		\Aimeos\MShop\Factory::setCache(true);
 	}
 
 
 	protected function tearDown()
 	{
-		\Aimeos\MShop\Factory::setCache( false );
+		\Aimeos\MShop\Factory::setCache(false);
 	}
 
 
@@ -26,27 +26,27 @@ 
 	{
 		$context = \TestHelperCntl::getContext();
 
-		$context->getConfig()->set( 'controller/common/subscription/process/processor/cgroup/groupids', ['1', '2'] );
+		$context->getConfig()->set('controller/common/subscription/process/processor/cgroup/groupids', ['1', '2']);
 
-		$customerStub = $this->getMockBuilder( '\\Aimeos\\MShop\\Customer\\Manager\\Standard' )
-			->setConstructorArgs( [$context] )
-			->setMethods( ['getItem', 'saveItem'] )
+		$customerStub = $this->getMockBuilder('\\Aimeos\\MShop\\Customer\\Manager\\Standard')
+			->setConstructorArgs([$context])
+			->setMethods(['getItem', 'saveItem'])
 			->getMock();
 
-		\Aimeos\MShop\Factory::injectManager( $context, 'customer', $customerStub );
+		\Aimeos\MShop\Factory::injectManager($context, 'customer', $customerStub);
 
 		$customerItem = $customerStub->createItem();
 
-		$customerStub->expects( $this->once() )->method( 'getItem' )
-			->will( $this->returnValue( $customerItem ) );
+		$customerStub->expects($this->once())->method('getItem')
+			->will($this->returnValue($customerItem));
 
-		$customerStub->expects( $this->once() )->method( 'saveItem' )
-			->with( $this->callback( function( $subject ){
+		$customerStub->expects($this->once())->method('saveItem')
+			->with($this->callback(function($subject) {
 				return $subject->getGroups() === ['1', '2'];
-			} ) );
+			} ));
 
-		$object = new \Aimeos\Controller\Common\Subscription\Process\Processor\Cgroup\Standard( $context );
-		$object->begin( $this->getSubscription( $context ) );
+		$object = new \Aimeos\Controller\Common\Subscription\Process\Processor\Cgroup\Standard($context);
+		$object->begin($this->getSubscription($context));
 	}
 
 
@@ -54,43 +54,43 @@ 
 	{
 		$context = \TestHelperCntl::getContext();
 
-		$context->getConfig()->set( 'controller/common/subscription/process/processor/cgroup/groupids', ['1', '2'] );
+		$context->getConfig()->set('controller/common/subscription/process/processor/cgroup/groupids', ['1', '2']);
 
-		$customerStub = $this->getMockBuilder( '\\Aimeos\\MShop\\Customer\\Manager\\Standard' )
-			->setConstructorArgs( [$context] )
-			->setMethods( ['getItem', 'saveItem'] )
+		$customerStub = $this->getMockBuilder('\\Aimeos\\MShop\\Customer\\Manager\\Standard')
+			->setConstructorArgs([$context])
+			->setMethods(['getItem', 'saveItem'])
 			->getMock();
 
-		\Aimeos\MShop\Factory::injectManager( $context, 'customer', $customerStub );
+		\Aimeos\MShop\Factory::injectManager($context, 'customer', $customerStub);
 
-		$customerItem = $customerStub->createItem()->setGroups( ['1', '2'] );
+		$customerItem = $customerStub->createItem()->setGroups(['1', '2']);
 
-		$customerStub->expects( $this->once() )->method( 'getItem' )
-			->will( $this->returnValue( $customerItem ) );
+		$customerStub->expects($this->once())->method('getItem')
+			->will($this->returnValue($customerItem));
 
-		$customerStub->expects( $this->once() )->method( 'saveItem' )
-			->with( $this->callback( function( $subject ){
+		$customerStub->expects($this->once())->method('saveItem')
+			->with($this->callback(function($subject) {
 				return $subject->getGroups() === [];
-			} ) );
+			} ));
 
-		$object = new \Aimeos\Controller\Common\Subscription\Process\Processor\Cgroup\Standard( $context );
-		$object->end( $this->getSubscription( $context ) );
+		$object = new \Aimeos\Controller\Common\Subscription\Process\Processor\Cgroup\Standard($context);
+		$object->end($this->getSubscription($context));
 	}
 
 
-	protected function getSubscription( $context )
+	protected function getSubscription($context)
 	{
-		$manager = \Aimeos\MShop\Factory::createManager( $context, 'subscription' );
+		$manager = \Aimeos\MShop\Factory::createManager($context, 'subscription');
 
 		$search = $manager->createSearch();
-		$search->setConditions( $search->compare( '==', 'subscription.dateend', '2010-01-01' ) );
+		$search->setConditions($search->compare('==', 'subscription.dateend', '2010-01-01'));
 
-		$items = $manager->searchItems( $search );
+		$items = $manager->searchItems($search);
 
-		if( ( $item = reset( $items ) ) !== false ) {
+		if (($item = reset($items)) !== false) {
 			return $item;
 		}
 
-		throw new \Exception( 'No subscription item found' );
+		throw new \Exception('No subscription item found');
 	}
 }