aimeos / ai-laravel

lib/custom/src/MShop/Customer/Manager/Lists/Laravel.php

Doc Comments +1 added lines, -1 removed lines

@@ -249,7 +249,7 @@
 	 *
 	 * @param string $manager Name of the sub manager type in lower case
 	 * @param string|null $name Name of the implementation, will be from configuration (or Default) if null
-	 * @return mixed Manager for different extensions, e.g stock, tags, locations, etc.
+	 * @return \Aimeos\MShop\Common\Manager\Iface Manager for different extensions, e.g stock, tags, locations, etc.
 	 */
 	public function getSubManager( $manager, $name = null )
 	{

Spacing +9 added lines, -9 removed lines

@@ -25,7 +25,7 @@ 
 		'customer.lists.id'=> array(
 			'code'=>'customer.lists.id',
 			'internalcode'=>'lvuli."id"',
-			'internaldeps' => array( 'LEFT JOIN "users_list" AS lvuli ON ( lvu."id" = lvuli."parentid" )' ),
+			'internaldeps' => array('LEFT JOIN "users_list" AS lvuli ON ( lvu."id" = lvuli."parentid" )'),
 			'label'=>'Customer list ID',
 			'type'=> 'integer',
 			'internaltype'=> \Aimeos\MW\DB\Statement\Base::PARAM_INT,
@@ -133,14 +133,14 @@ 
 	 *
 	 * @param array $siteids List of IDs for sites whose entries should be deleted
 	 */
-	public function cleanup( array $siteids )
+	public function cleanup(array $siteids)
 	{
 		$path = 'mshop/customer/manager/lists/submanagers';
-		foreach( $this->getContext()->getConfig()->get( $path, array( 'type' ) ) as $domain ) {
-			$this->getObject()->getSubManager( $domain )->cleanup( $siteids );
+		foreach ($this->getContext()->getConfig()->get($path, array('type')) as $domain) {
+			$this->getObject()->getSubManager($domain)->cleanup($siteids);
 		}
 
-		$this->cleanupBase( $siteids, 'mshop/customer/manager/lists/laravel/delete' );
+		$this->cleanupBase($siteids, 'mshop/customer/manager/lists/laravel/delete');
 	}
 
 
@@ -150,11 +150,11 @@ 
 	 * @param boolean $withsub Return also attributes of sub-managers if true
 	 * @return array List of attribute items implementing \Aimeos\MW\Criteria\Attribute\Iface
 	 */
-	public function getSearchAttributes( $withsub = true )
+	public function getSearchAttributes($withsub = true)
 	{
 		$path = 'mshop/customer/manager/lists/submanagers';
 
-		return $this->getSearchAttributesBase( $this->searchConfig, $path, array( 'type' ), $withsub );
+		return $this->getSearchAttributesBase($this->searchConfig, $path, array('type'), $withsub);
 	}
 
 
@@ -165,9 +165,9 @@ 
 	 * @param string|null $name Name of the implementation, will be from configuration (or Default) if null
 	 * @return mixed Manager for different extensions, e.g stock, tags, locations, etc.
 	 */
-	public function getSubManager( $manager, $name = null )
+	public function getSubManager($manager, $name = null)
 	{
-		return $this->getSubManagerBase( 'customer', 'lists/' . $manager, ( $name === null ? 'Laravel' : $name ) );
+		return $this->getSubManagerBase('customer', 'lists/'.$manager, ($name === null ? 'Laravel' : $name));
 	}
 
 

lib/custom/src/MShop/Customer/Manager/Lists/Type/Laravel.php

Doc Comments +1 added lines, -1 removed lines

@@ -249,7 +249,7 @@
 	 *
 	 * @param string $manager Name of the sub manager type in lower case
 	 * @param string|null $name Name of the implementation, will be from configuration (or Default) if null
-	 * @return mixed Manager for different extensions, e.g stock, tags, locations, etc.
+	 * @return \Aimeos\MShop\Common\Manager\Iface Manager for different extensions, e.g stock, tags, locations, etc.
 	 */
 	public function getSubManager( $manager, $name = null )
 	{

Spacing +9 added lines, -9 removed lines

@@ -25,7 +25,7 @@ 
 		'customer.lists.type.id' => array(
 			'code'=>'customer.lists.type.id',
 			'internalcode'=>'lvulity."id"',
-			'internaldeps'=>array( 'LEFT JOIN "users_list_type" AS lvulity ON ( lvuli."typeid" = lvulity."id" )' ),
+			'internaldeps'=>array('LEFT JOIN "users_list_type" AS lvulity ON ( lvuli."typeid" = lvulity."id" )'),
 			'label'=>'Customer list type ID',
 			'type'=> 'integer',
 			'internaltype' => \Aimeos\MW\DB\Statement\Base::PARAM_INT,
@@ -96,14 +96,14 @@ 
 	 *
 	 * @param array $siteids List of IDs for sites whose entries should be deleted
 	 */
-	public function cleanup( array $siteids )
+	public function cleanup(array $siteids)
 	{
 		$path = 'mshop/customer/manager/lists/type/submanagers';
-		foreach( $this->getContext()->getConfig()->get( $path, [] ) as $domain ) {
-			$this->getObject()->getSubManager( $domain )->cleanup( $siteids );
+		foreach ($this->getContext()->getConfig()->get($path, []) as $domain) {
+			$this->getObject()->getSubManager($domain)->cleanup($siteids);
 		}
 
-		$this->cleanupBase( $siteids, 'mshop/customer/manager/lists/type/laravel/delete' );
+		$this->cleanupBase($siteids, 'mshop/customer/manager/lists/type/laravel/delete');
 	}
 
 
@@ -113,11 +113,11 @@ 
 	 * @param boolean $withsub Return also attributes of sub-managers if true
 	 * @return array List of attribute items implementing \Aimeos\MW\Criteria\Attribute\Iface
 	 */
-	public function getSearchAttributes( $withsub = true )
+	public function getSearchAttributes($withsub = true)
 	{
 		$path = 'mshop/customer/manager/lists/type/submanagers';
 
-		return $this->getSearchAttributesBase( $this->searchConfig, $path, [], $withsub );
+		return $this->getSearchAttributesBase($this->searchConfig, $path, [], $withsub);
 	}
 
 
@@ -128,9 +128,9 @@ 
 	 * @param string|null $name Name of the implementation, will be from configuration (or Default) if null
 	 * @return mixed Manager for different extensions, e.g stock, tags, locations, etc.
 	 */
-	public function getSubManager( $manager, $name = null )
+	public function getSubManager($manager, $name = null)
 	{
-		return $this->getSubManagerBase( 'customer', 'lists/type/' . $manager, ( $name === null ? 'Laravel' : $name ) );
+		return $this->getSubManagerBase('customer', 'lists/type/'.$manager, ($name === null ? 'Laravel' : $name));
 	}
 
 

lib/custom/src/MW/Cache/Laravel5.php

Spacing +24 added lines, -24 removed lines

@@ -29,7 +29,7 @@ 
 	 *
 	 * @param \Illuminate\Contracts\Cache\Store $cache Laravel cache object
 	 */
-	public function __construct( \Illuminate\Contracts\Cache\Store $cache )
+	public function __construct(\Illuminate\Contracts\Cache\Store $cache)
 	{
 		$this->object = $cache;
 	}
@@ -53,9 +53,9 @@ 
 	 *
 	 * @param string $key Key string that identifies the single cache entry
 	 */
-	public function delete( $key )
+	public function delete($key)
 	{
-		$this->object->forget( $key );
+		$this->object->forget($key);
 	}
 
 
@@ -67,10 +67,10 @@ 
 	 * @param \Traversable|array $keys List of key strings that identify the cache entries
 	 * 	that should be removed
 	 */
-	public function deleteMultiple( $keys )
+	public function deleteMultiple($keys)
 	{
-		foreach( $keys as $key ) {
-			$this->object->forget( $key );
+		foreach ($keys as $key) {
+			$this->object->forget($key);
 		}
 	}
 
@@ -83,7 +83,7 @@ 
 	 * @param string[] $tags List of tag strings that are associated to one or more
 	 * 	cache entries that should be removed
 	 */
-	public function deleteByTags( array $tags )
+	public function deleteByTags(array $tags)
 	{
 		// $this->object->tags( $tag )->flush();
 		$this->object->flush();
@@ -99,9 +99,9 @@ 
 	 * @param string $default Value returned if requested key isn't found
 	 * @return mixed Value associated to the requested key
 	 */
-	public function get( $name, $default = null )
+	public function get($name, $default = null)
 	{
-		if( ( $entry = $this->object->get( $name ) ) !== null ) {
+		if (($entry = $this->object->get($name)) !== null) {
 			return $entry;
 		}
 
@@ -120,13 +120,13 @@ 
 	 * 	entries. If a cache entry doesn't exist, neither its key nor a value
 	 * 	will be in the result list
 	 */
-	public function getMultiple( $keys, $default = null )
+	public function getMultiple($keys, $default = null)
 	{
 		$result = [];
 
-		foreach( $keys as $key )
+		foreach ($keys as $key)
 		{
-			if( ( $entry = $this->object->get( $key ) ) !== false ) {
+			if (($entry = $this->object->get($key)) !== false) {
 				$result[$key] = $entry;
 			} else {
 				$result[$key] = $default;
@@ -147,7 +147,7 @@ 
 	 * 	entries. If a tag isn't associated to any cache entry, nothing is returned
 	 * 	for that tag
 	 */
-	public function getMultipleByTags( array $tags )
+	public function getMultipleByTags(array $tags)
 	{
 		return [];
 	}
@@ -165,14 +165,14 @@ 
 	 * @param array $tags List of tag strings that should be assoicated to the
 	 * 	given value in the cache
 	 */
-	public function set( $key, $value, $expires = null, array $tags = [] )
+	public function set($key, $value, $expires = null, array $tags = [])
 	{
-		if( is_string( $expires ) ) {
-			$this->object->put( $key, $value, (int) ( date_create( $expires )->getTimestamp() - time() ) / 60 );
-		} elseif( is_int( $expires ) ) {
-			$this->object->put( $key, $value, (int) $expires / 60 );
+		if (is_string($expires)) {
+			$this->object->put($key, $value, (int) (date_create($expires)->getTimestamp() - time())/60);
+		} elseif (is_int($expires)) {
+			$this->object->put($key, $value, (int) $expires/60);
 		} else {
-			$this->object->forever( $key, $value );
+			$this->object->forever($key, $value);
 		}
 	}
 
@@ -191,14 +191,14 @@ 
 	 *  should be associated to the values identified by their key. The value
 	 *  associated to the key can either be a tag string or an array of tag strings
 	 */
-	public function setMultiple( $pairs, $expires = null, array $tags = [] )
+	public function setMultiple($pairs, $expires = null, array $tags = [])
 	{
-		foreach( $pairs as $key => $value )
+		foreach ($pairs as $key => $value)
 		{
-			$tagList = ( isset( $tags[$key] ) ? (array) $tags[$key] : [] );
-			$keyExpire = ( isset( $expires[$key] ) ? $expires[$key] : null );
+			$tagList = (isset($tags[$key]) ? (array) $tags[$key] : []);
+			$keyExpire = (isset($expires[$key]) ? $expires[$key] : null);
 
-			$this->set( $key, $value, $keyExpire, $tagList );
+			$this->set($key, $value, $keyExpire, $tagList);
 		}
 	}
 }

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

Spacing +33 added lines, -33 removed lines

@@ -17,107 +17,107 @@
 
 	protected function setUp()
 	{
-		if( interface_exists( '\\Illuminate\\Contracts\\Cache\\Store' ) === false ) {
-			$this->markTestSkipped( 'Class \\Illuminate\\Contracts\\Cache\\Store not found' );
+		if (interface_exists('\\Illuminate\\Contracts\\Cache\\Store') === false) {
+			$this->markTestSkipped('Class \\Illuminate\\Contracts\\Cache\\Store not found');
 		}
 
-		$this->mock = $this->getMockBuilder( '\\Illuminate\\Contracts\\Cache\\Store' )->getMock();
-		$this->object = new \Aimeos\MW\Cache\Laravel5( $this->mock );
+		$this->mock = $this->getMockBuilder('\\Illuminate\\Contracts\\Cache\\Store')->getMock();
+		$this->object = new \Aimeos\MW\Cache\Laravel5($this->mock);
 	}
 
 
 	protected function tearDown()
 	{
-		unset( $this->mock, $this->object );
+		unset($this->mock, $this->object);
 	}
 
 
 	public function testDelete()
 	{
-		$this->mock->expects( $this->once() )->method( 'forget' )->with( $this->equalTo( 'key' ) );
-		$this->object->delete( 'key' );
+		$this->mock->expects($this->once())->method('forget')->with($this->equalTo('key'));
+		$this->object->delete('key');
 	}
 
 
 	public function testDeleteMultiple()
 	{
-		$this->mock->expects( $this->exactly( 2 ) )->method( 'forget' )->with( $this->equalTo( 'key' ) );
-		$this->object->deleteMultiple( array( 'key', 'key' ) );
+		$this->mock->expects($this->exactly(2))->method('forget')->with($this->equalTo('key'));
+		$this->object->deleteMultiple(array('key', 'key'));
 	}
 
 
 	public function testDeleteByTags()
 	{
-		$this->mock->expects( $this->once() )->method( 'flush' );
-		$this->object->deleteByTags( array( 'tag', 'tag' ) );
+		$this->mock->expects($this->once())->method('flush');
+		$this->object->deleteByTags(array('tag', 'tag'));
 	}
 
 
 	public function testClear()
 	{
-		$this->mock->expects( $this->once() )->method( 'flush' );
+		$this->mock->expects($this->once())->method('flush');
 		$this->object->clear();
 	}
 
 
 	public function testGet()
 	{
-		$this->mock->expects( $this->once() )->method( 'get' )
-			->with( $this->equalTo( 'key' ) )->will( $this->returnValue( 'value' ) );
+		$this->mock->expects($this->once())->method('get')
+			->with($this->equalTo('key'))->will($this->returnValue('value'));
 
-		$this->assertEquals( 'value', $this->object->get( 'key', 'default' ) );
+		$this->assertEquals('value', $this->object->get('key', 'default'));
 	}
 
 
 	public function testGetDefault()
 	{
-		$this->mock->expects( $this->once() )->method( 'get' )
-			->with( $this->equalTo( 'key' ) )->will( $this->returnValue( null ) );
+		$this->mock->expects($this->once())->method('get')
+			->with($this->equalTo('key'))->will($this->returnValue(null));
 
-		$this->assertEquals( 'default', $this->object->get( 'key', 'default' ) );
+		$this->assertEquals('default', $this->object->get('key', 'default'));
 	}
 
 
 	public function testGetMultiple()
 	{
-		$this->mock->expects( $this->exactly( 2 ) )->method( 'get' )
-			->will( $this->returnValue( 'value' ) );
+		$this->mock->expects($this->exactly(2))->method('get')
+			->will($this->returnValue('value'));
 
-		$expected = array( 'key1' => 'value', 'key2' => 'value' );
-		$this->assertEquals( $expected, $this->object->getMultiple( array( 'key1', 'key2' ) ) );
+		$expected = array('key1' => 'value', 'key2' => 'value');
+		$this->assertEquals($expected, $this->object->getMultiple(array('key1', 'key2')));
 	}
 
 
 	public function testGetMultipleByTags()
 	{
-		$this->assertEquals( [], $this->object->getMultipleByTags( array( 'key', 'key' ) ) );
+		$this->assertEquals([], $this->object->getMultipleByTags(array('key', 'key')));
 	}
 
 
 	public function testSet()
 	{
-		$this->mock->expects( $this->once() )->method( 'put' )
-			->with( $this->equalTo( 'key' ), $this->equalTo( 'value' ), $this->greaterThan( 0 ) );
+		$this->mock->expects($this->once())->method('put')
+			->with($this->equalTo('key'), $this->equalTo('value'), $this->greaterThan(0));
 
-		$this->object->set( 'key', 'value', '2100-01-01 00:00:00', array( 'tag' ) );
+		$this->object->set('key', 'value', '2100-01-01 00:00:00', array('tag'));
 	}
 
 
 	public function testSetForever()
 	{
-		$this->mock->expects( $this->once() )->method( 'forever' )
-			->with( $this->equalTo( 'key' ), $this->equalTo( 'value' ) );
+		$this->mock->expects($this->once())->method('forever')
+			->with($this->equalTo('key'), $this->equalTo('value'));
 
-		$this->object->set( 'key', 'value', null, array( 'tag' ) );
+		$this->object->set('key', 'value', null, array('tag'));
 	}
 
 
 	public function testSetMultiple()
 	{
-		$this->mock->expects( $this->once() )->method( 'put' )
-			->with( $this->equalTo( 'key' ), $this->equalTo( 'value' ), $this->greaterThan( 0 ) );
+		$this->mock->expects($this->once())->method('put')
+			->with($this->equalTo('key'), $this->equalTo('value'), $this->greaterThan(0));
 
-		$expires = array( 'key' => '2100-01-01 00:00:00' );
-		$this->object->setMultiple( array( 'key' => 'value' ), $expires, array( 'key' => array( 'tag' ) ) );
+		$expires = array('key' => '2100-01-01 00:00:00');
+		$this->object->setMultiple(array('key' => 'value'), $expires, array('key' => array('tag')));
 	}
 }

lib/custom/tests/MW/View/Engine/BladeTest.php

Spacing +15 added lines, -15 removed lines

@@ -16,41 +16,41 @@
 
 	protected function setUp()
 	{
-		if( !class_exists( '\Illuminate\View\Factory' ) ) {
-			$this->markTestSkipped( '\Illuminate\View\Factory is not available' );
+		if (!class_exists('\Illuminate\View\Factory')) {
+			$this->markTestSkipped('\Illuminate\View\Factory is not available');
 		}
 
-		$this->mock = $this->getMockBuilder( '\Illuminate\View\Factory' )
-			->setMethods( array( 'file' ) )
+		$this->mock = $this->getMockBuilder('\Illuminate\View\Factory')
+			->setMethods(array('file'))
 			->disableOriginalConstructor()
 			->getMock();
 
-		$this->object = new \Aimeos\MW\View\Engine\Blade( $this->mock );
+		$this->object = new \Aimeos\MW\View\Engine\Blade($this->mock);
 	}
 
 
 	protected function tearDown()
 	{
-		unset( $this->object, $this->mock );
+		unset($this->object, $this->mock);
 	}
 
 
 	public function testRender()
 	{
-		$v = new \Aimeos\MW\View\Standard( [] );
+		$v = new \Aimeos\MW\View\Standard([]);
 
-		$view = $this->getMockBuilder( '\Illuminate\View\View' )
-			->setMethods( array( 'render' ) )
+		$view = $this->getMockBuilder('\Illuminate\View\View')
+			->setMethods(array('render'))
 			->disableOriginalConstructor()
 			->getMock();
 
-		$view->expects( $this->once() )->method( 'render' )
-			->will( $this->returnValue( 'test' ) );
+		$view->expects($this->once())->method('render')
+			->will($this->returnValue('test'));
 
-		$this->mock->expects( $this->once() )->method( 'file' )
-			->will( $this->returnValue( $view) );
+		$this->mock->expects($this->once())->method('file')
+			->will($this->returnValue($view));
 
-		$result = $this->object->render( $v, 'filepath', array( 'key' => 'value' ) );
-		$this->assertEquals( 'test', $result );
+		$result = $this->object->render($v, 'filepath', array('key' => 'value'));
+		$this->assertEquals('test', $result);
 	}
 }

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( [], [], [], [], [], $param, 'Content' );
+		$param = array('HTTP_HOST' => 'localhost', 'REMOTE_ADDR' => '127.0.0.1');
+		$request = new \Illuminate\Http\Request([], [], [], [], [], $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/Filesystem/Manager/LaravelTest.php

Spacing +16 added lines, -16 removed lines

@@ -12,50 +12,50 @@
 
 	protected function setUp()
 	{
-		if( !class_exists( '\Illuminate\Filesystem\FilesystemManager' ) ) {
-			$this->markTestSkipped( 'Install the Laravel framework first' );
+		if (!class_exists('\Illuminate\Filesystem\FilesystemManager')) {
+			$this->markTestSkipped('Install the Laravel framework first');
 		}
 
-		$this->storage = $this->getMockBuilder( '\Illuminate\Filesystem\FilesystemManager' )
-			->setMethods( array( 'get' ) )
+		$this->storage = $this->getMockBuilder('\Illuminate\Filesystem\FilesystemManager')
+			->setMethods(array('get'))
 			->disableOriginalConstructor()
 			->getMock();
 
-		$this->config = new \Aimeos\MW\Config\Decorator\Memory( new \Aimeos\MW\Config\PHPArray( [], [] ) );
-		$this->object = new \Aimeos\MW\Filesystem\Manager\Laravel( $this->storage, $this->config, sys_get_temp_dir() );
+		$this->config = new \Aimeos\MW\Config\Decorator\Memory(new \Aimeos\MW\Config\PHPArray([], []));
+		$this->object = new \Aimeos\MW\Filesystem\Manager\Laravel($this->storage, $this->config, sys_get_temp_dir());
 	}
 
 
 	protected function tearDown()
 	{
-		unset( $this->config, $this->object, $this->storage );
+		unset($this->config, $this->object, $this->storage);
 	}
 
 
 	public function testGet()
 	{
-		$fs = $this->getMockBuilder( 'Illuminate\Contracts\Filesystem\Filesystem' )
+		$fs = $this->getMockBuilder('Illuminate\Contracts\Filesystem\Filesystem')
 			->disableOriginalConstructor()
 			->getMock();
 
-		$this->storage->expects( $this->once() )->method( 'get' )
-			->will( $this->returnValue( $fs ) );
+		$this->storage->expects($this->once())->method('get')
+			->will($this->returnValue($fs));
 
-		$this->config->set( 'resource/fs-media', 'local' );
-		$this->assertInstanceof( 'Aimeos\MW\Filesystem\Iface', $this->object->get( 'fs-media' ) );
+		$this->config->set('resource/fs-media', 'local');
+		$this->assertInstanceof('Aimeos\MW\Filesystem\Iface', $this->object->get('fs-media'));
 	}
 
 
 	public function testGetFallback()
 	{
-		$this->config->set( 'resource/fs', array( 'adapter' => 'Standard', 'basedir' => __DIR__ ) );
-		$this->assertInstanceof( 'Aimeos\MW\Filesystem\Iface', $this->object->get( 'fs-media' ) );
+		$this->config->set('resource/fs', array('adapter' => 'Standard', 'basedir' => __DIR__));
+		$this->assertInstanceof('Aimeos\MW\Filesystem\Iface', $this->object->get('fs-media'));
 	}
 
 
 	public function testGetException()
 	{
-		$this->setExpectedException( 'Aimeos\MW\Filesystem\Exception' );
-		$this->object->get( 'fs-media' );
+		$this->setExpectedException('Aimeos\MW\Filesystem\Exception');
+		$this->object->get('fs-media');
 	}
 }

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

Spacing +9 added lines, -9 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,13 +52,13 @@ 
 	 * @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 $trailing = [], array $config = [] )
+	public function transform($target = null, $controller = null, $action = null, array $params = [], array $trailing = [], array $config = [])
 	{
-		$params = $this->sanitize( $params );
-		$values = $this->getValues( $config );
-		$fragment = ( !empty( $trailing ) ? '#' . implode( '/', $trailing ) : '' );
+		$params = $this->sanitize($params);
+		$values = $this->getValues($config);
+		$fragment = (!empty($trailing) ? '#'.implode('/', $trailing) : '');
 
-		return $this->builder->route( $target, $params + $this->fixed, $values['absoluteUri'] ) . $fragment;
+		return $this->builder->route($target, $params + $this->fixed, $values['absoluteUri']).$fragment;
 	}
 
 
@@ -68,13 +68,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/View/Helper/Url/Laravel5Test.php

Spacing +12 added lines, -12 removed lines

@@ -17,37 +17,37 @@
 
 	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, [] );
+		$this->mock = $this->getMockBuilder('\\Illuminate\\Contracts\\Routing\\UrlGenerator')->getMock();
+		$this->object = new \Aimeos\MW\View\Helper\Url\Laravel5($view, $this->mock, []);
 	}
 
 
 	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( [] ), $this->equalTo( true ) );
+		$this->mock->expects($this->once())->method('route')
+			->with($this->equalTo('route'), $this->equalTo([]), $this->equalTo(true));
 
-		$options = array( 'absoluteUri' => true );
-		$this->object->transform( 'route', 'catalog', 'lists', [], [], $options );
+		$options = array('absoluteUri' => true);
+		$this->object->transform('route', 'catalog', 'lists', [], [], $options);
 	}
 }