aimeos / ai-admin-jqadm

admin/jqadm/src/Admin/JQAdm/Group/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 createClient( \Aimeos\MShop\Context\Item\Iface $context, $name = null )
+	public static function createClient(\Aimeos\MShop\Context\Item\Iface $context, $name = null)
 	{
 		/** admin/jqadm/group/name
 		 * Class name of the used account favorite client implementation
@@ -64,22 +64,22 @@ 
 		 * @since 2018.07
 		 * @category Developer
 		 */
-		if( $name === null ) {
-			$name = $context->getConfig()->get( 'admin/jqadm/group/name', 'Standard' );
+		if ($name === null) {
+			$name = $context->getConfig()->get('admin/jqadm/group/name', 'Standard');
 		}
 
-		if( ctype_alnum( $name ) === false )
+		if (ctype_alnum($name) === false)
 		{
-			$classname = is_string( $name ) ? '\\Aimeos\\Admin\\JQAdm\\Group\\' . $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\\Group\\' . $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\\Group\\' . $name;
 
-		$client = self::createClientBase( $context, $classname, $iface );
+		$client = self::createClientBase($context, $classname, $iface);
 
-		return self::addClientDecorators( $context, $client, 'group' );
+		return self::addClientDecorators($context, $client, 'group');
 	}
 
 }
\ No newline at end of file

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

Spacing +11 added lines, -11 removed lines

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

admin/jqadm/src/Admin/JQAdm/Attribute/Text/Standard.php

Braces +2 added lines, -4 removed lines

@@ -160,14 +160,12 @@
 			}
 
 			return;
-		}
-		catch( \Aimeos\MShop\Exception $e )
+		} catch( \Aimeos\MShop\Exception $e )
 		{
 			$error = array( 'attribute-item-text' => $this->getContext()->getI18n()->dt( 'mshop', $e->getMessage() ) );
 			$view->errors = $view->get( 'errors', [] ) + $error;
 			$this->logException( $e );
-		}
-		catch( \Exception $e )
+		} catch( \Exception $e )
 		{
 			$error = array( 'attribute-item-text' => $e->getMessage() . ', ' . $e->getFile() . ':' . $e->getLine() );
 			$view->errors = $view->get( 'errors', [] ) + $error;

Spacing +67 added lines, -67 removed lines

@@ -10,7 +10,7 @@ 
 
 namespace Aimeos\Admin\JQAdm\Attribute\Text;
 
-sprintf( 'text' ); // for translation
+sprintf('text'); // for translation
 
 
 /**
@@ -42,16 +42,16 @@ 
 	 */
 	public function copy()
 	{
-		$view = $this->addViewData( $this->getView() );
+		$view = $this->addViewData($this->getView());
 
-		$view->textData = $this->toArray( $view->item, true );
+		$view->textData = $this->toArray($view->item, true);
 		$view->textBody = '';
 
-		foreach( $this->getSubClients() as $client ) {
+		foreach ($this->getSubClients() as $client) {
 			$view->textBody .= $client->copy();
 		}
 
-		return $this->render( $view );
+		return $this->render($view);
 	}
 
 
@@ -62,11 +62,11 @@ 
 	 */
 	public function create()
 	{
-		$view = $this->addViewData( $this->getView() );
+		$view = $this->addViewData($this->getView());
 		$siteid = $this->getContext()->getLocale()->getSiteId();
-		$data = $view->param( 'text', [] );
+		$data = $view->param('text', []);
 
-		foreach( $data as $idx => $entry )
+		foreach ($data as $idx => $entry)
 		{
 			$data[$idx]['attribute.lists.siteid'] = $siteid;
 			$data[$idx]['text.siteid'] = $siteid;
@@ -75,11 +75,11 @@ 
 		$view->textData = $data;
 		$view->textBody = '';
 
-		foreach( $this->getSubClients() as $client ) {
+		foreach ($this->getSubClients() as $client) {
 			$view->textBody .= $client->create();
 		}
 
-		return $this->render( $view );
+		return $this->render($view);
 	}
 
 
@@ -91,7 +91,7 @@ 
 		parent::delete();
 
 		$item = $this->getView()->item;
-		$item->deleteListItems( $item->getListItems( 'text', null, null, false ), true );
+		$item->deleteListItems($item->getListItems('text', null, null, false), true);
 	}
 
 
@@ -102,16 +102,16 @@ 
 	 */
 	public function get()
 	{
-		$view = $this->addViewData( $this->getView() );
+		$view = $this->addViewData($this->getView());
 
-		$view->textData = $this->toArray( $view->item );
+		$view->textData = $this->toArray($view->item);
 		$view->textBody = '';
 
-		foreach( $this->getSubClients() as $client ) {
+		foreach ($this->getSubClients() as $client) {
 			$view->textBody .= $client->get();
 		}
 
-		return $this->render( $view );
+		return $this->render($view);
 	}
 
 
@@ -124,26 +124,26 @@ 
 
 		try
 		{
-			$view->item = $this->fromArray( $view->item, $view->param( 'text', [] ) );
+			$view->item = $this->fromArray($view->item, $view->param('text', []));
 			$view->textBody = '';
 
-			foreach( $this->getSubClients() as $client ) {
+			foreach ($this->getSubClients() as $client) {
 				$view->textBody .= $client->save();
 			}
 
 			return;
 		}
-		catch( \Aimeos\MShop\Exception $e )
+		catch (\Aimeos\MShop\Exception $e)
 		{
-			$error = array( 'attribute-item-text' => $this->getContext()->getI18n()->dt( 'mshop', $e->getMessage() ) );
-			$view->errors = $view->get( 'errors', [] ) + $error;
-			$this->logException( $e );
+			$error = array('attribute-item-text' => $this->getContext()->getI18n()->dt('mshop', $e->getMessage()));
+			$view->errors = $view->get('errors', []) + $error;
+			$this->logException($e);
 		}
-		catch( \Exception $e )
+		catch (\Exception $e)
 		{
-			$error = array( 'attribute-item-text' => $e->getMessage() . ', ' . $e->getFile() . ':' . $e->getLine() );
-			$view->errors = $view->get( 'errors', [] ) + $error;
-			$this->logException( $e );
+			$error = array('attribute-item-text' => $e->getMessage() . ', ' . $e->getFile() . ':' . $e->getLine());
+			$view->errors = $view->get('errors', []) + $error;
+			$this->logException($e);
 		}
 
 		throw new \Aimeos\Admin\JQAdm\Exception();
@@ -157,7 +157,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/attribute/text/decorators/excludes
 		 * Excludes decorators added by the "common" option from the attribute JQAdm client
@@ -232,7 +232,7 @@ 
 		 * @see admin/jqadm/attribute/text/decorators/excludes
 		 * @see admin/jqadm/attribute/text/decorators/global
 		 */
-		return $this->createSubClient( 'attribute/text/' . $type, $name );
+		return $this->createSubClient('attribute/text/' . $type, $name);
 	}
 
 
@@ -276,7 +276,7 @@ 
 		 * @since 2017.07
 		 * @category Developer
 		 */
-		return $this->getContext()->getConfig()->get( 'admin/jqadm/attribute/text/standard/subparts', [] );
+		return $this->getContext()->getConfig()->get('admin/jqadm/attribute/text/standard/subparts', []);
 	}
 
 
@@ -286,23 +286,23 @@ 
 	 * @param \Aimeos\MW\View\Iface $view View object
 	 * @return \Aimeos\MW\View\Iface View object with assigned parameters
 	 */
-	protected function addViewData( \Aimeos\MW\View\Iface $view )
+	protected function addViewData(\Aimeos\MW\View\Iface $view)
 	{
 		$context = $this->getContext();
 
-		$textTypeManager = \Aimeos\MShop\Factory::createManager( $context, 'text/type' );
-		$listTypeManager = \Aimeos\MShop\Factory::createManager( $context, 'attribute/lists/type' );
+		$textTypeManager = \Aimeos\MShop\Factory::createManager($context, 'text/type');
+		$listTypeManager = \Aimeos\MShop\Factory::createManager($context, 'attribute/lists/type');
 
-		$search = $textTypeManager->createSearch( true )->setSlice( 0, 0x7fffffff );
-		$search->setConditions( $search->compare( '==', 'text.type.domain', 'attribute' ) );
-		$search->setSortations( array( $search->sort( '+', 'text.type.label' ) ) );
+		$search = $textTypeManager->createSearch(true)->setSlice(0, 0x7fffffff);
+		$search->setConditions($search->compare('==', 'text.type.domain', 'attribute'));
+		$search->setSortations(array($search->sort('+', 'text.type.label')));
 
-		$listSearch = $listTypeManager->createSearch( true )->setSlice( 0, 0x7fffffff );
-		$listSearch->setConditions( $listSearch->compare( '==', 'attribute.lists.type.domain', 'text' ) );
-		$listSearch->setSortations( array( $listSearch->sort( '+', 'attribute.lists.type.label' ) ) );
+		$listSearch = $listTypeManager->createSearch(true)->setSlice(0, 0x7fffffff);
+		$listSearch->setConditions($listSearch->compare('==', 'attribute.lists.type.domain', 'text'));
+		$listSearch->setSortations(array($listSearch->sort('+', 'attribute.lists.type.label')));
 
-		$view->textTypes = $textTypeManager->searchItems( $search );
-		$view->textListTypes = $this->sortType( $listTypeManager->searchItems( $listSearch ) );
+		$view->textTypes = $textTypeManager->searchItems($search);
+		$view->textListTypes = $this->sortType($listTypeManager->searchItems($listSearch));
 
 		return $view;
 	}
@@ -314,50 +314,50 @@ 
 	 * @param \Aimeos\MShop\Attribute\Item\Iface $item Attribute item object without referenced domain items
 	 * @param string[] $data Data array
 	 */
-	protected function fromArray( \Aimeos\MShop\Attribute\Item\Iface $item, array $data )
+	protected function fromArray(\Aimeos\MShop\Attribute\Item\Iface $item, array $data)
 	{
 		$context = $this->getContext();
 
-		$textManager = \Aimeos\MShop\Factory::createManager( $context, 'text' );
-		$listManager = \Aimeos\MShop\Factory::createManager( $context, 'attribute/lists' );
+		$textManager = \Aimeos\MShop\Factory::createManager($context, 'text');
+		$listManager = \Aimeos\MShop\Factory::createManager($context, 'attribute/lists');
 
-		$listItems = $item->getListItems( 'text', null, null, false );
+		$listItems = $item->getListItems('text', null, null, false);
 
 
-		foreach( $data as $idx => $entry )
+		foreach ($data as $idx => $entry)
 		{
-			if( trim( $this->getValue( $entry, 'text.content', '' ) ) === '' ) {
+			if (trim($this->getValue($entry, 'text.content', '')) === '') {
 				continue;
 			}
 
-			if( ( $listItem = $item->getListItem( 'text', $entry['attribute.lists.type'], $entry['text.id'], false ) ) === null ) {
+			if (($listItem = $item->getListItem('text', $entry['attribute.lists.type'], $entry['text.id'], false)) === null) {
 				$listItem = $listManager->createItem();
 			}
 
-			if( ( $refItem = $listItem->getRefItem() ) === null ) {
+			if (($refItem = $listItem->getRefItem()) === null) {
 				$refItem = $textManager->createItem();
 			}
 
-			$refItem->fromArray( $entry );
+			$refItem->fromArray($entry);
 			$conf = [];
 
-			foreach( (array) $this->getValue( $entry, 'config/key' ) as $num => $key )
+			foreach ((array) $this->getValue($entry, 'config/key') as $num => $key)
 			{
-				if( trim( $key ) !== '' && ( $val = $this->getValue( $entry, 'config/val/' . $num ) ) !== null ) {
-					$conf[$key] = trim( $val );
+				if (trim($key) !== '' && ($val = $this->getValue($entry, 'config/val/' . $num)) !== null) {
+					$conf[$key] = trim($val);
 				}
 			}
 
-			$listItem->fromArray( $entry );
-			$listItem->setPosition( $idx );
-			$listItem->setConfig( $conf );
+			$listItem->fromArray($entry);
+			$listItem->setPosition($idx);
+			$listItem->setConfig($conf);
 
-			$item->addListItem( 'text', $listItem, $refItem );
+			$item->addListItem('text', $listItem, $refItem);
 
-			unset( $listItems[$listItem->getId()] );
+			unset($listItems[$listItem->getId()]);
 		}
 
-		return $item->deleteListItems( $listItems, true );
+		return $item->deleteListItems($listItems, true);
 	}
 
 
@@ -368,20 +368,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\Attribute\Item\Iface $item, $copy = false )
+	protected function toArray(\Aimeos\MShop\Attribute\Item\Iface $item, $copy = false)
 	{
 		$data = [];
 		$siteId = $this->getContext()->getLocale()->getSiteId();
 
-		foreach( $item->getListItems( 'text', null, null, false ) as $listItem )
+		foreach ($item->getListItems('text', null, 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['attribute.lists.siteid'] = $siteId;
 				$list['attribute.lists.id'] = '';
@@ -389,10 +389,10 @@ 
 				$list['text.id'] = null;
 			}
 
-			$list['attribute.lists.datestart'] = str_replace( ' ', 'T', $list['attribute.lists.datestart'] );
-			$list['attribute.lists.dateend'] = str_replace( ' ', 'T', $list['attribute.lists.dateend'] );
+			$list['attribute.lists.datestart'] = str_replace(' ', 'T', $list['attribute.lists.datestart']);
+			$list['attribute.lists.dateend'] = str_replace(' ', 'T', $list['attribute.lists.dateend']);
 
-			foreach( $list['attribute.lists.config'] as $key => $val )
+			foreach ($list['attribute.lists.config'] as $key => $val)
 			{
 				$list['config']['key'][] = $key;
 				$list['config']['val'][] = $val;
@@ -411,7 +411,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/attribute/text/template-item
 		 * Relative path to the HTML body template of the text subpart for attributes.
@@ -435,6 +435,6 @@ 
 		$tplconf = 'admin/jqadm/attribute/text/template-item';
 		$default = 'attribute/item-text-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/Text/Standard.php

Braces +2 added lines, -4 removed lines

@@ -160,14 +160,12 @@
 			}
 
 			return;
-		}
-		catch( \Aimeos\MShop\Exception $e )
+		} catch( \Aimeos\MShop\Exception $e )
 		{
 			$error = array( 'product-item-text' => $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-text' => $e->getMessage() . ', ' . $e->getFile() . ':' . $e->getLine() );
 			$view->errors = $view->get( 'errors', [] ) + $error;

Spacing +67 added lines, -67 removed lines

@@ -10,7 +10,7 @@ 
 
 namespace Aimeos\Admin\JQAdm\Product\Text;
 
-sprintf( 'text' ); // for translation
+sprintf('text'); // for translation
 
 
 /**
@@ -42,16 +42,16 @@ 
 	 */
 	public function copy()
 	{
-		$view = $this->addViewData( $this->getView() );
+		$view = $this->addViewData($this->getView());
 
-		$view->textData = $this->toArray( $view->item, true );
+		$view->textData = $this->toArray($view->item, true);
 		$view->textBody = '';
 
-		foreach( $this->getSubClients() as $client ) {
+		foreach ($this->getSubClients() as $client) {
 			$view->textBody .= $client->copy();
 		}
 
-		return $this->render( $view );
+		return $this->render($view);
 	}
 
 
@@ -62,11 +62,11 @@ 
 	 */
 	public function create()
 	{
-		$view = $this->addViewData( $this->getView() );
+		$view = $this->addViewData($this->getView());
 		$siteid = $this->getContext()->getLocale()->getSiteId();
-		$data = $view->param( 'text', [] );
+		$data = $view->param('text', []);
 
-		foreach( $data as $idx => $entry )
+		foreach ($data as $idx => $entry)
 		{
 			$data[$idx]['product.lists.siteid'] = $siteid;
 			$data[$idx]['text.siteid'] = $siteid;
@@ -75,11 +75,11 @@ 
 		$view->textData = $data;
 		$view->textBody = '';
 
-		foreach( $this->getSubClients() as $client ) {
+		foreach ($this->getSubClients() as $client) {
 			$view->textBody .= $client->create();
 		}
 
-		return $this->render( $view );
+		return $this->render($view);
 	}
 
 
@@ -91,7 +91,7 @@ 
 		parent::delete();
 
 		$item = $this->getView()->item;
-		$item->deleteListItems( $item->getListItems( 'text', null, null, false ), true );
+		$item->deleteListItems($item->getListItems('text', null, null, false), true);
 	}
 
 
@@ -102,16 +102,16 @@ 
 	 */
 	public function get()
 	{
-		$view = $this->addViewData( $this->getView() );
+		$view = $this->addViewData($this->getView());
 
-		$view->textData = $this->toArray( $view->item );
+		$view->textData = $this->toArray($view->item);
 		$view->textBody = '';
 
-		foreach( $this->getSubClients() as $client ) {
+		foreach ($this->getSubClients() as $client) {
 			$view->textBody .= $client->get();
 		}
 
-		return $this->render( $view );
+		return $this->render($view);
 	}
 
 
@@ -124,26 +124,26 @@ 
 
 		try
 		{
-			$view->item = $this->fromArray( $view->item, $view->param( 'text', [] ) );
+			$view->item = $this->fromArray($view->item, $view->param('text', []));
 			$view->textBody = '';
 
-			foreach( $this->getSubClients() as $client ) {
+			foreach ($this->getSubClients() as $client) {
 				$view->textBody .= $client->save();
 			}
 
 			return;
 		}
-		catch( \Aimeos\MShop\Exception $e )
+		catch (\Aimeos\MShop\Exception $e)
 		{
-			$error = array( 'product-item-text' => $this->getContext()->getI18n()->dt( 'mshop', $e->getMessage() ) );
-			$view->errors = $view->get( 'errors', [] ) + $error;
-			$this->logException( $e );
+			$error = array('product-item-text' => $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-text' => $e->getMessage() . ', ' . $e->getFile() . ':' . $e->getLine() );
-			$view->errors = $view->get( 'errors', [] ) + $error;
-			$this->logException( $e );
+			$error = array('product-item-text' => $e->getMessage() . ', ' . $e->getFile() . ':' . $e->getLine());
+			$view->errors = $view->get('errors', []) + $error;
+			$this->logException($e);
 		}
 
 		throw new \Aimeos\Admin\JQAdm\Exception();
@@ -157,7 +157,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/text/decorators/excludes
 		 * Excludes decorators added by the "common" option from the product JQAdm client
@@ -232,7 +232,7 @@ 
 		 * @see admin/jqadm/product/text/decorators/excludes
 		 * @see admin/jqadm/product/text/decorators/global
 		 */
-		return $this->createSubClient( 'product/text/' . $type, $name );
+		return $this->createSubClient('product/text/' . $type, $name);
 	}
 
 
@@ -276,7 +276,7 @@ 
 		 * @since 2017.07
 		 * @category Developer
 		 */
-		return $this->getContext()->getConfig()->get( 'admin/jqadm/product/text/standard/subparts', [] );
+		return $this->getContext()->getConfig()->get('admin/jqadm/product/text/standard/subparts', []);
 	}
 
 
@@ -286,23 +286,23 @@ 
 	 * @param \Aimeos\MW\View\Iface $view View object
 	 * @return \Aimeos\MW\View\Iface View object with assigned parameters
 	 */
-	protected function addViewData( \Aimeos\MW\View\Iface $view )
+	protected function addViewData(\Aimeos\MW\View\Iface $view)
 	{
 		$context = $this->getContext();
 
-		$textTypeManager = \Aimeos\MShop\Factory::createManager( $context, 'text/type' );
-		$listTypeManager = \Aimeos\MShop\Factory::createManager( $context, 'product/lists/type' );
+		$textTypeManager = \Aimeos\MShop\Factory::createManager($context, 'text/type');
+		$listTypeManager = \Aimeos\MShop\Factory::createManager($context, 'product/lists/type');
 
-		$search = $textTypeManager->createSearch( true )->setSlice( 0, 0x7fffffff );
-		$search->setConditions( $search->compare( '==', 'text.type.domain', 'product' ) );
-		$search->setSortations( array( $search->sort( '+', 'text.type.label' ) ) );
+		$search = $textTypeManager->createSearch(true)->setSlice(0, 0x7fffffff);
+		$search->setConditions($search->compare('==', 'text.type.domain', 'product'));
+		$search->setSortations(array($search->sort('+', 'text.type.label')));
 
-		$listSearch = $listTypeManager->createSearch( true )->setSlice( 0, 0x7fffffff );
-		$listSearch->setConditions( $listSearch->compare( '==', 'product.lists.type.domain', 'text' ) );
-		$listSearch->setSortations( array( $listSearch->sort( '+', 'product.lists.type.label' ) ) );
+		$listSearch = $listTypeManager->createSearch(true)->setSlice(0, 0x7fffffff);
+		$listSearch->setConditions($listSearch->compare('==', 'product.lists.type.domain', 'text'));
+		$listSearch->setSortations(array($listSearch->sort('+', 'product.lists.type.label')));
 
-		$view->textTypes = $textTypeManager->searchItems( $search );
-		$view->textListTypes = $this->sortType( $listTypeManager->searchItems( $listSearch ) );
+		$view->textTypes = $textTypeManager->searchItems($search);
+		$view->textListTypes = $this->sortType($listTypeManager->searchItems($listSearch));
 
 		return $view;
 	}
@@ -314,52 +314,52 @@ 
 	 * @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();
 
-		$textManager = \Aimeos\MShop\Factory::createManager( $context, 'text' );
-		$listManager = \Aimeos\MShop\Factory::createManager( $context, 'product/lists' );
+		$textManager = \Aimeos\MShop\Factory::createManager($context, 'text');
+		$listManager = \Aimeos\MShop\Factory::createManager($context, 'product/lists');
 
-		$listItems = $item->getListItems( 'text', null, null, false );
+		$listItems = $item->getListItems('text', null, null, false);
 
 
-		foreach( $data as $idx => $entry )
+		foreach ($data as $idx => $entry)
 		{
-			if( trim( $this->getValue( $entry, 'text.content', '' ) ) === '' ) {
+			if (trim($this->getValue($entry, 'text.content', '')) === '') {
 				continue;
 			}
 
 			$listType = $entry['product.lists.type'];
 
-			if( ( $listItem = $item->getListItem( 'text', $listType, $entry['text.id'], false ) ) === null ) {
+			if (($listItem = $item->getListItem('text', $listType, $entry['text.id'], false)) === null) {
 				$listItem = $listManager->createItem();
 			}
 
-			if( ( $refItem = $listItem->getRefItem() ) === null ) {
+			if (($refItem = $listItem->getRefItem()) === null) {
 				$refItem = $textManager->createItem();
 			}
 
-			$refItem->fromArray( $entry );
+			$refItem->fromArray($entry);
 			$conf = [];
 
-			foreach( (array) $this->getValue( $entry, 'config/key' ) as $num => $key )
+			foreach ((array) $this->getValue($entry, 'config/key') as $num => $key)
 			{
-				if( trim( $key ) !== '' && ( $val = $this->getValue( $entry, 'config/val/' . $num ) ) !== null ) {
-					$conf[$key] = trim( $val );
+				if (trim($key) !== '' && ($val = $this->getValue($entry, 'config/val/' . $num)) !== null) {
+					$conf[$key] = trim($val);
 				}
 			}
 
-			$listItem->fromArray( $entry );
-			$listItem->setPosition( $idx );
-			$listItem->setConfig( $conf );
+			$listItem->fromArray($entry);
+			$listItem->setPosition($idx);
+			$listItem->setConfig($conf);
 
-			$item->addListItem( 'text', $listItem, $refItem );
+			$item->addListItem('text', $listItem, $refItem);
 
-			unset( $listItems[$listItem->getId()] );
+			unset($listItems[$listItem->getId()]);
 		}
 
-		return $item->deleteListItems( $listItems, true );
+		return $item->deleteListItems($listItems, true);
 	}
 
 
@@ -370,20 +370,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 = [];
 		$siteId = $this->getContext()->getLocale()->getSiteId();
 
-		foreach( $item->getListItems( 'text', null, null, false ) as $listItem )
+		foreach ($item->getListItems('text', null, 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'] = '';
@@ -391,10 +391,10 @@ 
 				$list['text.id'] = null;
 			}
 
-			$list['product.lists.datestart'] = str_replace( ' ', 'T', $list['product.lists.datestart'] );
-			$list['product.lists.dateend'] = str_replace( ' ', 'T', $list['product.lists.dateend'] );
+			$list['product.lists.datestart'] = str_replace(' ', 'T', $list['product.lists.datestart']);
+			$list['product.lists.dateend'] = str_replace(' ', 'T', $list['product.lists.dateend']);
 
-			foreach( $list['product.lists.config'] as $key => $val )
+			foreach ($list['product.lists.config'] as $key => $val)
 			{
 				$list['config']['key'][] = $key;
 				$list['config']['val'][] = $val;
@@ -413,7 +413,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/text/template-item
 		 * Relative path to the HTML body template of the text subpart for products.
@@ -437,6 +437,6 @@ 
 		$tplconf = 'admin/jqadm/product/text/template-item';
 		$default = 'product/item-text-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/Service/Text/Standard.php

Braces +2 added lines, -4 removed lines

@@ -160,14 +160,12 @@
 			}
 
 			return;
-		}
-		catch( \Aimeos\MShop\Exception $e )
+		} catch( \Aimeos\MShop\Exception $e )
 		{
 			$error = array( 'service-item-text' => $this->getContext()->getI18n()->dt( 'mshop', $e->getMessage() ) );
 			$view->errors = $view->get( 'errors', [] ) + $error;
 			$this->logException( $e );
-		}
-		catch( \Exception $e )
+		} catch( \Exception $e )
 		{
 			$error = array( 'service-item-text' => $e->getMessage() . ', ' . $e->getFile() . ':' . $e->getLine() );
 			$view->errors = $view->get( 'errors', [] ) + $error;

Spacing +67 added lines, -67 removed lines

@@ -10,7 +10,7 @@ 
 
 namespace Aimeos\Admin\JQAdm\Service\Text;
 
-sprintf( 'text' ); // for translation
+sprintf('text'); // for translation
 
 
 /**
@@ -42,16 +42,16 @@ 
 	 */
 	public function copy()
 	{
-		$view = $this->addViewData( $this->getView() );
+		$view = $this->addViewData($this->getView());
 
-		$view->textData = $this->toArray( $view->item, true );
+		$view->textData = $this->toArray($view->item, true);
 		$view->textBody = '';
 
-		foreach( $this->getSubClients() as $client ) {
+		foreach ($this->getSubClients() as $client) {
 			$view->textBody .= $client->copy();
 		}
 
-		return $this->render( $view );
+		return $this->render($view);
 	}
 
 
@@ -62,11 +62,11 @@ 
 	 */
 	public function create()
 	{
-		$view = $this->addViewData( $this->getView() );
+		$view = $this->addViewData($this->getView());
 		$siteid = $this->getContext()->getLocale()->getSiteId();
-		$data = $view->param( 'text', [] );
+		$data = $view->param('text', []);
 
-		foreach( $data as $idx => $entry )
+		foreach ($data as $idx => $entry)
 		{
 			$data[$idx]['service.lists.siteid'] = $siteid;
 			$data[$idx]['text.siteid'] = $siteid;
@@ -75,11 +75,11 @@ 
 		$view->textData = $data;
 		$view->textBody = '';
 
-		foreach( $this->getSubClients() as $client ) {
+		foreach ($this->getSubClients() as $client) {
 			$view->textBody .= $client->create();
 		}
 
-		return $this->render( $view );
+		return $this->render($view);
 	}
 
 
@@ -91,7 +91,7 @@ 
 		parent::delete();
 
 		$item = $this->getView()->item;
-		$item->deleteListItems( $item->getListItems( 'text', null, null, false ), true );
+		$item->deleteListItems($item->getListItems('text', null, null, false), true);
 	}
 
 
@@ -102,16 +102,16 @@ 
 	 */
 	public function get()
 	{
-		$view = $this->addViewData( $this->getView() );
+		$view = $this->addViewData($this->getView());
 
-		$view->textData = $this->toArray( $view->item );
+		$view->textData = $this->toArray($view->item);
 		$view->textBody = '';
 
-		foreach( $this->getSubClients() as $client ) {
+		foreach ($this->getSubClients() as $client) {
 			$view->textBody .= $client->get();
 		}
 
-		return $this->render( $view );
+		return $this->render($view);
 	}
 
 
@@ -124,26 +124,26 @@ 
 
 		try
 		{
-			$view->item = $this->fromArray( $view->item, $view->param( 'text', [] ) );
+			$view->item = $this->fromArray($view->item, $view->param('text', []));
 			$view->textBody = '';
 
-			foreach( $this->getSubClients() as $client ) {
+			foreach ($this->getSubClients() as $client) {
 				$view->textBody .= $client->save();
 			}
 
 			return;
 		}
-		catch( \Aimeos\MShop\Exception $e )
+		catch (\Aimeos\MShop\Exception $e)
 		{
-			$error = array( 'service-item-text' => $this->getContext()->getI18n()->dt( 'mshop', $e->getMessage() ) );
-			$view->errors = $view->get( 'errors', [] ) + $error;
-			$this->logException( $e );
+			$error = array('service-item-text' => $this->getContext()->getI18n()->dt('mshop', $e->getMessage()));
+			$view->errors = $view->get('errors', []) + $error;
+			$this->logException($e);
 		}
-		catch( \Exception $e )
+		catch (\Exception $e)
 		{
-			$error = array( 'service-item-text' => $e->getMessage() . ', ' . $e->getFile() . ':' . $e->getLine() );
-			$view->errors = $view->get( 'errors', [] ) + $error;
-			$this->logException( $e );
+			$error = array('service-item-text' => $e->getMessage() . ', ' . $e->getFile() . ':' . $e->getLine());
+			$view->errors = $view->get('errors', []) + $error;
+			$this->logException($e);
 		}
 
 		throw new \Aimeos\Admin\JQAdm\Exception();
@@ -157,7 +157,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/service/text/decorators/excludes
 		 * Excludes decorators added by the "common" option from the service JQAdm client
@@ -232,7 +232,7 @@ 
 		 * @see admin/jqadm/service/text/decorators/excludes
 		 * @see admin/jqadm/service/text/decorators/global
 		 */
-		return $this->createSubClient( 'service/text/' . $type, $name );
+		return $this->createSubClient('service/text/' . $type, $name);
 	}
 
 
@@ -276,7 +276,7 @@ 
 		 * @since 2017.07
 		 * @category Developer
 		 */
-		return $this->getContext()->getConfig()->get( 'admin/jqadm/service/text/standard/subparts', [] );
+		return $this->getContext()->getConfig()->get('admin/jqadm/service/text/standard/subparts', []);
 	}
 
 
@@ -286,23 +286,23 @@ 
 	 * @param \Aimeos\MW\View\Iface $view View object
 	 * @return \Aimeos\MW\View\Iface View object with assigned parameters
 	 */
-	protected function addViewData( \Aimeos\MW\View\Iface $view )
+	protected function addViewData(\Aimeos\MW\View\Iface $view)
 	{
 		$context = $this->getContext();
 
-		$textTypeManager = \Aimeos\MShop\Factory::createManager( $context, 'text/type' );
-		$listTypeManager = \Aimeos\MShop\Factory::createManager( $context, 'service/lists/type' );
+		$textTypeManager = \Aimeos\MShop\Factory::createManager($context, 'text/type');
+		$listTypeManager = \Aimeos\MShop\Factory::createManager($context, 'service/lists/type');
 
-		$search = $textTypeManager->createSearch( true )->setSlice( 0, 0x7fffffff );
-		$search->setConditions( $search->compare( '==', 'text.type.domain', 'service' ) );
-		$search->setSortations( array( $search->sort( '+', 'text.type.label' ) ) );
+		$search = $textTypeManager->createSearch(true)->setSlice(0, 0x7fffffff);
+		$search->setConditions($search->compare('==', 'text.type.domain', 'service'));
+		$search->setSortations(array($search->sort('+', 'text.type.label')));
 
-		$listSearch = $listTypeManager->createSearch( true )->setSlice( 0, 0x7fffffff );
-		$listSearch->setConditions( $listSearch->compare( '==', 'service.lists.type.domain', 'text' ) );
-		$listSearch->setSortations( array( $listSearch->sort( '+', 'service.lists.type.label' ) ) );
+		$listSearch = $listTypeManager->createSearch(true)->setSlice(0, 0x7fffffff);
+		$listSearch->setConditions($listSearch->compare('==', 'service.lists.type.domain', 'text'));
+		$listSearch->setSortations(array($listSearch->sort('+', 'service.lists.type.label')));
 
-		$view->textTypes = $textTypeManager->searchItems( $search );
-		$view->textListTypes = $this->sortType( $listTypeManager->searchItems( $listSearch ) );
+		$view->textTypes = $textTypeManager->searchItems($search);
+		$view->textListTypes = $this->sortType($listTypeManager->searchItems($listSearch));
 
 		return $view;
 	}
@@ -314,50 +314,50 @@ 
 	 * @param \Aimeos\MShop\Service\Item\Iface $item Service item object without referenced domain items
 	 * @param string[] $data Data array
 	 */
-	protected function fromArray( \Aimeos\MShop\Service\Item\Iface $item, array $data )
+	protected function fromArray(\Aimeos\MShop\Service\Item\Iface $item, array $data)
 	{
 		$context = $this->getContext();
 
-		$textManager = \Aimeos\MShop\Factory::createManager( $context, 'text' );
-		$listManager = \Aimeos\MShop\Factory::createManager( $context, 'service/lists' );
+		$textManager = \Aimeos\MShop\Factory::createManager($context, 'text');
+		$listManager = \Aimeos\MShop\Factory::createManager($context, 'service/lists');
 
-		$listItems = $item->getListItems( 'text', null, null, false );
+		$listItems = $item->getListItems('text', null, null, false);
 
 
-		foreach( $data as $idx => $entry )
+		foreach ($data as $idx => $entry)
 		{
-			if( trim( $this->getValue( $entry, 'text.content', '' ) ) === '' ) {
+			if (trim($this->getValue($entry, 'text.content', '')) === '') {
 				continue;
 			}
 
-			if( ( $listItem = $item->getListItem( 'text', $entry['service.lists.type'], $entry['text.id'], false ) ) === null ) {
+			if (($listItem = $item->getListItem('text', $entry['service.lists.type'], $entry['text.id'], false)) === null) {
 				$listItem = $listManager->createItem();
 			}
 
-			if( ( $refItem = $listItem->getRefItem() ) === null ) {
+			if (($refItem = $listItem->getRefItem()) === null) {
 				$refItem = $textManager->createItem();
 			}
 
-			$refItem->fromArray( $entry );
+			$refItem->fromArray($entry);
 			$conf = [];
 
-			foreach( (array) $this->getValue( $entry, 'config/key' ) as $num => $key )
+			foreach ((array) $this->getValue($entry, 'config/key') as $num => $key)
 			{
-				if( trim( $key ) !== '' && ( $val = $this->getValue( $entry, 'config/val/' . $num ) ) !== null ) {
-					$conf[$key] = trim( $val );
+				if (trim($key) !== '' && ($val = $this->getValue($entry, 'config/val/' . $num)) !== null) {
+					$conf[$key] = trim($val);
 				}
 			}
 
-			$listItem->fromArray( $entry );
-			$listItem->setPosition( $idx );
-			$listItem->setConfig( $conf );
+			$listItem->fromArray($entry);
+			$listItem->setPosition($idx);
+			$listItem->setConfig($conf);
 
-			$item->addListItem( 'text', $listItem, $refItem );
+			$item->addListItem('text', $listItem, $refItem);
 
-			unset( $listItems[$listItem->getId()] );
+			unset($listItems[$listItem->getId()]);
 		}
 
-		return $item->deleteListItems( $listItems, true );
+		return $item->deleteListItems($listItems, true);
 	}
 
 
@@ -368,20 +368,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\Service\Item\Iface $item, $copy = false )
+	protected function toArray(\Aimeos\MShop\Service\Item\Iface $item, $copy = false)
 	{
 		$data = [];
 		$siteId = $this->getContext()->getLocale()->getSiteId();
 
-		foreach( $item->getListItems( 'text', null, null, false ) as $listItem )
+		foreach ($item->getListItems('text', null, 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['service.lists.siteid'] = $siteId;
 				$list['service.lists.id'] = '';
@@ -389,10 +389,10 @@ 
 				$list['text.id'] = null;
 			}
 
-			$list['service.lists.datestart'] = str_replace( ' ', 'T', $list['service.lists.datestart'] );
-			$list['service.lists.dateend'] = str_replace( ' ', 'T', $list['service.lists.dateend'] );
+			$list['service.lists.datestart'] = str_replace(' ', 'T', $list['service.lists.datestart']);
+			$list['service.lists.dateend'] = str_replace(' ', 'T', $list['service.lists.dateend']);
 
-			foreach( $list['service.lists.config'] as $key => $val )
+			foreach ($list['service.lists.config'] as $key => $val)
 			{
 				$list['config']['key'][] = $key;
 				$list['config']['val'][] = $val;
@@ -411,7 +411,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/service/text/template-item
 		 * Relative path to the HTML body template of the text subpart for services.
@@ -435,6 +435,6 @@ 
 		$tplconf = 'admin/jqadm/service/text/template-item';
 		$default = 'service/item-text-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/Catalog/Text/Standard.php

Braces +2 added lines, -4 removed lines

@@ -160,14 +160,12 @@
 			}
 
 			return;
-		}
-		catch( \Aimeos\MShop\Exception $e )
+		} catch( \Aimeos\MShop\Exception $e )
 		{
 			$error = array( 'catalog-item-text' => $this->getContext()->getI18n()->dt( 'mshop', $e->getMessage() ) );
 			$view->errors = $view->get( 'errors', [] ) + $error;
 			$this->logException( $e );
-		}
-		catch( \Exception $e )
+		} catch( \Exception $e )
 		{
 			$error = array( 'catalog-item-text' => $e->getMessage() . ', ' . $e->getFile() . ':' . $e->getLine() );
 			$view->errors = $view->get( 'errors', [] ) + $error;

Spacing +67 added lines, -67 removed lines

@@ -10,7 +10,7 @@ 
 
 namespace Aimeos\Admin\JQAdm\Catalog\Text;
 
-sprintf( 'text' ); // for translation
+sprintf('text'); // for translation
 
 
 /**
@@ -42,16 +42,16 @@ 
 	 */
 	public function copy()
 	{
-		$view = $this->addViewData( $this->getView() );
+		$view = $this->addViewData($this->getView());
 
-		$view->textData = $this->toArray( $view->item, true );
+		$view->textData = $this->toArray($view->item, true);
 		$view->textBody = '';
 
-		foreach( $this->getSubClients() as $client ) {
+		foreach ($this->getSubClients() as $client) {
 			$view->textBody .= $client->copy();
 		}
 
-		return $this->render( $view );
+		return $this->render($view);
 	}
 
 
@@ -62,11 +62,11 @@ 
 	 */
 	public function create()
 	{
-		$view = $this->addViewData( $this->getView() );
+		$view = $this->addViewData($this->getView());
 		$siteid = $this->getContext()->getLocale()->getSiteId();
-		$data = $view->param( 'text', [] );
+		$data = $view->param('text', []);
 
-		foreach( $data as $idx => $entry )
+		foreach ($data as $idx => $entry)
 		{
 			$data[$idx]['catalog.lists.siteid'] = $siteid;
 			$data[$idx]['text.siteid'] = $siteid;
@@ -75,11 +75,11 @@ 
 		$view->textData = $data;
 		$view->textBody = '';
 
-		foreach( $this->getSubClients() as $client ) {
+		foreach ($this->getSubClients() as $client) {
 			$view->textBody .= $client->create();
 		}
 
-		return $this->render( $view );
+		return $this->render($view);
 	}
 
 
@@ -91,7 +91,7 @@ 
 		parent::delete();
 
 		$item = $this->getView()->item;
-		$item->deleteListItems( $item->getListItems( 'text', null, null, false ), true );
+		$item->deleteListItems($item->getListItems('text', null, null, false), true);
 	}
 
 
@@ -102,16 +102,16 @@ 
 	 */
 	public function get()
 	{
-		$view = $this->addViewData( $this->getView() );
+		$view = $this->addViewData($this->getView());
 
-		$view->textData = $this->toArray( $view->item );
+		$view->textData = $this->toArray($view->item);
 		$view->textBody = '';
 
-		foreach( $this->getSubClients() as $client ) {
+		foreach ($this->getSubClients() as $client) {
 			$view->textBody .= $client->get();
 		}
 
-		return $this->render( $view );
+		return $this->render($view);
 	}
 
 
@@ -124,26 +124,26 @@ 
 
 		try
 		{
-			$view->item = $this->fromArray( $view->item, $view->param( 'text', [] ) );
+			$view->item = $this->fromArray($view->item, $view->param('text', []));
 			$view->textBody = '';
 
-			foreach( $this->getSubClients() as $client ) {
+			foreach ($this->getSubClients() as $client) {
 				$view->textBody .= $client->save();
 			}
 
 			return;
 		}
-		catch( \Aimeos\MShop\Exception $e )
+		catch (\Aimeos\MShop\Exception $e)
 		{
-			$error = array( 'catalog-item-text' => $this->getContext()->getI18n()->dt( 'mshop', $e->getMessage() ) );
-			$view->errors = $view->get( 'errors', [] ) + $error;
-			$this->logException( $e );
+			$error = array('catalog-item-text' => $this->getContext()->getI18n()->dt('mshop', $e->getMessage()));
+			$view->errors = $view->get('errors', []) + $error;
+			$this->logException($e);
 		}
-		catch( \Exception $e )
+		catch (\Exception $e)
 		{
-			$error = array( 'catalog-item-text' => $e->getMessage() . ', ' . $e->getFile() . ':' . $e->getLine() );
-			$view->errors = $view->get( 'errors', [] ) + $error;
-			$this->logException( $e );
+			$error = array('catalog-item-text' => $e->getMessage() . ', ' . $e->getFile() . ':' . $e->getLine());
+			$view->errors = $view->get('errors', []) + $error;
+			$this->logException($e);
 		}
 
 		throw new \Aimeos\Admin\JQAdm\Exception();
@@ -157,7 +157,7 @@ 
 	 * @param string|null $name Name of the sub-client (Default if null)
 	 * @return \Aimeos\Admin\JQAdm\Iface Sub-client object
 	 */
-	public function getSubClient( $type, $name = null )
+	public function getSubClient($type, $name = null)
 	{
 		/** admin/jqadm/catalog/text/decorators/excludes
 		 * Excludes decorators added by the "common" option from the catalog JQAdm client
@@ -232,7 +232,7 @@ 
 		 * @see admin/jqadm/catalog/text/decorators/excludes
 		 * @see admin/jqadm/catalog/text/decorators/global
 		 */
-		return $this->createSubClient( 'catalog/text/' . $type, $name );
+		return $this->createSubClient('catalog/text/' . $type, $name);
 	}
 
 
@@ -276,7 +276,7 @@ 
 		 * @since 2017.07
 		 * @category Developer
 		 */
-		return $this->getContext()->getConfig()->get( 'admin/jqadm/catalog/text/standard/subparts', [] );
+		return $this->getContext()->getConfig()->get('admin/jqadm/catalog/text/standard/subparts', []);
 	}
 
 
@@ -286,23 +286,23 @@ 
 	 * @param \Aimeos\MW\View\Iface $view View object
 	 * @return \Aimeos\MW\View\Iface View object with assigned parameters
 	 */
-	protected function addViewData( \Aimeos\MW\View\Iface $view )
+	protected function addViewData(\Aimeos\MW\View\Iface $view)
 	{
 		$context = $this->getContext();
 
-		$textTypeManager = \Aimeos\MShop\Factory::createManager( $context, 'text/type' );
-		$listTypeManager = \Aimeos\MShop\Factory::createManager( $context, 'catalog/lists/type' );
+		$textTypeManager = \Aimeos\MShop\Factory::createManager($context, 'text/type');
+		$listTypeManager = \Aimeos\MShop\Factory::createManager($context, 'catalog/lists/type');
 
-		$search = $textTypeManager->createSearch( true )->setSlice( 0, 0x7fffffff );
-		$search->setConditions( $search->compare( '==', 'text.type.domain', 'catalog' ) );
-		$search->setSortations( array( $search->sort( '+', 'text.type.label' ) ) );
+		$search = $textTypeManager->createSearch(true)->setSlice(0, 0x7fffffff);
+		$search->setConditions($search->compare('==', 'text.type.domain', 'catalog'));
+		$search->setSortations(array($search->sort('+', 'text.type.label')));
 
-		$listSearch = $listTypeManager->createSearch( true )->setSlice( 0, 0x7fffffff );
-		$listSearch->setConditions( $listSearch->compare( '==', 'catalog.lists.type.domain', 'text' ) );
-		$listSearch->setSortations( array( $listSearch->sort( '+', 'catalog.lists.type.label' ) ) );
+		$listSearch = $listTypeManager->createSearch(true)->setSlice(0, 0x7fffffff);
+		$listSearch->setConditions($listSearch->compare('==', 'catalog.lists.type.domain', 'text'));
+		$listSearch->setSortations(array($listSearch->sort('+', 'catalog.lists.type.label')));
 
-		$view->textTypes = $textTypeManager->searchItems( $search );
-		$view->textListTypes = $this->sortType( $listTypeManager->searchItems( $listSearch ) );
+		$view->textTypes = $textTypeManager->searchItems($search);
+		$view->textListTypes = $this->sortType($listTypeManager->searchItems($listSearch));
 
 		return $view;
 	}
@@ -314,50 +314,50 @@ 
 	 * @param \Aimeos\MShop\Catalog\Item\Iface $item Catalog item object without referenced domain items
 	 * @param string[] $data Data array
 	 */
-	protected function fromArray( \Aimeos\MShop\Catalog\Item\Iface $item, array $data )
+	protected function fromArray(\Aimeos\MShop\Catalog\Item\Iface $item, array $data)
 	{
 		$context = $this->getContext();
 
-		$textManager = \Aimeos\MShop\Factory::createManager( $context, 'text' );
-		$listManager = \Aimeos\MShop\Factory::createManager( $context, 'catalog/lists' );
+		$textManager = \Aimeos\MShop\Factory::createManager($context, 'text');
+		$listManager = \Aimeos\MShop\Factory::createManager($context, 'catalog/lists');
 
-		$listItems = $item->getListItems( 'text', null, null, false );
+		$listItems = $item->getListItems('text', null, null, false);
 
 
-		foreach( $data as $idx => $entry )
+		foreach ($data as $idx => $entry)
 		{
-			if( trim( $this->getValue( $entry, 'text.content', '' ) ) === '' ) {
+			if (trim($this->getValue($entry, 'text.content', '')) === '') {
 				continue;
 			}
 
-			if( ( $listItem = $item->getListItem( 'text', $entry['catalog.lists.type'], $entry['text.id'], false ) ) === null ) {
+			if (($listItem = $item->getListItem('text', $entry['catalog.lists.type'], $entry['text.id'], false)) === null) {
 				$listItem = $listManager->createItem();
 			}
 
-			if( ( $refItem = $listItem->getRefItem() ) === null ) {
+			if (($refItem = $listItem->getRefItem()) === null) {
 				$refItem = $textManager->createItem();
 			}
 
-			$refItem->fromArray( $entry );
+			$refItem->fromArray($entry);
 			$conf = [];
 
-			foreach( (array) $this->getValue( $entry, 'config/key' ) as $num => $key )
+			foreach ((array) $this->getValue($entry, 'config/key') as $num => $key)
 			{
-				if( trim( $key ) !== '' && ( $val = $this->getValue( $entry, 'config/val/' . $num ) ) !== null ) {
-					$conf[$key] = trim( $val );
+				if (trim($key) !== '' && ($val = $this->getValue($entry, 'config/val/' . $num)) !== null) {
+					$conf[$key] = trim($val);
 				}
 			}
 
-			$listItem->fromArray( $entry );
-			$listItem->setPosition( $idx );
-			$listItem->setConfig( $conf );
+			$listItem->fromArray($entry);
+			$listItem->setPosition($idx);
+			$listItem->setConfig($conf);
 
-			$item->addListItem( 'text', $listItem, $refItem );
+			$item->addListItem('text', $listItem, $refItem);
 
-			unset( $listItems[$listItem->getId()] );
+			unset($listItems[$listItem->getId()]);
 		}
 
-		return $item->deleteListItems( $listItems, true );
+		return $item->deleteListItems($listItems, true);
 	}
 
 
@@ -368,20 +368,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\Catalog\Item\Iface $item, $copy = false )
+	protected function toArray(\Aimeos\MShop\Catalog\Item\Iface $item, $copy = false)
 	{
 		$data = [];
 		$siteId = $this->getContext()->getLocale()->getSiteId();
 
-		foreach( $item->getListItems( 'text', null, null, false ) as $listItem )
+		foreach ($item->getListItems('text', null, 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['catalog.lists.siteid'] = $siteId;
 				$list['catalog.lists.id'] = '';
@@ -389,10 +389,10 @@ 
 				$list['text.id'] = null;
 			}
 
-			$list['catalog.lists.datestart'] = str_replace( ' ', 'T', $list['catalog.lists.datestart'] );
-			$list['catalog.lists.dateend'] = str_replace( ' ', 'T', $list['catalog.lists.dateend'] );
+			$list['catalog.lists.datestart'] = str_replace(' ', 'T', $list['catalog.lists.datestart']);
+			$list['catalog.lists.dateend'] = str_replace(' ', 'T', $list['catalog.lists.dateend']);
 
-			foreach( $list['catalog.lists.config'] as $key => $val )
+			foreach ($list['catalog.lists.config'] as $key => $val)
 			{
 				$list['config']['key'][] = $key;
 				$list['config']['val'][] = $val;
@@ -411,7 +411,7 @@ 
 	 * @param \Aimeos\MW\View\Iface $view View object with data assigned
 	 * @return string HTML output
 	 */
-	protected function render( \Aimeos\MW\View\Iface $view )
+	protected function render(\Aimeos\MW\View\Iface $view)
 	{
 		/** admin/jqadm/catalog/text/template-item
 		 * Relative path to the HTML body template of the text subpart for catalogs.
@@ -435,6 +435,6 @@ 
 		$tplconf = 'admin/jqadm/catalog/text/template-item';
 		$default = 'catalog/item-text-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/Supplier/Text/Standard.php

Braces +2 added lines, -4 removed lines

@@ -160,14 +160,12 @@
 			}
 
 			return;
-		}
-		catch( \Aimeos\MShop\Exception $e )
+		} catch( \Aimeos\MShop\Exception $e )
 		{
 			$error = array( 'supplier-item-text' => $this->getContext()->getI18n()->dt( 'mshop', $e->getMessage() ) );
 			$view->errors = $view->get( 'errors', [] ) + $error;
 			$this->logException( $e );
-		}
-		catch( \Exception $e )
+		} catch( \Exception $e )
 		{
 			$error = array( 'supplier-item-text' => $e->getMessage() . ', ' . $e->getFile() . ':' . $e->getLine() );
 			$view->errors = $view->get( 'errors', [] ) + $error;

Spacing +67 added lines, -67 removed lines

@@ -10,7 +10,7 @@ 
 
 namespace Aimeos\Admin\JQAdm\Supplier\Text;
 
-sprintf( 'text' ); // for translation
+sprintf('text'); // for translation
 
 
 /**
@@ -42,16 +42,16 @@ 
 	 */
 	public function copy()
 	{
-		$view = $this->addViewData( $this->getView() );
+		$view = $this->addViewData($this->getView());
 
-		$view->textData = $this->toArray( $view->item, true );
+		$view->textData = $this->toArray($view->item, true);
 		$view->textBody = '';
 
-		foreach( $this->getSubClients() as $client ) {
+		foreach ($this->getSubClients() as $client) {
 			$view->textBody .= $client->copy();
 		}
 
-		return $this->render( $view );
+		return $this->render($view);
 	}
 
 
@@ -62,11 +62,11 @@ 
 	 */
 	public function create()
 	{
-		$view = $this->addViewData( $this->getView() );
+		$view = $this->addViewData($this->getView());
 		$siteid = $this->getContext()->getLocale()->getSiteId();
-		$data = $view->param( 'text', [] );
+		$data = $view->param('text', []);
 
-		foreach( $data as $idx => $entry )
+		foreach ($data as $idx => $entry)
 		{
 			$data[$idx]['supplier.lists.siteid'] = $siteid;
 			$data[$idx]['text.siteid'] = $siteid;
@@ -75,11 +75,11 @@ 
 		$view->textData = $data;
 		$view->textBody = '';
 
-		foreach( $this->getSubClients() as $client ) {
+		foreach ($this->getSubClients() as $client) {
 			$view->textBody .= $client->create();
 		}
 
-		return $this->render( $view );
+		return $this->render($view);
 	}
 
 
@@ -91,7 +91,7 @@ 
 		parent::delete();
 
 		$item = $this->getView()->item;
-		$item->deleteListItems( $item->getListItems( 'text', null, null, false ), true );
+		$item->deleteListItems($item->getListItems('text', null, null, false), true);
 	}
 
 
@@ -102,16 +102,16 @@ 
 	 */
 	public function get()
 	{
-		$view = $this->addViewData( $this->getView() );
+		$view = $this->addViewData($this->getView());
 
-		$view->textData = $this->toArray( $view->item );
+		$view->textData = $this->toArray($view->item);
 		$view->textBody = '';
 
-		foreach( $this->getSubClients() as $client ) {
+		foreach ($this->getSubClients() as $client) {
 			$view->textBody .= $client->get();
 		}
 
-		return $this->render( $view );
+		return $this->render($view);
 	}
 
 
@@ -124,26 +124,26 @@ 
 
 		try
 		{
-			$view->item = $this->fromArray( $view->item, $view->param( 'text', [] ) );
+			$view->item = $this->fromArray($view->item, $view->param('text', []));
 			$view->textBody = '';
 
-			foreach( $this->getSubClients() as $client ) {
+			foreach ($this->getSubClients() as $client) {
 				$view->textBody .= $client->save();
 			}
 
 			return;
 		}
-		catch( \Aimeos\MShop\Exception $e )
+		catch (\Aimeos\MShop\Exception $e)
 		{
-			$error = array( 'supplier-item-text' => $this->getContext()->getI18n()->dt( 'mshop', $e->getMessage() ) );
-			$view->errors = $view->get( 'errors', [] ) + $error;
-			$this->logException( $e );
+			$error = array('supplier-item-text' => $this->getContext()->getI18n()->dt('mshop', $e->getMessage()));
+			$view->errors = $view->get('errors', []) + $error;
+			$this->logException($e);
 		}
-		catch( \Exception $e )
+		catch (\Exception $e)
 		{
-			$error = array( 'supplier-item-text' => $e->getMessage() . ', ' . $e->getFile() . ':' . $e->getLine() );
-			$view->errors = $view->get( 'errors', [] ) + $error;
-			$this->logException( $e );
+			$error = array('supplier-item-text' => $e->getMessage() . ', ' . $e->getFile() . ':' . $e->getLine());
+			$view->errors = $view->get('errors', []) + $error;
+			$this->logException($e);
 		}
 
 		throw new \Aimeos\Admin\JQAdm\Exception();
@@ -157,7 +157,7 @@ 
 	 * @param string|null $name Name of the sub-client (Default if null)
 	 * @return \Aimeos\Admin\JQAdm\Iface Sub-client object
 	 */
-	public function getSubClient( $type, $name = null )
+	public function getSubClient($type, $name = null)
 	{
 		/** admin/jqadm/supplier/text/decorators/excludes
 		 * Excludes decorators added by the "common" option from the supplier JQAdm client
@@ -232,7 +232,7 @@ 
 		 * @see admin/jqadm/supplier/text/decorators/excludes
 		 * @see admin/jqadm/supplier/text/decorators/global
 		 */
-		return $this->createSubClient( 'supplier/text/' . $type, $name );
+		return $this->createSubClient('supplier/text/' . $type, $name);
 	}
 
 
@@ -276,7 +276,7 @@ 
 		 * @since 2018.07
 		 * @category Developer
 		 */
-		return $this->getContext()->getConfig()->get( 'admin/jqadm/supplier/text/standard/subparts', [] );
+		return $this->getContext()->getConfig()->get('admin/jqadm/supplier/text/standard/subparts', []);
 	}
 
 
@@ -286,23 +286,23 @@ 
 	 * @param \Aimeos\MW\View\Iface $view View object
 	 * @return \Aimeos\MW\View\Iface View object with assigned parameters
 	 */
-	protected function addViewData( \Aimeos\MW\View\Iface $view )
+	protected function addViewData(\Aimeos\MW\View\Iface $view)
 	{
 		$context = $this->getContext();
 
-		$textTypeManager = \Aimeos\MShop\Factory::createManager( $context, 'text/type' );
-		$listTypeManager = \Aimeos\MShop\Factory::createManager( $context, 'supplier/lists/type' );
+		$textTypeManager = \Aimeos\MShop\Factory::createManager($context, 'text/type');
+		$listTypeManager = \Aimeos\MShop\Factory::createManager($context, 'supplier/lists/type');
 
-		$search = $textTypeManager->createSearch( true )->setSlice( 0, 0x7fffffff );
-		$search->setConditions( $search->compare( '==', 'text.type.domain', 'supplier' ) );
-		$search->setSortations( array( $search->sort( '+', 'text.type.label' ) ) );
+		$search = $textTypeManager->createSearch(true)->setSlice(0, 0x7fffffff);
+		$search->setConditions($search->compare('==', 'text.type.domain', 'supplier'));
+		$search->setSortations(array($search->sort('+', 'text.type.label')));
 
-		$listSearch = $listTypeManager->createSearch( true )->setSlice( 0, 0x7fffffff );
-		$listSearch->setConditions( $listSearch->compare( '==', 'supplier.lists.type.domain', 'text' ) );
-		$listSearch->setSortations( array( $listSearch->sort( '+', 'supplier.lists.type.label' ) ) );
+		$listSearch = $listTypeManager->createSearch(true)->setSlice(0, 0x7fffffff);
+		$listSearch->setConditions($listSearch->compare('==', 'supplier.lists.type.domain', 'text'));
+		$listSearch->setSortations(array($listSearch->sort('+', 'supplier.lists.type.label')));
 
-		$view->textTypes = $textTypeManager->searchItems( $search );
-		$view->textListTypes = $this->sortType( $listTypeManager->searchItems( $listSearch ) );
+		$view->textTypes = $textTypeManager->searchItems($search);
+		$view->textListTypes = $this->sortType($listTypeManager->searchItems($listSearch));
 
 		return $view;
 	}
@@ -314,50 +314,50 @@ 
 	 * @param \Aimeos\MShop\Supplier\Item\Iface $item Supplier item object without referenced domain items
 	 * @param string[] $data Data array
 	 */
-	protected function fromArray( \Aimeos\MShop\Supplier\Item\Iface $item, array $data )
+	protected function fromArray(\Aimeos\MShop\Supplier\Item\Iface $item, array $data)
 	{
 		$context = $this->getContext();
 
-		$textManager = \Aimeos\MShop\Factory::createManager( $context, 'text' );
-		$listManager = \Aimeos\MShop\Factory::createManager( $context, 'supplier/lists' );
+		$textManager = \Aimeos\MShop\Factory::createManager($context, 'text');
+		$listManager = \Aimeos\MShop\Factory::createManager($context, 'supplier/lists');
 
-		$listItems = $item->getListItems( 'text', null, null, false );
+		$listItems = $item->getListItems('text', null, null, false);
 
 
-		foreach( $data as $idx => $entry )
+		foreach ($data as $idx => $entry)
 		{
-			if( trim( $this->getValue( $entry, 'text.content', '' ) ) === '' ) {
+			if (trim($this->getValue($entry, 'text.content', '')) === '') {
 				continue;
 			}
 
-			if( ( $listItem = $item->getListItem( 'text', $entry['supplier.lists.type'], $entry['text.id'], false ) ) === null ) {
+			if (($listItem = $item->getListItem('text', $entry['supplier.lists.type'], $entry['text.id'], false)) === null) {
 				$listItem = $listManager->createItem();
 			}
 
-			if( ( $refItem = $listItem->getRefItem() ) === null ) {
+			if (($refItem = $listItem->getRefItem()) === null) {
 				$refItem = $textManager->createItem();
 			}
 
-			$refItem->fromArray( $entry );
+			$refItem->fromArray($entry);
 			$conf = [];
 
-			foreach( (array) $this->getValue( $entry, 'config/key' ) as $num => $key )
+			foreach ((array) $this->getValue($entry, 'config/key') as $num => $key)
 			{
-				if( trim( $key ) !== '' && ( $val = $this->getValue( $entry, 'config/val/' . $num ) ) !== null ) {
-					$conf[$key] = trim( $val );
+				if (trim($key) !== '' && ($val = $this->getValue($entry, 'config/val/' . $num)) !== null) {
+					$conf[$key] = trim($val);
 				}
 			}
 
-			$listItem->fromArray( $entry );
-			$listItem->setPosition( $idx );
-			$listItem->setConfig( $conf );
+			$listItem->fromArray($entry);
+			$listItem->setPosition($idx);
+			$listItem->setConfig($conf);
 
-			$item->addListItem( 'text', $listItem, $refItem );
+			$item->addListItem('text', $listItem, $refItem);
 
-			unset( $listItems[$listItem->getId()] );
+			unset($listItems[$listItem->getId()]);
 		}
 
-		return $item->deleteListItems( $listItems, true );
+		return $item->deleteListItems($listItems, true);
 	}
 
 
@@ -368,20 +368,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\Supplier\Item\Iface $item, $copy = false )
+	protected function toArray(\Aimeos\MShop\Supplier\Item\Iface $item, $copy = false)
 	{
 		$data = [];
 		$siteId = $this->getContext()->getLocale()->getSiteId();
 
-		foreach( $item->getListItems( 'text', null, null, false ) as $listItem )
+		foreach ($item->getListItems('text', null, 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['supplier.lists.siteid'] = $siteId;
 				$list['supplier.lists.id'] = '';
@@ -389,10 +389,10 @@ 
 				$list['text.id'] = null;
 			}
 
-			$list['supplier.lists.datestart'] = str_replace( ' ', 'T', $list['supplier.lists.datestart'] );
-			$list['supplier.lists.dateend'] = str_replace( ' ', 'T', $list['supplier.lists.dateend'] );
+			$list['supplier.lists.datestart'] = str_replace(' ', 'T', $list['supplier.lists.datestart']);
+			$list['supplier.lists.dateend'] = str_replace(' ', 'T', $list['supplier.lists.dateend']);
 
-			foreach( $list['supplier.lists.config'] as $key => $val )
+			foreach ($list['supplier.lists.config'] as $key => $val)
 			{
 				$list['config']['key'][] = $key;
 				$list['config']['val'][] = $val;
@@ -411,7 +411,7 @@ 
 	 * @param \Aimeos\MW\View\Iface $view View object with data assigned
 	 * @return string HTML output
 	 */
-	protected function render( \Aimeos\MW\View\Iface $view )
+	protected function render(\Aimeos\MW\View\Iface $view)
 	{
 		/** admin/jqadm/supplier/text/template-item
 		 * Relative path to the HTML body template of the text subpart for suppliers.
@@ -435,6 +435,6 @@ 
 		$tplconf = 'admin/jqadm/supplier/text/template-item';
 		$default = 'supplier/item-text-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/Attribute/Price/Standard.php

Braces +2 added lines, -4 removed lines

@@ -160,14 +160,12 @@
 			}
 
 			return;
-		}
-		catch( \Aimeos\MShop\Exception $e )
+		} catch( \Aimeos\MShop\Exception $e )
 		{
 			$error = array( 'attribute-item-price' => $this->getContext()->getI18n()->dt( 'mshop', $e->getMessage() ) );
 			$view->errors = $view->get( 'errors', [] ) + $error;
 			$this->logException( $e );
-		}
-		catch( \Exception $e )
+		} catch( \Exception $e )
 		{
 			$error = array( 'attribute-item-price' => $e->getMessage() . ', ' . $e->getFile() . ':' . $e->getLine() );
 			$view->errors = $view->get( 'errors', [] ) + $error;

Spacing +70 added lines, -70 removed lines

@@ -10,7 +10,7 @@ 
 
 namespace Aimeos\Admin\JQAdm\Attribute\Price;
 
-sprintf( 'price' ); // for translation
+sprintf('price'); // for translation
 
 
 /**
@@ -42,16 +42,16 @@ 
 	 */
 	public function copy()
 	{
-		$view = $this->addViewData( $this->getView() );
+		$view = $this->addViewData($this->getView());
 
-		$view->priceData = $this->toArray( $view->item, true );
+		$view->priceData = $this->toArray($view->item, true);
 		$view->priceBody = '';
 
-		foreach( $this->getSubClients() as $client ) {
+		foreach ($this->getSubClients() as $client) {
 			$view->priceBody .= $client->copy();
 		}
 
-		return $this->render( $view );
+		return $this->render($view);
 	}
 
 
@@ -62,11 +62,11 @@ 
 	 */
 	public function create()
 	{
-		$view = $this->addViewData( $this->getView() );
+		$view = $this->addViewData($this->getView());
 		$siteid = $this->getContext()->getLocale()->getSiteId();
-		$data = $view->param( 'price', [] );
+		$data = $view->param('price', []);
 
-		foreach( $data as $idx => $entry )
+		foreach ($data as $idx => $entry)
 		{
 			$data[$idx]['attribute.lists.siteid'] = $siteid;
 			$data[$idx]['price.siteid'] = $siteid;
@@ -75,11 +75,11 @@ 
 		$view->priceData = $data;
 		$view->priceBody = '';
 
-		foreach( $this->getSubClients() as $client ) {
+		foreach ($this->getSubClients() as $client) {
 			$view->priceBody .= $client->create();
 		}
 
-		return $this->render( $view );
+		return $this->render($view);
 	}
 
 
@@ -91,7 +91,7 @@ 
 		parent::delete();
 
 		$item = $this->getView()->item;
-		$item->deleteListItems( $item->getListItems( 'price', null, null, false ), true );
+		$item->deleteListItems($item->getListItems('price', null, null, false), true);
 	}
 
 
@@ -102,16 +102,16 @@ 
 	 */
 	public function get()
 	{
-		$view = $this->addViewData( $this->getView() );
+		$view = $this->addViewData($this->getView());
 
-		$view->priceData = $this->toArray( $view->item );
+		$view->priceData = $this->toArray($view->item);
 		$view->priceBody = '';
 
-		foreach( $this->getSubClients() as $client ) {
+		foreach ($this->getSubClients() as $client) {
 			$view->priceBody .= $client->get();
 		}
 
-		return $this->render( $view );
+		return $this->render($view);
 	}
 
 
@@ -124,26 +124,26 @@ 
 
 		try
 		{
-			$view->item = $this->fromArray( $view->item, $view->param( 'price', [] ) );
+			$view->item = $this->fromArray($view->item, $view->param('price', []));
 			$view->priceBody = '';
 
-			foreach( $this->getSubClients() as $client ) {
+			foreach ($this->getSubClients() as $client) {
 				$view->priceBody .= $client->save();
 			}
 
 			return;
 		}
-		catch( \Aimeos\MShop\Exception $e )
+		catch (\Aimeos\MShop\Exception $e)
 		{
-			$error = array( 'attribute-item-price' => $this->getContext()->getI18n()->dt( 'mshop', $e->getMessage() ) );
-			$view->errors = $view->get( 'errors', [] ) + $error;
-			$this->logException( $e );
+			$error = array('attribute-item-price' => $this->getContext()->getI18n()->dt('mshop', $e->getMessage()));
+			$view->errors = $view->get('errors', []) + $error;
+			$this->logException($e);
 		}
-		catch( \Exception $e )
+		catch (\Exception $e)
 		{
-			$error = array( 'attribute-item-price' => $e->getMessage() . ', ' . $e->getFile() . ':' . $e->getLine() );
-			$view->errors = $view->get( 'errors', [] ) + $error;
-			$this->logException( $e );
+			$error = array('attribute-item-price' => $e->getMessage() . ', ' . $e->getFile() . ':' . $e->getLine());
+			$view->errors = $view->get('errors', []) + $error;
+			$this->logException($e);
 		}
 
 		throw new \Aimeos\Admin\JQAdm\Exception();
@@ -157,7 +157,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/attribute/price/decorators/excludes
 		 * Excludes decorators added by the "common" option from the attribute JQAdm client
@@ -232,7 +232,7 @@ 
 		 * @see admin/jqadm/attribute/price/decorators/excludes
 		 * @see admin/jqadm/attribute/price/decorators/global
 		 */
-		return $this->createSubClient( 'attribute/price/' . $type, $name );
+		return $this->createSubClient('attribute/price/' . $type, $name);
 	}
 
 
@@ -276,7 +276,7 @@ 
 		 * @since 2016.01
 		 * @category Developer
 		 */
-		return $this->getContext()->getConfig()->get( 'admin/jqadm/attribute/price/standard/subparts', [] );
+		return $this->getContext()->getConfig()->get('admin/jqadm/attribute/price/standard/subparts', []);
 	}
 
 
@@ -286,28 +286,28 @@ 
 	 * @param \Aimeos\MW\View\Iface $view View object
 	 * @return \Aimeos\MW\View\Iface View object with assigned parameters
 	 */
-	protected function addViewData( \Aimeos\MW\View\Iface $view )
+	protected function addViewData(\Aimeos\MW\View\Iface $view)
 	{
 		$context = $this->getContext();
 
-		$priceTypeManager = \Aimeos\MShop\Factory::createManager( $context, 'price/type' );
-		$listTypeManager = \Aimeos\MShop\Factory::createManager( $context, 'attribute/lists/type' );
-		$currencyManager = \Aimeos\MShop\Factory::createManager( $context, 'locale/currency' );
+		$priceTypeManager = \Aimeos\MShop\Factory::createManager($context, 'price/type');
+		$listTypeManager = \Aimeos\MShop\Factory::createManager($context, 'attribute/lists/type');
+		$currencyManager = \Aimeos\MShop\Factory::createManager($context, 'locale/currency');
 
-		$search = $priceTypeManager->createSearch( true )->setSlice( 0, 0x7fffffff );
-		$search->setConditions( $search->compare( '==', 'price.type.domain', 'attribute' ) );
-		$search->setSortations( array( $search->sort( '+', 'price.type.label' ) ) );
+		$search = $priceTypeManager->createSearch(true)->setSlice(0, 0x7fffffff);
+		$search->setConditions($search->compare('==', 'price.type.domain', 'attribute'));
+		$search->setSortations(array($search->sort('+', 'price.type.label')));
 
-		$listSearch = $listTypeManager->createSearch( true )->setSlice( 0, 0x7fffffff );
-		$listSearch->setConditions( $listSearch->compare( '==', 'attribute.lists.type.domain', 'price' ) );
-		$listSearch->setSortations( array( $listSearch->sort( '+', 'attribute.lists.type.label' ) ) );
+		$listSearch = $listTypeManager->createSearch(true)->setSlice(0, 0x7fffffff);
+		$listSearch->setConditions($listSearch->compare('==', 'attribute.lists.type.domain', 'price'));
+		$listSearch->setSortations(array($listSearch->sort('+', 'attribute.lists.type.label')));
 
-		$view->priceTypes = $priceTypeManager->searchItems( $search );
-		$view->priceListTypes = $this->sortType( $listTypeManager->searchItems( $listSearch ) );
-		$view->priceCurrencies = $currencyManager->searchItems( $currencyManager->createSearch( true )->setSlice( 0, 0x7fffffff ) );
+		$view->priceTypes = $priceTypeManager->searchItems($search);
+		$view->priceListTypes = $this->sortType($listTypeManager->searchItems($listSearch));
+		$view->priceCurrencies = $currencyManager->searchItems($currencyManager->createSearch(true)->setSlice(0, 0x7fffffff));
 
-		if( $view->priceCurrencies === [] ) {
-			throw new \Aimeos\Admin\JQAdm\Exception( 'No currencies available. Please enable at least one currency' );
+		if ($view->priceCurrencies === []) {
+			throw new \Aimeos\Admin\JQAdm\Exception('No currencies available. Please enable at least one currency');
 		}
 
 		return $view;
@@ -320,46 +320,46 @@ 
 	 * @param \Aimeos\MShop\Attribute\Item\Iface $item Attribute item object without referenced domain items
 	 * @param string[] $data Data array
 	 */
-	protected function fromArray( \Aimeos\MShop\Attribute\Item\Iface $item, array $data )
+	protected function fromArray(\Aimeos\MShop\Attribute\Item\Iface $item, array $data)
 	{
 		$context = $this->getContext();
 
-		$priceManager = \Aimeos\MShop\Factory::createManager( $context, 'price' );
-		$listManager = \Aimeos\MShop\Factory::createManager( $context, 'attribute/lists' );
+		$priceManager = \Aimeos\MShop\Factory::createManager($context, 'price');
+		$listManager = \Aimeos\MShop\Factory::createManager($context, 'attribute/lists');
 
-		$listItems = $item->getListItems( 'price', null, null, false );
+		$listItems = $item->getListItems('price', null, null, false);
 
 
-		foreach( $data as $idx => $entry )
+		foreach ($data as $idx => $entry)
 		{
-			if( ( $listItem = $item->getListItem( 'price', $entry['attribute.lists.type'], $entry['price.id'], false ) ) === null ) {
+			if (($listItem = $item->getListItem('price', $entry['attribute.lists.type'], $entry['price.id'], false)) === null) {
 				$listItem = $listManager->createItem();
 			}
 
-			if( ( $refItem = $listItem->getRefItem() ) === null ) {
+			if (($refItem = $listItem->getRefItem()) === null) {
 				$refItem = $priceManager->createItem();
 			}
 
-			$refItem->fromArray( $entry );
+			$refItem->fromArray($entry);
 			$conf = [];
 
-			foreach( (array) $this->getValue( $entry, 'config/key' ) as $num => $key )
+			foreach ((array) $this->getValue($entry, 'config/key') as $num => $key)
 			{
-				if( trim( $key ) !== '' && ( $val = $this->getValue( $entry, 'config/val/' . $num ) ) !== null ) {
-					$conf[$key] = trim( $val );
+				if (trim($key) !== '' && ($val = $this->getValue($entry, 'config/val/' . $num)) !== null) {
+					$conf[$key] = trim($val);
 				}
 			}
 
-			$listItem->fromArray( $entry );
-			$listItem->setPosition( $idx );
-			$listItem->setConfig( $conf );
+			$listItem->fromArray($entry);
+			$listItem->setPosition($idx);
+			$listItem->setConfig($conf);
 
-			$item->addListItem( 'price', $listItem, $refItem );
+			$item->addListItem('price', $listItem, $refItem);
 
-			unset( $listItems[$listItem->getId()] );
+			unset($listItems[$listItem->getId()]);
 		}
 
-		return $item->deleteListItems( $listItems, true );
+		return $item->deleteListItems($listItems, true);
 	}
 
 
@@ -370,20 +370,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\Attribute\Item\Iface $item, $copy = false )
+	protected function toArray(\Aimeos\MShop\Attribute\Item\Iface $item, $copy = false)
 	{
 		$data = [];
 		$siteId = $this->getContext()->getLocale()->getSiteId();
 
-		foreach( $item->getListItems( 'price', null, null, false ) as $listItem )
+		foreach ($item->getListItems('price', null, 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['attribute.lists.siteid'] = $siteId;
 				$list['attribute.lists.id'] = '';
@@ -391,10 +391,10 @@ 
 				$list['price.id'] = null;
 			}
 
-			$list['attribute.lists.datestart'] = str_replace( ' ', 'T', $list['attribute.lists.datestart'] );
-			$list['attribute.lists.dateend'] = str_replace( ' ', 'T', $list['attribute.lists.dateend'] );
+			$list['attribute.lists.datestart'] = str_replace(' ', 'T', $list['attribute.lists.datestart']);
+			$list['attribute.lists.dateend'] = str_replace(' ', 'T', $list['attribute.lists.dateend']);
 
-			foreach( $list['attribute.lists.config'] as $key => $val )
+			foreach ($list['attribute.lists.config'] as $key => $val)
 			{
 				$list['config']['key'][] = $key;
 				$list['config']['val'][] = $val;
@@ -413,7 +413,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/attribute/price/template-item
 		 * Relative path to the HTML body template of the price subpart for attributes.
@@ -437,6 +437,6 @@ 
 		$tplconf = 'admin/jqadm/attribute/price/template-item';
 		$default = 'attribute/item-price-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/Service/Price/Standard.php

Braces +2 added lines, -4 removed lines

@@ -160,14 +160,12 @@
 			}
 
 			return;
-		}
-		catch( \Aimeos\MShop\Exception $e )
+		} catch( \Aimeos\MShop\Exception $e )
 		{
 			$error = array( 'service-item-price' => $this->getContext()->getI18n()->dt( 'mshop', $e->getMessage() ) );
 			$view->errors = $view->get( 'errors', [] ) + $error;
 			$this->logException( $e );
-		}
-		catch( \Exception $e )
+		} catch( \Exception $e )
 		{
 			$error = array( 'service-item-price' => $e->getMessage() . ', ' . $e->getFile() . ':' . $e->getLine() );
 			$view->errors = $view->get( 'errors', [] ) + $error;

Spacing +70 added lines, -70 removed lines

@@ -10,7 +10,7 @@ 
 
 namespace Aimeos\Admin\JQAdm\Service\Price;
 
-sprintf( 'price' ); // for translation
+sprintf('price'); // for translation
 
 
 /**
@@ -42,16 +42,16 @@ 
 	 */
 	public function copy()
 	{
-		$view = $this->addViewData( $this->getView() );
+		$view = $this->addViewData($this->getView());
 
-		$view->priceData = $this->toArray( $view->item, true );
+		$view->priceData = $this->toArray($view->item, true);
 		$view->priceBody = '';
 
-		foreach( $this->getSubClients() as $client ) {
+		foreach ($this->getSubClients() as $client) {
 			$view->priceBody .= $client->copy();
 		}
 
-		return $this->render( $view );
+		return $this->render($view);
 	}
 
 
@@ -62,11 +62,11 @@ 
 	 */
 	public function create()
 	{
-		$view = $this->addViewData( $this->getView() );
+		$view = $this->addViewData($this->getView());
 		$siteid = $this->getContext()->getLocale()->getSiteId();
-		$data = $view->param( 'price', [] );
+		$data = $view->param('price', []);
 
-		foreach( $data as $idx => $entry )
+		foreach ($data as $idx => $entry)
 		{
 			$data[$idx]['service.lists.siteid'] = $siteid;
 			$data[$idx]['price.siteid'] = $siteid;
@@ -75,11 +75,11 @@ 
 		$view->priceData = $data;
 		$view->priceBody = '';
 
-		foreach( $this->getSubClients() as $client ) {
+		foreach ($this->getSubClients() as $client) {
 			$view->priceBody .= $client->create();
 		}
 
-		return $this->render( $view );
+		return $this->render($view);
 	}
 
 
@@ -91,7 +91,7 @@ 
 		parent::delete();
 
 		$item = $this->getView()->item;
-		$item->deleteListItems( $item->getListItems( 'price', null, null, false ), true );
+		$item->deleteListItems($item->getListItems('price', null, null, false), true);
 	}
 
 
@@ -102,16 +102,16 @@ 
 	 */
 	public function get()
 	{
-		$view = $this->addViewData( $this->getView() );
+		$view = $this->addViewData($this->getView());
 
-		$view->priceData = $this->toArray( $view->item );
+		$view->priceData = $this->toArray($view->item);
 		$view->priceBody = '';
 
-		foreach( $this->getSubClients() as $client ) {
+		foreach ($this->getSubClients() as $client) {
 			$view->priceBody .= $client->get();
 		}
 
-		return $this->render( $view );
+		return $this->render($view);
 	}
 
 
@@ -124,26 +124,26 @@ 
 
 		try
 		{
-			$view->item = $this->fromArray( $view->item, $view->param( 'price', [] ) );
+			$view->item = $this->fromArray($view->item, $view->param('price', []));
 			$view->priceBody = '';
 
-			foreach( $this->getSubClients() as $client ) {
+			foreach ($this->getSubClients() as $client) {
 				$view->priceBody .= $client->save();
 			}
 
 			return;
 		}
-		catch( \Aimeos\MShop\Exception $e )
+		catch (\Aimeos\MShop\Exception $e)
 		{
-			$error = array( 'service-item-price' => $this->getContext()->getI18n()->dt( 'mshop', $e->getMessage() ) );
-			$view->errors = $view->get( 'errors', [] ) + $error;
-			$this->logException( $e );
+			$error = array('service-item-price' => $this->getContext()->getI18n()->dt('mshop', $e->getMessage()));
+			$view->errors = $view->get('errors', []) + $error;
+			$this->logException($e);
 		}
-		catch( \Exception $e )
+		catch (\Exception $e)
 		{
-			$error = array( 'service-item-price' => $e->getMessage() . ', ' . $e->getFile() . ':' . $e->getLine() );
-			$view->errors = $view->get( 'errors', [] ) + $error;
-			$this->logException( $e );
+			$error = array('service-item-price' => $e->getMessage() . ', ' . $e->getFile() . ':' . $e->getLine());
+			$view->errors = $view->get('errors', []) + $error;
+			$this->logException($e);
 		}
 
 		throw new \Aimeos\Admin\JQAdm\Exception();
@@ -157,7 +157,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/service/price/decorators/excludes
 		 * Excludes decorators added by the "common" option from the service JQAdm client
@@ -232,7 +232,7 @@ 
 		 * @see admin/jqadm/service/price/decorators/excludes
 		 * @see admin/jqadm/service/price/decorators/global
 		 */
-		return $this->createSubClient( 'service/price/' . $type, $name );
+		return $this->createSubClient('service/price/' . $type, $name);
 	}
 
 
@@ -276,7 +276,7 @@ 
 		 * @since 2016.01
 		 * @category Developer
 		 */
-		return $this->getContext()->getConfig()->get( 'admin/jqadm/service/price/standard/subparts', [] );
+		return $this->getContext()->getConfig()->get('admin/jqadm/service/price/standard/subparts', []);
 	}
 
 
@@ -286,28 +286,28 @@ 
 	 * @param \Aimeos\MW\View\Iface $view View object
 	 * @return \Aimeos\MW\View\Iface View object with assigned parameters
 	 */
-	protected function addViewData( \Aimeos\MW\View\Iface $view )
+	protected function addViewData(\Aimeos\MW\View\Iface $view)
 	{
 		$context = $this->getContext();
 
-		$priceTypeManager = \Aimeos\MShop\Factory::createManager( $context, 'price/type' );
-		$listTypeManager = \Aimeos\MShop\Factory::createManager( $context, 'service/lists/type' );
-		$currencyManager = \Aimeos\MShop\Factory::createManager( $context, 'locale/currency' );
+		$priceTypeManager = \Aimeos\MShop\Factory::createManager($context, 'price/type');
+		$listTypeManager = \Aimeos\MShop\Factory::createManager($context, 'service/lists/type');
+		$currencyManager = \Aimeos\MShop\Factory::createManager($context, 'locale/currency');
 
-		$search = $priceTypeManager->createSearch( true )->setSlice( 0, 0x7fffffff );
-		$search->setConditions( $search->compare( '==', 'price.type.domain', 'service' ) );
-		$search->setSortations( array( $search->sort( '+', 'price.type.label' ) ) );
+		$search = $priceTypeManager->createSearch(true)->setSlice(0, 0x7fffffff);
+		$search->setConditions($search->compare('==', 'price.type.domain', 'service'));
+		$search->setSortations(array($search->sort('+', 'price.type.label')));
 
-		$listSearch = $listTypeManager->createSearch( true )->setSlice( 0, 0x7fffffff );
-		$listSearch->setConditions( $listSearch->compare( '==', 'service.lists.type.domain', 'price' ) );
-		$listSearch->setSortations( array( $listSearch->sort( '+', 'service.lists.type.label' ) ) );
+		$listSearch = $listTypeManager->createSearch(true)->setSlice(0, 0x7fffffff);
+		$listSearch->setConditions($listSearch->compare('==', 'service.lists.type.domain', 'price'));
+		$listSearch->setSortations(array($listSearch->sort('+', 'service.lists.type.label')));
 
-		$view->priceTypes = $priceTypeManager->searchItems( $search );
-		$view->priceListTypes = $this->sortType( $listTypeManager->searchItems( $listSearch ) );
-		$view->priceCurrencies = $currencyManager->searchItems( $currencyManager->createSearch( true )->setSlice( 0, 0x7fffffff ) );
+		$view->priceTypes = $priceTypeManager->searchItems($search);
+		$view->priceListTypes = $this->sortType($listTypeManager->searchItems($listSearch));
+		$view->priceCurrencies = $currencyManager->searchItems($currencyManager->createSearch(true)->setSlice(0, 0x7fffffff));
 
-		if( $view->priceCurrencies === [] ) {
-			throw new \Aimeos\Admin\JQAdm\Exception( 'No currencies available. Please enable at least one currency' );
+		if ($view->priceCurrencies === []) {
+			throw new \Aimeos\Admin\JQAdm\Exception('No currencies available. Please enable at least one currency');
 		}
 
 		return $view;
@@ -320,46 +320,46 @@ 
 	 * @param \Aimeos\MShop\Service\Item\Iface $item Service item object without referenced domain items
 	 * @param string[] $data Data array
 	 */
-	protected function fromArray( \Aimeos\MShop\Service\Item\Iface $item, array $data )
+	protected function fromArray(\Aimeos\MShop\Service\Item\Iface $item, array $data)
 	{
 		$context = $this->getContext();
 
-		$priceManager = \Aimeos\MShop\Factory::createManager( $context, 'price' );
-		$listManager = \Aimeos\MShop\Factory::createManager( $context, 'service/lists' );
+		$priceManager = \Aimeos\MShop\Factory::createManager($context, 'price');
+		$listManager = \Aimeos\MShop\Factory::createManager($context, 'service/lists');
 
-		$listItems = $item->getListItems( 'price', null, null, false );
+		$listItems = $item->getListItems('price', null, null, false);
 
 
-		foreach( $data as $idx => $entry )
+		foreach ($data as $idx => $entry)
 		{
-			if( ( $listItem = $item->getListItem( 'price', $entry['service.lists.type'], $entry['price.id'], false ) ) === null ) {
+			if (($listItem = $item->getListItem('price', $entry['service.lists.type'], $entry['price.id'], false)) === null) {
 				$listItem = $listManager->createItem();
 			}
 
-			if( ( $refItem = $listItem->getRefItem() ) === null ) {
+			if (($refItem = $listItem->getRefItem()) === null) {
 				$refItem = $priceManager->createItem();
 			}
 
-			$refItem->fromArray( $entry );
+			$refItem->fromArray($entry);
 			$conf = [];
 
-			foreach( (array) $this->getValue( $entry, 'config/key' ) as $num => $key )
+			foreach ((array) $this->getValue($entry, 'config/key') as $num => $key)
 			{
-				if( trim( $key ) !== '' && ( $val = $this->getValue( $entry, 'config/val/' . $num ) ) !== null ) {
-					$conf[$key] = trim( $val );
+				if (trim($key) !== '' && ($val = $this->getValue($entry, 'config/val/' . $num)) !== null) {
+					$conf[$key] = trim($val);
 				}
 			}
 
-			$listItem->fromArray( $entry );
-			$listItem->setPosition( $idx );
-			$listItem->setConfig( $conf );
+			$listItem->fromArray($entry);
+			$listItem->setPosition($idx);
+			$listItem->setConfig($conf);
 
-			$item->addListItem( 'price', $listItem, $refItem );
+			$item->addListItem('price', $listItem, $refItem);
 
-			unset( $listItems[$listItem->getId()] );
+			unset($listItems[$listItem->getId()]);
 		}
 
-		return $item->deleteListItems( $listItems, true );
+		return $item->deleteListItems($listItems, true);
 	}
 
 
@@ -370,20 +370,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\Service\Item\Iface $item, $copy = false )
+	protected function toArray(\Aimeos\MShop\Service\Item\Iface $item, $copy = false)
 	{
 		$data = [];
 		$siteId = $this->getContext()->getLocale()->getSiteId();
 
-		foreach( $item->getListItems( 'price', null, null, false ) as $listItem )
+		foreach ($item->getListItems('price', null, 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['service.lists.siteid'] = $siteId;
 				$list['service.lists.id'] = '';
@@ -391,10 +391,10 @@ 
 				$list['price.id'] = null;
 			}
 
-			$list['service.lists.datestart'] = str_replace( ' ', 'T', $list['service.lists.datestart'] );
-			$list['service.lists.dateend'] = str_replace( ' ', 'T', $list['service.lists.dateend'] );
+			$list['service.lists.datestart'] = str_replace(' ', 'T', $list['service.lists.datestart']);
+			$list['service.lists.dateend'] = str_replace(' ', 'T', $list['service.lists.dateend']);
 
-			foreach( $list['service.lists.config'] as $key => $val )
+			foreach ($list['service.lists.config'] as $key => $val)
 			{
 				$list['config']['key'][] = $key;
 				$list['config']['val'][] = $val;
@@ -413,7 +413,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/service/price/template-item
 		 * Relative path to the HTML body template of the price subpart for services.
@@ -437,6 +437,6 @@ 
 		$tplconf = 'admin/jqadm/service/price/template-item';
 		$default = 'service/item-price-standard';
 
-		return $view->render( $view->config( $tplconf, $default ) );
+		return $view->render($view->config($tplconf, $default));
 	}
 }
\ No newline at end of file