aimeos / ai-cms-grapesjs

lib/custom/src/MShop/Cms/Item/Standard.php

Spacing +25 added lines, -25 removed lines

@@ -31,11 +31,11 @@ 
 	 * @param \Aimeos\MShop\Common\Item\Lists\Iface[] $listItems List of list items
 	 * @param \Aimeos\MShop\Common\Item\Iface[] $refItems List of referenced items
 	 */
-	public function __construct( array $values = [], array $listItems = [], array $refItems = [] )
+	public function __construct(array $values = [], array $listItems = [], array $refItems = [])
 	{
-		parent::__construct( 'cms.', $values );
+		parent::__construct('cms.', $values);
 
-		$this->initListItems( $listItems, $refItems );
+		$this->initListItems($listItems, $refItems);
 	}
 
 
@@ -46,7 +46,7 @@ 
 	 */
 	public function getUrl() : string
 	{
-		return $this->get( 'cms.url', '' );
+		return $this->get('cms.url', '');
 	}
 
 
@@ -56,13 +56,13 @@ 
 	 * @param string $value URL of the cms item
 	 * @return \Aimeos\MShop\Cms\Item\Iface Cms item for chaining method calls
 	 */
-	public function setUrl( string $value ) : \Aimeos\MShop\Cms\Item\Iface
+	public function setUrl(string $value) : \Aimeos\MShop\Cms\Item\Iface
 	{
-		$url = \Aimeos\Map::explode( '/', trim( $value, '/' ) )->map( function( $segment ) {
-			return \Aimeos\MW\Str::slug( $segment );
-		} )->join( '/' );
+		$url = \Aimeos\Map::explode('/', trim($value, '/'))->map(function($segment) {
+			return \Aimeos\MW\Str::slug($segment);
+		} )->join('/');
 
-		return $this->set( 'cms.url', '/' . $url );
+		return $this->set('cms.url', '/' . $url);
 	}
 
 
@@ -73,7 +73,7 @@ 
 	 */
 	public function getLabel() : string
 	{
-		return $this->get( 'cms.label', '' );
+		return $this->get('cms.label', '');
 	}
 
 
@@ -83,9 +83,9 @@ 
 	 * @param string $label Type label of the attribute item
 	 * @return \Aimeos\MShop\Cms\Item\Iface Cms item for chaining method calls
 	 */
-	public function setLabel( ?string $label ) : \Aimeos\MShop\Cms\Item\Iface
+	public function setLabel(?string $label) : \Aimeos\MShop\Cms\Item\Iface
 	{
-		return $this->set( 'cms.label', (string) $label );
+		return $this->set('cms.label', (string) $label);
 	}
 
 
@@ -96,7 +96,7 @@ 
 	 */
 	public function getStatus() : int
 	{
-		return $this->get( 'cms.status', 1 );
+		return $this->get('cms.status', 1);
 	}
 
 
@@ -106,9 +106,9 @@ 
 	 * @param int $status true/false for enabled/disabled
 	 * @return \Aimeos\MShop\Cms\Item\Iface Cms item for chaining method calls
 	 */
-	public function setStatus( int $status ) : \Aimeos\MShop\Common\Item\Iface
+	public function setStatus(int $status) : \Aimeos\MShop\Common\Item\Iface
 	{
-		return $this->set( 'cms.status', $status );
+		return $this->set('cms.status', $status);
 	}
 
 
@@ -141,21 +141,21 @@ 
 	 * @param bool True to set private properties too, false for public only
 	 * @return \Aimeos\MShop\Cms\Item\Iface Cms item for chaining method calls
 	 */
-	public function fromArray( array &$list, bool $private = false ) : \Aimeos\MShop\Common\Item\Iface
+	public function fromArray(array &$list, bool $private = false) : \Aimeos\MShop\Common\Item\Iface
 	{
-		$item = parent::fromArray( $list, $private );
+		$item = parent::fromArray($list, $private);
 
-		foreach( $list as $key => $value )
+		foreach ($list as $key => $value)
 		{
-			switch( $key )
+			switch ($key)
 			{
-				case 'cms.url': $item = $item->setUrl( $value ); break;
-				case 'cms.label': $item = $item->setLabel( $value ); break;
-				case 'cms.status': $item = $item->setStatus( (int) $value ); break;
+				case 'cms.url': $item = $item->setUrl($value); break;
+				case 'cms.label': $item = $item->setLabel($value); break;
+				case 'cms.status': $item = $item->setStatus((int) $value); break;
 				default: continue 2;
 			}
 
-			unset( $list[$key] );
+			unset($list[$key]);
 		}
 
 		return $item;
@@ -168,9 +168,9 @@ 
 	 * @param bool True to return private properties, false for public only
 	 * @return array Associative list of item properties and their values
 	 */
-	public function toArray( bool $private = false ) : array
+	public function toArray(bool $private = false) : array
 	{
-		$list = parent::toArray( $private );
+		$list = parent::toArray($private);
 
 		$list['cms.url'] = $this->getUrl();
 		$list['cms.label'] = $this->getLabel();

lib/custom/tests/TestHelper.php

Spacing +21 added lines, -21 removed lines

@@ -13,16 +13,16 @@ 
 
 		$includepaths = $aimeos->getIncludePaths();
 		$includepaths[] = get_include_path();
-		set_include_path( implode( PATH_SEPARATOR, $includepaths ) );
+		set_include_path(implode(PATH_SEPARATOR, $includepaths));
 	}
 
 
 	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');
 
 			self::$aimeos = new \Aimeos\Bootstrap();
 		}
@@ -31,10 +31,10 @@ 
 	}
 
 
-	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];
@@ -44,7 +44,7 @@ 
 	/**
 	 * @param string $site
 	 */
-	private static function createContext( $site )
+	private static function createContext($site)
 	{
 		$ctx = new \Aimeos\MShop\Context\Item\Standard();
 		$aimeos = self::getAimeos();
@@ -53,36 +53,36 @@ 
 		$paths = $aimeos->getConfigPaths();
 		$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\DBAL( $conf );
-		$ctx->setDatabaseManager( $dbm );
+		$dbm = new \Aimeos\MW\DB\Manager\DBAL($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);
 
 
 		$cache = new \Aimeos\MW\Cache\None();
-		$ctx->setCache( $cache );
+		$ctx->setCache($cache);
 
 
-		$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::create( $ctx );
-		$localeItem = $localeManager->bootstrap( $site, '', '', false );
+		$localeManager = \Aimeos\MShop\Locale\Manager\Factory::create($ctx);
+		$localeItem = $localeManager->bootstrap($site, '', '', false);
 
-		$ctx->setLocale( $localeItem );
+		$ctx->setLocale($localeItem);
 
-		$ctx->setEditor( 'ai-cms-grapejs:lib/custom' );
+		$ctx->setEditor('ai-cms-grapejs:lib/custom');
 
 		return $ctx;
 	}

lib/custom/tests/MShop/Cms/Manager/FactoryTest.php

Spacing +8 added lines, -8 removed lines

@@ -16,28 +16,28 @@
 {
 	public function testCreateManager()
 	{
-		$manager = \Aimeos\MShop\Cms\Manager\Factory::create( \TestHelperMShop::getContext() );
-		$this->assertInstanceOf( \Aimeos\MShop\Common\Manager\Iface::class, $manager );
+		$manager = \Aimeos\MShop\Cms\Manager\Factory::create(\TestHelperMShop::getContext());
+		$this->assertInstanceOf(\Aimeos\MShop\Common\Manager\Iface::class, $manager);
 	}
 
 
 	public function testCreateManagerName()
 	{
-		$manager = \Aimeos\MShop\Cms\Manager\Factory::create( \TestHelperMShop::getContext(), 'Standard' );
-		$this->assertInstanceOf( \Aimeos\MShop\Common\Manager\Iface::class, $manager );
+		$manager = \Aimeos\MShop\Cms\Manager\Factory::create(\TestHelperMShop::getContext(), 'Standard');
+		$this->assertInstanceOf(\Aimeos\MShop\Common\Manager\Iface::class, $manager);
 	}
 
 
 	public function testCreateManagerInvalidName()
 	{
-		$this->expectException( \Aimeos\MShop\Cms\Exception::class );
-		\Aimeos\MShop\Cms\Manager\Factory::create( \TestHelperMShop::getContext(), '%^&' );
+		$this->expectException(\Aimeos\MShop\Cms\Exception::class);
+		\Aimeos\MShop\Cms\Manager\Factory::create(\TestHelperMShop::getContext(), '%^&');
 	}
 
 
 	public function testCreateManagerNotExisting()
 	{
-		$this->expectException( \Aimeos\MShop\Exception::class );
-		\Aimeos\MShop\Cms\Manager\Factory::create( \TestHelperMShop::getContext(), 'unknown' );
+		$this->expectException(\Aimeos\MShop\Exception::class);
+		\Aimeos\MShop\Cms\Manager\Factory::create(\TestHelperMShop::getContext(), 'unknown');
 	}
 }

lib/custom/tests/MShop/Cms/Manager/Lists/Type/StandardTest.php

Spacing +72 added lines, -72 removed lines

@@ -18,22 +18,22 @@ 
 	protected function setUp() : void
 	{
 		$this->editor = \TestHelper::getContext()->getEditor();
-		$manager = \Aimeos\MShop\Cms\Manager\Factory::create( \TestHelper::getContext() );
+		$manager = \Aimeos\MShop\Cms\Manager\Factory::create(\TestHelper::getContext());
 
-		$listManager = $manager->getSubManager( 'lists' );
-		$this->object = $listManager->getSubManager( 'type' );
+		$listManager = $manager->getSubManager('lists');
+		$this->object = $listManager->getSubManager('type');
 	}
 
 
 	protected function tearDown() : void
 	{
-		unset( $this->object );
+		unset($this->object);
 	}
 
 
 	public function testClear()
 	{
-		$this->assertInstanceOf( \Aimeos\MShop\Common\Manager\Iface::class, $this->object->clear( [-1] ) );
+		$this->assertInstanceOf(\Aimeos\MShop\Common\Manager\Iface::class, $this->object->clear([-1]));
 	}
 
 
@@ -41,82 +41,82 @@ 
 	{
 		$result = $this->object->getResourceType();
 
-		$this->assertContains( 'cms/lists/type', $result );
+		$this->assertContains('cms/lists/type', $result);
 	}
 
 
 	public function testCreateItem()
 	{
 		$item = $this->object->create();
-		$this->assertInstanceOf( \Aimeos\MShop\Common\Item\Type\Iface::class, $item );
+		$this->assertInstanceOf(\Aimeos\MShop\Common\Item\Type\Iface::class, $item);
 	}
 
 
 	public function testGetItem()
 	{
-		$search = $this->object->filter()->slice( 0, 1 );
-		$search->setConditions( $search->compare( '==', 'cms.lists.type.editor', $this->editor ) );
-		$results = $this->object->search( $search )->toArray();
+		$search = $this->object->filter()->slice(0, 1);
+		$search->setConditions($search->compare('==', 'cms.lists.type.editor', $this->editor));
+		$results = $this->object->search($search)->toArray();
 
-		if( ( $expected = reset( $results ) ) === false ) {
-			throw new \RuntimeException( 'No cms list type item found' );
+		if (($expected = reset($results)) === false) {
+			throw new \RuntimeException('No cms list type item found');
 		}
 
-		$this->assertEquals( $expected, $this->object->get( $expected->getId() ) );
+		$this->assertEquals($expected, $this->object->get($expected->getId()));
 	}
 
 
 	public function testSaveUpdateDeleteItem()
 	{
 		$search = $this->object->filter();
-		$search->setConditions( $search->compare( '==', 'cms.lists.type.editor', $this->editor ) );
-		$results = $this->object->search( $search )->toArray();
+		$search->setConditions($search->compare('==', 'cms.lists.type.editor', $this->editor));
+		$results = $this->object->search($search)->toArray();
 
-		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( 'unitTestSave' );
-		$resultSaved = $this->object->save( $item );
-		$itemSaved = $this->object->get( $item->getId() );
+		$item->setId(null);
+		$item->setCode('unitTestSave');
+		$resultSaved = $this->object->save($item);
+		$itemSaved = $this->object->get($item->getId());
 
 		$itemExp = clone $itemSaved;
-		$itemExp->setCode( 'unitTestSave2' );
-		$resultUpd = $this->object->save( $itemExp );
-		$itemUpd = $this->object->get( $itemExp->getId() );
+		$itemExp->setCode('unitTestSave2');
+		$resultUpd = $this->object->save($itemExp);
+		$itemUpd = $this->object->get($itemExp->getId());
 
-		$this->object->delete( $itemSaved->getId() );
+		$this->object->delete($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->assertInstanceOf( \Aimeos\MShop\Common\Item\Iface::class, $resultSaved );
-		$this->assertInstanceOf( \Aimeos\MShop\Common\Item\Iface::class, $resultUpd );
+		$this->assertInstanceOf(\Aimeos\MShop\Common\Item\Iface::class, $resultSaved);
+		$this->assertInstanceOf(\Aimeos\MShop\Common\Item\Iface::class, $resultUpd);
 
-		$this->expectException( \Aimeos\MShop\Exception::class );
-		$this->object->get( $itemSaved->getId() );
+		$this->expectException(\Aimeos\MShop\Exception::class);
+		$this->object->get($itemSaved->getId());
 	}
 
 
@@ -126,19 +126,19 @@ 
 		$search = $this->object->filter();
 
 		$expr = [];
-		$expr[] = $search->compare( '!=', 'cms.lists.type.id', null );
-		$expr[] = $search->compare( '!=', 'cms.lists.type.siteid', null );
-		$expr[] = $search->compare( '==', 'cms.lists.type.code', 'default' );
-		$expr[] = $search->compare( '==', 'cms.lists.type.domain', 'text' );
-		$expr[] = $search->compare( '>', 'cms.lists.type.label', '' );
-		$expr[] = $search->compare( '>=', 'cms.lists.type.position', 0 );
-		$expr[] = $search->compare( '==', 'cms.lists.type.status', 1 );
-		$expr[] = $search->compare( '==', 'cms.lists.type.editor', $this->editor );
-
-		$search->setConditions( $search->and( $expr ) );
-
-		$results = $this->object->search( $search, [], $total )->toArray();
-		$this->assertEquals( 1, count( $results ) );
+		$expr[] = $search->compare('!=', 'cms.lists.type.id', null);
+		$expr[] = $search->compare('!=', 'cms.lists.type.siteid', null);
+		$expr[] = $search->compare('==', 'cms.lists.type.code', 'default');
+		$expr[] = $search->compare('==', 'cms.lists.type.domain', 'text');
+		$expr[] = $search->compare('>', 'cms.lists.type.label', '');
+		$expr[] = $search->compare('>=', 'cms.lists.type.position', 0);
+		$expr[] = $search->compare('==', 'cms.lists.type.status', 1);
+		$expr[] = $search->compare('==', 'cms.lists.type.editor', $this->editor);
+
+		$search->setConditions($search->and($expr));
+
+		$results = $this->object->search($search, [], $total)->toArray();
+		$this->assertEquals(1, count($results));
 	}
 
 
@@ -147,18 +147,18 @@ 
 		$total = 0;
 		$search = $this->object->filter();
 		$conditions = array(
-			$search->compare( '==', 'cms.lists.type.code', 'default' ),
-			$search->compare( '==', 'cms.lists.type.editor', $this->editor ),
+			$search->compare('==', 'cms.lists.type.code', 'default'),
+			$search->compare('==', 'cms.lists.type.editor', $this->editor),
 		);
-		$search->setConditions( $search->and( $conditions ) );
-		$search->setSortations( [$search->sort( '-', 'cms.lists.type.position' )] );
-		$search->slice( 0, 1 );
-		$results = $this->object->search( $search, [], $total )->toArray();
-		$this->assertEquals( 1, count( $results ) );
-		$this->assertEquals( 1, $total );
-
-		foreach( $results as $itemId => $item ) {
-			$this->assertEquals( $itemId, $item->getId() );
+		$search->setConditions($search->and($conditions));
+		$search->setSortations([$search->sort('-', 'cms.lists.type.position')]);
+		$search->slice(0, 1);
+		$results = $this->object->search($search, [], $total)->toArray();
+		$this->assertEquals(1, count($results));
+		$this->assertEquals(1, $total);
+
+		foreach ($results as $itemId => $item) {
+			$this->assertEquals($itemId, $item->getId());
 		}
 	}
 

lib/custom/tests/MShop/Cms/Manager/Lists/StandardTest.php

Spacing +98 added lines, -98 removed lines

@@ -20,20 +20,20 @@ 
 	{
 		$this->concms = \TestHelper::getContext();
 		$this->editor = $this->concms->getEditor();
-		$manager = \Aimeos\MShop\Cms\Manager\Factory::create( $this->concms, 'Standard' );
-		$this->object = $manager->getSubManager( 'lists', 'Standard' );
+		$manager = \Aimeos\MShop\Cms\Manager\Factory::create($this->concms, 'Standard');
+		$this->object = $manager->getSubManager('lists', 'Standard');
 	}
 
 
 	protected function tearDown() : void
 	{
-		unset( $this->object, $this->concms );
+		unset($this->object, $this->concms);
 	}
 
 
 	public function testClear()
 	{
-		$this->assertInstanceOf( \Aimeos\MShop\Common\Manager\Iface::class, $this->object->clear( [-1] ) );
+		$this->assertInstanceOf(\Aimeos\MShop\Common\Manager\Iface::class, $this->object->clear([-1]));
 	}
 
 
@@ -41,99 +41,99 @@ 
 	{
 		$result = $this->object->getResourceType();
 
-		$this->assertContains( 'cms/lists', $result );
+		$this->assertContains('cms/lists', $result);
 	}
 
 
 	public function testAggregate()
 	{
-		$search = $this->object->filter( true );
+		$search = $this->object->filter(true);
 		$expr = array(
 			$search->getConditions(),
-			$search->compare( '==', 'cms.lists.editor', $this->editor ),
+			$search->compare('==', 'cms.lists.editor', $this->editor),
 		);
-		$search->setConditions( $search->and( $expr ) );
+		$search->setConditions($search->and($expr));
 
-		$result = $this->object->aggregate( $search, 'cms.lists.domain' )->toArray();
+		$result = $this->object->aggregate($search, 'cms.lists.domain')->toArray();
 
-		$this->assertEquals( 1, count( $result ) );
-		$this->assertArrayHasKey( 'text', $result );
-		$this->assertEquals( 6, $result['text'] );
+		$this->assertEquals(1, count($result));
+		$this->assertArrayHasKey('text', $result);
+		$this->assertEquals(6, $result['text']);
 	}
 
 
 	public function testCreateItem()
 	{
 		$item = $this->object->create();
-		$this->assertInstanceOf( \Aimeos\MShop\Common\Item\Lists\Iface::class, $item );
+		$this->assertInstanceOf(\Aimeos\MShop\Common\Item\Lists\Iface::class, $item);
 	}
 
 
 	public function testGetItem()
 	{
-		$search = $this->object->filter()->slice( 0, 1 );
-		$search->setConditions( $search->compare( '==', 'cms.lists.editor', $this->editor ) );
-		$item = $this->object->search( $search )->first( new \RuntimeException( 'No item found' ) );
+		$search = $this->object->filter()->slice(0, 1);
+		$search->setConditions($search->compare('==', 'cms.lists.editor', $this->editor));
+		$item = $this->object->search($search)->first(new \RuntimeException('No item found'));
 
-		$this->assertEquals( $item, $this->object->get( $item->getId() ) );
+		$this->assertEquals($item, $this->object->get($item->getId()));
 	}
 
 
 	public function testSaveUpdateDeleteItem()
 	{
 		$search = $this->object->filter();
-		$search->setConditions( $search->compare( '==', 'cms.lists.editor', $this->editor ) );
-		$item = $this->object->search( $search )->first( new \RuntimeException( 'No item found' ) );
+		$search->setConditions($search->compare('==', 'cms.lists.editor', $this->editor));
+		$item = $this->object->search($search)->first(new \RuntimeException('No item found'));
 
-		$item->setId( null );
-		$item->setDomain( 'unittest' );
-		$resultSaved = $this->object->save( $item );
-		$itemSaved = $this->object->get( $item->getId() );
+		$item->setId(null);
+		$item->setDomain('unittest');
+		$resultSaved = $this->object->save($item);
+		$itemSaved = $this->object->get($item->getId());
 
 		$itemExp = clone $itemSaved;
-		$itemExp->setDomain( 'unittest1' );
-		$resultUpd = $this->object->save( $itemExp );
-		$itemUpd = $this->object->get( $itemExp->getId() );
-
-		$this->object->delete( $itemSaved->getId() );
-
-
-		$this->assertTrue( $item->getId() !== null );
-		$this->assertTrue( $itemSaved->getType() !== null );
-		$this->assertEquals( $item->getId(), $itemSaved->getId() );
-		$this->assertEquals( $item->getSiteId(), $itemSaved->getSiteId() );
-		$this->assertEquals( $item->getParentId(), $itemSaved->getParentId() );
-		$this->assertEquals( $item->getType(), $itemSaved->getType() );
-		$this->assertEquals( $item->getRefId(), $itemSaved->getRefId() );
-		$this->assertEquals( $item->getDomain(), $itemSaved->getDomain() );
-		$this->assertEquals( $item->getDateStart(), $itemSaved->getDateStart() );
-		$this->assertEquals( $item->getDateEnd(), $itemSaved->getDateEnd() );
-		$this->assertEquals( $item->getPosition(), $itemSaved->getPosition() );
-
-		$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->assertTrue( $itemUpd->getType() !== null );
-		$this->assertEquals( $itemExp->getId(), $itemUpd->getId() );
-		$this->assertEquals( $itemExp->getSiteId(), $itemUpd->getSiteId() );
-		$this->assertEquals( $itemExp->getParentId(), $itemUpd->getParentId() );
-		$this->assertEquals( $itemExp->getType(), $itemUpd->getType() );
-		$this->assertEquals( $itemExp->getRefId(), $itemUpd->getRefId() );
-		$this->assertEquals( $itemExp->getDomain(), $itemUpd->getDomain() );
-		$this->assertEquals( $itemExp->getDateStart(), $itemUpd->getDateStart() );
-		$this->assertEquals( $itemExp->getDateEnd(), $itemUpd->getDateEnd() );
-		$this->assertEquals( $itemExp->getPosition(), $itemUpd->getPosition() );
-
-		$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->assertInstanceOf( \Aimeos\MShop\Common\Item\Iface::class, $resultSaved );
-		$this->assertInstanceOf( \Aimeos\MShop\Common\Item\Iface::class, $resultUpd );
-
-		$this->expectException( \Aimeos\MShop\Exception::class );
-		$this->object->get( $itemSaved->getId() );
+		$itemExp->setDomain('unittest1');
+		$resultUpd = $this->object->save($itemExp);
+		$itemUpd = $this->object->get($itemExp->getId());
+
+		$this->object->delete($itemSaved->getId());
+
+
+		$this->assertTrue($item->getId() !== null);
+		$this->assertTrue($itemSaved->getType() !== null);
+		$this->assertEquals($item->getId(), $itemSaved->getId());
+		$this->assertEquals($item->getSiteId(), $itemSaved->getSiteId());
+		$this->assertEquals($item->getParentId(), $itemSaved->getParentId());
+		$this->assertEquals($item->getType(), $itemSaved->getType());
+		$this->assertEquals($item->getRefId(), $itemSaved->getRefId());
+		$this->assertEquals($item->getDomain(), $itemSaved->getDomain());
+		$this->assertEquals($item->getDateStart(), $itemSaved->getDateStart());
+		$this->assertEquals($item->getDateEnd(), $itemSaved->getDateEnd());
+		$this->assertEquals($item->getPosition(), $itemSaved->getPosition());
+
+		$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->assertTrue($itemUpd->getType() !== null);
+		$this->assertEquals($itemExp->getId(), $itemUpd->getId());
+		$this->assertEquals($itemExp->getSiteId(), $itemUpd->getSiteId());
+		$this->assertEquals($itemExp->getParentId(), $itemUpd->getParentId());
+		$this->assertEquals($itemExp->getType(), $itemUpd->getType());
+		$this->assertEquals($itemExp->getRefId(), $itemUpd->getRefId());
+		$this->assertEquals($itemExp->getDomain(), $itemUpd->getDomain());
+		$this->assertEquals($itemExp->getDateStart(), $itemUpd->getDateStart());
+		$this->assertEquals($itemExp->getDateEnd(), $itemUpd->getDateEnd());
+		$this->assertEquals($itemExp->getPosition(), $itemUpd->getPosition());
+
+		$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->assertInstanceOf(\Aimeos\MShop\Common\Item\Iface::class, $resultSaved);
+		$this->assertInstanceOf(\Aimeos\MShop\Common\Item\Iface::class, $resultUpd);
+
+		$this->expectException(\Aimeos\MShop\Exception::class);
+		$this->object->get($itemSaved->getId());
 	}
 
 
@@ -143,59 +143,59 @@ 
 		$search = $this->object->filter();
 
 		$expr = [];
-		$expr[] = $search->compare( '!=', 'cms.lists.id', null );
-		$expr[] = $search->compare( '!=', 'cms.lists.siteid', null );
-		$expr[] = $search->compare( '!=', 'cms.lists.parentid', null );
-		$expr[] = $search->compare( '==', 'cms.lists.domain', 'text' );
-		$expr[] = $search->compare( '==', 'cms.lists.type', 'default' );
-		$expr[] = $search->compare( '>', 'cms.lists.refid', 0 );
-		$expr[] = $search->compare( '==', 'cms.lists.datestart', null );
-		$expr[] = $search->compare( '==', 'cms.lists.dateend', null );
-		$expr[] = $search->compare( '!=', 'cms.lists.config', null );
-		$expr[] = $search->compare( '==', 'cms.lists.position', 0 );
-		$expr[] = $search->compare( '==', 'cms.lists.status', 1 );
-		$expr[] = $search->compare( '==', 'cms.lists.editor', $this->editor );
-
-		$search->setConditions( $search->and( $expr ) );
-		$results = $this->object->search( $search, [], $total )->toArray();
-		$this->assertEquals( 2, count( $results ) );
+		$expr[] = $search->compare('!=', 'cms.lists.id', null);
+		$expr[] = $search->compare('!=', 'cms.lists.siteid', null);
+		$expr[] = $search->compare('!=', 'cms.lists.parentid', null);
+		$expr[] = $search->compare('==', 'cms.lists.domain', 'text');
+		$expr[] = $search->compare('==', 'cms.lists.type', 'default');
+		$expr[] = $search->compare('>', 'cms.lists.refid', 0);
+		$expr[] = $search->compare('==', 'cms.lists.datestart', null);
+		$expr[] = $search->compare('==', 'cms.lists.dateend', null);
+		$expr[] = $search->compare('!=', 'cms.lists.config', null);
+		$expr[] = $search->compare('==', 'cms.lists.position', 0);
+		$expr[] = $search->compare('==', 'cms.lists.status', 1);
+		$expr[] = $search->compare('==', 'cms.lists.editor', $this->editor);
+
+		$search->setConditions($search->and($expr));
+		$results = $this->object->search($search, [], $total)->toArray();
+		$this->assertEquals(2, count($results));
 	}
 
 
 	public function testSearchItemsAll()
 	{
 		$search = $this->object->filter();
-		$search->setConditions( $search->compare( '==', 'cms.lists.editor', $this->editor ) );
-		$this->assertEquals( 6, count( $this->object->search( $search )->toArray() ) );
+		$search->setConditions($search->compare('==', 'cms.lists.editor', $this->editor));
+		$this->assertEquals(6, count($this->object->search($search)->toArray()));
 	}
 
 
 	public function testSearchItemsBase()
 	{
 		$total = 0;
-		$search = $this->object->filter( true );
+		$search = $this->object->filter(true);
 		$conditions = array(
-			$search->compare( '==', 'cms.lists.editor', $this->editor ),
+			$search->compare('==', 'cms.lists.editor', $this->editor),
 			$search->getConditions()
 		);
-		$search->setConditions( $search->and( $conditions ) );
-		$search->slice( 0, 1 );
-		$results = $this->object->search( $search, [], $total )->toArray();
-		$this->assertEquals( 1, count( $results ) );
-		$this->assertEquals( 6, $total );
-
-		foreach( $results as $itemId => $item ) {
-			$this->assertEquals( $itemId, $item->getId() );
+		$search->setConditions($search->and($conditions));
+		$search->slice(0, 1);
+		$results = $this->object->search($search, [], $total)->toArray();
+		$this->assertEquals(1, count($results));
+		$this->assertEquals(6, $total);
+
+		foreach ($results as $itemId => $item) {
+			$this->assertEquals($itemId, $item->getId());
 		}
 	}
 
 
 	public function testGetSubManager()
 	{
-		$this->assertInstanceOf( \Aimeos\MShop\Common\Manager\Iface::class, $this->object->getSubManager( 'type' ) );
-		$this->assertInstanceOf( \Aimeos\MShop\Common\Manager\Iface::class, $this->object->getSubManager( 'type', 'Standard' ) );
+		$this->assertInstanceOf(\Aimeos\MShop\Common\Manager\Iface::class, $this->object->getSubManager('type'));
+		$this->assertInstanceOf(\Aimeos\MShop\Common\Manager\Iface::class, $this->object->getSubManager('type', 'Standard'));
 
-		$this->expectException( \Aimeos\MShop\Exception::class );
-		$this->object->getSubManager( 'unknown' );
+		$this->expectException(\Aimeos\MShop\Exception::class);
+		$this->object->getSubManager('unknown');
 	}
 }

lib/custom/tests/MShop/Cms/Manager/StandardTest.php

Spacing +79 added lines, -79 removed lines

@@ -17,7 +17,7 @@ 
 	protected function setUp() : void
 	{
 		$this->editor = \TestHelper::getContext()->getEditor();
-		$this->object = new \Aimeos\MShop\Cms\Manager\Standard( \TestHelper::getContext() );
+		$this->object = new \Aimeos\MShop\Cms\Manager\Standard(\TestHelper::getContext());
 	}
 
 
@@ -29,27 +29,27 @@ 
 
 	public function testClear()
 	{
-		$this->assertInstanceOf( \Aimeos\MShop\Common\Manager\Iface::class, $this->object->clear( [-1] ) );
+		$this->assertInstanceOf(\Aimeos\MShop\Common\Manager\Iface::class, $this->object->clear([-1]));
 	}
 
 
 	public function testDeleteItems()
 	{
-		$this->assertInstanceOf( \Aimeos\MShop\Common\Manager\Iface::class, $this->object->delete( [-1] ) );
+		$this->assertInstanceOf(\Aimeos\MShop\Common\Manager\Iface::class, $this->object->delete([-1]));
 	}
 
 
 	public function testCreateItem()
 	{
 		$cmsItem = $this->object->create();
-		$this->assertInstanceOf( \Aimeos\MShop\Cms\Item\Iface::class, $cmsItem );
+		$this->assertInstanceOf(\Aimeos\MShop\Cms\Item\Iface::class, $cmsItem);
 	}
 
 
 	public function testCreateItemType()
 	{
-		$item = $this->object->create( ['cms.url' => '/test'] );
-		$this->assertEquals( '/test', $item->getUrl() );
+		$item = $this->object->create(['cms.url' => '/test']);
+		$this->assertEquals('/test', $item->getUrl());
 	}
 
 
@@ -57,150 +57,150 @@ 
 	{
 		$result = $this->object->getResourceType();
 
-		$this->assertContains( 'cms', $result );
-		$this->assertContains( 'cms/lists', $result );
+		$this->assertContains('cms', $result);
+		$this->assertContains('cms/lists', $result);
 	}
 
 
 	public function testGetSearchAttributes()
 	{
-		foreach( $this->object->getSearchAttributes() as $attribute ) {
-			$this->assertInstanceOf( \Aimeos\MW\Criteria\Attribute\Iface::class, $attribute );
+		foreach ($this->object->getSearchAttributes() as $attribute) {
+			$this->assertInstanceOf(\Aimeos\MW\Criteria\Attribute\Iface::class, $attribute);
 		}
 	}
 
 
 	public function testSearchItems()
 	{
-		$item = $this->object->find( '/contact', ['text'] );
-		$listItem = $item->getListItems( 'text', 'default' )->first( new \RuntimeException( 'No list item found' ) );
+		$item = $this->object->find('/contact', ['text']);
+		$listItem = $item->getListItems('text', 'default')->first(new \RuntimeException('No list item found'));
 
 		$total = 0;
 		$search = $this->object->filter();
 
 		$expr = [];
-		$expr[] = $search->compare( '!=', 'cms.id', null );
-		$expr[] = $search->compare( '!=', 'cms.siteid', null );
-		$expr[] = $search->compare( '==', 'cms.url', '/contact' );
-		$expr[] = $search->compare( '==', 'cms.label', 'Contact' );
-		$expr[] = $search->compare( '==', 'cms.status', 1 );
-		$expr[] = $search->compare( '>=', 'cms.mtime', '1970-01-01 00:00:00' );
-		$expr[] = $search->compare( '>=', 'cms.ctime', '1970-01-01 00:00:00' );
-		$expr[] = $search->compare( '==', 'cms.editor', $this->editor );
+		$expr[] = $search->compare('!=', 'cms.id', null);
+		$expr[] = $search->compare('!=', 'cms.siteid', null);
+		$expr[] = $search->compare('==', 'cms.url', '/contact');
+		$expr[] = $search->compare('==', 'cms.label', 'Contact');
+		$expr[] = $search->compare('==', 'cms.status', 1);
+		$expr[] = $search->compare('>=', 'cms.mtime', '1970-01-01 00:00:00');
+		$expr[] = $search->compare('>=', 'cms.ctime', '1970-01-01 00:00:00');
+		$expr[] = $search->compare('==', 'cms.editor', $this->editor);
 
 		$param = ['text', 'default', $listItem->getRefId()];
-		$expr[] = $search->compare( '!=', $search->make( 'cms:has', $param ), null );
+		$expr[] = $search->compare('!=', $search->make('cms:has', $param), null);
 
 		$param = ['text', 'default'];
-		$expr[] = $search->compare( '!=', $search->make( 'cms:has', $param ), null );
+		$expr[] = $search->compare('!=', $search->make('cms:has', $param), null);
 
 		$param = ['text'];
-		$expr[] = $search->compare( '!=', $search->make( 'cms:has', $param ), null );
+		$expr[] = $search->compare('!=', $search->make('cms:has', $param), null);
 
-		$search->setConditions( $search->and( $expr ) );
-		$result = $this->object->search( $search, [], $total )->toArray();
-		$this->assertEquals( 1, count( $result ) );
-		$this->assertEquals( 1, $total );
+		$search->setConditions($search->and($expr));
+		$result = $this->object->search($search, [], $total)->toArray();
+		$this->assertEquals(1, count($result));
+		$this->assertEquals(1, $total);
 	}
 
 
 	public function testSearchItemsAll()
 	{
 		$search = $this->object->filter();
-		$search->setConditions( $search->compare( '==', 'cms.editor', $this->editor ) );
-		$this->assertEquals( 2, count( $this->object->search( $search )->toArray() ) );
+		$search->setConditions($search->compare('==', 'cms.editor', $this->editor));
+		$this->assertEquals(2, count($this->object->search($search)->toArray()));
 	}
 
 
 	public function testSearchItemsBase()
 	{
 		$total = 0;
-		$search = $this->object->filter( true );
+		$search = $this->object->filter(true);
 		$conditions = array(
-			$search->compare( '==', 'cms.editor', $this->editor ),
+			$search->compare('==', 'cms.editor', $this->editor),
 			$search->getConditions()
 		);
-		$search->setConditions( $search->and( $conditions ) );
-		$search->slice( 0, 1 );
-		$results = $this->object->search( $search, [], $total )->toArray();
-		$this->assertEquals( 1, count( $results ) );
-		$this->assertEquals( 1, $total );
-
-		foreach( $results as $itemId => $item ) {
-			$this->assertEquals( $itemId, $item->getId() );
+		$search->setConditions($search->and($conditions));
+		$search->slice(0, 1);
+		$results = $this->object->search($search, [], $total)->toArray();
+		$this->assertEquals(1, count($results));
+		$this->assertEquals(1, $total);
+
+		foreach ($results as $itemId => $item) {
+			$this->assertEquals($itemId, $item->getId());
 		}
 	}
 
 
 	public function testGetItem()
 	{
-		$item = $this->object->find( '/contact' );
+		$item = $this->object->find('/contact');
 
-		$actual = $this->object->get( $item->getId() );
-		$this->assertEquals( $item, $actual );
+		$actual = $this->object->get($item->getId());
+		$this->assertEquals($item, $actual);
 	}
 
 
 	public function testSaveUpdateDeleteItem()
 	{
-		$item = $this->object->find( '/contact' );
+		$item = $this->object->find('/contact');
 
-		$item->setId( null );
-		$item->setUrl( '/contact-test' );
-		$item->setLabel( 'Contact test' );
-		$resultSaved = $this->object->save( $item );
-		$itemSaved = $this->object->get( $item->getId() );
+		$item->setId(null);
+		$item->setUrl('/contact-test');
+		$item->setLabel('Contact test');
+		$resultSaved = $this->object->save($item);
+		$itemSaved = $this->object->get($item->getId());
 
 		$itemExp = clone $itemSaved;
-		$itemExp->setLabel( 'Contact test 2' );
-		$resultUpd = $this->object->save( $itemExp );
-		$itemUpd = $this->object->get( $itemExp->getId() );
+		$itemExp->setLabel('Contact test 2');
+		$resultUpd = $this->object->save($itemExp);
+		$itemUpd = $this->object->get($itemExp->getId());
 
-		$this->object->delete( $itemSaved->getId() );
+		$this->object->delete($itemSaved->getId());
 
 
-		$this->assertTrue( $item->getId() !== null );
-		$this->assertEquals( $item->getId(), $itemSaved->getId() );
-		$this->assertEquals( $item->getSiteId(), $itemSaved->getSiteId() );
-		$this->assertEquals( $item->getUrl(), $itemSaved->getUrl() );
-		$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->getUrl(), $itemSaved->getUrl());
+		$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->getUrl(), $itemUpd->getUrl() );
-		$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->getUrl(), $itemUpd->getUrl());
+		$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->assertInstanceOf( \Aimeos\MShop\Common\Item\Iface::class, $resultSaved );
-		$this->assertInstanceOf( \Aimeos\MShop\Common\Item\Iface::class, $resultUpd );
+		$this->assertInstanceOf(\Aimeos\MShop\Common\Item\Iface::class, $resultSaved);
+		$this->assertInstanceOf(\Aimeos\MShop\Common\Item\Iface::class, $resultUpd);
 
-		$this->expectException( \Aimeos\MShop\Exception::class );
-		$this->object->get( $itemSaved->getId() );
+		$this->expectException(\Aimeos\MShop\Exception::class);
+		$this->object->get($itemSaved->getId());
 	}
 
 
 	public function testGetSubManager()
 	{
-		$this->assertInstanceOf( \Aimeos\MShop\Common\Manager\Iface::class, $this->object->getSubManager( 'lists' ) );
-		$this->assertInstanceOf( \Aimeos\MShop\Common\Manager\Iface::class, $this->object->getSubManager( 'lists', 'Standard' ) );
+		$this->assertInstanceOf(\Aimeos\MShop\Common\Manager\Iface::class, $this->object->getSubManager('lists'));
+		$this->assertInstanceOf(\Aimeos\MShop\Common\Manager\Iface::class, $this->object->getSubManager('lists', 'Standard'));
 
-		$this->expectException( \Aimeos\MShop\Exception::class );
-		$this->object->getSubManager( 'unknown' );
+		$this->expectException(\Aimeos\MShop\Exception::class);
+		$this->object->getSubManager('unknown');
 	}
 
 
 	public function testGetSubManagerInvalidName()
 	{
-		$this->expectException( \Aimeos\MShop\Exception::class );
-		$this->object->getSubManager( 'lists', 'unknown' );
+		$this->expectException(\Aimeos\MShop\Exception::class);
+		$this->object->getSubManager('lists', 'unknown');
 	}
 }

lib/custom/tests/MShop/Cms/Item/StandardTest.php

Spacing +56 added lines, -56 removed lines

@@ -28,123 +28,123 @@ 
 			'cms.editor' => 'unitTestUser',
 		);
 
-		$this->object = new \Aimeos\MShop\Cms\Item\Standard( $this->values );
+		$this->object = new \Aimeos\MShop\Cms\Item\Standard($this->values);
 	}
 
 
 	protected function tearDown() : void
 	{
-		unset( $this->object );
+		unset($this->object);
 	}
 
 
 	public function testIsModified()
 	{
-		$this->assertFalse( $this->object->isModified() );
+		$this->assertFalse($this->object->isModified());
 	}
 
 
 	public function testGetId()
 	{
-		$this->assertEquals( '10', $this->object->getId() );
+		$this->assertEquals('10', $this->object->getId());
 	}
 
 
 	public function testSetId()
 	{
-		$return = $this->object->setId( null );
+		$return = $this->object->setId(null);
 
-		$this->assertInstanceOf( \Aimeos\MShop\Cms\Item\Iface::class, $return );
-		$this->assertNull( $this->object->getId() );
-		$this->assertTrue( $this->object->isModified() );
+		$this->assertInstanceOf(\Aimeos\MShop\Cms\Item\Iface::class, $return);
+		$this->assertNull($this->object->getId());
+		$this->assertTrue($this->object->isModified());
 	}
 
 
 	public function testGetSiteId()
 	{
-		$this->assertEquals( 99, $this->object->getSiteId() );
+		$this->assertEquals(99, $this->object->getSiteId());
 	}
 
 
 	public function testGetUrl()
 	{
-		$this->assertEquals( '/test', $this->object->getUrl() );
+		$this->assertEquals('/test', $this->object->getUrl());
 	}
 
 
 	public function testSetUrl()
 	{
-		$return = $this->object->setUrl( '/test2' );
+		$return = $this->object->setUrl('/test2');
 
-		$this->assertInstanceOf( \Aimeos\MShop\Cms\Item\Iface::class, $return );
-		$this->assertEquals( '/test2', $this->object->getUrl() );
-		$this->assertTrue( $this->object->isModified() );
+		$this->assertInstanceOf(\Aimeos\MShop\Cms\Item\Iface::class, $return);
+		$this->assertEquals('/test2', $this->object->getUrl());
+		$this->assertTrue($this->object->isModified());
 	}
 
 
 	public function testSetUrlTransform()
 	{
-		$return = $this->object->setUrl( '/Äpfel/Birnen' );
+		$return = $this->object->setUrl('/Äpfel/Birnen');
 
-		$this->assertInstanceOf( \Aimeos\MShop\Cms\Item\Iface::class, $return );
-		$this->assertEquals( '/Apfel/Birnen', $this->object->getUrl() );
-		$this->assertTrue( $this->object->isModified() );
+		$this->assertInstanceOf(\Aimeos\MShop\Cms\Item\Iface::class, $return);
+		$this->assertEquals('/Apfel/Birnen', $this->object->getUrl());
+		$this->assertTrue($this->object->isModified());
 	}
 
 
 	public function testGetLabel()
 	{
-		$this->assertEquals( 'unittest label', $this->object->getLabel() );
+		$this->assertEquals('unittest label', $this->object->getLabel());
 	}
 
 
 	public function testSetLabel()
 	{
-		$return = $this->object->setLabel( 'unittest set label' );
+		$return = $this->object->setLabel('unittest set label');
 
-		$this->assertInstanceOf( \Aimeos\MShop\Cms\Item\Iface::class, $return );
-		$this->assertEquals( 'unittest set label', $this->object->getLabel() );
-		$this->assertTrue( $this->object->isModified() );
+		$this->assertInstanceOf(\Aimeos\MShop\Cms\Item\Iface::class, $return);
+		$this->assertEquals('unittest set label', $this->object->getLabel());
+		$this->assertTrue($this->object->isModified());
 	}
 
 
 	public function testGetStatus()
 	{
-		$this->assertEquals( 2, $this->object->getStatus() );
+		$this->assertEquals(2, $this->object->getStatus());
 	}
 
 
 	public function testSetStatus()
 	{
-		$return = $this->object->setStatus( 0 );
+		$return = $this->object->setStatus(0);
 
-		$this->assertInstanceOf( \Aimeos\MShop\Cms\Item\Iface::class, $return );
-		$this->assertEquals( 0, $this->object->getStatus() );
-		$this->assertTrue( $this->object->isModified() );
+		$this->assertInstanceOf(\Aimeos\MShop\Cms\Item\Iface::class, $return);
+		$this->assertEquals(0, $this->object->getStatus());
+		$this->assertTrue($this->object->isModified());
 	}
 
 
 	public function testGetTimeModified()
 	{
-		$this->assertEquals( '2011-01-01 00:00:02', $this->object->getTimeModified() );
+		$this->assertEquals('2011-01-01 00:00:02', $this->object->getTimeModified());
 	}
 
 
 	public function testGetTimeCreated()
 	{
-		$this->assertEquals( '2011-01-01 00:00:01', $this->object->getTimeCreated() );
+		$this->assertEquals('2011-01-01 00:00:01', $this->object->getTimeCreated());
 	}
 
 
 	public function testGetEditor()
 	{
-		$this->assertEquals( 'unitTestUser', $this->object->getEditor() );
+		$this->assertEquals('unitTestUser', $this->object->getEditor());
 	}
 
 
 	public function testGetResourceType()
 	{
-		$this->assertEquals( 'cms', $this->object->getResourceType() );
+		$this->assertEquals('cms', $this->object->getResourceType());
 	}
 
 
@@ -159,48 +159,48 @@ 
 			'cms.status' => 0,
 		);
 
-		$item = $item->fromArray( $entries, true );
+		$item = $item->fromArray($entries, true);
 
-		$this->assertEquals( [], $entries );
-		$this->assertEquals( '', $item->getSiteId() );
-		$this->assertEquals( $list['cms.id'], $item->getId() );
-		$this->assertEquals( $list['cms.url'], $item->getUrl() );
-		$this->assertEquals( $list['cms.label'], $item->getLabel() );
-		$this->assertEquals( $list['cms.status'], $item->getStatus() );
+		$this->assertEquals([], $entries);
+		$this->assertEquals('', $item->getSiteId());
+		$this->assertEquals($list['cms.id'], $item->getId());
+		$this->assertEquals($list['cms.url'], $item->getUrl());
+		$this->assertEquals($list['cms.label'], $item->getLabel());
+		$this->assertEquals($list['cms.status'], $item->getStatus());
 	}
 
 
 	public function testToArray()
 	{
-		$data = $this->object->toArray( true );
+		$data = $this->object->toArray(true);
 
-		$this->assertEquals( count( $this->values ), count( $data ) );
+		$this->assertEquals(count($this->values), count($data));
 
-		$this->assertEquals( $this->object->getId(), $data['cms.id'] );
-		$this->assertEquals( $this->object->getSiteId(), $data['cms.siteid'] );
-		$this->assertEquals( $this->object->getUrl(), $data['cms.url'] );
-		$this->assertEquals( $this->object->getLabel(), $data['cms.label'] );
-		$this->assertEquals( $this->object->getStatus(), $data['cms.status'] );
-		$this->assertEquals( $this->object->getTimeCreated(), $data['cms.ctime'] );
-		$this->assertEquals( $this->object->getTimeModified(), $data['cms.mtime'] );
-		$this->assertEquals( $this->object->getEditor(), $data['cms.editor'] );
+		$this->assertEquals($this->object->getId(), $data['cms.id']);
+		$this->assertEquals($this->object->getSiteId(), $data['cms.siteid']);
+		$this->assertEquals($this->object->getUrl(), $data['cms.url']);
+		$this->assertEquals($this->object->getLabel(), $data['cms.label']);
+		$this->assertEquals($this->object->getStatus(), $data['cms.status']);
+		$this->assertEquals($this->object->getTimeCreated(), $data['cms.ctime']);
+		$this->assertEquals($this->object->getTimeModified(), $data['cms.mtime']);
+		$this->assertEquals($this->object->getEditor(), $data['cms.editor']);
 	}
 
 
 	public function testIsAvailable()
 	{
-		$this->assertTrue( $this->object->isAvailable() );
-		$this->object->setAvailable( false );
-		$this->assertFalse( $this->object->isAvailable() );
+		$this->assertTrue($this->object->isAvailable());
+		$this->object->setAvailable(false);
+		$this->assertFalse($this->object->isAvailable());
 	}
 
 
 	public function testIsAvailableOnStatus()
 	{
-		$this->assertTrue( $this->object->isAvailable() );
-		$this->object->setStatus( 0 );
-		$this->assertFalse( $this->object->isAvailable() );
+		$this->assertTrue($this->object->isAvailable());
+		$this->object->setStatus(0);
+		$this->assertFalse($this->object->isAvailable());
 		$this->object->setStatus( -1 );
-		$this->assertFalse( $this->object->isAvailable() );
+		$this->assertFalse($this->object->isAvailable());
 	}
 }

lib/custom/tests/bootstrap.php

Spacing +6 added lines, -6 removed lines

@@ -4,17 +4,17 @@
  * Set error reporting to maximum
  */
 error_reporting( -1 );
-ini_set( 'display_errors', true );
+ini_set('display_errors', true);
 
-date_default_timezone_set( 'UTC' );
+date_default_timezone_set('UTC');
 
 /*
  * 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' );
+setlocale(LC_ALL, 'en_US.UTF-8');
+setlocale(LC_CTYPE, 'en_US.UTF-8');
+setlocale(LC_NUMERIC, 'POSIX');
+setlocale(LC_TIME, 'POSIX');
 
 
 require_once 'TestHelper.php';

lib/custom/setup/default/schema/cms.php

Spacing +61 added lines, -61 removed lines

@@ -8,81 +8,81 @@
 
 return [
 	'table' => [
-		'mshop_cms' => function ( \Doctrine\DBAL\Schema\Schema $schema ) {
+		'mshop_cms' => function(\Doctrine\DBAL\Schema\Schema $schema) {
 
-			$table = $schema->createTable( 'mshop_cms' );
-			$table->addOption( 'engine', 'InnoDB' );
+			$table = $schema->createTable('mshop_cms');
+			$table->addOption('engine', 'InnoDB');
 
-			$table->addColumn( 'id', 'integer', ['autoincrement' => true] );
-			$table->addColumn( 'siteid', 'string', ['length' => 255] );
-			$table->addColumn( 'url', 'string', ['length' => 255] );
-			$table->addColumn( 'label', 'string', ['length' => 255] );
-			$table->addColumn( 'status', 'smallint', [] );
-			$table->addColumn( 'ctime', 'datetime', [] );
-			$table->addColumn( 'mtime', 'datetime', [] );
-			$table->addColumn( 'editor', 'string', ['length' => 255] );
+			$table->addColumn('id', 'integer', ['autoincrement' => true]);
+			$table->addColumn('siteid', 'string', ['length' => 255]);
+			$table->addColumn('url', 'string', ['length' => 255]);
+			$table->addColumn('label', 'string', ['length' => 255]);
+			$table->addColumn('status', 'smallint', []);
+			$table->addColumn('ctime', 'datetime', []);
+			$table->addColumn('mtime', 'datetime', []);
+			$table->addColumn('editor', 'string', ['length' => 255]);
 
-			$table->setPrimaryKey( ['id'], 'pk_mscms_id' );
-			$table->addUniqueIndex( ['siteid', 'url'], 'unq_mscms_sid_url' );
-			$table->addIndex( ['siteid', 'status'], 'unq_mscms_sid_status' );
-			$table->addIndex( ['siteid', 'label'], 'unq_mscms_sid_label' );
+			$table->setPrimaryKey(['id'], 'pk_mscms_id');
+			$table->addUniqueIndex(['siteid', 'url'], 'unq_mscms_sid_url');
+			$table->addIndex(['siteid', 'status'], 'unq_mscms_sid_status');
+			$table->addIndex(['siteid', 'label'], 'unq_mscms_sid_label');
 
 			return $schema;
 		},
 
-		'mshop_cms_list_type' => function( \Doctrine\DBAL\Schema\Schema $schema ) {
+		'mshop_cms_list_type' => function(\Doctrine\DBAL\Schema\Schema $schema) {
 
-			$table = $schema->createTable( 'mshop_cms_list_type' );
-			$table->addOption( 'engine', 'InnoDB' );
+			$table = $schema->createTable('mshop_cms_list_type');
+			$table->addOption('engine', 'InnoDB');
 
-			$table->addColumn( 'id', 'integer', array( 'autoincrement' => true ) );
-			$table->addColumn( 'siteid', 'string', ['length' => 255] );
-			$table->addColumn( 'domain', 'string', array( 'length' => 32 ) );
-			$table->addColumn( 'code', 'string', array( 'length' => 64, 'customSchemaOptions' => ['charset' => 'binary'] ) );
-			$table->addColumn( 'label', 'string', array( 'length' => 255 ) );
-			$table->addColumn( 'pos', 'integer', ['default' => 0] );
-			$table->addColumn( 'status', 'smallint', [] );
-			$table->addColumn( 'mtime', 'datetime', [] );
-			$table->addColumn( 'ctime', 'datetime', [] );
-			$table->addColumn( 'editor', 'string', array( 'length' => 255 ) );
+			$table->addColumn('id', 'integer', array('autoincrement' => true));
+			$table->addColumn('siteid', 'string', ['length' => 255]);
+			$table->addColumn('domain', 'string', array('length' => 32));
+			$table->addColumn('code', 'string', array('length' => 64, 'customSchemaOptions' => ['charset' => 'binary']));
+			$table->addColumn('label', 'string', array('length' => 255));
+			$table->addColumn('pos', 'integer', ['default' => 0]);
+			$table->addColumn('status', 'smallint', []);
+			$table->addColumn('mtime', 'datetime', []);
+			$table->addColumn('ctime', 'datetime', []);
+			$table->addColumn('editor', 'string', array('length' => 255));
 
-			$table->setPrimaryKey( array( 'id' ), 'pk_mscmslity_id' );
-			$table->addUniqueIndex( array( 'siteid', 'domain', 'code' ), 'unq_mscmslity_sid_dom_code' );
-			$table->addIndex( array( 'siteid', 'status', 'pos' ), 'idx_mscmslity_sid_status_pos' );
-			$table->addIndex( array( 'siteid', 'label' ), 'idx_mscmslity_sid_label' );
-			$table->addIndex( array( 'siteid', 'code' ), 'idx_mscmslity_sid_code' );
+			$table->setPrimaryKey(array('id'), 'pk_mscmslity_id');
+			$table->addUniqueIndex(array('siteid', 'domain', 'code'), 'unq_mscmslity_sid_dom_code');
+			$table->addIndex(array('siteid', 'status', 'pos'), 'idx_mscmslity_sid_status_pos');
+			$table->addIndex(array('siteid', 'label'), 'idx_mscmslity_sid_label');
+			$table->addIndex(array('siteid', 'code'), 'idx_mscmslity_sid_code');
 
 			return $schema;
 		},
 
-		'mshop_cms_list' => function( \Doctrine\DBAL\Schema\Schema $schema ) {
-
-			$table = $schema->createTable( 'mshop_cms_list' );
-			$table->addOption( 'engine', 'InnoDB' );
-
-			$table->addColumn( 'id', 'integer', array( 'autoincrement' => true ) );
-			$table->addColumn( 'parentid', 'integer', [] );
-			$table->addColumn( 'siteid', 'string', ['length' => 255] );
-			$table->addColumn( 'key', 'string', array( 'length' => 134, 'default' => '', 'customSchemaOptions' => ['charset' => 'binary'] ) );
-			$table->addColumn( 'type', 'string', array( 'length' => 64, 'customSchemaOptions' => ['charset' => 'binary'] ) );
-			$table->addColumn( 'domain', 'string', array( 'length' => 32 ) );
-			$table->addColumn( 'refid', 'string', array( 'length' => 36, 'customSchemaOptions' => ['charset' => 'binary'] ) );
-			$table->addColumn( 'start', 'datetime', array( 'notnull' => false ) );
-			$table->addColumn( 'end', 'datetime', array( 'notnull' => false ) );
-			$table->addColumn( 'config', 'text', array( 'length' => 0xffff ) );
-			$table->addColumn( 'pos', 'integer', [] );
-			$table->addColumn( 'status', 'smallint', [] );
-			$table->addColumn( 'mtime', 'datetime', [] );
-			$table->addColumn( 'ctime', 'datetime', [] );
-			$table->addColumn( 'editor', 'string', array( 'length' => 255 ) );
-
-			$table->setPrimaryKey( array( 'id' ), 'pk_mscmsli_id' );
-			$table->addUniqueIndex( array( 'parentid', 'siteid', 'domain', 'type', 'refid' ), 'unq_mscmsli_pid_sid_dm_ty_rid' );
-			$table->addIndex( array( 'siteid', 'key' ), 'idx_mscmsli_sid_key' );
-			$table->addIndex( array( 'parentid' ), 'fk_mscmsli_pid' );
-
-			$table->addForeignKeyConstraint( 'mshop_cms', array( 'parentid' ), array( 'id' ),
-				array( 'onUpdate' => 'CASCADE', 'onDelete' => 'CASCADE' ), 'fk_mscmsli_pid' );
+		'mshop_cms_list' => function(\Doctrine\DBAL\Schema\Schema $schema) {
+
+			$table = $schema->createTable('mshop_cms_list');
+			$table->addOption('engine', 'InnoDB');
+
+			$table->addColumn('id', 'integer', array('autoincrement' => true));
+			$table->addColumn('parentid', 'integer', []);
+			$table->addColumn('siteid', 'string', ['length' => 255]);
+			$table->addColumn('key', 'string', array('length' => 134, 'default' => '', 'customSchemaOptions' => ['charset' => 'binary']));
+			$table->addColumn('type', 'string', array('length' => 64, 'customSchemaOptions' => ['charset' => 'binary']));
+			$table->addColumn('domain', 'string', array('length' => 32));
+			$table->addColumn('refid', 'string', array('length' => 36, 'customSchemaOptions' => ['charset' => 'binary']));
+			$table->addColumn('start', 'datetime', array('notnull' => false));
+			$table->addColumn('end', 'datetime', array('notnull' => false));
+			$table->addColumn('config', 'text', array('length' => 0xffff));
+			$table->addColumn('pos', 'integer', []);
+			$table->addColumn('status', 'smallint', []);
+			$table->addColumn('mtime', 'datetime', []);
+			$table->addColumn('ctime', 'datetime', []);
+			$table->addColumn('editor', 'string', array('length' => 255));
+
+			$table->setPrimaryKey(array('id'), 'pk_mscmsli_id');
+			$table->addUniqueIndex(array('parentid', 'siteid', 'domain', 'type', 'refid'), 'unq_mscmsli_pid_sid_dm_ty_rid');
+			$table->addIndex(array('siteid', 'key'), 'idx_mscmsli_sid_key');
+			$table->addIndex(array('parentid'), 'fk_mscmsli_pid');
+
+			$table->addForeignKeyConstraint('mshop_cms', array('parentid'), array('id'),
+				array('onUpdate' => 'CASCADE', 'onDelete' => 'CASCADE'), 'fk_mscmsli_pid');
 
 			return $schema;
 		},