aimeos / ai-client-html

client/html/src/Client/Html/Catalog/Stock/Factory.php

Spacing +9 added lines, -9 removed lines

@@ -31,7 +31,7 @@ 
 	 * @return \Aimeos\Client\Html\Iface Filter part implementing \Aimeos\Client\Html\Iface
 	 * @throws \Aimeos\Client\Html\Exception If requested client implementation couldn't be found or initialisation fails
 	 */
-	public static function createClient( \Aimeos\MShop\Context\Item\Iface $context, array $templatePaths, $name = null )
+	public static function createClient(\Aimeos\MShop\Context\Item\Iface $context, array $templatePaths, $name = null)
 	{
 		/** client/html/catalog/stock/name
 		 * Class name of the used catalog stock client implementation
@@ -66,22 +66,22 @@ 
 		 * @since 2014.03
 		 * @category Developer
 		 */
-		if( $name === null ) {
-			$name = $context->getConfig()->get( 'client/html/catalog/stock/name', 'Standard' );
+		if ($name === null) {
+			$name = $context->getConfig()->get('client/html/catalog/stock/name', 'Standard');
 		}
 
-		if( ctype_alnum( $name ) === false )
+		if (ctype_alnum($name) === false)
 		{
-			$classname = is_string( $name ) ? '\\Aimeos\\Client\\Html\\Catalog\\Stock\\' . $name : '<not a string>';
-			throw new \Aimeos\Client\Html\Exception( sprintf( 'Invalid characters in class name "%1$s"', $classname ) );
+			$classname = is_string($name) ? '\\Aimeos\\Client\\Html\\Catalog\\Stock\\'.$name : '<not a string>';
+			throw new \Aimeos\Client\Html\Exception(sprintf('Invalid characters in class name "%1$s"', $classname));
 		}
 
 		$iface = '\\Aimeos\\Client\\Html\\Iface';
-		$classname = '\\Aimeos\\Client\\Html\\Catalog\\Stock\\' . $name;
+		$classname = '\\Aimeos\\Client\\Html\\Catalog\\Stock\\'.$name;
 
-		$client = self::createClientBase( $context, $classname, $iface, $templatePaths );
+		$client = self::createClientBase($context, $classname, $iface, $templatePaths);
 
-		return self::addClientDecorators( $context, $client, $templatePaths, 'catalog/stock' );
+		return self::addClientDecorators($context, $client, $templatePaths, 'catalog/stock');
 	}
 }
 

client/html/src/Client/Html/Catalog/Stock/Standard.php

Spacing +43 added lines, -43 removed lines

@@ -68,15 +68,15 @@ 
 	 * @param string|null &$expire Result variable for the expiration date of the output (null for no expiry)
 	 * @return string HTML code
 	 */
-	public function getBody( $uid = '', array &$tags = array(), &$expire = null )
+	public function getBody($uid = '', array &$tags = array(), &$expire = null)
 	{
 		try
 		{
-			$view = $this->setViewParams( $this->getView(), $tags, $expire );
+			$view = $this->setViewParams($this->getView(), $tags, $expire);
 
 			$html = '';
-			foreach( $this->getSubClients() as $subclient ) {
-				$html .= $subclient->setView( $view )->getBody( $uid, $tags, $expire );
+			foreach ($this->getSubClients() as $subclient) {
+				$html .= $subclient->setView($view)->getBody($uid, $tags, $expire);
 			}
 			$view->stockBody = $html;
 
@@ -103,11 +103,11 @@ 
 			$tplconf = 'client/html/catalog/stock/standard/template-body';
 			$default = 'catalog/stock/body-default.php';
 
-			return $view->render( $view->config( $tplconf, $default ) );
+			return $view->render($view->config($tplconf, $default));
 		}
-		catch( \Exception $e )
+		catch (\Exception $e)
 		{
-			$this->getContext()->getLogger()->log( $e->getMessage() . PHP_EOL . $e->getTraceAsString() );
+			$this->getContext()->getLogger()->log($e->getMessage().PHP_EOL.$e->getTraceAsString());
 		}
 	}
 
@@ -120,15 +120,15 @@ 
 	 * @param string|null &$expire Result variable for the expiration date of the output (null for no expiry)
 	 * @return string|null String including HTML tags for the header on error
 	 */
-	public function getHeader( $uid = '', array &$tags = array(), &$expire = null )
+	public function getHeader($uid = '', array &$tags = array(), &$expire = null)
 	{
 		try
 		{
-			$view = $this->setViewParams( $this->getView(), $tags, $expire );
+			$view = $this->setViewParams($this->getView(), $tags, $expire);
 
 			$html = '';
-			foreach( $this->getSubClients() as $subclient ) {
-				$html .= $subclient->setView( $view )->getHeader( $uid, $tags, $expire );
+			foreach ($this->getSubClients() as $subclient) {
+				$html .= $subclient->setView($view)->getHeader($uid, $tags, $expire);
 			}
 			$view->stockHeader = $html;
 
@@ -156,11 +156,11 @@ 
 			$tplconf = 'client/html/catalog/stock/standard/template-header';
 			$default = 'catalog/stock/header-default.php';
 
-			return $view->render( $view->config( $tplconf, $default ) );
+			return $view->render($view->config($tplconf, $default));
 		}
-		catch( \Exception $e )
+		catch (\Exception $e)
 		{
-			$this->getContext()->getLogger()->log( $e->getMessage() . PHP_EOL . $e->getTraceAsString() );
+			$this->getContext()->getLogger()->log($e->getMessage().PHP_EOL.$e->getTraceAsString());
 		}
 	}
 
@@ -172,7 +172,7 @@ 
 	 * @param string|null $name Name of the sub-client (Default if null)
 	 * @return \Aimeos\Client\Html\Iface Sub-client object
 	 */
-	public function getSubClient( $type, $name = null )
+	public function getSubClient($type, $name = null)
 	{
 		/** client/html/catalog/stock/decorators/excludes
 		 * Excludes decorators added by the "common" option from the catalog stock html client
@@ -248,7 +248,7 @@ 
 		 * @see client/html/catalog/stock/decorators/global
 		 */
 
-		return $this->createSubClient( 'catalog/stock/' . $type, $name );
+		return $this->createSubClient('catalog/stock/'.$type, $name);
 	}
 
 
@@ -263,9 +263,9 @@ 
 		{
 			parent::process();
 		}
-		catch( \Exception $e )
+		catch (\Exception $e)
 		{
-			$this->getContext()->getLogger()->log( $e->getMessage() . PHP_EOL . $e->getTraceAsString() );
+			$this->getContext()->getLogger()->log($e->getMessage().PHP_EOL.$e->getTraceAsString());
 		}
 	}
 
@@ -277,7 +277,7 @@ 
 	 */
 	protected function getSubClientNames()
 	{
-		return $this->getContext()->getConfig()->get( $this->subPartPath, $this->subPartNames );
+		return $this->getContext()->getConfig()->get($this->subPartPath, $this->subPartNames);
 	}
 
 
@@ -289,9 +289,9 @@ 
 	 * @param string|null &$expire Result variable for the expiration date of the output (null for no expiry)
 	 * @return \Aimeos\MW\View\Iface Modified view object
 	 */
-	protected function setViewParams( \Aimeos\MW\View\Iface $view, array &$tags = array(), &$expire = null )
+	protected function setViewParams(\Aimeos\MW\View\Iface $view, array &$tags = array(), &$expire = null)
 	{
-		if( !isset( $this->cache ) )
+		if (!isset($this->cache))
 		{
 			$context = $this->getContext();
 			$siteConfig = $context->getLocale()->getSite()->getConfig();
@@ -310,58 +310,58 @@ 
 			 * @category Developer
 			 * @see client/html/catalog/stock/level/low
 			 */
-			$sortkey = $context->getConfig()->get( 'client/html/catalog/stock/sort', 'product.stock.warehouseid' );
-			$productIds = $view->param( 's_prodid' );
+			$sortkey = $context->getConfig()->get('client/html/catalog/stock/sort', 'product.stock.warehouseid');
+			$productIds = $view->param('s_prodid');
 
-			if( !is_array( $productIds ) ) {
-				$productIds = explode( ' ', $productIds );
+			if (!is_array($productIds)) {
+				$productIds = explode(' ', $productIds);
 			}
 
 
-			$stockManager = \Aimeos\MShop\Factory::createManager( $context, 'product/stock' );
+			$stockManager = \Aimeos\MShop\Factory::createManager($context, 'product/stock');
 
-			$search = $stockManager->createSearch( true );
-			$expr = array( $search->compare( '==', 'product.stock.parentid', $productIds ) );
+			$search = $stockManager->createSearch(true);
+			$expr = array($search->compare('==', 'product.stock.parentid', $productIds));
 
-			if( isset( $siteConfig['warehouse'] ) ) {
-				$expr[] = $search->compare( '==', 'product.stock.warehouse.code', $siteConfig['warehouse'] );
+			if (isset($siteConfig['warehouse'])) {
+				$expr[] = $search->compare('==', 'product.stock.warehouse.code', $siteConfig['warehouse']);
 			}
 
 			$expr[] = $search->getConditions();
 
 			$sortations = array(
-				$search->sort( '+', 'product.stock.parentid' ),
-				$search->sort( '+', $sortkey ),
+				$search->sort('+', 'product.stock.parentid'),
+				$search->sort('+', $sortkey),
 			);
 
-			$search->setConditions( $search->combine( '&&', $expr ) );
-			$search->setSortations( $sortations );
-			$search->setSlice( 0, 0x7fffffff );
+			$search->setConditions($search->combine('&&', $expr));
+			$search->setSortations($sortations);
+			$search->setSlice(0, 0x7fffffff);
 
-			$stockItems = $stockManager->searchItems( $search );
+			$stockItems = $stockManager->searchItems($search);
 
 
-			if( !empty( $stockItems ) )
+			if (!empty($stockItems))
 			{
 				$warehouseIds = $stockItemsByProducts = array();
 
-				foreach( $stockItems as $item )
+				foreach ($stockItems as $item)
 				{
 					$warehouseIds[$item->getWarehouseId()] = null;
 					$stockItemsByProducts[$item->getParentId()][] = $item;
 				}
 
-				$warehouseIds = array_keys( $warehouseIds );
+				$warehouseIds = array_keys($warehouseIds);
 
 
-				$warehouseManager = \Aimeos\MShop\Factory::createManager( $context, 'product/stock/warehouse' );
+				$warehouseManager = \Aimeos\MShop\Factory::createManager($context, 'product/stock/warehouse');
 
 				$search = $warehouseManager->createSearch();
-				$search->setConditions( $search->compare( '==', 'product.stock.warehouse.id', $warehouseIds ) );
-				$search->setSlice( 0, count( $warehouseIds ) );
+				$search->setConditions($search->compare('==', 'product.stock.warehouse.id', $warehouseIds));
+				$search->setSlice(0, count($warehouseIds));
 
 
-				$view->stockWarehouseItems = $warehouseManager->searchItems( $search );
+				$view->stockWarehouseItems = $warehouseManager->searchItems($search);
 				$view->stockItemsByProducts = $stockItemsByProducts;
 			}
 

client/html/src/Client/Html/Catalog/Count/Attribute/Standard.php

Spacing +21 added lines, -21 removed lines

@@ -68,13 +68,13 @@ 
 	 * @param string|null &$expire Result variable for the expiration date of the output (null for no expiry)
 	 * @return string HTML code
 	 */
-	public function getBody( $uid = '', array &$tags = array(), &$expire = null )
+	public function getBody($uid = '', array &$tags = array(), &$expire = null)
 	{
-		$view = $this->setViewParams( $this->getView(), $tags, $expire );
+		$view = $this->setViewParams($this->getView(), $tags, $expire);
 
 		$html = '';
-		foreach( $this->getSubClients() as $subclient ) {
-			$html .= $subclient->setView( $view )->getBody( $uid, $tags, $expire );
+		foreach ($this->getSubClients() as $subclient) {
+			$html .= $subclient->setView($view)->getBody($uid, $tags, $expire);
 		}
 		$view->attributeBody = $html;
 
@@ -101,7 +101,7 @@ 
 		$tplconf = 'client/html/catalog/count/attribute/standard/template-body';
 		$default = 'catalog/count/attribute-body-default.php';
 
-		return $view->render( $view->config( $tplconf, $default ) );
+		return $view->render($view->config($tplconf, $default));
 	}
 
 
@@ -113,13 +113,13 @@ 
 	 * @param string|null &$expire Result variable for the expiration date of the output (null for no expiry)
 	 * @return string|null String including HTML tags for the header on error
 	 */
-	public function getHeader( $uid = '', array &$tags = array(), &$expire = null )
+	public function getHeader($uid = '', array &$tags = array(), &$expire = null)
 	{
-		$view = $this->setViewParams( $this->getView(), $tags, $expire );
+		$view = $this->setViewParams($this->getView(), $tags, $expire);
 
 		$html = '';
-		foreach( $this->getSubClients() as $subclient ) {
-			$html .= $subclient->setView( $view )->getHeader( $uid, $tags, $expire );
+		foreach ($this->getSubClients() as $subclient) {
+			$html .= $subclient->setView($view)->getHeader($uid, $tags, $expire);
 		}
 		$view->attributeHeader = $html;
 
@@ -147,7 +147,7 @@ 
 		$tplconf = 'client/html/catalog/count/attribute/standard/template-header';
 		$default = 'catalog/count/attribute-header-default.php';
 
-		return $view->render( $view->config( $tplconf, $default ) );
+		return $view->render($view->config($tplconf, $default));
 	}
 
 
@@ -158,7 +158,7 @@ 
 	 * @param string|null $name Name of the sub-client (Default if null)
 	 * @return \Aimeos\Client\Html\Iface Sub-client object
 	 */
-	public function getSubClient( $type, $name = null )
+	public function getSubClient($type, $name = null)
 	{
 		/** client/html/catalog/count/attribute/decorators/excludes
 		 * Excludes decorators added by the "common" option from the catalog count attribute html client
@@ -234,7 +234,7 @@ 
 		 * @see client/html/catalog/count/attribute/decorators/global
 		 */
 
-		return $this->createSubClient( 'catalog/count/attribute/' . $type, $name );
+		return $this->createSubClient('catalog/count/attribute/'.$type, $name);
 	}
 
 
@@ -245,7 +245,7 @@ 
 	 */
 	protected function getSubClientNames()
 	{
-		return $this->getContext()->getConfig()->get( $this->subPartPath, $this->subPartNames );
+		return $this->getContext()->getConfig()->get($this->subPartPath, $this->subPartNames);
 	}
 
 
@@ -257,9 +257,9 @@ 
 	 * @param string|null &$expire Result variable for the expiration date of the output (null for no expiry)
 	 * @return \Aimeos\MW\View\Iface Modified view object
 	 */
-	protected function setViewParams( \Aimeos\MW\View\Iface $view, array &$tags = array(), &$expire = null )
+	protected function setViewParams(\Aimeos\MW\View\Iface $view, array &$tags = array(), &$expire = null)
 	{
-		if( !isset( $this->cache ) )
+		if (!isset($this->cache))
 		{
 			$context = $this->getContext();
 			$config = $context->getConfig();
@@ -276,7 +276,7 @@ 
 			 * @category Developer
 			 * @category User
 			 */
-			if( $config->get( 'client/html/catalog/count/attribute/aggregate', true ) == true )
+			if ($config->get('client/html/catalog/count/attribute/aggregate', true) == true)
 			{
 				/** client/html/catalog/count/limit
 				 * Limits the number of records that are used for product counts in the catalog filter
@@ -301,12 +301,12 @@ 
 				 * @category Developer
 				 * @category User
 				 */
-				$filter = $this->getProductListFilter( $view );
-				$filter->setSlice( 0, $config->get( 'client/html/catalog/count/limit', 10000 ) );
-				$filter->setSortations( array() ); // it's not necessary and slows down the query
+				$filter = $this->getProductListFilter($view);
+				$filter->setSlice(0, $config->get('client/html/catalog/count/limit', 10000));
+				$filter->setSortations(array()); // it's not necessary and slows down the query
 
-				$controller = \Aimeos\Controller\Frontend\Factory::createController( $context, 'catalog' );
-				$view->attributeCountList = $controller->aggregateIndex( $filter, 'index.attribute.id' );
+				$controller = \Aimeos\Controller\Frontend\Factory::createController($context, 'catalog');
+				$view->attributeCountList = $controller->aggregateIndex($filter, 'index.attribute.id');
 			}
 
 			$this->cache = $view;

client/html/src/Client/Html/Catalog/Count/Tree/Standard.php

Spacing +21 added lines, -21 removed lines

@@ -68,13 +68,13 @@ 
 	 * @param string|null &$expire Result variable for the expiration date of the output (null for no expiry)
 	 * @return string HTML code
 	 */
-	public function getBody( $uid = '', array &$tags = array(), &$expire = null )
+	public function getBody($uid = '', array &$tags = array(), &$expire = null)
 	{
-		$view = $this->setViewParams( $this->getView(), $tags, $expire );
+		$view = $this->setViewParams($this->getView(), $tags, $expire);
 
 		$html = '';
-		foreach( $this->getSubClients() as $subclient ) {
-			$html .= $subclient->setView( $view )->getBody( $uid, $tags, $expire );
+		foreach ($this->getSubClients() as $subclient) {
+			$html .= $subclient->setView($view)->getBody($uid, $tags, $expire);
 		}
 		$view->treeBody = $html;
 
@@ -101,7 +101,7 @@ 
 		$tplconf = 'client/html/catalog/count/tree/standard/template-body';
 		$default = 'catalog/count/tree-body-default.php';
 
-		return $view->render( $view->config( $tplconf, $default ) );
+		return $view->render($view->config($tplconf, $default));
 	}
 
 
@@ -113,13 +113,13 @@ 
 	 * @param string|null &$expire Result variable for the expiration date of the output (null for no expiry)
 	 * @return string|null String including HTML tags for the header on error
 	 */
-	public function getHeader( $uid = '', array &$tags = array(), &$expire = null )
+	public function getHeader($uid = '', array &$tags = array(), &$expire = null)
 	{
-		$view = $this->setViewParams( $this->getView(), $tags, $expire );
+		$view = $this->setViewParams($this->getView(), $tags, $expire);
 
 		$html = '';
-		foreach( $this->getSubClients() as $subclient ) {
-			$html .= $subclient->setView( $view )->getHeader( $uid, $tags, $expire );
+		foreach ($this->getSubClients() as $subclient) {
+			$html .= $subclient->setView($view)->getHeader($uid, $tags, $expire);
 		}
 		$view->treeHeader = $html;
 
@@ -147,7 +147,7 @@ 
 		$tplconf = 'client/html/catalog/count/tree/standard/template-header';
 		$default = 'catalog/count/tree-header-default.php';
 
-		return $view->render( $view->config( $tplconf, $default ) );
+		return $view->render($view->config($tplconf, $default));
 	}
 
 
@@ -158,7 +158,7 @@ 
 	 * @param string|null $name Name of the sub-client (Default if null)
 	 * @return \Aimeos\Client\Html\Iface Sub-client object
 	 */
-	public function getSubClient( $type, $name = null )
+	public function getSubClient($type, $name = null)
 	{
 		/** client/html/catalog/count/tree/decorators/excludes
 		 * Excludes decorators added by the "common" option from the catalog count tree html client
@@ -234,7 +234,7 @@ 
 		 * @see client/html/catalog/count/tree/decorators/global
 		 */
 
-		return $this->createSubClient( 'catalog/count/tree/' . $type, $name );
+		return $this->createSubClient('catalog/count/tree/'.$type, $name);
 	}
 
 
@@ -245,7 +245,7 @@ 
 	 */
 	protected function getSubClientNames()
 	{
-		return $this->getContext()->getConfig()->get( $this->subPartPath, $this->subPartNames );
+		return $this->getContext()->getConfig()->get($this->subPartPath, $this->subPartNames);
 	}
 
 
@@ -257,9 +257,9 @@ 
 	 * @param string|null &$expire Result variable for the expiration date of the output (null for no expiry)
 	 * @return \Aimeos\MW\View\Iface Modified view object
 	 */
-	protected function setViewParams( \Aimeos\MW\View\Iface $view, array &$tags = array(), &$expire = null )
+	protected function setViewParams(\Aimeos\MW\View\Iface $view, array &$tags = array(), &$expire = null)
 	{
-		if( !isset( $this->cache ) )
+		if (!isset($this->cache))
 		{
 			$context = $this->getContext();
 			$config = $context->getConfig();
@@ -274,17 +274,17 @@ 
 			 * @since 2014.03
 			 * @see client/html/catalog/count/limit
 			 */
-			if( $config->get( 'client/html/catalog/count/tree/aggregate', true ) == true )
+			if ($config->get('client/html/catalog/count/tree/aggregate', true) == true)
 			{
 				/** client/html/catalog/count/limit
 				 * @see client/html/catalog/count/tree/aggregate
 				 */
-				$filter = $this->getProductListFilter( $view, false );
-				$filter->setSlice( 0, $config->get( 'client/html/catalog/count/limit', 10000 ) );
-				$filter->setSortations( array() ); // it's not necessary and slows down the query
+				$filter = $this->getProductListFilter($view, false);
+				$filter->setSlice(0, $config->get('client/html/catalog/count/limit', 10000));
+				$filter->setSortations(array()); // it's not necessary and slows down the query
 
-				$controller = \Aimeos\Controller\Frontend\Factory::createController( $context, 'catalog' );
-				$view->treeCountList = $controller->aggregateIndex( $filter, 'index.catalog.id' );
+				$controller = \Aimeos\Controller\Frontend\Factory::createController($context, 'catalog');
+				$view->treeCountList = $controller->aggregateIndex($filter, 'index.catalog.id');
 			}
 
 			$this->cache = $view;

client/html/src/Client/Html/Catalog/Count/Factory.php

Spacing +9 added lines, -9 removed lines

@@ -31,7 +31,7 @@ 
 	 * @return \Aimeos\Client\Html\Iface Count part implementing \Aimeos\Client\Html\Iface
 	 * @throws \Aimeos\Client\Html\Exception If requested client implementation couldn't be found or initialisation fails
 	 */
-	public static function createClient( \Aimeos\MShop\Context\Item\Iface $context, array $templatePaths, $name = null )
+	public static function createClient(\Aimeos\MShop\Context\Item\Iface $context, array $templatePaths, $name = null)
 	{
 		/** client/html/catalog/count/name
 		 * Class name of the used catalog count client implementation
@@ -66,22 +66,22 @@ 
 		 * @since 2014.03
 		 * @category Developer
 		 */
-		if( $name === null ) {
-			$name = $context->getConfig()->get( 'client/html/catalog/count/name', 'Standard' );
+		if ($name === null) {
+			$name = $context->getConfig()->get('client/html/catalog/count/name', 'Standard');
 		}
 
-		if( ctype_alnum( $name ) === false )
+		if (ctype_alnum($name) === false)
 		{
-			$classname = is_string( $name ) ? '\\Aimeos\\Client\\Html\\Catalog\\Count\\' . $name : '<not a string>';
-			throw new \Aimeos\Client\Html\Exception( sprintf( 'Invalid characters in class name "%1$s"', $classname ) );
+			$classname = is_string($name) ? '\\Aimeos\\Client\\Html\\Catalog\\Count\\'.$name : '<not a string>';
+			throw new \Aimeos\Client\Html\Exception(sprintf('Invalid characters in class name "%1$s"', $classname));
 		}
 
 		$iface = '\\Aimeos\\Client\\Html\\Iface';
-		$classname = '\\Aimeos\\Client\\Html\\Catalog\\Count\\' . $name;
+		$classname = '\\Aimeos\\Client\\Html\\Catalog\\Count\\'.$name;
 
-		$client = self::createClientBase( $context, $classname, $iface, $templatePaths );
+		$client = self::createClientBase($context, $classname, $iface, $templatePaths);
 
-		return self::addClientDecorators( $context, $client, $templatePaths, 'catalog/count' );
+		return self::addClientDecorators($context, $client, $templatePaths, 'catalog/count');
 	}
 
 }

client/html/src/Client/Html/Catalog/Count/Standard.php

Spacing +20 added lines, -20 removed lines

@@ -78,7 +78,7 @@ 
 	 * @since 2014.03
 	 * @category Developer
 	 */
-	private $subPartNames = array( 'tree', 'attribute' );
+	private $subPartNames = array('tree', 'attribute');
 
 
 	/**
@@ -89,15 +89,15 @@ 
 	 * @param string|null &$expire Result variable for the expiration date of the output (null for no expiry)
 	 * @return string HTML code
 	 */
-	public function getBody( $uid = '', array &$tags = array(), &$expire = null )
+	public function getBody($uid = '', array &$tags = array(), &$expire = null)
 	{
 		try
 		{
-			$view = $this->setViewParams( $this->getView(), $tags, $expire );
+			$view = $this->setViewParams($this->getView(), $tags, $expire);
 
 			$html = '';
-			foreach( $this->getSubClients() as $subclient ) {
-				$html .= $subclient->setView( $view )->getBody( $uid, $tags, $expire );
+			foreach ($this->getSubClients() as $subclient) {
+				$html .= $subclient->setView($view)->getBody($uid, $tags, $expire);
 			}
 			$view->countBody = $html;
 
@@ -124,11 +124,11 @@ 
 			$tplconf = 'client/html/catalog/count/standard/template-body';
 			$default = 'catalog/count/body-default.php';
 
-			return $view->render( $view->config( $tplconf, $default ) );
+			return $view->render($view->config($tplconf, $default));
 		}
-		catch( \Exception $e )
+		catch (\Exception $e)
 		{
-			$this->getContext()->getLogger()->log( $e->getMessage() . PHP_EOL . $e->getTraceAsString() );
+			$this->getContext()->getLogger()->log($e->getMessage().PHP_EOL.$e->getTraceAsString());
 		}
 	}
 
@@ -141,15 +141,15 @@ 
 	 * @param string|null &$expire Result variable for the expiration date of the output (null for no expiry)
 	 * @return string|null String including HTML tags for the header on error
 	 */
-	public function getHeader( $uid = '', array &$tags = array(), &$expire = null )
+	public function getHeader($uid = '', array &$tags = array(), &$expire = null)
 	{
 		try
 		{
-			$view = $this->setViewParams( $this->getView(), $tags, $expire );
+			$view = $this->setViewParams($this->getView(), $tags, $expire);
 
 			$html = '';
-			foreach( $this->getSubClients() as $subclient ) {
-				$html .= $subclient->setView( $view )->getHeader( $uid, $tags, $expire );
+			foreach ($this->getSubClients() as $subclient) {
+				$html .= $subclient->setView($view)->getHeader($uid, $tags, $expire);
 			}
 			$view->countHeader = $html;
 
@@ -177,11 +177,11 @@ 
 			$tplconf = 'client/html/catalog/count/standard/template-header';
 			$default = 'catalog/count/header-default.php';
 
-			return $view->render( $view->config( $tplconf, $default ) );
+			return $view->render($view->config($tplconf, $default));
 		}
-		catch( \Exception $e )
+		catch (\Exception $e)
 		{
-			$this->getContext()->getLogger()->log( $e->getMessage() . PHP_EOL . $e->getTraceAsString() );
+			$this->getContext()->getLogger()->log($e->getMessage().PHP_EOL.$e->getTraceAsString());
 		}
 	}
 
@@ -193,7 +193,7 @@ 
 	 * @param string|null $name Name of the sub-client (Default if null)
 	 * @return \Aimeos\Client\Html\Iface Sub-client object
 	 */
-	public function getSubClient( $type, $name = null )
+	public function getSubClient($type, $name = null)
 	{
 		/** client/html/catalog/count/decorators/excludes
 		 * Excludes decorators added by the "common" option from the catalog count html client
@@ -269,7 +269,7 @@ 
 		 * @see client/html/catalog/count/decorators/global
 		 */
 
-		return $this->createSubClient( 'catalog/count/' . $type, $name );
+		return $this->createSubClient('catalog/count/'.$type, $name);
 	}
 
 
@@ -284,9 +284,9 @@ 
 		{
 			parent::process();
 		}
-		catch( \Exception $e )
+		catch (\Exception $e)
 		{
-			$this->getContext()->getLogger()->log( $e->getMessage() . PHP_EOL . $e->getTraceAsString() );
+			$this->getContext()->getLogger()->log($e->getMessage().PHP_EOL.$e->getTraceAsString());
 		}
 	}
 
@@ -298,6 +298,6 @@ 
 	 */
 	protected function getSubClientNames()
 	{
-		return $this->getContext()->getConfig()->get( $this->subPartPath, $this->subPartNames );
+		return $this->getContext()->getConfig()->get($this->subPartPath, $this->subPartNames);
 	}
 }
\ No newline at end of file

client/html/src/Client/Html/Catalog/Suggest/Factory.php

Spacing +9 added lines, -9 removed lines

@@ -31,7 +31,7 @@ 
 	 * @return \Aimeos\Client\Html\Iface Filter part implementing \Aimeos\Client\Html\Iface
 	 * @throws \Aimeos\Client\Html\Exception If requested client implementation couldn't be found or initialisation fails
 	 */
-	public static function createClient( \Aimeos\MShop\Context\Item\Iface $context, array $templatePaths, $name = null )
+	public static function createClient(\Aimeos\MShop\Context\Item\Iface $context, array $templatePaths, $name = null)
 	{
 		/** client/html/catalog/suggest/name
 		 * Class name of the used catalog suggest client implementation
@@ -66,22 +66,22 @@ 
 		 * @since 2015.02
 		 * @category Developer
 		 */
-		if( $name === null ) {
-			$name = $context->getConfig()->get( 'client/html/catalog/suggest/name', 'Standard' );
+		if ($name === null) {
+			$name = $context->getConfig()->get('client/html/catalog/suggest/name', 'Standard');
 		}
 
-		if( ctype_alnum( $name ) === false )
+		if (ctype_alnum($name) === false)
 		{
-			$classname = is_string( $name ) ? '\\Aimeos\\Client\\Html\\Catalog\\Suggest\\' . $name : '<not a string>';
-			throw new \Aimeos\Client\Html\Exception( sprintf( 'Invalid characters in class name "%1$s"', $classname ) );
+			$classname = is_string($name) ? '\\Aimeos\\Client\\Html\\Catalog\\Suggest\\'.$name : '<not a string>';
+			throw new \Aimeos\Client\Html\Exception(sprintf('Invalid characters in class name "%1$s"', $classname));
 		}
 
 		$iface = '\\Aimeos\\Client\\Html\\Iface';
-		$classname = '\\Aimeos\\Client\\Html\\Catalog\\Suggest\\' . $name;
+		$classname = '\\Aimeos\\Client\\Html\\Catalog\\Suggest\\'.$name;
 
-		$client = self::createClientBase( $context, $classname, $iface, $templatePaths );
+		$client = self::createClientBase($context, $classname, $iface, $templatePaths);
 
-		return self::addClientDecorators( $context, $client, $templatePaths, 'catalog/suggest' );
+		return self::addClientDecorators($context, $client, $templatePaths, 'catalog/suggest');
 	}
 }
 

client/html/src/Client/Html/Catalog/Suggest/Standard.php

Spacing +27 added lines, -27 removed lines

@@ -72,21 +72,21 @@ 
 	 * @param string|null &$expire Result variable for the expiration date of the output (null for no expiry)
 	 * @return string HTML code
 	 */
-	public function getBody( $uid = '', array &$tags = array(), &$expire = null )
+	public function getBody($uid = '', array &$tags = array(), &$expire = null)
 	{
 		try
 		{
-			$view = $this->setViewParams( $this->getView(), $tags, $expire );
+			$view = $this->setViewParams($this->getView(), $tags, $expire);
 
 			$html = '';
-			foreach( $this->getSubClients() as $subclient ) {
-				$html .= $subclient->setView( $view )->getBody( $uid, $tags, $expire );
+			foreach ($this->getSubClients() as $subclient) {
+				$html .= $subclient->setView($view)->getBody($uid, $tags, $expire);
 			}
 			$view->suggestBody = $html;
 		}
-		catch( \Exception $e )
+		catch (\Exception $e)
 		{
-			$this->getContext()->getLogger()->log( $e->getMessage() . PHP_EOL . $e->getTraceAsString() );
+			$this->getContext()->getLogger()->log($e->getMessage().PHP_EOL.$e->getTraceAsString());
 			return;
 		}
 
@@ -116,7 +116,7 @@ 
 		$tplconf = 'client/html/catalog/suggest/standard/template-body';
 		$default = 'catalog/suggest/body-default.php';
 
-		return $view->render( $view->config( $tplconf, $default ) );
+		return $view->render($view->config($tplconf, $default));
 	}
 
 
@@ -128,21 +128,21 @@ 
 	 * @param string|null &$expire Result variable for the expiration date of the output (null for no expiry)
 	 * @return string|null String including HTML tags for the header on error
 	 */
-	public function getHeader( $uid = '', array &$tags = array(), &$expire = null )
+	public function getHeader($uid = '', array &$tags = array(), &$expire = null)
 	{
 		try
 		{
-			$view = $this->setViewParams( $this->getView(), $tags, $expire );
+			$view = $this->setViewParams($this->getView(), $tags, $expire);
 
 			$html = '';
-			foreach( $this->getSubClients() as $subclient ) {
-				$html .= $subclient->setView( $view )->getHeader( $uid, $tags, $expire );
+			foreach ($this->getSubClients() as $subclient) {
+				$html .= $subclient->setView($view)->getHeader($uid, $tags, $expire);
 			}
 			$view->suggestHeader = $html;
 		}
-		catch( \Exception $e )
+		catch (\Exception $e)
 		{
-			$this->getContext()->getLogger()->log( $e->getMessage() . PHP_EOL . $e->getTraceAsString() );
+			$this->getContext()->getLogger()->log($e->getMessage().PHP_EOL.$e->getTraceAsString());
 			return;
 		}
 
@@ -173,7 +173,7 @@ 
 		$tplconf = 'client/html/catalog/suggest/standard/template-header';
 		$default = 'catalog/suggest/header-default.php';
 
-		return $view->render( $view->config( $tplconf, $default ) );
+		return $view->render($view->config($tplconf, $default));
 	}
 
 
@@ -184,7 +184,7 @@ 
 	 * @param string|null $name Name of the sub-client (Default if null)
 	 * @return \Aimeos\Client\Html\Iface Sub-client object
 	 */
-	public function getSubClient( $type, $name = null )
+	public function getSubClient($type, $name = null)
 	{
 		/** client/html/catalog/suggest/decorators/excludes
 		 * Excludes decorators added by the "common" option from the catalog suggest html client
@@ -260,7 +260,7 @@ 
 		 * @see client/html/catalog/suggest/decorators/global
 		 */
 
-		return $this->createSubClient( 'catalog/suggest/' . $type, $name );
+		return $this->createSubClient('catalog/suggest/'.$type, $name);
 	}
 
 
@@ -275,9 +275,9 @@ 
 		{
 			parent::process();
 		}
-		catch( \Exception $e )
+		catch (\Exception $e)
 		{
-			$this->getContext()->getLogger()->log( $e->getMessage() . PHP_EOL . $e->getTraceAsString() );
+			$this->getContext()->getLogger()->log($e->getMessage().PHP_EOL.$e->getTraceAsString());
 		}
 	}
 
@@ -289,7 +289,7 @@ 
 	 */
 	protected function getSubClientNames()
 	{
-		return $this->getContext()->getConfig()->get( $this->subPartPath, $this->subPartNames );
+		return $this->getContext()->getConfig()->get($this->subPartPath, $this->subPartNames);
 	}
 
 
@@ -301,20 +301,20 @@ 
 	 * @param string|null &$expire Result variable for the expiration date of the output (null for no expiry)
 	 * @return \Aimeos\MW\View\Iface Modified view object
 	 */
-	protected function setViewParams( \Aimeos\MW\View\Iface $view, array &$tags = array(), &$expire = null )
+	protected function setViewParams(\Aimeos\MW\View\Iface $view, array &$tags = array(), &$expire = null)
 	{
-		if( !isset( $this->cache ) )
+		if (!isset($this->cache))
 		{
-			$input = $view->param( 'f_search' );
+			$input = $view->param('f_search');
 
-			$controller = \Aimeos\Controller\Frontend\Factory::createController( $this->getContext(), 'catalog' );
+			$controller = \Aimeos\Controller\Frontend\Factory::createController($this->getContext(), 'catalog');
 
-			$filter = $controller->createTextFilter( $input );
-			$items = $controller->getTextList( $filter );
+			$filter = $controller->createTextFilter($input);
+			$items = $controller->getTextList($filter);
 
 			$suggestTextItems = array();
-			foreach( $items as $id => $name ) {
-				$suggestTextItems[] = array( "id" => $id, "name" => $name );
+			foreach ($items as $id => $name) {
+				$suggestTextItems[] = array("id" => $id, "name" => $name);
 			}
 
 			$view->suggestTextItems = $suggestTextItems;

client/html/src/Client/Html/Catalog/Filter/Attribute/Standard.php

Spacing +37 added lines, -37 removed lines

@@ -70,13 +70,13 @@ 
 	 * @param string|null &$expire Result variable for the expiration date of the output (null for no expiry)
 	 * @return string HTML code
 	 */
-	public function getBody( $uid = '', array &$tags = array(), &$expire = null )
+	public function getBody($uid = '', array &$tags = array(), &$expire = null)
 	{
-		$view = $this->setViewParams( $this->getView(), $tags, $expire );
+		$view = $this->setViewParams($this->getView(), $tags, $expire);
 
 		$html = '';
-		foreach( $this->getSubClients() as $subclient ) {
-			$html .= $subclient->setView( $view )->getBody( $uid, $tags, $expire );
+		foreach ($this->getSubClients() as $subclient) {
+			$html .= $subclient->setView($view)->getBody($uid, $tags, $expire);
 		}
 		$view->attributeBody = $html;
 
@@ -103,7 +103,7 @@ 
 		$tplconf = 'client/html/catalog/filter/attribute/standard/template-body';
 		$default = 'catalog/filter/attribute-body-default.php';
 
-		return $view->render( $view->config( $tplconf, $default ) );
+		return $view->render($view->config($tplconf, $default));
 	}
 
 
@@ -115,13 +115,13 @@ 
 	 * @param string|null &$expire Result variable for the expiration date of the output (null for no expiry)
 	 * @return string|null String including HTML tags for the header on error
 	 */
-	public function getHeader( $uid = '', array &$tags = array(), &$expire = null )
+	public function getHeader($uid = '', array &$tags = array(), &$expire = null)
 	{
-		$view = $this->setViewParams( $this->getView(), $tags, $expire );
+		$view = $this->setViewParams($this->getView(), $tags, $expire);
 
 		$html = '';
-		foreach( $this->getSubClients() as $subclient ) {
-			$html .= $subclient->setView( $view )->getHeader( $uid, $tags, $expire );
+		foreach ($this->getSubClients() as $subclient) {
+			$html .= $subclient->setView($view)->getHeader($uid, $tags, $expire);
 		}
 		$view->attributeHeader = $html;
 
@@ -149,7 +149,7 @@ 
 		$tplconf = 'client/html/catalog/filter/attribute/standard/template-header';
 		$default = 'catalog/filter/attribute-header-default.php';
 
-		return $view->render( $view->config( $tplconf, $default ) );
+		return $view->render($view->config($tplconf, $default));
 	}
 
 
@@ -160,7 +160,7 @@ 
 	 * @param string|null $name Name of the sub-client (Default if null)
 	 * @return \Aimeos\Client\Html\Iface Sub-client object
 	 */
-	public function getSubClient( $type, $name = null )
+	public function getSubClient($type, $name = null)
 	{
 		/** client/html/catalog/filter/attribute/decorators/excludes
 		 * Excludes decorators added by the "common" option from the catalog filter attribute html client
@@ -236,7 +236,7 @@ 
 		 * @see client/html/catalog/filter/attribute/decorators/global
 		 */
 
-		return $this->createSubClient( 'catalog/filter/attribute/' . $type, $name );
+		return $this->createSubClient('catalog/filter/attribute/'.$type, $name);
 	}
 
 
@@ -247,7 +247,7 @@ 
 	 */
 	protected function getSubClientNames()
 	{
-		return $this->getContext()->getConfig()->get( $this->subPartPath, $this->subPartNames );
+		return $this->getContext()->getConfig()->get($this->subPartPath, $this->subPartNames);
 	}
 
 
@@ -259,12 +259,12 @@ 
 	 * @param string|null &$expire Result variable for the expiration date of the output (null for no expiry)
 	 * @return \Aimeos\MW\View\Iface Modified view object
 	 */
-	protected function setViewParams( \Aimeos\MW\View\Iface $view, array &$tags = array(), &$expire = null )
+	protected function setViewParams(\Aimeos\MW\View\Iface $view, array &$tags = array(), &$expire = null)
 	{
-		if( !isset( $this->cache ) )
+		if (!isset($this->cache))
 		{
 			$attrMap = array();
-			$controller = \Aimeos\Controller\Frontend\Factory::createController( $this->getContext(), 'catalog' );
+			$controller = \Aimeos\Controller\Frontend\Factory::createController($this->getContext(), 'catalog');
 
 			/** client/html/catalog/filter/attribute/types
 			 * List of attribute types that should be displayed in this order in the catalog filter
@@ -286,24 +286,24 @@ 
 			 * @category Developer
 			 * @see client/html/catalog/filter/attribute/domains
 			 */
-			$attrTypes = $view->config( 'client/html/catalog/filter/attribute/types', array() );
+			$attrTypes = $view->config('client/html/catalog/filter/attribute/types', array());
 
-			$manager = $controller->createManager( 'attribute' );
-			$search = $manager->createSearch( true );
+			$manager = $controller->createManager('attribute');
+			$search = $manager->createSearch(true);
 
 			$expr = array();
-			if( !empty( $attrTypes ) ) {
-				$expr[] = $search->compare( '==', 'attribute.type.code', $attrTypes );
+			if (!empty($attrTypes)) {
+				$expr[] = $search->compare('==', 'attribute.type.code', $attrTypes);
 			}
 
-			$expr[] = $search->compare( '==', 'attribute.domain', 'product' );
+			$expr[] = $search->compare('==', 'attribute.domain', 'product');
 			$expr[] = $search->getConditions();
 
-			$sort = array( $search->sort( '+', 'attribute.position' ) );
+			$sort = array($search->sort('+', 'attribute.position'));
 
-			$search->setConditions( $search->combine( '&&', $expr ) );
-			$search->setSortations( $sort );
-			$search->setSlice( 0, 0x7fffffff );
+			$search->setConditions($search->combine('&&', $expr));
+			$search->setSortations($sort);
+			$search->setSlice(0, 0x7fffffff);
 
 			/** client/html/catalog/filter/attribute/domains
 			 * List of domain names whose items should be fetched with the filter attributes
@@ -321,21 +321,21 @@ 
 			 * @category Developer
 			 * @see client/html/catalog/filter/attribute/types
 			 */
-			$domains = $view->config( 'client/html/catalog/filter/attribute/domains', array( 'text', 'media' ) );
+			$domains = $view->config('client/html/catalog/filter/attribute/domains', array('text', 'media'));
 
-			$attributes = $manager->searchItems( $search, $domains );
+			$attributes = $manager->searchItems($search, $domains);
 
-			foreach( $attributes as $id => $item ) {
+			foreach ($attributes as $id => $item) {
 				$attrMap[$item->getType()][$id] = $item;
 			}
 
-			if( !empty( $attrTypes ) )
+			if (!empty($attrTypes))
 			{
 				$sortedMap = array();
 
-				foreach( $attrTypes as $type )
+				foreach ($attrTypes as $type)
 				{
-					if( isset( $attrMap[$type] ) ) {
+					if (isset($attrMap[$type])) {
 						$sortedMap[$type] = $attrMap[$type];
 					}
 				}
@@ -344,11 +344,11 @@ 
 			}
 			else
 			{
-				ksort( $attrMap );
+				ksort($attrMap);
 			}
 
-			$this->addMetaItem( $attributes, 'attribute', $this->expire, $this->tags );
-			$this->addMetaList( array_keys( $attributes ), 'attribute', $this->expire );
+			$this->addMetaItem($attributes, 'attribute', $this->expire, $this->tags);
+			$this->addMetaList(array_keys($attributes), 'attribute', $this->expire);
 
 			// Delete cache when attributes are added or deleted even in "tag-all" mode
 			$this->tags[] = 'attribute';
@@ -359,8 +359,8 @@ 
 			$this->cache = $view;
 		}
 
-		$expire = $this->expires( $this->expire, $expire );
-		$tags = array_merge( $tags, $this->tags );
+		$expire = $this->expires($this->expire, $expire);
+		$tags = array_merge($tags, $this->tags);
 
 		return $this->cache;
 	}