mundschenk-at / php-typography

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

Spacing +9 added lines, -9 removed lines

@@ -45,24 +45,24 @@
 	 * @param Settings $settings Required.
 	 * @param bool     $is_title Optional. Default false.
 	 */
-	public function apply( \DOMText $textnode, Settings $settings, $is_title = false ) {
-		if ( empty( $settings['smartDiacritics'] ) ) {
+	public function apply(\DOMText $textnode, Settings $settings, $is_title = false) {
+		if (empty($settings['smartDiacritics'])) {
 			return; // abort.
 		}
 
-		if ( ! empty( $settings['diacriticReplacement'] ) &&
-			 ! empty( $settings['diacriticReplacement']['patterns'] ) &&
-			 ! empty( $settings['diacriticReplacement']['replacements'] ) ) {
+		if ( ! empty($settings['diacriticReplacement']) &&
+			 ! empty($settings['diacriticReplacement']['patterns']) &&
+			 ! empty($settings['diacriticReplacement']['replacements'])) {
 
 			// Uses "word" => "replacement" pairs from an array to make fast preg_* replacements.
 			$replacements = $settings['diacriticReplacement']['replacements'];
-			$textnode->data = preg_replace_callback( $settings['diacriticReplacement']['patterns'], function( $match ) use ( $replacements ) {
-				if ( isset( $replacements[ $match[0] ] ) ) {
-					return $replacements[ $match[0] ];
+			$textnode->data = preg_replace_callback($settings['diacriticReplacement']['patterns'], function($match) use ($replacements) {
+				if (isset($replacements[$match[0]])) {
+					return $replacements[$match[0]];
 				} else {
 					return $match[0];
 				}
-			}, $textnode->data );
+			}, $textnode->data);
 		}
 	}
 }

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

Spacing +8 added lines, -8 removed lines

@@ -51,7 +51,7 @@ 
 	 *
 	 * @param bool $feed_compatible Optional. Default false.
 	 */
-	public function __construct( $feed_compatible = false ) {
+	public function __construct($feed_compatible = false) {
 		$this->feed_compatible = $feed_compatible;
 	}
 
@@ -64,7 +64,7 @@ 
 	 *
 	 * @return void
 	 */
-	abstract public function apply( \DOMText $textnode, Settings $settings, $is_title = false );
+	abstract public function apply(\DOMText $textnode, Settings $settings, $is_title = false);
 
 	/**
 	 * Determines whether the fix should be applied to (RSS) feeds.
@@ -86,18 +86,18 @@ 
 	 *
 	 * @return string            The string without `$prev_char` and `$next_char`.
 	 */
-	protected static function remove_adjacent_characters( $string, $prev_char = '', $next_char = '' ) {
+	protected static function remove_adjacent_characters($string, $prev_char = '', $next_char = '') {
 		// Use the most efficient string functions.
-		$func = Strings::functions( $string );
+		$func = Strings::functions($string);
 
 		// Remove previous character.
-		if ( '' !== $prev_char ) {
-			$string = $func['substr']( $string, 1, $func['strlen']( $string ) );
+		if ('' !== $prev_char) {
+			$string = $func['substr']($string, 1, $func['strlen']($string));
 		}
 
 		// Remove next character.
-		if ( '' !== $next_char ) {
-			$string = $func['substr']( $string, 0, $func['strlen']( $string ) - 1 );
+		if ('' !== $next_char) {
+			$string = $func['substr']($string, 0, $func['strlen']($string) - 1);
 		}
 
 		return $string;

src/fixes/node-fixes/class-style-numbers-fix.php

Spacing +2 added lines, -2 removed lines

@@ -53,7 +53,7 @@
 	 * @param string $css_class       HTML class used in markup.
 	 * @param bool   $feed_compatible Optional. Default false.
 	 */
-	public function __construct( $css_class, $feed_compatible = false ) {
-		parent::__construct( self::REGEX, self::SETTINGS_SWITCH, $css_class, $feed_compatible );
+	public function __construct($css_class, $feed_compatible = false) {
+		parent::__construct(self::REGEX, self::SETTINGS_SWITCH, $css_class, $feed_compatible);
 	}
 }

src/fixes/node-fixes/class-simple-regex-replacement-fix.php

Spacing +5 added lines, -5 removed lines

@@ -69,8 +69,8 @@ 
 	 * @param string $settings_switch On/off switch for fix.
 	 * @param bool   $feed_compatible Optional. Default false.
 	 */
-	public function __construct( $regex, $replacement, $settings_switch, $feed_compatible = false ) {
-		parent::__construct( $feed_compatible );
+	public function __construct($regex, $replacement, $settings_switch, $feed_compatible = false) {
+		parent::__construct($feed_compatible);
 
 		$this->regex           = $regex;
 		$this->settings_switch = $settings_switch;
@@ -84,11 +84,11 @@ 
 	 * @param Settings $settings Required.
 	 * @param bool     $is_title Optional. Default false.
 	 */
-	public function apply( \DOMText $textnode, Settings $settings, $is_title = false ) {
-		if ( empty( $settings[ $this->settings_switch ] ) ) {
+	public function apply(\DOMText $textnode, Settings $settings, $is_title = false) {
+		if (empty($settings[$this->settings_switch])) {
 			return;
 		}
 
-		$textnode->data = preg_replace( $this->regex, $this->replacement, $textnode->data );
+		$textnode->data = preg_replace($this->regex, $this->replacement, $textnode->data);
 	}
 }

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

Spacing +9 added lines, -9 removed lines

@@ -49,22 +49,22 @@
 	 * @param Settings $settings Required.
 	 * @param bool     $is_title Optional. Default false.
 	 */
-	public function apply( \DOMText $textnode, Settings $settings, $is_title = false ) {
-		if ( empty( $settings['smartMarks'] ) ) {
+	public function apply(\DOMText $textnode, Settings $settings, $is_title = false) {
+		if (empty($settings['smartMarks'])) {
 			return;
 		}
 
 		// Escape usage of "501(c)(1...29)" (US non-profit).
-		$textnode->data = preg_replace( self::ESCAPE_501C, '$1' . RE::ESCAPE_MARKER . '$2' . RE::ESCAPE_MARKER . '$3', $textnode->data );
+		$textnode->data = preg_replace(self::ESCAPE_501C, '$1' . RE::ESCAPE_MARKER . '$2' . RE::ESCAPE_MARKER . '$3', $textnode->data);
 
 		// Replace marks.
-		$textnode->data = str_replace( [ '(c)', '(C)' ],   U::COPYRIGHT,      $textnode->data );
-		$textnode->data = str_replace( [ '(r)', '(R)' ],   U::REGISTERED_MARK, $textnode->data );
-		$textnode->data = str_replace( [ '(p)', '(P)' ],   U::SOUND_COPY_MARK,  $textnode->data );
-		$textnode->data = str_replace( [ '(sm)', '(SM)' ], U::SERVICE_MARK,    $textnode->data );
-		$textnode->data = str_replace( [ '(tm)', '(TM)' ], U::TRADE_MARK,      $textnode->data );
+		$textnode->data = str_replace(['(c)', '(C)'], U::COPYRIGHT, $textnode->data);
+		$textnode->data = str_replace(['(r)', '(R)'], U::REGISTERED_MARK, $textnode->data);
+		$textnode->data = str_replace(['(p)', '(P)'], U::SOUND_COPY_MARK, $textnode->data);
+		$textnode->data = str_replace(['(sm)', '(SM)'], U::SERVICE_MARK, $textnode->data);
+		$textnode->data = str_replace(['(tm)', '(TM)'], U::TRADE_MARK, $textnode->data);
 
 		// Un-escape escaped sequences.
-		$textnode->data = str_replace( RE::ESCAPE_MARKER, '', $textnode->data );
+		$textnode->data = str_replace(RE::ESCAPE_MARKER, '', $textnode->data);
 	}
 }

src/fixes/node-fixes/class-simple-style-fix.php

Indentation -1 removed lines

@@ -40,7 +40,6 @@
 
 	/**
 	 * The setting string used to enable/disable the fix (e.g. 'styleAmpersands').
-
 	 * @var string
 	 */
 	protected $settings_switch;

Spacing +5 added lines, -5 removed lines

@@ -69,8 +69,8 @@ 
 	 * @param string $css_class       HTML class used in markup.
 	 * @param bool   $feed_compatible Optional. Default false.
 	 */
-	public function __construct( $regex, $settings_switch, $css_class, $feed_compatible = false ) {
-		parent::__construct( $css_class, $feed_compatible );
+	public function __construct($regex, $settings_switch, $css_class, $feed_compatible = false) {
+		parent::__construct($css_class, $feed_compatible);
 
 		$this->regex = $regex;
 		$this->settings_switch = $settings_switch;
@@ -84,11 +84,11 @@ 
 	 * @param Settings $settings Required.
 	 * @param bool     $is_title Optional. Default false.
 	 */
-	public function apply_internal( \DOMText $textnode, Settings $settings, $is_title = false ) {
-		if ( empty( $settings[ $this->settings_switch ] ) ) {
+	public function apply_internal(\DOMText $textnode, Settings $settings, $is_title = false) {
+		if (empty($settings[$this->settings_switch])) {
 			return;
 		}
 
-		$textnode->data = preg_replace( $this->regex, '<span class="' . $this->css_class . '">$1</span>', $textnode->data );
+		$textnode->data = preg_replace($this->regex, '<span class="' . $this->css_class . '">$1</span>', $textnode->data);
 	}
 }

src/fixes/node-fixes/class-dash-spacing-fix.php

Spacing +8 added lines, -8 removed lines

@@ -103,22 +103,22 @@ 
 	 * @param Settings $settings Required.
 	 * @param bool     $is_title Optional. Default false.
 	 */
-	public function apply( \DOMText $textnode, Settings $settings, $is_title = false ) {
-		if ( empty( $settings['dashSpacing'] ) ) {
+	public function apply(\DOMText $textnode, Settings $settings, $is_title = false) {
+		if (empty($settings['dashSpacing'])) {
 			return;
 		}
 
 		// Various special characters and regular expressions.
 		$s = $settings->dash_style();
 
-		if ( $s != $this->cached_dash_style ) {
-			$this->update_dash_spacing_regex( $s->parenthetical_dash(), $s->parenthetical_space(), $s->interval_dash(), $s->interval_space() );
+		if ($s != $this->cached_dash_style) {
+			$this->update_dash_spacing_regex($s->parenthetical_dash(), $s->parenthetical_space(), $s->interval_dash(), $s->interval_space());
 			$this->cached_dash_style = $s;
 		}
 
-		$textnode->data = preg_replace( self::EM_DASH_SPACING,             $this->em_dash_replacement,            $textnode->data );
-		$textnode->data = preg_replace( $this->parenthetical_dash_spacing, $this->parenthetical_dash_replacement, $textnode->data );
-		$textnode->data = preg_replace( $this->interval_dash_spacing,      $this->interval_dash_replacement,      $textnode->data );
+		$textnode->data = preg_replace(self::EM_DASH_SPACING, $this->em_dash_replacement, $textnode->data);
+		$textnode->data = preg_replace($this->parenthetical_dash_spacing, $this->parenthetical_dash_replacement, $textnode->data);
+		$textnode->data = preg_replace($this->interval_dash_spacing, $this->interval_dash_replacement, $textnode->data);
 	}
 
 	/**
@@ -129,7 +129,7 @@ 
 	 * @param string $interval            The dash character used for interval dashes.
 	 * @param string $interval_space      The space character used around interval dashes.
 	 */
-	private function update_dash_spacing_regex( $parenthetical, $parenthetical_space, $interval, $interval_space ) {
+	private function update_dash_spacing_regex($parenthetical, $parenthetical_space, $interval, $interval_space) {
 		$this->parenthetical_dash_spacing = "/
 			(?:
 				\s

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);
 	}
 }