aimeos / ai-admin-jqadm

admin/jqadm/templates/common/page-standard.php

Indentation -1 removed lines

@@ -161,7 +161,6 @@
  *
  * You can add, remove or reorder the links in the navigation bar by
  * setting a new list of client resource names.
-
  * In the configuration files of extensions, you should only add entries using
  * one of these lines:
  *

Braces +15 added lines, -6 removed lines

@@ -262,8 +262,11 @@ 
 							<?php foreach( $this->pageSiteList as $siteItem ) : ?>
 								<?php if( $siteItem->getId() === $this->pageSiteTree->getId() ) : ?>
 									<?php $siteFcn( $this->pageSiteTree ); ?>
-								<?php else : ?>
-									<?php $siteFcn( $siteItem ); ?>
+								<?php else {
+	: ?>
+									<?php $siteFcn( $siteItem );
+}
+?>
 								<?php endif; ?>
 							<?php endforeach; ?>
 						</ul>
@@ -295,9 +298,12 @@ 
 							</li>
 
 						<?php endif; ?>
-					<?php else : ?>
+					<?php else {
+	: ?>
 						<?php if( $this->access( $this->config( 'admin/jqadm/resource/' . $navitem . '/groups', [] ) ) ) : ?>
-							<?php $key = $this->config( 'admin/jqadm/resource/' . $navitem . '/key' ); ?>
+							<?php $key = $this->config( 'admin/jqadm/resource/' . $navitem . '/key' );
+}
+?>
 
 							<li class="<?= $enc->attr( $navitem ); ?> <?= $this->param( 'resource', $navfirst ) === $navitem ? 'active' : '' ?>">
 								<a href="<?= $enc->attr( $this->url( $searchTarget, $cntl, $action, array( 'resource' => $navitem ) + $params, [], $config ) ); ?>"
@@ -344,10 +350,13 @@ 
 							</li>
 
 						<?php endif; ?>
-					<?php else : ?>
+					<?php else {
+	: ?>
 						<?php if( $this->access( $this->config( 'admin/jqadm/resource/' . $navitem . '/groups', [] ) ) ) : ?>
 
-							<?php $key = $this->config( 'admin/jqadm/resource/' . $navitem . '/key' ); ?>
+							<?php $key = $this->config( 'admin/jqadm/resource/' . $navitem . '/key' );
+}
+?>
 							<li class="<?= $enc->attr( $navitem ); ?> <?= $this->param( 'resource', $navfirst ) === $navitem ? 'active' : '' ?>">
 								<a href="<?= $enc->attr( $this->url( $searchTarget, $cntl, $action, array( 'resource' => $navitem ) + $params, [], $config ) ); ?>"
 									title="<?= $enc->attr( sprintf( $title, $this->translate( 'admin', $navitem ), $key ) ); ?>"

Spacing +87 added lines, -87 removed lines

@@ -8,10 +8,10 @@ 
 $enc = $this->encoder();
 
 $target = $this->request()->getTarget();
-$searchTarget = $this->config( 'admin/jqadm/url/search/target' );
-$cntl = $this->config( 'admin/jqadm/url/search/controller', 'Jqadm' );
-$action = $this->config( 'admin/jqadm/url/search/action', 'search' );
-$config = $this->config( 'admin/jqadm/url/search/config', [] );
+$searchTarget = $this->config('admin/jqadm/url/search/target');
+$cntl = $this->config('admin/jqadm/url/search/controller', 'Jqadm');
+$action = $this->config('admin/jqadm/url/search/action', 'search');
+$config = $this->config('admin/jqadm/url/search/config', []);
 
 
 /** admin/jsonadm/url/options/target
@@ -28,7 +28,7 @@ 
  * @see admin/jsonadm/url/options/action
  * @see admin/jsonadm/url/options/config
  */
-$jsonTarget = $this->config( 'admin/jsonadm/url/options/target' );
+$jsonTarget = $this->config('admin/jsonadm/url/options/target');
 
 /** admin/jsonadm/url/options/controller
  * Name of the controller whose action should be called
@@ -44,7 +44,7 @@ 
  * @see admin/jsonadm/url/options/action
  * @see admin/jsonadm/url/options/config
  */
-$jsonCntl = $this->config( 'admin/jsonadm/url/options/controller', 'Jsonadm' );
+$jsonCntl = $this->config('admin/jsonadm/url/options/controller', 'Jsonadm');
 
 /** admin/jsonadm/url/options/action
  * Name of the action that should create the output
@@ -60,7 +60,7 @@ 
  * @see admin/jsonadm/url/options/controller
  * @see admin/jsonadm/url/options/config
  */
-$jsonAction = $this->config( 'admin/jsonadm/url/options/action', 'options' );
+$jsonAction = $this->config('admin/jsonadm/url/options/action', 'options');
 
 /** admin/jsonadm/url/options/config
  * Associative list of configuration options used for generating the URL
@@ -82,7 +82,7 @@ 
  * @see admin/jsonadm/url/options/controller
  * @see admin/jsonadm/url/options/action
  */
-$jsonConfig = $this->config( 'admin/jsonadm/url/options/config', [] );
+$jsonConfig = $this->config('admin/jsonadm/url/options/config', []);
 
 
 /** admin/jqadm/navbar
@@ -106,7 +106,7 @@ 
  * @category Developer
  * @see admin/jqadm/navbar-limit
  */
-$navlist = $this->config( 'admin/jqadm/navbar', [] );
+$navlist = $this->config('admin/jqadm/navbar', []);
 
 /** admin/jqadm/navbar-limit
  * Number of JQAdm client links in the navigation bar shown by default
@@ -122,26 +122,26 @@ 
  * @category Developer
  * @see admin/jqadm/navbar
  */
-$navlimit = $this->config( 'admin/jqadm/navbar-limit', 7 );
+$navlimit = $this->config('admin/jqadm/navbar-limit', 7);
 
 
-$navfirst = reset( $navlist );
-if( is_array( $navfirst ) ) {
-	$navfirst = key( $navlist );
+$navfirst = reset($navlist);
+if (is_array($navfirst)) {
+	$navfirst = key($navlist);
 }
 
-$resource = $this->param( 'resource', 'dashboard' );
-$site = $this->param( 'site', 'default' );
-$lang = $this->param( 'lang' );
+$resource = $this->param('resource', 'dashboard');
+$site = $this->param('site', 'default');
+$lang = $this->param('lang');
 
 $params = ['resource' => $resource, 'site' => $site];
 $extParams = ['site' => $site];
 
-if( $lang ) {
+if ($lang) {
 	$params['lang'] = $extParams['lang'] = $lang;
 }
 
-$title = $this->translate( 'admin', '%1$s (Ctrl+Alt+%2$s)' );
+$title = $this->translate('admin', '%1$s (Ctrl+Alt+%2$s)');
 
 
 /** admin/jqadm/partial/confirm
@@ -211,82 +211,82 @@ 
  */
 
 
-$infoMsgs = $this->get( 'info', [] );
+$infoMsgs = $this->get('info', []);
 
-switch( $this->param( 'act' ) )
+switch ($this->param('act'))
 {
 	case 'save':
-		$infoMsgs[] = $this->translate( 'admin', 'Item saved successfully' ); break;
+		$infoMsgs[] = $this->translate('admin', 'Item saved successfully'); break;
 	case 'delete':
-		$infoMsgs[] = $this->translate( 'admin', 'Item deleted successfully' ); break;
+		$infoMsgs[] = $this->translate('admin', 'Item deleted successfully'); break;
 }
 
 
 ?>
-<div class="aimeos" lang="<?= $this->param( 'lang' ); ?>" data-url="<?= $enc->attr( $this->url( $jsonTarget, $jsonCntl, $jsonAction, array( 'site' => $site ), [], $jsonConfig ) ); ?>">
+<div class="aimeos" lang="<?= $this->param('lang'); ?>" data-url="<?= $enc->attr($this->url($jsonTarget, $jsonCntl, $jsonAction, array('site' => $site), [], $jsonConfig)); ?>">
 
 	<nav class="main-sidebar">
 		<div class="sidebar-wrapper">
 
-			<a class="logo" target="_blank" href="https://aimeos.org/update/?type=<?= $this->get( 'aimeosType' ) ?>&version=<?= $this->get( 'aimeosVersion' ) ?>">
-				<img src="https://aimeos.org/check/?type=<?= $this->get( 'aimeosType' ) ?>&version=<?= $this->get( 'aimeosVersion' ) ?>&extensions=<?= $this->get( 'aimeosExtensions' ) ?>" alt="Aimeos update" title="Aimeos update">
+			<a class="logo" target="_blank" href="https://aimeos.org/update/?type=<?= $this->get('aimeosType') ?>&version=<?= $this->get('aimeosVersion') ?>">
+				<img src="https://aimeos.org/check/?type=<?= $this->get('aimeosType') ?>&version=<?= $this->get('aimeosVersion') ?>&extensions=<?= $this->get('aimeosExtensions') ?>" alt="Aimeos update" title="Aimeos update">
 			</a>
 
 			<ul class="sidebar-menu basic">
 
-				<?php if( ( count( $this->get( 'pageSiteList', [] ) ) > 1 || $this->pageSiteTree->getChildren() !== [] || count( $this->get( 'pageSitePath', [] ) ) > 1 ) && $this->access( $this->config( 'admin/jqadm/resource/site/groups', [] ) ) ) : ?>
+				<?php if ((count($this->get('pageSiteList', [])) > 1 || $this->pageSiteTree->getChildren() !== [] || count($this->get('pageSitePath', [])) > 1) && $this->access($this->config('admin/jqadm/resource/site/groups', []))) : ?>
 					<li class="site treeview">
 						<a href="#">
 							<i class="icon"></i>
-							<span class="title"><?= $enc->html( $this->site()->label() ); ?></span>
+							<span class="title"><?= $enc->html($this->site()->label()); ?></span>
 						</a>
 						<ul class="tree-menu">
-							<li class="menu-header"><strong><?= $enc->html( $this->translate( 'admin', 'Site' ) ); ?></strong></li>
+							<li class="menu-header"><strong><?= $enc->html($this->translate('admin', 'Site')); ?></strong></li>
 
-							<?php $siteFcn = function( \Aimeos\MShop\Locale\Item\Site\Iface $site ) use ( &$siteFcn, $enc, $searchTarget, $cntl, $action, $params, $config ) { ?>
+							<?php $siteFcn = function(\Aimeos\MShop\Locale\Item\Site\Iface $site) use (&$siteFcn, $enc, $searchTarget, $cntl, $action, $params, $config) { ?>
 
-								<li class="site-<?= $enc->attr( $site->getCode() ) ?>">
-									<a href="<?= $enc->attr( $this->url( $searchTarget, $cntl, $action, array( 'site' => $site->getCode() ) + $params, [], $config ) ); ?>">
-										<span class="name"><?= $enc->html( $site->getLabel() ); ?></span>
+								<li class="site-<?= $enc->attr($site->getCode()) ?>">
+									<a href="<?= $enc->attr($this->url($searchTarget, $cntl, $action, array('site' => $site->getCode()) + $params, [], $config)); ?>">
+										<span class="name"><?= $enc->html($site->getLabel()); ?></span>
 									</a>
 
-									<?php if( $site->getChildren() !== [] ) : ?>
+									<?php if ($site->getChildren() !== []) : ?>
 										<ul class="menu-sub">
-											<?php foreach( $site->getChildren() as $site ) { $siteFcn( $site ); } ?>
+											<?php foreach ($site->getChildren() as $site) { $siteFcn($site); } ?>
 										</ul>
 									<?php endif; ?>
 								</li>
 
 							<?php }; ?>
 
-							<?php foreach( $this->pageSiteList as $siteItem ) : ?>
-								<?php if( $siteItem->getId() === $this->pageSiteTree->getId() ) : ?>
-									<?php $siteFcn( $this->pageSiteTree ); ?>
+							<?php foreach ($this->pageSiteList as $siteItem) : ?>
+								<?php if ($siteItem->getId() === $this->pageSiteTree->getId()) : ?>
+									<?php $siteFcn($this->pageSiteTree); ?>
 								<?php else : ?>
-									<?php $siteFcn( $siteItem ); ?>
+									<?php $siteFcn($siteItem); ?>
 								<?php endif; ?>
 							<?php endforeach; ?>
 						</ul>
 					</li>
 				<?php endif; ?>
 
-				<?php foreach( array_splice( $navlist, 0, $navlimit ) as $nav => $navitem ) : ?>
-					<?php if( is_array( $navitem ) ) : ?>
-						<?php if( $this->access( $this->config( 'admin/jqadm/resource/' . $nav . '/groups', [] ) ) ) : ?>
+				<?php foreach (array_splice($navlist, 0, $navlimit) as $nav => $navitem) : ?>
+					<?php if (is_array($navitem)) : ?>
+						<?php if ($this->access($this->config('admin/jqadm/resource/' . $nav . '/groups', []))) : ?>
 
-							<li class="treeview <?= $enc->attr( $nav ) ?> <?= strncmp( $this->param( 'resource' ), $nav, strlen( $nav ) ) ? '' : 'active' ?>">
+							<li class="treeview <?= $enc->attr($nav) ?> <?= strncmp($this->param('resource'), $nav, strlen($nav)) ? '' : 'active' ?>">
 								<span>
 									<i class="icon"></i>
-									<span class="title"><?= $enc->attr( $this->translate( 'admin', $nav ) ); ?></span>
+									<span class="title"><?= $enc->attr($this->translate('admin', $nav)); ?></span>
 								</span>
 								<ul class="tree-menu">
-									<li class="menu-header"><strong><?= $enc->html( $this->translate( 'admin', $nav ) ); ?></strong></li>
+									<li class="menu-header"><strong><?= $enc->html($this->translate('admin', $nav)); ?></strong></li>
 
-									<?php foreach( $navitem as $subresource ) : ?>
-										<?php if( $this->access( $this->config( 'admin/jqadm/resource/' . $subresource . '/groups', [] ) ) ) : ?>
-											<li class="<?= str_replace( '/', '-', $subresource); ?>">
-												<a href="<?= $enc->attr( $this->url( $searchTarget, $cntl, $action, ['resource' => $subresource] + $params, [], $config ) ); ?>">
-													<span class="name"><?= $enc->html( $this->translate( 'admin', $subresource ) ); ?></span>
+									<?php foreach ($navitem as $subresource) : ?>
+										<?php if ($this->access($this->config('admin/jqadm/resource/' . $subresource . '/groups', []))) : ?>
+											<li class="<?= str_replace('/', '-', $subresource); ?>">
+												<a href="<?= $enc->attr($this->url($searchTarget, $cntl, $action, ['resource' => $subresource] + $params, [], $config)); ?>">
+													<span class="name"><?= $enc->html($this->translate('admin', $subresource)); ?></span>
 												</a>
 											</li>
 										<?php endif; ?>
@@ -296,15 +296,15 @@ 
 
 						<?php endif; ?>
 					<?php else : ?>
-						<?php if( $this->access( $this->config( 'admin/jqadm/resource/' . $navitem . '/groups', [] ) ) ) : ?>
-							<?php $key = $this->config( 'admin/jqadm/resource/' . $navitem . '/key' ); ?>
+						<?php if ($this->access($this->config('admin/jqadm/resource/' . $navitem . '/groups', []))) : ?>
+							<?php $key = $this->config('admin/jqadm/resource/' . $navitem . '/key'); ?>
 
-							<li class="<?= $enc->attr( $navitem ); ?> <?= $this->param( 'resource', $navfirst ) === $navitem ? 'active' : '' ?>">
-								<a href="<?= $enc->attr( $this->url( $searchTarget, $cntl, $action, array( 'resource' => $navitem ) + $params, [], $config ) ); ?>"
-									title="<?= $enc->attr( sprintf( $title, $this->translate( 'admin', $navitem ), $key ) ); ?>"
-									data-ctrlkey="<?= $enc->attr( strtolower( $key ) ); ?>">
+							<li class="<?= $enc->attr($navitem); ?> <?= $this->param('resource', $navfirst) === $navitem ? 'active' : '' ?>">
+								<a href="<?= $enc->attr($this->url($searchTarget, $cntl, $action, array('resource' => $navitem) + $params, [], $config)); ?>"
+									title="<?= $enc->attr(sprintf($title, $this->translate('admin', $navitem), $key)); ?>"
+									data-ctrlkey="<?= $enc->attr(strtolower($key)); ?>">
 									<i class="icon"></i>
-									<span class="title"><?= $enc->html( $this->translate( 'admin', $navitem ) ); ?></span>
+									<span class="title"><?= $enc->html($this->translate('admin', $navitem)); ?></span>
 								</a>
 							</li>
 
@@ -318,23 +318,23 @@ 
 
 			<ul class="sidebar-menu advanced">
 
-				<?php foreach( $navlist as $nav => $navitem ) : ?>
-					<?php if( is_array( $navitem ) ) : ?>
-						<?php if( $this->access( $this->config( 'admin/jqadm/resource/' . $nav . '/groups', [] ) ) ) : ?>
+				<?php foreach ($navlist as $nav => $navitem) : ?>
+					<?php if (is_array($navitem)) : ?>
+						<?php if ($this->access($this->config('admin/jqadm/resource/' . $nav . '/groups', []))) : ?>
 
-							<li class="treeview <?= $enc->attr( $nav ) ?> <?= strncmp( $this->param( 'resource' ), $nav, strlen( $nav ) ) ? '' : 'active' ?>">
+							<li class="treeview <?= $enc->attr($nav) ?> <?= strncmp($this->param('resource'), $nav, strlen($nav)) ? '' : 'active' ?>">
 								<span>
 									<i class="icon"></i>
-									<span class="title"><?= $enc->attr( $this->translate( 'admin', $nav ) ); ?></span>
+									<span class="title"><?= $enc->attr($this->translate('admin', $nav)); ?></span>
 								</span>
 								<ul class="tree-menu">
-									<li class="menu-header"><strong><?= $enc->html( $this->translate( 'admin', $nav ) ); ?></strong></li>
+									<li class="menu-header"><strong><?= $enc->html($this->translate('admin', $nav)); ?></strong></li>
 
-									<?php foreach( $navitem as $subresource ) : ?>
-										<?php if( $this->access( $this->config( 'admin/jqadm/resource/' . $subresource . '/groups', [] ) ) ) : ?>
-											<li class="<?= str_replace( '/', '-', $subresource); ?>">
-												<a href="<?= $enc->attr( $this->url( $searchTarget, $cntl, $action, ['resource' => $subresource] + $params, [], $config ) ); ?>">
-													<span class="name"><?= $enc->html( $this->translate( 'admin', $subresource ) ); ?></span>
+									<?php foreach ($navitem as $subresource) : ?>
+										<?php if ($this->access($this->config('admin/jqadm/resource/' . $subresource . '/groups', []))) : ?>
+											<li class="<?= str_replace('/', '-', $subresource); ?>">
+												<a href="<?= $enc->attr($this->url($searchTarget, $cntl, $action, ['resource' => $subresource] + $params, [], $config)); ?>">
+													<span class="name"><?= $enc->html($this->translate('admin', $subresource)); ?></span>
 												</a>
 											</li>
 										<?php endif; ?>
@@ -345,15 +345,15 @@ 
 
 						<?php endif; ?>
 					<?php else : ?>
-						<?php if( $this->access( $this->config( 'admin/jqadm/resource/' . $navitem . '/groups', [] ) ) ) : ?>
+						<?php if ($this->access($this->config('admin/jqadm/resource/' . $navitem . '/groups', []))) : ?>
 
-							<?php $key = $this->config( 'admin/jqadm/resource/' . $navitem . '/key' ); ?>
-							<li class="<?= $enc->attr( $navitem ); ?> <?= $this->param( 'resource', $navfirst ) === $navitem ? 'active' : '' ?>">
-								<a href="<?= $enc->attr( $this->url( $searchTarget, $cntl, $action, array( 'resource' => $navitem ) + $params, [], $config ) ); ?>"
-									title="<?= $enc->attr( sprintf( $title, $this->translate( 'admin', $navitem ), $key ) ); ?>"
-									data-ctrlkey="<?= $enc->attr( strtolower( $key ) ); ?>">
+							<?php $key = $this->config('admin/jqadm/resource/' . $navitem . '/key'); ?>
+							<li class="<?= $enc->attr($navitem); ?> <?= $this->param('resource', $navfirst) === $navitem ? 'active' : '' ?>">
+								<a href="<?= $enc->attr($this->url($searchTarget, $cntl, $action, array('resource' => $navitem) + $params, [], $config)); ?>"
+									title="<?= $enc->attr(sprintf($title, $this->translate('admin', $navitem), $key)); ?>"
+									data-ctrlkey="<?= $enc->attr(strtolower($key)); ?>">
 									<i class="icon"></i>
-									<span class="title"><?= $enc->html( $this->translate( 'admin', $navitem ) ); ?></span>
+									<span class="title"><?= $enc->html($this->translate('admin', $navitem)); ?></span>
 								</a>
 							</li>
 
@@ -361,18 +361,18 @@ 
 					<?php endif; ?>
 				<?php endforeach; ?>
 
-				<?php if( $this->access( $this->config( 'admin/jqadm/resource/language/groups', [] ) ) ) : ?>
+				<?php if ($this->access($this->config('admin/jqadm/resource/language/groups', []))) : ?>
 					<li class="language treeview">
 						<span>
 							<i class="icon"></i>
-							<span class="title"><?= $enc->attr( $this->translate( 'language', $this->param( 'lang', $this->translate( 'admin', 'Language' ) ) ) ); ?></span>
+							<span class="title"><?= $enc->attr($this->translate('language', $this->param('lang', $this->translate('admin', 'Language')))); ?></span>
 						</span>
 						<ul class="tree-menu">
-							<li class="menu-header"><strong><?= $enc->html( $this->translate( 'admin', 'Language' ) ); ?></strong></li>
-							<?php foreach( $this->get( 'pageI18nList', [] ) as $langid ) : ?>
-								<li class="lang-<?= $enc->attr( $langid ) ?>">
-									<a href="<?= $enc->attr( $this->url( $searchTarget, $cntl, $action, array( 'lang' => $langid ) + $params, [], $config ) ); ?>">
-										<span class="name"><?= $enc->html( $this->translate( 'language', $langid ) ); ?> (<?= $langid ?>)</span>
+							<li class="menu-header"><strong><?= $enc->html($this->translate('admin', 'Language')); ?></strong></li>
+							<?php foreach ($this->get('pageI18nList', []) as $langid) : ?>
+								<li class="lang-<?= $enc->attr($langid) ?>">
+									<a href="<?= $enc->attr($this->url($searchTarget, $cntl, $action, array('lang' => $langid) + $params, [], $config)); ?>">
+										<span class="name"><?= $enc->html($this->translate('language', $langid)); ?> (<?= $langid ?>)</span>
 									</a>
 								</li>
 							<?php endforeach; ?>
@@ -386,20 +386,20 @@ 
 
 	<main class="main-content">
 
-		<?= $this->partial( $this->config( 'admin/jqadm/partial/error', 'common/partials/error-standard' ), array( 'errors' => $this->get( 'errors', [] ) ) ); ?>
-		<?= $this->partial( $this->config( 'admin/jqadm/partial/info', 'common/partials/info-standard' ), array( 'info' => $infoMsgs ) ); ?>
+		<?= $this->partial($this->config('admin/jqadm/partial/error', 'common/partials/error-standard'), array('errors' => $this->get('errors', []))); ?>
+		<?= $this->partial($this->config('admin/jqadm/partial/info', 'common/partials/info-standard'), array('info' => $infoMsgs)); ?>
 
-		<?= $this->block()->get( 'jqadm_content' ); ?>
+		<?= $this->block()->get('jqadm_content'); ?>
 
 	</main>
 
 	<footer class="main-footer">
 		<a href="https://github.com/aimeos/ai-admin-jqadm/issues" target="_blank">
-			<?= $enc->html( $this->translate( 'admin', 'Bug or suggestion?' ) ); ?>
+			<?= $enc->html($this->translate('admin', 'Bug or suggestion?')); ?>
 		</a>
 	</footer>
 
-	<?= $this->partial( $this->config( 'admin/jqadm/partial/confirm', 'common/partials/confirm-standard' ) ); ?>
-	<?= $this->partial( $this->config( 'admin/jqadm/partial/problem', 'common/partials/problem-standard' ) ); ?>
+	<?= $this->partial($this->config('admin/jqadm/partial/confirm', 'common/partials/confirm-standard')); ?>
+	<?= $this->partial($this->config('admin/jqadm/partial/problem', 'common/partials/problem-standard')); ?>
 
 </div>

admin/jqadm/templates/supplier/item-product-standard.php

Braces +5 added lines, -2 removed lines

@@ -374,8 +374,11 @@
 									aria-label="<?= $enc->attr( $this->translate( 'admin', 'Show' ) ); ?>">
 									<?= $enc->html( $refId ); ?> - <?= $enc->html( $refItem->getLabel() . ' (' . $refItem->getCode() . ')' ); ?>
 								</a>
-							<?php else : ?>
-								<?= $enc->html( $refId ); ?> (<?= $enc->html( $this->translate( 'admin', 'not available any more' ) ); ?>)
+							<?php else {
+	: ?>
+								<?= $enc->html( $refId );
+}
+?> (<?= $enc->html( $this->translate( 'admin', 'not available any more' ) ); ?>)
 							<?php endif; ?>
 						</td>
 					<?php endif; ?>

Spacing +171 added lines, -171 removed lines

@@ -5,28 +5,28 @@ 
  * @copyright Aimeos (aimeos.org), 2017-2018
  */
 
-$selected = function( $key, $code ) {
-	return ( $key == $code ? 'selected="selected"' : '' );
+$selected = function($key, $code) {
+	return ($key == $code ? 'selected="selected"' : '');
 };
 
 
 $enc = $this->encoder();
-$params = $this->get( 'pageParams', [] );
+$params = $this->get('pageParams', []);
 
-$newTarget = $this->config( 'admin/jqadm/url/create/target' );
-$newCntl = $this->config( 'admin/jqadm/url/create/controller', 'Jqadm' );
-$newAction = $this->config( 'admin/jqadm/url/create/action', 'create' );
-$newConfig = $this->config( 'admin/jqadm/url/create/config', [] );
+$newTarget = $this->config('admin/jqadm/url/create/target');
+$newCntl = $this->config('admin/jqadm/url/create/controller', 'Jqadm');
+$newAction = $this->config('admin/jqadm/url/create/action', 'create');
+$newConfig = $this->config('admin/jqadm/url/create/config', []);
 
-$getTarget = $this->config( 'admin/jqadm/url/get/target' );
-$getCntl = $this->config( 'admin/jqadm/url/get/controller', 'Jqadm' );
-$getAction = $this->config( 'admin/jqadm/url/get/action', 'get' );
-$getConfig = $this->config( 'admin/jqadm/url/get/config', [] );
+$getTarget = $this->config('admin/jqadm/url/get/target');
+$getCntl = $this->config('admin/jqadm/url/get/controller', 'Jqadm');
+$getAction = $this->config('admin/jqadm/url/get/action', 'get');
+$getConfig = $this->config('admin/jqadm/url/get/config', []);
 
-$delTarget = $this->config( 'admin/jsonadm/url/target' );
-$delCntl = $this->config( 'admin/jsonadm/url/controller', 'Jsonadm' );
-$delAction = $this->config( 'admin/jsonadm/url/action', 'delete' );
-$delConfig = $this->config( 'admin/jsonadm/url/config', [] );
+$delTarget = $this->config('admin/jsonadm/url/target');
+$delCntl = $this->config('admin/jsonadm/url/controller', 'Jsonadm');
+$delAction = $this->config('admin/jsonadm/url/action', 'delete');
+$delConfig = $this->config('admin/jsonadm/url/config', []);
 
 
 /** admin/jqadm/supplier/product/fields
@@ -44,21 +44,21 @@ 
  * @category Developer
  */
 $default = ['supplier.lists.position', 'supplier.lists.status', 'supplier.lists.type', 'supplier.lists.config', 'supplier.lists.refid'];
-$default = $this->config( 'admin/jqadm/supplier/product/fields', $default );
-$fields = $this->session( 'aimeos/admin/jqadm/supplierproduct/fields', $default );
+$default = $this->config('admin/jqadm/supplier/product/fields', $default);
+$fields = $this->session('aimeos/admin/jqadm/supplierproduct/fields', $default);
 
-$listItems = $this->get( 'productListItems', [] );
-$refItems = $this->get( 'productItems', [] );
+$listItems = $this->get('productListItems', []);
+$refItems = $this->get('productItems', []);
 
 
 ?>
 <div id="product" class="item-product content-block tab-pane fade" role="tabpanel" aria-labelledby="product">
 
 	<?= $this->partial(
-			$this->config( 'admin/jqadm/partial/pagination', 'common/partials/pagination-standard' ),
-			['pageParams' => $params, 'pos' => 'top', 'total' => $this->get( 'productTotal' ),
+			$this->config('admin/jqadm/partial/pagination', 'common/partials/pagination-standard'),
+			['pageParams' => $params, 'pos' => 'top', 'total' => $this->get('productTotal'),
 			'group' => 'sp', 'action' => 'get', 'fragment' => 'product',
-			'page' => $this->session( 'aimeos/admin/jqadm/supplierproduct/page', [] )]
+			'page' => $this->session('aimeos/admin/jqadm/supplierproduct/page', [])]
 		);
 	?>
 
@@ -66,39 +66,39 @@ 
 		<thead class="list-header">
 			<tr>
 				<?= $this->partial(
-					$this->config( 'admin/jqadm/partial/listhead', 'common/partials/listhead-standard' ), [
-						'fields' => $fields, 'params' => $params, 'tabindex' => $this->get( 'tabindex' ),
+					$this->config('admin/jqadm/partial/listhead', 'common/partials/listhead-standard'), [
+						'fields' => $fields, 'params' => $params, 'tabindex' => $this->get('tabindex'),
 						'group' => 'sp', 'action' => 'get', 'fragment' => 'product',
-						'sort' => $this->session( 'aimeos/admin/jqadm/supplierproduct/sort' ),
+						'sort' => $this->session('aimeos/admin/jqadm/supplierproduct/sort'),
 						'data' => [
-							'supplier.lists.position' => $this->translate( 'admin', 'Position' ),
-							'supplier.lists.status' => $this->translate( 'admin', 'Status' ),
-							'supplier.lists.type' => $this->translate( 'admin', 'Type' ),
-							'supplier.lists.config' => $this->translate( 'admin', 'Config' ),
-							'supplier.lists.datestart' => $this->translate( 'admin', 'Start date' ),
-							'supplier.lists.dateend' => $this->translate( 'admin', 'End date' ),
-							'supplier.lists.refid' => $this->translate( 'admin', 'Product ID' ),
+							'supplier.lists.position' => $this->translate('admin', 'Position'),
+							'supplier.lists.status' => $this->translate('admin', 'Status'),
+							'supplier.lists.type' => $this->translate('admin', 'Type'),
+							'supplier.lists.config' => $this->translate('admin', 'Config'),
+							'supplier.lists.datestart' => $this->translate('admin', 'Start date'),
+							'supplier.lists.dateend' => $this->translate('admin', 'End date'),
+							'supplier.lists.refid' => $this->translate('admin', 'Product ID'),
 						]
 					] );
 				?>
 
 				<th class="actions">
-					<a class="btn fa act-add" href="#" tabindex="<?= $this->get( 'tabindex' ); ?>"
-						title="<?= $enc->attr( $this->translate( 'admin', 'Insert new entry (Ctrl+I)') ); ?>"
-						aria-label="<?= $enc->attr( $this->translate( 'admin', 'Add' ) ); ?>">
+					<a class="btn fa act-add" href="#" tabindex="<?= $this->get('tabindex'); ?>"
+						title="<?= $enc->attr($this->translate('admin', 'Insert new entry (Ctrl+I)')); ?>"
+						aria-label="<?= $enc->attr($this->translate('admin', 'Add')); ?>">
 					</a>
 
 					<?= $this->partial(
-						$this->config( 'admin/jqadm/partial/columns', 'common/partials/columns-standard' ), [
-							'fields' => $fields, 'group' => 'sp', 'tabindex' => $this->get( 'tabindex' ),
+						$this->config('admin/jqadm/partial/columns', 'common/partials/columns-standard'), [
+							'fields' => $fields, 'group' => 'sp', 'tabindex' => $this->get('tabindex'),
 							'data' => [
-								'supplier.lists.position' => $this->translate( 'admin', 'Position' ),
-								'supplier.lists.status' => $this->translate( 'admin', 'Status' ),
-								'supplier.lists.type' => $this->translate( 'admin', 'Type' ),
-								'supplier.lists.config' => $this->translate( 'admin', 'Config' ),
-								'supplier.lists.datestart' => $this->translate( 'admin', 'Start date' ),
-								'supplier.lists.dateend' => $this->translate( 'admin', 'End date' ),
-								'supplier.lists.refid' => $this->translate( 'admin', 'Product' ),
+								'supplier.lists.position' => $this->translate('admin', 'Position'),
+								'supplier.lists.status' => $this->translate('admin', 'Status'),
+								'supplier.lists.type' => $this->translate('admin', 'Type'),
+								'supplier.lists.config' => $this->translate('admin', 'Config'),
+								'supplier.lists.datestart' => $this->translate('admin', 'Start date'),
+								'supplier.lists.dateend' => $this->translate('admin', 'End date'),
+								'supplier.lists.refid' => $this->translate('admin', 'Product'),
 							]
 						] );
 					?>
@@ -107,18 +107,18 @@ 
 		</thead>
 		<tbody>
 			<?= $this->partial(
-				$this->config( 'admin/jqadm/partial/listsearch', 'common/partials/listsearch-standard' ), [
-					'filter' => $this->session( 'aimeos/admin/jqadm/supplierproduct/filter', [] ),
-					'fields' => $fields, 'group' => 'sp', 'tabindex' => $this->get( 'tabindex' ),
+				$this->config('admin/jqadm/partial/listsearch', 'common/partials/listsearch-standard'), [
+					'filter' => $this->session('aimeos/admin/jqadm/supplierproduct/filter', []),
+					'fields' => $fields, 'group' => 'sp', 'tabindex' => $this->get('tabindex'),
 					'data' => [
 						'supplier.lists.position' => ['op' => '>=', 'type' => 'number'],
 						'supplier.lists.status' => ['op' => '==', 'type' => 'select', 'val' => [
-							'1' => $this->translate( 'mshop/code', 'status:1' ),
-							'0' => $this->translate( 'mshop/code', 'status:0' ),
-							'-1' => $this->translate( 'mshop/code', 'status:-1' ),
-							'-2' => $this->translate( 'mshop/code', 'status:-2' ),
+							'1' => $this->translate('mshop/code', 'status:1'),
+							'0' => $this->translate('mshop/code', 'status:0'),
+							'-1' => $this->translate('mshop/code', 'status:-1'),
+							'-2' => $this->translate('mshop/code', 'status:-2'),
 						]],
-						'supplier.lists.type' => ['op' => '==', 'type' => 'select', 'val' => array_keys( $this->get( 'productListTypes', [] ) )],
+						'supplier.lists.type' => ['op' => '==', 'type' => 'select', 'val' => array_keys($this->get('productListTypes', []))],
 						'supplier.lists.config' => ['op' => '~='],
 						'supplier.lists.datestart' => ['op' => '>=', 'type' => 'datetime-local'],
 						'supplier.lists.dateend' => ['op' => '>=', 'type' => 'datetime-local'],
@@ -128,80 +128,80 @@ 
 			?>
 
 			<tr class="list-item-new prototype">
-				<td colspan="<?= count( $fields ); ?>">
+				<td colspan="<?= count($fields); ?>">
 					<div class="content-block row">
 						<div class="col-xl-6">
 							<div class="form-group row mandatory">
-								<label class="col-sm-4 form-control-label"><?= $enc->html( $this->translate( 'admin', 'Product' ) ); ?></label>
+								<label class="col-sm-4 form-control-label"><?= $enc->html($this->translate('admin', 'Product')); ?></label>
 								<div class="col-sm-8">
 									<input class="item-listid" type="hidden" disabled="disabled"
-										name="<?= $enc->attr( $this->formparam( array( 'product', 'supplier.lists.id', '' ) ) ); ?>" />
+										name="<?= $enc->attr($this->formparam(array('product', 'supplier.lists.id', ''))); ?>" />
 									<input class="item-config" type="hidden" disabled="disabled"
-										name="<?= $enc->attr( $this->formparam( array( 'product', 'supplier.lists.config', '' ) ) ); ?>" />
+										name="<?= $enc->attr($this->formparam(array('product', 'supplier.lists.config', ''))); ?>" />
 									<input class="item-label" type="hidden" disabled="disabled"
-										name="<?= $enc->attr( $this->formparam( array( 'product', 'product.label', '' ) ) ); ?>" />
-									<select class="combobox-prototype item-refid" tabindex="<?= $this->get( 'tabindex' ); ?>" disabled="disabled"
-										name="<?= $enc->attr( $this->formparam( array( 'product', 'supplier.lists.refid', '' ) ) ); ?>">
+										name="<?= $enc->attr($this->formparam(array('product', 'product.label', ''))); ?>" />
+									<select class="combobox-prototype item-refid" tabindex="<?= $this->get('tabindex'); ?>" disabled="disabled"
+										name="<?= $enc->attr($this->formparam(array('product', 'supplier.lists.refid', ''))); ?>">
 									</select>
 								</div>
 							</div>
 							<div class="form-group row mandatory">
-								<label class="col-sm-4 form-control-label"><?= $enc->html( $this->translate( 'admin', 'Status' ) ); ?></label>
+								<label class="col-sm-4 form-control-label"><?= $enc->html($this->translate('admin', 'Status')); ?></label>
 								<div class="col-sm-8">
-									<select class="form-control custom-select item-status" required="required" tabindex="<?= $this->get( 'tabindex' ); ?>" disabled="disabled"
-										name="<?= $enc->attr( $this->formparam( array( 'product', 'supplier.lists.status', '' ) ) ); ?>">
+									<select class="form-control custom-select item-status" required="required" tabindex="<?= $this->get('tabindex'); ?>" disabled="disabled"
+										name="<?= $enc->attr($this->formparam(array('product', 'supplier.lists.status', ''))); ?>">
 										<option value="">
-											<?= $enc->html( $this->translate( 'admin', 'Please select' ) ); ?>
+											<?= $enc->html($this->translate('admin', 'Please select')); ?>
 										</option>
 										<option value="1">
-											<?= $enc->html( $this->translate( 'mshop/code', 'status:1' ) ); ?>
+											<?= $enc->html($this->translate('mshop/code', 'status:1')); ?>
 										</option>
 										<option value="0">
-											<?= $enc->html( $this->translate( 'mshop/code', 'status:0' ) ); ?>
+											<?= $enc->html($this->translate('mshop/code', 'status:0')); ?>
 										</option>
 										<option value="-1">
-											<?= $enc->html( $this->translate( 'mshop/code', 'status:-1' ) ); ?>
+											<?= $enc->html($this->translate('mshop/code', 'status:-1')); ?>
 										</option>
 										<option value="-2">
-											<?= $enc->html( $this->translate( 'mshop/code', 'status:-2' ) ); ?>
+											<?= $enc->html($this->translate('mshop/code', 'status:-2')); ?>
 										</option>
 									</select>
 								</div>
 							</div>
 							<div class="form-group row mandatory">
-								<label class="col-sm-4 form-control-label"><?= $enc->html( $this->translate( 'admin', 'Type' ) ); ?></label>
+								<label class="col-sm-4 form-control-label"><?= $enc->html($this->translate('admin', 'Type')); ?></label>
 								<div class="col-sm-8">
-									<select class="form-control custom-select item-type" required="required" tabindex="<?= $this->get( 'tabindex' ); ?>" disabled="disabled"
-										name="<?= $enc->attr( $this->formparam( array( 'product', 'supplier.lists.type', '' ) ) ); ?>" >
+									<select class="form-control custom-select item-type" required="required" tabindex="<?= $this->get('tabindex'); ?>" disabled="disabled"
+										name="<?= $enc->attr($this->formparam(array('product', 'supplier.lists.type', ''))); ?>" >
 										<option value="">
-											<?= $enc->html( $this->translate( 'admin', 'Please select' ) ); ?>
+											<?= $enc->html($this->translate('admin', 'Please select')); ?>
 										</option>
 
-										<?php foreach( $this->get( 'productListTypes', [] ) as $type => $item ) : ?>
-											<option value="<?= $enc->attr( $type ); ?>"><?= $enc->html( $type ); ?></option>
+										<?php foreach ($this->get('productListTypes', []) as $type => $item) : ?>
+											<option value="<?= $enc->attr($type); ?>"><?= $enc->html($type); ?></option>
 										<?php endforeach; ?>
 									</select>
 								</div>
 							</div>
 							<div class="form-group row optional">
-								<label class="col-sm-4 form-control-label"><?= $enc->html( $this->translate( 'admin', 'Start date' ) ); ?></label>
+								<label class="col-sm-4 form-control-label"><?= $enc->html($this->translate('admin', 'Start date')); ?></label>
 								<div class="col-sm-8">
-									<input class="form-control item-datestart" type="datetime-local" tabindex="<?= $this->get( 'tabindex' ); ?>"
-										name="<?= $enc->attr( $this->formparam( array( 'product', 'supplier.lists.datestart', '' ) ) ); ?>" disabled="disabled" />
+									<input class="form-control item-datestart" type="datetime-local" tabindex="<?= $this->get('tabindex'); ?>"
+										name="<?= $enc->attr($this->formparam(array('product', 'supplier.lists.datestart', ''))); ?>" disabled="disabled" />
 								</div>
 							</div>
 							<div class="form-group row optional">
-								<label class="col-sm-4 form-control-label"><?= $enc->html( $this->translate( 'admin', 'End date' ) ); ?></label>
+								<label class="col-sm-4 form-control-label"><?= $enc->html($this->translate('admin', 'End date')); ?></label>
 								<div class="col-sm-8">
-									<input class="form-control item-dateend" type="datetime-local" tabindex="<?= $this->get( 'tabindex' ); ?>"
-										name="<?= $enc->attr( $this->formparam( array( 'product', 'supplier.lists.dateend', '' ) ) ); ?>" disabled="disabled" />
+									<input class="form-control item-dateend" type="datetime-local" tabindex="<?= $this->get('tabindex'); ?>"
+										name="<?= $enc->attr($this->formparam(array('product', 'supplier.lists.dateend', ''))); ?>" disabled="disabled" />
 								</div>
 							</div>
 							<div class="form-group row optional">
-								<label class="col-sm-4 form-control-label"><?= $enc->html( $this->translate( 'admin', 'Position' ) ); ?></label>
+								<label class="col-sm-4 form-control-label"><?= $enc->html($this->translate('admin', 'Position')); ?></label>
 								<div class="col-sm-8">
-									<input class="form-control item-position" type="number" step="1" tabindex="<?= $this->get( 'tabindex' ); ?>"
-										name="<?= $enc->attr( $this->formparam( array( 'product', 'supplier.lists.position', '' ) ) ); ?>" disabled="disabled" />
+									<input class="form-control item-position" type="number" step="1" tabindex="<?= $this->get('tabindex'); ?>"
+										name="<?= $enc->attr($this->formparam(array('product', 'supplier.lists.position', ''))); ?>" disabled="disabled" />
 								</div>
 							</div>
 						</div>
@@ -210,17 +210,17 @@ 
 								<thead>
 									<tr>
 										<th>
-											<span class="help"><?= $enc->html( $this->translate( 'admin', 'Option' ) ); ?></span>
+											<span class="help"><?= $enc->html($this->translate('admin', 'Option')); ?></span>
 											<div class="form-text text-muted help-text">
-												<?= $enc->html( $this->translate( 'admin', 'Article specific configuration options, will be available as key/value pairs in the templates' ) ); ?>
+												<?= $enc->html($this->translate('admin', 'Article specific configuration options, will be available as key/value pairs in the templates')); ?>
 											</div>
 										</th>
 										<th>
-											<?= $enc->html( $this->translate( 'admin', 'Value' ) ); ?>
+											<?= $enc->html($this->translate('admin', 'Value')); ?>
 										</th>
 										<th class="actions">
-											<div class="btn act-add fa" tabindex="<?= $this->get( 'tabindex' ); ?>"
-												title="<?= $enc->attr( $this->translate( 'admin', 'Insert new entry (Ctrl+I)') ); ?>">
+											<div class="btn act-add fa" tabindex="<?= $this->get('tabindex'); ?>"
+												title="<?= $enc->attr($this->translate('admin', 'Insert new entry (Ctrl+I)')); ?>">
 											</div>
 										</th>
 									</tr>
@@ -228,16 +228,16 @@ 
 								<tbody>
 									<tr class="prototype">
 										<td>
-											<input type="text" class="config-key form-control" tabindex="<?= $this->get( 'tabindex' ); ?>" disabled="disabled"
-												name="<?= $enc->attr( $this->formparam( array( 'product', 'config', 'idx', 'key', '' ) ) ); ?>" />
+											<input type="text" class="config-key form-control" tabindex="<?= $this->get('tabindex'); ?>" disabled="disabled"
+												name="<?= $enc->attr($this->formparam(array('product', 'config', 'idx', 'key', ''))); ?>" />
 										</td>
 										<td>
-											<input type="text" class="config-value form-control" tabindex="<?= $this->get( 'tabindex' ); ?>" disabled="disabled"
-												name="<?= $enc->attr( $this->formparam( array( 'product', 'config', 'idx', 'val', '' ) ) ); ?>" />
+											<input type="text" class="config-value form-control" tabindex="<?= $this->get('tabindex'); ?>" disabled="disabled"
+												name="<?= $enc->attr($this->formparam(array('product', 'config', 'idx', 'val', ''))); ?>" />
 										</td>
 										<td class="actions">
-											<div class="btn act-delete fa" tabindex="<?= $this->get( 'tabindex' ); ?>"
-												title="<?= $enc->attr( $this->translate( 'admin', 'Delete this entry') ); ?>">
+											<div class="btn act-delete fa" tabindex="<?= $this->get('tabindex'); ?>"
+												title="<?= $enc->attr($this->translate('admin', 'Delete this entry')); ?>">
 											</div>
 										</td>
 									</tr>
@@ -247,79 +247,79 @@ 
 					</div>
 				</td>
 				<td class="actions">
-					<a class="btn fa act-close" href="#" tabindex="<?= $this->get( 'tabindex' ); ?>"
-						title="<?= $enc->attr( $this->translate( 'admin', 'Close') ); ?>"
-						aria-label="<?= $enc->attr( $this->translate( 'admin', 'Close' ) ); ?>">
+					<a class="btn fa act-close" href="#" tabindex="<?= $this->get('tabindex'); ?>"
+						title="<?= $enc->attr($this->translate('admin', 'Close')); ?>"
+						aria-label="<?= $enc->attr($this->translate('admin', 'Close')); ?>">
 					</a>
 				</td>
 			</tr>
 
-			<?php foreach( $this->get( 'productData/supplier.lists.id', [] ) as $idx => $listId ) : ?>
-				<?php $siteId = $this->get( 'productData/supplier.lists.siteid/' . $idx ); ?>
-				<?php $refId = $this->get( 'productData/supplier.lists.refid/' . $idx ); ?>
+			<?php foreach ($this->get('productData/supplier.lists.id', []) as $idx => $listId) : ?>
+				<?php $siteId = $this->get('productData/supplier.lists.siteid/' . $idx); ?>
+				<?php $refId = $this->get('productData/supplier.lists.refid/' . $idx); ?>
 
-				<tr class="list-item <?= $this->site()->readonly( $siteId ); ?>">
-					<?php if( in_array( 'supplier.lists.position', $fields ) ) : ?>
+				<tr class="list-item <?= $this->site()->readonly($siteId); ?>">
+					<?php if (in_array('supplier.lists.position', $fields)) : ?>
 						<td class="supplier-lists-position">
-							<input class="form-control item-position" type="number" step="1" tabindex="<?= $this->get( 'tabindex' ); ?>"
-								name="<?= $enc->attr( $this->formparam( array( 'product', 'supplier.lists.position', '' ) ) ); ?>"
-								value="<?= $enc->attr( $this->get( 'productData/supplier.lists.position/' . $idx ) ); ?>"
-								<?= $this->site()->readonly( $siteId ); ?> disabled="disabled" />
+							<input class="form-control item-position" type="number" step="1" tabindex="<?= $this->get('tabindex'); ?>"
+								name="<?= $enc->attr($this->formparam(array('product', 'supplier.lists.position', ''))); ?>"
+								value="<?= $enc->attr($this->get('productData/supplier.lists.position/' . $idx)); ?>"
+								<?= $this->site()->readonly($siteId); ?> disabled="disabled" />
 						</td>
 					<?php endif; ?>
-					<?php if( in_array( 'supplier.lists.status', $fields ) ) : ?>
+					<?php if (in_array('supplier.lists.status', $fields)) : ?>
 						<td class="supplier-lists-status">
-							<select class="form-control custom-select item-status" required="required" tabindex="<?= $this->get( 'tabindex' ); ?>"
-								name="<?= $enc->attr( $this->formparam( array( 'product', 'supplier.lists.status', '' ) ) ); ?>"
-								<?= $this->site()->readonly( $siteId ); ?> disabled="disabled" >
+							<select class="form-control custom-select item-status" required="required" tabindex="<?= $this->get('tabindex'); ?>"
+								name="<?= $enc->attr($this->formparam(array('product', 'supplier.lists.status', ''))); ?>"
+								<?= $this->site()->readonly($siteId); ?> disabled="disabled" >
 								<option value="">
-									<?= $enc->html( $this->translate( 'admin', 'Please select' ) ); ?>
+									<?= $enc->html($this->translate('admin', 'Please select')); ?>
 								</option>
-								<option value="1" <?= $selected( $this->get( 'productData/supplier.lists.status/' . $idx, 1 ), 1 ); ?> >
-									<?= $enc->html( $this->translate( 'mshop/code', 'status:1' ) ); ?>
+								<option value="1" <?= $selected($this->get('productData/supplier.lists.status/' . $idx, 1), 1); ?> >
+									<?= $enc->html($this->translate('mshop/code', 'status:1')); ?>
 								</option>
-								<option value="0" <?= $selected( $this->get( 'productData/supplier.lists.status/' . $idx, 1 ), 0 ); ?> >
-									<?= $enc->html( $this->translate( 'mshop/code', 'status:0' ) ); ?>
+								<option value="0" <?= $selected($this->get('productData/supplier.lists.status/' . $idx, 1), 0); ?> >
+									<?= $enc->html($this->translate('mshop/code', 'status:0')); ?>
 								</option>
-								<option value="-1" <?= $selected( $this->get( 'productData/supplier.lists.status/' . $idx, 1 ), -1 ); ?> >
-									<?= $enc->html( $this->translate( 'mshop/code', 'status:-1' ) ); ?>
+								<option value="-1" <?= $selected($this->get('productData/supplier.lists.status/' . $idx, 1), -1); ?> >
+									<?= $enc->html($this->translate('mshop/code', 'status:-1')); ?>
 								</option>
-								<option value="-2" <?= $selected( $this->get( 'productData/supplier.lists.status/' . $idx, 1 ), -2 ); ?> >
-									<?= $enc->html( $this->translate( 'mshop/code', 'status:-2' ) ); ?>
+								<option value="-2" <?= $selected($this->get('productData/supplier.lists.status/' . $idx, 1), -2); ?> >
+									<?= $enc->html($this->translate('mshop/code', 'status:-2')); ?>
 								</option>
 							</select>
 						</td>
 					<?php endif; ?>
-					<?php if( in_array( 'supplier.lists.type', $fields ) ) : ?>
+					<?php if (in_array('supplier.lists.type', $fields)) : ?>
 						<td class="supplier-lists-type">
-							<select class="form-control custom-select item-type" required="required" tabindex="<?= $this->get( 'tabindex' ); ?>"
-								name="<?= $enc->attr( $this->formparam( array( 'product', 'supplier.lists.type', '' ) ) ); ?>"
-								<?= $this->site()->readonly( $siteId ); ?> disabled="disabled" >
+							<select class="form-control custom-select item-type" required="required" tabindex="<?= $this->get('tabindex'); ?>"
+								name="<?= $enc->attr($this->formparam(array('product', 'supplier.lists.type', ''))); ?>"
+								<?= $this->site()->readonly($siteId); ?> disabled="disabled" >
 								<option value="">
-									<?= $enc->html( $this->translate( 'admin', 'Please select' ) ); ?>
+									<?= $enc->html($this->translate('admin', 'Please select')); ?>
 								</option>
 
-								<?php foreach( $this->get( 'productListTypes', [] ) as $type => $item ) : ?>
-									<option value="<?= $enc->attr( $type ); ?>" <?= $selected( $this->get( 'productData/supplier.lists.type/' . $idx ), $type ); ?> >
-										<?= $enc->html( $type ); ?>
+								<?php foreach ($this->get('productListTypes', []) as $type => $item) : ?>
+									<option value="<?= $enc->attr($type); ?>" <?= $selected($this->get('productData/supplier.lists.type/' . $idx), $type); ?> >
+										<?= $enc->html($type); ?>
 									</option>
 								<?php endforeach; ?>
 							</select>
 						</td>
 					<?php endif; ?>
-					<?php if( in_array( 'supplier.lists.config', $fields ) ) : ?>
+					<?php if (in_array('supplier.lists.config', $fields)) : ?>
 						<td class="supplier-lists-config item-config">
 							<div class="config-type config-type-map">
-								<input type="text" class="config-value form-control" tabindex="<?= $this->get( 'tabindex' ); ?>"
-									name="<?= $enc->attr( $this->formparam( array( 'product', 'supplier.lists.config', '' ) ) ); ?>"
-									value="<?= $enc->attr( json_encode( $this->get( 'productData/supplier.lists.config/' . $idx ) ) ); ?>"
-									<?= $this->site()->readonly( $siteId ); ?> disabled="disabled" />
+								<input type="text" class="config-value form-control" tabindex="<?= $this->get('tabindex'); ?>"
+									name="<?= $enc->attr($this->formparam(array('product', 'supplier.lists.config', ''))); ?>"
+									value="<?= $enc->attr(json_encode($this->get('productData/supplier.lists.config/' . $idx))); ?>"
+									<?= $this->site()->readonly($siteId); ?> disabled="disabled" />
 
 								<table class="table table-striped config-map-table">
 									<tr class="config-map-row prototype-map">
 										<td class="config-map-actions">
 											<div class="btn act-delete fa" tabindex="1"
-												title="<?= $enc->attr( $this->translate( 'admin', 'Delete this entry') ); ?>">
+												title="<?= $enc->attr($this->translate('admin', 'Delete this entry')); ?>">
 											</div>
 										</td>
 										<td class="config-map-row-key">
@@ -332,12 +332,12 @@ 
 									<tr class="config-map-actions">
 										<td class="config-map-action-add">
 											<div class="btn act-add fa" tabindex="1"
-												title="<?= $enc->attr( $this->translate( 'admin', 'Insert new entry') ); ?>">
+												title="<?= $enc->attr($this->translate('admin', 'Insert new entry')); ?>">
 											</div>
 										</td>
 										<td class="config-map-action-update" colspan="2">
 											<div class="btn btn-primary act-update" tabindex="1">
-												<?= $enc->attr( $this->translate( 'admin', 'OK') ); ?>
+												<?= $enc->attr($this->translate('admin', 'OK')); ?>
 											</div>
 										</td>
 									</tr>
@@ -345,55 +345,55 @@ 
 							</div>
 						</td>
 					<?php endif; ?>
-					<?php if( in_array( 'supplier.lists.datestart', $fields ) ) : ?>
+					<?php if (in_array('supplier.lists.datestart', $fields)) : ?>
 						<td class="supplier-lists-datestart">
-							<input class="form-control item-datestart" type="datetime-local" tabindex="<?= $this->get( 'tabindex' ); ?>"
-								name="<?= $enc->attr( $this->formparam( array( 'product', 'supplier.lists.datestart', '' ) ) ); ?>"
-								value="<?= $enc->attr( str_replace( ' ', 'T', $this->get( 'productData/supplier.lists.datestart/' . $idx ) ) ); ?>"
-								<?= $this->site()->readonly( $siteId ); ?> disabled="disabled" />
+							<input class="form-control item-datestart" type="datetime-local" tabindex="<?= $this->get('tabindex'); ?>"
+								name="<?= $enc->attr($this->formparam(array('product', 'supplier.lists.datestart', ''))); ?>"
+								value="<?= $enc->attr(str_replace(' ', 'T', $this->get('productData/supplier.lists.datestart/' . $idx))); ?>"
+								<?= $this->site()->readonly($siteId); ?> disabled="disabled" />
 						</td>
 					<?php endif; ?>
-					<?php if( in_array( 'supplier.lists.dateend', $fields ) ) : ?>
+					<?php if (in_array('supplier.lists.dateend', $fields)) : ?>
 						<td class="supplier-lists-dateend">
-							<input class="form-control item-dateend" type="datetime-local" tabindex="<?= $this->get( 'tabindex' ); ?>"
-								name="<?= $enc->attr( $this->formparam( array( 'product', 'supplier.lists.dateend', '' ) ) ); ?>"
-								value="<?= $enc->attr( str_replace( ' ', 'T', $this->get( 'productData/supplier.lists.dateend/' . $idx ) ) ); ?>"
-								<?= $this->site()->readonly( $siteId ); ?> disabled="disabled" />
+							<input class="form-control item-dateend" type="datetime-local" tabindex="<?= $this->get('tabindex'); ?>"
+								name="<?= $enc->attr($this->formparam(array('product', 'supplier.lists.dateend', ''))); ?>"
+								value="<?= $enc->attr(str_replace(' ', 'T', $this->get('productData/supplier.lists.dateend/' . $idx))); ?>"
+								<?= $this->site()->readonly($siteId); ?> disabled="disabled" />
 						</td>
 					<?php endif; ?>
 
-					<?php $refItem = ( isset( $refItems[$refId] ) ? $refItems[$refId] : null ); ?>
+					<?php $refItem = (isset($refItems[$refId]) ? $refItems[$refId] : null); ?>
 
-					<?php if( in_array( 'supplier.lists.refid', $fields ) ) : ?>
+					<?php if (in_array('supplier.lists.refid', $fields)) : ?>
 						<td class="supplier-lists-refid">
 							<input class="form-control item-refid" type="hidden"
-								name="<?= $enc->attr( $this->formparam( array( 'product', 'supplier.lists.refid', '' ) ) ); ?>"
-								value="<?= $enc->attr( $refId ); ?>" disabled="disabled" />
-							<?php if( $refItem ) : ?>
-								<a class="btn act-view fa item-refid" tabindex="<?= $this->get( 'tabindex' ); ?>" target="_blank"
-									href="<?= $enc->attr( $this->url( $getTarget, $getCntl, $getAction, ['resource' => 'product', 'id' => $refId] + $params, [], $getConfig ) ); ?>"
-									title="<?= $enc->attr( $this->translate( 'admin', 'Show entry') ); ?>"
-									aria-label="<?= $enc->attr( $this->translate( 'admin', 'Show' ) ); ?>">
-									<?= $enc->html( $refId ); ?> - <?= $enc->html( $refItem->getLabel() . ' (' . $refItem->getCode() . ')' ); ?>
+								name="<?= $enc->attr($this->formparam(array('product', 'supplier.lists.refid', ''))); ?>"
+								value="<?= $enc->attr($refId); ?>" disabled="disabled" />
+							<?php if ($refItem) : ?>
+								<a class="btn act-view fa item-refid" tabindex="<?= $this->get('tabindex'); ?>" target="_blank"
+									href="<?= $enc->attr($this->url($getTarget, $getCntl, $getAction, ['resource' => 'product', 'id' => $refId] + $params, [], $getConfig)); ?>"
+									title="<?= $enc->attr($this->translate('admin', 'Show entry')); ?>"
+									aria-label="<?= $enc->attr($this->translate('admin', 'Show')); ?>">
+									<?= $enc->html($refId); ?> - <?= $enc->html($refItem->getLabel() . ' (' . $refItem->getCode() . ')'); ?>
 								</a>
 							<?php else : ?>
-								<?= $enc->html( $refId ); ?> (<?= $enc->html( $this->translate( 'admin', 'not available any more' ) ); ?>)
+								<?= $enc->html($refId); ?> (<?= $enc->html($this->translate('admin', 'not available any more')); ?>)
 							<?php endif; ?>
 						</td>
 					<?php endif; ?>
 
 					<td class="actions">
-						<input type="hidden" value="<?= $enc->attr( $listId ); ?>" disabled="disabled"
-							name="<?= $enc->attr( $this->formparam( array( 'product', 'supplier.lists.id', '' ) ) ); ?>" />
+						<input type="hidden" value="<?= $enc->attr($listId); ?>" disabled="disabled"
+							name="<?= $enc->attr($this->formparam(array('product', 'supplier.lists.id', ''))); ?>" />
 
-						<?php if( !$this->site()->readonly( $siteId ) ) : ?>
-							<a class="btn act-edit fa" tabindex="<?= $this->get( 'tabindex' ); ?>" href="#"
-								title="<?= $enc->attr( $this->translate( 'admin', 'Edit this entry') ); ?>"
-								aria-label="<?= $enc->attr( $this->translate( 'admin', 'Edit' ) ); ?>"></a>
-							<a class="btn act-delete fa" tabindex="<?= $this->get( 'tabindex' ); ?>"
-								href="<?= $enc->attr( $this->url( $delTarget, $delCntl, $delAction, ['resource' => 'supplier/lists', 'id' => $listId] + $params, [], $delConfig ) ); ?>"
-								title="<?= $enc->attr( $this->translate( 'admin', 'Delete this entry') ); ?>"
-								aria-label="<?= $enc->attr( $this->translate( 'admin', 'Delete' ) ); ?>"></a>
+						<?php if (!$this->site()->readonly($siteId)) : ?>
+							<a class="btn act-edit fa" tabindex="<?= $this->get('tabindex'); ?>" href="#"
+								title="<?= $enc->attr($this->translate('admin', 'Edit this entry')); ?>"
+								aria-label="<?= $enc->attr($this->translate('admin', 'Edit')); ?>"></a>
+							<a class="btn act-delete fa" tabindex="<?= $this->get('tabindex'); ?>"
+								href="<?= $enc->attr($this->url($delTarget, $delCntl, $delAction, ['resource' => 'supplier/lists', 'id' => $listId] + $params, [], $delConfig)); ?>"
+								title="<?= $enc->attr($this->translate('admin', 'Delete this entry')); ?>"
+								aria-label="<?= $enc->attr($this->translate('admin', 'Delete')); ?>"></a>
 						<?php endif; ?>
 					</td>
 				</tr>
@@ -402,17 +402,17 @@ 
 		</tbody>
 	</table>
 
-	<?php if( $this->get( 'productData', [] ) === [] ) : ?>
-		<div class="noitems"><?= $enc->html( sprintf( $this->translate( 'admin', 'No items found' ) ) ); ?></div>
+	<?php if ($this->get('productData', []) === []) : ?>
+		<div class="noitems"><?= $enc->html(sprintf($this->translate('admin', 'No items found'))); ?></div>
 	<?php endif; ?>
 
 	<?= $this->partial(
-			$this->config( 'admin/jqadm/partial/pagination', 'common/partials/pagination-standard' ),
-			['pageParams' => $params, 'pos' => 'bottom', 'total' => $this->get( 'productTotal' ),
+			$this->config('admin/jqadm/partial/pagination', 'common/partials/pagination-standard'),
+			['pageParams' => $params, 'pos' => 'bottom', 'total' => $this->get('productTotal'),
 			'group' => 'sp', 'action' => 'get', 'fragment' => 'product',
-			'page' =>$this->session( 'aimeos/admin/jqadm/supplierproduct/page', [] )]
+			'page' =>$this->session('aimeos/admin/jqadm/supplierproduct/page', [])]
 		);
 	?>
 
 </div>
-<?= $this->get( 'productBody' ); ?>
+<?= $this->get('productBody'); ?>

admin/jqadm/templates/dashboard/list-order-standard.php

Spacing +3 added lines, -3 removed lines

@@ -6,10 +6,10 @@
  */
 
 $enc = $this->encoder();
-$currencies = array_keys( $this->get( 'orderCurrencyItems', [] ) );
+$currencies = array_keys($this->get('orderCurrencyItems', []));
 
 
 ?>
-<div class="dashboard-order row" data-currencies="<?= $enc->attr( json_encode( $currencies ) ) ?>">
-	<?= $this->get( 'orderBody' ); ?>
+<div class="dashboard-order row" data-currencies="<?= $enc->attr(json_encode($currencies)) ?>">
+	<?= $this->get('orderBody'); ?>
 </div>

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

Braces +13 added lines, -26 removed lines

@@ -92,14 +92,12 @@ 
 				$view->tabindex = ++$idx + 1;
 				$view->itemBody .= $client->copy();
 			}
-		}
-		catch( \Aimeos\MShop\Exception $e )
+		} catch( \Aimeos\MShop\Exception $e )
 		{
 			$error = array( 'plugin-item' => $context->getI18n()->dt( 'mshop', $e->getMessage() ) );
 			$view->errors = $view->get( 'errors', [] ) + $error;
 			$this->logException( $e );
-		}
-		catch( \Exception $e )
+		} catch( \Exception $e )
 		{
 			$error = array( 'plugin-item' => $e->getMessage() . ', ' . $e->getFile() . ':' . $e->getLine() );
 			$view->errors = $view->get( 'errors', [] ) + $error;
@@ -144,14 +142,12 @@ 
 				$view->tabindex = ++$idx + 1;
 				$view->itemBody .= $client->create();
 			}
-		}
-		catch( \Aimeos\MShop\Exception $e )
+		} catch( \Aimeos\MShop\Exception $e )
 		{
 			$error = array( 'plugin-item' => $context->getI18n()->dt( 'mshop', $e->getMessage() ) );
 			$view->errors = $view->get( 'errors', [] ) + $error;
 			$this->logException( $e );
-		}
-		catch( \Exception $e )
+		} catch( \Exception $e )
 		{
 			$error = array( 'plugin-item' => $e->getMessage() . ', ' . $e->getFile() . ':' . $e->getLine() );
 			$view->errors = $view->get( 'errors', [] ) + $error;
@@ -192,14 +188,12 @@ 
 
 			$this->nextAction( $view, 'search', 'plugin', null, 'delete' );
 			return;
-		}
-		catch( \Aimeos\MShop\Exception $e )
+		} catch( \Aimeos\MShop\Exception $e )
 		{
 			$error = array( 'plugin-item' => $context->getI18n()->dt( 'mshop', $e->getMessage() ) );
 			$view->errors = $view->get( 'errors', [] ) + $error;
 			$this->logException( $e );
-		}
-		catch( \Exception $e )
+		} catch( \Exception $e )
 		{
 			$error = array( 'plugin-item' => $e->getMessage() . ', ' . $e->getFile() . ':' . $e->getLine() );
 			$view->errors = $view->get( 'errors', [] ) + $error;
@@ -244,14 +238,12 @@ 
 				$view->tabindex = ++$idx + 1;
 				$view->itemBody .= $client->get();
 			}
-		}
-		catch( \Aimeos\MShop\Exception $e )
+		} catch( \Aimeos\MShop\Exception $e )
 		{
 			$error = array( 'plugin-item' => $context->getI18n()->dt( 'mshop', $e->getMessage() ) );
 			$view->errors = $view->get( 'errors', [] ) + $error;
 			$this->logException( $e );
-		}
-		catch( \Exception $e )
+		} catch( \Exception $e )
 		{
 			$error = array( 'plugin-item' => $e->getMessage() . ', ' . $e->getFile() . ':' . $e->getLine() );
 			$view->errors = $view->get( 'errors', [] ) + $error;
@@ -288,18 +280,15 @@ 
 
 			$this->nextAction( $view, $view->param( 'next' ), 'plugin', $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( 'plugin-item' => $context->getI18n()->dt( 'mshop', $e->getMessage() ) );
 			$view->errors = $view->get( 'errors', [] ) + $error;
 			$this->logException( $e );
-		}
-		catch( \Exception $e )
+		} catch( \Exception $e )
 		{
 			$error = array( 'plugin-item' => $e->getMessage() . ', ' . $e->getFile() . ':' . $e->getLine() );
 			$view->errors = $view->get( 'errors', [] ) + $error;
@@ -342,14 +331,12 @@ 
 			foreach( $this->getSubClients() as $client ) {
 				$view->itemBody .= $client->search();
 			}
-		}
-		catch( \Aimeos\MShop\Exception $e )
+		} catch( \Aimeos\MShop\Exception $e )
 		{
 			$error = array( 'plugin-item' => $context->getI18n()->dt( 'mshop', $e->getMessage() ) );
 			$view->errors = $view->get( 'errors', [] ) + $error;
 			$this->logException( $e );
-		}
-		catch( \Exception $e )
+		} catch( \Exception $e )
 		{
 			$error = array( 'plugin-item' => $e->getMessage() . ', ' . $e->getFile() . ':' . $e->getLine() );
 			$view->errors = $view->get( 'errors', [] ) + $error;

Doc Comments +1 added lines, -1 removed lines

@@ -522,7 +522,7 @@
 	/**
 	 * Returns the available plugin type items
 	 *
-	 * @return array List of item implementing \Aimeos\MShop\Common\Type\Iface
+	 * @return \Aimeos\MShop\Common\Item\Type\Iface[] List of item implementing \Aimeos\MShop\Common\Type\Iface
 	 */
 	protected function getTypeItems()
 	{

Spacing +122 added lines, -122 removed lines

@@ -10,7 +10,7 @@ 
 
 namespace Aimeos\Admin\JQAdm\Plugin;
 
-sprintf( 'plugin' ); // for translation
+sprintf('plugin'); // for translation
 
 
 /**
@@ -35,41 +35,41 @@ 
 
 		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, 'plugin' );
+			$manager = \Aimeos\MShop::create($context, 'plugin');
 
-			$view->item = $manager->getItem( $id );
-			$view->itemData = $this->toArray( $view->item, true );
+			$view->item = $manager->getItem($id);
+			$view->itemData = $this->toArray($view->item, true);
 			$view->itemSubparts = $this->getSubClientNames();
 			$view->itemProviders = $this->getProviderNames();
 			$view->itemDecorators = $this->getDecoratorNames();
-			$view->itemAttributes = $this->getConfigAttributes( $view->item );
+			$view->itemAttributes = $this->getConfigAttributes($view->item);
 			$view->itemTypes = $this->getTypeItems();
 			$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( 'plugin-item' => $context->getI18n()->dt( 'mshop', $e->getMessage() ) );
-			$view->errors = $view->get( 'errors', [] ) + $error;
-			$this->logException( $e );
+			$error = array('plugin-item' => $context->getI18n()->dt('mshop', $e->getMessage()));
+			$view->errors = $view->get('errors', []) + $error;
+			$this->logException($e);
 		}
-		catch( \Exception $e )
+		catch (\Exception $e)
 		{
-			$error = array( 'plugin-item' => $e->getMessage() . ', ' . $e->getFile() . ':' . $e->getLine() );
-			$view->errors = $view->get( 'errors', [] ) + $error;
-			$this->logException( $e );
+			$error = array('plugin-item' => $e->getMessage() . ', ' . $e->getFile() . ':' . $e->getLine());
+			$view->errors = $view->get('errors', []) + $error;
+			$this->logException($e);
 		}
 
-		return $this->render( $view );
+		return $this->render($view);
 	}
 
 
@@ -85,10 +85,10 @@ 
 
 		try
 		{
-			$data = $view->param( 'item', [] );
+			$data = $view->param('item', []);
 
-			if( !isset( $view->item ) ) {
-				$view->item = \Aimeos\MShop::create( $context, 'plugin' )->createItem();
+			if (!isset($view->item)) {
+				$view->item = \Aimeos\MShop::create($context, 'plugin')->createItem();
 			}
 
 			$data['plugin.siteid'] = $view->item->getSiteId();
@@ -100,26 +100,26 @@ 
 			$view->itemData = $data;
 			$view->itemBody = '';
 
-			foreach( $this->getSubClients() as $idx => $client )
+			foreach ($this->getSubClients() as $idx => $client)
 			{
 				$view->tabindex = ++$idx + 1;
 				$view->itemBody .= $client->create();
 			}
 		}
-		catch( \Aimeos\MShop\Exception $e )
+		catch (\Aimeos\MShop\Exception $e)
 		{
-			$error = array( 'plugin-item' => $context->getI18n()->dt( 'mshop', $e->getMessage() ) );
-			$view->errors = $view->get( 'errors', [] ) + $error;
-			$this->logException( $e );
+			$error = array('plugin-item' => $context->getI18n()->dt('mshop', $e->getMessage()));
+			$view->errors = $view->get('errors', []) + $error;
+			$this->logException($e);
 		}
-		catch( \Exception $e )
+		catch (\Exception $e)
 		{
-			$error = array( 'plugin-item' => $e->getMessage() . ', ' . $e->getFile() . ':' . $e->getLine() );
-			$view->errors = $view->get( 'errors', [] ) + $error;
-			$this->logException( $e );
+			$error = array('plugin-item' => $e->getMessage() . ', ' . $e->getFile() . ':' . $e->getLine());
+			$view->errors = $view->get('errors', []) + $error;
+			$this->logException($e);
 		}
 
-		return $this->render( $view );
+		return $this->render($view);
 	}
 
 
@@ -133,38 +133,38 @@ 
 		$view = $this->getView();
 		$context = $this->getContext();
 
-		$manager = \Aimeos\MShop::create( $context, 'plugin' );
+		$manager = \Aimeos\MShop::create($context, 'plugin');
 		$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', 'plugin', null, 'delete' );
+			$this->nextAction($view, 'search', 'plugin', null, 'delete');
 			return;
 		}
-		catch( \Aimeos\MShop\Exception $e )
+		catch (\Aimeos\MShop\Exception $e)
 		{
-			$error = array( 'plugin-item' => $context->getI18n()->dt( 'mshop', $e->getMessage() ) );
-			$view->errors = $view->get( 'errors', [] ) + $error;
-			$this->logException( $e );
+			$error = array('plugin-item' => $context->getI18n()->dt('mshop', $e->getMessage()));
+			$view->errors = $view->get('errors', []) + $error;
+			$this->logException($e);
 		}
-		catch( \Exception $e )
+		catch (\Exception $e)
 		{
-			$error = array( 'plugin-item' => $e->getMessage() . ', ' . $e->getFile() . ':' . $e->getLine() );
-			$view->errors = $view->get( 'errors', [] ) + $error;
-			$this->logException( $e );
+			$error = array('plugin-item' => $e->getMessage() . ', ' . $e->getFile() . ':' . $e->getLine());
+			$view->errors = $view->get('errors', []) + $error;
+			$this->logException($e);
 		}
 
 		$manager->rollback();
@@ -185,41 +185,41 @@ 
 
 		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, 'plugin' );
+			$manager = \Aimeos\MShop::create($context, 'plugin');
 
-			$view->item = $manager->getItem( $id );
-			$view->itemData = $this->toArray( $view->item );
+			$view->item = $manager->getItem($id);
+			$view->itemData = $this->toArray($view->item);
 			$view->itemSubparts = $this->getSubClientNames();
 			$view->itemDecorators = $this->getDecoratorNames();
 			$view->itemProviders = $this->getProviderNames();
-			$view->itemAttributes = $this->getConfigAttributes( $view->item );
+			$view->itemAttributes = $this->getConfigAttributes($view->item);
 			$view->itemTypes = $this->getTypeItems();
 			$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( 'plugin-item' => $context->getI18n()->dt( 'mshop', $e->getMessage() ) );
-			$view->errors = $view->get( 'errors', [] ) + $error;
-			$this->logException( $e );
+			$error = array('plugin-item' => $context->getI18n()->dt('mshop', $e->getMessage()));
+			$view->errors = $view->get('errors', []) + $error;
+			$this->logException($e);
 		}
-		catch( \Exception $e )
+		catch (\Exception $e)
 		{
-			$error = array( 'plugin-item' => $e->getMessage() . ', ' . $e->getFile() . ':' . $e->getLine() );
-			$view->errors = $view->get( 'errors', [] ) + $error;
-			$this->logException( $e );
+			$error = array('plugin-item' => $e->getMessage() . ', ' . $e->getFile() . ':' . $e->getLine());
+			$view->errors = $view->get('errors', []) + $error;
+			$this->logException($e);
 		}
 
-		return $this->render( $view );
+		return $this->render($view);
 	}
 
 
@@ -233,40 +233,40 @@ 
 		$view = $this->getView();
 		$context = $this->getContext();
 
-		$manager = \Aimeos\MShop::create( $context, 'plugin' );
+		$manager = \Aimeos\MShop::create($context, 'plugin');
 		$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' ), 'plugin', $view->item->getId(), 'save' );
+			$this->nextAction($view, $view->param('next'), 'plugin', $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( 'plugin-item' => $context->getI18n()->dt( 'mshop', $e->getMessage() ) );
-			$view->errors = $view->get( 'errors', [] ) + $error;
-			$this->logException( $e );
+			$error = array('plugin-item' => $context->getI18n()->dt('mshop', $e->getMessage()));
+			$view->errors = $view->get('errors', []) + $error;
+			$this->logException($e);
 		}
-		catch( \Exception $e )
+		catch (\Exception $e)
 		{
-			$error = array( 'plugin-item' => $e->getMessage() . ', ' . $e->getFile() . ':' . $e->getLine() );
-			$view->errors = $view->get( 'errors', [] ) + $error;
-			$this->logException( $e );
+			$error = array('plugin-item' => $e->getMessage() . ', ' . $e->getFile() . ':' . $e->getLine());
+			$view->errors = $view->get('errors', []) + $error;
+			$this->logException($e);
 		}
 
 		$manager->rollback();
@@ -288,35 +288,35 @@ 
 		try
 		{
 			$total = 0;
-			$params = $this->storeSearchParams( $view->param(), 'plugin' );
-			$manager = \Aimeos\MShop::create( $context, 'plugin' );
+			$params = $this->storeSearchParams($view->param(), 'plugin');
+			$manager = \Aimeos\MShop::create($context, 'plugin');
 
 			$search = $manager->createSearch();
-			$search->setSortations( [$search->sort( '+', 'plugin.type' ), $search->sort( '+', 'plugin.position' )] );
-			$search = $this->initCriteria( $search, $params );
+			$search->setSortations([$search->sort('+', 'plugin.type'), $search->sort('+', 'plugin.position')]);
+			$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->itemTypes = $this->getTypeItems();
 			$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( 'plugin-item' => $context->getI18n()->dt( 'mshop', $e->getMessage() ) );
-			$view->errors = $view->get( 'errors', [] ) + $error;
-			$this->logException( $e );
+			$error = array('plugin-item' => $context->getI18n()->dt('mshop', $e->getMessage()));
+			$view->errors = $view->get('errors', []) + $error;
+			$this->logException($e);
 		}
-		catch( \Exception $e )
+		catch (\Exception $e)
 		{
-			$error = array( 'plugin-item' => $e->getMessage() . ', ' . $e->getFile() . ':' . $e->getLine() );
-			$view->errors = $view->get( 'errors', [] ) + $error;
-			$this->logException( $e );
+			$error = array('plugin-item' => $e->getMessage() . ', ' . $e->getFile() . ':' . $e->getLine());
+			$view->errors = $view->get('errors', []) + $error;
+			$this->logException($e);
 		}
 
 		/** admin/jqadm/plugin/template-list
@@ -341,7 +341,7 @@ 
 		$tplconf = 'admin/jqadm/plugin/template-list';
 		$default = 'plugin/list-standard';
 
-		return $view->render( $view->config( $tplconf, $default ) );
+		return $view->render($view->config($tplconf, $default));
 	}
 
 
@@ -352,7 +352,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/plugin/decorators/excludes
 		 * Excludes decorators added by the "common" option from the plugin JQAdm client
@@ -427,7 +427,7 @@ 
 		 * @see admin/jqadm/plugin/decorators/excludes
 		 * @see admin/jqadm/plugin/decorators/global
 		 */
-		return $this->createSubClient( 'plugin/' . $type, $name );
+		return $this->createSubClient('plugin/' . $type, $name);
 	}
 
 
@@ -437,13 +437,13 @@ 
 	 * @param \Aimeos\MShop\Plugin\Item\Iface $item Plugin item incl. provider/decorator property
 	 * @return \Aimeos\MW\Common\Critera\Attribute\Iface[] List of configuration attributes
 	 */
-	public function getConfigAttributes( \Aimeos\MShop\Plugin\Item\Iface $item )
+	public function getConfigAttributes(\Aimeos\MShop\Plugin\Item\Iface $item)
 	{
-		$manager = \Aimeos\MShop::create( $this->getContext(), 'plugin' );
+		$manager = \Aimeos\MShop::create($this->getContext(), 'plugin');
 
 		try {
-			return $manager->getProvider( $item, $item->getType() )->getConfigBE();
-		} catch( \Aimeos\MShop\Exception $e ) {
+			return $manager->getProvider($item, $item->getType())->getConfigBE();
+		} catch (\Aimeos\MShop\Exception $e) {
 			return [];
 		}
 	}
@@ -457,7 +457,7 @@ 
 	protected function getDecoratorNames()
 	{
 		$ds = DIRECTORY_SEPARATOR;
-		return $this->getClassNames( 'MShop' . $ds . 'Plugin' . $ds . 'Provider' . $ds . 'Decorator' );
+		return $this->getClassNames('MShop' . $ds . 'Plugin' . $ds . 'Provider' . $ds . 'Decorator');
 	}
 
 
@@ -470,7 +470,7 @@ 
 	{
 		$ds = DIRECTORY_SEPARATOR;
 		return [
-			'order' => $this->getClassNames( 'MShop' . $ds . 'Plugin' . $ds . 'Provider' . $ds . 'Order' )
+			'order' => $this->getClassNames('MShop' . $ds . 'Plugin' . $ds . 'Provider' . $ds . 'Order')
 		];
 	}
 
@@ -515,7 +515,7 @@ 
 		 * @since 2017.10
 		 * @category Developer
 		 */
-		return $this->getContext()->getConfig()->get( 'admin/jqadm/plugin/standard/subparts', [] );
+		return $this->getContext()->getConfig()->get('admin/jqadm/plugin/standard/subparts', []);
 	}
 
 
@@ -526,12 +526,12 @@ 
 	 */
 	protected function getTypeItems()
 	{
-		$typeManager = \Aimeos\MShop::create( $this->getContext(), 'plugin/type' );
+		$typeManager = \Aimeos\MShop::create($this->getContext(), 'plugin/type');
 
-		$search = $typeManager->createSearch( true )->setSlice( 0, 10000 );
-		$search->setSortations( [$search->sort( '+', 'plugin.type.position' )] );
+		$search = $typeManager->createSearch(true)->setSlice(0, 10000);
+		$search->setSortations([$search->sort('+', 'plugin.type.position')]);
 
-		return $this->map( $typeManager->searchItems( $search ) );
+		return $this->map($typeManager->searchItems($search));
 	}
 
 
@@ -541,17 +541,17 @@ 
 	 * @param string[] Data array
 	 * @return \Aimeos\MShop\Plugin\Item\Iface New plugin item object
 	 */
-	protected function fromArray( array $data )
+	protected function fromArray(array $data)
 	{
 		$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]))
 				{
-					if( ( $val = json_decode( $data['config']['val'][$idx] ) ) === null ) {
+					if (($val = json_decode($data['config']['val'][$idx])) === null) {
 						$conf[$key] = $data['config']['val'][$idx];
 					} else {
 						$conf[$key] = $val;
@@ -560,16 +560,16 @@ 
 			}
 		}
 
-		$manager = \Aimeos\MShop::create( $this->getContext(), 'plugin' );
+		$manager = \Aimeos\MShop::create($this->getContext(), 'plugin');
 
-		if( isset( $data['plugin.id'] ) && $data['plugin.id'] != '' ) {
-			$item = $manager->getItem( $data['plugin.id'] );
+		if (isset($data['plugin.id']) && $data['plugin.id'] != '') {
+			$item = $manager->getItem($data['plugin.id']);
 		} else {
 			$item = $manager->createItem();
 		}
 
-		$item->fromArray( $data );
-		$item->setConfig( $conf );
+		$item->fromArray($data);
+		$item->setConfig($conf);
 
 		return $item;
 	}
@@ -581,21 +581,21 @@ 
 	 * @param \Aimeos\MShop\Plugin\Item\Iface $item Plugin item object
 	 * @return string[] Multi-dimensional associative list of item data
 	 */
-	protected function toArray( \Aimeos\MShop\Plugin\Item\Iface $item, $copy = false )
+	protected function toArray(\Aimeos\MShop\Plugin\Item\Iface $item, $copy = false)
 	{
 		$config = $item->getConfig();
-		$data = $item->toArray( true );
+		$data = $item->toArray(true);
 		$data['config'] = [];
 
-		if( $copy === true )
+		if ($copy === true)
 		{
 			$data['plugin.siteid'] = $this->getContext()->getLocale()->getSiteId();
 			$data['plugin.id'] = '';
 		}
 
-		ksort( $config );
+		ksort($config);
 
-		foreach( $config as $key => $value )
+		foreach ($config as $key => $value)
 		{
 			$data['config']['key'][] = $key;
 			$data['config']['val'][] = $value;
@@ -611,7 +611,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/plugin/template-item
 		 * Relative path to the HTML body template for the plugin item.
@@ -635,6 +635,6 @@ 
 		$tplconf = 'admin/jqadm/plugin/template-item';
 		$default = 'plugin/item-standard';
 
-		return $view->render( $view->config( $tplconf, $default ) );
+		return $view->render($view->config($tplconf, $default));
 	}
 }

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

Braces +2 added lines, -4 removed lines

@@ -111,14 +111,12 @@
 			foreach( $this->getSubClients() as $client ) {
 				$view->listBody .= $client->search();
 			}
-		}
-		catch( \Aimeos\MShop\Exception $e )
+		} catch( \Aimeos\MShop\Exception $e )
 		{
 			$error = array( 'dashboard' => $context->getI18n()->dt( 'mshop', $e->getMessage() ) );
 			$view->errors = $view->get( 'errors', [] ) + $error;
 			$this->logException( $e );
-		}
-		catch( \Exception $e )
+		} catch( \Exception $e )
 		{
 			$error = array( 'dashboard' => $e->getMessage() );
 			$view->errors = $view->get( 'errors', [] ) + $error;

Spacing +14 added lines, -14 removed lines

@@ -10,7 +10,7 @@ 
 
 namespace Aimeos\Admin\JQAdm\Dashboard;
 
-sprintf( 'dashboard' ); // for translation
+sprintf('dashboard'); // for translation
 
 
 /**
@@ -49,21 +49,21 @@ 
 		{
 			$view->listBody = '';
 
-			foreach( $this->getSubClients() as $client ) {
+			foreach ($this->getSubClients() as $client) {
 				$view->listBody .= $client->search();
 			}
 		}
-		catch( \Aimeos\MShop\Exception $e )
+		catch (\Aimeos\MShop\Exception $e)
 		{
-			$error = array( 'dashboard' => $context->getI18n()->dt( 'mshop', $e->getMessage() ) );
-			$view->errors = $view->get( 'errors', [] ) + $error;
-			$this->logException( $e );
+			$error = array('dashboard' => $context->getI18n()->dt('mshop', $e->getMessage()));
+			$view->errors = $view->get('errors', []) + $error;
+			$this->logException($e);
 		}
-		catch( \Exception $e )
+		catch (\Exception $e)
 		{
-			$error = array( 'dashboard' => $e->getMessage() );
-			$view->errors = $view->get( 'errors', [] ) + $error;
-			$this->logException( $e );
+			$error = array('dashboard' => $e->getMessage());
+			$view->errors = $view->get('errors', []) + $error;
+			$this->logException($e);
 		}
 
 		/** admin/jqadm/dashboard/template-list
@@ -88,7 +88,7 @@ 
 		$tplconf = 'admin/jqadm/dashboard/template-list';
 		$default = 'dashboard/list-standard';
 
-		return $view->render( $view->config( $tplconf, $default ) );
+		return $view->render($view->config($tplconf, $default));
 	}
 
 
@@ -99,7 +99,7 @@ 
 	 * @param string|null $name Name of the sub-client (Default if null)
 	 * @return \Aimeos\Admin\JQAdm\Iface Sub-client object
 	 */
-	public function getSubClient( $type, $name = null )
+	public function getSubClient($type, $name = null)
 	{
 		/** admin/jqadm/dashboard/decorators/excludes
 		 * Excludes decorators added by the "common" option from the dashboard JQAdm client
@@ -174,7 +174,7 @@ 
 		 * @see admin/jqadm/dashboard/decorators/excludes
 		 * @see admin/jqadm/dashboard/decorators/global
 		 */
-		return $this->createSubClient( 'dashboard/' . $type, $name );
+		return $this->createSubClient('dashboard/' . $type, $name);
 	}
 
 
@@ -218,6 +218,6 @@ 
 		 * @since 2016.07
 		 * @category Developer
 		 */
-		return $this->getContext()->getConfig()->get( 'admin/jqadm/dashboard/standard/subparts', [] );
+		return $this->getContext()->getConfig()->get('admin/jqadm/dashboard/standard/subparts', []);
 	}
 }

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

Braces +13 added lines, -26 removed lines

@@ -125,14 +125,12 @@ 
 				$view->tabindex = ++$idx + 1;
 				$view->itemBody .= $client->copy();
 			}
-		}
-		catch( \Aimeos\MShop\Exception $e )
+		} catch( \Aimeos\MShop\Exception $e )
 		{
 			$error = array( 'service-item' => $context->getI18n()->dt( 'mshop', $e->getMessage() ) );
 			$view->errors = $view->get( 'errors', [] ) + $error;
 			$this->logException( $e );
-		}
-		catch( \Exception $e )
+		} catch( \Exception $e )
 		{
 			$error = array( 'service-item' => $e->getMessage() . ', ' . $e->getFile() . ':' . $e->getLine() );
 			$view->errors = $view->get( 'errors', [] ) + $error;
@@ -177,14 +175,12 @@ 
 				$view->tabindex = ++$idx + 1;
 				$view->itemBody .= $client->create();
 			}
-		}
-		catch( \Aimeos\MShop\Exception $e )
+		} catch( \Aimeos\MShop\Exception $e )
 		{
 			$error = array( 'service-item' => $context->getI18n()->dt( 'mshop', $e->getMessage() ) );
 			$view->errors = $view->get( 'errors', [] ) + $error;
 			$this->logException( $e );
-		}
-		catch( \Exception $e )
+		} catch( \Exception $e )
 		{
 			$error = array( 'service-item' => $e->getMessage() . ', ' . $e->getFile() . ':' . $e->getLine() );
 			$view->errors = $view->get( 'errors', [] ) + $error;
@@ -225,14 +221,12 @@ 
 
 			$this->nextAction( $view, 'search', 'service', null, 'delete' );
 			return;
-		}
-		catch( \Aimeos\MShop\Exception $e )
+		} catch( \Aimeos\MShop\Exception $e )
 		{
 			$error = array( 'service-item' => $context->getI18n()->dt( 'mshop', $e->getMessage() ) );
 			$view->errors = $view->get( 'errors', [] ) + $error;
 			$this->logException( $e );
-		}
-		catch( \Exception $e )
+		} catch( \Exception $e )
 		{
 			$error = array( 'service-item' => $e->getMessage() . ', ' . $e->getFile() . ':' . $e->getLine() );
 			$view->errors = $view->get( 'errors', [] ) + $error;
@@ -277,14 +271,12 @@ 
 				$view->tabindex = ++$idx + 1;
 				$view->itemBody .= $client->get();
 			}
-		}
-		catch( \Aimeos\MShop\Exception $e )
+		} catch( \Aimeos\MShop\Exception $e )
 		{
 			$error = array( 'service-item' => $context->getI18n()->dt( 'mshop', $e->getMessage() ) );
 			$view->errors = $view->get( 'errors', [] ) + $error;
 			$this->logException( $e );
-		}
-		catch( \Exception $e )
+		} catch( \Exception $e )
 		{
 			$error = array( 'service-item' => $e->getMessage() . ', ' . $e->getFile() . ':' . $e->getLine() );
 			$view->errors = $view->get( 'errors', [] ) + $error;
@@ -321,18 +313,15 @@ 
 
 			$this->nextAction( $view, $view->param( 'next' ), 'service', $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( 'service-item' => $context->getI18n()->dt( 'mshop', $e->getMessage() ) );
 			$view->errors = $view->get( 'errors', [] ) + $error;
 			$this->logException( $e );
-		}
-		catch( \Exception $e )
+		} catch( \Exception $e )
 		{
 			$error = array( 'service-item' => $e->getMessage() . ', ' . $e->getFile() . ':' . $e->getLine() );
 			$view->errors = $view->get( 'errors', [] ) + $error;
@@ -375,14 +364,12 @@ 
 			foreach( $this->getSubClients() as $client ) {
 				$view->itemBody .= $client->search();
 			}
-		}
-		catch( \Aimeos\MShop\Exception $e )
+		} catch( \Aimeos\MShop\Exception $e )
 		{
 			$error = array( 'service-item' => $context->getI18n()->dt( 'mshop', $e->getMessage() ) );
 			$view->errors = $view->get( 'errors', [] ) + $error;
 			$this->logException( $e );
-		}
-		catch( \Exception $e )
+		} catch( \Exception $e )
 		{
 			$error = array( 'service-item' => $e->getMessage() . ', ' . $e->getFile() . ':' . $e->getLine() );
 			$view->errors = $view->get( 'errors', [] ) + $error;

Doc Comments +1 added lines, -1 removed lines

@@ -548,7 +548,7 @@
 	/**
 	 * Returns the available service type items
 	 *
-	 * @return array List of item implementing \Aimeos\MShop\Common\Type\Iface
+	 * @return \Aimeos\MShop\Common\Item\Type\Iface[] List of item implementing \Aimeos\MShop\Common\Type\Iface
 	 */
 	protected function getTypeItems()
 	{

Spacing +126 added lines, -126 removed lines

@@ -10,7 +10,7 @@ 
 
 namespace Aimeos\Admin\JQAdm\Service;
 
-sprintf( 'service' ); // for translation
+sprintf('service'); // for translation
 
 
 /**
@@ -35,41 +35,41 @@ 
 
 		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, 'service' );
+			$manager = \Aimeos\MShop::create($context, 'service');
 
-			$view->item = $manager->getItem( $id, $this->getDomains() );
-			$view->itemData = $this->toArray( $view->item, true );
+			$view->item = $manager->getItem($id, $this->getDomains());
+			$view->itemData = $this->toArray($view->item, true);
 			$view->itemSubparts = $this->getSubClientNames();
 			$view->itemProviders = $this->getProviderNames();
 			$view->itemDecorators = $this->getDecoratorNames();
-			$view->itemAttributes = $this->getConfigAttributes( $view->item );
+			$view->itemAttributes = $this->getConfigAttributes($view->item);
 			$view->itemTypes = $this->getTypeItems();
 			$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( 'service-item' => $context->getI18n()->dt( 'mshop', $e->getMessage() ) );
-			$view->errors = $view->get( 'errors', [] ) + $error;
-			$this->logException( $e );
+			$error = array('service-item' => $context->getI18n()->dt('mshop', $e->getMessage()));
+			$view->errors = $view->get('errors', []) + $error;
+			$this->logException($e);
 		}
-		catch( \Exception $e )
+		catch (\Exception $e)
 		{
-			$error = array( 'service-item' => $e->getMessage() . ', ' . $e->getFile() . ':' . $e->getLine() );
-			$view->errors = $view->get( 'errors', [] ) + $error;
-			$this->logException( $e );
+			$error = array('service-item' => $e->getMessage() . ', ' . $e->getFile() . ':' . $e->getLine());
+			$view->errors = $view->get('errors', []) + $error;
+			$this->logException($e);
 		}
 
-		return $this->render( $view );
+		return $this->render($view);
 	}
 
 
@@ -85,10 +85,10 @@ 
 
 		try
 		{
-			$data = $view->param( 'item', [] );
+			$data = $view->param('item', []);
 
-			if( !isset( $view->item ) ) {
-				$view->item = \Aimeos\MShop::create( $context, 'service' )->createItem();
+			if (!isset($view->item)) {
+				$view->item = \Aimeos\MShop::create($context, 'service')->createItem();
 			}
 
 			$data['service.siteid'] = $view->item->getSiteId();
@@ -100,26 +100,26 @@ 
 			$view->itemData = $data;
 			$view->itemBody = '';
 
-			foreach( $this->getSubClients() as $idx => $client )
+			foreach ($this->getSubClients() as $idx => $client)
 			{
 				$view->tabindex = ++$idx + 1;
 				$view->itemBody .= $client->create();
 			}
 		}
-		catch( \Aimeos\MShop\Exception $e )
+		catch (\Aimeos\MShop\Exception $e)
 		{
-			$error = array( 'service-item' => $context->getI18n()->dt( 'mshop', $e->getMessage() ) );
-			$view->errors = $view->get( 'errors', [] ) + $error;
-			$this->logException( $e );
+			$error = array('service-item' => $context->getI18n()->dt('mshop', $e->getMessage()));
+			$view->errors = $view->get('errors', []) + $error;
+			$this->logException($e);
 		}
-		catch( \Exception $e )
+		catch (\Exception $e)
 		{
-			$error = array( 'service-item' => $e->getMessage() . ', ' . $e->getFile() . ':' . $e->getLine() );
-			$view->errors = $view->get( 'errors', [] ) + $error;
-			$this->logException( $e );
+			$error = array('service-item' => $e->getMessage() . ', ' . $e->getFile() . ':' . $e->getLine());
+			$view->errors = $view->get('errors', []) + $error;
+			$this->logException($e);
 		}
 
-		return $this->render( $view );
+		return $this->render($view);
 	}
 
 
@@ -133,39 +133,39 @@ 
 		$view = $this->getView();
 		$context = $this->getContext();
 
-		$manager = \Aimeos\MShop::create( $context, 'service' );
+		$manager = \Aimeos\MShop::create($context, 'service');
 		$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, $this->getDomains() );
+			$view->item = $manager->getItem($id, $this->getDomains());
 
-			foreach( $this->getSubClients() as $client ) {
+			foreach ($this->getSubClients() as $client) {
 				$client->delete();
 			}
 
-			$manager->saveItem( $view->item );
-			$manager->deleteItem( $id );
+			$manager->saveItem($view->item);
+			$manager->deleteItem($id);
 			$manager->commit();
 
-			$this->nextAction( $view, 'search', 'service', null, 'delete' );
+			$this->nextAction($view, 'search', 'service', null, 'delete');
 			return;
 		}
-		catch( \Aimeos\MShop\Exception $e )
+		catch (\Aimeos\MShop\Exception $e)
 		{
-			$error = array( 'service-item' => $context->getI18n()->dt( 'mshop', $e->getMessage() ) );
-			$view->errors = $view->get( 'errors', [] ) + $error;
-			$this->logException( $e );
+			$error = array('service-item' => $context->getI18n()->dt('mshop', $e->getMessage()));
+			$view->errors = $view->get('errors', []) + $error;
+			$this->logException($e);
 		}
-		catch( \Exception $e )
+		catch (\Exception $e)
 		{
-			$error = array( 'service-item' => $e->getMessage() . ', ' . $e->getFile() . ':' . $e->getLine() );
-			$view->errors = $view->get( 'errors', [] ) + $error;
-			$this->logException( $e );
+			$error = array('service-item' => $e->getMessage() . ', ' . $e->getFile() . ':' . $e->getLine());
+			$view->errors = $view->get('errors', []) + $error;
+			$this->logException($e);
 		}
 
 		$manager->rollback();
@@ -186,41 +186,41 @@ 
 
 		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, 'service' );
+			$manager = \Aimeos\MShop::create($context, 'service');
 
-			$view->item = $manager->getItem( $id, $this->getDomains() );
-			$view->itemData = $this->toArray( $view->item );
+			$view->item = $manager->getItem($id, $this->getDomains());
+			$view->itemData = $this->toArray($view->item);
 			$view->itemSubparts = $this->getSubClientNames();
 			$view->itemDecorators = $this->getDecoratorNames();
 			$view->itemProviders = $this->getProviderNames();
-			$view->itemAttributes = $this->getConfigAttributes( $view->item );
+			$view->itemAttributes = $this->getConfigAttributes($view->item);
 			$view->itemTypes = $this->getTypeItems();
 			$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( 'service-item' => $context->getI18n()->dt( 'mshop', $e->getMessage() ) );
-			$view->errors = $view->get( 'errors', [] ) + $error;
-			$this->logException( $e );
+			$error = array('service-item' => $context->getI18n()->dt('mshop', $e->getMessage()));
+			$view->errors = $view->get('errors', []) + $error;
+			$this->logException($e);
 		}
-		catch( \Exception $e )
+		catch (\Exception $e)
 		{
-			$error = array( 'service-item' => $e->getMessage() . ', ' . $e->getFile() . ':' . $e->getLine() );
-			$view->errors = $view->get( 'errors', [] ) + $error;
-			$this->logException( $e );
+			$error = array('service-item' => $e->getMessage() . ', ' . $e->getFile() . ':' . $e->getLine());
+			$view->errors = $view->get('errors', []) + $error;
+			$this->logException($e);
 		}
 
-		return $this->render( $view );
+		return $this->render($view);
 	}
 
 
@@ -234,40 +234,40 @@ 
 		$view = $this->getView();
 		$context = $this->getContext();
 
-		$manager = \Aimeos\MShop::create( $context, 'service' );
+		$manager = \Aimeos\MShop::create($context, 'service');
 		$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' ), 'service', $view->item->getId(), 'save' );
+			$this->nextAction($view, $view->param('next'), 'service', $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( 'service-item' => $context->getI18n()->dt( 'mshop', $e->getMessage() ) );
-			$view->errors = $view->get( 'errors', [] ) + $error;
-			$this->logException( $e );
+			$error = array('service-item' => $context->getI18n()->dt('mshop', $e->getMessage()));
+			$view->errors = $view->get('errors', []) + $error;
+			$this->logException($e);
 		}
-		catch( \Exception $e )
+		catch (\Exception $e)
 		{
-			$error = array( 'service-item' => $e->getMessage() . ', ' . $e->getFile() . ':' . $e->getLine() );
-			$view->errors = $view->get( 'errors', [] ) + $error;
-			$this->logException( $e );
+			$error = array('service-item' => $e->getMessage() . ', ' . $e->getFile() . ':' . $e->getLine());
+			$view->errors = $view->get('errors', []) + $error;
+			$this->logException($e);
 		}
 
 		$manager->rollback();
@@ -289,35 +289,35 @@ 
 		try
 		{
 			$total = 0;
-			$params = $this->storeSearchParams( $view->param(), 'service' );
-			$manager = \Aimeos\MShop::create( $context, 'service' );
+			$params = $this->storeSearchParams($view->param(), 'service');
+			$manager = \Aimeos\MShop::create($context, 'service');
 
 			$search = $manager->createSearch();
-			$search->setSortations( [$search->sort( '+', 'service.type' ), $search->sort( '+', 'service.position' )] );
-			$search = $this->initCriteria( $search, $params );
+			$search->setSortations([$search->sort('+', 'service.type'), $search->sort('+', 'service.position')]);
+			$search = $this->initCriteria($search, $params);
 
-			$view->items = $manager->searchItems( $search, $this->getDomains(), $total );
-			$view->filterAttributes = $manager->getSearchAttributes( true );
+			$view->items = $manager->searchItems($search, $this->getDomains(), $total);
+			$view->filterAttributes = $manager->getSearchAttributes(true);
 			$view->filterOperators = $search->getOperators();
 			$view->itemTypes = $this->getTypeItems();
 			$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( 'service-item' => $context->getI18n()->dt( 'mshop', $e->getMessage() ) );
-			$view->errors = $view->get( 'errors', [] ) + $error;
-			$this->logException( $e );
+			$error = array('service-item' => $context->getI18n()->dt('mshop', $e->getMessage()));
+			$view->errors = $view->get('errors', []) + $error;
+			$this->logException($e);
 		}
-		catch( \Exception $e )
+		catch (\Exception $e)
 		{
-			$error = array( 'service-item' => $e->getMessage() . ', ' . $e->getFile() . ':' . $e->getLine() );
-			$view->errors = $view->get( 'errors', [] ) + $error;
-			$this->logException( $e );
+			$error = array('service-item' => $e->getMessage() . ', ' . $e->getFile() . ':' . $e->getLine());
+			$view->errors = $view->get('errors', []) + $error;
+			$this->logException($e);
 		}
 
 		/** admin/jqadm/service/template-list
@@ -342,7 +342,7 @@ 
 		$tplconf = 'admin/jqadm/service/template-list';
 		$default = 'service/list-standard';
 
-		return $view->render( $view->config( $tplconf, $default ) );
+		return $view->render($view->config($tplconf, $default));
 	}
 
 
@@ -353,7 +353,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/decorators/excludes
 		 * Excludes decorators added by the "common" option from the service JQAdm client
@@ -428,7 +428,7 @@ 
 		 * @see admin/jqadm/service/decorators/excludes
 		 * @see admin/jqadm/service/decorators/global
 		 */
-		return $this->createSubClient( 'service/' . $type, $name );
+		return $this->createSubClient('service/' . $type, $name);
 	}
 
 
@@ -438,13 +438,13 @@ 
 	 * @param \Aimeos\MShop\Service\Item\Iface $item Service item incl. provider/decorator property
 	 * @return \Aimeos\MW\Common\Critera\Attribute\Iface[] List of configuration attributes
 	 */
-	public function getConfigAttributes( \Aimeos\MShop\Service\Item\Iface $item )
+	public function getConfigAttributes(\Aimeos\MShop\Service\Item\Iface $item)
 	{
-		$manager = \Aimeos\MShop::create( $this->getContext(), 'service' );
+		$manager = \Aimeos\MShop::create($this->getContext(), 'service');
 
 		try {
-			return $manager->getProvider( $item, $item->getType() )->getConfigBE();
-		} catch( \Aimeos\MShop\Exception $e ) {
+			return $manager->getProvider($item, $item->getType())->getConfigBE();
+		} catch (\Aimeos\MShop\Exception $e) {
 			return [];
 		}
 	}
@@ -468,9 +468,9 @@ 
 		 * @since 2017.10
 		 * @category Developer
 		 */
-		$domains = array( 'media', 'price', 'text' );
+		$domains = array('media', 'price', 'text');
 
-		return $this->getContext()->getConfig()->get( 'admin/jqadm/service/domains', $domains );
+		return $this->getContext()->getConfig()->get('admin/jqadm/service/domains', $domains);
 	}
 
 
@@ -482,7 +482,7 @@ 
 	protected function getDecoratorNames()
 	{
 		$ds = DIRECTORY_SEPARATOR;
-		return $this->getClassNames( 'MShop' . $ds . 'Service' . $ds . 'Provider' . $ds . 'Decorator' );
+		return $this->getClassNames('MShop' . $ds . 'Service' . $ds . 'Provider' . $ds . 'Decorator');
 	}
 
 
@@ -495,8 +495,8 @@ 
 	{
 		$ds = DIRECTORY_SEPARATOR;
 		return [
-			'delivery' => $this->getClassNames( 'MShop' . $ds . 'Service' . $ds . 'Provider' . $ds . 'Delivery' ),
-			'payment' => $this->getClassNames( 'MShop' . $ds . 'Service' . $ds . 'Provider' . $ds . 'Payment' ),
+			'delivery' => $this->getClassNames('MShop' . $ds . 'Service' . $ds . 'Provider' . $ds . 'Delivery'),
+			'payment' => $this->getClassNames('MShop' . $ds . 'Service' . $ds . 'Provider' . $ds . 'Payment'),
 		];
 	}
 
@@ -541,7 +541,7 @@ 
 		 * @since 2017.10
 		 * @category Developer
 		 */
-		return $this->getContext()->getConfig()->get( 'admin/jqadm/service/standard/subparts', [] );
+		return $this->getContext()->getConfig()->get('admin/jqadm/service/standard/subparts', []);
 	}
 
 
@@ -552,12 +552,12 @@ 
 	 */
 	protected function getTypeItems()
 	{
-		$typeManager = \Aimeos\MShop::create( $this->getContext(), 'service/type' );
+		$typeManager = \Aimeos\MShop::create($this->getContext(), 'service/type');
 
-		$search = $typeManager->createSearch( true )->setSlice( 0, 10000 );
-		$search->setSortations( [$search->sort( '+', 'service.type.position' )] );
+		$search = $typeManager->createSearch(true)->setSlice(0, 10000);
+		$search->setSortations([$search->sort('+', 'service.type.position')]);
 
-		return $this->map( $typeManager->searchItems( $search ) );
+		return $this->map($typeManager->searchItems($search));
 	}
 
 
@@ -567,17 +567,17 @@ 
 	 * @param string[] Data array
 	 * @return \Aimeos\MShop\Service\Item\Iface New service item object
 	 */
-	protected function fromArray( array $data )
+	protected function fromArray(array $data)
 	{
 		$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]))
 				{
-					if( ( $val = json_decode( $data['config']['val'][$idx] ) ) === null ) {
+					if (($val = json_decode($data['config']['val'][$idx])) === null) {
 						$conf[$key] = $data['config']['val'][$idx];
 					} else {
 						$conf[$key] = $val;
@@ -586,16 +586,16 @@ 
 			}
 		}
 
-		$manager = \Aimeos\MShop::create( $this->getContext(), 'service' );
+		$manager = \Aimeos\MShop::create($this->getContext(), 'service');
 
-		if( isset( $data['service.id'] ) && $data['service.id'] != '' ) {
-			$item = $manager->getItem( $data['service.id'], $this->getDomains() );
+		if (isset($data['service.id']) && $data['service.id'] != '') {
+			$item = $manager->getItem($data['service.id'], $this->getDomains());
 		} else {
 			$item = $manager->createItem();
 		}
 
-		$item->fromArray( $data );
-		$item->setConfig( $conf );
+		$item->fromArray($data);
+		$item->setConfig($conf);
 
 		return $item;
 	}
@@ -607,22 +607,22 @@ 
 	 * @param \Aimeos\MShop\Service\Item\Iface $item Service item object
 	 * @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)
 	{
 		$config = $item->getConfig();
-		$data = $item->toArray( true );
+		$data = $item->toArray(true);
 		$data['config'] = [];
 
-		if( $copy === true )
+		if ($copy === true)
 		{
 			$data['service.siteid'] = $this->getContext()->getLocale()->getSiteId();
 			$data['service.code'] = $data['service.code'] . '_copy';
 			$data['service.id'] = '';
 		}
 
-		ksort( $config );
+		ksort($config);
 
-		foreach( $config as $key => $value )
+		foreach ($config as $key => $value)
 		{
 			$data['config']['key'][] = $key;
 			$data['config']['val'][] = $value;
@@ -638,7 +638,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/template-item
 		 * Relative path to the HTML body template for the service item.
@@ -662,6 +662,6 @@ 
 		$tplconf = 'admin/jqadm/service/template-item';
 		$default = 'service/item-standard';
 
-		return $view->render( $view->config( $tplconf, $default ) );
+		return $view->render($view->config($tplconf, $default));
 	}
 }

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

Braces +13 added lines, -26 removed lines

@@ -122,14 +122,12 @@ 
 				$view->tabindex = ++$idx + 1;
 				$view->itemBody .= $client->copy();
 			}
-		}
-		catch( \Aimeos\MShop\Exception $e )
+		} catch( \Aimeos\MShop\Exception $e )
 		{
 			$error = array( 'customer-item' => $context->getI18n()->dt( 'mshop', $e->getMessage() ) );
 			$view->errors = $view->get( 'errors', [] ) + $error;
 			$this->logException( $e );
-		}
-		catch( \Exception $e )
+		} catch( \Exception $e )
 		{
 			$error = array( 'customer-item' => $e->getMessage() . ', ' . $e->getFile() . ':' . $e->getLine() );
 			$view->errors = $view->get( 'errors', [] ) + $error;
@@ -172,14 +170,12 @@ 
 				$view->tabindex = ++$idx + 1;
 				$view->itemBody .= $client->create();
 			}
-		}
-		catch( \Aimeos\MShop\Exception $e )
+		} catch( \Aimeos\MShop\Exception $e )
 		{
 			$error = array( 'customer-item' => $context->getI18n()->dt( 'mshop', $e->getMessage() ) );
 			$view->errors = $view->get( 'errors', [] ) + $error;
 			$this->logException( $e );
-		}
-		catch( \Exception $e )
+		} catch( \Exception $e )
 		{
 			$error = array( 'customer-item' => $e->getMessage() . ', ' . $e->getFile() . ':' . $e->getLine() );
 			$view->errors = $view->get( 'errors', [] ) + $error;
@@ -220,14 +216,12 @@ 
 
 			$this->nextAction( $view, 'search', 'customer', null, 'delete' );
 			return;
-		}
-		catch( \Aimeos\MShop\Exception $e )
+		} catch( \Aimeos\MShop\Exception $e )
 		{
 			$error = array( 'customer-item' => $context->getI18n()->dt( 'mshop', $e->getMessage() ) );
 			$view->errors = $view->get( 'errors', [] ) + $error;
 			$this->logException( $e );
-		}
-		catch( \Exception $e )
+		} catch( \Exception $e )
 		{
 			$error = array( 'customer-item' => $e->getMessage() . ', ' . $e->getFile() . ':' . $e->getLine() );
 			$view->errors = $view->get( 'errors', [] ) + $error;
@@ -269,14 +263,12 @@ 
 				$view->tabindex = ++$idx + 1;
 				$view->itemBody .= $client->get();
 			}
-		}
-		catch( \Aimeos\MShop\Exception $e )
+		} catch( \Aimeos\MShop\Exception $e )
 		{
 			$error = array( 'customer-item' => $context->getI18n()->dt( 'mshop', $e->getMessage() ) );
 			$view->errors = $view->get( 'errors', [] ) + $error;
 			$this->logException( $e );
-		}
-		catch( \Exception $e )
+		} catch( \Exception $e )
 		{
 			$error = array( 'customer-item' => $e->getMessage() . ', ' . $e->getFile() . ':' . $e->getLine() );
 			$view->errors = $view->get( 'errors', [] ) + $error;
@@ -313,18 +305,15 @@ 
 
 			$this->nextAction( $view, $view->param( 'next' ), 'customer', $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( 'customer-item' => $context->getI18n()->dt( 'mshop', $e->getMessage() ) );
 			$view->errors = $view->get( 'errors', [] ) + $error;
 			$this->logException( $e );
-		}
-		catch( \Exception $e )
+		} catch( \Exception $e )
 		{
 			$error = array( 'customer-item' => $e->getMessage() . ', ' . $e->getFile() . ':' . $e->getLine() );
 			$view->errors = $view->get( 'errors', [] ) + $error;
@@ -363,14 +352,12 @@ 
 			foreach( $this->getSubClients() as $client ) {
 				$view->itemBody .= $client->search();
 			}
-		}
-		catch( \Aimeos\MShop\Exception $e )
+		} catch( \Aimeos\MShop\Exception $e )
 		{
 			$error = array( 'customer-item' => $context->getI18n()->dt( 'mshop', $e->getMessage() ) );
 			$view->errors = $view->get( 'errors', [] ) + $error;
 			$this->logException( $e );
-		}
-		catch( \Exception $e )
+		} catch( \Exception $e )
 		{
 			$error = array( 'customer-item' => $e->getMessage() . ', ' . $e->getFile() . ':' . $e->getLine() );
 			$view->errors = $view->get( 'errors', [] ) + $error;

Spacing +112 added lines, -112 removed lines

@@ -10,7 +10,7 @@ 
 
 namespace Aimeos\Admin\JQAdm\Customer;
 
-sprintf( 'customer' ); // for translation
+sprintf('customer'); // for translation
 
 
 /**
@@ -35,38 +35,38 @@ 
 
 		try
 		{
-			if( ( $id = $view->param( 'id' ) ) === null ) {
-				throw new \Aimeos\Admin\JQAdm\Exception( sprintf( 'Required parameter "%1$s" is missing', 'id' ) );
+			if (($id = $view->param('id')) === null) {
+				throw new \Aimeos\Admin\JQAdm\Exception(sprintf('Required parameter "%1$s" is missing', 'id'));
 			}
 
-			$manager = \Aimeos\MShop::create( $context, 'customer' );
-			$view->item = $manager->getItem( $id, $this->getDomains() );
+			$manager = \Aimeos\MShop::create($context, 'customer');
+			$view->item = $manager->getItem($id, $this->getDomains());
 
-			$view->itemData = $this->toArray( $view->item, true );
+			$view->itemData = $this->toArray($view->item, true);
 			$view->itemSubparts = $this->getSubClientNames();
 			$view->itemGroups = $this->getGroupItems();
 			$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( 'customer-item' => $context->getI18n()->dt( 'mshop', $e->getMessage() ) );
-			$view->errors = $view->get( 'errors', [] ) + $error;
-			$this->logException( $e );
+			$error = array('customer-item' => $context->getI18n()->dt('mshop', $e->getMessage()));
+			$view->errors = $view->get('errors', []) + $error;
+			$this->logException($e);
 		}
-		catch( \Exception $e )
+		catch (\Exception $e)
 		{
-			$error = array( 'customer-item' => $e->getMessage() . ', ' . $e->getFile() . ':' . $e->getLine() );
-			$view->errors = $view->get( 'errors', [] ) + $error;
-			$this->logException( $e );
+			$error = array('customer-item' => $e->getMessage() . ', ' . $e->getFile() . ':' . $e->getLine());
+			$view->errors = $view->get('errors', []) + $error;
+			$this->logException($e);
 		}
 
-		return $this->render( $view );
+		return $this->render($view);
 	}
 
 
@@ -82,10 +82,10 @@ 
 
 		try
 		{
-			$data = $view->param( 'item', [] );
+			$data = $view->param('item', []);
 
-			if( !isset( $view->item ) ) {
-				$view->item = \Aimeos\MShop::create( $context, 'customer' )->createItem();
+			if (!isset($view->item)) {
+				$view->item = \Aimeos\MShop::create($context, 'customer')->createItem();
 			}
 
 			$data['customer.siteid'] = $view->item->getSiteId();
@@ -95,26 +95,26 @@ 
 			$view->itemData = $data;
 			$view->itemBody = '';
 
-			foreach( $this->getSubClients() as $idx => $client )
+			foreach ($this->getSubClients() as $idx => $client)
 			{
 				$view->tabindex = ++$idx + 1;
 				$view->itemBody .= $client->create();
 			}
 		}
-		catch( \Aimeos\MShop\Exception $e )
+		catch (\Aimeos\MShop\Exception $e)
 		{
-			$error = array( 'customer-item' => $context->getI18n()->dt( 'mshop', $e->getMessage() ) );
-			$view->errors = $view->get( 'errors', [] ) + $error;
-			$this->logException( $e );
+			$error = array('customer-item' => $context->getI18n()->dt('mshop', $e->getMessage()));
+			$view->errors = $view->get('errors', []) + $error;
+			$this->logException($e);
 		}
-		catch( \Exception $e )
+		catch (\Exception $e)
 		{
-			$error = array( 'customer-item' => $e->getMessage() . ', ' . $e->getFile() . ':' . $e->getLine() );
-			$view->errors = $view->get( 'errors', [] ) + $error;
-			$this->logException( $e );
+			$error = array('customer-item' => $e->getMessage() . ', ' . $e->getFile() . ':' . $e->getLine());
+			$view->errors = $view->get('errors', []) + $error;
+			$this->logException($e);
 		}
 
-		return $this->render( $view );
+		return $this->render($view);
 	}
 
 
@@ -128,39 +128,39 @@ 
 		$view = $this->getView();
 		$context = $this->getContext();
 
-		$manager = \Aimeos\MShop::create( $context, 'customer' );
+		$manager = \Aimeos\MShop::create($context, 'customer');
 		$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, $this->getDomains() );
+			$view->item = $manager->getItem($id, $this->getDomains());
 
-			foreach( $this->getSubClients() as $client ) {
+			foreach ($this->getSubClients() as $client) {
 				$client->delete();
 			}
 
-			$manager->saveItem( $view->item );
-			$manager->deleteItem( $id );
+			$manager->saveItem($view->item);
+			$manager->deleteItem($id);
 			$manager->commit();
 
-			$this->nextAction( $view, 'search', 'customer', null, 'delete' );
+			$this->nextAction($view, 'search', 'customer', null, 'delete');
 			return;
 		}
-		catch( \Aimeos\MShop\Exception $e )
+		catch (\Aimeos\MShop\Exception $e)
 		{
-			$error = array( 'customer-item' => $context->getI18n()->dt( 'mshop', $e->getMessage() ) );
-			$view->errors = $view->get( 'errors', [] ) + $error;
-			$this->logException( $e );
+			$error = array('customer-item' => $context->getI18n()->dt('mshop', $e->getMessage()));
+			$view->errors = $view->get('errors', []) + $error;
+			$this->logException($e);
 		}
-		catch( \Exception $e )
+		catch (\Exception $e)
 		{
-			$error = array( 'customer-item' => $e->getMessage() . ', ' . $e->getFile() . ':' . $e->getLine() );
-			$view->errors = $view->get( 'errors', [] ) + $error;
-			$this->logException( $e );
+			$error = array('customer-item' => $e->getMessage() . ', ' . $e->getFile() . ':' . $e->getLine());
+			$view->errors = $view->get('errors', []) + $error;
+			$this->logException($e);
 		}
 
 		$manager->rollback();
@@ -181,38 +181,38 @@ 
 
 		try
 		{
-			if( ( $id = $view->param( 'id' ) ) === null ) {
-				throw new \Aimeos\Admin\JQAdm\Exception( sprintf( 'Required parameter "%1$s" is missing', 'id' ) );
+			if (($id = $view->param('id')) === null) {
+				throw new \Aimeos\Admin\JQAdm\Exception(sprintf('Required parameter "%1$s" is missing', 'id'));
 			}
 
-			$manager = \Aimeos\MShop::create( $context, 'customer' );
+			$manager = \Aimeos\MShop::create($context, 'customer');
 
-			$view->item = $manager->getItem( $id, $this->getDomains() );
+			$view->item = $manager->getItem($id, $this->getDomains());
 			$view->itemSubparts = $this->getSubClientNames();
-			$view->itemData = $this->toArray( $view->item );
+			$view->itemData = $this->toArray($view->item);
 			$view->itemGroups = $this->getGroupItems();
 			$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( 'customer-item' => $context->getI18n()->dt( 'mshop', $e->getMessage() ) );
-			$view->errors = $view->get( 'errors', [] ) + $error;
-			$this->logException( $e );
+			$error = array('customer-item' => $context->getI18n()->dt('mshop', $e->getMessage()));
+			$view->errors = $view->get('errors', []) + $error;
+			$this->logException($e);
 		}
-		catch( \Exception $e )
+		catch (\Exception $e)
 		{
-			$error = array( 'customer-item' => $e->getMessage() . ', ' . $e->getFile() . ':' . $e->getLine() );
-			$view->errors = $view->get( 'errors', [] ) + $error;
-			$this->logException( $e );
+			$error = array('customer-item' => $e->getMessage() . ', ' . $e->getFile() . ':' . $e->getLine());
+			$view->errors = $view->get('errors', []) + $error;
+			$this->logException($e);
 		}
 
-		return $this->render( $view );
+		return $this->render($view);
 	}
 
 
@@ -226,40 +226,40 @@ 
 		$view = $this->getView();
 		$context = $this->getContext();
 
-		$manager = \Aimeos\MShop::create( $context, 'customer' );
+		$manager = \Aimeos\MShop::create($context, 'customer');
 		$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' ), 'customer', $view->item->getId(), 'save' );
+			$this->nextAction($view, $view->param('next'), 'customer', $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( 'customer-item' => $context->getI18n()->dt( 'mshop', $e->getMessage() ) );
-			$view->errors = $view->get( 'errors', [] ) + $error;
-			$this->logException( $e );
+			$error = array('customer-item' => $context->getI18n()->dt('mshop', $e->getMessage()));
+			$view->errors = $view->get('errors', []) + $error;
+			$this->logException($e);
 		}
-		catch( \Exception $e )
+		catch (\Exception $e)
 		{
-			$error = array( 'customer-item' => $e->getMessage() . ', ' . $e->getFile() . ':' . $e->getLine() );
-			$view->errors = $view->get( 'errors', [] ) + $error;
-			$this->logException( $e );
+			$error = array('customer-item' => $e->getMessage() . ', ' . $e->getFile() . ':' . $e->getLine());
+			$view->errors = $view->get('errors', []) + $error;
+			$this->logException($e);
 		}
 
 		$manager->rollback();
@@ -281,31 +281,31 @@ 
 		try
 		{
 			$total = 0;
-			$params = $this->storeSearchParams( $view->param(), 'customer' );
-			$manager = \Aimeos\MShop::create( $context, 'customer' );
-			$search = $this->initCriteria( $manager->createSearch(), $params );
+			$params = $this->storeSearchParams($view->param(), 'customer');
+			$manager = \Aimeos\MShop::create($context, 'customer');
+			$search = $this->initCriteria($manager->createSearch(), $params);
 
-			$view->items = $manager->searchItems( $search, $this->getDomains(), $total );
-			$view->filterAttributes = $manager->getSearchAttributes( true );
+			$view->items = $manager->searchItems($search, $this->getDomains(), $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( 'customer-item' => $context->getI18n()->dt( 'mshop', $e->getMessage() ) );
-			$view->errors = $view->get( 'errors', [] ) + $error;
-			$this->logException( $e );
+			$error = array('customer-item' => $context->getI18n()->dt('mshop', $e->getMessage()));
+			$view->errors = $view->get('errors', []) + $error;
+			$this->logException($e);
 		}
-		catch( \Exception $e )
+		catch (\Exception $e)
 		{
-			$error = array( 'customer-item' => $e->getMessage() . ', ' . $e->getFile() . ':' . $e->getLine() );
-			$view->errors = $view->get( 'errors', [] ) + $error;
-			$this->logException( $e );
+			$error = array('customer-item' => $e->getMessage() . ', ' . $e->getFile() . ':' . $e->getLine());
+			$view->errors = $view->get('errors', []) + $error;
+			$this->logException($e);
 		}
 
 		/** admin/jqadm/customer/template-list
@@ -330,7 +330,7 @@ 
 		$tplconf = 'admin/jqadm/customer/template-list';
 		$default = 'customer/list-standard';
 
-		return $view->render( $view->config( $tplconf, $default ) );
+		return $view->render($view->config($tplconf, $default));
 	}
 
 
@@ -341,7 +341,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/customer/decorators/excludes
 		 * Excludes decorators added by the "common" option from the customer JQAdm client
@@ -416,7 +416,7 @@ 
 		 * @see admin/jqadm/customer/decorators/excludes
 		 * @see admin/jqadm/customer/decorators/global
 		 */
-		return $this->createSubClient( 'customer/' . $type, $name );
+		return $this->createSubClient('customer/' . $type, $name);
 	}
 
 
@@ -440,7 +440,7 @@ 
 		 */
 		$domains = ['customer/address', 'customer/group', 'customer/property'];
 
-		return $this->getContext()->getConfig()->get( 'admin/jqadm/customer/domains', $domains );
+		return $this->getContext()->getConfig()->get('admin/jqadm/customer/domains', $domains);
 	}
 
 
@@ -452,20 +452,20 @@ 
 	protected function getGroupItems()
 	{
 		$list = [];
-		$isSuper = $this->getView()->access( ['super'] );
-		$isAdmin = $this->getView()->access( ['admin'] );
+		$isSuper = $this->getView()->access(['super']);
+		$isAdmin = $this->getView()->access(['admin']);
 
-		$manager = \Aimeos\MShop::create( $this->getContext(), 'customer/group' );
+		$manager = \Aimeos\MShop::create($this->getContext(), 'customer/group');
 		$search = $manager->createSearch();
-		$search->setSortations( [$search->sort( '+', 'customer.group.label' )] );
+		$search->setSortations([$search->sort('+', 'customer.group.label')]);
 
-		foreach( $manager->searchItems( $search ) as $groupId => $groupItem )
+		foreach ($manager->searchItems($search) as $groupId => $groupItem)
 		{
-			if( !$isSuper && in_array( $groupItem->getCode(), ['super'] ) ) {
+			if (!$isSuper && in_array($groupItem->getCode(), ['super'])) {
 				continue;
 			}
 
-			if( !$isSuper && !$isAdmin && in_array( $groupItem->getCode(), ['super', 'admin', 'editor'] ) ) {
+			if (!$isSuper && !$isAdmin && in_array($groupItem->getCode(), ['super', 'admin', 'editor'])) {
 				continue;
 			}
 
@@ -516,7 +516,7 @@ 
 		 * @since 2017.07
 		 * @category Developer
 		 */
-		return $this->getContext()->getConfig()->get( 'admin/jqadm/customer/standard/subparts', [] );
+		return $this->getContext()->getConfig()->get('admin/jqadm/customer/standard/subparts', []);
 	}
 
 
@@ -527,21 +527,21 @@ 
 	 * @param string[] Data array
 	 * @return \Aimeos\MShop\Customer\Item\Iface New customer item object
 	 */
-	protected function fromArray( array $data )
+	protected function fromArray(array $data)
 	{
 		$data['customer.label'] = $data['customer.firstname'] . ' ' . $data['customer.lastname'];
 		$data['customer.code'] = $data['customer.email'];
 
-		$manager = \Aimeos\MShop::create( $this->getContext(), 'customer' );
+		$manager = \Aimeos\MShop::create($this->getContext(), 'customer');
 
-		if( isset( $data['customer.id'] ) && $data['customer.id'] != '' ) {
-			$item = $manager->getItem( $data['customer.id'], $this->getDomains() );
+		if (isset($data['customer.id']) && $data['customer.id'] != '') {
+			$item = $manager->getItem($data['customer.id'], $this->getDomains());
 		} else {
 			$item = $manager->createItem();
 		}
 
-		$item->fromArray( $data );
-		$item->setGroups( array_intersect( array_keys( $this->getGroupItems() ), $item->getGroups() ) );
+		$item->fromArray($data);
+		$item->setGroups(array_intersect(array_keys($this->getGroupItems()), $item->getGroups()));
 
 		return $item;
 	}
@@ -553,11 +553,11 @@ 
 	 * @param \Aimeos\MShop\Customer\Item\Iface $item Customer item object
 	 * @return string[] Multi-dimensional associative list of item data
 	 */
-	protected function toArray( \Aimeos\MShop\Customer\Item\Iface $item, $copy = false )
+	protected function toArray(\Aimeos\MShop\Customer\Item\Iface $item, $copy = false)
 	{
-		$data = $item->toArray( true );
+		$data = $item->toArray(true);
 
-		if( $copy === true )
+		if ($copy === true)
 		{
 			$data['customer.siteid'] = $this->getContext()->getLocale()->getSiteId();
 			$data['customer.email'] = '';
@@ -574,7 +574,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/customer/template-item
 		 * Relative path to the HTML body template for the customer item.
@@ -598,6 +598,6 @@ 
 		$tplconf = 'admin/jqadm/customer/template-item';
 		$default = 'customer/item-standard';
 
-		return $view->render( $view->config( $tplconf, $default ) );
+		return $view->render($view->config($tplconf, $default));
 	}
 }

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

Braces +2 added lines, -4 removed lines

@@ -84,14 +84,12 @@
 			foreach( $this->getSubClients() as $client ) {
 				$view->itemBody .= $client->search();
 			}
-		}
-		catch( \Aimeos\MAdmin\Exception $e )
+		} catch( \Aimeos\MAdmin\Exception $e )
 		{
 			$error = array( 'log-item' => $context->getI18n()->dt( 'mshop', $e->getMessage() ) );
 			$view->errors = $view->get( 'errors', [] ) + $error;
 			$this->logException( $e );
-		}
-		catch( \Exception $e )
+		} catch( \Exception $e )
 		{
 			$error = array( 'log-item' => $e->getMessage() . ', ' . $e->getFile() . ':' . $e->getLine() );
 			$view->errors = $view->get( 'errors', [] ) + $error;

Spacing +21 added lines, -21 removed lines

@@ -10,7 +10,7 @@ 
 
 namespace Aimeos\Admin\JQAdm\Log;
 
-sprintf( 'log' ); // for translation
+sprintf('log'); // for translation
 
 
 /**
@@ -36,32 +36,32 @@ 
 		try
 		{
 			$total = 0;
-			$params = $this->storeSearchParams( $view->param(), 'log' );
-			$manager = \Aimeos\MAdmin::create( $context, 'log' );
+			$params = $this->storeSearchParams($view->param(), 'log');
+			$manager = \Aimeos\MAdmin::create($context, 'log');
 
 			$search = $manager->createSearch();
-			$search->setSortations( [$search->sort( '-', 'log.timestamp')] );
-			$search = $this->initCriteria( $search, $params );
+			$search->setSortations([$search->sort('-', 'log.timestamp')]);
+			$search = $this->initCriteria($search, $params);
 
-			$view->items = $manager->searchItems( $search, [], $total );
+			$view->items = $manager->searchItems($search, [], $total);
 			$view->total = $total;
 			$view->itemBody = '';
 
-			foreach( $this->getSubClients() as $client ) {
+			foreach ($this->getSubClients() as $client) {
 				$view->itemBody .= $client->search();
 			}
 		}
-		catch( \Aimeos\MAdmin\Exception $e )
+		catch (\Aimeos\MAdmin\Exception $e)
 		{
-			$error = array( 'log-item' => $context->getI18n()->dt( 'mshop', $e->getMessage() ) );
-			$view->errors = $view->get( 'errors', [] ) + $error;
-			$this->logException( $e );
+			$error = array('log-item' => $context->getI18n()->dt('mshop', $e->getMessage()));
+			$view->errors = $view->get('errors', []) + $error;
+			$this->logException($e);
 		}
-		catch( \Exception $e )
+		catch (\Exception $e)
 		{
-			$error = array( 'log-item' => $e->getMessage() . ', ' . $e->getFile() . ':' . $e->getLine() );
-			$view->errors = $view->get( 'errors', [] ) + $error;
-			$this->logException( $e );
+			$error = array('log-item' => $e->getMessage() . ', ' . $e->getFile() . ':' . $e->getLine());
+			$view->errors = $view->get('errors', []) + $error;
+			$this->logException($e);
 		}
 
 		/** admin/jqadm/log/template-list
@@ -86,7 +86,7 @@ 
 		$tplconf = 'admin/jqadm/log/template-list';
 		$default = 'log/list-standard';
 
-		return $view->render( $view->config( $tplconf, $default ) );
+		return $view->render($view->config($tplconf, $default));
 	}
 
 
@@ -97,7 +97,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/log/decorators/excludes
 		 * Excludes decorators added by the "common" option from the log JQAdm client
@@ -172,7 +172,7 @@ 
 		 * @see admin/jqadm/log/decorators/excludes
 		 * @see admin/jqadm/log/decorators/global
 		 */
-		return $this->createSubClient( 'log/' . $type, $name );
+		return $this->createSubClient('log/' . $type, $name);
 	}
 
 
@@ -216,7 +216,7 @@ 
 		 * @since 2018.04
 		 * @category Developer
 		 */
-		return $this->getContext()->getConfig()->get( 'admin/jqadm/log/standard/subparts', [] );
+		return $this->getContext()->getConfig()->get('admin/jqadm/log/standard/subparts', []);
 	}
 
 
@@ -226,7 +226,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/log/template-item
 		 * Relative path to the HTML body template for the log item.
@@ -250,6 +250,6 @@ 
 		$tplconf = 'admin/jqadm/log/template-item';
 		$default = 'log/item-standard';
 
-		return $view->render( $view->config( $tplconf, $default ) );
+		return $view->render($view->config($tplconf, $default));
 	}
 }

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

Braces +13 added lines, -26 removed lines

@@ -121,14 +121,12 @@ 
 				$view->tabindex = ++$idx + 1;
 				$view->itemBody .= $client->copy();
 			}
-		}
-		catch( \Aimeos\MShop\Exception $e )
+		} catch( \Aimeos\MShop\Exception $e )
 		{
 			$error = array( 'supplier-item' => $context->getI18n()->dt( 'mshop', $e->getMessage() ) );
 			$view->errors = $view->get( 'errors', [] ) + $error;
 			$this->logException( $e );
-		}
-		catch( \Exception $e )
+		} catch( \Exception $e )
 		{
 			$error = array( 'supplier-item' => $e->getMessage() . ', ' . $e->getFile() . ':' . $e->getLine() );
 			$view->errors = $view->get( 'errors', [] ) + $error;
@@ -170,14 +168,12 @@ 
 				$view->tabindex = ++$idx + 1;
 				$view->itemBody .= $client->create();
 			}
-		}
-		catch( \Aimeos\MShop\Exception $e )
+		} catch( \Aimeos\MShop\Exception $e )
 		{
 			$error = array( 'supplier-item' => $context->getI18n()->dt( 'mshop', $e->getMessage() ) );
 			$view->errors = $view->get( 'errors', [] ) + $error;
 			$this->logException( $e );
-		}
-		catch( \Exception $e )
+		} catch( \Exception $e )
 		{
 			$error = array( 'supplier-item' => $e->getMessage() . ', ' . $e->getFile() . ':' . $e->getLine() );
 			$view->errors = $view->get( 'errors', [] ) + $error;
@@ -218,14 +214,12 @@ 
 
 			$this->nextAction( $view, 'search', 'supplier', null, 'delete' );
 			return;
-		}
-		catch( \Aimeos\MShop\Exception $e )
+		} catch( \Aimeos\MShop\Exception $e )
 		{
 			$error = array( 'supplier-item' => $context->getI18n()->dt( 'mshop', $e->getMessage() ) );
 			$view->errors = $view->get( 'errors', [] ) + $error;
 			$this->logException( $e );
-		}
-		catch( \Exception $e )
+		} catch( \Exception $e )
 		{
 			$error = array( 'supplier-item' => $e->getMessage() . ', ' . $e->getFile() . ':' . $e->getLine() );
 			$view->errors = $view->get( 'errors', [] ) + $error;
@@ -266,14 +260,12 @@ 
 				$view->tabindex = ++$idx + 1;
 				$view->itemBody .= $client->get();
 			}
-		}
-		catch( \Aimeos\MShop\Exception $e )
+		} catch( \Aimeos\MShop\Exception $e )
 		{
 			$error = array( 'supplier-item' => $context->getI18n()->dt( 'mshop', $e->getMessage() ) );
 			$view->errors = $view->get( 'errors', [] ) + $error;
 			$this->logException( $e );
-		}
-		catch( \Exception $e )
+		} catch( \Exception $e )
 		{
 			$error = array( 'supplier-item' => $e->getMessage() . ', ' . $e->getFile() . ':' . $e->getLine() );
 			$view->errors = $view->get( 'errors', [] ) + $error;
@@ -310,18 +302,15 @@ 
 
 			$this->nextAction( $view, $view->param( 'next' ), 'supplier', $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( 'supplier-item' => $context->getI18n()->dt( 'mshop', $e->getMessage() ) );
 			$view->errors = $view->get( 'errors', [] ) + $error;
 			$this->logException( $e );
-		}
-		catch( \Exception $e )
+		} catch( \Exception $e )
 		{
 			$error = array( 'supplier-item' => $e->getMessage() . ', ' . $e->getFile() . ':' . $e->getLine() );
 			$view->errors = $view->get( 'errors', [] ) + $error;
@@ -360,14 +349,12 @@ 
 			foreach( $this->getSubClients() as $client ) {
 				$view->itemBody .= $client->search();
 			}
-		}
-		catch( \Aimeos\MShop\Exception $e )
+		} catch( \Aimeos\MShop\Exception $e )
 		{
 			$error = array( 'supplier-item' => $context->getI18n()->dt( 'mshop', $e->getMessage() ) );
 			$view->errors = $view->get( 'errors', [] ) + $error;
 			$this->logException( $e );
-		}
-		catch( \Exception $e )
+		} catch( \Exception $e )
 		{
 			$error = array( 'supplier-item' => $e->getMessage() . ', ' . $e->getFile() . ':' . $e->getLine() );
 			$view->errors = $view->get( 'errors', [] ) + $error;

Spacing +104 added lines, -104 removed lines

@@ -10,7 +10,7 @@ 
 
 namespace Aimeos\Admin\JQAdm\Supplier;
 
-sprintf( 'supplier' ); // for translation
+sprintf('supplier'); // 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, 'supplier' );
-			$view->item = $manager->getItem( $id, $this->getDomains() );
+			$manager = \Aimeos\MShop::create($context, 'supplier');
+			$view->item = $manager->getItem($id, $this->getDomains());
 
-			$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( 'supplier-item' => $context->getI18n()->dt( 'mshop', $e->getMessage() ) );
-			$view->errors = $view->get( 'errors', [] ) + $error;
-			$this->logException( $e );
+			$error = array('supplier-item' => $context->getI18n()->dt('mshop', $e->getMessage()));
+			$view->errors = $view->get('errors', []) + $error;
+			$this->logException($e);
 		}
-		catch( \Exception $e )
+		catch (\Exception $e)
 		{
-			$error = array( 'supplier-item' => $e->getMessage() . ', ' . $e->getFile() . ':' . $e->getLine() );
-			$view->errors = $view->get( 'errors', [] ) + $error;
-			$this->logException( $e );
+			$error = array('supplier-item' => $e->getMessage() . ', ' . $e->getFile() . ':' . $e->getLine());
+			$view->errors = $view->get('errors', []) + $error;
+			$this->logException($e);
 		}
 
-		return $this->render( $view );
+		return $this->render($view);
 	}
 
 
@@ -81,10 +81,10 @@ 
 
 		try
 		{
-			$data = $view->param( 'item', [] );
+			$data = $view->param('item', []);
 
-			if( !isset( $view->item ) ) {
-				$view->item = \Aimeos\MShop::create( $context, 'supplier' )->createItem();
+			if (!isset($view->item)) {
+				$view->item = \Aimeos\MShop::create($context, 'supplier')->createItem();
 			}
 
 			$data['supplier.siteid'] = $view->item->getSiteId();
@@ -93,26 +93,26 @@ 
 			$view->itemData = $data;
 			$view->itemBody = '';
 
-			foreach( $this->getSubClients() as $idx => $client )
+			foreach ($this->getSubClients() as $idx => $client)
 			{
 				$view->tabindex = ++$idx + 1;
 				$view->itemBody .= $client->create();
 			}
 		}
-		catch( \Aimeos\MShop\Exception $e )
+		catch (\Aimeos\MShop\Exception $e)
 		{
-			$error = array( 'supplier-item' => $context->getI18n()->dt( 'mshop', $e->getMessage() ) );
-			$view->errors = $view->get( 'errors', [] ) + $error;
-			$this->logException( $e );
+			$error = array('supplier-item' => $context->getI18n()->dt('mshop', $e->getMessage()));
+			$view->errors = $view->get('errors', []) + $error;
+			$this->logException($e);
 		}
-		catch( \Exception $e )
+		catch (\Exception $e)
 		{
-			$error = array( 'supplier-item' => $e->getMessage() . ', ' . $e->getFile() . ':' . $e->getLine() );
-			$view->errors = $view->get( 'errors', [] ) + $error;
-			$this->logException( $e );
+			$error = array('supplier-item' => $e->getMessage() . ', ' . $e->getFile() . ':' . $e->getLine());
+			$view->errors = $view->get('errors', []) + $error;
+			$this->logException($e);
 		}
 
-		return $this->render( $view );
+		return $this->render($view);
 	}
 
 
@@ -126,39 +126,39 @@ 
 		$view = $this->getView();
 		$context = $this->getContext();
 
-		$manager = \Aimeos\MShop::create( $context, 'supplier' );
+		$manager = \Aimeos\MShop::create($context, 'supplier');
 		$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, $this->getDomains() );
+			$view->item = $manager->getItem($id, $this->getDomains());
 
-			foreach( $this->getSubClients() as $client ) {
+			foreach ($this->getSubClients() as $client) {
 				$client->delete();
 			}
 
-			$manager->saveItem( $view->item );
-			$manager->deleteItem( $id );
+			$manager->saveItem($view->item);
+			$manager->deleteItem($id);
 			$manager->commit();
 
-			$this->nextAction( $view, 'search', 'supplier', null, 'delete' );
+			$this->nextAction($view, 'search', 'supplier', null, 'delete');
 			return;
 		}
-		catch( \Aimeos\MShop\Exception $e )
+		catch (\Aimeos\MShop\Exception $e)
 		{
-			$error = array( 'supplier-item' => $context->getI18n()->dt( 'mshop', $e->getMessage() ) );
-			$view->errors = $view->get( 'errors', [] ) + $error;
-			$this->logException( $e );
+			$error = array('supplier-item' => $context->getI18n()->dt('mshop', $e->getMessage()));
+			$view->errors = $view->get('errors', []) + $error;
+			$this->logException($e);
 		}
-		catch( \Exception $e )
+		catch (\Exception $e)
 		{
-			$error = array( 'supplier-item' => $e->getMessage() . ', ' . $e->getFile() . ':' . $e->getLine() );
-			$view->errors = $view->get( 'errors', [] ) + $error;
-			$this->logException( $e );
+			$error = array('supplier-item' => $e->getMessage() . ', ' . $e->getFile() . ':' . $e->getLine());
+			$view->errors = $view->get('errors', []) + $error;
+			$this->logException($e);
 		}
 
 		$manager->rollback();
@@ -179,37 +179,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, 'supplier' );
+			$manager = \Aimeos\MShop::create($context, 'supplier');
 
-			$view->item = $manager->getItem( $id, $this->getDomains() );
+			$view->item = $manager->getItem($id, $this->getDomains());
 			$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( 'supplier-item' => $context->getI18n()->dt( 'mshop', $e->getMessage() ) );
-			$view->errors = $view->get( 'errors', [] ) + $error;
-			$this->logException( $e );
+			$error = array('supplier-item' => $context->getI18n()->dt('mshop', $e->getMessage()));
+			$view->errors = $view->get('errors', []) + $error;
+			$this->logException($e);
 		}
-		catch( \Exception $e )
+		catch (\Exception $e)
 		{
-			$error = array( 'supplier-item' => $e->getMessage() . ', ' . $e->getFile() . ':' . $e->getLine() );
-			$view->errors = $view->get( 'errors', [] ) + $error;
-			$this->logException( $e );
+			$error = array('supplier-item' => $e->getMessage() . ', ' . $e->getFile() . ':' . $e->getLine());
+			$view->errors = $view->get('errors', []) + $error;
+			$this->logException($e);
 		}
 
-		return $this->render( $view );
+		return $this->render($view);
 	}
 
 
@@ -223,40 +223,40 @@ 
 		$view = $this->getView();
 		$context = $this->getContext();
 
-		$manager = \Aimeos\MShop::create( $context, 'supplier' );
+		$manager = \Aimeos\MShop::create($context, 'supplier');
 		$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' ), 'supplier', $view->item->getId(), 'save' );
+			$this->nextAction($view, $view->param('next'), 'supplier', $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( 'supplier-item' => $context->getI18n()->dt( 'mshop', $e->getMessage() ) );
-			$view->errors = $view->get( 'errors', [] ) + $error;
-			$this->logException( $e );
+			$error = array('supplier-item' => $context->getI18n()->dt('mshop', $e->getMessage()));
+			$view->errors = $view->get('errors', []) + $error;
+			$this->logException($e);
 		}
-		catch( \Exception $e )
+		catch (\Exception $e)
 		{
-			$error = array( 'supplier-item' => $e->getMessage() . ', ' . $e->getFile() . ':' . $e->getLine() );
-			$view->errors = $view->get( 'errors', [] ) + $error;
-			$this->logException( $e );
+			$error = array('supplier-item' => $e->getMessage() . ', ' . $e->getFile() . ':' . $e->getLine());
+			$view->errors = $view->get('errors', []) + $error;
+			$this->logException($e);
 		}
 
 		$manager->rollback();
@@ -278,31 +278,31 @@ 
 		try
 		{
 			$total = 0;
-			$params = $this->storeSearchParams( $view->param(), 'supplier' );
-			$manager = \Aimeos\MShop::create( $context, 'supplier' );
-			$search = $this->initCriteria( $manager->createSearch(), $params );
+			$params = $this->storeSearchParams($view->param(), 'supplier');
+			$manager = \Aimeos\MShop::create($context, 'supplier');
+			$search = $this->initCriteria($manager->createSearch(), $params);
 
-			$view->items = $manager->searchItems( $search, $this->getDomains(), $total );
-			$view->filterAttributes = $manager->getSearchAttributes( true );
+			$view->items = $manager->searchItems($search, $this->getDomains(), $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( 'supplier-item' => $context->getI18n()->dt( 'mshop', $e->getMessage() ) );
-			$view->errors = $view->get( 'errors', [] ) + $error;
-			$this->logException( $e );
+			$error = array('supplier-item' => $context->getI18n()->dt('mshop', $e->getMessage()));
+			$view->errors = $view->get('errors', []) + $error;
+			$this->logException($e);
 		}
-		catch( \Exception $e )
+		catch (\Exception $e)
 		{
-			$error = array( 'supplier-item' => $e->getMessage() . ', ' . $e->getFile() . ':' . $e->getLine() );
-			$view->errors = $view->get( 'errors', [] ) + $error;
-			$this->logException( $e );
+			$error = array('supplier-item' => $e->getMessage() . ', ' . $e->getFile() . ':' . $e->getLine());
+			$view->errors = $view->get('errors', []) + $error;
+			$this->logException($e);
 		}
 
 		/** admin/jqadm/supplier/template-list
@@ -327,7 +327,7 @@ 
 		$tplconf = 'admin/jqadm/supplier/template-list';
 		$default = 'supplier/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/supplier/decorators/excludes
 		 * Excludes decorators added by the "common" option from the supplier JQAdm client
@@ -413,7 +413,7 @@ 
 		 * @see admin/jqadm/supplier/decorators/excludes
 		 * @see admin/jqadm/supplier/decorators/global
 		 */
-		return $this->createSubClient( 'supplier/' . $type, $name );
+		return $this->createSubClient('supplier/' . $type, $name);
 	}
 
 
@@ -437,7 +437,7 @@ 
 		 */
 		$domains = ['media', 'supplier/address', 'text'];
 
-		return $this->getContext()->getConfig()->get( 'admin/jqadm/supplier/domains', $domains );
+		return $this->getContext()->getConfig()->get('admin/jqadm/supplier/domains', $domains);
 	}
 
 
@@ -481,7 +481,7 @@ 
 		 * @since 2017.10
 		 * @category Developer
 		 */
-		return $this->getContext()->getConfig()->get( 'admin/jqadm/supplier/standard/subparts', [] );
+		return $this->getContext()->getConfig()->get('admin/jqadm/supplier/standard/subparts', []);
 	}
 
 
@@ -492,17 +492,17 @@ 
 	 * @param string[] Data array
 	 * @return \Aimeos\MShop\Supplier\Item\Iface New supplier item object
 	 */
-	protected function fromArray( array $data )
+	protected function fromArray(array $data)
 	{
-		$manager = \Aimeos\MShop::create( $this->getContext(), 'supplier' );
+		$manager = \Aimeos\MShop::create($this->getContext(), 'supplier');
 
-		if( isset( $data['supplier.id'] ) && $data['supplier.id'] != '' ) {
-			$item = $manager->getItem( $data['supplier.id'], $this->getDomains() );
+		if (isset($data['supplier.id']) && $data['supplier.id'] != '') {
+			$item = $manager->getItem($data['supplier.id'], $this->getDomains());
 		} else {
 			$item = $manager->createItem();
 		}
 
-		$item->fromArray( $data );
+		$item->fromArray($data);
 
 		return $item;
 	}
@@ -514,11 +514,11 @@ 
 	 * @param \Aimeos\MShop\Supplier\Item\Iface $item Supplier item object
 	 * @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 = $item->toArray( true );
+		$data = $item->toArray(true);
 
-		if( $copy === true )
+		if ($copy === true)
 		{
 			$data['supplier.siteid'] = $this->getContext()->getLocale()->getSiteId();
 			$data['supplier.code'] = $data['supplier.code'] . '_copy';
@@ -535,7 +535,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/template-item
 		 * Relative path to the HTML body template for the supplier item.
@@ -559,6 +559,6 @@ 
 		$tplconf = 'admin/jqadm/supplier/template-item';
 		$default = 'supplier/item-standard';
 
-		return $view->render( $view->config( $tplconf, $default ) );
+		return $view->render($view->config($tplconf, $default));
 	}
 }