aimeos / ai-laravel

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);
 		}
 	}
 }

Doc Comments +2 added lines, -2 removed lines

@@ -64,7 +64,7 @@ 
 	 *
 	 * @inheritDoc
 	 *
-	 * @param \Traversable|array $keys List of key strings that identify the cache entries
+	 * @param string[] $keys List of key strings that identify the cache entries
 	 * 	that should be removed
 	 */
 	public function deleteMultiple( $keys )
@@ -114,7 +114,7 @@ 
 	 *
 	 * @inheritDoc
 	 *
-	 * @param \Traversable|array $keys List of key strings for the requested cache entries
+	 * @param string[] $keys List of key strings for the requested cache entries
 	 * @param mixed $default Default value to return for keys that do not exist
 	 * @return array Associative list of key/value pairs for the requested cache
 	 * 	entries. If a cache entry doesn't exist, neither its key nor a value

lib/custom/tests/TestHelper.php

Spacing +24 added lines, -24 removed lines

@@ -12,14 +12,14 @@ 
 
 		$includepaths = $aimeos->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];
@@ -28,54 +28,54 @@ 
 
 	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;
 	}
 
 
-	private static function createContext( $site )
+	private static function createContext($site)
 	{
 		$ctx = new \Aimeos\MShop\Context\Item\Standard();
 		$aimeos = self::getAimeos();
 
 
-		$paths = $aimeos->getConfigPaths( 'mysql' );
-		$paths[] = __DIR__ . DIRECTORY_SEPARATOR . 'config';
+		$paths = $aimeos->getConfigPaths('mysql');
+		$paths[] = __DIR__.DIRECTORY_SEPARATOR.'config';
 
-		$conf = new \Aimeos\MW\Config\PHPArray( [], $paths );
-		$ctx->setConfig( $conf );
+		$conf = new \Aimeos\MW\Config\PHPArray([], $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);
 
 
-		$i18n = new \Aimeos\MW\Translation\None( 'de' );
-		$ctx->setI18n( array( 'de' => $i18n ) );
+		$i18n = new \Aimeos\MW\Translation\None('de');
+		$ctx->setI18n(array('de' => $i18n));
 
 
 		$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-laravel:unittest' );
+		$ctx->setEditor('ai-laravel:unittest');
 
 		return $ctx;
 	}

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/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/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);
 	}
 }

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

Spacing +10 added lines, -10 removed lines

@@ -17,32 +17,32 @@
 
 	protected function setUp()
 	{
-		if( interface_exists( '\\Illuminate\\Session\\Store' ) === false ) {
-			$this->markTestSkipped( 'Class \\Illuminate\\Session\\Store not found' );
+		if (interface_exists('\\Illuminate\\Session\\Store') === false) {
+			$this->markTestSkipped('Class \\Illuminate\\Session\\Store not found');
 		}
 
-		$this->mock = $this->getMockBuilder( '\\Illuminate\\Session\\Store' )->getMock();
-		$this->object = new \Aimeos\MW\Session\Laravel5( $this->mock );
+		$this->mock = $this->getMockBuilder('\\Illuminate\\Session\\Store')->getMock();
+		$this->object = new \Aimeos\MW\Session\Laravel5($this->mock);
 	}
 
 
 	protected function tearDown()
 	{
-		unset( $this->object );
+		unset($this->object);
 	}
 
 
 	public function testGetDefault()
 	{
-		$this->mock->expects( $this->once() )->method( 'get' )->with( $this->equalTo( 'notexist' ) );
-		$this->object->get( 'notexist' );
+		$this->mock->expects($this->once())->method('get')->with($this->equalTo('notexist'));
+		$this->object->get('notexist');
 	}
 
 
 	public function testSet()
 	{
-		$this->mock->expects( $this->once() )->method( 'put' )
-			->with( $this->equalTo( 'key' ), $this->equalTo( 'value' ) );
-		$this->object->set( 'key', 'value' );
+		$this->mock->expects($this->once())->method('put')
+			->with($this->equalTo('key'), $this->equalTo('value'));
+		$this->object->set('key', 'value');
 	}
 }

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

Spacing +21 added lines, -21 removed lines

@@ -17,53 +17,53 @@
 
 	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);
 	}
 
 
 	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);
 	}
 }