aimeos / ai-ezpublish

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/EzuserAddAddress.php

Braces +1 added lines, -2 removed lines

@@ -50,8 +50,7 @@
 		{
 			$this->executeList( $sql, 'db-customer' );
 			$this->status( 'done' );
-		}
-		else
+		} else
 		{
 			$this->status( 'OK' );
 		}

Spacing +50 added lines, -50 removed lines

@@ -21,7 +21,7 @@ 
 	 */
 	public function getPreDependencies()
 	{
-		return array( 'TablesCreateEzpublish' );
+		return array('TablesCreateEzpublish');
 	}
 
 
@@ -41,29 +41,29 @@ 
 	 */
 	public function migrate()
 	{
-		$this->msg( 'Adding address fields to ezuser table', 0 );
+		$this->msg('Adding address fields to ezuser table', 0);
 
-		$dbal = $this->getConnection( 'db-customer' )->getRawObject();
+		$dbal = $this->getConnection('db-customer')->getRawObject();
 
-		if( !( $dbal instanceof \Doctrine\DBAL\Connection ) ) {
-			throw new \Aimeos\MW\Setup\Exception( 'Not a DBAL connection' );
+		if (!($dbal instanceof \Doctrine\DBAL\Connection)) {
+			throw new \Aimeos\MW\Setup\Exception('Not a DBAL connection');
 		}
 
 
 		$fromSchema = $dbal->getSchemaManager()->createSchema();
 		$toSchema = clone $fromSchema;
 
-		$this->addIndexes( $this->addColumns( $toSchema->getTable( 'ezuser' ) ) );
-		$sql = $fromSchema->getMigrateToSql( $toSchema, $dbal->getDatabasePlatform() );
+		$this->addIndexes($this->addColumns($toSchema->getTable('ezuser')));
+		$sql = $fromSchema->getMigrateToSql($toSchema, $dbal->getDatabasePlatform());
 
-		if( $sql !== [] )
+		if ($sql !== [])
 		{
-			$this->executeList( $sql, 'db-customer' );
-			$this->status( 'done' );
+			$this->executeList($sql, 'db-customer');
+			$this->status('done');
 		}
 		else
 		{
-			$this->status( 'OK' );
+			$this->status('OK');
 		}
 	}
 
@@ -74,38 +74,38 @@ 
 	 * @param \Doctrine\DBAL\Schema\Table $table Table object
 	 * @return \Doctrine\DBAL\Schema\Table Updated table object
 	 */
-	protected function addColumns( \Doctrine\DBAL\Schema\Table $table )
+	protected function addColumns(\Doctrine\DBAL\Schema\Table $table)
 	{
 		$columns = array(
-			'company' => array( 'string', array( 'length' => 100 ) ),
-			'vatid' => array( 'string', array( 'length' => 32 ) ),
-			'salutation' => array( 'string', array( 'length' => 8 ) ),
-			'title' => array( 'string', array( 'length' => 64 ) ),
-			'firstname' => array( 'string', array( 'length' => 64 ) ),
-			'lastname' => array( 'string', array( 'length' => 64 ) ),
-			'address1' => array( 'string', array( 'length' => 255 ) ),
-			'address2' => array( 'string', array( 'length' => 255 ) ),
-			'address3' => array( 'string', array( 'length' => 255 ) ),
-			'postal' => array( 'string', array( 'length' => 16 ) ),
-			'city' => array( 'string', array( 'length' => 255 ) ),
-			'state' => array( 'string', array( 'length' => 255 ) ),
-			'langid' => array( 'string', array( 'length' => 5, 'notnull' => false ) ),
-			'countryid' => array( 'string', array( 'length' => 2, 'notnull' => false, 'fixed' => true ) ),
-			'telephone' => array( 'string', array( 'length' => 32 ) ),
-			'telefax' => array( 'string', array( 'length' => 32 ) ),
-			'website' => array( 'string', array( 'length' => 255 ) ),
-			'birthday' => array( 'date', array( 'notnull' => false ) ),
-			'vdate' => array( 'date', array( 'notnull' => false ) ),
-			'status' => array( 'smallint', [] ),
-			'mtime' => array( 'datetime', [] ),
-			'ctime' => array( 'datetime', [] ),
-			'editor' => array( 'string', array( 'length' => 255 ) ),
+			'company' => array('string', array('length' => 100)),
+			'vatid' => array('string', array('length' => 32)),
+			'salutation' => array('string', array('length' => 8)),
+			'title' => array('string', array('length' => 64)),
+			'firstname' => array('string', array('length' => 64)),
+			'lastname' => array('string', array('length' => 64)),
+			'address1' => array('string', array('length' => 255)),
+			'address2' => array('string', array('length' => 255)),
+			'address3' => array('string', array('length' => 255)),
+			'postal' => array('string', array('length' => 16)),
+			'city' => array('string', array('length' => 255)),
+			'state' => array('string', array('length' => 255)),
+			'langid' => array('string', array('length' => 5, 'notnull' => false)),
+			'countryid' => array('string', array('length' => 2, 'notnull' => false, 'fixed' => true)),
+			'telephone' => array('string', array('length' => 32)),
+			'telefax' => array('string', array('length' => 32)),
+			'website' => array('string', array('length' => 255)),
+			'birthday' => array('date', array('notnull' => false)),
+			'vdate' => array('date', array('notnull' => false)),
+			'status' => array('smallint', []),
+			'mtime' => array('datetime', []),
+			'ctime' => array('datetime', []),
+			'editor' => array('string', array('length' => 255)),
 		);
 
-		foreach( $columns as $name => $def )
+		foreach ($columns as $name => $def)
 		{
-			if( $table->hasColumn( $name ) === false ) {
-				$table->addColumn( $name, $def[0], $def[1] );
+			if ($table->hasColumn($name) === false) {
+				$table->addColumn($name, $def[0], $def[1]);
 			}
 		}
 
@@ -119,23 +119,23 @@ 
 	 * @param \Doctrine\DBAL\Schema\Table $table Table object
 	 * @return \Doctrine\DBAL\Schema\Table Updated table object
 	 */
-	protected function addIndexes( \Doctrine\DBAL\Schema\Table $table )
+	protected function addIndexes(\Doctrine\DBAL\Schema\Table $table)
 	{
 		$indexes = array(
-			'idx_ezpus_langid' => array( 'langid' ),
-			'idx_ezpus_status_ln_fn' => array( 'status', 'lastname', 'firstname' ),
-			'idx_ezpus_status_ad1_ad2' => array( 'status', 'address1', 'address2' ),
-			'idx_ezpus_status_postal_city' => array( 'status', 'postal', 'city' ),
-			'idx_ezpus_lastname' => array( 'lastname' ),
-			'idx_ezpus_address1' => array( 'address1' ),
-			'idx_ezpus_postal' => array( 'postal' ),
-			'idx_ezpus_city' => array( 'city' ),
+			'idx_ezpus_langid' => array('langid'),
+			'idx_ezpus_status_ln_fn' => array('status', 'lastname', 'firstname'),
+			'idx_ezpus_status_ad1_ad2' => array('status', 'address1', 'address2'),
+			'idx_ezpus_status_postal_city' => array('status', 'postal', 'city'),
+			'idx_ezpus_lastname' => array('lastname'),
+			'idx_ezpus_address1' => array('address1'),
+			'idx_ezpus_postal' => array('postal'),
+			'idx_ezpus_city' => array('city'),
 		);
 
-		foreach( $indexes as $name => $def )
+		foreach ($indexes as $name => $def)
 		{
-			if( $table->hasIndex( $name ) === false ) {
-				$table->addIndex( $def, $name );
+			if ($table->hasIndex($name) === false) {
+				$table->addIndex($def, $name);
 			}
 		}
 

lib/custom/tests/MShop/Customer/Manager/Lists/Type/EzpublishTest.php

Spacing +10 added lines, -10 removed lines

@@ -16,39 +16,39 @@
 
 	protected function setUp()
 	{
-		$manager = \Aimeos\MShop\Customer\Manager\Factory::createManager( \TestHelper::getContext(), 'Ezpublish' );
+		$manager = \Aimeos\MShop\Customer\Manager\Factory::createManager(\TestHelper::getContext(), 'Ezpublish');
 
-		$listManager = $manager->getSubManager( 'lists', 'Ezpublish' );
-		$this->object = $listManager->getSubManager( 'type', 'Ezpublish' );
+		$listManager = $manager->getSubManager('lists', 'Ezpublish');
+		$this->object = $listManager->getSubManager('type', 'Ezpublish');
 	}
 
 
 	protected function tearDown()
 	{
-		unset( $this->object );
+		unset($this->object);
 	}
 
 
 	public function testCleanup()
 	{
-		$this->object->cleanup( array( -1 ) );
+		$this->object->cleanup(array( -1 ));
 	}
 
 
 	public function testGetSearchAttributes()
 	{
 		$attributes = $this->object->getSearchAttributes();
-		$this->assertGreaterThan( 0, count( $attributes ) );
+		$this->assertGreaterThan(0, count($attributes));
 
-		foreach( $attributes as $attribute ) {
-			$this->assertInstanceOf( '\Aimeos\MW\Criteria\Attribute\Iface', $attribute );
+		foreach ($attributes as $attribute) {
+			$this->assertInstanceOf('\Aimeos\MW\Criteria\Attribute\Iface', $attribute);
 		}
 	}
 
 
 	public function testGetSubManager()
 	{
-		$this->setExpectedException( '\\Aimeos\\MShop\\Exception' );
-		$this->object->getSubManager( 'unknown' );
+		$this->setExpectedException('\\Aimeos\\MShop\\Exception');
+		$this->object->getSubManager('unknown');
 	}
 }

lib/custom/tests/MShop/Customer/Manager/Lists/EzpublishTest.php

Spacing +16 added lines, -16 removed lines

@@ -16,57 +16,57 @@
 
 	protected function setUp()
 	{
-		$manager = \Aimeos\MShop\Customer\Manager\Factory::createManager( \TestHelper::getContext(), 'Ezpublish' );
-		$this->object = $manager->getSubManager( 'lists', 'Ezpublish' );
+		$manager = \Aimeos\MShop\Customer\Manager\Factory::createManager(\TestHelper::getContext(), 'Ezpublish');
+		$this->object = $manager->getSubManager('lists', 'Ezpublish');
 	}
 
 
 	protected function tearDown()
 	{
-		unset( $this->object );
+		unset($this->object);
 	}
 
 
 	public function testCleanup()
 	{
-		$this->object->cleanup( array( -1 ) );
+		$this->object->cleanup(array( -1 ));
 	}
 
 
 	public function testGetSearchAttributes()
 	{
 		$attributes = $this->object->getSearchAttributes();
-		$this->assertGreaterThan( 0, count( $attributes ) );
+		$this->assertGreaterThan(0, count($attributes));
 
-		foreach( $attributes as $attribute ) {
-			$this->assertInstanceOf( '\Aimeos\MW\Criteria\Attribute\Iface', $attribute );
+		foreach ($attributes as $attribute) {
+			$this->assertInstanceOf('\Aimeos\MW\Criteria\Attribute\Iface', $attribute);
 		}
 	}
 
 
 	public function testGetSubManager()
 	{
-		$this->assertInstanceOf( '\\Aimeos\\MShop\\Common\\Manager\\Iface', $this->object->getSubManager('type') );
-		$this->assertInstanceOf( '\\Aimeos\\MShop\\Common\\Manager\\Iface', $this->object->getSubManager('type', 'Standard') );
+		$this->assertInstanceOf('\\Aimeos\\MShop\\Common\\Manager\\Iface', $this->object->getSubManager('type'));
+		$this->assertInstanceOf('\\Aimeos\\MShop\\Common\\Manager\\Iface', $this->object->getSubManager('type', 'Standard'));
 
-		$this->setExpectedException( '\\Aimeos\\MShop\\Exception' );
-		$this->object->getSubManager( 'unknown' );
+		$this->setExpectedException('\\Aimeos\\MShop\\Exception');
+		$this->object->getSubManager('unknown');
 	}
 
 
 	public function testSaveItemInvalidItem()
 	{
-		$this->setExpectedException( '\Aimeos\MShop\Exception' );
-		$this->object->saveItem( new \Aimeos\MShop\Common\Item\Type\Standard( 'common.lists.type.' ) );
+		$this->setExpectedException('\Aimeos\MShop\Exception');
+		$this->object->saveItem(new \Aimeos\MShop\Common\Item\Type\Standard('common.lists.type.'));
 	}
 
 
 	public function testSaveItemInvalidDomain()
 	{
 		$item = $this->object->createItem();
-		$item->setDomain( 'customer/group' );
+		$item->setDomain('customer/group');
 
-		$this->setExpectedException( '\Aimeos\MShop\Customer\Exception' );
-		$this->object->saveItem( $item );
+		$this->setExpectedException('\Aimeos\MShop\Customer\Exception');
+		$this->object->saveItem($item);
 	}
 }

lib/custom/tests/MShop/Customer/Manager/Address/EzpublishTest.php

Spacing +10 added lines, -10 removed lines

@@ -16,44 +16,44 @@
 
 	protected function setUp()
 	{
-		$customer = new \Aimeos\MShop\Customer\Manager\Ezpublish( \TestHelper::getContext() );
+		$customer = new \Aimeos\MShop\Customer\Manager\Ezpublish(\TestHelper::getContext());
 
-		$this->object = $customer->getSubManager( 'address', 'Ezpublish' );
+		$this->object = $customer->getSubManager('address', 'Ezpublish');
 	}
 
 
 	protected function tearDown()
 	{
-		unset( $this->object );
+		unset($this->object);
 	}
 
 
 	public function testCleanup()
 	{
-		$this->object->cleanup( array( -1 ) );
+		$this->object->cleanup(array( -1 ));
 	}
 
 
 	public function testDeleteItems()
 	{
-		$this->object->deleteItems( array( -1 ) );
+		$this->object->deleteItems(array( -1 ));
 	}
 
 
 	public function testGetSearchAttributes()
 	{
 		$attributes = $this->object->getSearchAttributes();
-		$this->assertGreaterThan( 0, count( $attributes ) );
+		$this->assertGreaterThan(0, count($attributes));
 
-		foreach( $attributes as $attribute ) {
-			$this->assertInstanceOf( '\\Aimeos\\MW\\Criteria\\Attribute\\Iface', $attribute );
+		foreach ($attributes as $attribute) {
+			$this->assertInstanceOf('\\Aimeos\\MW\\Criteria\\Attribute\\Iface', $attribute);
 		}
 	}
 
 
 	public function testGetSubManager()
 	{
-		$this->setExpectedException( '\\Aimeos\\MShop\\Exception' );
-		$this->object->getSubManager( 'unknown' );
+		$this->setExpectedException('\\Aimeos\\MShop\\Exception');
+		$this->object->getSubManager('unknown');
 	}
 }

lib/custom/tests/MShop/Customer/Manager/Group/EzpublishTest.php

Spacing +32 added lines, -32 removed lines

@@ -18,91 +18,91 @@
 	protected function setUp()
 	{
 		$this->context = \TestHelper::getContext();
-		$customer = new \Aimeos\MShop\Customer\Manager\Ezpublish( $this->context );
+		$customer = new \Aimeos\MShop\Customer\Manager\Ezpublish($this->context);
 
-		$this->object = $customer->getSubManager( 'group', 'Ezpublish' );
+		$this->object = $customer->getSubManager('group', 'Ezpublish');
 	}
 
 
 	protected function tearDown()
 	{
-		unset( $this->object );
+		unset($this->object);
 	}
 
 
 	public function testCleanup()
 	{
-		$this->object->cleanup( array( -1 ) );
+		$this->object->cleanup(array( -1 ));
 	}
 
 
 	public function testDeleteItems()
 	{
-		$this->setExpectedException( '\\Aimeos\\MShop\\Exception' );
-		$this->object->deleteItems( array( -1 ) );
+		$this->setExpectedException('\\Aimeos\\MShop\\Exception');
+		$this->object->deleteItems(array( -1 ));
 	}
 
 
 	public function testGetSearchAttributes()
 	{
 		$attributes = $this->object->getSearchAttributes();
-		$this->assertGreaterThan( 0, count( $attributes ) );
+		$this->assertGreaterThan(0, count($attributes));
 
-		foreach( $attributes as $attribute ) {
-			$this->assertInstanceOf( '\\Aimeos\\MW\\Criteria\\Attribute\\Iface', $attribute );
+		foreach ($attributes as $attribute) {
+			$this->assertInstanceOf('\\Aimeos\\MW\\Criteria\\Attribute\\Iface', $attribute);
 		}
 	}
 
 
 	public function testGetSubManager()
 	{
-		$this->setExpectedException( '\\Aimeos\\MShop\\Exception' );
-		$this->object->getSubManager( 'unknown' );
+		$this->setExpectedException('\\Aimeos\\MShop\\Exception');
+		$this->object->getSubManager('unknown');
 	}
 
 
 	public function testSaveItem()
 	{
-		$this->setExpectedException( '\\Aimeos\\MShop\\Customer\\Exception' );
-		$this->object->saveItem( new \Aimeos\MShop\Common\Item\Lists\Standard( 'common.lists.' ) );
+		$this->setExpectedException('\\Aimeos\\MShop\\Customer\\Exception');
+		$this->object->saveItem(new \Aimeos\MShop\Common\Item\Lists\Standard('common.lists.'));
 	}
 
 
 	public function testSearchItems()
 	{
-		$mock = $this->getMockBuilder( '\Aimeos\MShop\Customer\Manager\Group\Ezpublish' )
-			->setConstructorArgs( array( \TestHelper::getContext() ) )
-			->setMethods( array( 'searchItemsBase' ) )
+		$mock = $this->getMockBuilder('\Aimeos\MShop\Customer\Manager\Group\Ezpublish')
+			->setConstructorArgs(array(\TestHelper::getContext()))
+			->setMethods(array('searchItemsBase'))
 			->getMock();
 
-		$result = $this->getMockBuilder( '\Aimeos\MW\DB\Result\Iface' )
-			->setMethods( array( 'affectedRows', 'fetch', 'finish', 'nextResult' ) )
+		$result = $this->getMockBuilder('\Aimeos\MW\DB\Result\Iface')
+			->setMethods(array('affectedRows', 'fetch', 'finish', 'nextResult'))
 			->getMock();
 
-		$mock->expects( $this->once() )->method( 'searchItemsBase' )
-			->will( $this->returnValue( $result ) );
+		$mock->expects($this->once())->method('searchItemsBase')
+			->will($this->returnValue($result));
 
-		$result->expects( $this->exactly( 2 ) )->method( 'fetch' )
-			->will( $this->onConsecutiveCalls( array( 'customer.group.id' => 1 ), false ) );
+		$result->expects($this->exactly(2))->method('fetch')
+			->will($this->onConsecutiveCalls(array('customer.group.id' => 1), false));
 
-		$result = $mock->searchItems( $mock->createSearch() );
+		$result = $mock->searchItems($mock->createSearch());
 
-		$this->assertEquals( 1, count( $result ) );
-		$this->assertInstanceOf( '\Aimeos\MShop\Customer\Item\Group\Iface', $result[1] );
+		$this->assertEquals(1, count($result));
+		$this->assertInstanceOf('\Aimeos\MShop\Customer\Item\Group\Iface', $result[1]);
 	}
 
 
 	public function testSearchItemsException()
 	{
-		$mock = $this->getMockBuilder( '\Aimeos\MShop\Customer\Manager\Group\Ezpublish' )
-			->setConstructorArgs( array( \TestHelper::getContext() ) )
-			->setMethods( array( 'searchItemsBase' ) )
+		$mock = $this->getMockBuilder('\Aimeos\MShop\Customer\Manager\Group\Ezpublish')
+			->setConstructorArgs(array(\TestHelper::getContext()))
+			->setMethods(array('searchItemsBase'))
 			->getMock();
 
-		$mock->expects( $this->once() )->method( 'searchItemsBase' )
-			->will( $this->throwException( new \Aimeos\MShop\Exception() ) );
+		$mock->expects($this->once())->method('searchItemsBase')
+			->will($this->throwException(new \Aimeos\MShop\Exception()));
 
-		$this->setExpectedException( '\\Aimeos\\MShop\\Exception' );
-		$mock->searchItems( $mock->createSearch() );
+		$this->setExpectedException('\\Aimeos\\MShop\\Exception');
+		$mock->searchItems($mock->createSearch());
 	}
 }

lib/custom/tests/MShop/Customer/Manager/EzpublishTest.php

Spacing +99 added lines, -99 removed lines

@@ -18,206 +18,206 @@
 
 	protected function setUp()
 	{
-		if( !interface_exists( 'eZ\Publish\API\Repository\UserService' ) ) {
-			$this->markTestSkipped( 'Install ezsystems/ezpublish-api first' );
+		if (!interface_exists('eZ\Publish\API\Repository\UserService')) {
+			$this->markTestSkipped('Install ezsystems/ezpublish-api first');
 		}
 
 		$this->context = \TestHelper::getContext();
 
-		$this->object = new \Aimeos\MShop\Customer\Manager\Ezpublish( $this->context );
-		$this->address = new \Aimeos\MShop\Common\Item\Address\Standard( 'common.address.' );
+		$this->object = new \Aimeos\MShop\Customer\Manager\Ezpublish($this->context);
+		$this->address = new \Aimeos\MShop\Common\Item\Address\Standard('common.address.');
 	}
 
 
 	protected function tearDown()
 	{
-		unset( $this->object, $this->context, $this->address );
+		unset($this->object, $this->context, $this->address);
 	}
 
 
 	public function testCleanup()
 	{
-		$this->object->cleanup( array( -1 ) );
+		$this->object->cleanup(array( -1 ));
 	}
 
 
 	public function testCreateItem()
 	{
-		$this->assertInstanceOf( 'Aimeos\MShop\Customer\Item\Iface', $this->object->createItem() );
+		$this->assertInstanceOf('Aimeos\MShop\Customer\Item\Iface', $this->object->createItem());
 	}
 
 
 	public function testDeleteItems()
 	{
-		$mock = $this->getMockBuilder( 'eZ\Publish\API\Repository\UserService' )->getMock();
-		$user = $this->getMockBuilder( 'eZ\Publish\API\Repository\Values\User\User' )->getMock();
+		$mock = $this->getMockBuilder('eZ\Publish\API\Repository\UserService')->getMock();
+		$user = $this->getMockBuilder('eZ\Publish\API\Repository\Values\User\User')->getMock();
 
-		$mock->expects( $this->once() )->method( 'loadUser' )->will( $this->returnValue( $user ) );
-		$mock->expects( $this->once() )->method( 'deleteUser' );
+		$mock->expects($this->once())->method('loadUser')->will($this->returnValue($user));
+		$mock->expects($this->once())->method('deleteUser');
 
-		$this->context->setEzUserService( $mock );
-		$this->object->deleteItems( array( -1 ) );
+		$this->context->setEzUserService($mock);
+		$this->object->deleteItems(array( -1 ));
 	}
 
 
 	public function testGetSearchAttributes()
 	{
 		$attributes = $this->object->getSearchAttributes();
-		$this->assertGreaterThan( 0, count( $attributes ) );
+		$this->assertGreaterThan(0, count($attributes));
 
-		foreach( $attributes as $attribute ) {
-			$this->assertInstanceOf( '\Aimeos\MW\Criteria\Attribute\Iface', $attribute );
+		foreach ($attributes as $attribute) {
+			$this->assertInstanceOf('\Aimeos\MW\Criteria\Attribute\Iface', $attribute);
 		}
 	}
 
 
 	public function testGetSubManager()
 	{
-		$this->assertInstanceOf( '\Aimeos\MShop\Common\Manager\Iface', $this->object->getSubManager( 'address' ) );
-		$this->assertInstanceOf( '\Aimeos\MShop\Common\Manager\Iface', $this->object->getSubManager( 'address', 'Standard' ) );
+		$this->assertInstanceOf('\Aimeos\MShop\Common\Manager\Iface', $this->object->getSubManager('address'));
+		$this->assertInstanceOf('\Aimeos\MShop\Common\Manager\Iface', $this->object->getSubManager('address', 'Standard'));
 
-		$this->setExpectedException( '\Aimeos\MShop\Exception' );
-		$this->object->getSubManager( 'unknown' );
+		$this->setExpectedException('\Aimeos\MShop\Exception');
+		$this->object->getSubManager('unknown');
 	}
 
 
 	public function testGetSubManagerInvalidName()
 	{
-		$this->setExpectedException( '\Aimeos\MShop\Exception' );
-		$this->object->getSubManager( 'address', 'unknown' );
+		$this->setExpectedException('\Aimeos\MShop\Exception');
+		$this->object->getSubManager('address', 'unknown');
 	}
 
 
 	public function testSaveItemInsert()
 	{
-		$service = $this->getMockBuilder( 'eZ\Publish\API\Repository\UserService' )->getMock();
-		$struct = $this->getMockBuilder( 'eZ\Publish\API\Repository\Values\User\UserCreateStruct' )->getMock();
-		$user = $this->getMockBuilder( '\eZ\Publish\API\Repository\Values\User\User' )
-			->setMethods( array( 'getUserId', 'getVersionInfo', 'getFieldValue', 'getFields', 'getFieldsByLanguage' ) )
+		$service = $this->getMockBuilder('eZ\Publish\API\Repository\UserService')->getMock();
+		$struct = $this->getMockBuilder('eZ\Publish\API\Repository\Values\User\UserCreateStruct')->getMock();
+		$user = $this->getMockBuilder('\eZ\Publish\API\Repository\Values\User\User')
+			->setMethods(array('getUserId', 'getVersionInfo', 'getFieldValue', 'getFields', 'getFieldsByLanguage'))
 			->getMock();
 
-		$dbm = $this->getMockBuilder( 'Aimeos\MW\DB\Manager\Iface' )->getMock();
-		$conn = $this->getMockBuilder( 'Aimeos\MW\DB\Connection\Iface' )->getMock();
-		$stmt = $this->getMockBuilder( 'Aimeos\MW\DB\Statement\Iface' )->getMock();
-		$result = $this->getMockBuilder( 'Aimeos\MW\DB\Result\Iface' )->getMock();
+		$dbm = $this->getMockBuilder('Aimeos\MW\DB\Manager\Iface')->getMock();
+		$conn = $this->getMockBuilder('Aimeos\MW\DB\Connection\Iface')->getMock();
+		$stmt = $this->getMockBuilder('Aimeos\MW\DB\Statement\Iface')->getMock();
+		$result = $this->getMockBuilder('Aimeos\MW\DB\Result\Iface')->getMock();
 
-		$service->expects( $this->once() )->method( 'newUserCreateStruct' )->will( $this->returnValue( $struct ) );
-		$service->expects( $this->once() )->method( 'createUser' )->will( $this->returnValue( $user ) );
-		$user->expects( $this->once() )->method( 'getUserId' )->will( $this->returnValue( 1 ) );
-		$dbm->expects( $this->once() )->method( 'acquire' )->will( $this->returnValue( $conn ) );
-		$conn->expects( $this->once() )->method( 'create' )->will( $this->returnValue( $stmt ) );
-		$stmt->expects( $this->once() )->method( 'execute' )->will( $this->returnValue( $result ) );
+		$service->expects($this->once())->method('newUserCreateStruct')->will($this->returnValue($struct));
+		$service->expects($this->once())->method('createUser')->will($this->returnValue($user));
+		$user->expects($this->once())->method('getUserId')->will($this->returnValue(1));
+		$dbm->expects($this->once())->method('acquire')->will($this->returnValue($conn));
+		$conn->expects($this->once())->method('create')->will($this->returnValue($stmt));
+		$stmt->expects($this->once())->method('execute')->will($this->returnValue($result));
 
 
-		$this->context->setDatabaseManager( $dbm );
-		$this->context->setEzUserService( $service );
+		$this->context->setDatabaseManager($dbm);
+		$this->context->setEzUserService($service);
 
-		$this->object->saveItem( new \Aimeos\MShop\Customer\Item\Standard( $this->address ) );
+		$this->object->saveItem(new \Aimeos\MShop\Customer\Item\Standard($this->address));
 	}
 
 
 	public function testSaveItemUpdate()
 	{
-		$service = $this->getMockBuilder( 'eZ\Publish\API\Repository\UserService' )->getMock();
-		$user = $this->getMockBuilder( 'eZ\Publish\API\Repository\Values\User\User' )->getMock();
-		$struct = $this->getMockBuilder( 'eZ\Publish\API\Repository\Values\User\UserUpdateStruct' )->getMock();
-		$dbm = $this->getMockBuilder( 'Aimeos\MW\DB\Manager\Iface' )->getMock();
-		$conn = $this->getMockBuilder( 'Aimeos\MW\DB\Connection\Iface' )->getMock();
-		$stmt = $this->getMockBuilder( 'Aimeos\MW\DB\Statement\Iface' )->getMock();
-		$result = $this->getMockBuilder( 'Aimeos\MW\DB\Result\Iface' )->getMock();
-
-		$service->expects( $this->once() )->method( 'updateUser' );
-		$service->expects( $this->once() )->method( 'loadUser' )->will( $this->returnValue( $user ) );
-		$service->expects( $this->once() )->method( 'newUserUpdateStruct' )->will( $this->returnValue( $struct ) );
-		$dbm->expects( $this->once() )->method( 'acquire' )->will( $this->returnValue( $conn ) );
-		$conn->expects( $this->once() )->method( 'create' )->will( $this->returnValue( $stmt ) );
-		$stmt->expects( $this->once() )->method( 'execute' )->will( $this->returnValue( $result ) );
-		$result->expects( $this->once() )->method( 'finish' );
-		$dbm->expects( $this->once() )->method( 'release' );
-
-
-		$item = new \Aimeos\MShop\Customer\Item\Standard( $this->address, array( 'customer.id' => 1 ) );
-		$item->setStatus( 1 );
-
-		$this->context->setDatabaseManager( $dbm );
-		$this->context->setEzUserService( $service );
-		$this->object->saveItem( $item );
+		$service = $this->getMockBuilder('eZ\Publish\API\Repository\UserService')->getMock();
+		$user = $this->getMockBuilder('eZ\Publish\API\Repository\Values\User\User')->getMock();
+		$struct = $this->getMockBuilder('eZ\Publish\API\Repository\Values\User\UserUpdateStruct')->getMock();
+		$dbm = $this->getMockBuilder('Aimeos\MW\DB\Manager\Iface')->getMock();
+		$conn = $this->getMockBuilder('Aimeos\MW\DB\Connection\Iface')->getMock();
+		$stmt = $this->getMockBuilder('Aimeos\MW\DB\Statement\Iface')->getMock();
+		$result = $this->getMockBuilder('Aimeos\MW\DB\Result\Iface')->getMock();
+
+		$service->expects($this->once())->method('updateUser');
+		$service->expects($this->once())->method('loadUser')->will($this->returnValue($user));
+		$service->expects($this->once())->method('newUserUpdateStruct')->will($this->returnValue($struct));
+		$dbm->expects($this->once())->method('acquire')->will($this->returnValue($conn));
+		$conn->expects($this->once())->method('create')->will($this->returnValue($stmt));
+		$stmt->expects($this->once())->method('execute')->will($this->returnValue($result));
+		$result->expects($this->once())->method('finish');
+		$dbm->expects($this->once())->method('release');
+
+
+		$item = new \Aimeos\MShop\Customer\Item\Standard($this->address, array('customer.id' => 1));
+		$item->setStatus(1);
+
+		$this->context->setDatabaseManager($dbm);
+		$this->context->setEzUserService($service);
+		$this->object->saveItem($item);
 	}
 
 
 	public function testSaveItemInvalidItem()
 	{
-		$this->setExpectedException( '\Aimeos\MShop\Exception' );
-		$this->object->saveItem( new \Aimeos\MShop\Common\Item\Lists\Standard( 'common.lists.' ) );
+		$this->setExpectedException('\Aimeos\MShop\Exception');
+		$this->object->saveItem(new \Aimeos\MShop\Common\Item\Lists\Standard('common.lists.'));
 	}
 
 
 	public function testSaveItemNotModified()
 	{
-		$item = new \Aimeos\MShop\Customer\Item\Standard( $this->address, array( 'customer.id' => 1 ) );
-		$this->object->saveItem( $item );
+		$item = new \Aimeos\MShop\Customer\Item\Standard($this->address, array('customer.id' => 1));
+		$this->object->saveItem($item);
 	}
 
 
 	public function testSaveItemInvalidContext()
 	{
 		$context = new \Aimeos\MShop\Context\Item\Standard();
-		$context->setConfig( $this->context->getConfig() );
+		$context->setConfig($this->context->getConfig());
 
-		$object = new \Aimeos\MShop\Customer\Manager\Ezpublish( $context );
+		$object = new \Aimeos\MShop\Customer\Manager\Ezpublish($context);
 
-		$this->setExpectedException( '\Aimeos\MShop\Customer\Exception' );
-		$object->saveItem( new \Aimeos\MShop\Customer\Item\Standard( $this->address ) );
+		$this->setExpectedException('\Aimeos\MShop\Customer\Exception');
+		$object->saveItem(new \Aimeos\MShop\Customer\Item\Standard($this->address));
 	}
 
 
 	public function testSearchItems()
 	{
-		$mock = $this->getMockBuilder( '\Aimeos\MShop\Customer\Manager\Ezpublish' )
-			->setConstructorArgs( array( $this->context ) )
-			->setMethods( array( 'searchItemsBase' ) )
+		$mock = $this->getMockBuilder('\Aimeos\MShop\Customer\Manager\Ezpublish')
+			->setConstructorArgs(array($this->context))
+			->setMethods(array('searchItemsBase'))
 			->getMock();
 
-		$result1 = $this->getMockBuilder( '\Aimeos\MW\DB\Result\Iface' )->getMock();
+		$result1 = $this->getMockBuilder('\Aimeos\MW\DB\Result\Iface')->getMock();
 
-		$mock->expects( $this->once() )->method( 'searchItemsBase' )->will( $this->returnValue( $result1 ) );
-		$result1->expects( $this->exactly( 2 ) )->method( 'fetch' )
-			->will( $this->onConsecutiveCalls( array( 'customer.id' => -1 ), false ) );
+		$mock->expects($this->once())->method('searchItemsBase')->will($this->returnValue($result1));
+		$result1->expects($this->exactly(2))->method('fetch')
+			->will($this->onConsecutiveCalls(array('customer.id' => -1), false));
 
 
-		$dbm = $this->getMockBuilder( 'Aimeos\MW\DB\Manager\Iface' )->getMock();
-		$conn = $this->getMockBuilder( 'Aimeos\MW\DB\Connection\Iface' )->getMock();
-		$stmt = $this->getMockBuilder( 'Aimeos\MW\DB\Statement\Iface' )->getMock();
-		$result2 = $this->getMockBuilder( '\Aimeos\MW\DB\Result\Iface' )->getMock();
+		$dbm = $this->getMockBuilder('Aimeos\MW\DB\Manager\Iface')->getMock();
+		$conn = $this->getMockBuilder('Aimeos\MW\DB\Connection\Iface')->getMock();
+		$stmt = $this->getMockBuilder('Aimeos\MW\DB\Statement\Iface')->getMock();
+		$result2 = $this->getMockBuilder('\Aimeos\MW\DB\Result\Iface')->getMock();
 
-		$dbm->expects( $this->any() )->method( 'acquire' )->will( $this->returnValue( $conn ) );
-		$conn->expects( $this->once() )->method( 'create' )->will( $this->returnValue( $stmt ) );
-		$stmt->expects( $this->once() )->method( 'execute' )->will( $this->returnValue( $result2 ) );
-		$result2->expects( $this->exactly( 2 ) )->method( 'fetch' )
-			->will( $this->onConsecutiveCalls( array( 'contentobject_id' => -1, 'role_id' => -2 ), false ) );
+		$dbm->expects($this->any())->method('acquire')->will($this->returnValue($conn));
+		$conn->expects($this->once())->method('create')->will($this->returnValue($stmt));
+		$stmt->expects($this->once())->method('execute')->will($this->returnValue($result2));
+		$result2->expects($this->exactly(2))->method('fetch')
+			->will($this->onConsecutiveCalls(array('contentobject_id' => -1, 'role_id' => -2), false));
 
-		$dbm->expects( $this->any() )->method( 'release' );
+		$dbm->expects($this->any())->method('release');
 
 
-		$this->context->setDatabaseManager( $dbm );
-		$list = $mock->searchItems( $mock->createSearch() );
+		$this->context->setDatabaseManager($dbm);
+		$list = $mock->searchItems($mock->createSearch());
 
-		$this->assertEquals( 1, count( $list ) );
-		$this->assertInstanceOf( '\Aimeos\MShop\Customer\Item\Iface', $list[-1] );
+		$this->assertEquals(1, count($list));
+		$this->assertInstanceOf('\Aimeos\MShop\Customer\Item\Iface', $list[-1]);
 	}
 
 
 	public function testSearchItemsException()
 	{
-		$object = $this->getMockBuilder( '\Aimeos\MShop\Customer\Manager\Ezpublish' )
-			->setConstructorArgs( array( $this->context ) )
-			->setMethods( array( 'searchItemsBase' ) )
+		$object = $this->getMockBuilder('\Aimeos\MShop\Customer\Manager\Ezpublish')
+			->setConstructorArgs(array($this->context))
+			->setMethods(array('searchItemsBase'))
 			->getMock();
 
-		$object->expects( $this->once() )->method( 'searchItemsBase' )
-			->will( $this->throwException( new \Aimeos\MShop\Exception() ) );
+		$object->expects($this->once())->method('searchItemsBase')
+			->will($this->throwException(new \Aimeos\MShop\Exception()));
 
-		$this->setExpectedException( '\Aimeos\MShop\Exception' );
-		$object->searchItems( $object->createSearch() );
+		$this->setExpectedException('\Aimeos\MShop\Exception');
+		$object->searchItems($object->createSearch());
 	}
 }

lib/custom/tests/MShop/Context/Item/EzpublishTest.php

Spacing +8 added lines, -8 removed lines

@@ -16,33 +16,33 @@
 
 	protected function setUp()
 	{
-		if( !interface_exists( 'eZ\Publish\API\Repository\UserService' ) ) {
-			$this->markTestSkipped( 'Install ezsystems/ezpublish-api first' );
+		if (!interface_exists('eZ\Publish\API\Repository\UserService')) {
+			$this->markTestSkipped('Install ezsystems/ezpublish-api first');
 		}
 
-		$this->mock = $this->getMockBuilder( 'eZ\Publish\API\Repository\UserService' )->getMock();
+		$this->mock = $this->getMockBuilder('eZ\Publish\API\Repository\UserService')->getMock();
 		$this->object = new \Aimeos\MShop\Context\Item\Ezpublish();
 	}
 
 
 	protected function tearDown()
 	{
-		unset( $this->object, $this->mock );
+		unset($this->object, $this->mock);
 	}
 
 
 	public function testGetEzUserService()
 	{
-		$this->setExpectedException( '\\Aimeos\\MShop\\Exception' );
+		$this->setExpectedException('\\Aimeos\\MShop\\Exception');
 		$this->object->getEzUserService();
 	}
 
 
 	public function testSetEzUserService()
 	{
-		$return = $this->object->setEzUserService( $this->mock );
+		$return = $this->object->setEzUserService($this->mock);
 
-		$this->assertSame( $this->mock, $this->object->getEzUserService() );
-		$this->assertInstanceOf( '\Aimeos\MShop\Context\Item\Iface', $return );
+		$this->assertSame($this->mock, $this->object->getEzUserService());
+		$this->assertInstanceOf('\Aimeos\MShop\Context\Item\Iface', $return);
 	}
 }

lib/custom/src/MShop/Customer/Item/Ezpublish.php

Spacing +2 added lines, -2 removed lines

@@ -39,9 +39,9 @@
 	 * @param string $value password of the customer item
 	 * @return \Aimeos\MShop\Customer\Item\Iface Customer item for chaining method calls
 	 */
-	public function setPassword( $value )
+	public function setPassword($value)
 	{
-		if( $value !== '' && $value != $this->getPassword() )
+		if ($value !== '' && $value != $this->getPassword())
 		{
 			$this->password = (string) $value;
 			$this->setModified();