aimeos / ai-admin-jqadm

admin/jqadm/src/Admin/JQAdm/Dashboard/Order/Standard.php

Spacing +11 added lines, -11 removed lines

@@ -55,7 +55,7 @@ 
 	 * @category Developer
 	 */
 	private $subPartPath = 'admin/jqadm/dashboard/order/standard/subparts';
-	private $subPartNames = array( 'paymentstatus' );
+	private $subPartNames = array('paymentstatus');
 
 
 	/**
@@ -65,7 +65,7 @@ 
 	 */
 	public function copy()
 	{
-		throw new \Aimeos\Admin\JQAdm\Exception( 'The resource can not be copied' );
+		throw new \Aimeos\Admin\JQAdm\Exception('The resource can not be copied');
 	}
 
 
@@ -76,7 +76,7 @@ 
 	 */
 	public function create()
 	{
-		throw new \Aimeos\Admin\JQAdm\Exception( 'New resources can not be created' );
+		throw new \Aimeos\Admin\JQAdm\Exception('New resources can not be created');
 	}
 
 
@@ -87,7 +87,7 @@ 
 	 */
 	public function delete()
 	{
-		throw new \Aimeos\Admin\JQAdm\Exception( 'The resource can not be deleted' );
+		throw new \Aimeos\Admin\JQAdm\Exception('The resource can not be deleted');
 	}
 
 
@@ -98,7 +98,7 @@ 
 	 */
 	public function get()
 	{
-		throw new \Aimeos\Admin\JQAdm\Exception( 'The resource can not be retrieved' );
+		throw new \Aimeos\Admin\JQAdm\Exception('The resource can not be retrieved');
 	}
 
 
@@ -109,7 +109,7 @@ 
 	 */
 	public function save()
 	{
-		throw new \Aimeos\Admin\JQAdm\Exception( 'The resource can not be modified' );
+		throw new \Aimeos\Admin\JQAdm\Exception('The resource can not be modified');
 	}
 
 
@@ -123,14 +123,14 @@ 
 		$view = $this->getView();
 		$view->orderBody = '';
 
-		foreach( $this->getSubClients() as $client ) {
+		foreach ($this->getSubClients() as $client) {
 			$view->orderBody .= $client->search();
 		}
 
 		$tplconf = 'admin/jqadm/dashboard/order/template-list';
 		$default = 'dashboard/list-order-default.php';
 
-		return $view->render( $view->config( $tplconf, $default ) );
+		return $view->render($view->config($tplconf, $default));
 	}
 
 
@@ -141,7 +141,7 @@ 
 	 * @param string|null $name Name of the sub-client (Default if null)
 	 * @return \Aimeos\Admin\JQAdm\Iface Sub-client object
 	 */
-	public function getSubClient( $type, $name = null )
+	public function getSubClient($type, $name = null)
 	{
 		/** admin/jqadm/dashboard/order/decorators/excludes
 		 * Excludes decorators added by the "common" option from the dashboard JQAdm client
@@ -216,7 +216,7 @@ 
 		 * @see admin/jqadm/dashboard/order/decorators/excludes
 		 * @see admin/jqadm/dashboard/order/decorators/global
 		 */
-		return $this->createSubClient( 'dashboard/order/' . $type, $name );
+		return $this->createSubClient('dashboard/order/' . $type, $name);
 	}
 
 
@@ -227,6 +227,6 @@ 
 	 */
 	protected function getSubClientNames()
 	{
-		return $this->getContext()->getConfig()->get( $this->subPartPath, $this->subPartNames );
+		return $this->getContext()->getConfig()->get($this->subPartPath, $this->subPartNames);
 	}
 }

admin/jqadm/src/Admin/JQAdm/Dashboard/Order/Paymentstatus/Standard.php

Spacing +16 added lines, -16 removed lines

@@ -65,7 +65,7 @@ 
 	 */
 	public function copy()
 	{
-		throw new \Aimeos\Admin\JQAdm\Exception( 'The resource can not be copied' );
+		throw new \Aimeos\Admin\JQAdm\Exception('The resource can not be copied');
 	}
 
 
@@ -76,7 +76,7 @@ 
 	 */
 	public function create()
 	{
-		throw new \Aimeos\Admin\JQAdm\Exception( 'New resources can not be created' );
+		throw new \Aimeos\Admin\JQAdm\Exception('New resources can not be created');
 	}
 
 
@@ -87,7 +87,7 @@ 
 	 */
 	public function delete()
 	{
-		throw new \Aimeos\Admin\JQAdm\Exception( 'The resource can not be deleted' );
+		throw new \Aimeos\Admin\JQAdm\Exception('The resource can not be deleted');
 	}
 
 
@@ -98,7 +98,7 @@ 
 	 */
 	public function get()
 	{
-		throw new \Aimeos\Admin\JQAdm\Exception( 'The resource can not be retrieved' );
+		throw new \Aimeos\Admin\JQAdm\Exception('The resource can not be retrieved');
 	}
 
 
@@ -109,7 +109,7 @@ 
 	 */
 	public function save()
 	{
-		throw new \Aimeos\Admin\JQAdm\Exception( 'The resource can not be modified' );
+		throw new \Aimeos\Admin\JQAdm\Exception('The resource can not be modified');
 	}
 
 
@@ -127,25 +127,25 @@ 
 		{
 			$view->orderpaymentstatusBody = '';
 
-			foreach( $this->getSubClients() as $client ) {
+			foreach ($this->getSubClients() as $client) {
 				$view->orderpaymentstatusBody .= $client->search();
 			}
 		}
-		catch( \Aimeos\MShop\Exception $e )
+		catch (\Aimeos\MShop\Exception $e)
 		{
-			$error = array( 'order-paymentstatus' => $context->getI18n()->dt( 'mshop', $e->getMessage() ) );
-			$view->errors = $view->get( 'errors', array() ) + $error;
+			$error = array('order-paymentstatus' => $context->getI18n()->dt('mshop', $e->getMessage()));
+			$view->errors = $view->get('errors', array()) + $error;
 		}
-		catch( \Exception $e )
+		catch (\Exception $e)
 		{
-			$error = array( 'order-paymentstatus' => $e->getMessage() );
-			$view->errors = $view->get( 'errors', array() ) + $error;
+			$error = array('order-paymentstatus' => $e->getMessage());
+			$view->errors = $view->get('errors', array()) + $error;
 		}
 
 		$tplconf = 'admin/jqadm/dashboard/order/paymentstatus/template-item';
 		$default = 'dashboard/item-order-paymentstatus-default.php';
 
-		return $view->render( $view->config( $tplconf, $default ) );
+		return $view->render($view->config($tplconf, $default));
 	}
 
 
@@ -156,7 +156,7 @@ 
 	 * @param string|null $name Name of the sub-client (Default if null)
 	 * @return \Aimeos\Admin\JQAdm\Iface Sub-client object
 	 */
-	public function getSubClient( $type, $name = null )
+	public function getSubClient($type, $name = null)
 	{
 		/** admin/jqadm/dashboard/order/paymentstatus/decorators/excludes
 		 * Excludes decorators added by the "common" option from the dashboard JQAdm client
@@ -231,7 +231,7 @@ 
 		 * @see admin/jqadm/dashboard/order/paymentstatus/decorators/excludes
 		 * @see admin/jqadm/dashboard/order/paymentstatus/decorators/global
 		 */
-		return $this->createSubClient( 'dashboard/order/paymentstatus/' . $type, $name );
+		return $this->createSubClient('dashboard/order/paymentstatus/' . $type, $name);
 	}
 
 
@@ -242,6 +242,6 @@ 
 	 */
 	protected function getSubClientNames()
 	{
-		return $this->getContext()->getConfig()->get( $this->subPartPath, $this->subPartNames );
+		return $this->getContext()->getConfig()->get($this->subPartPath, $this->subPartNames);
 	}
 }

Braces +2 added lines, -4 removed lines

@@ -151,14 +151,12 @@
 
 			$manager->commit();
 			return;
-		}
-		catch( \Aimeos\MShop\Exception $e )
+		} catch( \Aimeos\MShop\Exception $e )
 		{
 			$error = array( 'product-item-category' => $context->getI18n()->dt( 'mshop', $e->getMessage() ) );
 			$view->errors = $view->get( 'errors', array() ) + $error;
 			$manager->rollback();
-		}
-		catch( \Exception $e )
+		} catch( \Exception $e )
 		{
 			$context->getLogger()->log( $e->getMessage() . ' - ' . $e->getTraceAsString() );
 			$error = array( 'product-item-category' => $e->getMessage() );

admin/jqadm/src/Admin/JQAdm/Dashboard/Order/Latest/Standard.php

Spacing +25 added lines, -25 removed lines

@@ -65,7 +65,7 @@ 
 	 */
 	public function copy()
 	{
-		throw new \Aimeos\Admin\JQAdm\Exception( 'The resource can not be copied' );
+		throw new \Aimeos\Admin\JQAdm\Exception('The resource can not be copied');
 	}
 
 
@@ -76,7 +76,7 @@ 
 	 */
 	public function create()
 	{
-		throw new \Aimeos\Admin\JQAdm\Exception( 'New resources can not be created' );
+		throw new \Aimeos\Admin\JQAdm\Exception('New resources can not be created');
 	}
 
 
@@ -87,7 +87,7 @@ 
 	 */
 	public function delete()
 	{
-		throw new \Aimeos\Admin\JQAdm\Exception( 'The resource can not be deleted' );
+		throw new \Aimeos\Admin\JQAdm\Exception('The resource can not be deleted');
 	}
 
 
@@ -98,7 +98,7 @@ 
 	 */
 	public function get()
 	{
-		throw new \Aimeos\Admin\JQAdm\Exception( 'The resource can not be retrieved' );
+		throw new \Aimeos\Admin\JQAdm\Exception('The resource can not be retrieved');
 	}
 
 
@@ -109,7 +109,7 @@ 
 	 */
 	public function save()
 	{
-		throw new \Aimeos\Admin\JQAdm\Exception( 'The resource can not be modified' );
+		throw new \Aimeos\Admin\JQAdm\Exception('The resource can not be modified');
 	}
 
 
@@ -124,28 +124,28 @@ 
 
 		try
 		{
-			$this->addOrders( $view );
+			$this->addOrders($view);
 			$view->orderlatestBody = '';
 
-			foreach( $this->getSubClients() as $client ) {
+			foreach ($this->getSubClients() as $client) {
 				$view->orderlatestBody .= $client->search();
 			}
 		}
-		catch( \Aimeos\MShop\Exception $e )
+		catch (\Aimeos\MShop\Exception $e)
 		{
-			$error = array( 'order-latest' => $this->getContext()->getI18n()->dt( 'mshop', $e->getMessage() ) );
-			$view->errors = $view->get( 'errors', array() ) + $error;
+			$error = array('order-latest' => $this->getContext()->getI18n()->dt('mshop', $e->getMessage()));
+			$view->errors = $view->get('errors', array()) + $error;
 		}
-		catch( \Exception $e )
+		catch (\Exception $e)
 		{
-			$error = array( 'order-latest' => $e->getMessage() );
-			$view->errors = $view->get( 'errors', array() ) + $error;
+			$error = array('order-latest' => $e->getMessage());
+			$view->errors = $view->get('errors', array()) + $error;
 		}
 
 		$tplconf = 'admin/jqadm/dashboard/order/latest/template-item';
 		$default = 'dashboard/item-order-latest-default.php';
 
-		return $view->render( $view->config( $tplconf, $default ) );
+		return $view->render($view->config($tplconf, $default));
 	}
 
 
@@ -156,7 +156,7 @@ 
 	 * @param string|null $name Name of the sub-client (Default if null)
 	 * @return \Aimeos\Admin\JQAdm\Iface Sub-client object
 	 */
-	public function getSubClient( $type, $name = null )
+	public function getSubClient($type, $name = null)
 	{
 		/** admin/jqadm/dashboard/order/latest/decorators/excludes
 		 * Excludes decorators added by the "common" option from the dashboard JQAdm client
@@ -231,7 +231,7 @@ 
 		 * @see admin/jqadm/dashboard/order/latest/decorators/excludes
 		 * @see admin/jqadm/dashboard/order/latest/decorators/global
 		 */
-		return $this->createSubClient( 'dashboard/order/latest/' . $type, $name );
+		return $this->createSubClient('dashboard/order/latest/' . $type, $name);
 	}
 
 
@@ -240,23 +240,23 @@ 
 	 *
 	 * @param \Aimeos\MW\View\Iface $view View object to add the parameters to
 	 */
-	protected function addOrders( \Aimeos\MW\View\Iface $view )
+	protected function addOrders(\Aimeos\MW\View\Iface $view)
 	{
-		$manager = \Aimeos\MShop\Factory::createManager( $this->getContext(), 'order/base' );
+		$manager = \Aimeos\MShop\Factory::createManager($this->getContext(), 'order/base');
 
 		$search = $manager->createSearch();
-		$search->setSortations( array( $search->sort( '-', 'order.base.ctime' ) ) );
-		$search->setSlice( 0, 10 );
+		$search->setSortations(array($search->sort('-', 'order.base.ctime')));
+		$search->setSlice(0, 10);
 
-		$orders = $manager->searchItems( $search );
+		$orders = $manager->searchItems($search);
 
 
-		$manager = \Aimeos\MShop\Factory::createManager( $this->getContext(), 'order' );
+		$manager = \Aimeos\MShop\Factory::createManager($this->getContext(), 'order');
 
 		$search = $manager->createSearch();
-		$search->setConditions( $search->compare( '==', 'order.baseid', array_keys( $orders ) ) );
+		$search->setConditions($search->compare('==', 'order.baseid', array_keys($orders)));
 
-		$items = $manager->searchItems( $search );
+		$items = $manager->searchItems($search);
 
 
 		$view->orderlatestOrders = $orders;
@@ -271,6 +271,6 @@ 
 	 */
 	protected function getSubClientNames()
 	{
-		return $this->getContext()->getConfig()->get( $this->subPartPath, $this->subPartNames );
+		return $this->getContext()->getConfig()->get($this->subPartPath, $this->subPartNames);
 	}
 }

Braces +2 added lines, -4 removed lines

@@ -151,14 +151,12 @@
 
 			$manager->commit();
 			return;
-		}
-		catch( \Aimeos\MShop\Exception $e )
+		} catch( \Aimeos\MShop\Exception $e )
 		{
 			$error = array( 'product-item-category' => $context->getI18n()->dt( 'mshop', $e->getMessage() ) );
 			$view->errors = $view->get( 'errors', array() ) + $error;
 			$manager->rollback();
-		}
-		catch( \Exception $e )
+		} catch( \Exception $e )
 		{
 			$context->getLogger()->log( $e->getMessage() . ' - ' . $e->getTraceAsString() );
 			$error = array( 'product-item-category' => $e->getMessage() );

admin/jqadm/src/Admin/JQAdm/Dashboard/Factory.php

Spacing +8 added lines, -8 removed lines

@@ -30,7 +30,7 @@ 
 	 * @return \Aimeos\Admin\JQAdm\Iface Filter part implementing \Aimeos\Admin\JQAdm\Iface
 	 * @throws \Aimeos\Admin\JQAdm\Exception If requested client implementation couldn't be found or initialisation fails
 	 */
-	public static function createClient( \Aimeos\MShop\Context\Item\Iface $context, array $templatePaths, $name = null )
+	public static function createClient(\Aimeos\MShop\Context\Item\Iface $context, array $templatePaths, $name = null)
 	{
 		/** admin/jqadm/dashboard/name
 		 * Class name of the used account favorite client implementation
@@ -65,22 +65,22 @@ 
 		 * @since 2016.01
 		 * @category Developer
 		 */
-		if( $name === null ) {
-			$name = $context->getConfig()->get( 'admin/jqadm/dashboard/name', 'Standard' );
+		if ($name === null) {
+			$name = $context->getConfig()->get('admin/jqadm/dashboard/name', 'Standard');
 		}
 
-		if( ctype_alnum( $name ) === false )
+		if (ctype_alnum($name) === false)
 		{
-			$classname = is_string( $name ) ? '\\Aimeos\\Admin\\JQAdm\\Dashboard\\' . $name : '<not a string>';
-			throw new \Aimeos\Admin\JQAdm\Exception( sprintf( 'Invalid characters in class name "%1$s"', $classname ) );
+			$classname = is_string($name) ? '\\Aimeos\\Admin\\JQAdm\\Dashboard\\' . $name : '<not a string>';
+			throw new \Aimeos\Admin\JQAdm\Exception(sprintf('Invalid characters in class name "%1$s"', $classname));
 		}
 
 		$iface = '\\Aimeos\\Admin\\JQAdm\\Iface';
 		$classname = '\\Aimeos\\Admin\\JQAdm\\Dashboard\\' . $name;
 
-		$client = self::createClientBase( $context, $classname, $iface, $templatePaths );
+		$client = self::createClientBase($context, $classname, $iface, $templatePaths);
 
-		return self::addClientDecorators( $context, $client, $templatePaths, 'dashboard' );
+		return self::addClientDecorators($context, $client, $templatePaths, 'dashboard');
 	}
 
 }

admin/jqadm/tests/Admin/JQAdm/Product/StandardTest.php

Spacing +145 added lines, -145 removed lines

@@ -17,21 +17,21 @@ 
 	protected function setUp()
 	{
 		$this->view = \TestHelperJqadm::getView();
-		$request = $this->getMockBuilder( '\Psr\Http\Message\ServerRequestInterface' )->getMock();
-		$helper = new \Aimeos\MW\View\Helper\Request\Standard( $this->view, $request, '127.0.0.1', 'test' );
-		$this->view ->addHelper( 'request', $helper );
+		$request = $this->getMockBuilder('\Psr\Http\Message\ServerRequestInterface')->getMock();
+		$helper = new \Aimeos\MW\View\Helper\Request\Standard($this->view, $request, '127.0.0.1', 'test');
+		$this->view ->addHelper('request', $helper);
 
 		$this->context = \TestHelperJqadm::getContext();
 		$templatePaths = \TestHelperJqadm::getTemplatePaths();
 
-		$this->object = new \Aimeos\Admin\JQAdm\Product\Standard( $this->context, $templatePaths );
-		$this->object->setView( $this->view );
+		$this->object = new \Aimeos\Admin\JQAdm\Product\Standard($this->context, $templatePaths);
+		$this->object->setView($this->view);
 	}
 
 
 	protected function tearDown()
 	{
-		unset( $this->object );
+		unset($this->object);
 	}
 
 
@@ -43,15 +43,15 @@ 
 
 	public function testCreateException()
 	{
-		$object = $this->getMockBuilder( '\Aimeos\Admin\JQAdm\Product\Standard' )
-			->setConstructorArgs( array( $this->context, \TestHelperJqadm::getTemplatePaths() ) )
-			->setMethods( array( 'getSubClients' ) )
+		$object = $this->getMockBuilder('\Aimeos\Admin\JQAdm\Product\Standard')
+			->setConstructorArgs(array($this->context, \TestHelperJqadm::getTemplatePaths()))
+			->setMethods(array('getSubClients'))
 			->getMock();
 
-		$object->expects( $this->once() )->method( 'getSubClients' )
-			->will( $this->throwException( new \Exception() ) );
+		$object->expects($this->once())->method('getSubClients')
+			->will($this->throwException(new \Exception()));
 
-		$object->setView( $this->getViewNoRender() );
+		$object->setView($this->getViewNoRender());
 
 		$object->create();
 	}
@@ -59,15 +59,15 @@ 
 
 	public function testCreateMShopException()
 	{
-		$object = $this->getMockBuilder( '\Aimeos\Admin\JQAdm\Product\Standard' )
-			->setConstructorArgs( array( $this->context, \TestHelperJqadm::getTemplatePaths() ) )
-			->setMethods( array( 'getSubClients' ) )
+		$object = $this->getMockBuilder('\Aimeos\Admin\JQAdm\Product\Standard')
+			->setConstructorArgs(array($this->context, \TestHelperJqadm::getTemplatePaths()))
+			->setMethods(array('getSubClients'))
 			->getMock();
 
-		$object->expects( $this->once() )->method( 'getSubClients' )
-			->will( $this->throwException( new \Aimeos\MShop\Exception() ) );
+		$object->expects($this->once())->method('getSubClients')
+			->will($this->throwException(new \Aimeos\MShop\Exception()));
 
-		$object->setView( $this->getViewNoRender() );
+		$object->setView($this->getViewNoRender());
 
 		$object->create();
 	}
@@ -75,29 +75,29 @@ 
 
 	public function testCopy()
 	{
-		$manager = \Aimeos\MShop\Factory::createManager( $this->context, 'product' );
+		$manager = \Aimeos\MShop\Factory::createManager($this->context, 'product');
 
-		$param = array( 'id' => $manager->findItem( 'CNC' )->getId() );
-		$helper = new \Aimeos\MW\View\Helper\Param\Standard( $this->view, $param );
-		$this->view->addHelper( 'param', $helper );
+		$param = array('id' => $manager->findItem('CNC')->getId());
+		$helper = new \Aimeos\MW\View\Helper\Param\Standard($this->view, $param);
+		$this->view->addHelper('param', $helper);
 
 		$result = $this->object->copy();
 
-		$this->assertContains( 'CNC_copy', $result );
+		$this->assertContains('CNC_copy', $result);
 	}
 
 
 	public function testCopyException()
 	{
-		$object = $this->getMockBuilder( '\Aimeos\Admin\JQAdm\Product\Standard' )
-			->setConstructorArgs( array( $this->context, \TestHelperJqadm::getTemplatePaths() ) )
-			->setMethods( array( 'getSubClients' ) )
+		$object = $this->getMockBuilder('\Aimeos\Admin\JQAdm\Product\Standard')
+			->setConstructorArgs(array($this->context, \TestHelperJqadm::getTemplatePaths()))
+			->setMethods(array('getSubClients'))
 			->getMock();
 
-		$object->expects( $this->once() )->method( 'getSubClients' )
-			->will( $this->throwException( new \Exception() ) );
+		$object->expects($this->once())->method('getSubClients')
+			->will($this->throwException(new \Exception()));
 
-		$object->setView( $this->getViewNoRender() );
+		$object->setView($this->getViewNoRender());
 
 		$object->copy();
 	}
@@ -105,15 +105,15 @@ 
 
 	public function testCopyMShopException()
 	{
-		$object = $this->getMockBuilder( '\Aimeos\Admin\JQAdm\Product\Standard' )
-			->setConstructorArgs( array( $this->context, \TestHelperJqadm::getTemplatePaths() ) )
-			->setMethods( array( 'getSubClients' ) )
+		$object = $this->getMockBuilder('\Aimeos\Admin\JQAdm\Product\Standard')
+			->setConstructorArgs(array($this->context, \TestHelperJqadm::getTemplatePaths()))
+			->setMethods(array('getSubClients'))
 			->getMock();
 
-		$object->expects( $this->once() )->method( 'getSubClients' )
-			->will( $this->throwException( new \Aimeos\MShop\Exception() ) );
+		$object->expects($this->once())->method('getSubClients')
+			->will($this->throwException(new \Aimeos\MShop\Exception()));
 
-		$object->setView( $this->getViewNoRender() );
+		$object->setView($this->getViewNoRender());
 
 		$object->copy();
 	}
@@ -121,21 +121,21 @@ 
 
 	public function testDelete()
 	{
-		$this->assertNull( $this->object->delete() );
+		$this->assertNull($this->object->delete());
 	}
 
 
 	public function testDeleteException()
 	{
-		$object = $this->getMockBuilder( '\Aimeos\Admin\JQAdm\Product\Standard' )
-			->setConstructorArgs( array( $this->context, \TestHelperJqadm::getTemplatePaths() ) )
-			->setMethods( array( 'getSubClients' ) )
+		$object = $this->getMockBuilder('\Aimeos\Admin\JQAdm\Product\Standard')
+			->setConstructorArgs(array($this->context, \TestHelperJqadm::getTemplatePaths()))
+			->setMethods(array('getSubClients'))
 			->getMock();
 
-		$object->expects( $this->once() )->method( 'getSubClients' )
-			->will( $this->throwException( new \Exception() ) );
+		$object->expects($this->once())->method('getSubClients')
+			->will($this->throwException(new \Exception()));
 
-		$object->setView( $this->getViewNoRender() );
+		$object->setView($this->getViewNoRender());
 
 		$object->delete();
 	}
@@ -143,15 +143,15 @@ 
 
 	public function testDeleteMShopException()
 	{
-		$object = $this->getMockBuilder( '\Aimeos\Admin\JQAdm\Product\Standard' )
-			->setConstructorArgs( array( $this->context, \TestHelperJqadm::getTemplatePaths() ) )
-			->setMethods( array( 'getSubClients' ) )
+		$object = $this->getMockBuilder('\Aimeos\Admin\JQAdm\Product\Standard')
+			->setConstructorArgs(array($this->context, \TestHelperJqadm::getTemplatePaths()))
+			->setMethods(array('getSubClients'))
 			->getMock();
 
-		$object->expects( $this->once() )->method( 'getSubClients' )
-			->will( $this->throwException( new \Aimeos\MShop\Exception() ) );
+		$object->expects($this->once())->method('getSubClients')
+			->will($this->throwException(new \Aimeos\MShop\Exception()));
 
-		$object->setView( $this->getViewNoRender() );
+		$object->setView($this->getViewNoRender());
 
 		$object->delete();
 	}
@@ -159,29 +159,29 @@ 
 
 	public function testGet()
 	{
-		$manager = \Aimeos\MShop\Factory::createManager( $this->context, 'product' );
+		$manager = \Aimeos\MShop\Factory::createManager($this->context, 'product');
 
-		$param = array( 'id' => $manager->findItem( 'CNC' )->getId() );
-		$helper = new \Aimeos\MW\View\Helper\Param\Standard( $this->view, $param );
-		$this->view->addHelper( 'param', $helper );
+		$param = array('id' => $manager->findItem('CNC')->getId());
+		$helper = new \Aimeos\MW\View\Helper\Param\Standard($this->view, $param);
+		$this->view->addHelper('param', $helper);
 
 		$result = $this->object->get();
 
-		$this->assertContains( 'CNC', $result );
+		$this->assertContains('CNC', $result);
 	}
 
 
 	public function testGetException()
 	{
-		$object = $this->getMockBuilder( '\Aimeos\Admin\JQAdm\Product\Standard' )
-			->setConstructorArgs( array( $this->context, \TestHelperJqadm::getTemplatePaths() ) )
-			->setMethods( array( 'getSubClients' ) )
+		$object = $this->getMockBuilder('\Aimeos\Admin\JQAdm\Product\Standard')
+			->setConstructorArgs(array($this->context, \TestHelperJqadm::getTemplatePaths()))
+			->setMethods(array('getSubClients'))
 			->getMock();
 
-		$object->expects( $this->once() )->method( 'getSubClients' )
-			->will( $this->throwException( new \Exception() ) );
+		$object->expects($this->once())->method('getSubClients')
+			->will($this->throwException(new \Exception()));
 
-		$object->setView( $this->getViewNoRender() );
+		$object->setView($this->getViewNoRender());
 
 		$object->get();
 	}
@@ -189,15 +189,15 @@ 
 
 	public function testGetMShopException()
 	{
-		$object = $this->getMockBuilder( '\Aimeos\Admin\JQAdm\Product\Standard' )
-			->setConstructorArgs( array( $this->context, \TestHelperJqadm::getTemplatePaths() ) )
-			->setMethods( array( 'getSubClients' ) )
+		$object = $this->getMockBuilder('\Aimeos\Admin\JQAdm\Product\Standard')
+			->setConstructorArgs(array($this->context, \TestHelperJqadm::getTemplatePaths()))
+			->setMethods(array('getSubClients'))
 			->getMock();
 
-		$object->expects( $this->once() )->method( 'getSubClients' )
-			->will( $this->throwException( new \Aimeos\MShop\Exception() ) );
+		$object->expects($this->once())->method('getSubClients')
+			->will($this->throwException(new \Aimeos\MShop\Exception()));
 
-		$object->setView( $this->getViewNoRender() );
+		$object->setView($this->getViewNoRender());
 
 		$object->get();
 	}
@@ -205,24 +205,24 @@ 
 
 	public function testGetViewException()
 	{
-		$object = new \Aimeos\Admin\JQAdm\Product\Standard( $this->context, array() );
+		$object = new \Aimeos\Admin\JQAdm\Product\Standard($this->context, array());
 
-		$this->setExpectedException( '\Aimeos\Admin\JQAdm\Exception' );
+		$this->setExpectedException('\Aimeos\Admin\JQAdm\Exception');
 		$object->getView();
 	}
 
 
 	public function testSave()
 	{
-		$manager = \Aimeos\MShop\Factory::createManager( $this->context, 'product' );
-		$typeManager = \Aimeos\MShop\Factory::createManager( $this->context, 'product/type' );
+		$manager = \Aimeos\MShop\Factory::createManager($this->context, 'product');
+		$typeManager = \Aimeos\MShop\Factory::createManager($this->context, 'product/type');
 
 		$search = $typeManager->createSearch();
-		$search->setSlice( 0, 1 );
-		$typeItems = $typeManager->searchItems( $search );
+		$search->setSlice(0, 1);
+		$typeItems = $typeManager->searchItems($search);
 
-		if( ( $typeItem = reset( $typeItems ) ) === false ) {
-			throw new \Exception( 'No product type item found' );
+		if (($typeItem = reset($typeItems)) === false) {
+			throw new \Exception('No product type item found');
 		}
 
 
@@ -235,42 +235,42 @@ 
 				'product.datestart' => null,
 				'product.dateend' => null,
 				'config' => array(
-					'key' => array( 0 => 'test key' ),
-					'val' => array( 0 => 'test value' ),
+					'key' => array(0 => 'test key'),
+					'val' => array(0 => 'test value'),
 				),
 			),
 		);
 
-		$helper = new \Aimeos\MW\View\Helper\Param\Standard( $this->view, $param );
-		$this->view->addHelper( 'param', $helper );
+		$helper = new \Aimeos\MW\View\Helper\Param\Standard($this->view, $param);
+		$this->view->addHelper('param', $helper);
 
 		$this->object->save();
 
-		$manager->deleteItem( $manager->findItem( 'test' )->getId() );
+		$manager->deleteItem($manager->findItem('test')->getId());
 	}
 
 
 	public function testSaveException()
 	{
-		$object = $this->getMockBuilder( '\Aimeos\Admin\JQAdm\Product\Standard' )
-			->setConstructorArgs( array( $this->context, \TestHelperJqadm::getTemplatePaths() ) )
-			->setMethods( array( 'getSubClients' ) )
+		$object = $this->getMockBuilder('\Aimeos\Admin\JQAdm\Product\Standard')
+			->setConstructorArgs(array($this->context, \TestHelperJqadm::getTemplatePaths()))
+			->setMethods(array('getSubClients'))
 			->getMock();
 
 		$name = 'AdminJQAdmStandard';
-		$this->context->getConfig()->set( 'mshop/product/manager/name', $name );
+		$this->context->getConfig()->set('mshop/product/manager/name', $name);
 
-		$mock = $this->getMockBuilder( '\Aimeos\MShop\Product\Manager\Standard' )
-			->setConstructorArgs( array( $this->context ) )
-			->setMethods( array( 'createItem' ) )
+		$mock = $this->getMockBuilder('\Aimeos\MShop\Product\Manager\Standard')
+			->setConstructorArgs(array($this->context))
+			->setMethods(array('createItem'))
 			->getMock();
 
-		$mock->expects( $this->exactly( 2 ) )->method( 'createItem' )
-			->will( $this->throwException( new \Exception() ) );
+		$mock->expects($this->exactly(2))->method('createItem')
+			->will($this->throwException(new \Exception()));
 
-		\Aimeos\MShop\Product\Manager\Factory::injectManager( '\\Aimeos\\MShop\\Product\\Manager\\' . $name, $mock );
+		\Aimeos\MShop\Product\Manager\Factory::injectManager('\\Aimeos\\MShop\\Product\\Manager\\' . $name, $mock);
 
-		$object->setView( $this->getViewNoRender() );
+		$object->setView($this->getViewNoRender());
 
 		$object->save();
 	}
@@ -278,25 +278,25 @@ 
 
 	public function testSaveMShopException()
 	{
-		$object = $this->getMockBuilder( '\Aimeos\Admin\JQAdm\Product\Standard' )
-			->setConstructorArgs( array( $this->context, \TestHelperJqadm::getTemplatePaths() ) )
-			->setMethods( array( 'getSubClients' ) )
+		$object = $this->getMockBuilder('\Aimeos\Admin\JQAdm\Product\Standard')
+			->setConstructorArgs(array($this->context, \TestHelperJqadm::getTemplatePaths()))
+			->setMethods(array('getSubClients'))
 			->getMock();
 
 		$name = 'AdminJQAdmStandard';
-		$this->context->getConfig()->set( 'mshop/product/manager/name', $name );
+		$this->context->getConfig()->set('mshop/product/manager/name', $name);
 
-		$mock = $this->getMockBuilder( '\Aimeos\MShop\Product\Manager\Standard' )
-			->setConstructorArgs( array( $this->context ) )
-			->setMethods( array( 'createItem' ) )
+		$mock = $this->getMockBuilder('\Aimeos\MShop\Product\Manager\Standard')
+			->setConstructorArgs(array($this->context))
+			->setMethods(array('createItem'))
 			->getMock();
 
-		$mock->expects( $this->exactly( 2 ) )->method( 'createItem' )
-			->will( $this->throwException( new \Aimeos\MShop\Exception() ) );
+		$mock->expects($this->exactly(2))->method('createItem')
+			->will($this->throwException(new \Aimeos\MShop\Exception()));
 
-		\Aimeos\MShop\Product\Manager\Factory::injectManager( '\\Aimeos\\MShop\\Product\\Manager\\' . $name, $mock );
+		\Aimeos\MShop\Product\Manager\Factory::injectManager('\\Aimeos\\MShop\\Product\\Manager\\' . $name, $mock);
 
-		$object->setView( $this->getViewNoRender() );
+		$object->setView($this->getViewNoRender());
 
 		$object->save();
 	}
@@ -304,25 +304,25 @@ 
 
 	public function testSaveJQAdmException()
 	{
-		$object = $this->getMockBuilder( '\Aimeos\Admin\JQAdm\Product\Standard' )
-			->setConstructorArgs( array( $this->context, \TestHelperJqadm::getTemplatePaths() ) )
-			->setMethods( array( 'getSubClients' ) )
+		$object = $this->getMockBuilder('\Aimeos\Admin\JQAdm\Product\Standard')
+			->setConstructorArgs(array($this->context, \TestHelperJqadm::getTemplatePaths()))
+			->setMethods(array('getSubClients'))
 			->getMock();
 
 		$name = 'AdminJQAdmStandard';
-		$this->context->getConfig()->set( 'mshop/product/manager/name', $name );
+		$this->context->getConfig()->set('mshop/product/manager/name', $name);
 
-		$mock = $this->getMockBuilder( '\Aimeos\MShop\Product\Manager\Standard' )
-			->setConstructorArgs( array( $this->context ) )
-			->setMethods( array( 'createItem' ) )
+		$mock = $this->getMockBuilder('\Aimeos\MShop\Product\Manager\Standard')
+			->setConstructorArgs(array($this->context))
+			->setMethods(array('createItem'))
 			->getMock();
 
-		$mock->expects( $this->exactly( 2 ) )->method( 'createItem' )
-			->will( $this->throwException( new \Aimeos\Admin\JQAdm\Exception() ) );
+		$mock->expects($this->exactly(2))->method('createItem')
+			->will($this->throwException(new \Aimeos\Admin\JQAdm\Exception()));
 
-		\Aimeos\MShop\Product\Manager\Factory::injectManager( '\\Aimeos\\MShop\\Product\\Manager\\' . $name, $mock );
+		\Aimeos\MShop\Product\Manager\Factory::injectManager('\\Aimeos\\MShop\\Product\\Manager\\' . $name, $mock);
 
-		$object->setView( $this->getViewNoRender() );
+		$object->setView($this->getViewNoRender());
 
 		$object->save();
 	}
@@ -333,32 +333,32 @@ 
 		$param = array(
 			'site' => 'unittest', 'lang' => 'de',
 			'filter' => array(
-				'key' => array( 0 => 'product.code' ),
-				'op' => array( 0 => '==' ),
-				'val' => array( 0 => 'CNE' ),
+				'key' => array(0 => 'product.code'),
+				'op' => array(0 => '=='),
+				'val' => array(0 => 'CNE'),
 			),
-			'sort' => array( 'product.label', '-product.id' ),
+			'sort' => array('product.label', '-product.id'),
 		);
-		$helper = new \Aimeos\MW\View\Helper\Param\Standard( $this->view, $param );
-		$this->view->addHelper( 'param', $helper );
+		$helper = new \Aimeos\MW\View\Helper\Param\Standard($this->view, $param);
+		$this->view->addHelper('param', $helper);
 
 		$result = $this->object->search();
 
-		$this->assertContains( '>CNE<', $result );
+		$this->assertContains('>CNE<', $result);
 	}
 
 
 	public function testSearchException()
 	{
-		$object = $this->getMockBuilder( '\Aimeos\Admin\JQAdm\Product\Standard' )
-			->setConstructorArgs( array( $this->context, \TestHelperJqadm::getTemplatePaths() ) )
-			->setMethods( array( 'getSubClients' ) )
+		$object = $this->getMockBuilder('\Aimeos\Admin\JQAdm\Product\Standard')
+			->setConstructorArgs(array($this->context, \TestHelperJqadm::getTemplatePaths()))
+			->setMethods(array('getSubClients'))
 			->getMock();
 
-		$object->expects( $this->once() )->method( 'getSubClients' )
-			->will( $this->throwException( new \Exception() ) );
+		$object->expects($this->once())->method('getSubClients')
+			->will($this->throwException(new \Exception()));
 
-		$object->setView( $this->getViewNoRender() );
+		$object->setView($this->getViewNoRender());
 
 		$object->search();
 	}
@@ -366,15 +366,15 @@ 
 
 	public function testSearchMShopException()
 	{
-		$object = $this->getMockBuilder( '\Aimeos\Admin\JQAdm\Product\Standard' )
-			->setConstructorArgs( array( $this->context, \TestHelperJqadm::getTemplatePaths() ) )
-			->setMethods( array( 'getSubClients' ) )
+		$object = $this->getMockBuilder('\Aimeos\Admin\JQAdm\Product\Standard')
+			->setConstructorArgs(array($this->context, \TestHelperJqadm::getTemplatePaths()))
+			->setMethods(array('getSubClients'))
 			->getMock();
 
-		$object->expects( $this->once() )->method( 'getSubClients' )
-			->will( $this->throwException( new \Aimeos\MShop\Exception() ) );
+		$object->expects($this->once())->method('getSubClients')
+			->will($this->throwException(new \Aimeos\MShop\Exception()));
 
-		$object->setView( $this->getViewNoRender() );
+		$object->setView($this->getViewNoRender());
 
 		$object->search();
 	}
@@ -382,48 +382,48 @@ 
 
 	public function testGetSubClient()
 	{
-		$result = $this->object->getSubClient( 'image' );
-		$this->assertInstanceOf( '\Aimeos\Admin\JQAdm\Iface', $result );
+		$result = $this->object->getSubClient('image');
+		$this->assertInstanceOf('\Aimeos\Admin\JQAdm\Iface', $result);
 	}
 
 
 	public function testGetSubClientEmpty()
 	{
-		$this->setExpectedException( '\Aimeos\Admin\JQAdm\Exception' );
-		$this->object->getSubClient( '' );
+		$this->setExpectedException('\Aimeos\Admin\JQAdm\Exception');
+		$this->object->getSubClient('');
 	}
 
 
 	public function testGetSubClientInvalid()
 	{
-		$this->setExpectedException( '\Aimeos\Admin\JQAdm\Exception' );
-		$this->object->getSubClient( 'invalid' );
+		$this->setExpectedException('\Aimeos\Admin\JQAdm\Exception');
+		$this->object->getSubClient('invalid');
 	}
 
 
 	public function testGetSubClientDecorators()
 	{
-		$this->context->getConfig()->set( 'admin/jqadm/product/image/decorators/global', array( 'Cache' ) );
+		$this->context->getConfig()->set('admin/jqadm/product/image/decorators/global', array('Cache'));
 
-		$result = $this->object->getSubClient( 'image' );
-		$this->assertInstanceOf( '\Aimeos\Admin\JQAdm\Iface', $result );
+		$result = $this->object->getSubClient('image');
+		$this->assertInstanceOf('\Aimeos\Admin\JQAdm\Iface', $result);
 	}
 
 
 	public function testGetSubClientDecoratorInvalid()
 	{
-		$this->context->getConfig()->set( 'admin/jqadm/product/image/decorators/global', array( 'Invalid' ) );
+		$this->context->getConfig()->set('admin/jqadm/product/image/decorators/global', array('Invalid'));
 
-		$this->setExpectedException( '\Aimeos\Admin\JQAdm\Exception' );
-		$this->object->getSubClient( 'image' );
+		$this->setExpectedException('\Aimeos\Admin\JQAdm\Exception');
+		$this->object->getSubClient('image');
 	}
 
 
 	protected function getViewNoRender()
 	{
-		return $this->getMockBuilder( '\Aimeos\MW\View\Standard' )
-			->setConstructorArgs( array( array() ) )
-			->setMethods( array( 'render', 'config' ) )
+		return $this->getMockBuilder('\Aimeos\MW\View\Standard')
+			->setConstructorArgs(array(array()))
+			->setMethods(array('render', 'config'))
 			->getMock();
 	}
 }

admin/jqadm/tests/Admin/JQAdm/Dashboard/StandardTest.php

Spacing +29 added lines, -29 removed lines

@@ -19,55 +19,55 @@ 
 	protected function setUp()
 	{
 		$this->view = \TestHelperJqadm::getView();
-		$request = $this->getMockBuilder( '\Psr\Http\Message\ServerRequestInterface' )->getMock();
-		$helper = new \Aimeos\MW\View\Helper\Request\Standard( $this->view, $request, '127.0.0.1', 'test' );
-		$this->view ->addHelper( 'request', $helper );
+		$request = $this->getMockBuilder('\Psr\Http\Message\ServerRequestInterface')->getMock();
+		$helper = new \Aimeos\MW\View\Helper\Request\Standard($this->view, $request, '127.0.0.1', 'test');
+		$this->view ->addHelper('request', $helper);
 
 		$this->context = \TestHelperJqadm::getContext();
 		$templatePaths = \TestHelperJqadm::getTemplatePaths();
 
-		$this->object = new \Aimeos\Admin\JQAdm\Dashboard\Standard( $this->context, $templatePaths );
-		$this->object->setView( $this->view );
+		$this->object = new \Aimeos\Admin\JQAdm\Dashboard\Standard($this->context, $templatePaths);
+		$this->object->setView($this->view);
 	}
 
 
 	protected function tearDown()
 	{
-		unset( $this->object );
+		unset($this->object);
 	}
 
 
 	public function testCreate()
 	{
-		$this->setExpectedException( '\Aimeos\Admin\JQAdm\Exception' );
+		$this->setExpectedException('\Aimeos\Admin\JQAdm\Exception');
 		$this->object->create();
 	}
 
 
 	public function testCopy()
 	{
-		$this->setExpectedException( '\Aimeos\Admin\JQAdm\Exception' );
+		$this->setExpectedException('\Aimeos\Admin\JQAdm\Exception');
 		$this->object->copy();
 	}
 
 
 	public function testDelete()
 	{
-		$this->setExpectedException( '\Aimeos\Admin\JQAdm\Exception' );
+		$this->setExpectedException('\Aimeos\Admin\JQAdm\Exception');
 		$this->object->delete();
 	}
 
 
 	public function testGet()
 	{
-		$this->setExpectedException( '\Aimeos\Admin\JQAdm\Exception' );
+		$this->setExpectedException('\Aimeos\Admin\JQAdm\Exception');
 		$this->object->get();
 	}
 
 
 	public function testSave()
 	{
-		$this->setExpectedException( '\Aimeos\Admin\JQAdm\Exception' );
+		$this->setExpectedException('\Aimeos\Admin\JQAdm\Exception');
 		$this->object->save();
 	}
 
@@ -76,21 +76,21 @@ 
 	{
 		$result = $this->object->search();
 
-		$this->assertContains( '<div class="dashboard">', $result );
+		$this->assertContains('<div class="dashboard">', $result);
 	}
 
 
 	public function testSearchException()
 	{
-		$object = $this->getMockBuilder( '\Aimeos\Admin\JQAdm\Dashboard\Standard' )
-			->setConstructorArgs( array( $this->context, \TestHelperJqadm::getTemplatePaths() ) )
-			->setMethods( array( 'getSubClients' ) )
+		$object = $this->getMockBuilder('\Aimeos\Admin\JQAdm\Dashboard\Standard')
+			->setConstructorArgs(array($this->context, \TestHelperJqadm::getTemplatePaths()))
+			->setMethods(array('getSubClients'))
 			->getMock();
 
-		$object->expects( $this->once() )->method( 'getSubClients' )
-			->will( $this->throwException( new \Exception() ) );
+		$object->expects($this->once())->method('getSubClients')
+			->will($this->throwException(new \Exception()));
 
-		$object->setView( $this->getViewNoRender() );
+		$object->setView($this->getViewNoRender());
 
 		$object->search();
 	}
@@ -98,15 +98,15 @@ 
 
 	public function testSearchMShopException()
 	{
-		$object = $this->getMockBuilder( '\Aimeos\Admin\JQAdm\Dashboard\Standard' )
-			->setConstructorArgs( array( $this->context, \TestHelperJqadm::getTemplatePaths() ) )
-			->setMethods( array( 'getSubClients' ) )
+		$object = $this->getMockBuilder('\Aimeos\Admin\JQAdm\Dashboard\Standard')
+			->setConstructorArgs(array($this->context, \TestHelperJqadm::getTemplatePaths()))
+			->setMethods(array('getSubClients'))
 			->getMock();
 
-		$object->expects( $this->once() )->method( 'getSubClients' )
-			->will( $this->throwException( new \Aimeos\MShop\Exception() ) );
+		$object->expects($this->once())->method('getSubClients')
+			->will($this->throwException(new \Aimeos\MShop\Exception()));
 
-		$object->setView( $this->getViewNoRender() );
+		$object->setView($this->getViewNoRender());
 
 		$object->search();
 	}
@@ -114,16 +114,16 @@ 
 
 	public function testGetSubClient()
 	{
-		$result = $this->object->getSubClient( 'order' );
-		$this->assertInstanceOf( '\Aimeos\Admin\JQAdm\Iface', $result );
+		$result = $this->object->getSubClient('order');
+		$this->assertInstanceOf('\Aimeos\Admin\JQAdm\Iface', $result);
 	}
 
 
 	protected function getViewNoRender()
 	{
-		return $this->getMockBuilder( '\Aimeos\MW\View\Standard' )
-			->setConstructorArgs( array( array() ) )
-			->setMethods( array( 'render', 'config' ) )
+		return $this->getMockBuilder('\Aimeos\MW\View\Standard')
+			->setConstructorArgs(array(array()))
+			->setMethods(array('render', 'config'))
 			->getMock();
 	}
 }