aimeos / ai-admin-jqadm

admin/jqadm/tests/Admin/JQAdm/Catalog/Image/StandardTest.php

Spacing +64 added lines, -64 removed lines

@@ -21,70 +21,70 @@ 
 		$this->view = \TestHelperJqadm::getView();
 		$this->context = \TestHelperJqadm::getContext();
 
-		$this->object = new \Aimeos\Admin\JQAdm\Catalog\Media\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\Catalog\Media\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);
 	}
 
 
 	public function testCreate()
 	{
-		$manager = \Aimeos\MShop::create( $this->context, 'catalog' );
+		$manager = \Aimeos\MShop::create($this->context, 'catalog');
 
 		$this->view->item = $manager->createItem();
 		$result = $this->object->create();
 
-		$this->assertContains( 'item-media', $result );
-		$this->assertNull( $this->view->get( 'errors' ) );
+		$this->assertContains('item-media', $result);
+		$this->assertNull($this->view->get('errors'));
 	}
 
 
 	public function testCopy()
 	{
-		$manager = \Aimeos\MShop::create( $this->context, 'catalog' );
+		$manager = \Aimeos\MShop::create($this->context, 'catalog');
 
-		$this->view->item = $manager->findItem( 'cafe', ['media'] );
+		$this->view->item = $manager->findItem('cafe', ['media']);
 		$result = $this->object->copy();
 
-		$this->assertNull( $this->view->get( 'errors' ) );
-		$this->assertContains( '"media.preview":"prod_123x103\/195_prod_123x103.jpg"', $result );
+		$this->assertNull($this->view->get('errors'));
+		$this->assertContains('"media.preview":"prod_123x103\/195_prod_123x103.jpg"', $result);
 	}
 
 
 	public function testDelete()
 	{
-		$manager = \Aimeos\MShop::create( $this->context, 'catalog' );
+		$manager = \Aimeos\MShop::create($this->context, 'catalog');
 
 		$this->view->item = $manager->createItem();
 		$result = $this->object->delete();
 
-		$this->assertNull( $this->view->get( 'errors' ) );
-		$this->assertNull( $result );
+		$this->assertNull($this->view->get('errors'));
+		$this->assertNull($result);
 	}
 
 
 	public function testGet()
 	{
-		$manager = \Aimeos\MShop::create( $this->context, 'catalog' );
+		$manager = \Aimeos\MShop::create($this->context, 'catalog');
 
-		$this->view->item = $manager->findItem( 'cafe', ['media'] );
+		$this->view->item = $manager->findItem('cafe', ['media']);
 		$result = $this->object->get();
 
-		$this->assertNull( $this->view->get( 'errors' ) );
-		$this->assertContains( '"media.preview":"prod_123x103\/195_prod_123x103.jpg"', $result );
+		$this->assertNull($this->view->get('errors'));
+		$this->assertContains('"media.preview":"prod_123x103\/195_prod_123x103.jpg"', $result);
 	}
 
 
 	public function testSave()
 	{
-		$manager = \Aimeos\MShop::create( $this->context, 'catalog' );
+		$manager = \Aimeos\MShop::create($this->context, 'catalog');
 		$this->view->item = $manager->createItem();
 
 		$param = array(
@@ -98,108 +98,108 @@ 
 			]],
 		);
 
-		$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);
 
-		$file = $this->getMockBuilder( \Psr\Http\Message\UploadedFileInterface::class )->getMock();
-		$request = $this->getMockBuilder( \Psr\Http\Message\ServerRequestInterface::class )->getMock();
-		$request->expects( $this->any() )->method( 'getUploadedFiles' )
-			->will( $this->returnValue( ['media' => [0 => ['file' => $file] ] ] ) );
+		$file = $this->getMockBuilder(\Psr\Http\Message\UploadedFileInterface::class)->getMock();
+		$request = $this->getMockBuilder(\Psr\Http\Message\ServerRequestInterface::class)->getMock();
+		$request->expects($this->any())->method('getUploadedFiles')
+			->will($this->returnValue(['media' => [0 => ['file' => $file]]]));
 
-		$helper = new \Aimeos\MW\View\Helper\Request\Standard( $this->view, $request, '127.0.0.1', 'test' );
-		$this->view ->addHelper( 'request', $helper );
+		$helper = new \Aimeos\MW\View\Helper\Request\Standard($this->view, $request, '127.0.0.1', 'test');
+		$this->view ->addHelper('request', $helper);
 
 
 		$name = 'AdminJQAdmCatalogMediaSave';
-		$this->context->getConfig()->set( 'controller/common/media/name', $name );
+		$this->context->getConfig()->set('controller/common/media/name', $name);
 
-		$cntlStub = $this->getMockBuilder( '\\Aimeos\\Controller\\Common\\Media\\Standard' )
-			->setConstructorArgs( array( $this->context ) )
-			->setMethods( array( 'add' ) )
+		$cntlStub = $this->getMockBuilder('\\Aimeos\\Controller\\Common\\Media\\Standard')
+			->setConstructorArgs(array($this->context))
+			->setMethods(array('add'))
 			->getMock();
 
-		\Aimeos\Controller\Common\Media\Factory::injectController( '\\Aimeos\\Controller\\Common\\Media\\' . $name, $cntlStub );
+		\Aimeos\Controller\Common\Media\Factory::injectController('\\Aimeos\\Controller\\Common\\Media\\' . $name, $cntlStub);
 
-		$cntlStub->expects( $this->once() )->method( 'add' );
+		$cntlStub->expects($this->once())->method('add');
 
 
 		$result = $this->object->save();
 
 
-		$this->assertNull( $this->view->get( 'errors' ) );
-		$this->assertNull( $result );
-		$this->assertEquals( 1, count( $this->view->item->getListItems() ) );
+		$this->assertNull($this->view->get('errors'));
+		$this->assertNull($result);
+		$this->assertEquals(1, count($this->view->item->getListItems()));
 
-		foreach( $this->view->item->getListItems( 'media' ) as $listItem )
+		foreach ($this->view->item->getListItems('media') as $listItem)
 		{
-			$this->assertEquals( 'media', $listItem->getDomain() );
+			$this->assertEquals('media', $listItem->getDomain());
 
 			$refItem = $listItem->getRefItem();
-			$this->assertEquals( 'de', $refItem->getLanguageId() );
-			$this->assertEquals( 'test', $refItem->getLabel() );
+			$this->assertEquals('de', $refItem->getLanguageId());
+			$this->assertEquals('test', $refItem->getLabel());
 		}
 
 
-		$helper = new \Aimeos\MW\View\Helper\Param\Standard( $this->view, ['site' => 'unittest', 'media' => []] );
-		$this->view->addHelper( 'param', $helper );
+		$helper = new \Aimeos\MW\View\Helper\Param\Standard($this->view, ['site' => 'unittest', 'media' => []]);
+		$this->view->addHelper('param', $helper);
 
 		$result = $this->object->save();
 
-		$this->assertNull( $this->view->get( 'errors' ) );
-		$this->assertNull( $result );
-		$this->assertEquals( 0, count( $this->view->item->getListItems() ) );
+		$this->assertNull($this->view->get('errors'));
+		$this->assertNull($result);
+		$this->assertEquals(0, count($this->view->item->getListItems()));
 	}
 
 
 	public function testSaveException()
 	{
-		$object = $this->getClientMock( 'fromArray' );
+		$object = $this->getClientMock('fromArray');
 
-		$object->expects( $this->once() )->method( 'fromArray' )
-			->will( $this->throwException( new \RuntimeException() ) );
+		$object->expects($this->once())->method('fromArray')
+			->will($this->throwException(new \RuntimeException()));
 
-		$this->setExpectedException( \Aimeos\Admin\JQAdm\Exception::class );
+		$this->setExpectedException(\Aimeos\Admin\JQAdm\Exception::class);
 		$object->save();
 	}
 
 
 	public function testSaveMShopException()
 	{
-		$object = $this->getClientMock( 'fromArray' );
+		$object = $this->getClientMock('fromArray');
 
-		$object->expects( $this->once() )->method( 'fromArray' )
-			->will( $this->throwException( new \Aimeos\MShop\Exception() ) );
+		$object->expects($this->once())->method('fromArray')
+			->will($this->throwException(new \Aimeos\MShop\Exception()));
 
-		$this->setExpectedException( \Aimeos\Admin\JQAdm\Exception::class );
+		$this->setExpectedException(\Aimeos\Admin\JQAdm\Exception::class);
 		$object->save();
 	}
 
 
 	public function testSearch()
 	{
-		$this->assertNull( $this->object->search() );
+		$this->assertNull($this->object->search());
 	}
 
 
 	public function testGetSubClient()
 	{
-		$this->setExpectedException( \Aimeos\Admin\JQAdm\Exception::class );
-		$this->object->getSubClient( 'unknown' );
+		$this->setExpectedException(\Aimeos\Admin\JQAdm\Exception::class);
+		$this->object->getSubClient('unknown');
 	}
 
 
-	public function getClientMock( $method )
+	public function getClientMock($method)
 	{
-		$object = $this->getMockBuilder( \Aimeos\Admin\JQAdm\Catalog\Media\Standard::class )
-			->setConstructorArgs( array( $this->context, \TestHelperJqadm::getTemplatePaths() ) )
-			->setMethods( [$method] )
+		$object = $this->getMockBuilder(\Aimeos\Admin\JQAdm\Catalog\Media\Standard::class)
+			->setConstructorArgs(array($this->context, \TestHelperJqadm::getTemplatePaths()))
+			->setMethods([$method])
 			->getMock();
 
 		$view = \TestHelperJqadm::getView();
-		$view->item = \Aimeos\MShop::create( $this->context, 'catalog' )->createItem();
+		$view->item = \Aimeos\MShop::create($this->context, 'catalog')->createItem();
 
-		$object->setAimeos( \TestHelperJqadm::getAimeos() );
-		$object->setView( $view );
+		$object->setAimeos(\TestHelperJqadm::getAimeos());
+		$object->setView($view);
 
 		return $object;
 	}

admin/jqadm/tests/Admin/JQAdm/Catalog/Image/Property/StandardTest.php

Spacing +43 added lines, -43 removed lines

@@ -21,40 +21,40 @@ 
 		$this->view = \TestHelperJqadm::getView();
 		$this->context = \TestHelperJqadm::getContext();
 
-		$this->object = new \Aimeos\Admin\JQAdm\Catalog\Media\Property\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\Catalog\Media\Property\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);
 	}
 
 
 	public function testCreate()
 	{
-		$manager = \Aimeos\MShop::create( $this->context, 'catalog' );
+		$manager = \Aimeos\MShop::create($this->context, 'catalog');
 
 		$this->view->item = $manager->createItem();
 		$result = $this->object->create();
 
-		$this->assertContains( 'Media properties', $result );
-		$this->assertNull( $this->view->get( 'errors' ) );
+		$this->assertContains('Media properties', $result);
+		$this->assertNull($this->view->get('errors'));
 	}
 
 
 	public function testCopy()
 	{
-		$manager = \Aimeos\MShop::create( $this->context, 'catalog' );
+		$manager = \Aimeos\MShop::create($this->context, 'catalog');
 
-		$this->view->item = $manager->findItem( 'cafe', ['media'] );
+		$this->view->item = $manager->findItem('cafe', ['media']);
 		$result = $this->object->copy();
 
-		$this->assertNull( $this->view->get( 'errors' ) );
-		$this->assertContains( 'item-media-property', $result );
+		$this->assertNull($this->view->get('errors'));
+		$this->assertContains('item-media-property', $result);
 	}
 
 
@@ -62,33 +62,33 @@ 
 	{
 		$result = $this->object->delete();
 
-		$this->assertNull( $this->view->get( 'errors' ) );
-		$this->assertNull( $result );
+		$this->assertNull($this->view->get('errors'));
+		$this->assertNull($result);
 	}
 
 
 	public function testGet()
 	{
-		$manager = \Aimeos\MShop::create( $this->context, 'catalog' );
+		$manager = \Aimeos\MShop::create($this->context, 'catalog');
 
-		$this->view->item = $manager->findItem( 'cafe', ['media'] );
+		$this->view->item = $manager->findItem('cafe', ['media']);
 		$result = $this->object->get();
 
-		$this->assertNull( $this->view->get( 'errors' ) );
-		$this->assertContains( 'item-media-property', $result );
+		$this->assertNull($this->view->get('errors'));
+		$this->assertContains('item-media-property', $result);
 	}
 
 
 	public function testSave()
 	{
-		$manager = \Aimeos\MShop::create( $this->context, 'catalog' );
-		$typeManager = \Aimeos\MShop::create( $this->context, 'media/property/type' );
+		$manager = \Aimeos\MShop::create($this->context, 'catalog');
+		$typeManager = \Aimeos\MShop::create($this->context, 'media/property/type');
 
-		$item = $manager->findItem( 'cafe', ['media'] );
-		$item->setCode( 'jqadm-test-media-property' );
-		$item->setId( null );
+		$item = $manager->findItem('cafe', ['media']);
+		$item->setCode('jqadm-test-media-property');
+		$item->setId(null);
 
-		$this->view->item = $manager->insertItem( $item );
+		$this->view->item = $manager->insertItem($item);
 
 		$param = array(
 			'site' => 'unittest',
@@ -104,53 +104,53 @@ 
 			),
 		);
 
-		$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->save();
 
 
-		$manager->deleteItem( $this->view->item->getId() );
+		$manager->deleteItem($this->view->item->getId());
 
-		$this->assertNull( $this->view->get( 'errors' ) );
-		$this->assertNull( $result );
+		$this->assertNull($this->view->get('errors'));
+		$this->assertNull($result);
 
-		$mediaItems = $this->view->item->getRefItems( 'media' );
-		$this->assertEquals( 2, count( $mediaItems ) );
-		$this->assertEquals( 1, count( reset( $mediaItems )->getPropertyItems() ) );
+		$mediaItems = $this->view->item->getRefItems('media');
+		$this->assertEquals(2, count($mediaItems));
+		$this->assertEquals(1, count(reset($mediaItems)->getPropertyItems()));
 	}
 
 
 	public function testSaveException()
 	{
-		$object = $this->getMockBuilder( \Aimeos\Admin\JQAdm\Catalog\Media\Property\Standard::class )
-			->setConstructorArgs( array( $this->context, \TestHelperJqadm::getTemplatePaths() ) )
-			->setMethods( array( 'fromArray' ) )
+		$object = $this->getMockBuilder(\Aimeos\Admin\JQAdm\Catalog\Media\Property\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()));
 
 		$this->view = \TestHelperJqadm::getView();
-		$this->view->item = \Aimeos\MShop::create( $this->context, 'catalog' )->createItem();
+		$this->view->item = \Aimeos\MShop::create($this->context, 'catalog')->createItem();
 
-		$object->setView( $this->view );
+		$object->setView($this->view);
 
-		$this->setExpectedException( \RuntimeException::class );
+		$this->setExpectedException(\RuntimeException::class);
 		$object->save();
 	}
 
 
 	public function testSearch()
 	{
-		$this->assertNull( $this->object->search() );
+		$this->assertNull($this->object->search());
 	}
 
 
 	public function testGetSubClient()
 	{
-		$this->setExpectedException( \Aimeos\Admin\JQAdm\Exception::class );
-		$this->object->getSubClient( 'unknown' );
+		$this->setExpectedException(\Aimeos\Admin\JQAdm\Exception::class);
+		$this->object->getSubClient('unknown');
 	}
 }

admin/jqadm/tests/Admin/JQAdm/Service/FactoryTest.php

Spacing +11 added lines, -11 removed lines

@@ -17,42 +17,42 @@
 	protected function setUp()
 	{
 		$this->context = \TestHelperJqadm::getContext();
-		$this->context->setView( \TestHelperJqadm::getView() );
+		$this->context->setView(\TestHelperJqadm::getView());
 	}
 
 
 	public function testCreateClient()
 	{
-		$client = \Aimeos\Admin\JQAdm\Service\Factory::create( $this->context );
-		$this->assertInstanceOf( '\\Aimeos\\Admin\\JQAdm\\Iface', $client );
+		$client = \Aimeos\Admin\JQAdm\Service\Factory::create($this->context);
+		$this->assertInstanceOf('\\Aimeos\\Admin\\JQAdm\\Iface', $client);
 	}
 
 
 	public function testCreateClientName()
 	{
-		$client = \Aimeos\Admin\JQAdm\Service\Factory::create( $this->context, 'Standard' );
-		$this->assertInstanceOf( '\\Aimeos\\Admin\\JQAdm\\Iface', $client );
+		$client = \Aimeos\Admin\JQAdm\Service\Factory::create($this->context, 'Standard');
+		$this->assertInstanceOf('\\Aimeos\\Admin\\JQAdm\\Iface', $client);
 	}
 
 
 	public function testCreateClientNameEmpty()
 	{
-		$this->setExpectedException( '\\Aimeos\\Admin\\JQAdm\\Exception' );
-		\Aimeos\Admin\JQAdm\Service\Factory::create( $this->context, '' );
+		$this->setExpectedException('\\Aimeos\\Admin\\JQAdm\\Exception');
+		\Aimeos\Admin\JQAdm\Service\Factory::create($this->context, '');
 	}
 
 
 	public function testCreateClientNameInvalid()
 	{
-		$this->setExpectedException( '\\Aimeos\\Admin\\JQAdm\\Exception' );
-		\Aimeos\Admin\JQAdm\Service\Factory::create( $this->context, '%service' );
+		$this->setExpectedException('\\Aimeos\\Admin\\JQAdm\\Exception');
+		\Aimeos\Admin\JQAdm\Service\Factory::create($this->context, '%service');
 	}
 
 
 	public function testCreateClientNameNotFound()
 	{
-		$this->setExpectedException( '\\Aimeos\\Admin\\JQAdm\\Exception' );
-		\Aimeos\Admin\JQAdm\Service\Factory::create( $this->context, 'test' );
+		$this->setExpectedException('\\Aimeos\\Admin\\JQAdm\\Exception');
+		\Aimeos\Admin\JQAdm\Service\Factory::create($this->context, 'test');
 	}
 
 }

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

Spacing +94 added lines, -94 removed lines

@@ -19,22 +19,22 @@ 
 	protected function setUp()
 	{
 		$this->view = \TestHelperJqadm::getView();
-		$request = $this->getMockBuilder( \Psr\Http\Message\ServerRequestInterface::class )->getMock();
-		$helper = new \Aimeos\MW\View\Helper\Request\Standard( $this->view, $request, '127.0.0.1', 'test' );
-		$this->view ->addHelper( 'request', $helper );
+		$request = $this->getMockBuilder(\Psr\Http\Message\ServerRequestInterface::class)->getMock();
+		$helper = new \Aimeos\MW\View\Helper\Request\Standard($this->view, $request, '127.0.0.1', 'test');
+		$this->view ->addHelper('request', $helper);
 
 		$this->context = \TestHelperJqadm::getContext();
 
-		$this->object = new \Aimeos\Admin\JQAdm\Service\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\Service\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);
 	}
 
 
@@ -46,10 +46,10 @@ 
 
 	public function testCreateException()
 	{
-		$object = $this->getClientMock( 'getSubClients' );
+		$object = $this->getClientMock('getSubClients');
 
-		$object->expects( $this->once() )->method( 'getSubClients' )
-			->will( $this->throwException( new \RuntimeException() ) );
+		$object->expects($this->once())->method('getSubClients')
+			->will($this->throwException(new \RuntimeException()));
 
 		$object->create();
 	}
@@ -57,10 +57,10 @@ 
 
 	public function testCreateMShopException()
 	{
-		$object = $this->getClientMock( 'getSubClients' );
+		$object = $this->getClientMock('getSubClients');
 
-		$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->create();
 	}
@@ -68,24 +68,24 @@ 
 
 	public function testCopy()
 	{
-		$manager = \Aimeos\MShop::create( $this->context, 'service' );
+		$manager = \Aimeos\MShop::create($this->context, 'service');
 
-		$param = ['site' => 'unittest', 'id' => $manager->findItem( 'unitpaymentcode' )->getId()];
-		$helper = new \Aimeos\MW\View\Helper\Param\Standard( $this->view, $param );
-		$this->view->addHelper( 'param', $helper );
+		$param = ['site' => 'unittest', 'id' => $manager->findItem('unitpaymentcode')->getId()];
+		$helper = new \Aimeos\MW\View\Helper\Param\Standard($this->view, $param);
+		$this->view->addHelper('param', $helper);
 
 		$result = $this->object->copy();
 
-		$this->assertContains( 'unitpaymentcode_copy', $result );
+		$this->assertContains('unitpaymentcode_copy', $result);
 	}
 
 
 	public function testCopyException()
 	{
-		$object = $this->getClientMock( 'getSubClients' );
+		$object = $this->getClientMock('getSubClients');
 
-		$object->expects( $this->once() )->method( 'getSubClients' )
-			->will( $this->throwException( new \RuntimeException() ) );
+		$object->expects($this->once())->method('getSubClients')
+			->will($this->throwException(new \RuntimeException()));
 
 		$object->copy();
 	}
@@ -93,10 +93,10 @@ 
 
 	public function testCopyMShopException()
 	{
-		$object = $this->getClientMock( 'getSubClients' );
+		$object = $this->getClientMock('getSubClients');
 
-		$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->copy();
 	}
@@ -104,16 +104,16 @@ 
 
 	public function testDelete()
 	{
-		$this->assertNotNull( $this->object->delete() );
+		$this->assertNotNull($this->object->delete());
 	}
 
 
 	public function testDeleteException()
 	{
-		$object = $this->getClientMock( 'getSubClients' );
+		$object = $this->getClientMock('getSubClients');
 
-		$object->expects( $this->exactly( 2 ) )->method( 'getSubClients' )
-			->will( $this->throwException( new \RuntimeException() ) );
+		$object->expects($this->exactly(2))->method('getSubClients')
+			->will($this->throwException(new \RuntimeException()));
 
 		$object->delete();
 	}
@@ -121,10 +121,10 @@ 
 
 	public function testDeleteMShopException()
 	{
-		$object = $this->getClientMock( 'getSubClients' );
+		$object = $this->getClientMock('getSubClients');
 
-		$object->expects( $this->exactly( 2 ) )->method( 'getSubClients' )
-			->will( $this->throwException( new \Aimeos\MShop\Exception() ) );
+		$object->expects($this->exactly(2))->method('getSubClients')
+			->will($this->throwException(new \Aimeos\MShop\Exception()));
 
 		$object->delete();
 	}
@@ -132,24 +132,24 @@ 
 
 	public function testGet()
 	{
-		$manager = \Aimeos\MShop::create( $this->context, 'service' );
+		$manager = \Aimeos\MShop::create($this->context, 'service');
 
-		$param = ['site' => 'unittest', 'id' => $manager->findItem( 'unitpaymentcode' )->getId()];
-		$helper = new \Aimeos\MW\View\Helper\Param\Standard( $this->view, $param );
-		$this->view->addHelper( 'param', $helper );
+		$param = ['site' => 'unittest', 'id' => $manager->findItem('unitpaymentcode')->getId()];
+		$helper = new \Aimeos\MW\View\Helper\Param\Standard($this->view, $param);
+		$this->view->addHelper('param', $helper);
 
 		$result = $this->object->get();
 
-		$this->assertContains( 'unitpaymentlabel', $result );
+		$this->assertContains('unitpaymentlabel', $result);
 	}
 
 
 	public function testGetException()
 	{
-		$object = $this->getClientMock( 'getSubClients' );
+		$object = $this->getClientMock('getSubClients');
 
-		$object->expects( $this->once() )->method( 'getSubClients' )
-			->will( $this->throwException( new \RuntimeException() ) );
+		$object->expects($this->once())->method('getSubClients')
+			->will($this->throwException(new \RuntimeException()));
 
 		$object->get();
 	}
@@ -157,10 +157,10 @@ 
 
 	public function testGetMShopException()
 	{
-		$object = $this->getClientMock( 'getSubClients' );
+		$object = $this->getClientMock('getSubClients');
 
-		$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->get();
 	}
@@ -168,16 +168,16 @@ 
 
 	public function testGetViewException()
 	{
-		$object = new \Aimeos\Admin\JQAdm\Service\Standard( $this->context, [] );
+		$object = new \Aimeos\Admin\JQAdm\Service\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, 'service' );
+		$manager = \Aimeos\MShop::create($this->context, 'service');
 
 		$param = array(
 			'site' => 'unittest',
@@ -191,27 +191,27 @@ 
 				'service.dateend' => null,
 				'service.position' => '2',
 				'config' => array(
-					'key' => array( 0 => 'test key' ),
-					'val' => array( 0 => 'test value' ),
+					'key' => array(0 => 'test key'),
+					'val' => array(0 => 'test value'),
 				),
 			),
 		);
 
-		$helper = new \Aimeos\MW\View\Helper\Param\Standard( $this->view, $param );
-		$this->view->addHelper( 'param', $helper );
+		$helper = new \Aimeos\MW\View\Helper\Param\Standard($this->view, $param);
+		$this->view->addHelper('param', $helper);
 
 		$this->object->save();
 
-		$manager->deleteItem( $manager->findItem( 'test' )->getId() );
+		$manager->deleteItem($manager->findItem('test')->getId());
 	}
 
 
 	public function testSaveException()
 	{
-		$object = $this->getClientMock( 'fromArray' );
+		$object = $this->getClientMock('fromArray');
 
-		$object->expects( $this->once() )->method( 'fromArray' )
-			->will( $this->throwException( new \RuntimeException() ) );
+		$object->expects($this->once())->method('fromArray')
+			->will($this->throwException(new \RuntimeException()));
 
 		$object->save();
 	}
@@ -219,10 +219,10 @@ 
 
 	public function testSaveMShopException()
 	{
-		$object = $this->getClientMock( 'fromArray' );
+		$object = $this->getClientMock('fromArray');
 
-		$object->expects( $this->once() )->method( 'fromArray' )
-			->will( $this->throwException( new \RuntimeException() ) );
+		$object->expects($this->once())->method('fromArray')
+			->will($this->throwException(new \RuntimeException()));
 
 		$object->save();
 	}
@@ -230,10 +230,10 @@ 
 
 	public function testSaveJQAdmException()
 	{
-		$object = $this->getClientMock( 'fromArray' );
+		$object = $this->getClientMock('fromArray');
 
-		$object->expects( $this->once() )->method( 'fromArray' )
-			->will( $this->throwException( new \RuntimeException() ) );
+		$object->expects($this->once())->method('fromArray')
+			->will($this->throwException(new \RuntimeException()));
 
 		$object->save();
 	}
@@ -244,27 +244,27 @@ 
 		$param = array(
 			'site' => 'unittest', 'lang' => 'de',
 			'filter' => array(
-				'key' => array( 0 => 'service.code' ),
-				'op' => array( 0 => '==' ),
-				'val' => array( 0 => 'unitpaymentcode' ),
+				'key' => array(0 => 'service.code'),
+				'op' => array(0 => '=='),
+				'val' => array(0 => 'unitpaymentcode'),
 			),
-			'sort' => array( 'service.label', '-service.id' ),
+			'sort' => array('service.label', '-service.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( '>unitpaymentlabel<', $result );
+		$this->assertContains('>unitpaymentlabel<', $result);
 	}
 
 
 	public function testSearchException()
 	{
-		$object = $this->getClientMock( 'initCriteria' );
+		$object = $this->getClientMock('initCriteria');
 
-		$object->expects( $this->once() )->method( 'initCriteria' )
-			->will( $this->throwException( new \RuntimeException() ) );
+		$object->expects($this->once())->method('initCriteria')
+			->will($this->throwException(new \RuntimeException()));
 
 		$object->search();
 	}
@@ -272,10 +272,10 @@ 
 
 	public function testSearchMShopException()
 	{
-		$object = $this->getClientMock( 'initCriteria' );
+		$object = $this->getClientMock('initCriteria');
 
-		$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->search();
 	}
@@ -283,34 +283,34 @@ 
 
 	public function testGetSubClient()
 	{
-		$result = $this->object->getSubClient( 'media' );
-		$this->assertInstanceOf( \Aimeos\Admin\JQAdm\Iface::class, $result );
+		$result = $this->object->getSubClient('media');
+		$this->assertInstanceOf(\Aimeos\Admin\JQAdm\Iface::class, $result);
 	}
 
 
 	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');
 	}
 
 
-	public function getClientMock( $method )
+	public function getClientMock($method)
 	{
-		$object = $this->getMockBuilder( \Aimeos\Admin\JQAdm\Service\Standard::class )
-			->setConstructorArgs( array( $this->context, \TestHelperJqadm::getTemplatePaths() ) )
-			->setMethods( [$method] )
+		$object = $this->getMockBuilder(\Aimeos\Admin\JQAdm\Service\Standard::class)
+			->setConstructorArgs(array($this->context, \TestHelperJqadm::getTemplatePaths()))
+			->setMethods([$method])
 			->getMock();
 
-		$object->setAimeos( \TestHelperJqadm::getAimeos() );
-		$object->setView( $this->getViewNoRender() );
+		$object->setAimeos(\TestHelperJqadm::getAimeos());
+		$object->setView($this->getViewNoRender());
 
 		return $object;
 	}
@@ -318,19 +318,19 @@ 
 
 	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, 'service' );
+		$manager = \Aimeos\MShop::create($this->context, 'service');
 
-		$param = ['site' => 'unittest', 'id' => $manager->findItem( 'unitpaymentcode' )->getId()];
-		$helper = new \Aimeos\MW\View\Helper\Param\Standard( $view, $param );
-		$view->addHelper( 'param', $helper );
+		$param = ['site' => 'unittest', 'id' => $manager->findItem('unitpaymentcode')->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/Service/Image/StandardTest.php

Spacing +64 added lines, -64 removed lines

@@ -21,70 +21,70 @@ 
 		$this->view = \TestHelperJqadm::getView();
 		$this->context = \TestHelperJqadm::getContext();
 
-		$this->object = new \Aimeos\Admin\JQAdm\Service\Media\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\Service\Media\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);
 	}
 
 
 	public function testCreate()
 	{
-		$manager = \Aimeos\MShop::create( $this->context, 'service' );
+		$manager = \Aimeos\MShop::create($this->context, 'service');
 
 		$this->view->item = $manager->createItem();
 		$result = $this->object->create();
 
-		$this->assertContains( 'item-media', $result );
-		$this->assertNull( $this->view->get( 'errors' ) );
+		$this->assertContains('item-media', $result);
+		$this->assertNull($this->view->get('errors'));
 	}
 
 
 	public function testCopy()
 	{
-		$manager = \Aimeos\MShop::create( $this->context, 'service' );
+		$manager = \Aimeos\MShop::create($this->context, 'service');
 
-		$this->view->item = $manager->findItem( 'unitcode', ['media'] );
+		$this->view->item = $manager->findItem('unitcode', ['media']);
 		$result = $this->object->copy();
 
-		$this->assertNull( $this->view->get( 'errors' ) );
-		$this->assertContains( '"media.preview":"path\/to\/service.png"', $result );
+		$this->assertNull($this->view->get('errors'));
+		$this->assertContains('"media.preview":"path\/to\/service.png"', $result);
 	}
 
 
 	public function testDelete()
 	{
-		$manager = \Aimeos\MShop::create( $this->context, 'service' );
+		$manager = \Aimeos\MShop::create($this->context, 'service');
 
 		$this->view->item = $manager->createItem();
 		$result = $this->object->delete();
 
-		$this->assertNull( $this->view->get( 'errors' ) );
-		$this->assertNull( $result );
+		$this->assertNull($this->view->get('errors'));
+		$this->assertNull($result);
 	}
 
 
 	public function testGet()
 	{
-		$manager = \Aimeos\MShop::create( $this->context, 'service' );
+		$manager = \Aimeos\MShop::create($this->context, 'service');
 
-		$this->view->item = $manager->findItem( 'unitcode', ['media'] );
+		$this->view->item = $manager->findItem('unitcode', ['media']);
 		$result = $this->object->get();
 
-		$this->assertNull( $this->view->get( 'errors' ) );
-		$this->assertContains( '"media.preview":"path\/to\/service.png"', $result );
+		$this->assertNull($this->view->get('errors'));
+		$this->assertContains('"media.preview":"path\/to\/service.png"', $result);
 	}
 
 
 	public function testSave()
 	{
-		$manager = \Aimeos\MShop::create( $this->context, 'service' );
+		$manager = \Aimeos\MShop::create($this->context, 'service');
 		$this->view->item = $manager->createItem();
 
 
@@ -99,108 +99,108 @@ 
 			]],
 		);
 
-		$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);
 
-		$file = $this->getMockBuilder( \Psr\Http\Message\UploadedFileInterface::class )->getMock();
-		$request = $this->getMockBuilder( \Psr\Http\Message\ServerRequestInterface::class )->getMock();
-		$request->expects( $this->any() )->method( 'getUploadedFiles' )
-			->will( $this->returnValue( ['media' => [0 => ['file' => $file] ] ] ) );
+		$file = $this->getMockBuilder(\Psr\Http\Message\UploadedFileInterface::class)->getMock();
+		$request = $this->getMockBuilder(\Psr\Http\Message\ServerRequestInterface::class)->getMock();
+		$request->expects($this->any())->method('getUploadedFiles')
+			->will($this->returnValue(['media' => [0 => ['file' => $file]]]));
 
-		$helper = new \Aimeos\MW\View\Helper\Request\Standard( $this->view, $request, '127.0.0.1', 'test' );
-		$this->view ->addHelper( 'request', $helper );
+		$helper = new \Aimeos\MW\View\Helper\Request\Standard($this->view, $request, '127.0.0.1', 'test');
+		$this->view ->addHelper('request', $helper);
 
 
 		$name = 'AdminJQAdmServiceMediaSave';
-		$this->context->getConfig()->set( 'controller/common/media/name', $name );
+		$this->context->getConfig()->set('controller/common/media/name', $name);
 
-		$cntlStub = $this->getMockBuilder( '\\Aimeos\\Controller\\Common\\Media\\Standard' )
-			->setConstructorArgs( array( $this->context ) )
-			->setMethods( array( 'add' ) )
+		$cntlStub = $this->getMockBuilder('\\Aimeos\\Controller\\Common\\Media\\Standard')
+			->setConstructorArgs(array($this->context))
+			->setMethods(array('add'))
 			->getMock();
 
-		\Aimeos\Controller\Common\Media\Factory::injectController( '\\Aimeos\\Controller\\Common\\Media\\' . $name, $cntlStub );
+		\Aimeos\Controller\Common\Media\Factory::injectController('\\Aimeos\\Controller\\Common\\Media\\' . $name, $cntlStub);
 
-		$cntlStub->expects( $this->once() )->method( 'add' );
+		$cntlStub->expects($this->once())->method('add');
 
 
 		$result = $this->object->save();
 
 
-		$this->assertNull( $this->view->get( 'errors' ) );
-		$this->assertNull( $result );
-		$this->assertEquals( 1, count( $this->view->item->getListItems() ) );
+		$this->assertNull($this->view->get('errors'));
+		$this->assertNull($result);
+		$this->assertEquals(1, count($this->view->item->getListItems()));
 
-		foreach( $this->view->item->getListItems( 'media' ) as $listItem )
+		foreach ($this->view->item->getListItems('media') as $listItem)
 		{
-			$this->assertEquals( 'media', $listItem->getDomain() );
+			$this->assertEquals('media', $listItem->getDomain());
 
 			$refItem = $listItem->getRefItem();
-			$this->assertEquals( 'de', $refItem->getLanguageId() );
-			$this->assertEquals( 'test', $refItem->getLabel() );
+			$this->assertEquals('de', $refItem->getLanguageId());
+			$this->assertEquals('test', $refItem->getLabel());
 		}
 
 
-		$helper = new \Aimeos\MW\View\Helper\Param\Standard( $this->view, ['site' => 'unittest', 'media' => []] );
-		$this->view->addHelper( 'param', $helper );
+		$helper = new \Aimeos\MW\View\Helper\Param\Standard($this->view, ['site' => 'unittest', 'media' => []]);
+		$this->view->addHelper('param', $helper);
 
 		$result = $this->object->save();
 
-		$this->assertNull( $this->view->get( 'errors' ) );
-		$this->assertNull( $result );
-		$this->assertEquals( 0, count( $this->view->item->getListItems() ) );
+		$this->assertNull($this->view->get('errors'));
+		$this->assertNull($result);
+		$this->assertEquals(0, count($this->view->item->getListItems()));
 	}
 
 
 	public function testSaveException()
 	{
-		$object = $this->getClientMock( 'fromArray' );
+		$object = $this->getClientMock('fromArray');
 
-		$object->expects( $this->once() )->method( 'fromArray' )
-			->will( $this->throwException( new \RuntimeException() ) );
+		$object->expects($this->once())->method('fromArray')
+			->will($this->throwException(new \RuntimeException()));
 
-		$this->setExpectedException( \Aimeos\Admin\JQAdm\Exception::class );
+		$this->setExpectedException(\Aimeos\Admin\JQAdm\Exception::class);
 		$object->save();
 	}
 
 
 	public function testSaveMShopException()
 	{
-		$object = $this->getClientMock( 'fromArray' );
+		$object = $this->getClientMock('fromArray');
 
-		$object->expects( $this->once() )->method( 'fromArray' )
-			->will( $this->throwException( new \Aimeos\MShop\Exception() ) );
+		$object->expects($this->once())->method('fromArray')
+			->will($this->throwException(new \Aimeos\MShop\Exception()));
 
-		$this->setExpectedException( \Aimeos\Admin\JQAdm\Exception::class );
+		$this->setExpectedException(\Aimeos\Admin\JQAdm\Exception::class);
 		$object->save();
 	}
 
 
 	public function testSearch()
 	{
-		$this->assertNull( $this->object->search() );
+		$this->assertNull($this->object->search());
 	}
 
 
 	public function testGetSubClient()
 	{
-		$this->setExpectedException( \Aimeos\Admin\JQAdm\Exception::class );
-		$this->object->getSubClient( 'unknown' );
+		$this->setExpectedException(\Aimeos\Admin\JQAdm\Exception::class);
+		$this->object->getSubClient('unknown');
 	}
 
 
-	public function getClientMock( $method )
+	public function getClientMock($method)
 	{
-		$object = $this->getMockBuilder( \Aimeos\Admin\JQAdm\Service\Media\Standard::class )
-			->setConstructorArgs( array( $this->context, \TestHelperJqadm::getTemplatePaths() ) )
-			->setMethods( [$method] )
+		$object = $this->getMockBuilder(\Aimeos\Admin\JQAdm\Service\Media\Standard::class)
+			->setConstructorArgs(array($this->context, \TestHelperJqadm::getTemplatePaths()))
+			->setMethods([$method])
 			->getMock();
 
 		$view = \TestHelperJqadm::getView();
-		$view->item = \Aimeos\MShop::create( $this->context, 'service' )->createItem();
+		$view->item = \Aimeos\MShop::create($this->context, 'service')->createItem();
 
-		$object->setAimeos( \TestHelperJqadm::getAimeos() );
-		$object->setView( $view );
+		$object->setAimeos(\TestHelperJqadm::getAimeos());
+		$object->setView($view);
 
 		return $object;
 	}

admin/jqadm/tests/Admin/JQAdm/Service/Image/Property/StandardTest.php

Spacing +42 added lines, -42 removed lines

@@ -21,40 +21,40 @@ 
 		$this->view = \TestHelperJqadm::getView();
 		$this->context = \TestHelperJqadm::getContext();
 
-		$this->object = new \Aimeos\Admin\JQAdm\Service\Media\Property\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\Service\Media\Property\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);
 	}
 
 
 	public function testCreate()
 	{
-		$manager = \Aimeos\MShop::create( $this->context, 'service' );
+		$manager = \Aimeos\MShop::create($this->context, 'service');
 
 		$this->view->item = $manager->createItem();
 		$result = $this->object->create();
 
-		$this->assertContains( 'Media properties', $result );
-		$this->assertNull( $this->view->get( 'errors' ) );
+		$this->assertContains('Media properties', $result);
+		$this->assertNull($this->view->get('errors'));
 	}
 
 
 	public function testCopy()
 	{
-		$manager = \Aimeos\MShop::create( $this->context, 'service' );
+		$manager = \Aimeos\MShop::create($this->context, 'service');
 
-		$this->view->item = $manager->findItem( 'unitcode', ['media'] );
+		$this->view->item = $manager->findItem('unitcode', ['media']);
 		$result = $this->object->copy();
 
-		$this->assertNull( $this->view->get( 'errors' ) );
-		$this->assertContains( 'item-media-property', $result );
+		$this->assertNull($this->view->get('errors'));
+		$this->assertContains('item-media-property', $result);
 	}
 
 
@@ -62,32 +62,32 @@ 
 	{
 		$result = $this->object->delete();
 
-		$this->assertNull( $this->view->get( 'errors' ) );
-		$this->assertNull( $result );
+		$this->assertNull($this->view->get('errors'));
+		$this->assertNull($result);
 	}
 
 
 	public function testGet()
 	{
-		$manager = \Aimeos\MShop::create( $this->context, 'service' );
+		$manager = \Aimeos\MShop::create($this->context, 'service');
 
-		$this->view->item = $manager->findItem( 'unitcode', ['media'] );
+		$this->view->item = $manager->findItem('unitcode', ['media']);
 		$result = $this->object->get();
 
-		$this->assertNull( $this->view->get( 'errors' ) );
-		$this->assertContains( 'item-media-property', $result );
+		$this->assertNull($this->view->get('errors'));
+		$this->assertContains('item-media-property', $result);
 	}
 
 
 	public function testSave()
 	{
-		$manager = \Aimeos\MShop::create( $this->context, 'service' );
+		$manager = \Aimeos\MShop::create($this->context, 'service');
 
-		$item = $manager->findItem( 'unitcode', ['media'] );
-		$item->setCode( 'jqadm-test-media-property' );
-		$item->setId( null );
+		$item = $manager->findItem('unitcode', ['media']);
+		$item->setCode('jqadm-test-media-property');
+		$item->setId(null);
 
-		$this->view->item = $manager->saveItem( $item );
+		$this->view->item = $manager->saveItem($item);
 
 		$param = array(
 			'site' => 'unittest',
@@ -103,53 +103,53 @@ 
 			),
 		);
 
-		$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->save();
 
 
-		$manager->deleteItem( $this->view->item->getId() );
+		$manager->deleteItem($this->view->item->getId());
 
-		$this->assertNull( $this->view->get( 'errors' ) );
-		$this->assertNull( $result );
+		$this->assertNull($this->view->get('errors'));
+		$this->assertNull($result);
 
-		$mediaItems = $this->view->item->getRefItems( 'media' );
-		$this->assertEquals( 1, count( $mediaItems ) );
-		$this->assertEquals( 1, count( reset( $mediaItems )->getPropertyItems() ) );
+		$mediaItems = $this->view->item->getRefItems('media');
+		$this->assertEquals(1, count($mediaItems));
+		$this->assertEquals(1, count(reset($mediaItems)->getPropertyItems()));
 	}
 
 
 	public function testSaveException()
 	{
-		$object = $this->getMockBuilder( \Aimeos\Admin\JQAdm\Service\Media\Property\Standard::class )
-			->setConstructorArgs( array( $this->context, \TestHelperJqadm::getTemplatePaths() ) )
-			->setMethods( array( 'fromArray' ) )
+		$object = $this->getMockBuilder(\Aimeos\Admin\JQAdm\Service\Media\Property\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()));
 
 		$this->view = \TestHelperJqadm::getView();
-		$this->view->item = \Aimeos\MShop::create( $this->context, 'service' )->createItem();
+		$this->view->item = \Aimeos\MShop::create($this->context, 'service')->createItem();
 
-		$object->setView( $this->view );
+		$object->setView($this->view);
 
-		$this->setExpectedException( \RuntimeException::class );
+		$this->setExpectedException(\RuntimeException::class);
 		$object->save();
 	}
 
 
 	public function testSearch()
 	{
-		$this->assertNull( $this->object->search() );
+		$this->assertNull($this->object->search());
 	}
 
 
 	public function testGetSubClient()
 	{
-		$this->setExpectedException( \Aimeos\Admin\JQAdm\Exception::class );
-		$this->object->getSubClient( 'unknown' );
+		$this->setExpectedException(\Aimeos\Admin\JQAdm\Exception::class);
+		$this->object->getSubClient('unknown');
 	}
 }

admin/jqadm/tests/Admin/JQAdm/Customer/FactoryTest.php

Spacing +11 added lines, -11 removed lines

@@ -17,42 +17,42 @@
 	protected function setUp()
 	{
 		$this->context = \TestHelperJqadm::getContext();
-		$this->context->setView( \TestHelperJqadm::getView() );
+		$this->context->setView(\TestHelperJqadm::getView());
 	}
 
 
 	public function testCreateClient()
 	{
-		$client = \Aimeos\Admin\JQAdm\Customer\Factory::create( $this->context );
-		$this->assertInstanceOf( '\\Aimeos\\Admin\\JQAdm\\Iface', $client );
+		$client = \Aimeos\Admin\JQAdm\Customer\Factory::create($this->context);
+		$this->assertInstanceOf('\\Aimeos\\Admin\\JQAdm\\Iface', $client);
 	}
 
 
 	public function testCreateClientName()
 	{
-		$client = \Aimeos\Admin\JQAdm\Customer\Factory::create( $this->context, 'Standard' );
-		$this->assertInstanceOf( '\\Aimeos\\Admin\\JQAdm\\Iface', $client );
+		$client = \Aimeos\Admin\JQAdm\Customer\Factory::create($this->context, 'Standard');
+		$this->assertInstanceOf('\\Aimeos\\Admin\\JQAdm\\Iface', $client);
 	}
 
 
 	public function testCreateClientNameEmpty()
 	{
-		$this->setExpectedException( '\\Aimeos\\Admin\\JQAdm\\Exception' );
-		\Aimeos\Admin\JQAdm\Customer\Factory::create( $this->context, '' );
+		$this->setExpectedException('\\Aimeos\\Admin\\JQAdm\\Exception');
+		\Aimeos\Admin\JQAdm\Customer\Factory::create($this->context, '');
 	}
 
 
 	public function testCreateClientNameInvalid()
 	{
-		$this->setExpectedException( '\\Aimeos\\Admin\\JQAdm\\Exception' );
-		\Aimeos\Admin\JQAdm\Customer\Factory::create( $this->context, '%customer' );
+		$this->setExpectedException('\\Aimeos\\Admin\\JQAdm\\Exception');
+		\Aimeos\Admin\JQAdm\Customer\Factory::create($this->context, '%customer');
 	}
 
 
 	public function testCreateClientNameNotFound()
 	{
-		$this->setExpectedException( '\\Aimeos\\Admin\\JQAdm\\Exception' );
-		\Aimeos\Admin\JQAdm\Customer\Factory::create( $this->context, 'test' );
+		$this->setExpectedException('\\Aimeos\\Admin\\JQAdm\\Exception');
+		\Aimeos\Admin\JQAdm\Customer\Factory::create($this->context, 'test');
 	}
 
 }

admin/jqadm/tests/Admin/JQAdm/Supplier/FactoryTest.php

Spacing +11 added lines, -11 removed lines

@@ -17,42 +17,42 @@
 	protected function setUp()
 	{
 		$this->context = \TestHelperJqadm::getContext();
-		$this->context->setView( \TestHelperJqadm::getView() );
+		$this->context->setView(\TestHelperJqadm::getView());
 	}
 
 
 	public function testCreateClient()
 	{
-		$client = \Aimeos\Admin\JQAdm\Supplier\Factory::create( $this->context );
-		$this->assertInstanceOf( '\\Aimeos\\Admin\\JQAdm\\Iface', $client );
+		$client = \Aimeos\Admin\JQAdm\Supplier\Factory::create($this->context);
+		$this->assertInstanceOf('\\Aimeos\\Admin\\JQAdm\\Iface', $client);
 	}
 
 
 	public function testCreateClientName()
 	{
-		$client = \Aimeos\Admin\JQAdm\Supplier\Factory::create( $this->context, 'Standard' );
-		$this->assertInstanceOf( '\\Aimeos\\Admin\\JQAdm\\Iface', $client );
+		$client = \Aimeos\Admin\JQAdm\Supplier\Factory::create($this->context, 'Standard');
+		$this->assertInstanceOf('\\Aimeos\\Admin\\JQAdm\\Iface', $client);
 	}
 
 
 	public function testCreateClientNameEmpty()
 	{
-		$this->setExpectedException( '\\Aimeos\\Admin\\JQAdm\\Exception' );
-		\Aimeos\Admin\JQAdm\Supplier\Factory::create( $this->context, '' );
+		$this->setExpectedException('\\Aimeos\\Admin\\JQAdm\\Exception');
+		\Aimeos\Admin\JQAdm\Supplier\Factory::create($this->context, '');
 	}
 
 
 	public function testCreateClientNameInvalid()
 	{
-		$this->setExpectedException( '\\Aimeos\\Admin\\JQAdm\\Exception' );
-		\Aimeos\Admin\JQAdm\Supplier\Factory::create( $this->context, '%supplier' );
+		$this->setExpectedException('\\Aimeos\\Admin\\JQAdm\\Exception');
+		\Aimeos\Admin\JQAdm\Supplier\Factory::create($this->context, '%supplier');
 	}
 
 
 	public function testCreateClientNameNotFound()
 	{
-		$this->setExpectedException( '\\Aimeos\\Admin\\JQAdm\\Exception' );
-		\Aimeos\Admin\JQAdm\Supplier\Factory::create( $this->context, 'test' );
+		$this->setExpectedException('\\Aimeos\\Admin\\JQAdm\\Exception');
+		\Aimeos\Admin\JQAdm\Supplier\Factory::create($this->context, 'test');
 	}
 
 }

admin/jqadm/tests/Admin/JQAdm/Supplier/Image/StandardTest.php

Spacing +64 added lines, -64 removed lines

@@ -21,70 +21,70 @@ 
 		$this->view = \TestHelperJqadm::getView();
 		$this->context = \TestHelperJqadm::getContext();
 
-		$this->object = new \Aimeos\Admin\JQAdm\Supplier\Media\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\Supplier\Media\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);
 	}
 
 
 	public function testCreate()
 	{
-		$manager = \Aimeos\MShop::create( $this->context, 'supplier' );
+		$manager = \Aimeos\MShop::create($this->context, 'supplier');
 
 		$this->view->item = $manager->createItem();
 		$result = $this->object->create();
 
-		$this->assertContains( 'item-media', $result );
-		$this->assertNull( $this->view->get( 'errors' ) );
+		$this->assertContains('item-media', $result);
+		$this->assertNull($this->view->get('errors'));
 	}
 
 
 	public function testCopy()
 	{
-		$manager = \Aimeos\MShop::create( $this->context, 'supplier' );
+		$manager = \Aimeos\MShop::create($this->context, 'supplier');
 
-		$this->view->item = $manager->findItem( 'unitCode001', ['media'] );
+		$this->view->item = $manager->findItem('unitCode001', ['media']);
 		$result = $this->object->copy();
 
-		$this->assertNull( $this->view->get( 'errors' ) );
-		$this->assertContains( '"media.preview":"path\/to\/supplier.jpg"', $result );
+		$this->assertNull($this->view->get('errors'));
+		$this->assertContains('"media.preview":"path\/to\/supplier.jpg"', $result);
 	}
 
 
 	public function testDelete()
 	{
-		$manager = \Aimeos\MShop::create( $this->context, 'supplier' );
+		$manager = \Aimeos\MShop::create($this->context, 'supplier');
 
 		$this->view->item = $manager->createItem();
 		$result = $this->object->delete();
 
-		$this->assertNull( $this->view->get( 'errors' ) );
-		$this->assertNull( $result );
+		$this->assertNull($this->view->get('errors'));
+		$this->assertNull($result);
 	}
 
 
 	public function testGet()
 	{
-		$manager = \Aimeos\MShop::create( $this->context, 'supplier' );
+		$manager = \Aimeos\MShop::create($this->context, 'supplier');
 
-		$this->view->item = $manager->findItem( 'unitCode001', ['media'] );
+		$this->view->item = $manager->findItem('unitCode001', ['media']);
 		$result = $this->object->get();
 
-		$this->assertNull( $this->view->get( 'errors' ) );
-		$this->assertContains( '"media.preview":"path\/to\/supplier.jpg"', $result );
+		$this->assertNull($this->view->get('errors'));
+		$this->assertContains('"media.preview":"path\/to\/supplier.jpg"', $result);
 	}
 
 
 	public function testSave()
 	{
-		$manager = \Aimeos\MShop::create( $this->context, 'supplier' );
+		$manager = \Aimeos\MShop::create($this->context, 'supplier');
 		$this->view->item = $manager->createItem();
 
 		$param = array(
@@ -98,108 +98,108 @@ 
 			]],
 		);
 
-		$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);
 
-		$file = $this->getMockBuilder( \Psr\Http\Message\UploadedFileInterface::class )->getMock();
-		$request = $this->getMockBuilder( \Psr\Http\Message\ServerRequestInterface::class )->getMock();
-		$request->expects( $this->any() )->method( 'getUploadedFiles' )
-			->will( $this->returnValue( ['media' => [0 => ['file' => $file] ] ] ) );
+		$file = $this->getMockBuilder(\Psr\Http\Message\UploadedFileInterface::class)->getMock();
+		$request = $this->getMockBuilder(\Psr\Http\Message\ServerRequestInterface::class)->getMock();
+		$request->expects($this->any())->method('getUploadedFiles')
+			->will($this->returnValue(['media' => [0 => ['file' => $file]]]));
 
-		$helper = new \Aimeos\MW\View\Helper\Request\Standard( $this->view, $request, '127.0.0.1', 'test' );
-		$this->view ->addHelper( 'request', $helper );
+		$helper = new \Aimeos\MW\View\Helper\Request\Standard($this->view, $request, '127.0.0.1', 'test');
+		$this->view ->addHelper('request', $helper);
 
 
 		$name = 'AdminJQAdmSupplierMediaSave';
-		$this->context->getConfig()->set( 'controller/common/media/name', $name );
+		$this->context->getConfig()->set('controller/common/media/name', $name);
 
-		$cntlStub = $this->getMockBuilder( '\\Aimeos\\Controller\\Common\\Media\\Standard' )
-			->setConstructorArgs( array( $this->context ) )
-			->setMethods( array( 'add' ) )
+		$cntlStub = $this->getMockBuilder('\\Aimeos\\Controller\\Common\\Media\\Standard')
+			->setConstructorArgs(array($this->context))
+			->setMethods(array('add'))
 			->getMock();
 
-		\Aimeos\Controller\Common\Media\Factory::injectController( '\\Aimeos\\Controller\\Common\\Media\\' . $name, $cntlStub );
+		\Aimeos\Controller\Common\Media\Factory::injectController('\\Aimeos\\Controller\\Common\\Media\\' . $name, $cntlStub);
 
-		$cntlStub->expects( $this->once() )->method( 'add' );
+		$cntlStub->expects($this->once())->method('add');
 
 
 		$result = $this->object->save();
 
 
-		$this->assertNull( $this->view->get( 'errors' ) );
-		$this->assertNull( $result );
-		$this->assertEquals( 1, count( $this->view->item->getListItems() ) );
+		$this->assertNull($this->view->get('errors'));
+		$this->assertNull($result);
+		$this->assertEquals(1, count($this->view->item->getListItems()));
 
-		foreach( $this->view->item->getListItems( 'media' ) as $listItem )
+		foreach ($this->view->item->getListItems('media') as $listItem)
 		{
-			$this->assertEquals( 'media', $listItem->getDomain() );
+			$this->assertEquals('media', $listItem->getDomain());
 
 			$refItem = $listItem->getRefItem();
-			$this->assertEquals( 'de', $refItem->getLanguageId() );
-			$this->assertEquals( 'test', $refItem->getLabel() );
+			$this->assertEquals('de', $refItem->getLanguageId());
+			$this->assertEquals('test', $refItem->getLabel());
 		}
 
 
-		$helper = new \Aimeos\MW\View\Helper\Param\Standard( $this->view, ['site' => 'unittest', 'media' => []] );
-		$this->view->addHelper( 'param', $helper );
+		$helper = new \Aimeos\MW\View\Helper\Param\Standard($this->view, ['site' => 'unittest', 'media' => []]);
+		$this->view->addHelper('param', $helper);
 
 		$result = $this->object->save();
 
-		$this->assertNull( $this->view->get( 'errors' ) );
-		$this->assertNull( $result );
-		$this->assertEquals( 0, count( $this->view->item->getListItems() ) );
+		$this->assertNull($this->view->get('errors'));
+		$this->assertNull($result);
+		$this->assertEquals(0, count($this->view->item->getListItems()));
 	}
 
 
 	public function testSaveException()
 	{
-		$object = $this->getClientMock( 'fromArray' );
+		$object = $this->getClientMock('fromArray');
 
-		$object->expects( $this->once() )->method( 'fromArray' )
-			->will( $this->throwException( new \RuntimeException() ) );
+		$object->expects($this->once())->method('fromArray')
+			->will($this->throwException(new \RuntimeException()));
 
-		$this->setExpectedException( \Aimeos\Admin\JQAdm\Exception::class );
+		$this->setExpectedException(\Aimeos\Admin\JQAdm\Exception::class);
 		$object->save();
 	}
 
 
 	public function testSaveMShopException()
 	{
-		$object = $this->getClientMock( 'fromArray' );
+		$object = $this->getClientMock('fromArray');
 
-		$object->expects( $this->once() )->method( 'fromArray' )
-			->will( $this->throwException( new \Aimeos\MShop\Exception() ) );
+		$object->expects($this->once())->method('fromArray')
+			->will($this->throwException(new \Aimeos\MShop\Exception()));
 
-		$this->setExpectedException( \Aimeos\Admin\JQAdm\Exception::class );
+		$this->setExpectedException(\Aimeos\Admin\JQAdm\Exception::class);
 		$object->save();
 	}
 
 
 	public function testSearch()
 	{
-		$this->assertNull( $this->object->search() );
+		$this->assertNull($this->object->search());
 	}
 
 
 	public function testGetSubClient()
 	{
-		$this->setExpectedException( \Aimeos\Admin\JQAdm\Exception::class );
-		$this->object->getSubClient( 'unknown' );
+		$this->setExpectedException(\Aimeos\Admin\JQAdm\Exception::class);
+		$this->object->getSubClient('unknown');
 	}
 
 
-	public function getClientMock( $method )
+	public function getClientMock($method)
 	{
-		$object = $this->getMockBuilder( \Aimeos\Admin\JQAdm\Supplier\Media\Standard::class )
-			->setConstructorArgs( array( $this->context, \TestHelperJqadm::getTemplatePaths() ) )
-			->setMethods( [$method] )
+		$object = $this->getMockBuilder(\Aimeos\Admin\JQAdm\Supplier\Media\Standard::class)
+			->setConstructorArgs(array($this->context, \TestHelperJqadm::getTemplatePaths()))
+			->setMethods([$method])
 			->getMock();
 
 		$view = \TestHelperJqadm::getView();
-		$view->item = \Aimeos\MShop::create( $this->context, 'supplier' )->createItem();
+		$view->item = \Aimeos\MShop::create($this->context, 'supplier')->createItem();
 
-		$object->setAimeos( \TestHelperJqadm::getAimeos() );
-		$object->setView( $view );
+		$object->setAimeos(\TestHelperJqadm::getAimeos());
+		$object->setView($view);
 
 		return $object;
 	}