aimeos / ai-fosuser

lib/custom/tests/MShop/Customer/Item/FosUserTest.php

Spacing +23 added lines, -23 removed lines

@@ -22,7 +22,7 @@ 
 			'customer.address.position' => 1,
 		);
 
-		$this->address = new \Aimeos\MShop\Common\Item\Address\Standard( 'common.address.', $addressValues );
+		$this->address = new \Aimeos\MShop\Common\Item\Address\Standard('common.address.', $addressValues);
 
 		$values = array(
 			'customer.id' => 541,
@@ -54,71 +54,71 @@ 
 			'customer.mtime'=> '2010-01-05 00:00:05',
 			'customer.ctime'=> '2010-01-01 00:00:00',
 			'customer.editor' => 'unitTestUser',
-			'roles' => array( 'ROLE_ADMIN' ),
+			'roles' => array('ROLE_ADMIN'),
 			'salt' => 'test',
 		);
 
-		$this->object = new \Aimeos\MShop\Customer\Item\FosUser( $this->address, $values );
+		$this->object = new \Aimeos\MShop\Customer\Item\FosUser($this->address, $values);
 	}
 
 	protected function tearDown()
 	{
-		unset( $this->object );
+		unset($this->object);
 	}
 
 	public function testGetPassword()
 	{
-		$this->assertEquals( 'testpwd', $this->object->getPassword() );
+		$this->assertEquals('testpwd', $this->object->getPassword());
 	}
 
 	public function testSetPassword()
 	{
-		$this->object->setPassword( 'new' );
-		$this->assertTrue( $this->object->isModified() );
-		$this->assertEquals( 'new', $this->object->getPassword() );
+		$this->object->setPassword('new');
+		$this->assertTrue($this->object->isModified());
+		$this->assertEquals('new', $this->object->getPassword());
 	}
 
 	public function testSetPasswordGenerated()
 	{
-		$helper = new \Aimeos\MShop\Common\Item\Helper\Password\Standard( array( 'format' => '%1$s{%2$s}' ) );
-		$object = new \Aimeos\MShop\Customer\Item\FosUser( $this->address, [], [], [], null, $helper );
+		$helper = new \Aimeos\MShop\Common\Item\Helper\Password\Standard(array('format' => '%1$s{%2$s}'));
+		$object = new \Aimeos\MShop\Customer\Item\FosUser($this->address, [], [], [], null, $helper);
 
-		$object->setPassword( 'newpwd' );
-		$this->assertEquals( sha1( 'newpwd{' . $object->getSalt() . '}' ), $object->getPassword() );
+		$object->setPassword('newpwd');
+		$this->assertEquals(sha1('newpwd{' . $object->getSalt() . '}'), $object->getPassword());
 	}
 
 	public function testGetRoles()
 	{
-		$this->assertEquals( array( 'ROLE_ADMIN' ), $this->object->getRoles() );
+		$this->assertEquals(array('ROLE_ADMIN'), $this->object->getRoles());
 	}
 
 	public function testSetRoles()
 	{
-		$this->object->setRoles( array( 'ROLE_USER' ) );
-		$this->assertTrue( $this->object->isModified() );
-		$this->assertEquals( array( 'ROLE_USER' ), $this->object->getRoles() );
+		$this->object->setRoles(array('ROLE_USER'));
+		$this->assertTrue($this->object->isModified());
+		$this->assertEquals(array('ROLE_USER'), $this->object->getRoles());
 	}
 
 	public function testGetSalt()
 	{
-		$this->assertEquals( 'test', $this->object->getSalt() );
+		$this->assertEquals('test', $this->object->getSalt());
 	}
 
 	public function testGetSaltGenerated()
 	{
-		$object = new \Aimeos\MShop\Customer\Item\FosUser( $this->address, [] );
-		$this->assertEquals( '', $object->getSalt() );
+		$object = new \Aimeos\MShop\Customer\Item\FosUser($this->address, []);
+		$this->assertEquals('', $object->getSalt());
 	}
 
 	public function testSetSalt()
 	{
-		$this->object->setSalt( 'new' );
-		$this->assertTrue( $this->object->isModified() );
-		$this->assertEquals( 'new', $this->object->getSalt() );
+		$this->object->setSalt('new');
+		$this->assertTrue($this->object->isModified());
+		$this->assertEquals('new', $this->object->getSalt());
 	}
 
 	public function testIsModified()
 	{
-		$this->assertFalse( $this->object->isModified() );
+		$this->assertFalse($this->object->isModified());
 	}
 }

lib/custom/setup/CustomerChangeAddressRefidParentidFosUser.php

Spacing +11 added lines, -11 removed lines

@@ -40,7 +40,7 @@ 
 	 */
 	public function getPostDependencies()
 	{
-		return array( 'TablesCreateFosUser' );
+		return array('TablesCreateFosUser');
 	}
 
 
@@ -49,7 +49,7 @@ 
 	 */
 	public function migrate()
 	{
-		$this->process( $this->mysql );
+		$this->process($this->mysql);
 	}
 
 
@@ -58,22 +58,22 @@ 
 	 *
 	 * array string $stmts List of SQL statements for changing the columns
 	 */
-	protected function process( array $stmts )
+	protected function process(array $stmts)
 	{
 		$table = 'fos_user_address';
-		$this->msg( sprintf( 'Rename "refid" to "parentid" in table "%1$s"', $table ), 0 ); $this->status( '' );
+		$this->msg(sprintf('Rename "refid" to "parentid" in table "%1$s"', $table), 0); $this->status('');
 
-		foreach( $stmts as $column => $stmts )
+		foreach ($stmts as $column => $stmts)
 		{
-			$this->msg( sprintf( 'Checking column "%1$s"', $column ), 1 );
+			$this->msg(sprintf('Checking column "%1$s"', $column), 1);
 
-			if( $this->schema->tableExists( $table )
-				&& $this->schema->columnExists( $table, $column ) === true
+			if ($this->schema->tableExists($table)
+				&& $this->schema->columnExists($table, $column) === true
 			) {
-				$this->executeList( $stmts );
-				$this->status( 'done' );
+				$this->executeList($stmts);
+				$this->status('done');
 			} else {
-				$this->status( 'OK' );
+				$this->status('OK');
 			}
 		}
 	}

lib/custom/setup/unittest/data/customer-property.php

Spacing +3 added lines, -3 removed lines

@@ -5,12 +5,12 @@
  * @copyright Aimeos (aimeos.org), 2018
  */
 
-return array (
+return array(
 	'customer/property/type' => array(
-		'customer/property/type/newsletter' => array( 'domain' => 'customer', 'code' => 'newsletter', 'label' => 'Newsletter', 'status' => 1 ),
+		'customer/property/type/newsletter' => array('domain' => 'customer', 'code' => 'newsletter', 'label' => 'Newsletter', 'status' => 1),
 	),
 
 	'customer/property' => array(
-		'customer/property/UTC001/newsletter' => array( 'parentid' => 'customer/UTC001', 'typeid' => 'customer/property/type/newsletter', 'langid' => null, 'value' => '1' ),
+		'customer/property/UTC001/newsletter' => array('parentid' => 'customer/UTC001', 'typeid' => 'customer/property/type/newsletter', 'langid' => null, 'value' => '1'),
 	),
 );
\ No newline at end of file

lib/custom/setup/CustomerRemoveLostUserDataFosuser.php

Spacing +11 added lines, -11 removed lines

@@ -38,7 +38,7 @@ 
 	 */
 	public function getPostDependencies()
 	{
-		return array( 'TablesCreateMShop' );
+		return array('TablesCreateMShop');
 	}
 
 
@@ -47,29 +47,29 @@ 
 	 */
 	public function migrate()
 	{
-		$this->msg( 'Remove left over FOS user address records', 0 );
+		$this->msg('Remove left over FOS user address records', 0);
 
-		if( $this->schema->tableExists( 'fos_user' ) && $this->schema->tableExists( 'fos_user_address' ) )
+		if ($this->schema->tableExists('fos_user') && $this->schema->tableExists('fos_user_address'))
 		{
-			$this->execute( $this->sql['address'] );
-			$this->status( 'done' );
+			$this->execute($this->sql['address']);
+			$this->status('done');
 		}
 		else
 		{
-			$this->status( 'OK' );
+			$this->status('OK');
 		}
 
 
-		$this->msg( 'Remove left over FOS user list records', 0 );
+		$this->msg('Remove left over FOS user list records', 0);
 
-		if( $this->schema->tableExists( 'fos_user' ) && $this->schema->tableExists( 'fos_user_list' ) )
+		if ($this->schema->tableExists('fos_user') && $this->schema->tableExists('fos_user_list'))
 		{
-			$this->execute( $this->sql['list'] );
-			$this->status( 'done' );
+			$this->execute($this->sql['list']);
+			$this->status('done');
 		}
 		else
 		{
-			$this->status( 'OK' );
+			$this->status('OK');
 		}
 	}
 }

Braces +2 added lines, -4 removed lines

@@ -53,8 +53,7 @@ 
 		{
 			$this->execute( $this->sql['address'] );
 			$this->status( 'done' );
-		}
-		else
+		} else
 		{
 			$this->status( 'OK' );
 		}
@@ -66,8 +65,7 @@ 
 		{
 			$this->execute( $this->sql['list'] );
 			$this->status( 'done' );
-		}
-		else
+		} else
 		{
 			$this->status( 'OK' );
 		}

lib/custom/src/MShop/Customer/Manager/Property/Type/FosUser.php

Spacing +9 added lines, -9 removed lines

@@ -24,7 +24,7 @@ 
 		'customer.property.type.id' => array(
 			'code' => 'customer.property.type.id',
 			'internalcode' => 'fosprty."id"',
-			'internaldeps' => array( 'LEFT JOIN "fos_user_property_type" AS fosprty ON ( fospr."typeid" = fosprty."id" )' ),
+			'internaldeps' => array('LEFT JOIN "fos_user_property_type" AS fosprty ON ( fospr."typeid" = fosprty."id" )'),
 			'label' => 'Property type ID',
 			'type' => 'integer',
 			'internaltype' => \Aimeos\MW\DB\Statement\Base::PARAM_INT,
@@ -105,14 +105,14 @@ 
 	 *
 	 * @param array $siteids List of IDs for sites whose entries should be deleted
 	 */
-	public function cleanup( array $siteids )
+	public function cleanup(array $siteids)
 	{
 		$path = 'mshop/customer/manager/property/type/submanagers';
-		foreach( $this->getContext()->getConfig()->get( $path, [] ) as $domain ) {
-			$this->getObject()->getSubManager( $domain )->cleanup( $siteids );
+		foreach ($this->getContext()->getConfig()->get($path, []) as $domain) {
+			$this->getObject()->getSubManager($domain)->cleanup($siteids);
 		}
 
-		$this->cleanupBase( $siteids, 'mshop/customer/manager/property/type/fosuser/delete' );
+		$this->cleanupBase($siteids, 'mshop/customer/manager/property/type/fosuser/delete');
 	}
 
 
@@ -122,11 +122,11 @@ 
 	 * @param boolean $withsub Return also attributes of sub-managers if true
 	 * @return array Returns a list of attribtes implementing \Aimeos\MW\Criteria\Attribute\Iface
 	 */
-	public function getSearchAttributes( $withsub = true )
+	public function getSearchAttributes($withsub = true)
 	{
 		$path = 'mshop/customer/manager/property/type/submanagers';
 
-		return $this->getSearchAttributesBase( $this->searchConfig, $path, [], $withsub );
+		return $this->getSearchAttributesBase($this->searchConfig, $path, [], $withsub);
 	}
 
 
@@ -137,9 +137,9 @@ 
 	 * @param string|null $name Name of the implementation, will be from configuration (or Default) if null
 	 * @return \Aimeos\MShop\Common\Manager\Iface Manager for different extensions, e.g types, lists etc.
 	 */
-	public function getSubManager( $manager, $name = null )
+	public function getSubManager($manager, $name = null)
 	{
-		return $this->getSubManagerBase( 'customer', 'property/type/' . $manager, ( $name === null ? 'FosUser' : $name ) );
+		return $this->getSubManagerBase('customer', 'property/type/' . $manager, ($name === null ? 'FosUser' : $name));
 	}
 
 

lib/custom/src/MShop/Customer/Manager/Property/FosUser.php

Spacing +9 added lines, -9 removed lines

@@ -24,7 +24,7 @@ 
 		'customer.property.id' => array(
 			'code' => 'customer.property.id',
 			'internalcode' => 'fospr."id"',
-			'internaldeps'=>array( 'LEFT JOIN "fos_user_property" AS fospr ON ( fospr."parentid" = fos."id" )' ),
+			'internaldeps'=>array('LEFT JOIN "fos_user_property" AS fospr ON ( fospr."parentid" = fos."id" )'),
 			'label' => 'Property ID',
 			'type' => 'integer',
 			'internaltype' => \Aimeos\MW\DB\Statement\Base::PARAM_INT,
@@ -100,14 +100,14 @@ 
 	 *
 	 * @param integer[] $siteids List of IDs for sites whose entries should be deleted
 	 */
-	public function cleanup( array $siteids )
+	public function cleanup(array $siteids)
 	{
 		$path = 'mshop/customer/manager/property/submanagers';
-		foreach( $this->getContext()->getConfig()->get( $path, [] ) as $domain ) {
-			$this->getObject()->getSubManager( $domain )->cleanup( $siteids );
+		foreach ($this->getContext()->getConfig()->get($path, []) as $domain) {
+			$this->getObject()->getSubManager($domain)->cleanup($siteids);
 		}
 
-		$this->cleanupBase( $siteids, 'mshop/customer/manager/property/fosuser/delete' );
+		$this->cleanupBase($siteids, 'mshop/customer/manager/property/fosuser/delete');
 	}
 
 
@@ -117,11 +117,11 @@ 
 	 * @param boolean $withsub Return also attributes of sub-managers if true
 	 * @return array Returns a list of attribtes implementing \Aimeos\MW\Criteria\Attribute\Iface
 	 */
-	public function getSearchAttributes( $withsub = true )
+	public function getSearchAttributes($withsub = true)
 	{
 		$path = 'mshop/customer/manager/property/submanagers';
 
-		return $this->getSearchAttributesBase( $this->searchConfig, $path, ['type'], $withsub );
+		return $this->getSearchAttributesBase($this->searchConfig, $path, ['type'], $withsub);
 	}
 
 
@@ -133,9 +133,9 @@ 
 	 * configuration (or Default) if null
 	 * @return \Aimeos\MShop\Common\Manager\Iface Manager for different extensions, e.g property types, property lists etc.
 	 */
-	public function getSubManager( $manager, $name = null )
+	public function getSubManager($manager, $name = null)
 	{
-		return $this->getSubManagerBase( 'customer', 'property/' . $manager, ( $name === null ? 'FosUser' : $name ) );
+		return $this->getSubManagerBase('customer', 'property/' . $manager, ($name === null ? 'FosUser' : $name));
 	}
 
 

lib/custom/setup/unittest/CustomerAddFosUserTestData.php

Spacing +14 added lines, -14 removed lines

@@ -21,7 +21,7 @@ 
 	 */
 	public function getPreDependencies()
 	{
-		return array( 'TablesCreateFosUser' );
+		return array('TablesCreateFosUser');
 	}
 
 
@@ -30,36 +30,36 @@ 
 	 */
 	public function migrate()
 	{
-		\Aimeos\MW\Common\Base::checkClass( '\\Aimeos\\MShop\\Context\\Item\\Iface', $this->additional );
+		\Aimeos\MW\Common\Base::checkClass('\\Aimeos\\MShop\\Context\\Item\\Iface', $this->additional);
 
-		$this->msg( 'Adding Fos user bundle customer test data', 0 );
+		$this->msg('Adding Fos user bundle customer test data', 0);
 
 		$parentIds = [];
 		$ds = DIRECTORY_SEPARATOR;
-		$this->additional->setEditor( 'ai-fosuser:unittest' );
+		$this->additional->setEditor('ai-fosuser:unittest');
 		$path = __DIR__ . $ds . 'data' . $ds . 'customer.php';
 
-		if( ( $testdata = include( $path ) ) === false ){
-			throw new \Aimeos\MShop\Exception( sprintf( 'No file "%1$s" found for customer domain', $path ) );
+		if (($testdata = include($path)) === false) {
+			throw new \Aimeos\MShop\Exception(sprintf('No file "%1$s" found for customer domain', $path));
 		}
 
 
-		$customerManager = \Aimeos\MShop\Customer\Manager\Factory::createManager( $this->additional, 'FosUser' );
-		$customerAddressManager = $customerManager->getSubManager( 'address', 'FosUser' );
+		$customerManager = \Aimeos\MShop\Customer\Manager\Factory::createManager($this->additional, 'FosUser');
+		$customerAddressManager = $customerManager->getSubManager('address', 'FosUser');
 
 		$search = $customerManager->createSearch();
-		$search->setConditions( $search->compare( '==', 'customer.code', array( 'UTC001', 'UTC002', 'UTC003' ) ) );
-		$items = $customerManager->searchItems( $search );
+		$search->setConditions($search->compare('==', 'customer.code', array('UTC001', 'UTC002', 'UTC003')));
+		$items = $customerManager->searchItems($search);
 
 		$this->conn->begin();
 
-		$customerManager->deleteItems( array_keys( $items ) );
-		$parentIds = $this->addCustomerData( $testdata, $customerManager, $customerAddressManager->createItem() );
-		$this->addCustomerAddressData( $testdata, $customerAddressManager, $parentIds );
+		$customerManager->deleteItems(array_keys($items));
+		$parentIds = $this->addCustomerData($testdata, $customerManager, $customerAddressManager->createItem());
+		$this->addCustomerAddressData($testdata, $customerAddressManager, $parentIds);
 
 		$this->conn->commit();
 
 
-		$this->status( 'done' );
+		$this->status('done');
 	}
 }

lib/custom/setup/unittest/CustomerListAddFosUserTestData.php

Spacing +12 added lines, -12 removed lines

@@ -22,7 +22,7 @@ 
 	 */
 	public function getPreDependencies()
 	{
-		return array( 'TablesCreateFosUser', 'CustomerAddFosUserTestData', 'LocaleAddTestData', 'TextAddTestData' );
+		return array('TablesCreateFosUser', 'CustomerAddFosUserTestData', 'LocaleAddTestData', 'TextAddTestData');
 	}
 
 
@@ -31,27 +31,27 @@ 
 	 */
 	public function migrate()
 	{
-		\Aimeos\MW\Common\Base::checkClass( '\\Aimeos\\MShop\\Context\\Item\\Iface', $this->additional );
+		\Aimeos\MW\Common\Base::checkClass('\\Aimeos\\MShop\\Context\\Item\\Iface', $this->additional);
 
-		$this->msg( 'Adding customer-list Fos user bundle test data', 0 );
-		$this->additional->setEditor( 'ai-fosuser:unittest' );
+		$this->msg('Adding customer-list Fos user bundle test data', 0);
+		$this->additional->setEditor('ai-fosuser:unittest');
 
 		$ds = DIRECTORY_SEPARATOR;
-		$path = dirname( __FILE__ ) . $ds . 'data' . $ds . 'customer-list.php';
+		$path = dirname(__FILE__) . $ds . 'data' . $ds . 'customer-list.php';
 
-		if( ( $testdata = include( $path ) ) === false ){
-			throw new \Aimeos\MShop\Exception( sprintf( 'No file "%1$s" found for customer list domain', $path ) );
+		if (($testdata = include($path)) === false) {
+			throw new \Aimeos\MShop\Exception(sprintf('No file "%1$s" found for customer list domain', $path));
 		}
 
 		$refKeys = [];
-		foreach( $testdata['customer/lists'] as $dataset ) {
-			$refKeys[ $dataset['domain'] ][] = $dataset['refid'];
+		foreach ($testdata['customer/lists'] as $dataset) {
+			$refKeys[$dataset['domain']][] = $dataset['refid'];
 		}
 
 		$refIds = [];
-		$refIds['text'] = $this->getTextData( $refKeys['text'] );
-		$this->addCustomerListData( $testdata, $refIds, 'FosUser' );
+		$refIds['text'] = $this->getTextData($refKeys['text']);
+		$this->addCustomerListData($testdata, $refIds, 'FosUser');
 
-		$this->status( 'done' );
+		$this->status('done');
 	}
 }

lib/custom/setup/unittest/CustomerPropertyAddFosUserTestData.php

Spacing +8 added lines, -8 removed lines

@@ -23,7 +23,7 @@ 
 	 */
 	public function getPreDependencies()
 	{
-		return array( 'TablesCreateFosUser', 'CustomerAddFosUserTestData', 'LocaleAddTestData' );
+		return array('TablesCreateFosUser', 'CustomerAddFosUserTestData', 'LocaleAddTestData');
 	}
 
 
@@ -32,20 +32,20 @@ 
 	 */
 	public function migrate()
 	{
-		\Aimeos\MW\Common\Base::checkClass( '\\Aimeos\\MShop\\Context\\Item\\Iface', $this->additional );
+		\Aimeos\MW\Common\Base::checkClass('\\Aimeos\\MShop\\Context\\Item\\Iface', $this->additional);
 
-		$this->msg( 'Adding customer property Fos user bundle test data', 0 );
-		$this->additional->setEditor( 'ai-fosuser:unittest' );
+		$this->msg('Adding customer property Fos user bundle test data', 0);
+		$this->additional->setEditor('ai-fosuser:unittest');
 
 		$ds = DIRECTORY_SEPARATOR;
 		$path = __DIR__ . $ds . 'data' . $ds . 'customer-property.php';
 
-		if( ( $testdata = include( $path ) ) == false ) {
-			throw new \Aimeos\MShop\Exception( sprintf( 'No file "%1$s" found for customer domain', $path ) );
+		if (($testdata = include($path)) == false) {
+			throw new \Aimeos\MShop\Exception(sprintf('No file "%1$s" found for customer domain', $path));
 		}
 
-		$this->addCustomerPropertyData( $testdata, 'FosUser' );
+		$this->addCustomerPropertyData($testdata, 'FosUser');
 
-		$this->status( 'done' );
+		$this->status('done');
 	}
 }
\ No newline at end of file