aimeos / ai-flow

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

Spacing +9 added lines, -9 removed lines

@@ -25,14 +25,14 @@ 
 	 */
 	protected function setUp()
 	{
-		if( class_exists( '\\TYPO3\\Flow\\Session\\Session' ) === false ) {
-			$this->markTestSkipped( 'Class \\TYPO3\\Flow\\Session\\Session not found' );
+		if (class_exists('\\TYPO3\\Flow\\Session\\Session') === false) {
+			$this->markTestSkipped('Class \\TYPO3\\Flow\\Session\\Session not found');
 		}
 
 		$session = new \TYPO3\Flow\Session\TransientSession();
 		$session->start();
 
-		$this->object = new \Aimeos\MW\Session\Flow( $session );
+		$this->object = new \Aimeos\MW\Session\Flow($session);
 	}
 
 
@@ -44,26 +44,26 @@ 
 	 */
 	protected function tearDown()
 	{
-		unset( $this->object );
+		unset($this->object);
 	}
 
 
 	public function testGetDefault()
 	{
-		$this->assertEquals( null, $this->object->get( 'notexist' ) );
+		$this->assertEquals(null, $this->object->get('notexist'));
 	}
 
 
 	public function testGetSet()
 	{
-		$this->object->set( 'key', 'value' );
-		$this->assertEquals( 'value', $this->object->get( 'key' ) );
+		$this->object->set('key', 'value');
+		$this->assertEquals('value', $this->object->get('key'));
 	}
 
 
 	public function testGetSetArray()
 	{
-		$this->object->set( 'key', array( 'value' ) );
-		$this->assertEquals( array( 'value' ), $this->object->get( 'key' ) );
+		$this->object->set('key', array('value'));
+		$this->assertEquals(array('value'), $this->object->get('key'));
 	}
 }

lib/custom/tests/bootstrap.php

Spacing +6 added lines, -6 removed lines

@@ -9,17 +9,17 @@
  * Set error reporting to maximum
  */
 error_reporting( -1 );
-ini_set( 'display_errors', '1' );
+ini_set('display_errors', '1');
 
 
 /*
  * 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/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( dirname( __FILE__ ) ) ) );
-			self::$aimeos = new \Aimeos\Bootstrap( array( $extdir ), false );
+			$extdir = dirname(dirname(dirname(dirname(__FILE__))));
+			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-flow:unittest' );
+		$ctx->setEditor('ai-flow:unittest');
 
 		return $ctx;
 	}

lib/custom/src/MW/Logger/Flow.php

Spacing +10 added lines, -10 removed lines

@@ -29,7 +29,7 @@ 
 	 *
 	 * @param \TYPO3\Flow\Log\SystemLoggerInterface $logger Flow logger object
 	 */
-	public function __construct( \TYPO3\Flow\Log\SystemLoggerInterface $logger )
+	public function __construct(\TYPO3\Flow\Log\SystemLoggerInterface $logger)
 	{
 		$this->logger = $logger;
 	}
@@ -44,18 +44,18 @@ 
 	 * @throws \Aimeos\MW\Logger\Exception If an error occurs in Zend_Log
 	 * @see \Aimeos\MW\Logger\Base for available log level constants
 	 */
-	public function log( $message, $priority = \Aimeos\MW\Logger\Base::ERR, $facility = 'message' )
+	public function log($message, $priority = \Aimeos\MW\Logger\Base::ERR, $facility = 'message')
 	{
 		try
 		{
-			if( !is_scalar( $message ) ) {
-				$message = json_encode( $message );
+			if (!is_scalar($message)) {
+				$message = json_encode($message);
 			}
 
-			$this->logger->log( $message, $this->translatePriority( $priority ) );
+			$this->logger->log($message, $this->translatePriority($priority));
 		}
-		catch( \Exception $e )	{
-			throw new \Aimeos\MW\Logger\Exception( $e->getMessage(), $e->getCode(), $e );
+		catch (\Exception $e) {
+			throw new \Aimeos\MW\Logger\Exception($e->getMessage(), $e->getCode(), $e);
 		}
 	}
 
@@ -67,9 +67,9 @@ 
 	 * @return integer Log level from Monolog\Logger
 	 * @throws \Aimeos\MW\Logger\Exception If log level is unknown
 	 */
-	protected function translatePriority( $priority )
+	protected function translatePriority($priority)
 	{
-		switch( $priority )
+		switch ($priority)
 		{
 			case \Aimeos\MW\Logger\Base::EMERG:
 				return LOG_EMERG;
@@ -88,7 +88,7 @@ 
 			case \Aimeos\MW\Logger\Base::DEBUG:
 				return LOG_DEBUG;
 			default:
-				throw new \Aimeos\MW\Logger\Exception( 'Invalid log level' );
+				throw new \Aimeos\MW\Logger\Exception('Invalid log level');
 		}
 	}
 }
\ No newline at end of file

Braces +1 added lines, -2 removed lines

@@ -53,8 +53,7 @@
 			}
 
 			$this->logger->log( $message, $this->translatePriority( $priority ) );
-		}
-		catch( \Exception $e )	{
+		} catch( \Exception $e )	{
 			throw new \Aimeos\MW\Logger\Exception( $e->getMessage(), $e->getCode(), $e );
 		}
 	}

lib/custom/src/MW/Session/Flow.php

Spacing +6 added lines, -6 removed lines

@@ -28,7 +28,7 @@ 
 	 *
 	 * @param \TYPO3\Flow\Session\SessionInterface $object Flow session object
 	 */
-	public function __construct( \TYPO3\Flow\Session\SessionInterface $object )
+	public function __construct(\TYPO3\Flow\Session\SessionInterface $object)
 	{
 		$this->object = $object;
 	}
@@ -43,13 +43,13 @@ 
 	 * @param mixed $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( $this->object->hasKey( $name ) !== true ) {
+		if ($this->object->hasKey($name) !== true) {
 			return $default;
 		}
 
-		return $this->object->getData( $name );
+		return $this->object->getData($name);
 	}
 
 
@@ -62,8 +62,8 @@ 
 	 * @param mixed $value Value that should be associated with the given key
 	 * @return void
 	 */
-	public function set( $name, $value )
+	public function set($name, $value)
 	{
-		$this->object->putData( $name, $value );
+		$this->object->putData($name, $value);
 	}
 }

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

Spacing +33 added lines, -33 removed lines

@@ -31,9 +31,9 @@ 
 	 * @param array $config List of configuration values
 	 * @param \TYPO3\Flow\Cache\Frontend\FrontendInterface $cache TYPO3 cache object
 	 */
-	public function __construct( array $config, \TYPO3\Flow\Cache\Frontend\FrontendInterface $cache )
+	public function __construct(array $config, \TYPO3\Flow\Cache\Frontend\FrontendInterface $cache)
 	{
-		$this->prefix = ( isset( $config['siteid'] ) ? $config['siteid'] . '-' : '' );
+		$this->prefix = (isset($config['siteid']) ? $config['siteid'] . '-' : '');
 		$this->object = $cache;
 	}
 
@@ -45,9 +45,9 @@ 
 	 *
 	 * @param string $key Key string that identifies the single cache entry
 	 */
-	public function delete( $key )
+	public function delete($key)
 	{
-		$this->object->remove( $this->prefix . $key );
+		$this->object->remove($this->prefix . $key);
 	}
 
 
@@ -59,10 +59,10 @@ 
 	 * @param string[] $keys List of key strings that identify the cache entries
 	 * 	that should be removed
 	 */
-	public function deleteList( array $keys )
+	public function deleteList(array $keys)
 	{
-		foreach( $keys as $key ) {
-			$this->object->remove( $this->prefix . $key );
+		foreach ($keys as $key) {
+			$this->object->remove($this->prefix . $key);
 		}
 	}
 
@@ -75,10 +75,10 @@ 
 	 * @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)
 	{
-		foreach( $tags as $tag ) {
-			$this->object->flushByTag( $this->prefix . $tag );
+		foreach ($tags as $tag) {
+			$this->object->flushByTag($this->prefix . $tag);
 		}
 	}
 
@@ -90,8 +90,8 @@ 
 	 */
 	public function flush()
 	{
-		if( $this->prefix ) {
-			$this->object->flushByTag( $this->prefix . 'siteid' );
+		if ($this->prefix) {
+			$this->object->flushByTag($this->prefix . 'siteid');
 		} else {
 			$this->object->flush();
 		}
@@ -107,9 +107,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( $this->prefix . $name ) ) !== false ) {
+		if (($entry = $this->object->get($this->prefix . $name)) !== false) {
 			return $entry;
 		}
 
@@ -127,13 +127,13 @@ 
 	 * 	entries. If a cache entry doesn't exist, neither its key nor a value
 	 * 	will be in the result list
 	 */
-	public function getList( array $keys )
+	public function getList(array $keys)
 	{
 		$result = array();
 
-		foreach( $keys as $key )
+		foreach ($keys as $key)
 		{
-			if( ( $entry = $this->object->get( $this->prefix . $key ) ) !== false ) {
+			if (($entry = $this->object->get($this->prefix . $key)) !== false) {
 				$result[$key] = $entry;
 			}
 		}
@@ -152,17 +152,17 @@ 
 	 * 	entries. If a tag isn't associated to any cache entry, nothing is returned
 	 * 	for that tag
 	 */
-	public function getListByTags( array $tags )
+	public function getListByTags(array $tags)
 	{
 		$result = array();
-		$len = strlen( $this->prefix );
+		$len = strlen($this->prefix);
 
-		foreach( $tags as $tag )
+		foreach ($tags as $tag)
 		{
-			foreach( $this->object->getByTag( $this->prefix . $tag ) as $key => $value )
+			foreach ($this->object->getByTag($this->prefix . $tag) as $key => $value)
 			{
-				if( strncmp( $key, $this->prefix, $len ) === 0 ) {
-					$result[ substr( $key, $len ) ] = $value;
+				if (strncmp($key, $this->prefix, $len) === 0) {
+					$result[substr($key, $len)] = $value;
 				} else {
 					$result[$key] = $value;
 				}
@@ -186,21 +186,21 @@ 
 	 * @param string|null $expires Date/time string in "YYYY-MM-DD HH:mm:ss"
 	 * 	format when the cache entry expires
 	 */
-	public function set( $name, $value, array $tags = array(), $expires = null )
+	public function set($name, $value, array $tags = array(), $expires = null)
 	{
-		if( $expires !== null && ( $timestamp = strtotime( $expires ) ) !== false ) {
+		if ($expires !== null && ($timestamp = strtotime($expires)) !== false) {
 			$expires = $timestamp;
 		} else {
 			$expires = null;
 		}
 
-		$tagList = ( $this->prefix ? array( $this->prefix . 'siteid' ) : array() );
+		$tagList = ($this->prefix ? array($this->prefix . 'siteid') : array());
 
-		foreach( $tags as $tag ) {
+		foreach ($tags as $tag) {
 			$tagList[] = $this->prefix . $tag;
 		}
 
-		$this->object->set( $this->prefix . $name, $value, $tagList, $expires );
+		$this->object->set($this->prefix . $name, $value, $tagList, $expires);
 	}
 
 
@@ -217,14 +217,14 @@ 
 	 * 	to the key can either be a tag string or an array of tag strings
 	 * @param array $expires Associative list of key/datetime pairs.
 	 */
-	public function setList( array $pairs, array $tags = array(), array $expires = array() )
+	public function setList(array $pairs, array $tags = array(), array $expires = array())
 	{
-		foreach( $pairs as $key => $value )
+		foreach ($pairs as $key => $value)
 		{
-			$tagList = ( isset( $tags[$key] ) ? (array) $tags[$key] : array() );
-			$keyExpire = ( isset( $expires[$key] ) ? $expires[$key] : null );
+			$tagList = (isset($tags[$key]) ? (array) $tags[$key] : array());
+			$keyExpire = (isset($expires[$key]) ? $expires[$key] : null);
 
-			$this->set( $key, $value, $tagList, $keyExpire );
+			$this->set($key, $value, $tagList, $keyExpire);
 		}
 	}
 }

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

Spacing +13 added lines, -13 removed lines

@@ -31,9 +31,9 @@ 
 	 * @param \TYPO3\Flow\Mvc\Routing\UriBuilder $builder Flow URI builder object
 	 * @param array Associative list of fixed parameters that should be available for all routes
 	 */
-	public function __construct( $view, \TYPO3\Flow\Mvc\Routing\UriBuilder $builder, array $fixed )
+	public function __construct($view, \TYPO3\Flow\Mvc\Routing\UriBuilder $builder, array $fixed)
 	{
-		parent::__construct( $view );
+		parent::__construct($view);
 
 		$this->builder = $builder;
 		$this->fixed = $fixed;
@@ -51,19 +51,19 @@ 
 	 * @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())
 	{
-		$values = $this->getValues( $config );
+		$values = $this->getValues($config);
 
 		$this->builder
 			->reset()
-			->setSection( join( '/', $trailing ) )
-			->setCreateAbsoluteUri( $values['absoluteUri'] )
-			->setFormat( $values['format'] );
+			->setSection(join('/', $trailing))
+			->setCreateAbsoluteUri($values['absoluteUri'])
+			->setFormat($values['format']);
 
 		$params['node'] = $target;
 
-		return $this->builder->uriFor( $action, $params + $this->fixed, $controller, $values['package'], $values['subpackage'] );
+		return $this->builder->uriFor($action, $params + $this->fixed, $controller, $values['package'], $values['subpackage']);
 	}
 
 
@@ -73,7 +73,7 @@ 
 	 * @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(
 			'package' => null,
@@ -82,19 +82,19 @@ 
 			'format' => 'html',
 		);
 
-		if( isset( $config['package'] ) ) {
+		if (isset($config['package'])) {
 			$values['package'] = (string) $config['package'];
 		}
 
-		if( isset( $config['subpackage'] ) ) {
+		if (isset($config['subpackage'])) {
 			$values['subpackage'] = (string) $config['subpackage'];
 		}
 
-		if( isset( $config['absoluteUri'] ) ) {
+		if (isset($config['absoluteUri'])) {
 			$values['absoluteUri'] = (bool) $config['absoluteUri'];
 		}
 
-		if( isset( $config['format'] ) ) {
+		if (isset($config['format'])) {
 			$values['format'] = (string) $config['format'];
 		}
 

lib/custom/src/MAdmin/Cache/Proxy/Flow.php

Spacing +4 added lines, -4 removed lines

@@ -32,7 +32,7 @@ 
 	 * @param \Aimeos\MShop\Context\Item\Iface $context MShop context object
 	 * @param \TYPO3\Flow\Cache\Frontend\StringFrontend $cache Flow cache object
 	 */
-	public function __construct( \Aimeos\MShop\Context\Item\Iface $context, \TYPO3\Flow\Cache\Frontend\StringFrontend $cache )
+	public function __construct(\Aimeos\MShop\Context\Item\Iface $context, \TYPO3\Flow\Cache\Frontend\StringFrontend $cache)
 	{
 		$this->context = $context;
 		$this->cache = $cache;
@@ -46,11 +46,11 @@ 
 	 */
 	protected function getObject()
 	{
-		if( !isset( $this->object ) )
+		if (!isset($this->object))
 		{
 			$siteid = $this->context->getLocale()->getSiteId();
-			$conf = array( 'siteid' => $this->context->getConfig()->get( 'madmin/cache/prefix' ) . $siteid );
-			$this->object = \Aimeos\MW\Cache\Factory::createManager( 'Flow', $conf, $this->cache );
+			$conf = array('siteid' => $this->context->getConfig()->get('madmin/cache/prefix') . $siteid);
+			$this->object = \Aimeos\MW\Cache\Factory::createManager('Flow', $conf, $this->cache);
 		}
 
 		return $this->object;

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

Spacing +15 added lines, -15 removed lines

@@ -21,15 +21,15 @@ 
 	 */
 	protected function setUp()
 	{
-		if( !class_exists( '\TYPO3\Flow\Http\Request' ) ) {
-			$this->markTestSkipped( '\TYPO3\Flow\Http\Request is not available' );
+		if (!class_exists('\TYPO3\Flow\Http\Request')) {
+			$this->markTestSkipped('\TYPO3\Flow\Http\Request is not available');
 		}
 
-		$this->mock = $this->getMockBuilder( '\TYPO3\Flow\Http\Request' )
-			->setConstructorArgs( array( array(), array(), array(), array() ) )->getMock();
+		$this->mock = $this->getMockBuilder('\TYPO3\Flow\Http\Request')
+			->setConstructorArgs(array(array(), array(), array(), array()))->getMock();
 
 		$view = new \Aimeos\MW\View\Standard();
-		$this->object = new \Aimeos\MW\View\Helper\Request\Flow( $view, $this->mock, array() );
+		$this->object = new \Aimeos\MW\View\Helper\Request\Flow($view, $this->mock, array());
 	}
 
 
@@ -41,37 +41,37 @@ 
 	 */
 	protected function tearDown()
 	{
-		unset( $this->object, $this->mock );
+		unset($this->object, $this->mock);
 	}
 
 
 	public function testTransform()
 	{
-		$this->assertInstanceOf( '\\Aimeos\\MW\\View\\Helper\\Request\\Flow', $this->object->transform() );
+		$this->assertInstanceOf('\\Aimeos\\MW\\View\\Helper\\Request\\Flow', $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( 'getClientIpAddress' )
-			->will( $this->returnValue( '127.0.0.1' ) );
+		$this->mock->expects($this->once())->method('getClientIpAddress')
+			->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->assertEquals( null, $this->object->transform()->getTarget() );
+		$this->assertEquals(null, $this->object->transform()->getTarget());
 	}
 
 
@@ -79,6 +79,6 @@ 
 	{
 		$files = $this->object->transform()->getUploadedFiles();
 
-		$this->assertEquals( array(), $files );
+		$this->assertEquals(array(), $files);
 	}
 }