aimeos / ai-admin-jqadm

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

Spacing +102 added lines, -102 removed lines

@@ -10,7 +10,7 @@ 
 
 namespace Aimeos\Admin\JQAdm\Group;
 
-sprintf( 'group' ); // for translation
+sprintf('group'); // 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, 'customer/group' );
-			$view->item = $manager->getItem( $id );
+			$manager = \Aimeos\MShop::create($context, 'customer/group');
+			$view->item = $manager->getItem($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( 'group-item' => $context->getI18n()->dt( 'mshop', $e->getMessage() ) );
-			$view->errors = $view->get( 'errors', [] ) + $error;
-			$this->logException( $e );
+			$error = array('group-item' => $context->getI18n()->dt('mshop', $e->getMessage()));
+			$view->errors = $view->get('errors', []) + $error;
+			$this->logException($e);
 		}
-		catch( \Exception $e )
+		catch (\Exception $e)
 		{
-			$error = array( 'group-item' => $e->getMessage() . ', ' . $e->getFile() . ':' . $e->getLine() );
-			$view->errors = $view->get( 'errors', [] ) + $error;
-			$this->logException( $e );
+			$error = array('group-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, 'customer/group' )->createItem();
+			if (!isset($view->item)) {
+				$view->item = \Aimeos\MShop::create($context, 'customer/group')->createItem();
 			}
 
 			$data['customer.group.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( 'group-item' => $context->getI18n()->dt( 'mshop', $e->getMessage() ) );
-			$view->errors = $view->get( 'errors', [] ) + $error;
-			$this->logException( $e );
+			$error = array('group-item' => $context->getI18n()->dt('mshop', $e->getMessage()));
+			$view->errors = $view->get('errors', []) + $error;
+			$this->logException($e);
 		}
-		catch( \Exception $e )
+		catch (\Exception $e)
 		{
-			$error = array( 'group-item' => $e->getMessage() . ', ' . $e->getFile() . ':' . $e->getLine() );
-			$view->errors = $view->get( 'errors', [] ) + $error;
-			$this->logException( $e );
+			$error = array('group-item' => $e->getMessage() . ', ' . $e->getFile() . ':' . $e->getLine());
+			$view->errors = $view->get('errors', []) + $error;
+			$this->logException($e);
 		}
 
-		return $this->render( $view );
+		return $this->render($view);
 	}
 
 
@@ -126,38 +126,38 @@ 
 		$view = $this->getView();
 		$context = $this->getContext();
 
-		$manager = \Aimeos\MShop::create( $context, 'customer/group' );
+		$manager = \Aimeos\MShop::create($context, 'customer/group');
 		$manager->begin();
 
 		try
 		{
-			if( ( $id = $view->param( 'id' ) ) === null ) {
-				throw new \Aimeos\Admin\JQAdm\Exception( sprintf( 'Required parameter "%1$s" is missing', 'id' ) );
+			if (($id = $view->param('id')) === null) {
+				throw new \Aimeos\Admin\JQAdm\Exception(sprintf('Required parameter "%1$s" is missing', 'id'));
 			}
 
-			$view->item = $manager->getItem( $id );
+			$view->item = $manager->getItem($id);
 
-			foreach( $this->getSubClients() as $client ) {
+			foreach ($this->getSubClients() as $client) {
 				$client->delete();
 			}
 
-			$manager->deleteItem( $id );
+			$manager->deleteItem($id);
 			$manager->commit();
 
-			$this->nextAction( $view, 'search', 'group', null, 'delete' );
+			$this->nextAction($view, 'search', 'group', null, 'delete');
 			return;
 		}
-		catch( \Aimeos\MShop\Exception $e )
+		catch (\Aimeos\MShop\Exception $e)
 		{
-			$error = array( 'group-item' => $context->getI18n()->dt( 'mshop', $e->getMessage() ) );
-			$view->errors = $view->get( 'errors', [] ) + $error;
-			$this->logException( $e );
+			$error = array('group-item' => $context->getI18n()->dt('mshop', $e->getMessage()));
+			$view->errors = $view->get('errors', []) + $error;
+			$this->logException($e);
 		}
-		catch( \Exception $e )
+		catch (\Exception $e)
 		{
-			$error = array( 'group-item' => $e->getMessage() . ', ' . $e->getFile() . ':' . $e->getLine() );
-			$view->errors = $view->get( 'errors', [] ) + $error;
-			$this->logException( $e );
+			$error = array('group-item' => $e->getMessage() . ', ' . $e->getFile() . ':' . $e->getLine());
+			$view->errors = $view->get('errors', []) + $error;
+			$this->logException($e);
 		}
 
 		$manager->rollback();
@@ -178,37 +178,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, 'customer/group' );
+			$manager = \Aimeos\MShop::create($context, 'customer/group');
 
-			$view->item = $manager->getItem( $id );
+			$view->item = $manager->getItem($id);
 			$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( 'group-item' => $context->getI18n()->dt( 'mshop', $e->getMessage() ) );
-			$view->errors = $view->get( 'errors', [] ) + $error;
-			$this->logException( $e );
+			$error = array('group-item' => $context->getI18n()->dt('mshop', $e->getMessage()));
+			$view->errors = $view->get('errors', []) + $error;
+			$this->logException($e);
 		}
-		catch( \Exception $e )
+		catch (\Exception $e)
 		{
-			$error = array( 'group-item' => $e->getMessage() . ', ' . $e->getFile() . ':' . $e->getLine() );
-			$view->errors = $view->get( 'errors', [] ) + $error;
-			$this->logException( $e );
+			$error = array('group-item' => $e->getMessage() . ', ' . $e->getFile() . ':' . $e->getLine());
+			$view->errors = $view->get('errors', []) + $error;
+			$this->logException($e);
 		}
 
-		return $this->render( $view );
+		return $this->render($view);
 	}
 
 
@@ -222,40 +222,40 @@ 
 		$view = $this->getView();
 		$context = $this->getContext();
 
-		$manager = \Aimeos\MShop::create( $context, 'customer/group' );
+		$manager = \Aimeos\MShop::create($context, 'customer/group');
 		$manager->begin();
 
 		try
 		{
-			$item = $this->fromArray( $view->param( 'item', [] ) );
-			$view->item = $item->getId() ? $item : $manager->saveItem( $item );
+			$item = $this->fromArray($view->param('item', []));
+			$view->item = $item->getId() ? $item : $manager->saveItem($item);
 			$view->itemBody = '';
 
-			foreach( $this->getSubClients() as $client ) {
+			foreach ($this->getSubClients() as $client) {
 				$view->itemBody .= $client->save();
 			}
 
-			$manager->saveItem( clone $view->item );
+			$manager->saveItem(clone $view->item);
 			$manager->commit();
 
-			$this->nextAction( $view, $view->param( 'next' ), 'group', $view->item->getId(), 'save' );
+			$this->nextAction($view, $view->param('next'), 'group', $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( 'group-item' => $context->getI18n()->dt( 'mshop', $e->getMessage() ) );
-			$view->errors = $view->get( 'errors', [] ) + $error;
-			$this->logException( $e );
+			$error = array('group-item' => $context->getI18n()->dt('mshop', $e->getMessage()));
+			$view->errors = $view->get('errors', []) + $error;
+			$this->logException($e);
 		}
-		catch( \Exception $e )
+		catch (\Exception $e)
 		{
-			$error = array( 'group-item' => $e->getMessage() . ', ' . $e->getFile() . ':' . $e->getLine() );
-			$view->errors = $view->get( 'errors', [] ) + $error;
-			$this->logException( $e );
+			$error = array('group-item' => $e->getMessage() . ', ' . $e->getFile() . ':' . $e->getLine());
+			$view->errors = $view->get('errors', []) + $error;
+			$this->logException($e);
 		}
 
 		$manager->rollback();
@@ -277,31 +277,31 @@ 
 		try
 		{
 			$total = 0;
-			$params = $this->storeSearchParams( $view->param(), 'group' );
-			$manager = \Aimeos\MShop::create( $context, 'customer/group' );
-			$search = $this->initCriteria( $manager->createSearch(), $params );
+			$params = $this->storeSearchParams($view->param(), 'group');
+			$manager = \Aimeos\MShop::create($context, 'customer/group');
+			$search = $this->initCriteria($manager->createSearch(), $params);
 
-			$view->items = $manager->searchItems( $search, [], $total );
-			$view->filterAttributes = $manager->getSearchAttributes( true );
+			$view->items = $manager->searchItems($search, [], $total);
+			$view->filterAttributes = $manager->getSearchAttributes(true);
 			$view->filterOperators = $search->getOperators();
 			$view->total = $total;
 			$view->itemBody = '';
 
-			foreach( $this->getSubClients() as $client ) {
+			foreach ($this->getSubClients() as $client) {
 				$view->itemBody .= $client->search();
 			}
 		}
-		catch( \Aimeos\MShop\Exception $e )
+		catch (\Aimeos\MShop\Exception $e)
 		{
-			$error = array( 'group-item' => $context->getI18n()->dt( 'mshop', $e->getMessage() ) );
-			$view->errors = $view->get( 'errors', [] ) + $error;
-			$this->logException( $e );
+			$error = array('group-item' => $context->getI18n()->dt('mshop', $e->getMessage()));
+			$view->errors = $view->get('errors', []) + $error;
+			$this->logException($e);
 		}
-		catch( \Exception $e )
+		catch (\Exception $e)
 		{
-			$error = array( 'group-item' => $e->getMessage() . ', ' . $e->getFile() . ':' . $e->getLine() );
-			$view->errors = $view->get( 'errors', [] ) + $error;
-			$this->logException( $e );
+			$error = array('group-item' => $e->getMessage() . ', ' . $e->getFile() . ':' . $e->getLine());
+			$view->errors = $view->get('errors', []) + $error;
+			$this->logException($e);
 		}
 
 		/** admin/jqadm/group/template-list
@@ -326,7 +326,7 @@ 
 		$tplconf = 'admin/jqadm/group/template-list';
 		$default = 'group/list-standard';
 
-		return $view->render( $view->config( $tplconf, $default ) );
+		return $view->render($view->config($tplconf, $default));
 	}
 
 
@@ -337,7 +337,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/group/decorators/excludes
 		 * Excludes decorators added by the "common" option from the group JQAdm client
@@ -412,7 +412,7 @@ 
 		 * @see admin/jqadm/group/decorators/excludes
 		 * @see admin/jqadm/group/decorators/global
 		 */
-		return $this->createSubClient( 'group/' . $type, $name );
+		return $this->createSubClient('group/' . $type, $name);
 	}
 
 
@@ -456,7 +456,7 @@ 
 		 * @since 2018.07
 		 * @category Developer
 		 */
-		return $this->getContext()->getConfig()->get( 'admin/jqadm/group/standard/subparts', [] );
+		return $this->getContext()->getConfig()->get('admin/jqadm/group/standard/subparts', []);
 	}
 
 
@@ -467,17 +467,17 @@ 
 	 * @param string[] Data array
 	 * @return \Aimeos\MShop\Group\Item\Iface New group item object
 	 */
-	protected function fromArray( array $data )
+	protected function fromArray(array $data)
 	{
-		$manager = \Aimeos\MShop::create( $this->getContext(), 'customer/group' );
+		$manager = \Aimeos\MShop::create($this->getContext(), 'customer/group');
 
-		if( isset( $data['customer.group.id'] ) && $data['customer.group.id'] != '' ) {
-			$item = $manager->getItem( $data['customer.group.id'] );
+		if (isset($data['customer.group.id']) && $data['customer.group.id'] != '') {
+			$item = $manager->getItem($data['customer.group.id']);
 		} else {
 			$item = $manager->createItem();
 		}
 
-		$item->fromArray( $data );
+		$item->fromArray($data);
 
 		return $item;
 	}
@@ -489,11 +489,11 @@ 
 	 * @param \Aimeos\MShop\Customer\Item\Group\Iface $item Group item object
 	 * @return string[] Multi-dimensional associative list of item data
 	 */
-	protected function toArray( \Aimeos\MShop\Customer\Item\Group\Iface $item, $copy = false )
+	protected function toArray(\Aimeos\MShop\Customer\Item\Group\Iface $item, $copy = false)
 	{
-		$data = $item->toArray( true );
+		$data = $item->toArray(true);
 
-		if( $copy === true )
+		if ($copy === true)
 		{
 			$data['customer.group.siteid'] = $this->getContext()->getLocale()->getSiteId();
 			$data['customer.group.id'] = '';
@@ -509,7 +509,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/group/template-item
 		 * Relative path to the HTML body template for the group item.
@@ -533,6 +533,6 @@ 
 		$tplconf = 'admin/jqadm/group/template-item';
 		$default = 'group/item-standard';
 
-		return $view->render( $view->config( $tplconf, $default ) );
+		return $view->render($view->config($tplconf, $default));
 	}
 }

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

Spacing +63 added lines, -63 removed lines

@@ -10,7 +10,7 @@ 
 
 namespace Aimeos\Admin\JQAdm\Order\Invoice;
 
-sprintf( 'invoice' ); // for translation
+sprintf('invoice'); // for translation
 
 
 /**
@@ -69,31 +69,31 @@ 
 		try
 		{
 			$total = 0;
-			$params = $this->storeSearchParams( $view->param( 'oi', [] ), 'orderinvoice' );
-			$orderItems = $this->getOrderItems( $view->item, $params, $total );
+			$params = $this->storeSearchParams($view->param('oi', []), 'orderinvoice');
+			$orderItems = $this->getOrderItems($view->item, $params, $total);
 
-			$view->invoiceData = $this->toArray( $orderItems );
+			$view->invoiceData = $this->toArray($orderItems);
 			$view->invoiceTotal = $total;
 			$view->invoiceBody = '';
 
-			foreach( $this->getSubClients() as $client ) {
+			foreach ($this->getSubClients() as $client) {
 				$view->invoiceBody .= $client->search();
 			}
 		}
-		catch( \Aimeos\MShop\Exception $e )
+		catch (\Aimeos\MShop\Exception $e)
 		{
-			$error = array( 'order-item-invoice' => $this->getContext()->getI18n()->dt( 'mshop', $e->getMessage() ) );
-			$view->errors = $view->get( 'errors', [] ) + $error;
-			$this->logException( $e );
+			$error = array('order-item-invoice' => $this->getContext()->getI18n()->dt('mshop', $e->getMessage()));
+			$view->errors = $view->get('errors', []) + $error;
+			$this->logException($e);
 		}
-		catch( \Exception $e )
+		catch (\Exception $e)
 		{
-			$error = array( 'order-item-invoice' => $e->getMessage() . ', ' . $e->getFile() . ':' . $e->getLine() );
-			$view->errors = $view->get( 'errors', [] ) + $error;
-			$this->logException( $e );
+			$error = array('order-item-invoice' => $e->getMessage() . ', ' . $e->getFile() . ':' . $e->getLine());
+			$view->errors = $view->get('errors', []) + $error;
+			$this->logException($e);
 		}
 
-		return $this->render( $view );
+		return $this->render($view);
 	}
 
 
@@ -105,33 +105,33 @@ 
 		$view = $this->getView();
 		$context = $this->getContext();
 
-		$manager = \Aimeos\MShop::create( $context, 'order' );
+		$manager = \Aimeos\MShop::create($context, 'order');
 		$manager->begin();
 
 		try
 		{
-			$this->storeSearchParams( $view->param( 'oi', [] ), 'orderinvoice' );
-			$this->fromArray( $view->item, $view->param( 'invoice', [] ) );
+			$this->storeSearchParams($view->param('oi', []), 'orderinvoice');
+			$this->fromArray($view->item, $view->param('invoice', []));
 			$view->invoiceBody = '';
 
-			foreach( $this->getSubClients() as $client ) {
+			foreach ($this->getSubClients() as $client) {
 				$view->invoiceBody .= $client->save();
 			}
 
 			$manager->commit();
 			return;
 		}
-		catch( \Aimeos\MShop\Exception $e )
+		catch (\Aimeos\MShop\Exception $e)
 		{
-			$error = array( 'order-item-invoice' => $context->getI18n()->dt( 'mshop', $e->getMessage() ) );
-			$view->errors = $view->get( 'errors', [] ) + $error;
-			$this->logException( $e );
+			$error = array('order-item-invoice' => $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-invoice' => $e->getMessage() . ', ' . $e->getFile() . ':' . $e->getLine() );
-			$view->errors = $view->get( 'errors', [] ) + $error;
-			$this->logException( $e );
+			$error = array('order-item-invoice' => $e->getMessage() . ', ' . $e->getFile() . ':' . $e->getLine());
+			$view->errors = $view->get('errors', []) + $error;
+			$this->logException($e);
 		}
 
 		$manager->rollback();
@@ -147,7 +147,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/invoice/decorators/excludes
 		 * Excludes decorators added by the "common" option from the order JQAdm client
@@ -222,7 +222,7 @@ 
 		 * @see admin/jqadm/order/invoice/decorators/excludes
 		 * @see admin/jqadm/order/invoice/decorators/global
 		 */
-		return $this->createSubClient( 'order/invoice/' . $type, $name );
+		return $this->createSubClient('order/invoice/' . $type, $name);
 	}
 
 
@@ -266,7 +266,7 @@ 
 		 * @since 2017.07
 		 * @category Developer
 		 */
-		return $this->getContext()->getConfig()->get( 'admin/jqadm/order/invoice/standard/subparts', [] );
+		return $this->getContext()->getConfig()->get('admin/jqadm/order/invoice/standard/subparts', []);
 	}
 
 
@@ -278,21 +278,21 @@ 
 	 * @param integer $total Value/result parameter that will contain the item total afterwards
 	 * @return \Aimeos\MShop\Order\Item\Iface[] Associative list of order IDs as keys and items as values
 	 */
-	protected function getOrderItems( \Aimeos\MShop\Order\Item\Base\Iface $order, array $params, &$total )
+	protected function getOrderItems(\Aimeos\MShop\Order\Item\Base\Iface $order, array $params, &$total)
 	{
-		$manager = \Aimeos\MShop::create( $this->getContext(), 'order' );
+		$manager = \Aimeos\MShop::create($this->getContext(), 'order');
 
 		$search = $manager->createSearch();
-		$search->setSortations( [$search->sort( '-', 'order.ctime' )] );
+		$search->setSortations([$search->sort('-', 'order.ctime')]);
 
-		$search = $this->initCriteria( $search, $params, 'orderinvoice' );
+		$search = $this->initCriteria($search, $params, 'orderinvoice');
 		$expr = [
-			$search->compare( '==', 'order.baseid', $order->getId() ),
+			$search->compare('==', 'order.baseid', $order->getId()),
 			$search->getConditions(),
 		];
-		$search->setConditions( $search->combine( '&&', $expr ) );
+		$search->setConditions($search->combine('&&', $expr));
 
-		return $manager->searchItems( $search, [], $total );
+		return $manager->searchItems($search, [], $total);
 	}
 
 
@@ -302,52 +302,52 @@ 
 	 * @param \Aimeos\MShop\Order\Item\Base\Iface $order Order base item object
 	 * @param string[] $data Data array
 	 */
-	protected function fromArray( \Aimeos\MShop\Order\Item\Base\Iface $order, array $data )
+	protected function fromArray(\Aimeos\MShop\Order\Item\Base\Iface $order, array $data)
 	{
-		$invoiceIds = $this->getValue( $data, 'order.id', [] );
-		$manager = \Aimeos\MShop::create( $this->getContext(), 'order' );
+		$invoiceIds = $this->getValue($data, 'order.id', []);
+		$manager = \Aimeos\MShop::create($this->getContext(), 'order');
 
-		$search = $manager->createSearch()->setSlice( 0, count( $invoiceIds ) );
-		$search->setConditions( $search->compare( '==', 'order.id', $invoiceIds ) );
+		$search = $manager->createSearch()->setSlice(0, count($invoiceIds));
+		$search->setConditions($search->compare('==', 'order.id', $invoiceIds));
 
-		$items = $manager->searchItems( $search );
+		$items = $manager->searchItems($search);
 
 
-		foreach( $invoiceIds as $idx => $id )
+		foreach ($invoiceIds as $idx => $id)
 		{
-			if( !isset( $items[$id] ) ) {
+			if (!isset($items[$id])) {
 				$item = $manager->createItem();
 			} else {
 				$item = $items[$id];
 			}
 
-			if( isset( $data['order.statusdelivery'][$idx] ) ) {
-				$item->setDeliveryStatus( $data['order.statusdelivery'][$idx] );
+			if (isset($data['order.statusdelivery'][$idx])) {
+				$item->setDeliveryStatus($data['order.statusdelivery'][$idx]);
 			}
 
-			if( isset( $data['order.statuspayment'][$idx] ) ) {
-				$item->setPaymentStatus( $data['order.statuspayment'][$idx] );
+			if (isset($data['order.statuspayment'][$idx])) {
+				$item->setPaymentStatus($data['order.statuspayment'][$idx]);
 			}
 
-			if( isset( $data['order.datedelivery'][$idx] ) ) {
-				$item->setDateDelivery( $data['order.datedelivery'][$idx] );
+			if (isset($data['order.datedelivery'][$idx])) {
+				$item->setDateDelivery($data['order.datedelivery'][$idx]);
 			}
 
-			if( isset( $data['order.datepayment'][$idx] ) ) {
-				$item->setDatePayment( $data['order.datepayment'][$idx] );
+			if (isset($data['order.datepayment'][$idx])) {
+				$item->setDatePayment($data['order.datepayment'][$idx]);
 			}
 
-			if( isset( $data['order.relatedid'][$idx] ) ) {
-				$item->setRelatedId( $data['order.relatedid'][$idx] );
+			if (isset($data['order.relatedid'][$idx])) {
+				$item->setRelatedId($data['order.relatedid'][$idx]);
 			}
 
-			if( isset( $data['order.type'][$idx] ) ) {
-				$item->setType( $data['order.type'][$idx] );
+			if (isset($data['order.type'][$idx])) {
+				$item->setType($data['order.type'][$idx]);
 			}
 
-			$item->setBaseId( $order->getId() );
+			$item->setBaseId($order->getId());
 
-			$manager->saveItem( $item );
+			$manager->saveItem($item);
 		}
 	}
 
@@ -358,13 +358,13 @@ 
 	 * @param \Aimeos\MShop\Order\Item\Iface[] $invoices List of invoices belonging to the order
 	 * @return string[] Multi-dimensional associative list of item data
 	 */
-	protected function toArray( array $invoices )
+	protected function toArray(array $invoices)
 	{
 		$data = [];
 
-		foreach( $invoices as $item )
+		foreach ($invoices as $item)
 		{
-			foreach( $item->toArray( true ) as $key => $value ) {
+			foreach ($item->toArray(true) as $key => $value) {
 				$data[$key][] = $value;
 			}
 		}
@@ -379,7 +379,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/invoice/template-item
 		 * Relative path to the HTML body template of the invoice subpart for orders.
@@ -403,6 +403,6 @@ 
 		$tplconf = 'admin/jqadm/order/invoice/template-item';
 		$default = 'order/item-invoice-standard';
 
-		return $view->render( $view->config( $tplconf, $default ) );
+		return $view->render($view->config($tplconf, $default));
 	}
 }

admin/jqadm/src/Admin/JQAdm/Product/Related/Suggest/Standard.php

Spacing +35 added lines, -35 removed lines

@@ -42,14 +42,14 @@ 
 	{
 		$view = $this->getView();
 
-		$view->suggestData = $this->toArray( $view->item, true );
+		$view->suggestData = $this->toArray($view->item, true);
 		$view->suggestBody = '';
 
-		foreach( $this->getSubClients() as $client ) {
+		foreach ($this->getSubClients() as $client) {
 			$view->suggestBody .= $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( 'related/suggest', [] );
+		$data = $view->param('related/suggest', []);
 
-		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->suggestData = $data;
 		$view->suggestBody = '';
 
-		foreach( $this->getSubClients() as $client ) {
+		foreach ($this->getSubClients() as $client) {
 			$view->suggestBody .= $client->create();
 		}
 
-		return $this->render( $view );
+		return $this->render($view);
 	}
 
 
@@ -88,14 +88,14 @@ 
 	{
 		$view = $this->getView();
 
-		$view->suggestData = $this->toArray( $view->item );
+		$view->suggestData = $this->toArray($view->item);
 		$view->suggestBody = '';
 
-		foreach( $this->getSubClients() as $client ) {
+		foreach ($this->getSubClients() as $client) {
 			$view->suggestBody .= $client->get();
 		}
 
-		return $this->render( $view );
+		return $this->render($view);
 	}
 
 
@@ -106,10 +106,10 @@ 
 	{
 		$view = $this->getView();
 
-		$this->fromArray( $view->item, $view->param( 'related/suggest', [] ) );
+		$this->fromArray($view->item, $view->param('related/suggest', []));
 		$view->suggestBody = '';
 
-		foreach( $this->getSubClients() as $client ) {
+		foreach ($this->getSubClients() as $client) {
 			$view->suggestBody .= $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/related/suggest/decorators/excludes
 		 * Excludes decorators added by the "common" option from the product JQAdm client
@@ -197,7 +197,7 @@ 
 		 * @see admin/jqadm/product/related/suggest/decorators/excludes
 		 * @see admin/jqadm/product/related/suggest/decorators/global
 		 */
-		return $this->createSubClient( 'product/related/suggest/' . $type, $name );
+		return $this->createSubClient('product/related/suggest/' . $type, $name);
 	}
 
 
@@ -241,7 +241,7 @@ 
 		 * @since 2016.01
 		 * @category Developer
 		 */
-		return $this->getContext()->getConfig()->get( 'admin/jqadm/product/related/suggest/standard/subparts', [] );
+		return $this->getContext()->getConfig()->get('admin/jqadm/product/related/suggest/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( 'suggestion' );
-		$listItems = $item->getListItems( 'product', 'suggestion', null, false );
+		$listItem = $listManager->createItem()->setType('suggestion');
+		$listItems = $item->getListItems('product', 'suggestion', 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( 'product', $litem, $litem->getRefItem() );
+			$item->addListItem('product', $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( 'product', 'suggestion', null, false ) as $listItem )
+		foreach ($item->getListItems('product', 'suggestion', 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/related/suggest/template-item
 		 * Relative path to the HTML body template of the suggest related subpart for products.
@@ -342,6 +342,6 @@ 
 		$tplconf = 'admin/jqadm/product/related/suggest/template-item';
 		$default = 'product/item-related-suggest-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/Related/Bought/Standard.php

Spacing +35 added lines, -35 removed lines

@@ -42,14 +42,14 @@ 
 	{
 		$view = $this->getView();
 
-		$view->boughtData = $this->toArray( $view->item, true );
+		$view->boughtData = $this->toArray($view->item, true);
 		$view->boughtBody = '';
 
-		foreach( $this->getSubClients() as $client ) {
+		foreach ($this->getSubClients() as $client) {
 			$view->boughtBody .= $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( 'related/bought', [] );
+		$data = $view->param('related/bought', []);
 
-		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->boughtData = $data;
 		$view->boughtBody = '';
 
-		foreach( $this->getSubClients() as $client ) {
+		foreach ($this->getSubClients() as $client) {
 			$view->boughtBody .= $client->create();
 		}
 
-		return $this->render( $view );
+		return $this->render($view);
 	}
 
 
@@ -88,14 +88,14 @@ 
 	{
 		$view = $this->getView();
 
-		$view->boughtData = $this->toArray( $view->item );
+		$view->boughtData = $this->toArray($view->item);
 		$view->boughtBody = '';
 
-		foreach( $this->getSubClients() as $client ) {
+		foreach ($this->getSubClients() as $client) {
 			$view->boughtBody .= $client->get();
 		}
 
-		return $this->render( $view );
+		return $this->render($view);
 	}
 
 
@@ -106,10 +106,10 @@ 
 	{
 		$view = $this->getView();
 
-		$this->fromArray( $view->item, $view->param( 'related/bought', [] ) );
+		$this->fromArray($view->item, $view->param('related/bought', []));
 		$view->boughtBody = '';
 
-		foreach( $this->getSubClients() as $client ) {
+		foreach ($this->getSubClients() as $client) {
 			$view->boughtBody .= $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/related/bought/decorators/excludes
 		 * Excludes decorators added by the "common" option from the product JQAdm client
@@ -197,7 +197,7 @@ 
 		 * @see admin/jqadm/product/related/bought/decorators/excludes
 		 * @see admin/jqadm/product/related/bought/decorators/global
 		 */
-		return $this->createSubClient( 'product/related/bought/' . $type, $name );
+		return $this->createSubClient('product/related/bought/' . $type, $name);
 	}
 
 
@@ -241,7 +241,7 @@ 
 		 * @since 2016.01
 		 * @category Developer
 		 */
-		return $this->getContext()->getConfig()->get( 'admin/jqadm/product/related/bought/standard/subparts', [] );
+		return $this->getContext()->getConfig()->get('admin/jqadm/product/related/bought/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( 'bought-together' );
-		$listItems = $item->getListItems( 'product', 'bought-together', null, false );
+		$listItem = $listManager->createItem()->setType('bought-together');
+		$listItems = $item->getListItems('product', 'bought-together', 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( 'product', $litem, $litem->getRefItem() );
+			$item->addListItem('product', $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( 'product', 'bought-together', null, false ) as $listItem )
+		foreach ($item->getListItems('product', 'bought-together', 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/related/bought/template-item
 		 * Relative path to the HTML body template of the bought related subpart for products.
@@ -342,6 +342,6 @@ 
 		$tplconf = 'admin/jqadm/product/related/bought/template-item';
 		$default = 'product/item-related-bought-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/Subscription/Standard.php

Spacing +52 added lines, -52 removed lines

@@ -10,7 +10,7 @@ 
 
 namespace Aimeos\Admin\JQAdm\Product\Subscription;
 
-sprintf( 'subscription' ); // for translation
+sprintf('subscription'); // for translation
 
 
 /**
@@ -44,14 +44,14 @@ 
 	{
 		$view = $this->getView();
 
-		$view->subscriptionData = $this->toArray( $view->item, true );
+		$view->subscriptionData = $this->toArray($view->item, true);
 		$view->subscriptionBody = '';
 
-		foreach( $this->getSubClients() as $client ) {
+		foreach ($this->getSubClients() as $client) {
 			$view->subscriptionBody .= $client->copy();
 		}
 
-		return $this->render( $view );
+		return $this->render($view);
 	}
 
 
@@ -64,20 +64,20 @@ 
 	{
 		$view = $this->getView();
 		$siteid = $this->getContext()->getLocale()->getSiteId();
-		$data = $view->param( 'subscription', [] );
+		$data = $view->param('subscription', []);
 
-		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->subscriptionData = $data;
 		$view->subscriptionBody = '';
 
-		foreach( $this->getSubClients() as $client ) {
+		foreach ($this->getSubClients() as $client) {
 			$view->subscriptionBody .= $client->create();
 		}
 
-		return $this->render( $view );
+		return $this->render($view);
 	}
 
 
@@ -90,14 +90,14 @@ 
 	{
 		$view = $this->getView();
 
-		$view->subscriptionData = $this->toArray( $view->item );
+		$view->subscriptionData = $this->toArray($view->item);
 		$view->subscriptionBody = '';
 
-		foreach( $this->getSubClients() as $client ) {
+		foreach ($this->getSubClients() as $client) {
 			$view->subscriptionBody .= $client->get();
 		}
 
-		return $this->render( $view );
+		return $this->render($view);
 	}
 
 
@@ -110,26 +110,26 @@ 
 
 		try
 		{
-			$this->fromArray( $view->item, $view->param( 'subscription', [] ) );
+			$this->fromArray($view->item, $view->param('subscription', []));
 			$view->subscriptionBody = '';
 
-			foreach( $this->getSubClients() as $client ) {
+			foreach ($this->getSubClients() as $client) {
 				$view->subscriptionBody .= $client->save();
 			}
 
 			return;
 		}
-		catch( \Aimeos\MShop\Exception $e )
+		catch (\Aimeos\MShop\Exception $e)
 		{
-			$error = array( 'product-item-subscription' => $this->getContext()->getI18n()->dt( 'mshop', $e->getMessage() ) );
-			$view->errors = $view->get( 'errors', [] ) + $error;
-			$this->logException( $e );
+			$error = array('product-item-subscription' => $this->getContext()->getI18n()->dt('mshop', $e->getMessage()));
+			$view->errors = $view->get('errors', []) + $error;
+			$this->logException($e);
 		}
-		catch( \Exception $e )
+		catch (\Exception $e)
 		{
-			$error = array( 'product-item-subscription' => $e->getMessage() . ', ' . $e->getFile() . ':' . $e->getLine() );
-			$view->errors = $view->get( 'errors', [] ) + $error;
-			$this->logException( $e );
+			$error = array('product-item-subscription' => $e->getMessage() . ', ' . $e->getFile() . ':' . $e->getLine());
+			$view->errors = $view->get('errors', []) + $error;
+			$this->logException($e);
 		}
 
 		throw new \Aimeos\Admin\JQAdm\Exception();
@@ -143,7 +143,7 @@ 
 	 * @param string|null $name Name of the sub-client (Default if null)
 	 * @return \Aimeos\Admin\JQAdm\Iface Sub-client object
 	 */
-	public function getSubClient( $type, $name = null )
+	public function getSubClient($type, $name = null)
 	{
 		/** admin/jqadm/product/subscription/decorators/excludes
 		 * Excludes decorators added by the "common" subscription from the product JQAdm client
@@ -218,7 +218,7 @@ 
 		 * @see admin/jqadm/product/subscription/decorators/excludes
 		 * @see admin/jqadm/product/subscription/decorators/global
 		 */
-		return $this->createSubClient( 'product/subscription/' . $type, $name );
+		return $this->createSubClient('product/subscription/' . $type, $name);
 	}
 
 
@@ -229,17 +229,17 @@ 
 	 */
 	protected function getIntervalItems()
 	{
-		$manager = \Aimeos\MShop::create( $this->getContext(), 'attribute' );
+		$manager = \Aimeos\MShop::create($this->getContext(), 'attribute');
 
 		$search = $manager->createSearch();
 		$expr = [
-			$search->compare( '==', 'attribute.type', 'interval' ),
-			$search->compare( '==', 'attribute.domain', 'product' ),
+			$search->compare('==', 'attribute.type', 'interval'),
+			$search->compare('==', 'attribute.domain', 'product'),
 		];
-		$search->setConditions( $search->combine( '&&', $expr ) );
-		$search->setSortations( [$search->sort( '+', 'attribute.code' )] );
+		$search->setConditions($search->combine('&&', $expr));
+		$search->setSortations([$search->sort('+', 'attribute.code')]);
 
-		return $manager->searchItems( $search );
+		return $manager->searchItems($search);
 	}
 
 
@@ -283,7 +283,7 @@ 
 		 * @since 2018.04
 		 * @category Developer
 		 */
-		return $this->getContext()->getConfig()->get( 'admin/jqadm/product/subscription/standard/subparts', [] );
+		return $this->getContext()->getConfig()->get('admin/jqadm/product/subscription/standard/subparts', []);
 	}
 
 
@@ -293,37 +293,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)
 	{
 		$context = $this->getContext();
 
-		$attrManager = \Aimeos\MShop::create( $context, 'attribute' );
-		$listManager = \Aimeos\MShop::create( $context, 'product/lists' );
+		$attrManager = \Aimeos\MShop::create($context, 'attribute');
+		$listManager = \Aimeos\MShop::create($context, 'product/lists');
 
-		$listItems = $item->getListItems( 'attribute', 'config', 'interval', false );
+		$listItems = $item->getListItems('attribute', 'config', 'interval', false);
 
-		foreach( $data as $idx => $entry )
+		foreach ($data as $idx => $entry)
 		{
-			if( !array_key_exists( 'attribute.id', $entry ) ) {
+			if (!array_key_exists('attribute.id', $entry)) {
 				continue;
 			}
 
-			if( $entry['attribute.id'] == '' || ( $listItem = $item->getListItem( 'attribute', 'config', $entry['attribute.id'], false ) ) === null ) {
-				$listItem = $listManager->createItem()->setType( 'config' );
+			if ($entry['attribute.id'] == '' || ($listItem = $item->getListItem('attribute', 'config', $entry['attribute.id'], false)) === null) {
+				$listItem = $listManager->createItem()->setType('config');
 			}
 
-			if( $entry['attribute.id'] == '' || ( $refItem = $listItem->getRefItem() ) === null )
+			if ($entry['attribute.id'] == '' || ($refItem = $listItem->getRefItem()) === null)
 			{
-				$refItem = $attrManager->createItem()->setType( 'interval' );
-				$refItem->fromArray( $entry );
+				$refItem = $attrManager->createItem()->setType('interval');
+				$refItem->fromArray($entry);
 			}
 
-			unset( $listItems[$listItem->getId()] );
+			unset($listItems[$listItem->getId()]);
 
-			$item->addListItem( 'attribute', $listItem->setPosition( $idx ), $refItem );
+			$item->addListItem('attribute', $listItem->setPosition($idx), $refItem);
 		}
 
-		$item->deleteListItems( $listItems );
+		$item->deleteListItems($listItems);
 	}
 
 
@@ -334,20 +334,20 @@ 
 	 * @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 = $map = [];
 		$siteId = $this->getContext()->getLocale()->getSiteId();
 
-		foreach( $item->getListItems( 'attribute', 'config', 'interval', false ) as $listItem ) {
+		foreach ($item->getListItems('attribute', 'config', 'interval', false) as $listItem) {
 			$map[$listItem->getRefId()] = $listItem;
 		}
 
-		foreach( $this->getIntervalItems() as $attrId => $attrItem )
+		foreach ($this->getIntervalItems() as $attrId => $attrItem)
 		{
-			$list = $attrItem->toArray( true );
+			$list = $attrItem->toArray(true);
 
-			if( isset( $map[$attrId] ) && $copy !== true )
+			if (isset($map[$attrId]) && $copy !== true)
 			{
 				$list['product.lists.siteid'] = (string) $map[$attrId]->getSiteId();
 				$list['product.lists.id'] = (string) $map[$attrId]->getId();
@@ -361,7 +361,7 @@ 
 			$matches = [];
 			$list['Y'] = $list['M'] = $list['W'] = $list['D'] = 0;
 
-			if( preg_match( '/^P([0-9]+)Y([0-9]+)M([0-9]+)W([0-9]+)D$/', $list['attribute.code'], $matches ) === 1 )
+			if (preg_match('/^P([0-9]+)Y([0-9]+)M([0-9]+)W([0-9]+)D$/', $list['attribute.code'], $matches) === 1)
 			{
 				$list['Y'] = $matches[1];
 				$list['M'] = $matches[2];
@@ -382,7 +382,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/subscription/template-item
 		 * Relative path to the HTML body template of the subscription subpart for products.
@@ -406,6 +406,6 @@ 
 		$tplconf = 'admin/jqadm/product/subscription/template-item';
 		$default = 'product/item-subscription-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/Option/Custom/Standard.php

Spacing +35 added lines, -35 removed lines

@@ -42,14 +42,14 @@ 
 	{
 		$view = $this->getView();
 
-		$view->customData = $this->toArray( $view->item, true );
+		$view->customData = $this->toArray($view->item, true);
 		$view->customBody = '';
 
-		foreach( $this->getSubClients() as $client ) {
+		foreach ($this->getSubClients() as $client) {
 			$view->customBody .= $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( 'option/custom', [] );
+		$data = $view->param('option/custom', []);
 
-		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->customData = $data;
 		$view->customBody = '';
 
-		foreach( $this->getSubClients() as $client ) {
+		foreach ($this->getSubClients() as $client) {
 			$view->customBody .= $client->create();
 		}
 
-		return $this->render( $view );
+		return $this->render($view);
 	}
 
 
@@ -88,14 +88,14 @@ 
 	{
 		$view = $this->getView();
 
-		$view->customData = $this->toArray( $view->item );
+		$view->customData = $this->toArray($view->item);
 		$view->customBody = '';
 
-		foreach( $this->getSubClients() as $client ) {
+		foreach ($this->getSubClients() as $client) {
 			$view->customBody .= $client->get();
 		}
 
-		return $this->render( $view );
+		return $this->render($view);
 	}
 
 
@@ -106,10 +106,10 @@ 
 	{
 		$view = $this->getView();
 
-		$this->fromArray( $view->item, $view->param( 'option/custom', [] ) );
+		$this->fromArray($view->item, $view->param('option/custom', []));
 		$view->customBody = '';
 
-		foreach( $this->getSubClients() as $client ) {
+		foreach ($this->getSubClients() as $client) {
 			$view->customBody .= $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/option/custom/decorators/excludes
 		 * Excludes decorators added by the "common" option from the product JQAdm client
@@ -197,7 +197,7 @@ 
 		 * @see admin/jqadm/product/option/custom/decorators/excludes
 		 * @see admin/jqadm/product/option/custom/decorators/global
 		 */
-		return $this->createSubClient( 'product/option/custom/' . $type, $name );
+		return $this->createSubClient('product/option/custom/' . $type, $name);
 	}
 
 
@@ -241,7 +241,7 @@ 
 		 * @since 2017.03
 		 * @category Developer
 		 */
-		return $this->getContext()->getConfig()->get( 'admin/jqadm/product/option/custom/standard/subparts', [] );
+		return $this->getContext()->getConfig()->get('admin/jqadm/product/option/custom/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( 'custom' );
-		$listItems = $item->getListItems( 'attribute', 'custom', null, false );
+		$listItem = $listManager->createItem()->setType('custom');
+		$listItems = $item->getListItems('attribute', 'custom', 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', 'custom', null, false ) as $listItem )
+		foreach ($item->getListItems('attribute', 'custom', 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/option/custom/template-item
 		 * Relative path to the HTML body template of the custom option subpart for products.
@@ -342,6 +342,6 @@ 
 		$tplconf = 'admin/jqadm/product/option/custom/template-item';
 		$default = 'product/item-option-custom-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/Option/Config/Standard.php

Spacing +35 added lines, -35 removed lines

@@ -42,14 +42,14 @@ 
 	{
 		$view = $this->getView();
 
-		$view->configData = $this->toArray( $view->item, true );
+		$view->configData = $this->toArray($view->item, true);
 		$view->configBody = '';
 
-		foreach( $this->getSubClients() as $client ) {
+		foreach ($this->getSubClients() as $client) {
 			$view->configBody .= $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( 'option/config', [] );
+		$data = $view->param('option/config', []);
 
-		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->configData = $data;
 		$view->configBody = '';
 
-		foreach( $this->getSubClients() as $client ) {
+		foreach ($this->getSubClients() as $client) {
 			$view->configBody .= $client->create();
 		}
 
-		return $this->render( $view );
+		return $this->render($view);
 	}
 
 
@@ -88,14 +88,14 @@ 
 	{
 		$view = $this->getView();
 
-		$view->configData = $this->toArray( $view->item );
+		$view->configData = $this->toArray($view->item);
 		$view->configBody = '';
 
-		foreach( $this->getSubClients() as $client ) {
+		foreach ($this->getSubClients() as $client) {
 			$view->configBody .= $client->get();
 		}
 
-		return $this->render( $view );
+		return $this->render($view);
 	}
 
 
@@ -106,10 +106,10 @@ 
 	{
 		$view = $this->getView();
 
-		$this->fromArray( $view->item, $view->param( 'option/config', [] ) );
+		$this->fromArray($view->item, $view->param('option/config', []));
 		$view->configBody = '';
 
-		foreach( $this->getSubClients() as $client ) {
+		foreach ($this->getSubClients() as $client) {
 			$view->configBody .= $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/option/config/decorators/excludes
 		 * Excludes decorators added by the "common" option from the product JQAdm client
@@ -197,7 +197,7 @@ 
 		 * @see admin/jqadm/product/option/config/decorators/excludes
 		 * @see admin/jqadm/product/option/config/decorators/global
 		 */
-		return $this->createSubClient( 'product/option/config/' . $type, $name );
+		return $this->createSubClient('product/option/config/' . $type, $name);
 	}
 
 
@@ -241,7 +241,7 @@ 
 		 * @since 2017.03
 		 * @category Developer
 		 */
-		return $this->getContext()->getConfig()->get( 'admin/jqadm/product/option/config/standard/subparts', [] );
+		return $this->getContext()->getConfig()->get('admin/jqadm/product/option/config/standard/subparts', []);
 	}
 
 
@@ -251,31 +251,31 @@ 
 	 * @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( 'config' );
-		$listItems = $item->getListItems( 'attribute', 'config', null, false );
+		$listItem = $listManager->createItem()->setType('config');
+		$listItems = $item->getListItems('attribute', 'config', 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);
 	}
 
 
@@ -286,26 +286,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', 'config', null, false ) as $listItem )
+		foreach ($item->getListItems('attribute', 'config', 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;
 			}
 		}
@@ -320,7 +320,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/option/config/template-item
 		 * Relative path to the HTML body template of the config option subpart for products.
@@ -344,6 +344,6 @@ 
 		$tplconf = 'admin/jqadm/product/option/config/template-item';
 		$default = 'product/item-option-config-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/Hidden/Standard.php

Spacing +35 added lines, -35 removed lines

@@ -42,14 +42,14 @@ 
 	{
 		$view = $this->getView();
 
-		$view->hiddenData = $this->toArray( $view->item, true );
+		$view->hiddenData = $this->toArray($view->item, true);
 		$view->hiddenBody = '';
 
-		foreach( $this->getSubClients() as $client ) {
+		foreach ($this->getSubClients() as $client) {
 			$view->hiddenBody .= $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/hidden', [] );
+		$data = $view->param('characteristic/hidden', []);
 
-		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->hiddenData = $data;
 		$view->hiddenBody = '';
 
-		foreach( $this->getSubClients() as $client ) {
+		foreach ($this->getSubClients() as $client) {
 			$view->hiddenBody .= $client->create();
 		}
 
-		return $this->render( $view );
+		return $this->render($view);
 	}
 
 
@@ -88,14 +88,14 @@ 
 	{
 		$view = $this->getView();
 
-		$view->hiddenData = $this->toArray( $view->item );
+		$view->hiddenData = $this->toArray($view->item);
 		$view->hiddenBody = '';
 
-		foreach( $this->getSubClients() as $client ) {
+		foreach ($this->getSubClients() as $client) {
 			$view->hiddenBody .= $client->get();
 		}
 
-		return $this->render( $view );
+		return $this->render($view);
 	}
 
 
@@ -106,10 +106,10 @@ 
 	{
 		$view = $this->getView();
 
-		$this->fromArray( $view->item, $view->param( 'characteristic/hidden', [] ) );
+		$this->fromArray($view->item, $view->param('characteristic/hidden', []));
 		$view->hiddenBody = '';
 
-		foreach( $this->getSubClients() as $client ) {
+		foreach ($this->getSubClients() as $client) {
 			$view->hiddenBody .= $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/hidden/decorators/excludes
 		 * Excludes decorators added by the "common" option from the product JQAdm client
@@ -197,7 +197,7 @@ 
 		 * @see admin/jqadm/product/characteristic/hidden/decorators/excludes
 		 * @see admin/jqadm/product/characteristic/hidden/decorators/global
 		 */
-		return $this->createSubClient( 'product/characteristic/hidden/' . $type, $name );
+		return $this->createSubClient('product/characteristic/hidden/' . $type, $name);
 	}
 
 
@@ -241,7 +241,7 @@ 
 		 * @since 2016.01
 		 * @category Developer
 		 */
-		return $this->getContext()->getConfig()->get( 'admin/jqadm/product/characteristic/hidden/standard/subparts', [] );
+		return $this->getContext()->getConfig()->get('admin/jqadm/product/characteristic/hidden/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( 'hidden' );
-		$listItems = $item->getListItems( 'attribute', 'hidden', null, false );
+		$listItem = $listManager->createItem()->setType('hidden');
+		$listItems = $item->getListItems('attribute', 'hidden', 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', 'hidden', null, false ) as $listItem )
+		foreach ($item->getListItems('attribute', 'hidden', 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/hidden/template-item
 		 * Relative path to the HTML body template of the hidden characteristic subpart for products.
@@ -342,6 +342,6 @@ 
 		$tplconf = 'admin/jqadm/product/characteristic/hidden/template-item';
 		$default = 'product/item-characteristic-hidden-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/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