aimeos / ai-symfony

lib/custom/tests/MW/View/Helper/Url/Symfony2Test.php

Spacing +13 added lines, -13 removed lines

@@ -25,17 +25,17 @@ 
 	 */
 	protected function setUp()
 	{
-		if( !class_exists( '\Symfony\Component\Routing\Router' ) ) {
-			$this->markTestSkipped( 'Symfony\Component\Routing\Router is not available' );
+		if (!class_exists('\Symfony\Component\Routing\Router')) {
+			$this->markTestSkipped('Symfony\Component\Routing\Router is not available');
 		}
 
 		$view = new \Aimeos\MW\View\Standard();
 
-		$loc = new \Symfony\Component\Config\FileLocator( array( __DIR__ . DIRECTORY_SEPARATOR . '_testfiles' ) );
-		$loader = new \Symfony\Component\Routing\Loader\PhpFileLoader( $loc );
-		$router = new \Symfony\Component\Routing\Router( $loader, 'routing.php' );
+		$loc = new \Symfony\Component\Config\FileLocator(array(__DIR__ . DIRECTORY_SEPARATOR . '_testfiles'));
+		$loader = new \Symfony\Component\Routing\Loader\PhpFileLoader($loc);
+		$router = new \Symfony\Component\Routing\Router($loader, 'routing.php');
 
-		$this->object = new \Aimeos\MW\View\Helper\Url\Symfony2( $view, $router, array( 'site' => 'unittest' ) );
+		$this->object = new \Aimeos\MW\View\Helper\Url\Symfony2($view, $router, array('site' => 'unittest'));
 	}
 
 
@@ -53,32 +53,32 @@ 
 
 	public function testTransform()
 	{
-		$this->assertEquals( '/unittest/lists', $this->object->transform( 'catalog_list' ) );
+		$this->assertEquals('/unittest/lists', $this->object->transform('catalog_list'));
 	}
 
 
 	public function testTransformSlashes()
 	{
-		$this->assertEquals( '/unittest/lists?test=a/b', $this->object->transform( 'catalog_list', null, null, array( 'test' => 'a/b' ) ) );
+		$this->assertEquals('/unittest/lists?test=a/b', $this->object->transform('catalog_list', null, null, array('test' => 'a/b')));
 	}
 
 
 	public function testTransformArrays()
 	{
-		$this->assertEquals( '/unittest/lists?test%5B0%5D=a&test%5B1%5D=b', $this->object->transform( 'catalog_list', null, null, array( 'test' => array( 'a', 'b' ) ) ) );
+		$this->assertEquals('/unittest/lists?test%5B0%5D=a&test%5B1%5D=b', $this->object->transform('catalog_list', null, null, array('test' => array('a', 'b'))));
 	}
 
 
 	public function testTransformTrailing()
 	{
-		$this->assertEquals( '/unittest/lists?trailing=a_b', $this->object->transform( 'catalog_list', null, null, array(), array( 'a', 'b' ) ) );
+		$this->assertEquals('/unittest/lists?trailing=a_b', $this->object->transform('catalog_list', null, null, array(), array('a', 'b')));
 	}
 
 
 	public function testTransformAbsolute()
 	{
-		$options = array( 'absoluteUri' => true );
-		$result = $this->object->transform( 'catalog_list', null, null, array(), array(), $options );
-		$this->assertEquals( 'http://localhost/unittest/lists', $result );
+		$options = array('absoluteUri' => true);
+		$result = $this->object->transform('catalog_list', null, null, array(), array(), $options);
+		$this->assertEquals('http://localhost/unittest/lists', $result);
 	}
 }

lib/custom/tests/MW/View/Helper/Url/_testfiles/routing.php

Spacing +2 added lines, -2 removed lines

@@ -11,7 +11,7 @@
 
 $collection = new RouteCollection();
 
-$route = new Route( '/{site}/lists', array( '_controller' => 'AimeosShopBundle:Catalog:list' ) );
-$collection->add( 'catalog_list', $route );
+$route = new Route('/{site}/lists', array('_controller' => 'AimeosShopBundle:Catalog:list'));
+$collection->add('catalog_list', $route);
 
 return $collection;

lib/custom/tests/TestHelper.php

Spacing +21 added lines, -21 removed lines

@@ -17,14 +17,14 @@ 
 
 		$includepaths = $mshop->getIncludePaths();
 		$includepaths[] = get_include_path();
-		set_include_path( implode( PATH_SEPARATOR, $includepaths ) );
+		set_include_path(implode(PATH_SEPARATOR, $includepaths));
 	}
 
 
-	public static function getContext( $site = 'unittest' )
+	public static function getContext($site = 'unittest')
 	{
-		if( !isset( self::$context[$site] ) ) {
-			self::$context[$site] = self::createContext( $site );
+		if (!isset(self::$context[$site])) {
+			self::$context[$site] = self::createContext($site);
 		}
 
 		return clone self::$context[$site];
@@ -33,13 +33,13 @@ 
 
 	private static function getAimeos()
 	{
-		if( !isset( self::$aimeos ) )
+		if (!isset(self::$aimeos))
 		{
 			require_once 'Bootstrap.php';
-			spl_autoload_register( 'Aimeos\\Bootstrap::autoload' );
+			spl_autoload_register('Aimeos\\Bootstrap::autoload');
 
-			$extdir = dirname( dirname( dirname( __DIR__ ) ) );
-			self::$aimeos = new \Aimeos\Bootstrap( array( $extdir ), false );
+			$extdir = dirname(dirname(dirname(__DIR__)));
+			self::$aimeos = new \Aimeos\Bootstrap(array($extdir), false);
 		}
 
 		return self::$aimeos;
@@ -52,37 +52,37 @@ 
 	 * @param string $site Unique site code
 	 * @return \\Aimeos\MShop\Context\Item\Iface Context object
 	 */
-		private static function createContext( $site )
+		private static function createContext($site)
 	{
 		$ctx = new \Aimeos\MShop\Context\Item\Standard();
 		$mshop = self::getAimeos();
 
 
-		$paths = $mshop->getConfigPaths( 'mysql' );
+		$paths = $mshop->getConfigPaths('mysql');
 		$paths[] = __DIR__ . DIRECTORY_SEPARATOR . 'config';
 
-		$conf = new \Aimeos\MW\Config\PHPArray( array(), $paths );
-		$ctx->setConfig( $conf );
+		$conf = new \Aimeos\MW\Config\PHPArray(array(), $paths);
+		$ctx->setConfig($conf);
 
 
-		$dbm = new \Aimeos\MW\DB\Manager\PDO( $conf );
-		$ctx->setDatabaseManager( $dbm );
+		$dbm = new \Aimeos\MW\DB\Manager\PDO($conf);
+		$ctx->setDatabaseManager($dbm);
 
 
-		$logger = new \Aimeos\MW\Logger\File( $site . '.log', \Aimeos\MW\Logger\Base::DEBUG );
-		$ctx->setLogger( $logger );
+		$logger = new \Aimeos\MW\Logger\File($site . '.log', \Aimeos\MW\Logger\Base::DEBUG);
+		$ctx->setLogger($logger);
 
 
 		$session = new \Aimeos\MW\Session\None();
-		$ctx->setSession( $session );
+		$ctx->setSession($session);
 
 
-		$localeManager = \Aimeos\MShop\Locale\Manager\Factory::createManager( $ctx );
-		$localeItem = $localeManager->bootstrap( $site, '', '', false );
+		$localeManager = \Aimeos\MShop\Locale\Manager\Factory::createManager($ctx);
+		$localeItem = $localeManager->bootstrap($site, '', '', false);
 
-		$ctx->setLocale( $localeItem );
+		$ctx->setLocale($localeItem);
 
-		$ctx->setEditor( 'ai-symfony:unittest' );
+		$ctx->setEditor('ai-symfony:unittest');
 
 		return $ctx;
 	}

lib/custom/tests/bootstrap.php

Spacing +6 added lines, -6 removed lines

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

lib/custom/tests/MW/View/Helper/Request/Symfony2Test.php

Spacing +15 added lines, -15 removed lines

@@ -15,51 +15,51 @@
 
 	protected function setUp()
 	{
-		if( !class_exists( '\Symfony\Component\HttpFoundation\Request' ) ) {
-			$this->markTestSkipped( '\Symfony\Component\HttpFoundation\Request is not available' );
+		if (!class_exists('\Symfony\Component\HttpFoundation\Request')) {
+			$this->markTestSkipped('\Symfony\Component\HttpFoundation\Request is not available');
 		}
 
 		$view = new \Aimeos\MW\View\Standard();
-		$this->mock = $this->getMock( '\Symfony\Component\HttpFoundation\Request' );
-		$this->object = new \Aimeos\MW\View\Helper\Request\Symfony2( $view, $this->mock, array() );
+		$this->mock = $this->getMock('\Symfony\Component\HttpFoundation\Request');
+		$this->object = new \Aimeos\MW\View\Helper\Request\Symfony2($view, $this->mock, array());
 	}
 
 
 	public function testTransform()
 	{
-		$this->assertInstanceOf( '\\Aimeos\\MW\\View\\Helper\\Request\\Symfony2', $this->object->transform() );
+		$this->assertInstanceOf('\\Aimeos\\MW\\View\\Helper\\Request\\Symfony2', $this->object->transform());
 	}
 
 
 	public function testGetBody()
 	{
-		$this->mock->expects( $this->once() )->method( 'getContent' )
-			->will( $this->returnValue( 'body' ) );
+		$this->mock->expects($this->once())->method('getContent')
+			->will($this->returnValue('body'));
 
-		$this->assertEquals( 'body', $this->object->transform()->getBody() );
+		$this->assertEquals('body', $this->object->transform()->getBody());
 	}
 
 
 	public function testGetClientAddress()
 	{
-		$this->mock->expects( $this->once() )->method( 'getClientIp' )
-			->will( $this->returnValue( '127.0.0.1' ) );
+		$this->mock->expects($this->once())->method('getClientIp')
+			->will($this->returnValue('127.0.0.1'));
 
-		$this->assertEquals( '127.0.0.1', $this->object->transform()->getClientAddress() );
+		$this->assertEquals('127.0.0.1', $this->object->transform()->getClientAddress());
 	}
 
 
 	public function testGetTarget()
 	{
-		$this->mock->expects( $this->once() )->method( 'get' )
-			->will( $this->returnValue( 'test' ) );
+		$this->mock->expects($this->once())->method('get')
+			->will($this->returnValue('test'));
 
-		$this->assertEquals( 'test', $this->object->transform()->getTarget() );
+		$this->assertEquals('test', $this->object->transform()->getTarget());
 	}
 
 
 	public function testGetUploadedFiles()
 	{
-		$this->assertEquals( array(), $this->object->transform()->getUploadedFiles() );
+		$this->assertEquals(array(), $this->object->transform()->getUploadedFiles());
 	}
 }

lib/custom/src/MW/View/Helper/Request/Symfony2.php

Spacing +8 added lines, -8 removed lines

@@ -33,9 +33,9 @@ 
 	 * @param \Aimeos\MW\View\Iface $view View instance with registered view helpers
 	 * @param \Symfony\Component\HttpFoundation\Request $request Symfony2 request object
 	 */
-	public function __construct( $view, \Symfony\Component\HttpFoundation\Request $request )
+	public function __construct($view, \Symfony\Component\HttpFoundation\Request $request)
 	{
-		parent::__construct( $view, null, null, null, $request->files );
+		parent::__construct($view, null, null, null, $request->files);
 
 		$this->request = $request;
 	}
@@ -70,7 +70,7 @@ 
 	 */
 	public function getTarget()
 	{
-		return $this->request->get( '_route' );
+		return $this->request->get('_route');
 	}
 
 
@@ -80,13 +80,13 @@ 
 	 * @param \Traversable|array $files File upload data
 	 * @return array Multi-dimensional list of file objects
 	 */
-	protected function createUploadedFiles( $files )
+	protected function createUploadedFiles($files)
 	{
 		$list = array();
 
-		foreach( $files as $key => $value )
+		foreach ($files as $key => $value)
 		{
-			if( $value instanceof UploadedFile )
+			if ($value instanceof UploadedFile)
 			{
 				$list[$key] = new \Aimeos\MW\View\Helper\Request\File\Standard(
 					$value->getRealPath(),
@@ -96,9 +96,9 @@ 
 					$value->getError()
 				);
 			}
-			elseif( is_array( $value ) )
+			elseif (is_array($value))
 			{
-				$list[$key] = $this->createUploadedFiles( $value );
+				$list[$key] = $this->createUploadedFiles($value);
 			}
 		}
 

Braces +1 added lines, -2 removed lines

@@ -95,8 +95,7 @@
 					$value->getClientMimeType(),
 					$value->getError()
 				);
-			}
-			elseif( is_array( $value ) )
+			} elseif( is_array( $value ) )
 			{
 				$list[$key] = $this->createUploadedFiles( $value );
 			}