mundschenk-at / php-typography

src/fixes/node-fixes/class-space-collapse-fix.php

Spacing +7 added lines, -7 removed lines

@@ -53,23 +53,23 @@
 	 * @param Settings $settings Required.
 	 * @param bool     $is_title Optional. Default false.
 	 */
-	public function apply( \DOMText $textnode, Settings $settings, $is_title = false ) {
-		if ( empty( $settings['spaceCollapse'] ) ) {
+	public function apply(\DOMText $textnode, Settings $settings, $is_title = false) {
+		if (empty($settings['spaceCollapse'])) {
 			return;
 		}
 
 		// Normal spacing.
-		$textnode->data = preg_replace( self::COLLAPSE_NORMAL_SPACES, ' ', $textnode->data );
+		$textnode->data = preg_replace(self::COLLAPSE_NORMAL_SPACES, ' ', $textnode->data);
 
 		// Non-breakable space get's priority. If non-breakable space exists in a string of spaces, it collapses to a single non-breakable space.
-		$textnode->data = preg_replace( self::COLLAPSE_NON_BREAKABLE_SPACES, U::NO_BREAK_SPACE, $textnode->data );
+		$textnode->data = preg_replace(self::COLLAPSE_NON_BREAKABLE_SPACES, U::NO_BREAK_SPACE, $textnode->data);
 
 		// For any other spaceing, replace with the first occurance of an unusual space character.
-		$textnode->data = preg_replace( self::COLLAPSE_OTHER_SPACES, '$1', $textnode->data );
+		$textnode->data = preg_replace(self::COLLAPSE_OTHER_SPACES, '$1', $textnode->data);
 
 		// Remove all spacing at beginning of block level elements.
-		if ( '' === DOM::get_prev_chr( $textnode ) ) { // we have the first text in a block level element.
-			$textnode->data = preg_replace( self::COLLAPSE_SPACES_AT_START_OF_BLOCK, '', $textnode->data );
+		if ('' === DOM::get_prev_chr($textnode)) { // we have the first text in a block level element.
+			$textnode->data = preg_replace(self::COLLAPSE_SPACES_AT_START_OF_BLOCK, '', $textnode->data);
 		}
 	}
 }

src/fixes/node-fixes/class-smart-ellipses-fix.php

Spacing +4 added lines, -4 removed lines

@@ -46,12 +46,12 @@
 	 * @param Settings $settings Required.
 	 * @param bool     $is_title Optional. Default false.
 	 */
-	public function apply( \DOMText $textnode, Settings $settings, $is_title = false ) {
-		if ( empty( $settings['smartEllipses'] ) ) {
+	public function apply(\DOMText $textnode, Settings $settings, $is_title = false) {
+		if (empty($settings['smartEllipses'])) {
 			return;
 		}
 
-		$textnode->data = str_replace( [ '....', '. . . .' ], '.' . U::ELLIPSIS, $textnode->data );
-		$textnode->data = str_replace( [ '...', '. . .' ],          U::ELLIPSIS, $textnode->data );
+		$textnode->data = str_replace(['....', '. . . .'], '.' . U::ELLIPSIS, $textnode->data);
+		$textnode->data = str_replace(['...', '. . .'], U::ELLIPSIS, $textnode->data);
 	}
 }

src/fixes/node-fixes/class-smart-maths-fix.php

Spacing +14 added lines, -14 removed lines

@@ -187,37 +187,37 @@
 	 * @param Settings $settings Required.
 	 * @param bool     $is_title Optional. Default false.
 	 */
-	public function apply( \DOMText $textnode, Settings $settings, $is_title = false ) {
-		if ( empty( $settings['smartMath'] ) ) {
+	public function apply(\DOMText $textnode, Settings $settings, $is_title = false) {
+		if (empty($settings['smartMath'])) {
 			return;
 		}
 
 		// First, let's find math equations.
-		$textnode->data = preg_replace_callback( self::MATH_EQUATION, function( array $matches ) {
-			$matches[0] = str_replace( '-', U::MINUS,          $matches[0] );
-			$matches[0] = str_replace( '/', U::DIVISION,       $matches[0] );
-			$matches[0] = str_replace( 'x', U::MULTIPLICATION, $matches[0] );
-			$matches[0] = str_replace( '*', U::MULTIPLICATION, $matches[0] );
+		$textnode->data = preg_replace_callback(self::MATH_EQUATION, function(array $matches) {
+			$matches[0] = str_replace('-', U::MINUS, $matches[0]);
+			$matches[0] = str_replace('/', U::DIVISION, $matches[0]);
+			$matches[0] = str_replace('x', U::MULTIPLICATION, $matches[0]);
+			$matches[0] = str_replace('*', U::MULTIPLICATION, $matches[0]);
 
 			return $matches[0];
 
-		}, $textnode->data );
+		}, $textnode->data);
 
 		// Revert 4-4 to plain minus-hyphen so as to not mess with ranges of numbers (i.e. pp. 46-50).
-		$textnode->data = preg_replace( self::REVERT_RANGE, '$1-$2', $textnode->data );
+		$textnode->data = preg_replace(self::REVERT_RANGE, '$1-$2', $textnode->data);
 
 		// Revert fractions to basic slash.
 		// We'll leave styling fractions to smart_fractions.
-		$textnode->data = preg_replace( self::REVERT_FRACTION, '$1/$2', $textnode->data );
+		$textnode->data = preg_replace(self::REVERT_FRACTION, '$1/$2', $textnode->data);
 
 		// Revert date back to original formats.
 		// YYYY-MM-DD.
-		$textnode->data = preg_replace( self::REVERT_DATE_YYYY_MM_DD,         '$1-$2-$3',     $textnode->data );
+		$textnode->data = preg_replace(self::REVERT_DATE_YYYY_MM_DD, '$1-$2-$3', $textnode->data);
 		// MM-DD-YYYY or DD-MM-YYYY.
-		$textnode->data = preg_replace( self::REVERT_DATE_MM_DD_YYYY,         '$1$3-$2$4-$5', $textnode->data );
+		$textnode->data = preg_replace(self::REVERT_DATE_MM_DD_YYYY, '$1$3-$2$4-$5', $textnode->data);
 		// YYYY-MM or YYYY-DDD next.
-		$textnode->data = preg_replace( self::REVERT_DATE_YYYY_MM,            '$1-$2',        $textnode->data );
+		$textnode->data = preg_replace(self::REVERT_DATE_YYYY_MM, '$1-$2', $textnode->data);
 		// MM/DD/YYYY or DD/MM/YYYY.
-		$textnode->data = preg_replace( self::REVERT_DATE_MM_DD_YYYY_SLASHED, '$1$3/$2$4/$5', $textnode->data );
+		$textnode->data = preg_replace(self::REVERT_DATE_MM_DD_YYYY_SLASHED, '$1$3/$2$4/$5', $textnode->data);
 	}
 }

src/fixes/token-fixes/class-wrap-hard-hyphens-fix.php

Spacing +15 added lines, -15 removed lines

@@ -58,11 +58,11 @@ 
 	 *
 	 * @param bool $feed_compatible Optional. Default false.
 	 */
-	public function __construct( $feed_compatible = false ) {
-		parent::__construct( Token_Fix::MIXED_WORDS, $feed_compatible );
+	public function __construct($feed_compatible = false) {
+		parent::__construct(Token_Fix::MIXED_WORDS, $feed_compatible);
 
-		$this->hyphens_array = array_unique( [ '-', U::HYPHEN ] );
-		$this->remove_ending_space_regex = '/(' . implode( '|', $this->hyphens_array ) . ')' . U::ZERO_WIDTH_SPACE . '$/';
+		$this->hyphens_array = array_unique(['-', U::HYPHEN]);
+		$this->remove_ending_space_regex = '/(' . implode('|', $this->hyphens_array) . ')' . U::ZERO_WIDTH_SPACE . '$/';
 	}
 
 	/**
@@ -75,26 +75,26 @@ 
 	 *
 	 * @return array An array of tokens.
 	 */
-	public function apply( array $tokens, Settings $settings, $is_title = false, \DOMText $textnode = null ) {
-		if ( ! empty( $settings['hyphenHardWrap'] ) || ! empty( $settings['smartDashes'] ) ) {
+	public function apply(array $tokens, Settings $settings, $is_title = false, \DOMText $textnode = null) {
+		if ( ! empty($settings['hyphenHardWrap']) || ! empty($settings['smartDashes'])) {
 
-			foreach ( $tokens as $index => $text_token ) {
+			foreach ($tokens as $index => $text_token) {
 				$value = $text_token->value;
 
-				if ( isset( $settings['hyphenHardWrap'] ) && $settings['hyphenHardWrap'] ) {
-					$value = str_replace( $this->hyphens_array, '-' . U::ZERO_WIDTH_SPACE, $value );
-					$value = str_replace( '_', '_' . U::ZERO_WIDTH_SPACE, $value );
-					$value = str_replace( '/', '/' . U::ZERO_WIDTH_SPACE, $value );
+				if (isset($settings['hyphenHardWrap']) && $settings['hyphenHardWrap']) {
+					$value = str_replace($this->hyphens_array, '-' . U::ZERO_WIDTH_SPACE, $value);
+					$value = str_replace('_', '_' . U::ZERO_WIDTH_SPACE, $value);
+					$value = str_replace('/', '/' . U::ZERO_WIDTH_SPACE, $value);
 
-					$value = preg_replace( $this->remove_ending_space_regex, '$1', $value );
+					$value = preg_replace($this->remove_ending_space_regex, '$1', $value);
 				}
 
-				if ( ! empty( $settings['smartDashes'] ) ) {
+				if ( ! empty($settings['smartDashes'])) {
 					// Handled here because we need to know we are inside a word and not a URL.
-					$value = str_replace( '-', U::HYPHEN, $value );
+					$value = str_replace('-', U::HYPHEN, $value);
 				}
 
-				$tokens[ $index ] = $text_token->with_value( $value );
+				$tokens[$index] = $text_token->with_value($value);
 			}
 		}
 

src/fixes/token-fixes/class-hyphenate-fix.php

Spacing +21 added lines, -21 removed lines

@@ -62,10 +62,10 @@ 
 	 * @param int                   $target          Optional. Default Token_Fix::WORDS.
 	 * @param bool                  $feed_compatible Optional. Default false.
 	 */
-	public function __construct( Hyphenator_Cache $cache = null, $target = Token_Fix::WORDS, $feed_compatible = false ) {
-		parent::__construct( $target, $feed_compatible );
+	public function __construct(Hyphenator_Cache $cache = null, $target = Token_Fix::WORDS, $feed_compatible = false) {
+		parent::__construct($target, $feed_compatible);
 
-		if ( empty( $cache ) ) {
+		if (empty($cache)) {
 			$cache = new Hyphenator_Cache();
 		}
 
@@ -89,26 +89,26 @@ 
 	 *
 	 * @return array An array of tokens.
 	 */
-	public function apply( array $tokens, Settings $settings, $is_title = false, \DOMText $textnode = null ) {
-		if ( empty( $settings['hyphenation'] ) ) {
+	public function apply(array $tokens, Settings $settings, $is_title = false, \DOMText $textnode = null) {
+		if (empty($settings['hyphenation'])) {
 			return $tokens; // abort.
 		}
 
 		$is_heading = false;
-		if ( ! empty( $textnode ) && ! empty( $textnode->parentNode ) ) {
-			$block_level_parent = DOM::get_block_parent_name( $textnode );
+		if ( ! empty($textnode) && ! empty($textnode->parentNode)) {
+			$block_level_parent = DOM::get_block_parent_name($textnode);
 
-			if ( ! empty( $block_level_parent ) && isset( $this->heading_tags[ $block_level_parent ] ) ) {
+			if ( ! empty($block_level_parent) && isset($this->heading_tags[$block_level_parent])) {
 				$is_heading = true;
 			}
 		}
 
-		if ( empty( $settings['hyphenateTitle'] ) && ( $is_title || $is_heading ) ) {
+		if (empty($settings['hyphenateTitle']) && ($is_title || $is_heading)) {
 			return $tokens; // abort.
 		}
 
 		// Call functionality as seperate function so it can be run without test for setting['hyphenation'] - such as with url wrapping.
-		return $this->do_hyphenate( $tokens, $settings );
+		return $this->do_hyphenate($tokens, $settings);
 	}
 
 	/**
@@ -120,12 +120,12 @@ 
 	 *
 	 * @return array The hyphenated text tokens.
 	 */
-	protected function do_hyphenate( array $tokens, Settings $settings, $hyphen = U::SOFT_HYPHEN ) {
-		if ( empty( $settings['hyphenMinLength'] ) || empty( $settings['hyphenMinBefore'] ) ) {
+	protected function do_hyphenate(array $tokens, Settings $settings, $hyphen = U::SOFT_HYPHEN) {
+		if (empty($settings['hyphenMinLength']) || empty($settings['hyphenMinBefore'])) {
 			return $tokens;
 		}
 
-		return $this->get_hyphenator( $settings )->hyphenate( $tokens, $hyphen, ! empty( $settings['hyphenateTitleCase'] ), $settings['hyphenMinLength'], $settings['hyphenMinBefore'], $settings['hyphenMinAfter'] );
+		return $this->get_hyphenator($settings)->hyphenate($tokens, $hyphen, ! empty($settings['hyphenateTitleCase']), $settings['hyphenMinLength'], $settings['hyphenMinBefore'], $settings['hyphenMinAfter']);
 	}
 
 	/**
@@ -135,17 +135,17 @@ 
 	 *
 	 * @return Hyphenator
 	 */
-	public function get_hyphenator( Settings $settings ) {
+	public function get_hyphenator(Settings $settings) {
 		$lang       = $settings['hyphenLanguage'];
 		$exceptions = (array) $settings['hyphenationCustomExceptions'];
-		$hyphenator = $this->cache->get_hyphenator( $lang );
+		$hyphenator = $this->cache->get_hyphenator($lang);
 
-		if ( empty( $hyphenator ) ) {
-			$hyphenator = new Hyphenator( $lang, $exceptions );
-			$this->cache->set_hyphenator( $lang, $hyphenator );
+		if (empty($hyphenator)) {
+			$hyphenator = new Hyphenator($lang, $exceptions);
+			$this->cache->set_hyphenator($lang, $hyphenator);
 		} else {
-			$hyphenator->set_language( $lang ); // just for insurance.
-			$hyphenator->set_custom_exceptions( $exceptions );
+			$hyphenator->set_language($lang); // just for insurance.
+			$hyphenator->set_custom_exceptions($exceptions);
 		}
 
 		return $hyphenator;
@@ -156,7 +156,7 @@ 
 	 *
 	 * @param Hyphenator_Cache $cache Required.
 	 */
-	public function set_hyphenator_cache( Hyphenator_Cache $cache ) {
+	public function set_hyphenator_cache(Hyphenator_Cache $cache) {
 		$this->cache = $cache;
 	}
 }

src/fixes/token-fixes/class-wrap-emails-fix.php

Spacing +7 added lines, -7 removed lines

@@ -54,8 +54,8 @@ 
 	 *
 	 * @param bool $feed_compatible Optional. Default false.
 	 */
-	public function __construct( $feed_compatible = false ) {
-		parent::__construct( Token_Fix::OTHER, $feed_compatible );
+	public function __construct($feed_compatible = false) {
+		parent::__construct(Token_Fix::OTHER, $feed_compatible);
 
 		$this->email_pattern = "/(?:
 			\A
@@ -88,16 +88,16 @@ 
 	 *
 	 * @return array An array of tokens.
 	 */
-	public function apply( array $tokens, Settings $settings, $is_title = false, \DOMText $textnode = null ) {
-		if ( empty( $settings['emailWrap'] ) ) {
+	public function apply(array $tokens, Settings $settings, $is_title = false, \DOMText $textnode = null) {
+		if (empty($settings['emailWrap'])) {
 			return $tokens;
 		}
 
 		// Test for and parse urls.
-		foreach ( $tokens as $index => $token ) {
+		foreach ($tokens as $index => $token) {
 			$value = $token->value;
-			if ( preg_match( $this->email_pattern, $value, $email_match ) ) {
-				$tokens[ $index ] = $token->with_value( preg_replace( self::REPLACE_EMAILS, '$1' . U::ZERO_WIDTH_SPACE, $value ) );
+			if (preg_match($this->email_pattern, $value, $email_match)) {
+				$tokens[$index] = $token->with_value(preg_replace(self::REPLACE_EMAILS, '$1' . U::ZERO_WIDTH_SPACE, $value));
 			}
 		}
 

src/fixes/token-fixes/class-hyphenate-compounds-fix.php

Spacing +9 added lines, -9 removed lines

@@ -48,8 +48,8 @@ 
 	 * @param Hyphenator_Cache|null $cache           Optional. Default null.
 	 * @param bool                  $feed_compatible Optional. Default false.
 	 */
-	public function __construct( Hyphenator_Cache $cache = null, $feed_compatible = false ) {
-		parent::__construct( $cache, Token_Fix::COMPOUND_WORDS, $feed_compatible );
+	public function __construct(Hyphenator_Cache $cache = null, $feed_compatible = false) {
+		parent::__construct($cache, Token_Fix::COMPOUND_WORDS, $feed_compatible);
 	}
 
 	/**
@@ -62,21 +62,21 @@ 
 	 *
 	 * @return array An array of tokens.
 	 */
-	public function apply( array $tokens, Settings $settings, $is_title = false, \DOMText $textnode = null ) {
-		if ( empty( $settings['hyphenateCompounds'] ) ) {
+	public function apply(array $tokens, Settings $settings, $is_title = false, \DOMText $textnode = null) {
+		if (empty($settings['hyphenateCompounds'])) {
 			return $tokens; // abort.
 		}
 
 		// Hyphenate compound words.
-		foreach ( $tokens as $key => $word_token ) {
+		foreach ($tokens as $key => $word_token) {
 			$component_words = [];
-			foreach ( preg_split( '/(-)/', $word_token->value, -1, PREG_SPLIT_NO_EMPTY | PREG_SPLIT_DELIM_CAPTURE ) as $word_part ) {
-				$component_words[] = new Text_Parser\Token( $word_part, Text_Parser\Token::WORD );
+			foreach (preg_split('/(-)/', $word_token->value, -1, PREG_SPLIT_NO_EMPTY | PREG_SPLIT_DELIM_CAPTURE) as $word_part) {
+				$component_words[] = new Text_Parser\Token($word_part, Text_Parser\Token::WORD);
 			}
 
-			$tokens[ $key ] = $word_token->with_value( array_reduce( parent::apply( $component_words, $settings, $is_title, $textnode ), function( $carry, $item ) {
+			$tokens[$key] = $word_token->with_value(array_reduce(parent::apply($component_words, $settings, $is_title, $textnode), function($carry, $item) {
 				return $carry . $item->value;
-			} ) );
+			} ));
 		}
 
 		return $tokens;

src/fixes/token-fixes/class-wrap-urls-fix.php

Spacing +21 added lines, -21 removed lines

@@ -60,8 +60,8 @@ 
 	 * @param Hyphenator_Cache|null $cache           Optional. Default null.
 	 * @param bool                  $feed_compatible Optional. Default false.
 	 */
-	public function __construct( Hyphenator_Cache $cache = null, $feed_compatible = false ) {
-		parent::__construct( $cache, Token_Fix::OTHER, $feed_compatible );
+	public function __construct(Hyphenator_Cache $cache = null, $feed_compatible = false) {
+		parent::__construct($cache, Token_Fix::OTHER, $feed_compatible);
 
 		// Combined URL pattern.
 		$this->url_pattern = '`(?:
@@ -106,59 +106,59 @@ 
 	 *
 	 * @return array An array of tokens.
 	 */
-	public function apply( array $tokens, Settings $settings, $is_title = false, \DOMText $textnode = null ) {
-		if ( empty( $settings['urlWrap'] ) || empty( $settings['urlMinAfterWrap'] ) ) {
+	public function apply(array $tokens, Settings $settings, $is_title = false, \DOMText $textnode = null) {
+		if (empty($settings['urlWrap']) || empty($settings['urlMinAfterWrap'])) {
 			return $tokens;
 		}
 
 		// Test for and parse urls.
-		foreach ( $tokens as $token_index => $text_token ) {
-			if ( preg_match( $this->url_pattern, $text_token->value, $url_match ) ) {
+		foreach ($tokens as $token_index => $text_token) {
+			if (preg_match($this->url_pattern, $text_token->value, $url_match)) {
 
 				// $url_match['schema'] holds "http://".
 				// $url_match['domain'] holds "subdomains.domain.tld".
 				// $url_match['path']   holds the path after the domain.
-				$http = ( $url_match['schema'] ) ? $url_match[1] . U::ZERO_WIDTH_SPACE : '';
+				$http = ($url_match['schema']) ? $url_match[1] . U::ZERO_WIDTH_SPACE : '';
 
-				$domain_parts = preg_split( self::WRAP_URLS_DOMAIN_PARTS, $url_match['domain'], -1, PREG_SPLIT_DELIM_CAPTURE );
+				$domain_parts = preg_split(self::WRAP_URLS_DOMAIN_PARTS, $url_match['domain'], -1, PREG_SPLIT_DELIM_CAPTURE);
 
 				// This is a hack, but it works.
 				// First, we hyphenate each part, we need it formated like a group of words.
 				$parsed_words_like = [];
-				foreach ( $domain_parts as $key => $part ) {
-					$parsed_words_like[ $key ] = new Text_Parser\Token( $part, Text_Parser\Token::OTHER );
+				foreach ($domain_parts as $key => $part) {
+					$parsed_words_like[$key] = new Text_Parser\Token($part, Text_Parser\Token::OTHER);
 				}
 
 				// Do the hyphenation.
-				$parsed_words_like = $this->do_hyphenate( $parsed_words_like, $settings, U::ZERO_WIDTH_SPACE );
+				$parsed_words_like = $this->do_hyphenate($parsed_words_like, $settings, U::ZERO_WIDTH_SPACE);
 
 				// Restore format.
-				foreach ( $parsed_words_like as $key => $parsed_word ) {
+				foreach ($parsed_words_like as $key => $parsed_word) {
 					$value = $parsed_word->value;
 
-					if ( $key > 0 && 1 === strlen( $value ) ) {
-						$domain_parts[ $key ] = U::ZERO_WIDTH_SPACE . $value;
+					if ($key > 0 && 1 === strlen($value)) {
+						$domain_parts[$key] = U::ZERO_WIDTH_SPACE . $value;
 					} else {
-						$domain_parts[ $key ] = $value;
+						$domain_parts[$key] = $value;
 					}
 				}
 
 				// Lastly let's recombine.
-				$domain = implode( $domain_parts );
+				$domain = implode($domain_parts);
 
 				// Break up the URL path to individual characters.
-				$path_parts = str_split( $url_match['path'], 1 );
-				$path_count = count( $path_parts );
+				$path_parts = str_split($url_match['path'], 1);
+				$path_count = count($path_parts);
 				$path = '';
-				foreach ( $path_parts as $index => $path_part ) {
-					if ( 0 === $index || $path_count - $index < $settings['urlMinAfterWrap'] ) {
+				foreach ($path_parts as $index => $path_part) {
+					if (0 === $index || $path_count - $index < $settings['urlMinAfterWrap']) {
 						$path .= $path_part;
 					} else {
 						$path .= U::ZERO_WIDTH_SPACE . $path_part;
 					}
 				}
 
-				$tokens[ $token_index ] = $text_token->with_value( $http . $domain . $path );
+				$tokens[$token_index] = $text_token->with_value($http . $domain . $path);
 			}
 		}
 

src/fixes/token-fixes/class-abstract-token-fix.php

Spacing +2 added lines, -2 removed lines

@@ -58,7 +58,7 @@ 
 	 * @param int  $target          Required.
 	 * @param bool $feed_compatible Optional. Default false.
 	 */
-	protected function __construct( $target, $feed_compatible = false ) {
+	protected function __construct($target, $feed_compatible = false) {
 		$this->target          = $target;
 		$this->feed_compatible = $feed_compatible;
 	}
@@ -73,7 +73,7 @@ 
 	 *
 	 * @return array An array of tokens.
 	 */
-	abstract public function apply( array $tokens, Settings $settings, $is_title = false, \DOMText $textnode = null );
+	abstract public function apply(array $tokens, Settings $settings, $is_title = false, \DOMText $textnode = null);
 
 	/**
 	 * Determines whether the fix should be applied to (RSS) feeds.