aimeos / ai-admin-jqadm

admin/jqadm/src/Admin/JQAdm/Type/Text/Factory.php

Spacing +8 added lines, -8 removed lines

@@ -29,7 +29,7 @@ 
 	 * @return \Aimeos\Admin\JQAdm\Iface Filter part implementing \Aimeos\Admin\JQAdm\Iface
 	 * @throws \Aimeos\Admin\JQAdm\Exception If requested client implementation couldn't be found or initialisation fails
 	 */
-	public static function create( \Aimeos\MShop\Context\Item\Iface $context, $name = null )
+	public static function create(\Aimeos\MShop\Context\Item\Iface $context, $name = null)
 	{
 		/** admin/jqadm/type/text/name
 		 * Class name of the used account favorite client implementation
@@ -64,22 +64,22 @@ 
 		 * @since 2017.10
 		 * @category Developer
 		 */
-		if( $name === null ) {
-			$name = $context->getConfig()->get( 'admin/jqadm/type/text/name', 'Standard' );
+		if ($name === null) {
+			$name = $context->getConfig()->get('admin/jqadm/type/text/name', 'Standard');
 		}
 
-		if( ctype_alnum( $name ) === false )
+		if (ctype_alnum($name) === false)
 		{
-			$classname = is_string( $name ) ? '\\Aimeos\\Admin\\JQAdm\\Type\\Text\\' . $name : '<not a string>';
-			throw new \Aimeos\Admin\JQAdm\Exception( sprintf( 'Invalid characters in class name "%1$s"', $classname ) );
+			$classname = is_string($name) ? '\\Aimeos\\Admin\\JQAdm\\Type\\Text\\' . $name : '<not a string>';
+			throw new \Aimeos\Admin\JQAdm\Exception(sprintf('Invalid characters in class name "%1$s"', $classname));
 		}
 
 		$iface = '\\Aimeos\\Admin\\JQAdm\\Iface';
 		$classname = '\\Aimeos\\Admin\\JQAdm\\Type\\Text\\' . $name;
 
-		$client = self::createAdmin( $context, $classname, $iface );
+		$client = self::createAdmin($context, $classname, $iface);
 
-		return self::addClientDecorators( $context, $client, 'type/text' );
+		return self::addClientDecorators($context, $client, 'type/text');
 	}
 
 }
\ No newline at end of file

admin/jqadm/src/Admin/JQAdm/Type/Price/Lists/Factory.php

Spacing +8 added lines, -8 removed lines

@@ -29,7 +29,7 @@ 
 	 * @return \Aimeos\Admin\JQAdm\Iface Filter part implementing \Aimeos\Admin\JQAdm\Iface
 	 * @throws \Aimeos\Admin\JQAdm\Exception If requested client implementation couldn't be found or initialisation fails
 	 */
-	public static function create( \Aimeos\MShop\Context\Item\Iface $context, $name = null )
+	public static function create(\Aimeos\MShop\Context\Item\Iface $context, $name = null)
 	{
 		/** admin/jqadm/type/price/lists/name
 		 * Class name of the used account favorite client implementation
@@ -64,22 +64,22 @@ 
 		 * @since 2017.10
 		 * @category Developer
 		 */
-		if( $name === null ) {
-			$name = $context->getConfig()->get( 'admin/jqadm/type/price/lists/name', 'Standard' );
+		if ($name === null) {
+			$name = $context->getConfig()->get('admin/jqadm/type/price/lists/name', 'Standard');
 		}
 
-		if( ctype_alnum( $name ) === false )
+		if (ctype_alnum($name) === false)
 		{
-			$classname = is_string( $name ) ? '\\Aimeos\\Admin\\JQAdm\\Type\\Price\\Lists\\' . $name : '<not a string>';
-			throw new \Aimeos\Admin\JQAdm\Exception( sprintf( 'Invalid characters in class name "%1$s"', $classname ) );
+			$classname = is_string($name) ? '\\Aimeos\\Admin\\JQAdm\\Type\\Price\\Lists\\' . $name : '<not a string>';
+			throw new \Aimeos\Admin\JQAdm\Exception(sprintf('Invalid characters in class name "%1$s"', $classname));
 		}
 
 		$iface = '\\Aimeos\\Admin\\JQAdm\\Iface';
 		$classname = '\\Aimeos\\Admin\\JQAdm\\Type\\Price\\Lists\\' . $name;
 
-		$client = self::createAdmin( $context, $classname, $iface );
+		$client = self::createAdmin($context, $classname, $iface);
 
-		return self::addClientDecorators( $context, $client, 'type/price/lists' );
+		return self::addClientDecorators($context, $client, 'type/price/lists');
 	}
 
 }
\ No newline at end of file

admin/jqadm/src/Admin/JQAdm/Type/Price/Factory.php

Spacing +8 added lines, -8 removed lines

@@ -29,7 +29,7 @@ 
 	 * @return \Aimeos\Admin\JQAdm\Iface Filter part implementing \Aimeos\Admin\JQAdm\Iface
 	 * @throws \Aimeos\Admin\JQAdm\Exception If requested client implementation couldn't be found or initialisation fails
 	 */
-	public static function create( \Aimeos\MShop\Context\Item\Iface $context, $name = null )
+	public static function create(\Aimeos\MShop\Context\Item\Iface $context, $name = null)
 	{
 		/** admin/jqadm/type/price/name
 		 * Class name of the used account favorite client implementation
@@ -64,22 +64,22 @@ 
 		 * @since 2017.10
 		 * @category Developer
 		 */
-		if( $name === null ) {
-			$name = $context->getConfig()->get( 'admin/jqadm/type/price/name', 'Standard' );
+		if ($name === null) {
+			$name = $context->getConfig()->get('admin/jqadm/type/price/name', 'Standard');
 		}
 
-		if( ctype_alnum( $name ) === false )
+		if (ctype_alnum($name) === false)
 		{
-			$classname = is_string( $name ) ? '\\Aimeos\\Admin\\JQAdm\\Type\\Price\\' . $name : '<not a string>';
-			throw new \Aimeos\Admin\JQAdm\Exception( sprintf( 'Invalid characters in class name "%1$s"', $classname ) );
+			$classname = is_string($name) ? '\\Aimeos\\Admin\\JQAdm\\Type\\Price\\' . $name : '<not a string>';
+			throw new \Aimeos\Admin\JQAdm\Exception(sprintf('Invalid characters in class name "%1$s"', $classname));
 		}
 
 		$iface = '\\Aimeos\\Admin\\JQAdm\\Iface';
 		$classname = '\\Aimeos\\Admin\\JQAdm\\Type\\Price\\' . $name;
 
-		$client = self::createAdmin( $context, $classname, $iface );
+		$client = self::createAdmin($context, $classname, $iface);
 
-		return self::addClientDecorators( $context, $client, 'type/price' );
+		return self::addClientDecorators($context, $client, 'type/price');
 	}
 
 }
\ No newline at end of file

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

Spacing +146 added lines, -146 removed lines

@@ -10,7 +10,7 @@ 
 
 namespace Aimeos\Admin\JQAdm\Order;
 
-sprintf( 'order' ); // for translation
+sprintf('order'); // for translation
 
 
 /**
@@ -35,37 +35,37 @@ 
 
 		try
 		{
-			if( ( $id = $view->param( 'id' ) ) === null ) {
-				throw new \Aimeos\Admin\JQAdm\Exception( sprintf( 'Required parameter "%1$s" is missing', 'id' ) );
+			if (($id = $view->param('id')) === null) {
+				throw new \Aimeos\Admin\JQAdm\Exception(sprintf('Required parameter "%1$s" is missing', 'id'));
 			}
 
-			$manager = \Aimeos\MShop::create( $context, 'order/base' );
-			$view->item = $manager->load( $id );
+			$manager = \Aimeos\MShop::create($context, 'order/base');
+			$view->item = $manager->load($id);
 
-			$view->itemData = $this->toArray( $view->item, true );
+			$view->itemData = $this->toArray($view->item, true);
 			$view->itemSubparts = $this->getSubClientNames();
 			$view->itemBody = '';
 
-			foreach( $this->getSubClients() as $idx => $client )
+			foreach ($this->getSubClients() as $idx => $client)
 			{
 				$view->tabindex = ++$idx + 1;
 				$view->itemBody .= $client->copy();
 			}
 		}
-		catch( \Aimeos\MShop\Exception $e )
+		catch (\Aimeos\MShop\Exception $e)
 		{
-			$error = array( 'order-item' => $context->getI18n()->dt( 'mshop', $e->getMessage() ) );
-			$view->errors = $view->get( 'errors', [] ) + $error;
-			$this->logException( $e );
+			$error = array('order-item' => $context->getI18n()->dt('mshop', $e->getMessage()));
+			$view->errors = $view->get('errors', []) + $error;
+			$this->logException($e);
 		}
-		catch( \Exception $e )
+		catch (\Exception $e)
 		{
-			$error = array( 'order-item' => $e->getMessage() . ', ' . $e->getFile() . ':' . $e->getLine() );
-			$view->errors = $view->get( 'errors', [] ) + $error;
-			$this->logException( $e );
+			$error = array('order-item' => $e->getMessage() . ', ' . $e->getFile() . ':' . $e->getLine());
+			$view->errors = $view->get('errors', []) + $error;
+			$this->logException($e);
 		}
 
-		return $this->render( $view );
+		return $this->render($view);
 	}
 
 
@@ -81,10 +81,10 @@ 
 
 		try
 		{
-			$data = $view->param( 'item', [] );
+			$data = $view->param('item', []);
 
-			if( !isset( $view->item ) ) {
-				$view->item = \Aimeos\MShop::create( $context, 'order/base' )->createItem();
+			if (!isset($view->item)) {
+				$view->item = \Aimeos\MShop::create($context, 'order/base')->createItem();
 			}
 
 			$data['order.siteid'] = $view->item->getSiteId();
@@ -93,26 +93,26 @@ 
 			$view->itemData = $data;
 			$view->itemBody = '';
 
-			foreach( $this->getSubClients() as $idx => $client )
+			foreach ($this->getSubClients() as $idx => $client)
 			{
 				$view->tabindex = ++$idx + 1;
 				$view->itemBody .= $client->create();
 			}
 		}
-		catch( \Aimeos\MShop\Exception $e )
+		catch (\Aimeos\MShop\Exception $e)
 		{
-			$error = array( 'order-item' => $context->getI18n()->dt( 'mshop', $e->getMessage() ) );
-			$view->errors = $view->get( 'errors', [] ) + $error;
-			$this->logException( $e );
+			$error = array('order-item' => $context->getI18n()->dt('mshop', $e->getMessage()));
+			$view->errors = $view->get('errors', []) + $error;
+			$this->logException($e);
 		}
-		catch( \Exception $e )
+		catch (\Exception $e)
 		{
-			$error = array( 'order-item' => $e->getMessage() . ', ' . $e->getFile() . ':' . $e->getLine() );
-			$view->errors = $view->get( 'errors', [] ) + $error;
-			$this->logException( $e );
+			$error = array('order-item' => $e->getMessage() . ', ' . $e->getFile() . ':' . $e->getLine());
+			$view->errors = $view->get('errors', []) + $error;
+			$this->logException($e);
 		}
 
-		return $this->render( $view );
+		return $this->render($view);
 	}
 
 
@@ -128,34 +128,34 @@ 
 
 		try
 		{
-			$params = $this->storeSearchParams( $view->param(), 'order' );
+			$params = $this->storeSearchParams($view->param(), 'order');
 			$msg = ['sitecode' => $context->getLocale()->getSite()->getCode()];
 
-			if( isset( $params['filter'] ) ) {
-				$msg['filter'] = $this->getCriteriaConditions( $params['filter'] );
+			if (isset($params['filter'])) {
+				$msg['filter'] = $this->getCriteriaConditions($params['filter']);
 			}
 
-			if( isset( $params['sort'] ) ) {
-				$msg['sort'] = $this->getCriteriaSortations( $params['sort'] );
+			if (isset($params['sort'])) {
+				$msg['sort'] = $this->getCriteriaSortations($params['sort']);
 			}
 
-			$mq = $context->getMessageQueueManager()->get( 'mq-admin' )->getQueue( 'order-export' );
-			$mq->add( json_encode( $msg ) );
+			$mq = $context->getMessageQueueManager()->get('mq-admin')->getQueue('order-export');
+			$mq->add(json_encode($msg));
 
-			$msg = $context->getI18n()->dt( 'admin', 'Your export will be available in a few minutes for download' );
-			$view->info = $view->get( 'info', [] ) + ['order-item' => $msg];
+			$msg = $context->getI18n()->dt('admin', 'Your export will be available in a few minutes for download');
+			$view->info = $view->get('info', []) + ['order-item' => $msg];
 		}
-		catch( \Aimeos\MShop\Exception $e )
+		catch (\Aimeos\MShop\Exception $e)
 		{
-			$error = array( 'order-item' => $context->getI18n()->dt( 'mshop', $e->getMessage() ) );
-			$view->errors = $view->get( 'errors', [] ) + $error;
-			$this->logException( $e );
+			$error = array('order-item' => $context->getI18n()->dt('mshop', $e->getMessage()));
+			$view->errors = $view->get('errors', []) + $error;
+			$this->logException($e);
 		}
-		catch( \Exception $e )
+		catch (\Exception $e)
 		{
-			$error = array( 'order-item' => $e->getMessage() . ', ' . $e->getFile() . ':' . $e->getLine() );
-			$view->errors = $view->get( 'errors', [] ) + $error;
-			$this->logException( $e );
+			$error = array('order-item' => $e->getMessage() . ', ' . $e->getFile() . ':' . $e->getLine());
+			$view->errors = $view->get('errors', []) + $error;
+			$this->logException($e);
 		}
 
 		return $this->search();
@@ -174,38 +174,38 @@ 
 
 		try
 		{
-			if( ( $id = $view->param( 'id' ) ) === null ) {
-				throw new \Aimeos\Admin\JQAdm\Exception( sprintf( 'Required parameter "%1$s" is missing', 'id' ) );
+			if (($id = $view->param('id')) === null) {
+				throw new \Aimeos\Admin\JQAdm\Exception(sprintf('Required parameter "%1$s" is missing', 'id'));
 			}
 
-			$manager = \Aimeos\MShop::create( $context, 'order/base' );
+			$manager = \Aimeos\MShop::create($context, 'order/base');
 			$refs = ['order/base/address', 'order/base/coupon', 'order/base/product', 'order/base/service'];
 
-			$view->item = $manager->getItem( $id, $refs );
+			$view->item = $manager->getItem($id, $refs);
 			$view->itemSubparts = $this->getSubClientNames();
-			$view->itemData = $this->toArray( $view->item );
+			$view->itemData = $this->toArray($view->item);
 			$view->itemBody = '';
 
-			foreach( $this->getSubClients() as $idx => $client )
+			foreach ($this->getSubClients() as $idx => $client)
 			{
 				$view->tabindex = ++$idx + 1;
 				$view->itemBody .= $client->get();
 			}
 		}
-		catch( \Aimeos\MShop\Exception $e )
+		catch (\Aimeos\MShop\Exception $e)
 		{
-			$error = array( 'order-item' => $context->getI18n()->dt( 'mshop', $e->getMessage() ) );
-			$view->errors = $view->get( 'errors', [] ) + $error;
-			$this->logException( $e );
+			$error = array('order-item' => $context->getI18n()->dt('mshop', $e->getMessage()));
+			$view->errors = $view->get('errors', []) + $error;
+			$this->logException($e);
 		}
-		catch( \Exception $e )
+		catch (\Exception $e)
 		{
-			$error = array( 'order-item' => $e->getMessage() . ', ' . $e->getFile() . ':' . $e->getLine() );
-			$view->errors = $view->get( 'errors', [] ) + $error;
-			$this->logException( $e );
+			$error = array('order-item' => $e->getMessage() . ', ' . $e->getFile() . ':' . $e->getLine());
+			$view->errors = $view->get('errors', []) + $error;
+			$this->logException($e);
 		}
 
-		return $this->render( $view );
+		return $this->render($view);
 	}
 
 
@@ -219,40 +219,40 @@ 
 		$view = $this->getView();
 		$context = $this->getContext();
 
-		$manager = \Aimeos\MShop::create( $context, 'order/base' );
+		$manager = \Aimeos\MShop::create($context, 'order/base');
 		$manager->begin();
 
 		try
 		{
-			$item = $this->fromArray( $view->param( 'item', [] ) );
-			$view->item = $item->getId() ? $item : $manager->store( clone $item );
+			$item = $this->fromArray($view->param('item', []));
+			$view->item = $item->getId() ? $item : $manager->store(clone $item);
 			$view->itemBody = '';
 
-			foreach( $this->getSubClients() as $client ) {
+			foreach ($this->getSubClients() as $client) {
 				$view->itemBody .= $client->save();
 			}
 
-			$manager->store( clone $view->item );
+			$manager->store(clone $view->item);
 			$manager->commit();
 
-			$this->nextAction( $view, $view->param( 'next' ), 'order', $view->item->getId(), 'save' );
+			$this->nextAction($view, $view->param('next'), 'order', $view->item->getId(), 'save');
 			return;
 		}
-		catch( \Aimeos\Admin\JQAdm\Exception $e )
+		catch (\Aimeos\Admin\JQAdm\Exception $e)
 		{
 			// fall through to create
 		}
-		catch( \Aimeos\MShop\Exception $e )
+		catch (\Aimeos\MShop\Exception $e)
 		{
-			$error = array( 'order-item' => $context->getI18n()->dt( 'mshop', $e->getMessage() ) );
-			$view->errors = $view->get( 'errors', [] ) + $error;
-			$this->logException( $e );
+			$error = array('order-item' => $context->getI18n()->dt('mshop', $e->getMessage()));
+			$view->errors = $view->get('errors', []) + $error;
+			$this->logException($e);
 		}
-		catch( \Exception $e )
+		catch (\Exception $e)
 		{
-			$error = array( 'order-item' => $e->getMessage() . ', ' . $e->getFile() . ':' . $e->getLine() );
-			$view->errors = $view->get( 'errors', [] ) + $error;
-			$this->logException( $e );
+			$error = array('order-item' => $e->getMessage() . ', ' . $e->getFile() . ':' . $e->getLine());
+			$view->errors = $view->get('errors', []) + $error;
+			$this->logException($e);
 		}
 
 		$manager->rollback();
@@ -274,35 +274,35 @@ 
 		try
 		{
 			$total = 0;
-			$params = $this->storeSearchParams( $view->param(), 'order' );
-			$manager = \Aimeos\MShop::create( $context, 'order' );
+			$params = $this->storeSearchParams($view->param(), 'order');
+			$manager = \Aimeos\MShop::create($context, 'order');
 
 			$search = $manager->createSearch();
-			$search->setSortations( [$search->sort( '-', 'order.id' )] );
-			$search = $this->initCriteria( $search, $params );
+			$search->setSortations([$search->sort('-', 'order.id')]);
+			$search = $this->initCriteria($search, $params);
 
-			$view->items = $manager->searchItems( $search, [], $total );
-			$view->baseItems = $this->getOrderBaseItems( $view->items );
-			$view->filterAttributes = $manager->getSearchAttributes( true );
+			$view->items = $manager->searchItems($search, [], $total);
+			$view->baseItems = $this->getOrderBaseItems($view->items);
+			$view->filterAttributes = $manager->getSearchAttributes(true);
 			$view->filterOperators = $search->getOperators();
 			$view->total = $total;
 			$view->itemBody = '';
 
-			foreach( $this->getSubClients() as $client ) {
+			foreach ($this->getSubClients() as $client) {
 				$view->itemBody .= $client->search();
 			}
 		}
-		catch( \Aimeos\MShop\Exception $e )
+		catch (\Aimeos\MShop\Exception $e)
 		{
-			$error = array( 'order-item' => $context->getI18n()->dt( 'mshop', $e->getMessage() ) );
-			$view->errors = $view->get( 'errors', [] ) + $error;
-			$this->logException( $e );
+			$error = array('order-item' => $context->getI18n()->dt('mshop', $e->getMessage()));
+			$view->errors = $view->get('errors', []) + $error;
+			$this->logException($e);
 		}
-		catch( \Exception $e )
+		catch (\Exception $e)
 		{
-			$error = array( 'order-item' => $e->getMessage() . ', ' . $e->getFile() . ':' . $e->getLine() );
-			$view->errors = $view->get( 'errors', [] ) + $error;
-			$this->logException( $e );
+			$error = array('order-item' => $e->getMessage() . ', ' . $e->getFile() . ':' . $e->getLine());
+			$view->errors = $view->get('errors', []) + $error;
+			$this->logException($e);
 		}
 
 		/** admin/jqadm/order/template-list
@@ -327,7 +327,7 @@ 
 		$tplconf = 'admin/jqadm/order/template-list';
 		$default = 'order/list-standard';
 
-		return $view->render( $view->config( $tplconf, $default ) );
+		return $view->render($view->config($tplconf, $default));
 	}
 
 
@@ -338,7 +338,7 @@ 
 	 * @param string|null $name Name of the sub-client (Default if null)
 	 * @return \Aimeos\Admin\JQAdm\Iface Sub-client object
 	 */
-	public function getSubClient( $type, $name = null )
+	public function getSubClient($type, $name = null)
 	{
 		/** admin/jqadm/order/decorators/excludes
 		 * Excludes decorators added by the "common" option from the order JQAdm client
@@ -413,7 +413,7 @@ 
 		 * @see admin/jqadm/order/decorators/excludes
 		 * @see admin/jqadm/order/decorators/global
 		 */
-		return $this->createSubClient( 'order/' . $type, $name );
+		return $this->createSubClient('order/' . $type, $name);
 	}
 
 
@@ -423,19 +423,19 @@ 
 	 * @param \Aimeos\MShop\Order\Item\Iface[] $orderItems List of order items
 	 * @param \Aimeos\MShop\Order\Item\Base\Iface[] List of order base items
 	 */
-	protected function getOrderBaseItems( array $orderItems )
+	protected function getOrderBaseItems(array $orderItems)
 	{
 		$baseIds = [];
-		foreach( $orderItems as $item ) {
+		foreach ($orderItems as $item) {
 			$baseIds[] = $item->getBaseId();
 		}
 
-		$manager = \Aimeos\MShop::create( $this->getContext(), 'order/base' );
+		$manager = \Aimeos\MShop::create($this->getContext(), 'order/base');
 
-		$search = $manager->createSearch()->setSlice( 0, count( $baseIds ) );
-		$search->setConditions( $search->compare( '==', 'order.base.id', $baseIds ) );
+		$search = $manager->createSearch()->setSlice(0, count($baseIds));
+		$search->setConditions($search->compare('==', 'order.base.id', $baseIds));
 
-		return $manager->searchItems( $search, ['order/base/address', 'order/base/service'] );
+		return $manager->searchItems($search, ['order/base/address', 'order/base/service']);
 	}
 
 
@@ -479,7 +479,7 @@ 
 		 * @since 2016.01
 		 * @category Developer
 		 */
-		return $this->getContext()->getConfig()->get( 'admin/jqadm/order/standard/subparts', [] );
+		return $this->getContext()->getConfig()->get('admin/jqadm/order/standard/subparts', []);
 	}
 
 
@@ -489,72 +489,72 @@ 
 	 * @param string[] Data array
 	 * @return \Aimeos\MShop\Order\Item\Iface New order item object
 	 */
-	protected function fromArray( array $data )
+	protected function fromArray(array $data)
 	{
-		$manager = \Aimeos\MShop::create( $this->getContext(), 'order/base' );
-		$attrManager = \Aimeos\MShop::create( $this->getContext(), 'order/base/service/attribute' );
+		$manager = \Aimeos\MShop::create($this->getContext(), 'order/base');
+		$attrManager = \Aimeos\MShop::create($this->getContext(), 'order/base/service/attribute');
 
-		if( isset( $data['order.base.id'] ) ) {
-			$basket = $manager->load( $data['order.base.id'] );
+		if (isset($data['order.base.id'])) {
+			$basket = $manager->load($data['order.base.id']);
 		} else {
 			$basket = $manager->createItem();
 		}
 
-		$basket->fromArray( $data );
+		$basket->fromArray($data);
 
-		foreach( $basket->getProducts() as $pos => $product )
+		foreach ($basket->getProducts() as $pos => $product)
 		{
-			if( isset( $data['product'][$pos]['order.base.product.status'] ) ) {
-				$product->setStatus( $data['product'][$pos]['order.base.product.status'] );
+			if (isset($data['product'][$pos]['order.base.product.status'])) {
+				$product->setStatus($data['product'][$pos]['order.base.product.status']);
 			}
 		}
 
-		foreach( $basket->getAddresses() as $type => $address )
+		foreach ($basket->getAddresses() as $type => $address)
 		{
-			if( isset( $data['address'][$type] ) ) {
+			if (isset($data['address'][$type])) {
 				$list = (array) $data['address'][$type];
-				$basket->setAddress( $address->fromArray( $list ), $type );
+				$basket->setAddress($address->fromArray($list), $type);
 			} else {
-				$basket->deleteAddress( $type );
+				$basket->deleteAddress($type);
 			}
 		}
 
-		foreach( $basket->getServices() as $type => $services )
+		foreach ($basket->getServices() as $type => $services)
 		{
-			foreach( $services as $serviceId => $service )
+			foreach ($services as $serviceId => $service)
 			{
 				$list = [];
 				$attrItems = $service->getAttributeItems();
 
-				if( isset( $data['service'][$type][$serviceId] ) )
+				if (isset($data['service'][$type][$serviceId]))
 				{
-					foreach( (array) $data['service'][$type][$serviceId] as $key => $pair )
+					foreach ((array) $data['service'][$type][$serviceId] as $key => $pair)
 					{
-						foreach( $pair as $pos => $value ) {
+						foreach ($pair as $pos => $value) {
 							$list[$pos][$key] = $value;
 						}
 					}
 
-					foreach( $list as $array )
+					foreach ($list as $array)
 					{
-						if( isset( $attrItems[$array['order.base.service.attribute.id']] ) )
+						if (isset($attrItems[$array['order.base.service.attribute.id']]))
 						{
 							$attrItem = $attrItems[$array['order.base.service.attribute.id']];
-							unset( $attrItems[$array['order.base.service.attribute.id']] );
+							unset($attrItems[$array['order.base.service.attribute.id']]);
 						}
 						else
 						{
 							$attrItem = $attrManager->createItem();
 						}
 
-						$attrItem->fromArray( $array );
-						$attrItem->setParentId( $service->getId() );
+						$attrItem->fromArray($array);
+						$attrItem->setParentId($service->getId());
 
-						$item = $attrManager->saveItem( $attrItem );
+						$item = $attrManager->saveItem($attrItem);
 					}
 				}
 
-				$attrManager->deleteItems( array_keys( $attrItems ) );
+				$attrManager->deleteItems(array_keys($attrItems));
 			}
 		}
 
@@ -568,53 +568,53 @@ 
 	 * @param \Aimeos\MShop\Order\Item\Base\Iface $item Order base item object
 	 * @return string[] Multi-dimensional associative list of item data
 	 */
-	protected function toArray( \Aimeos\MShop\Order\Item\Base\Iface $item, $copy = false )
+	protected function toArray(\Aimeos\MShop\Order\Item\Base\Iface $item, $copy = false)
 	{
 		$siteId = $this->getContext()->getLocale()->getSiteId();
-		$data = $item->toArray( true );
+		$data = $item->toArray(true);
 
-		if( $item->getCustomerId() != '' )
+		if ($item->getCustomerId() != '')
 		{
-			$manager = \Aimeos\MShop::create( $this->getContext(), 'customer' );
+			$manager = \Aimeos\MShop::create($this->getContext(), 'customer');
 
 			try {
-				$data += $manager->getItem( $item->getCustomerId() )->toArray();
-			} catch( \Exception $e ) {};
+				$data += $manager->getItem($item->getCustomerId())->toArray();
+			} catch (\Exception $e) {};
 		}
 
 
-		if( $copy === true )
+		if ($copy === true)
 		{
 			$data['order.base.siteid'] = $siteId;
 			$data['order.base.id'] = '';
 		}
 
-		foreach( $item->getAddresses() as $type => $addrItem )
+		foreach ($item->getAddresses() as $type => $addrItem)
 		{
-			$list = $addrItem->toArray( true );
+			$list = $addrItem->toArray(true);
 
-			foreach( $list as $key => $value ) {
+			foreach ($list as $key => $value) {
 				$data['address'][$type][$key] = $value;
 			}
 
-			if( $copy === true )
+			if ($copy === true)
 			{
 				$data['address'][$type]['order.base.address.siteid'] = $siteId;
 				$data['address'][$type]['order.base.address.id'] = '';
 			}
 		}
 
-		if( $copy !== true )
+		if ($copy !== true)
 		{
-			foreach( $item->getServices() as $type => $services )
+			foreach ($item->getServices() as $type => $services)
 			{
-				foreach( $services as $serviceItem )
+				foreach ($services as $serviceItem)
 				{
 					$serviceId = $serviceItem->getServiceId();
 
-					foreach( $serviceItem->getAttributeItems() as $attrItem )
+					foreach ($serviceItem->getAttributeItems() as $attrItem)
 					{
-						foreach( $attrItem->toArray( true ) as $key => $value ) {
+						foreach ($attrItem->toArray(true) as $key => $value) {
 							$data['service'][$type][$serviceId][$key][] = $value;
 						}
 					}
@@ -622,7 +622,7 @@ 
 			}
 		}
 
-		foreach( $item->getProducts() as $pos => $productItem ) {
+		foreach ($item->getProducts() as $pos => $productItem) {
 			$data['product'][$pos]['order.base.product.status'] = $productItem->getStatus();
 		}
 
@@ -636,7 +636,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/order/template-item
 		 * Relative path to the HTML body template for the order item.
@@ -660,6 +660,6 @@ 
 		$tplconf = 'admin/jqadm/order/template-item';
 		$default = 'order/item-standard';
 
-		return $view->render( $view->config( $tplconf, $default ) );
+		return $view->render($view->config($tplconf, $default));
 	}
 }

admin/jqadm/tests/TestHelperJqadm.php

Spacing +61 added lines, -61 removed lines

@@ -13,63 +13,63 @@ 
 	public static function bootstrap()
 	{
 		self::getAimeos();
-		\Aimeos\MShop::cache( false );
+		\Aimeos\MShop::cache(false);
 	}
 
 
-	public static function getContext( $site = 'unittest' )
+	public static function getContext($site = 'unittest')
 	{
-		if( !isset( self::$context[$site] ) ) {
-			self::$context[$site] = self::createContext( $site );
+		if (!isset(self::$context[$site])) {
+			self::$context[$site] = self::createContext($site);
 		}
 
 		return clone self::$context[$site];
 	}
 
 
-	public static function getView( $site = 'unittest', \Aimeos\MW\Config\Iface $config = null )
+	public static function getView($site = 'unittest', \Aimeos\MW\Config\Iface $config = null)
 	{
-		if( $config === null ) {
-			$config = self::getContext( $site )->getConfig();
+		if ($config === null) {
+			$config = self::getContext($site)->getConfig();
 		}
 
-		$view = new \Aimeos\MW\View\Standard( self::getTemplatePaths() );
+		$view = new \Aimeos\MW\View\Standard(self::getTemplatePaths());
 
-		$helper = new \Aimeos\MW\View\Helper\Param\Standard( $view, ['site' => 'unittest'] );
-		$view->addHelper( 'param', $helper );
+		$helper = new \Aimeos\MW\View\Helper\Param\Standard($view, ['site' => 'unittest']);
+		$view->addHelper('param', $helper);
 
-		$trans = new \Aimeos\MW\Translation\None( 'de_DE' );
-		$helper = new \Aimeos\MW\View\Helper\Translate\Standard( $view, $trans );
-		$view->addHelper( 'translate', $helper );
+		$trans = new \Aimeos\MW\Translation\None('de_DE');
+		$helper = new \Aimeos\MW\View\Helper\Translate\Standard($view, $trans);
+		$view->addHelper('translate', $helper);
 
-		$helper = new \Aimeos\MW\View\Helper\Url\Standard( $view, 'http://baseurl' );
-		$view->addHelper( 'url', $helper );
+		$helper = new \Aimeos\MW\View\Helper\Url\Standard($view, 'http://baseurl');
+		$view->addHelper('url', $helper);
 
-		$helper = new \Aimeos\MW\View\Helper\Number\Standard( $view, '.', '' );
-		$view->addHelper( 'number', $helper );
+		$helper = new \Aimeos\MW\View\Helper\Number\Standard($view, '.', '');
+		$view->addHelper('number', $helper);
 
-		$helper = new \Aimeos\MW\View\Helper\Date\Standard( $view, 'Y-m-d' );
-		$view->addHelper( 'date', $helper );
+		$helper = new \Aimeos\MW\View\Helper\Date\Standard($view, 'Y-m-d');
+		$view->addHelper('date', $helper);
 
-		$config = new \Aimeos\MW\Config\Decorator\Protect( $config, ['admin', 'resource/fs/baseurl'] );
-		$helper = new \Aimeos\MW\View\Helper\Config\Standard( $view, $config );
-		$view->addHelper( 'config', $helper );
+		$config = new \Aimeos\MW\Config\Decorator\Protect($config, ['admin', 'resource/fs/baseurl']);
+		$helper = new \Aimeos\MW\View\Helper\Config\Standard($view, $config);
+		$view->addHelper('config', $helper);
 
-		$helper = new \Aimeos\MW\View\Helper\Session\Standard( $view, new \Aimeos\MW\Session\None() );
-		$view->addHelper( 'session', $helper );
+		$helper = new \Aimeos\MW\View\Helper\Session\Standard($view, new \Aimeos\MW\Session\None());
+		$view->addHelper('session', $helper);
 
-		$helper = new \Aimeos\MW\View\Helper\Request\Standard( $view, new \Zend\Diactoros\ServerRequest() );
-		$view->addHelper( 'request', $helper );
+		$helper = new \Aimeos\MW\View\Helper\Request\Standard($view, new \Zend\Diactoros\ServerRequest());
+		$view->addHelper('request', $helper);
 
-		$helper = new \Aimeos\MW\View\Helper\Response\Standard( $view, new \Zend\Diactoros\Response() );
-		$view->addHelper( 'response', $helper );
+		$helper = new \Aimeos\MW\View\Helper\Response\Standard($view, new \Zend\Diactoros\Response());
+		$view->addHelper('response', $helper);
 
-		$helper = new \Aimeos\MW\View\Helper\Csrf\Standard( $view, '_csrf_token', '_csrf_value' );
-		$view->addHelper( 'csrf', $helper );
+		$helper = new \Aimeos\MW\View\Helper\Csrf\Standard($view, '_csrf_token', '_csrf_value');
+		$view->addHelper('csrf', $helper);
 
-		$fcn = function() { return array( 'admin' ); };
-		$helper = new \Aimeos\MW\View\Helper\Access\Standard( $view, $fcn );
-		$view->addHelper( 'access', $helper );
+		$fcn = function() { return array('admin'); };
+		$helper = new \Aimeos\MW\View\Helper\Access\Standard($view, $fcn);
+		$view->addHelper('access', $helper);
 
 		$view->pageSitePath = [];
 
@@ -79,19 +79,19 @@ 
 
 	public static function getTemplatePaths()
 	{
-		return self::getAimeos()->getCustomPaths( 'admin/jqadm/templates' );
+		return self::getAimeos()->getCustomPaths('admin/jqadm/templates');
 	}
 
 
 	public static function getAimeos()
 	{
-		if( !isset( self::$aimeos ) )
+		if (!isset(self::$aimeos))
 		{
 			require_once 'Bootstrap.php';
-			spl_autoload_register( 'Aimeos\\Bootstrap::autoload' );
+			spl_autoload_register('Aimeos\\Bootstrap::autoload');
 
-			$extdir = dirname( dirname( dirname( dirname( __FILE__ ) ) ) );
-			self::$aimeos = new \Aimeos\Bootstrap( array( $extdir ), false );
+			$extdir = dirname(dirname(dirname(dirname(__FILE__))));
+			self::$aimeos = new \Aimeos\Bootstrap(array($extdir), false);
 		}
 
 		return self::$aimeos;
@@ -101,57 +101,57 @@ 
 	/**
 	 * @param string $site
 	 */
-	private static function createContext( $site )
+	private static function createContext($site)
 	{
 		$ctx = new \Aimeos\MShop\Context\Item\Standard();
 		$aimeos = self::getAimeos();
 
 
-		$paths = $aimeos->getConfigPaths( 'mysql' );
+		$paths = $aimeos->getConfigPaths('mysql');
 		$paths[] = __DIR__ . DIRECTORY_SEPARATOR . 'config';
 		$file = __DIR__ . DIRECTORY_SEPARATOR . 'confdoc.ser';
-		$local = array( 'resource' => array( 'fs' => array( 'adapter' => 'Standard', 'basedir' => __DIR__ . '/tmp' ) ) );
+		$local = array('resource' => array('fs' => array('adapter' => 'Standard', 'basedir' => __DIR__ . '/tmp')));
 
-		$conf = new \Aimeos\MW\Config\PHPArray( $local, $paths );
-		$conf = new \Aimeos\MW\Config\Decorator\Memory( $conf );
-		$conf = new \Aimeos\MW\Config\Decorator\Documentor( $conf, $file );
-		$ctx->setConfig( $conf );
+		$conf = new \Aimeos\MW\Config\PHPArray($local, $paths);
+		$conf = new \Aimeos\MW\Config\Decorator\Memory($conf);
+		$conf = new \Aimeos\MW\Config\Decorator\Documentor($conf, $file);
+		$ctx->setConfig($conf);
 
 
-		$dbm = new \Aimeos\MW\DB\Manager\PDO( $conf );
-		$ctx->setDatabaseManager( $dbm );
+		$dbm = new \Aimeos\MW\DB\Manager\PDO($conf);
+		$ctx->setDatabaseManager($dbm);
 
 
-		$fs = new \Aimeos\MW\Filesystem\Manager\Standard( $conf );
-		$ctx->setFilesystemManager( $fs );
+		$fs = new \Aimeos\MW\Filesystem\Manager\Standard($conf);
+		$ctx->setFilesystemManager($fs);
 
 
-		$mq = new \Aimeos\MW\MQueue\Manager\Standard( $conf );
-		$ctx->setMessageQueueManager( $mq );
+		$mq = new \Aimeos\MW\MQueue\Manager\Standard($conf);
+		$ctx->setMessageQueueManager($mq);
 
 
-		$logger = new \Aimeos\MW\Logger\File( $site . '.log', \Aimeos\MW\Logger\Base::DEBUG );
-		$ctx->setLogger( $logger );
+		$logger = new \Aimeos\MW\Logger\File($site . '.log', \Aimeos\MW\Logger\Base::DEBUG);
+		$ctx->setLogger($logger);
 
 
 		$cache = new \Aimeos\MW\Cache\None();
-		$ctx->setCache( $cache );
+		$ctx->setCache($cache);
 
 
-		$i18n = new \Aimeos\MW\Translation\None( 'de' );
-		$ctx->setI18n( array( 'de' => $i18n ) );
+		$i18n = new \Aimeos\MW\Translation\None('de');
+		$ctx->setI18n(array('de' => $i18n));
 
 
 		$session = new \Aimeos\MW\Session\None();
-		$ctx->setSession( $session );
+		$ctx->setSession($session);
 
 
-		$localeManager = \Aimeos\MShop::create( $ctx, 'locale' );
-		$locale = $localeManager->bootstrap( $site, '', '', false );
-		$ctx->setLocale( $locale );
+		$localeManager = \Aimeos\MShop::create($ctx, 'locale');
+		$locale = $localeManager->bootstrap($site, '', '', false);
+		$ctx->setLocale($locale);
 
 
-		$ctx->setEditor( 'ai-admin-jqadm:admin/jqadm' );
+		$ctx->setEditor('ai-admin-jqadm:admin/jqadm');
 
 		return $ctx;
 	}

lib/custom/tests/TestHelperCustom.php

Spacing +52 added lines, -52 removed lines

@@ -13,55 +13,55 @@ 
 	public static function bootstrap()
 	{
 		self::getAimeos();
-		\Aimeos\MShop::cache( false );
+		\Aimeos\MShop::cache(false);
 	}
 
 
-	public static function getContext( $site = 'unittest' )
+	public static function getContext($site = 'unittest')
 	{
-		if( !isset( self::$context[$site] ) ) {
-			self::$context[$site] = self::createContext( $site );
+		if (!isset(self::$context[$site])) {
+			self::$context[$site] = self::createContext($site);
 		}
 
 		return clone self::$context[$site];
 	}
 
 
-	public static function getView( $site = 'unittest', \Aimeos\MW\Config\Iface $config = null )
+	public static function getView($site = 'unittest', \Aimeos\MW\Config\Iface $config = null)
 	{
-		if( $config === null ) {
-			$config = self::getContext( $site )->getConfig();
+		if ($config === null) {
+			$config = self::getContext($site)->getConfig();
 		}
 
-		$view = new \Aimeos\MW\View\Standard( self::getTemplatePaths() );
+		$view = new \Aimeos\MW\View\Standard(self::getTemplatePaths());
 
 		$param = ['site' => 'unittest'];
-		$helper = new \Aimeos\MW\View\Helper\Param\Standard( $view, $param );
-		$view->addHelper( 'param', $helper );
+		$helper = new \Aimeos\MW\View\Helper\Param\Standard($view, $param);
+		$view->addHelper('param', $helper);
 
-		$trans = new \Aimeos\MW\Translation\None( 'de_DE' );
-		$helper = new \Aimeos\MW\View\Helper\Translate\Standard( $view, $trans );
-		$view->addHelper( 'translate', $helper );
+		$trans = new \Aimeos\MW\Translation\None('de_DE');
+		$helper = new \Aimeos\MW\View\Helper\Translate\Standard($view, $trans);
+		$view->addHelper('translate', $helper);
 
-		$helper = new \Aimeos\MW\View\Helper\Url\Standard( $view, 'http://baseurl' );
-		$view->addHelper( 'url', $helper );
+		$helper = new \Aimeos\MW\View\Helper\Url\Standard($view, 'http://baseurl');
+		$view->addHelper('url', $helper);
 
-		$helper = new \Aimeos\MW\View\Helper\Number\Standard( $view, '.', '' );
-		$view->addHelper( 'number', $helper );
+		$helper = new \Aimeos\MW\View\Helper\Number\Standard($view, '.', '');
+		$view->addHelper('number', $helper);
 
-		$helper = new \Aimeos\MW\View\Helper\Date\Standard( $view, 'Y-m-d' );
-		$view->addHelper( 'date', $helper );
+		$helper = new \Aimeos\MW\View\Helper\Date\Standard($view, 'Y-m-d');
+		$view->addHelper('date', $helper);
 
-		$config = new \Aimeos\MW\Config\Decorator\Protect( $config, array( 'admin', 'client/html', 'controller/jsonadm' ) );
-		$helper = new \Aimeos\MW\View\Helper\Config\Standard( $view, $config );
-		$view->addHelper( 'config', $helper );
+		$config = new \Aimeos\MW\Config\Decorator\Protect($config, array('admin', 'client/html', 'controller/jsonadm'));
+		$helper = new \Aimeos\MW\View\Helper\Config\Standard($view, $config);
+		$view->addHelper('config', $helper);
 
-		$helper = new \Aimeos\MW\View\Helper\Csrf\Standard( $view, '_csrf_token', '_csrf_value' );
-		$view->addHelper( 'csrf', $helper );
+		$helper = new \Aimeos\MW\View\Helper\Csrf\Standard($view, '_csrf_token', '_csrf_value');
+		$view->addHelper('csrf', $helper);
 
-		$fcn = function() { return array( 'admin' ); };
-		$helper = new \Aimeos\MW\View\Helper\Access\Standard( $view, $fcn );
-		$view->addHelper( 'access', $helper );
+		$fcn = function() { return array('admin'); };
+		$helper = new \Aimeos\MW\View\Helper\Access\Standard($view, $fcn);
+		$view->addHelper('access', $helper);
 
 		return $view;
 	}
@@ -69,19 +69,19 @@ 
 
 	public static function getTemplatePaths()
 	{
-		return self::getAimeos()->getCustomPaths( 'admin/jqadm/templates' );
+		return self::getAimeos()->getCustomPaths('admin/jqadm/templates');
 	}
 
 
 	private static function getAimeos()
 	{
-		if( !isset( self::$aimeos ) )
+		if (!isset(self::$aimeos))
 		{
 			require_once 'Bootstrap.php';
-			spl_autoload_register( 'Aimeos\\Bootstrap::autoload' );
+			spl_autoload_register('Aimeos\\Bootstrap::autoload');
 
-			$extdir = dirname( dirname( dirname( dirname( __FILE__ ) ) ) );
-			self::$aimeos = new \Aimeos\Bootstrap( array( $extdir ), false );
+			$extdir = dirname(dirname(dirname(dirname(__FILE__))));
+			self::$aimeos = new \Aimeos\Bootstrap(array($extdir), false);
 		}
 
 		return self::$aimeos;
@@ -91,7 +91,7 @@ 
 	/**
 	 * @param string $site
 	 */
-	private static function createContext( $site )
+	private static function createContext($site)
 	{
 		$ctx = new \Aimeos\MShop\Context\Item\Standard();
 		$aimeos = self::getAimeos();
@@ -100,44 +100,44 @@ 
 		$paths = $aimeos->getConfigPaths();
 		$paths[] = __DIR__ . DIRECTORY_SEPARATOR . 'config';
 		$file = __DIR__ . DIRECTORY_SEPARATOR . 'confdoc.ser';
-		$local = array( 'resource' => array( 'fs' => array( 'adapter' => 'Standard', 'basedir' => __DIR__ . '/tmp' ) ) );
+		$local = array('resource' => array('fs' => array('adapter' => 'Standard', 'basedir' => __DIR__ . '/tmp')));
 
-		$conf = new \Aimeos\MW\Config\PHPArray( $local, $paths );
-		$conf = new \Aimeos\MW\Config\Decorator\Memory( $conf );
-		$conf = new \Aimeos\MW\Config\Decorator\Documentor( $conf, $file );
-		$ctx->setConfig( $conf );
+		$conf = new \Aimeos\MW\Config\PHPArray($local, $paths);
+		$conf = new \Aimeos\MW\Config\Decorator\Memory($conf);
+		$conf = new \Aimeos\MW\Config\Decorator\Documentor($conf, $file);
+		$ctx->setConfig($conf);
 
 
-		$dbm = new \Aimeos\MW\DB\Manager\PDO( $conf );
-		$ctx->setDatabaseManager( $dbm );
+		$dbm = new \Aimeos\MW\DB\Manager\PDO($conf);
+		$ctx->setDatabaseManager($dbm);
 
 
-		$fs = new \Aimeos\MW\Filesystem\Manager\Standard( $conf );
-		$ctx->setFilesystemManager( $fs );
+		$fs = new \Aimeos\MW\Filesystem\Manager\Standard($conf);
+		$ctx->setFilesystemManager($fs);
 
 
-		$logger = new \Aimeos\MW\Logger\File( $site . '.log', \Aimeos\MW\Logger\Base::DEBUG );
-		$ctx->setLogger( $logger );
+		$logger = new \Aimeos\MW\Logger\File($site . '.log', \Aimeos\MW\Logger\Base::DEBUG);
+		$ctx->setLogger($logger);
 
 
 		$cache = new \Aimeos\MW\Cache\None();
-		$ctx->setCache( $cache );
+		$ctx->setCache($cache);
 
 
-		$i18n = new \Aimeos\MW\Translation\None( 'de' );
-		$ctx->setI18n( array( 'de' => $i18n ) );
+		$i18n = new \Aimeos\MW\Translation\None('de');
+		$ctx->setI18n(array('de' => $i18n));
 
 
 		$session = new \Aimeos\MW\Session\None();
-		$ctx->setSession( $session );
+		$ctx->setSession($session);
 
 
-		$localeManager = \Aimeos\MShop::create( $ctx, 'locale' );
-		$locale = $localeManager->bootstrap( $site, '', '', false );
-		$ctx->setLocale( $locale );
+		$localeManager = \Aimeos\MShop::create($ctx, 'locale');
+		$locale = $localeManager->bootstrap($site, '', '', false);
+		$ctx->setLocale($locale);
 
 
-		$ctx->setEditor( 'ai-admin-jqadm:lib/custom' );
+		$ctx->setEditor('ai-admin-jqadm:lib/custom');
 
 		return $ctx;
 	}

admin/jqadm/src/Admin/JQAdm.php

Spacing +16 added lines, -16 removed lines

@@ -29,40 +29,40 @@
 	 * @return \Aimeos\Admin\JQAdm\Iface admin client implementing \Aimeos\Admin\JQAdm\Iface
 	 * @throws \Aimeos\Admin\JQAdm\Exception If requested client implementation couldn't be found or initialisation fails
 	 */
-	public static function create( \Aimeos\MShop\Context\Item\Iface $context, \Aimeos\Bootstrap $aimeos, $path, $name = null )
+	public static function create(\Aimeos\MShop\Context\Item\Iface $context, \Aimeos\Bootstrap $aimeos, $path, $name = null)
 	{
-		if( empty( $path ) ) {
-			throw new \Aimeos\Admin\JQAdm\Exception( sprintf( 'Admin JQAdm type is empty' ) );
+		if (empty($path)) {
+			throw new \Aimeos\Admin\JQAdm\Exception(sprintf('Admin JQAdm type is empty'));
 		}
 
 		$view = $context->getView();
 		$config = $context->getConfig();
-		$parts = explode( '/', $path );
+		$parts = explode('/', $path);
 
-		foreach( $parts as $idx => $part )
+		foreach ($parts as $idx => $part)
 		{
-			if( ctype_alnum( $part ) === false ) {
-				throw new \Aimeos\Admin\JQAdm\Exception( sprintf( 'Invalid characters in client name "%1$s"', $path ) );
+			if (ctype_alnum($part) === false) {
+				throw new \Aimeos\Admin\JQAdm\Exception(sprintf('Invalid characters in client name "%1$s"', $path));
 			}
 
-			$parts[$idx] = ucwords( $part );
+			$parts[$idx] = ucwords($part);
 		}
 
-		if( $view->access( $config->get( 'admin/jqadm/resource/' . $path . '/groups', [] ) ) !== true ) {
-			throw new \Aimeos\Admin\JQAdm\Exception( sprintf( 'Not allowed to access JQAdm "%1$s" client', $path ) );
+		if ($view->access($config->get('admin/jqadm/resource/' . $path . '/groups', [])) !== true) {
+			throw new \Aimeos\Admin\JQAdm\Exception(sprintf('Not allowed to access JQAdm "%1$s" client', $path));
 		}
 
-		$factory = '\\Aimeos\\Admin\\JQAdm\\' . implode( '\\', $parts ) . '\\Factory';
+		$factory = '\\Aimeos\\Admin\\JQAdm\\' . implode('\\', $parts) . '\\Factory';
 
-		if( class_exists( $factory ) === false ) {
-			throw new \Aimeos\Admin\JQAdm\Exception( sprintf( 'Class "%1$s" not available', $factory ) );
+		if (class_exists($factory) === false) {
+			throw new \Aimeos\Admin\JQAdm\Exception(sprintf('Class "%1$s" not available', $factory));
 		}
 
-		if( ( $client = @call_user_func_array( [$factory, 'create'], [$context, $name] ) ) === false ) {
-			throw new \Aimeos\Admin\JQAdm\Exception( sprintf( 'Invalid factory "%1$s"', $factory ) );
+		if (($client = @call_user_func_array([$factory, 'create'], [$context, $name])) === false) {
+			throw new \Aimeos\Admin\JQAdm\Exception(sprintf('Invalid factory "%1$s"', $factory));
 		}
 
-		return $client->setAimeos( $aimeos )->setView( $view );;
+		return $client->setAimeos($aimeos)->setView($view); ;
 	}
 
 }
\ No newline at end of file

admin/jqadm/templates/subscription/list-standard.php

Spacing +182 added lines, -182 removed lines

@@ -8,35 +8,35 @@ 
 $enc = $this->encoder();
 
 
-$target = $this->config( 'admin/jqadm/url/search/target' );
-$controller = $this->config( 'admin/jqadm/url/search/controller', 'Jqadm' );
-$action = $this->config( 'admin/jqadm/url/search/action', 'search' );
-$config = $this->config( 'admin/jqadm/url/search/config', [] );
+$target = $this->config('admin/jqadm/url/search/target');
+$controller = $this->config('admin/jqadm/url/search/controller', 'Jqadm');
+$action = $this->config('admin/jqadm/url/search/action', 'search');
+$config = $this->config('admin/jqadm/url/search/config', []);
 
-$newTarget = $this->config( 'admin/jqadm/url/create/target' );
-$newCntl = $this->config( 'admin/jqadm/url/create/controller', 'Jqadm' );
-$newAction = $this->config( 'admin/jqadm/url/create/action', 'create' );
-$newConfig = $this->config( 'admin/jqadm/url/create/config', [] );
+$newTarget = $this->config('admin/jqadm/url/create/target');
+$newCntl = $this->config('admin/jqadm/url/create/controller', 'Jqadm');
+$newAction = $this->config('admin/jqadm/url/create/action', 'create');
+$newConfig = $this->config('admin/jqadm/url/create/config', []);
 
-$getTarget = $this->config( 'admin/jqadm/url/get/target' );
-$getCntl = $this->config( 'admin/jqadm/url/get/controller', 'Jqadm' );
-$getAction = $this->config( 'admin/jqadm/url/get/action', 'get' );
-$getConfig = $this->config( 'admin/jqadm/url/get/config', [] );
+$getTarget = $this->config('admin/jqadm/url/get/target');
+$getCntl = $this->config('admin/jqadm/url/get/controller', 'Jqadm');
+$getAction = $this->config('admin/jqadm/url/get/action', 'get');
+$getConfig = $this->config('admin/jqadm/url/get/config', []);
 
-$copyTarget = $this->config( 'admin/jqadm/url/copy/target' );
-$copyCntl = $this->config( 'admin/jqadm/url/copy/controller', 'Jqadm' );
-$copyAction = $this->config( 'admin/jqadm/url/copy/action', 'copy' );
-$copyConfig = $this->config( 'admin/jqadm/url/copy/config', [] );
+$copyTarget = $this->config('admin/jqadm/url/copy/target');
+$copyCntl = $this->config('admin/jqadm/url/copy/controller', 'Jqadm');
+$copyAction = $this->config('admin/jqadm/url/copy/action', 'copy');
+$copyConfig = $this->config('admin/jqadm/url/copy/config', []);
 
-$delTarget = $this->config( 'admin/jqadm/url/delete/target' );
-$delCntl = $this->config( 'admin/jqadm/url/delete/controller', 'Jqadm' );
-$delAction = $this->config( 'admin/jqadm/url/delete/action', 'delete' );
-$delConfig = $this->config( 'admin/jqadm/url/delete/config', [] );
+$delTarget = $this->config('admin/jqadm/url/delete/target');
+$delCntl = $this->config('admin/jqadm/url/delete/controller', 'Jqadm');
+$delAction = $this->config('admin/jqadm/url/delete/action', 'delete');
+$delConfig = $this->config('admin/jqadm/url/delete/config', []);
 
-$expTarget = $this->config( 'admin/jqadm/url/export/target' );
-$expCntl = $this->config( 'admin/jqadm/url/export/controller', 'Jqadm' );
-$expAction = $this->config( 'admin/jqadm/url/export/action', 'export' );
-$expConfig = $this->config( 'admin/jqadm/url/export/config', [] );
+$expTarget = $this->config('admin/jqadm/url/export/target');
+$expCntl = $this->config('admin/jqadm/url/export/controller', 'Jqadm');
+$expAction = $this->config('admin/jqadm/url/export/action', 'export');
+$expConfig = $this->config('admin/jqadm/url/export/config', []);
 
 
 /** admin/jqadm/subscription/fields
@@ -54,90 +54,90 @@ 
  * @category Developer
  */
 $default = ['subscription.status', 'subscription.datenext', 'subscription.dateend', 'subscription.interval'];
-$default = $this->config( 'admin/jqadm/subscription/fields', $default );
-$fields = $this->session( 'aimeos/admin/jqadm/subscription/fields', $default );
+$default = $this->config('admin/jqadm/subscription/fields', $default);
+$fields = $this->session('aimeos/admin/jqadm/subscription/fields', $default);
 
-$searchParams = $params = $this->get( 'pageParams', [] );
+$searchParams = $params = $this->get('pageParams', []);
 $searchParams['page']['start'] = 0;
 
-$baseItems = $this->get( 'baseItems', [] );
+$baseItems = $this->get('baseItems', []);
 
 $columnList = [
-	'subscription.id' => $this->translate( 'admin', 'ID' ),
-	'subscription.status' => $this->translate( 'admin', 'Status' ),
-	'subscription.interval' => $this->translate( 'admin', 'Interval' ),
-	'subscription.datenext' => $this->translate( 'admin', 'Next date' ),
-	'subscription.dateend' => $this->translate( 'admin', 'End date' ),
-	'subscription.reason' => $this->translate( 'admin', 'Reason' ),
-	'subscription.ctime' => $this->translate( 'admin', 'Created' ),
-	'subscription.mtime' => $this->translate( 'admin', 'Modified' ),
-	'subscription.editor' => $this->translate( 'admin', 'Editor' ),
-	'order.base.customerid' => $this->translate( 'admin', 'Customer ID' ),
-	'order.base.sitecode' => $this->translate( 'admin', 'Site' ),
-	'order.base.languageid' => $this->translate( 'admin', 'Language' ),
-	'order.base.currencyid' => $this->translate( 'admin', 'Currency' ),
-	'order.base.taxflag' => $this->translate( 'admin', 'Incl. tax' ),
-	'order.base.comment' => $this->translate( 'admin', 'Comment' ),
-	'order.base.address.salutation' => $this->translate( 'admin', 'Salutation' ),
-	'order.base.address.company' => $this->translate( 'admin', 'Company' ),
-	'order.base.address.vatid' => $this->translate( 'admin', 'VAT ID' ),
-	'order.base.address.title' => $this->translate( 'admin', 'Title' ),
-	'order.base.address.firstname' => $this->translate( 'admin', 'First name' ),
-	'order.base.address.lastname' => $this->translate( 'admin', 'Last name' ),
-	'order.base.address.address1' => $this->translate( 'admin', 'Street' ),
-	'order.base.address.address2' => $this->translate( 'admin', 'House number' ),
-	'order.base.address.address3' => $this->translate( 'admin', 'Floor' ),
-	'order.base.address.postal' => $this->translate( 'admin', 'Zip code' ),
-	'order.base.address.city' => $this->translate( 'admin', 'City' ),
-	'order.base.address.state' => $this->translate( 'admin', 'State' ),
-	'order.base.address.countryid' => $this->translate( 'admin', 'Country' ),
-	'order.base.address.languageid' => $this->translate( 'admin', 'Language' ),
-	'order.base.address.telephone' => $this->translate( 'admin', 'Telephone' ),
-	'order.base.address.telefax' => $this->translate( 'admin', 'Facsimile' ),
-	'order.base.address.email' => $this->translate( 'admin', 'E-Mail' ),
-	'order.base.address.website' => $this->translate( 'admin', 'Web site' ),
+	'subscription.id' => $this->translate('admin', 'ID'),
+	'subscription.status' => $this->translate('admin', 'Status'),
+	'subscription.interval' => $this->translate('admin', 'Interval'),
+	'subscription.datenext' => $this->translate('admin', 'Next date'),
+	'subscription.dateend' => $this->translate('admin', 'End date'),
+	'subscription.reason' => $this->translate('admin', 'Reason'),
+	'subscription.ctime' => $this->translate('admin', 'Created'),
+	'subscription.mtime' => $this->translate('admin', 'Modified'),
+	'subscription.editor' => $this->translate('admin', 'Editor'),
+	'order.base.customerid' => $this->translate('admin', 'Customer ID'),
+	'order.base.sitecode' => $this->translate('admin', 'Site'),
+	'order.base.languageid' => $this->translate('admin', 'Language'),
+	'order.base.currencyid' => $this->translate('admin', 'Currency'),
+	'order.base.taxflag' => $this->translate('admin', 'Incl. tax'),
+	'order.base.comment' => $this->translate('admin', 'Comment'),
+	'order.base.address.salutation' => $this->translate('admin', 'Salutation'),
+	'order.base.address.company' => $this->translate('admin', 'Company'),
+	'order.base.address.vatid' => $this->translate('admin', 'VAT ID'),
+	'order.base.address.title' => $this->translate('admin', 'Title'),
+	'order.base.address.firstname' => $this->translate('admin', 'First name'),
+	'order.base.address.lastname' => $this->translate('admin', 'Last name'),
+	'order.base.address.address1' => $this->translate('admin', 'Street'),
+	'order.base.address.address2' => $this->translate('admin', 'House number'),
+	'order.base.address.address3' => $this->translate('admin', 'Floor'),
+	'order.base.address.postal' => $this->translate('admin', 'Zip code'),
+	'order.base.address.city' => $this->translate('admin', 'City'),
+	'order.base.address.state' => $this->translate('admin', 'State'),
+	'order.base.address.countryid' => $this->translate('admin', 'Country'),
+	'order.base.address.languageid' => $this->translate('admin', 'Language'),
+	'order.base.address.telephone' => $this->translate('admin', 'Telephone'),
+	'order.base.address.telefax' => $this->translate('admin', 'Facsimile'),
+	'order.base.address.email' => $this->translate('admin', 'E-Mail'),
+	'order.base.address.website' => $this->translate('admin', 'Web site'),
 ];
 
 $deliveryStatusList = [
-	'-1' => $this->translate( 'mshop/code', 'stat:-1' ),
-	'0' => $this->translate( 'mshop/code', 'stat:0' ),
-	'1' => $this->translate( 'mshop/code', 'stat:1' ),
-	'2' => $this->translate( 'mshop/code', 'stat:2' ),
-	'3' => $this->translate( 'mshop/code', 'stat:3' ),
-	'4' => $this->translate( 'mshop/code', 'stat:4' ),
-	'5' => $this->translate( 'mshop/code', 'stat:5' ),
-	'6' => $this->translate( 'mshop/code', 'stat:6' ),
-	'7' => $this->translate( 'mshop/code', 'stat:7' ),
+	'-1' => $this->translate('mshop/code', 'stat:-1'),
+	'0' => $this->translate('mshop/code', 'stat:0'),
+	'1' => $this->translate('mshop/code', 'stat:1'),
+	'2' => $this->translate('mshop/code', 'stat:2'),
+	'3' => $this->translate('mshop/code', 'stat:3'),
+	'4' => $this->translate('mshop/code', 'stat:4'),
+	'5' => $this->translate('mshop/code', 'stat:5'),
+	'6' => $this->translate('mshop/code', 'stat:6'),
+	'7' => $this->translate('mshop/code', 'stat:7'),
 ];
 
 $statusList = [
-	0 => $this->translate( 'admin', 'no' ),
-	1 => $this->translate( 'admin', 'yes' ),
+	0 => $this->translate('admin', 'no'),
+	1 => $this->translate('admin', 'yes'),
 ];
 
 $reasonList = [
 	null => '',
-	-1 => $this->translate( 'mshop/code', 'reason:-1' ),
-	0 => $this->translate( 'mshop/code', 'reason:0' ),
-	1 => $this->translate( 'mshop/code', 'reason:1' ),
+	-1 => $this->translate('mshop/code', 'reason:-1'),
+	0 => $this->translate('mshop/code', 'reason:0'),
+	1 => $this->translate('mshop/code', 'reason:1'),
 ];
 
 
 ?>
-<?php $this->block()->start( 'jqadm_content' ); ?>
+<?php $this->block()->start('jqadm_content'); ?>
 
 <nav class="main-navbar">
 
 	<span class="navbar-brand">
-		<?= $enc->html( $this->translate( 'admin', 'Subscription' ) ); ?>
-		<span class="navbar-secondary">(<?= $enc->html( $this->site()->label() ); ?>)</span>
+		<?= $enc->html($this->translate('admin', 'Subscription')); ?>
+		<span class="navbar-secondary">(<?= $enc->html($this->site()->label()); ?>)</span>
 	</span>
 
 	<?= $this->partial(
-		$this->config( 'admin/jqadm/partial/navsearch', 'common/partials/navsearch-standard' ), [
-			'filter' => $this->session( 'aimeos/admin/jqadm/subscription/filter', [] ),
-			'filterAttributes' => $this->get( 'filterAttributes', [] ),
-			'filterOperators' => $this->get( 'filterOperators', [] ),
+		$this->config('admin/jqadm/partial/navsearch', 'common/partials/navsearch-standard'), [
+			'filter' => $this->session('aimeos/admin/jqadm/subscription/filter', []),
+			'filterAttributes' => $this->get('filterAttributes', []),
+			'filterOperators' => $this->get('filterOperators', []),
 			'params' => $params,
 		]
 	); ?>
@@ -145,13 +145,13 @@ 
 
 
 <?= $this->partial(
-		$this->config( 'admin/jqadm/partial/pagination', 'common/partials/pagination-standard' ),
-		['pageParams' => $params, 'pos' => 'top', 'total' => $this->get( 'total' ),
-		'page' => $this->session( 'aimeos/admin/jqadm/subscription/page', [] )]
+		$this->config('admin/jqadm/partial/pagination', 'common/partials/pagination-standard'),
+		['pageParams' => $params, 'pos' => 'top', 'total' => $this->get('total'),
+		'page' => $this->session('aimeos/admin/jqadm/subscription/page', [])]
 	);
 ?>
 
-<form class="list list-subscription" method="POST" action="<?= $enc->attr( $this->url( $target, $controller, $action, $searchParams, [], $config ) ); ?>">
+<form class="list list-subscription" method="POST" action="<?= $enc->attr($this->url($target, $controller, $action, $searchParams, [], $config)); ?>">
 	<?= $this->csrf()->formfield(); ?>
 
 	<table class="list-items table table-hover table-striped">
@@ -159,20 +159,20 @@ 
 			<tr>
 
 				<?= $this->partial(
-						$this->config( 'admin/jqadm/partial/listhead', 'common/partials/listhead-standard' ),
-						['fields' => $fields, 'params' => $params, 'data' => $columnList, 'sort' => $this->session( 'aimeos/admin/jqadm/subscription/sort' )]
+						$this->config('admin/jqadm/partial/listhead', 'common/partials/listhead-standard'),
+						['fields' => $fields, 'params' => $params, 'data' => $columnList, 'sort' => $this->session('aimeos/admin/jqadm/subscription/sort')]
 					);
 				?>
 
 				<th class="actions">
 					<a class="btn fa act-download" tabindex="1"
-						href="<?= $enc->attr( $this->url( $expTarget, $expCntl, $expAction, $params, [], $expConfig ) ); ?>"
-						title="<?= $enc->attr( $this->translate( 'admin', 'Download') ); ?>"
-						aria-label="<?= $enc->attr( $this->translate( 'admin', 'Download' ) ); ?>">
+						href="<?= $enc->attr($this->url($expTarget, $expCntl, $expAction, $params, [], $expConfig)); ?>"
+						title="<?= $enc->attr($this->translate('admin', 'Download')); ?>"
+						aria-label="<?= $enc->attr($this->translate('admin', 'Download')); ?>">
 					</a>
 
 					<?= $this->partial(
-							$this->config( 'admin/jqadm/partial/columns', 'common/partials/columns-standard' ),
+							$this->config('admin/jqadm/partial/columns', 'common/partials/columns-standard'),
 							['fields' => $fields, 'data' => $columnList]
 						);
 					?>
@@ -182,15 +182,15 @@ 
 		<tbody>
 
 			<?= $this->partial(
-				$this->config( 'admin/jqadm/partial/listsearch', 'common/partials/listsearch-standard' ), [
-					'fields' => $fields, 'filter' => $this->session( 'aimeos/admin/jqadm/subscription/filter', [] ),
+				$this->config('admin/jqadm/partial/listsearch', 'common/partials/listsearch-standard'), [
+					'fields' => $fields, 'filter' => $this->session('aimeos/admin/jqadm/subscription/filter', []),
 					'data' => [
 						'subscription.id' => ['op' => '=='],
 						'subscription.status' => ['op' => '==', 'type' => 'select', 'val' => [
-							'1' => $this->translate( 'mshop/code', 'status:1' ),
-							'0' => $this->translate( 'mshop/code', 'status:0' ),
-							'-1' => $this->translate( 'mshop/code', 'status:-1' ),
-							'-2' => $this->translate( 'mshop/code', 'status:-2' ),
+							'1' => $this->translate('mshop/code', 'status:1'),
+							'0' => $this->translate('mshop/code', 'status:0'),
+							'-1' => $this->translate('mshop/code', 'status:-1'),
+							'-2' => $this->translate('mshop/code', 'status:-2'),
 						]],
 						'subscription.interval' => ['op' => '=~', 'type' => 'string'],
 						'subscription.datenext' => ['op' => '>=', 'type' => 'date'],
@@ -229,129 +229,129 @@ 
 				] );
 			?>
 
-			<?php foreach( $this->get( 'items', [] ) as $id => $item ) : ?>
-				<?php $url = $enc->attr( $this->url( $getTarget, $getCntl, $getAction, ['id' => $id] + $params, [], $getConfig ) ); ?>
-				<tr class="<?= $this->site()->readonly( $item->getSiteId() ); ?>">
-					<?php if( in_array( 'subscription.id', $fields ) ) : ?>
-						<td class="subscription-id"><a class="items-field" href="<?= $url; ?>"><?= $enc->html( $item->getId() ); ?></a></td>
+			<?php foreach ($this->get('items', []) as $id => $item) : ?>
+				<?php $url = $enc->attr($this->url($getTarget, $getCntl, $getAction, ['id' => $id] + $params, [], $getConfig)); ?>
+				<tr class="<?= $this->site()->readonly($item->getSiteId()); ?>">
+					<?php if (in_array('subscription.id', $fields)) : ?>
+						<td class="subscription-id"><a class="items-field" href="<?= $url; ?>"><?= $enc->html($item->getId()); ?></a></td>
 					<?php endif; ?>
-					<?php if( in_array( 'subscription.status', $fields ) ) : ?>
-						<td class="subscription-status"><a class="items-field" href="<?= $url; ?>"><div class="fa status-<?= $enc->attr( $item->getStatus() ); ?>"></div></a></td>
+					<?php if (in_array('subscription.status', $fields)) : ?>
+						<td class="subscription-status"><a class="items-field" href="<?= $url; ?>"><div class="fa status-<?= $enc->attr($item->getStatus()); ?>"></div></a></td>
 					<?php endif; ?>
-					<?php if( in_array( 'subscription.interval', $fields ) ) : ?>
-						<td class="subscription-interval"><a class="items-field" href="<?= $url; ?>"><?= $enc->html( $item->getInterval() ); ?></a></td>
+					<?php if (in_array('subscription.interval', $fields)) : ?>
+						<td class="subscription-interval"><a class="items-field" href="<?= $url; ?>"><?= $enc->html($item->getInterval()); ?></a></td>
 					<?php endif; ?>
-					<?php if( in_array( 'subscription.datenext', $fields ) ) : ?>
-						<td class="subscription-datenext"><a class="items-field" href="<?= $url; ?>"><?= $enc->html( $item->getDateNext() ); ?></a></td>
+					<?php if (in_array('subscription.datenext', $fields)) : ?>
+						<td class="subscription-datenext"><a class="items-field" href="<?= $url; ?>"><?= $enc->html($item->getDateNext()); ?></a></td>
 					<?php endif; ?>
-					<?php if( in_array( 'subscription.dateend', $fields ) ) : ?>
-						<td class="subscription-dateend"><a class="items-field" href="<?= $url; ?>"><?= $enc->html( $item->getDateEnd() ); ?></a></td>
+					<?php if (in_array('subscription.dateend', $fields)) : ?>
+						<td class="subscription-dateend"><a class="items-field" href="<?= $url; ?>"><?= $enc->html($item->getDateEnd()); ?></a></td>
 					<?php endif; ?>
-					<?php if( in_array( 'subscription.reason', $fields ) ) : ?>
-						<td class="subscription-reason"><a class="items-field" href="<?= $url; ?>"><?= $enc->html( $reasonList[$item->getReason()] ); ?></td>
+					<?php if (in_array('subscription.reason', $fields)) : ?>
+						<td class="subscription-reason"><a class="items-field" href="<?= $url; ?>"><?= $enc->html($reasonList[$item->getReason()]); ?></td>
 					<?php endif; ?>
-					<?php if( in_array( 'subscription.ctime', $fields ) ) : ?>
-						<td class="subscription-ctime"><a class="items-field" href="<?= $url; ?>"><?= $enc->html( $item->getTimeCreated() ); ?></a></td>
+					<?php if (in_array('subscription.ctime', $fields)) : ?>
+						<td class="subscription-ctime"><a class="items-field" href="<?= $url; ?>"><?= $enc->html($item->getTimeCreated()); ?></a></td>
 					<?php endif; ?>
-					<?php if( in_array( 'subscription.mtime', $fields ) ) : ?>
-						<td class="subscription-mtime"><a class="items-field" href="<?= $url; ?>"><?= $enc->html( $item->getTimeModified() ); ?></a></td>
+					<?php if (in_array('subscription.mtime', $fields)) : ?>
+						<td class="subscription-mtime"><a class="items-field" href="<?= $url; ?>"><?= $enc->html($item->getTimeModified()); ?></a></td>
 					<?php endif; ?>
-					<?php if( in_array( 'subscription.editor', $fields ) ) : ?>
-						<td class="subscription-editor"><a class="items-field" href="<?= $url; ?>"><?= $enc->html( $item->getEditor() ); ?></a></td>
+					<?php if (in_array('subscription.editor', $fields)) : ?>
+						<td class="subscription-editor"><a class="items-field" href="<?= $url; ?>"><?= $enc->html($item->getEditor()); ?></a></td>
 					<?php endif; ?>
 
-					<?php $baseItem = ( isset( $baseItems[$item->getOrderBaseId()] ) ? $baseItems[$item->getOrderBaseId()] : null ); ?>
+					<?php $baseItem = (isset($baseItems[$item->getOrderBaseId()]) ? $baseItems[$item->getOrderBaseId()] : null); ?>
 
-					<?php if( in_array( 'order.base.customerid', $fields ) ) : ?>
-						<td class="order-base-customerid"><a class="items-field" href="<?= $url; ?>"><?= $baseItem ? $enc->html( $baseItem->getCustomerId() ) : ''; ?></a></td>
+					<?php if (in_array('order.base.customerid', $fields)) : ?>
+						<td class="order-base-customerid"><a class="items-field" href="<?= $url; ?>"><?= $baseItem ? $enc->html($baseItem->getCustomerId()) : ''; ?></a></td>
 					<?php endif; ?>
-					<?php if( in_array( 'order.base.sitecode', $fields ) ) : ?>
-						<td class="order-base-sitecode"><a class="items-field" href="<?= $url; ?>"><?= $baseItem ? $enc->html( $baseItem->getSiteCode() ) : ''; ?></a></td>
+					<?php if (in_array('order.base.sitecode', $fields)) : ?>
+						<td class="order-base-sitecode"><a class="items-field" href="<?= $url; ?>"><?= $baseItem ? $enc->html($baseItem->getSiteCode()) : ''; ?></a></td>
 					<?php endif; ?>
-					<?php if( in_array( 'order.base.languageid', $fields ) ) : ?>
-						<td class="order-base-languageid"><a class="items-field" href="<?= $url; ?>"><?= $baseItem ? $enc->html( $baseItem->getLocale()->getLanguageId() ) : ''; ?></a></td>
+					<?php if (in_array('order.base.languageid', $fields)) : ?>
+						<td class="order-base-languageid"><a class="items-field" href="<?= $url; ?>"><?= $baseItem ? $enc->html($baseItem->getLocale()->getLanguageId()) : ''; ?></a></td>
 					<?php endif; ?>
-					<?php if( in_array( 'order.base.currencyid', $fields ) ) : ?>
-						<td class="order-base-currencyid"><a class="items-field" href="<?= $url; ?>"><?= $baseItem ? $enc->html( $baseItem->getLocale()->getCurrencyId() ) : ''; ?></a></td>
+					<?php if (in_array('order.base.currencyid', $fields)) : ?>
+						<td class="order-base-currencyid"><a class="items-field" href="<?= $url; ?>"><?= $baseItem ? $enc->html($baseItem->getLocale()->getCurrencyId()) : ''; ?></a></td>
 					<?php endif; ?>
-					<?php if( in_array( 'order.base.taxflag', $fields ) ) : ?>
-						<td class="order-base-taxflag"><a class="items-field" href="<?= $url; ?>"><?= $baseItem ? $enc->html( $statusList[$baseItem->getPrice()->getTaxFlag()] ) : ''; ?></a></td>
+					<?php if (in_array('order.base.taxflag', $fields)) : ?>
+						<td class="order-base-taxflag"><a class="items-field" href="<?= $url; ?>"><?= $baseItem ? $enc->html($statusList[$baseItem->getPrice()->getTaxFlag()]) : ''; ?></a></td>
 					<?php endif; ?>
-					<?php if( in_array( 'order.base.comment', $fields ) ) : ?>
-						<td class="order-base-comment"><a class="items-field" href="<?= $url; ?>"><?= $baseItem ? $enc->html( $baseItem->getComment() ) : ''; ?></a></td>
+					<?php if (in_array('order.base.comment', $fields)) : ?>
+						<td class="order-base-comment"><a class="items-field" href="<?= $url; ?>"><?= $baseItem ? $enc->html($baseItem->getComment()) : ''; ?></a></td>
 					<?php endif; ?>
 
 					<?php $addrItem = null;
-						if( $baseItem && ( $addresses = $baseItem->getAddresses() ) && isset( $addresses[\Aimeos\MShop\Order\Item\Base\Address\Base::TYPE_PAYMENT] ) ) {
+						if ($baseItem && ($addresses = $baseItem->getAddresses()) && isset($addresses[\Aimeos\MShop\Order\Item\Base\Address\Base::TYPE_PAYMENT])) {
 							$addrItem = $addresses[\Aimeos\MShop\Order\Item\Base\Address\Base::TYPE_PAYMENT];
 						}
 					?>
 
-					<?php if( in_array( 'order.base.address.salutation', $fields ) ) : ?>
-						<td class="order-base-address-salutation"><a class="items-field" href="<?= $url; ?>"><?= $addrItem ? $enc->html( $addrItem->getSalutation() ) : ''; ?></a></td>
+					<?php if (in_array('order.base.address.salutation', $fields)) : ?>
+						<td class="order-base-address-salutation"><a class="items-field" href="<?= $url; ?>"><?= $addrItem ? $enc->html($addrItem->getSalutation()) : ''; ?></a></td>
 					<?php endif; ?>
-					<?php if( in_array( 'order.base.address.company', $fields ) ) : ?>
-						<td class="order-base-address-company"><a class="items-field" href="<?= $url; ?>"><?= $addrItem ? $enc->html( $addrItem->getCompany() ) : ''; ?></a></td>
+					<?php if (in_array('order.base.address.company', $fields)) : ?>
+						<td class="order-base-address-company"><a class="items-field" href="<?= $url; ?>"><?= $addrItem ? $enc->html($addrItem->getCompany()) : ''; ?></a></td>
 					<?php endif; ?>
-					<?php if( in_array( 'order.base.address.vatid', $fields ) ) : ?>
-						<td class="order-base-address-vatid"><a class="items-field" href="<?= $url; ?>"><?= $addrItem ? $enc->html( $addrItem->getVatID() ) : ''; ?></a></td>
+					<?php if (in_array('order.base.address.vatid', $fields)) : ?>
+						<td class="order-base-address-vatid"><a class="items-field" href="<?= $url; ?>"><?= $addrItem ? $enc->html($addrItem->getVatID()) : ''; ?></a></td>
 					<?php endif; ?>
-					<?php if( in_array( 'order.base.address.title', $fields ) ) : ?>
-						<td class="order-base-address-title"><a class="items-field" href="<?= $url; ?>"><?= $addrItem ? $enc->html( $addrItem->getTitle() ) : ''; ?></a></td>
+					<?php if (in_array('order.base.address.title', $fields)) : ?>
+						<td class="order-base-address-title"><a class="items-field" href="<?= $url; ?>"><?= $addrItem ? $enc->html($addrItem->getTitle()) : ''; ?></a></td>
 					<?php endif; ?>
-					<?php if( in_array( 'order.base.address.firstname', $fields ) ) : ?>
-						<td class="order-base-address-firstname"><a class="items-field" href="<?= $url; ?>"><?= $addrItem ? $enc->html( $addrItem->getFirstname() ) : ''; ?></a></td>
+					<?php if (in_array('order.base.address.firstname', $fields)) : ?>
+						<td class="order-base-address-firstname"><a class="items-field" href="<?= $url; ?>"><?= $addrItem ? $enc->html($addrItem->getFirstname()) : ''; ?></a></td>
 					<?php endif; ?>
-					<?php if( in_array( 'order.base.address.lastname', $fields ) ) : ?>
-						<td class="order-base-address-lastname"><a class="items-field" href="<?= $url; ?>"><?= $addrItem ? $enc->html( $addrItem->getLastname() ) : ''; ?></a></td>
+					<?php if (in_array('order.base.address.lastname', $fields)) : ?>
+						<td class="order-base-address-lastname"><a class="items-field" href="<?= $url; ?>"><?= $addrItem ? $enc->html($addrItem->getLastname()) : ''; ?></a></td>
 					<?php endif; ?>
-					<?php if( in_array( 'order.base.address.address1', $fields ) ) : ?>
-						<td class="order-base-address-address1"><a class="items-field" href="<?= $url; ?>"><?= $addrItem ? $enc->html( $addrItem->getAddress1() ) : ''; ?></a></td>
+					<?php if (in_array('order.base.address.address1', $fields)) : ?>
+						<td class="order-base-address-address1"><a class="items-field" href="<?= $url; ?>"><?= $addrItem ? $enc->html($addrItem->getAddress1()) : ''; ?></a></td>
 					<?php endif; ?>
-					<?php if( in_array( 'order.base.address.address2', $fields ) ) : ?>
-						<td class="order-base-address-address2"><a class="items-field" href="<?= $url; ?>"><?= $addrItem ? $enc->html( $addrItem->getAddress2() ) : ''; ?></a></td>
+					<?php if (in_array('order.base.address.address2', $fields)) : ?>
+						<td class="order-base-address-address2"><a class="items-field" href="<?= $url; ?>"><?= $addrItem ? $enc->html($addrItem->getAddress2()) : ''; ?></a></td>
 					<?php endif; ?>
-					<?php if( in_array( 'order.base.address.address3', $fields ) ) : ?>
-						<td class="order-base-address-address3"><a class="items-field" href="<?= $url; ?>"><?= $addrItem ? $enc->html( $addrItem->getAddress3() ) : ''; ?></a></td>
+					<?php if (in_array('order.base.address.address3', $fields)) : ?>
+						<td class="order-base-address-address3"><a class="items-field" href="<?= $url; ?>"><?= $addrItem ? $enc->html($addrItem->getAddress3()) : ''; ?></a></td>
 					<?php endif; ?>
-					<?php if( in_array( 'order.base.address.postal', $fields ) ) : ?>
-						<td class="order-base-address-postal"><a class="items-field" href="<?= $url; ?>"><?= $addrItem ? $enc->html( $addrItem->getPostal() ) : ''; ?></a></td>
+					<?php if (in_array('order.base.address.postal', $fields)) : ?>
+						<td class="order-base-address-postal"><a class="items-field" href="<?= $url; ?>"><?= $addrItem ? $enc->html($addrItem->getPostal()) : ''; ?></a></td>
 					<?php endif; ?>
-					<?php if( in_array( 'order.base.address.city', $fields ) ) : ?>
-						<td class="order-base-address-city"><a class="items-field" href="<?= $url; ?>"><?= $addrItem ? $enc->html( $addrItem->getCity() ) : ''; ?></a></td>
+					<?php if (in_array('order.base.address.city', $fields)) : ?>
+						<td class="order-base-address-city"><a class="items-field" href="<?= $url; ?>"><?= $addrItem ? $enc->html($addrItem->getCity()) : ''; ?></a></td>
 					<?php endif; ?>
-					<?php if( in_array( 'order.base.address.state', $fields ) ) : ?>
-						<td class="order-base-address-state"><a class="items-field" href="<?= $url; ?>"><?= $addrItem ? $enc->html( $addrItem->getState() ) : ''; ?></a></td>
+					<?php if (in_array('order.base.address.state', $fields)) : ?>
+						<td class="order-base-address-state"><a class="items-field" href="<?= $url; ?>"><?= $addrItem ? $enc->html($addrItem->getState()) : ''; ?></a></td>
 					<?php endif; ?>
-					<?php if( in_array( 'order.base.address.countryid', $fields ) ) : ?>
-						<td class="order-base-address-countryid"><a class="items-field" href="<?= $url; ?>"><?= $addrItem ? $enc->html( $addrItem->getCountryId() ) : ''; ?></a></td>
+					<?php if (in_array('order.base.address.countryid', $fields)) : ?>
+						<td class="order-base-address-countryid"><a class="items-field" href="<?= $url; ?>"><?= $addrItem ? $enc->html($addrItem->getCountryId()) : ''; ?></a></td>
 					<?php endif; ?>
-					<?php if( in_array( 'order.base.address.languageid', $fields ) ) : ?>
-						<td class="order-base-address-languageid"><a class="items-field" href="<?= $url; ?>"><?= $addrItem ? $enc->html( $addrItem->getLanguageId() ) : ''; ?></a></td>
+					<?php if (in_array('order.base.address.languageid', $fields)) : ?>
+						<td class="order-base-address-languageid"><a class="items-field" href="<?= $url; ?>"><?= $addrItem ? $enc->html($addrItem->getLanguageId()) : ''; ?></a></td>
 					<?php endif; ?>
-					<?php if( in_array( 'order.base.address.telephone', $fields ) ) : ?>
-						<td class="order-base-address-telephone"><a class="items-field" href="<?= $url; ?>"><?= $addrItem ? $enc->html( $addrItem->getTelephone() ) : ''; ?></a></td>
+					<?php if (in_array('order.base.address.telephone', $fields)) : ?>
+						<td class="order-base-address-telephone"><a class="items-field" href="<?= $url; ?>"><?= $addrItem ? $enc->html($addrItem->getTelephone()) : ''; ?></a></td>
 					<?php endif; ?>
-					<?php if( in_array( 'order.base.address.telefax', $fields ) ) : ?>
-						<td class="order-base-address-telefax"><a class="items-field" href="<?= $url; ?>"><?= $addrItem ? $enc->html( $addrItem->getTelefax() ) : ''; ?></a></td>
+					<?php if (in_array('order.base.address.telefax', $fields)) : ?>
+						<td class="order-base-address-telefax"><a class="items-field" href="<?= $url; ?>"><?= $addrItem ? $enc->html($addrItem->getTelefax()) : ''; ?></a></td>
 					<?php endif; ?>
-					<?php if( in_array( 'order.base.address.email', $fields ) ) : ?>
-						<td class="order-base-address-email"><a class="items-field" href="<?= $url; ?>"><?= $addrItem ? $enc->html( $addrItem->getEmail() ) : ''; ?></a></td>
+					<?php if (in_array('order.base.address.email', $fields)) : ?>
+						<td class="order-base-address-email"><a class="items-field" href="<?= $url; ?>"><?= $addrItem ? $enc->html($addrItem->getEmail()) : ''; ?></a></td>
 					<?php endif; ?>
-					<?php if( in_array( 'order.base.address.website', $fields ) ) : ?>
-						<td class="order-base-address-website"><a class="items-field" href="<?= $url; ?>"><?= $addrItem ? $enc->html( $addrItem->getWebsite() ) : ''; ?></a></td>
+					<?php if (in_array('order.base.address.website', $fields)) : ?>
+						<td class="order-base-address-website"><a class="items-field" href="<?= $url; ?>"><?= $addrItem ? $enc->html($addrItem->getWebsite()) : ''; ?></a></td>
 					<?php endif; ?>
 
 					<td class="actions">
 						<a class="btn act-copy fa" tabindex="1"
-							href="<?= $enc->attr( $this->url( $copyTarget, $copyCntl, $copyAction, ['id' => $id] + $params, [], $copyConfig ) ); ?>"
-							title="<?= $enc->attr( $this->translate( 'admin', 'Copy this entry') ); ?>"
-							aria-label="<?= $enc->attr( $this->translate( 'admin', 'Copy' ) ); ?>"></a>
-						<?php if( !$this->site()->readonly( $item->getSiteId() ) ) : ?>
+							href="<?= $enc->attr($this->url($copyTarget, $copyCntl, $copyAction, ['id' => $id] + $params, [], $copyConfig)); ?>"
+							title="<?= $enc->attr($this->translate('admin', 'Copy this entry')); ?>"
+							aria-label="<?= $enc->attr($this->translate('admin', 'Copy')); ?>"></a>
+						<?php if (!$this->site()->readonly($item->getSiteId())) : ?>
 							<a class="btn act-delete fa" tabindex="1"
-								href="<?= $enc->attr( $this->url( $delTarget, $delCntl, $delAction, ['resource' => 'subscription', 'id' => $id] + $params, [], $delConfig ) ); ?>"
-								title="<?= $enc->attr( $this->translate( 'admin', 'Delete this entry') ); ?>"
-								aria-label="<?= $enc->attr( $this->translate( 'admin', 'Delete' ) ); ?>"></a>
+								href="<?= $enc->attr($this->url($delTarget, $delCntl, $delAction, ['resource' => 'subscription', 'id' => $id] + $params, [], $delConfig)); ?>"
+								title="<?= $enc->attr($this->translate('admin', 'Delete this entry')); ?>"
+								aria-label="<?= $enc->attr($this->translate('admin', 'Delete')); ?>"></a>
 						<?php endif; ?>
 					</td>
 				</tr>
@@ -359,18 +359,18 @@ 
 		</tbody>
 	</table>
 
-	<?php if( $this->get( 'items', [] ) === [] ) : ?>
-		<div class="noitems"><?= $enc->html( sprintf( $this->translate( 'admin', 'No items found' ) ) ); ?></div>
+	<?php if ($this->get('items', []) === []) : ?>
+		<div class="noitems"><?= $enc->html(sprintf($this->translate('admin', 'No items found'))); ?></div>
 	<?php endif; ?>
 </form>
 
 <?= $this->partial(
-		$this->config( 'admin/jqadm/partial/pagination', 'common/partials/pagination-standard' ),
-		['pageParams' => $params, 'pos' => 'bottom', 'total' => $this->get( 'total' ),
-		'page' => $this->session( 'aimeos/admin/jqadm/subscription/page', [] )]
+		$this->config('admin/jqadm/partial/pagination', 'common/partials/pagination-standard'),
+		['pageParams' => $params, 'pos' => 'bottom', 'total' => $this->get('total'),
+		'page' => $this->session('aimeos/admin/jqadm/subscription/page', [])]
 	);
 ?>
 
 <?php $this->block()->stop(); ?>
 
-<?= $this->render( $this->config( 'admin/jqadm/template/page', 'common/page-standard' ) ); ?>
+<?= $this->render($this->config('admin/jqadm/template/page', 'common/page-standard')); ?>

admin/jqadm/src/Admin/JQAdm/Product/Category/Standard.php

Spacing +78 added lines, -78 removed lines

@@ -10,7 +10,7 @@ 
 
 namespace Aimeos\Admin\JQAdm\Product\Category;
 
-sprintf( 'category' ); // for translation
+sprintf('category'); // for translation
 
 
 /**
@@ -44,15 +44,15 @@ 
 	{
 		$view = $this->getView();
 
-		$view->categoryData = $this->toArray( $view->item, true );
+		$view->categoryData = $this->toArray($view->item, true);
 		$view->categoryListTypes = $this->getListTypes();
 		$view->categoryBody = '';
 
-		foreach( $this->getSubClients() as $client ) {
+		foreach ($this->getSubClients() as $client) {
 			$view->categoryBody .= $client->copy();
 		}
 
-		return $this->render( $view );
+		return $this->render($view);
 	}
 
 
@@ -64,10 +64,10 @@ 
 	public function create()
 	{
 		$view = $this->getView();
-		$data = $view->param( 'category', [] );
+		$data = $view->param('category', []);
 		$siteid = $this->getContext()->getLocale()->getSiteId();
 
-		foreach( $view->value( $data, 'catalog.lists.id', [] ) as $idx => $value ) {
+		foreach ($view->value($data, 'catalog.lists.id', []) as $idx => $value) {
 			$data['catalog.lists.siteid'][$idx] = $siteid;
 		}
 
@@ -75,11 +75,11 @@ 
 		$view->categoryListTypes = $this->getListTypes();
 		$view->categoryBody = '';
 
-		foreach( $this->getSubClients() as $client ) {
+		foreach ($this->getSubClients() as $client) {
 			$view->categoryBody .= $client->create();
 		}
 
-		return $this->render( $view );
+		return $this->render($view);
 	}
 
 
@@ -93,29 +93,29 @@ 
 		$view = $this->getView();
 		$context = $this->getContext();
 
-		$manager = \Aimeos\MShop::create( $context, 'catalog/lists' );
+		$manager = \Aimeos\MShop::create($context, 'catalog/lists');
 
 		$search = $manager->createSearch();
 		$expr = array(
-			$search->compare( '==', 'catalog.lists.refid', $view->param( 'id' ) ),
-			$search->compare( '==', 'catalog.lists.domain', 'product' )
+			$search->compare('==', 'catalog.lists.refid', $view->param('id')),
+			$search->compare('==', 'catalog.lists.domain', 'product')
 		);
-		$search->setConditions( $search->combine( '&&', $expr ) );
-		$search->setSlice( 0, 0x7fffffff );
+		$search->setConditions($search->combine('&&', $expr));
+		$search->setSlice(0, 0x7fffffff);
 
 		$start = 0;
 
 		do
 		{
-			$search->setSlice( $start );
+			$search->setSlice($start);
 
-			$result = $manager->searchItems( $search );
-			$manager->deleteItems( array_keys( $result ) );
+			$result = $manager->searchItems($search);
+			$manager->deleteItems(array_keys($result));
 
-			$count = count( $result );
+			$count = count($result);
 			$start += $count;
 		}
-		while( $count >= $search->getSliceSize() );
+		while ($count >= $search->getSliceSize());
 	}
 
 
@@ -128,15 +128,15 @@ 
 	{
 		$view = $this->getView();
 
-		$view->categoryData = $this->toArray( $view->item );
+		$view->categoryData = $this->toArray($view->item);
 		$view->categoryListTypes = $this->getListTypes();
 		$view->categoryBody = '';
 
-		foreach( $this->getSubClients() as $client ) {
+		foreach ($this->getSubClients() as $client) {
 			$view->categoryBody .= $client->get();
 		}
 
-		return $this->render( $view );
+		return $this->render($view);
 	}
 
 
@@ -148,32 +148,32 @@ 
 		$view = $this->getView();
 		$context = $this->getContext();
 
-		$manager = \Aimeos\MShop::create( $context, 'product/lists' );
+		$manager = \Aimeos\MShop::create($context, 'product/lists');
 		$manager->begin();
 
 		try
 		{
-			$this->fromArray( $view->item, $view->param( 'category', [] ) );
+			$this->fromArray($view->item, $view->param('category', []));
 			$view->categoryBody = '';
 
-			foreach( $this->getSubClients() as $client ) {
+			foreach ($this->getSubClients() as $client) {
 				$view->categoryBody .= $client->save();
 			}
 
 			$manager->commit();
 			return;
 		}
-		catch( \Aimeos\MShop\Exception $e )
+		catch (\Aimeos\MShop\Exception $e)
 		{
-			$error = array( 'product-item-category' => $context->getI18n()->dt( 'mshop', $e->getMessage() ) );
-			$view->errors = $view->get( 'errors', [] ) + $error;
-			$this->logException( $e );
+			$error = array('product-item-category' => $context->getI18n()->dt('mshop', $e->getMessage()));
+			$view->errors = $view->get('errors', []) + $error;
+			$this->logException($e);
 		}
-		catch( \Exception $e )
+		catch (\Exception $e)
 		{
-			$error = array( 'product-item-category' => $e->getMessage() . ', ' . $e->getFile() . ':' . $e->getLine() );
-			$view->errors = $view->get( 'errors', [] ) + $error;
-			$this->logException( $e );
+			$error = array('product-item-category' => $e->getMessage() . ', ' . $e->getFile() . ':' . $e->getLine());
+			$view->errors = $view->get('errors', []) + $error;
+			$this->logException($e);
 		}
 
 		$manager->rollback();
@@ -189,7 +189,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/category/decorators/excludes
 		 * Excludes decorators added by the "common" option from the product JQAdm client
@@ -264,7 +264,7 @@ 
 		 * @see admin/jqadm/product/category/decorators/excludes
 		 * @see admin/jqadm/product/category/decorators/global
 		 */
-		return $this->createSubClient( 'product/category/' . $type, $name );
+		return $this->createSubClient('product/category/' . $type, $name);
 	}
 
 
@@ -308,7 +308,7 @@ 
 		 * @since 2016.01
 		 * @category Developer
 		 */
-		return $this->getContext()->getConfig()->get( 'admin/jqadm/product/category/standard/subparts', [] );
+		return $this->getContext()->getConfig()->get('admin/jqadm/product/category/standard/subparts', []);
 	}
 
 
@@ -318,20 +318,20 @@ 
 	 * @param array $listItems List of items implementing \Aimeos\Common\Item\Lists\Iface
 	 * @return array Associative list of catalog IDs as keys and items implementing \Aimeos\Catalog\Item\Iface as values
 	 */
-	protected function getCatalogItems( array $listItems )
+	protected function getCatalogItems(array $listItems)
 	{
 		$ids = [];
 
-		foreach( $listItems as $listItem ) {
+		foreach ($listItems as $listItem) {
 			$ids[] = $listItem->getParentId();
 		}
 
-		$manager = \Aimeos\MShop::create( $this->getContext(), 'catalog' );
+		$manager = \Aimeos\MShop::create($this->getContext(), 'catalog');
 
 		$search = $manager->createSearch();
-		$search->setConditions( $search->compare( '==', 'catalog.id', $ids ) );
+		$search->setConditions($search->compare('==', 'catalog.id', $ids));
 
-		return $manager->searchItems( $search );
+		return $manager->searchItems($search);
 	}
 
 
@@ -341,18 +341,18 @@ 
 	 * @param string $prodid Unique product ID
 	 * @return array Associative list of category list IDs as keys and list items as values
 	 */
-	protected function getListItems( $prodid )
+	protected function getListItems($prodid)
 	{
-		$manager = \Aimeos\MShop::create( $this->getContext(), 'catalog/lists' );
+		$manager = \Aimeos\MShop::create($this->getContext(), 'catalog/lists');
 
-		$search = $manager->createSearch()->setSlice( 0, 0x7fffffff );
+		$search = $manager->createSearch()->setSlice(0, 0x7fffffff);
 		$expr = array(
-			$search->compare( '==', 'catalog.lists.refid', $prodid ),
-			$search->compare( '==', 'catalog.lists.domain', 'product' ),
+			$search->compare('==', 'catalog.lists.refid', $prodid),
+			$search->compare('==', 'catalog.lists.domain', 'product'),
 		);
-		$search->setConditions( $search->combine( '&&', $expr ) );
+		$search->setConditions($search->combine('&&', $expr));
 
-		return $manager->searchItems( $search );
+		return $manager->searchItems($search);
 	}
 
 
@@ -363,13 +363,13 @@ 
 	 */
 	protected function getListTypes()
 	{
-		$manager = \Aimeos\MShop::create( $this->getContext(), 'catalog/lists/type' );
+		$manager = \Aimeos\MShop::create($this->getContext(), 'catalog/lists/type');
 
-		$search = $manager->createSearch( true )->setSlice( 0, 10000 );
-		$search->setConditions( $search->compare( '==', 'catalog.lists.type.domain', 'product' ) );
-		$search->setSortations( [$search->sort( '+', 'catalog.lists.type.position' )] );
+		$search = $manager->createSearch(true)->setSlice(0, 10000);
+		$search->setConditions($search->compare('==', 'catalog.lists.type.domain', 'product'));
+		$search->setSortations([$search->sort('+', 'catalog.lists.type.position')]);
 
-		return $this->map( $manager->searchItems( $search ) );
+		return $this->map($manager->searchItems($search));
 	}
 
 
@@ -379,37 +379,37 @@ 
 	 * @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)
 	{
-		$manager = \Aimeos\MShop::create( $this->getContext(), 'catalog/lists' );
-		$listIds = (array) $this->getValue( $data, 'catalog.lists.id', [] );
-		$listItems = $map = $this->getListItems( $item->getId() );
+		$manager = \Aimeos\MShop::create($this->getContext(), 'catalog/lists');
+		$listIds = (array) $this->getValue($data, 'catalog.lists.id', []);
+		$listItems = $map = $this->getListItems($item->getId());
 
 
-		foreach( $listIds as $idx => $listid )
+		foreach ($listIds as $idx => $listid)
 		{
-			if( isset( $map[$listid] ) ) {
-				unset( $map[$listid] );
+			if (isset($map[$listid])) {
+				unset($map[$listid]);
 			}
 		}
 
-		$manager->deleteItems( array_keys( $map ) );
+		$manager->deleteItems(array_keys($map));
 
 
-		foreach( $listIds as $idx => $listid )
+		foreach ($listIds as $idx => $listid)
 		{
-			if( isset( $listItems[$listid] ) ) {
+			if (isset($listItems[$listid])) {
 				$litem = $listItems[$listid];
 			} else {
 				$litem = $manager->createItem();
 			}
 
-			$litem->setDomain( 'product' );
-			$litem->setRefId( $item->getId() );
-			$litem->setParentId( $this->getValue( $data, 'catalog.id/' . $idx ) );
-			$litem->setType( $this->getValue( $data, 'catalog.lists.type/' . $idx ) );
+			$litem->setDomain('product');
+			$litem->setRefId($item->getId());
+			$litem->setParentId($this->getValue($data, 'catalog.id/' . $idx));
+			$litem->setType($this->getValue($data, 'catalog.lists.type/' . $idx));
 
-			$manager->saveItem( $litem, false );
+			$manager->saveItem($litem, false);
 		}
 	}
 
@@ -421,34 +421,34 @@ 
 	 * @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();
-		$listItems = $this->getListItems( $item->getId() );
-		$catItems = $this->getCatalogItems( $listItems );
+		$listItems = $this->getListItems($item->getId());
+		$catItems = $this->getCatalogItems($listItems);
 		$data = [];
 
-		foreach( $listItems as $listItem )
+		foreach ($listItems as $listItem)
 		{
 			$catId = $listItem->getParentId();
 
-			if( !isset( $catItems[$catId] ) ) {
+			if (!isset($catItems[$catId])) {
 				continue;
 			}
 
-			$list = $listItem->toArray( true );
+			$list = $listItem->toArray(true);
 
-			if( $copy === true )
+			if ($copy === true)
 			{
 				$list['catalog.lists.siteid'] = $siteId;
 				$list['catalog.lists.id'] = '';
 			}
 
-			foreach( $list as $key => $value ) {
+			foreach ($list as $key => $value) {
 				$data[$key][] = $value;
 			}
 
-			foreach( $catItems[$catId]->toArray( true ) as $key => $value ) {
+			foreach ($catItems[$catId]->toArray(true) as $key => $value) {
 				$data[$key][] = $value;
 			}
 		}
@@ -463,7 +463,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/category/template-item
 		 * Relative path to the HTML body template of the category subpart for products.
@@ -487,6 +487,6 @@ 
 		$tplconf = 'admin/jqadm/product/category/template-item';
 		$default = 'product/item-category-standard';
 
-		return $view->render( $view->config( $tplconf, $default ) );
+		return $view->render($view->config($tplconf, $default));
 	}
 }
\ No newline at end of file