matecat / subfiltering

src/Filters/Html/HtmlParser.php

Spacing +58 added lines, -58 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.
@@ -201,50 +201,50 @@ 
     /**
      * Handles character processing when in the STATE_HTML.
      */
-    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 '<':
                 // 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;
                 break;
             case '>':
                 // 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;
                     break;
                 }
 
                 $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 = '';
                 break;
             case '"':
             case '\'':
                 // 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;
                 break;
             case '-':
                 // Detect HTML comment opening ('<!--').
-                if ( $html_buffer === '<!-' ) {
+                if ($html_buffer === '<!-') {
                     $state = static::STATE_COMMENT;
                 }
                 $html_buffer .= $char;
@@ -252,11 +252,11 @@ 
             case ' ':
             case "\n":
                 // 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();
                     }
                     break;
@@ -266,9 +266,9 @@ 
             default:
                 $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       = '';
                 }
                 break;
@@ -278,14 +278,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();
             }
         }
@@ -294,16 +294,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();
                 }
             }
@@ -313,8 +313,8 @@ 
     /**
      * 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 ']);
     }
 }

src/Filters/DataRefRestore.php

Spacing +22 added lines, -22 removed lines

@@ -25,23 +25,23 @@ 
      * @inheritDoc
      * @throws Exception
      */
-    public function transform( string $segment ): string {
+    public function transform(string $segment): string {
 
-        if ( empty( $this->dataRefMap ) ) {
+        if (empty($this->dataRefMap)) {
             $this->dataRefMap = $this->pipeline->getDataRefMap();
         }
 
-        if ( empty( $this->dataRefMap ) ) {
-            $segment = $this->restoreXliffPhTagsFromMatecatPhTags( $segment );
+        if (empty($this->dataRefMap)) {
+            $segment = $this->restoreXliffPhTagsFromMatecatPhTags($segment);
 
-            return $this->restoreXliffPcTagsFromMatecatPhTags( $segment );
+            return $this->restoreXliffPcTagsFromMatecatPhTags($segment);
         }
 
-        $dataRefReplacer = new DataRefReplacer( $this->dataRefMap );
-        $segment         = $dataRefReplacer->restore( $segment );
-        $segment         = $this->restoreXliffPhTagsFromMatecatPhTags( $segment );
+        $dataRefReplacer = new DataRefReplacer($this->dataRefMap);
+        $segment         = $dataRefReplacer->restore($segment);
+        $segment         = $this->restoreXliffPhTagsFromMatecatPhTags($segment);
 
-        return $this->restoreXliffPcTagsFromMatecatPhTags( $segment );
+        return $this->restoreXliffPcTagsFromMatecatPhTags($segment);
     }
 
     /**
@@ -60,15 +60,15 @@ 
      *
      * @return string
      */
-    private function restoreXliffPhTagsFromMatecatPhTags( string $segment ): string {
-        preg_match_all( '|<ph[^>]+? ctype="' . CTypeEnum::ORIGINAL_PH_OR_NOT_DATA_REF . '" equiv-text="base64:(.*?)"/>|iu', $segment, $matches );
+    private function restoreXliffPhTagsFromMatecatPhTags(string $segment): string {
+        preg_match_all('|<ph[^>]+? ctype="'.CTypeEnum::ORIGINAL_PH_OR_NOT_DATA_REF.'" equiv-text="base64:(.*?)"/>|iu', $segment, $matches);
 
-        if ( empty( $matches[ 0 ] ) ) {
+        if (empty($matches[0])) {
             return $segment;
         }
 
-        foreach ( $matches[ 0 ] as $index => $match ) {
-            $segment = str_replace( $match, base64_decode( $matches[ 1 ][ $index ] ), $segment );
+        foreach ($matches[0] as $index => $match) {
+            $segment = str_replace($match, base64_decode($matches[1][$index]), $segment);
         }
 
         return $segment;
@@ -90,21 +90,21 @@ 
      *
      * @return string
      */
-    private function restoreXliffPcTagsFromMatecatPhTags( string $segment ): string {
+    private function restoreXliffPcTagsFromMatecatPhTags(string $segment): string {
 
         $matches = [];
-        preg_match_all( '|<ph[^>]+? ctype="' . CTypeEnum::ORIGINAL_PC_OPEN_NO_DATA_REF . '" equiv-text="base64:(.*?)"/>|iu', $segment, $open_matches );
-        preg_match_all( '|<ph[^>]+? ctype="' . CTypeEnum::ORIGINAL_PC_CLOSE_NO_DATA_REF . '" equiv-text="base64:(.*?)"/>|iu', $segment, $close_matches );
+        preg_match_all('|<ph[^>]+? ctype="'.CTypeEnum::ORIGINAL_PC_OPEN_NO_DATA_REF.'" equiv-text="base64:(.*?)"/>|iu', $segment, $open_matches);
+        preg_match_all('|<ph[^>]+? ctype="'.CTypeEnum::ORIGINAL_PC_CLOSE_NO_DATA_REF.'" equiv-text="base64:(.*?)"/>|iu', $segment, $close_matches);
 
-        $matches[ 0 ] = array_merge( $open_matches[ 0 ], $close_matches[ 0 ] );
-        $matches[ 1 ] = array_merge( $open_matches[ 1 ], $close_matches[ 1 ] );
+        $matches[0] = array_merge($open_matches[0], $close_matches[0]);
+        $matches[1] = array_merge($open_matches[1], $close_matches[1]);
 
-        if ( empty( $matches[ 0 ] ) ) {
+        if (empty($matches[0])) {
             return $segment;
         }
 
-        foreach ( $matches[ 0 ] as $index => $match ) {
-            $segment = str_replace( $match, base64_decode( $matches[ 1 ][ $index ] ), $segment );
+        foreach ($matches[0] as $index => $match) {
+            $segment = str_replace($match, base64_decode($matches[1][$index]), $segment);
         }
 
         return $segment;

src/Utils/Utils.php

Spacing +21 added lines, -21 removed lines

@@ -9,13 +9,13 @@ 
      *
      * @return bool
      */
-    public static function array_is_list( $array ): bool {
+    public static function array_is_list($array): bool {
 
-        if ( $array === [] ) {
+        if ($array === []) {
             return true;
         }
 
-        return array_keys( $array ) === range( 0, count( $array ) - 1 );
+        return array_keys($array) === range(0, count($array) - 1);
 
     }
 
@@ -25,8 +25,8 @@ 
      *
      * @return bool
      */
-    public static function contains( string $needle, string $haystack ): bool {
-        return strpos( $haystack, $needle ) !== false;
+    public static function contains(string $needle, string $haystack): bool {
+        return strpos($haystack, $needle) !== false;
     }
 
     /**
@@ -39,22 +39,22 @@ 
      * @return int
      *
      */
-    public static function fastUnicode2ord( string $mb_char ) {
-        switch ( strlen( $mb_char ) ) {
+    public static function fastUnicode2ord(string $mb_char) {
+        switch (strlen($mb_char)) {
             case 1:
-                return ord( $mb_char );
+                return ord($mb_char);
             case 2:
-                return ( ord( $mb_char[ 0 ] ) - 0xC0 ) * 0x40 +
-                        ord( $mb_char[ 1 ] ) - 0x80;
+                return (ord($mb_char[0]) - 0xC0) * 0x40 +
+                        ord($mb_char[1]) - 0x80;
             case 3:
-                return ( ord( $mb_char[ 0 ] ) - 0xE0 ) * 0x1000 +
-                        ( ord( $mb_char[ 1 ] ) - 0x80 ) * 0x40 +
-                        ord( $mb_char[ 2 ] ) - 0x80;
+                return (ord($mb_char[0]) - 0xE0) * 0x1000 +
+                        (ord($mb_char[1]) - 0x80) * 0x40 +
+                        ord($mb_char[2]) - 0x80;
             case 4:
-                return ( ord( $mb_char[ 0 ] ) - 0xF0 ) * 0x40000 +
-                        ( ord( $mb_char[ 1 ] ) - 0x80 ) * 0x1000 +
-                        ( ord( $mb_char[ 2 ] ) - 0x80 ) * 0x40 +
-                        ord( $mb_char[ 3 ] ) - 0x80;
+                return (ord($mb_char[0]) - 0xF0) * 0x40000 +
+                        (ord($mb_char[1]) - 0x80) * 0x1000 +
+                        (ord($mb_char[2]) - 0x80) * 0x40 +
+                        ord($mb_char[3]) - 0x80;
         }
 
         return 20; //as default, return a space (should never happen)
@@ -65,8 +65,8 @@ 
      *
      * @return string
      */
-    public static function htmlentitiesFromUnicode( array $str ): string {
-        return "&#" . self::fastUnicode2ord( $str[ 1 ] ) . ";";
+    public static function htmlentitiesFromUnicode(array $str): string {
+        return "&#".self::fastUnicode2ord($str[1]).";";
     }
 
     /**
@@ -79,8 +79,8 @@ 
      *
      * @return string
      */
-    public static function unicode2chr( int $o ): string {
-        return (string)mb_convert_encoding( '&#' . $o . ';', 'UTF-8', 'HTML-ENTITIES' );
+    public static function unicode2chr(int $o): string {
+        return (string) mb_convert_encoding('&#'.$o.';', 'UTF-8', 'HTML-ENTITIES');
     }
 
 }
\ No newline at end of file

src/Utils/DataRefReplacer.php

Spacing +92 added lines, -92 removed lines

@@ -27,8 +27,8 @@ 
      *
      * @param array $map
      */
-    public function __construct( array $map = [] ) {
-        $this->map = Map::instance( $this->sanitizeMap( $map ) );
+    public function __construct(array $map = []) {
+        $this->map = Map::instance($this->sanitizeMap($map));
     }
 
     /**
@@ -42,42 +42,42 @@ 
      *
      * @return string
      */
-    public function replace( string $string ): string {
+    public function replace(string $string): string {
 
         // if the map is empty
         // or the string has not a dataRef attribute
         // return string as is
-        if ( $this->map->isEmpty() || !$this->hasAnyDataRefAttribute( $string ) ) {
+        if ($this->map->isEmpty() || !$this->hasAnyDataRefAttribute($string)) {
             return $string;
         }
 
         // try not to throw exception for wrong segments with opening tags and no closing
         try {
 
-            $html = XmlParser::parse( $string, true );
+            $html = XmlParser::parse($string, true);
 
             $dataRefEndMap = new ArrayList();
 
-            foreach ( $html as $node ) {
+            foreach ($html as $node) {
 
                 // 1. Replace <ph>|<sc>|<ec> tags
-                $string = $this->recursiveTransformDataRefToPhTag( $node, $string );
+                $string = $this->recursiveTransformDataRefToPhTag($node, $string);
 
                 // 2. Replace self-closed <pc dataRefStart="xyz" /> tags
-                $string = $this->recursiveReplaceSelfClosedPcTags( $node, $string );
+                $string = $this->recursiveReplaceSelfClosedPcTags($node, $string);
 
                 // 3. Build the DataRefEndMap needed by replaceClosingPcTags function
                 // (needed for correct handling of </pc> closing tags)
                 // make this inline with one foreach cycle
-                $this->extractDataRefMapRecursively( $node, $string, $dataRefEndMap );
+                $this->extractDataRefMapRecursively($node, $string, $dataRefEndMap);
 
             }
 
             // 4. replace pc tags
-            $string = $this->replaceOpeningPcTags( $string );
-            $string = $this->replaceClosingPcTags( $string, $dataRefEndMap );
+            $string = $this->replaceOpeningPcTags($string);
+            $string = $this->replaceClosingPcTags($string, $dataRefEndMap);
 
-        } catch ( Exception $ignore ) {
+        } catch (Exception $ignore) {
             // if something fails here, do not throw an exception and return the original string instead
 //            var_dump( $ignore );
         } finally {
@@ -91,8 +91,8 @@ 
      *
      * @return bool
      */
-    private function hasAnyDataRefAttribute( string $string ): bool {
-        return (bool)preg_match( '/(dataRef|dataRefStart|dataRefEnd)=[\'"].*?[\'"]/', $string );
+    private function hasAnyDataRefAttribute(string $string): bool {
+        return (bool) preg_match('/(dataRef|dataRefStart|dataRefEnd)=[\'"].*?[\'"]/', $string);
     }
 
     /**
@@ -108,18 +108,18 @@ 
      *
      * @return string
      */
-    private function recursiveTransformDataRefToPhTag( object $node, string $string ): string {
+    private function recursiveTransformDataRefToPhTag(object $node, string $string): string {
 
-        if ( $node->has_children ) {
+        if ($node->has_children) {
 
-            foreach ( $node->inner_html as $childNode ) {
-                $string = $this->recursiveTransformDataRefToPhTag( $childNode, $string );
+            foreach ($node->inner_html as $childNode) {
+                $string = $this->recursiveTransformDataRefToPhTag($childNode, $string);
             }
 
         } else {
 
             // accept only those tags
-            switch ( $node->tagName ) {
+            switch ($node->tagName) {
                 case 'ph':
                     $ctype = CTypeEnum::PH_DATA_REF;
                     break;
@@ -134,17 +134,17 @@ 
             }
 
             // if isset a value in the map, proceed with conversion, otherwise skip
-            $attributesMap = Map::instance( $node->attributes );
-            if ( !$this->map->get( $attributesMap->get( 'dataRef' ) ) ) {
+            $attributesMap = Map::instance($node->attributes);
+            if (!$this->map->get($attributesMap->get('dataRef'))) {
                 return $string;
             }
 
-            $dataRefName = $node->attributes[ 'dataRef' ];   // map identifier. Eg: source1
+            $dataRefName = $node->attributes['dataRef']; // map identifier. Eg: source1
 
             return $this->replaceNewTagString(
                     $node->node,
-                    $attributesMap->getOrDefault( 'id', $dataRefName ),
-                    $this->map->getOrDefault( $node->attributes[ 'dataRef' ], 'NULL' ),
+                    $attributesMap->getOrDefault('id', $dataRefName),
+                    $this->map->getOrDefault($node->attributes['dataRef'], 'NULL'),
                     $ctype,
                     $string,
                     null
@@ -162,11 +162,11 @@ 
      *
      * @return array
      */
-    private function sanitizeMap( array $map ): array {
+    private function sanitizeMap(array $map): array {
 
-        foreach ( $map as $name => $value ) {
-            if ( is_null( $value ) || $value === '' ) {
-                $map[ $name ] = 'NULL';
+        foreach ($map as $name => $value) {
+            if (is_null($value) || $value === '') {
+                $map[$name] = 'NULL';
             }
         }
 
@@ -182,23 +182,23 @@ 
      * @throws InvalidXmlException
      * @throws XmlParsingException
      */
-    private function recursiveReplaceSelfClosedPcTags( object $node, string $string ): string {
+    private function recursiveReplaceSelfClosedPcTags(object $node, string $string): string {
 
-        if ( $node->has_children ) {
+        if ($node->has_children) {
 
-            foreach ( $node->inner_html as $childNode ) {
-                $string = $this->recursiveReplaceSelfClosedPcTags( $childNode, $string );
+            foreach ($node->inner_html as $childNode) {
+                $string = $this->recursiveReplaceSelfClosedPcTags($childNode, $string);
             }
 
-        } elseif ( $node->tagName == 'pc' && $node->self_closed === true ) {
+        } elseif ($node->tagName == 'pc' && $node->self_closed === true) {
 
-            $attributesMap = Map::instance( $node->attributes );
+            $attributesMap = Map::instance($node->attributes);
 
-            if ( $dataRefStartValue = $this->map->get( $node->attributes[ 'dataRefStart' ] ) ) {
+            if ($dataRefStartValue = $this->map->get($node->attributes['dataRefStart'])) {
 
                 $string = $this->replaceNewTagString(
                         $node->node,
-                        $attributesMap->get( 'id' ),
+                        $attributesMap->get('id'),
                         $dataRefStartValue,
                         CTypeEnum::PC_SELF_CLOSE_DATA_REF,
                         $string
@@ -219,12 +219,12 @@ 
      * @param string    $completeString
      * @param ArrayList $dataRefEndMap
      */
-    private function extractDataRefMapRecursively( object $node, string $completeString, ArrayList $dataRefEndMap ) {
+    private function extractDataRefMapRecursively(object $node, string $completeString, ArrayList $dataRefEndMap) {
 
         // we have to build the map for the closing pc tag, so get the children first
-        if ( $node->has_children ) {
-            foreach ( $node->inner_html as $nestedNode ) {
-                $this->extractDataRefMapRecursively( $nestedNode, $completeString, $dataRefEndMap );
+        if ($node->has_children) {
+            foreach ($node->inner_html as $nestedNode) {
+                $this->extractDataRefMapRecursively($nestedNode, $completeString, $dataRefEndMap);
             }
         }
 
@@ -237,15 +237,15 @@ 
         //   becomes
         // <pc id="source5" dataRefStart="source5"></pc>
         //
-        $isATagPairWithEmptyTextNode = strpos( $completeString, substr( $node->node, 0, -2 ) . '></pc>' ) !== false;
+        $isATagPairWithEmptyTextNode = strpos($completeString, substr($node->node, 0, -2).'></pc>') !== false;
 
-        if ( $node->tagName === 'pc' && ( $node->self_closed === false || $isATagPairWithEmptyTextNode ) ) {
+        if ($node->tagName === 'pc' && ($node->self_closed === false || $isATagPairWithEmptyTextNode)) {
 
-            $attributesMap = Map::instance( $node->attributes );
-            $dataRefEnd    = $attributesMap->getOrDefault( 'dataRefEnd', $attributesMap->get( 'dataRefStart' ) );
+            $attributesMap = Map::instance($node->attributes);
+            $dataRefEnd    = $attributesMap->getOrDefault('dataRefEnd', $attributesMap->get('dataRefStart'));
 
             $dataRefEndMap[] = [
-                    'id'         => $attributesMap->get( 'id' ),
+                    'id'         => $attributesMap->get('id'),
                     'dataRefEnd' => $dataRefEnd,
             ];
 
@@ -263,31 +263,31 @@ 
      * @throws InvalidXmlException
      * @throws XmlParsingException
      */
-    private function replaceOpeningPcTags( string $string ): string {
+    private function replaceOpeningPcTags(string $string): string {
 
-        preg_match_all( '|<pc ([^>/]+?)>|iu', $string, $openingPcMatches );
+        preg_match_all('|<pc ([^>/]+?)>|iu', $string, $openingPcMatches);
 
-        foreach ( $openingPcMatches[ 0 ] as $match ) {
+        foreach ($openingPcMatches[0] as $match) {
 
-            $node = XmlParser::parse( $match . '</pc>', true )[ 0 ]; // add a closing tag to not break XML integrity
+            $node = XmlParser::parse($match.'</pc>', true)[0]; // add a closing tag to not break XML integrity
 
             // CASE 1 - Missing `dataRefStart`
-            if ( isset( $node->attributes[ 'dataRefEnd' ] ) && !isset( $node->attributes[ 'dataRefStart' ] ) ) {
-                $node->attributes[ 'dataRefStart' ] = $node->attributes[ 'dataRefEnd' ];
+            if (isset($node->attributes['dataRefEnd']) && !isset($node->attributes['dataRefStart'])) {
+                $node->attributes['dataRefStart'] = $node->attributes['dataRefEnd'];
             }
 
             // CASE 2 - Missing `dataRefEnd`
-            if ( isset( $node->attributes[ 'dataRefStart' ] ) && !isset( $node->attributes[ 'dataRefEnd' ] ) ) {
-                $node->attributes[ 'dataRefEnd' ] = $node->attributes[ 'dataRefStart' ];
+            if (isset($node->attributes['dataRefStart']) && !isset($node->attributes['dataRefEnd'])) {
+                $node->attributes['dataRefEnd'] = $node->attributes['dataRefStart'];
             }
 
-            if ( isset( $node->attributes[ 'dataRefStart' ] ) ) {
+            if (isset($node->attributes['dataRefStart'])) {
 
-                $attributesMap = Map::instance( $node->attributes );
+                $attributesMap = Map::instance($node->attributes);
                 $string        = $this->replaceNewTagString(
                         $match,
-                        $attributesMap->get( 'id' ),
-                        $this->map->getOrDefault( $node->attributes[ 'dataRefStart' ], 'NULL' ),
+                        $attributesMap->get('id'),
+                        $this->map->getOrDefault($node->attributes['dataRefStart'], 'NULL'),
                         CTypeEnum::PC_OPEN_DATA_REF,
                         $string
                 );
@@ -307,31 +307,31 @@ 
      *
      * @return string
      */
-    private function replaceClosingPcTags( string $string, ArrayList $dataRefEndMap ): string {
+    private function replaceClosingPcTags(string $string, ArrayList $dataRefEndMap): string {
 
-        preg_match_all( '|</pc>|iu', $string, $closingPcMatches, PREG_OFFSET_CAPTURE );
+        preg_match_all('|</pc>|iu', $string, $closingPcMatches, PREG_OFFSET_CAPTURE);
         $delta = 0;
 
-        foreach ( $closingPcMatches[ 0 ] as $index => $match ) {
+        foreach ($closingPcMatches[0] as $index => $match) {
 
-            $offset = $match[ 1 ];
+            $offset = $match[1];
             $length = 5; // strlen of '</pc>'
 
-            $attr = $dataRefEndMap->get( $index );
-            if ( !empty( $attr ) && isset( $attr[ 'dataRefEnd' ] ) ) {
+            $attr = $dataRefEndMap->get($index);
+            if (!empty($attr) && isset($attr['dataRefEnd'])) {
 
                 // conversion for opening <pc> tag
                 $completeTag = $this->getNewTagString(
                         '</pc>',
-                        $attr[ 'id' ],
-                        $this->map->getOrDefault( $attr[ 'dataRefEnd' ], 'NULL' ),
+                        $attr['id'],
+                        $this->map->getOrDefault($attr['dataRefEnd'], 'NULL'),
                         CTypeEnum::PC_CLOSE_DATA_REF,
                         '_2'
                 );
 
-                $realOffset = ( $delta === 0 ) ? $offset : ( $offset + $delta );
-                $string     = (string)substr_replace( $string, $completeTag, $realOffset, $length );
-                $delta      = $delta + strlen( $completeTag ) - $length;
+                $realOffset = ($delta === 0) ? $offset : ($offset + $delta);
+                $string     = (string) substr_replace($string, $completeTag, $realOffset, $length);
+                $delta      = $delta + strlen($completeTag) - $length;
 
             }
 
@@ -349,17 +349,17 @@ 
      * @throws InvalidXmlException
      * @throws XmlParsingException
      */
-    public function restore( string $string ): string {
+    public function restore(string $string): string {
 
         // if the map is empty return string as is
-        if ( $this->map->isEmpty() ) {
+        if ($this->map->isEmpty()) {
             return $string;
         }
 
-        $html = XmlParser::parse( $string, true );
+        $html = XmlParser::parse($string, true);
 
-        foreach ( $html as $node ) {
-            $string = $this->recursiveRestoreOriginalTags( $node, $string );
+        foreach ($html as $node) {
+            $string = $this->recursiveRestoreOriginalTags($node, $string);
         }
 
         return $string;
@@ -371,26 +371,26 @@ 
      *
      * @return string
      */
-    private function recursiveRestoreOriginalTags( object $node, string $string ): string {
+    private function recursiveRestoreOriginalTags(object $node, string $string): string {
 
-        if ( $node->has_children ) {
+        if ($node->has_children) {
 
-            foreach ( $node->inner_html as $childNode ) {
-                $string = $this->recursiveRestoreOriginalTags( $childNode, $string );
+            foreach ($node->inner_html as $childNode) {
+                $string = $this->recursiveRestoreOriginalTags($childNode, $string);
             }
 
         } else {
 
-            $nodeAttributesMap = Map::instance( $node->attributes );
+            $nodeAttributesMap = Map::instance($node->attributes);
 
-            if ( !$nodeAttributesMap->get( 'x-orig' ) ) {
+            if (!$nodeAttributesMap->get('x-orig')) {
                 return $string;
             }
 
-            $cType = $nodeAttributesMap->get( 'ctype' );
+            $cType = $nodeAttributesMap->get('ctype');
 
-            if ( CTypeEnum::isLayer2Constant( $cType ?? '' ) ) {
-                return preg_replace( '/' . preg_quote( $node->node, '/' ) . '/', base64_decode( $nodeAttributesMap->get( 'x-orig' ) ), $string, 1 );
+            if (CTypeEnum::isLayer2Constant($cType ?? '')) {
+                return preg_replace('/'.preg_quote($node->node, '/').'/', base64_decode($nodeAttributesMap->get('x-orig')), $string, 1);
             }
 
         }
@@ -408,24 +408,24 @@ 
      *
      * @return string
      */
-    private function getNewTagString( string $actualNodeString, ?string $id, string $dataRefValue, string $ctype, ?string $upCountIdValue = null ): string {
+    private function getNewTagString(string $actualNodeString, ?string $id, string $dataRefValue, string $ctype, ?string $upCountIdValue = null): string {
 
-        $newTag = [ '<ph' ];
+        $newTag = ['<ph'];
 
-        if ( isset( $id ) ) {
-            $newTag[] = 'id="' . $id . $upCountIdValue . '"';
+        if (isset($id)) {
+            $newTag[] = 'id="'.$id.$upCountIdValue.'"';
         }
 
-        $newTag[] = 'ctype="' . $ctype . '"';
-        $newTag[] = 'equiv-text="base64:' . base64_encode( $dataRefValue ) . '"';
-        $newTag[] = 'x-orig="' . base64_encode( $actualNodeString ) . '"';
+        $newTag[] = 'ctype="'.$ctype.'"';
+        $newTag[] = 'equiv-text="base64:'.base64_encode($dataRefValue).'"';
+        $newTag[] = 'x-orig="'.base64_encode($actualNodeString).'"';
 
-        return implode( " ", $newTag ) . '/>';
+        return implode(" ", $newTag).'/>';
 
     }
 
-    private function replaceNewTagString( $actualNodeString, $id, $dataRefValue, $ctype, $originalString, $upCountIdValue = '_1' ) {
-        return str_replace( $actualNodeString, $this->getNewTagString( $actualNodeString, $id, $dataRefValue, $ctype, $upCountIdValue ), $originalString );
+    private function replaceNewTagString($actualNodeString, $id, $dataRefValue, $ctype, $originalString, $upCountIdValue = '_1') {
+        return str_replace($actualNodeString, $this->getNewTagString($actualNodeString, $id, $dataRefValue, $ctype, $upCountIdValue), $originalString);
     }
 
 }
\ No newline at end of file