aimeos / ai-flow

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

Spacing +35 added lines, -35 removed lines

@@ -31,9 +31,9 @@ 
 	 * @param array $config List of configuration values
 	 * @param \Neos\Cache\Frontend\FrontendInterface $cache Neos cache object
 	 */
-	public function __construct( array $config, \Neos\Cache\Frontend\FrontendInterface $cache )
+	public function __construct(array $config, \Neos\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 \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->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 clear()
 	{
-		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;
 		}
 
@@ -129,13 +129,13 @@ 
 	 * 	will be in the result list
 	 * @throws \Aimeos\MW\Cache\Exception If the cache server doesn't respond
 	 */
-	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( $this->prefix . $key ) ) !== false ) {
+			if (($entry = $this->object->get($this->prefix.$key)) !== false) {
 				$result[$key] = $entry;
 			} else {
 				$result[$key] = $default;
@@ -156,17 +156,17 @@ 
 	 * 	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)
 	{
 		$result = [];
-		$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;
 				}
@@ -190,19 +190,19 @@ 
 	 * @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 ) ) {
-			$expires = date_create( $expires )->getTimestamp() - time();
+		if (is_string($expires)) {
+			$expires = date_create($expires)->getTimestamp() - time();
 		}
 
-		$tagList = ( $this->prefix ? array( $this->prefix . 'siteid' ) : [] );
+		$tagList = ($this->prefix ? array($this->prefix.'siteid') : []);
 
-		foreach( $tags as $tag ) {
-			$tagList[] = $this->prefix . $tag;
+		foreach ($tags as $tag) {
+			$tagList[] = $this->prefix.$tag;
 		}
 
-		$this->object->set( $this->prefix . $key, $value, $tagList, $expires );
+		$this->object->set($this->prefix.$key, $value, $tagList, $expires);
 	}
 
 
@@ -222,14 +222,14 @@ 
 	 * @return null
 	 * @throws \Aimeos\MW\Cache\Exception If the cache server doesn't respond
 	 */
-	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] : $expires );
+			$tagList = (isset($tags[$key]) ? (array) $tags[$key] : []);
+			$keyExpire = (isset($expires[$key]) ? $expires[$key] : $expires);
 
-			$this->set( $key, $value, $keyExpire, $tagList );
+			$this->set($key, $value, $keyExpire, $tagList);
 		}
 	}
 }

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

Spacing +27 added lines, -27 removed lines

@@ -17,18 +17,18 @@ 
 
 	protected function setUp()
 	{
-		if( !class_exists( '\\Neos\\Flow\\Mvc\\Routing\\UriBuilder' ) ) {
-			$this->markTestSkipped( '\\Neos\\Flow\\Mvc\\Routing\\UriBuilder is not available' );
+		if (!class_exists('\\Neos\\Flow\\Mvc\\Routing\\UriBuilder')) {
+			$this->markTestSkipped('\\Neos\\Flow\\Mvc\\Routing\\UriBuilder is not available');
 		}
 
 		$view = new \Aimeos\MW\View\Standard();
 
-		$this->buildMock = $this->getMockbuilder( '\\Neos\\Flow\\Mvc\\Routing\\UriBuilder' )
-			->setMethods( array( 'uriFor' ) )
+		$this->buildMock = $this->getMockbuilder('\\Neos\\Flow\\Mvc\\Routing\\UriBuilder')
+			->setMethods(array('uriFor'))
 			->disableOriginalConstructor()
 			->getMock();
 
-		$this->object = new \Aimeos\MW\View\Helper\Url\Flow( $view, $this->buildMock, array( 'site' => 'unittest' ) );
+		$this->object = new \Aimeos\MW\View\Helper\Url\Flow($view, $this->buildMock, array('site' => 'unittest'));
 	}
 
 
@@ -40,60 +40,60 @@ 
 
 	public function testTransform()
 	{
-		$this->buildMock->expects( $this->once() )->method( 'uriFor' )
-			->will( $this->returnValue( '/shop/catalog/lists') );
+		$this->buildMock->expects($this->once())->method('uriFor')
+			->will($this->returnValue('/shop/catalog/lists'));
 
-		$this->assertEquals( '/shop/catalog/lists', $this->object->transform( 'shop', 'catalog', 'lists' ) );
+		$this->assertEquals('/shop/catalog/lists', $this->object->transform('shop', 'catalog', 'lists'));
 	}
 
 
 	public function testTransformSlashes()
 	{
-		$this->buildMock->expects( $this->once() )->method( 'uriFor' )
-			->will( $this->returnValue( '/shop/catalog/lists?test=a%2Fb') );
+		$this->buildMock->expects($this->once())->method('uriFor')
+			->will($this->returnValue('/shop/catalog/lists?test=a%2Fb'));
 
-		$this->assertEquals( '/shop/catalog/lists?test=a%2Fb', $this->object->transform( 'shop', 'catalog', 'lists', array( 'test' => 'a/b' ) ) );
+		$this->assertEquals('/shop/catalog/lists?test=a%2Fb', $this->object->transform('shop', 'catalog', 'lists', array('test' => 'a/b')));
 	}
 
 
 	public function testTransformArrays()
 	{
-		$this->buildMock->expects( $this->once() )->method( 'uriFor' )
-			->will( $this->returnValue( '/shop/catalog/list?test%5B0%5D=a&test%5B1%5D=b') );
+		$this->buildMock->expects($this->once())->method('uriFor')
+			->will($this->returnValue('/shop/catalog/list?test%5B0%5D=a&test%5B1%5D=b'));
 
-		$this->assertEquals( '/shop/catalog/list?test%5B0%5D=a&test%5B1%5D=b', $this->object->transform( 'shop', 'catalog', 'lists', array( 'test' => array( 'a', 'b' ) ) ) );
+		$this->assertEquals('/shop/catalog/list?test%5B0%5D=a&test%5B1%5D=b', $this->object->transform('shop', 'catalog', 'lists', array('test' => array('a', 'b'))));
 	}
 
 
 	public function testTransformTrailing()
 	{
-		$this->buildMock->expects( $this->once() )->method( 'uriFor' )
-			->will( $this->returnValue( '/shop/catalog/lists#a/b') );
+		$this->buildMock->expects($this->once())->method('uriFor')
+			->will($this->returnValue('/shop/catalog/lists#a/b'));
 
-		$this->assertEquals( '/shop/catalog/lists#a/b', $this->object->transform( 'shop', 'catalog', 'lists', [], array( 'a', 'b' ) ) );
+		$this->assertEquals('/shop/catalog/lists#a/b', $this->object->transform('shop', 'catalog', 'lists', [], array('a', 'b')));
 	}
 
 
 	public function testTransformAbsolute()
 	{
-		$this->buildMock->expects( $this->once() )->method( 'uriFor' )
-			->will( $this->returnValue( 'http://localhost/shop/catalog/lists') );
+		$this->buildMock->expects($this->once())->method('uriFor')
+			->will($this->returnValue('http://localhost/shop/catalog/lists'));
 
-		$options = array( 'absoluteUri' => true );
-		$result = $this->object->transform( 'shop', 'catalog', 'lists', [], [], $options );
+		$options = array('absoluteUri' => true);
+		$result = $this->object->transform('shop', 'catalog', 'lists', [], [], $options);
 
-		$this->assertEquals( 'http://localhost/shop/catalog/lists', $result );
+		$this->assertEquals('http://localhost/shop/catalog/lists', $result);
 	}
 
 
 	public function testTransformConfig()
 	{
-		$this->buildMock->expects( $this->once() )->method( 'uriFor' )
-			->will( $this->returnValue( '/shop/catalog/lists') );
+		$this->buildMock->expects($this->once())->method('uriFor')
+			->will($this->returnValue('/shop/catalog/lists'));
 
-		$options = array( 'package' => 'test', 'subpackage' => 'subtest', 'format' => 'fmt' );
-		$result = $this->object->transform( 'shop', 'catalog', 'lists', [], [], $options );
+		$options = array('package' => 'test', 'subpackage' => 'subtest', 'format' => 'fmt');
+		$result = $this->object->transform('shop', 'catalog', 'lists', [], [], $options);
 
-		$this->assertEquals( '/shop/catalog/lists', $result );
+		$this->assertEquals('/shop/catalog/lists', $result);
 	}
 }

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

Spacing +12 added lines, -12 removed lines

@@ -16,32 +16,32 @@
 
 	protected function setUp()
 	{
-		$this->mock = $this->getMockBuilder( \Neos\FluidAdaptor\View\StandaloneView::class )
-			->setMethods( array( 'assign', 'assignMultiple', 'render', 'setTemplatePathAndFilename' ) )
+		$this->mock = $this->getMockBuilder(\Neos\FluidAdaptor\View\StandaloneView::class)
+			->setMethods(array('assign', 'assignMultiple', 'render', 'setTemplatePathAndFilename'))
 			->disableOriginalConstructor()
 			->getMock();
 
-		$this->object = new \Aimeos\MW\View\Engine\Flow( $this->mock );
+		$this->object = new \Aimeos\MW\View\Engine\Flow($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([]);
 
-		$this->mock->expects( $this->once() )->method( 'setTemplatePathAndFilename' );
-		$this->mock->expects( $this->once() )->method( 'assignMultiple' );
-		$this->mock->expects( $this->once() )->method( 'assign' );
-		$this->mock->expects( $this->once() )->method( 'render' )
-			->will( $this->returnValue( 'test' ) );
+		$this->mock->expects($this->once())->method('setTemplatePathAndFilename');
+		$this->mock->expects($this->once())->method('assignMultiple');
+		$this->mock->expects($this->once())->method('assign');
+		$this->mock->expects($this->once())->method('render')
+			->will($this->returnValue('test'));
 
-		$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/FlowTest.php

Spacing +11 added lines, -11 removed lines

@@ -14,41 +14,41 @@
 
 	protected function setUp()
 	{
-		if( !class_exists( '\Neos\Flow\Http\Request' ) ) {
-			$this->markTestSkipped( '\Neos\Flow\Http\Request is not available' );
+		if (!class_exists('\Neos\Flow\Http\Request')) {
+			$this->markTestSkipped('\Neos\Flow\Http\Request is not available');
 		}
 
-		if( !class_exists( '\Zend\Diactoros\ServerRequestFactory' ) ) {
-			$this->markTestSkipped( '\Zend\Diactoros\ServerRequestFactory is not available' );
+		if (!class_exists('\Zend\Diactoros\ServerRequestFactory')) {
+			$this->markTestSkipped('\Zend\Diactoros\ServerRequestFactory is not available');
 		}
 
 		$view = new \Aimeos\MW\View\Standard();
-		$server = array( 'REMOTE_ADDR' => '127.0.0.1' );
-		$request = new \Neos\Flow\Http\Request( [], [], [], $server );
-		$this->object = new \Aimeos\MW\View\Helper\Request\Flow( $view, $request, [], [], [], [], $server );
+		$server = array('REMOTE_ADDR' => '127.0.0.1');
+		$request = new \Neos\Flow\Http\Request([], [], [], $server);
+		$this->object = new \Aimeos\MW\View\Helper\Request\Flow($view, $request, [], [], [], [], $server);
 	}
 
 
 	protected function tearDown()
 	{
-		unset( $this->object, $this->mock );
+		unset($this->object, $this->mock);
 	}
 
 
 	public function testTransform()
 	{
-		$this->assertInstanceOf( \Aimeos\MW\View\Helper\Request\Flow::class, $this->object->transform() );
+		$this->assertInstanceOf(\Aimeos\MW\View\Helper\Request\Flow::class, $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/FlowTest.php

Spacing +5 added lines, -5 removed lines

@@ -16,23 +16,23 @@
 
 	protected function setUp()
 	{
-		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\Flow( $view );
+		$this->object = new \Aimeos\MW\View\Helper\Response\Flow($view);
 	}
 
 
 	protected function tearDown()
 	{
-		unset( $this->object );
+		unset($this->object);
 	}
 
 
 	public function testTransform()
 	{
-		$this->assertInstanceOf( \Aimeos\MW\View\Helper\Response\Flow::class, $this->object->transform() );
+		$this->assertInstanceOf(\Aimeos\MW\View\Helper\Response\Flow::class, $this->object->transform());
 	}
 }

lib/custom/tests/MAdmin/Cache/Proxy/FlowTest.php

Spacing +13 added lines, -13 removed lines

@@ -17,38 +17,38 @@
 
 	protected function setUp()
 	{
-		if( class_exists( 'Neos\Cache\Frontend\StringFrontend' ) === false ) {
-			$this->markTestSkipped( 'Class \\Neos\\Cache\\Frontend\\StringFrontend not found' );
+		if (class_exists('Neos\Cache\Frontend\StringFrontend') === false) {
+			$this->markTestSkipped('Class \\Neos\\Cache\\Frontend\\StringFrontend not found');
 		}
 
-		$localeItem = $this->getMockBuilder( \Aimeos\MShop\Locale\Item\Standard::class )
-			->setMethods( ['getSiteId'])
+		$localeItem = $this->getMockBuilder(\Aimeos\MShop\Locale\Item\Standard::class)
+			->setMethods(['getSiteId'])
 			->getMock();
 
-		$this->mock = $this->getMockBuilder( 'Neos\Cache\Frontend\StringFrontend' )
+		$this->mock = $this->getMockBuilder('Neos\Cache\Frontend\StringFrontend')
 			->disableOriginalConstructor()
 			->getMock();
 
 		$context = \TestHelper::getContext();
-		$context->setLocale( $localeItem );
+		$context->setLocale($localeItem);
 
-		$this->object = new \Aimeos\MAdmin\Cache\Proxy\Flow( $context, $this->mock );
+		$this->object = new \Aimeos\MAdmin\Cache\Proxy\Flow($context, $this->mock);
 	}
 
 
 	protected function tearDown()
 	{
-		unset( $this->mock, $this->object );
+		unset($this->mock, $this->object);
 	}
 
 
 	public function testGetObject()
 	{
-		$class = new \ReflectionClass( \Aimeos\MAdmin\Cache\Proxy\Flow::class );
-		$method = $class->getMethod( 'getObject' );
-		$method->setAccessible( true );
+		$class = new \ReflectionClass(\Aimeos\MAdmin\Cache\Proxy\Flow::class);
+		$method = $class->getMethod('getObject');
+		$method->setAccessible(true);
 
-		$result = $method->invokeArgs( $this->object, [] );
-		$this->assertInstanceOf( \Aimeos\MW\Cache\Iface::class, $result );
+		$result = $method->invokeArgs($this->object, []);
+		$this->assertInstanceOf(\Aimeos\MW\Cache\Iface::class, $result);
 	}
 }