JeroenDeDauw / TermStore

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

src/IdLookup.php

Spacing +29 added lines, -29 removed lines

@@ -15,7 +15,7 @@ 
 	private $labelTable;
 	private $aliasesTable;
 
-	public function __construct( TableQueryExecutor $labelTable, TableQueryExecutor $aliasesTable ) {
+	public function __construct(TableQueryExecutor $labelTable, TableQueryExecutor $aliasesTable) {
 		$this->labelTable = $labelTable;
 		$this->aliasesTable = $aliasesTable;
 	}
@@ -29,8 +29,8 @@ 
 	 * @return string|null
 	 * @throws TermStoreException
 	 */
-	public function getIdByLabel( $labelLanguageCode, $labelText ) {
-		return $this->getEntityIdByLabel( $labelLanguageCode, $labelText );
+	public function getIdByLabel($labelLanguageCode, $labelText) {
+		return $this->getEntityIdByLabel($labelLanguageCode, $labelText);
 	}
 
 	/**
@@ -42,8 +42,8 @@ 
 	 * @return string|null
 	 * @throws TermStoreException
 	 */
-	public function getItemIdByLabel( $labelLanguageCode, $labelText ) {
-		return $this->getEntityIdByLabel( $labelLanguageCode, $labelText, 'item' );
+	public function getItemIdByLabel($labelLanguageCode, $labelText) {
+		return $this->getEntityIdByLabel($labelLanguageCode, $labelText, 'item');
 	}
 
 	/**
@@ -55,8 +55,8 @@ 
 	 * @return string|null
 	 * @throws TermStoreException
 	 */
-	public function getPropertyIdByLabel( $labelLanguageCode, $labelText ) {
-		return $this->getEntityIdByLabel( $labelLanguageCode, $labelText, 'property' );
+	public function getPropertyIdByLabel($labelLanguageCode, $labelText) {
+		return $this->getEntityIdByLabel($labelLanguageCode, $labelText, 'property');
 	}
 
 	/**
@@ -67,14 +67,14 @@ 
 	 * @return string|null
 	 * @throws TermStoreException
 	 */
-	private function getEntityIdByLabel( $labelLanguageCode, $labelText, $entityTypeFilter = null ) {
+	private function getEntityIdByLabel($labelLanguageCode, $labelText, $entityTypeFilter = null) {
 		$conditions = [
-			'text_lowercase' => strtolower( $labelText ),
+			'text_lowercase' => strtolower($labelText),
 			'language' => $labelLanguageCode,
 		];
 
-		if ( $entityTypeFilter !== null ) {
-			$conditions['entity_type'] = $entityTypeFilter;
+		if ($entityTypeFilter !== null) {
+			$conditions[ 'entity_type' ] = $entityTypeFilter;
 		}
 
 		try {
@@ -83,8 +83,8 @@ 
 				$conditions
 			);
 		}
-		catch ( DBALException $ex ) {
-			throw new TermStoreException( $ex->getMessage(), $ex );
+		catch (DBALException $ex) {
+			throw new TermStoreException($ex->getMessage(), $ex);
 		}
 	}
 
@@ -97,8 +97,8 @@ 
 	 * @return string|null
 	 * @throws TermStoreException
 	 */
-	public function getIdByText( $languageCode, $termText ) {
-		return $this->getEntityIdByText( $languageCode, $termText );
+	public function getIdByText($languageCode, $termText) {
+		return $this->getEntityIdByText($languageCode, $termText);
 	}
 
 	/**
@@ -110,8 +110,8 @@ 
 	 * @return string|null
 	 * @throws TermStoreException
 	 */
-	public function getItemIdByText( $languageCode, $termText ) {
-		return $this->getEntityIdByText( $languageCode, $termText, 'item' );
+	public function getItemIdByText($languageCode, $termText) {
+		return $this->getEntityIdByText($languageCode, $termText, 'item');
 	}
 
 	/**
@@ -123,8 +123,8 @@ 
 	 * @return string|null
 	 * @throws TermStoreException
 	 */
-	public function getPropertyIdByText( $languageCode, $termText ) {
-		return $this->getEntityIdByText( $languageCode, $termText, 'property' );
+	public function getPropertyIdByText($languageCode, $termText) {
+		return $this->getEntityIdByText($languageCode, $termText, 'property');
 	}
 
 	/**
@@ -135,14 +135,14 @@ 
 	 * @return string|null
 	 * @throws TermStoreException
 	 */
-	private function getEntityIdByText( $languageCode, $termText, $entityTypeFilter = null ) {
-		$labelMatch = $this->getEntityIdByLabel( $languageCode, $termText, $entityTypeFilter );
+	private function getEntityIdByText($languageCode, $termText, $entityTypeFilter = null) {
+		$labelMatch = $this->getEntityIdByLabel($languageCode, $termText, $entityTypeFilter);
 
-		if ( $labelMatch !== null ) {
+		if ($labelMatch !== null) {
 			return $labelMatch;
 		}
 
-		return $this->getIdByAlias( $languageCode, $termText, $entityTypeFilter );
+		return $this->getIdByAlias($languageCode, $termText, $entityTypeFilter);
 	}
 
 	/**
@@ -153,14 +153,14 @@ 
 	 * @return string|null
 	 * @throws TermStoreException
 	 */
-	private function getIdByAlias( $aliasLanguageCode, $aliasText, $entityTypeFilter = null ) {
+	private function getIdByAlias($aliasLanguageCode, $aliasText, $entityTypeFilter = null) {
 		$conditions = [
-			'text_lowercase' => strtolower( $aliasText ),
+			'text_lowercase' => strtolower($aliasText),
 			'language' => $aliasLanguageCode
 		];
 
-		if ( $entityTypeFilter !== null ) {
-			$conditions['entity_type'] = $entityTypeFilter;
+		if ($entityTypeFilter !== null) {
+			$conditions[ 'entity_type' ] = $entityTypeFilter;
 		}
 
 		try {
@@ -169,8 +169,8 @@ 
 				$conditions
 			);
 		}
-		catch ( DBALException $ex ) {
-			throw new TermStoreException( $ex->getMessage(), $ex );
+		catch (DBALException $ex) {
+			throw new TermStoreException($ex->getMessage(), $ex);
 		}
 	}
 

Braces +2 added lines, -4 removed lines

@@ -82,8 +82,7 @@ 
 				'entity_id',
 				$conditions
 			);
-		}
-		catch ( DBALException $ex ) {
+		} catch ( DBALException $ex ) {
 			throw new TermStoreException( $ex->getMessage(), $ex );
 		}
 	}
@@ -168,8 +167,7 @@ 
 				'entity_id',
 				$conditions
 			);
-		}
-		catch ( DBALException $ex ) {
+		} catch ( DBALException $ex ) {
 			throw new TermStoreException( $ex->getMessage(), $ex );
 		}
 	}

src/TermStoreInstaller.php

Spacing +23 added lines, -23 removed lines

@@ -19,7 +19,7 @@ 
 	private $schemaManager;
 	private $config;
 
-	public function __construct( AbstractSchemaManager $schemaManager, TermStoreConfig $config ) {
+	public function __construct(AbstractSchemaManager $schemaManager, TermStoreConfig $config) {
 		$this->schemaManager = $schemaManager;
 		$this->config = $config;
 	}
@@ -28,45 +28,45 @@ 
 	 * @throws DBALException
 	 */
 	public function install() {
-		$this->schemaManager->createTable( $this->newLabelTable() );
-		$this->schemaManager->createTable( $this->newAliasesTable() );
+		$this->schemaManager->createTable($this->newLabelTable());
+		$this->schemaManager->createTable($this->newAliasesTable());
 	}
 
 	private function newLabelTable() {
-		$table = new Table( $this->config->getLabelTableName() );
+		$table = new Table($this->config->getLabelTableName());
 
-		$table->addColumn( 'text', Type::STRING, array( 'length' => 255 ) );
-		$table->addColumn( 'text_lowercase', Type::STRING, array( 'length' => 255 ) );
-		$table->addColumn( 'language', Type::STRING, array( 'length' => 16 ) );
-		$table->addColumn( 'entity_id', Type::STRING, array( 'length' => 32 ) );
-		$table->addColumn( 'entity_type', Type::STRING, array( 'length' => 16 ) );
+		$table->addColumn('text', Type::STRING, array('length' => 255));
+		$table->addColumn('text_lowercase', Type::STRING, array('length' => 255));
+		$table->addColumn('language', Type::STRING, array('length' => 16));
+		$table->addColumn('entity_id', Type::STRING, array('length' => 32));
+		$table->addColumn('entity_type', Type::STRING, array('length' => 16));
 
-		$table->addIndex( array( 'text_lowercase', 'language' ) );
-		$table->addIndex( array( 'entity_id', 'language' ) );
-		$table->addIndex( array( 'entity_type' ) );
+		$table->addIndex(array('text_lowercase', 'language'));
+		$table->addIndex(array('entity_id', 'language'));
+		$table->addIndex(array('entity_type'));
 
 		return $table;
 	}
 
 	private function newAliasesTable() {
-		$table = new Table( $this->config->getAliasesTableName() );
+		$table = new Table($this->config->getAliasesTableName());
 
-		$table->addColumn( 'text', Type::STRING, array( 'length' => 255 ) );
-		$table->addColumn( 'text_lowercase', Type::STRING, array( 'length' => 255 ) );
-		$table->addColumn( 'language', Type::STRING, array( 'length' => 16 ) );
-		$table->addColumn( 'entity_id', Type::STRING, array( 'length' => 32 ) );
-		$table->addColumn( 'entity_type', Type::STRING, array( 'length' => 16 ) );
+		$table->addColumn('text', Type::STRING, array('length' => 255));
+		$table->addColumn('text_lowercase', Type::STRING, array('length' => 255));
+		$table->addColumn('language', Type::STRING, array('length' => 16));
+		$table->addColumn('entity_id', Type::STRING, array('length' => 32));
+		$table->addColumn('entity_type', Type::STRING, array('length' => 16));
 
-		$table->addIndex( array( 'text_lowercase', 'language' ) );
-		$table->addIndex( array( 'entity_id', 'language' ) );
-		$table->addIndex( array( 'entity_type' ) );
+		$table->addIndex(array('text_lowercase', 'language'));
+		$table->addIndex(array('entity_id', 'language'));
+		$table->addIndex(array('entity_type'));
 
 		return $table;
 	}
 
 	public function uninstall() {
-		$this->schemaManager->dropTable( $this->config->getLabelTableName() );
-		$this->schemaManager->dropTable( $this->config->getAliasesTableName() );
+		$this->schemaManager->dropTable($this->config->getLabelTableName());
+		$this->schemaManager->dropTable($this->config->getAliasesTableName());
 	}
 
 }
\ No newline at end of file

src/TermStoreFactory.php

Spacing +6 added lines, -6 removed lines

@@ -16,7 +16,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,17 +32,17 @@ 
 	 * @return EntityIdLookup
 	 */
 	public function newEntityIdLookup() {
-		$labelTable = new TableQueryExecutor( $this->connection, $this->config->getLabelTableName() );
-		$aliasesTable = new TableQueryExecutor( $this->connection, $this->config->getAliasesTableName() );
+		$labelTable = new TableQueryExecutor($this->connection, $this->config->getLabelTableName());
+		$aliasesTable = new TableQueryExecutor($this->connection, $this->config->getAliasesTableName());
 
-		return new IdLookup( $labelTable, $aliasesTable );
+		return new IdLookup($labelTable, $aliasesTable);
 	}
 
 	/**
 	 * @return TermStoreWriter
 	 */
 	public function newTermStoreWriter() {
-		return new TermStoreWriter( $this->connection, $this->config );
+		return new TermStoreWriter($this->connection, $this->config);
 	}
 
 	/**
@@ -60,7 +60,7 @@ 
 	 * @return TermStore
 	 */
 	public function newTermStore() {
-		return new TermStore( $this->connection, $this->config );
+		return new TermStore($this->connection, $this->config);
 	}
 
 }
\ No newline at end of file

src/LabelLookup.php

Spacing +1 added lines, -1 removed lines

@@ -20,6 +20,6 @@
 	 * @return string|null
 	 * @throws TermStoreException
 	 */
-	public function getLabelByIdAndLanguage( EntityId $id, $languageCode );
+	public function getLabelByIdAndLanguage(EntityId $id, $languageCode);
 
 }
\ No newline at end of file