aimeos / ai-fosuser

lib/custom/setup/FosuserAddIndexes.php

Braces +1 added lines, -2 removed lines

@@ -66,8 +66,7 @@
 			{
 				$this->execute( $stmt );
 				$this->status( 'done' );
-			}
-			else
+			} else
 			{
 				$this->status( 'OK' );
 			}

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

Spacing +46 added lines, -46 removed lines

@@ -132,56 +132,56 @@ 
 
 		$this->assertTrue( $item->getId() !== null );
 		$this->assertEquals( $item->getId(), $itemSaved->getId() );
-		$this->assertEquals( $item->getParentId(), $itemSaved->getParentId());
-		$this->assertEquals( $item->getPosition(), $itemSaved->getPosition());
-		$this->assertEquals( $item->getCompany(), $itemSaved->getCompany());
-		$this->assertEquals( $item->getVatID(), $itemSaved->getVatID());
-		$this->assertEquals( $item->getSalutation(), $itemSaved->getSalutation());
-		$this->assertEquals( $item->getTitle(), $itemSaved->getTitle());
-		$this->assertEquals( $item->getFirstname(), $itemSaved->getFirstname());
-		$this->assertEquals( $item->getLastname(), $itemSaved->getLastname());
-		$this->assertEquals( $item->getAddress1(), $itemSaved->getAddress1());
-		$this->assertEquals( $item->getAddress2(), $itemSaved->getAddress2());
-		$this->assertEquals( $item->getAddress3(), $itemSaved->getAddress3());
-		$this->assertEquals( $item->getPostal(), $itemSaved->getPostal());
-		$this->assertEquals( $item->getCity(), $itemSaved->getCity());
-		$this->assertEquals( $item->getState(), $itemSaved->getState());
-		$this->assertEquals( $item->getCountryId(), $itemSaved->getCountryId());
-		$this->assertEquals( $item->getLanguageId(), $itemSaved->getLanguageId());
-		$this->assertEquals( $item->getTelephone(), $itemSaved->getTelephone());
-		$this->assertEquals( $item->getEmail(), $itemSaved->getEmail());
-		$this->assertEquals( $item->getTelefax(), $itemSaved->getTelefax());
-		$this->assertEquals( $item->getWebsite(), $itemSaved->getWebsite());
-		$this->assertEquals( $item->getLongitude(), $itemSaved->getLongitude());
-		$this->assertEquals( $item->getLatitude(), $itemSaved->getLatitude());
+		$this->assertEquals( $item->getParentId(), $itemSaved->getParentId() );
+		$this->assertEquals( $item->getPosition(), $itemSaved->getPosition() );
+		$this->assertEquals( $item->getCompany(), $itemSaved->getCompany() );
+		$this->assertEquals( $item->getVatID(), $itemSaved->getVatID() );
+		$this->assertEquals( $item->getSalutation(), $itemSaved->getSalutation() );
+		$this->assertEquals( $item->getTitle(), $itemSaved->getTitle() );
+		$this->assertEquals( $item->getFirstname(), $itemSaved->getFirstname() );
+		$this->assertEquals( $item->getLastname(), $itemSaved->getLastname() );
+		$this->assertEquals( $item->getAddress1(), $itemSaved->getAddress1() );
+		$this->assertEquals( $item->getAddress2(), $itemSaved->getAddress2() );
+		$this->assertEquals( $item->getAddress3(), $itemSaved->getAddress3() );
+		$this->assertEquals( $item->getPostal(), $itemSaved->getPostal() );
+		$this->assertEquals( $item->getCity(), $itemSaved->getCity() );
+		$this->assertEquals( $item->getState(), $itemSaved->getState() );
+		$this->assertEquals( $item->getCountryId(), $itemSaved->getCountryId() );
+		$this->assertEquals( $item->getLanguageId(), $itemSaved->getLanguageId() );
+		$this->assertEquals( $item->getTelephone(), $itemSaved->getTelephone() );
+		$this->assertEquals( $item->getEmail(), $itemSaved->getEmail() );
+		$this->assertEquals( $item->getTelefax(), $itemSaved->getTelefax() );
+		$this->assertEquals( $item->getWebsite(), $itemSaved->getWebsite() );
+		$this->assertEquals( $item->getLongitude(), $itemSaved->getLongitude() );
+		$this->assertEquals( $item->getLatitude(), $itemSaved->getLatitude() );
 
 		$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->assertRegExp( '/\d{4}-\d{2}-\d{2} \d{2}:\d{2}:\d{2}/', $itemSaved->getTimeModified() );
 
 		$this->assertEquals( $itemExp->getId(), $itemUpd->getId() );
-		$this->assertEquals( $itemExp->getParentId(), $itemUpd->getParentId());
-		$this->assertEquals( $itemExp->getPosition(), $itemUpd->getPosition());
-		$this->assertEquals( $itemExp->getCompany(), $itemUpd->getCompany());
-		$this->assertEquals( $itemExp->getVatID(), $itemUpd->getVatID());
-		$this->assertEquals( $itemExp->getSalutation(), $itemUpd->getSalutation());
-		$this->assertEquals( $itemExp->getTitle(), $itemUpd->getTitle());
-		$this->assertEquals( $itemExp->getFirstname(), $itemUpd->getFirstname());
-		$this->assertEquals( $itemExp->getLastname(), $itemUpd->getLastname());
-		$this->assertEquals( $itemExp->getAddress1(), $itemUpd->getAddress1());
-		$this->assertEquals( $itemExp->getAddress2(), $itemUpd->getAddress2());
-		$this->assertEquals( $itemExp->getAddress3(), $itemUpd->getAddress3());
-		$this->assertEquals( $itemExp->getPostal(), $itemUpd->getPostal());
-		$this->assertEquals( $itemExp->getCity(), $itemUpd->getCity());
-		$this->assertEquals( $itemExp->getState(), $itemUpd->getState());
-		$this->assertEquals( $itemExp->getCountryId(), $itemUpd->getCountryId());
-		$this->assertEquals( $itemExp->getLanguageId(), $itemUpd->getLanguageId());
-		$this->assertEquals( $itemExp->getTelephone(), $itemUpd->getTelephone());
-		$this->assertEquals( $itemExp->getEmail(), $itemUpd->getEmail());
-		$this->assertEquals( $itemExp->getTelefax(), $itemUpd->getTelefax());
-		$this->assertEquals( $itemExp->getWebsite(), $itemUpd->getWebsite());
-		$this->assertEquals( $itemExp->getLongitude(), $itemUpd->getLongitude());
-		$this->assertEquals( $itemExp->getLatitude(), $itemUpd->getLatitude());
+		$this->assertEquals( $itemExp->getParentId(), $itemUpd->getParentId() );
+		$this->assertEquals( $itemExp->getPosition(), $itemUpd->getPosition() );
+		$this->assertEquals( $itemExp->getCompany(), $itemUpd->getCompany() );
+		$this->assertEquals( $itemExp->getVatID(), $itemUpd->getVatID() );
+		$this->assertEquals( $itemExp->getSalutation(), $itemUpd->getSalutation() );
+		$this->assertEquals( $itemExp->getTitle(), $itemUpd->getTitle() );
+		$this->assertEquals( $itemExp->getFirstname(), $itemUpd->getFirstname() );
+		$this->assertEquals( $itemExp->getLastname(), $itemUpd->getLastname() );
+		$this->assertEquals( $itemExp->getAddress1(), $itemUpd->getAddress1() );
+		$this->assertEquals( $itemExp->getAddress2(), $itemUpd->getAddress2() );
+		$this->assertEquals( $itemExp->getAddress3(), $itemUpd->getAddress3() );
+		$this->assertEquals( $itemExp->getPostal(), $itemUpd->getPostal() );
+		$this->assertEquals( $itemExp->getCity(), $itemUpd->getCity() );
+		$this->assertEquals( $itemExp->getState(), $itemUpd->getState() );
+		$this->assertEquals( $itemExp->getCountryId(), $itemUpd->getCountryId() );
+		$this->assertEquals( $itemExp->getLanguageId(), $itemUpd->getLanguageId() );
+		$this->assertEquals( $itemExp->getTelephone(), $itemUpd->getTelephone() );
+		$this->assertEquals( $itemExp->getEmail(), $itemUpd->getEmail() );
+		$this->assertEquals( $itemExp->getTelefax(), $itemUpd->getTelefax() );
+		$this->assertEquals( $itemExp->getWebsite(), $itemUpd->getWebsite() );
+		$this->assertEquals( $itemExp->getLongitude(), $itemUpd->getLongitude() );
+		$this->assertEquals( $itemExp->getLatitude(), $itemUpd->getLatitude() );
 
 		$this->assertEquals( $this->editor, $itemUpd->getEditor() );
 		$this->assertEquals( $itemExp->getTimeCreated(), $itemUpd->getTimeCreated() );
@@ -190,7 +190,7 @@ 
 		$this->assertInstanceOf( \Aimeos\MShop\Common\Item\Iface::class, $resultSaved );
 		$this->assertInstanceOf( \Aimeos\MShop\Common\Item\Iface::class, $resultUpd );
 
-		$this->setExpectedException('\\Aimeos\\MShop\\Exception');
+		$this->setExpectedException( '\\Aimeos\\MShop\\Exception' );
 		$this->object->getItem( $itemSaved->getId() );
 	}
 

lib/custom/tests/MShop/Customer/Manager/Property/FosUserTest.php

Spacing +11 added lines, -11 removed lines

@@ -57,11 +57,11 @@ 
 		$search->setConditions( $search->compare( '==', 'customer.property.editor', $this->editor ) );
 		$results = $this->object->searchItems( $search );
 
-		if( ( $item = reset($results) ) === false ) {
+		if( ( $item = reset( $results ) ) === false ) {
 			throw new \RuntimeException( 'No property item found' );
 		}
 
-		$item->setId(null);
+		$item->setId( null );
 		$item->setLanguageId( 'en' );
 		$resultSaved = $this->object->saveItem( $item );
 		$itemSaved = $this->object->getItem( $item->getId() );
@@ -110,13 +110,13 @@ 
 	{
 		$search = $this->object->createSearch();
 		$conditions = array(
-			$search->compare( '~=', 'customer.property.value', '1'),
+			$search->compare( '~=', 'customer.property.value', '1' ),
 			$search->compare( '==', 'customer.property.editor', $this->editor )
 		);
 		$search->setConditions( $search->combine( '&&', $conditions ) );
 		$results = $this->object->searchItems( $search );
 
-		if( ($expected = reset($results)) === false ) {
+		if( ( $expected = reset( $results ) ) === false ) {
 			throw new \RuntimeException( sprintf( 'No customer property item found for value "%1$s".', '1' ) );
 		}
 
@@ -155,7 +155,7 @@ 
 		$expr[] = $search->compare( '==', 'customer.property.value', '1' );
 		$expr[] = $search->compare( '==', 'customer.property.editor', $this->editor );
 
-		$search->setConditions( $search->combine('&&', $expr) );
+		$search->setConditions( $search->combine( '&&', $expr ) );
 		$results = $this->object->searchItems( $search, [], $total );
 		$this->assertEquals( 1, count( $results ) );
 	}
@@ -163,17 +163,17 @@ 
 
 	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 testGetSubManagerInvalidName()
 	{
-		$this->setExpectedException('\\Aimeos\\MShop\\Exception');
-		$this->object->getSubManager('type', 'unknown');
+		$this->setExpectedException( '\\Aimeos\\MShop\\Exception' );
+		$this->object->getSubManager( 'type', 'unknown' );
 	}
 }

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

Spacing +2 added lines, -2 removed lines

@@ -492,14 +492,14 @@
 
 			$results = $this->searchItemsBase( $conn, $search, $cfgPathSearch, $cfgPathCount, $required, $total, $level );
 			while( ( $row = $results->fetch() ) !== false ) {
-				$map[ $row['customer.id'] ] = $row;
+				$map[$row['customer.id']] = $row;
 			}
 
 			$dbm->release( $conn, $dbname );
 		}
 		catch( \Exception $e )
 		{
-			$dbm->release( $conn, $dbname  );
+			$dbm->release( $conn, $dbname );
 			throw $e;
 		}
 

Braces +1 added lines, -2 removed lines

@@ -340,8 +340,7 @@
 				 * @see mshop/customer/manager/fosuser/count
 				 */
 				$path = 'mshop/customer/manager/fosuser/insert';
-			}
-			else
+			} else
 			{
 				/** mshop/customer/manager/fosuser/update
 				 * Updates an existing customer record in the database

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

Spacing +8 added lines, -8 removed lines

@@ -8,7 +8,7 @@ 
 
 return array(
 	'table' => array(
-		'fos_user' => function ( \Doctrine\DBAL\Schema\Schema $schema ) {
+		'fos_user' => function( \Doctrine\DBAL\Schema\Schema $schema ) {
 
 			$table = $schema->createTable( 'fos_user' );
 
@@ -48,7 +48,7 @@ 
 			$table->addColumn( 'vdate', 'date', array( 'notnull' => false ) );
 			$table->addColumn( 'mtime', 'datetime', array( 'notnull' => false ) );
 			$table->addColumn( 'ctime', 'datetime', array( 'notnull' => false ) );
-			$table->addColumn( 'editor', 'string', array('length' => 255 ) );
+			$table->addColumn( 'editor', 'string', array( 'length' => 255 ) );
 
 			$table->setPrimaryKey( array( 'id' ), 'pk_fosus_id' );
 			$table->addUniqueIndex( array( 'confirmation_token' ), 'unq_fosus_confirmtoken' );
@@ -65,7 +65,7 @@ 
 			return $schema;
 		},
 
-		'fos_user_address' => function ( \Doctrine\DBAL\Schema\Schema $schema ) {
+		'fos_user_address' => function( \Doctrine\DBAL\Schema\Schema $schema ) {
 
 			$table = $schema->createTable( 'fos_user_address' );
 
@@ -95,7 +95,7 @@ 
 			$table->addColumn( 'latitude', 'float', array( 'notnull' => false ) );
 			$table->addColumn( 'mtime', 'datetime', [] );
 			$table->addColumn( 'ctime', 'datetime', [] );
-			$table->addColumn( 'editor', 'string', array('length' => 255 ) );
+			$table->addColumn( 'editor', 'string', array( 'length' => 255 ) );
 
 			$table->setPrimaryKey( array( 'id' ), 'pk_fosad_id' );
 			$table->addIndex( array( 'parentid' ), 'idx_fosad_pid' );
@@ -112,7 +112,7 @@ 
 			return $schema;
 		},
 
-		'fos_user_list_type' => function ( \Doctrine\DBAL\Schema\Schema $schema ) {
+		'fos_user_list_type' => function( \Doctrine\DBAL\Schema\Schema $schema ) {
 
 			$table = $schema->createTable( 'fos_user_list_type' );
 
@@ -136,7 +136,7 @@ 
 			return $schema;
 		},
 
-		'fos_user_list' => function ( \Doctrine\DBAL\Schema\Schema $schema ) {
+		'fos_user_list' => function( \Doctrine\DBAL\Schema\Schema $schema ) {
 
 			$table = $schema->createTable( 'fos_user_list' );
 
@@ -165,7 +165,7 @@ 
 			return $schema;
 		},
 
-		'fos_user_property_type' => function ( \Doctrine\DBAL\Schema\Schema $schema ) {
+		'fos_user_property_type' => function( \Doctrine\DBAL\Schema\Schema $schema ) {
 
 			$table = $schema->createTable( 'fos_user_property_type' );
 
@@ -189,7 +189,7 @@ 
 			return $schema;
 		},
 
-		'fos_user_property' => function ( \Doctrine\DBAL\Schema\Schema $schema ) {
+		'fos_user_property' => function( \Doctrine\DBAL\Schema\Schema $schema ) {
 
 			$table = $schema->createTable( 'fos_user_property' );
 

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

Spacing +8 added lines, -8 removed lines

@@ -63,7 +63,7 @@ 
 	public function testGetItem()
 	{
 		$search = $this->object->createSearch();
-		$search->setSlice(0, 1);
+		$search->setSlice( 0, 1 );
 		$results = $this->object->searchItems( $search );
 
 		if( ( $item = reset( $results ) ) === false ) {
@@ -77,7 +77,7 @@ 
 	public function testSaveUpdateDeleteItem()
 	{
 		$search = $this->object->createSearch();
-		$search->setSlice(0, 1);
+		$search->setSlice( 0, 1 );
 		$items = $this->object->searchItems( $search );
 
 		if( ( $item = reset( $items ) ) === false ) {
@@ -108,12 +108,12 @@ 
 		$this->assertEquals( $item->getDateEnd(), $itemSaved->getDateEnd() );
 		$this->assertEquals( $item->getPosition(), $itemSaved->getPosition() );
 		$this->assertEquals( $this->editor, $itemSaved->getEditor() );
-		$this->assertStringStartsWith(date('Y-m-d', time()), $itemSaved->getTimeCreated());
-		$this->assertStringStartsWith(date('Y-m-d', time()), $itemSaved->getTimeModified());
+		$this->assertStringStartsWith( date( 'Y-m-d', time() ), $itemSaved->getTimeCreated() );
+		$this->assertStringStartsWith( date( 'Y-m-d', time() ), $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->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() );
@@ -132,7 +132,7 @@ 
 		$this->assertInstanceOf( \Aimeos\MShop\Common\Item\Iface::class, $resultSaved );
 		$this->assertInstanceOf( \Aimeos\MShop\Common\Item\Iface::class, $resultUpd );
 
-		$this->setExpectedException('\\Aimeos\\MShop\\Exception');
+		$this->setExpectedException( '\\Aimeos\\MShop\\Exception' );
 		$this->object->getItem( $itemSaved->getId() );
 	}
 
@@ -278,8 +278,8 @@ 
 
 	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' );

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

Spacing +1 added lines, -1 removed lines

@@ -75,7 +75,7 @@
 				'customer.address.telefax' => '055544332212', 'customer.address.website' => 'www.example.com',
 				'customer.address.longitude' => '10.5', 'customer.address.latitude' => '51.0',
 				'customer.address.position' => '0',
-			],[
+			], [
 				'customer.address.company' => 'Example company LLC', 'customer.address.vatid' => 'DE999999999',
 				'customer.address.salutation' => 'mr', 'customer.address.title' => 'Dr.',
 				'customer.address.firstname' => 'Good', 'customer.address.lastname' => 'Unittest',

lib/custom/setup/CustomerRemoveLostUserDataFosuser.php

Braces +1 added lines, -2 removed lines

@@ -66,8 +66,7 @@
 			{
 				$this->execute( $stmt );
 				$this->status( 'done' );
-			}
-			else
+			} else
 			{
 				$this->status( 'OK' );
 			}

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

Spacing +3 added lines, -3 removed lines

@@ -220,13 +220,13 @@
 		$expr[] = $search->compare( '==', 'customer.longitude', '10.0' );
 		$expr[] = $search->compare( '==', 'customer.latitude', '50.0' );
 
-		$param = ['text','default', 0];
+		$param = ['text', 'default', 0];
 		$expr[] = $search->compare( '==', $search->createFunction( 'customer:has', $param ), null );
 
-		$param = ['text','default', $listItem->getRefId()];
+		$param = ['text', 'default', $listItem->getRefId()];
 		$expr[] = $search->compare( '!=', $search->createFunction( 'customer:has', $param ), null );
 
-		$param = ['text','default'];
+		$param = ['text', 'default'];
 		$expr[] = $search->compare( '!=', $search->createFunction( 'customer:has', $param ), null );
 
 		$param = ['text'];