JeroenDeDauw / TermStore

tests/integration/TestEnvironment.php

Spacing +4 added lines, -4 removed lines

@@ -30,13 +30,13 @@
 	private $factory;
 
 	private function __construct() {
-		$connection = DriverManager::getConnection( array(
+		$connection = DriverManager::getConnection(array(
 				'driver' => 'pdo_sqlite',
 				'memory' => true,
-		) );
+		));
 
-		$config = new TermStoreConfig( 'ts_' );
-		$this->factory = new TermStoreFactory( $connection, $config );
+		$config = new TermStoreConfig('ts_');
+		$this->factory = new TermStoreFactory($connection, $config);
 	}
 
 	public function getFactory() {

tests/integration/TermStoreTest.php

Spacing +80 added lines, -80 removed lines

@@ -29,236 +29,236 @@
 	}
 
 	public function testGivenNotMatchingArgs_getTermByIdAndLanguageReturnsNull() {
-		$this->assertNull( $this->store->getLabelByIdAndLanguage( new ItemId( 'Q1337' ), 'en' ) );
+		$this->assertNull($this->store->getLabelByIdAndLanguage(new ItemId('Q1337'), 'en'));
 	}
 
 	public function testStoreIdAndFingerprint() {
-		$id = new ItemId( 'Q1337' );
+		$id = new ItemId('Q1337');
 
 		$fingerprint = new Fingerprint();
-		$fingerprint->setLabel( 'en', 'en label' );
-		$fingerprint->setLabel( 'de', 'de label' );
-		$fingerprint->setDescription( 'en', 'en description' );
-		$fingerprint->setAliasGroup( 'en', [ 'first en alias', 'second en alias' ] );
+		$fingerprint->setLabel('en', 'en label');
+		$fingerprint->setLabel('de', 'de label');
+		$fingerprint->setDescription('en', 'en description');
+		$fingerprint->setAliasGroup('en', [ 'first en alias', 'second en alias' ]);
 
-		$this->store->storeEntityFingerprint( $id, $fingerprint );
+		$this->store->storeEntityFingerprint($id, $fingerprint);
 
 		$this->assertSame(
 			'en label',
-			$this->store->getLabelByIdAndLanguage( $id, 'en' )
+			$this->store->getLabelByIdAndLanguage($id, 'en')
 		);
 
 		$this->assertSame(
 			'de label',
-			$this->store->getLabelByIdAndLanguage( $id, 'de' )
+			$this->store->getLabelByIdAndLanguage($id, 'de')
 		);
 
 		$this->assertSame(
 			[ 'first en alias', 'second en alias' ],
-			$this->store->getAliasesByIdAndLanguage( $id, 'en' )
+			$this->store->getAliasesByIdAndLanguage($id, 'en')
 		);
 	}
 
 	public function testGetIdByLabelAndLanguage() {
-		$id = new ItemId( 'Q1337' );
+		$id = new ItemId('Q1337');
 
 		$fingerprint = new Fingerprint();
-		$fingerprint->setLabel( 'en', 'en label' );
-		$fingerprint->setLabel( 'de', 'de label' );
+		$fingerprint->setLabel('en', 'en label');
+		$fingerprint->setLabel('de', 'de label');
 
-		$this->store->storeEntityFingerprint( $id, $fingerprint );
+		$this->store->storeEntityFingerprint($id, $fingerprint);
 
 		$this->assertSame(
 			'Q1337',
-			$this->store->getIdByLabel( 'en', 'en label' )
+			$this->store->getIdByLabel('en', 'en label')
 		);
 	}
 
 	public function testStoreFingerprintRemovesOldData() {
-		$id = new ItemId( 'Q1337' );
+		$id = new ItemId('Q1337');
 
 		$fingerprint = new Fingerprint();
-		$fingerprint->setLabel( 'en', 'en label' );
-		$fingerprint->setLabel( 'de', 'de label' );
-		$fingerprint->setAliasGroup( 'en', [ 'first en alias', 'second en alias' ] );
+		$fingerprint->setLabel('en', 'en label');
+		$fingerprint->setLabel('de', 'de label');
+		$fingerprint->setAliasGroup('en', [ 'first en alias', 'second en alias' ]);
 
-		$this->store->storeEntityFingerprint( $id, $fingerprint );
+		$this->store->storeEntityFingerprint($id, $fingerprint);
 
 		$fingerprint = new Fingerprint();
-		$fingerprint->setLabel( 'de', 'new de label' );
+		$fingerprint->setLabel('de', 'new de label');
 
-		$this->store->storeEntityFingerprint( $id, $fingerprint );
+		$this->store->storeEntityFingerprint($id, $fingerprint);
 
 		$this->assertEquals(
 			'new de label',
-			$this->store->getLabelByIdAndLanguage( $id, 'de' )
+			$this->store->getLabelByIdAndLanguage($id, 'de')
 		);
 
-		$this->assertNull( $this->store->getLabelByIdAndLanguage( $id, 'en' ) );
-		$this->assertEmpty( $this->store->getAliasesByIdAndLanguage( $id, 'en' ) );
+		$this->assertNull($this->store->getLabelByIdAndLanguage($id, 'en'));
+		$this->assertEmpty($this->store->getAliasesByIdAndLanguage($id, 'en'));
 	}
 
 	public function testGivenNonMatchingArgs_getAliasesReturnsEmptyArray() {
-		$this->assertSame( [], $this->store->getAliasesByIdAndLanguage( new ItemId( 'Q1337' ), 'en' ) );
+		$this->assertSame([ ], $this->store->getAliasesByIdAndLanguage(new ItemId('Q1337'), 'en'));
 	}
 
 	public function testGetIdByTextReturnsMatchBasedOnLabel() {
-		$id = new ItemId( 'Q1337' );
+		$id = new ItemId('Q1337');
 
 		$fingerprint = new Fingerprint();
-		$fingerprint->setLabel( 'en', 'kittens' );
-		$fingerprint->setAliasGroup( 'en', [ 'first en alias', 'second en alias' ] );
+		$fingerprint->setLabel('en', 'kittens');
+		$fingerprint->setAliasGroup('en', [ 'first en alias', 'second en alias' ]);
 
-		$this->store->storeEntityFingerprint( $id, $fingerprint );
+		$this->store->storeEntityFingerprint($id, $fingerprint);
 
-		$id = new ItemId( 'Q42' );
+		$id = new ItemId('Q42');
 
 		$fingerprint = new Fingerprint();
-		$fingerprint->setLabel( 'en', 'foobar' );
-		$fingerprint->setAliasGroup( 'en', [ 'kittens', 'first en alias' ] );
+		$fingerprint->setLabel('en', 'foobar');
+		$fingerprint->setAliasGroup('en', [ 'kittens', 'first en alias' ]);
 
-		$this->store->storeEntityFingerprint( $id, $fingerprint );
+		$this->store->storeEntityFingerprint($id, $fingerprint);
 
 		$this->assertSame(
 			'Q1337',
-			$this->store->getIdByText( 'en', 'kittens' )
+			$this->store->getIdByText('en', 'kittens')
 		);
 	}
 
 	public function testGetIdByTextReturnsAliasBasedMatchIfNoLabelsMatch() {
-		$id = new ItemId( 'Q1337' );
+		$id = new ItemId('Q1337');
 
 		$fingerprint = new Fingerprint();
-		$fingerprint->setLabel( 'en', 'foobar' );
-		$fingerprint->setAliasGroup( 'en', [ 'first en alias', 'second en alias' ] );
+		$fingerprint->setLabel('en', 'foobar');
+		$fingerprint->setAliasGroup('en', [ 'first en alias', 'second en alias' ]);
 
-		$this->store->storeEntityFingerprint( $id, $fingerprint );
+		$this->store->storeEntityFingerprint($id, $fingerprint);
 
-		$id = new ItemId( 'Q42' );
+		$id = new ItemId('Q42');
 
 		$fingerprint = new Fingerprint();
-		$fingerprint->setLabel( 'en', 'foobar' );
-		$fingerprint->setAliasGroup( 'en', [ 'kittens', 'first en alias' ] );
+		$fingerprint->setLabel('en', 'foobar');
+		$fingerprint->setAliasGroup('en', [ 'kittens', 'first en alias' ]);
 
-		$this->store->storeEntityFingerprint( $id, $fingerprint );
+		$this->store->storeEntityFingerprint($id, $fingerprint);
 
 		$this->assertSame(
 			'Q42',
-			$this->store->getIdByText( 'en', 'kittens' )
+			$this->store->getIdByText('en', 'kittens')
 		);
 	}
 
 	public function testByLabelLookupIsCaseInsensitive() {
-		$id = new ItemId( 'Q1337' );
+		$id = new ItemId('Q1337');
 
 		$fingerprint = new Fingerprint();
-		$fingerprint->setLabel( 'en', 'EN label' );
+		$fingerprint->setLabel('en', 'EN label');
 
-		$this->store->storeEntityFingerprint( $id, $fingerprint );
+		$this->store->storeEntityFingerprint($id, $fingerprint);
 
 		$this->assertSame(
 			'Q1337',
-			$this->store->getIdByLabel( 'en', 'en LABEL' )
+			$this->store->getIdByLabel('en', 'en LABEL')
 		);
 	}
 
 	public function testGetItemIdByLabelReturnsNoPropertyIds() {
-		$id = new PropertyId( 'P1337' );
+		$id = new PropertyId('P1337');
 
 		$fingerprint = new Fingerprint();
-		$fingerprint->setLabel( 'en', 'some label' );
+		$fingerprint->setLabel('en', 'some label');
 
-		$this->store->storeEntityFingerprint( $id, $fingerprint );
+		$this->store->storeEntityFingerprint($id, $fingerprint);
 
-		$this->assertNull( $this->store->getItemIdByLabel( 'en', 'some label' ) );
+		$this->assertNull($this->store->getItemIdByLabel('en', 'some label'));
 	}
 
 	public function testGetPropertyIdByLabelReturnsNoItemIds() {
-		$id = new ItemId( 'Q1337' );
+		$id = new ItemId('Q1337');
 
 		$fingerprint = new Fingerprint();
-		$fingerprint->setLabel( 'en', 'some label' );
+		$fingerprint->setLabel('en', 'some label');
 
-		$this->store->storeEntityFingerprint( $id, $fingerprint );
+		$this->store->storeEntityFingerprint($id, $fingerprint);
 
-		$this->assertNull( $this->store->getPropertyIdByLabel( 'en', 'some label' ) );
+		$this->assertNull($this->store->getPropertyIdByLabel('en', 'some label'));
 	}
 
 	public function testGetItemIdByLabelReturnsItemIds() {
-		$id = new ItemId( 'Q1337' );
+		$id = new ItemId('Q1337');
 
 		$fingerprint = new Fingerprint();
-		$fingerprint->setLabel( 'en', 'some label' );
+		$fingerprint->setLabel('en', 'some label');
 
-		$this->store->storeEntityFingerprint( $id, $fingerprint );
+		$this->store->storeEntityFingerprint($id, $fingerprint);
 
 		$this->assertSame(
 			'Q1337',
-			$this->store->getItemIdByLabel( 'en', 'some label' )
+			$this->store->getItemIdByLabel('en', 'some label')
 		);
 	}
 
 	public function testGetPropertyIdByLabelReturnsPropertyIds() {
-		$id = new PropertyId( 'P1337' );
+		$id = new PropertyId('P1337');
 
 		$fingerprint = new Fingerprint();
-		$fingerprint->setLabel( 'en', 'some label' );
+		$fingerprint->setLabel('en', 'some label');
 
-		$this->store->storeEntityFingerprint( $id, $fingerprint );
+		$this->store->storeEntityFingerprint($id, $fingerprint);
 
 		$this->assertSame(
 			'P1337',
-			$this->store->getPropertyIdByLabel( 'en', 'some label' )
+			$this->store->getPropertyIdByLabel('en', 'some label')
 		);
 	}
 
 	public function testGetItemIdByTextReturnsNoPropertyIds() {
-		$id = new PropertyId( 'P1337' );
+		$id = new PropertyId('P1337');
 
 		$fingerprint = new Fingerprint();
-		$fingerprint->setLabel( 'en', 'some label' );
+		$fingerprint->setLabel('en', 'some label');
 
-		$this->store->storeEntityFingerprint( $id, $fingerprint );
+		$this->store->storeEntityFingerprint($id, $fingerprint);
 
-		$this->assertNull( $this->store->getItemIdByText( 'en', 'some label' ) );
+		$this->assertNull($this->store->getItemIdByText('en', 'some label'));
 	}
 
 	public function testGetPropertyIdByTextReturnsNoItemIds() {
-		$id = new ItemId( 'Q1337' );
+		$id = new ItemId('Q1337');
 
 		$fingerprint = new Fingerprint();
-		$fingerprint->setLabel( 'en', 'some label' );
+		$fingerprint->setLabel('en', 'some label');
 
-		$this->store->storeEntityFingerprint( $id, $fingerprint );
+		$this->store->storeEntityFingerprint($id, $fingerprint);
 
-		$this->assertNull( $this->store->getPropertyIdByText( 'en', 'some label' ) );
+		$this->assertNull($this->store->getPropertyIdByText('en', 'some label'));
 	}
 
 	public function testGetItemIdByTextReturnsItemIds() {
-		$id = new ItemId( 'Q1337' );
+		$id = new ItemId('Q1337');
 
 		$fingerprint = new Fingerprint();
-		$fingerprint->setLabel( 'en', 'some label' );
+		$fingerprint->setLabel('en', 'some label');
 
-		$this->store->storeEntityFingerprint( $id, $fingerprint );
+		$this->store->storeEntityFingerprint($id, $fingerprint);
 
 		$this->assertSame(
 			'Q1337',
-			$this->store->getItemIdByText( 'en', 'some label' )
+			$this->store->getItemIdByText('en', 'some label')
 		);
 	}
 
 	public function testGetPropertyIdByTextReturnsPropertyIds() {
-		$id = new PropertyId( 'P1337' );
+		$id = new PropertyId('P1337');
 
 		$fingerprint = new Fingerprint();
-		$fingerprint->setLabel( 'en', 'some label' );
+		$fingerprint->setLabel('en', 'some label');
 
-		$this->store->storeEntityFingerprint( $id, $fingerprint );
+		$this->store->storeEntityFingerprint($id, $fingerprint);
 
 		$this->assertSame(
 			'P1337',
-			$this->store->getPropertyIdByText( 'en', 'some label' )
+			$this->store->getPropertyIdByText('en', 'some label')
 		);
 	}
 

tests/bootstrap.php

Spacing +9 added lines, -9 removed lines

@@ -1,18 +1,18 @@
 <?php
 
-if ( PHP_SAPI !== 'cli' ) {
-	die( 'Not an entry point' );
+if (PHP_SAPI !== 'cli') {
+	die('Not an entry point');
 }
 
-error_reporting( E_ALL | E_STRICT );
-ini_set( 'display_errors', 1 );
+error_reporting(E_ALL | E_STRICT);
+ini_set('display_errors', 1);
 
-if ( !is_readable( __DIR__ . '/../vendor/autoload.php' ) ) {
-	die( 'You need to install this package with Composer before you can run the tests' );
+if (!is_readable(__DIR__.'/../vendor/autoload.php')) {
+	die('You need to install this package with Composer before you can run the tests');
 }
 
-$autoLoader = require __DIR__ . '/../vendor/autoload.php';
+$autoLoader = require __DIR__.'/../vendor/autoload.php';
 
-$autoLoader->addPsr4( 'Tests\\Queryr\\TermStore\\', __DIR__ . '/integration' );
+$autoLoader->addPsr4('Tests\\Queryr\\TermStore\\', __DIR__.'/integration');
 
-unset( $autoLoader );
\ No newline at end of file
+unset($autoLoader);
\ No newline at end of file

src/TableQueryExecutor.php

Spacing +28 added lines, -28 removed lines

@@ -15,70 +15,70 @@
 	private $connection;
 	private $tableName;
 
-	public function __construct( Connection $connection, $tableName ) {
+	public function __construct(Connection $connection, $tableName) {
 		$this->connection = $connection;
 		$this->tableName = $tableName;
 	}
 
-	public function selectOneField( $fieldName, array $conditions = [] ) {
-		return $this->selectOne( [ $fieldName ], $conditions )[$fieldName];
+	public function selectOneField($fieldName, array $conditions = [ ]) {
+		return $this->selectOne([ $fieldName ], $conditions)[ $fieldName ];
 	}
 
-	public function selectOne( array $fieldNames = null, array $conditions = [] ) {
-		$statement = $this->executeSelect( $fieldNames, $conditions, 1 );
+	public function selectOne(array $fieldNames = null, array $conditions = [ ]) {
+		$statement = $this->executeSelect($fieldNames, $conditions, 1);
 
 		$result = $statement->fetch();
 		return $result === false ? null : $result;
 	}
 
-	public function select( array $fieldNames = null, array $conditions = [] ) {
-		$statement = $this->executeSelect( $fieldNames, $conditions );
+	public function select(array $fieldNames = null, array $conditions = [ ]) {
+		$statement = $this->executeSelect($fieldNames, $conditions);
 		return $statement->fetchAll();
 	}
 
-	public function selectField( $fieldName, array $conditions = [] ) {
+	public function selectField($fieldName, array $conditions = [ ]) {
 		return array_map(
-			function( array $resultRow ) use ( $fieldName ) {
-				return $resultRow[$fieldName];
+			function(array $resultRow) use ($fieldName) {
+				return $resultRow[ $fieldName ];
 			},
-			$this->select( [ $fieldName ], $conditions )
+			$this->select([ $fieldName ], $conditions)
 		);
 	}
 
-	private function executeSelect( array $fieldNames = null, array $conditions = [], $limit = null ) {
-		$sql = $this->buildSelectSql( $fieldNames, $conditions, $limit );
+	private function executeSelect(array $fieldNames = null, array $conditions = [ ], $limit = null) {
+		$sql = $this->buildSelectSql($fieldNames, $conditions, $limit);
 
-		$statement = $this->connection->prepare( $sql );
-		$statement->execute( array_values( $conditions ) );
+		$statement = $this->connection->prepare($sql);
+		$statement->execute(array_values($conditions));
 
 		return $statement;
 	}
 
-	private function buildSelectSql( array $fieldNames = null, array $conditions = [], $limit = null ) {
-		$fieldSql = $this->getFieldSql( $fieldNames );
-		$conditionSql = $this->getConditionSql( $conditions );
+	private function buildSelectSql(array $fieldNames = null, array $conditions = [ ], $limit = null) {
+		$fieldSql = $this->getFieldSql($fieldNames);
+		$conditionSql = $this->getConditionSql($conditions);
 
-		$sql = 'SELECT ' . $fieldSql . ' FROM ' . $this->tableName . ' WHERE ' . $conditionSql;
+		$sql = 'SELECT '.$fieldSql.' FROM '.$this->tableName.' WHERE '.$conditionSql;
 
-		if ( $limit !== null ) {
-			$sql .= ' LIMIT ' . (int)$limit;
+		if ($limit !== null) {
+			$sql .= ' LIMIT '.(int)$limit;
 		}
 
 		return $sql;
 	}
 
-	private function getFieldSql( array $fields = null ) {
-		return $fields === null || $fields === [] ? '*' : implode( ', ', (array)$fields );
+	private function getFieldSql(array $fields = null) {
+		return $fields === null || $fields === [ ] ? '*' : implode(', ', (array)$fields);
 	}
 
-	private function getConditionSql( array $conditions ) {
-		$wherePredicates = [];
+	private function getConditionSql(array $conditions) {
+		$wherePredicates = [ ];
 
-		foreach ( $conditions as $columnName => $columnValue ) {
-			$wherePredicates[] = $columnName . ' = ?';
+		foreach ($conditions as $columnName => $columnValue) {
+			$wherePredicates[ ] = $columnName.' = ?';
 		}
 
-		return implode( ' AND ', $wherePredicates );
+		return implode(' AND ', $wherePredicates);
 	}
 
 }

src/EntityIdLookup.php

Spacing +6 added lines, -6 removed lines

@@ -20,7 +20,7 @@ 
 	 * @return string|null
 	 * @throws TermStoreException
 	 */
-	public function getIdByLabel( $labelLanguageCode, $labelText );
+	public function getIdByLabel($labelLanguageCode, $labelText);
 
 	/**
 	 * Returns the first matching item id. Case insensitive.
@@ -31,7 +31,7 @@ 
 	 * @return string|null
 	 * @throws TermStoreException
 	 */
-	public function getItemIdByLabel( $labelLanguageCode, $labelText );
+	public function getItemIdByLabel($labelLanguageCode, $labelText);
 
 	/**
 	 * Returns the first matching property id. Case insensitive.
@@ -42,7 +42,7 @@ 
 	 * @return string|null
 	 * @throws TermStoreException
 	 */
-	public function getPropertyIdByLabel( $labelLanguageCode, $labelText );
+	public function getPropertyIdByLabel($labelLanguageCode, $labelText);
 
 	/**
 	 * Returns the first matching entity id. Case insensitive.
@@ -53,7 +53,7 @@ 
 	 * @return string|null
 	 * @throws TermStoreException
 	 */
-	public function getIdByText( $languageCode, $termText );
+	public function getIdByText($languageCode, $termText);
 
 	/**
 	 * Returns the first matching item id. Case insensitive.
@@ -64,7 +64,7 @@ 
 	 * @return string|null
 	 * @throws TermStoreException
 	 */
-	public function getItemIdByText( $languageCode, $termText );
+	public function getItemIdByText($languageCode, $termText);
 
 	/**
 	 * Returns the first matching property id. Case insensitive.
@@ -75,6 +75,6 @@ 
 	 * @return string|null
 	 * @throws TermStoreException
 	 */
-	public function getPropertyIdByText( $languageCode, $termText );
+	public function getPropertyIdByText($languageCode, $termText);
 
 }
\ No newline at end of file

src/TermStoreConfig.php

Spacing +3 added lines, -3 removed lines

@@ -13,16 +13,16 @@
 
 	private $prefix;
 
-	public function __construct( $tablePrefix = '' ) {
+	public function __construct($tablePrefix = '') {
 		$this->prefix = $tablePrefix;
 	}
 
 	public function getLabelTableName() {
-		return $this->prefix . 'labels';
+		return $this->prefix.'labels';
 	}
 
 	public function getAliasesTableName() {
-		return $this->prefix . 'aliases';
+		return $this->prefix.'aliases';
 	}
 
 }
\ No newline at end of file

src/TermStore.php

Spacing +27 added lines, -27 removed lines

@@ -22,12 +22,12 @@ 
 	private $labelTable;
 	private $aliasesTable;
 
-	public function __construct( Connection $connection, TermStoreConfig $config ) {
-		$this->labelTable = new TableQueryExecutor( $connection, $config->getLabelTableName() );
-		$this->aliasesTable = new TableQueryExecutor( $connection, $config->getAliasesTableName() );
+	public function __construct(Connection $connection, TermStoreConfig $config) {
+		$this->labelTable = new TableQueryExecutor($connection, $config->getLabelTableName());
+		$this->aliasesTable = new TableQueryExecutor($connection, $config->getAliasesTableName());
 
-		$this->storeWriter = new TermStoreWriter( $connection, $config );
-		$this->idLookup = new IdLookup( $this->labelTable, $this->aliasesTable );
+		$this->storeWriter = new TermStoreWriter($connection, $config);
+		$this->idLookup = new IdLookup($this->labelTable, $this->aliasesTable);
 	}
 
 	/**
@@ -36,8 +36,8 @@ 
 	 *
 	 * @throws TermStoreException
 	 */
-	public function storeEntityFingerprint( EntityId $id, Fingerprint $fingerprint ) {
-		$this->storeWriter->storeEntityFingerprint( $id, $fingerprint );
+	public function storeEntityFingerprint(EntityId $id, Fingerprint $fingerprint) {
+		$this->storeWriter->storeEntityFingerprint($id, $fingerprint);
 	}
 
 	/**
@@ -45,8 +45,8 @@ 
 	 *
 	 * @throws TermStoreException
 	 */
-	public function dropTermsForId( EntityId $id ) {
-		$this->storeWriter->dropTermsForId( $id );
+	public function dropTermsForId(EntityId $id) {
+		$this->storeWriter->dropTermsForId($id);
 	}
 
 	/**
@@ -56,7 +56,7 @@ 
 	 * @return string|null
 	 * @throws TermStoreException
 	 */
-	public function getLabelByIdAndLanguage( EntityId $id, $languageCode ) {
+	public function getLabelByIdAndLanguage(EntityId $id, $languageCode) {
 		try {
 			return $this->labelTable->selectOneField(
 				'text',
@@ -66,8 +66,8 @@ 
 				]
 			);
 		}
-		catch ( DBALException $ex ) {
-			throw new TermStoreException( $ex->getMessage(), $ex );
+		catch (DBALException $ex) {
+			throw new TermStoreException($ex->getMessage(), $ex);
 		}
 	}
 
@@ -78,7 +78,7 @@ 
 	 * @return string[]
 	 * @throws TermStoreException
 	 */
-	public function getAliasesByIdAndLanguage( EntityId $id, $languageCode ) {
+	public function getAliasesByIdAndLanguage(EntityId $id, $languageCode) {
 		try {
 			return $this->aliasesTable->selectField(
 				'text',
@@ -88,8 +88,8 @@ 
 				]
 			);
 		}
-		catch ( DBALException $ex ) {
-			throw new TermStoreException( $ex->getMessage(), $ex );
+		catch (DBALException $ex) {
+			throw new TermStoreException($ex->getMessage(), $ex);
 		}
 	}
 
@@ -102,8 +102,8 @@ 
 	 * @return string|null
 	 * @throws TermStoreException
 	 */
-	public function getIdByLabel( $labelLanguageCode, $labelText ) {
-		return $this->idLookup->getIdByLabel( $labelLanguageCode, $labelText );
+	public function getIdByLabel($labelLanguageCode, $labelText) {
+		return $this->idLookup->getIdByLabel($labelLanguageCode, $labelText);
 	}
 
 	/**
@@ -115,8 +115,8 @@ 
 	 * @return string|null
 	 * @throws TermStoreException
 	 */
-	public function getItemIdByLabel( $labelLanguageCode, $labelText ) {
-		return $this->idLookup->getItemIdByLabel( $labelLanguageCode, $labelText );
+	public function getItemIdByLabel($labelLanguageCode, $labelText) {
+		return $this->idLookup->getItemIdByLabel($labelLanguageCode, $labelText);
 	}
 
 	/**
@@ -128,8 +128,8 @@ 
 	 * @return string|null
 	 * @throws TermStoreException
 	 */
-	public function getPropertyIdByLabel( $labelLanguageCode, $labelText ) {
-		return $this->idLookup->getPropertyIdByLabel( $labelLanguageCode, $labelText );
+	public function getPropertyIdByLabel($labelLanguageCode, $labelText) {
+		return $this->idLookup->getPropertyIdByLabel($labelLanguageCode, $labelText);
 	}
 
 	/**
@@ -141,8 +141,8 @@ 
 	 * @return string|null
 	 * @throws TermStoreException
 	 */
-	public function getIdByText( $languageCode, $termText ) {
-		return $this->idLookup->getIdByText( $languageCode, $termText );
+	public function getIdByText($languageCode, $termText) {
+		return $this->idLookup->getIdByText($languageCode, $termText);
 	}
 
 	/**
@@ -154,8 +154,8 @@ 
 	 * @return string|null
 	 * @throws TermStoreException
 	 */
-	public function getItemIdByText( $languageCode, $termText ) {
-		return $this->idLookup->getItemIdByText( $languageCode, $termText );
+	public function getItemIdByText($languageCode, $termText) {
+		return $this->idLookup->getItemIdByText($languageCode, $termText);
 	}
 
 	/**
@@ -167,8 +167,8 @@ 
 	 * @return string|null
 	 * @throws TermStoreException
 	 */
-	public function getPropertyIdByText( $languageCode, $termText ) {
-		return $this->idLookup->getPropertyIdByText( $languageCode, $termText );
+	public function getPropertyIdByText($languageCode, $termText) {
+		return $this->idLookup->getPropertyIdByText($languageCode, $termText);
 	}
 
 }

Braces +2 added lines, -4 removed lines

@@ -65,8 +65,7 @@ 
 					'language' => $languageCode
 				]
 			);
-		}
-		catch ( DBALException $ex ) {
+		} catch ( DBALException $ex ) {
 			throw new TermStoreException( $ex->getMessage(), $ex );
 		}
 	}
@@ -87,8 +86,7 @@ 
 					'language' => $languageCode
 				]
 			);
-		}
-		catch ( DBALException $ex ) {
+		} catch ( DBALException $ex ) {
 			throw new TermStoreException( $ex->getMessage(), $ex );
 		}
 	}

src/TermStoreWriter.php

Spacing +19 added lines, -19 removed lines

@@ -21,7 +21,7 @@ 
 	private $connection;
 	private $config;
 
-	public function __construct( Connection $connection, TermStoreConfig $config ) {
+	public function __construct(Connection $connection, TermStoreConfig $config) {
 		$this->connection = $connection;
 		$this->config = $config;
 	}
@@ -32,35 +32,35 @@ 
 	 *
 	 * @throws TermStoreException
 	 */
-	public function storeEntityFingerprint( EntityId $id, Fingerprint $fingerprint ) {
+	public function storeEntityFingerprint(EntityId $id, Fingerprint $fingerprint) {
 		$this->connection->beginTransaction();
 
-		$this->dropTermsForId( $id );
+		$this->dropTermsForId($id);
 
 		try {
-			$this->storeFingerprintParts( $id, $fingerprint );
+			$this->storeFingerprintParts($id, $fingerprint);
 		}
-		catch ( DBALException $ex ) {
+		catch (DBALException $ex) {
 			$this->connection->rollBack();
-			throw new TermStoreException( $ex->getMessage(), $ex );
+			throw new TermStoreException($ex->getMessage(), $ex);
 		}
 
 		$this->connection->commit();
 	}
 
-	private function storeFingerprintParts( EntityId $id, Fingerprint $fingerprint ) {
+	private function storeFingerprintParts(EntityId $id, Fingerprint $fingerprint) {
 		/**
 		 * @var Term $label
 		 */
-		foreach ( $fingerprint->getLabels() as $label ) {
-			$this->storeLabel( $label->getLanguageCode(), $label->getText(), $id );
+		foreach ($fingerprint->getLabels() as $label) {
+			$this->storeLabel($label->getLanguageCode(), $label->getText(), $id);
 		}
 
 		/**
 		 * @var AliasGroup $aliasGroup
 		 */
-		foreach ( $fingerprint->getAliasGroups() as $aliasGroup ) {
-			$this->storeAliases( $aliasGroup, $id );
+		foreach ($fingerprint->getAliasGroups() as $aliasGroup) {
+			$this->storeAliases($aliasGroup, $id);
 		}
 	}
 
@@ -69,7 +69,7 @@ 
 	 *
 	 * @throws TermStoreException
 	 */
-	public function dropTermsForId( EntityId $id ) {
+	public function dropTermsForId(EntityId $id) {
 		try {
 			$this->connection->delete(
 				$this->config->getLabelTableName(),
@@ -81,17 +81,17 @@ 
 				[ 'entity_id' => $id->getSerialization() ]
 			);
 		}
-		catch ( DBALException $ex ) {
-			throw new TermStoreException( $ex->getMessage(), $ex );
+		catch (DBALException $ex) {
+			throw new TermStoreException($ex->getMessage(), $ex);
 		}
 	}
 
-	private function storeLabel( $languageCode, $text, EntityId $id ) {
+	private function storeLabel($languageCode, $text, EntityId $id) {
 		$this->connection->insert(
 			$this->config->getLabelTableName(),
 			[
 				'text' => $text,
-				'text_lowercase' => strtolower( $text ),
+				'text_lowercase' => strtolower($text),
 				'language' => $languageCode,
 				'entity_id' => $id->getSerialization(),
 				'entity_type' => $id->getEntityType()
@@ -99,13 +99,13 @@ 
 		);
 	}
 
-	private function storeAliases( AliasGroup $aliasGroup, EntityId $id ) {
-		foreach ( $aliasGroup->getAliases() as $alias ) {
+	private function storeAliases(AliasGroup $aliasGroup, EntityId $id) {
+		foreach ($aliasGroup->getAliases() as $alias) {
 			$this->connection->insert(
 				$this->config->getAliasesTableName(),
 				[
 					'text' => $alias,
-					'text_lowercase' => strtolower( $alias ),
+					'text_lowercase' => strtolower($alias),
 					'language' => $aliasGroup->getLanguageCode(),
 					'entity_id' => $id->getSerialization(),
 					'entity_type' => $id->getEntityType()

Braces +2 added lines, -4 removed lines

@@ -39,8 +39,7 @@ 
 
 		try {
 			$this->storeFingerprintParts( $id, $fingerprint );
-		}
-		catch ( DBALException $ex ) {
+		} catch ( DBALException $ex ) {
 			$this->connection->rollBack();
 			throw new TermStoreException( $ex->getMessage(), $ex );
 		}
@@ -80,8 +79,7 @@ 
 				$this->config->getAliasesTableName(),
 				[ 'entity_id' => $id->getSerialization() ]
 			);
-		}
-		catch ( DBALException $ex ) {
+		} catch ( DBALException $ex ) {
 			throw new TermStoreException( $ex->getMessage(), $ex );
 		}
 	}

src/TermStoreException.php

Spacing +2 added lines, -2 removed lines

@@ -11,8 +11,8 @@
  */
 class TermStoreException extends \RuntimeException {
 
-	public function __construct( $message, \Exception $previous = null ) {
-		parent::__construct( $message, 0, $previous );
+	public function __construct($message, \Exception $previous = null) {
+		parent::__construct($message, 0, $previous);
 	}
 
 }
\ No newline at end of file