aimeos / ai-controller-frontend

controller/frontend/src/Controller/Frontend/Factory.php

Spacing +4 added lines, -4 removed lines

@@ -26,9 +26,9 @@ 
 	 * @return \Aimeos\Controller\Frontend\Iface New frontend controller
 	 * @throws \Aimeos\Controller\Frontend\Exception If the given path is invalid or the manager wasn't found
 	 */
-	static public function createController( \Aimeos\MShop\Context\Item\Iface $context, $path )
+	static public function createController(\Aimeos\MShop\Context\Item\Iface $context, $path)
 	{
-		return self::create( $context, $path );
+		return self::create($context, $path);
 	}
 
 
@@ -38,8 +38,8 @@ 
 	 * @param boolean $value True to enable caching, false to disable it.
 	 * @return boolean Previous cache setting
 	 */
-	static public function setCache( $value )
+	static public function setCache($value)
 	{
-		return self::cache( $value );
+		return self::cache($value);
 	}
 }

controller/frontend/src/Controller/Frontend.php

Spacing +19 added lines, -19 removed lines

@@ -27,7 +27,7 @@ 
 	 * @param boolean $value True to enable caching, false to disable it.
 	 * @return boolean Previous cache setting
 	 */
-	static public function cache( $value )
+	static public function cache($value)
 	{
 		$old = self::$cache;
 		self::$cache = (boolean) $value;
@@ -44,11 +44,11 @@ 
 	 * @param integer $id Context ID the objects have been created with (string of \Aimeos\MShop\Context\Item\Iface)
 	 * @param string $path Path describing the controller to clear, e.g. "basket"
 	 */
-	static public function clear( $id = null, $path = null )
+	static public function clear($id = null, $path = null)
 	{
-		if( $id !== null )
+		if ($id !== null)
 		{
-			if( $path !== null ) {
+			if ($path !== null) {
 				self::$controllers[$id][$path] = null;
 			} else {
 				self::$controllers[$id] = [];
@@ -76,37 +76,37 @@ 
 	 * @return \Aimeos\Controller\Frontend\Iface New frontend controller
 	 * @throws \Aimeos\Controller\Frontend\Exception If the given path is invalid or the manager wasn't found
 	 */
-	static public function create( \Aimeos\MShop\Context\Item\Iface $context, $path )
+	static public function create(\Aimeos\MShop\Context\Item\Iface $context, $path)
 	{
-		if( empty( $path ) ) {
-			throw new \Aimeos\Controller\Frontend\Exception( sprintf( 'Controller path is empty' ) );
+		if (empty($path)) {
+			throw new \Aimeos\Controller\Frontend\Exception(sprintf('Controller path is empty'));
 		}
 
 		$id = (string) $context;
 
-		if( self::$cache === false || !isset( self::$controllers[$id][$path] ) )
+		if (self::$cache === false || !isset(self::$controllers[$id][$path]))
 		{
-			$parts = explode( '/', $path );
+			$parts = explode('/', $path);
 
-			foreach( $parts as $key => $part )
+			foreach ($parts as $key => $part)
 			{
-				if( ctype_alnum( $part ) === false ) {
-					throw new \Aimeos\Controller\Frontend\Exception( sprintf( 'Invalid characters in controller name "%1$s" in "%2$s"', $part, $path ) );
+				if (ctype_alnum($part) === false) {
+					throw new \Aimeos\Controller\Frontend\Exception(sprintf('Invalid characters in controller name "%1$s" in "%2$s"', $part, $path));
 				}
 
-				$parts[$key] = ucwords( $part );
+				$parts[$key] = ucwords($part);
 			}
 
-			$factory = '\\Aimeos\\Controller\\Frontend\\' . join( '\\', $parts ) . '\\Factory';
+			$factory = '\\Aimeos\\Controller\\Frontend\\' . join('\\', $parts) . '\\Factory';
 
-			if( class_exists( $factory ) === false ) {
-				throw new \Aimeos\Controller\Frontend\Exception( sprintf( 'Class "%1$s" not available', $factory ) );
+			if (class_exists($factory) === false) {
+				throw new \Aimeos\Controller\Frontend\Exception(sprintf('Class "%1$s" not available', $factory));
 			}
 
-			$manager = call_user_func_array( array( $factory, 'createController' ), array( $context ) );
+			$manager = call_user_func_array(array($factory, 'createController'), array($context));
 
-			if( $manager === false ) {
-				throw new \Aimeos\Controller\Frontend\Exception( sprintf( 'Invalid factory "%1$s"', $factory ) );
+			if ($manager === false) {
+				throw new \Aimeos\Controller\Frontend\Exception(sprintf('Invalid factory "%1$s"', $factory));
 			}
 
 			self::$controllers[$id][$path] = $manager;

controller/frontend/tests/Controller/FrontendTest.php

Spacing +33 added lines, -33 removed lines

@@ -13,94 +13,94 @@
 {
 	public function testCreateController()
 	{
-		$controller = \Aimeos\Controller\Frontend::create( \TestHelperFrontend::getContext(), 'basket' );
-		$this->assertInstanceOf( '\\Aimeos\\Controller\\Frontend\\Iface', $controller );
+		$controller = \Aimeos\Controller\Frontend::create(\TestHelperFrontend::getContext(), 'basket');
+		$this->assertInstanceOf('\\Aimeos\\Controller\\Frontend\\Iface', $controller);
 	}
 
 
 	public function testCreateControllerEmpty()
 	{
-		$this->setExpectedException( '\\Aimeos\\Controller\\Frontend\\Exception' );
-		\Aimeos\Controller\Frontend::create( \TestHelperFrontend::getContext(), '' );
+		$this->setExpectedException('\\Aimeos\\Controller\\Frontend\\Exception');
+		\Aimeos\Controller\Frontend::create(\TestHelperFrontend::getContext(), '');
 	}
 
 
 	public function testCreateControllerInvalidName()
 	{
-		$this->setExpectedException( '\\Aimeos\\Controller\\Frontend\\Exception' );
-		\Aimeos\Controller\Frontend::create( \TestHelperFrontend::getContext(), '%^' );
+		$this->setExpectedException('\\Aimeos\\Controller\\Frontend\\Exception');
+		\Aimeos\Controller\Frontend::create(\TestHelperFrontend::getContext(), '%^');
 	}
 
 
 	public function testCreateControllerNotExisting()
 	{
-		$this->setExpectedException( '\\Aimeos\\Controller\\Frontend\\Exception' );
-		\Aimeos\Controller\Frontend::create( \TestHelperFrontend::getContext(), 'notexist' );
+		$this->setExpectedException('\\Aimeos\\Controller\\Frontend\\Exception');
+		\Aimeos\Controller\Frontend::create(\TestHelperFrontend::getContext(), 'notexist');
 	}
 
 
 	public function testCreateSubControllerNotExisting()
 	{
-		$this->setExpectedException( '\\Aimeos\\Controller\\Frontend\\Exception' );
-		\Aimeos\Controller\Frontend::create( \TestHelperFrontend::getContext(), 'basket/notexist' );
+		$this->setExpectedException('\\Aimeos\\Controller\\Frontend\\Exception');
+		\Aimeos\Controller\Frontend::create(\TestHelperFrontend::getContext(), 'basket/notexist');
 	}
 
 
 	public function testClear()
 	{
-		$cache = \Aimeos\Controller\Frontend::cache( true );
+		$cache = \Aimeos\Controller\Frontend::cache(true);
 
 		$context = \TestHelperFrontend::getContext();
 
-		$controller1 = \Aimeos\Controller\Frontend::create( $context, 'basket' );
+		$controller1 = \Aimeos\Controller\Frontend::create($context, 'basket');
 		\Aimeos\Controller\Frontend::clear();
-		$controller2 = \Aimeos\Controller\Frontend::create( $context, 'basket' );
+		$controller2 = \Aimeos\Controller\Frontend::create($context, 'basket');
 
-		\Aimeos\Controller\Frontend::cache( $cache );
+		\Aimeos\Controller\Frontend::cache($cache);
 
-		$this->assertNotSame( $controller1, $controller2 );
+		$this->assertNotSame($controller1, $controller2);
 	}
 
 
 	public function testClearSite()
 	{
-		$cache = \Aimeos\Controller\Frontend::cache( true );
+		$cache = \Aimeos\Controller\Frontend::cache(true);
 
 		$context = \TestHelperFrontend::getContext();
 
-		$basket1 = \Aimeos\Controller\Frontend::create( $context, 'basket' );
-		$catalog1 = \Aimeos\Controller\Frontend::create( $context, 'catalog' );
+		$basket1 = \Aimeos\Controller\Frontend::create($context, 'basket');
+		$catalog1 = \Aimeos\Controller\Frontend::create($context, 'catalog');
 
-		\Aimeos\Controller\Frontend::clear( (string) $context );
+		\Aimeos\Controller\Frontend::clear((string) $context);
 
-		$basket2 = \Aimeos\Controller\Frontend::create( $context, 'basket' );
-		$catalog2 = \Aimeos\Controller\Frontend::create( $context, 'catalog' );
+		$basket2 = \Aimeos\Controller\Frontend::create($context, 'basket');
+		$catalog2 = \Aimeos\Controller\Frontend::create($context, 'catalog');
 
-		\Aimeos\Controller\Frontend::cache( $cache );
+		\Aimeos\Controller\Frontend::cache($cache);
 
-		$this->assertNotSame( $basket1, $basket2 );
-		$this->assertNotSame( $catalog1, $catalog2 );
+		$this->assertNotSame($basket1, $basket2);
+		$this->assertNotSame($catalog1, $catalog2);
 	}
 
 
 	public function testClearSpecific()
 	{
-		$cache = \Aimeos\Controller\Frontend::cache( true );
+		$cache = \Aimeos\Controller\Frontend::cache(true);
 
 		$context = \TestHelperFrontend::getContext();
 
-		$basket1 = \Aimeos\Controller\Frontend::create( $context, 'basket' );
-		$catalog1 = \Aimeos\Controller\Frontend::create( $context, 'catalog' );
+		$basket1 = \Aimeos\Controller\Frontend::create($context, 'basket');
+		$catalog1 = \Aimeos\Controller\Frontend::create($context, 'catalog');
 
-		\Aimeos\Controller\Frontend::clear( (string) $context, 'basket' );
+		\Aimeos\Controller\Frontend::clear((string) $context, 'basket');
 
-		$basket2 = \Aimeos\Controller\Frontend::create( $context, 'basket' );
-		$catalog2 = \Aimeos\Controller\Frontend::create( $context, 'catalog' );
+		$basket2 = \Aimeos\Controller\Frontend::create($context, 'basket');
+		$catalog2 = \Aimeos\Controller\Frontend::create($context, 'catalog');
 
-		\Aimeos\Controller\Frontend::cache( $cache );
+		\Aimeos\Controller\Frontend::cache($cache);
 
-		$this->assertNotSame( $basket1, $basket2 );
-		$this->assertSame( $catalog1, $catalog2 );
+		$this->assertNotSame($basket1, $basket2);
+		$this->assertSame($catalog1, $catalog2);
 	}
 
 }
\ No newline at end of file