SemanticMediaWiki / SemanticCite

src/DataValues/CitationReferenceValue.php

Spacing +29 added lines, -29 removed lines

@@ -41,8 +41,8 @@ 
 	/**
 	 * @param string $typeid
 	 */
-	public function __construct( $typeid = '' ) {
-		parent::__construct( '_sci_ref' );
+	public function __construct($typeid = '') {
+		parent::__construct('_sci_ref');
 
 		// Currently there is no good way to inject the setting
 		$this->captionFormat = $GLOBALS['scigCitationReferenceCaptionFormat'];
@@ -53,42 +53,42 @@ 
 	 *
 	 * @param  integer $captionFormat
 	 */
-	public function setCaptionFormat( $captionFormat ) {
+	public function setCaptionFormat($captionFormat) {
 		$this->captionFormat = $captionFormat;
 	}
 
 	/**
 	 * @see StringValue::parseUserValue
 	 */
-	protected function parseUserValue( $value ) {
+	protected function parseUserValue($value) {
 
-		if ( method_exists( $this, 'getCallable' ) ) {
-			$citationReferencePositionJournal = $this->getCallable( 'sci.citationreferencepositionjournal' );
+		if (method_exists($this, 'getCallable')) {
+			$citationReferencePositionJournal = $this->getCallable('sci.citationreferencepositionjournal');
 			$this->citationReferencePositionJournal = $citationReferencePositionJournal();
 		} else {
-			$this->citationReferencePositionJournal = $this->getExtraneousFunctionFor( '\SCI\CitationReferencePositionJournal' );
+			$this->citationReferencePositionJournal = $this->getExtraneousFunctionFor('\SCI\CitationReferencePositionJournal');
 		}
 
-		$value = trim( $value );
+		$value = trim($value);
 
-		if ( $value === '' ) {
-			$this->addError( wfMessage( 'sci-datavalue-empty-reference' )->inContentLanguage()->text() );
-			$this->m_dataitem = new DIBlob( 'ERROR' );
+		if ($value === '') {
+			$this->addError(wfMessage('sci-datavalue-empty-reference')->inContentLanguage()->text());
+			$this->m_dataitem = new DIBlob('ERROR');
 			return;
 		}
 
-		if ( $this->m_contextPage === null ) {
-			parent::parseUserValue( $value );
+		if ($this->m_contextPage === null) {
+			parent::parseUserValue($value);
 			return;
 		}
 
 		$this->reference = $value;
 
-		if ( $this->m_caption && $this->captionFormat === SCI_CITEREF_NUM ) {
+		if ($this->m_caption && $this->captionFormat === SCI_CITEREF_NUM) {
 			$this->usesShortFormCaption = true;
 		}
 
-		if ( !$this->m_caption ) {
+		if (!$this->m_caption) {
 			$this->m_caption = $this->reference;
 		}
 
@@ -99,20 +99,20 @@ 
 			$this->reference
 		);
 
-		parent::parseUserValue( $this->reference );
+		parent::parseUserValue($this->reference);
 	}
 
 	/**
 	 * @see StringValue::parseUserValue
 	 */
-	public function getShortWikiText( $linked = null ) {
+	public function getShortWikiText($linked = null) {
 
-		if ( $this->citationReferencePositionJournal === null ) {
-			if ( method_exists( $this, 'getCallable' ) ) {
-				$citationReferencePositionJournal = $this->getCallable( 'sci.citationreferencepositionjournal' );
+		if ($this->citationReferencePositionJournal === null) {
+			if (method_exists($this, 'getCallable')) {
+				$citationReferencePositionJournal = $this->getCallable('sci.citationreferencepositionjournal');
 				$this->citationReferencePositionJournal = $citationReferencePositionJournal();
 			} else {
-				$this->citationReferencePositionJournal = $this->getExtraneousFunctionFor( '\SCI\CitationReferencePositionJournal' );
+				$this->citationReferencePositionJournal = $this->getExtraneousFunctionFor('\SCI\CitationReferencePositionJournal');
 			}
 		}
 
@@ -123,14 +123,14 @@ 
 			$this->reference
 		);
 
-		if ( $referencePosition === null || $this->m_caption === false ) {
+		if ($referencePosition === null || $this->m_caption === false) {
 			return '';
 		}
 
-		$referenceHash = md5( $this->reference );
+		$referenceHash = md5($this->reference);
 
-		if ( $this->captionFormat === SCI_CITEREF_NUM ) {
-			list( $major, $minor ) = explode( '-', $referencePosition );
+		if ($this->captionFormat === SCI_CITEREF_NUM) {
+			list($major, $minor) = explode('-', $referencePosition);
 			$caption = $major;
 			$captionClass = 'number';
 
@@ -142,10 +142,10 @@ 
 			$shortFormCaption = ''; // Never has a short form
 		}
 
-		if ( $shortFormCaption !== '' ) {
+		if ($shortFormCaption !== '') {
 			$shortFormCaption = Html::rawElement(
 				'span',
-				[ 'class' => 'scite-citeref-shortcaption' ],
+				['class' => 'scite-citeref-shortcaption'],
 				$shortFormCaption
 			);
 		}
@@ -154,11 +154,11 @@ 
 		$html = Html::rawElement(
 			'span',
 			[
-				'id'    => 'scite-ref-'. $referenceHash . '-' . $referencePosition,
+				'id'    => 'scite-ref-' . $referenceHash . '-' . $referencePosition,
 				'class' => 'scite-citeref-' . $captionClass,
 				'data-reference' => $this->reference
 			],
-			'[[' .'#scite-' . $referenceHash . '|' . $caption . ']]'
+			'[[' . '#scite-' . $referenceHash . '|' . $caption . ']]'
 		) . $shortFormCaption;
 
 		return $html;

src/DataValues/ResourceIdentifierStringValue.php

Spacing +26 added lines, -26 removed lines

@@ -22,46 +22,46 @@ 
 	/**
 	 * @param string $typeid
 	 */
-	public function __construct( $typeid = '' ) {
-		parent::__construct( $typeid );
-		$this->resourceIdentifierStringValueParser = new ResourceIdentifierStringValueParser( $typeid );
+	public function __construct($typeid = '') {
+		parent::__construct($typeid);
+		$this->resourceIdentifierStringValueParser = new ResourceIdentifierStringValueParser($typeid);
 	}
 
 	/**
 	 * @see StringValue::parseUserValue
 	 */
-	protected function parseUserValue( $value ) {
+	protected function parseUserValue($value) {
 
 		$inputValue = $value;
 	//	$this->m_caption = $value;
 
-		if ( !$this->resourceIdentifierStringValueParser->parse( $value ) ) {
-			$this->addError( wfMessage( 'sci-datavalue-invalid-id-value', $inputValue, $this->resourceIdentifierStringValueParser->getCanonicalName() )->inContentLanguage()->escaped() );
-			$this->m_dataitem = new DIBlob( 'ERROR' );
+		if (!$this->resourceIdentifierStringValueParser->parse($value)) {
+			$this->addError(wfMessage('sci-datavalue-invalid-id-value', $inputValue, $this->resourceIdentifierStringValueParser->getCanonicalName())->inContentLanguage()->escaped());
+			$this->m_dataitem = new DIBlob('ERROR');
 			return;
 		}
 
-		parent::parseUserValue( $value );
+		parent::parseUserValue($value);
 	}
 
 	/**
 	 * @see StringValue::getShortWikiText
 	 */
-	public function getShortWikiText( $linker = null ) {
+	public function getShortWikiText($linker = null) {
 
-		if ( !$this->isValid() ) {
+		if (!$this->isValid()) {
 			return '';
 		}
 
-		if ( !$this->m_caption ) {
+		if (!$this->m_caption) {
 			$this->m_caption = $this->m_dataitem->getString();
 		}
 
-		if ( $preferredCaption = $this->getPreferredCaption() ) {
+		if ($preferredCaption = $this->getPreferredCaption()) {
 			return $preferredCaption;
 		}
 
-		if ( $linker === null ) {
+		if ($linker === null) {
 			return $this->m_caption;
 		}
 
@@ -70,35 +70,35 @@ 
 			[
 				'class' => 'plainlinks'
 			],
-			'[' . $this->getTargetLink( urlencode( $this->m_caption ) ) . ' ' . $this->m_caption .']'
+			'[' . $this->getTargetLink(urlencode($this->m_caption)) . ' ' . $this->m_caption . ']'
 		);
 	}
 
 	/**
 	 * @see StringValue::getShortHTMLText
 	 */
-	public function getShortHTMLText( $linker = null ) {
+	public function getShortHTMLText($linker = null) {
 
-		if ( !$this->isValid() ) {
+		if (!$this->isValid()) {
 			return '';
 		}
 
-		if ( !$this->m_caption ) {
+		if (!$this->m_caption) {
 			$this->m_caption = $this->m_dataitem->getString();
 		}
 
-		if ( $preferredCaption = $this->getPreferredCaption() ) {
+		if ($preferredCaption = $this->getPreferredCaption()) {
 			return $preferredCaption;
 		}
 
-		if ( $linker === null ) {
+		if ($linker === null) {
 			return $this->m_caption;
 		}
 
 		return Html::rawElement(
 			'a',
 			[
-				'href'   => $this->getTargetLink( $this->m_caption ),
+				'href'   => $this->getTargetLink($this->m_caption),
 				'target' => '_blank'
 			],
 			$this->m_caption
@@ -108,15 +108,15 @@ 
 	/**
 	 * @see StringValue::getLongWikiText
 	 */
-	public function getLongWikiText( $linked = null ) {
-		return $this->getShortWikiText( $linked );
+	public function getLongWikiText($linked = null) {
+		return $this->getShortWikiText($linked);
 	}
 
 	/**
 	 * @see StringValue::getLongHTMLText
 	 */
-	public function getLongHTMLText( $linker = null ) {
-		return $this->getShortHTMLText( $linker );
+	public function getLongHTMLText($linker = null) {
+		return $this->getShortHTMLText($linker);
 	}
 
 	/**
@@ -124,14 +124,14 @@ 
 	 */
 	public function getPreferredCaption() {
 
-		if ( $this->m_outformat == '-' ) {
+		if ($this->m_outformat == '-') {
 			return $this->m_caption;
 		}
 
 		return false;
 	}
 
-	private function getTargetLink( $target ) {
+	private function getTargetLink($target) {
 		return $this->resourceIdentifierStringValueParser->getResourceTargetUri() . $target;
 	}
 

src/DataValues/ResourceIdentifierFactory.php

Spacing +10 added lines, -10 removed lines

@@ -23,36 +23,36 @@
 	 * @return ResourceIdentifierStringValue
 	 * @throws RuntimeException
 	 */
-	public function newResourceIdentifierStringValueForType( $type ) {
+	public function newResourceIdentifierStringValueForType($type) {
 
 		$propertyRegistry = PropertyRegistry::getInstance();
 		$property = null;
 
-		switch ( strtoupper( $type ) ) {
+		switch (strtoupper($type)) {
 			case 'OCLC':
 			case 'VIAF':
 			case 'DOI':
 			case 'OLID':
-				$property = $propertyRegistry->findPropertyIdByLabel( $type );
+				$property = $propertyRegistry->findPropertyIdByLabel($type);
 				break;
 			case 'PUBMED':
 			case 'PMID':
-				$property = $propertyRegistry->findPropertyIdByLabel( 'PMID' );
+				$property = $propertyRegistry->findPropertyIdByLabel('PMID');
 				break;
 			case 'PMC':
 			case 'PMCID':
-				$property = $propertyRegistry->findPropertyIdByLabel( 'PMCID' );
+				$property = $propertyRegistry->findPropertyIdByLabel('PMCID');
 				break;
 		}
 
-		$typeId = $propertyRegistry->getPropertyTypeId( $property );
+		$typeId = $propertyRegistry->getPropertyTypeId($property);
 
-		if ( $typeId === null || $typeId === '' ) {
-			throw new RuntimeException( "{$type} is an unmatched type for ResourceIdentifierStringValue" );
+		if ($typeId === null || $typeId === '') {
+			throw new RuntimeException("{$type} is an unmatched type for ResourceIdentifierStringValue");
 		}
 
-		$resourceIdentifierStringValue = DataValueFactory::getInstance()->newDataValueByType( $typeId );
-		$resourceIdentifierStringValue->setProperty( new DIProperty( $property ) );
+		$resourceIdentifierStringValue = DataValueFactory::getInstance()->newDataValueByType($typeId);
+		$resourceIdentifierStringValue->setProperty(new DIProperty($property));
 
 		return $resourceIdentifierStringValue;
 	}

src/DataValues/ResourceIdentifierStringValueParser.php

Spacing +18 added lines, -18 removed lines

@@ -18,7 +18,7 @@ 
 	/**
 	 * @param string $typeid
 	 */
-	public function __construct( $typeid ) {
+	public function __construct($typeid) {
 		$this->typeid = $typeid;
 	}
 
@@ -27,16 +27,16 @@ 
 	 *
 	 * @return true
 	 */
-	public function parse( &$value ) {
+	public function parse(&$value) {
 
 		// Remove a possible url-prefix
 
-		if ( $this->typeid ===  '_sci_doi' ) {
+		if ($this->typeid === '_sci_doi') {
 			// Remove anything before 10 as in http://dx.doi.org/10.1000/123
-			$value = substr( $value, strcspn( $value, '10' ) );
-		} elseif ( strrpos( $value, '://' ) !== false ) {
+			$value = substr($value, strcspn($value, '10'));
+		} elseif (strrpos($value, '://') !== false) {
 			// Remove anything before the last / as in http://foo/bar123
-			$value = substr( $value, strrpos( $value, '/' ) + 1 );
+			$value = substr($value, strrpos($value, '/') + 1);
 		}
 
 		// Remove other possible prefixes
@@ -45,13 +45,13 @@ 
 		// All DOI names are converted to upper case upon registration
 		// but since it system is case insensitive, we store it as lower
 		// case as it "looks" better
-		if ( $this->typeid ===  '_sci_doi' ) {
-			$value = strtolower( $value );
+		if ($this->typeid === '_sci_doi') {
+			$value = strtolower($value);
 		} else {
-			$value = str_replace( [ 'VIAF', 'OCLC', 'PMID' ], '', strtoupper( $value ) );
+			$value = str_replace(['VIAF', 'OCLC', 'PMID'], '', strtoupper($value));
 		}
 
-		return $this->canMatchValueToPattern( $value );
+		return $this->canMatchValueToPattern($value);
 	}
 
 	/**
@@ -59,7 +59,7 @@ 
 	 */
 	public function getCanonicalName() {
 
-		switch ( $this->typeid ) {
+		switch ($this->typeid) {
 			case '_sci_viaf':
 				return 'VIAF';
 			case '_sci_oclc':
@@ -82,7 +82,7 @@ 
 	 */
 	public function getResourceTargetUri() {
 
-		switch ( $this->typeid ) {
+		switch ($this->typeid) {
 			case '_sci_viaf':
 	 			// http://www.oclc.org/research/activities/viaf.html
 	 			// http://id.loc.gov/vocabulary/identifiers/viaf.html
@@ -103,20 +103,20 @@ 
 		}
 	}
 
-	private function canMatchValueToPattern( &$value ) {
+	private function canMatchValueToPattern(&$value) {
 
-		switch ( $this->typeid ) {
+		switch ($this->typeid) {
 			case '_sci_viaf':
 			case '_sci_oclc':
 			case '_sci_pmid':
-	 			return preg_match( "/^[0-9]*$/", $value );
+	 			return preg_match("/^[0-9]*$/", $value);
 			case '_sci_pmcid':
-	 			return preg_match( "/PMC[\d]+/", $value );
+	 			return preg_match("/PMC[\d]+/", $value);
 			case '_sci_olid':
-	 			return preg_match( "/OL[A-Z0-9]+/", $value );
+	 			return preg_match("/OL[A-Z0-9]+/", $value);
 			case '_sci_doi':
 				// http://stackoverflow.com/questions/27910/finding-a-doi-in-a-document-or-page#
-	 			return preg_match( "/\b(10[.][0-9]{4,}(?:[.][0-9]+)*\/(?:(?![\"&\'])\S)+)\b/", $value );
+	 			return preg_match("/\b(10[.][0-9]{4,}(?:[.][0-9]+)*\/(?:(?![\"&\'])\S)+)\b/", $value);
 		}
 
 		return false;

src/PreTextFormatter.php

Spacing +7 added lines, -7 removed lines

@@ -18,8 +18,8 @@ 
 	 *
 	 * @return string
 	 */
-	public function getFormattedSciteFuncFrom( array $parameters ) {
-		return "<pre>{{#scite:\n" . implode( "\n", $this->format( $parameters ) ) ."\n}}</pre>";
+	public function getFormattedSciteFuncFrom(array $parameters) {
+		return "<pre>{{#scite:\n" . implode("\n", $this->format($parameters)) . "\n}}</pre>";
 	}
 
 	/**
@@ -27,18 +27,18 @@ 
 	 *
 	 * @return array
 	 */
-	public function format( array $parameters ) {
+	public function format(array $parameters) {
 
 		$formatted = [];
 
-		foreach ( $parameters as $key => $value ) {
+		foreach ($parameters as $key => $value) {
 
-			if ( $value === '' || $value[0] === '@' ) {
+			if ($value === '' || $value[0] === '@') {
 				continue;
 			}
 
-			if ( $value[0] === '+' ) {
-				$formatted[$key-1] = $formatted[$key-1] . '|' . $value;
+			if ($value[0] === '+') {
+				$formatted[$key - 1] = $formatted[$key - 1] . '|' . $value;
 				continue;
 			}
 

src/CitationReferencePositionJournal.php

Spacing +26 added lines, -26 removed lines

@@ -36,7 +36,7 @@ 
 	 * @param Cache $cache
 	 * @param CacheKeyProvider $cacheKeyProvider
 	 */
-	public function __construct( Cache $cache, CacheKeyProvider $cacheKeyProvider ) {
+	public function __construct(Cache $cache, CacheKeyProvider $cacheKeyProvider) {
 		$this->cache = $cache;
 		$this->cacheKeyProvider = $cacheKeyProvider;
 	}
@@ -48,8 +48,8 @@ 
 	 *
 	 * @return boolean
 	 */
-	public function hasCitationReference( SemanticData $semanticData ) {
-		return $semanticData->getPropertyValues( new DIProperty( PropertyRegistry::SCI_CITE_REFERENCE ) ) !== [];
+	public function hasCitationReference(SemanticData $semanticData) {
+		return $semanticData->getPropertyValues(new DIProperty(PropertyRegistry::SCI_CITE_REFERENCE)) !== [];
 	}
 
 	/**
@@ -59,8 +59,8 @@ 
 	 *
 	 * @return array|null
 	 */
-	public function getJournalBySubject( DIWikiPage $subject ) {
-		return $this->hasJournalForHash( $subject->getHash() ) ? self::$citationReferenceJournal[$subject->getHash()] : null;
+	public function getJournalBySubject(DIWikiPage $subject) {
+		return $this->hasJournalForHash($subject->getHash()) ? self::$citationReferenceJournal[$subject->getHash()] : null;
 	}
 
 	/**
@@ -74,9 +74,9 @@ 
 	 *
 	 * @return array|null
 	 */
-	public function buildJournalForUnboundReferenceList( array $referenceList ) {
+	public function buildJournalForUnboundReferenceList(array $referenceList) {
 
-		if ( $referenceList === [] ) {
+		if ($referenceList === []) {
 			return null;
 		}
 
@@ -86,9 +86,9 @@ 
 			'reference-pos'  => []
 		];
 
-		$journal['total'] = count( $referenceList );
+		$journal['total'] = count($referenceList);
 
-		foreach ( $referenceList as $reference ) {
+		foreach ($referenceList as $reference) {
 			$journal['reference-pos'][$reference] = [];
 			$journal['reference-list'][$reference] = $reference;
 		}
@@ -104,22 +104,22 @@ 
 	 *
 	 * @return string|null
 	 */
-	public function findLastReferencePositionEntryFor( DIWikiPage $subject = null, $reference ) {
+	public function findLastReferencePositionEntryFor(DIWikiPage $subject = null, $reference) {
 
-		if ( $subject === null ) {
+		if ($subject === null) {
 			return;
 		}
 
-		$referenceHash = md5( $reference );
+		$referenceHash = md5($reference);
 		$hash = $subject->getHash();
 
 		// Tracks the current parser run invoked by the InTextAnnotationParser
 		$uniqid = $subject->getContextReference();
 
-		if ( $this->hasJournalForHash( $hash ) &&
+		if ($this->hasJournalForHash($hash) &&
 			self::$citationReferenceJournal[$hash]['uniqid'] === $uniqid &&
-			isset( self::$citationReferenceJournal[$hash]['reference-pos'][$referenceHash] ) ) {
-			return end( self::$citationReferenceJournal[$hash]['reference-pos'][$referenceHash] );
+			isset(self::$citationReferenceJournal[$hash]['reference-pos'][$referenceHash])) {
+			return end(self::$citationReferenceJournal[$hash]['reference-pos'][$referenceHash]);
 		}
 
 		return null;
@@ -131,19 +131,19 @@ 
 	 * @param DIWikiPage|null $subject
 	 * @param string $reference
 	 */
-	public function addJournalEntryFor( DIWikiPage $subject = null, $reference ) {
+	public function addJournalEntryFor(DIWikiPage $subject = null, $reference) {
 
-		if ( $subject === null ) {
+		if ($subject === null) {
 			return;
 		}
 
-		$referenceHash = md5( $reference );
+		$referenceHash = md5($reference);
 		$hash = $subject->getHash();
 
 		// Tracks the current parser run invoked by the InTextAnnotationParser
 		$uniqid = $subject->getContextReference();
 
-		if ( !$this->hasJournalForHash( $hash ) || self::$citationReferenceJournal[$hash]['uniqid'] !== $uniqid ) {
+		if (!$this->hasJournalForHash($hash) || self::$citationReferenceJournal[$hash]['uniqid'] !== $uniqid) {
 			$journal = [
 				'uniqid'     => $uniqid,
 				'total'      => 0,
@@ -160,7 +160,7 @@ 
 
 		// New reference, increase the total of existing citations for this
 		// subject and set the position level to 'a'
-		if ( !isset( $journal['reference-pos'][$referenceHash] ) || $journal['reference-pos'][$referenceHash] === [] ) {
+		if (!isset($journal['reference-pos'][$referenceHash]) || $journal['reference-pos'][$referenceHash] === []) {
 			$journal['total']++;
 			$journal['reference-pos'][$referenceHash][] = $journal['total'] . '-' . 'a';
 		} else {
@@ -173,7 +173,7 @@ 
 			// The exact position is specified by something like
 			// like 57d0dc056f9e5d5c6cfc77fc27091f60-1-a and to be used within
 			// html as link anchor
-			list( $major, $minor ) = explode( '-', end( $journal['reference-pos'][$referenceHash] ) );
+			list($major, $minor) = explode('-', end($journal['reference-pos'][$referenceHash]));
 			$journal['reference-pos'][$referenceHash][] = $major . '-' . ++$minor;
 		}
 
@@ -182,20 +182,20 @@ 
 	 	// Safeguard against a repeated call to the hashlist for when the static
 	 	// cache is empty
 		$this->cache->save(
-			$this->cacheKeyProvider->getCacheKeyForCitationReference( $hash ),
+			$this->cacheKeyProvider->getCacheKeyForCitationReference($hash),
 			self::$citationReferenceJournal
 		);
 	}
 
-	private function hasJournalForHash( $hash ) {
+	private function hasJournalForHash($hash) {
 
-		if ( self::$citationReferenceJournal === [] ) {
+		if (self::$citationReferenceJournal === []) {
 			self::$citationReferenceJournal = $this->cache->fetch(
-				$this->cacheKeyProvider->getCacheKeyForCitationReference( $hash )
+				$this->cacheKeyProvider->getCacheKeyForCitationReference($hash)
 			);
 		}
 
-		return isset( self::$citationReferenceJournal[$hash] );
+		return isset(self::$citationReferenceJournal[$hash]);
 	}
 
 }

src/Options.php

Spacing +7 added lines, -7 removed lines

@@ -20,7 +20,7 @@ 
 	/**
 	 * @since 1.0
 	 */
-	public function __construct( array $options = [] ) {
+	public function __construct(array $options = []) {
 		$this->options = $options;
 	}
 
@@ -30,7 +30,7 @@ 
 	 * @param string $key
 	 * @param mixed $value
 	 */
-	public function set( $key, $value ) {
+	public function set($key, $value) {
 		$this->options[$key] = $value;
 	}
 
@@ -41,8 +41,8 @@ 
 	 *
 	 * @return boolean
 	 */
-	public function has( $key ) {
-		return isset( $this->options[$key] ) || array_key_exists( $key, $this->options );
+	public function has($key) {
+		return isset($this->options[$key]) || array_key_exists($key, $this->options);
 	}
 
 	/**
@@ -53,13 +53,13 @@ 
 	 * @return string
 	 * @throws InvalidArgumentException
 	 */
-	public function get( $key ) {
+	public function get($key) {
 
-		if ( $this->has( $key ) ) {
+		if ($this->has($key)) {
 			return $this->options[$key];
 		}
 
-		throw new InvalidArgumentException( "{$key} is an unregistered option" );
+		throw new InvalidArgumentException("{$key} is an unregistered option");
 	}
 
 }

src/FilteredMetadata/OLResponseParser.php

Spacing +15 added lines, -15 removed lines

@@ -24,7 +24,7 @@ 
 	 *
 	 * @param OLFilteredHttpResponseParser $olFilteredHttpResponseParser
 	 */
-	public function __construct( OLFilteredHttpResponseParser $olFilteredHttpResponseParser ) {
+	public function __construct(OLFilteredHttpResponseParser $olFilteredHttpResponseParser) {
 		$this->olFilteredHttpResponseParser = $olFilteredHttpResponseParser;
 	}
 
@@ -60,8 +60,8 @@ 
 	 *
 	 * {@inheritDoc}
 	 */
-	public function getRawResponse( $id ) {
-		return $this->olFilteredHttpResponseParser->getRawResponse( $id );
+	public function getRawResponse($id) {
+		return $this->olFilteredHttpResponseParser->getRawResponse($id);
 	}
 
 	/**
@@ -69,39 +69,39 @@ 
 	 *
 	 * {@inheritDoc}
 	 */
-	public function doFilterResponseFor( $olID ) {
+	public function doFilterResponseFor($olID) {
 
-		$this->olFilteredHttpResponseParser->doFilterResponseFor( $olID );
+		$this->olFilteredHttpResponseParser->doFilterResponseFor($olID);
 		$filteredRecord = $this->olFilteredHttpResponseParser->getFilteredRecord();
 
 		// Fetch the OLID has one could search for an ISBN as well
-		if ( $filteredRecord->has( 'olid' ) ) {
-			$olID = $filteredRecord->get( 'olid' );
+		if ($filteredRecord->has('olid')) {
+			$olID = $filteredRecord->get('olid');
 		}
 
-		if ( is_array( $olID ) ) {
-			$olID = end( $olID );
+		if (is_array($olID)) {
+			$olID = end($olID);
 		}
 
 		$filteredRecord->setTitleForPageCreation(
-			'OL:' . str_replace( 'OL', '', $olID )
+			'OL:' . str_replace('OL', '', $olID)
 		);
 
 		$filteredRecord->setSciteTransclusionHead(
 			$olID
 		);
 
-		$filteredRecord->addSearchMatchSet( 'olid', $olID );
+		$filteredRecord->addSearchMatchSet('olid', $olID);
 
-		if ( $filteredRecord->has( 'lccn' ) ) {
+		if ($filteredRecord->has('lccn')) {
 			$filteredRecord->addSearchMatchSet(
 				'lccn',
-				$filteredRecord->get( 'lccn' )
+				$filteredRecord->get('lccn')
 			);
 		}
 
-		$dateTimeUtc = new \DateTime( 'now', new \DateTimeZone( 'UTC' ) );
-		$filteredRecord->set( 'retrieved-on', $dateTimeUtc->format( 'Y-m-d' ) );
+		$dateTimeUtc = new \DateTime('now', new \DateTimeZone('UTC'));
+		$filteredRecord->set('retrieved-on', $dateTimeUtc->format('Y-m-d'));
 	}
 
 }

src/FilteredMetadata/OclcResponseParser.php

Spacing +14 added lines, -14 removed lines

@@ -24,7 +24,7 @@ 
 	 *
 	 * @param OclcFilteredHttpResponseParser $oclcFilteredHttpResponseParser
 	 */
-	public function __construct( OclcFilteredHttpResponseParser $oclcFilteredHttpResponseParser ) {
+	public function __construct(OclcFilteredHttpResponseParser $oclcFilteredHttpResponseParser) {
 		$this->oclcFilteredHttpResponseParser = $oclcFilteredHttpResponseParser;
 	}
 
@@ -60,8 +60,8 @@ 
 	 *
 	 * {@inheritDoc}
 	 */
-	public function getRawResponse( $oclcID ) {
-		return $this->oclcFilteredHttpResponseParser->getRawResponse( $oclcID );
+	public function getRawResponse($oclcID) {
+		return $this->oclcFilteredHttpResponseParser->getRawResponse($oclcID);
 	}
 
 	/**
@@ -69,32 +69,32 @@ 
 	 *
 	 * {@inheritDoc}
 	 */
-	public function doFilterResponseFor( $oclcID ) {
+	public function doFilterResponseFor($oclcID) {
 
 		$resourceIdentifierFactory = new ResourceIdentifierFactory();
 
-		$oclcValue = $resourceIdentifierFactory->newResourceIdentifierStringValueForType( 'oclc' );
-		$oclcValue->setUserValue( $oclcID );
+		$oclcValue = $resourceIdentifierFactory->newResourceIdentifierStringValueForType('oclc');
+		$oclcValue->setUserValue($oclcID);
 
-		if ( !$oclcValue->isValid() ) {
-			return $this->oclcFilteredHttpResponseParser->addMessage( $oclcValue->getErrors() );
+		if (!$oclcValue->isValid()) {
+			return $this->oclcFilteredHttpResponseParser->addMessage($oclcValue->getErrors());
 		}
 
 		$oclcID = $oclcValue->getWikiValue();
 
-		$this->oclcFilteredHttpResponseParser->doFilterResponseFor( $oclcID );
+		$this->oclcFilteredHttpResponseParser->doFilterResponseFor($oclcID);
 		$filteredRecord = $this->oclcFilteredHttpResponseParser->getFilteredRecord();
 
-		$filteredRecord->setTitleForPageCreation( 'OCLC:' . $oclcID );
+		$filteredRecord->setTitleForPageCreation('OCLC:' . $oclcID);
 		$filteredRecord->setSciteTransclusionHead(
 			'OCLC' . $oclcID
 		);
 
-		$filteredRecord->addSearchMatchSet( 'oclc', $oclcID );
-		$filteredRecord->set( 'oclc', $oclcID );
+		$filteredRecord->addSearchMatchSet('oclc', $oclcID);
+		$filteredRecord->set('oclc', $oclcID);
 
-		$dateTimeUtc = new \DateTime( 'now', new \DateTimeZone( 'UTC' ) );
-		$filteredRecord->set( 'retrieved-on', $dateTimeUtc->format( 'Y-m-d' ) );
+		$dateTimeUtc = new \DateTime('now', new \DateTimeZone('UTC'));
+		$filteredRecord->set('retrieved-on', $dateTimeUtc->format('Y-m-d'));
 	}
 
 }