aimeos / ai-client-html

client/html/src/Client/Html/Locale/Select/Language/Standard.php

Spacing +16 added lines, -16 removed lines

@@ -67,13 +67,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->languageBody = $html;
 
@@ -100,7 +100,7 @@ 
 		$tplconf = 'client/html/locale/select/language/standard/template-body';
 		$default = 'locale/select/language-body-default.php';
 
-		return $view->render( $view->config( $tplconf, $default ) );
+		return $view->render($view->config($tplconf, $default));
 	}
 
 
@@ -112,13 +112,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->languageHeader = $html;
 
@@ -146,7 +146,7 @@ 
 		$tplconf = 'client/html/locale/select/language/standard/template-header';
 		$default = 'locale/select/language-header-default.php';
 
-		return $view->render( $view->config( $tplconf, $default ) );
+		return $view->render($view->config($tplconf, $default));
 	}
 
 
@@ -157,7 +157,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/locale/select/language/decorators/excludes
 		 * Excludes decorators added by the "common" option from the locale select language html client
@@ -233,7 +233,7 @@ 
 		 * @see client/html/locale/select/language/decorators/global
 		 */
 
-		return $this->createSubClient( 'locale/select/language/' . $type, $name );
+		return $this->createSubClient('locale/select/language/'.$type, $name);
 	}
 
 
@@ -249,10 +249,10 @@ 
 		$config = $context->getConfig();
 		$session = $context->getSession();
 
-		$name = $config->get( 'client/html/locale/select/language/param-name', 'loc_languageid' );
+		$name = $config->get('client/html/locale/select/language/param-name', 'loc_languageid');
 
-		if( ( $languageId = $view->param( $name ) ) !== null ) {
-			$session->set( 'aimeos/locale/languageid', $languageId );
+		if (($languageId = $view->param($name)) !== null) {
+			$session->set('aimeos/locale/languageid', $languageId);
 		}
 
 		parent::process();
@@ -266,6 +266,6 @@ 
 	 */
 	protected function getSubClientNames()
 	{
-		return $this->getContext()->getConfig()->get( $this->subPartPath, $this->subPartNames );
+		return $this->getContext()->getConfig()->get($this->subPartPath, $this->subPartNames);
 	}
 }

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

Spacing +15 added lines, -15 removed lines

@@ -29,35 +29,35 @@
 	 * @return \Aimeos\Client\Html\Iface HTML client 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, $type, $name = null )
+	public static function createClient(\Aimeos\MShop\Context\Item\Iface $context, array $templatePaths, $type, $name = null)
 	{
-		if( empty( $type ) ) {
-			throw new \Aimeos\Client\Html\Exception( sprintf( 'Client HTML type is empty' ) );
+		if (empty($type)) {
+			throw new \Aimeos\Client\Html\Exception(sprintf('Client HTML type is empty'));
 		}
 
-		$parts = explode( '/', $type );
+		$parts = explode('/', $type);
 
-		if( count( $parts ) !== 2 ) {
-			throw new \Aimeos\Client\Html\Exception( sprintf( 'Client type "%1$s" must consist of two parts separated by "/"', $type ) );
+		if (count($parts) !== 2) {
+			throw new \Aimeos\Client\Html\Exception(sprintf('Client type "%1$s" must consist of two parts separated by "/"', $type));
 		}
 
-		foreach( $parts as $part )
+		foreach ($parts as $part)
 		{
-			if( ctype_alnum( $part ) === false ) {
-				throw new \Aimeos\Client\Html\Exception( sprintf( 'Invalid characters in client name "%1$s" in "%2$s"', $part, $type ) );
+			if (ctype_alnum($part) === false) {
+				throw new \Aimeos\Client\Html\Exception(sprintf('Invalid characters in client name "%1$s" in "%2$s"', $part, $type));
 			}
 		}
 
-		$factory = '\\Aimeos\\Client\\Html\\' . ucwords( $parts[0] ) . '\\' . ucwords( $parts[1] ) . '\\Factory';
+		$factory = '\\Aimeos\\Client\\Html\\'.ucwords($parts[0]).'\\'.ucwords($parts[1]).'\\Factory';
 
-		if( class_exists( $factory ) === false ) {
-			throw new \Aimeos\Client\Html\Exception( sprintf( 'Class "%1$s" not available', $factory ) );
+		if (class_exists($factory) === false) {
+			throw new \Aimeos\Client\Html\Exception(sprintf('Class "%1$s" not available', $factory));
 		}
 
-		$client = @call_user_func_array( array( $factory, 'createClient' ), array( $context, $templatePaths, $name ) );
+		$client = @call_user_func_array(array($factory, 'createClient'), array($context, $templatePaths, $name));
 
-		if( $client === false ) {
-			throw new \Aimeos\Client\Html\Exception( sprintf( 'Invalid factory "%1$s"', $factory ) );
+		if ($client === false) {
+			throw new \Aimeos\Client\Html\Exception(sprintf('Invalid factory "%1$s"', $factory));
 		}
 
 		return $client;

client/html/src/Client/Html/Basket/Base.php

Spacing +9 added lines, -9 removed lines

@@ -28,8 +28,8 @@ 
 	{
 		$session = $this->getContext()->getSession();
 
-		foreach( $session->get( 'aimeos/basket/cache', array() ) as $key => $value ) {
-			$session->set( $key, null );
+		foreach ($session->get('aimeos/basket/cache', array()) as $key => $value) {
+			$session->set($key, null);
 		}
 	}
 
@@ -42,9 +42,9 @@ 
 	 * @return mixed Value associated to the requested key. If no value for the
 	 *	key is found in the cache, the given default value is returned
 	 */
-	protected function getBasketCached( $key, $default = null )
+	protected function getBasketCached($key, $default = null)
 	{
-		return $this->getContext()->getSession()->get( $key, $default );
+		return $this->getContext()->getSession()->get($key, $default);
 	}
 
 
@@ -54,7 +54,7 @@ 
 	 * @param string $key Path the cache entry should be stored in
 	 * @param mixed $value Value stored in the cache for the path
 	 */
-	protected function setBasketCached( $key, $value )
+	protected function setBasketCached($key, $value)
 	{
 		$context = $this->getContext();
 
@@ -72,13 +72,13 @@ 
 		 * @category Developer
 		 * @since 2014.11
 		 */
-		if( $context->getConfig()->get( 'client/html/basket/cache/enable', true ) != false )
+		if ($context->getConfig()->get('client/html/basket/cache/enable', true) != false)
 		{
 			$session = $context->getSession();
 
-			$cached = $session->get( 'aimeos/basket/cache', array() ) + array( $key => true );
-			$session->set( 'aimeos/basket/cache', $cached );
-			$session->set( $key, $value );
+			$cached = $session->get('aimeos/basket/cache', array()) + array($key => true);
+			$session->set('aimeos/basket/cache', $cached);
+			$session->set($key, $value);
 		}
 	}
 }

client/html/src/Client/Html/Basket/Standard/Coupon/Standard.php

Spacing +23 added lines, -23 removed lines

@@ -67,13 +67,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->couponBody = $html;
 
@@ -100,7 +100,7 @@ 
 		$tplconf = 'client/html/basket/standard/coupon/standard/template-body';
 		$default = 'basket/standard/coupon-body-default.php';
 
-		return $view->render( $view->config( $tplconf, $default ) );
+		return $view->render($view->config($tplconf, $default));
 	}
 
 
@@ -112,13 +112,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->couponHeader = $html;
 
@@ -146,7 +146,7 @@ 
 		$tplconf = 'client/html/basket/standard/coupon/standard/template-header';
 		$default = 'basket/standard/coupon-header-default.php';
 
-		return $view->render( $view->config( $tplconf, $default ) );
+		return $view->render($view->config($tplconf, $default));
 	}
 
 
@@ -157,7 +157,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/basket/standard/coupon/decorators/excludes
 		 * Excludes decorators added by the "common" option from the basket standard coupon html client
@@ -232,7 +232,7 @@ 
 		 * @see client/html/basket/standard/coupon/decorators/excludes
 		 * @see client/html/basket/standard/coupon/decorators/global
 		 */
-		return $this->createSubClient( 'basket/standard/coupon/' . $type, $name );
+		return $this->createSubClient('basket/standard/coupon/'.$type, $name);
 	}
 
 
@@ -246,25 +246,25 @@ 
 		$view = $this->getView();
 		$context = $this->getContext();
 
-		switch( $view->param( 'b_action' ) )
+		switch ($view->param('b_action'))
 		{
 			case 'coupon-delete':
 
-				if( ( $coupon = $view->param( 'b_coupon' ) ) != '' )
+				if (($coupon = $view->param('b_coupon')) != '')
 				{
 					$this->clearCached();
-					$cntl = \Aimeos\Controller\Frontend\Factory::createController( $context, 'basket' );
-					$cntl->deleteCoupon( $coupon );
+					$cntl = \Aimeos\Controller\Frontend\Factory::createController($context, 'basket');
+					$cntl->deleteCoupon($coupon);
 				}
 
 				break;
 
 			default:
 
-				if( ( $coupon = $view->param( 'b_coupon' ) ) != '' )
+				if (($coupon = $view->param('b_coupon')) != '')
 				{
 					$this->clearCached();
-					$cntl = \Aimeos\Controller\Frontend\Factory::createController( $context, 'basket' );
+					$cntl = \Aimeos\Controller\Frontend\Factory::createController($context, 'basket');
 
 					/** client/html/basket/standard/coupon/allowed
 					 * Number of coupon codes a customer is allowed to enter
@@ -282,13 +282,13 @@ 
 					 * @category User
 					 * @category Developer
 					 */
-					$allowed = $context->getConfig()->get( 'client/html/basket/standard/coupon/allowed', 1 );
+					$allowed = $context->getConfig()->get('client/html/basket/standard/coupon/allowed', 1);
 
-					if( $allowed <= count( $cntl->get()->getCoupons() ) ) {
-						throw new \Aimeos\Client\Html\Exception( sprintf( 'Number of coupon codes exceeds the limit' ) );
+					if ($allowed <= count($cntl->get()->getCoupons())) {
+						throw new \Aimeos\Client\Html\Exception(sprintf('Number of coupon codes exceeds the limit'));
 					}
 
-					$cntl->addCoupon( $coupon );
+					$cntl->addCoupon($coupon);
 				}
 
 				break;
@@ -305,6 +305,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/Basket/Standard/Detail/Standard.php

Spacing +24 added lines, -24 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->detailBody = $html;
 
@@ -102,7 +102,7 @@ 
 		$tplconf = 'client/html/basket/standard/detail/standard/template-body';
 		$default = 'common/summary/detail-body-default.php';
 
-		return $view->render( $view->config( $tplconf, $default ) );
+		return $view->render($view->config($tplconf, $default));
 	}
 
 
@@ -114,13 +114,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->detailHeader = $html;
 
@@ -149,7 +149,7 @@ 
 		$tplconf = 'client/html/basket/standard/detail/standard/template-header';
 		$default = 'common/summary/detail-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/basket/standard/detail/decorators/excludes
 		 * Excludes decorators added by the "common" option from the basket standard detail html client
@@ -235,7 +235,7 @@ 
 		 * @see client/html/basket/standard/detail/decorators/excludes
 		 * @see client/html/basket/standard/detail/decorators/global
 		 */
-		return $this->createSubClient( 'basket/standard/detail/' . $type, $name );
+		return $this->createSubClient('basket/standard/detail/'.$type, $name);
 	}
 
 
@@ -246,7 +246,7 @@ 
 	 */
 	protected function getSubClientNames()
 	{
-		return $this->getContext()->getConfig()->get( $this->subPartPath, $this->subPartNames );
+		return $this->getContext()->getConfig()->get($this->subPartPath, $this->subPartNames);
 	}
 
 
@@ -258,23 +258,23 @@ 
 	 * @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)
 	{
-		$view = parent::setViewParams( $view );
+		$view = parent::setViewParams($view);
 
-		if( !isset( $this->cache ) )
+		if (!isset($this->cache))
 		{
 			$basket = $view->standardBasket;
 
-			$target = $view->config( 'client/html/checkout/standard/url/target' );
-			$cntl = $view->config( 'client/html/checkout/standard/url/controller', 'checkout' );
-			$action = $view->config( 'client/html/checkout/standard/url/action', 'index' );
-			$config = $view->config( 'client/html/checkout/standard/url/config', array() );
+			$target = $view->config('client/html/checkout/standard/url/target');
+			$cntl = $view->config('client/html/checkout/standard/url/controller', 'checkout');
+			$action = $view->config('client/html/checkout/standard/url/action', 'index');
+			$config = $view->config('client/html/checkout/standard/url/config', array());
 
-			$view->summaryParams = $view->get( 'standardParams', array() );
-			$view->summaryUrlServicePayment = $view->url( $target, $cntl, $action, array( 'c_step' => 'payment' ), array(), $config );
-			$view->summaryUrlServiceDelivery = $view->url( $target, $cntl, $action, array( 'c_step' => 'delivery' ), array(), $config );
-			$view->summaryTaxRates = $this->getTaxRates( $basket );
+			$view->summaryParams = $view->get('standardParams', array());
+			$view->summaryUrlServicePayment = $view->url($target, $cntl, $action, array('c_step' => 'payment'), array(), $config);
+			$view->summaryUrlServiceDelivery = $view->url($target, $cntl, $action, array('c_step' => 'delivery'), array(), $config);
+			$view->summaryTaxRates = $this->getTaxRates($basket);
 			$view->summaryEnableModify = true;
 			$view->summaryBasket = $basket;
 

client/html/src/Client/Html/Basket/Standard/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/basket/standard/name
 		 * Class name of the used basket standard client implementation
@@ -66,22 +66,22 @@ 
 		 * @since 2014.03
 		 * @category Developer
 		 */
-		if( $name === null ) {
-			$name = $context->getConfig()->get( 'client/html/basket/standard/name', 'Standard' );
+		if ($name === null) {
+			$name = $context->getConfig()->get('client/html/basket/standard/name', 'Standard');
 		}
 
-		if( ctype_alnum( $name ) === false )
+		if (ctype_alnum($name) === false)
 		{
-			$classname = is_string( $name ) ? '\\Aimeos\\Client\\Html\\Basket\\Standard\\' . $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\\Basket\\Standard\\'.$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\\Basket\\Standard\\' . $name;
+		$classname = '\\Aimeos\\Client\\Html\\Basket\\Standard\\'.$name;
 
-		$client = self::createClientBase( $context, $classname, $iface, $templatePaths );
+		$client = self::createClientBase($context, $classname, $iface, $templatePaths);
 
-		return self::addClientDecorators( $context, $client, $templatePaths, 'basket/standard' );
+		return self::addClientDecorators($context, $client, $templatePaths, 'basket/standard');
 	}
 }
 

client/html/src/Client/Html/Basket/Related/Bought/Standard.php

Spacing +45 added lines, -45 removed lines

@@ -67,13 +67,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->boughtBody = $html;
 
@@ -100,7 +100,7 @@ 
 		$tplconf = 'client/html/basket/related/bought/standard/template-body';
 		$default = 'basket/related/bought-body-default.php';
 
-		return $view->render( $view->config( $tplconf, $default ) );
+		return $view->render($view->config($tplconf, $default));
 	}
 
 
@@ -112,13 +112,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->boughtHeader = $html;
 
@@ -146,7 +146,7 @@ 
 		$tplconf = 'client/html/basket/related/bought/standard/template-header';
 		$default = 'basket/related/bought-header-default.php';
 
-		return $view->render( $view->config( $tplconf, $default ) );
+		return $view->render($view->config($tplconf, $default));
 	}
 
 
@@ -157,7 +157,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/basket/related/bought/decorators/excludes
 		 * Excludes decorators added by the "common" option from the basket related bought html client
@@ -233,7 +233,7 @@ 
 		 * @see client/html/basket/related/bought/decorators/global
 		 */
 
-		return $this->createSubClient( 'basket/related/bought/' . $type, $name );
+		return $this->createSubClient('basket/related/bought/'.$type, $name);
 	}
 
 
@@ -244,7 +244,7 @@ 
 	 */
 	protected function getSubClientNames()
 	{
-		return $this->getContext()->getConfig()->get( $this->subPartPath, $this->subPartNames );
+		return $this->getContext()->getConfig()->get($this->subPartPath, $this->subPartNames);
 	}
 
 
@@ -256,31 +256,31 @@ 
 	 * @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))
 		{
-			if( isset( $view->relatedBasket ) )
+			if (isset($view->relatedBasket))
 			{
 				$refIds = $items = array();
 				$context = $this->getContext();
 
-				$prodIds = $this->getProductIdsFromBasket( $view->relatedBasket );
+				$prodIds = $this->getProductIdsFromBasket($view->relatedBasket);
 
-				foreach( $this->getListItems( $prodIds ) as $listItem )
+				foreach ($this->getListItems($prodIds) as $listItem)
 				{
 					$refId = $listItem->getRefId();
 
-					if( !isset( $prodIds[$refId] ) ) {
+					if (!isset($prodIds[$refId])) {
 						$refIds[$refId] = $refId;
 					}
 				}
 
-				$products = $this->getProductItems( $refIds );
+				$products = $this->getProductItems($refIds);
 
-				foreach( $refIds as $id )
+				foreach ($refIds as $id)
 				{
-					if( isset( $products[$id] ) ) {
+					if (isset($products[$id])) {
 						$items[$id] = $products[$id];
 					}
 				}
@@ -300,10 +300,10 @@ 
 				 * @param integer Number of products
 				 * @since 2014.09
 				 */
-				$size = $context->getConfig()->get( 'client/html/basket/related/bought/standard/limit', 6 );
+				$size = $context->getConfig()->get('client/html/basket/related/bought/standard/limit', 6);
 
 
-				$view->boughtItems = array_slice( $items, 0, $size, true );
+				$view->boughtItems = array_slice($items, 0, $size, true);
 			}
 
 			$this->cache = $view;
@@ -319,22 +319,22 @@ 
 	 * @param string[] $prodIds List of product IDs
 	 * @return \Aimeos\MShop\Product\Item\Lists\Iface[] List of product list items
 	 */
-	protected function getListItems( array $prodIds )
+	protected function getListItems(array $prodIds)
 	{
-		$typeItem = $this->getTypeItem( 'product/lists/type', 'product', 'bought-together' );
-		$manager = \Aimeos\MShop\Factory::createManager( $this->getContext(), 'product/lists' );
+		$typeItem = $this->getTypeItem('product/lists/type', 'product', 'bought-together');
+		$manager = \Aimeos\MShop\Factory::createManager($this->getContext(), 'product/lists');
 
-		$search = $manager->createSearch( true );
+		$search = $manager->createSearch(true);
 		$expr = array(
-				$search->compare( '==', 'product.lists.parentid', $prodIds ),
-				$search->compare( '==', 'product.lists.typeid', $typeItem->getId() ),
-				$search->compare( '==', 'product.lists.domain', 'product' ),
+				$search->compare('==', 'product.lists.parentid', $prodIds),
+				$search->compare('==', 'product.lists.typeid', $typeItem->getId()),
+				$search->compare('==', 'product.lists.domain', 'product'),
 				$search->getConditions(),
 		);
-		$search->setConditions( $search->combine( '&&', $expr ) );
-		$search->setSortations( array( $search->sort( '+', 'product.lists.position' ) ) );
+		$search->setConditions($search->combine('&&', $expr));
+		$search->setSortations(array($search->sort('+', 'product.lists.position')));
 
-		return $manager->searchItems( $search );
+		return $manager->searchItems($search);
 	}
 
 
@@ -344,20 +344,20 @@ 
 	 * @param \Aimeos\MShop\Order\Item\Base\Iface $basket Basket object
 	 * @return string[] List of product IDs
 	 */
-	protected function getProductIdsFromBasket( \Aimeos\MShop\Order\Item\Base\Iface $basket )
+	protected function getProductIdsFromBasket(\Aimeos\MShop\Order\Item\Base\Iface $basket)
 	{
 		$list = array();
 
-		foreach( $basket->getProducts() as $orderProduct )
+		foreach ($basket->getProducts() as $orderProduct)
 		{
-			$list[ $orderProduct->getProductId() ] = true;
+			$list[$orderProduct->getProductId()] = true;
 
-			foreach( $orderProduct->getProducts() as $subProduct ) {
-				$list[ $subProduct->getProductId() ] = true;
+			foreach ($orderProduct->getProducts() as $subProduct) {
+				$list[$subProduct->getProductId()] = true;
 			}
 		}
 
-		return array_keys( $list );
+		return array_keys($list);
 	}
 
 
@@ -367,7 +367,7 @@ 
 	 * @param string[] $ids List of product IDs
 	 * @return \Aimeos\MShop\Product\Item\Iface[] List of product items
 	 */
-	protected function getProductItems( array $ids )
+	protected function getProductItems(array $ids)
 	{
 		$context = $this->getContext();
 		$config = $context->getConfig();
@@ -387,11 +387,11 @@ 
 		 * @since 2014.09
 		 * @category Developer
 		 */
-		$domains = array( 'text', 'price', 'media' );
-		$domains = $config->get( 'client/html/basket/related/bought/standard/domains', $domains );
+		$domains = array('text', 'price', 'media');
+		$domains = $config->get('client/html/basket/related/bought/standard/domains', $domains);
 
-		$controller = \Aimeos\Controller\Frontend\Factory::createController( $context, 'catalog' );
+		$controller = \Aimeos\Controller\Frontend\Factory::createController($context, 'catalog');
 
-		return $controller->getProductItems( $ids, $domains );
+		return $controller->getProductItems($ids, $domains);
 	}
 }
\ No newline at end of file

client/html/src/Client/Html/Basket/Related/Factory.php

Spacing +9 added lines, -9 removed lines

@@ -30,7 +30,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/basket/related/name
 		 * Class name of the used basket related client implementation
@@ -65,22 +65,22 @@ 
 		 * @since 2014.03
 		 * @category Developer
 		 */
-		if( $name === null ) {
-			$name = $context->getConfig()->get( 'client/html/basket/related/name', 'Standard' );
+		if ($name === null) {
+			$name = $context->getConfig()->get('client/html/basket/related/name', 'Standard');
 		}
 
-		if( ctype_alnum( $name ) === false )
+		if (ctype_alnum($name) === false)
 		{
-			$classname = is_string( $name ) ? '\\Aimeos\\Client\\Html\\Basket\\Related\\' . $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\\Basket\\Related\\'.$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\\Basket\\Related\\' . $name;
+		$classname = '\\Aimeos\\Client\\Html\\Basket\\Related\\'.$name;
 
-		$client = self::createClientBase( $context, $classname, $iface, $templatePaths );
+		$client = self::createClientBase($context, $classname, $iface, $templatePaths);
 
-		return self::addClientDecorators( $context, $client, $templatePaths, 'basket/related' );
+		return self::addClientDecorators($context, $client, $templatePaths, 'basket/related');
 	}
 }
 

client/html/src/Client/Html/Basket/Related/Standard.php

Spacing +32 added lines, -32 removed lines

@@ -66,7 +66,7 @@ 
 	 * @since 2014.09
 	 * @category Developer
 	 */
-	private $subPartNames = array( 'bought' );
+	private $subPartNames = array('bought');
 	private $cache;
 
 
@@ -78,42 +78,42 @@ 
 	 * @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)
 	{
 		$context = $this->getContext();
 		$view = $this->getView();
 
 		try
 		{
-			$view = $this->setViewParams( $view, $tags, $expire );
+			$view = $this->setViewParams($view, $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->relatedBody = $html;
 		}
-		catch( \Aimeos\Client\Html\Exception $e )
+		catch (\Aimeos\Client\Html\Exception $e)
 		{
-			$error = array( $this->getContext()->getI18n()->dt( 'client', $e->getMessage() ) );
-			$view->relatedErrorList = $view->get( 'relatedErrorList', array() ) + $error;
+			$error = array($this->getContext()->getI18n()->dt('client', $e->getMessage()));
+			$view->relatedErrorList = $view->get('relatedErrorList', array()) + $error;
 		}
-		catch( \Aimeos\Controller\Frontend\Exception $e )
+		catch (\Aimeos\Controller\Frontend\Exception $e)
 		{
-			$error = array( $this->getContext()->getI18n()->dt( 'controller/frontend', $e->getMessage() ) );
-			$view->relatedErrorList = $view->get( 'relatedErrorList', array() ) + $error;
+			$error = array($this->getContext()->getI18n()->dt('controller/frontend', $e->getMessage()));
+			$view->relatedErrorList = $view->get('relatedErrorList', array()) + $error;
 		}
-		catch( \Aimeos\MShop\Exception $e )
+		catch (\Aimeos\MShop\Exception $e)
 		{
-			$error = array( $this->getContext()->getI18n()->dt( 'mshop', $e->getMessage() ) );
-			$view->relatedErrorList = $view->get( 'relatedErrorList', array() ) + $error;
+			$error = array($this->getContext()->getI18n()->dt('mshop', $e->getMessage()));
+			$view->relatedErrorList = $view->get('relatedErrorList', array()) + $error;
 		}
-		catch( \Exception $e )
+		catch (\Exception $e)
 		{
-			$context->getLogger()->log( $e->getMessage() . PHP_EOL . $e->getTraceAsString() );
+			$context->getLogger()->log($e->getMessage().PHP_EOL.$e->getTraceAsString());
 
-			$error = array( $context->getI18n()->dt( 'client', 'A non-recoverable error occured' ) );
-			$view->relatedErrorList = $view->get( 'relatedErrorList', array() ) + $error;
+			$error = array($context->getI18n()->dt('client', 'A non-recoverable error occured'));
+			$view->relatedErrorList = $view->get('relatedErrorList', array()) + $error;
 		}
 
 		/** client/html/basket/related/standard/template-body
@@ -139,7 +139,7 @@ 
 		$tplconf = 'client/html/basket/related/standard/template-body';
 		$default = 'basket/related/body-default.php';
 
-		return $view->render( $view->config( $tplconf, $default ) );
+		return $view->render($view->config($tplconf, $default));
 	}
 
 
@@ -151,21 +151,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->relatedHeader = $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 '';
 		}
 
@@ -193,7 +193,7 @@ 
 		$tplconf = 'client/html/basket/related/standard/template-header';
 		$default = 'basket/related/header-default.php';
 
-		return $view->render( $view->config( $tplconf, $default ) );
+		return $view->render($view->config($tplconf, $default));
 	}
 
 
@@ -204,7 +204,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/basket/related/decorators/excludes
 		 * Excludes decorators added by the "common" option from the basket related html client
@@ -280,7 +280,7 @@ 
 		 * @see client/html/basket/related/decorators/global
 		 */
 
-		return $this->createSubClient( 'basket/related/' . $type, $name );
+		return $this->createSubClient('basket/related/'.$type, $name);
 	}
 
 
@@ -291,7 +291,7 @@ 
 	 */
 	protected function getSubClientNames()
 	{
-		return $this->getContext()->getConfig()->get( $this->subPartPath, $this->subPartNames );
+		return $this->getContext()->getConfig()->get($this->subPartPath, $this->subPartNames);
 	}
 
 
@@ -303,13 +303,13 @@ 
 	 * @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();
 
-			$view->relatedBasket = \Aimeos\Controller\Frontend\Factory::createController( $context, 'basket' )->get();
+			$view->relatedBasket = \Aimeos\Controller\Frontend\Factory::createController($context, 'basket')->get();
 
 			$this->cache = $view;
 		}