aimeos / ai-laravel

lib/custom/src/MW/View/Helper/Response/Laravel5.php

Spacing +2 added lines, -2 removed lines

@@ -26,8 +26,8 @@
 	 *
 	 * @param \Aimeos\MW\View\Iface $view View instance with registered view helpers
 	 */
-	public function __construct( \Aimeos\MW\View\Iface $view )
+	public function __construct(\Aimeos\MW\View\Iface $view)
 	{
-		parent::__construct( $view, new \Zend\Diactoros\Response() );
+		parent::__construct($view, new \Zend\Diactoros\Response());
 	}
 }

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

Spacing +11 added lines, -11 removed lines

@@ -17,42 +17,42 @@
 
 	protected function setUp()
 	{
-		if( !class_exists( '\Illuminate\Http\Request' ) ) {
-			$this->markTestSkipped( '\Illuminate\Http\Request is not available' );
+		if (!class_exists('\Illuminate\Http\Request')) {
+			$this->markTestSkipped('\Illuminate\Http\Request is not available');
 		}
 
-		if( !class_exists( '\Zend\Diactoros\Response' ) ) {
-			$this->markTestSkipped( '\Zend\Diactoros\Response is not available' );
+		if (!class_exists('\Zend\Diactoros\Response')) {
+			$this->markTestSkipped('\Zend\Diactoros\Response is not available');
 		}
 
 		$view = new \Aimeos\MW\View\Standard();
-		$param = array( 'HTTP_HOST' => 'localhost', 'REMOTE_ADDR' => '127.0.0.1' );
-		$request = new \Illuminate\Http\Request( array(), array(), array(), array(), array(), $param, 'Content' );
+		$param = array('HTTP_HOST' => 'localhost', 'REMOTE_ADDR' => '127.0.0.1');
+		$request = new \Illuminate\Http\Request(array(), array(), array(), array(), array(), $param, 'Content');
 
-		$this->object = new \Aimeos\MW\View\Helper\Request\Laravel5( $view, $request );
+		$this->object = new \Aimeos\MW\View\Helper\Request\Laravel5($view, $request);
 	}
 
 
 	protected function tearDown()
 	{
-		unset( $this->object, $this->mock );
+		unset($this->object, $this->mock);
 	}
 
 
 	public function testTransform()
 	{
-		$this->assertInstanceOf( '\Aimeos\MW\View\Helper\Request\Laravel5', $this->object->transform() );
+		$this->assertInstanceOf('\Aimeos\MW\View\Helper\Request\Laravel5', $this->object->transform());
 	}
 
 
 	public function testGetClientAddress()
 	{
-		$this->assertEquals( '127.0.0.1', $this->object->getClientAddress() );
+		$this->assertEquals('127.0.0.1', $this->object->getClientAddress());
 	}
 
 
 	public function testGetTarget()
 	{
-		$this->assertEquals( null, $this->object->getTarget() );
+		$this->assertEquals(null, $this->object->getTarget());
 	}
 }

lib/custom/tests/MW/View/Helper/Response/Laravel5Test.php

Spacing +7 added lines, -7 removed lines

@@ -17,27 +17,27 @@
 
 	protected function setUp()
 	{
-		if( !class_exists( '\Illuminate\Http\Response' ) ) {
-			$this->markTestSkipped( '\Illuminate\Http\Response is not available' );
+		if (!class_exists('\Illuminate\Http\Response')) {
+			$this->markTestSkipped('\Illuminate\Http\Response is not available');
 		}
 
-		if( !class_exists( '\Zend\Diactoros\Response' ) ) {
-			$this->markTestSkipped( '\Zend\Diactoros\Response is not available' );
+		if (!class_exists('\Zend\Diactoros\Response')) {
+			$this->markTestSkipped('\Zend\Diactoros\Response is not available');
 		}
 
 		$view = new \Aimeos\MW\View\Standard();
-		$this->object = new \Aimeos\MW\View\Helper\Response\Laravel5( $view );
+		$this->object = new \Aimeos\MW\View\Helper\Response\Laravel5($view);
 	}
 
 
 	protected function tearDown()
 	{
-		unset( $this->object, $this->mock );
+		unset($this->object, $this->mock);
 	}
 
 
 	public function testTransform()
 	{
-		$this->assertInstanceOf( '\Aimeos\MW\View\Helper\Response\Laravel5', $this->object->transform() );
+		$this->assertInstanceOf('\Aimeos\MW\View\Helper\Response\Laravel5', $this->object->transform());
 	}
 }

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

Spacing +16 added lines, -16 removed lines

@@ -34,11 +34,11 @@ 
 	 * @param \\Aimeos\MW\View\Iface $view View instance with registered view helpers
 	 * @param \Illuminate\Http\Request $request Laravel request object
 	 */
-	public function __construct( \Aimeos\MW\View\Iface $view, \Illuminate\Http\Request $request )
+	public function __construct(\Aimeos\MW\View\Iface $view, \Illuminate\Http\Request $request)
 	{
 		$this->request = $request;
 
-		parent::__construct( $view, $this->createRequest( $request ) );
+		parent::__construct($view, $this->createRequest($request));
 	}
 
 
@@ -60,7 +60,7 @@ 
 	 */
 	public function getTarget()
 	{
-		if( ( $route = $this->request->route() ) !== null ) {
+		if (($route = $this->request->route()) !== null) {
 			return $route->getName();
 		}
 	}
@@ -72,10 +72,10 @@ 
 	 * @param \Illuminate\Http\Request $nativeRequest Laravel request object
 	 * @return \Psr\Http\Message\ServerRequestInterface PSR-7 request object
 	 */
-	protected function createRequest( \Illuminate\Http\Request $nativeRequest )
+	protected function createRequest(\Illuminate\Http\Request $nativeRequest)
 	{
-		$files = ServerRequestFactory::normalizeFiles( $this->getFiles( $nativeRequest->files->all() ) );
-		$server = ServerRequestFactory::normalizeServer( $nativeRequest->server->all() );
+		$files = ServerRequestFactory::normalizeFiles($this->getFiles($nativeRequest->files->all()));
+		$server = ServerRequestFactory::normalizeServer($nativeRequest->server->all());
 		$headers = $nativeRequest->headers->all();
 		$cookies = $nativeRequest->cookies->all();
 		$post = $nativeRequest->request->all();
@@ -83,13 +83,13 @@ 
 		$method = $nativeRequest->getMethod();
 		$uri = $nativeRequest->getUri();
 
-		$body = new Stream( 'php://temp', 'wb+' );
-		$body->write( $nativeRequest->getContent() );
+		$body = new Stream('php://temp', 'wb+');
+		$body->write($nativeRequest->getContent());
 
-		$request = new ServerRequest( $server, $files, $uri, $method, $body, $headers, $cookies, $query, $post );
+		$request = new ServerRequest($server, $files, $uri, $method, $body, $headers, $cookies, $query, $post);
 
-		foreach( $nativeRequest->attributes->all() as $key => $value ) {
-			$request = $request->withAttribute( $key, $value );
+		foreach ($nativeRequest->attributes->all() as $key => $value) {
+			$request = $request->withAttribute($key, $value);
 		}
 
 		return $request;
@@ -102,13 +102,13 @@ 
 	 * @param array $files Multi-dimensional list of uploaded files from Symfony request
 	 * @return array Multi-dimensional list of uploaded files as PSR-7 objects
 	 */
-	protected function getFiles( array $files )
+	protected function getFiles(array $files)
 	{
 		$list = array();
 
-		foreach( $files as $key => $value )
+		foreach ($files as $key => $value)
 		{
-			if( $value instanceof \Symfony\Component\HttpFoundation\File\UploadedFile )
+			if ($value instanceof \Symfony\Component\HttpFoundation\File\UploadedFile)
 			{
 				$list[$key] = new \Zend\Diactoros\UploadedFile(
 					$value->getRealPath(),
@@ -118,9 +118,9 @@ 
 					$value->getClientMimeType()
 				);
 			}
-			elseif( is_array( $value ) )
+			elseif (is_array($value))
 			{
-				$list[$key] = $this->getFiles( $value );
+				$list[$key] = $this->getFiles($value);
 			}
 		}
 

Doc Comments +1 added lines, -1 removed lines

@@ -31,7 +31,7 @@
 	/**
 	 * Initializes the request view helper.
 	 *
-	 * @param \\Aimeos\MW\View\Iface $view View instance with registered view helpers
+	 * @param \Aimeos\MW\View\Iface $view View instance with registered view helpers
 	 * @param \Illuminate\Http\Request $request Laravel request object
 	 */
 	public function __construct( \Aimeos\MW\View\Iface $view, \Illuminate\Http\Request $request )

Braces +1 added lines, -2 removed lines

@@ -117,8 +117,7 @@
 					$value->getClientOriginalName(),
 					$value->getClientMimeType()
 				);
-			}
-			elseif( is_array( $value ) )
+			} elseif( is_array( $value ) )
 			{
 				$list[$key] = $this->getFiles( $value );
 			}

lib/custom/src/MW/View/Helper/Url/Laravel5.php

Spacing +8 added lines, -8 removed lines

@@ -32,9 +32,9 @@ 
 	 * @param \Illuminate\Contracts\Routing\UrlGenerator $builder Laravel URL builder object
 	 * @param array Associative list of fixed parameters that should be available for all routes
 	 */
-	public function __construct( \Aimeos\MW\View\Iface $view, \Illuminate\Contracts\Routing\UrlGenerator $builder, array $fixed )
+	public function __construct(\Aimeos\MW\View\Iface $view, \Illuminate\Contracts\Routing\UrlGenerator $builder, array $fixed)
 	{
-		parent::__construct( $view );
+		parent::__construct($view);
 
 		$this->builder = $builder;
 		$this->fixed = $fixed;
@@ -52,12 +52,12 @@ 
 	 * @param array $config Additional configuration parameter per URL
 	 * @return string Complete URL that can be used in the template
 	 */
-	public function transform( $target = null, $controller = null, $action = null, array $params = array(), array $trailing = array(), array $config = array() )
+	public function transform($target = null, $controller = null, $action = null, array $params = array(), array $trailing = array(), array $config = array())
 	{
-		$params = $this->sanitize( $params );
-		$values = $this->getValues( $config );
+		$params = $this->sanitize($params);
+		$values = $this->getValues($config);
 
-		return $this->builder->route( $target, $params + $this->fixed, $values['absoluteUri'] );
+		return $this->builder->route($target, $params + $this->fixed, $values['absoluteUri']);
 	}
 
 
@@ -67,13 +67,13 @@ 
 	 * @param array $config Associative list of key/value pairs
 	 * @return array Associative list of sanitized key/value pairs
 	 */
-	protected function getValues( array $config )
+	protected function getValues(array $config)
 	{
 		$values = array(
 			'absoluteUri' => false,
 		);
 
-		if( isset( $config['absoluteUri'] ) ) {
+		if (isset($config['absoluteUri'])) {
 			$values['absoluteUri'] = (bool) $config['absoluteUri'];
 		}
 

lib/custom/tests/MW/Logger/Laravel5Test.php

Spacing +21 added lines, -21 removed lines

@@ -20,12 +20,12 @@ 
 	 */
 	protected function setUp()
 	{
-		if( interface_exists( '\\Illuminate\\Contracts\\Logging\\Log' ) === false ) {
-			$this->markTestSkipped( 'Class \\Illuminate\\Contracts\\Logging\\Log not found' );
+		if (interface_exists('\\Illuminate\\Contracts\\Logging\\Log') === false) {
+			$this->markTestSkipped('Class \\Illuminate\\Contracts\\Logging\\Log not found');
 		}
 
-		$this->mock = $this->getMockBuilder( '\\Illuminate\\Contracts\\Logging\\Log' )->getMock();
-		$this->object = new \Aimeos\MW\Logger\Laravel5( $this->mock );
+		$this->mock = $this->getMockBuilder('\\Illuminate\\Contracts\\Logging\\Log')->getMock();
+		$this->object = new \Aimeos\MW\Logger\Laravel5($this->mock);
 	}
 
 
@@ -37,42 +37,42 @@ 
 	 */
 	protected function tearDown()
 	{
-		unset( $this->mock, $this->object );
+		unset($this->mock, $this->object);
 	}
 
 
 	public function testLog()
 	{
-		$this->mock->expects( $this->once() )->method( 'log' )
-			->with( $this->equalTo( 'msg' ), $this->equalTo( 'error' ) );
-		$this->object->log( 'msg' );
+		$this->mock->expects($this->once())->method('log')
+			->with($this->equalTo('msg'), $this->equalTo('error'));
+		$this->object->log('msg');
 	}
 
 
 	public function testNonScalarLog()
 	{
-		$this->mock->expects( $this->once() )->method( 'log' )
-			->with( $this->equalTo( '["error","error2",2]' ), $this->equalTo( 'error' ) );
-		$this->object->log( array( 'error', 'error2', 2 ) );
+		$this->mock->expects($this->once())->method('log')
+			->with($this->equalTo('["error","error2",2]'), $this->equalTo('error'));
+		$this->object->log(array('error', 'error2', 2));
 	}
 
 
 	public function testLogDebug()
 	{
-		$this->object->log( 'emergency', \Aimeos\MW\Logger\Base::EMERG );
-		$this->object->log( 'alert', \Aimeos\MW\Logger\Base::ALERT );
-		$this->object->log( 'critical', \Aimeos\MW\Logger\Base::CRIT );
-		$this->object->log( 'error', \Aimeos\MW\Logger\Base::ERR );
-		$this->object->log( 'warning', \Aimeos\MW\Logger\Base::WARN );
-		$this->object->log( 'notice', \Aimeos\MW\Logger\Base::NOTICE );
-		$this->object->log( 'info', \Aimeos\MW\Logger\Base::INFO );
-		$this->object->log( 'debug', \Aimeos\MW\Logger\Base::DEBUG );
+		$this->object->log('emergency', \Aimeos\MW\Logger\Base::EMERG);
+		$this->object->log('alert', \Aimeos\MW\Logger\Base::ALERT);
+		$this->object->log('critical', \Aimeos\MW\Logger\Base::CRIT);
+		$this->object->log('error', \Aimeos\MW\Logger\Base::ERR);
+		$this->object->log('warning', \Aimeos\MW\Logger\Base::WARN);
+		$this->object->log('notice', \Aimeos\MW\Logger\Base::NOTICE);
+		$this->object->log('info', \Aimeos\MW\Logger\Base::INFO);
+		$this->object->log('debug', \Aimeos\MW\Logger\Base::DEBUG);
 	}
 
 
 	public function testBadPriority()
 	{
-		$this->setExpectedException( '\\Aimeos\\MW\\Logger\\Exception' );
-		$this->object->log( 'error', -1 );
+		$this->setExpectedException('\\Aimeos\\MW\\Logger\\Exception');
+		$this->object->log('error', -1);
 	}
 }

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

Spacing +12 added lines, -12 removed lines

@@ -21,13 +21,13 @@ 
 	 */
 	protected function setUp()
 	{
-		if( !interface_exists( '\\Illuminate\\Contracts\\Routing\\UrlGenerator' ) ) {
-			$this->markTestSkipped( '\\Illuminate\\Contracts\\Routing\\UrlGenerator is not available' );
+		if (!interface_exists('\\Illuminate\\Contracts\\Routing\\UrlGenerator')) {
+			$this->markTestSkipped('\\Illuminate\\Contracts\\Routing\\UrlGenerator is not available');
 		}
 
 		$view = new \Aimeos\MW\View\Standard();
-		$this->mock = $this->getMockBuilder( '\\Illuminate\\Contracts\\Routing\\UrlGenerator' )->getMock();
-		$this->object = new \Aimeos\MW\View\Helper\Url\Laravel5( $view, $this->mock, array() );
+		$this->mock = $this->getMockBuilder('\\Illuminate\\Contracts\\Routing\\UrlGenerator')->getMock();
+		$this->object = new \Aimeos\MW\View\Helper\Url\Laravel5($view, $this->mock, array());
 	}
 
 
@@ -39,25 +39,25 @@ 
 	 */
 	protected function tearDown()
 	{
-		unset( $this->object, $this->mock );
+		unset($this->object, $this->mock);
 	}
 
 
 	public function testTransform()
 	{
-		$this->mock->expects( $this->once() )->method( 'route' )
-			->with( $this->equalTo( 'route'), $this->equalTo( array( 'key' => 'value' ) ), $this->equalTo( false ) );
+		$this->mock->expects($this->once())->method('route')
+			->with($this->equalTo('route'), $this->equalTo(array('key' => 'value')), $this->equalTo(false));
 
-		$this->object->transform( 'route', 'catalog', 'lists', array( 'key' => 'value' ) );
+		$this->object->transform('route', 'catalog', 'lists', array('key' => 'value'));
 	}
 
 
 	public function testTransformAbsolute()
 	{
-		$this->mock->expects( $this->once() )->method( 'route' )
-			->with( $this->equalTo( 'route'), $this->equalTo( array() ), $this->equalTo( true ) );
+		$this->mock->expects($this->once())->method('route')
+			->with($this->equalTo('route'), $this->equalTo(array()), $this->equalTo(true));
 
-		$options = array( 'absoluteUri' => true );
-		$this->object->transform( 'route', 'catalog', 'lists', array(), array(), $options );
+		$options = array('absoluteUri' => true);
+		$this->object->transform('route', 'catalog', 'lists', array(), array(), $options);
 	}
 }

lib/custom/tests/MW/Filesystem/LaravelTest.php

Spacing +124 added lines, -124 removed lines

@@ -11,348 +11,348 @@
 
 	protected function setUp()
 	{
-		if( !interface_exists( '\\Illuminate\\Contracts\\Filesystem\\Filesystem' ) ) {
-			$this->markTestSkipped( 'Install Laravel framework first' );
+		if (!interface_exists('\\Illuminate\\Contracts\\Filesystem\\Filesystem')) {
+			$this->markTestSkipped('Install Laravel framework first');
 		}
 
-		$this->mock = $this->getMockBuilder( '\\Illuminate\\Contracts\\Filesystem\\Filesystem' )
+		$this->mock = $this->getMockBuilder('\\Illuminate\\Contracts\\Filesystem\\Filesystem')
 			->disableOriginalConstructor()
 			->getMock();
 
-		$this->object = new \Aimeos\MW\Filesystem\Laravel( $this->mock, sys_get_temp_dir() );
+		$this->object = new \Aimeos\MW\Filesystem\Laravel($this->mock, sys_get_temp_dir());
 	}
 
 
 	protected function tearDown()
 	{
-		unset( $this->object );
+		unset($this->object);
 	}
 
 
 	public function testIsdir()
 	{
-		$this->mock->expects( $this->once() )->method( 'directories' )
-			->will( $this->returnValue( array( 't', 'test', 'es' ) ) );
+		$this->mock->expects($this->once())->method('directories')
+			->will($this->returnValue(array('t', 'test', 'es')));
 
-		$this->assertTrue( $this->object->isdir( 'test' ) );
+		$this->assertTrue($this->object->isdir('test'));
 	}
 
 
 	public function testIsdirFalse()
 	{
-		$this->mock->expects( $this->once() )->method( 'directories' )
-			->will( $this->returnValue( array( 't', 'es' ) ) );
+		$this->mock->expects($this->once())->method('directories')
+			->will($this->returnValue(array('t', 'es')));
 
-		$this->assertFalse( $this->object->isdir( 'test' ) );
+		$this->assertFalse($this->object->isdir('test'));
 	}
 
 
 	public function testMkdir()
 	{
-		$this->mock->expects( $this->once() )->method( 'makeDirectory' );
+		$this->mock->expects($this->once())->method('makeDirectory');
 
-		$this->object->mkdir( 'test' );
+		$this->object->mkdir('test');
 	}
 
 
 	public function testMkdirException()
 	{
-		$this->mock->expects( $this->once() )->method( 'makeDirectory' )
-			->will( $this->throwException( new \RuntimeException() ) );
+		$this->mock->expects($this->once())->method('makeDirectory')
+			->will($this->throwException(new \RuntimeException()));
 
-		$this->setExpectedException( '\Aimeos\MW\Filesystem\Exception' );
-		$this->object->mkdir( 'test' );
+		$this->setExpectedException('\Aimeos\MW\Filesystem\Exception');
+		$this->object->mkdir('test');
 	}
 
 
 	public function testRmdir()
 	{
-		$this->mock->expects( $this->once() )->method( 'deleteDirectory' );
+		$this->mock->expects($this->once())->method('deleteDirectory');
 
-		$this->object->rmdir( 'test' );
+		$this->object->rmdir('test');
 	}
 
 
 	public function testRmdirException()
 	{
-		$this->mock->expects( $this->once() )->method( 'deleteDirectory' )
-			->will( $this->throwException( new \RuntimeException() ) );
+		$this->mock->expects($this->once())->method('deleteDirectory')
+			->will($this->throwException(new \RuntimeException()));
 
-		$this->setExpectedException( '\Aimeos\MW\Filesystem\Exception' );
-		$this->object->rmdir( 'test' );
+		$this->setExpectedException('\Aimeos\MW\Filesystem\Exception');
+		$this->object->rmdir('test');
 	}
 
 
 	public function testScan()
 	{
-		$list = array( 't', 'es', 'st' );
+		$list = array('t', 'es', 'st');
 
-		$this->mock->expects( $this->once() )->method( 'directories' )
-			->will( $this->returnValue( array( 't', 'es' ) ) );
+		$this->mock->expects($this->once())->method('directories')
+			->will($this->returnValue(array('t', 'es')));
 
-		$this->mock->expects( $this->once() )->method( 'files' )
-			->will( $this->returnValue( array( 'st' ) ) );
+		$this->mock->expects($this->once())->method('files')
+			->will($this->returnValue(array('st')));
 
 		$result = $this->object->scan();
 
-		$this->assertInternalType( 'array', $result );
+		$this->assertInternalType('array', $result);
 
-		foreach( $result as $entry ) {
-			$this->assertTrue( in_array( $entry, $list ) );
+		foreach ($result as $entry) {
+			$this->assertTrue(in_array($entry, $list));
 		}
 	}
 
 
 	public function testScanException()
 	{
-		$this->mock->expects( $this->once() )->method( 'directories' )
-			->will( $this->throwException( new \RuntimeException() ) );
+		$this->mock->expects($this->once())->method('directories')
+			->will($this->throwException(new \RuntimeException()));
 
-		$this->setExpectedException( '\Aimeos\MW\Filesystem\Exception' );
-		$this->object->scan( 'test' );
+		$this->setExpectedException('\Aimeos\MW\Filesystem\Exception');
+		$this->object->scan('test');
 	}
 
 
 	public function testSize()
 	{
-		$this->mock->expects( $this->once() )->method( 'size' )
-			->will( $this->returnValue( 4 ) );
+		$this->mock->expects($this->once())->method('size')
+			->will($this->returnValue(4));
 
-		$result = $this->object->size( 'test' );
+		$result = $this->object->size('test');
 
-		$this->assertEquals( 4, $result );
+		$this->assertEquals(4, $result);
 	}
 
 
 	public function testSizeException()
 	{
-		$this->mock->expects( $this->once() )->method( 'size' )
-			->will( $this->throwException( new \RuntimeException() ) );
+		$this->mock->expects($this->once())->method('size')
+			->will($this->throwException(new \RuntimeException()));
 
-		$this->setExpectedException( '\Aimeos\MW\Filesystem\Exception' );
-		$this->object->size( 'test' );
+		$this->setExpectedException('\Aimeos\MW\Filesystem\Exception');
+		$this->object->size('test');
 	}
 
 
 	public function testTime()
 	{
-		$this->mock->expects( $this->once() )->method( 'lastModified' )
-			->will( $this->returnValue( 1 ) );
+		$this->mock->expects($this->once())->method('lastModified')
+			->will($this->returnValue(1));
 
-		$result = $this->object->time( 'test' );
+		$result = $this->object->time('test');
 
-		$this->assertGreaterThan( 0, $result );
+		$this->assertGreaterThan(0, $result);
 	}
 
 
 	public function testTimeException()
 	{
-		$this->mock->expects( $this->once() )->method( 'lastModified' )
-			->will( $this->throwException( new \RuntimeException() ) );
+		$this->mock->expects($this->once())->method('lastModified')
+			->will($this->throwException(new \RuntimeException()));
 
-		$this->setExpectedException( '\Aimeos\MW\Filesystem\Exception' );
-		$this->object->time( 'test' );
+		$this->setExpectedException('\Aimeos\MW\Filesystem\Exception');
+		$this->object->time('test');
 	}
 
 
 	public function testRm()
 	{
-		$this->mock->expects( $this->once() )->method( 'delete' );
+		$this->mock->expects($this->once())->method('delete');
 
-		$this->object->rm( 'test' );
+		$this->object->rm('test');
 	}
 
 
 	public function testRmException()
 	{
-		$this->mock->expects( $this->once() )->method( 'delete' )
-			->will( $this->throwException( new \RuntimeException() ) );
+		$this->mock->expects($this->once())->method('delete')
+			->will($this->throwException(new \RuntimeException()));
 
-		$this->setExpectedException( '\Aimeos\MW\Filesystem\Exception' );
-		$this->object->rm( 'test' );
+		$this->setExpectedException('\Aimeos\MW\Filesystem\Exception');
+		$this->object->rm('test');
 	}
 
 
 	public function testHas()
 	{
-		$this->mock->expects( $this->once() )->method( 'exists' )
-			->will( $this->returnValue( true ) );
+		$this->mock->expects($this->once())->method('exists')
+			->will($this->returnValue(true));
 
-		$result = $this->object->has( 'test' );
+		$result = $this->object->has('test');
 
-		$this->assertTrue( $result );
+		$this->assertTrue($result);
 	}
 
 
 	public function testHasFalse()
 	{
-		$this->mock->expects( $this->once() )->method( 'exists' )
-			->will( $this->returnValue( false ) );
+		$this->mock->expects($this->once())->method('exists')
+			->will($this->returnValue(false));
 
-		$result = $this->object->has( 'test' );
+		$result = $this->object->has('test');
 
-		$this->assertFalse( $result );
+		$this->assertFalse($result);
 	}
 
 
 	public function testRead()
 	{
-		$this->mock->expects( $this->once() )->method( 'get' )
-			->will( $this->returnValue( 'test' ) );
+		$this->mock->expects($this->once())->method('get')
+			->will($this->returnValue('test'));
 
-		$result = $this->object->read( 'file' );
+		$result = $this->object->read('file');
 
-		$this->assertEquals( 'test', $result );
+		$this->assertEquals('test', $result);
 	}
 
 
 	public function testReadException()
 	{
-		$this->mock->expects( $this->once() )->method( 'get' )
-			->will( $this->throwException( new \RuntimeException() ) );
+		$this->mock->expects($this->once())->method('get')
+			->will($this->throwException(new \RuntimeException()));
 
-		$this->setExpectedException( '\Aimeos\MW\Filesystem\Exception' );
-		$this->object->read( 'readinvalid' );
+		$this->setExpectedException('\Aimeos\MW\Filesystem\Exception');
+		$this->object->read('readinvalid');
 	}
 
 
 	public function testReadf()
 	{
-		$this->mock->expects( $this->once() )->method( 'get' )
-			->will( $this->returnValue( 'test' ) );
+		$this->mock->expects($this->once())->method('get')
+			->will($this->returnValue('test'));
 
-		$result = $this->object->readf( 'file' );
+		$result = $this->object->readf('file');
 
-		$this->assertEquals( 'test', file_get_contents( $result ) );
-		unlink( $result );
+		$this->assertEquals('test', file_get_contents($result));
+		unlink($result);
 	}
 
 
 	public function testReads()
 	{
-		$this->mock->expects( $this->once() )->method( 'get' )
-			->will( $this->returnValue( 'test' ) );
+		$this->mock->expects($this->once())->method('get')
+			->will($this->returnValue('test'));
 
-		$handle = $this->object->reads( 'file' );
+		$handle = $this->object->reads('file');
 
-		$this->assertInternalType( 'resource', $handle );
-		$this->assertEquals( 'test', fgets( $handle ) );
+		$this->assertInternalType('resource', $handle);
+		$this->assertEquals('test', fgets($handle));
 
-		fclose( $handle );
+		fclose($handle);
 	}
 
 
 	public function testReadsException()
 	{
-		$this->mock->expects( $this->once() )->method( 'get' )
-			->will( $this->throwException( new \RuntimeException() ) );
+		$this->mock->expects($this->once())->method('get')
+			->will($this->throwException(new \RuntimeException()));
 
-		$this->setExpectedException( '\Aimeos\MW\Filesystem\Exception' );
-		$this->object->reads( 'readinvalid' );
+		$this->setExpectedException('\Aimeos\MW\Filesystem\Exception');
+		$this->object->reads('readinvalid');
 	}
 
 
 	public function testWrite()
 	{
-		$this->mock->expects( $this->once() )->method( 'put' );
+		$this->mock->expects($this->once())->method('put');
 
-		$this->object->write( 'file', 'test' );
+		$this->object->write('file', 'test');
 	}
 
 
 	public function testWriteException()
 	{
-		$this->mock->expects( $this->once() )->method( 'put' )
-			->will( $this->throwException( new \RuntimeException() ) );
+		$this->mock->expects($this->once())->method('put')
+			->will($this->throwException(new \RuntimeException()));
 
-		$this->setExpectedException( '\Aimeos\MW\Filesystem\Exception' );
-		$this->object->write( '', 'test' );
+		$this->setExpectedException('\Aimeos\MW\Filesystem\Exception');
+		$this->object->write('', 'test');
 	}
 
 
 	public function testWritef()
 	{
-		$file = sys_get_temp_dir() . DIRECTORY_SEPARATOR . 'file99';
-		file_put_contents( $file, 'test' );
+		$file = sys_get_temp_dir().DIRECTORY_SEPARATOR.'file99';
+		file_put_contents($file, 'test');
 
-		$this->mock->expects( $this->once() )->method( 'put' );
+		$this->mock->expects($this->once())->method('put');
 
-		$this->object->writef( 'file', $file );
+		$this->object->writef('file', $file);
 
-		unlink( $file );
+		unlink($file);
 	}
 
 
 	public function testWritefException()
 	{
-		$this->setExpectedException( '\Aimeos\MW\Filesystem\Exception' );
-		$this->object->writef( '', 'invalid' );
+		$this->setExpectedException('\Aimeos\MW\Filesystem\Exception');
+		$this->object->writef('', 'invalid');
 	}
 
 
 	public function testWrites()
 	{
-		$this->mock->expects( $this->once() )->method( 'put' );
+		$this->mock->expects($this->once())->method('put');
 
-		$handle = fopen( __FILE__, 'r' );
+		$handle = fopen(__FILE__, 'r');
 
-		$this->object->writes( 'file', $handle );
+		$this->object->writes('file', $handle);
 
-		fclose( $handle );
+		fclose($handle);
 	}
 
 
 	public function testWritesException()
 	{
-		$this->mock->expects( $this->once() )->method( 'put' )
-			->will( $this->throwException( new \RuntimeException() ) );
+		$this->mock->expects($this->once())->method('put')
+			->will($this->throwException(new \RuntimeException()));
 
-		$handle = fopen( __FILE__, 'r' );
+		$handle = fopen(__FILE__, 'r');
 
-		$this->setExpectedException( '\Aimeos\MW\Filesystem\Exception' );
-		$this->object->writes( 'file', $handle );
+		$this->setExpectedException('\Aimeos\MW\Filesystem\Exception');
+		$this->object->writes('file', $handle);
 	}
 
 
 	public function testWritesException2()
 	{
-		$this->setExpectedException( '\Aimeos\MW\Filesystem\Exception' );
-		$this->object->writes( 'file', null );
+		$this->setExpectedException('\Aimeos\MW\Filesystem\Exception');
+		$this->object->writes('file', null);
 	}
 
 
 	public function testMove()
 	{
-		$this->mock->expects( $this->once() )->method( 'move' );
+		$this->mock->expects($this->once())->method('move');
 
-		$this->object->move( 'file1', 'file2' );
+		$this->object->move('file1', 'file2');
 	}
 
 
 	public function testMoveException()
 	{
-		$this->mock->expects( $this->once() )->method( 'move' )
-			->will( $this->throwException( new \RuntimeException() ) );
+		$this->mock->expects($this->once())->method('move')
+			->will($this->throwException(new \RuntimeException()));
 
-		$this->setExpectedException( '\Aimeos\MW\Filesystem\Exception' );
-		$this->object->move( 'file1', 'file2' );
+		$this->setExpectedException('\Aimeos\MW\Filesystem\Exception');
+		$this->object->move('file1', 'file2');
 	}
 
 
 	public function testCopy()
 	{
-		$this->mock->expects( $this->once() )->method( 'copy' );
+		$this->mock->expects($this->once())->method('copy');
 
-		$this->object->copy( 'file1', 'file2' );
+		$this->object->copy('file1', 'file2');
 	}
 
 
 	public function testCopyException()
 	{
-		$this->mock->expects( $this->once() )->method( 'copy' )
-			->will( $this->throwException( new \RuntimeException() ) );
+		$this->mock->expects($this->once())->method('copy')
+			->will($this->throwException(new \RuntimeException()));
 
-		$this->setExpectedException( '\Aimeos\MW\Filesystem\Exception' );
-		$this->object->copy( 'file1', 'file2' );
+		$this->setExpectedException('\Aimeos\MW\Filesystem\Exception');
+		$this->object->copy('file1', 'file2');
 	}
 }

lib/custom/tests/MShop/Customer/Manager/Lists/Type/LaravelTest.php

Spacing +56 added lines, -56 removed lines

@@ -22,10 +22,10 @@ 
 	protected function setUp()
 	{
 		$this->editor = \TestHelper::getContext()->getEditor();
-		$manager = \Aimeos\MShop\Customer\Manager\Factory::createManager( \TestHelper::getContext(), 'Laravel' );
+		$manager = \Aimeos\MShop\Customer\Manager\Factory::createManager(\TestHelper::getContext(), 'Laravel');
 
-		$listManager = $manager->getSubManager( 'lists', 'Laravel' );
-		$this->object = $listManager->getSubManager( 'type', 'Laravel' );
+		$listManager = $manager->getSubManager('lists', 'Laravel');
+		$this->object = $listManager->getSubManager('type', 'Laravel');
 	}
 
 
@@ -37,20 +37,20 @@ 
 	 */
 	protected function tearDown()
 	{
-		unset( $this->object );
+		unset($this->object);
 	}
 
 
 	public function testCleanup()
 	{
-		$this->object->cleanup( array( -1 ) );
+		$this->object->cleanup(array( -1 ));
 	}
 
 
 	public function testCreateItem()
 	{
 		$item = $this->object->createItem();
-		$this->assertInstanceOf( '\\Aimeos\\MShop\\Common\\Item\\Type\\Iface', $item );
+		$this->assertInstanceOf('\\Aimeos\\MShop\\Common\\Item\\Type\\Iface', $item);
 	}
 
 
@@ -60,18 +60,18 @@ 
 		$search->setSlice(0, 1);
 		$results = $this->object->searchItems($search);
 
-		if( ( $expected = reset($results) ) === false ) {
-			throw new \RuntimeException( 'No list type item found' );
+		if (($expected = reset($results)) === false) {
+			throw new \RuntimeException('No list type item found');
 		}
 
-		$this->assertEquals( $expected, $this->object->getItem( $expected->getId() ) );
+		$this->assertEquals($expected, $this->object->getItem($expected->getId()));
 	}
 
 
 	public function testGetSubManager()
 	{
-		$this->setExpectedException( '\\Aimeos\\MShop\\Exception' );
-		$this->object->getSubManager( 'unknown' );
+		$this->setExpectedException('\\Aimeos\\MShop\\Exception');
+		$this->object->getSubManager('unknown');
 	}
 
 
@@ -81,48 +81,48 @@ 
 		$search->setSlice(0, 1);
 		$results = $this->object->searchItems($search);
 
-		if( ( $item = reset($results) ) === false ) {
-			throw new \RuntimeException( 'No type item found' );
+		if (($item = reset($results)) === false) {
+			throw new \RuntimeException('No type item found');
 		}
 
 		$item->setId(null);
-		$item->setCode( 'unitTestInit' );
-		$this->object->saveItem( $item );
-		$itemSaved = $this->object->getItem( $item->getId() );
+		$item->setCode('unitTestInit');
+		$this->object->saveItem($item);
+		$itemSaved = $this->object->getItem($item->getId());
 
 		$itemExp = clone $itemSaved;
-		$itemExp->setCode( 'unitTestSave' );
-		$this->object->saveItem( $itemExp );
-		$itemUpd = $this->object->getItem( $itemExp->getId() );
+		$itemExp->setCode('unitTestSave');
+		$this->object->saveItem($itemExp);
+		$itemUpd = $this->object->getItem($itemExp->getId());
 
-		$this->object->deleteItem( $itemSaved->getId() );
+		$this->object->deleteItem($itemSaved->getId());
 
 
-		$this->assertTrue( $item->getId() !== null );
-		$this->assertEquals( $item->getId(), $itemSaved->getId() );
-		$this->assertEquals( $item->getSiteId(), $itemSaved->getSiteId() );
-		$this->assertEquals( $item->getCode(), $itemSaved->getCode() );
-		$this->assertEquals( $item->getDomain(), $itemSaved->getDomain() );
-		$this->assertEquals( $item->getLabel(), $itemSaved->getLabel() );
-		$this->assertEquals( $item->getStatus(), $itemSaved->getStatus() );
+		$this->assertTrue($item->getId() !== null);
+		$this->assertEquals($item->getId(), $itemSaved->getId());
+		$this->assertEquals($item->getSiteId(), $itemSaved->getSiteId());
+		$this->assertEquals($item->getCode(), $itemSaved->getCode());
+		$this->assertEquals($item->getDomain(), $itemSaved->getDomain());
+		$this->assertEquals($item->getLabel(), $itemSaved->getLabel());
+		$this->assertEquals($item->getStatus(), $itemSaved->getStatus());
 
-		$this->assertEquals( $this->editor, $itemSaved->getEditor() );
-		$this->assertRegExp( '/\d{4}-\d{2}-\d{2} \d{2}:\d{2}:\d{2}/', $itemSaved->getTimeCreated() );
-		$this->assertRegExp( '/\d{4}-\d{2}-\d{2} \d{2}:\d{2}:\d{2}/', $itemSaved->getTimeModified() );
+		$this->assertEquals($this->editor, $itemSaved->getEditor());
+		$this->assertRegExp('/\d{4}-\d{2}-\d{2} \d{2}:\d{2}:\d{2}/', $itemSaved->getTimeCreated());
+		$this->assertRegExp('/\d{4}-\d{2}-\d{2} \d{2}:\d{2}:\d{2}/', $itemSaved->getTimeModified());
 
-		$this->assertEquals( $itemExp->getId(), $itemUpd->getId() );
-		$this->assertEquals( $itemExp->getSiteId(), $itemUpd->getSiteId() );
-		$this->assertEquals( $itemExp->getCode(), $itemUpd->getCode() );
-		$this->assertEquals( $itemExp->getDomain(), $itemUpd->getDomain() );
-		$this->assertEquals( $itemExp->getLabel(), $itemUpd->getLabel() );
-		$this->assertEquals( $itemExp->getStatus(), $itemUpd->getStatus() );
+		$this->assertEquals($itemExp->getId(), $itemUpd->getId());
+		$this->assertEquals($itemExp->getSiteId(), $itemUpd->getSiteId());
+		$this->assertEquals($itemExp->getCode(), $itemUpd->getCode());
+		$this->assertEquals($itemExp->getDomain(), $itemUpd->getDomain());
+		$this->assertEquals($itemExp->getLabel(), $itemUpd->getLabel());
+		$this->assertEquals($itemExp->getStatus(), $itemUpd->getStatus());
 
-		$this->assertEquals( $this->editor, $itemUpd->getEditor() );
-		$this->assertEquals( $itemExp->getTimeCreated(), $itemUpd->getTimeCreated() );
-		$this->assertRegExp( '/\d{4}-\d{2}-\d{2} \d{2}:\d{2}:\d{2}/', $itemUpd->getTimeModified() );
+		$this->assertEquals($this->editor, $itemUpd->getEditor());
+		$this->assertEquals($itemExp->getTimeCreated(), $itemUpd->getTimeCreated());
+		$this->assertRegExp('/\d{4}-\d{2}-\d{2} \d{2}:\d{2}:\d{2}/', $itemUpd->getTimeModified());
 
 		$this->setExpectedException('\\Aimeos\\MShop\\Exception');
-		$this->object->getItem( $itemSaved->getId() );
+		$this->object->getItem($itemSaved->getId());
 	}
 
 
@@ -132,25 +132,25 @@ 
 		$search = $this->object->createSearch();
 
 		$expr = array();
-		$expr[] = $search->compare( '!=', 'customer.lists.type.id', 0 );
-		$expr[] = $search->compare( '!=', 'customer.lists.type.siteid', null );
-		$expr[] = $search->compare( '==', 'customer.lists.type.code', 'default' );
-		$expr[] = $search->compare( '==', 'customer.lists.type.domain', 'text' );
-		$expr[] = $search->compare( '==', 'customer.lists.type.label', 'Standard' );
-		$expr[] = $search->compare( '==', 'customer.lists.type.status', 1 );
-		$expr[] = $search->compare( '>=', 'customer.lists.type.mtime', '1970-01-01 00:00:00' );
-		$expr[] = $search->compare( '>=', 'customer.lists.type.ctime', '1970-01-01 00:00:00' );
-		$expr[] = $search->compare( '==', 'customer.lists.type.editor', $this->editor );
-
-		$search->setConditions( $search->combine( '&&', $expr ) );
+		$expr[] = $search->compare('!=', 'customer.lists.type.id', 0);
+		$expr[] = $search->compare('!=', 'customer.lists.type.siteid', null);
+		$expr[] = $search->compare('==', 'customer.lists.type.code', 'default');
+		$expr[] = $search->compare('==', 'customer.lists.type.domain', 'text');
+		$expr[] = $search->compare('==', 'customer.lists.type.label', 'Standard');
+		$expr[] = $search->compare('==', 'customer.lists.type.status', 1);
+		$expr[] = $search->compare('>=', 'customer.lists.type.mtime', '1970-01-01 00:00:00');
+		$expr[] = $search->compare('>=', 'customer.lists.type.ctime', '1970-01-01 00:00:00');
+		$expr[] = $search->compare('==', 'customer.lists.type.editor', $this->editor);
+
+		$search->setConditions($search->combine('&&', $expr));
 		$search->setSlice(0, 1);
 
-		$results = $this->object->searchItems( $search, array(), $total );
-		$this->assertEquals( 1, count( $results ) );
-		$this->assertEquals( 1, $total );
+		$results = $this->object->searchItems($search, array(), $total);
+		$this->assertEquals(1, count($results));
+		$this->assertEquals(1, $total);
 
-		foreach($results as $itemId => $item) {
-			$this->assertEquals( $itemId, $item->getId() );
+		foreach ($results as $itemId => $item) {
+			$this->assertEquals($itemId, $item->getId());
 		}
 	}