JeroenDeDauw / TermStore

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

tests/integration/TermStoreWriterTest.php

Spacing +4 added lines, -4 removed lines

@@ -28,15 +28,15 @@
 	public function testWhenNoConnection_storeEntityFingerprintThrowsException() {
 		$writer = TestEnvironment::newInstanceWithoutTables()->getFactory()->newTermStoreWriter();
 
-		$this->expectException( TermStoreException::class );
-		$writer->storeEntityFingerprint( new ItemId( 'Q1' ), new Fingerprint() );
+		$this->expectException(TermStoreException::class);
+		$writer->storeEntityFingerprint(new ItemId('Q1'), new Fingerprint());
 	}
 
 	public function testWhenNoConnection_dropTermsForIdThrowsException() {
 		$writer = TestEnvironment::newInstanceWithoutTables()->getFactory()->newTermStoreWriter();
 
-		$this->expectException( TermStoreException::class );
-		$writer->dropTermsForId( new ItemId( 'Q1' ) );
+		$this->expectException(TermStoreException::class);
+		$writer->dropTermsForId(new ItemId('Q1'));
 	}
 
 }

tests/integration/TermStoreExceptionTest.php

Spacing +7 added lines, -7 removed lines

@@ -24,22 +24,22 @@
 	private $store;
 
 	public function setUp() {
-		$connection = DriverManager::getConnection( array(
+		$connection = DriverManager::getConnection(array(
 			'driver' => 'pdo_sqlite',
 			'memory' => true,
-		) );
+		));
 
-		$this->store = new TermStore( $connection, new TermStoreConfig( '' ) );
+		$this->store = new TermStore($connection, new TermStoreConfig(''));
 	}
 
 	public function testInsertWhenStoreNotInstalledCausesTermStoreException() {
-		$id = new ItemId( 'Q1337' );
+		$id = new ItemId('Q1337');
 
 		$fingerprint = new Fingerprint();
-		$fingerprint->setLabel( 'en', 'EN label' );
+		$fingerprint->setLabel('en', 'EN label');
 
-		$this->expectException( TermStoreException::class );
-		$this->store->storeEntityFingerprint( $id, $fingerprint );
+		$this->expectException(TermStoreException::class);
+		$this->store->storeEntityFingerprint($id, $fingerprint);
 	}
 
 }

tests/integration/TermStoreInstallerTest.php

Spacing +8 added lines, -8 removed lines

@@ -27,30 +27,30 @@
 	private $schemaManager;
 
 	public function setUp() {
-		$connection = DriverManager::getConnection( array(
+		$connection = DriverManager::getConnection(array(
 			'driver' => 'pdo_sqlite',
 			'memory' => true,
-		) );
+		));
 
 		$this->schemaManager = $connection->getSchemaManager();
-		$this->storeInstaller = new TermStoreInstaller( $this->schemaManager, new TermStoreConfig( 'kittens_' ) );
+		$this->storeInstaller = new TermStoreInstaller($this->schemaManager, new TermStoreConfig('kittens_'));
 	}
 
 	public function testInstallationAndRemoval() {
 		$this->storeInstaller->install();
 
-		$this->assertTrue( $this->schemaManager->tablesExist( 'kittens_labels' ) );
-		$this->assertTrue( $this->schemaManager->tablesExist( 'kittens_aliases' ) );
+		$this->assertTrue($this->schemaManager->tablesExist('kittens_labels'));
+		$this->assertTrue($this->schemaManager->tablesExist('kittens_aliases'));
 
 		$this->storeInstaller->uninstall();
 
-		$this->assertFalse( $this->schemaManager->tablesExist( 'kittens_labels' ) );
-		$this->assertFalse( $this->schemaManager->tablesExist( 'kittens_aliases' ) );
+		$this->assertFalse($this->schemaManager->tablesExist('kittens_labels'));
+		$this->assertFalse($this->schemaManager->tablesExist('kittens_aliases'));
 	}
 
 	public function testStoresPage() {
 		$this->storeInstaller->install();
-		$this->assertTrue( true );
+		$this->assertTrue(true);
 	}
 
 }