aimeos / ai-admin-jqadm

admin/jqadm/tests/Admin/JQAdm/Type/Text/Lists/StandardTest.php

Spacing +120 added lines, -120 removed lines

@@ -21,16 +21,16 @@ 
 		$this->view = \TestHelperJqadm::getView();
 		$this->context = \TestHelperJqadm::getContext();
 
-		$this->object = new \Aimeos\Admin\JQAdm\Type\Text\Lists\Standard( $this->context );
-		$this->object = new \Aimeos\Admin\JQAdm\Common\Decorator\Page( $this->object, $this->context );
-		$this->object->setAimeos( \TestHelperJqadm::getAimeos() );
-		$this->object->setView( $this->view );
+		$this->object = new \Aimeos\Admin\JQAdm\Type\Text\Lists\Standard($this->context);
+		$this->object = new \Aimeos\Admin\JQAdm\Common\Decorator\Page($this->object, $this->context);
+		$this->object->setAimeos(\TestHelperJqadm::getAimeos());
+		$this->object->setView($this->view);
 	}
 
 
 	protected function tearDown()
 	{
-		unset( $this->object, $this->view, $this->context );
+		unset($this->object, $this->view, $this->context);
 	}
 
 
@@ -42,15 +42,15 @@ 
 
 	public function testCreateException()
 	{
-		$object = $this->getMockBuilder( \Aimeos\Admin\JQAdm\Type\Text\Lists\Standard::class )
-			->setConstructorArgs( array( $this->context, \TestHelperJqadm::getTemplatePaths() ) )
-			->setMethods( array( 'getSubClients' ) )
+		$object = $this->getMockBuilder(\Aimeos\Admin\JQAdm\Type\Text\Lists\Standard::class)
+			->setConstructorArgs(array($this->context, \TestHelperJqadm::getTemplatePaths()))
+			->setMethods(array('getSubClients'))
 			->getMock();
 
-		$object->expects( $this->once() )->method( 'getSubClients' )
-			->will( $this->throwException( new \RuntimeException() ) );
+		$object->expects($this->once())->method('getSubClients')
+			->will($this->throwException(new \RuntimeException()));
 
-		$object->setView( $this->getViewNoRender() );
+		$object->setView($this->getViewNoRender());
 
 		$object->create();
 	}
@@ -58,15 +58,15 @@ 
 
 	public function testCreateMShopException()
 	{
-		$object = $this->getMockBuilder( \Aimeos\Admin\JQAdm\Type\Text\Lists\Standard::class )
-			->setConstructorArgs( array( $this->context, \TestHelperJqadm::getTemplatePaths() ) )
-			->setMethods( array( 'getSubClients' ) )
+		$object = $this->getMockBuilder(\Aimeos\Admin\JQAdm\Type\Text\Lists\Standard::class)
+			->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();
 	}
@@ -74,29 +74,29 @@ 
 
 	public function testCopy()
 	{
-		$manager = \Aimeos\MShop::create( $this->context, 'text/lists/type' );
+		$manager = \Aimeos\MShop::create($this->context, 'text/lists/type');
 
-		$param = ['type' => 'unittest', 'id' => $manager->findItem( 'default', [], 'product' )->getId()];
-		$helper = new \Aimeos\MW\View\Helper\Param\Standard( $this->view, $param );
-		$this->view->addHelper( 'param', $helper );
+		$param = ['type' => 'unittest', 'id' => $manager->findItem('default', [], 'product')->getId()];
+		$helper = new \Aimeos\MW\View\Helper\Param\Standard($this->view, $param);
+		$this->view->addHelper('param', $helper);
 
 		$result = $this->object->copy();
 
-		$this->assertContains( 'default', $result );
+		$this->assertContains('default', $result);
 	}
 
 
 	public function testCopyException()
 	{
-		$object = $this->getMockBuilder( \Aimeos\Admin\JQAdm\Type\Text\Lists\Standard::class )
-			->setConstructorArgs( array( $this->context, \TestHelperJqadm::getTemplatePaths() ) )
-			->setMethods( array( 'getSubClients' ) )
+		$object = $this->getMockBuilder(\Aimeos\Admin\JQAdm\Type\Text\Lists\Standard::class)
+			->setConstructorArgs(array($this->context, \TestHelperJqadm::getTemplatePaths()))
+			->setMethods(array('getSubClients'))
 			->getMock();
 
-		$object->expects( $this->once() )->method( 'getSubClients' )
-			->will( $this->throwException( new \RuntimeException() ) );
+		$object->expects($this->once())->method('getSubClients')
+			->will($this->throwException(new \RuntimeException()));
 
-		$object->setView( $this->getViewNoRender() );
+		$object->setView($this->getViewNoRender());
 
 		$object->copy();
 	}
@@ -104,15 +104,15 @@ 
 
 	public function testCopyMShopException()
 	{
-		$object = $this->getMockBuilder( \Aimeos\Admin\JQAdm\Type\Text\Lists\Standard::class )
-			->setConstructorArgs( array( $this->context, \TestHelperJqadm::getTemplatePaths() ) )
-			->setMethods( array( 'getSubClients' ) )
+		$object = $this->getMockBuilder(\Aimeos\Admin\JQAdm\Type\Text\Lists\Standard::class)
+			->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();
 	}
@@ -120,21 +120,21 @@ 
 
 	public function testDelete()
 	{
-		$this->assertNotNull( $this->object->delete() );
+		$this->assertNotNull($this->object->delete());
 	}
 
 
 	public function testDeleteException()
 	{
-		$object = $this->getMockBuilder( \Aimeos\Admin\JQAdm\Type\Text\Lists\Standard::class )
-			->setConstructorArgs( array( $this->context, \TestHelperJqadm::getTemplatePaths() ) )
-			->setMethods( array( 'getSubClients', 'search' ) )
+		$object = $this->getMockBuilder(\Aimeos\Admin\JQAdm\Type\Text\Lists\Standard::class)
+			->setConstructorArgs(array($this->context, \TestHelperJqadm::getTemplatePaths()))
+			->setMethods(array('getSubClients', 'search'))
 			->getMock();
 
-		$object->expects( $this->once() )->method( 'getSubClients' )
-			->will( $this->throwException( new \RuntimeException() ) );
+		$object->expects($this->once())->method('getSubClients')
+			->will($this->throwException(new \RuntimeException()));
 
-		$object->setView( $this->getViewNoRender() );
+		$object->setView($this->getViewNoRender());
 
 		$object->delete();
 	}
@@ -142,15 +142,15 @@ 
 
 	public function testDeleteMShopException()
 	{
-		$object = $this->getMockBuilder( \Aimeos\Admin\JQAdm\Type\Text\Lists\Standard::class )
-			->setConstructorArgs( array( $this->context, \TestHelperJqadm::getTemplatePaths() ) )
-			->setMethods( array( 'getSubClients', 'search' ) )
+		$object = $this->getMockBuilder(\Aimeos\Admin\JQAdm\Type\Text\Lists\Standard::class)
+			->setConstructorArgs(array($this->context, \TestHelperJqadm::getTemplatePaths()))
+			->setMethods(array('getSubClients', 'search'))
 			->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();
 	}
@@ -158,29 +158,29 @@ 
 
 	public function testGet()
 	{
-		$manager = \Aimeos\MShop::create( $this->context, 'text/lists/type' );
+		$manager = \Aimeos\MShop::create($this->context, 'text/lists/type');
 
-		$param = ['type' => 'unittest', 'id' => $manager->findItem( 'default', [], 'product' )->getId()];
-		$helper = new \Aimeos\MW\View\Helper\Param\Standard( $this->view, $param );
-		$this->view->addHelper( 'param', $helper );
+		$param = ['type' => 'unittest', 'id' => $manager->findItem('default', [], 'product')->getId()];
+		$helper = new \Aimeos\MW\View\Helper\Param\Standard($this->view, $param);
+		$this->view->addHelper('param', $helper);
 
 		$result = $this->object->get();
 
-		$this->assertContains( 'default', $result );
+		$this->assertContains('default', $result);
 	}
 
 
 	public function testGetException()
 	{
-		$object = $this->getMockBuilder( \Aimeos\Admin\JQAdm\Type\Text\Lists\Standard::class )
-			->setConstructorArgs( array( $this->context, \TestHelperJqadm::getTemplatePaths() ) )
-			->setMethods( array( 'getSubClients' ) )
+		$object = $this->getMockBuilder(\Aimeos\Admin\JQAdm\Type\Text\Lists\Standard::class)
+			->setConstructorArgs(array($this->context, \TestHelperJqadm::getTemplatePaths()))
+			->setMethods(array('getSubClients'))
 			->getMock();
 
-		$object->expects( $this->once() )->method( 'getSubClients' )
-			->will( $this->throwException( new \RuntimeException() ) );
+		$object->expects($this->once())->method('getSubClients')
+			->will($this->throwException(new \RuntimeException()));
 
-		$object->setView( $this->getViewNoRender() );
+		$object->setView($this->getViewNoRender());
 
 		$object->get();
 	}
@@ -188,15 +188,15 @@ 
 
 	public function testGetMShopException()
 	{
-		$object = $this->getMockBuilder( \Aimeos\Admin\JQAdm\Type\Text\Lists\Standard::class )
-			->setConstructorArgs( array( $this->context, \TestHelperJqadm::getTemplatePaths() ) )
-			->setMethods( array( 'getSubClients' ) )
+		$object = $this->getMockBuilder(\Aimeos\Admin\JQAdm\Type\Text\Lists\Standard::class)
+			->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();
 	}
@@ -204,16 +204,16 @@ 
 
 	public function testGetViewException()
 	{
-		$object = new \Aimeos\Admin\JQAdm\Type\Text\Lists\Standard( $this->context, [] );
+		$object = new \Aimeos\Admin\JQAdm\Type\Text\Lists\Standard($this->context, []);
 
-		$this->setExpectedException( \Aimeos\Admin\JQAdm\Exception::class );
+		$this->setExpectedException(\Aimeos\Admin\JQAdm\Exception::class);
 		$object->getView();
 	}
 
 
 	public function testSave()
 	{
-		$manager = \Aimeos\MShop::create( $this->context, 'text/lists/type' );
+		$manager = \Aimeos\MShop::create($this->context, 'text/lists/type');
 
 		$param = array(
 			'type' => 'unittest',
@@ -226,26 +226,26 @@ 
 			),
 		);
 
-		$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( 'jqadm@test', [], 'product' )->getId() );
+		$manager->deleteItem($manager->findItem('jqadm@test', [], 'product')->getId());
 	}
 
 
 	public function testSaveException()
 	{
-		$object = $this->getMockBuilder( \Aimeos\Admin\JQAdm\Type\Text\Lists\Standard::class )
-			->setConstructorArgs( array( $this->context, \TestHelperJqadm::getTemplatePaths() ) )
-			->setMethods( array( 'fromArray' ) )
+		$object = $this->getMockBuilder(\Aimeos\Admin\JQAdm\Type\Text\Lists\Standard::class)
+			->setConstructorArgs(array($this->context, \TestHelperJqadm::getTemplatePaths()))
+			->setMethods(array('fromArray'))
 			->getMock();
 
-		$object->expects( $this->once() )->method( 'fromArray' )
-			->will( $this->throwException( new \RuntimeException() ) );
+		$object->expects($this->once())->method('fromArray')
+			->will($this->throwException(new \RuntimeException()));
 
-		$object->setView( $this->getViewNoRender() );
+		$object->setView($this->getViewNoRender());
 
 		$object->save();
 	}
@@ -253,15 +253,15 @@ 
 
 	public function testSaveMShopException()
 	{
-		$object = $this->getMockBuilder( \Aimeos\Admin\JQAdm\Type\Text\Lists\Standard::class )
-			->setConstructorArgs( array( $this->context, \TestHelperJqadm::getTemplatePaths() ) )
-			->setMethods( array( 'fromArray' ) )
+		$object = $this->getMockBuilder(\Aimeos\Admin\JQAdm\Type\Text\Lists\Standard::class)
+			->setConstructorArgs(array($this->context, \TestHelperJqadm::getTemplatePaths()))
+			->setMethods(array('fromArray'))
 			->getMock();
 
-		$object->expects( $this->once() )->method( 'fromArray' )
-			->will( $this->throwException( new \RuntimeException() ) );
+		$object->expects($this->once())->method('fromArray')
+			->will($this->throwException(new \RuntimeException()));
 
-		$object->setView( $this->getViewNoRender() );
+		$object->setView($this->getViewNoRender());
 
 		$object->save();
 	}
@@ -269,15 +269,15 @@ 
 
 	public function testSaveJQAdmException()
 	{
-		$object = $this->getMockBuilder( \Aimeos\Admin\JQAdm\Type\Text\Lists\Standard::class )
-			->setConstructorArgs( array( $this->context, \TestHelperJqadm::getTemplatePaths() ) )
-			->setMethods( array( 'fromArray' ) )
+		$object = $this->getMockBuilder(\Aimeos\Admin\JQAdm\Type\Text\Lists\Standard::class)
+			->setConstructorArgs(array($this->context, \TestHelperJqadm::getTemplatePaths()))
+			->setMethods(array('fromArray'))
 			->getMock();
 
-		$object->expects( $this->once() )->method( 'fromArray' )
-			->will( $this->throwException( new \RuntimeException() ) );
+		$object->expects($this->once())->method('fromArray')
+			->will($this->throwException(new \RuntimeException()));
 
-		$object->setView( $this->getViewNoRender() );
+		$object->setView($this->getViewNoRender());
 
 		$object->save();
 	}
@@ -288,32 +288,32 @@ 
 		$param = array(
 			'type' => 'unittest', 'lang' => 'de',
 			'filter' => array(
-				'key' => array( 0 => 'text.lists.type.code' ),
-				'op' => array( 0 => '==' ),
-				'val' => array( 0 => 'default' ),
+				'key' => array(0 => 'text.lists.type.code'),
+				'op' => array(0 => '=='),
+				'val' => array(0 => 'default'),
 			),
-			'sort' => array( '-text.lists.type.id' ),
+			'sort' => array('-text.lists.type.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( '>default<', $result );
+		$this->assertContains('>default<', $result);
 	}
 
 
 	public function testSearchException()
 	{
-		$object = $this->getMockBuilder( \Aimeos\Admin\JQAdm\Type\Text\Lists\Standard::class )
-			->setConstructorArgs( array( $this->context, \TestHelperJqadm::getTemplatePaths() ) )
-			->setMethods( array( 'initCriteria' ) )
+		$object = $this->getMockBuilder(\Aimeos\Admin\JQAdm\Type\Text\Lists\Standard::class)
+			->setConstructorArgs(array($this->context, \TestHelperJqadm::getTemplatePaths()))
+			->setMethods(array('initCriteria'))
 			->getMock();
 
-		$object->expects( $this->once() )->method( 'initCriteria' )
-			->will( $this->throwException( new \RuntimeException() ) );
+		$object->expects($this->once())->method('initCriteria')
+			->will($this->throwException(new \RuntimeException()));
 
-		$object->setView( $this->getViewNoRender() );
+		$object->setView($this->getViewNoRender());
 
 		$object->search();
 	}
@@ -321,15 +321,15 @@ 
 
 	public function testSearchMShopException()
 	{
-		$object = $this->getMockBuilder( \Aimeos\Admin\JQAdm\Type\Text\Lists\Standard::class )
-			->setConstructorArgs( array( $this->context, \TestHelperJqadm::getTemplatePaths() ) )
-			->setMethods( array( 'initCriteria' ) )
+		$object = $this->getMockBuilder(\Aimeos\Admin\JQAdm\Type\Text\Lists\Standard::class)
+			->setConstructorArgs(array($this->context, \TestHelperJqadm::getTemplatePaths()))
+			->setMethods(array('initCriteria'))
 			->getMock();
 
-		$object->expects( $this->once() )->method( 'initCriteria' )
-			->will( $this->throwException( new \Aimeos\MShop\Exception() ) );
+		$object->expects($this->once())->method('initCriteria')
+			->will($this->throwException(new \Aimeos\MShop\Exception()));
 
-		$object->setView( $this->getViewNoRender() );
+		$object->setView($this->getViewNoRender());
 
 		$object->search();
 	}
@@ -337,33 +337,33 @@ 
 
 	public function testGetSubClientInvalid()
 	{
-		$this->setExpectedException( \Aimeos\Admin\JQAdm\Exception::class );
-		$this->object->getSubClient( '$unknown$' );
+		$this->setExpectedException(\Aimeos\Admin\JQAdm\Exception::class);
+		$this->object->getSubClient('$unknown$');
 	}
 
 
 	public function testGetSubClientUnknown()
 	{
-		$this->setExpectedException( \Aimeos\Admin\JQAdm\Exception::class );
-		$this->object->getSubClient( 'unknown' );
+		$this->setExpectedException(\Aimeos\Admin\JQAdm\Exception::class);
+		$this->object->getSubClient('unknown');
 	}
 
 
 	protected function getViewNoRender()
 	{
-		$view = $this->getMockBuilder( \Aimeos\MW\View\Standard::class )
-			->setConstructorArgs( array( [] ) )
-			->setMethods( array( 'render', 'config' ) )
+		$view = $this->getMockBuilder(\Aimeos\MW\View\Standard::class)
+			->setConstructorArgs(array([]))
+			->setMethods(array('render', 'config'))
 			->getMock();
 
-		$manager = \Aimeos\MShop::create( $this->context, 'text/lists/type' );
+		$manager = \Aimeos\MShop::create($this->context, 'text/lists/type');
 
-		$param = ['type' => 'unittest', 'id' => $manager->findItem( 'default', [], 'product' )->getId()];
-		$helper = new \Aimeos\MW\View\Helper\Param\Standard( $view, $param );
-		$view->addHelper( 'param', $helper );
+		$param = ['type' => 'unittest', 'id' => $manager->findItem('default', [], 'product')->getId()];
+		$helper = new \Aimeos\MW\View\Helper\Param\Standard($view, $param);
+		$view->addHelper('param', $helper);
 
-		$helper = new \Aimeos\MW\View\Helper\Access\Standard( $view, [] );
-		$view->addHelper( 'access', $helper );
+		$helper = new \Aimeos\MW\View\Helper\Access\Standard($view, []);
+		$view->addHelper('access', $helper);
 
 		return $view;
 	}

admin/jqadm/tests/Admin/JQAdm/Type/Price/StandardTest.php

Spacing +120 added lines, -120 removed lines

@@ -21,16 +21,16 @@ 
 		$this->view = \TestHelperJqadm::getView();
 		$this->context = \TestHelperJqadm::getContext();
 
-		$this->object = new \Aimeos\Admin\JQAdm\Type\Price\Standard( $this->context );
-		$this->object = new \Aimeos\Admin\JQAdm\Common\Decorator\Page( $this->object, $this->context );
-		$this->object->setAimeos( \TestHelperJqadm::getAimeos() );
-		$this->object->setView( $this->view );
+		$this->object = new \Aimeos\Admin\JQAdm\Type\Price\Standard($this->context);
+		$this->object = new \Aimeos\Admin\JQAdm\Common\Decorator\Page($this->object, $this->context);
+		$this->object->setAimeos(\TestHelperJqadm::getAimeos());
+		$this->object->setView($this->view);
 	}
 
 
 	protected function tearDown()
 	{
-		unset( $this->object, $this->view, $this->context );
+		unset($this->object, $this->view, $this->context);
 	}
 
 
@@ -42,15 +42,15 @@ 
 
 	public function testCreateException()
 	{
-		$object = $this->getMockBuilder( \Aimeos\Admin\JQAdm\Type\Price\Standard::class )
-			->setConstructorArgs( array( $this->context, \TestHelperJqadm::getTemplatePaths() ) )
-			->setMethods( array( 'getSubClients' ) )
+		$object = $this->getMockBuilder(\Aimeos\Admin\JQAdm\Type\Price\Standard::class)
+			->setConstructorArgs(array($this->context, \TestHelperJqadm::getTemplatePaths()))
+			->setMethods(array('getSubClients'))
 			->getMock();
 
-		$object->expects( $this->once() )->method( 'getSubClients' )
-			->will( $this->throwException( new \RuntimeException() ) );
+		$object->expects($this->once())->method('getSubClients')
+			->will($this->throwException(new \RuntimeException()));
 
-		$object->setView( $this->getViewNoRender() );
+		$object->setView($this->getViewNoRender());
 
 		$object->create();
 	}
@@ -58,15 +58,15 @@ 
 
 	public function testCreateMShopException()
 	{
-		$object = $this->getMockBuilder( \Aimeos\Admin\JQAdm\Type\Price\Standard::class )
-			->setConstructorArgs( array( $this->context, \TestHelperJqadm::getTemplatePaths() ) )
-			->setMethods( array( 'getSubClients' ) )
+		$object = $this->getMockBuilder(\Aimeos\Admin\JQAdm\Type\Price\Standard::class)
+			->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();
 	}
@@ -74,29 +74,29 @@ 
 
 	public function testCopy()
 	{
-		$manager = \Aimeos\MShop::create( $this->context, 'price/type' );
+		$manager = \Aimeos\MShop::create($this->context, 'price/type');
 
-		$param = ['type' => 'unittest', 'id' => $manager->findItem( 'default', [], 'product' )->getId()];
-		$helper = new \Aimeos\MW\View\Helper\Param\Standard( $this->view, $param );
-		$this->view->addHelper( 'param', $helper );
+		$param = ['type' => 'unittest', 'id' => $manager->findItem('default', [], 'product')->getId()];
+		$helper = new \Aimeos\MW\View\Helper\Param\Standard($this->view, $param);
+		$this->view->addHelper('param', $helper);
 
 		$result = $this->object->copy();
 
-		$this->assertContains( 'default', $result );
+		$this->assertContains('default', $result);
 	}
 
 
 	public function testCopyException()
 	{
-		$object = $this->getMockBuilder( \Aimeos\Admin\JQAdm\Type\Price\Standard::class )
-			->setConstructorArgs( array( $this->context, \TestHelperJqadm::getTemplatePaths() ) )
-			->setMethods( array( 'getSubClients' ) )
+		$object = $this->getMockBuilder(\Aimeos\Admin\JQAdm\Type\Price\Standard::class)
+			->setConstructorArgs(array($this->context, \TestHelperJqadm::getTemplatePaths()))
+			->setMethods(array('getSubClients'))
 			->getMock();
 
-		$object->expects( $this->once() )->method( 'getSubClients' )
-			->will( $this->throwException( new \RuntimeException() ) );
+		$object->expects($this->once())->method('getSubClients')
+			->will($this->throwException(new \RuntimeException()));
 
-		$object->setView( $this->getViewNoRender() );
+		$object->setView($this->getViewNoRender());
 
 		$object->copy();
 	}
@@ -104,15 +104,15 @@ 
 
 	public function testCopyMShopException()
 	{
-		$object = $this->getMockBuilder( \Aimeos\Admin\JQAdm\Type\Price\Standard::class )
-			->setConstructorArgs( array( $this->context, \TestHelperJqadm::getTemplatePaths() ) )
-			->setMethods( array( 'getSubClients' ) )
+		$object = $this->getMockBuilder(\Aimeos\Admin\JQAdm\Type\Price\Standard::class)
+			->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();
 	}
@@ -120,21 +120,21 @@ 
 
 	public function testDelete()
 	{
-		$this->assertNotNull( $this->object->delete() );
+		$this->assertNotNull($this->object->delete());
 	}
 
 
 	public function testDeleteException()
 	{
-		$object = $this->getMockBuilder( \Aimeos\Admin\JQAdm\Type\Price\Standard::class )
-			->setConstructorArgs( array( $this->context, \TestHelperJqadm::getTemplatePaths() ) )
-			->setMethods( array( 'getSubClients', 'search' ) )
+		$object = $this->getMockBuilder(\Aimeos\Admin\JQAdm\Type\Price\Standard::class)
+			->setConstructorArgs(array($this->context, \TestHelperJqadm::getTemplatePaths()))
+			->setMethods(array('getSubClients', 'search'))
 			->getMock();
 
-		$object->expects( $this->once() )->method( 'getSubClients' )
-			->will( $this->throwException( new \RuntimeException() ) );
+		$object->expects($this->once())->method('getSubClients')
+			->will($this->throwException(new \RuntimeException()));
 
-		$object->setView( $this->getViewNoRender() );
+		$object->setView($this->getViewNoRender());
 
 		$object->delete();
 	}
@@ -142,15 +142,15 @@ 
 
 	public function testDeleteMShopException()
 	{
-		$object = $this->getMockBuilder( \Aimeos\Admin\JQAdm\Type\Price\Standard::class )
-			->setConstructorArgs( array( $this->context, \TestHelperJqadm::getTemplatePaths() ) )
-			->setMethods( array( 'getSubClients', 'search' ) )
+		$object = $this->getMockBuilder(\Aimeos\Admin\JQAdm\Type\Price\Standard::class)
+			->setConstructorArgs(array($this->context, \TestHelperJqadm::getTemplatePaths()))
+			->setMethods(array('getSubClients', 'search'))
 			->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();
 	}
@@ -158,29 +158,29 @@ 
 
 	public function testGet()
 	{
-		$manager = \Aimeos\MShop::create( $this->context, 'price/type' );
+		$manager = \Aimeos\MShop::create($this->context, 'price/type');
 
-		$param = ['type' => 'unittest', 'id' => $manager->findItem( 'default', [], 'product' )->getId()];
-		$helper = new \Aimeos\MW\View\Helper\Param\Standard( $this->view, $param );
-		$this->view->addHelper( 'param', $helper );
+		$param = ['type' => 'unittest', 'id' => $manager->findItem('default', [], 'product')->getId()];
+		$helper = new \Aimeos\MW\View\Helper\Param\Standard($this->view, $param);
+		$this->view->addHelper('param', $helper);
 
 		$result = $this->object->get();
 
-		$this->assertContains( 'default', $result );
+		$this->assertContains('default', $result);
 	}
 
 
 	public function testGetException()
 	{
-		$object = $this->getMockBuilder( \Aimeos\Admin\JQAdm\Type\Price\Standard::class )
-			->setConstructorArgs( array( $this->context, \TestHelperJqadm::getTemplatePaths() ) )
-			->setMethods( array( 'getSubClients' ) )
+		$object = $this->getMockBuilder(\Aimeos\Admin\JQAdm\Type\Price\Standard::class)
+			->setConstructorArgs(array($this->context, \TestHelperJqadm::getTemplatePaths()))
+			->setMethods(array('getSubClients'))
 			->getMock();
 
-		$object->expects( $this->once() )->method( 'getSubClients' )
-			->will( $this->throwException( new \RuntimeException() ) );
+		$object->expects($this->once())->method('getSubClients')
+			->will($this->throwException(new \RuntimeException()));
 
-		$object->setView( $this->getViewNoRender() );
+		$object->setView($this->getViewNoRender());
 
 		$object->get();
 	}
@@ -188,15 +188,15 @@ 
 
 	public function testGetMShopException()
 	{
-		$object = $this->getMockBuilder( \Aimeos\Admin\JQAdm\Type\Price\Standard::class )
-			->setConstructorArgs( array( $this->context, \TestHelperJqadm::getTemplatePaths() ) )
-			->setMethods( array( 'getSubClients' ) )
+		$object = $this->getMockBuilder(\Aimeos\Admin\JQAdm\Type\Price\Standard::class)
+			->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();
 	}
@@ -204,16 +204,16 @@ 
 
 	public function testGetViewException()
 	{
-		$object = new \Aimeos\Admin\JQAdm\Type\Price\Standard( $this->context, [] );
+		$object = new \Aimeos\Admin\JQAdm\Type\Price\Standard($this->context, []);
 
-		$this->setExpectedException( \Aimeos\Admin\JQAdm\Exception::class );
+		$this->setExpectedException(\Aimeos\Admin\JQAdm\Exception::class);
 		$object->getView();
 	}
 
 
 	public function testSave()
 	{
-		$manager = \Aimeos\MShop::create( $this->context, 'price/type' );
+		$manager = \Aimeos\MShop::create($this->context, 'price/type');
 
 		$param = array(
 			'type' => 'unittest',
@@ -226,26 +226,26 @@ 
 			),
 		);
 
-		$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( 'jqadm@test', [], 'price' )->getId() );
+		$manager->deleteItem($manager->findItem('jqadm@test', [], 'price')->getId());
 	}
 
 
 	public function testSaveException()
 	{
-		$object = $this->getMockBuilder( \Aimeos\Admin\JQAdm\Type\Price\Standard::class )
-			->setConstructorArgs( array( $this->context, \TestHelperJqadm::getTemplatePaths() ) )
-			->setMethods( array( 'fromArray' ) )
+		$object = $this->getMockBuilder(\Aimeos\Admin\JQAdm\Type\Price\Standard::class)
+			->setConstructorArgs(array($this->context, \TestHelperJqadm::getTemplatePaths()))
+			->setMethods(array('fromArray'))
 			->getMock();
 
-		$object->expects( $this->once() )->method( 'fromArray' )
-			->will( $this->throwException( new \RuntimeException() ) );
+		$object->expects($this->once())->method('fromArray')
+			->will($this->throwException(new \RuntimeException()));
 
-		$object->setView( $this->getViewNoRender() );
+		$object->setView($this->getViewNoRender());
 
 		$object->save();
 	}
@@ -253,15 +253,15 @@ 
 
 	public function testSaveMShopException()
 	{
-		$object = $this->getMockBuilder( \Aimeos\Admin\JQAdm\Type\Price\Standard::class )
-			->setConstructorArgs( array( $this->context, \TestHelperJqadm::getTemplatePaths() ) )
-			->setMethods( array( 'fromArray' ) )
+		$object = $this->getMockBuilder(\Aimeos\Admin\JQAdm\Type\Price\Standard::class)
+			->setConstructorArgs(array($this->context, \TestHelperJqadm::getTemplatePaths()))
+			->setMethods(array('fromArray'))
 			->getMock();
 
-		$object->expects( $this->once() )->method( 'fromArray' )
-			->will( $this->throwException( new \RuntimeException() ) );
+		$object->expects($this->once())->method('fromArray')
+			->will($this->throwException(new \RuntimeException()));
 
-		$object->setView( $this->getViewNoRender() );
+		$object->setView($this->getViewNoRender());
 
 		$object->save();
 	}
@@ -269,15 +269,15 @@ 
 
 	public function testSaveJQAdmException()
 	{
-		$object = $this->getMockBuilder( \Aimeos\Admin\JQAdm\Type\Price\Standard::class )
-			->setConstructorArgs( array( $this->context, \TestHelperJqadm::getTemplatePaths() ) )
-			->setMethods( array( 'fromArray' ) )
+		$object = $this->getMockBuilder(\Aimeos\Admin\JQAdm\Type\Price\Standard::class)
+			->setConstructorArgs(array($this->context, \TestHelperJqadm::getTemplatePaths()))
+			->setMethods(array('fromArray'))
 			->getMock();
 
-		$object->expects( $this->once() )->method( 'fromArray' )
-			->will( $this->throwException( new \RuntimeException() ) );
+		$object->expects($this->once())->method('fromArray')
+			->will($this->throwException(new \RuntimeException()));
 
-		$object->setView( $this->getViewNoRender() );
+		$object->setView($this->getViewNoRender());
 
 		$object->save();
 	}
@@ -288,32 +288,32 @@ 
 		$param = array(
 			'type' => 'unittest', 'lang' => 'de',
 			'filter' => array(
-				'key' => array( 0 => 'price.type.code' ),
-				'op' => array( 0 => '==' ),
-				'val' => array( 0 => 'default' ),
+				'key' => array(0 => 'price.type.code'),
+				'op' => array(0 => '=='),
+				'val' => array(0 => 'default'),
 			),
-			'sort' => array( '-price.type.id' ),
+			'sort' => array('-price.type.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( '>default<', $result );
+		$this->assertContains('>default<', $result);
 	}
 
 
 	public function testSearchException()
 	{
-		$object = $this->getMockBuilder( \Aimeos\Admin\JQAdm\Type\Price\Standard::class )
-			->setConstructorArgs( array( $this->context, \TestHelperJqadm::getTemplatePaths() ) )
-			->setMethods( array( 'initCriteria' ) )
+		$object = $this->getMockBuilder(\Aimeos\Admin\JQAdm\Type\Price\Standard::class)
+			->setConstructorArgs(array($this->context, \TestHelperJqadm::getTemplatePaths()))
+			->setMethods(array('initCriteria'))
 			->getMock();
 
-		$object->expects( $this->once() )->method( 'initCriteria' )
-			->will( $this->throwException( new \RuntimeException() ) );
+		$object->expects($this->once())->method('initCriteria')
+			->will($this->throwException(new \RuntimeException()));
 
-		$object->setView( $this->getViewNoRender() );
+		$object->setView($this->getViewNoRender());
 
 		$object->search();
 	}
@@ -321,15 +321,15 @@ 
 
 	public function testSearchMShopException()
 	{
-		$object = $this->getMockBuilder( \Aimeos\Admin\JQAdm\Type\Price\Standard::class )
-			->setConstructorArgs( array( $this->context, \TestHelperJqadm::getTemplatePaths() ) )
-			->setMethods( array( 'initCriteria' ) )
+		$object = $this->getMockBuilder(\Aimeos\Admin\JQAdm\Type\Price\Standard::class)
+			->setConstructorArgs(array($this->context, \TestHelperJqadm::getTemplatePaths()))
+			->setMethods(array('initCriteria'))
 			->getMock();
 
-		$object->expects( $this->once() )->method( 'initCriteria' )
-			->will( $this->throwException( new \Aimeos\MShop\Exception() ) );
+		$object->expects($this->once())->method('initCriteria')
+			->will($this->throwException(new \Aimeos\MShop\Exception()));
 
-		$object->setView( $this->getViewNoRender() );
+		$object->setView($this->getViewNoRender());
 
 		$object->search();
 	}
@@ -337,33 +337,33 @@ 
 
 	public function testGetSubClientInvalid()
 	{
-		$this->setExpectedException( \Aimeos\Admin\JQAdm\Exception::class );
-		$this->object->getSubClient( '$unknown$' );
+		$this->setExpectedException(\Aimeos\Admin\JQAdm\Exception::class);
+		$this->object->getSubClient('$unknown$');
 	}
 
 
 	public function testGetSubClientUnknown()
 	{
-		$this->setExpectedException( \Aimeos\Admin\JQAdm\Exception::class );
-		$this->object->getSubClient( 'unknown' );
+		$this->setExpectedException(\Aimeos\Admin\JQAdm\Exception::class);
+		$this->object->getSubClient('unknown');
 	}
 
 
 	protected function getViewNoRender()
 	{
-		$view = $this->getMockBuilder( \Aimeos\MW\View\Standard::class )
-			->setConstructorArgs( array( [] ) )
-			->setMethods( array( 'render', 'config' ) )
+		$view = $this->getMockBuilder(\Aimeos\MW\View\Standard::class)
+			->setConstructorArgs(array([]))
+			->setMethods(array('render', 'config'))
 			->getMock();
 
-		$manager = \Aimeos\MShop::create( $this->context, 'price/type' );
+		$manager = \Aimeos\MShop::create($this->context, 'price/type');
 
-		$param = ['type' => 'unittest', 'id' => $manager->findItem( 'default', [], 'product' )->getId()];
-		$helper = new \Aimeos\MW\View\Helper\Param\Standard( $view, $param );
-		$view->addHelper( 'param', $helper );
+		$param = ['type' => 'unittest', 'id' => $manager->findItem('default', [], 'product')->getId()];
+		$helper = new \Aimeos\MW\View\Helper\Param\Standard($view, $param);
+		$view->addHelper('param', $helper);
 
-		$helper = new \Aimeos\MW\View\Helper\Access\Standard( $view, [] );
-		$view->addHelper( 'access', $helper );
+		$helper = new \Aimeos\MW\View\Helper\Access\Standard($view, []);
+		$view->addHelper('access', $helper);
 
 		return $view;
 	}

admin/jqadm/tests/Admin/JQAdm/Type/Price/Lists/StandardTest.php

Spacing +120 added lines, -120 removed lines

@@ -21,16 +21,16 @@ 
 		$this->view = \TestHelperJqadm::getView();
 		$this->context = \TestHelperJqadm::getContext();
 
-		$this->object = new \Aimeos\Admin\JQAdm\Type\Price\Lists\Standard( $this->context );
-		$this->object = new \Aimeos\Admin\JQAdm\Common\Decorator\Page( $this->object, $this->context );
-		$this->object->setAimeos( \TestHelperJqadm::getAimeos() );
-		$this->object->setView( $this->view );
+		$this->object = new \Aimeos\Admin\JQAdm\Type\Price\Lists\Standard($this->context);
+		$this->object = new \Aimeos\Admin\JQAdm\Common\Decorator\Page($this->object, $this->context);
+		$this->object->setAimeos(\TestHelperJqadm::getAimeos());
+		$this->object->setView($this->view);
 	}
 
 
 	protected function tearDown()
 	{
-		unset( $this->object, $this->view, $this->context );
+		unset($this->object, $this->view, $this->context);
 	}
 
 
@@ -42,15 +42,15 @@ 
 
 	public function testCreateException()
 	{
-		$object = $this->getMockBuilder( \Aimeos\Admin\JQAdm\Type\Price\Lists\Standard::class )
-			->setConstructorArgs( array( $this->context, \TestHelperJqadm::getTemplatePaths() ) )
-			->setMethods( array( 'getSubClients' ) )
+		$object = $this->getMockBuilder(\Aimeos\Admin\JQAdm\Type\Price\Lists\Standard::class)
+			->setConstructorArgs(array($this->context, \TestHelperJqadm::getTemplatePaths()))
+			->setMethods(array('getSubClients'))
 			->getMock();
 
-		$object->expects( $this->once() )->method( 'getSubClients' )
-			->will( $this->throwException( new \RuntimeException() ) );
+		$object->expects($this->once())->method('getSubClients')
+			->will($this->throwException(new \RuntimeException()));
 
-		$object->setView( $this->getViewNoRender() );
+		$object->setView($this->getViewNoRender());
 
 		$object->create();
 	}
@@ -58,15 +58,15 @@ 
 
 	public function testCreateMShopException()
 	{
-		$object = $this->getMockBuilder( \Aimeos\Admin\JQAdm\Type\Price\Lists\Standard::class )
-			->setConstructorArgs( array( $this->context, \TestHelperJqadm::getTemplatePaths() ) )
-			->setMethods( array( 'getSubClients' ) )
+		$object = $this->getMockBuilder(\Aimeos\Admin\JQAdm\Type\Price\Lists\Standard::class)
+			->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();
 	}
@@ -74,29 +74,29 @@ 
 
 	public function testCopy()
 	{
-		$manager = \Aimeos\MShop::create( $this->context, 'price/lists/type' );
+		$manager = \Aimeos\MShop::create($this->context, 'price/lists/type');
 
-		$param = ['type' => 'unittest', 'id' => $manager->findItem( 'default', [], 'product' )->getId()];
-		$helper = new \Aimeos\MW\View\Helper\Param\Standard( $this->view, $param );
-		$this->view->addHelper( 'param', $helper );
+		$param = ['type' => 'unittest', 'id' => $manager->findItem('default', [], 'product')->getId()];
+		$helper = new \Aimeos\MW\View\Helper\Param\Standard($this->view, $param);
+		$this->view->addHelper('param', $helper);
 
 		$result = $this->object->copy();
 
-		$this->assertContains( 'default', $result );
+		$this->assertContains('default', $result);
 	}
 
 
 	public function testCopyException()
 	{
-		$object = $this->getMockBuilder( \Aimeos\Admin\JQAdm\Type\Price\Lists\Standard::class )
-			->setConstructorArgs( array( $this->context, \TestHelperJqadm::getTemplatePaths() ) )
-			->setMethods( array( 'getSubClients' ) )
+		$object = $this->getMockBuilder(\Aimeos\Admin\JQAdm\Type\Price\Lists\Standard::class)
+			->setConstructorArgs(array($this->context, \TestHelperJqadm::getTemplatePaths()))
+			->setMethods(array('getSubClients'))
 			->getMock();
 
-		$object->expects( $this->once() )->method( 'getSubClients' )
-			->will( $this->throwException( new \RuntimeException() ) );
+		$object->expects($this->once())->method('getSubClients')
+			->will($this->throwException(new \RuntimeException()));
 
-		$object->setView( $this->getViewNoRender() );
+		$object->setView($this->getViewNoRender());
 
 		$object->copy();
 	}
@@ -104,15 +104,15 @@ 
 
 	public function testCopyMShopException()
 	{
-		$object = $this->getMockBuilder( \Aimeos\Admin\JQAdm\Type\Price\Lists\Standard::class )
-			->setConstructorArgs( array( $this->context, \TestHelperJqadm::getTemplatePaths() ) )
-			->setMethods( array( 'getSubClients' ) )
+		$object = $this->getMockBuilder(\Aimeos\Admin\JQAdm\Type\Price\Lists\Standard::class)
+			->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();
 	}
@@ -120,21 +120,21 @@ 
 
 	public function testDelete()
 	{
-		$this->assertNotNull( $this->object->delete() );
+		$this->assertNotNull($this->object->delete());
 	}
 
 
 	public function testDeleteException()
 	{
-		$object = $this->getMockBuilder( \Aimeos\Admin\JQAdm\Type\Price\Lists\Standard::class )
-			->setConstructorArgs( array( $this->context, \TestHelperJqadm::getTemplatePaths() ) )
-			->setMethods( array( 'getSubClients', 'search' ) )
+		$object = $this->getMockBuilder(\Aimeos\Admin\JQAdm\Type\Price\Lists\Standard::class)
+			->setConstructorArgs(array($this->context, \TestHelperJqadm::getTemplatePaths()))
+			->setMethods(array('getSubClients', 'search'))
 			->getMock();
 
-		$object->expects( $this->once() )->method( 'getSubClients' )
-			->will( $this->throwException( new \RuntimeException() ) );
+		$object->expects($this->once())->method('getSubClients')
+			->will($this->throwException(new \RuntimeException()));
 
-		$object->setView( $this->getViewNoRender() );
+		$object->setView($this->getViewNoRender());
 
 		$object->delete();
 	}
@@ -142,15 +142,15 @@ 
 
 	public function testDeleteMShopException()
 	{
-		$object = $this->getMockBuilder( \Aimeos\Admin\JQAdm\Type\Price\Lists\Standard::class )
-			->setConstructorArgs( array( $this->context, \TestHelperJqadm::getTemplatePaths() ) )
-			->setMethods( array( 'getSubClients', 'search' ) )
+		$object = $this->getMockBuilder(\Aimeos\Admin\JQAdm\Type\Price\Lists\Standard::class)
+			->setConstructorArgs(array($this->context, \TestHelperJqadm::getTemplatePaths()))
+			->setMethods(array('getSubClients', 'search'))
 			->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();
 	}
@@ -158,29 +158,29 @@ 
 
 	public function testGet()
 	{
-		$manager = \Aimeos\MShop::create( $this->context, 'price/lists/type' );
+		$manager = \Aimeos\MShop::create($this->context, 'price/lists/type');
 
-		$param = ['type' => 'unittest', 'id' => $manager->findItem( 'default', [], 'product' )->getId()];
-		$helper = new \Aimeos\MW\View\Helper\Param\Standard( $this->view, $param );
-		$this->view->addHelper( 'param', $helper );
+		$param = ['type' => 'unittest', 'id' => $manager->findItem('default', [], 'product')->getId()];
+		$helper = new \Aimeos\MW\View\Helper\Param\Standard($this->view, $param);
+		$this->view->addHelper('param', $helper);
 
 		$result = $this->object->get();
 
-		$this->assertContains( 'default', $result );
+		$this->assertContains('default', $result);
 	}
 
 
 	public function testGetException()
 	{
-		$object = $this->getMockBuilder( \Aimeos\Admin\JQAdm\Type\Price\Lists\Standard::class )
-			->setConstructorArgs( array( $this->context, \TestHelperJqadm::getTemplatePaths() ) )
-			->setMethods( array( 'getSubClients' ) )
+		$object = $this->getMockBuilder(\Aimeos\Admin\JQAdm\Type\Price\Lists\Standard::class)
+			->setConstructorArgs(array($this->context, \TestHelperJqadm::getTemplatePaths()))
+			->setMethods(array('getSubClients'))
 			->getMock();
 
-		$object->expects( $this->once() )->method( 'getSubClients' )
-			->will( $this->throwException( new \RuntimeException() ) );
+		$object->expects($this->once())->method('getSubClients')
+			->will($this->throwException(new \RuntimeException()));
 
-		$object->setView( $this->getViewNoRender() );
+		$object->setView($this->getViewNoRender());
 
 		$object->get();
 	}
@@ -188,15 +188,15 @@ 
 
 	public function testGetMShopException()
 	{
-		$object = $this->getMockBuilder( \Aimeos\Admin\JQAdm\Type\Price\Lists\Standard::class )
-			->setConstructorArgs( array( $this->context, \TestHelperJqadm::getTemplatePaths() ) )
-			->setMethods( array( 'getSubClients' ) )
+		$object = $this->getMockBuilder(\Aimeos\Admin\JQAdm\Type\Price\Lists\Standard::class)
+			->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();
 	}
@@ -204,16 +204,16 @@ 
 
 	public function testGetViewException()
 	{
-		$object = new \Aimeos\Admin\JQAdm\Type\Price\Lists\Standard( $this->context, [] );
+		$object = new \Aimeos\Admin\JQAdm\Type\Price\Lists\Standard($this->context, []);
 
-		$this->setExpectedException( \Aimeos\Admin\JQAdm\Exception::class );
+		$this->setExpectedException(\Aimeos\Admin\JQAdm\Exception::class);
 		$object->getView();
 	}
 
 
 	public function testSave()
 	{
-		$manager = \Aimeos\MShop::create( $this->context, 'price/lists/type' );
+		$manager = \Aimeos\MShop::create($this->context, 'price/lists/type');
 
 		$param = array(
 			'type' => 'unittest',
@@ -226,26 +226,26 @@ 
 			),
 		);
 
-		$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( 'jqadm@test', [], 'product' )->getId() );
+		$manager->deleteItem($manager->findItem('jqadm@test', [], 'product')->getId());
 	}
 
 
 	public function testSaveException()
 	{
-		$object = $this->getMockBuilder( \Aimeos\Admin\JQAdm\Type\Price\Lists\Standard::class )
-			->setConstructorArgs( array( $this->context, \TestHelperJqadm::getTemplatePaths() ) )
-			->setMethods( array( 'fromArray' ) )
+		$object = $this->getMockBuilder(\Aimeos\Admin\JQAdm\Type\Price\Lists\Standard::class)
+			->setConstructorArgs(array($this->context, \TestHelperJqadm::getTemplatePaths()))
+			->setMethods(array('fromArray'))
 			->getMock();
 
-		$object->expects( $this->once() )->method( 'fromArray' )
-			->will( $this->throwException( new \RuntimeException() ) );
+		$object->expects($this->once())->method('fromArray')
+			->will($this->throwException(new \RuntimeException()));
 
-		$object->setView( $this->getViewNoRender() );
+		$object->setView($this->getViewNoRender());
 
 		$object->save();
 	}
@@ -253,15 +253,15 @@ 
 
 	public function testSaveMShopException()
 	{
-		$object = $this->getMockBuilder( \Aimeos\Admin\JQAdm\Type\Price\Lists\Standard::class )
-			->setConstructorArgs( array( $this->context, \TestHelperJqadm::getTemplatePaths() ) )
-			->setMethods( array( 'fromArray' ) )
+		$object = $this->getMockBuilder(\Aimeos\Admin\JQAdm\Type\Price\Lists\Standard::class)
+			->setConstructorArgs(array($this->context, \TestHelperJqadm::getTemplatePaths()))
+			->setMethods(array('fromArray'))
 			->getMock();
 
-		$object->expects( $this->once() )->method( 'fromArray' )
-			->will( $this->throwException( new \RuntimeException() ) );
+		$object->expects($this->once())->method('fromArray')
+			->will($this->throwException(new \RuntimeException()));
 
-		$object->setView( $this->getViewNoRender() );
+		$object->setView($this->getViewNoRender());
 
 		$object->save();
 	}
@@ -269,15 +269,15 @@ 
 
 	public function testSaveJQAdmException()
 	{
-		$object = $this->getMockBuilder( \Aimeos\Admin\JQAdm\Type\Price\Lists\Standard::class )
-			->setConstructorArgs( array( $this->context, \TestHelperJqadm::getTemplatePaths() ) )
-			->setMethods( array( 'fromArray' ) )
+		$object = $this->getMockBuilder(\Aimeos\Admin\JQAdm\Type\Price\Lists\Standard::class)
+			->setConstructorArgs(array($this->context, \TestHelperJqadm::getTemplatePaths()))
+			->setMethods(array('fromArray'))
 			->getMock();
 
-		$object->expects( $this->once() )->method( 'fromArray' )
-			->will( $this->throwException( new \RuntimeException() ) );
+		$object->expects($this->once())->method('fromArray')
+			->will($this->throwException(new \RuntimeException()));
 
-		$object->setView( $this->getViewNoRender() );
+		$object->setView($this->getViewNoRender());
 
 		$object->save();
 	}
@@ -288,32 +288,32 @@ 
 		$param = array(
 			'type' => 'unittest', 'lang' => 'de',
 			'filter' => array(
-				'key' => array( 0 => 'price.lists.type.code' ),
-				'op' => array( 0 => '==' ),
-				'val' => array( 0 => 'default' ),
+				'key' => array(0 => 'price.lists.type.code'),
+				'op' => array(0 => '=='),
+				'val' => array(0 => 'default'),
 			),
-			'sort' => array( '-price.lists.type.id' ),
+			'sort' => array('-price.lists.type.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( '>default<', $result );
+		$this->assertContains('>default<', $result);
 	}
 
 
 	public function testSearchException()
 	{
-		$object = $this->getMockBuilder( \Aimeos\Admin\JQAdm\Type\Price\Lists\Standard::class )
-			->setConstructorArgs( array( $this->context, \TestHelperJqadm::getTemplatePaths() ) )
-			->setMethods( array( 'initCriteria' ) )
+		$object = $this->getMockBuilder(\Aimeos\Admin\JQAdm\Type\Price\Lists\Standard::class)
+			->setConstructorArgs(array($this->context, \TestHelperJqadm::getTemplatePaths()))
+			->setMethods(array('initCriteria'))
 			->getMock();
 
-		$object->expects( $this->once() )->method( 'initCriteria' )
-			->will( $this->throwException( new \RuntimeException() ) );
+		$object->expects($this->once())->method('initCriteria')
+			->will($this->throwException(new \RuntimeException()));
 
-		$object->setView( $this->getViewNoRender() );
+		$object->setView($this->getViewNoRender());
 
 		$object->search();
 	}
@@ -321,15 +321,15 @@ 
 
 	public function testSearchMShopException()
 	{
-		$object = $this->getMockBuilder( \Aimeos\Admin\JQAdm\Type\Price\Lists\Standard::class )
-			->setConstructorArgs( array( $this->context, \TestHelperJqadm::getTemplatePaths() ) )
-			->setMethods( array( 'initCriteria' ) )
+		$object = $this->getMockBuilder(\Aimeos\Admin\JQAdm\Type\Price\Lists\Standard::class)
+			->setConstructorArgs(array($this->context, \TestHelperJqadm::getTemplatePaths()))
+			->setMethods(array('initCriteria'))
 			->getMock();
 
-		$object->expects( $this->once() )->method( 'initCriteria' )
-			->will( $this->throwException( new \Aimeos\MShop\Exception() ) );
+		$object->expects($this->once())->method('initCriteria')
+			->will($this->throwException(new \Aimeos\MShop\Exception()));
 
-		$object->setView( $this->getViewNoRender() );
+		$object->setView($this->getViewNoRender());
 
 		$object->search();
 	}
@@ -337,33 +337,33 @@ 
 
 	public function testGetSubClientInvalid()
 	{
-		$this->setExpectedException( \Aimeos\Admin\JQAdm\Exception::class );
-		$this->object->getSubClient( '$unknown$' );
+		$this->setExpectedException(\Aimeos\Admin\JQAdm\Exception::class);
+		$this->object->getSubClient('$unknown$');
 	}
 
 
 	public function testGetSubClientUnknown()
 	{
-		$this->setExpectedException( \Aimeos\Admin\JQAdm\Exception::class );
-		$this->object->getSubClient( 'unknown' );
+		$this->setExpectedException(\Aimeos\Admin\JQAdm\Exception::class);
+		$this->object->getSubClient('unknown');
 	}
 
 
 	protected function getViewNoRender()
 	{
-		$view = $this->getMockBuilder( \Aimeos\MW\View\Standard::class )
-			->setConstructorArgs( array( [] ) )
-			->setMethods( array( 'render', 'config' ) )
+		$view = $this->getMockBuilder(\Aimeos\MW\View\Standard::class)
+			->setConstructorArgs(array([]))
+			->setMethods(array('render', 'config'))
 			->getMock();
 
-		$manager = \Aimeos\MShop::create( $this->context, 'price/lists/type' );
+		$manager = \Aimeos\MShop::create($this->context, 'price/lists/type');
 
-		$param = ['type' => 'unittest', 'id' => $manager->findItem( 'default', [], 'product' )->getId()];
-		$helper = new \Aimeos\MW\View\Helper\Param\Standard( $view, $param );
-		$view->addHelper( 'param', $helper );
+		$param = ['type' => 'unittest', 'id' => $manager->findItem('default', [], 'product')->getId()];
+		$helper = new \Aimeos\MW\View\Helper\Param\Standard($view, $param);
+		$view->addHelper('param', $helper);
 
-		$helper = new \Aimeos\MW\View\Helper\Access\Standard( $view, [] );
-		$view->addHelper( 'access', $helper );
+		$helper = new \Aimeos\MW\View\Helper\Access\Standard($view, []);
+		$view->addHelper('access', $helper);
 
 		return $view;
 	}

admin/jqadm/src/Admin/JQAdm/Catalog/Standard.php

Spacing +111 added lines, -111 removed lines

@@ -10,7 +10,7 @@ 
 
 namespace Aimeos\Admin\JQAdm\Catalog;
 
-sprintf( 'catalog' ); // for translation
+sprintf('catalog'); // for translation
 
 
 /**
@@ -35,38 +35,38 @@ 
 
 		try
 		{
-			if( ( $id = $view->param( 'id' ) ) === null ) {
-				throw new \Aimeos\Admin\JQAdm\Exception( sprintf( 'Required parameter "%1$s" is missing', 'id' ) );
+			if (($id = $view->param('id')) === null) {
+				throw new \Aimeos\Admin\JQAdm\Exception(sprintf('Required parameter "%1$s" is missing', 'id'));
 			}
 
-			$manager = \Aimeos\MShop::create( $context, 'catalog' );
-			$view->item = $manager->getItem( $id, $this->getDomains() );
+			$manager = \Aimeos\MShop::create($context, 'catalog');
+			$view->item = $manager->getItem($id, $this->getDomains());
 
-			$view->itemData = $this->toArray( $view->item, true );
+			$view->itemData = $this->toArray($view->item, true);
 			$view->itemSubparts = $this->getSubClientNames();
 			$view->itemRootId = $this->getRootId();
 			$view->itemBody = '';
 
-			foreach( $this->getSubClients() as $idx => $client )
+			foreach ($this->getSubClients() as $idx => $client)
 			{
 				$view->tabindex = ++$idx + 1;
 				$view->itemBody .= $client->copy();
 			}
 		}
-		catch( \Aimeos\MShop\Exception $e )
+		catch (\Aimeos\MShop\Exception $e)
 		{
-			$error = array( 'catalog-item' => $context->getI18n()->dt( 'mshop', $e->getMessage() ) );
-			$view->errors = $view->get( 'errors', [] ) + $error;
-			$this->logException( $e );
+			$error = array('catalog-item' => $context->getI18n()->dt('mshop', $e->getMessage()));
+			$view->errors = $view->get('errors', []) + $error;
+			$this->logException($e);
 		}
-		catch( \Exception $e )
+		catch (\Exception $e)
 		{
-			$error = array( 'catalog-item' => $e->getMessage() . ', ' . $e->getFile() . ':' . $e->getLine() );
-			$view->errors = $view->get( 'errors', [] ) + $error;
-			$this->logException( $e );
+			$error = array('catalog-item' => $e->getMessage() . ', ' . $e->getFile() . ':' . $e->getLine());
+			$view->errors = $view->get('errors', []) + $error;
+			$this->logException($e);
 		}
 
-		return $this->render( $view );
+		return $this->render($view);
 	}
 
 
@@ -82,40 +82,40 @@ 
 
 		try
 		{
-			$data = $view->param( 'item', [] );
+			$data = $view->param('item', []);
 
-			if( !isset( $view->item ) ) {
-				$view->item = \Aimeos\MShop::create( $context, 'catalog' )->createItem();
+			if (!isset($view->item)) {
+				$view->item = \Aimeos\MShop::create($context, 'catalog')->createItem();
 			}
 
 			$data['catalog.siteid'] = $view->item->getSiteId();
-			$data['catalog.parentid'] = $view->item->getParentId() ?: $view->param( 'item/catalog.parentid' );
+			$data['catalog.parentid'] = $view->item->getParentId() ?: $view->param('item/catalog.parentid');
 
 			$view->itemSubparts = $this->getSubClientNames();
 			$view->itemRootId = $this->getRootId();
 			$view->itemData = $data;
 			$view->itemBody = '';
 
-			foreach( $this->getSubClients() as $idx => $client )
+			foreach ($this->getSubClients() as $idx => $client)
 			{
 				$view->tabindex = ++$idx + 1;
 				$view->itemBody .= $client->create();
 			}
 		}
-		catch( \Aimeos\MShop\Exception $e )
+		catch (\Aimeos\MShop\Exception $e)
 		{
-			$error = array( 'catalog-item' => $context->getI18n()->dt( 'mshop', $e->getMessage() ) );
-			$view->errors = $view->get( 'errors', [] ) + $error;
-			$this->logException( $e );
+			$error = array('catalog-item' => $context->getI18n()->dt('mshop', $e->getMessage()));
+			$view->errors = $view->get('errors', []) + $error;
+			$this->logException($e);
 		}
-		catch( \Exception $e )
+		catch (\Exception $e)
 		{
-			$error = array( 'catalog-item' => $e->getMessage() . ', ' . $e->getFile() . ':' . $e->getLine() );
-			$view->errors = $view->get( 'errors', [] ) + $error;
-			$this->logException( $e );
+			$error = array('catalog-item' => $e->getMessage() . ', ' . $e->getFile() . ':' . $e->getLine());
+			$view->errors = $view->get('errors', []) + $error;
+			$this->logException($e);
 		}
 
-		return $this->render( $view );
+		return $this->render($view);
 	}
 
 
@@ -129,39 +129,39 @@ 
 		$view = $this->getView();
 		$context = $this->getContext();
 
-		$manager = \Aimeos\MShop::create( $context, 'catalog' );
+		$manager = \Aimeos\MShop::create($context, 'catalog');
 		$manager->begin();
 
 		try
 		{
-			if( ( $id = $view->param( 'id' ) ) === null ) {
-				throw new \Aimeos\Admin\JQAdm\Exception( sprintf( 'Required parameter "%1$s" is missing', 'id' ) );
+			if (($id = $view->param('id')) === null) {
+				throw new \Aimeos\Admin\JQAdm\Exception(sprintf('Required parameter "%1$s" is missing', 'id'));
 			}
 
-			$view->item = $manager->getItem( $id, $this->getDomains() );
+			$view->item = $manager->getItem($id, $this->getDomains());
 
-			foreach( $this->getSubClients() as $client ) {
+			foreach ($this->getSubClients() as $client) {
 				$client->delete();
 			}
 
-			$manager->saveItem( $view->item );
-			$manager->deleteItem( $id );
+			$manager->saveItem($view->item);
+			$manager->deleteItem($id);
 			$manager->commit();
 
-			$this->nextAction( $view, 'search', 'catalog', null, 'delete' );
+			$this->nextAction($view, 'search', 'catalog', null, 'delete');
 			return;
 		}
-		catch( \Aimeos\MShop\Exception $e )
+		catch (\Aimeos\MShop\Exception $e)
 		{
-			$error = array( 'catalog-item' => $context->getI18n()->dt( 'mshop', $e->getMessage() ) );
-			$view->errors = $view->get( 'errors', [] ) + $error;
-			$this->logException( $e );
+			$error = array('catalog-item' => $context->getI18n()->dt('mshop', $e->getMessage()));
+			$view->errors = $view->get('errors', []) + $error;
+			$this->logException($e);
 		}
-		catch( \Exception $e )
+		catch (\Exception $e)
 		{
-			$error = array( 'catalog-item' => $e->getMessage() . ', ' . $e->getFile() . ':' . $e->getLine() );
-			$view->errors = $view->get( 'errors', [] ) + $error;
-			$this->logException( $e );
+			$error = array('catalog-item' => $e->getMessage() . ', ' . $e->getFile() . ':' . $e->getLine());
+			$view->errors = $view->get('errors', []) + $error;
+			$this->logException($e);
 		}
 
 		$manager->rollback();
@@ -182,38 +182,38 @@ 
 
 		try
 		{
-			if( ( $id = $view->param( 'id' ) ) === null ) {
-				throw new \Aimeos\Admin\JQAdm\Exception( sprintf( 'Required parameter "%1$s" is missing', 'id' ) );
+			if (($id = $view->param('id')) === null) {
+				throw new \Aimeos\Admin\JQAdm\Exception(sprintf('Required parameter "%1$s" is missing', 'id'));
 			}
 
-			$manager = \Aimeos\MShop::create( $context, 'catalog' );
+			$manager = \Aimeos\MShop::create($context, 'catalog');
 
-			$view->item = $manager->getItem( $id, $this->getDomains() );
+			$view->item = $manager->getItem($id, $this->getDomains());
 			$view->itemSubparts = $this->getSubClientNames();
-			$view->itemData = $this->toArray( $view->item );
+			$view->itemData = $this->toArray($view->item);
 			$view->itemRootId = $this->getRootId();
 			$view->itemBody = '';
 
-			foreach( $this->getSubClients() as $idx => $client )
+			foreach ($this->getSubClients() as $idx => $client)
 			{
 				$view->tabindex = ++$idx + 1;
 				$view->itemBody .= $client->get();
 			}
 		}
-		catch( \Aimeos\MShop\Exception $e )
+		catch (\Aimeos\MShop\Exception $e)
 		{
-			$error = array( 'catalog-item' => $context->getI18n()->dt( 'mshop', $e->getMessage() ) );
-			$view->errors = $view->get( 'errors', [] ) + $error;
-			$this->logException( $e );
+			$error = array('catalog-item' => $context->getI18n()->dt('mshop', $e->getMessage()));
+			$view->errors = $view->get('errors', []) + $error;
+			$this->logException($e);
 		}
-		catch( \Exception $e )
+		catch (\Exception $e)
 		{
-			$error = array( 'catalog-item' => $e->getMessage() . ', ' . $e->getFile() . ':' . $e->getLine() );
-			$view->errors = $view->get( 'errors', [] ) + $error;
-			$this->logException( $e );
+			$error = array('catalog-item' => $e->getMessage() . ', ' . $e->getFile() . ':' . $e->getLine());
+			$view->errors = $view->get('errors', []) + $error;
+			$this->logException($e);
 		}
 
-		return $this->render( $view );
+		return $this->render($view);
 	}
 
 
@@ -227,40 +227,40 @@ 
 		$view = $this->getView();
 		$context = $this->getContext();
 
-		$manager = \Aimeos\MShop::create( $context, 'catalog' );
+		$manager = \Aimeos\MShop::create($context, 'catalog');
 		$manager->begin();
 
 		try
 		{
-			$item = $this->fromArray( $view->param( 'item', [] ) );
-			$view->item = $item->getId() ? $item : $manager->saveItem( $item );
+			$item = $this->fromArray($view->param('item', []));
+			$view->item = $item->getId() ? $item : $manager->saveItem($item);
 			$view->itemBody = '';
 
-			foreach( $this->getSubClients() as $client ) {
+			foreach ($this->getSubClients() as $client) {
 				$view->itemBody .= $client->save();
 			}
 
-			$manager->saveItem( clone $view->item );
+			$manager->saveItem(clone $view->item);
 			$manager->commit();
 
-			$this->nextAction( $view, $view->param( 'next' ), 'catalog', $view->item->getId(), 'save' );
+			$this->nextAction($view, $view->param('next'), 'catalog', $view->item->getId(), 'save');
 			return;
 		}
-		catch( \Aimeos\Admin\JQAdm\Exception $e )
+		catch (\Aimeos\Admin\JQAdm\Exception $e)
 		{
 			// fall through to create
 		}
-		catch( \Aimeos\MShop\Exception $e )
+		catch (\Aimeos\MShop\Exception $e)
 		{
-			$error = array( 'catalog-item' => $context->getI18n()->dt( 'mshop', $e->getMessage() ) );
-			$view->errors = $view->get( 'errors', [] ) + $error;
-			$this->logException( $e );
+			$error = array('catalog-item' => $context->getI18n()->dt('mshop', $e->getMessage()));
+			$view->errors = $view->get('errors', []) + $error;
+			$this->logException($e);
 		}
-		catch( \Exception $e )
+		catch (\Exception $e)
 		{
-			$error = array( 'catalog-item' => $e->getMessage() . ', ' . $e->getFile() . ':' . $e->getLine() );
-			$view->errors = $view->get( 'errors', [] ) + $error;
-			$this->logException( $e );
+			$error = array('catalog-item' => $e->getMessage() . ', ' . $e->getFile() . ':' . $e->getLine());
+			$view->errors = $view->get('errors', []) + $error;
+			$this->logException($e);
 		}
 
 		$manager->rollback();
@@ -281,24 +281,24 @@ 
 
 		try
 		{
-			$view->item = \Aimeos\MShop::create( $context, 'catalog' )->createItem();
+			$view->item = \Aimeos\MShop::create($context, 'catalog')->createItem();
 			$view->itemRootId = $this->getRootId();
 			$view->itemBody = '';
 		}
-		catch( \Aimeos\MShop\Exception $e )
+		catch (\Aimeos\MShop\Exception $e)
 		{
-			$error = array( 'catalog-item' => $context->getI18n()->dt( 'mshop', $e->getMessage() ) );
-			$view->errors = $view->get( 'errors', [] ) + $error;
-			$this->logException( $e );
+			$error = array('catalog-item' => $context->getI18n()->dt('mshop', $e->getMessage()));
+			$view->errors = $view->get('errors', []) + $error;
+			$this->logException($e);
 		}
-		catch( \Exception $e )
+		catch (\Exception $e)
 		{
-			$error = array( 'catalog-item' => $e->getMessage() . ', ' . $e->getFile() . ':' . $e->getLine() );
-			$view->errors = $view->get( 'errors', [] ) + $error;
-			$this->logException( $e );
+			$error = array('catalog-item' => $e->getMessage() . ', ' . $e->getFile() . ':' . $e->getLine());
+			$view->errors = $view->get('errors', []) + $error;
+			$this->logException($e);
 		}
 
-		return $this->render( $view );
+		return $this->render($view);
 	}
 
 
@@ -309,7 +309,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/catalog/decorators/excludes
 		 * Excludes decorators added by the "common" option from the catalog JQAdm client
@@ -384,7 +384,7 @@ 
 		 * @see admin/jqadm/catalog/decorators/excludes
 		 * @see admin/jqadm/catalog/decorators/global
 		 */
-		return $this->createSubClient( 'catalog/' . $type, $name );
+		return $this->createSubClient('catalog/' . $type, $name);
 	}
 
 
@@ -406,9 +406,9 @@ 
 		 * @since 2016.01
 		 * @category Developer
 		 */
-		$domains = array( 'media', 'text' );
+		$domains = array('media', 'text');
 
-		return $this->getContext()->getConfig()->get( 'admin/jqadm/catalog/domains', $domains );
+		return $this->getContext()->getConfig()->get('admin/jqadm/catalog/domains', $domains);
 	}
 
 
@@ -419,11 +419,11 @@ 
 	 */
 	protected function getRootId()
 	{
-		$manager = \Aimeos\MShop::create( $this->getContext(), 'catalog' );
+		$manager = \Aimeos\MShop::create($this->getContext(), 'catalog');
 
 		try {
-			return $manager->getTree( null, [], \Aimeos\MW\Tree\Manager\Base::LEVEL_ONE )->getId();
-		} catch( \Exception $e ) {
+			return $manager->getTree(null, [], \Aimeos\MW\Tree\Manager\Base::LEVEL_ONE)->getId();
+		} catch (\Exception $e) {
 			return null;
 		}
 	}
@@ -469,7 +469,7 @@ 
 		 * @since 2016.01
 		 * @category Developer
 		 */
-		return $this->getContext()->getConfig()->get( 'admin/jqadm/catalog/standard/subparts', [] );
+		return $this->getContext()->getConfig()->get('admin/jqadm/catalog/standard/subparts', []);
 	}
 
 
@@ -479,33 +479,33 @@ 
 	 * @param string[] Data array
 	 * @return \Aimeos\MShop\Catalog\Item\Iface New catalog item object
 	 */
-	protected function fromArray( array $data )
+	protected function fromArray(array $data)
 	{
 		$conf = [];
 
-		if( isset( $data['config']['key'] ) )
+		if (isset($data['config']['key']))
 		{
-			foreach( (array) $data['config']['key'] as $idx => $key )
+			foreach ((array) $data['config']['key'] as $idx => $key)
 			{
-				if( trim( $key ) !== '' && isset( $data['config']['val'][$idx] ) ) {
+				if (trim($key) !== '' && isset($data['config']['val'][$idx])) {
 					$conf[$key] = $data['config']['val'][$idx];
 				}
 			}
 		}
 
-		$manager = \Aimeos\MShop::create( $this->getContext(), 'catalog' );
+		$manager = \Aimeos\MShop::create($this->getContext(), 'catalog');
 
-		if( isset( $data['catalog.id'] ) && $data['catalog.id'] != '' ) {
-			$item = $manager->getItem( $data['catalog.id'], $this->getDomains() );
+		if (isset($data['catalog.id']) && $data['catalog.id'] != '') {
+			$item = $manager->getItem($data['catalog.id'], $this->getDomains());
 		} else {
 			$item = $manager->createItem();
 		}
 
-		$item->fromArray( $data );
-		$item->setConfig( $conf );
+		$item->fromArray($data);
+		$item->setConfig($conf);
 
-		if( $item->getId() == null ) {
-			return $manager->insertItem( $item, $data['catalog.parentid'] ?: null );
+		if ($item->getId() == null) {
+			return $manager->insertItem($item, $data['catalog.parentid'] ?: null);
 		}
 
 		return $item;
@@ -518,19 +518,19 @@ 
 	 * @param \Aimeos\MShop\Catalog\Item\Iface $item Catalog item object
 	 * @return string[] Multi-dimensional associative list of item data
 	 */
-	protected function toArray( \Aimeos\MShop\Catalog\Item\Iface $item, $copy = false )
+	protected function toArray(\Aimeos\MShop\Catalog\Item\Iface $item, $copy = false)
 	{
-		$data = $item->toArray( true );
+		$data = $item->toArray(true);
 		$data['config'] = [];
 
-		if( $copy === true )
+		if ($copy === true)
 		{
 			$data['catalog.id'] = '';
 			$data['catalog.siteid'] = $item->getSiteId();
 			$data['catalog.code'] = $data['catalog.code'] . '_copy';
 		}
 
-		foreach( $item->getConfig() as $key => $value )
+		foreach ($item->getConfig() as $key => $value)
 		{
 			$data['config']['key'][] = $key;
 			$data['config']['val'][] = $value;
@@ -546,7 +546,7 @@ 
 	 * @param \Aimeos\MW\View\Iface $view View object with data assigned
 	 * @return string HTML output
 	 */
-	protected function render( \Aimeos\MW\View\Iface $view )
+	protected function render(\Aimeos\MW\View\Iface $view)
 	{
 		/** admin/jqadm/catalog/template-item
 		 * Relative path to the HTML body template for the catalog item.
@@ -570,6 +570,6 @@ 
 		$tplconf = 'admin/jqadm/catalog/template-item';
 		$default = 'catalog/item-standard';
 
-		return $view->render( $view->config( $tplconf, $default ) );
+		return $view->render($view->config($tplconf, $default));
 	}
 }

admin/jqadm/src/Admin/JQAdm/Customer/Address/Standard.php

Spacing +38 added lines, -38 removed lines

@@ -10,7 +10,7 @@ 
 
 namespace Aimeos\Admin\JQAdm\Customer\Address;
 
-sprintf( 'address' ); // for translation
+sprintf('address'); // for translation
 
 
 /**
@@ -44,14 +44,14 @@ 
 	{
 		$view = $this->getView();
 
-		$view->addressData = $this->toArray( $view->item, true );
+		$view->addressData = $this->toArray($view->item, true);
 		$view->addressBody = '';
 
-		foreach( $this->getSubClients() as $client ) {
+		foreach ($this->getSubClients() as $client) {
 			$view->addressBody .= $client->copy();
 		}
 
-		return $this->render( $view );
+		return $this->render($view);
 	}
 
 
@@ -64,20 +64,20 @@ 
 	{
 		$view = $this->getView();
 		$siteid = $this->getContext()->getLocale()->getSiteId();
-		$data = $view->param( 'address', [] );
+		$data = $view->param('address', []);
 
-		foreach( $data as $idx => $entry ) {
+		foreach ($data as $idx => $entry) {
 			$data[$idx]['customer.address.siteid'] = $siteid;
 		}
 
 		$view->addressData = $data;
 		$view->addressBody = '';
 
-		foreach( $this->getSubClients() as $client ) {
+		foreach ($this->getSubClients() as $client) {
 			$view->addressBody .= $client->create();
 		}
 
-		return $this->render( $view );
+		return $this->render($view);
 	}
 
 
@@ -90,14 +90,14 @@ 
 	{
 		$view = $this->getView();
 
-		$view->addressData = $this->toArray( $view->item );
+		$view->addressData = $this->toArray($view->item);
 		$view->addressBody = '';
 
-		foreach( $this->getSubClients() as $client ) {
+		foreach ($this->getSubClients() as $client) {
 			$view->addressBody .= $client->get();
 		}
 
-		return $this->render( $view );
+		return $this->render($view);
 	}
 
 
@@ -110,26 +110,26 @@ 
 
 		try
 		{
-			$this->fromArray( $view->item, $view->param( 'address', [] ) );
+			$this->fromArray($view->item, $view->param('address', []));
 			$view->addressBody = '';
 
-			foreach( $this->getSubClients() as $client ) {
+			foreach ($this->getSubClients() as $client) {
 				$view->addressBody .= $client->save();
 			}
 
 			return;
 		}
-		catch( \Aimeos\MShop\Exception $e )
+		catch (\Aimeos\MShop\Exception $e)
 		{
-			$error = array( 'customer-item-address' => $this->getContext()->getI18n()->dt( 'mshop', $e->getMessage() ) );
-			$view->errors = $view->get( 'errors', [] ) + $error;
-			$this->logException( $e );
+			$error = array('customer-item-address' => $this->getContext()->getI18n()->dt('mshop', $e->getMessage()));
+			$view->errors = $view->get('errors', []) + $error;
+			$this->logException($e);
 		}
-		catch( \Exception $e )
+		catch (\Exception $e)
 		{
-			$error = array( 'customer-item-address' => $e->getMessage() . ', ' . $e->getFile() . ':' . $e->getLine() );
-			$view->errors = $view->get( 'errors', [] ) + $error;
-			$this->logException( $e );
+			$error = array('customer-item-address' => $e->getMessage() . ', ' . $e->getFile() . ':' . $e->getLine());
+			$view->errors = $view->get('errors', []) + $error;
+			$this->logException($e);
 		}
 
 		throw new \Aimeos\Admin\JQAdm\Exception();
@@ -143,7 +143,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/customer/address/decorators/excludes
 		 * Excludes decorators added by the "common" option from the customer JQAdm client
@@ -218,7 +218,7 @@ 
 		 * @see admin/jqadm/customer/address/decorators/excludes
 		 * @see admin/jqadm/customer/address/decorators/global
 		 */
-		return $this->createSubClient( 'customer/address/' . $type, $name );
+		return $this->createSubClient('customer/address/' . $type, $name);
 	}
 
 
@@ -262,7 +262,7 @@ 
 		 * @since 2016.01
 		 * @category Developer
 		 */
-		return $this->getContext()->getConfig()->get( 'admin/jqadm/customer/address/standard/subparts', [] );
+		return $this->getContext()->getConfig()->get('admin/jqadm/customer/address/standard/subparts', []);
 	}
 
 
@@ -272,29 +272,29 @@ 
 	 * @param \Aimeos\MShop\Customer\Item\Iface $item Customer item object without referenced domain items
 	 * @param string[] $data Data array
 	 */
-	protected function fromArray( \Aimeos\MShop\Customer\Item\Iface $item, array $data )
+	protected function fromArray(\Aimeos\MShop\Customer\Item\Iface $item, array $data)
 	{
-		$manager = \Aimeos\MShop::create( $this->getContext(), 'customer/address' );
+		$manager = \Aimeos\MShop::create($this->getContext(), 'customer/address');
 
 		$addrItems = $item->getAddressItems();
 
-		foreach( $data as $entry )
+		foreach ($data as $entry)
 		{
-			if( isset( $addrItems[$entry['customer.address.id']] ) )
+			if (isset($addrItems[$entry['customer.address.id']]))
 			{
 				$addrItem = $addrItems[$entry['customer.address.id']];
-				unset( $addrItems[$entry['customer.address.id']] );
+				unset($addrItems[$entry['customer.address.id']]);
 			}
 			else
 			{
 				$addrItem = $manager->createItem();
 			}
 
-			$addrItem->fromArray( $entry );
-			$item->addAddressItem( $addrItem );
+			$addrItem->fromArray($entry);
+			$item->addAddressItem($addrItem);
 		}
 
-		$item->deleteAddressItems( $addrItems );
+		$item->deleteAddressItems($addrItems);
 	}
 
 
@@ -305,16 +305,16 @@ 
 	 * @param boolean $copy True if items should be copied, false if not
 	 * @return string[] Multi-dimensional associative list of item data
 	 */
-	protected function toArray( \Aimeos\MShop\Customer\Item\Iface $item, $copy = false )
+	protected function toArray(\Aimeos\MShop\Customer\Item\Iface $item, $copy = false)
 	{
 		$siteId = $this->getContext()->getLocale()->getSiteId();
 		$data = [];
 
-		foreach( $item->getAddressItems() as $addrItem )
+		foreach ($item->getAddressItems() as $addrItem)
 		{
-			$list = $addrItem->toArray( true );
+			$list = $addrItem->toArray(true);
 
-			if( $copy === true )
+			if ($copy === true)
 			{
 				$list['customer.address.siteid'] = $siteId;
 				$list['customer.address.id'] = '';
@@ -333,7 +333,7 @@ 
 	 * @param \Aimeos\MW\View\Iface $view View object with data assigned
 	 * @return string HTML output
 	 */
-	protected function render( \Aimeos\MW\View\Iface $view )
+	protected function render(\Aimeos\MW\View\Iface $view)
 	{
 		/** admin/jqadm/customer/address/template-item
 		 * Relative path to the HTML body template of the address subpart for customers.
@@ -357,6 +357,6 @@ 
 		$tplconf = 'admin/jqadm/customer/address/template-item';
 		$default = 'customer/item-address-standard';
 
-		return $view->render( $view->config( $tplconf, $default ) );
+		return $view->render($view->config($tplconf, $default));
 	}
 }

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

Spacing +32 added lines, -32 removed lines

@@ -10,7 +10,7 @@ 
 
 namespace Aimeos\Admin\JQAdm\Customer\Order;
 
-sprintf( 'order' ); // for translation
+sprintf('order'); // for translation
 
 
 /**
@@ -69,33 +69,33 @@ 
 		try
 		{
 			$total = 0;
-			$params = $this->storeSearchParams( $view->param( 'uo', [] ), 'customerorder' );
-			$orderItems = $this->getOrderItems( $view->item, $params, $total );
-			$baseItems = $this->getOrderBaseItems( $orderItems );
+			$params = $this->storeSearchParams($view->param('uo', []), 'customerorder');
+			$orderItems = $this->getOrderItems($view->item, $params, $total);
+			$baseItems = $this->getOrderBaseItems($orderItems);
 
 			$view->orderItems = $orderItems;
 			$view->orderBaseItems = $baseItems;
 			$view->orderTotal = $total;
 			$view->orderBody = '';
 
-			foreach( $this->getSubClients() as $client ) {
+			foreach ($this->getSubClients() as $client) {
 				$view->orderBody .= $client->search();
 			}
 		}
-		catch( \Aimeos\MShop\Exception $e )
+		catch (\Aimeos\MShop\Exception $e)
 		{
-			$error = array( 'customer-order' => $this->getContext()->getI18n()->dt( 'mshop', $e->getMessage() ) );
-			$view->errors = $view->get( 'errors', [] ) + $error;
-			$this->logException( $e );
+			$error = array('customer-order' => $this->getContext()->getI18n()->dt('mshop', $e->getMessage()));
+			$view->errors = $view->get('errors', []) + $error;
+			$this->logException($e);
 		}
-		catch( \Exception $e )
+		catch (\Exception $e)
 		{
-			$error = array( 'customer-order' => $e->getMessage() . ', ' . $e->getFile() . ':' . $e->getLine() );
-			$view->errors = $view->get( 'errors', [] ) + $error;
-			$this->logException( $e );
+			$error = array('customer-order' => $e->getMessage() . ', ' . $e->getFile() . ':' . $e->getLine());
+			$view->errors = $view->get('errors', []) + $error;
+			$this->logException($e);
 		}
 
-		return $this->render( $view );
+		return $this->render($view);
 	}
 
 
@@ -115,7 +115,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/customer/order/decorators/excludes
 		 * Excludes decorators added by the "common" option from the customer JQAdm client
@@ -190,7 +190,7 @@ 
 		 * @see admin/jqadm/customer/order/decorators/excludes
 		 * @see admin/jqadm/customer/order/decorators/global
 		 */
-		return $this->createSubClient( 'customer/order/' . $type, $name );
+		return $this->createSubClient('customer/order/' . $type, $name);
 	}
 
 
@@ -200,20 +200,20 @@ 
 	 * @param \Aimeos\MShop\Order\Item\Iface[] $items Order item objects
 	 * @return \Aimeos\MShop\Order\Item\Base\Iface[] Basket items
 	 */
-	protected function getOrderBaseItems( array $items )
+	protected function getOrderBaseItems(array $items)
 	{
 		$ids = [];
 
-		foreach( $items as $item ) {
+		foreach ($items as $item) {
 			$ids[] = $item->getBaseId();
 		}
 
-		$manager = \Aimeos\MShop::create( $this->getContext(), 'order/base' );
+		$manager = \Aimeos\MShop::create($this->getContext(), 'order/base');
 
-		$search = $manager->createSearch()->setSlice( 0, count( $ids ) );
-		$search->setConditions( $search->compare( '==', 'order.base.id', $ids ) );
+		$search = $manager->createSearch()->setSlice(0, count($ids));
+		$search->setConditions($search->compare('==', 'order.base.id', $ids));
 
-		return $manager->searchItems( $search );
+		return $manager->searchItems($search);
 	}
 
 
@@ -225,21 +225,21 @@ 
 	 * @param integer $total Value/result parameter that will contain the item total afterwards
 	 * @return \Aimeos\MShop\Order\Item\Iface[] Order items of the customer
 	 */
-	protected function getOrderItems( \Aimeos\MShop\Customer\Item\Iface $item, array $params = [], &$total )
+	protected function getOrderItems(\Aimeos\MShop\Customer\Item\Iface $item, array $params = [], &$total)
 	{
-		$manager = \Aimeos\MShop::create( $this->getContext(), 'order' );
+		$manager = \Aimeos\MShop::create($this->getContext(), 'order');
 
 		$search = $manager->createSearch();
-		$search->setSortations( [$search->sort( '-', 'order.ctime' )] );
+		$search->setSortations([$search->sort('-', 'order.ctime')]);
 
-		$search = $this->initCriteria( $search, $params );
+		$search = $this->initCriteria($search, $params);
 		$expr = [
-			$search->compare( '==', 'order.base.customerid', $item->getId() ),
+			$search->compare('==', 'order.base.customerid', $item->getId()),
 			$search->getConditions(),
 		];
-		$search->setConditions( $search->combine( '&&', $expr ) );
+		$search->setConditions($search->combine('&&', $expr));
 
-		return $manager->searchItems( $search, [], $total );
+		return $manager->searchItems($search, [], $total);
 	}
 
 
@@ -283,7 +283,7 @@ 
 		 * @since 2017.07
 		 * @category Developer
 		 */
-		return $this->getContext()->getConfig()->get( 'admin/jqadm/customer/order/standard/subparts', [] );
+		return $this->getContext()->getConfig()->get('admin/jqadm/customer/order/standard/subparts', []);
 	}
 
 
@@ -293,7 +293,7 @@ 
 	 * @param \Aimeos\MW\View\Iface $view View object with data assigned
 	 * @return string HTML output
 	 */
-	protected function render( \Aimeos\MW\View\Iface $view )
+	protected function render(\Aimeos\MW\View\Iface $view)
 	{
 		/** admin/jqadm/customer/order/template-item
 		 * Relative path to the HTML body template of the order subpart for customers.
@@ -317,6 +317,6 @@ 
 		$tplconf = 'admin/jqadm/customer/order/template-item';
 		$default = 'customer/item-order-standard';
 
-		return $view->render( $view->config( $tplconf, $default ) );
+		return $view->render($view->config($tplconf, $default));
 	}
 }

admin/jqadm/src/Admin/JQAdm/Customer/Standard.php

Spacing +112 added lines, -112 removed lines

@@ -10,7 +10,7 @@ 
 
 namespace Aimeos\Admin\JQAdm\Customer;
 
-sprintf( 'customer' ); // for translation
+sprintf('customer'); // for translation
 
 
 /**
@@ -35,38 +35,38 @@ 
 
 		try
 		{
-			if( ( $id = $view->param( 'id' ) ) === null ) {
-				throw new \Aimeos\Admin\JQAdm\Exception( sprintf( 'Required parameter "%1$s" is missing', 'id' ) );
+			if (($id = $view->param('id')) === null) {
+				throw new \Aimeos\Admin\JQAdm\Exception(sprintf('Required parameter "%1$s" is missing', 'id'));
 			}
 
-			$manager = \Aimeos\MShop::create( $context, 'customer' );
-			$view->item = $manager->getItem( $id, $this->getDomains() );
+			$manager = \Aimeos\MShop::create($context, 'customer');
+			$view->item = $manager->getItem($id, $this->getDomains());
 
-			$view->itemData = $this->toArray( $view->item, true );
+			$view->itemData = $this->toArray($view->item, true);
 			$view->itemSubparts = $this->getSubClientNames();
 			$view->itemGroups = $this->getGroupItems();
 			$view->itemBody = '';
 
-			foreach( $this->getSubClients() as $idx => $client )
+			foreach ($this->getSubClients() as $idx => $client)
 			{
 				$view->tabindex = ++$idx + 1;
 				$view->itemBody .= $client->copy();
 			}
 		}
-		catch( \Aimeos\MShop\Exception $e )
+		catch (\Aimeos\MShop\Exception $e)
 		{
-			$error = array( 'customer-item' => $context->getI18n()->dt( 'mshop', $e->getMessage() ) );
-			$view->errors = $view->get( 'errors', [] ) + $error;
-			$this->logException( $e );
+			$error = array('customer-item' => $context->getI18n()->dt('mshop', $e->getMessage()));
+			$view->errors = $view->get('errors', []) + $error;
+			$this->logException($e);
 		}
-		catch( \Exception $e )
+		catch (\Exception $e)
 		{
-			$error = array( 'customer-item' => $e->getMessage() . ', ' . $e->getFile() . ':' . $e->getLine() );
-			$view->errors = $view->get( 'errors', [] ) + $error;
-			$this->logException( $e );
+			$error = array('customer-item' => $e->getMessage() . ', ' . $e->getFile() . ':' . $e->getLine());
+			$view->errors = $view->get('errors', []) + $error;
+			$this->logException($e);
 		}
 
-		return $this->render( $view );
+		return $this->render($view);
 	}
 
 
@@ -82,10 +82,10 @@ 
 
 		try
 		{
-			$data = $view->param( 'item', [] );
+			$data = $view->param('item', []);
 
-			if( !isset( $view->item ) ) {
-				$view->item = \Aimeos\MShop::create( $context, 'customer' )->createItem();
+			if (!isset($view->item)) {
+				$view->item = \Aimeos\MShop::create($context, 'customer')->createItem();
 			}
 
 			$data['customer.siteid'] = $view->item->getSiteId();
@@ -95,26 +95,26 @@ 
 			$view->itemData = $data;
 			$view->itemBody = '';
 
-			foreach( $this->getSubClients() as $idx => $client )
+			foreach ($this->getSubClients() as $idx => $client)
 			{
 				$view->tabindex = ++$idx + 1;
 				$view->itemBody .= $client->create();
 			}
 		}
-		catch( \Aimeos\MShop\Exception $e )
+		catch (\Aimeos\MShop\Exception $e)
 		{
-			$error = array( 'customer-item' => $context->getI18n()->dt( 'mshop', $e->getMessage() ) );
-			$view->errors = $view->get( 'errors', [] ) + $error;
-			$this->logException( $e );
+			$error = array('customer-item' => $context->getI18n()->dt('mshop', $e->getMessage()));
+			$view->errors = $view->get('errors', []) + $error;
+			$this->logException($e);
 		}
-		catch( \Exception $e )
+		catch (\Exception $e)
 		{
-			$error = array( 'customer-item' => $e->getMessage() . ', ' . $e->getFile() . ':' . $e->getLine() );
-			$view->errors = $view->get( 'errors', [] ) + $error;
-			$this->logException( $e );
+			$error = array('customer-item' => $e->getMessage() . ', ' . $e->getFile() . ':' . $e->getLine());
+			$view->errors = $view->get('errors', []) + $error;
+			$this->logException($e);
 		}
 
-		return $this->render( $view );
+		return $this->render($view);
 	}
 
 
@@ -128,39 +128,39 @@ 
 		$view = $this->getView();
 		$context = $this->getContext();
 
-		$manager = \Aimeos\MShop::create( $context, 'customer' );
+		$manager = \Aimeos\MShop::create($context, 'customer');
 		$manager->begin();
 
 		try
 		{
-			if( ( $id = $view->param( 'id' ) ) === null ) {
-				throw new \Aimeos\Admin\JQAdm\Exception( sprintf( 'Required parameter "%1$s" is missing', 'id' ) );
+			if (($id = $view->param('id')) === null) {
+				throw new \Aimeos\Admin\JQAdm\Exception(sprintf('Required parameter "%1$s" is missing', 'id'));
 			}
 
-			$view->item = $manager->getItem( $id, $this->getDomains() );
+			$view->item = $manager->getItem($id, $this->getDomains());
 
-			foreach( $this->getSubClients() as $client ) {
+			foreach ($this->getSubClients() as $client) {
 				$client->delete();
 			}
 
-			$manager->saveItem( $view->item );
-			$manager->deleteItem( $id );
+			$manager->saveItem($view->item);
+			$manager->deleteItem($id);
 			$manager->commit();
 
-			$this->nextAction( $view, 'search', 'customer', null, 'delete' );
+			$this->nextAction($view, 'search', 'customer', null, 'delete');
 			return;
 		}
-		catch( \Aimeos\MShop\Exception $e )
+		catch (\Aimeos\MShop\Exception $e)
 		{
-			$error = array( 'customer-item' => $context->getI18n()->dt( 'mshop', $e->getMessage() ) );
-			$view->errors = $view->get( 'errors', [] ) + $error;
-			$this->logException( $e );
+			$error = array('customer-item' => $context->getI18n()->dt('mshop', $e->getMessage()));
+			$view->errors = $view->get('errors', []) + $error;
+			$this->logException($e);
 		}
-		catch( \Exception $e )
+		catch (\Exception $e)
 		{
-			$error = array( 'customer-item' => $e->getMessage() . ', ' . $e->getFile() . ':' . $e->getLine() );
-			$view->errors = $view->get( 'errors', [] ) + $error;
-			$this->logException( $e );
+			$error = array('customer-item' => $e->getMessage() . ', ' . $e->getFile() . ':' . $e->getLine());
+			$view->errors = $view->get('errors', []) + $error;
+			$this->logException($e);
 		}
 
 		$manager->rollback();
@@ -181,38 +181,38 @@ 
 
 		try
 		{
-			if( ( $id = $view->param( 'id' ) ) === null ) {
-				throw new \Aimeos\Admin\JQAdm\Exception( sprintf( 'Required parameter "%1$s" is missing', 'id' ) );
+			if (($id = $view->param('id')) === null) {
+				throw new \Aimeos\Admin\JQAdm\Exception(sprintf('Required parameter "%1$s" is missing', 'id'));
 			}
 
-			$manager = \Aimeos\MShop::create( $context, 'customer' );
+			$manager = \Aimeos\MShop::create($context, 'customer');
 
-			$view->item = $manager->getItem( $id, $this->getDomains() );
+			$view->item = $manager->getItem($id, $this->getDomains());
 			$view->itemSubparts = $this->getSubClientNames();
-			$view->itemData = $this->toArray( $view->item );
+			$view->itemData = $this->toArray($view->item);
 			$view->itemGroups = $this->getGroupItems();
 			$view->itemBody = '';
 
-			foreach( $this->getSubClients() as $idx => $client )
+			foreach ($this->getSubClients() as $idx => $client)
 			{
 				$view->tabindex = ++$idx + 1;
 				$view->itemBody .= $client->get();
 			}
 		}
-		catch( \Aimeos\MShop\Exception $e )
+		catch (\Aimeos\MShop\Exception $e)
 		{
-			$error = array( 'customer-item' => $context->getI18n()->dt( 'mshop', $e->getMessage() ) );
-			$view->errors = $view->get( 'errors', [] ) + $error;
-			$this->logException( $e );
+			$error = array('customer-item' => $context->getI18n()->dt('mshop', $e->getMessage()));
+			$view->errors = $view->get('errors', []) + $error;
+			$this->logException($e);
 		}
-		catch( \Exception $e )
+		catch (\Exception $e)
 		{
-			$error = array( 'customer-item' => $e->getMessage() . ', ' . $e->getFile() . ':' . $e->getLine() );
-			$view->errors = $view->get( 'errors', [] ) + $error;
-			$this->logException( $e );
+			$error = array('customer-item' => $e->getMessage() . ', ' . $e->getFile() . ':' . $e->getLine());
+			$view->errors = $view->get('errors', []) + $error;
+			$this->logException($e);
 		}
 
-		return $this->render( $view );
+		return $this->render($view);
 	}
 
 
@@ -226,40 +226,40 @@ 
 		$view = $this->getView();
 		$context = $this->getContext();
 
-		$manager = \Aimeos\MShop::create( $context, 'customer' );
+		$manager = \Aimeos\MShop::create($context, 'customer');
 		$manager->begin();
 
 		try
 		{
-			$item = $this->fromArray( $view->param( 'item', [] ) );
-			$view->item = $item->getId() ? $item : $manager->saveItem( $item );
+			$item = $this->fromArray($view->param('item', []));
+			$view->item = $item->getId() ? $item : $manager->saveItem($item);
 			$view->itemBody = '';
 
-			foreach( $this->getSubClients() as $client ) {
+			foreach ($this->getSubClients() as $client) {
 				$view->itemBody .= $client->save();
 			}
 
-			$manager->saveItem( clone $view->item );
+			$manager->saveItem(clone $view->item);
 			$manager->commit();
 
-			$this->nextAction( $view, $view->param( 'next' ), 'customer', $view->item->getId(), 'save' );
+			$this->nextAction($view, $view->param('next'), 'customer', $view->item->getId(), 'save');
 			return;
 		}
-		catch( \Aimeos\Admin\JQAdm\Exception $e )
+		catch (\Aimeos\Admin\JQAdm\Exception $e)
 		{
 			// fall through to create
 		}
-		catch( \Aimeos\MShop\Exception $e )
+		catch (\Aimeos\MShop\Exception $e)
 		{
-			$error = array( 'customer-item' => $context->getI18n()->dt( 'mshop', $e->getMessage() ) );
-			$view->errors = $view->get( 'errors', [] ) + $error;
-			$this->logException( $e );
+			$error = array('customer-item' => $context->getI18n()->dt('mshop', $e->getMessage()));
+			$view->errors = $view->get('errors', []) + $error;
+			$this->logException($e);
 		}
-		catch( \Exception $e )
+		catch (\Exception $e)
 		{
-			$error = array( 'customer-item' => $e->getMessage() . ', ' . $e->getFile() . ':' . $e->getLine() );
-			$view->errors = $view->get( 'errors', [] ) + $error;
-			$this->logException( $e );
+			$error = array('customer-item' => $e->getMessage() . ', ' . $e->getFile() . ':' . $e->getLine());
+			$view->errors = $view->get('errors', []) + $error;
+			$this->logException($e);
 		}
 
 		$manager->rollback();
@@ -281,31 +281,31 @@ 
 		try
 		{
 			$total = 0;
-			$params = $this->storeSearchParams( $view->param(), 'customer' );
-			$manager = \Aimeos\MShop::create( $context, 'customer' );
-			$search = $this->initCriteria( $manager->createSearch(), $params );
+			$params = $this->storeSearchParams($view->param(), 'customer');
+			$manager = \Aimeos\MShop::create($context, 'customer');
+			$search = $this->initCriteria($manager->createSearch(), $params);
 
-			$view->items = $manager->searchItems( $search, $this->getDomains(), $total );
-			$view->filterAttributes = $manager->getSearchAttributes( true );
+			$view->items = $manager->searchItems($search, $this->getDomains(), $total);
+			$view->filterAttributes = $manager->getSearchAttributes(true);
 			$view->filterOperators = $search->getOperators();
 			$view->total = $total;
 			$view->itemBody = '';
 
-			foreach( $this->getSubClients() as $client ) {
+			foreach ($this->getSubClients() as $client) {
 				$view->itemBody .= $client->search();
 			}
 		}
-		catch( \Aimeos\MShop\Exception $e )
+		catch (\Aimeos\MShop\Exception $e)
 		{
-			$error = array( 'customer-item' => $context->getI18n()->dt( 'mshop', $e->getMessage() ) );
-			$view->errors = $view->get( 'errors', [] ) + $error;
-			$this->logException( $e );
+			$error = array('customer-item' => $context->getI18n()->dt('mshop', $e->getMessage()));
+			$view->errors = $view->get('errors', []) + $error;
+			$this->logException($e);
 		}
-		catch( \Exception $e )
+		catch (\Exception $e)
 		{
-			$error = array( 'customer-item' => $e->getMessage() . ', ' . $e->getFile() . ':' . $e->getLine() );
-			$view->errors = $view->get( 'errors', [] ) + $error;
-			$this->logException( $e );
+			$error = array('customer-item' => $e->getMessage() . ', ' . $e->getFile() . ':' . $e->getLine());
+			$view->errors = $view->get('errors', []) + $error;
+			$this->logException($e);
 		}
 
 		/** admin/jqadm/customer/template-list
@@ -330,7 +330,7 @@ 
 		$tplconf = 'admin/jqadm/customer/template-list';
 		$default = 'customer/list-standard';
 
-		return $view->render( $view->config( $tplconf, $default ) );
+		return $view->render($view->config($tplconf, $default));
 	}
 
 
@@ -341,7 +341,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/customer/decorators/excludes
 		 * Excludes decorators added by the "common" option from the customer JQAdm client
@@ -416,7 +416,7 @@ 
 		 * @see admin/jqadm/customer/decorators/excludes
 		 * @see admin/jqadm/customer/decorators/global
 		 */
-		return $this->createSubClient( 'customer/' . $type, $name );
+		return $this->createSubClient('customer/' . $type, $name);
 	}
 
 
@@ -440,7 +440,7 @@ 
 		 */
 		$domains = ['customer/address', 'customer/group', 'customer/property'];
 
-		return $this->getContext()->getConfig()->get( 'admin/jqadm/customer/domains', $domains );
+		return $this->getContext()->getConfig()->get('admin/jqadm/customer/domains', $domains);
 	}
 
 
@@ -452,20 +452,20 @@ 
 	protected function getGroupItems()
 	{
 		$list = [];
-		$isSuper = $this->getView()->access( ['super'] );
-		$isAdmin = $this->getView()->access( ['admin'] );
+		$isSuper = $this->getView()->access(['super']);
+		$isAdmin = $this->getView()->access(['admin']);
 
-		$manager = \Aimeos\MShop::create( $this->getContext(), 'customer/group' );
+		$manager = \Aimeos\MShop::create($this->getContext(), 'customer/group');
 		$search = $manager->createSearch();
-		$search->setSortations( [$search->sort( '+', 'customer.group.label' )] );
+		$search->setSortations([$search->sort('+', 'customer.group.label')]);
 
-		foreach( $manager->searchItems( $search ) as $groupId => $groupItem )
+		foreach ($manager->searchItems($search) as $groupId => $groupItem)
 		{
-			if( !$isSuper && in_array( $groupItem->getCode(), ['super'] ) ) {
+			if (!$isSuper && in_array($groupItem->getCode(), ['super'])) {
 				continue;
 			}
 
-			if( !$isSuper && !$isAdmin && in_array( $groupItem->getCode(), ['super', 'admin', 'editor'] ) ) {
+			if (!$isSuper && !$isAdmin && in_array($groupItem->getCode(), ['super', 'admin', 'editor'])) {
 				continue;
 			}
 
@@ -516,7 +516,7 @@ 
 		 * @since 2017.07
 		 * @category Developer
 		 */
-		return $this->getContext()->getConfig()->get( 'admin/jqadm/customer/standard/subparts', [] );
+		return $this->getContext()->getConfig()->get('admin/jqadm/customer/standard/subparts', []);
 	}
 
 
@@ -527,21 +527,21 @@ 
 	 * @param string[] Data array
 	 * @return \Aimeos\MShop\Customer\Item\Iface New customer item object
 	 */
-	protected function fromArray( array $data )
+	protected function fromArray(array $data)
 	{
 		$data['customer.label'] = $data['customer.firstname'] . ' ' . $data['customer.lastname'];
 		$data['customer.code'] = $data['customer.email'];
 
-		$manager = \Aimeos\MShop::create( $this->getContext(), 'customer' );
+		$manager = \Aimeos\MShop::create($this->getContext(), 'customer');
 
-		if( isset( $data['customer.id'] ) && $data['customer.id'] != '' ) {
-			$item = $manager->getItem( $data['customer.id'], $this->getDomains() );
+		if (isset($data['customer.id']) && $data['customer.id'] != '') {
+			$item = $manager->getItem($data['customer.id'], $this->getDomains());
 		} else {
 			$item = $manager->createItem();
 		}
 
-		$item->fromArray( $data );
-		$item->setGroups( array_intersect( array_keys( $this->getGroupItems() ), $item->getGroups() ) );
+		$item->fromArray($data);
+		$item->setGroups(array_intersect(array_keys($this->getGroupItems()), $item->getGroups()));
 
 		return $item;
 	}
@@ -553,11 +553,11 @@ 
 	 * @param \Aimeos\MShop\Customer\Item\Iface $item Customer item object
 	 * @return string[] Multi-dimensional associative list of item data
 	 */
-	protected function toArray( \Aimeos\MShop\Customer\Item\Iface $item, $copy = false )
+	protected function toArray(\Aimeos\MShop\Customer\Item\Iface $item, $copy = false)
 	{
-		$data = $item->toArray( true );
+		$data = $item->toArray(true);
 
-		if( $copy === true )
+		if ($copy === true)
 		{
 			$data['customer.siteid'] = $this->getContext()->getLocale()->getSiteId();
 			$data['customer.email'] = '';
@@ -574,7 +574,7 @@ 
 	 * @param \Aimeos\MW\View\Iface $view View object with data assigned
 	 * @return string HTML output
 	 */
-	protected function render( \Aimeos\MW\View\Iface $view )
+	protected function render(\Aimeos\MW\View\Iface $view)
 	{
 		/** admin/jqadm/customer/template-item
 		 * Relative path to the HTML body template for the customer item.
@@ -598,6 +598,6 @@ 
 		$tplconf = 'admin/jqadm/customer/template-item';
 		$default = 'customer/item-standard';
 
-		return $view->render( $view->config( $tplconf, $default ) );
+		return $view->render($view->config($tplconf, $default));
 	}
 }

admin/jqadm/src/Admin/JQAdm/Supplier/Address/Standard.php

Spacing +38 added lines, -38 removed lines

@@ -10,7 +10,7 @@ 
 
 namespace Aimeos\Admin\JQAdm\Supplier\Address;
 
-sprintf( 'address' ); // for translation
+sprintf('address'); // for translation
 
 
 /**
@@ -44,14 +44,14 @@ 
 	{
 		$view = $this->getView();
 
-		$view->addressData = $this->toArray( $view->item, true );
+		$view->addressData = $this->toArray($view->item, true);
 		$view->addressBody = '';
 
-		foreach( $this->getSubClients() as $client ) {
+		foreach ($this->getSubClients() as $client) {
 			$view->addressBody .= $client->copy();
 		}
 
-		return $this->render( $view );
+		return $this->render($view);
 	}
 
 
@@ -64,20 +64,20 @@ 
 	{
 		$view = $this->getView();
 		$siteid = $this->getContext()->getLocale()->getSiteId();
-		$data = $view->param( 'address', [] );
+		$data = $view->param('address', []);
 
-		foreach( $data as $idx => $entry ) {
+		foreach ($data as $idx => $entry) {
 			$data[$idx]['supplier.address.siteid'] = $siteid;
 		}
 
 		$view->addressData = $data;
 		$view->addressBody = '';
 
-		foreach( $this->getSubClients() as $client ) {
+		foreach ($this->getSubClients() as $client) {
 			$view->addressBody .= $client->create();
 		}
 
-		return $this->render( $view );
+		return $this->render($view);
 	}
 
 
@@ -90,14 +90,14 @@ 
 	{
 		$view = $this->getView();
 
-		$view->addressData = $this->toArray( $view->item );
+		$view->addressData = $this->toArray($view->item);
 		$view->addressBody = '';
 
-		foreach( $this->getSubClients() as $client ) {
+		foreach ($this->getSubClients() as $client) {
 			$view->addressBody .= $client->get();
 		}
 
-		return $this->render( $view );
+		return $this->render($view);
 	}
 
 
@@ -110,26 +110,26 @@ 
 
 		try
 		{
-			$this->fromArray( $view->item, $view->param( 'address', [] ) );
+			$this->fromArray($view->item, $view->param('address', []));
 			$view->addressBody = '';
 
-			foreach( $this->getSubClients() as $client ) {
+			foreach ($this->getSubClients() as $client) {
 				$view->addressBody .= $client->save();
 			}
 
 			return;
 		}
-		catch( \Aimeos\MShop\Exception $e )
+		catch (\Aimeos\MShop\Exception $e)
 		{
-			$error = array( 'supplier-item-address' => $this->getContext()->getI18n()->dt( 'mshop', $e->getMessage() ) );
-			$view->errors = $view->get( 'errors', [] ) + $error;
-			$this->logException( $e );
+			$error = array('supplier-item-address' => $this->getContext()->getI18n()->dt('mshop', $e->getMessage()));
+			$view->errors = $view->get('errors', []) + $error;
+			$this->logException($e);
 		}
-		catch( \Exception $e )
+		catch (\Exception $e)
 		{
-			$error = array( 'supplier-item-address' => $e->getMessage() . ', ' . $e->getFile() . ':' . $e->getLine() );
-			$view->errors = $view->get( 'errors', [] ) + $error;
-			$this->logException( $e );
+			$error = array('supplier-item-address' => $e->getMessage() . ', ' . $e->getFile() . ':' . $e->getLine());
+			$view->errors = $view->get('errors', []) + $error;
+			$this->logException($e);
 		}
 
 		throw new \Aimeos\Admin\JQAdm\Exception();
@@ -143,7 +143,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/supplier/address/decorators/excludes
 		 * Excludes decorators added by the "common" option from the supplier JQAdm client
@@ -218,7 +218,7 @@ 
 		 * @see admin/jqadm/supplier/address/decorators/excludes
 		 * @see admin/jqadm/supplier/address/decorators/global
 		 */
-		return $this->createSubClient( 'supplier/address/' . $type, $name );
+		return $this->createSubClient('supplier/address/' . $type, $name);
 	}
 
 
@@ -262,7 +262,7 @@ 
 		 * @since 2017.10
 		 * @category Developer
 		 */
-		return $this->getContext()->getConfig()->get( 'admin/jqadm/supplier/address/standard/subparts', [] );
+		return $this->getContext()->getConfig()->get('admin/jqadm/supplier/address/standard/subparts', []);
 	}
 
 
@@ -272,29 +272,29 @@ 
 	 * @param \Aimeos\MShop\Supplier\Item\Iface $item Supplier item object without referenced domain items
 	 * @param string[] $data Data array
 	 */
-	protected function fromArray( \Aimeos\MShop\Supplier\Item\Iface $item, array $data )
+	protected function fromArray(\Aimeos\MShop\Supplier\Item\Iface $item, array $data)
 	{
-		$manager = \Aimeos\MShop::create( $this->getContext(), 'supplier/address' );
+		$manager = \Aimeos\MShop::create($this->getContext(), 'supplier/address');
 
 		$addrItems = $item->getAddressItems();
 
-		foreach( $data as $entry )
+		foreach ($data as $entry)
 		{
-			if( isset( $addrItems[$entry['supplier.address.id']] ) )
+			if (isset($addrItems[$entry['supplier.address.id']]))
 			{
 				$addrItem = $addrItems[$entry['supplier.address.id']];
-				unset( $addrItems[$entry['supplier.address.id']] );
+				unset($addrItems[$entry['supplier.address.id']]);
 			}
 			else
 			{
 				$addrItem = $manager->createItem();
 			}
 
-			$addrItem->fromArray( $entry );
-			$item->addAddressItem( $addrItem );
+			$addrItem->fromArray($entry);
+			$item->addAddressItem($addrItem);
 		}
 
-		$item->deleteAddressItems( $addrItems );
+		$item->deleteAddressItems($addrItems);
 	}
 
 
@@ -305,16 +305,16 @@ 
 	 * @param boolean $copy True if items should be copied, false if not
 	 * @return string[] Multi-dimensional associative list of item data
 	 */
-	protected function toArray( \Aimeos\MShop\Supplier\Item\Iface $item, $copy = false )
+	protected function toArray(\Aimeos\MShop\Supplier\Item\Iface $item, $copy = false)
 	{
 		$siteId = $this->getContext()->getLocale()->getSiteId();
 		$data = [];
 
-		foreach( $item->getAddressItems() as $addrItem )
+		foreach ($item->getAddressItems() as $addrItem)
 		{
-			$list = $addrItem->toArray( true );
+			$list = $addrItem->toArray(true);
 
-			if( $copy === true )
+			if ($copy === true)
 			{
 				$list['supplier.address.siteid'] = $siteId;
 				$list['supplier.address.id'] = '';
@@ -333,7 +333,7 @@ 
 	 * @param \Aimeos\MW\View\Iface $view View object with data assigned
 	 * @return string HTML output
 	 */
-	protected function render( \Aimeos\MW\View\Iface $view )
+	protected function render(\Aimeos\MW\View\Iface $view)
 	{
 		/** admin/jqadm/supplier/address/template-item
 		 * Relative path to the HTML body template of the address subpart for suppliers.
@@ -357,6 +357,6 @@ 
 		$tplconf = 'admin/jqadm/supplier/address/template-item';
 		$default = 'supplier/item-address-standard';
 
-		return $view->render( $view->config( $tplconf, $default ) );
+		return $view->render($view->config($tplconf, $default));
 	}
 }

admin/jqadm/src/Admin/JQAdm/Supplier/Standard.php

Spacing +104 added lines, -104 removed lines

@@ -10,7 +10,7 @@ 
 
 namespace Aimeos\Admin\JQAdm\Supplier;
 
-sprintf( 'supplier' ); // for translation
+sprintf('supplier'); // for translation
 
 
 /**
@@ -35,37 +35,37 @@ 
 
 		try
 		{
-			if( ( $id = $view->param( 'id' ) ) === null ) {
-				throw new \Aimeos\Admin\JQAdm\Exception( sprintf( 'Required parameter "%1$s" is missing', 'id' ) );
+			if (($id = $view->param('id')) === null) {
+				throw new \Aimeos\Admin\JQAdm\Exception(sprintf('Required parameter "%1$s" is missing', 'id'));
 			}
 
-			$manager = \Aimeos\MShop::create( $context, 'supplier' );
-			$view->item = $manager->getItem( $id, $this->getDomains() );
+			$manager = \Aimeos\MShop::create($context, 'supplier');
+			$view->item = $manager->getItem($id, $this->getDomains());
 
-			$view->itemData = $this->toArray( $view->item, true );
+			$view->itemData = $this->toArray($view->item, true);
 			$view->itemSubparts = $this->getSubClientNames();
 			$view->itemBody = '';
 
-			foreach( $this->getSubClients() as $idx => $client )
+			foreach ($this->getSubClients() as $idx => $client)
 			{
 				$view->tabindex = ++$idx + 1;
 				$view->itemBody .= $client->copy();
 			}
 		}
-		catch( \Aimeos\MShop\Exception $e )
+		catch (\Aimeos\MShop\Exception $e)
 		{
-			$error = array( 'supplier-item' => $context->getI18n()->dt( 'mshop', $e->getMessage() ) );
-			$view->errors = $view->get( 'errors', [] ) + $error;
-			$this->logException( $e );
+			$error = array('supplier-item' => $context->getI18n()->dt('mshop', $e->getMessage()));
+			$view->errors = $view->get('errors', []) + $error;
+			$this->logException($e);
 		}
-		catch( \Exception $e )
+		catch (\Exception $e)
 		{
-			$error = array( 'supplier-item' => $e->getMessage() . ', ' . $e->getFile() . ':' . $e->getLine() );
-			$view->errors = $view->get( 'errors', [] ) + $error;
-			$this->logException( $e );
+			$error = array('supplier-item' => $e->getMessage() . ', ' . $e->getFile() . ':' . $e->getLine());
+			$view->errors = $view->get('errors', []) + $error;
+			$this->logException($e);
 		}
 
-		return $this->render( $view );
+		return $this->render($view);
 	}
 
 
@@ -81,10 +81,10 @@ 
 
 		try
 		{
-			$data = $view->param( 'item', [] );
+			$data = $view->param('item', []);
 
-			if( !isset( $view->item ) ) {
-				$view->item = \Aimeos\MShop::create( $context, 'supplier' )->createItem();
+			if (!isset($view->item)) {
+				$view->item = \Aimeos\MShop::create($context, 'supplier')->createItem();
 			}
 
 			$data['supplier.siteid'] = $view->item->getSiteId();
@@ -93,26 +93,26 @@ 
 			$view->itemData = $data;
 			$view->itemBody = '';
 
-			foreach( $this->getSubClients() as $idx => $client )
+			foreach ($this->getSubClients() as $idx => $client)
 			{
 				$view->tabindex = ++$idx + 1;
 				$view->itemBody .= $client->create();
 			}
 		}
-		catch( \Aimeos\MShop\Exception $e )
+		catch (\Aimeos\MShop\Exception $e)
 		{
-			$error = array( 'supplier-item' => $context->getI18n()->dt( 'mshop', $e->getMessage() ) );
-			$view->errors = $view->get( 'errors', [] ) + $error;
-			$this->logException( $e );
+			$error = array('supplier-item' => $context->getI18n()->dt('mshop', $e->getMessage()));
+			$view->errors = $view->get('errors', []) + $error;
+			$this->logException($e);
 		}
-		catch( \Exception $e )
+		catch (\Exception $e)
 		{
-			$error = array( 'supplier-item' => $e->getMessage() . ', ' . $e->getFile() . ':' . $e->getLine() );
-			$view->errors = $view->get( 'errors', [] ) + $error;
-			$this->logException( $e );
+			$error = array('supplier-item' => $e->getMessage() . ', ' . $e->getFile() . ':' . $e->getLine());
+			$view->errors = $view->get('errors', []) + $error;
+			$this->logException($e);
 		}
 
-		return $this->render( $view );
+		return $this->render($view);
 	}
 
 
@@ -126,39 +126,39 @@ 
 		$view = $this->getView();
 		$context = $this->getContext();
 
-		$manager = \Aimeos\MShop::create( $context, 'supplier' );
+		$manager = \Aimeos\MShop::create($context, 'supplier');
 		$manager->begin();
 
 		try
 		{
-			if( ( $id = $view->param( 'id' ) ) === null ) {
-				throw new \Aimeos\Admin\JQAdm\Exception( sprintf( 'Required parameter "%1$s" is missing', 'id' ) );
+			if (($id = $view->param('id')) === null) {
+				throw new \Aimeos\Admin\JQAdm\Exception(sprintf('Required parameter "%1$s" is missing', 'id'));
 			}
 
-			$view->item = $manager->getItem( $id, $this->getDomains() );
+			$view->item = $manager->getItem($id, $this->getDomains());
 
-			foreach( $this->getSubClients() as $client ) {
+			foreach ($this->getSubClients() as $client) {
 				$client->delete();
 			}
 
-			$manager->saveItem( $view->item );
-			$manager->deleteItem( $id );
+			$manager->saveItem($view->item);
+			$manager->deleteItem($id);
 			$manager->commit();
 
-			$this->nextAction( $view, 'search', 'supplier', null, 'delete' );
+			$this->nextAction($view, 'search', 'supplier', null, 'delete');
 			return;
 		}
-		catch( \Aimeos\MShop\Exception $e )
+		catch (\Aimeos\MShop\Exception $e)
 		{
-			$error = array( 'supplier-item' => $context->getI18n()->dt( 'mshop', $e->getMessage() ) );
-			$view->errors = $view->get( 'errors', [] ) + $error;
-			$this->logException( $e );
+			$error = array('supplier-item' => $context->getI18n()->dt('mshop', $e->getMessage()));
+			$view->errors = $view->get('errors', []) + $error;
+			$this->logException($e);
 		}
-		catch( \Exception $e )
+		catch (\Exception $e)
 		{
-			$error = array( 'supplier-item' => $e->getMessage() . ', ' . $e->getFile() . ':' . $e->getLine() );
-			$view->errors = $view->get( 'errors', [] ) + $error;
-			$this->logException( $e );
+			$error = array('supplier-item' => $e->getMessage() . ', ' . $e->getFile() . ':' . $e->getLine());
+			$view->errors = $view->get('errors', []) + $error;
+			$this->logException($e);
 		}
 
 		$manager->rollback();
@@ -179,37 +179,37 @@ 
 
 		try
 		{
-			if( ( $id = $view->param( 'id' ) ) === null ) {
-				throw new \Aimeos\Admin\JQAdm\Exception( sprintf( 'Required parameter "%1$s" is missing', 'id' ) );
+			if (($id = $view->param('id')) === null) {
+				throw new \Aimeos\Admin\JQAdm\Exception(sprintf('Required parameter "%1$s" is missing', 'id'));
 			}
 
-			$manager = \Aimeos\MShop::create( $context, 'supplier' );
+			$manager = \Aimeos\MShop::create($context, 'supplier');
 
-			$view->item = $manager->getItem( $id, $this->getDomains() );
+			$view->item = $manager->getItem($id, $this->getDomains());
 			$view->itemSubparts = $this->getSubClientNames();
-			$view->itemData = $this->toArray( $view->item );
+			$view->itemData = $this->toArray($view->item);
 			$view->itemBody = '';
 
-			foreach( $this->getSubClients() as $idx => $client )
+			foreach ($this->getSubClients() as $idx => $client)
 			{
 				$view->tabindex = ++$idx + 1;
 				$view->itemBody .= $client->get();
 			}
 		}
-		catch( \Aimeos\MShop\Exception $e )
+		catch (\Aimeos\MShop\Exception $e)
 		{
-			$error = array( 'supplier-item' => $context->getI18n()->dt( 'mshop', $e->getMessage() ) );
-			$view->errors = $view->get( 'errors', [] ) + $error;
-			$this->logException( $e );
+			$error = array('supplier-item' => $context->getI18n()->dt('mshop', $e->getMessage()));
+			$view->errors = $view->get('errors', []) + $error;
+			$this->logException($e);
 		}
-		catch( \Exception $e )
+		catch (\Exception $e)
 		{
-			$error = array( 'supplier-item' => $e->getMessage() . ', ' . $e->getFile() . ':' . $e->getLine() );
-			$view->errors = $view->get( 'errors', [] ) + $error;
-			$this->logException( $e );
+			$error = array('supplier-item' => $e->getMessage() . ', ' . $e->getFile() . ':' . $e->getLine());
+			$view->errors = $view->get('errors', []) + $error;
+			$this->logException($e);
 		}
 
-		return $this->render( $view );
+		return $this->render($view);
 	}
 
 
@@ -223,40 +223,40 @@ 
 		$view = $this->getView();
 		$context = $this->getContext();
 
-		$manager = \Aimeos\MShop::create( $context, 'supplier' );
+		$manager = \Aimeos\MShop::create($context, 'supplier');
 		$manager->begin();
 
 		try
 		{
-			$item = $this->fromArray( $view->param( 'item', [] ) );
-			$view->item = $item->getId() ? $item : $manager->saveItem( $item );
+			$item = $this->fromArray($view->param('item', []));
+			$view->item = $item->getId() ? $item : $manager->saveItem($item);
 			$view->itemBody = '';
 
-			foreach( $this->getSubClients() as $client ) {
+			foreach ($this->getSubClients() as $client) {
 				$view->itemBody .= $client->save();
 			}
 
-			$manager->saveItem( clone $view->item );
+			$manager->saveItem(clone $view->item);
 			$manager->commit();
 
-			$this->nextAction( $view, $view->param( 'next' ), 'supplier', $view->item->getId(), 'save' );
+			$this->nextAction($view, $view->param('next'), 'supplier', $view->item->getId(), 'save');
 			return;
 		}
-		catch( \Aimeos\Admin\JQAdm\Exception $e )
+		catch (\Aimeos\Admin\JQAdm\Exception $e)
 		{
 			// fall through to create
 		}
-		catch( \Aimeos\MShop\Exception $e )
+		catch (\Aimeos\MShop\Exception $e)
 		{
-			$error = array( 'supplier-item' => $context->getI18n()->dt( 'mshop', $e->getMessage() ) );
-			$view->errors = $view->get( 'errors', [] ) + $error;
-			$this->logException( $e );
+			$error = array('supplier-item' => $context->getI18n()->dt('mshop', $e->getMessage()));
+			$view->errors = $view->get('errors', []) + $error;
+			$this->logException($e);
 		}
-		catch( \Exception $e )
+		catch (\Exception $e)
 		{
-			$error = array( 'supplier-item' => $e->getMessage() . ', ' . $e->getFile() . ':' . $e->getLine() );
-			$view->errors = $view->get( 'errors', [] ) + $error;
-			$this->logException( $e );
+			$error = array('supplier-item' => $e->getMessage() . ', ' . $e->getFile() . ':' . $e->getLine());
+			$view->errors = $view->get('errors', []) + $error;
+			$this->logException($e);
 		}
 
 		$manager->rollback();
@@ -278,31 +278,31 @@ 
 		try
 		{
 			$total = 0;
-			$params = $this->storeSearchParams( $view->param(), 'supplier' );
-			$manager = \Aimeos\MShop::create( $context, 'supplier' );
-			$search = $this->initCriteria( $manager->createSearch(), $params );
+			$params = $this->storeSearchParams($view->param(), 'supplier');
+			$manager = \Aimeos\MShop::create($context, 'supplier');
+			$search = $this->initCriteria($manager->createSearch(), $params);
 
-			$view->items = $manager->searchItems( $search, $this->getDomains(), $total );
-			$view->filterAttributes = $manager->getSearchAttributes( true );
+			$view->items = $manager->searchItems($search, $this->getDomains(), $total);
+			$view->filterAttributes = $manager->getSearchAttributes(true);
 			$view->filterOperators = $search->getOperators();
 			$view->total = $total;
 			$view->itemBody = '';
 
-			foreach( $this->getSubClients() as $client ) {
+			foreach ($this->getSubClients() as $client) {
 				$view->itemBody .= $client->search();
 			}
 		}
-		catch( \Aimeos\MShop\Exception $e )
+		catch (\Aimeos\MShop\Exception $e)
 		{
-			$error = array( 'supplier-item' => $context->getI18n()->dt( 'mshop', $e->getMessage() ) );
-			$view->errors = $view->get( 'errors', [] ) + $error;
-			$this->logException( $e );
+			$error = array('supplier-item' => $context->getI18n()->dt('mshop', $e->getMessage()));
+			$view->errors = $view->get('errors', []) + $error;
+			$this->logException($e);
 		}
-		catch( \Exception $e )
+		catch (\Exception $e)
 		{
-			$error = array( 'supplier-item' => $e->getMessage() . ', ' . $e->getFile() . ':' . $e->getLine() );
-			$view->errors = $view->get( 'errors', [] ) + $error;
-			$this->logException( $e );
+			$error = array('supplier-item' => $e->getMessage() . ', ' . $e->getFile() . ':' . $e->getLine());
+			$view->errors = $view->get('errors', []) + $error;
+			$this->logException($e);
 		}
 
 		/** admin/jqadm/supplier/template-list
@@ -327,7 +327,7 @@ 
 		$tplconf = 'admin/jqadm/supplier/template-list';
 		$default = 'supplier/list-standard';
 
-		return $view->render( $view->config( $tplconf, $default ) );
+		return $view->render($view->config($tplconf, $default));
 	}
 
 
@@ -338,7 +338,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/supplier/decorators/excludes
 		 * Excludes decorators added by the "common" option from the supplier JQAdm client
@@ -413,7 +413,7 @@ 
 		 * @see admin/jqadm/supplier/decorators/excludes
 		 * @see admin/jqadm/supplier/decorators/global
 		 */
-		return $this->createSubClient( 'supplier/' . $type, $name );
+		return $this->createSubClient('supplier/' . $type, $name);
 	}
 
 
@@ -437,7 +437,7 @@ 
 		 */
 		$domains = ['media', 'supplier/address', 'text'];
 
-		return $this->getContext()->getConfig()->get( 'admin/jqadm/supplier/domains', $domains );
+		return $this->getContext()->getConfig()->get('admin/jqadm/supplier/domains', $domains);
 	}
 
 
@@ -481,7 +481,7 @@ 
 		 * @since 2017.10
 		 * @category Developer
 		 */
-		return $this->getContext()->getConfig()->get( 'admin/jqadm/supplier/standard/subparts', [] );
+		return $this->getContext()->getConfig()->get('admin/jqadm/supplier/standard/subparts', []);
 	}
 
 
@@ -492,17 +492,17 @@ 
 	 * @param string[] Data array
 	 * @return \Aimeos\MShop\Supplier\Item\Iface New supplier item object
 	 */
-	protected function fromArray( array $data )
+	protected function fromArray(array $data)
 	{
-		$manager = \Aimeos\MShop::create( $this->getContext(), 'supplier' );
+		$manager = \Aimeos\MShop::create($this->getContext(), 'supplier');
 
-		if( isset( $data['supplier.id'] ) && $data['supplier.id'] != '' ) {
-			$item = $manager->getItem( $data['supplier.id'], $this->getDomains() );
+		if (isset($data['supplier.id']) && $data['supplier.id'] != '') {
+			$item = $manager->getItem($data['supplier.id'], $this->getDomains());
 		} else {
 			$item = $manager->createItem();
 		}
 
-		$item->fromArray( $data );
+		$item->fromArray($data);
 
 		return $item;
 	}
@@ -514,11 +514,11 @@ 
 	 * @param \Aimeos\MShop\Supplier\Item\Iface $item Supplier item object
 	 * @return string[] Multi-dimensional associative list of item data
 	 */
-	protected function toArray( \Aimeos\MShop\Supplier\Item\Iface $item, $copy = false )
+	protected function toArray(\Aimeos\MShop\Supplier\Item\Iface $item, $copy = false)
 	{
-		$data = $item->toArray( true );
+		$data = $item->toArray(true);
 
-		if( $copy === true )
+		if ($copy === true)
 		{
 			$data['supplier.siteid'] = $this->getContext()->getLocale()->getSiteId();
 			$data['supplier.code'] = $data['supplier.code'] . '_copy';
@@ -535,7 +535,7 @@ 
 	 * @param \Aimeos\MW\View\Iface $view View object with data assigned
 	 * @return string HTML output
 	 */
-	protected function render( \Aimeos\MW\View\Iface $view )
+	protected function render(\Aimeos\MW\View\Iface $view)
 	{
 		/** admin/jqadm/supplier/template-item
 		 * Relative path to the HTML body template for the supplier item.
@@ -559,6 +559,6 @@ 
 		$tplconf = 'admin/jqadm/supplier/template-item';
 		$default = 'supplier/item-standard';
 
-		return $view->render( $view->config( $tplconf, $default ) );
+		return $view->render($view->config($tplconf, $default));
 	}
 }