aimeos / ai-admin-jqadm

admin/jqadm/src/Admin/JQAdm/Product/Characteristic/Attribute/Standard.php

Spacing +35 added lines, -35 removed lines

@@ -42,14 +42,14 @@ 
 	{
 		$view = $this->getView();
 
-		$view->attributeData = $this->toArray( $view->item, true );
+		$view->attributeData = $this->toArray($view->item, true);
 		$view->attributeBody = '';
 
-		foreach( $this->getSubClients() as $client ) {
+		foreach ($this->getSubClients() as $client) {
 			$view->attributeBody .= $client->copy();
 		}
 
-		return $this->render( $view );
+		return $this->render($view);
 	}
 
 
@@ -62,20 +62,20 @@ 
 	{
 		$view = $this->getView();
 		$siteid = $this->getContext()->getLocale()->getSiteId();
-		$data = $view->param( 'characteristic/attribute', [] );
+		$data = $view->param('characteristic/attribute', []);
 
-		foreach( $view->value( $data, 'product.lists.id', [] ) as $idx => $value ) {
+		foreach ($view->value($data, 'product.lists.id', []) as $idx => $value) {
 			$data['product.lists.siteid'][$idx] = $siteid;
 		}
 
 		$view->attributeData = $data;
 		$view->attributeBody = '';
 
-		foreach( $this->getSubClients() as $client ) {
+		foreach ($this->getSubClients() as $client) {
 			$view->attributeBody .= $client->create();
 		}
 
-		return $this->render( $view );
+		return $this->render($view);
 	}
 
 
@@ -88,14 +88,14 @@ 
 	{
 		$view = $this->getView();
 
-		$view->attributeData = $this->toArray( $view->item );
+		$view->attributeData = $this->toArray($view->item);
 		$view->attributeBody = '';
 
-		foreach( $this->getSubClients() as $client ) {
+		foreach ($this->getSubClients() as $client) {
 			$view->attributeBody .= $client->get();
 		}
 
-		return $this->render( $view );
+		return $this->render($view);
 	}
 
 
@@ -106,10 +106,10 @@ 
 	{
 		$view = $this->getView();
 
-		$this->fromArray( $view->item, $view->param( 'characteristic/attribute', [] ) );
+		$this->fromArray($view->item, $view->param('characteristic/attribute', []));
 		$view->attributeBody = '';
 
-		foreach( $this->getSubClients() as $client ) {
+		foreach ($this->getSubClients() as $client) {
 			$view->attributeBody .= $client->save();
 		}
 	}
@@ -122,7 +122,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/product/characteristic/attribute/decorators/excludes
 		 * Excludes decorators added by the "common" option from the product JQAdm client
@@ -197,7 +197,7 @@ 
 		 * @see admin/jqadm/product/characteristic/attribute/decorators/excludes
 		 * @see admin/jqadm/product/characteristic/attribute/decorators/global
 		 */
-		return $this->createSubClient( 'product/characteristic/attribute/' . $type, $name );
+		return $this->createSubClient('product/characteristic/attribute/' . $type, $name);
 	}
 
 
@@ -241,7 +241,7 @@ 
 		 * @since 2016.01
 		 * @category Developer
 		 */
-		return $this->getContext()->getConfig()->get( 'admin/jqadm/product/characteristic/attribute/standard/subparts', [] );
+		return $this->getContext()->getConfig()->get('admin/jqadm/product/characteristic/attribute/standard/subparts', []);
 	}
 
 
@@ -251,29 +251,29 @@ 
 	 * @param \Aimeos\MShop\Product\Item\Iface $item Product item object without referenced domain items
 	 * @param string[] $data Data array
 	 */
-	protected function fromArray( \Aimeos\MShop\Product\Item\Iface $item, array $data )
+	protected function fromArray(\Aimeos\MShop\Product\Item\Iface $item, array $data)
 	{
-		$listManager = \Aimeos\MShop::create( $this->getContext(), 'product/lists' );
+		$listManager = \Aimeos\MShop::create($this->getContext(), 'product/lists');
 
-		$listItem = $listManager->createItem()->setType( 'default' );
-		$listItems = $item->getListItems( 'attribute', 'default', null, false );
+		$listItem = $listManager->createItem()->setType('default');
+		$listItems = $item->getListItems('attribute', 'default', null, false);
 
-		foreach( $this->getValue( $data, 'product.lists.id', [] ) as $idx => $id )
+		foreach ($this->getValue($data, 'product.lists.id', []) as $idx => $id)
 		{
-			if( isset( $listItems[$id] ) ) {
-				$litem = $listItems[$id]; unset( $listItems[$id] );
+			if (isset($listItems[$id])) {
+				$litem = $listItems[$id]; unset($listItems[$id]);
 			} else {
 				$litem = clone $listItem;
 			}
 
-			$litem->setId( $id );
-			$litem->setPosition( $idx );
-			$litem->setRefId( $this->getValue( $data, 'product.lists.refid/' . $idx ) );
+			$litem->setId($id);
+			$litem->setPosition($idx);
+			$litem->setRefId($this->getValue($data, 'product.lists.refid/' . $idx));
 
-			$item->addListItem( 'attribute', $litem, $litem->getRefItem() );
+			$item->addListItem('attribute', $litem, $litem->getRefItem());
 		}
 
-		return $item->deleteListItems( $listItems );
+		return $item->deleteListItems($listItems);
 	}
 
 
@@ -284,26 +284,26 @@ 
 	 * @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\Product\Item\Iface $item, $copy = false )
+	protected function toArray(\Aimeos\MShop\Product\Item\Iface $item, $copy = false)
 	{
 		$data = [];
 		$siteId = $this->getContext()->getLocale()->getSiteId();
 
-		foreach( $item->getListItems( 'attribute', 'default', null, false ) as $listItem )
+		foreach ($item->getListItems('attribute', 'default', null, false) as $listItem)
 		{
-			if( ( $refItem = $listItem->getRefItem() ) === null ) {
+			if (($refItem = $listItem->getRefItem()) === null) {
 				continue;
 			}
 
-			$list = $listItem->toArray( true ) + $refItem->toArray( true );
+			$list = $listItem->toArray(true) + $refItem->toArray(true);
 
-			if( $copy === true )
+			if ($copy === true)
 			{
 				$list['product.lists.siteid'] = $siteId;
 				$list['product.lists.id'] = '';
 			}
 
-			foreach( $list as $key => $value ) {
+			foreach ($list as $key => $value) {
 				$data[$key][] = $value;
 			}
 		}
@@ -318,7 +318,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/product/characteristic/attribute/template-item
 		 * Relative path to the HTML body template of the attribute characteristic subpart for products.
@@ -342,6 +342,6 @@ 
 		$tplconf = 'admin/jqadm/product/characteristic/attribute/template-item';
 		$default = 'product/item-characteristic-attribute-standard';
 
-		return $view->render( $view->config( $tplconf, $default ) );
+		return $view->render($view->config($tplconf, $default));
 	}
 }
\ No newline at end of file

admin/jqadm/src/Admin/JQAdm/Product/Characteristic/Variant/Standard.php

Spacing +35 added lines, -35 removed lines

@@ -42,14 +42,14 @@ 
 	{
 		$view = $this->getView();
 
-		$view->variantData = $this->toArray( $view->item, true );
+		$view->variantData = $this->toArray($view->item, true);
 		$view->variantBody = '';
 
-		foreach( $this->getSubClients() as $client ) {
+		foreach ($this->getSubClients() as $client) {
 			$view->variantBody .= $client->copy();
 		}
 
-		return $this->render( $view );
+		return $this->render($view);
 	}
 
 
@@ -62,20 +62,20 @@ 
 	{
 		$view = $this->getView();
 		$siteid = $this->getContext()->getLocale()->getSiteId();
-		$data = $view->param( 'characteristic/variant', [] );
+		$data = $view->param('characteristic/variant', []);
 
-		foreach( $view->value( $data, 'product.lists.id', [] ) as $idx => $value ) {
+		foreach ($view->value($data, 'product.lists.id', []) as $idx => $value) {
 			$data['product.lists.siteid'][$idx] = $siteid;
 		}
 
 		$view->variantData = $data;
 		$view->variantBody = '';
 
-		foreach( $this->getSubClients() as $client ) {
+		foreach ($this->getSubClients() as $client) {
 			$view->variantBody .= $client->create();
 		}
 
-		return $this->render( $view );
+		return $this->render($view);
 	}
 
 
@@ -88,14 +88,14 @@ 
 	{
 		$view = $this->getView();
 
-		$view->variantData = $this->toArray( $view->item );
+		$view->variantData = $this->toArray($view->item);
 		$view->variantBody = '';
 
-		foreach( $this->getSubClients() as $client ) {
+		foreach ($this->getSubClients() as $client) {
 			$view->variantBody .= $client->get();
 		}
 
-		return $this->render( $view );
+		return $this->render($view);
 	}
 
 
@@ -106,10 +106,10 @@ 
 	{
 		$view = $this->getView();
 
-		$this->fromArray( $view->item, $view->param( 'characteristic/variant', [] ) );
+		$this->fromArray($view->item, $view->param('characteristic/variant', []));
 		$view->variantBody = '';
 
-		foreach( $this->getSubClients() as $client ) {
+		foreach ($this->getSubClients() as $client) {
 			$view->variantBody .= $client->save();
 		}
 	}
@@ -122,7 +122,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/product/characteristic/variant/decorators/excludes
 		 * Excludes decorators added by the "common" option from the product JQAdm client
@@ -197,7 +197,7 @@ 
 		 * @see admin/jqadm/product/characteristic/variant/decorators/excludes
 		 * @see admin/jqadm/product/characteristic/variant/decorators/global
 		 */
-		return $this->createSubClient( 'product/characteristic/variant/' . $type, $name );
+		return $this->createSubClient('product/characteristic/variant/' . $type, $name);
 	}
 
 
@@ -241,7 +241,7 @@ 
 		 * @since 2016.01
 		 * @category Developer
 		 */
-		return $this->getContext()->getConfig()->get( 'admin/jqadm/product/characteristic/variant/standard/subparts', [] );
+		return $this->getContext()->getConfig()->get('admin/jqadm/product/characteristic/variant/standard/subparts', []);
 	}
 
 
@@ -251,29 +251,29 @@ 
 	 * @param \Aimeos\MShop\Product\Item\Iface $item Product item object without referenced domain items
 	 * @param string[] $data Data array
 	 */
-	protected function fromArray( \Aimeos\MShop\Product\Item\Iface $item, array $data )
+	protected function fromArray(\Aimeos\MShop\Product\Item\Iface $item, array $data)
 	{
-		$listManager = \Aimeos\MShop::create( $this->getContext(), 'product/lists' );
+		$listManager = \Aimeos\MShop::create($this->getContext(), 'product/lists');
 
-		$listItem = $listManager->createItem()->setType( 'variant' );
-		$listItems = $item->getListItems( 'attribute', 'variant', null, false );
+		$listItem = $listManager->createItem()->setType('variant');
+		$listItems = $item->getListItems('attribute', 'variant', null, false);
 
-		foreach( $this->getValue( $data, 'product.lists.id', [] ) as $idx => $id )
+		foreach ($this->getValue($data, 'product.lists.id', []) as $idx => $id)
 		{
-			if( isset( $listItems[$id] ) ) {
-				$litem = $listItems[$id]; unset( $listItems[$id] );
+			if (isset($listItems[$id])) {
+				$litem = $listItems[$id]; unset($listItems[$id]);
 			} else {
 				$litem = clone $listItem;
 			}
 
-			$litem->setId( $id );
-			$litem->setPosition( $idx );
-			$litem->setRefId( $this->getValue( $data, 'product.lists.refid/' . $idx ) );
+			$litem->setId($id);
+			$litem->setPosition($idx);
+			$litem->setRefId($this->getValue($data, 'product.lists.refid/' . $idx));
 
-			$item->addListItem( 'attribute', $litem, $litem->getRefItem() );
+			$item->addListItem('attribute', $litem, $litem->getRefItem());
 		}
 
-		return $item->deleteListItems( $listItems );
+		return $item->deleteListItems($listItems);
 	}
 
 
@@ -284,26 +284,26 @@ 
 	 * @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\Product\Item\Iface $item, $copy = false )
+	protected function toArray(\Aimeos\MShop\Product\Item\Iface $item, $copy = false)
 	{
 		$siteId = $this->getContext()->getLocale()->getSiteId();
 		$data = [];
 
-		foreach( $item->getListItems( 'attribute', 'variant', null, false ) as $listItem )
+		foreach ($item->getListItems('attribute', 'variant', null, false) as $listItem)
 		{
-			if( ( $refItem = $listItem->getRefItem() ) === null ) {
+			if (($refItem = $listItem->getRefItem()) === null) {
 				continue;
 			}
 
-			$list = $listItem->toArray( true ) + $refItem->toArray( true );
+			$list = $listItem->toArray(true) + $refItem->toArray(true);
 
-			if( $copy === true )
+			if ($copy === true)
 			{
 				$list['product.lists.siteid'] = $siteId;
 				$list['product.lists.id'] = '';
 			}
 
-			foreach( $list as $key => $value ) {
+			foreach ($list as $key => $value) {
 				$data[$key][] = $value;
 			}
 		}
@@ -318,7 +318,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/product/characteristic/variant/template-item
 		 * Relative path to the HTML body template of the variant characteristic subpart for products.
@@ -342,6 +342,6 @@ 
 		$tplconf = 'admin/jqadm/product/characteristic/variant/template-item';
 		$default = 'product/item-characteristic-variant-standard';
 
-		return $view->render( $view->config( $tplconf, $default ) );
+		return $view->render($view->config($tplconf, $default));
 	}
 }
\ No newline at end of file

admin/jqadm/src/Admin/JQAdm/Common/Decorator/Index.php

Spacing +4 added lines, -4 removed lines

@@ -28,8 +28,8 @@ 
 	{
 		$result = $this->getClient()->delete();
 
-		$ids = (array) $this->getView()->param( 'id' );
-		\Aimeos\MShop::create( $this->getContext(), 'index' )->deleteItems( $ids );
+		$ids = (array) $this->getView()->param('id');
+		\Aimeos\MShop::create($this->getContext(), 'index')->deleteItems($ids);
 
 		return $result;
 	}
@@ -45,8 +45,8 @@ 
 		$result = $this->getClient()->save();
 		$item = $this->getView()->item;
 
-		if( $item->getId() !== null ) {
-			\Aimeos\MShop::create( $this->getContext(), 'index' )->saveItem( $item );
+		if ($item->getId() !== null) {
+			\Aimeos\MShop::create($this->getContext(), 'index')->saveItem($item);
 		}
 
 		return $result;

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

Spacing +7 added lines, -7 removed lines

@@ -43,10 +43,10 @@ 
 		$view = $this->getView();
 		$view->orderBody = '';
 
-		$manager = \Aimeos\MShop::create( $this->getContext(), 'locale/currency' );
-		$view->orderCurrencyItems = $manager->searchItems( $manager->createSearch( true ) );
+		$manager = \Aimeos\MShop::create($this->getContext(), 'locale/currency');
+		$view->orderCurrencyItems = $manager->searchItems($manager->createSearch(true));
 
-		foreach( $this->getSubClients() as $client ) {
+		foreach ($this->getSubClients() as $client) {
 			$view->orderBody .= $client->search();
 		}
 
@@ -72,7 +72,7 @@ 
 		$tplconf = 'admin/jqadm/dashboard/order/template-list';
 		$default = 'dashboard/list-order-standard';
 
-		return $view->render( $view->config( $tplconf, $default ) );
+		return $view->render($view->config($tplconf, $default));
 	}
 
 
@@ -83,7 +83,7 @@ 
 	 * @param string|null $name Name of the sub-client (Default if null)
 	 * @return \Aimeos\Admin\JQAdm\Iface Sub-client object
 	 */
-	public function getSubClient( $type, $name = null )
+	public function getSubClient($type, $name = null)
 	{
 		/** admin/jqadm/dashboard/order/decorators/excludes
 		 * Excludes decorators added by the "common" option from the dashboard JQAdm client
@@ -158,7 +158,7 @@ 
 		 * @see admin/jqadm/dashboard/order/decorators/excludes
 		 * @see admin/jqadm/dashboard/order/decorators/global
 		 */
-		return $this->createSubClient( 'dashboard/order/' . $type, $name );
+		return $this->createSubClient('dashboard/order/' . $type, $name);
 	}
 
 
@@ -202,6 +202,6 @@ 
 		 * @since 2018.01
 		 * @category Developer
 		 */
-		return $this->getContext()->getConfig()->get( 'admin/jqadm/dashboard/order/standard/subparts', [] );
+		return $this->getContext()->getConfig()->get('admin/jqadm/dashboard/order/standard/subparts', []);
 	}
 }

admin/jqadm/tests/Admin/JQAdm/Common/Factory/BaseTest.php

Doc Comments +7 added lines

@@ -126,12 +126,19 @@
 		return self::addDecorators( $context, $client, $decorators, $classprefix );
 	}
 
+	/**
+	 * @param string $path
+	 */
 	public static function addClientDecoratorsPublic( \Aimeos\MShop\Context\Item\Iface $context,
 		\Aimeos\Admin\JQAdm\Iface $client, $path )
 	{
 		return self::addClientDecorators( $context, $client, $path );
 	}
 
+	/**
+	 * @param string $classname
+	 * @param string $interface
+	 */
 	public static function createAdminPublic( \Aimeos\MShop\Context\Item\Iface $context,
 		$classname, $interface )
 	{

Spacing +46 added lines, -46 removed lines

@@ -17,102 +17,102 @@ 
 	protected function setUp()
 	{
 		$this->context = \TestHelperJqadm::getContext();
-		$this->context->setView( \TestHelperJqadm::getView() );
+		$this->context->setView(\TestHelperJqadm::getView());
 
 		$config = $this->context->getConfig();
-		$config->set( 'admin/jqadm/common/decorators/default', [] );
-		$config->set( 'admin/jqadm/product/decorators/global', [] );
-		$config->set( 'admin/jqadm/product/decorators/local', [] );
+		$config->set('admin/jqadm/common/decorators/default', []);
+		$config->set('admin/jqadm/product/decorators/global', []);
+		$config->set('admin/jqadm/product/decorators/local', []);
 
 	}
 
 
 	public function testInjectClient()
 	{
-		$client = \Aimeos\Admin\JQAdm\Product\Factory::create( $this->context, 'Standard' );
-		\Aimeos\Admin\JQAdm\Product\Factory::injectClient( '\\Aimeos\\Admin\\JQAdm\\Product\\Standard', $client );
+		$client = \Aimeos\Admin\JQAdm\Product\Factory::create($this->context, 'Standard');
+		\Aimeos\Admin\JQAdm\Product\Factory::injectClient('\\Aimeos\\Admin\\JQAdm\\Product\\Standard', $client);
 
-		$iClient = \Aimeos\Admin\JQAdm\Product\Factory::create( $this->context, 'Standard' );
+		$iClient = \Aimeos\Admin\JQAdm\Product\Factory::create($this->context, 'Standard');
 
-		$this->assertSame( $client, $iClient );
+		$this->assertSame($client, $iClient);
 	}
 
 
 	public function testInjectClientReset()
 	{
-		$client = \Aimeos\Admin\JQAdm\Product\Factory::create( $this->context, 'Standard' );
-		\Aimeos\Admin\JQAdm\Product\Factory::injectClient( '\\Aimeos\\Admin\\JQAdm\\Product\\Standard', $client );
-		\Aimeos\Admin\JQAdm\Product\Factory::injectClient( '\\Aimeos\\Admin\\JQAdm\\Product\\Standard', null );
+		$client = \Aimeos\Admin\JQAdm\Product\Factory::create($this->context, 'Standard');
+		\Aimeos\Admin\JQAdm\Product\Factory::injectClient('\\Aimeos\\Admin\\JQAdm\\Product\\Standard', $client);
+		\Aimeos\Admin\JQAdm\Product\Factory::injectClient('\\Aimeos\\Admin\\JQAdm\\Product\\Standard', null);
 
-		$new = \Aimeos\Admin\JQAdm\Product\Factory::create( $this->context, 'Standard' );
+		$new = \Aimeos\Admin\JQAdm\Product\Factory::create($this->context, 'Standard');
 
-		$this->assertNotSame( $client, $new );
+		$this->assertNotSame($client, $new);
 	}
 
 
 	public function testAddDecorators()
 	{
-		$client = \Aimeos\Admin\JQAdm\Product\Factory::create( $this->context, 'Standard' );
+		$client = \Aimeos\Admin\JQAdm\Product\Factory::create($this->context, 'Standard');
 
-		$result = \Aimeos\Admin\JQAdm\Common\Factory\TestAbstract::addDecoratorsPublic( $this->context, $client,
-			[], array( 'Cache' ), '\\Aimeos\\Admin\\JQAdm\\Common\\Decorator\\' );
+		$result = \Aimeos\Admin\JQAdm\Common\Factory\TestAbstract::addDecoratorsPublic($this->context, $client,
+			[], array('Cache'), '\\Aimeos\\Admin\\JQAdm\\Common\\Decorator\\');
 
-		$this->assertInstanceOf( \Aimeos\Admin\JQAdm\Iface::class, $result );
+		$this->assertInstanceOf(\Aimeos\Admin\JQAdm\Iface::class, $result);
 	}
 
 
 	public function testAddDecoratorsInvalidName()
 	{
-		$client = \Aimeos\Admin\JQAdm\Product\Factory::create( $this->context, 'Standard' );
+		$client = \Aimeos\Admin\JQAdm\Product\Factory::create($this->context, 'Standard');
 
-		$this->setExpectedException( '\\Aimeos\\Admin\\JQAdm\\Exception' );
-		\Aimeos\Admin\JQAdm\Common\Factory\TestAbstract::addDecoratorsPublic( $this->context, $client, ['$'], 'Test' );
+		$this->setExpectedException('\\Aimeos\\Admin\\JQAdm\\Exception');
+		\Aimeos\Admin\JQAdm\Common\Factory\TestAbstract::addDecoratorsPublic($this->context, $client, ['$'], 'Test');
 	}
 
 
 	public function testAddDecoratorsInvalidClass()
 	{
-		$client = \Aimeos\Admin\JQAdm\Product\Factory::create( $this->context, 'Standard' );
+		$client = \Aimeos\Admin\JQAdm\Product\Factory::create($this->context, 'Standard');
 
-		$this->setExpectedException( '\\Aimeos\\Admin\\JQAdm\\Exception' );
-		\Aimeos\Admin\JQAdm\Common\Factory\TestAbstract::addDecoratorsPublic( $this->context, $client,
-			array( 'Test' ), 'TestDecorator' );
+		$this->setExpectedException('\\Aimeos\\Admin\\JQAdm\\Exception');
+		\Aimeos\Admin\JQAdm\Common\Factory\TestAbstract::addDecoratorsPublic($this->context, $client,
+			array('Test'), 'TestDecorator');
 	}
 
 
 	public function testAddDecoratorsInvalidInterface()
 	{
-		$client = \Aimeos\Admin\JQAdm\Product\Factory::create( $this->context, 'Standard' );
+		$client = \Aimeos\Admin\JQAdm\Product\Factory::create($this->context, 'Standard');
 
-		$this->setExpectedException( '\\Aimeos\\Admin\\JQAdm\\Exception' );
-		\Aimeos\Admin\JQAdm\Common\Factory\TestAbstract::addDecoratorsPublic( $this->context, $client,
-			array( 'Test' ), '\\Aimeos\\Admin\\JQAdm\\Common\\Decorator\\' );
+		$this->setExpectedException('\\Aimeos\\Admin\\JQAdm\\Exception');
+		\Aimeos\Admin\JQAdm\Common\Factory\TestAbstract::addDecoratorsPublic($this->context, $client,
+			array('Test'), '\\Aimeos\\Admin\\JQAdm\\Common\\Decorator\\');
 	}
 
 
 	public function testAddClientDecoratorsExcludes()
 	{
-		$this->context->getConfig()->set( 'admin/jqadm/decorators/excludes', array( 'TestDecorator' ) );
-		$this->context->getConfig()->set( 'admin/jqadm/common/decorators/default', array( 'TestDecorator' ) );
+		$this->context->getConfig()->set('admin/jqadm/decorators/excludes', array('TestDecorator'));
+		$this->context->getConfig()->set('admin/jqadm/common/decorators/default', array('TestDecorator'));
 
-		$this->setExpectedException( '\\Aimeos\\Admin\\JQAdm\\Exception' );
-		\Aimeos\Admin\JQAdm\Product\Factory::create( $this->context, 'Standard' );
+		$this->setExpectedException('\\Aimeos\\Admin\\JQAdm\\Exception');
+		\Aimeos\Admin\JQAdm\Product\Factory::create($this->context, 'Standard');
 	}
 
 
 	public function testAddClientDecoratorsEmptyPath()
 	{
-		$client = \Aimeos\Admin\JQAdm\Product\Factory::create( $this->context, 'Standard' );
+		$client = \Aimeos\Admin\JQAdm\Product\Factory::create($this->context, 'Standard');
 
-		$this->setExpectedException( '\\Aimeos\\Admin\\JQAdm\\Exception' );
-		\Aimeos\Admin\JQAdm\Common\Factory\TestAbstract::addClientDecoratorsPublic( $this->context, $client, '' );
+		$this->setExpectedException('\\Aimeos\\Admin\\JQAdm\\Exception');
+		\Aimeos\Admin\JQAdm\Common\Factory\TestAbstract::addClientDecoratorsPublic($this->context, $client, '');
 	}
 
 
 	public function testCreateClientBase()
 	{
-		$this->setExpectedException( '\\Aimeos\\Admin\\JQAdm\\Exception' );
-		\Aimeos\Admin\JQAdm\Common\Factory\TestAbstract::createAdminPublic( $this->context, 'Test', 'Test' );
+		$this->setExpectedException('\\Aimeos\\Admin\\JQAdm\\Exception');
+		\Aimeos\Admin\JQAdm\Common\Factory\TestAbstract::createAdminPublic($this->context, 'Test', 'Test');
 	}
 }
 
@@ -120,22 +120,22 @@ 
 class TestAbstract
 	extends \Aimeos\Admin\JQAdm\Common\Factory\Base
 {
-	public static function addDecoratorsPublic( \Aimeos\MShop\Context\Item\Iface $context,
-		\Aimeos\Admin\JQAdm\Iface $client, array $decorators, $classprefix )
+	public static function addDecoratorsPublic(\Aimeos\MShop\Context\Item\Iface $context,
+		\Aimeos\Admin\JQAdm\Iface $client, array $decorators, $classprefix)
 	{
-		return self::addDecorators( $context, $client, $decorators, $classprefix );
+		return self::addDecorators($context, $client, $decorators, $classprefix);
 	}
 
-	public static function addClientDecoratorsPublic( \Aimeos\MShop\Context\Item\Iface $context,
-		\Aimeos\Admin\JQAdm\Iface $client, $path )
+	public static function addClientDecoratorsPublic(\Aimeos\MShop\Context\Item\Iface $context,
+		\Aimeos\Admin\JQAdm\Iface $client, $path)
 	{
-		return self::addClientDecorators( $context, $client, $path );
+		return self::addClientDecorators($context, $client, $path);
 	}
 
-	public static function createAdminPublic( \Aimeos\MShop\Context\Item\Iface $context,
-		$classname, $interface )
+	public static function createAdminPublic(\Aimeos\MShop\Context\Item\Iface $context,
+		$classname, $interface)
 	{
-		return self::createAdmin( $context, $classname, $interface );
+		return self::createAdmin($context, $classname, $interface);
 	}
 }
 

admin/jqadm/templates/attribute/item-media-property-standard.php

Spacing +22 added lines, -22 removed lines

@@ -17,14 +17,14 @@ 
 		<thead>
 			<tr>
 				<th colspan="3">
-					<span class="help"><?= $enc->html( $this->translate( 'admin', 'Media properties' ) ); ?></span>
+					<span class="help"><?= $enc->html($this->translate('admin', 'Media properties')); ?></span>
 					<div class="form-text text-muted help-text">
-						<?= $enc->html( $this->translate( 'admin', 'Non-shared properties for the media item' ) ); ?>
+						<?= $enc->html($this->translate('admin', 'Non-shared properties for the media item')); ?>
 					</div>
 				</th>
 				<th class="actions">
-					<div class="btn act-add fa" tabindex="<?= $this->get( 'tabindex' ); ?>"
-						title="<?= $enc->attr( $this->translate( 'admin', 'Insert new entry (Ctrl+I)') ); ?>"
+					<div class="btn act-add fa" tabindex="<?= $this->get('tabindex'); ?>"
+						title="<?= $enc->attr($this->translate('admin', 'Insert new entry (Ctrl+I)')); ?>"
 						v-on:click="addPropertyItem(idx)">
 					</div>
 				</th>
@@ -36,48 +36,48 @@ 
 
 				<td class="property-type">
 					<input class="item-propertyid" type="hidden" v-bind:value="propdata['media.property.id']"
-						v-bind:name="'<?= $enc->attr( $this->formparam( array( 'media', 'idx', 'property', 'propidx', 'media.property.id' ) ) ); ?>'.replace( 'idx', idx ).replace( 'propidx', propidx )" />
-					<select class="form-control custom-select item-type" required="required" tabindex="<?= $this->get( 'tabindex' ); ?>"
-						v-bind:name="'<?= $enc->attr( $this->formparam( array( 'media', 'idx', 'property', 'propidx', 'media.property.type' ) ) ); ?>'.replace( 'idx', idx ).replace( 'propidx', propidx )"
+						v-bind:name="'<?= $enc->attr($this->formparam(array('media', 'idx', 'property', 'propidx', 'media.property.id'))); ?>'.replace( 'idx', idx ).replace( 'propidx', propidx )" />
+					<select class="form-control custom-select item-type" required="required" tabindex="<?= $this->get('tabindex'); ?>"
+						v-bind:name="'<?= $enc->attr($this->formparam(array('media', 'idx', 'property', 'propidx', 'media.property.type'))); ?>'.replace( 'idx', idx ).replace( 'propidx', propidx )"
 						v-bind:readonly="checkSite('media.siteid', idx)"
 						v-model="items[idx]['property'][propidx]['media.property.type']" >
 
-						<?php foreach( $this->get( 'propertyTypes', [] ) as $id => $item ) : ?>
-							<option value="<?= $enc->attr( $id ); ?>" v-bind:selected="items[idx]['property'][propidx]['media.property.type'] == '<?= $enc->attr( $id ) ?>'" >
-								<?= $enc->html( $item->getLabel() ); ?>
+						<?php foreach ($this->get('propertyTypes', []) as $id => $item) : ?>
+							<option value="<?= $enc->attr($id); ?>" v-bind:selected="items[idx]['property'][propidx]['media.property.type'] == '<?= $enc->attr($id) ?>'" >
+								<?= $enc->html($item->getLabel()); ?>
 							</option>
 						<?php endforeach; ?>
 
 					</select>
 				</td>
 				<td class="property-language">
-					<select class="form-control custom-select item-languageid" tabindex="<?= $this->get( 'tabindex' ); ?>"
-						v-bind:name="'<?= $enc->attr( $this->formparam( array( 'media', 'idx', 'property', 'propidx', 'media.property.languageid' ) ) ); ?>'.replace( 'idx', idx ).replace( 'propidx', propidx )"
+					<select class="form-control custom-select item-languageid" tabindex="<?= $this->get('tabindex'); ?>"
+						v-bind:name="'<?= $enc->attr($this->formparam(array('media', 'idx', 'property', 'propidx', 'media.property.languageid'))); ?>'.replace( 'idx', idx ).replace( 'propidx', propidx )"
 						v-bind:readonly="checkSite('media.siteid', idx)"
 						v-model="items[idx]['property'][propidx]['media.property.languageid']" >
 
 						<option v-bind:value="null">
-							<?= $enc->html( $this->translate( 'admin', 'All' ) ); ?>
+							<?= $enc->html($this->translate('admin', 'All')); ?>
 						</option>
 
-						<?php foreach( $this->get( 'pageLangItems', [] ) as $langId => $langItem ) : ?>
-							<option value="<?= $enc->attr( $langId ); ?>" v-bind:selected="items[idx]['property'][propidx]['media.property.languageid'] == '<?= $enc->attr( $langId ) ?>'" >
-								<?= $enc->html( $langItem->getLabel() ); ?>
+						<?php foreach ($this->get('pageLangItems', []) as $langId => $langItem) : ?>
+							<option value="<?= $enc->attr($langId); ?>" v-bind:selected="items[idx]['property'][propidx]['media.property.languageid'] == '<?= $enc->attr($langId) ?>'" >
+								<?= $enc->html($langItem->getLabel()); ?>
 							</option>
 						<?php endforeach; ?>
 					</select>
 				</td>
 				<td class="property-value">
-					<input class="form-control item-value" type="text" required="required" tabindex="<?= $this->get( 'tabindex' ); ?>"
-						v-bind:name="'<?= $enc->attr( $this->formparam( array( 'media', 'idx', 'property', 'propidx', 'media.property.value' ) ) ); ?>'.replace( 'idx', idx ).replace( 'propidx', propidx )"
-						placeholder="<?= $enc->attr( $this->translate( 'admin', 'Property value (required)' ) ); ?>"
+					<input class="form-control item-value" type="text" required="required" tabindex="<?= $this->get('tabindex'); ?>"
+						v-bind:name="'<?= $enc->attr($this->formparam(array('media', 'idx', 'property', 'propidx', 'media.property.value'))); ?>'.replace( 'idx', idx ).replace( 'propidx', propidx )"
+						placeholder="<?= $enc->attr($this->translate('admin', 'Property value (required)')); ?>"
 						v-bind:readonly="checkSite('media.siteid', idx)"
 						v-model="items[idx]['property'][propidx]['media.property.value']" >
 				</td>
 
 				<td class="actions">
-					<div v-if="!checkSite('media.siteid', idx)" class="btn act-delete fa" tabindex="<?= $this->get( 'tabindex' ); ?>"
-						title="<?= $enc->attr( $this->translate( 'admin', 'Delete this entry') ); ?>"
+					<div v-if="!checkSite('media.siteid', idx)" class="btn act-delete fa" tabindex="<?= $this->get('tabindex'); ?>"
+						title="<?= $enc->attr($this->translate('admin', 'Delete this entry')); ?>"
 						v-on:click.stop="removePropertyItem(idx, propidx)">
 					</div>
 				</td>
@@ -86,6 +86,6 @@ 
 		</tbody>
 	</table>
 
-	<?= $this->get( 'propertyBody' ); ?>
+	<?= $this->get('propertyBody'); ?>
 
 </div>

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

Spacing +16 added lines, -16 removed lines

@@ -19,52 +19,52 @@
 	{
 		$this->aimeos = \TestHelperJqadm::getAimeos();
 		$this->context = \TestHelperJqadm::getContext();
-		$this->context->setView( \TestHelperJqadm::getView() );
+		$this->context->setView(\TestHelperJqadm::getView());
 	}
 
 
 	public function testCreateClient()
 	{
-		$this->context->getConfig()->set( 'admin/jqadm/resources', ['product'] );
-		$client = \Aimeos\Admin\JQAdm\Factory::create( $this->context, $this->aimeos, 'product' );
-		$this->assertInstanceOf( '\\Aimeos\\Admin\\JQAdm\\Iface', $client );
+		$this->context->getConfig()->set('admin/jqadm/resources', ['product']);
+		$client = \Aimeos\Admin\JQAdm\Factory::create($this->context, $this->aimeos, 'product');
+		$this->assertInstanceOf('\\Aimeos\\Admin\\JQAdm\\Iface', $client);
 	}
 
 
 	public function testCreateClientName()
 	{
-		$this->context->getConfig()->set( 'admin/jqadm/resources', ['product'] );
-		$client = \Aimeos\Admin\JQAdm\Factory::create( $this->context, $this->aimeos, 'product', 'Standard' );
-		$this->assertInstanceOf( '\\Aimeos\\Admin\\JQAdm\\Iface', $client );
+		$this->context->getConfig()->set('admin/jqadm/resources', ['product']);
+		$client = \Aimeos\Admin\JQAdm\Factory::create($this->context, $this->aimeos, 'product', 'Standard');
+		$this->assertInstanceOf('\\Aimeos\\Admin\\JQAdm\\Iface', $client);
 	}
 
 
 	public function testCreateSubClient()
 	{
-		$this->context->getConfig()->set( 'admin/jqadm/resources', ['locale/site'] );
-		$client = \Aimeos\Admin\JQAdm\Factory::create( $this->context, $this->aimeos, 'locale/site' );
-		$this->assertInstanceOf( '\\Aimeos\\Admin\\JQAdm\\Iface', $client );
+		$this->context->getConfig()->set('admin/jqadm/resources', ['locale/site']);
+		$client = \Aimeos\Admin\JQAdm\Factory::create($this->context, $this->aimeos, 'locale/site');
+		$this->assertInstanceOf('\\Aimeos\\Admin\\JQAdm\\Iface', $client);
 	}
 
 
 	public function testCreateClientNameEmpty()
 	{
-		$this->setExpectedException( '\\Aimeos\\Admin\\JQAdm\\Exception' );
-		\Aimeos\Admin\JQAdm\Factory::create( $this->context, $this->aimeos, '' );
+		$this->setExpectedException('\\Aimeos\\Admin\\JQAdm\\Exception');
+		\Aimeos\Admin\JQAdm\Factory::create($this->context, $this->aimeos, '');
 	}
 
 
 	public function testCreateClientNameInvalid()
 	{
-		$this->setExpectedException( '\\Aimeos\\Admin\\JQAdm\\Exception' );
-		\Aimeos\Admin\JQAdm\Factory::create( $this->context, $this->aimeos, '%product' );
+		$this->setExpectedException('\\Aimeos\\Admin\\JQAdm\\Exception');
+		\Aimeos\Admin\JQAdm\Factory::create($this->context, $this->aimeos, '%product');
 	}
 
 
 	public function testCreateClientNameNotFound()
 	{
-		$this->setExpectedException( '\\Aimeos\\Admin\\JQAdm\\Exception' );
-		\Aimeos\Admin\JQAdm\Factory::create( $this->context, $this->aimeos, 'unknown' );
+		$this->setExpectedException('\\Aimeos\\Admin\\JQAdm\\Exception');
+		\Aimeos\Admin\JQAdm\Factory::create($this->context, $this->aimeos, 'unknown');
 	}
 
 }

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

Spacing +11 added lines, -11 removed lines

@@ -16,42 +16,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\Catalog\Factory::create( $this->context );
-		$this->assertInstanceOf( '\\Aimeos\\Admin\\JQAdm\\Iface', $client );
+		$client = \Aimeos\Admin\JQAdm\Catalog\Factory::create($this->context);
+		$this->assertInstanceOf('\\Aimeos\\Admin\\JQAdm\\Iface', $client);
 	}
 
 
 	public function testCreateClientName()
 	{
-		$client = \Aimeos\Admin\JQAdm\Catalog\Factory::create( $this->context, 'Standard' );
-		$this->assertInstanceOf( '\\Aimeos\\Admin\\JQAdm\\Iface', $client );
+		$client = \Aimeos\Admin\JQAdm\Catalog\Factory::create($this->context, 'Standard');
+		$this->assertInstanceOf('\\Aimeos\\Admin\\JQAdm\\Iface', $client);
 	}
 
 
 	public function testCreateClientNameEmpty()
 	{
-		$this->setExpectedException( '\\Aimeos\\Admin\\JQAdm\\Exception' );
-		\Aimeos\Admin\JQAdm\Catalog\Factory::create( $this->context, '' );
+		$this->setExpectedException('\\Aimeos\\Admin\\JQAdm\\Exception');
+		\Aimeos\Admin\JQAdm\Catalog\Factory::create($this->context, '');
 	}
 
 
 	public function testCreateClientNameInvalid()
 	{
-		$this->setExpectedException( '\\Aimeos\\Admin\\JQAdm\\Exception' );
-		\Aimeos\Admin\JQAdm\Catalog\Factory::create( $this->context, '%catalog' );
+		$this->setExpectedException('\\Aimeos\\Admin\\JQAdm\\Exception');
+		\Aimeos\Admin\JQAdm\Catalog\Factory::create($this->context, '%catalog');
 	}
 
 
 	public function testCreateClientNameNotFound()
 	{
-		$this->setExpectedException( '\\Aimeos\\Admin\\JQAdm\\Exception' );
-		\Aimeos\Admin\JQAdm\Catalog\Factory::create( $this->context, 'test' );
+		$this->setExpectedException('\\Aimeos\\Admin\\JQAdm\\Exception');
+		\Aimeos\Admin\JQAdm\Catalog\Factory::create($this->context, 'test');
 	}
 
 }

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;
 	}