matecat / subfiltering

src/Filters/Html/HtmlParser.php

Spacing +70 added lines, -70 removed lines

@@ -71,7 +71,7 @@ 
      *
      * @param Pipeline|null $pipeline
      */
-    public function __construct( ?Pipeline $pipeline = null ) {
+    public function __construct(?Pipeline $pipeline = null) {
         $this->pipeline = $pipeline;
     }
 
@@ -83,10 +83,10 @@ 
      *
      * @throws RuntimeException If the handler does not use the CallbacksHandler trait.
      */
-    public function registerCallbacksHandler( AbstractHandler $class ) {
+    public function registerCallbacksHandler(AbstractHandler $class) {
         //check: $class must use CallbacksHandler trait
-        if ( !in_array( CallbacksHandler::class, array_merge( class_uses( $class ), class_uses( get_parent_class( $class ) ) ) ) ) {
-            throw new RuntimeException( "Class passed to " . __METHOD__ . " must use " . CallbacksHandler::class . " trait." );
+        if (!in_array(CallbacksHandler::class, array_merge(class_uses($class), class_uses(get_parent_class($class))))) {
+            throw new RuntimeException("Class passed to ".__METHOD__." must use ".CallbacksHandler::class." trait.");
         }
         $this->callbacksHandler = $class;
         $this->pipeline         = $this->callbacksHandler->getPipeline();
@@ -103,18 +103,18 @@ 
      * @return mixed             Return value from the handler's method.
      * @throws ReflectionException If a method cannot be found/reflected.
      */
-    public function __call( string $name, array $arguments = [] ) {
+    public function __call(string $name, array $arguments = []) {
 
         // Create a ReflectionMethod instance for the method being called on the callback handler
-        $reflector = new ReflectionMethod( $this->callbacksHandler, $name );
+        $reflector = new ReflectionMethod($this->callbacksHandler, $name);
 
         // If the method is not public, make it accessible
-        if ( !$reflector->isPublic() ) {
-            $reflector->setAccessible( true );
+        if (!$reflector->isPublic()) {
+            $reflector->setAccessible(true);
         }
 
         // Invoke the method on the callback handler with the provided arguments
-        return $reflector->invoke( $this->callbacksHandler, $arguments[ 0 ] ?? null );
+        return $reflector->invoke($this->callbacksHandler, $arguments[0] ?? null);
     }
 
     /**
@@ -128,10 +128,10 @@ 
      * @return string         The processed segment, with tags and text handled appropriately.
      */
 
-    public function transform( string $segment ): string {
+    public function transform(string $segment): string {
         // Split input into Unicode codepoints for accurate char-by-char iteration.
-        $originalSplit = preg_split( '//u', $segment, -1, PREG_SPLIT_NO_EMPTY );
-        if ( empty( $originalSplit ) ) {
+        $originalSplit = preg_split('//u', $segment, -1, PREG_SPLIT_NO_EMPTY);
+        if (empty($originalSplit)) {
             return '';
         }
 
@@ -140,36 +140,36 @@ 
         $plain_text_buffer = '';
         $in_quote_char     = '';
         $output            = '';
-        $charCount         = count( $originalSplit );
+        $charCount         = count($originalSplit);
 
-        foreach ( $originalSplit as $idx => $char ) {
-            switch ( $state ) {
+        foreach ($originalSplit as $idx => $char) {
+            switch ($state) {
                 case static::STATE_PLAINTEXT:
-                    $this->handlePlainTextState( $char, $state, $html_buffer, $plain_text_buffer, $output );
+                    $this->handlePlainTextState($char, $state, $html_buffer, $plain_text_buffer, $output);
                     break;
                 case static::STATE_HTML:
-                    $this->handleHtmlState( $char, $idx, $charCount, $state, $html_buffer, $plain_text_buffer, $output, $in_quote_char );
+                    $this->handleHtmlState($char, $idx, $charCount, $state, $html_buffer, $plain_text_buffer, $output, $in_quote_char);
                     break;
                 case static::STATE_COMMENT:
-                    $this->handleCommentState( $char, $state, $html_buffer, $output );
+                    $this->handleCommentState($char, $state, $html_buffer, $output);
                     break;
                 case static::STATE_JS_CSS:
-                    $this->handleJsCssState( $char, $state, $html_buffer, $output );
+                    $this->handleJsCssState($char, $state, $html_buffer, $output);
                     break;
             }
         }
 
         // HTML Partial at the end, treat as invalid and preserve the string content
-        if ( !empty( $html_buffer ) ) {
-            if ( $this->_isTagValid( $html_buffer ) && null !== $this->pipeline ) {
+        if (!empty($html_buffer)) {
+            if ($this->_isTagValid($html_buffer) && null !== $this->pipeline) {
                 $this->_setSegmentContainsMarkup();
             }
-            $output .= $this->_fixWrongBuffer( $html_buffer );
+            $output .= $this->_fixWrongBuffer($html_buffer);
         }
 
         // Any trailing plain text: finalize it.
-        if ( '' !== $plain_text_buffer ) {
-            $output .= $this->_finalizePlainText( $plain_text_buffer );
+        if ('' !== $plain_text_buffer) {
+            $output .= $this->_finalizePlainText($plain_text_buffer);
         }
 
         return $output;
@@ -178,18 +178,18 @@ 
     /**
      * Handles character processing when in the STATE_PLAINTEXT.
      */
-    private function handlePlainTextState( string $char, int &$state, string &$html_buffer, string &$plain_text_buffer, string &$output ): void {
-        switch ( $char ) {
+    private function handlePlainTextState(string $char, int &$state, string &$html_buffer, string &$plain_text_buffer, string &$output): void {
+        switch ($char) {
             case '<':
                 // Potential new tag starts; finalize plain text so far.
                 $state             = static::STATE_HTML;
                 $html_buffer       .= $char;
-                $output            .= $this->_finalizePlainText( $plain_text_buffer );
+                $output            .= $this->_finalizePlainText($plain_text_buffer);
                 $plain_text_buffer = '';
                 break;
             case '>':
                 // Unescaped '>' in plaintext; treat as literal via error handing.
-                $plain_text_buffer .= $this->_fixWrongBuffer( $char );
+                $plain_text_buffer .= $this->_fixWrongBuffer($char);
                 break;
             default:
                 // Collect as plain text.
@@ -203,27 +203,27 @@ 
      * This method acts as a dispatcher based on the character.
      * Assumes parser state variables (state, html_buffer, etc.) are now class properties.
      */
-    private function handleHtmlState( string $char, int $idx, int $charCount, int &$state, string &$html_buffer, string &$plain_text_buffer, string &$output, string &$in_quote_char ): void {
-        switch ( $char ) {
+    private function handleHtmlState(string $char, int $idx, int $charCount, int &$state, string &$html_buffer, string &$plain_text_buffer, string &$output, string &$in_quote_char): void {
+        switch ($char) {
             case '<':
-                $this->onLessThanInHtml( $char, $output, $html_buffer );
+                $this->onLessThanInHtml($char, $output, $html_buffer);
                 break;
             case '>':
-                $this->onGreaterThanInHtml( $char, $state, $html_buffer, $output, $in_quote_char );
+                $this->onGreaterThanInHtml($char, $state, $html_buffer, $output, $in_quote_char);
                 break;
             case '"':
             case '\'':
-                $this->onQuoteInHtml( $char, $html_buffer, $in_quote_char );
+                $this->onQuoteInHtml($char, $html_buffer, $in_quote_char);
                 break;
             case '-':
-                $this->onDashInHtml( $char, $state, $html_buffer );
+                $this->onDashInHtml($char, $state, $html_buffer);
                 break;
             case ' ':
             case "\n":
-                $this->onWhitespaceInHtml( $char, $state, $html_buffer, $output );
+                $this->onWhitespaceInHtml($char, $state, $html_buffer, $output);
                 break;
             default:
-                $this->onDefaultCharInHtml( $char, $idx, $charCount, $state, $html_buffer, $plain_text_buffer );
+                $this->onDefaultCharInHtml($char, $idx, $charCount, $state, $html_buffer, $plain_text_buffer);
                 break;
         }
     }
@@ -231,37 +231,37 @@ 
     /**
      * Handles the '<' character in the HTML state.
      */
-    private function onLessThanInHtml( string $char, string &$output, string &$html_buffer ): void {
+    private function onLessThanInHtml(string $char, string &$output, string &$html_buffer): void {
         // If we found a second less than symbol, the first one IS NOT a tag.
         // See https://www.w3.org/TR/xml/#charsets
-        $output      .= $this->_fixWrongBuffer( $html_buffer );
+        $output .= $this->_fixWrongBuffer($html_buffer);
         $html_buffer = $char;
     }
 
     /**
      * Handles the '>' character in the HTML state.
      */
-    private function onGreaterThanInHtml( string $char, int &$state, string &$html_buffer, string &$output, string &$in_quote_char ): void {
+    private function onGreaterThanInHtml(string $char, int &$state, string &$html_buffer, string &$output, string &$in_quote_char): void {
         // End of current tag. Special-case for <script> or <style> blocks.
-        if ( $this->isScriptOrStyleTag( $html_buffer ) ) {
+        if ($this->isScriptOrStyleTag($html_buffer)) {
             $html_buffer .= $char;
-            $state       = static::STATE_JS_CSS;
+            $state = static::STATE_JS_CSS;
 
             return;
         }
 
         $in_quote_char = '';
         $state         = static::STATE_PLAINTEXT;
-        $html_buffer   .= $char;
+        $html_buffer .= $char;
 
         // Validate and finalize HTML tag. Invalid tags are corrected/errors handled.
-        if ( $this->_isTagValid( $html_buffer ) ) {
-            $output .= $this->_finalizeMarkupTag( $html_buffer );
-            if ( null !== $this->pipeline ) {
+        if ($this->_isTagValid($html_buffer)) {
+            $output .= $this->_finalizeMarkupTag($html_buffer);
+            if (null !== $this->pipeline) {
                 $this->_setSegmentContainsMarkup();
             }
         } else {
-            $output .= $this->_fixWrongBuffer( $html_buffer );
+            $output .= $this->_fixWrongBuffer($html_buffer);
         }
         $html_buffer = '';
     }
@@ -269,11 +269,11 @@ 
     /**
      * Handles quote characters ('"' or "'") in the HTML state.
      */
-    private function onQuoteInHtml( string $char, string &$html_buffer, string &$in_quote_char ): void {
+    private function onQuoteInHtml(string $char, string &$html_buffer, string &$in_quote_char): void {
         // Track entry/exit into quoted attributes.
-        if ( $char == $in_quote_char ) {
+        if ($char == $in_quote_char) {
             $in_quote_char = ''; // Exiting quote
-        } elseif ( $in_quote_char == '' ) {
+        } elseif ($in_quote_char == '') {
             $in_quote_char = $char; // Entering quote
         }
         $html_buffer .= $char;
@@ -282,9 +282,9 @@ 
     /**
      * Handles the '-' character in the HTML state.
      */
-    private function onDashInHtml( string $char, int &$state, string &$html_buffer ): void {
+    private function onDashInHtml(string $char, int &$state, string &$html_buffer): void {
         // Detect HTML comment opening ('<!--').
-        if ( $html_buffer === '<!-' ) {
+        if ($html_buffer === '<!-') {
             $state = static::STATE_COMMENT;
         }
         $html_buffer .= $char;
@@ -293,13 +293,13 @@ 
     /**
      * Handles whitespace characters in the HTML state.
      */
-    private function onWhitespaceInHtml( string $char, int &$state, string &$html_buffer, string &$output ): void {
+    private function onWhitespaceInHtml(string $char, int &$state, string &$html_buffer, string &$output): void {
         // Space or newline immediately after '<' is invalid.
-        if ( $html_buffer === '<' ) {
+        if ($html_buffer === '<') {
             $state       = static::STATE_PLAINTEXT;
-            $output      .= $this->_fixWrongBuffer( '<' . $char );
+            $output .= $this->_fixWrongBuffer('<'.$char);
             $html_buffer = '';
-            if ( null !== $this->pipeline ) {
+            if (null !== $this->pipeline) {
                 $this->_setSegmentContainsMarkup();
             }
 
@@ -311,12 +311,12 @@ 
     /**
      * Handles any other default character in the HTML state.
      */
-    private function onDefaultCharInHtml( string $char, int $idx, int $charCount, int &$state, string &$html_buffer, string &$plain_text_buffer ): void {
+    private function onDefaultCharInHtml(string $char, int $idx, int $charCount, int &$state, string &$html_buffer, string &$plain_text_buffer): void {
         $html_buffer .= $char;
         // End of input: treat buffer as plain text if not a valid tag.
-        if ( $idx === ( $charCount - 1 ) && !$this->_isTagValid( $html_buffer ) ) {
+        if ($idx === ($charCount - 1) && !$this->_isTagValid($html_buffer)) {
             $state             = static::STATE_PLAINTEXT; // Error: not a valid tag
-            $plain_text_buffer .= $this->_fixWrongBuffer( $html_buffer );
+            $plain_text_buffer .= $this->_fixWrongBuffer($html_buffer);
             $html_buffer       = '';
         }
     }
@@ -324,14 +324,14 @@ 
     /**
      * Handles character processing when in the STATE_COMMENT.
      */
-    private function handleCommentState( string $char, int &$state, string &$html_buffer, string &$output ): void {
+    private function handleCommentState(string $char, int &$state, string &$html_buffer, string &$output): void {
         $html_buffer .= $char;
         // Check for the end of a comment: '-->'
-        if ( $char === '>' && substr( $html_buffer, -3 ) === '-->' ) {
+        if ($char === '>' && substr($html_buffer, -3) === '-->') {
             $state       = static::STATE_PLAINTEXT;
-            $output      .= $this->_finalizeScriptTag( $html_buffer );
+            $output .= $this->_finalizeScriptTag($html_buffer);
             $html_buffer = '';
-            if ( null !== $this->pipeline ) {
+            if (null !== $this->pipeline) {
                 $this->_setSegmentContainsMarkup();
             }
         }
@@ -340,16 +340,16 @@ 
     /**
      * Handles character processing when in the STATE_JS_CSS.
      */
-    private function handleJsCssState( string $char, int &$state, string &$html_buffer, string &$output ): void {
+    private function handleJsCssState(string $char, int &$state, string &$html_buffer, string &$output): void {
         $html_buffer .= $char;
         // Detect close: e.g., '</script>' or '</style>'
-        if ( $char === '>' ) {
-            if ( in_array( substr( $html_buffer, -6 ), [ 'cript>', 'style>' ], true ) ) {
+        if ($char === '>') {
+            if (in_array(substr($html_buffer, -6), ['cript>', 'style>'], true)) {
                 $state = static::STATE_PLAINTEXT;
-                $this->_isTagValid( $html_buffer );
-                $output      .= $this->_finalizeScriptTag( $html_buffer );
+                $this->_isTagValid($html_buffer);
+                $output .= $this->_finalizeScriptTag($html_buffer);
                 $html_buffer = '';
-                if ( null !== $this->pipeline ) {
+                if (null !== $this->pipeline) {
                     $this->_setSegmentContainsMarkup();
                 }
             }
@@ -359,9 +359,9 @@ 
     /**
      * Checks if the buffered HTML is the beginning of a script or style tag.
      */
-    private function isScriptOrStyleTag( string $html_buffer ): bool {
+    private function isScriptOrStyleTag(string $html_buffer): bool {
         // A tag starts with '<script' or '<style'. This also covers variants with spaces or attributes.
-        return in_array( substr( $html_buffer, 0, 8 ), [ '<script ', '<style', '<script', '<style ' ] );
+        return in_array(substr($html_buffer, 0, 8), ['<script ', '<style', '<script', '<style ']);
     }
 
 }