JeroenDeDauw / EntityStore

src/PropertyTypeLookup.php

Spacing +15 added lines, -15 removed lines

@@ -21,7 +21,7 @@ 
 	 * @param Connection $connection
 	 * @param string $tableName
 	 */
-	public function __construct( Connection $connection, $tableName ) {
+	public function __construct(Connection $connection, $tableName) {
 		$this->connection = $connection;
 		$this->tableName = $tableName;
 	}
@@ -32,35 +32,35 @@ 
 	 * @return string|null
 	 * @throws PropertyTypeLookupException
 	 */
-	public function getTypeOfProperty( PropertyId $id ) {
-		$query = $this->buildQuery( $id );
+	public function getTypeOfProperty(PropertyId $id) {
+		$query = $this->buildQuery($id);
 
 		try {
 			$queryResult = $query->execute();
 		}
-		catch ( DBALException $ex ) {
-			throw new PropertyTypeLookupException( $ex->getMessage(), $ex );
+		catch (DBALException $ex) {
+			throw new PropertyTypeLookupException($ex->getMessage(), $ex);
 		}
 
-		return $this->getTypeFromResult( $queryResult );
+		return $this->getTypeFromResult($queryResult);
 	}
 
-	private function buildQuery( PropertyId $id ) {
+	private function buildQuery(PropertyId $id) {
 		return $this->connection->createQueryBuilder()
-			->select( 'property_type' )
-			->from( $this->tableName )
-			->where( 'property_id = ?' )
-			->setParameter( 0, (int)$id->getNumericId() );
+			->select('property_type')
+			->from($this->tableName)
+			->where('property_id = ?')
+			->setParameter(0, (int)$id->getNumericId());
 	}
 
-	private function getTypeFromResult( \Traversable $rows ) {
-		$rows = iterator_to_array( $rows );
+	private function getTypeFromResult(\Traversable $rows) {
+		$rows = iterator_to_array($rows);
 
-		if ( count( $rows ) < 1 ) {
+		if (count($rows) < 1) {
 			return null;
 		}
 
-		return reset( $rows )['property_type'];
+		return reset($rows)['property_type'];
 	}
 
 }
\ No newline at end of file

Braces +1 added lines, -2 removed lines

@@ -37,8 +37,7 @@
 
 		try {
 			$queryResult = $query->execute();
-		}
-		catch ( DBALException $ex ) {
+		} catch ( DBALException $ex ) {
 			throw new PropertyTypeLookupException( $ex->getMessage(), $ex );
 		}
 

src/EntityStore.php

Spacing +13 added lines, -13 removed lines

@@ -22,7 +22,7 @@ 
 	 * @param ItemStore $itemStore
 	 * @param PropertyStore $propertyStore
 	 */
-	public function __construct( ItemStore $itemStore, PropertyStore $propertyStore ) {
+	public function __construct(ItemStore $itemStore, PropertyStore $propertyStore) {
 		$this->itemStore = $itemStore;
 		$this->propertyStore = $propertyStore;
 	}
@@ -32,8 +32,8 @@ 
 	 *
 	 * @throws EntityStoreException
 	 */
-	public function storeItemRow( ItemRow $itemRow ) {
-		$this->itemStore->storeItemRow( $itemRow );
+	public function storeItemRow(ItemRow $itemRow) {
+		$this->itemStore->storeItemRow($itemRow);
 	}
 
 	/**
@@ -41,8 +41,8 @@ 
 	 *
 	 * @throws EntityStoreException
 	 */
-	public function storePropertyRow( PropertyRow $propertyRow ) {
-		$this->propertyStore->storePropertyRow( $propertyRow );
+	public function storePropertyRow(PropertyRow $propertyRow) {
+		$this->propertyStore->storePropertyRow($propertyRow);
 	}
 
 	/**
@@ -50,8 +50,8 @@ 
 	 * @return ItemRow|null
 	 * @throws EntityStoreException
 	 */
-	public function getItemRowByNumericItemId( $numericItemId ) {
-		return $this->itemStore->getItemRowByNumericItemId( $numericItemId );
+	public function getItemRowByNumericItemId($numericItemId) {
+		return $this->itemStore->getItemRowByNumericItemId($numericItemId);
 	}
 
 	/**
@@ -60,8 +60,8 @@ 
 	 * @return PropertyRow|null
 	 * @throws EntityStoreException
 	 */
-	public function getPropertyRowByNumericPropertyId( $numericPropertyId ) {
-		return $this->propertyStore->getPropertyRowByNumericPropertyId( $numericPropertyId );
+	public function getPropertyRowByNumericPropertyId($numericPropertyId) {
+		return $this->propertyStore->getPropertyRowByNumericPropertyId($numericPropertyId);
 	}
 
 	/**
@@ -71,8 +71,8 @@ 
 	 * @return PropertyInfo[]
 	 * @throws EntityStoreException
 	 */
-	public function getPropertyInfo( $limit, $offset ) {
-		return $this->propertyStore->getPropertyInfo( $limit, $offset );
+	public function getPropertyInfo($limit, $offset) {
+		return $this->propertyStore->getPropertyInfo($limit, $offset);
 	}
 
 	/**
@@ -82,8 +82,8 @@ 
 	 * @return ItemInfo[]
 	 * @throws EntityStoreException
 	 */
-	public function getItemInfo( $limit, $offset ) {
-		return $this->itemStore->getItemInfo( $limit, $offset );
+	public function getItemInfo($limit, $offset) {
+		return $this->itemStore->getItemInfo($limit, $offset);
 	}
 
 }
\ No newline at end of file

src/ItemRowFactory.php

Spacing +23 added lines, -23 removed lines

@@ -17,7 +17,7 @@ 
 	private $itemSerializer;
 	private $typeExtractor;
 
-	public function __construct( Serializer $itemSerializer, ItemTypeExtractor $typeExtractor ) {
+	public function __construct(Serializer $itemSerializer, ItemTypeExtractor $typeExtractor) {
 		$this->itemSerializer = $itemSerializer;
 		$this->typeExtractor = $typeExtractor;
 	}
@@ -29,39 +29,39 @@ 
 	 * @return ItemRow
 	 * @throws InvalidArgumentException
 	 */
-	public function newFromItemAndPageInfo( Item $item, EntityPageInfo $pageInfo ) {
-		if ( $item->getId() === null ) {
-			throw new InvalidArgumentException( 'The items id cannot be null' );
+	public function newFromItemAndPageInfo(Item $item, EntityPageInfo $pageInfo) {
+		if ($item->getId() === null) {
+			throw new InvalidArgumentException('The items id cannot be null');
 		}
 
-		return ( new ItemRow() )
-			->setPageTitle( $pageInfo->getPageTitle() )
-			->setRevisionId( $pageInfo->getRevisionId() )
-			->setRevisionTime( $pageInfo->getRevisionTime() )
-			->setEnglishLabel( $this->getEnglishLabel( $item ) )
-			->setItemType( $this->getItemType( $item ) )
-			->setNumericItemId( $item->getId()->getNumericId() )
-			->setItemJson( $this->getItemJson( $item ) )
-			->setEnglishWikipediaTitle( $this->getEnWikiTitle( $item ) );
+		return (new ItemRow())
+			->setPageTitle($pageInfo->getPageTitle())
+			->setRevisionId($pageInfo->getRevisionId())
+			->setRevisionTime($pageInfo->getRevisionTime())
+			->setEnglishLabel($this->getEnglishLabel($item))
+			->setItemType($this->getItemType($item))
+			->setNumericItemId($item->getId()->getNumericId())
+			->setItemJson($this->getItemJson($item))
+			->setEnglishWikipediaTitle($this->getEnWikiTitle($item));
 	}
 
-	private function getItemJson( Item $item ) {
-		return json_encode( $this->itemSerializer->serialize( $item ) );
+	private function getItemJson(Item $item) {
+		return json_encode($this->itemSerializer->serialize($item));
 	}
 
-	private function getItemType( Item $item ) {
-		return $this->typeExtractor->getTypeOfItem( $item );
+	private function getItemType(Item $item) {
+		return $this->typeExtractor->getTypeOfItem($item);
 	}
 
-	private function getEnglishLabel( Item $item ) {
-		return $item->getFingerprint()->hasLabel( 'en' ) ?
-			$item->getFingerprint()->getLabel( 'en' )->getText()
+	private function getEnglishLabel(Item $item) {
+		return $item->getFingerprint()->hasLabel('en') ?
+			$item->getFingerprint()->getLabel('en')->getText()
 			: null;
 	}
 
-	private function getEnWikiTitle( Item $item ) {
-		return $item->getSiteLinkList()->hasLinkWithSiteId( 'enwiki' ) ?
-			$item->getSiteLinkList()->getBySiteId( 'enwiki' )->getPageName() : null;
+	private function getEnWikiTitle(Item $item) {
+		return $item->getSiteLinkList()->hasLinkWithSiteId('enwiki') ?
+			$item->getSiteLinkList()->getBySiteId('enwiki')->getPageName() : null;
 	}
 
 }
\ No newline at end of file

src/ItemStore.php

Spacing +72 added lines, -72 removed lines

@@ -23,7 +23,7 @@ 
 	 * @param Connection $connection
 	 * @param string $tableName
 	 */
-	public function __construct( Connection $connection, $tableName ) {
+	public function __construct(Connection $connection, $tableName) {
 		$this->connection = $connection;
 		$this->tableName = $tableName;
 	}
@@ -33,8 +33,8 @@ 
 	 *
 	 * @throws EntityStoreException
 	 */
-	public function storeItemRow( ItemRow $itemRow ) {
-		$this->deleteItemById( ItemId::newFromNumber( $itemRow->getNumericItemId() ) );
+	public function storeItemRow(ItemRow $itemRow) {
+		$this->deleteItemById(ItemId::newFromNumber($itemRow->getNumericItemId()));
 
 		try {
 			$this->connection->insert(
@@ -53,8 +53,8 @@ 
 				]
 			);
 		}
-		catch ( DBALException $ex ) {
-			throw new EntityStoreException( $ex->getMessage(), $ex );
+		catch (DBALException $ex) {
+			throw new EntityStoreException($ex->getMessage(), $ex);
 		}
 	}
 
@@ -63,7 +63,7 @@ 
 	 *
 	 * @throws EntityStoreException
 	 */
-	public function deleteItemById( ItemId $itemId ) {
+	public function deleteItemById(ItemId $itemId) {
 		try {
 			$this->connection->delete(
 				$this->tableName,
@@ -72,8 +72,8 @@ 
 				]
 			);
 		}
-		catch ( DBALException $ex ) {
-			throw new EntityStoreException( $ex->getMessage(), $ex );
+		catch (DBALException $ex) {
+			throw new EntityStoreException($ex->getMessage(), $ex);
 		}
 	}
 
@@ -82,18 +82,18 @@ 
 	 * @return ItemRow|null
 	 * @throws EntityStoreException
 	 */
-	public function getItemRowByNumericItemId( $numericItemId ) {
+	public function getItemRowByNumericItemId($numericItemId) {
 		try {
 			$rows = $this->selectItems()
-				->where( 'item_id = ?' )
-				->setParameter( 0, (int)$numericItemId )
+				->where('item_id = ?')
+				->setParameter(0, (int)$numericItemId)
 				->execute();
 		}
-		catch ( DBALException $ex ) {
-			throw new EntityStoreException( $ex->getMessage(), $ex );
+		catch (DBALException $ex) {
+			throw new EntityStoreException($ex->getMessage(), $ex);
 		}
 
-		return $this->newItemRowFromResult( $rows );
+		return $this->newItemRowFromResult($rows);
 	}
 
 	private function selectItems() {
@@ -106,38 +106,38 @@ 
 			'item_type',
 			'item_label_en',
 			'wp_title_en'
-		)->from( $this->tableName );
+		)->from($this->tableName);
 	}
 
-	private function newItemRowFromResult( \Traversable $rows ) {
-		$rows = iterator_to_array( $rows );
+	private function newItemRowFromResult(\Traversable $rows) {
+		$rows = iterator_to_array($rows);
 
-		if ( count( $rows ) < 1 ) {
+		if (count($rows) < 1) {
 			return null;
 		}
 
-		$row = reset( $rows );
+		$row = reset($rows);
 
-		return ( new ItemRow() )
-			->setItemJson( $row['item_json'] )
-			->setNumericItemId( $row['item_id'] )
-			->setPageTitle( $row['page_title'] )
-			->setRevisionId( $row['revision_id'] )
-			->setRevisionTime( $row['revision_time'] )
-			->setItemType( $row['item_type'] )
-			->setEnglishWikipediaTitle( $row['wp_title_en'] )
-			->setEnglishLabel( $row['item_label_en'] );
+		return (new ItemRow())
+			->setItemJson($row['item_json'])
+			->setNumericItemId($row['item_id'])
+			->setPageTitle($row['page_title'])
+			->setRevisionId($row['revision_id'])
+			->setRevisionTime($row['revision_time'])
+			->setItemType($row['item_type'])
+			->setEnglishWikipediaTitle($row['wp_title_en'])
+			->setEnglishLabel($row['item_label_en']);
 	}
 
-	private function newItemInfoFromResultRow( array $row ) {
-		return ( new ItemInfo() )
-			->setNumericItemId( $row['item_id'] )
-			->setPageTitle( $row['page_title'] )
-			->setRevisionId( $row['revision_id'] )
-			->setRevisionTime( $row['revision_time'] )
-			->setItemType( $row['item_type'] )
-			->setEnglishWikipediaTitle( $row['wp_title_en'] )
-			->setEnglishLabel( $row['item_label_en'] );
+	private function newItemInfoFromResultRow(array $row) {
+		return (new ItemInfo())
+			->setNumericItemId($row['item_id'])
+			->setPageTitle($row['page_title'])
+			->setRevisionId($row['revision_id'])
+			->setRevisionTime($row['revision_time'])
+			->setItemType($row['item_type'])
+			->setEnglishWikipediaTitle($row['wp_title_en'])
+			->setEnglishLabel($row['item_label_en']);
 	}
 
 	private function selectItemInfoSets() {
@@ -149,7 +149,7 @@ 
 			'item_type',
 			'item_label_en',
 			'wp_title_en'
-		)->from( $this->tableName );
+		)->from($this->tableName);
 	}
 
 	/**
@@ -160,31 +160,31 @@ 
 	 * @return ItemInfo[]
 	 * @throws EntityStoreException
 	 */
-	public function getItemInfo( $limit, $offset, $itemType = null ) {
+	public function getItemInfo($limit, $offset, $itemType = null) {
 		$query = $this->selectItemInfoSets()
-			->orderBy( 'item_id', 'asc' )
-			->setMaxResults( $limit )
-			->setFirstResult( $offset );
+			->orderBy('item_id', 'asc')
+			->setMaxResults($limit)
+			->setFirstResult($offset);
 
-		if ( is_int( $itemType ) ) {
-			$query->where( 'item_type = ?' )->setParameter( 0, $itemType );
+		if (is_int($itemType)) {
+			$query->where('item_type = ?')->setParameter(0, $itemType);
 		}
 
 		try {
 			$rows = $query->execute();
 		}
-		catch ( DBALException $ex ) {
-			throw new EntityStoreException( $ex->getMessage(), $ex );
+		catch (DBALException $ex) {
+			throw new EntityStoreException($ex->getMessage(), $ex);
 		}
 
-		return $this->newItemInfoArrayFromResult( $rows );
+		return $this->newItemInfoArrayFromResult($rows);
 	}
 
-	private function newItemInfoArrayFromResult( \Traversable $rows ) {
+	private function newItemInfoArrayFromResult(\Traversable $rows) {
 		$infoList = [];
 
-		foreach ( $rows as $resultRow ) {
-			$infoList[] = $this->newItemInfoFromResultRow( $resultRow );
+		foreach ($rows as $resultRow) {
+			$infoList[] = $this->newItemInfoFromResultRow($resultRow);
 		}
 
 		return $infoList;
@@ -197,28 +197,28 @@ 
 	 * @return int[]
 	 * @throws EntityStoreException
 	 */
-	public function getItemTypes( $limit = 100, $offset = 0 ) {
+	public function getItemTypes($limit = 100, $offset = 0) {
 		try {
 			$rows = $this->connection->createQueryBuilder()
-				->select( 'DISTINCT item_type' )
-				->from( $this->tableName )
-				->where( 'item_type IS NOT NULL' )
-				->orderBy( 'item_type', 'ASC' )
-				->setMaxResults( $limit )
-				->setFirstResult( $offset )
+				->select('DISTINCT item_type')
+				->from($this->tableName)
+				->where('item_type IS NOT NULL')
+				->orderBy('item_type', 'ASC')
+				->setMaxResults($limit)
+				->setFirstResult($offset)
 				->execute();
 		}
-		catch ( DBALException $ex ) {
-			throw new EntityStoreException( $ex->getMessage(), $ex );
+		catch (DBALException $ex) {
+			throw new EntityStoreException($ex->getMessage(), $ex);
 		}
 
-		return $this->getTypeArrayFromRows( $rows );
+		return $this->getTypeArrayFromRows($rows);
 	}
 
-	private function getTypeArrayFromRows( \Traversable $rows ) {
+	private function getTypeArrayFromRows(\Traversable $rows) {
 		$types = [];
 
-		foreach ( $rows as $row ) {
+		foreach ($rows as $row) {
 			$types[] = (int)$row['item_type'];
 		}
 
@@ -231,22 +231,22 @@ 
 	 * @return ItemId|null
 	 * @throws EntityStoreException
 	 */
-	public function getIdForEnWikiPage( $pageName ) {
+	public function getIdForEnWikiPage($pageName) {
 		try {
 			$rows = $this->connection->createQueryBuilder()
-				->select( 'item_id' )
-				->from( $this->tableName )
-				->where( 'wp_title_en = ?' )
-				->setParameter( 0, $pageName )
-				->setMaxResults( 1 )
+				->select('item_id')
+				->from($this->tableName)
+				->where('wp_title_en = ?')
+				->setParameter(0, $pageName)
+				->setMaxResults(1)
 				->execute();
 		}
-		catch ( DBALException $ex ) {
-			throw new EntityStoreException( $ex->getMessage(), $ex );
+		catch (DBALException $ex) {
+			throw new EntityStoreException($ex->getMessage(), $ex);
 		}
 
-		foreach ( $rows as $row ) {
-			return ItemId::newFromNumber( $row['item_id'] );
+		foreach ($rows as $row) {
+			return ItemId::newFromNumber($row['item_id']);
 		}
 
 		return null;

Braces +6 added lines, -12 removed lines

@@ -52,8 +52,7 @@ 
 					'item_json' => $itemRow->getItemJson(),
 				]
 			);
-		}
-		catch ( DBALException $ex ) {
+		} catch ( DBALException $ex ) {
 			throw new EntityStoreException( $ex->getMessage(), $ex );
 		}
 	}
@@ -71,8 +70,7 @@ 
 					'item_id' => $itemId->getNumericId()
 				]
 			);
-		}
-		catch ( DBALException $ex ) {
+		} catch ( DBALException $ex ) {
 			throw new EntityStoreException( $ex->getMessage(), $ex );
 		}
 	}
@@ -88,8 +86,7 @@ 
 				->where( 'item_id = ?' )
 				->setParameter( 0, (int)$numericItemId )
 				->execute();
-		}
-		catch ( DBALException $ex ) {
+		} catch ( DBALException $ex ) {
 			throw new EntityStoreException( $ex->getMessage(), $ex );
 		}
 
@@ -172,8 +169,7 @@ 
 
 		try {
 			$rows = $query->execute();
-		}
-		catch ( DBALException $ex ) {
+		} catch ( DBALException $ex ) {
 			throw new EntityStoreException( $ex->getMessage(), $ex );
 		}
 
@@ -207,8 +203,7 @@ 
 				->setMaxResults( $limit )
 				->setFirstResult( $offset )
 				->execute();
-		}
-		catch ( DBALException $ex ) {
+		} catch ( DBALException $ex ) {
 			throw new EntityStoreException( $ex->getMessage(), $ex );
 		}
 
@@ -240,8 +235,7 @@ 
 				->setParameter( 0, $pageName )
 				->setMaxResults( 1 )
 				->execute();
-		}
-		catch ( DBALException $ex ) {
+		} catch ( DBALException $ex ) {
 			throw new EntityStoreException( $ex->getMessage(), $ex );
 		}
 

src/ItemTypeExtractor.php

Spacing +1 added lines, -1 removed lines

@@ -17,6 +17,6 @@
 	 *
 	 * @return int|null
 	 */
-	public function getTypeOfItem( Item $item );
+	public function getTypeOfItem(Item $item);
 
 }
\ No newline at end of file

src/EntityStoreInstaller.php

Spacing +34 added lines, -34 removed lines

@@ -16,7 +16,7 @@ 
 	private $schemaManager;
 	private $config;
 
-	public function __construct( AbstractSchemaManager $schemaManager, EntityStoreConfig $config ) {
+	public function __construct(AbstractSchemaManager $schemaManager, EntityStoreConfig $config) {
 		$this->schemaManager = $schemaManager;
 		$this->config = $config;
 	}
@@ -25,54 +25,54 @@ 
 	 * @throws DBALException
 	 */
 	public function install() {
-		$this->schemaManager->createTable( $this->newItemTable() );
-		$this->schemaManager->createTable( $this->newPropertyTable() );
+		$this->schemaManager->createTable($this->newItemTable());
+		$this->schemaManager->createTable($this->newPropertyTable());
 	}
 
 	private function newItemTable() {
-		$table = new Table( $this->config->getItemTableName() );
-
-		$table->addColumn( 'item_id', Type::BIGINT );
-		$table->addColumn( 'item_type', Type::BIGINT, [ 'notnull' => false ] );
-		$table->addColumn( 'page_title', Type::STRING, [ 'length' => 255 ] );
-		$table->addColumn( 'revision_id', Type::BIGINT );
-		$table->addColumn( 'revision_time', Type::STRING, [ 'length' => 25 ] );
-		$table->addColumn( 'item_json', Type::BLOB );
-		$table->addColumn( 'item_label_en', Type::STRING, [ 'length' => 255, 'notnull' => false ] );
-		$table->addColumn( 'wp_title_en', Type::STRING, [ 'length' => 255, 'notnull' => false ] );
-
-		$table->addIndex( [ 'item_id' ] );
-		$table->addIndex( [ 'item_type' ] );
-		$table->addIndex( [ 'page_title' ] );
-		$table->addIndex( [ 'revision_id' ] );
-		$table->addIndex( [ 'revision_time' ] );
-		$table->addIndex( [ 'wp_title_en' ] );
+		$table = new Table($this->config->getItemTableName());
+
+		$table->addColumn('item_id', Type::BIGINT);
+		$table->addColumn('item_type', Type::BIGINT, ['notnull' => false]);
+		$table->addColumn('page_title', Type::STRING, ['length' => 255]);
+		$table->addColumn('revision_id', Type::BIGINT);
+		$table->addColumn('revision_time', Type::STRING, ['length' => 25]);
+		$table->addColumn('item_json', Type::BLOB);
+		$table->addColumn('item_label_en', Type::STRING, ['length' => 255, 'notnull' => false]);
+		$table->addColumn('wp_title_en', Type::STRING, ['length' => 255, 'notnull' => false]);
+
+		$table->addIndex(['item_id']);
+		$table->addIndex(['item_type']);
+		$table->addIndex(['page_title']);
+		$table->addIndex(['revision_id']);
+		$table->addIndex(['revision_time']);
+		$table->addIndex(['wp_title_en']);
 
 		return $table;
 	}
 
 	private function newPropertyTable() {
-		$table = new Table( $this->config->getPropertyTableName() );
+		$table = new Table($this->config->getPropertyTableName());
 
-		$table->addColumn( 'property_id', Type::BIGINT );
-		$table->addColumn( 'property_json', Type::BLOB );
-		$table->addColumn( 'page_title', Type::STRING, [ 'length' => 255 ] );
-		$table->addColumn( 'revision_id', Type::BIGINT );
-		$table->addColumn( 'revision_time', Type::STRING, [ 'length' => 25 ] );
-		$table->addColumn( 'property_type', Type::STRING, [ 'length' => 30 ] );
+		$table->addColumn('property_id', Type::BIGINT);
+		$table->addColumn('property_json', Type::BLOB);
+		$table->addColumn('page_title', Type::STRING, ['length' => 255]);
+		$table->addColumn('revision_id', Type::BIGINT);
+		$table->addColumn('revision_time', Type::STRING, ['length' => 25]);
+		$table->addColumn('property_type', Type::STRING, ['length' => 30]);
 
-		$table->addIndex( [ 'property_id' ] );
-		$table->addIndex( [ 'page_title' ] );
-		$table->addIndex( [ 'revision_id' ] );
-		$table->addIndex( [ 'revision_time' ] );
-		$table->addIndex( [ 'property_type' ] );
+		$table->addIndex(['property_id']);
+		$table->addIndex(['page_title']);
+		$table->addIndex(['revision_id']);
+		$table->addIndex(['revision_time']);
+		$table->addIndex(['property_type']);
 
 		return $table;
 	}
 
 	public function uninstall() {
-		$this->schemaManager->dropTable( $this->config->getItemTableName() );
-		$this->schemaManager->dropTable( $this->config->getPropertyTableName() );
+		$this->schemaManager->dropTable($this->config->getItemTableName());
+		$this->schemaManager->dropTable($this->config->getPropertyTableName());
 	}
 
 }
\ No newline at end of file

src/Data/ItemRowInfo.php

Spacing +7 added lines, -7 removed lines

@@ -23,7 +23,7 @@ 
 	 * @param string|null $enLabel
 	 * @return $this
 	 */
-	public function setEnglishLabel( $enLabel ) {
+	public function setEnglishLabel($enLabel) {
 		$this->enLabel = $enLabel;
 		return $this;
 	}
@@ -32,7 +32,7 @@ 
 	 * @param int|string $itemId
 	 * @return $this
 	 */
-	public function setNumericItemId( $itemId ) {
+	public function setNumericItemId($itemId) {
 		$this->itemId = (int)$itemId;
 		return $this;
 	}
@@ -41,7 +41,7 @@ 
 	 * @param int|null $itemType
 	 * @return $this
 	 */
-	public function setItemType( $itemType ) {
+	public function setItemType($itemType) {
 		$this->itemType = $itemType === null ? null : (int)$itemType;
 		return $this;
 	}
@@ -50,7 +50,7 @@ 
 	 * @param string $pageTitle
 	 * @return $this
 	 */
-	public function setPageTitle( $pageTitle ) {
+	public function setPageTitle($pageTitle) {
 		$this->pageTitle = $pageTitle;
 		return $this;
 	}
@@ -59,7 +59,7 @@ 
 	 * @param int|string $revisionId
 	 * @return $this
 	 */
-	public function setRevisionId( $revisionId ) {
+	public function setRevisionId($revisionId) {
 		$this->revisionId = (int)$revisionId;
 		return $this;
 	}
@@ -68,7 +68,7 @@ 
 	 * @param string $revisionTime
 	 * @return $this
 	 */
-	public function setRevisionTime( $revisionTime ) {
+	public function setRevisionTime($revisionTime) {
 		$this->revisionTime = $revisionTime;
 		return $this;
 	}
@@ -77,7 +77,7 @@ 
 	 * @param string|null $enWikiTitle
 	 * @return $this
 	 */
-	public function setEnglishWikipediaTitle( $enWikiTitle ) {
+	public function setEnglishWikipediaTitle($enWikiTitle) {
 		$this->enWikiTitle = $enWikiTitle;
 		return $this;
 	}

src/Data/PropertyRow.php

Spacing +1 added lines, -1 removed lines

@@ -17,7 +17,7 @@
 	 * @param string $propertyJson
 	 * @param PropertyInfo $info
 	 */
-	public function __construct( $propertyJson, PropertyInfo $info ) {
+	public function __construct($propertyJson, PropertyInfo $info) {
 		$this->propertyJson = $propertyJson;
 		$this->propertyInfo = $info;
 	}

src/Data/PropertyInfo.php

Spacing +1 added lines, -1 removed lines

@@ -23,7 +23,7 @@
 	 * @param string $revisionTime
 	 * @param string $propertyType
 	 */
-	public function __construct( $numericPropertyId, $pageTitle, $revisionId, $revisionTime, $propertyType ) {
+	public function __construct($numericPropertyId, $pageTitle, $revisionId, $revisionTime, $propertyType) {
 		$this->propertyId = (int)$numericPropertyId;
 		$this->pageTitle = $pageTitle;
 		$this->revisionId = (int)$revisionId;