aimeos / ai-admin-jqadm

admin/jqadm/tests/bootstrap.php

Spacing +6 added lines, -6 removed lines

@@ -11,17 +11,17 @@
  * Set error reporting to maximum
  */
 error_reporting( -1 );
-ini_set( 'display_errors', '1' );
+ini_set('display_errors', '1');
 
-date_default_timezone_set( 'UTC' );
+date_default_timezone_set('UTC');
 
 /*
  * Set locale settings to reasonable defaults
  */
-setlocale( LC_ALL, 'en_US.UTF-8' );
-setlocale( LC_NUMERIC, 'POSIX' );
-setlocale( LC_CTYPE, 'en_US.UTF-8' );
-setlocale( LC_TIME, 'POSIX' );
+setlocale(LC_ALL, 'en_US.UTF-8');
+setlocale(LC_NUMERIC, 'POSIX');
+setlocale(LC_CTYPE, 'en_US.UTF-8');
+setlocale(LC_TIME, 'POSIX');
 
 
 require_once 'TestHelperJqadm.php';

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

Indentation +5 added lines, -5 removed lines

@@ -74,7 +74,7 @@ 
 	 * Creates a new resource
 	 *
 	 * @return string|null admin output to display or null for redirecting to the list
-	*/
+	 */
 	public function create()
 	{
 		return $this->client->create();
@@ -85,7 +85,7 @@ 
 	 * Deletes a resource
 	 *
 	 * @return string|null admin output to display or null for redirecting to the list
-	*/
+	 */
 	public function delete()
 	{
 		return $this->client->delete();
@@ -96,7 +96,7 @@ 
 	 * Returns a single resource
 	 *
 	 * @return string|null admin output to display or null for redirecting to the list
-	*/
+	 */
 	public function get()
 	{
 		return $this->client->get();
@@ -107,7 +107,7 @@ 
 	 * Saves the data
 	 *
 	 * @return string|null admin output to display or null for redirecting to the list
-	*/
+	 */
 	public function save()
 	{
 		return $this->client->save();
@@ -118,7 +118,7 @@ 
 	 * Returns a list of resource according to the conditions
 	 *
 	 * @return string admin output to display
-	*/
+	 */
 	public function search()
 	{
 		return $this->client->search();

Spacing +13 added lines, -13 removed lines

@@ -30,10 +30,10 @@ 
 	 * @param \Aimeos\Admin\JQAdm\Iface $client Admin object
 	 * @param \Aimeos\MShop\Context\Item\Iface $context Context object with required objects
 	 */
-	public function __construct( \Aimeos\Admin\JQAdm\Iface $client,
-		\Aimeos\MShop\Context\Item\Iface $context )
+	public function __construct(\Aimeos\Admin\JQAdm\Iface $client,
+		\Aimeos\MShop\Context\Item\Iface $context)
 	{
-		parent::__construct( $context );
+		parent::__construct($context);
 
 		$this->client = $client;
 	}
@@ -47,9 +47,9 @@ 
 	 * @return mixed Returns the value of the called method
 	 * @throws \Aimeos\Admin\JQAdm\Exception If method call failed
 	 */
-	public function __call( $name, array $param )
+	public function __call($name, array $param)
 	{
-		return @call_user_func_array( array( $this->client, $name ), $param );
+		return @call_user_func_array(array($this->client, $name), $param);
 	}
 
 
@@ -148,9 +148,9 @@ 
 	 * @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)
 	{
-		return $this->client->getSubClient( $type, $name );
+		return $this->client->getSubClient($type, $name);
 	}
 
 
@@ -171,11 +171,11 @@ 
 	 * @param \Aimeos\MW\View\Iface $view The view object which generates the admin output
 	 * @return \Aimeos\Admin\JQAdm\Iface Reference to this object for fluent calls
 	 */
-	public function setView( \Aimeos\MW\View\Iface $view )
+	public function setView(\Aimeos\MW\View\Iface $view)
 	{
-		parent::setView( $view );
+		parent::setView($view);
 
-		$this->client->setView( $view );
+		$this->client->setView($view);
 		return $this;
 	}
 
@@ -197,11 +197,11 @@ 
 	 * @param \Aimeos\Bootstrap $aimeos The Aimeos bootstrap object
 	 * @return \Aimeos\Admin\JQAdm\Iface Reference to this object for fluent calls
 	 */
-	public function setAimeos( \Aimeos\Bootstrap $aimeos )
+	public function setAimeos(\Aimeos\Bootstrap $aimeos)
 	{
-		parent::setAimeos( $aimeos );
+		parent::setAimeos($aimeos);
 
-		$this->client->setAimeos( $aimeos );
+		$this->client->setAimeos($aimeos);
 		return $this;
 	}
 

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

Spacing +6 added lines, -6 removed lines

@@ -28,14 +28,14 @@ 
 	{
 		$result = $this->getClient()->delete();
 
-		$ids = (array) $this->getView()->param( 'id' );
-		$tags = array( 'product' );
+		$ids = (array) $this->getView()->param('id');
+		$tags = array('product');
 
-		foreach( $ids as $id ) {
+		foreach ($ids as $id) {
 			$tags[] = 'product-' . $id;
 		}
 
-		$this->getContext()->getCache()->deleteByTags( $tags );
+		$this->getContext()->getCache()->deleteByTags($tags);
 
 		return $result;
 	}
@@ -51,10 +51,10 @@ 
 		$result = $this->getClient()->save();
 		$item = $this->getView()->item;
 
-		if( $item->getId() !== null )
+		if ($item->getId() !== null)
 		{
 			$idtag = 'product-' . $item->getId();
-			$this->getContext()->getCache()->deleteByTags( array( 'product', $idtag ) );
+			$this->getContext()->getCache()->deleteByTags(array('product', $idtag));
 		}
 
 		return $result;

admin/jqadm/src/Admin/JQAdm/Iface.php

Spacing +2 added lines, -2 removed lines

@@ -26,7 +26,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);
 
 	/**
 	 * Returns the view object that will generate the admin output.
@@ -41,7 +41,7 @@ 
 	 * @param \Aimeos\MW\View\Iface $view The view object which generates the admin output
 	 * @return \Aimeos\Admin\JQAdm\Iface Reference to this object for fluent calls
 	 */
-	public function setView( \Aimeos\MW\View\Iface $view );
+	public function setView(\Aimeos\MW\View\Iface $view);
 
 	/**
 	 * Copies a resource

admin/jqadm/src/Admin/JQAdm/Product/Category/Decorator/Cache.php

Spacing +3 added lines, -3 removed lines

@@ -27,13 +27,13 @@
 	public function save()
 	{
 		$result = $this->getClient()->save();
-		$ids = array( 'catalog' );
+		$ids = array('catalog');
 
-		foreach( $this->getView()->param( 'category/catalog.id', [] ) as $id ) {
+		foreach ($this->getView()->param('category/catalog.id', []) as $id) {
 			$ids[] = 'catalog-' . $id;
 		}
 
-		$this->getContext()->getCache()->deleteByTags( $ids );
+		$this->getContext()->getCache()->deleteByTags($ids);
 
 		return $result;
 	}

lib/custom/tests/bootstrap.php

Spacing +6 added lines, -6 removed lines

@@ -10,17 +10,17 @@
  * Set error reporting to maximum
  */
 error_reporting( -1 );
-ini_set( 'display_errors', '1' );
+ini_set('display_errors', '1');
 
-date_default_timezone_set( 'UTC' );
+date_default_timezone_set('UTC');
 
 /*
  * Set locale settings to reasonable defaults
  */
-setlocale( LC_ALL, 'en_US.UTF-8' );
-setlocale( LC_NUMERIC, 'POSIX' );
-setlocale( LC_CTYPE, 'en_US.UTF-8' );
-setlocale( LC_TIME, 'POSIX' );
+setlocale(LC_ALL, 'en_US.UTF-8');
+setlocale(LC_NUMERIC, 'POSIX');
+setlocale(LC_CTYPE, 'en_US.UTF-8');
+setlocale(LC_TIME, 'POSIX');
 
 
 require_once 'TestHelperCustom.php';

lib/custom/src/MW/View/Helper/Site/Standard.php

Doc Comments +1 added lines, -1 removed lines

@@ -37,7 +37,7 @@
 	/**
 	 * Returns the site view helper
 	 *
-	 * @return Aimeos\MW\View\Helper\Site\Iface Site view helper
+	 * @return Standard Site view helper
 	 */
 	public function transform()
 	{

Spacing +6 added lines, -6 removed lines

@@ -27,9 +27,9 @@ 
 	 *
 	 * @param \Aimeos\MW\View\Iface $view View object
 	 */
-	public function __construct( \Aimeos\MW\View\Iface $view )
+	public function __construct(\Aimeos\MW\View\Iface $view)
 	{
-		parent::__construct( $view );
+		parent::__construct($view);
 		$this->siteItem = $view->pageSiteItem;
 	}
 
@@ -62,9 +62,9 @@ 
 	 * @param string $siteid ID of a site item
 	 * @return string|null Label of the site item or null if not found
 	 */
-	public function match( $siteid )
+	public function match($siteid)
 	{
-		if( $this->siteItem->getId() == $siteid ) {
+		if ($this->siteItem->getId() == $siteid) {
 			return $this->siteItem->getLabel();
 		}
 	}
@@ -76,9 +76,9 @@ 
 	 * @param string $siteid ID of a site item
 	 * @return string|null "readonly" if item is from a parent site, null if not
 	 */
-	public function readonly( $siteid )
+	public function readonly($siteid)
 	{
-		if( $this->siteItem->getId() != $siteid ) {
+		if ($this->siteItem->getId() != $siteid) {
 			return 'readonly';
 		}
 	}

admin/jqadm/src/Admin/JQAdm/Locale/Site/Standard.php

Doc Comments +1 added lines, -1 removed lines

@@ -498,7 +498,7 @@
 	/**
 	 * Constructs the data array for the view from the given item
 	 *
-	 * @param \Aimeos\MShop\Locale\Item\Iface $item Locale item object
+	 * @param \Aimeos\MShop\Locale\Item\Site\Iface $item Locale item object
 	 * @return string[] Multi-dimensional associative list of item data
 	 */
 	protected function toArray( \Aimeos\MShop\Locale\Item\Site\Iface $item, $copy = false )

Braces +18 added lines, -36 removed lines

@@ -90,20 +90,17 @@ 
 				$view->tabindex = ++$idx + 1;
 				$view->itemBody .= $client->copy();
 			}
-		}
-		catch( \Aimeos\Admin\JQAdm\Exception $e )
+		} catch( \Aimeos\Admin\JQAdm\Exception $e )
 		{
 			$error = array( 'locale-site-item' => $context->getI18n()->dt( 'admin', $e->getMessage() ) );
 			$view->errors = $view->get( 'errors', [] ) + $error;
 			$this->logException( $e );
-		}
-		catch( \Aimeos\MShop\Exception $e )
+		} catch( \Aimeos\MShop\Exception $e )
 		{
 			$error = array( 'locale-site-item' => $context->getI18n()->dt( 'mshop', $e->getMessage() ) );
 			$view->errors = $view->get( 'errors', [] ) + $error;
 			$this->logException( $e );
-		}
-		catch( \Exception $e )
+		} catch( \Exception $e )
 		{
 			$error = array( 'locale-site-item' => $e->getMessage() . ', ' . $e->getFile() . ':' . $e->getLine() );
 			$view->errors = $view->get( 'errors', [] ) + $error;
@@ -145,20 +142,17 @@ 
 				$view->tabindex = ++$idx + 1;
 				$view->itemBody .= $client->create();
 			}
-		}
-		catch( \Aimeos\Admin\JQAdm\Exception $e )
+		} catch( \Aimeos\Admin\JQAdm\Exception $e )
 		{
 			$error = array( 'locale-site-item' => $context->getI18n()->dt( 'admin', $e->getMessage() ) );
 			$view->errors = $view->get( 'errors', [] ) + $error;
 			$this->logException( $e );
-		}
-		catch( \Aimeos\MShop\Exception $e )
+		} catch( \Aimeos\MShop\Exception $e )
 		{
 			$error = array( 'locale-site-item' => $context->getI18n()->dt( 'mshop', $e->getMessage() ) );
 			$view->errors = $view->get( 'errors', [] ) + $error;
 			$this->logException( $e );
-		}
-		catch( \Exception $e )
+		} catch( \Exception $e )
 		{
 			$error = array( 'locale-site-item' => $e->getMessage() . ', ' . $e->getFile() . ':' . $e->getLine() );
 			$view->errors = $view->get( 'errors', [] ) + $error;
@@ -201,20 +195,17 @@ 
 
 			$this->nextAction( $view, 'search', 'locale/site', null, 'delete' );
 			return;
-		}
-		catch( \Aimeos\Admin\JQAdm\Exception $e )
+		} catch( \Aimeos\Admin\JQAdm\Exception $e )
 		{
 			$error = array( 'locale-site-item' => $context->getI18n()->dt( 'admin', $e->getMessage() ) );
 			$view->errors = $view->get( 'errors', [] ) + $error;
 			$this->logException( $e );
-		}
-		catch( \Aimeos\MShop\Exception $e )
+		} catch( \Aimeos\MShop\Exception $e )
 		{
 			$error = array( 'locale-site-item' => $context->getI18n()->dt( 'mshop', $e->getMessage() ) );
 			$view->errors = $view->get( 'errors', [] ) + $error;
 			$this->logException( $e );
-		}
-		catch( \Exception $e )
+		} catch( \Exception $e )
 		{
 			$error = array( 'locale-site-item' => $e->getMessage() . ', ' . $e->getFile() . ':' . $e->getLine() );
 			$view->errors = $view->get( 'errors', [] ) + $error;
@@ -257,20 +248,17 @@ 
 				$view->tabindex = ++$idx + 1;
 				$view->itemBody .= $client->get();
 			}
-		}
-		catch( \Aimeos\Admin\JQAdm\Exception $e )
+		} catch( \Aimeos\Admin\JQAdm\Exception $e )
 		{
 			$error = array( 'locale-site-item' => $context->getI18n()->dt( 'admin', $e->getMessage() ) );
 			$view->errors = $view->get( 'errors', [] ) + $error;
 			$this->logException( $e );
-		}
-		catch( \Aimeos\MShop\Exception $e )
+		} catch( \Aimeos\MShop\Exception $e )
 		{
 			$error = array( 'locale-site-item' => $context->getI18n()->dt( 'mshop', $e->getMessage() ) );
 			$view->errors = $view->get( 'errors', [] ) + $error;
 			$this->logException( $e );
-		}
-		catch( \Exception $e )
+		} catch( \Exception $e )
 		{
 			$error = array( 'locale-site-item' => $e->getMessage() . ', ' . $e->getFile() . ':' . $e->getLine() );
 			$view->errors = $view->get( 'errors', [] ) + $error;
@@ -308,20 +296,17 @@ 
 
 			$this->nextAction( $view, $view->param( 'next' ), 'locale/site', $view->item->getId(), 'save' );
 			return;
-		}
-		catch( \Aimeos\Admin\JQAdm\Exception $e )
+		} catch( \Aimeos\Admin\JQAdm\Exception $e )
 		{
 			$error = array( 'locale-site-item' => $context->getI18n()->dt( 'admin', $e->getMessage() ) );
 			$view->errors = $view->get( 'errors', [] ) + $error;
 			$this->logException( $e );
-		}
-		catch( \Aimeos\MShop\Exception $e )
+		} catch( \Aimeos\MShop\Exception $e )
 		{
 			$error = array( 'locale-site-item' => $context->getI18n()->dt( 'mshop', $e->getMessage() ) );
 			$view->errors = $view->get( 'errors', [] ) + $error;
 			$this->logException( $e );
-		}
-		catch( \Exception $e )
+		} catch( \Exception $e )
 		{
 			$error = array( 'locale-site-item' => $e->getMessage() . ', ' . $e->getFile() . ':' . $e->getLine() );
 			$view->errors = $view->get( 'errors', [] ) + $error;
@@ -368,14 +353,12 @@ 
 			foreach( $this->getSubClients() as $client ) {
 				$view->itemBody .= $client->search();
 			}
-		}
-		catch( \Aimeos\MShop\Exception $e )
+		} catch( \Aimeos\MShop\Exception $e )
 		{
 			$error = array( 'locale-item' => $context->getI18n()->dt( 'mshop', $e->getMessage() ) );
 			$view->errors = $view->get( 'errors', [] ) + $error;
 			$this->logException( $e );
-		}
-		catch( \Exception $e )
+		} catch( \Exception $e )
 		{
 			$error = array( 'locale-item' => $e->getMessage() . ', ' . $e->getFile() . ':' . $e->getLine() );
 			$view->errors = $view->get( 'errors', [] ) + $error;
@@ -563,8 +546,7 @@ 
 		{
 			$this->checkSite( $super, $data['locale.site.id'] );
 			$item = $manager->getItem( $data['locale.site.id'] );
-		}
-		else
+		} else
 		{
 			$this->checkSite( $super );
 			$item = $manager->createItem();

Spacing +143 added lines, -143 removed lines

@@ -10,7 +10,7 @@ 
 
 namespace Aimeos\Admin\JQAdm\Locale\Site;
 
-sprintf( 'locale/site' ); // for translation
+sprintf('locale/site'); // for translation
 
 
 /**
@@ -35,45 +35,45 @@ 
 
 		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'));
 			}
 
-			$this->checkSite( $view->access( 'super' ), $id );
+			$this->checkSite($view->access('super'), $id);
 
-			$manager = \Aimeos\MShop::create( $context, 'locale/site' );
-			$view->item = $manager->getItem( $id );
+			$manager = \Aimeos\MShop::create($context, 'locale/site');
+			$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\Admin\JQAdm\Exception $e )
+		catch (\Aimeos\Admin\JQAdm\Exception $e)
 		{
-			$error = array( 'locale-site-item' => $context->getI18n()->dt( 'admin', $e->getMessage() ) );
-			$view->errors = $view->get( 'errors', [] ) + $error;
-			$this->logException( $e );
+			$error = array('locale-site-item' => $context->getI18n()->dt('admin', $e->getMessage()));
+			$view->errors = $view->get('errors', []) + $error;
+			$this->logException($e);
 		}
-		catch( \Aimeos\MShop\Exception $e )
+		catch (\Aimeos\MShop\Exception $e)
 		{
-			$error = array( 'locale-site-item' => $context->getI18n()->dt( 'mshop', $e->getMessage() ) );
-			$view->errors = $view->get( 'errors', [] ) + $error;
-			$this->logException( $e );
+			$error = array('locale-site-item' => $context->getI18n()->dt('mshop', $e->getMessage()));
+			$view->errors = $view->get('errors', []) + $error;
+			$this->logException($e);
 		}
-		catch( \Exception $e )
+		catch (\Exception $e)
 		{
-			$error = array( 'locale-site-item' => $e->getMessage() . ', ' . $e->getFile() . ':' . $e->getLine() );
-			$view->errors = $view->get( 'errors', [] ) + $error;
-			$this->logException( $e );
+			$error = array('locale-site-item' => $e->getMessage() . ', ' . $e->getFile() . ':' . $e->getLine());
+			$view->errors = $view->get('errors', []) + $error;
+			$this->logException($e);
 		}
 
-		return $this->render( $view );
+		return $this->render($view);
 	}
 
 
@@ -89,44 +89,44 @@ 
 
 		try
 		{
-			$this->checkSite( $view->access( 'super' ) );
+			$this->checkSite($view->access('super'));
 
-			$data = $view->param( 'item', [] );
+			$data = $view->param('item', []);
 
-			if( !isset( $view->item ) ) {
-				$view->item = \Aimeos\MShop::create( $context, 'locale/site' )->createItem();
+			if (!isset($view->item)) {
+				$view->item = \Aimeos\MShop::create($context, 'locale/site')->createItem();
 			}
 
 			$view->itemSubparts = $this->getSubClientNames();
 			$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\Admin\JQAdm\Exception $e )
+		catch (\Aimeos\Admin\JQAdm\Exception $e)
 		{
-			$error = array( 'locale-site-item' => $context->getI18n()->dt( 'admin', $e->getMessage() ) );
-			$view->errors = $view->get( 'errors', [] ) + $error;
-			$this->logException( $e );
+			$error = array('locale-site-item' => $context->getI18n()->dt('admin', $e->getMessage()));
+			$view->errors = $view->get('errors', []) + $error;
+			$this->logException($e);
 		}
-		catch( \Aimeos\MShop\Exception $e )
+		catch (\Aimeos\MShop\Exception $e)
 		{
-			$error = array( 'locale-site-item' => $context->getI18n()->dt( 'mshop', $e->getMessage() ) );
-			$view->errors = $view->get( 'errors', [] ) + $error;
-			$this->logException( $e );
+			$error = array('locale-site-item' => $context->getI18n()->dt('mshop', $e->getMessage()));
+			$view->errors = $view->get('errors', []) + $error;
+			$this->logException($e);
 		}
-		catch( \Exception $e )
+		catch (\Exception $e)
 		{
-			$error = array( 'locale-site-item' => $e->getMessage() . ', ' . $e->getFile() . ':' . $e->getLine() );
-			$view->errors = $view->get( 'errors', [] ) + $error;
-			$this->logException( $e );
+			$error = array('locale-site-item' => $e->getMessage() . ', ' . $e->getFile() . ':' . $e->getLine());
+			$view->errors = $view->get('errors', []) + $error;
+			$this->logException($e);
 		}
 
-		return $this->render( $view );
+		return $this->render($view);
 	}
 
 
@@ -140,46 +140,46 @@ 
 		$view = $this->getView();
 		$context = $this->getContext();
 
-		$manager = \Aimeos\MShop::create( $context, 'locale/site' );
+		$manager = \Aimeos\MShop::create($context, 'locale/site');
 		$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'));
 			}
 
-			$this->checkSite( $view->access( 'super' ), $id );
+			$this->checkSite($view->access('super'), $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', 'locale/site', null, 'delete' );
+			$this->nextAction($view, 'search', 'locale/site', null, 'delete');
 			return;
 		}
-		catch( \Aimeos\Admin\JQAdm\Exception $e )
+		catch (\Aimeos\Admin\JQAdm\Exception $e)
 		{
-			$error = array( 'locale-site-item' => $context->getI18n()->dt( 'admin', $e->getMessage() ) );
-			$view->errors = $view->get( 'errors', [] ) + $error;
-			$this->logException( $e );
+			$error = array('locale-site-item' => $context->getI18n()->dt('admin', $e->getMessage()));
+			$view->errors = $view->get('errors', []) + $error;
+			$this->logException($e);
 		}
-		catch( \Aimeos\MShop\Exception $e )
+		catch (\Aimeos\MShop\Exception $e)
 		{
-			$error = array( 'locale-site-item' => $context->getI18n()->dt( 'mshop', $e->getMessage() ) );
-			$view->errors = $view->get( 'errors', [] ) + $error;
-			$this->logException( $e );
+			$error = array('locale-site-item' => $context->getI18n()->dt('mshop', $e->getMessage()));
+			$view->errors = $view->get('errors', []) + $error;
+			$this->logException($e);
 		}
-		catch( \Exception $e )
+		catch (\Exception $e)
 		{
-			$error = array( 'locale-site-item' => $e->getMessage() . ', ' . $e->getFile() . ':' . $e->getLine() );
-			$view->errors = $view->get( 'errors', [] ) + $error;
-			$this->logException( $e );
+			$error = array('locale-site-item' => $e->getMessage() . ', ' . $e->getFile() . ':' . $e->getLine());
+			$view->errors = $view->get('errors', []) + $error;
+			$this->logException($e);
 		}
 
 		$manager->rollback();
@@ -200,45 +200,45 @@ 
 
 		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'));
 			}
 
-			$this->checkSite( $view->access( 'super' ), $id );
+			$this->checkSite($view->access('super'), $id);
 
-			$manager = \Aimeos\MShop::create( $context, 'locale/site' );
+			$manager = \Aimeos\MShop::create($context, 'locale/site');
 
-			$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\Admin\JQAdm\Exception $e )
+		catch (\Aimeos\Admin\JQAdm\Exception $e)
 		{
-			$error = array( 'locale-site-item' => $context->getI18n()->dt( 'admin', $e->getMessage() ) );
-			$view->errors = $view->get( 'errors', [] ) + $error;
-			$this->logException( $e );
+			$error = array('locale-site-item' => $context->getI18n()->dt('admin', $e->getMessage()));
+			$view->errors = $view->get('errors', []) + $error;
+			$this->logException($e);
 		}
-		catch( \Aimeos\MShop\Exception $e )
+		catch (\Aimeos\MShop\Exception $e)
 		{
-			$error = array( 'locale-site-item' => $context->getI18n()->dt( 'mshop', $e->getMessage() ) );
-			$view->errors = $view->get( 'errors', [] ) + $error;
-			$this->logException( $e );
+			$error = array('locale-site-item' => $context->getI18n()->dt('mshop', $e->getMessage()));
+			$view->errors = $view->get('errors', []) + $error;
+			$this->logException($e);
 		}
-		catch( \Exception $e )
+		catch (\Exception $e)
 		{
-			$error = array( 'locale-site-item' => $e->getMessage() . ', ' . $e->getFile() . ':' . $e->getLine() );
-			$view->errors = $view->get( 'errors', [] ) + $error;
-			$this->logException( $e );
+			$error = array('locale-site-item' => $e->getMessage() . ', ' . $e->getFile() . ':' . $e->getLine());
+			$view->errors = $view->get('errors', []) + $error;
+			$this->logException($e);
 		}
 
-		return $this->render( $view );
+		return $this->render($view);
 	}
 
 
@@ -252,42 +252,42 @@ 
 		$view = $this->getView();
 		$context = $this->getContext();
 
-		$manager = \Aimeos\MShop::create( $context, 'locale/site' );
+		$manager = \Aimeos\MShop::create($context, 'locale/site');
 		$manager->begin();
 
 		try
 		{
-			$item = $this->fromArray( $view->param( 'item', [] ), $view->access( 'super' ) );
-			$view->item = $item->getId() ? $item : $manager->saveItem( $item );
+			$item = $this->fromArray($view->param('item', []), $view->access('super'));
+			$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' ), 'locale/site', $view->item->getId(), 'save' );
+			$this->nextAction($view, $view->param('next'), 'locale/site', $view->item->getId(), 'save');
 			return;
 		}
-		catch( \Aimeos\Admin\JQAdm\Exception $e )
+		catch (\Aimeos\Admin\JQAdm\Exception $e)
 		{
-			$error = array( 'locale-site-item' => $context->getI18n()->dt( 'admin', $e->getMessage() ) );
-			$view->errors = $view->get( 'errors', [] ) + $error;
-			$this->logException( $e );
+			$error = array('locale-site-item' => $context->getI18n()->dt('admin', $e->getMessage()));
+			$view->errors = $view->get('errors', []) + $error;
+			$this->logException($e);
 		}
-		catch( \Aimeos\MShop\Exception $e )
+		catch (\Aimeos\MShop\Exception $e)
 		{
-			$error = array( 'locale-site-item' => $context->getI18n()->dt( 'mshop', $e->getMessage() ) );
-			$view->errors = $view->get( 'errors', [] ) + $error;
-			$this->logException( $e );
+			$error = array('locale-site-item' => $context->getI18n()->dt('mshop', $e->getMessage()));
+			$view->errors = $view->get('errors', []) + $error;
+			$this->logException($e);
 		}
-		catch( \Exception $e )
+		catch (\Exception $e)
 		{
-			$error = array( 'locale-site-item' => $e->getMessage() . ', ' . $e->getFile() . ':' . $e->getLine() );
-			$view->errors = $view->get( 'errors', [] ) + $error;
-			$this->logException( $e );
+			$error = array('locale-site-item' => $e->getMessage() . ', ' . $e->getFile() . ':' . $e->getLine());
+			$view->errors = $view->get('errors', []) + $error;
+			$this->logException($e);
 		}
 
 		$manager->rollback();
@@ -309,39 +309,39 @@ 
 		try
 		{
 			$total = 0;
-			$params = $this->storeSearchParams( $view->param(), 'locale/site' );
-			$manager = \Aimeos\MShop::create( $context, 'locale/site' );
-			$search = $this->initCriteria( $manager->createSearch(), $params );
+			$params = $this->storeSearchParams($view->param(), 'locale/site');
+			$manager = \Aimeos\MShop::create($context, 'locale/site');
+			$search = $this->initCriteria($manager->createSearch(), $params);
 
-			if( $view->access( 'super' ) === false )
+			if ($view->access('super') === false)
 			{
-				$search->setConditions( $search->combine( '&&', [
-					$search->compare( '==', 'locale.site.id', $this->getUserSiteId() ),
+				$search->setConditions($search->combine('&&', [
+					$search->compare('==', 'locale.site.id', $this->getUserSiteId()),
 					$search->getConditions(),
-				] ) );
+				]));
 			}
 
-			$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( 'locale-item' => $context->getI18n()->dt( 'mshop', $e->getMessage() ) );
-			$view->errors = $view->get( 'errors', [] ) + $error;
-			$this->logException( $e );
+			$error = array('locale-item' => $context->getI18n()->dt('mshop', $e->getMessage()));
+			$view->errors = $view->get('errors', []) + $error;
+			$this->logException($e);
 		}
-		catch( \Exception $e )
+		catch (\Exception $e)
 		{
-			$error = array( 'locale-item' => $e->getMessage() . ', ' . $e->getFile() . ':' . $e->getLine() );
-			$view->errors = $view->get( 'errors', [] ) + $error;
-			$this->logException( $e );
+			$error = array('locale-item' => $e->getMessage() . ', ' . $e->getFile() . ':' . $e->getLine());
+			$view->errors = $view->get('errors', []) + $error;
+			$this->logException($e);
 		}
 
 		/** admin/jqadm/locale/site/template-list
@@ -366,7 +366,7 @@ 
 		$tplconf = 'admin/jqadm/locale/site/template-list';
 		$default = 'locale/site/list-standard';
 
-		return $view->render( $view->config( $tplconf, $default ) );
+		return $view->render($view->config($tplconf, $default));
 	}
 
 
@@ -377,7 +377,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/locale/site/decorators/excludes
 		 * Excludes decorators added by the "common" option from the locale JQAdm client
@@ -452,7 +452,7 @@ 
 		 * @see admin/jqadm/locale/site/decorators/excludes
 		 * @see admin/jqadm/locale/site/decorators/global
 		 */
-		return $this->createSubClient( 'locale/site' . $type, $name );
+		return $this->createSubClient('locale/site' . $type, $name);
 	}
 
 
@@ -463,13 +463,13 @@ 
 	 * @param string $id ID of the site to access
 	 * @throws \Aimeos\Admin\JQAdm\Exception If user isn't allowed to access the site
 	 */
-	protected function checkSite( $super, $id = null )
+	protected function checkSite($super, $id = null)
 	{
-		if( $super === true || (string) $this->getUserSiteId() === (string) $id ) {
+		if ($super === true || (string) $this->getUserSiteId() === (string) $id) {
 			return;
 		}
 
-		throw new \Aimeos\Admin\JQAdm\Exception( sprintf( 'Permission denied' ) );
+		throw new \Aimeos\Admin\JQAdm\Exception(sprintf('Permission denied'));
 	}
 
 
@@ -481,9 +481,9 @@ 
 	protected function getUserSiteId()
 	{
 		$context = $this->getContext();
-		$manager = \Aimeos\MShop::create( $context, 'customer' );
+		$manager = \Aimeos\MShop::create($context, 'customer');
 
-		return $manager->getItem( $context->getUserId() )->getSiteId();
+		return $manager->getItem($context->getUserId())->getSiteId();
 	}
 
 
@@ -527,7 +527,7 @@ 
 		 * @since 2017.10
 		 * @category Developer
 		 */
-		return $this->getContext()->getConfig()->get( 'admin/jqadm/locale/site/standard/subparts', [] );
+		return $this->getContext()->getConfig()->get('admin/jqadm/locale/site/standard/subparts', []);
 	}
 
 
@@ -538,38 +538,38 @@ 
 	 * @param boolean $super If current user is a super user
 	 * @return \Aimeos\MShop\Locale\Item\Iface New locale item object
 	 */
-	protected function fromArray( array $data, $super )
+	protected function fromArray(array $data, $super)
 	{
 		$conf = [];
 
-		if( isset( $data['config']['key'] ) )
+		if (isset($data['config']['key']))
 		{
-			foreach( (array) $data['config']['key'] as $idx => $key )
+			foreach ((array) $data['config']['key'] as $idx => $key)
 			{
-				if( trim( $key ) !== '' && isset( $data['config']['val'][$idx] ) ) {
+				if (trim($key) !== '' && isset($data['config']['val'][$idx])) {
 					$conf[$key] = $data['config']['val'][$idx];
 				}
 			}
 		}
 
-		$manager = \Aimeos\MShop::create( $this->getContext(), 'locale/site' );
+		$manager = \Aimeos\MShop::create($this->getContext(), 'locale/site');
 
-		if( isset( $data['locale.site.id'] ) && $data['locale.site.id'] != '' )
+		if (isset($data['locale.site.id']) && $data['locale.site.id'] != '')
 		{
-			$this->checkSite( $super, $data['locale.site.id'] );
-			$item = $manager->getItem( $data['locale.site.id'] );
+			$this->checkSite($super, $data['locale.site.id']);
+			$item = $manager->getItem($data['locale.site.id']);
 		}
 		else
 		{
-			$this->checkSite( $super );
+			$this->checkSite($super);
 			$item = $manager->createItem();
 		}
 
-		$item->fromArray( $data );
-		$item->setConfig( $conf );
+		$item->fromArray($data);
+		$item->setConfig($conf);
 
-		if( $item->getId() == null ) {
-			return $manager->insertItem( $item );
+		if ($item->getId() == null) {
+			return $manager->insertItem($item);
 		}
 
 		return $item;
@@ -582,18 +582,18 @@ 
 	 * @param \Aimeos\MShop\Locale\Item\Iface $item Locale item object
 	 * @return string[] Multi-dimensional associative list of item data
 	 */
-	protected function toArray( \Aimeos\MShop\Locale\Item\Site\Iface $item, $copy = false )
+	protected function toArray(\Aimeos\MShop\Locale\Item\Site\Iface $item, $copy = false)
 	{
-		$data = $item->toArray( true );
+		$data = $item->toArray(true);
 		$data['config'] = [];
 
-		if( $copy === true )
+		if ($copy === true)
 		{
 			$data['locale.site.code'] = $data['locale.site.code'] . '_copy';
 			$data['locale.site.id'] = '';
 		}
 
-		foreach( $item->getConfig() as $key => $value )
+		foreach ($item->getConfig() as $key => $value)
 		{
 			$data['config']['key'][] = $key;
 			$data['config']['val'][] = $value;
@@ -608,7 +608,7 @@ 
 	 *
 	 * @return string HTML output
 	 */
-	protected function render( \Aimeos\MW\View\Iface $view )
+	protected function render(\Aimeos\MW\View\Iface $view)
 	{
 		/** admin/jqadm/locale/site/template-item
 		 * Relative path to the HTML body template for the locale item.
@@ -632,6 +632,6 @@ 
 		$tplconf = 'admin/jqadm/locale/site/template-item';
 		$default = 'locale/site/item-standard';
 
-		return $view->render( $view->config( $tplconf, $default ) );
+		return $view->render($view->config($tplconf, $default));
 	}
 }

admin/jqadm/src/Admin/JQAdm/Locale/Language/Standard.php

Doc Comments +1 added lines, -1 removed lines

@@ -476,7 +476,7 @@
 	/**
 	 * Constructs the data array for the view from the given item
 	 *
-	 * @param \Aimeos\MShop\Locale\Item\Iface $item Locale item object
+	 * @param \Aimeos\MShop\Locale\Item\Language\Iface $item Locale item object
 	 * @return string[] Multi-dimensional associative list of item data
 	 */
 	protected function toArray( \Aimeos\MShop\Locale\Item\Language\Iface $item, $copy = false )

Braces +13 added lines, -26 removed lines

@@ -51,14 +51,12 @@ 
 				$view->tabindex = ++$idx + 1;
 				$view->itemBody .= $client->copy();
 			}
-		}
-		catch( \Aimeos\MShop\Exception $e )
+		} catch( \Aimeos\MShop\Exception $e )
 		{
 			$error = array( 'locale-language-item' => $context->getI18n()->dt( 'mshop', $e->getMessage() ) );
 			$view->errors = $view->get( 'errors', [] ) + $error;
 			$this->logException( $e );
-		}
-		catch( \Exception $e )
+		} catch( \Exception $e )
 		{
 			$error = array( 'locale-language-item' => $e->getMessage() . ', ' . $e->getFile() . ':' . $e->getLine() );
 			$view->errors = $view->get( 'errors', [] ) + $error;
@@ -96,14 +94,12 @@ 
 				$view->tabindex = ++$idx + 1;
 				$view->itemBody .= $client->create();
 			}
-		}
-		catch( \Aimeos\MShop\Exception $e )
+		} catch( \Aimeos\MShop\Exception $e )
 		{
 			$error = array( 'locale-language-item' => $context->getI18n()->dt( 'mshop', $e->getMessage() ) );
 			$view->errors = $view->get( 'errors', [] ) + $error;
 			$this->logException( $e );
-		}
-		catch( \Exception $e )
+		} catch( \Exception $e )
 		{
 			$error = array( 'locale-language-item' => $e->getMessage() . ', ' . $e->getFile() . ':' . $e->getLine() );
 			$view->errors = $view->get( 'errors', [] ) + $error;
@@ -144,14 +140,12 @@ 
 
 			$this->nextAction( $view, 'search', 'locale/language', null, 'delete' );
 			return;
-		}
-		catch( \Aimeos\MShop\Exception $e )
+		} catch( \Aimeos\MShop\Exception $e )
 		{
 			$error = array( 'locale-language-item' => $context->getI18n()->dt( 'mshop', $e->getMessage() ) );
 			$view->errors = $view->get( 'errors', [] ) + $error;
 			$this->logException( $e );
-		}
-		catch( \Exception $e )
+		} catch( \Exception $e )
 		{
 			$error = array( 'locale-language-item' => $e->getMessage() . ', ' . $e->getFile() . ':' . $e->getLine() );
 			$view->errors = $view->get( 'errors', [] ) + $error;
@@ -192,14 +186,12 @@ 
 				$view->tabindex = ++$idx + 1;
 				$view->itemBody .= $client->get();
 			}
-		}
-		catch( \Aimeos\MShop\Exception $e )
+		} catch( \Aimeos\MShop\Exception $e )
 		{
 			$error = array( 'locale-language-item' => $context->getI18n()->dt( 'mshop', $e->getMessage() ) );
 			$view->errors = $view->get( 'errors', [] ) + $error;
 			$this->logException( $e );
-		}
-		catch( \Exception $e )
+		} catch( \Exception $e )
 		{
 			$error = array( 'locale-language-item' => $e->getMessage() . ', ' . $e->getFile() . ':' . $e->getLine() );
 			$view->errors = $view->get( 'errors', [] ) + $error;
@@ -238,18 +230,15 @@ 
 
 			$this->nextAction( $view, $view->param( 'next' ), 'locale/language', $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( 'locale-language-item' => $context->getI18n()->dt( 'mshop', $e->getMessage() ) );
 			$view->errors = $view->get( 'errors', [] ) + $error;
 			$this->logException( $e );
-		}
-		catch( \Exception $e )
+		} catch( \Exception $e )
 		{
 			$error = array( 'locale-language-item' => $e->getMessage() . ', ' . $e->getFile() . ':' . $e->getLine() );
 			$view->errors = $view->get( 'errors', [] ) + $error;
@@ -291,14 +280,12 @@ 
 			foreach( $this->getSubClients() as $client ) {
 				$view->itemBody .= $client->search();
 			}
-		}
-		catch( \Aimeos\MShop\Exception $e )
+		} catch( \Aimeos\MShop\Exception $e )
 		{
 			$error = array( 'locale-language-item' => $context->getI18n()->dt( 'mshop', $e->getMessage() ) );
 			$view->errors = $view->get( 'errors', [] ) + $error;
 			$this->logException( $e );
-		}
-		catch( \Exception $e )
+		} catch( \Exception $e )
 		{
 			$error = array( 'locale-language-item' => $e->getMessage() . ', ' . $e->getFile() . ':' . $e->getLine() );
 			$view->errors = $view->get( 'errors', [] ) + $error;

Spacing +103 added lines, -103 removed lines

@@ -10,7 +10,7 @@ 
 
 namespace Aimeos\Admin\JQAdm\Locale\Language;
 
-sprintf( 'locale/language' ); // for translation
+sprintf('locale/language'); // 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, 'locale/language' );
-			$view->item = $manager->getItem( $id );
+			$manager = \Aimeos\MShop::create($context, 'locale/language');
+			$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( 'locale-language-item' => $context->getI18n()->dt( 'mshop', $e->getMessage() ) );
-			$view->errors = $view->get( 'errors', [] ) + $error;
-			$this->logException( $e );
+			$error = array('locale-language-item' => $context->getI18n()->dt('mshop', $e->getMessage()));
+			$view->errors = $view->get('errors', []) + $error;
+			$this->logException($e);
 		}
-		catch( \Exception $e )
+		catch (\Exception $e)
 		{
-			$error = array( 'locale-language-item' => $e->getMessage() . ', ' . $e->getFile() . ':' . $e->getLine() );
-			$view->errors = $view->get( 'errors', [] ) + $error;
-			$this->logException( $e );
+			$error = array('locale-language-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,36 +81,36 @@ 
 
 		try
 		{
-			$data = $view->param( 'item', [] );
+			$data = $view->param('item', []);
 
-			if( !isset( $view->item ) ) {
-				$view->item = \Aimeos\MShop::create( $context, 'locale/language' )->createItem();
+			if (!isset($view->item)) {
+				$view->item = \Aimeos\MShop::create($context, 'locale/language')->createItem();
 			}
 
 			$view->itemSubparts = $this->getSubClientNames();
 			$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( 'locale-language-item' => $context->getI18n()->dt( 'mshop', $e->getMessage() ) );
-			$view->errors = $view->get( 'errors', [] ) + $error;
-			$this->logException( $e );
+			$error = array('locale-language-item' => $context->getI18n()->dt('mshop', $e->getMessage()));
+			$view->errors = $view->get('errors', []) + $error;
+			$this->logException($e);
 		}
-		catch( \Exception $e )
+		catch (\Exception $e)
 		{
-			$error = array( 'locale-language-item' => $e->getMessage() . ', ' . $e->getFile() . ':' . $e->getLine() );
-			$view->errors = $view->get( 'errors', [] ) + $error;
-			$this->logException( $e );
+			$error = array('locale-language-item' => $e->getMessage() . ', ' . $e->getFile() . ':' . $e->getLine());
+			$view->errors = $view->get('errors', []) + $error;
+			$this->logException($e);
 		}
 
-		return $this->render( $view );
+		return $this->render($view);
 	}
 
 
@@ -124,38 +124,38 @@ 
 		$view = $this->getView();
 		$context = $this->getContext();
 
-		$manager = \Aimeos\MShop::create( $context, 'locale/language' );
+		$manager = \Aimeos\MShop::create($context, 'locale/language');
 		$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', 'locale/language', null, 'delete' );
+			$this->nextAction($view, 'search', 'locale/language', null, 'delete');
 			return;
 		}
-		catch( \Aimeos\MShop\Exception $e )
+		catch (\Aimeos\MShop\Exception $e)
 		{
-			$error = array( 'locale-language-item' => $context->getI18n()->dt( 'mshop', $e->getMessage() ) );
-			$view->errors = $view->get( 'errors', [] ) + $error;
-			$this->logException( $e );
+			$error = array('locale-language-item' => $context->getI18n()->dt('mshop', $e->getMessage()));
+			$view->errors = $view->get('errors', []) + $error;
+			$this->logException($e);
 		}
-		catch( \Exception $e )
+		catch (\Exception $e)
 		{
-			$error = array( 'locale-language-item' => $e->getMessage() . ', ' . $e->getFile() . ':' . $e->getLine() );
-			$view->errors = $view->get( 'errors', [] ) + $error;
-			$this->logException( $e );
+			$error = array('locale-language-item' => $e->getMessage() . ', ' . $e->getFile() . ':' . $e->getLine());
+			$view->errors = $view->get('errors', []) + $error;
+			$this->logException($e);
 		}
 
 		$manager->rollback();
@@ -176,37 +176,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, 'locale/language' );
+			$manager = \Aimeos\MShop::create($context, 'locale/language');
 
-			$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( 'locale-language-item' => $context->getI18n()->dt( 'mshop', $e->getMessage() ) );
-			$view->errors = $view->get( 'errors', [] ) + $error;
-			$this->logException( $e );
+			$error = array('locale-language-item' => $context->getI18n()->dt('mshop', $e->getMessage()));
+			$view->errors = $view->get('errors', []) + $error;
+			$this->logException($e);
 		}
-		catch( \Exception $e )
+		catch (\Exception $e)
 		{
-			$error = array( 'locale-language-item' => $e->getMessage() . ', ' . $e->getFile() . ':' . $e->getLine() );
-			$view->errors = $view->get( 'errors', [] ) + $error;
-			$this->logException( $e );
+			$error = array('locale-language-item' => $e->getMessage() . ', ' . $e->getFile() . ':' . $e->getLine());
+			$view->errors = $view->get('errors', []) + $error;
+			$this->logException($e);
 		}
 
-		return $this->render( $view );
+		return $this->render($view);
 	}
 
 
@@ -220,40 +220,40 @@ 
 		$view = $this->getView();
 		$context = $this->getContext();
 
-		$manager = \Aimeos\MShop::create( $context, 'locale/language' );
+		$manager = \Aimeos\MShop::create($context, 'locale/language');
 		$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' ), 'locale/language', $view->item->getId(), 'save' );
+			$this->nextAction($view, $view->param('next'), 'locale/language', $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( 'locale-language-item' => $context->getI18n()->dt( 'mshop', $e->getMessage() ) );
-			$view->errors = $view->get( 'errors', [] ) + $error;
-			$this->logException( $e );
+			$error = array('locale-language-item' => $context->getI18n()->dt('mshop', $e->getMessage()));
+			$view->errors = $view->get('errors', []) + $error;
+			$this->logException($e);
 		}
-		catch( \Exception $e )
+		catch (\Exception $e)
 		{
-			$error = array( 'locale-language-item' => $e->getMessage() . ', ' . $e->getFile() . ':' . $e->getLine() );
-			$view->errors = $view->get( 'errors', [] ) + $error;
-			$this->logException( $e );
+			$error = array('locale-language-item' => $e->getMessage() . ', ' . $e->getFile() . ':' . $e->getLine());
+			$view->errors = $view->get('errors', []) + $error;
+			$this->logException($e);
 		}
 
 		$manager->rollback();
@@ -275,34 +275,34 @@ 
 		try
 		{
 			$total = 0;
-			$params = $this->storeSearchParams( $view->param(), 'locale/language' );
-			$manager = \Aimeos\MShop::create( $context, 'locale/language' );
+			$params = $this->storeSearchParams($view->param(), 'locale/language');
+			$manager = \Aimeos\MShop::create($context, 'locale/language');
 
 			$search = $manager->createSearch();
-			$search->setSortations( [$search->sort( '-', 'locale.language.status'), $search->sort( '+', 'locale.language.id')] );
-			$search = $this->initCriteria( $search, $params );
+			$search->setSortations([$search->sort('-', 'locale.language.status'), $search->sort('+', 'locale.language.id')]);
+			$search = $this->initCriteria($search, $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( 'locale-language-item' => $context->getI18n()->dt( 'mshop', $e->getMessage() ) );
-			$view->errors = $view->get( 'errors', [] ) + $error;
-			$this->logException( $e );
+			$error = array('locale-language-item' => $context->getI18n()->dt('mshop', $e->getMessage()));
+			$view->errors = $view->get('errors', []) + $error;
+			$this->logException($e);
 		}
-		catch( \Exception $e )
+		catch (\Exception $e)
 		{
-			$error = array( 'locale-language-item' => $e->getMessage() . ', ' . $e->getFile() . ':' . $e->getLine() );
-			$view->errors = $view->get( 'errors', [] ) + $error;
-			$this->logException( $e );
+			$error = array('locale-language-item' => $e->getMessage() . ', ' . $e->getFile() . ':' . $e->getLine());
+			$view->errors = $view->get('errors', []) + $error;
+			$this->logException($e);
 		}
 
 		/** admin/jqadm/locale/language/template-list
@@ -327,7 +327,7 @@ 
 		$tplconf = 'admin/jqadm/locale/language/template-list';
 		$default = 'locale/language/list-standard';
 
-		return $view->render( $view->config( $tplconf, $default ) );
+		return $view->render($view->config($tplconf, $default));
 	}
 
 
@@ -338,7 +338,7 @@ 
 	 * @param string|null $name Name of the sub-client (Default if null)
 	 * @return \Aimeos\Admin\JQAdm\Iface Sub-client object
 	 */
-	public function getSubClient( $type, $name = null )
+	public function getSubClient($type, $name = null)
 	{
 		/** admin/jqadm/locale/language/decorators/excludes
 		 * Excludes decorators added by the "common" option from the locale JQAdm client
@@ -413,7 +413,7 @@ 
 		 * @see admin/jqadm/locale/language/decorators/excludes
 		 * @see admin/jqadm/locale/language/decorators/global
 		 */
-		return $this->createSubClient( 'locale/language' . $type, $name );
+		return $this->createSubClient('locale/language' . $type, $name);
 	}
 
 
@@ -457,7 +457,7 @@ 
 		 * @since 2017.10
 		 * @category Developer
 		 */
-		return $this->getContext()->getConfig()->get( 'admin/jqadm/locale/language/standard/subparts', [] );
+		return $this->getContext()->getConfig()->get('admin/jqadm/locale/language/standard/subparts', []);
 	}
 
 
@@ -468,17 +468,17 @@ 
 	 * @param string[] Data array
 	 * @return \Aimeos\MShop\Locale\Item\Iface New locale item object
 	 */
-	protected function fromArray( array $data )
+	protected function fromArray(array $data)
 	{
-		$manager = \Aimeos\MShop::create( $this->getContext(), 'locale/language' );
+		$manager = \Aimeos\MShop::create($this->getContext(), 'locale/language');
 
-		if( isset( $data['locale.language.id'] ) && $data['locale.language.id'] != '' ) {
-			$item = $manager->getItem( $data['locale.language.id'] );
+		if (isset($data['locale.language.id']) && $data['locale.language.id'] != '') {
+			$item = $manager->getItem($data['locale.language.id']);
 		} else {
 			$item = $manager->createItem();
 		}
 
-		$item->fromArray( $data );
+		$item->fromArray($data);
 
 		return $item;
 	}
@@ -490,11 +490,11 @@ 
 	 * @param \Aimeos\MShop\Locale\Item\Iface $item Locale item object
 	 * @return string[] Multi-dimensional associative list of item data
 	 */
-	protected function toArray( \Aimeos\MShop\Locale\Item\Language\Iface $item, $copy = false )
+	protected function toArray(\Aimeos\MShop\Locale\Item\Language\Iface $item, $copy = false)
 	{
-		$data = $item->toArray( true );
+		$data = $item->toArray(true);
 
-		if( $copy === true )
+		if ($copy === true)
 		{
 			$data['locale.language.id'] = '';
 			$data['locale.language.code'] = '';
@@ -510,7 +510,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/locale/language/template-item
 		 * Relative path to the HTML body template for the locale item.
@@ -534,6 +534,6 @@ 
 		$tplconf = 'admin/jqadm/locale/language/template-item';
 		$default = 'locale/language/item-standard';
 
-		return $view->render( $view->config( $tplconf, $default ) );
+		return $view->render($view->config($tplconf, $default));
 	}
 }