mundschenk-at / php-typography

src/class-dom.php

Indentation -1 removed lines

@@ -332,7 +332,6 @@
 	 * Retrieves the tag name of the nearest block-level parent.
 	 *
 	 * @param \DOMNode $element A node.
-
 	 * @return string The tag name (or the empty string).
 	 */
 	public static function get_block_parent_name( \DOMNode $element ) {

Spacing +70 added lines, -70 removed lines

@@ -48,13 +48,13 @@ 
 	 *
 	 * @return array
 	 */
-	public static function block_tags( $reset = false ) {
-		if ( empty( self::$block_tags ) || $reset ) {
+	public static function block_tags($reset = false) {
+		if (empty(self::$block_tags) || $reset) {
 			self::$block_tags = array_merge(
-				array_flip( array_filter( array_keys( \Masterminds\HTML5\Elements::$html5 ), function( $tag ) {
-					return \Masterminds\HTML5\Elements::isA( $tag, \Masterminds\HTML5\Elements::BLOCK_TAG );
-				} ) ),
-				array_flip( [ 'li', 'td', 'dt' ] ) // not included as "block tags" in current HTML5-PHP version.
+				array_flip(array_filter(array_keys(\Masterminds\HTML5\Elements::$html5), function($tag) {
+					return \Masterminds\HTML5\Elements::isA($tag, \Masterminds\HTML5\Elements::BLOCK_TAG);
+				} )),
+				array_flip(['li', 'td', 'dt']) // not included as "block tags" in current HTML5-PHP version.
 			);
 		}
 
@@ -69,11 +69,11 @@ 
 	 *
 	 * @return array An associative array in the form ( $spl_object_hash => $node ).
 	 */
-	public static function nodelist_to_array( \DOMNodeList $list ) {
+	public static function nodelist_to_array(\DOMNodeList $list) {
 		$out = [];
 
-		foreach ( $list as $node ) {
-			$out[ spl_object_hash( $node ) ] = $node;
+		foreach ($list as $node) {
+			$out[spl_object_hash($node)] = $node;
 		}
 
 		return $out;
@@ -87,10 +87,10 @@ 
 	 *
 	 * @return array An array of \DOMNode.
 	 */
-	public static function get_ancestors( \DOMNode $node ) {
+	public static function get_ancestors(\DOMNode $node) {
 		$result = [];
 
-		while ( ( $node = $node->parentNode ) && ( $node instanceof \DOMElement ) ) { // @codingStandardsIgnoreLine.
+		while (($node = $node->parentNode) && ($node instanceof \DOMElement)) { // @codingStandardsIgnoreLine.
 			$result[] = $node;
 		}
 
@@ -106,26 +106,26 @@ 
 	 *
 	 * @return boolean True if the element has any of the given class(es).
 	 */
-	public static function has_class( \DOMNode $tag, $classnames ) {
-		if ( $tag instanceof \DOMText ) {
+	public static function has_class(\DOMNode $tag, $classnames) {
+		if ($tag instanceof \DOMText) {
 			$tag = $tag->parentNode; // @codingStandardsIgnoreLine.
 		}
 
 		// Bail if we are not working with a tag or if there is no classname.
-		if ( ! ( $tag instanceof \DOMElement ) || empty( $classnames ) ) {
+		if ( ! ($tag instanceof \DOMElement) || empty($classnames)) {
 			return false;
 		}
 
 		// Ensure we always have an array of classnames.
-		if ( ! is_array( $classnames ) ) {
-			$classnames = [ $classnames ];
+		if ( ! is_array($classnames)) {
+			$classnames = [$classnames];
 		}
 
-		if ( $tag->hasAttribute( 'class' ) ) {
-			$tag_classes = array_flip( explode( ' ', $tag->getAttribute( 'class' ) ) );
+		if ($tag->hasAttribute('class')) {
+			$tag_classes = array_flip(explode(' ', $tag->getAttribute('class')));
 
-			foreach ( $classnames as $classname ) {
-				if ( isset( $tag_classes[ $classname ] ) ) {
+			foreach ($classnames as $classname) {
+				if (isset($tag_classes[$classname])) {
 					return true;
 				}
 			}
@@ -141,15 +141,15 @@ 
 	 *
 	 * @return string A single character (or the empty string).
 	 */
-	public static function get_prev_chr( \DOMNode $element ) {
-		$previous_textnode = self::get_previous_textnode( $element );
+	public static function get_prev_chr(\DOMNode $element) {
+		$previous_textnode = self::get_previous_textnode($element);
 
-		if ( isset( $previous_textnode ) && isset( $previous_textnode->data ) ) {
+		if (isset($previous_textnode) && isset($previous_textnode->data)) {
 			// First determine encoding.
-			$func = Strings::functions( $previous_textnode->data );
+			$func = Strings::functions($previous_textnode->data);
 
-			if ( ! empty( $func ) ) {
-				return preg_replace( '/\p{C}/Su', '', $func['substr']( $previous_textnode->data, - 1 ) );
+			if ( ! empty($func)) {
+				return preg_replace('/\p{C}/Su', '', $func['substr']($previous_textnode->data, - 1));
 			}
 		} // @codeCoverageIgnore
 
@@ -163,15 +163,15 @@ 
 	 *
 	 * @return string A single character (or the empty string).
 	 */
-	public static function get_next_chr( \DOMNode $element ) {
-		$next_textnode = self::get_next_textnode( $element );
+	public static function get_next_chr(\DOMNode $element) {
+		$next_textnode = self::get_next_textnode($element);
 
-		if ( isset( $next_textnode ) && isset( $next_textnode->data ) ) {
+		if (isset($next_textnode) && isset($next_textnode->data)) {
 			// First determine encoding.
-			$func = Strings::functions( $next_textnode->data );
+			$func = Strings::functions($next_textnode->data);
 
-			if ( ! empty( $func ) ) {
-				return preg_replace( '/\p{C}/Su', '', $func['substr']( $next_textnode->data, 0, 1 ) );
+			if ( ! empty($func)) {
+				return preg_replace('/\p{C}/Su', '', $func['substr']($next_textnode->data, 0, 1));
 			}
 		} // @codeCoverageIgnore
 
@@ -185,11 +185,11 @@ 
 	 *
 	 * @return \DOMText|null Null if $element is a block-level element or no text sibling exists.
 	 */
-	public static function get_previous_textnode( \DOMNode $element = null ) {
-		return self::get_adjacent_textnode( function( &$node = null ) {
+	public static function get_previous_textnode(\DOMNode $element = null) {
+		return self::get_adjacent_textnode(function(&$node = null) {
 			$node = $node->previousSibling;
-			return self::get_last_textnode( $node );
-		}, __METHOD__, $element );
+			return self::get_last_textnode($node);
+		}, __METHOD__, $element);
 	}
 
 	/**
@@ -199,11 +199,11 @@ 
 	 *
 	 * @return \DOMText|null Null if $element is a block-level element or no text sibling exists.
 	 */
-	public static function get_next_textnode( \DOMNode $element = null ) {
-		return self::get_adjacent_textnode( function( &$node = null ) {
+	public static function get_next_textnode(\DOMNode $element = null) {
+		return self::get_adjacent_textnode(function(&$node = null) {
 			$node = $node->nextSibling;
-			return self::get_first_textnode( $node );
-		}, __METHOD__, $element );
+			return self::get_first_textnode($node);
+		}, __METHOD__, $element);
 	}
 
 	/**
@@ -215,22 +215,22 @@ 
 	 *
 	 * @return \DOMText|null Null if $element is a block-level element or no text sibling exists.
 	 */
-	private static function get_adjacent_textnode( callable $iterate, callable $get_adjacent_parent, \DOMNode $element = null ) {
-		if ( ! isset( $element ) ) {
+	private static function get_adjacent_textnode(callable $iterate, callable $get_adjacent_parent, \DOMNode $element = null) {
+		if ( ! isset($element)) {
 			return null;
-		} elseif ( $element instanceof \DOMElement && isset( self::$block_tags[ $element->tagName ] ) ) {
+		} elseif ($element instanceof \DOMElement && isset(self::$block_tags[$element->tagName])) {
 			return null;
 		}
 
 		$adjacent = null;
 		$node     = $element;
 
-		while ( ! empty( $node ) && empty( $adjacent ) ) {
-			$adjacent = $iterate( $node );
+		while ( ! empty($node) && empty($adjacent)) {
+			$adjacent = $iterate($node);
 		}
 
-		if ( empty( $adjacent ) ) {
-			$adjacent = $get_adjacent_parent( $element->parentNode );
+		if (empty($adjacent)) {
+			$adjacent = $get_adjacent_parent($element->parentNode);
 		}
 
 		return $adjacent;
@@ -244,15 +244,15 @@ 
 	 *
 	 * @return \DOMText|null The first child of type \DOMText, the element itself if it is of type \DOMText or null.
 	 */
-	public static function get_first_textnode( \DOMNode $element = null, $recursive = false ) {
-		return self::get_edge_textnode( function( \DOMNodeList $children, \DOMText &$first_textnode = null ) {
+	public static function get_first_textnode(\DOMNode $element = null, $recursive = false) {
+		return self::get_edge_textnode(function(\DOMNodeList $children, \DOMText & $first_textnode = null) {
 			$i = 0;
 
-			while ( $i < $children->length && empty( $first_textnode ) ) {
-				$first_textnode = self::get_first_textnode( $children->item( $i ), true );
+			while ($i < $children->length && empty($first_textnode)) {
+				$first_textnode = self::get_first_textnode($children->item($i), true);
 				$i++;
 			}
-		}, $element, $recursive );
+		}, $element, $recursive);
 	}
 
 	/**
@@ -263,15 +263,15 @@ 
 	 *
 	 * @return \DOMText|null The last child of type \DOMText, the element itself if it is of type \DOMText or null.
 	 */
-	public static function get_last_textnode( \DOMNode $element = null, $recursive = false ) {
-		return self::get_edge_textnode( function( \DOMNodeList $children, \DOMText &$last_textnode = null ) {
+	public static function get_last_textnode(\DOMNode $element = null, $recursive = false) {
+		return self::get_edge_textnode(function(\DOMNodeList $children, \DOMText & $last_textnode = null) {
 			$i = $children->length - 1;
 
-			while ( $i >= 0 && empty( $last_textnode ) ) {
-				$last_textnode = self::get_last_textnode( $children->item( $i ), true );
+			while ($i >= 0 && empty($last_textnode)) {
+				$last_textnode = self::get_last_textnode($children->item($i), true);
 				$i--;
 			}
-		}, $element, $recursive );
+		}, $element, $recursive);
 	}
 
 	/**
@@ -285,24 +285,24 @@ 
 	 *
 	 * @return \DOMText|null The last child of type \DOMText, the element itself if it is of type \DOMText or null.
 	 */
-	private static function get_edge_textnode( callable $iteration, \DOMNode $element = null, $recursive = false ) {
-		if ( ! isset( $element ) ) {
+	private static function get_edge_textnode(callable $iteration, \DOMNode $element = null, $recursive = false) {
+		if ( ! isset($element)) {
 			return null;
 		}
 
-		if ( $element instanceof \DOMText ) {
+		if ($element instanceof \DOMText) {
 			return $element;
-		} elseif ( ! $element instanceof \DOMElement ) {
+		} elseif ( ! $element instanceof \DOMElement) {
 			// Return null if $element is neither \DOMText nor \DOMElement.
 			return null;
-		} elseif ( $recursive && isset( self::$block_tags[ $element->tagName ] ) ) {
+		} elseif ($recursive && isset(self::$block_tags[$element->tagName])) {
 			return null;
 		}
 
 		$edge_textnode = null;
 
-		if ( $element->hasChildNodes() ) {
-			$iteration( $element->childNodes, $edge_textnode );
+		if ($element->hasChildNodes()) {
+			$iteration($element->childNodes, $edge_textnode);
 		}
 
 		return $edge_textnode;
@@ -315,13 +315,13 @@ 
 	 *
 	 * @return \DOMElement|null
 	 */
-	public static function get_block_parent( \DOMNode $element ) {
+	public static function get_block_parent(\DOMNode $element) {
 		$parent = $element->parentNode;
-		if ( ! $parent instanceof \DOMElement ) {
+		if ( ! $parent instanceof \DOMElement) {
 			return null;
 		}
 
-		while ( ! isset( self::$block_tags[ $parent->tagName ] ) && ! empty( $parent->parentNode ) && $parent->parentNode instanceof \DOMElement ) {
+		while ( ! isset(self::$block_tags[$parent->tagName]) && ! empty($parent->parentNode) && $parent->parentNode instanceof \DOMElement) {
 			$parent = $parent->parentNode;
 		}
 
@@ -335,10 +335,10 @@ 
 
 	 * @return string The tag name (or the empty string).
 	 */
-	public static function get_block_parent_name( \DOMNode $element ) {
-		$parent = self::get_block_parent( $element );
+	public static function get_block_parent_name(\DOMNode $element) {
+		$parent = self::get_block_parent($element);
 
-		if ( ! empty( $parent ) ) {
+		if ( ! empty($parent)) {
 			return $parent->tagName;
 		} else {
 			return '';

src/class-text-parser.php

Spacing +64 added lines, -64 removed lines

@@ -299,28 +299,28 @@ 
 	 *
 	 * @return bool Returns `true` on successful completion, `false` otherwise.
 	 */
-	public function load( $raw_text ) {
-		if ( ! is_string( $raw_text ) ) {
+	public function load($raw_text) {
+		if ( ! is_string($raw_text)) {
 			return false; // we have an error, abort.
 		}
 
 		// Abort if a simple string exceeds 500 characters (security concern).
-		if ( preg_match( self::_RE_MAX_STRING_LENGTH, $raw_text ) ) {
+		if (preg_match(self::_RE_MAX_STRING_LENGTH, $raw_text)) {
 			return false;
 		}
 
 		// Detect encoding.
-		$str_functions = Strings::functions( $raw_text );
-		if ( empty( $str_functions ) ) {
+		$str_functions = Strings::functions($raw_text);
+		if (empty($str_functions)) {
 			return false; // unknown encoding.
 		}
 		$this->current_strtoupper = $str_functions['strtoupper'];
 
 		// Tokenize the raw text parts.
-		$this->text = self::tokenize( preg_split( self::_RE_ANY_TEXT, $raw_text, -1, PREG_SPLIT_DELIM_CAPTURE | PREG_SPLIT_NO_EMPTY ) );
+		$this->text = self::tokenize(preg_split(self::_RE_ANY_TEXT, $raw_text, -1, PREG_SPLIT_DELIM_CAPTURE | PREG_SPLIT_NO_EMPTY));
 
 		// The token array should never be empty.
-		return ! empty( $this->text );
+		return ! empty($this->text);
 	}
 
 	/**
@@ -334,23 +334,23 @@ 
 	 *         @type Token $index A token may combine several input parts.
 	 * }
 	 */
-	protected static function tokenize( array $parts ) {
+	protected static function tokenize(array $parts) {
 		$tokens = [];
 		$index  = 0;
 
-		foreach ( $parts as $part ) {
-			if ( preg_match( self::_RE_SPACE, $part ) ) {
-				$tokens[ $index ] = new Token( $part, Token::SPACE );
-			} elseif ( preg_match( self::_RE_PUNCTUATION, $part ) ) {
-				$tokens[ $index ] = new Token( $part, Token::PUNCTUATION );
-			} elseif ( preg_match( self::_RE_WORD, $part ) ) {
+		foreach ($parts as $part) {
+			if (preg_match(self::_RE_SPACE, $part)) {
+				$tokens[$index] = new Token($part, Token::SPACE);
+			} elseif (preg_match(self::_RE_PUNCTUATION, $part)) {
+				$tokens[$index] = new Token($part, Token::PUNCTUATION);
+			} elseif (preg_match(self::_RE_WORD, $part)) {
 				// Make sure that things like email addresses and URLs are not broken up
 				// into words and punctuation not preceeded by an 'other'.
-				self::parse_ambiguous_token( Token::WORD, $part, $tokens, $index );
+				self::parse_ambiguous_token(Token::WORD, $part, $tokens, $index);
 			} else {
 				// Make sure that things like email addresses and URLs are not broken up into words
 				// and punctuation not preceeded by an 'other' or 'word'.
-				self::parse_ambiguous_token( Token::OTHER, $part, $tokens, $index );
+				self::parse_ambiguous_token(Token::OTHER, $part, $tokens, $index);
 			}
 
 			$index++;
@@ -367,25 +367,25 @@ 
 	 * @param array  $tokens        The token array. Passed by reference.
 	 * @param int    $index         The current index. Passed by reference.
 	 */
-	protected static function parse_ambiguous_token( $expected_type, $part, array &$tokens, &$index ) {
+	protected static function parse_ambiguous_token($expected_type, $part, array &$tokens, &$index) {
 
 		// Make sure that things like email addresses and URLs are not broken up incorrectly.
-		if ( self::is_preceeded_by( Token::OTHER, $tokens, $index ) || ( Token::OTHER === $expected_type && self::is_preceeded_by( Token::WORD, $tokens, $index ) ) ) {
+		if (self::is_preceeded_by(Token::OTHER, $tokens, $index) || (Token::OTHER === $expected_type && self::is_preceeded_by(Token::WORD, $tokens, $index))) {
 			$index--;
-			$old_part = $tokens[ $index ]->value;
-			$tokens[ $index ] = new Token( $old_part . $part, Token::OTHER );
+			$old_part = $tokens[$index]->value;
+			$tokens[$index] = new Token($old_part . $part, Token::OTHER);
 
 		// Not preceeded by a non-space + punctuation.
-		} elseif ( self::is_preceeded_by( Token::PUNCTUATION, $tokens, $index ) && self::is_not_preceeded_by( Token::SPACE, $tokens, $index, 2 ) ) {
-			$old_part   = $tokens[ $index - 1 ]->value;
-			$older_part = $tokens[ $index - 2 ]->value;
-			$tokens[ $index - 2 ] = new Token( $older_part . $old_part . $part, Token::OTHER );
-			unset( $tokens[ $index - 1 ] );
+		} elseif (self::is_preceeded_by(Token::PUNCTUATION, $tokens, $index) && self::is_not_preceeded_by(Token::SPACE, $tokens, $index, 2)) {
+			$old_part   = $tokens[$index - 1]->value;
+			$older_part = $tokens[$index - 2]->value;
+			$tokens[$index - 2] = new Token($older_part . $old_part . $part, Token::OTHER);
+			unset($tokens[$index - 1]);
 			$index = $index - 2;
 
 		// All good.
 		} else {
-			$tokens[ $index ] = new Token( $part, $expected_type );
+			$tokens[$index] = new Token($part, $expected_type);
 		}
 	}
 
@@ -399,8 +399,8 @@ 
 	 *
 	 * @return bool
 	 */
-	protected static function is_preceeded_by( $type, array $tokens, $index, $steps = 1 ) {
-		return $index - $steps >= 0 && $type === $tokens[ $index - $steps ]->type;
+	protected static function is_preceeded_by($type, array $tokens, $index, $steps = 1) {
+		return $index - $steps >= 0 && $type === $tokens[$index - $steps]->type;
 	}
 
 	/**
@@ -413,8 +413,8 @@ 
 	 *
 	 * @return bool
 	 */
-	protected static function is_not_preceeded_by( $type, array $tokens, $index, $steps = 1 ) {
-		return $index - $steps >= 0 && $type !== $tokens[ $index - $steps ]->type;
+	protected static function is_not_preceeded_by($type, array $tokens, $index, $steps = 1) {
+		return $index - $steps >= 0 && $type !== $tokens[$index - $steps]->type;
 	}
 
 
@@ -426,7 +426,7 @@ 
 	 * @return bool Returns true on successful completion.
 	 */
 	public function reload() {
-		return $this->load( $this->unload() );
+		return $this->load($this->unload());
 	}
 
 	/**
@@ -437,7 +437,7 @@ 
 	public function unload() {
 		$reassembled_text = '';
 
-		foreach ( $this->text as $token ) {
+		foreach ($this->text as $token) {
 			$reassembled_text .= $token->value;
 		}
 
@@ -463,9 +463,9 @@ 
 	 *      @type Text_Parser\Token $index
 	 * }
 	 */
-	public function update( $tokens ) {
-		foreach ( $tokens as $index => $token ) {
-			$this->text[ $index ] = $this->text[ $index ]->with_value( $token->value );
+	public function update($tokens) {
+		foreach ($tokens as $index => $token) {
+			$this->text[$index] = $this->text[$index]->with_value($token->value);
 		}
 	}
 
@@ -484,7 +484,7 @@ 
 	 * @return array    An array of Text_Parser\Token.
 	 */
 	public function get_spaces() {
-		return $this->get_type( Token::SPACE );
+		return $this->get_type(Token::SPACE);
 	}
 
 	/**
@@ -493,7 +493,7 @@ 
 	 * @return array    An array of Text_Parser\Token.
 	 */
 	public function get_punctuation() {
-		return $this->get_type( Token::PUNCTUATION );
+		return $this->get_type(Token::PUNCTUATION);
 	}
 
 	/**
@@ -505,22 +505,22 @@ 
 	 *
 	 * @return array    An array of Text_Parser\Token.
 	 */
-	public function get_words( $abc = self::ALLOW_ALL_LETTERS, $caps = self::ALLOW_ALL_CAPS, $comps = self::ALLOW_COMPOUNDS ) {
+	public function get_words($abc = self::ALLOW_ALL_LETTERS, $caps = self::ALLOW_ALL_CAPS, $comps = self::ALLOW_COMPOUNDS) {
 		// Return early if no text has been loaded.
-		if ( ! isset( $this->text ) || ! is_callable( $this->current_strtoupper ) ) {
+		if ( ! isset($this->text) || ! is_callable($this->current_strtoupper)) {
 			return []; // abort.
 		}
 
 		// Result set.
 		$tokens = [];
 
-		foreach ( $this->get_type( Token::WORD ) as $index => $token ) {
+		foreach ($this->get_type(Token::WORD) as $index => $token) {
 
-			if ( $this->conforms_to_letters_policy( $token, $abc ) &&
-				 $this->conforms_to_caps_policy( $token, $caps ) &&
-				 $this->conforms_to_compounds_policy( $token, $comps ) ) {
+			if ($this->conforms_to_letters_policy($token, $abc) &&
+				 $this->conforms_to_caps_policy($token, $caps) &&
+				 $this->conforms_to_compounds_policy($token, $comps)) {
 
-				$tokens[ $index ] = $token;
+				$tokens[$index] = $token;
 			}
 		}
 
@@ -535,18 +535,18 @@ 
 	 *
 	 * @return bool
 	 */
-	protected function conforms_to_letters_policy( Token $token, $policy ) {
+	protected function conforms_to_letters_policy(Token $token, $policy) {
 
 		// Short circuit.
-		if ( self::ALLOW_ALL_LETTERS === $policy ) {
+		if (self::ALLOW_ALL_LETTERS === $policy) {
 			return true;
 		}
 
-		$lettered = preg_replace( self::_RE_HTML_LETTER_CONNECTORS, '', $token->value );
+		$lettered = preg_replace(self::_RE_HTML_LETTER_CONNECTORS, '', $token->value);
 
 		return
-			( self::REQUIRE_ALL_LETTERS === $policy && $lettered === $token->value ) ||
-			( self::NO_ALL_LETTERS === $policy && $lettered !== $token->value );
+			(self::REQUIRE_ALL_LETTERS === $policy && $lettered === $token->value) ||
+			(self::NO_ALL_LETTERS === $policy && $lettered !== $token->value);
 	}
 
 	/**
@@ -557,19 +557,19 @@ 
 	 *
 	 * @return bool
 	 */
-	protected function conforms_to_caps_policy( Token $token, $policy ) {
+	protected function conforms_to_caps_policy(Token $token, $policy) {
 
 		// Short circuit.
-		if ( self::ALLOW_ALL_CAPS === $policy ) {
+		if (self::ALLOW_ALL_CAPS === $policy) {
 			return true;
 		}
 
 		// Token value in all-caps.
-		$capped = call_user_func( $this->current_strtoupper, $token->value );
+		$capped = call_user_func($this->current_strtoupper, $token->value);
 
 		return
-			( self::REQUIRE_ALL_CAPS === $policy && $capped === $token->value ) ||
-			( self::NO_ALL_CAPS === $policy && $capped !== $token->value );
+			(self::REQUIRE_ALL_CAPS === $policy && $capped === $token->value) ||
+			(self::NO_ALL_CAPS === $policy && $capped !== $token->value);
 	}
 
 	/**
@@ -580,18 +580,18 @@ 
 	 *
 	 * @return bool
 	 */
-	protected function conforms_to_compounds_policy( Token $token, $policy ) {
+	protected function conforms_to_compounds_policy(Token $token, $policy) {
 
 		// Short circuit.
-		if ( self::ALLOW_COMPOUNDS === $policy ) {
+		if (self::ALLOW_COMPOUNDS === $policy) {
 			return true;
 		}
 
-		$uncompound = preg_replace( '/-/S', '', $token->value );
+		$uncompound = preg_replace('/-/S', '', $token->value);
 
 		return
-			( self::REQUIRE_COMPOUNDS === $policy && $uncompound !== $token->value ) ||
-			( self::NO_COMPOUNDS === $policy && $uncompound === $token->value );
+			(self::REQUIRE_COMPOUNDS === $policy && $uncompound !== $token->value) ||
+			(self::NO_COMPOUNDS === $policy && $uncompound === $token->value);
 	}
 
 	/**
@@ -600,7 +600,7 @@ 
 	 * @return array    An array of Text_Parser\Token.
 	 */
 	public function get_other() {
-		return $this->get_type( Token::OTHER );
+		return $this->get_type(Token::OTHER);
 	}
 
 	/**
@@ -610,12 +610,12 @@ 
 	 *
 	 * @return array    An array of Text_Parser\Token.
 	 */
-	public function get_type( $type ) {
+	public function get_type($type) {
 		$tokens = [];
 
-		foreach ( $this->text as $index => $token ) {
-			if ( $token->type === $type ) {
-				$tokens[ $index ] = $token;
+		foreach ($this->text as $index => $token) {
+			if ($token->type === $type) {
+				$tokens[$index] = $token;
 			}
 		}
 

src/class-re.php

Spacing +10 added lines, -10 removed lines

@@ -103,23 +103,23 @@ 
 	 *
 	 * @return string A list of top-level domains concatenated with '|'.
 	 */
-	private static function get_top_level_domains_from_file( $path ) {
+	private static function get_top_level_domains_from_file($path) {
 		$domains = [];
 
-		if ( file_exists( $path ) ) {
-			$file = new \SplFileObject( $path );
+		if (file_exists($path)) {
+			$file = new \SplFileObject($path);
 
-			while ( ! $file->eof() ) {
+			while ( ! $file->eof()) {
 				$line = $file->fgets();
 
-				if ( preg_match( '#^[a-zA-Z0-9][a-zA-Z0-9-]*$#', $line, $matches ) ) {
-					$domains[] = strtolower( $matches[0] );
+				if (preg_match('#^[a-zA-Z0-9][a-zA-Z0-9-]*$#', $line, $matches)) {
+					$domains[] = strtolower($matches[0]);
 				}
 			}
 		}
 
-		if ( count( $domains ) > 0 ) {
-			return implode( '|', $domains );
+		if (count($domains) > 0) {
+			return implode('|', $domains);
 		} else {
 			return 'ac|ad|aero|ae|af|ag|ai|al|am|an|ao|aq|arpa|ar|asia|as|at|au|aw|ax|az|ba|bb|bd|be|bf|bg|bh|biz|bi|bj|bm|bn|bo|br|bs|bt|bv|bw|by|bz|cat|ca|cc|cd|cf|cg|ch|ci|ck|cl|cm|cn|com|coop|co|cr|cu|cv|cx|cy|cz|de|dj|dk|dm|do|dz|ec|edu|ee|eg|er|es|et|eu|fi|fj|fk|fm|fo|fr|ga|gb|gd|ge|gf|gg|gh|gi|gl|gm|gn|gov|gp|gq|gr|gs|gt|gu|gw|gy|hk|hm|hn|hr|ht|hu|id|ie|il|im|info|int|in|io|iq|ir|is|it|je|jm|jobs|jo|jp|ke|kg|kh|ki|km|kn|kp|kr|kw|ky|kz|la|lb|lc|li|lk|lr|ls|lt|lu|lv|ly|ma|mc|md|me|mg|mh|mil|mk|ml|mm|mn|mobi|mo|mp|mq|mr|ms|mt|museum|mu|mv|mw|mx|my|mz|name|na|nc|net|ne|nf|ng|ni|nl|no|np|nr|nu|nz|om|org|pa|pe|pf|pg|ph|pk|pl|pm|pn|pro|pr|ps|pt|pw|py|qa|re|ro|rs|ru|rw|sa|sb|sc|sd|se|sg|sh|si|sj|sk|sl|sm|sn|so|sr|st|su|sv|sy|sz|tc|td|tel|tf|tg|th|tj|tk|tl|tm|tn|to|tp|travel|tr|tt|tv|tw|tz|ua|ug|uk|um|us|uy|uz|va|vc|ve|vg|vi|vn|vu|wf|ws|ye|yt|yu|za|zm|zw';
 		}
@@ -131,9 +131,9 @@ 
 	 * @return string
 	 */
 	public static function top_level_domains() {
-		if ( empty( self::$top_level_domains_pattern ) ) {
+		if (empty(self::$top_level_domains_pattern)) {
 			// Initialize valid top level domains from IANA list.
-			self::$top_level_domains_pattern = self::get_top_level_domains_from_file( __DIR__ . '/IANA/tlds-alpha-by-domain.txt' );
+			self::$top_level_domains_pattern = self::get_top_level_domains_from_file(__DIR__ . '/IANA/tlds-alpha-by-domain.txt');
 		}
 
 		return self::$top_level_domains_pattern;

src/settings/class-simple-quotes.php

Spacing +1 added lines, -1 removed lines

@@ -55,7 +55,7 @@
 	 * @param string $open  Opening quote character(s).
 	 * @param string $close Closing quote character(s).
 	 */
-	public function __construct( $open, $close ) {
+	public function __construct($open, $close) {
 		$this->open  = $open;
 		$this->close = $close;
 	}

src/settings/class-simple-dashes.php

Spacing +1 added lines, -1 removed lines

@@ -71,7 +71,7 @@
 	 * @param string $interval            The dash character used for interval dashes.
 	 * @param string $interval_space      The space character used around interval dashes.
 	 */
-	public function __construct( $parenthetical, $parenthetical_space, $interval, $interval_space ) {
+	public function __construct($parenthetical, $parenthetical_space, $interval, $interval_space) {
 		$this->parenthetical_dash  = $parenthetical;
 		$this->parenthetical_space = $parenthetical_space;
 		$this->interval_dash       = $interval;

src/settings/class-quote-style.php

Spacing +5 added lines, -5 removed lines

@@ -149,15 +149,15 @@
 	 *
 	 * @return Quotes|null Returns null in case of an invalid $style parameter.
 	 */
-	public static function get_styled_quotes( $style, Settings $settings ) {
-		if ( isset( self::$styles[ $style ] ) ) {
-			return new Simple_Quotes( self::$styles[ $style ][ self::_OPEN ], self::$styles[ $style ][ self::_CLOSE ] );
+	public static function get_styled_quotes($style, Settings $settings) {
+		if (isset(self::$styles[$style])) {
+			return new Simple_Quotes(self::$styles[$style][self::_OPEN], self::$styles[$style][self::_CLOSE]);
 		}
 
-		if ( self::DOUBLE_GUILLEMETS_FRENCH === $style ) {
+		if (self::DOUBLE_GUILLEMETS_FRENCH === $style) {
 			$space = $settings->no_break_narrow_space();
 
-			return new Simple_Quotes( U::GUILLEMET_OPEN . $space, $space . U::GUILLEMET_CLOSE );
+			return new Simple_Quotes(U::GUILLEMET_OPEN . $space, $space . U::GUILLEMET_CLOSE);
 		}
 
 		return null;

src/class-hyphenator.php

Spacing +72 added lines, -72 removed lines

@@ -92,14 +92,14 @@ 
 	 * @param string|null $language   Optional. Short-form language name. Default null.
 	 * @param array       $exceptions Optional. Custom hyphenation exceptions. Default empty array.
 	 */
-	public function __construct( $language = null, array $exceptions = [] ) {
+	public function __construct($language = null, array $exceptions = []) {
 
-		if ( ! empty( $language ) ) {
-			$this->set_language( $language );
+		if ( ! empty($language)) {
+			$this->set_language($language);
 		}
 
-		if ( ! empty( $exceptions ) ) {
-			$this->set_custom_exceptions( $exceptions );
+		if ( ! empty($exceptions)) {
+			$this->set_custom_exceptions($exceptions);
 		}
 	}
 
@@ -108,31 +108,31 @@ 
 	 *
 	 * @param array $exceptions Optional. An array of words with all hyphenation points marked with a hard hyphen. Default empty array.
 	 */
-	public function set_custom_exceptions( array $exceptions = [] ) {
-		if ( empty( $exceptions ) && empty( $this->custom_exceptions ) ) {
+	public function set_custom_exceptions(array $exceptions = []) {
+		if (empty($exceptions) && empty($this->custom_exceptions)) {
 			return; // Nothing to do at all.
 		}
 
 		// Calculate hash & check against previous exceptions.
-		$new_hash = self::get_object_hash( $exceptions );
-		if ( $this->custom_exceptions_hash === $new_hash ) {
+		$new_hash = self::get_object_hash($exceptions);
+		if ($this->custom_exceptions_hash === $new_hash) {
 			return; // No need to update exceptions.
 		}
 
 		// Do our thing.
 		$exception_keys = [];
-		foreach ( $exceptions as $exception ) {
-			$func = Strings::functions( $exception );
-			if ( empty( $func ) ) {
+		foreach ($exceptions as $exception) {
+			$func = Strings::functions($exception);
+			if (empty($func)) {
 				continue; // unknown encoding, abort.
 			}
 
-			$exception = $func['strtolower']( $exception );
-			$exception_keys[ $exception ] = preg_replace( "#-#{$func['u']}", '', $exception );
+			$exception = $func['strtolower']($exception);
+			$exception_keys[$exception] = preg_replace("#-#{$func['u']}", '', $exception);
 		}
 
 		// Update exceptions.
-		$this->custom_exceptions      = array_flip( $exception_keys );
+		$this->custom_exceptions      = array_flip($exception_keys);
 		$this->custom_exceptions_hash = $new_hash;
 
 		// Force remerging of patgen and custom exception patterns.
@@ -146,8 +146,8 @@ 
 	 *
 	 * @return string
 	 */
-	protected static function get_object_hash( $object ) {
-		return md5( json_encode( $object ), false );
+	protected static function get_object_hash($object) {
+		return md5(json_encode($object), false);
 	}
 
 	/**
@@ -157,23 +157,23 @@ 
 	 *
 	 * @return bool Whether loading the pattern file was successful.
 	 */
-	public function set_language( $lang = 'en-US' ) {
-		if ( isset( $this->language ) && $this->language === $lang ) {
+	public function set_language($lang = 'en-US') {
+		if (isset($this->language) && $this->language === $lang) {
 			return true; // Bail out, no need to do anything.
 		}
 
 		$success = false;
-		$language_file_name = dirname( __FILE__ ) . '/lang/' . $lang . '.json';
+		$language_file_name = dirname(__FILE__) . '/lang/' . $lang . '.json';
 
-		if ( file_exists( $language_file_name ) ) {
-			$raw_language_file = file_get_contents( $language_file_name );
+		if (file_exists($language_file_name)) {
+			$raw_language_file = file_get_contents($language_file_name);
 
-			if ( false !== $raw_language_file ) {
-				$language_file = json_decode( $raw_language_file, true );
+			if (false !== $raw_language_file) {
+				$language_file = json_decode($raw_language_file, true);
 
-				if ( false !== $language_file ) {
+				if (false !== $language_file) {
 					$this->language           = $lang;
-					$this->pattern_trie       = Trie_Node::build_trie( $language_file['patterns'] );
+					$this->pattern_trie       = Trie_Node::build_trie($language_file['patterns']);
 					$this->pattern_exceptions = $language_file['exceptions'];
 
 					$success = true;
@@ -182,7 +182,7 @@ 
 		}
 
 		// Clean up.
-		if ( ! $success ) {
+		if ( ! $success) {
 			$this->language = null;
 			$this->pattern_trie = null;
 			$this->pattern_exceptions = [];
@@ -206,18 +206,18 @@ 
 	 *
 	 * @return array The modified text tokens.
 	 */
-	public function hyphenate( $parsed_text_tokens, $hyphen = '-', $hyphenate_title_case = false, $min_length = 2, $min_before = 2, $min_after = 2 ) {
-		if ( empty( $min_length ) || empty( $min_before ) || ! isset( $this->pattern_trie ) || ! isset( $this->pattern_exceptions ) ) {
+	public function hyphenate($parsed_text_tokens, $hyphen = '-', $hyphenate_title_case = false, $min_length = 2, $min_before = 2, $min_after = 2) {
+		if (empty($min_length) || empty($min_before) || ! isset($this->pattern_trie) || ! isset($this->pattern_exceptions)) {
 			return $parsed_text_tokens;
 		}
 
 		// Make sure we have full exceptions list.
-		if ( ! isset( $this->merged_exception_patterns ) ) {
+		if ( ! isset($this->merged_exception_patterns)) {
 			$this->merge_hyphenation_exceptions();
 		}
 
-		foreach ( $parsed_text_tokens as $key => $text_token ) {
-			$parsed_text_tokens[ $key ] = $text_token->with_value( $this->hyphenate_word( $text_token->value, $hyphen, $hyphenate_title_case, $min_length, $min_before, $min_after ) );
+		foreach ($parsed_text_tokens as $key => $text_token) {
+			$parsed_text_tokens[$key] = $text_token->with_value($this->hyphenate_word($text_token->value, $hyphen, $hyphenate_title_case, $min_length, $min_before, $min_after));
 		}
 
 		return $parsed_text_tokens;
@@ -235,48 +235,48 @@ 
 	 *
 	 * @return string
 	 */
-	protected function hyphenate_word( $word, $hyphen, $hyphenate_title_case, $min_length, $min_before, $min_after ) {
+	protected function hyphenate_word($word, $hyphen, $hyphenate_title_case, $min_length, $min_before, $min_after) {
 		// Quickly reference string functions according to encoding.
-		$func = Strings::functions( $word );
-		if ( empty( $func ) ) {
+		$func = Strings::functions($word);
+		if (empty($func)) {
 			return $word; // unknown encoding, abort.
 		}
 
 		// Check word length.
-		$word_length = $func['strlen']( $word );
-		if ( $word_length < $min_length ) {
+		$word_length = $func['strlen']($word);
+		if ($word_length < $min_length) {
 			return $word;
 		}
 
 		// Trie lookup requires a lowercase search term.
-		$the_key = $func['strtolower']( $word );
+		$the_key = $func['strtolower']($word);
 
 		// If this is a capitalized word, and settings do not allow hyphenation of such, abort!
 		// Note: This is different than uppercase words, where we are looking for title case.
-		if ( ! $hyphenate_title_case && $func['substr']( $the_key, 0, 1 ) !== $func['substr']( $word, 0, 1 ) ) {
+		if ( ! $hyphenate_title_case && $func['substr']($the_key, 0, 1) !== $func['substr']($word, 0, 1)) {
 			return $word;
 		}
 
 		// Give exceptions preference.
-		if ( isset( $this->merged_exception_patterns[ $the_key ] ) ) {
-			$word_pattern = $this->merged_exception_patterns[ $the_key ];
+		if (isset($this->merged_exception_patterns[$the_key])) {
+			$word_pattern = $this->merged_exception_patterns[$the_key];
 		}
 
 		// Lookup word pattern if there is no exception.
-		if ( ! isset( $word_pattern ) ) {
-			$word_pattern = $this->lookup_word_pattern( $the_key, $func['strlen'], $func['str_split'] );
+		if ( ! isset($word_pattern)) {
+			$word_pattern = $this->lookup_word_pattern($the_key, $func['strlen'], $func['str_split']);
 		}
 
 		// Add hyphen character based on $word_pattern.
-		$word_parts      = $func['str_split']( $word, 1 );
+		$word_parts      = $func['str_split']($word, 1);
 		$hyphenated_word = '';
 
-		for ( $i = 0; $i < $word_length; $i++ ) {
-			if ( isset( $word_pattern[ $i ] ) && self::is_odd( $word_pattern[ $i ] ) && ( $i >= $min_before) && ( $i <= $word_length - $min_after ) ) {
+		for ($i = 0; $i < $word_length; $i++) {
+			if (isset($word_pattern[$i]) && self::is_odd($word_pattern[$i]) && ($i >= $min_before) && ($i <= $word_length - $min_after)) {
 				$hyphenated_word .= $hyphen;
 			}
 
-			$hyphenated_word .= $word_parts[ $i ];
+			$hyphenated_word .= $word_parts[$i];
 		}
 
 		return $hyphenated_word;
@@ -291,33 +291,33 @@ 
 	 *
 	 * @return array The hyphenation pattern.
 	 */
-	protected function lookup_word_pattern( $key, callable $strlen, callable $str_split ) {
+	protected function lookup_word_pattern($key, callable $strlen, callable $str_split) {
 		// Add underscores to make out-of-index checks unnecessary,
 		// also hyphenation is done in lower case.
 		$search        = '_' . $key . '_';
-		$search_length = $strlen( $search );
-		$chars         = $str_split( $search );
+		$search_length = $strlen($search);
+		$chars         = $str_split($search);
 		$word_pattern  = [];
 
-		for ( $start = 0; $start < $search_length; ++$start ) {
+		for ($start = 0; $start < $search_length; ++$start) {
 			// Start from the trie root node.
 			$node = $this->pattern_trie;
 
 			// Walk through the trie while storing detected patterns.
-			for ( $step = $start; $step < $search_length; ++$step ) {
+			for ($step = $start; $step < $search_length; ++$step) {
 				// No further path in the trie.
-				if ( ! $node->exists( $chars[ $step ] ) ) {
+				if ( ! $node->exists($chars[$step])) {
 					break;
 				}
 
 				// Look for next character.
-				$node = $node->get_node( $chars[ $step ] );
+				$node = $node->get_node($chars[$step]);
 
 				// Merge different offset values and keep maximum.
-				foreach ( $node->offsets() as $pattern_offset ) {
+				foreach ($node->offsets() as $pattern_offset) {
 					$value  = $pattern_offset[0];
 					$offset = $pattern_offset[1] + $start - 1;
-					$word_pattern[ $offset ] = isset( $word_pattern[ $offset ] ) ? max( $word_pattern[ $offset ], $value ) : $value;
+					$word_pattern[$offset] = isset($word_pattern[$offset]) ? max($word_pattern[$offset], $value) : $value;
 				}
 			}
 		}
@@ -333,18 +333,18 @@ 
 		$exceptions = [];
 
 		// Merge custom and language specific word hyphenations.
-		if ( ! empty( $this->pattern_exceptions ) && ! empty( $this->custom_exceptions ) ) {
-			$exceptions = array_merge( $this->custom_exceptions, $this->pattern_exceptions );
-		} elseif ( ! empty( $this->pattern_exceptions ) ) {
+		if ( ! empty($this->pattern_exceptions) && ! empty($this->custom_exceptions)) {
+			$exceptions = array_merge($this->custom_exceptions, $this->pattern_exceptions);
+		} elseif ( ! empty($this->pattern_exceptions)) {
 			$exceptions = $this->pattern_exceptions;
-		} elseif ( ! empty( $this->custom_exceptions ) ) {
+		} elseif ( ! empty($this->custom_exceptions)) {
 			$exceptions = $this->custom_exceptions;
 		}
 
 		// Update patterns as well.
 		$exception_patterns = [];
-		foreach ( $exceptions as $exception_key => $exception ) {
-			$exception_patterns[ $exception_key ] = self::convert_hyphenation_exception_to_pattern( $exception );
+		foreach ($exceptions as $exception_key => $exception) {
+			$exception_patterns[$exception_key] = self::convert_hyphenation_exception_to_pattern($exception);
 		}
 
 		$this->merged_exception_patterns = $exception_patterns;
@@ -357,22 +357,22 @@ 
 	 *
 	 * @return array|null Returns the hyphenation pattern or null if `$exception` is using an invalid encoding.
 	 */
-	protected static function convert_hyphenation_exception_to_pattern( $exception ) {
-		$func = Strings::functions( $exception );
-		if ( empty( $func ) ) {
+	protected static function convert_hyphenation_exception_to_pattern($exception) {
+		$func = Strings::functions($exception);
+		if (empty($func)) {
 			return null; // unknown encoding, abort.
 		}
 
 		// Set the word_pattern - this method keeps any contextually important capitalization.
-		$lowercase_hyphened_word_parts  = $func['str_split']( $exception, 1 );
-		$lowercase_hyphened_word_length = $func['strlen']( $exception );
+		$lowercase_hyphened_word_parts  = $func['str_split']($exception, 1);
+		$lowercase_hyphened_word_length = $func['strlen']($exception);
 
 		$word_pattern = [];
 		$index = 0;
 
-		for ( $i = 0; $i < $lowercase_hyphened_word_length; $i++ ) {
-			if ( '-' === $lowercase_hyphened_word_parts[ $i ] ) {
-				$word_pattern[ $index ] = 9;
+		for ($i = 0; $i < $lowercase_hyphened_word_length; $i++) {
+			if ('-' === $lowercase_hyphened_word_parts[$i]) {
+				$word_pattern[$index] = 9;
 			} else {
 				$index++;
 			}
@@ -388,7 +388,7 @@ 
 	 *
 	 * @return bool true if $number is odd, false if it is even.
 	 */
-	protected static function is_odd( $number ) {
-		return (bool) ( $number % 2 );
+	protected static function is_odd($number) {
+		return (bool) ($number % 2);
 	}
 }

src/fixes/class-node-fix.php

Spacing +1 added lines, -1 removed lines

@@ -46,7 +46,7 @@
 	 *
 	 * @return void
 	 */
-	public function apply( \DOMText $textnode, Settings $settings, $is_title = false );
+	public function apply(\DOMText $textnode, Settings $settings, $is_title = false);
 
 	/**
 	 * Determines whether the fix should be applied to (RSS) feeds.

src/fixes/class-token-fix.php

Spacing +1 added lines, -1 removed lines

@@ -52,7 +52,7 @@
 	 *
 	 * @return array An array of tokens.
 	 */
-	public function apply( array $tokens, Settings $settings, $is_title = false, \DOMText $textnode = null );
+	public function apply(array $tokens, Settings $settings, $is_title = false, \DOMText $textnode = null);
 
 	/**
 	 * Determines whether the fix should be applied to (RSS) feeds.