matecat / subfiltering

src/Filters/StandardPHToMateCatCustomPH.php

Spacing +16 added lines, -16 removed lines

@@ -14,10 +14,10 @@ 
 
 class StandardPHToMateCatCustomPH extends AbstractHandler {
 
-    public function transform( $segment ) {
+    public function transform($segment) {
 
-        $segment = $this->filterPhTagContent( $segment );
-        $segment = $this->filterOriginalSelfClosePhTagsWithEquivText( $segment );
+        $segment = $this->filterPhTagContent($segment);
+        $segment = $this->filterOriginalSelfClosePhTagsWithEquivText($segment);
 
         return $segment;
 
@@ -28,15 +28,15 @@ 
      *
      * @return string
      */
-    private function filterPhTagContent( $segment ) {
+    private function filterPhTagContent($segment) {
 
-        if ( preg_match( '|</ph>|s', $segment ) ) {
-            preg_match_all( '|<ph id=["\']([^\'"]+?)["\'].*?>(.*?)</ph>|', $segment, $phTags, PREG_SET_ORDER );
-            foreach ( $phTags as $group ) {
+        if (preg_match('|</ph>|s', $segment)) {
+            preg_match_all('|<ph id=["\']([^\'"]+?)["\'].*?>(.*?)</ph>|', $segment, $phTags, PREG_SET_ORDER);
+            foreach ($phTags as $group) {
                 $segment = preg_replace(
-                        '/' . preg_quote( $group[ 0 ], '/' ) . '/',
-                        '<ph id="' . $this->getPipeline()->getNextId() . '" ctype="' . CTypeEnum::ORIGINAL_PH_CONTENT . '" x-orig="' . base64_encode( $group[ 0 ] ) . '" equiv-text="base64:' .
-                        base64_encode( htmlentities( $group[ 2 ], ENT_NOQUOTES | 16 /* ENT_XML1 */, 'UTF-8' ) ) .
+                        '/'.preg_quote($group[0], '/').'/',
+                        '<ph id="'.$this->getPipeline()->getNextId().'" ctype="'.CTypeEnum::ORIGINAL_PH_CONTENT.'" x-orig="'.base64_encode($group[0]).'" equiv-text="base64:'.
+                        base64_encode(htmlentities($group[2], ENT_NOQUOTES | 16 /* ENT_XML1 */, 'UTF-8')).
                         '"/>',
                         $segment,
                         1
@@ -55,16 +55,16 @@ 
      *
      * @return string
      */
-    private function filterOriginalSelfClosePhTagsWithEquivText( $segment ) {
+    private function filterOriginalSelfClosePhTagsWithEquivText($segment) {
 
-        preg_match_all( '|<ph[^>]+?equiv-text\s*?=\s*?(["\'])(?!base64:)(.*?)?\1[^>]*?/>|', $segment, $html, PREG_SET_ORDER );
-        foreach ( $html as $tag_attribute ) {
+        preg_match_all('|<ph[^>]+?equiv-text\s*?=\s*?(["\'])(?!base64:)(.*?)?\1[^>]*?/>|', $segment, $html, PREG_SET_ORDER);
+        foreach ($html as $tag_attribute) {
 
             //replace subsequent elements excluding already encoded
             $segment = preg_replace(
-                    '/' . preg_quote( $tag_attribute[ 0 ], '/' ) . '/',
-                    '<ph id="' . $this->getPipeline()->getNextId() . '" ctype="' . CTypeEnum::ORIGINAL_SELF_CLOSE_PH_WITH_EQUIV_TEXT . '" x-orig="' . base64_encode( $tag_attribute[ 0 ] ) . '" equiv-text="base64:' .
-                    base64_encode( $tag_attribute[ 2 ] != '' ? $tag_attribute[ 2 ] : 'NULL' ) .
+                    '/'.preg_quote($tag_attribute[0], '/').'/',
+                    '<ph id="'.$this->getPipeline()->getNextId().'" ctype="'.CTypeEnum::ORIGINAL_SELF_CLOSE_PH_WITH_EQUIV_TEXT.'" x-orig="'.base64_encode($tag_attribute[0]).'" equiv-text="base64:'.
+                    base64_encode($tag_attribute[2] != '' ? $tag_attribute[2] : 'NULL').
                     '"/>',
                     $segment,
                     1

src/Filters/DataRefRestore.php

Spacing +20 added lines, -20 removed lines

@@ -23,23 +23,23 @@ 
     /**
      * @inheritDoc
      */
-    public function transform( $segment ) {
+    public function transform($segment) {
 
-        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);
     }
 
     /**
@@ -58,15 +58,15 @@ 
      *
      * @return string
      */
-    private function restoreXliffPhTagsFromMatecatPhTags( $segment ) {
-        preg_match_all( '|<ph id="mtc_[0-9]+" ctype="' . CTypeEnum::ORIGINAL_PH_OR_NOT_DATA_REF . '" equiv-text="base64:(.*?)"/>|iu', $segment, $matches );
+    private function restoreXliffPhTagsFromMatecatPhTags($segment) {
+        preg_match_all('|<ph id="mtc_[0-9]+" 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;
@@ -88,16 +88,16 @@ 
      *
      * @return string
      */
-    private function restoreXliffPcTagsFromMatecatPhTags( $segment ) {
-        preg_match_all( '|<ph.+? ctype="' . CTypeEnum::ORIGINAL_PC_OPEN . '" equiv-text="base64:(.*?)"/>|iu', $segment, $matches );
-        preg_match_all( '|<ph.+? ctype="' . CTypeEnum::ORIGINAL_PC_CLOSE . '" equiv-text="base64:(.*?)"/>|iu', $segment, $matches );
+    private function restoreXliffPcTagsFromMatecatPhTags($segment) {
+        preg_match_all('|<ph.+? ctype="'.CTypeEnum::ORIGINAL_PC_OPEN.'" equiv-text="base64:(.*?)"/>|iu', $segment, $matches);
+        preg_match_all('|<ph.+? ctype="'.CTypeEnum::ORIGINAL_PC_CLOSE.'" 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;

src/Filters/DataRefReplace.php

Spacing +41 added lines, -41 removed lines

@@ -28,24 +28,24 @@ 
     /**
      * @inheritDoc
      */
-    public function transform( $segment ) {
+    public function transform($segment) {
 
-        if ( empty( $this->dataRefMap ) ) {
+        if (empty($this->dataRefMap)) {
             $this->dataRefMap = $this->pipeline->getDataRefMap();
         }
 
         // dataRefMap is present only in xliff 2.0 files
-        if ( empty( $this->dataRefMap ) ) {
-            $segment = $this->replace_Ph_TagsWithoutDataRefCorrespondenceToMatecatPhTags( $segment );
+        if (empty($this->dataRefMap)) {
+            $segment = $this->replace_Ph_TagsWithoutDataRefCorrespondenceToMatecatPhTags($segment);
 
-            return $this->replace_Pc_TagsWithoutDataRefCorrespondenceToMatecatPhTags( $segment );
+            return $this->replace_Pc_TagsWithoutDataRefCorrespondenceToMatecatPhTags($segment);
         }
 
-        $dataRefReplacer = new DataRefReplacer( $this->dataRefMap );
-        $segment         = $dataRefReplacer->replace( $segment );
-        $segment         = $this->replace_Ph_TagsWithoutDataRefCorrespondenceToMatecatPhTags( $segment );
+        $dataRefReplacer = new DataRefReplacer($this->dataRefMap);
+        $segment         = $dataRefReplacer->replace($segment);
+        $segment         = $this->replace_Ph_TagsWithoutDataRefCorrespondenceToMatecatPhTags($segment);
 
-        return $this->replace_Pc_TagsWithoutDataRefCorrespondenceToMatecatPhTags( $segment );
+        return $this->replace_Pc_TagsWithoutDataRefCorrespondenceToMatecatPhTags($segment);
     }
 
     /**
@@ -64,23 +64,23 @@ 
      *
      * @return string|string[]
      */
-    private function replace_Ph_TagsWithoutDataRefCorrespondenceToMatecatPhTags( $segment ) {
+    private function replace_Ph_TagsWithoutDataRefCorrespondenceToMatecatPhTags($segment) {
 
-        preg_match_all( '/<(ph .*?)>/iu', $segment, $phTags );
+        preg_match_all('/<(ph .*?)>/iu', $segment, $phTags);
 
-        if ( count( $phTags[ 0 ] ) === 0 ) {
+        if (count($phTags[0]) === 0) {
             return $segment;
         }
 
-        foreach ( $phTags[ 0 ] as $phTag ) {
+        foreach ($phTags[0] as $phTag) {
 
             // check if phTag has not any correspondence on dataRef map
-            if ( $this->isAValidPhTag( $phTag ) ) {
+            if ($this->isAValidPhTag($phTag)) {
                 $segment = preg_replace(
-                        '/' . preg_quote( $phTag, '/' ) . '/',
-                        '<ph id="' . $this->getPipeline()->getNextId() .
-                        '" ctype="' . CTypeEnum::ORIGINAL_PH_OR_NOT_DATA_REF .
-                        '" equiv-text="base64:' . base64_encode( $phTag ) .
+                        '/'.preg_quote($phTag, '/').'/',
+                        '<ph id="'.$this->getPipeline()->getNextId().
+                        '" ctype="'.CTypeEnum::ORIGINAL_PH_OR_NOT_DATA_REF.
+                        '" equiv-text="base64:'.base64_encode($phTag).
                         '"/>',
                         $segment,
                         1 // replace ONLY ONE occurrence
@@ -100,32 +100,32 @@ 
      *
      * @return bool
      */
-    private function isAValidPhTag( $phTag ) {
+    private function isAValidPhTag($phTag) {
 
         // try not to throw exception for wrong segments with opening tags and no closing
         try {
-            $parsed = XmlParser::parse( $phTag, true );
-        } catch ( Exception $e ){
+            $parsed = XmlParser::parse($phTag, true);
+        } catch (Exception $e) {
             return false;
         }
 
-        if ( $parsed->count() == 0 ) {
+        if ($parsed->count() == 0) {
             return false;
         }
 
         // check for matecat ctype
-        $cType = isset( $parsed[ 0 ]->attributes[ 'ctype' ] ) ? $parsed[ 0 ]->attributes[ 'ctype' ] : null;
-        if ( CTypeEnum::isMatecatCType( $cType ) ) {
+        $cType = isset($parsed[0]->attributes['ctype']) ? $parsed[0]->attributes['ctype'] : null;
+        if (CTypeEnum::isMatecatCType($cType)) {
             return false;
         }
 
         // if has equiv-text don't touch
-        if ( isset( $parsed[ 0 ]->attributes[ 'equiv-text' ] ) ) {
+        if (isset($parsed[0]->attributes['equiv-text'])) {
             return false;
         }
 
-        if ( isset( $parsed[ 0 ]->attributes[ 'dataRef' ] ) ) {
-            return !array_key_exists( $parsed[ 0 ]->attributes[ 'dataRef' ], $this->dataRefMap );
+        if (isset($parsed[0]->attributes['dataRef'])) {
+            return !array_key_exists($parsed[0]->attributes['dataRef'], $this->dataRefMap);
         }
 
         return true;
@@ -148,33 +148,33 @@ 
      *
      * @return string|string[]
      */
-    private function replace_Pc_TagsWithoutDataRefCorrespondenceToMatecatPhTags( $segment ) {
+    private function replace_Pc_TagsWithoutDataRefCorrespondenceToMatecatPhTags($segment) {
 
-        preg_match_all( '/<(pc .*?)>/iu', $segment, $openingPcTags );
-        preg_match_all( '|<(/pc)>|iu', $segment, $closingPcTags );
+        preg_match_all('/<(pc .*?)>/iu', $segment, $openingPcTags);
+        preg_match_all('|<(/pc)>|iu', $segment, $closingPcTags);
 
-        if ( count( $openingPcTags[ 0 ] ) === 0 ) {
+        if (count($openingPcTags[0]) === 0) {
             return $segment;
         }
 
-        foreach ( $openingPcTags[ 0 ] as $openingPcTag ) {
+        foreach ($openingPcTags[0] as $openingPcTag) {
             $segment = preg_replace(
-                    '/' . preg_quote( $openingPcTag, '/' ) . '/',
-                    '<ph id="' . $this->getPipeline()->getNextId() .
-                    '" ctype="' . CTypeEnum::ORIGINAL_PC_OPEN .
-                    '" equiv-text="base64:' . base64_encode( $openingPcTag ) .
+                    '/'.preg_quote($openingPcTag, '/').'/',
+                    '<ph id="'.$this->getPipeline()->getNextId().
+                    '" ctype="'.CTypeEnum::ORIGINAL_PC_OPEN.
+                    '" equiv-text="base64:'.base64_encode($openingPcTag).
                     '"/>',
                     $segment,
                     1
             );
         }
 
-        foreach ( $closingPcTags[ 0 ] as $closingPcTag ) {
+        foreach ($closingPcTags[0] as $closingPcTag) {
             $segment = preg_replace(
-                    '/' . preg_quote( $closingPcTag, '/' ) . '/',
-                    '<ph id="' . $this->getPipeline()->getNextId() .
-                    '" ctype="' . CTypeEnum::ORIGINAL_PC_CLOSE .
-                    '" equiv-text="base64:' . base64_encode( $closingPcTag ) .
+                    '/'.preg_quote($closingPcTag, '/').'/',
+                    '<ph id="'.$this->getPipeline()->getNextId().
+                    '" ctype="'.CTypeEnum::ORIGINAL_PC_CLOSE.
+                    '" equiv-text="base64:'.base64_encode($closingPcTag).
                     '"/>',
                     $segment,
                     1

src/Utils/DataRefReplacer.php

Spacing +106 added lines, -106 removed lines

@@ -27,7 +27,7 @@ 
      *
      * @param array $map
      */
-    public function __construct( array $map = null ) {
+    public function __construct(array $map = null) {
         $this->map = $map;
     }
 
@@ -42,42 +42,42 @@ 
      *
      * @return string
      */
-    public function replace( $string ) {
+    public function replace($string) {
 
         // if map is empty
         // or the string has not a dataRef attribute
         // return string as is
-        if ( empty( $this->map ) || !$this->hasAnyDataRefAttribute( $string ) ) {
+        if (empty($this->map) || !$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 = [];
 
-            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, $dataRefEndMap );
+                $this->extractDataRefMapRecursively($node, $dataRefEndMap);
 
             }
 
             // 4. replace pc tags
-            $string = $this->replaceOpeningPcTags( null, $string );
-            $string = $this->replaceClosingPcTags( $string, $dataRefEndMap );
+            $string = $this->replaceOpeningPcTags(null, $string);
+            $string = $this->replaceClosingPcTags($string, $dataRefEndMap);
 
-        } catch ( Exception $ignore ) {
+        } catch (Exception $ignore) {
         } finally {
             return $string;
         }
@@ -89,8 +89,8 @@ 
      *
      * @return bool
      */
-    private function hasAnyDataRefAttribute( $string ) {
-        return (bool)preg_match( '/(dataRef|dataRefStart|dataRefEnd)=[\'"].*?[\'"]/', $string );
+    private function hasAnyDataRefAttribute($string) {
+        return (bool) preg_match('/(dataRef|dataRefStart|dataRefEnd)=[\'"].*?[\'"]/', $string);
     }
 
     /**
@@ -106,17 +106,17 @@ 
      *
      * @return string
      */
-    private function recursiveTransformDataRefToPhTag( $node, $string ) {
+    private function recursiveTransformDataRefToPhTag($node, $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 {
 
-            switch ( $node->tagName ) {
+            switch ($node->tagName) {
                 case 'ph':
                 case 'sc':
                 case 'ec':
@@ -125,48 +125,48 @@ 
                     return $string;
             }
 
-            if ( !isset( $node->attributes[ 'dataRef' ] ) ) {
+            if (!isset($node->attributes['dataRef'])) {
                 return $string;
             }
 
             // if isset a value in the map calculate base64 encoded value
             // otherwise skip
-            if ( !in_array( $node->attributes[ 'dataRef' ], array_keys( $this->map ) ) ) {
+            if (!in_array($node->attributes['dataRef'], array_keys($this->map))) {
                 return $string;
             }
 
-            $dataRefName  = $node->attributes[ 'dataRef' ];   // map identifier. Eg: source1
-            $dataRefValue = $this->map[ $dataRefName ];   // map identifier. Eg: source1
+            $dataRefName  = $node->attributes['dataRef']; // map identifier. Eg: source1
+            $dataRefValue = $this->map[$dataRefName]; // map identifier. Eg: source1
 
             // check if is null or an empty string, in this case, convert it to NULL string
-            if ( is_null( $dataRefValue ) || $dataRefValue === '' ) {
-                $this->map[ $dataRefName ] = 'NULL';
+            if (is_null($dataRefValue) || $dataRefValue === '') {
+                $this->map[$dataRefName] = 'NULL';
             }
 
 
-            $newTag = [ '<ph' ];
+            $newTag = ['<ph'];
 
             // if there is no id copy it from dataRef
-            if ( !isset( $node->attributes[ 'id' ] ) ) {
-                $newTag[] = 'id="' . $dataRefName . '"';
+            if (!isset($node->attributes['id'])) {
+                $newTag[] = 'id="'.$dataRefName.'"';
                 $newTag[] = 'x-removeId="true"';
             } else {
-                $newTag[] = 'id="' . $node->attributes[ 'id' ] . '"';
+                $newTag[] = 'id="'.$node->attributes['id'].'"';
             }
 
             // introduce dataType for <ec>/<sc> tag handling
-            if ( $node->tagName === 'ec' ) {
-                $newTag[] = 'ctype="' . CTypeEnum::EC_DATA_REF . '"';
-            } elseif ( $node->tagName === 'sc' ) {
-                $newTag[] = 'ctype="' . CTypeEnum::SC_DATA_REF . '"';
+            if ($node->tagName === 'ec') {
+                $newTag[] = 'ctype="'.CTypeEnum::EC_DATA_REF.'"';
+            } elseif ($node->tagName === 'sc') {
+                $newTag[] = 'ctype="'.CTypeEnum::SC_DATA_REF.'"';
             } else {
-                $newTag[] = 'ctype="' . CTypeEnum::PH_DATA_REF . '"';
+                $newTag[] = 'ctype="'.CTypeEnum::PH_DATA_REF.'"';
             }
 
-            $newTag[] = 'equiv-text="base64:' . base64_encode( $dataRefValue ) . '"';
-            $newTag[] = 'x-orig="' . base64_encode( $node->node ) . '"';
+            $newTag[] = 'equiv-text="base64:'.base64_encode($dataRefValue).'"';
+            $newTag[] = 'x-orig="'.base64_encode($node->node).'"';
 
-            return str_replace( $node->node, implode( " ", $newTag ) . '/>', $string );
+            return str_replace($node->node, implode(" ", $newTag).'/>', $string);
 
         }
 
@@ -182,29 +182,29 @@ 
      * @throws InvalidXmlException
      * @throws XmlParsingException
      */
-    private function recursiveReplaceSelfClosedPcTags( $node, $string ) {
+    private function recursiveReplaceSelfClosedPcTags($node, $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) {
 
-            if ( isset( $node->attributes[ 'dataRefStart' ] ) && array_key_exists( $node->attributes[ 'dataRefStart' ], $this->map ) ) {
+            if (isset($node->attributes['dataRefStart']) && array_key_exists($node->attributes['dataRefStart'], $this->map)) {
 
-                $newTag = [ '<ph' ];
+                $newTag = ['<ph'];
 
-                if ( isset( $node->attributes[ 'id' ] ) ) {
-                    $newTag[] = 'id="' . $node->attributes[ 'id' ] . '_1"';
+                if (isset($node->attributes['id'])) {
+                    $newTag[] = 'id="'.$node->attributes['id'].'_1"';
                 }
 
-                $newTag[] = 'ctype="' . CTypeEnum::PC_SELF_CLOSE_DATA_REF . '"';
-                $newTag[] = 'equiv-text="base64:' . base64_encode( $this->map[ $node->attributes[ 'dataRefStart' ] ] ) . '"';
-                $newTag[] = 'x-orig="' . base64_encode( $node->node ) . '"';
+                $newTag[] = 'ctype="'.CTypeEnum::PC_SELF_CLOSE_DATA_REF.'"';
+                $newTag[] = 'equiv-text="base64:'.base64_encode($this->map[$node->attributes['dataRefStart']]).'"';
+                $newTag[] = 'x-orig="'.base64_encode($node->node).'"';
 
-                $string = str_replace( $node->node, implode( " ", $newTag ) . '/>', $string );
+                $string = str_replace($node->node, implode(" ", $newTag).'/>', $string);
             }
 
         }
@@ -219,27 +219,27 @@ 
      * @param object $node
      * @param        $dataRefEndMap
      */
-    private function extractDataRefMapRecursively( $node, &$dataRefEndMap ) {
+    private function extractDataRefMapRecursively($node, &$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, $dataRefEndMap );
+        if ($node->has_children) {
+            foreach ($node->inner_html as $nestedNode) {
+                $this->extractDataRefMapRecursively($nestedNode, $dataRefEndMap);
             }
         }
 
         // EXCLUDE self closed <pc/>
-        if ( $node->tagName === 'pc' && $node->self_closed === false ) {
-            if ( isset( $node->attributes[ 'dataRefEnd' ] ) ) {
-                $dataRefEnd = $node->attributes[ 'dataRefEnd' ];
-            } elseif ( isset( $node->attributes[ 'dataRefStart' ] ) ) {
-                $dataRefEnd = $node->attributes[ 'dataRefStart' ];
+        if ($node->tagName === 'pc' && $node->self_closed === false) {
+            if (isset($node->attributes['dataRefEnd'])) {
+                $dataRefEnd = $node->attributes['dataRefEnd'];
+            } elseif (isset($node->attributes['dataRefStart'])) {
+                $dataRefEnd = $node->attributes['dataRefStart'];
             } else {
                 $dataRefEnd = null;
             }
 
             $dataRefEndMap[] = [
-                    'id'         => isset( $node->attributes[ 'id' ] ) ? $node->attributes[ 'id' ] : null,
+                    'id'         => isset($node->attributes['id']) ? $node->attributes['id'] : null,
                     'dataRefEnd' => $dataRefEnd,
             ];
 
@@ -257,40 +257,40 @@ 
      * @throws InvalidXmlException
      * @throws XmlParsingException
      */
-    private function replaceOpeningPcTags( $node, $string ) {
+    private function replaceOpeningPcTags($node, $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'])) {
 
-                $startValue = $this->map[ $node->attributes[ 'dataRefStart' ] ] ?: 'NULL'; //handling null values in original data map
+                $startValue = $this->map[$node->attributes['dataRefStart']] ?: 'NULL'; //handling null values in original data map
 
-                $newTag = [ '<ph' ];
+                $newTag = ['<ph'];
 
-                if ( isset( $node->attributes[ 'id' ] ) ) {
-                    $newTag[] = 'id="' . $node->attributes[ 'id' ] . '_1"';
+                if (isset($node->attributes['id'])) {
+                    $newTag[] = 'id="'.$node->attributes['id'].'_1"';
                 }
 
-                $newTag[] = 'ctype="' . CTypeEnum::PC_OPEN_DATA_REF . '"';
-                $newTag[] = 'equiv-text="base64:' . base64_encode( $startValue ) . '"';
-                $newTag[] = 'x-orig="' . base64_encode( $match ) . '"';
+                $newTag[] = 'ctype="'.CTypeEnum::PC_OPEN_DATA_REF.'"';
+                $newTag[] = 'equiv-text="base64:'.base64_encode($startValue).'"';
+                $newTag[] = 'x-orig="'.base64_encode($match).'"';
 
                 // conversion for opening <pc> tag
-                $string = str_replace( $match, implode( " ", $newTag ) . '/>', $string );
+                $string = str_replace($match, implode(" ", $newTag).'/>', $string);
 
             }
         }
@@ -307,37 +307,37 @@ 
      *
      * @return string
      */
-    private function replaceClosingPcTags( $string, $dataRefEndMap = [] ) {
+    private function replaceClosingPcTags($string, $dataRefEndMap = []) {
 
-        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 = isset( $dataRefEndMap[ $index ] ) ? $dataRefEndMap[ $index ] : null;
+            $attr = isset($dataRefEndMap[$index]) ? $dataRefEndMap[$index] : null;
 
-            if ( !empty( $attr ) && isset( $attr[ 'dataRefEnd' ] ) ) {
+            if (!empty($attr) && isset($attr['dataRefEnd'])) {
 
-                $endValue = !empty( $this->map[ $attr[ 'dataRefEnd' ] ] ) ? $this->map[ $attr[ 'dataRefEnd' ] ] : 'NULL';
+                $endValue = !empty($this->map[$attr['dataRefEnd']]) ? $this->map[$attr['dataRefEnd']] : 'NULL';
 
-                $newTag = [ '<ph' ];
+                $newTag = ['<ph'];
 
-                if ( isset( $attr[ 'id' ] ) ) {
-                    $newTag[] = 'id="' . $attr[ 'id' ] . '_2"';
+                if (isset($attr['id'])) {
+                    $newTag[] = 'id="'.$attr['id'].'_2"';
                 }
 
-                $newTag[] = 'ctype="' . CTypeEnum::PC_CLOSE_DATA_REF . '"';
-                $newTag[] = 'equiv-text="base64:' . base64_encode( $endValue ) . '"';
-                $newTag[] = 'x-orig="' . base64_encode( '</pc>' ) . '"';
+                $newTag[] = 'ctype="'.CTypeEnum::PC_CLOSE_DATA_REF.'"';
+                $newTag[] = 'equiv-text="base64:'.base64_encode($endValue).'"';
+                $newTag[] = 'x-orig="'.base64_encode('</pc>').'"';
 
                 // conversion for opening <pc> tag
-                $completeTag = implode( " ", $newTag ) . '/>';
-                $realOffset  = ( $delta === 0 ) ? $offset : ( $offset + $delta );
-                $string      = substr_replace( $string, $completeTag, $realOffset, $length );
-                $delta       = $delta + strlen( $completeTag ) - $length;
+                $completeTag = implode(" ", $newTag).'/>';
+                $realOffset  = ($delta === 0) ? $offset : ($offset + $delta);
+                $string      = substr_replace($string, $completeTag, $realOffset, $length);
+                $delta       = $delta + strlen($completeTag) - $length;
 
             }
 
@@ -355,19 +355,19 @@ 
      * @throws InvalidXmlException
      * @throws XmlParsingException
      */
-    public function restore( $string ) {
+    public function restore($string) {
 
         // if map is empty return string as is
-        if ( empty( $this->map ) ) {
+        if (empty($this->map)) {
             return $string;
         }
 
         // replace eventual empty equiv-text=""
 //        $string = str_replace( ' equiv-text=""', '', $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;
@@ -379,21 +379,21 @@ 
      *
      * @return string
      */
-    private function recursiveRestoreOriginalTags( $node, $string ) {
+    private function recursiveRestoreOriginalTags($node, $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 {
 
-            $cType = isset( $node->attributes[ 'ctype' ] ) ? $node->attributes[ 'ctype' ] : null;
+            $cType = isset($node->attributes['ctype']) ? $node->attributes['ctype'] : null;
 
-            if ( $cType ) {
+            if ($cType) {
 
-                switch ( $node->attributes[ 'ctype' ] ) {
+                switch ($node->attributes['ctype']) {
                     case CTypeEnum::ORIGINAL_PC_OPEN:
                     case CTypeEnum::ORIGINAL_PC_CLOSE:
                     case CTypeEnum::ORIGINAL_PH_OR_NOT_DATA_REF:
@@ -403,7 +403,7 @@ 
                     case CTypeEnum::PC_SELF_CLOSE_DATA_REF:
                     case CTypeEnum::SC_DATA_REF:
                     case CTypeEnum::EC_DATA_REF:
-                        return preg_replace( '/' . preg_quote( $node->node, '/' ) . '/', base64_decode( $node->attributes[ 'x-orig' ] ), $string, 1 );
+                        return preg_replace('/'.preg_quote($node->node, '/').'/', base64_decode($node->attributes['x-orig']), $string, 1);
                 }
 
             }

src/Enum/CTypeEnum.php

Spacing +5 added lines, -5 removed lines

@@ -39,16 +39,16 @@
     protected static $constantsValues = [];
 
     protected static function getConstantsMap() {
-        if ( empty( static::$constantsValues ) ) {
-            $reflectedProperty       = ( new ReflectionClass( static::class ) )->getConstants();
-            static::$constantsValues = array_flip( $reflectedProperty );
+        if (empty(static::$constantsValues)) {
+            $reflectedProperty       = (new ReflectionClass(static::class))->getConstants();
+            static::$constantsValues = array_flip($reflectedProperty);
         }
 
         return static::$constantsValues;
     }
 
-    public static function isMatecatCType( $ctype ) {
-        return array_key_exists( $ctype, static::getConstantsMap() );
+    public static function isMatecatCType($ctype) {
+        return array_key_exists($ctype, static::getConstantsMap());
     }
 
 }
\ No newline at end of file