matecat / xliff-parser

src/XliffUtils/DataRefReplacer.php

Spacing +164 added lines, -164 removed lines

@@ -16,7 +16,7 @@ 
      *
      * @param array $map
      */
-    public function __construct( array $map ) {
+    public function __construct(array $map) {
         $this->map = $map;
     }
 
@@ -31,38 +31,38 @@ 
      *
      * @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;
         }
 
         // (recursively) clean string from equiv-text eventually present
-        $string = $this->cleanFromEquivText( $string );
+        $string = $this->cleanFromEquivText($string);
 
-        $html = HtmlParser::parse( $string );
+        $html = HtmlParser::parse($string);
 
         // 1. Replace <ph>|<sc>|<ec> tags
-        foreach ( $html as $node ) {
-            $string = $this->recursiveAddEquivTextToPhTag( $node, $string );
+        foreach ($html as $node) {
+            $string = $this->recursiveAddEquivTextToPhTag($node, $string);
         }
 
         // 2. Replace <pc> tags
-        $toBeEscaped = Strings::isAnEscapedHTML( $string );
+        $toBeEscaped = Strings::isAnEscapedHTML($string);
 
-        if ( $this->stringContainsPcTags( $string, $toBeEscaped ) ) {
+        if ($this->stringContainsPcTags($string, $toBeEscaped)) {
 
             // replace self-closed <pc />
-            $string = $this->replaceSelfClosedPcTags( $string, $toBeEscaped );
+            $string = $this->replaceSelfClosedPcTags($string, $toBeEscaped);
 
             // create a dataRefEnd map
             // (needed for correct handling of </pc> closing tags)
-            $dataRefEndMap = $this->buildDataRefEndMap( $html );
-            $string        = $this->replaceOpeningPcTags( $string, $toBeEscaped );
-            $string        = $this->replaceClosingPcTags( $string, $toBeEscaped, $dataRefEndMap );
-            $string        = ( $toBeEscaped ) ? Strings::escapeOnlyHTMLTags( $string ) : $string;
+            $dataRefEndMap = $this->buildDataRefEndMap($html);
+            $string        = $this->replaceOpeningPcTags($string, $toBeEscaped);
+            $string        = $this->replaceClosingPcTags($string, $toBeEscaped, $dataRefEndMap);
+            $string        = ($toBeEscaped) ? Strings::escapeOnlyHTMLTags($string) : $string;
         }
 
         return $string;
@@ -73,17 +73,17 @@ 
      *
      * @return bool
      */
-    private function hasAnyDataRefAttribute( $string ) {
+    private function hasAnyDataRefAttribute($string) {
         $dataRefTags = [
                 'dataRef',
                 'dataRefStart',
                 'dataRefEnd',
         ];
 
-        foreach ( $dataRefTags as $tag ) {
-            preg_match( '/ ' . $tag . '=[\\\\"](.*?)[\\\\"]/', $string, $matches );
+        foreach ($dataRefTags as $tag) {
+            preg_match('/ ' . $tag . '=[\\\\"](.*?)[\\\\"]/', $string, $matches);
 
-            if ( count( $matches ) > 0 ) {
+            if (count($matches) > 0) {
                 return true;
             }
         }
@@ -94,11 +94,11 @@ 
      *
      * @return string
      */
-    private function cleanFromEquivText( $string ) {
-        $html = HtmlParser::parse( $string );
+    private function cleanFromEquivText($string) {
+        $html = HtmlParser::parse($string);
 
-        foreach ( $html as $node ) {
-            $string = $this->recursiveCleanFromEquivText( $node, $string );
+        foreach ($html as $node) {
+            $string = $this->recursiveCleanFromEquivText($node, $string);
         }
 
         return $string;
@@ -117,57 +117,57 @@ 
      *
      * @return string
      */
-    private function recursiveAddEquivTextToPhTag( $node, $string ) {
-        if ( $node->has_children ) {
-            foreach ( $node->inner_html as $childNode ) {
-                $string = $this->recursiveAddEquivTextToPhTag( $childNode, $string );
+    private function recursiveAddEquivTextToPhTag($node, $string) {
+        if ($node->has_children) {
+            foreach ($node->inner_html as $childNode) {
+                $string = $this->recursiveAddEquivTextToPhTag($childNode, $string);
             }
         } else {
-            if ( $node->tagname === 'ph' || $node->tagname === 'sc' || $node->tagname === 'ec' ) {
-                if ( !isset( $node->attributes[ 'dataRef' ] ) ) {
+            if ($node->tagname === 'ph' || $node->tagname === 'sc' || $node->tagname === 'ec') {
+                if (!isset($node->attributes['dataRef'])) {
                     return $string;
                 }
 
-                $a = $node->node;  // complete match. Eg:  <ph id="source1" dataRef="source1"/>
-                $b = $node->attributes[ 'dataRef' ];   // map identifier. Eg: source1
+                $a = $node->node; // complete match. Eg:  <ph id="source1" dataRef="source1"/>
+                $b = $node->attributes['dataRef']; // map identifier. Eg: source1
 
 
                 // if isset a value in the map calculate base64 encoded value
                 // otherwise skip
-                if ( !in_array( $b, array_keys( $this->map ) ) ) {
+                if (!in_array($b, array_keys($this->map))) {
                     return $string;
                 }
 
                 // check if is null, in this case convert it to NULL string
-                if ( is_null( $this->map[ $b ] ) ) {
-                    $this->map[ $b ] = 'NULL';
+                if (is_null($this->map[$b])) {
+                    $this->map[$b] = 'NULL';
                 }
 
-                $value              = $this->map[ $b ];
-                $base64EncodedValue = base64_encode( $value );
+                $value              = $this->map[$b];
+                $base64EncodedValue = base64_encode($value);
 
-                if ( empty( $base64EncodedValue ) || $base64EncodedValue === '' ) {
+                if (empty($base64EncodedValue) || $base64EncodedValue === '') {
                     return $string;
                 }
 
                 // if there is no id copy it from dataRef
-                $id = ( !isset( $node->attributes[ 'id' ] ) ) ? ' id="' . $b . '" removeId="true"' : '';
+                $id = (!isset($node->attributes['id'])) ? ' id="' . $b . '" removeId="true"' : '';
 
                 // introduce dataType for <ec>/<sc> tag handling
-                $dataType = ( $this->isAEcOrScTag( $node ) ) ? ' dataType="' . $node->tagname . '"' : '';
+                $dataType = ($this->isAEcOrScTag($node)) ? ' dataType="' . $node->tagname . '"' : '';
 
                 // replacement
-                $d = str_replace( '/', $id . $dataType . ' equiv-text="base64:' . $base64EncodedValue . '"/', $a );
-                $a = str_replace( [ '<', '>', '&gt;', '&lt;' ], '', $a );
-                $d = str_replace( [ '<', '>', '&gt;', '&lt;' ], '', $d );
+                $d = str_replace('/', $id . $dataType . ' equiv-text="base64:' . $base64EncodedValue . '"/', $a);
+                $a = str_replace(['<', '>', '&gt;', '&lt;'], '', $a);
+                $d = str_replace(['<', '>', '&gt;', '&lt;'], '', $d);
 
                 // convert <ec>/<sc> into <ph>
-                if ( $this->isAEcOrScTag( $node ) ) {
-                    $d = 'ph' . substr( $d, 2 );
-                    $d = trim( $d );
+                if ($this->isAEcOrScTag($node)) {
+                    $d = 'ph' . substr($d, 2);
+                    $d = trim($d);
                 }
 
-                return str_replace( $a, $d, $string );
+                return str_replace($a, $d, $string);
             }
         }
 
@@ -180,11 +180,11 @@ 
      *
      * @return bool
      */
-    private function stringContainsPcTags( $string, $toBeEscaped ) {
-        $regex = ( $toBeEscaped ) ? '/&lt;pc (.*?)&gt;/iu' : '/<pc (.*?)>/iu';
-        preg_match_all( $regex, $string, $openingPcMatches );
+    private function stringContainsPcTags($string, $toBeEscaped) {
+        $regex = ($toBeEscaped) ? '/&lt;pc (.*?)&gt;/iu' : '/<pc (.*?)>/iu';
+        preg_match_all($regex, $string, $openingPcMatches);
 
-        return ( isset( $openingPcMatches[ 0 ] ) && count( $openingPcMatches[ 0 ] ) > 0 );
+        return (isset($openingPcMatches[0]) && count($openingPcMatches[0]) > 0);
     }
 
     /**
@@ -193,28 +193,28 @@ 
      *
      * @return mixed
      */
-    private function replaceSelfClosedPcTags( $string, $toBeEscaped ) {
-        if ( $toBeEscaped ) {
-            $string = str_replace( [ '&lt;', '&gt;' ], [ '<', '>' ], $string );
+    private function replaceSelfClosedPcTags($string, $toBeEscaped) {
+        if ($toBeEscaped) {
+            $string = str_replace(['&lt;', '&gt;'], ['<', '>'], $string);
         }
 
         $regex = '/<pc[^>]+?\/>/iu';
-        preg_match_all( $regex, $string, $selfClosedPcMatches );
+        preg_match_all($regex, $string, $selfClosedPcMatches);
 
-        foreach ( $selfClosedPcMatches[ 0 ] as $match ) {
+        foreach ($selfClosedPcMatches[0] as $match) {
 
-            $html       = HtmlParser::parse( $match );
-            $node       = $html[ 0 ];
+            $html       = HtmlParser::parse($match);
+            $node       = $html[0];
             $attributes = $node->attributes;
 
-            if ( isset( $attributes[ 'dataRefStart' ] ) && array_key_exists( $node->attributes[ 'dataRefStart' ], $this->map ) ) {
-                $replacement = '<ph id="' . $attributes[ 'id' ] . '" dataType="pcSelf" originalData="' . base64_encode( $match ) . '" dataRef="' . $attributes[ 'dataRefStart' ] . '" equiv-text="base64:' . base64_encode( $this->map[ $node->attributes[ 'dataRefStart' ] ] ) . '"/>';
-                $string      = str_replace( $match, $replacement, $string );
+            if (isset($attributes['dataRefStart']) && array_key_exists($node->attributes['dataRefStart'], $this->map)) {
+                $replacement = '<ph id="' . $attributes['id'] . '" dataType="pcSelf" originalData="' . base64_encode($match) . '" dataRef="' . $attributes['dataRefStart'] . '" equiv-text="base64:' . base64_encode($this->map[$node->attributes['dataRefStart']]) . '"/>';
+                $string      = str_replace($match, $replacement, $string);
             }
         }
 
-        if ( $toBeEscaped ) {
-            $string = str_replace( [ '<', '>' ], [ '&lt;', '&gt;' ], $string );
+        if ($toBeEscaped) {
+            $string = str_replace(['<', '>'], ['&lt;', '&gt;'], $string);
         }
 
         return $string;
@@ -228,12 +228,12 @@ 
      *
      * @return array
      */
-    private function buildDataRefEndMap( $html ) {
+    private function buildDataRefEndMap($html) {
         $dataRefEndMap = [];
 
-        foreach ( $html as $index => $node ) {
-            if ( $node->tagname === 'pc' ) {
-                $this->extractDataRefMapRecursively( $node, $dataRefEndMap );
+        foreach ($html as $index => $node) {
+            if ($node->tagname === 'pc') {
+                $this->extractDataRefMapRecursively($node, $dataRefEndMap);
             }
         }
 
@@ -246,25 +246,25 @@ 
      * @param object $node
      * @param        $dataRefEndMap
      */
-    private function extractDataRefMapRecursively( $node, &$dataRefEndMap ) {
-        if ( $this->nodeContainsNestedPcTags( $node ) ) {
-            foreach ( $node->inner_html as $nestedNode ) {
-                $this->extractDataRefMapRecursively( $nestedNode, $dataRefEndMap );
+    private function extractDataRefMapRecursively($node, &$dataRefEndMap) {
+        if ($this->nodeContainsNestedPcTags($node)) {
+            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,
             ];
         }
@@ -276,15 +276,15 @@ 
      *
      * @return string|string[]
      */
-    private function recursiveCleanFromEquivText( $node, $string ) {
-        if ( $node->has_children ) {
-            foreach ( $node->inner_html as $childNode ) {
-                $string = $this->recursiveCleanFromEquivText( $childNode, $string );
+    private function recursiveCleanFromEquivText($node, $string) {
+        if ($node->has_children) {
+            foreach ($node->inner_html as $childNode) {
+                $string = $this->recursiveCleanFromEquivText($childNode, $string);
             }
         } else {
-            if ( isset( $node->attributes[ 'dataRef' ] ) && array_key_exists( $node->attributes[ 'dataRef' ], $this->map ) ) {
-                $cleaned = preg_replace( '/ equiv-text="(.*?)"/', '', $node->node );
-                $string  = str_replace( $node->node, $cleaned, $string );
+            if (isset($node->attributes['dataRef']) && array_key_exists($node->attributes['dataRef'], $this->map)) {
+                $cleaned = preg_replace('/ equiv-text="(.*?)"/', '', $node->node);
+                $string  = str_replace($node->node, $cleaned, $string);
             }
         }
 
@@ -299,35 +299,35 @@ 
      *
      * @return string
      */
-    private function replaceOpeningPcTags( $string, $toBeEscaped ) {
-        $regex = ( $toBeEscaped ) ? '/&lt;pc (.*?)&gt;/iu' : '/<pc (.*?)>/iu';
-        preg_match_all( $regex, $string, $openingPcMatches );
+    private function replaceOpeningPcTags($string, $toBeEscaped) {
+        $regex = ($toBeEscaped) ? '/&lt;pc (.*?)&gt;/iu' : '/<pc (.*?)>/iu';
+        preg_match_all($regex, $string, $openingPcMatches);
 
-        foreach ( $openingPcMatches[ 0 ] as $index => $match ) {
-            $attr = HtmlParser::getAttributes( $openingPcMatches[ 1 ][ $index ] );
+        foreach ($openingPcMatches[0] as $index => $match) {
+            $attr = HtmlParser::getAttributes($openingPcMatches[1][$index]);
 
             // CASE 1 - Missing `dataRefStart`
-            if ( isset( $attr[ 'dataRefEnd' ] ) && !isset( $attr[ 'dataRefStart' ] ) ) {
-                $attr[ 'dataRefStart' ] = $attr[ 'dataRefEnd' ];
+            if (isset($attr['dataRefEnd']) && !isset($attr['dataRefStart'])) {
+                $attr['dataRefStart'] = $attr['dataRefEnd'];
             }
 
             // CASE 2 - Missing `dataRefEnd`
-            if ( isset( $attr[ 'dataRefStart' ] ) && !isset( $attr[ 'dataRefEnd' ] ) ) {
-                $attr[ 'dataRefEnd' ] = $attr[ 'dataRefStart' ];
+            if (isset($attr['dataRefStart']) && !isset($attr['dataRefEnd'])) {
+                $attr['dataRefEnd'] = $attr['dataRefStart'];
             }
 
-            if ( isset( $attr[ 'dataRefStart' ] ) ) {
+            if (isset($attr['dataRefStart'])) {
                 $startOriginalData       = $match; // opening <pc>
-                $startValue              = $this->map[ $attr[ 'dataRefStart' ] ] ? $this->map[ $attr[ 'dataRefStart' ] ] : 'NULL'; //handling null values in original data map
-                $base64EncodedStartValue = base64_encode( $startValue );
-                $base64StartOriginalData = base64_encode( $startOriginalData );
+                $startValue              = $this->map[$attr['dataRefStart']] ? $this->map[$attr['dataRefStart']] : 'NULL'; //handling null values in original data map
+                $base64EncodedStartValue = base64_encode($startValue);
+                $base64StartOriginalData = base64_encode($startOriginalData);
 
                 // conversion for opening <pc> tag
-                $openingPcConverted = '<ph ' . ( ( isset( $attr[ 'id' ] ) ) ? 'id="' . $attr[ 'id' ] . '_1"' : '' ) . ' dataType="pcStart" originalData="' . $base64StartOriginalData . '" dataRef="'
-                        . $attr[ 'dataRefStart' ] . '" equiv-text="base64:'
+                $openingPcConverted = '<ph ' . ((isset($attr['id'])) ? 'id="' . $attr['id'] . '_1"' : '') . ' dataType="pcStart" originalData="' . $base64StartOriginalData . '" dataRef="'
+                        . $attr['dataRefStart'] . '" equiv-text="base64:'
                         . $base64EncodedStartValue . '"/>';
 
-                $string = str_replace( $startOriginalData, $openingPcConverted, $string );
+                $string = str_replace($startOriginalData, $openingPcConverted, $string);
             }
         }
 
@@ -344,34 +344,34 @@ 
      *
      * @return string
      */
-    private function replaceClosingPcTags( $string, $toBeEscaped, $dataRefEndMap = [] ) {
-        $regex = ( $toBeEscaped ) ? '/&lt;\/pc&gt;/iu' : '/<\/pc>/iu';
-        preg_match_all( $regex, $string, $closingPcMatches, PREG_OFFSET_CAPTURE );
+    private function replaceClosingPcTags($string, $toBeEscaped, $dataRefEndMap = []) {
+        $regex = ($toBeEscaped) ? '/&lt;\/pc&gt;/iu' : '/<\/pc>/iu';
+        preg_match_all($regex, $string, $closingPcMatches, PREG_OFFSET_CAPTURE);
         $delta = 0;
 
-        foreach ( $closingPcMatches[ 0 ] as $index => $match ) {
-            $offset = $match[ 1 ];
-            $length = strlen( $match[ 0 ] );
-            $attr   = $dataRefEndMap[ $index ];
+        foreach ($closingPcMatches[0] as $index => $match) {
+            $offset = $match[1];
+            $length = strlen($match[0]);
+            $attr   = $dataRefEndMap[$index];
 
-            if ( !empty( $attr ) && isset( $attr[ 'dataRefEnd' ] ) ) {
-                $endOriginalData       = $match[ 0 ]; // </pc>
-                $endValue              = $this->map[ $attr[ 'dataRefEnd' ] ] ?: 'NULL';
-                $base64EncodedEndValue = base64_encode( $endValue );
-                $base64EndOriginalData = base64_encode( $endOriginalData );
+            if (!empty($attr) && isset($attr['dataRefEnd'])) {
+                $endOriginalData       = $match[0]; // </pc>
+                $endValue              = $this->map[$attr['dataRefEnd']] ?: 'NULL';
+                $base64EncodedEndValue = base64_encode($endValue);
+                $base64EndOriginalData = base64_encode($endOriginalData);
 
                 // conversion for closing <pc> tag
-                $closingPcConverted = '<ph ' . ( ( isset( $attr[ 'id' ] ) ) ? 'id="' . $attr[ 'id' ] . '_2"' : '' ) . ' dataType="pcEnd" originalData="' . $base64EndOriginalData . '" dataRef="'
-                        . $attr[ 'dataRefEnd' ] . '" equiv-text="base64:' . $base64EncodedEndValue . '"/>';
+                $closingPcConverted = '<ph ' . ((isset($attr['id'])) ? 'id="' . $attr['id'] . '_2"' : '') . ' dataType="pcEnd" originalData="' . $base64EndOriginalData . '" dataRef="'
+                        . $attr['dataRefEnd'] . '" equiv-text="base64:' . $base64EncodedEndValue . '"/>';
 
-                $realOffset = ( $delta === 0 ) ? $offset : ( $offset + $delta );
+                $realOffset = ($delta === 0) ? $offset : ($offset + $delta);
 
-                $string = substr_replace( $string, $closingPcConverted, $realOffset, $length );
-                $delta  = $delta + strlen( $closingPcConverted ) - $length;
+                $string = substr_replace($string, $closingPcConverted, $realOffset, $length);
+                $delta  = $delta + strlen($closingPcConverted) - $length;
             }
         }
 
-        return !is_array( $string ) ? $string : implode( $string );
+        return !is_array($string) ? $string : implode($string);
     }
 
     /**
@@ -379,13 +379,13 @@ 
      *
      * @return bool
      */
-    private function nodeContainsNestedPcTags( $node ) {
-        if ( !$node->has_children ) {
+    private function nodeContainsNestedPcTags($node) {
+        if (!$node->has_children) {
             return false;
         }
 
-        foreach ( $node->inner_html as $nestedNode ) {
-            if ( $nestedNode->tagname === 'pc' && ( isset( $node->attributes[ 'dataRefEnd' ] ) || isset( $node->attributes[ 'dataRefStart' ] ) ) ) {
+        foreach ($node->inner_html as $nestedNode) {
+            if ($nestedNode->tagname === 'pc' && (isset($node->attributes['dataRefEnd']) || isset($node->attributes['dataRefStart']))) {
                 return true;
             }
         }
@@ -398,18 +398,18 @@ 
      *
      * @return string
      */
-    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   = HtmlParser::parse( $string );
+        $string = str_replace(' equiv-text=""', '', $string);
+        $html   = HtmlParser::parse($string);
 
-        foreach ( $html as $node ) {
-            $string = $this->recursiveRemoveOriginalData( $node, $string );
+        foreach ($html as $node) {
+            $string = $this->recursiveRemoveOriginalData($node, $string);
         }
 
         return $string;
@@ -421,76 +421,76 @@ 
      *
      * @return string|string[]
      */
-    private function recursiveRemoveOriginalData( $node, $string ) {
-        if ( $node->has_children ) {
-            foreach ( $node->inner_html as $childNode ) {
-                $string = $this->recursiveRemoveOriginalData( $childNode, $string );
+    private function recursiveRemoveOriginalData($node, $string) {
+        if ($node->has_children) {
+            foreach ($node->inner_html as $childNode) {
+                $string = $this->recursiveRemoveOriginalData($childNode, $string);
             }
         } else {
 
-            if ( !isset( $node->attributes[ 'dataRef' ] ) ) {
+            if (!isset($node->attributes['dataRef'])) {
                 return $string;
             }
 
-            $a = $node->node;                  // complete match. Eg:  <ph id="source1" dataRef="source1"/>
-            $b = $node->attributes[ 'dataRef' ]; // map identifier. Eg: source1
-            $c = $node->terminator;            // terminator: Eg: >
+            $a = $node->node; // complete match. Eg:  <ph id="source1" dataRef="source1"/>
+            $b = $node->attributes['dataRef']; // map identifier. Eg: source1
+            $c = $node->terminator; // terminator: Eg: >
 
             // if isset a value in the map calculate base64 encoded value
             // or it is an empty string
             // otherwise skip
-            if ( !in_array( $b, array_keys( $this->map ) ) ) {
+            if (!in_array($b, array_keys($this->map))) {
                 return $string;
             }
 
             // check if is null, in this case convert it to NULL string
-            if ( is_null( $this->map[ $b ] ) ) {
-                $this->map[ $b ] = 'NULL';
+            if (is_null($this->map[$b])) {
+                $this->map[$b] = 'NULL';
             }
 
             // remove id?
-            $removeId = ( isset( $node->attributes[ 'removeId' ] ) && $node->attributes[ 'removeId' ] === "true" ) ? ' id="' . $b . '" removeId="true"' : '';
+            $removeId = (isset($node->attributes['removeId']) && $node->attributes['removeId'] === "true") ? ' id="' . $b . '" removeId="true"' : '';
 
             // grab dataType attribute for <ec>/<sc> tag handling
-            $dataType = ( $this->wasAEcOrScTag( $node ) ) ? ' dataType="' . $node->attributes[ 'dataType' ] . '"' : '';
+            $dataType = ($this->wasAEcOrScTag($node)) ? ' dataType="' . $node->attributes['dataType'] . '"' : '';
 
-            $d = str_replace( $removeId . $dataType . ' equiv-text="base64:' . base64_encode( $this->map[ $b ] ) . '"/' . $c, '/' . $c, $a );
+            $d = str_replace($removeId . $dataType . ' equiv-text="base64:' . base64_encode($this->map[$b]) . '"/' . $c, '/' . $c, $a);
 
             // replace original <ec>/<sc> tag
-            if ( $this->wasAEcOrScTag( $node ) ) {
-                $d = $node->attributes[ 'dataType' ] . substr( $d, 3 );
-                $d = trim( $d );
+            if ($this->wasAEcOrScTag($node)) {
+                $d = $node->attributes['dataType'] . substr($d, 3);
+                $d = trim($d);
             }
 
             // replace only content tag, no matter if the string is encoded or not
             // in this way we can handle string with mixed tags (encoded and not-encoded)
             // in the same string
-            $a = $this->purgeTags( $a );
-            $d = $this->purgeTags( $d );
+            $a = $this->purgeTags($a);
+            $d = $this->purgeTags($d);
 
-            $string = str_replace( $a, $d, $string );
+            $string = str_replace($a, $d, $string);
 
             // restoring <pc/> self-closed here
-            if ( Strings::contains( 'dataType="pcSelf"', $d ) ) {
-                preg_match( '/\s?originalData="(.*?)"\s?/', $d, $originalDataMatches );
+            if (Strings::contains('dataType="pcSelf"', $d)) {
+                preg_match('/\s?originalData="(.*?)"\s?/', $d, $originalDataMatches);
 
-                if ( isset( $originalDataMatches[ 1 ] ) ) {
-                    $originalData = base64_decode( $originalDataMatches[ 1 ] );
-                    $originalData = $this->purgeTags( $originalData );
-                    $string       = str_replace( $d, $originalData, $string );
+                if (isset($originalDataMatches[1])) {
+                    $originalData = base64_decode($originalDataMatches[1]);
+                    $originalData = $this->purgeTags($originalData);
+                    $string       = str_replace($d, $originalData, $string);
                 }
             }
 
             // restoring <pc> tags here
             // if <ph> tag has originalData and originalType is pcStart or pcEnd,
             // replace with original data
-            if ( Strings::contains( 'dataType="pcStart"', $d ) || Strings::contains( 'dataType="pcEnd"', $d ) ) {
-                preg_match( '/\s?originalData="(.*?)"\s?/', $d, $originalDataMatches );
+            if (Strings::contains('dataType="pcStart"', $d) || Strings::contains('dataType="pcEnd"', $d)) {
+                preg_match('/\s?originalData="(.*?)"\s?/', $d, $originalDataMatches);
 
-                if ( isset( $originalDataMatches[ 1 ] ) ) {
-                    $originalData = base64_decode( $originalDataMatches[ 1 ] );
-                    $originalData = $this->purgeTags( $originalData );
-                    $string       = str_replace( $d, $originalData, $string );
+                if (isset($originalDataMatches[1])) {
+                    $originalData = base64_decode($originalDataMatches[1]);
+                    $originalData = $this->purgeTags($originalData);
+                    $string       = str_replace($d, $originalData, $string);
                 }
             }
         }
@@ -503,8 +503,8 @@ 
      *
      * @return string
      */
-    private function purgeTags( $string ) {
-        return str_replace( [ '<', '>', '&lt;', '&gt;' ], '', $string );
+    private function purgeTags($string) {
+        return str_replace(['<', '>', '&lt;', '&gt;'], '', $string);
     }
 
     /**
@@ -514,8 +514,8 @@ 
      *
      * @return bool
      */
-    private function isAEcOrScTag( $node ) {
-        return ( $node->tagname === 'ec' || $node->tagname === 'sc' );
+    private function isAEcOrScTag($node) {
+        return ($node->tagname === 'ec' || $node->tagname === 'sc');
     }
 
     /**
@@ -526,7 +526,7 @@ 
      *
      * @return bool
      */
-    private function wasAEcOrScTag( $node ) {
-        return ( isset( $node->attributes[ 'dataType' ] ) && ( $node->attributes[ 'dataType' ] === 'ec' || $node->attributes[ 'dataType' ] === 'sc' ) );
+    private function wasAEcOrScTag($node) {
+        return (isset($node->attributes['dataType']) && ($node->attributes['dataType'] === 'ec' || $node->attributes['dataType'] === 'sc'));
     }
 }

src/XliffUtils/XmlParser.php

Spacing +34 added lines, -34 removed lines

@@ -27,65 +27,65 @@ 
      * @throws InvalidXmlException When parsing of XML with schema or callable produces any errors unrelated to the XML parsing itself
      * @throws RuntimeException   When DOM extension is missing
      */
-    public static function parse( $content, $schemaOrCallable = null ) {
-        if ( !extension_loaded( 'dom' ) ) {
-            throw new RuntimeException( 'Extension DOM is required.' );
+    public static function parse($content, $schemaOrCallable = null) {
+        if (!extension_loaded('dom')) {
+            throw new RuntimeException('Extension DOM is required.');
         }
 
-        $internalErrors  = libxml_use_internal_errors( true );
-        $disableEntities = libxml_disable_entity_loader( true );
+        $internalErrors  = libxml_use_internal_errors(true);
+        $disableEntities = libxml_disable_entity_loader(true);
         libxml_clear_errors();
 
-        $dom                  = new DOMDocument( '1.0', 'UTF-8' );
+        $dom                  = new DOMDocument('1.0', 'UTF-8');
         $dom->validateOnParse = true;
-        if ( !$dom->loadXML( $content, LIBXML_NONET | ( defined( 'LIBXML_COMPACT' ) ? LIBXML_COMPACT : 0 ) ) ) {
-            libxml_disable_entity_loader( $disableEntities );
+        if (!$dom->loadXML($content, LIBXML_NONET | (defined('LIBXML_COMPACT') ? LIBXML_COMPACT : 0))) {
+            libxml_disable_entity_loader($disableEntities);
 
-            throw new XmlParsingException( implode( "\n", static::getXmlErrors( $internalErrors ) ) );
+            throw new XmlParsingException(implode("\n", static::getXmlErrors($internalErrors)));
         }
 
         $dom->normalizeDocument();
 
-        libxml_use_internal_errors( $internalErrors );
-        libxml_disable_entity_loader( $disableEntities );
+        libxml_use_internal_errors($internalErrors);
+        libxml_disable_entity_loader($disableEntities);
 
-        foreach ( $dom->childNodes as $child ) {
-            if ( XML_DOCUMENT_TYPE_NODE === $child->nodeType ) {
-                throw new XmlParsingException( 'Document types are not allowed.' );
+        foreach ($dom->childNodes as $child) {
+            if (XML_DOCUMENT_TYPE_NODE === $child->nodeType) {
+                throw new XmlParsingException('Document types are not allowed.');
             }
         }
 
-        if ( null !== $schemaOrCallable ) {
-            $internalErrors = libxml_use_internal_errors( true );
+        if (null !== $schemaOrCallable) {
+            $internalErrors = libxml_use_internal_errors(true);
             libxml_clear_errors();
 
             $e = null;
-            if ( is_callable( $schemaOrCallable ) ) {
+            if (is_callable($schemaOrCallable)) {
                 try {
-                    $valid = call_user_func( $schemaOrCallable, $dom, $internalErrors );
-                } catch ( Exception $e ) {
+                    $valid = call_user_func($schemaOrCallable, $dom, $internalErrors);
+                } catch (Exception $e) {
                     $valid = false;
                 }
-            } elseif ( !is_array( $schemaOrCallable ) && is_file( (string)$schemaOrCallable ) ) {
-                $schemaSource = file_get_contents( (string)$schemaOrCallable );
-                $valid        = @$dom->schemaValidateSource( $schemaSource );
+            } elseif (!is_array($schemaOrCallable) && is_file((string)$schemaOrCallable)) {
+                $schemaSource = file_get_contents((string)$schemaOrCallable);
+                $valid        = @$dom->schemaValidateSource($schemaSource);
             } else {
-                libxml_use_internal_errors( $internalErrors );
+                libxml_use_internal_errors($internalErrors);
 
-                throw new XmlParsingException( 'The schemaOrCallable argument has to be a valid path to XSD file or callable.' );
+                throw new XmlParsingException('The schemaOrCallable argument has to be a valid path to XSD file or callable.');
             }
 
-            if ( !$valid ) {
-                $messages = static::getXmlErrors( $internalErrors );
-                if ( empty( $messages ) ) {
-                    throw new InvalidXmlException( 'The XML is not valid.', 0, $e );
+            if (!$valid) {
+                $messages = static::getXmlErrors($internalErrors);
+                if (empty($messages)) {
+                    throw new InvalidXmlException('The XML is not valid.', 0, $e);
                 }
-                throw new XmlParsingException( implode( "\n", $messages ), 0, $e );
+                throw new XmlParsingException(implode("\n", $messages), 0, $e);
             }
         }
 
         libxml_clear_errors();
-        libxml_use_internal_errors( $internalErrors );
+        libxml_use_internal_errors($internalErrors);
 
         return $dom;
     }
@@ -95,14 +95,14 @@ 
      *
      * @return array
      */
-    private static function getXmlErrors( $internalErrors ) {
+    private static function getXmlErrors($internalErrors) {
         $errors = [];
-        foreach ( libxml_get_errors() as $error ) {
+        foreach (libxml_get_errors() as $error) {
             $errors[] = sprintf(
                     '[%s %s] %s (in %s - line %d, column %d)',
                     LIBXML_ERR_WARNING == $error->level ? 'WARNING' : 'ERROR',
                     $error->code,
-                    trim( $error->message ),
+                    trim($error->message),
                     $error->file ?: 'n/a',
                     $error->line,
                     $error->column
@@ -110,7 +110,7 @@ 
         }
 
         libxml_clear_errors();
-        libxml_use_internal_errors( $internalErrors );
+        libxml_use_internal_errors($internalErrors);
 
         return $errors;
     }

src/XliffReplacer/SdlXliffSAXTranslationReplacer.php

Spacing +19 added lines, -19 removed lines

@@ -19,15 +19,15 @@ 
 
             // get id
             // trim to first 100 characters because this is the limit on Matecat's DB
-            $this->currentTransUnitId = substr($attr[ 'id' ], 0, 100);
+            $this->currentTransUnitId = substr($attr['id'], 0, 100);
 
             // current 'translate' attribute of the current trans-unit
-            $this->currentTransUnitTranslate = isset($attr[ 'translate' ]) ? $attr[ 'translate' ] : 'yes';
+            $this->currentTransUnitTranslate = isset($attr['translate']) ? $attr['translate'] : 'yes';
         }
 
         // check if we are entering into a <target>
         if ('target' == $name) {
-            if($this->currentTransUnitTranslate === 'no'){
+            if ($this->currentTransUnitTranslate === 'no') {
                 $this->inTarget = false;
             } else {
                 $this->inTarget = true;
@@ -64,10 +64,10 @@ 
                 } elseif ('sdl:seg' == $name) {
 
                     // write the confidence level for this segment ( Translated, Draft, etc. )
-                    if (isset($this->segments[ 'matecat|' . $this->currentTransUnitId ]) && $_sdlStatus_confWritten === false) {
+                    if (isset($this->segments['matecat|' . $this->currentTransUnitId]) && $_sdlStatus_confWritten === false) {
 
                         // append definition attribute
-                        $tag .= $this->prepareTargetStatuses($this->lastTransUnit[ $this->markerPos ]);
+                        $tag .= $this->prepareTargetStatuses($this->lastTransUnit[$this->markerPos]);
 
                         //prepare for an eventual next cycle
                         $this->markerPos++;
@@ -94,15 +94,15 @@ 
             // is outside current buffer (in the latter case, it's in next buffer to be read by the while loop);
             // this check is necessary because we may have truncated a tag in half with current read,
             // and the other half may be encountered in the next buffer it will be passed
-            if (isset($this->currentBuffer[ $idx - $this->offset ])) {
+            if (isset($this->currentBuffer[$idx - $this->offset])) {
                 // if this tag entire lenght fitted in the buffer, the last char must be the last
                 // symbol before the '>'; if it's an empty tag, it is assumed that it's a '/'
                 $tmp_offset = $idx - $this->offset;
-                $lastChar   = $this->currentBuffer[ $tmp_offset ];
+                $lastChar   = $this->currentBuffer[$tmp_offset];
             } else {
                 //if it's out, simple use the last character of the chunk
                 $tmp_offset = $this->len - 1;
-                $lastChar   = $this->currentBuffer[ $tmp_offset ];
+                $lastChar   = $this->currentBuffer[$tmp_offset];
             }
 
             // trim last space
@@ -136,7 +136,7 @@ 
                 } else {
                     //these are NOT source/seg-source/value empty tags, THERE IS A CONTENT, write it in buffer
                     $this->bufferIsActive = true;
-                    $this->CDATABuffer    .= $tag;
+                    $this->CDATABuffer .= $tag;
                 }
             } else {
                 $this->postProcAndFlush($this->outputFP, $tag);
@@ -156,11 +156,11 @@ 
     {
         $endTags = "";
 
-        $segment     = Strings::removeDangerousChars($seg [ 'segment' ]);
-        $translation = Strings::removeDangerousChars($seg [ 'translation' ]);
+        $segment     = Strings::removeDangerousChars($seg ['segment']);
+        $translation = Strings::removeDangerousChars($seg ['translation']);
         $dataRefMap  = (isset($seg['data_ref_map']) && $seg['data_ref_map'] !== null) ? Strings::jsonToArray($seg['data_ref_map']) : [];
 
-        if (is_null($seg [ 'translation' ]) || $seg [ 'translation' ] == '') {
+        if (is_null($seg ['translation']) || $seg ['translation'] == '') {
             $translation = $segment;
         } else {
             if ($this->callback) {
@@ -173,20 +173,20 @@ 
         // for Trados the trailing spaces after </mrk> are meaningful
         // so we trim the translation from Matecat DB and add them after </mrk>
         $trailingSpaces = '';
-        for ($s=0; $s < Strings::getTheNumberOfTrailingSpaces($translation); $s++){
+        for ($s = 0; $s < Strings::getTheNumberOfTrailingSpaces($translation); $s++) {
             $trailingSpaces .= ' ';
         }
 
-        if ($seg[ 'mrk_id' ] !== null && $seg[ 'mrk_id' ] != '') {
+        if ($seg['mrk_id'] !== null && $seg['mrk_id'] != '') {
             if ($this->targetLang === 'ja-JP') {
-                $seg[ 'mrk_succ_tags' ] = ltrim($seg[ 'mrk_succ_tags' ]);
+                $seg['mrk_succ_tags'] = ltrim($seg['mrk_succ_tags']);
             }
 
-            $translation = "<mrk mid=\"" . $seg[ 'mrk_id' ] . "\" mtype=\"seg\">" . $seg[ 'mrk_prev_tags' ] . rtrim($translation) . $seg[ 'mrk_succ_tags' ] . "</mrk>" . $trailingSpaces;
+            $translation = "<mrk mid=\"" . $seg['mrk_id'] . "\" mtype=\"seg\">" . $seg['mrk_prev_tags'] . rtrim($translation) . $seg['mrk_succ_tags'] . "</mrk>" . $trailingSpaces;
         }
 
         // we need to trim succ_tags here because we already added the trailing spaces after </mrk>
-        $transUnitTranslation .= $seg[ 'prev_tags' ] . $translation . $endTags . ltrim($seg[ 'succ_tags' ]);
+        $transUnitTranslation .= $seg['prev_tags'] . $translation . $endTags . ltrim($seg['succ_tags']);
 
         return $transUnitTranslation;
     }
@@ -206,7 +206,7 @@ 
             'REJECTED'   => 'RejectedTranslation',
         ];
 
-        return "conf=\"{$statusMap[ $segment[ 'status' ] ]}\" ";
+        return "conf=\"{$statusMap[$segment['status']]}\" ";
     }
 
     /**
@@ -218,7 +218,7 @@ 
      */
     protected function setTransUnitState($seg, $state_prop, $lastMrkState)
     {
-        return [ null, null ];
+        return [null, null];
     }
 
     /**

src/XliffReplacer/AbstractXliffReplacer.php

Spacing +52 added lines, -52 removed lines

@@ -8,28 +8,28 @@ 
 abstract class AbstractXliffReplacer {
     protected $originalFP;
 
-    protected $tuTagName;                 // <trans-unit> (forXliff v 1.*) or <unit> (forXliff v 2.*)
-    protected $inTU                = false;  // flag to check whether we are in a <trans-unit>
-    protected $inTarget            = false;  // flag to check whether we are in a <target>, to ignore everything
-    protected $isEmpty             = false;  // flag to check whether we are in an empty tag (<tag/>)
-    protected $targetWasWritten    = false;  // flag to check is <target> was written in the current unit
-    protected $segmentPositionInTu = -1;  // the current position of segment in the current <unit> (forXliff v 2.*)
-
-    protected $CDATABuffer    = "";       // buffer for special tag
-    protected $bufferIsActive = false;    // buffer for special tag
-
-    protected $offset              = 0;         // offset for SAX pointer
-    protected $outputFP;                  // output stream pointer
-    protected $currentBuffer;             // the current piece of text it's been parsed
-    protected $len;                       // length of the currentBuffer
-    protected $segments;                  // array of translations
+    protected $tuTagName; // <trans-unit> (forXliff v 1.*) or <unit> (forXliff v 2.*)
+    protected $inTU                = false; // flag to check whether we are in a <trans-unit>
+    protected $inTarget            = false; // flag to check whether we are in a <target>, to ignore everything
+    protected $isEmpty             = false; // flag to check whether we are in an empty tag (<tag/>)
+    protected $targetWasWritten    = false; // flag to check is <target> was written in the current unit
+    protected $segmentPositionInTu = -1; // the current position of segment in the current <unit> (forXliff v 2.*)
+
+    protected $CDATABuffer    = ""; // buffer for special tag
+    protected $bufferIsActive = false; // buffer for special tag
+
+    protected $offset = 0; // offset for SAX pointer
+    protected $outputFP; // output stream pointer
+    protected $currentBuffer; // the current piece of text it's been parsed
+    protected $len; // length of the currentBuffer
+    protected $segments; // array of translations
     protected $lastTransUnit       = [];
-    protected $currentTransUnitId;        // id of current <trans-unit>
+    protected $currentTransUnitId; // id of current <trans-unit>
     protected $currentTransUnitTranslate; // 'translate' attribute of current <trans-unit>
-    protected $currentSegmentArray = [];  // id of current <segment> (forXliff v 2.*)
-    protected $unitContainsMda     = false;   // check if <unit> already contains a <mda:metadata> (forXliff v 2.*)
-    protected $hasWrittenCounts    = false;  // check if <unit> already wrote segment counts (forXliff v 2.*)
-    protected $sourceAttributes    = [];     // current <source> attributes (needed when handling xliff files without <target>)
+    protected $currentSegmentArray = []; // id of current <segment> (forXliff v 2.*)
+    protected $unitContainsMda     = false; // check if <unit> already contains a <mda:metadata> (forXliff v 2.*)
+    protected $hasWrittenCounts    = false; // check if <unit> already wrote segment counts (forXliff v 2.*)
+    protected $sourceAttributes    = []; // current <source> attributes (needed when handling xliff files without <target>)
 
     protected $targetLang;
 
@@ -76,8 +76,8 @@ 
             XliffReplacerCallbackInterface $callback = null
     ) {
         self::$INTERNAL_TAG_PLACEHOLDER = $this->getInternalTagPlaceholder();
-        $this->createOutputFileIfDoesNotExist( $outputFilePath );
-        $this->setFileDescriptors( $originalXliffPath, $outputFilePath );
+        $this->createOutputFileIfDoesNotExist($outputFilePath);
+        $this->setFileDescriptors($originalXliffPath, $outputFilePath);
         $this->xliffVersion = $xliffVersion;
         $this->setTuTagName();
         $this->segments       = $segments;
@@ -95,19 +95,19 @@ 
         return "§" .
                 substr(
                         str_replace(
-                                [ '+', '/' ],
+                                ['+', '/'],
                                 '',
-                                base64_encode( openssl_random_pseudo_bytes( 10, $_crypto_strong ) )
+                                base64_encode(openssl_random_pseudo_bytes(10, $_crypto_strong))
                         ),
                         0,
                         4
                 );
     }
 
-    private function createOutputFileIfDoesNotExist( $outputFilePath ) {
+    private function createOutputFileIfDoesNotExist($outputFilePath) {
         // create output file
-        if ( !file_exists( $outputFilePath ) ) {
-            touch( $outputFilePath );
+        if (!file_exists($outputFilePath)) {
+            touch($outputFilePath);
         }
     }
 
@@ -115,13 +115,13 @@ 
      * @param $originalXliffPath
      * @param $outputFilePath
      */
-    private function setFileDescriptors( $originalXliffPath, $outputFilePath ) {
-        $this->outputFP = fopen( $outputFilePath, 'w+' );
+    private function setFileDescriptors($originalXliffPath, $outputFilePath) {
+        $this->outputFP = fopen($outputFilePath, 'w+');
 
         $streamArgs = null;
 
-        if ( !( $this->originalFP = fopen( $originalXliffPath, "r", false, stream_context_create( $streamArgs ) ) ) ) {
-            throw new RuntimeException( "could not open XML input" );
+        if (!($this->originalFP = fopen($originalXliffPath, "r", false, stream_context_create($streamArgs)))) {
+            throw new RuntimeException("could not open XML input");
         }
     }
 
@@ -130,7 +130,7 @@ 
      * <trans-unit> (xliff v1.*) || <unit> (xliff v2.*)
      */
     private function setTuTagName() {
-        $this->tuTagName = ( $this->xliffVersion === 2 ) ? 'unit' : 'trans-unit';
+        $this->tuTagName = ($this->xliffVersion === 2) ? 'unit' : 'trans-unit';
     }
 
     /**
@@ -139,8 +139,8 @@ 
     public function __destruct() {
         //this stream can be closed outside the class
         //to permit multiple concurrent downloads, so suppress warnings
-        @fclose( $this->originalFP );
-        fclose( $this->outputFP );
+        @fclose($this->originalFP);
+        fclose($this->outputFP);
     }
 
     abstract public function replaceTranslation();
@@ -151,11 +151,11 @@ 
      * @return resource
      */
     protected function initSaxParser() {
-        $xmlSaxParser = xml_parser_create( 'UTF-8' );
-        xml_set_object( $xmlSaxParser, $this );
-        xml_parser_set_option( $xmlSaxParser, XML_OPTION_CASE_FOLDING, false );
-        xml_set_element_handler( $xmlSaxParser, 'tagOpen', 'tagClose' );
-        xml_set_character_data_handler( $xmlSaxParser, 'characterData' );
+        $xmlSaxParser = xml_parser_create('UTF-8');
+        xml_set_object($xmlSaxParser, $this);
+        xml_parser_set_option($xmlSaxParser, XML_OPTION_CASE_FOLDING, false);
+        xml_set_element_handler($xmlSaxParser, 'tagOpen', 'tagClose');
+        xml_set_character_data_handler($xmlSaxParser, 'characterData');
 
         return $xmlSaxParser;
     }
@@ -163,8 +163,8 @@ 
     /**
      * @param resource $xmlSaxParser
      */
-    protected function closeSaxParser( $xmlSaxParser ) {
-        xml_parser_free( $xmlSaxParser );
+    protected function closeSaxParser($xmlSaxParser) {
+        xml_parser_free($xmlSaxParser);
     }
 
     /**
@@ -174,7 +174,7 @@ 
      *
      * @return mixed
      */
-    abstract protected function tagOpen( $parser, $name, $attr );
+    abstract protected function tagOpen($parser, $name, $attr);
 
     /**
      * @param $parser
@@ -182,7 +182,7 @@ 
      *
      * @return mixed
      */
-    abstract protected function tagClose( $parser, $name );
+    abstract protected function tagClose($parser, $name);
 
     /**
      * @param $parser
@@ -190,7 +190,7 @@ 
      *
      * @return mixed
      */
-    abstract protected function characterData( $parser, $data );
+    abstract protected function characterData($parser, $data);
 
     /**
      * postprocess escaped data and write to disk
@@ -199,18 +199,18 @@ 
      * @param string   $data
      * @param bool     $treatAsCDATA
      */
-    protected function postProcAndFlush( $fp, $data, $treatAsCDATA = false ) {
+    protected function postProcAndFlush($fp, $data, $treatAsCDATA = false) {
         //postprocess string
-        $data = preg_replace( "/" . self::$INTERNAL_TAG_PLACEHOLDER . '(.*?)' . self::$INTERNAL_TAG_PLACEHOLDER . "/", '&$1;', $data );
-        $data = str_replace( '&nbsp;', ' ', $data );
-        if ( !$treatAsCDATA ) {
+        $data = preg_replace("/" . self::$INTERNAL_TAG_PLACEHOLDER . '(.*?)' . self::$INTERNAL_TAG_PLACEHOLDER . "/", '&$1;', $data);
+        $data = str_replace('&nbsp;', ' ', $data);
+        if (!$treatAsCDATA) {
             //unix2dos
-            $data = str_replace( "\r\n", "\r", $data );
-            $data = str_replace( "\n", "\r", $data );
-            $data = str_replace( "\r", "\r\n", $data );
+            $data = str_replace("\r\n", "\r", $data);
+            $data = str_replace("\n", "\r", $data);
+            $data = str_replace("\r", "\r\n", $data);
         }
 
         //flush to disk
-        fwrite( $fp, $data );
+        fwrite($fp, $data);
     }
 }

src/XliffReplacer/XliffSAXTranslationReplacer.php

Spacing +191 added lines, -191 removed lines

@@ -31,17 +31,17 @@ 
     ];
 
     public function replaceTranslation() {
-        fwrite( $this->outputFP, '<?xml version="1.0" encoding="UTF-8"?>' );
+        fwrite($this->outputFP, '<?xml version="1.0" encoding="UTF-8"?>');
 
         //create Sax parser
         $xmlParser = $this->initSaxParser();
 
-        while ( $this->currentBuffer = fread( $this->originalFP, 4096 ) ) {
+        while ($this->currentBuffer = fread($this->originalFP, 4096)) {
             /*
                preprocess file
              */
             // obfuscate entities because sax automatically does html_entity_decode
-            $temporary_check_buffer = preg_replace( "/&(.*?);/", self::$INTERNAL_TAG_PLACEHOLDER . '$1' . self::$INTERNAL_TAG_PLACEHOLDER, $this->currentBuffer );
+            $temporary_check_buffer = preg_replace("/&(.*?);/", self::$INTERNAL_TAG_PLACEHOLDER . '$1' . self::$INTERNAL_TAG_PLACEHOLDER, $this->currentBuffer);
 
             //avoid cutting entities in half:
             //the last fread could have truncated an entity (say, '&lt;' in '&l'), thus invalidating the escaping
@@ -53,84 +53,84 @@ 
             // add 9 Bytes and substitute the entities, if the & is present, and it is not at the end
             //it can't be an entity, exit the loop
 
-            while ( true ) {
-                $_ampPos = strpos( $temporary_check_buffer, '&' );
+            while (true) {
+                $_ampPos = strpos($temporary_check_buffer, '&');
 
                 //check for real entity or escape it to safely exit from the loop!!!
-                if ( $_ampPos === false || strlen( substr( $temporary_check_buffer, $_ampPos ) ) > 9 ) {
+                if ($_ampPos === false || strlen(substr($temporary_check_buffer, $_ampPos)) > 9) {
                     $escape_AMP = true;
                     break;
                 }
 
                 //if an entity is still present, fetch some more and repeat the escaping
-                $this->currentBuffer    .= fread( $this->originalFP, 9 );
-                $temporary_check_buffer = preg_replace( "/&(.*?);/", self::$INTERNAL_TAG_PLACEHOLDER . '$1' . self::$INTERNAL_TAG_PLACEHOLDER, $this->currentBuffer );
+                $this->currentBuffer .= fread($this->originalFP, 9);
+                $temporary_check_buffer = preg_replace("/&(.*?);/", self::$INTERNAL_TAG_PLACEHOLDER . '$1' . self::$INTERNAL_TAG_PLACEHOLDER, $this->currentBuffer);
             }
 
             //free stuff outside the loop
-            unset( $temporary_check_buffer );
+            unset($temporary_check_buffer);
 
-            $this->currentBuffer = preg_replace( "/&(.*?);/", self::$INTERNAL_TAG_PLACEHOLDER . '$1' . self::$INTERNAL_TAG_PLACEHOLDER, $this->currentBuffer );
-            if ( $escape_AMP ) {
-                $this->currentBuffer = str_replace( "&", self::$INTERNAL_TAG_PLACEHOLDER . 'amp' . self::$INTERNAL_TAG_PLACEHOLDER, $this->currentBuffer );
+            $this->currentBuffer = preg_replace("/&(.*?);/", self::$INTERNAL_TAG_PLACEHOLDER . '$1' . self::$INTERNAL_TAG_PLACEHOLDER, $this->currentBuffer);
+            if ($escape_AMP) {
+                $this->currentBuffer = str_replace("&", self::$INTERNAL_TAG_PLACEHOLDER . 'amp' . self::$INTERNAL_TAG_PLACEHOLDER, $this->currentBuffer);
             }
 
             //get length of chunk
-            $this->len = strlen( $this->currentBuffer );
+            $this->len = strlen($this->currentBuffer);
 
             //parse chunk of text
-            if ( !xml_parse( $xmlParser, $this->currentBuffer, feof( $this->originalFP ) ) ) {
+            if (!xml_parse($xmlParser, $this->currentBuffer, feof($this->originalFP))) {
                 //if unable, raise an exception
-                throw new RuntimeException( sprintf(
+                throw new RuntimeException(sprintf(
                         "XML error: %s at line %d",
-                        xml_error_string( xml_get_error_code( $xmlParser ) ),
-                        xml_get_current_line_number( $xmlParser )
-                ) );
+                        xml_error_string(xml_get_error_code($xmlParser)),
+                        xml_get_current_line_number($xmlParser)
+                ));
             }
             //get accumulated this->offset in document: as long as SAX pointer advances, we keep track of total bytes it has seen so far; this way, we can translate its global pointer in an address local to the current buffer of text to retrieve last char of tag
             $this->offset += $this->len;
         }
 
         // close Sax parser
-        $this->closeSaxParser( $xmlParser );
+        $this->closeSaxParser($xmlParser);
 
     }
 
     /**
      * @inheritDoc
      */
-    protected function tagOpen( $parser, $name, $attr ) {
+    protected function tagOpen($parser, $name, $attr) {
         // check if we are entering into a <trans-unit> (xliff v1.*) or <unit> (xliff v2.*)
-        if ( $this->tuTagName === $name ) {
+        if ($this->tuTagName === $name) {
             $this->inTU = true;
 
             // get id
             // trim to first 100 characters because this is the limit on Matecat's DB
-            $this->currentTransUnitId = substr( $attr[ 'id' ], 0, 100 );
+            $this->currentTransUnitId = substr($attr['id'], 0, 100);
 
             // `translate` attribute can be only yes or no
-            if ( isset( $attr[ 'translate' ] ) && $attr[ 'translate' ] === 'no' ) {
-                $attr[ 'translate' ] = 'no';
+            if (isset($attr['translate']) && $attr['translate'] === 'no') {
+                $attr['translate'] = 'no';
             } else {
-                $attr[ 'translate' ] = 'yes';
+                $attr['translate'] = 'yes';
             }
 
             // current 'translate' attribute of the current trans-unit
-            $this->currentTransUnitTranslate = $attr[ 'translate' ];
+            $this->currentTransUnitTranslate = $attr['translate'];
         }
 
-        if ( 'source' === $name ) {
+        if ('source' === $name) {
             $this->sourceAttributes = $attr;
         }
 
-        if ( 'mda:metadata' === $name ) {
+        if ('mda:metadata' === $name) {
             $this->unitContainsMda = true;
         }
 
         // check if we are entering into a <target>
-        if ( 'target' === $name ) {
+        if ('target' === $name) {
 
-            if ( $this->currentTransUnitTranslate === 'no' ) {
+            if ($this->currentTransUnitTranslate === 'no') {
                 $this->inTarget = false;
             } else {
                 $this->inTarget = true;
@@ -139,7 +139,7 @@ 
 
         // check if we are inside a <target>, obviously this happen only if there are targets inside the trans-unit
         // <target> must be stripped to be replaced, so this check avoids <target> reconstruction
-        if ( !$this->inTarget ) {
+        if (!$this->inTarget) {
 
             $tag = '';
 
@@ -161,14 +161,14 @@ 
             //
             // http://docs.oasis-open.org/xliff/xliff-core/v2.0/os/xliff-core-v2.0-os.html#unit
             //
-            if ( $this->xliffVersion === 2 && ( $name === 'notes' || $name === 'originalData' || $name === 'segment' || $name === 'ignorable' ) && $this->unitContainsMda === false ) {
-                if ( isset( $this->transUnits[ $this->currentTransUnitId ] ) && !empty( $this->transUnits[ $this->currentTransUnitId ] ) && !$this->hasWrittenCounts ) {
+            if ($this->xliffVersion === 2 && ($name === 'notes' || $name === 'originalData' || $name === 'segment' || $name === 'ignorable') && $this->unitContainsMda === false) {
+                if (isset($this->transUnits[$this->currentTransUnitId]) && !empty($this->transUnits[$this->currentTransUnitId]) && !$this->hasWrittenCounts) {
 
                     // we need to update counts here
                     $this->updateCounts();
                     $this->hasWrittenCounts = true;
 
-                    $tag                   .= $this->getWordCountGroupForXliffV2( $this->counts[ 'raw_word_count' ], $this->counts[ 'eq_word_count' ] );
+                    $tag                   .= $this->getWordCountGroupForXliffV2($this->counts['raw_word_count'], $this->counts['eq_word_count']);
                     $this->unitContainsMda = true;
                 }
             }
@@ -179,88 +179,88 @@ 
             $lastMrkState = null;
             $stateProp    = '';
 
-            foreach ( $attr as $k => $v ) {
+            foreach ($attr as $k => $v) {
 
                 //if tag name is file, we must replace the target-language attribute
-                if ( $name === 'file' && $k === 'target-language' && !empty( $this->targetLang ) ) {
+                if ($name === 'file' && $k === 'target-language' && !empty($this->targetLang)) {
                     //replace Target language with job language provided from constructor
                     $tag .= "$k=\"$this->targetLang\" ";
                 } else {
                     $pos = 0;
-                    if ( $this->currentTransUnitId ) {
-                        $pos = current( $this->transUnits[ $this->currentTransUnitId ] );
+                    if ($this->currentTransUnitId) {
+                        $pos = current($this->transUnits[$this->currentTransUnitId]);
                     }
 
-                    if ( $name === $this->tuTagName ) {
+                    if ($name === $this->tuTagName) {
 
-                        $sid = $this->segments[ $pos ][ 'sid' ];
+                        $sid = $this->segments[$pos]['sid'];
 
                         // add `help-id` to xliff v.1*
                         // add `mtc:segment-id` to xliff v.2*
-                        if ( $this->xliffVersion === 1 && strpos( $tag, 'help-id' ) === false ) {
-                            if ( !empty( $sid ) ) {
+                        if ($this->xliffVersion === 1 && strpos($tag, 'help-id') === false) {
+                            if (!empty($sid)) {
                                 $tag .= "help-id=\"$sid\" ";
                             }
-                        } elseif ( $this->xliffVersion === 2 && strpos( $tag, 'mtc:segment-id' ) === false ) {
-                            if ( !empty( $sid ) ) {
+                        } elseif ($this->xliffVersion === 2 && strpos($tag, 'mtc:segment-id') === false) {
+                            if (!empty($sid)) {
                                 $tag .= "mtc:segment-id=\"$sid\" ";
                             }
                         }
 
-                    } elseif ( 'segment' === $name && $this->xliffVersion === 2 ) { // add state to segment in Xliff v2
-                        list( $stateProp, $lastMrkState ) = $this->setTransUnitState( $this->segments[ $pos ], $stateProp, $lastMrkState );
+                    } elseif ('segment' === $name && $this->xliffVersion === 2) { // add state to segment in Xliff v2
+                        list($stateProp, $lastMrkState) = $this->setTransUnitState($this->segments[$pos], $stateProp, $lastMrkState);
                     }
 
                     //normal tag flux, put attributes in it
                     $tag .= "$k=\"$v\" ";
 
                     // replace state for xliff v2
-                    if ( $stateProp ) {
+                    if ($stateProp) {
                         $pattern = '/state=\"(.*)\"/i';
-                        $tag     = preg_replace( $pattern, $stateProp, $tag );
+                        $tag     = preg_replace($pattern, $stateProp, $tag);
                     }
                 }
             }
 
             // add oasis xliff 20 namespace
-            if ( $this->xliffVersion === 2 && $name === 'xliff' && !array_key_exists( 'xmlns:mda', $attr ) ) {
+            if ($this->xliffVersion === 2 && $name === 'xliff' && !array_key_exists('xmlns:mda', $attr)) {
                 $tag .= 'xmlns:mda="urn:oasis:names:tc:xliff:metadata:2.0"';
             }
 
             // add MateCat specific namespace, we want maybe add non-XLIFF attributes
-            if ( $name === 'xliff' && !array_key_exists( 'xmlns:mtc', $attr ) ) {
+            if ($name === 'xliff' && !array_key_exists('xmlns:mtc', $attr)) {
                 $tag .= ' xmlns:mtc="https://www.matecat.com" ';
             }
 
             // trgLang
-            if ( $name === 'xliff' ) {
-                $tag = preg_replace( '/trgLang="(.*?)"/', 'trgLang="' . $this->targetLang . '"', $tag );
+            if ($name === 'xliff') {
+                $tag = preg_replace('/trgLang="(.*?)"/', 'trgLang="' . $this->targetLang . '"', $tag);
             }
 
             //this logic helps detecting empty tags
             //get current position of SAX pointer in all the stream of data is has read so far:
             //it points at the end of current tag
-            $idx = xml_get_current_byte_index( $parser );
+            $idx = xml_get_current_byte_index($parser);
 
             //check whether the bounds of current tag are entirely in current buffer or the end of the current tag
             //is outside current buffer (in the latter case, it's in next buffer to be read by the while loop);
             //this check is necessary because we may have truncated a tag in half with current read,
             //and the other half may be encountered in the next buffer it will be passed
-            if ( isset( $this->currentBuffer[ $idx - $this->offset ] ) ) {
+            if (isset($this->currentBuffer[$idx - $this->offset])) {
                 //if this tag entire lenght fitted in the buffer, the last char must be the last
                 //symbol before the '>'; if it's an empty tag, it is assumed that it's a '/'
-                $lastChar = $this->currentBuffer[ $idx - $this->offset ];
+                $lastChar = $this->currentBuffer[$idx - $this->offset];
             } else {
                 //if it's out, simple use the last character of the chunk
-                $lastChar = $this->currentBuffer[ $this->len - 1 ];
+                $lastChar = $this->currentBuffer[$this->len - 1];
             }
 
             //trim last space
-            $tag = rtrim( $tag );
+            $tag = rtrim($tag);
 
             //detect empty tag
-            $this->isEmpty = ( $lastChar == '/' || $name == 'x' );
-            if ( $this->isEmpty ) {
+            $this->isEmpty = ($lastChar == '/' || $name == 'x');
+            if ($this->isEmpty) {
                 $tag .= '/';
             }
 
@@ -268,29 +268,29 @@ 
             $tag .= ">";
 
             //set a a Buffer for the segSource Source tag
-            if ( $this->bufferIsActive || in_array( $name, $this->nodesToCopy ) ) { // we are opening a critical CDATA section
+            if ($this->bufferIsActive || in_array($name, $this->nodesToCopy)) { // we are opening a critical CDATA section
 
                 //WARNING BECAUSE SOURCE AND SEG-SOURCE TAGS CAN BE EMPTY IN SOME CASES!!!!!
                 //so check for isEmpty also in conjunction with name
-                if ( $this->isEmpty && ( 'source' === $name || 'seg-source' === $name ) ) {
-                    $this->postProcAndFlush( $this->outputFP, $tag );
+                if ($this->isEmpty && ('source' === $name || 'seg-source' === $name)) {
+                    $this->postProcAndFlush($this->outputFP, $tag);
                 } else {
                     //these are NOT source/seg-source/value empty tags, THERE IS A CONTENT, write it in buffer
                     $this->bufferIsActive = true;
-                    $this->CDATABuffer    .= $tag;
+                    $this->CDATABuffer .= $tag;
                 }
             } else {
-                $this->postProcAndFlush( $this->outputFP, $tag );
+                $this->postProcAndFlush($this->outputFP, $tag);
             }
         }
 
         // update segmentPositionInTu
 
-        if ( $this->xliffVersion === 1 && $this->inTU && $name === 'source' ) {
+        if ($this->xliffVersion === 1 && $this->inTU && $name === 'source') {
             $this->segmentPositionInTu++;
         }
 
-        if ( $this->xliffVersion === 2 && $this->inTU && $name === 'segment' ) {
+        if ($this->xliffVersion === 2 && $this->inTU && $name === 'segment') {
             $this->segmentPositionInTu++;
         }
     }
@@ -298,7 +298,7 @@ 
     /**
      * @inheritDoc
      */
-    protected function tagClose( $parser, $name ) {
+    protected function tagClose($parser, $name) {
         $tag = '';
 
         /**
@@ -307,26 +307,26 @@ 
          *
          * self::tagOpen method
          */
-        if ( !$this->isEmpty && !( $this->inTarget && $name !== 'target' ) ) {
+        if (!$this->isEmpty && !($this->inTarget && $name !== 'target')) {
 
-            if ( !$this->inTarget ) {
+            if (!$this->inTarget) {
                 $tag = "</$name>";
             }
 
-            if ( 'target' == $name ) {
+            if ('target' == $name) {
 
-                if ( $this->currentTransUnitTranslate === 'no' ) {
+                if ($this->currentTransUnitTranslate === 'no') {
                     // do nothing
-                } elseif ( isset( $this->transUnits[ $this->currentTransUnitId ] ) ) {
+                } elseif (isset($this->transUnits[$this->currentTransUnitId])) {
 
                     // get translation of current segment, by indirect indexing: id -> positional index -> segment
                     // actually there may be more that one segment to that ID if there are two mrk of the same source segment
 
-                    $listOfSegmentsIds = $this->transUnits[ $this->currentTransUnitId ];
+                    $listOfSegmentsIds = $this->transUnits[$this->currentTransUnitId];
 
                     // $currentSegmentId
-                    if ( !empty( $listOfSegmentsIds ) ) {
-                        $this->setCurrentSegmentArray( $listOfSegmentsIds );
+                    if (!empty($listOfSegmentsIds)) {
+                        $this->setCurrentSegmentArray($listOfSegmentsIds);
                     }
 
                     /*
@@ -343,12 +343,12 @@ 
                     $this->lastTransUnit = [];
 
                     $last_value    = null;
-                    $segmentsCount = count( $listOfSegmentsIds );
-                    for ( $i = 0; $i < $segmentsCount; $i++ ) {
-                        $id = $listOfSegmentsIds[ $i ];
-                        if ( isset( $this->segments[ $id ] ) && ( $i == 0 || $last_value + 1 == $listOfSegmentsIds[ $i ] ) ) {
-                            $last_value            = $listOfSegmentsIds[ $i ];
-                            $this->lastTransUnit[] = $this->segments[ $id ];
+                    $segmentsCount = count($listOfSegmentsIds);
+                    for ($i = 0; $i < $segmentsCount; $i++) {
+                        $id = $listOfSegmentsIds[$i];
+                        if (isset($this->segments[$id]) && ($i == 0 || $last_value + 1 == $listOfSegmentsIds[$i])) {
+                            $last_value            = $listOfSegmentsIds[$i];
+                            $this->lastTransUnit[] = $this->segments[$id];
                         }
                     }
 
@@ -360,27 +360,27 @@ 
                     // we must reset the lastMrkId found because this is a new segment.
                     $lastMrkId = -1;
 
-                    if ( $this->xliffVersion === 2 ) {
-                        $seg = $this->segments[ $this->currentSegmentArray[ 'sid' ] ];
+                    if ($this->xliffVersion === 2) {
+                        $seg = $this->segments[$this->currentSegmentArray['sid']];
 
                         // update counts
-                        if ( !$this->hasWrittenCounts && !empty( $seg ) ) {
-                            $this->updateSegmentCounts( $seg );
+                        if (!$this->hasWrittenCounts && !empty($seg)) {
+                            $this->updateSegmentCounts($seg);
                         }
 
                         // delete translations so the prepareSegment
                         // will put source content in target tag
-                        if ( $this->sourceInTarget ) {
-                            $seg[ 'translation' ] = '';
+                        if ($this->sourceInTarget) {
+                            $seg['translation'] = '';
                             $this->resetCounts();
                         }
 
                         // append $translation
-                        $translation = $this->prepareTranslation( $seg, $translation );
+                        $translation = $this->prepareTranslation($seg, $translation);
 
-                        list( $stateProp, $lastMrkState ) = $this->setTransUnitState( $seg, $stateProp, $lastMrkState );
+                        list($stateProp, $lastMrkState) = $this->setTransUnitState($seg, $stateProp, $lastMrkState);
                     } else {
-                        foreach ( $listOfSegmentsIds as $pos => $id ) {
+                        foreach ($listOfSegmentsIds as $pos => $id) {
 
                             /*
                              * This routine works to respect the positional orders of markers.
@@ -393,8 +393,8 @@ 
                              * pre-assign zero to the new mrk if this is the first one ( in this segment )
                              * If it is null leave it NULL
                              */
-                            if ( (int)$this->segments[ $id ][ "mrk_id" ] < 0 && $this->segments[ $id ][ "mrk_id" ] !== null ) {
-                                $this->segments[ $id ][ "mrk_id" ] = 0;
+                            if ((int)$this->segments[$id]["mrk_id"] < 0 && $this->segments[$id]["mrk_id"] !== null) {
+                                $this->segments[$id]["mrk_id"] = 0;
                             }
 
                             /*
@@ -403,65 +403,65 @@ 
                              * ( null <= -1 ) === true
                              * so, cast to int
                              */
-                            if ( (int)$this->segments[ $id ][ "mrk_id" ] <= $lastMrkId ) {
+                            if ((int)$this->segments[$id]["mrk_id"] <= $lastMrkId) {
                                 break;
                             }
 
                             // set $this->currentSegment
-                            $seg = $this->segments[ $id ];
+                            $seg = $this->segments[$id];
 
                             // update counts
-                            if ( !empty( $seg ) ) {
-                                $this->updateSegmentCounts( $seg );
+                            if (!empty($seg)) {
+                                $this->updateSegmentCounts($seg);
                             }
 
                             // delete translations so the prepareSegment
                             // will put source content in target tag
-                            if ( $this->sourceInTarget ) {
-                                $seg[ 'translation' ] = '';
+                            if ($this->sourceInTarget) {
+                                $seg['translation'] = '';
                                 $this->resetCounts();
                             }
 
                             // append $translation
-                            $translation = $this->prepareTranslation( $seg, $translation );
+                            $translation = $this->prepareTranslation($seg, $translation);
 
                             // for xliff 2 we need $this->transUnits[ $this->currentId ] [ $pos ] for populating metadata
 
-                            unset( $this->transUnits[ $this->currentTransUnitId ] [ $pos ] );
+                            unset($this->transUnits[$this->currentTransUnitId] [$pos]);
 
-                            $lastMrkId = $this->segments[ $id ][ "mrk_id" ];
+                            $lastMrkId = $this->segments[$id]["mrk_id"];
 
-                            list( $stateProp, $lastMrkState ) = $this->setTransUnitState( $seg, $stateProp, $lastMrkState );
+                            list($stateProp, $lastMrkState) = $this->setTransUnitState($seg, $stateProp, $lastMrkState);
                         }
                     }
 
                     //append translation
                     $targetLang = '';
-                    if ( $this->xliffVersion === 1 ) {
+                    if ($this->xliffVersion === 1) {
                         $targetLang = ' xml:lang="' . $this->targetLang . '"';
                     }
 
-                    $tag = $this->buildTranslateTag( $targetLang, $stateProp, $translation, $this->counts[ 'raw_word_count' ], $this->counts[ 'eq_word_count' ] );
+                    $tag = $this->buildTranslateTag($targetLang, $stateProp, $translation, $this->counts['raw_word_count'], $this->counts['eq_word_count']);
                 }
 
                 // signal we are leaving a target
                 $this->targetWasWritten = true;
                 $this->inTarget         = false;
-                $this->postProcAndFlush( $this->outputFP, $tag, $treatAsCDATA = true );
-            } elseif ( in_array( $name, $this->nodesToCopy ) ) { // we are closing a critical CDATA section
+                $this->postProcAndFlush($this->outputFP, $tag, $treatAsCDATA = true);
+            } elseif (in_array($name, $this->nodesToCopy)) { // we are closing a critical CDATA section
 
                 $this->bufferIsActive = false;
 
                 // only for Xliff 2.*
                 // write here <mda:metaGroup> and <mda:meta> if already present in the <unit>
-                if ( 'mda:metadata' === $name && $this->unitContainsMda && $this->xliffVersion === 2 && !$this->hasWrittenCounts ) {
+                if ('mda:metadata' === $name && $this->unitContainsMda && $this->xliffVersion === 2 && !$this->hasWrittenCounts) {
 
                     // we need to update counts here
                     $this->updateCounts();
                     $this->hasWrittenCounts = true;
 
                     $tag = $this->CDATABuffer;
-                    $tag .= $this->getWordCountGroupForXliffV2( $this->counts[ 'raw_word_count' ], $this->counts[ 'eq_word_count' ], false );
+                    $tag .= $this->getWordCountGroupForXliffV2($this->counts['raw_word_count'], $this->counts['eq_word_count'], false);
                     $tag .= "    </mda:metadata>";
 
                 } else {
@@ -471,59 +471,59 @@ 
                 $this->CDATABuffer = "";
 
                 //flush to pointer
-                $this->postProcAndFlush( $this->outputFP, $tag );
-            } elseif ( 'segment' === $name ) {
+                $this->postProcAndFlush($this->outputFP, $tag);
+            } elseif ('segment' === $name) {
 
                 // only for Xliff 2.*
                 // if segment has no <target> add it BEFORE </segment>
-                if ( $this->xliffVersion === 2 && !$this->targetWasWritten ) {
+                if ($this->xliffVersion === 2 && !$this->targetWasWritten) {
 
                     $seg = $this->getCurrentSegment();
 
                     // copy attr from <source>
                     $tag = '<target';
-                    foreach ( $this->sourceAttributes as $k => $v ) {
+                    foreach ($this->sourceAttributes as $k => $v) {
                         $tag .= " $k=\"$v\"";
                     }
 
-                    $tag .= '>' . $seg[ 'translation' ] . '</target></segment>';
+                    $tag .= '>' . $seg['translation'] . '</target></segment>';
                 }
 
-                $this->postProcAndFlush( $this->outputFP, $tag );
+                $this->postProcAndFlush($this->outputFP, $tag);
 
                 // we are leaving <segment>, reset $segmentHasTarget
                 $this->targetWasWritten = false;
 
-            } elseif ( $name === 'trans-unit' ) {
+            } elseif ($name === 'trans-unit') {
 
                 // only for Xliff 1.*
                 // handling </trans-unit> closure
-                if ( !$this->targetWasWritten ) {
+                if (!$this->targetWasWritten) {
                     $seg          = $this->getCurrentSegment();
                     $lastMrkState = null;
                     $stateProp    = '';
                     $tag          = '';
 
                     // if there is translation available insert <target> BEFORE </trans-unit>
-                    if ( isset( $seg[ 'translation' ] ) ) {
-                        list( $stateProp, $lastMrkState ) = $this->setTransUnitState( $seg, $stateProp, $lastMrkState );
-                        $tag .= $this->createTargetTag( $seg[ 'translation' ], $stateProp );
+                    if (isset($seg['translation'])) {
+                        list($stateProp, $lastMrkState) = $this->setTransUnitState($seg, $stateProp, $lastMrkState);
+                        $tag .= $this->createTargetTag($seg['translation'], $stateProp);
                     }
 
                     $tag .= '</trans-unit>';
-                    $this->postProcAndFlush( $this->outputFP, $tag );
+                    $this->postProcAndFlush($this->outputFP, $tag);
                 } else {
-                    $this->postProcAndFlush( $this->outputFP, '</trans-unit>' );
+                    $this->postProcAndFlush($this->outputFP, '</trans-unit>');
                 }
-            } elseif ( $this->bufferIsActive ) { // this is a tag ( <g | <mrk ) inside a seg or seg-source tag
+            } elseif ($this->bufferIsActive) { // this is a tag ( <g | <mrk ) inside a seg or seg-source tag
                 $this->CDATABuffer .= "</$name>";
                 // Do NOT Flush
             } else { //generic tag closure do Nothing
                 // flush to pointer
-                $this->postProcAndFlush( $this->outputFP, $tag );
+                $this->postProcAndFlush($this->outputFP, $tag);
             }
-        } elseif ( $this->CDATABuffer === '<note/>' && $this->bufferIsActive === true ) {
-            $this->postProcAndFlush( $this->outputFP, '<note/>' );
+        } elseif ($this->CDATABuffer === '<note/>' && $this->bufferIsActive === true) {
+            $this->postProcAndFlush($this->outputFP, '<note/>');
             $this->bufferIsActive = false;
             $this->CDATABuffer    = '';
             $this->isEmpty        = false;
@@ -533,7 +533,7 @@ 
         }
 
         // check if we are leaving a <trans-unit> (xliff v1.*) or <unit> (xliff v2.*)
-        if ( $this->tuTagName === $name ) {
+        if ($this->tuTagName === $name) {
             $this->currentTransUnitTranslate = null;
             $this->inTU                      = false;
             $this->segmentPositionInTu       = -1;
@@ -548,21 +548,21 @@ 
      *
      * @param array $listOfSegmentsIds
      */
-    private function setCurrentSegmentArray( array $listOfSegmentsIds = [] ) {
+    private function setCurrentSegmentArray(array $listOfSegmentsIds = []) {
         // $currentSegmentId
-        if ( empty( $this->currentSegmentArray ) ) {
+        if (empty($this->currentSegmentArray)) {
             $this->currentSegmentArray = [
-                    'sid' => $listOfSegmentsIds[ 0 ],
+                    'sid' => $listOfSegmentsIds[0],
                     'tid' => $this->currentTransUnitId,
             ];
         } else {
-            if ( $this->currentSegmentArray[ 'tid' ] === $this->currentTransUnitId ) {
-                $key                                = array_search( $this->currentSegmentArray[ 'sid' ], $listOfSegmentsIds );
-                $this->currentSegmentArray[ 'sid' ] = $listOfSegmentsIds[ $key + 1 ];
-                $this->currentSegmentArray[ 'tid' ] = $this->currentTransUnitId;
+            if ($this->currentSegmentArray['tid'] === $this->currentTransUnitId) {
+                $key                                = array_search($this->currentSegmentArray['sid'], $listOfSegmentsIds);
+                $this->currentSegmentArray['sid'] = $listOfSegmentsIds[$key + 1];
+                $this->currentSegmentArray['tid'] = $this->currentTransUnitId;
             } else {
                 $this->currentSegmentArray = [
-                        'sid' => $listOfSegmentsIds[ 0 ],
+                        'sid' => $listOfSegmentsIds[0],
                         'tid' => $this->currentTransUnitId,
                 ];
             }
@@ -574,23 +574,23 @@ 
      */
     private function updateCounts() {
         // populate counts
-        $listOfSegmentsIds = $this->transUnits[ $this->currentTransUnitId ];
+        $listOfSegmentsIds = $this->transUnits[$this->currentTransUnitId];
 
         // $currentSegmentId
-        if ( !empty( $listOfSegmentsIds ) ) {
-            $this->setCurrentSegmentArray( $listOfSegmentsIds );
+        if (!empty($listOfSegmentsIds)) {
+            $this->setCurrentSegmentArray($listOfSegmentsIds);
         }
 
-        if ( $this->xliffVersion === 2 ) {
-            $seg = $this->segments[ $this->currentSegmentArray[ 'sid' ] ];
-            if ( !empty( $seg ) ) {
-                $this->updateSegmentCounts( $seg );
+        if ($this->xliffVersion === 2) {
+            $seg = $this->segments[$this->currentSegmentArray['sid']];
+            if (!empty($seg)) {
+                $this->updateSegmentCounts($seg);
             }
         } else {
-            foreach ( $listOfSegmentsIds as $pos => $id ) {
-                $seg = $this->segments[ $id ];
-                if ( !empty( $seg ) ) {
-                    $this->updateSegmentCounts( $seg );
+            foreach ($listOfSegmentsIds as $pos => $id) {
+                $seg = $this->segments[$id];
+                if (!empty($seg)) {
+                    $this->updateSegmentCounts($seg);
                 }
             }
         }
@@ -601,14 +601,14 @@ 
     /**
      * @param array $seg
      */
-    private function updateSegmentCounts( array $seg = [] ) {
-        $this->counts[ 'raw_word_count' ] += $seg[ 'raw_word_count' ];
-        $this->counts[ 'eq_word_count' ]  += ( floor( $seg[ 'eq_word_count' ] * 100 ) / 100 );
+    private function updateSegmentCounts(array $seg = []) {
+        $this->counts['raw_word_count'] += $seg['raw_word_count'];
+        $this->counts['eq_word_count']  += (floor($seg['eq_word_count'] * 100) / 100);
     }
 
     private function resetCounts() {
-        $this->counts[ 'raw_word_count' ] = 0;
-        $this->counts[ 'eq_word_count' ]  = 0;
+        $this->counts['raw_word_count'] = 0;
+        $this->counts['eq_word_count']  = 0;
     }
 
     /**
@@ -619,37 +619,37 @@ 
      *
      * @return string
      */
-    protected function prepareTranslation( $seg, $transUnitTranslation = "" ) {
+    protected function prepareTranslation($seg, $transUnitTranslation = "") {
         $endTags = "";
 
-        $segment     = Strings::removeDangerousChars( $seg [ 'segment' ] );
-        $translation = Strings::removeDangerousChars( $seg [ 'translation' ] );
-        $dataRefMap  = ( isset( $seg[ 'data_ref_map' ] ) && $seg[ 'data_ref_map' ] !== null ) ? Strings::jsonToArray( $seg[ 'data_ref_map' ] ) : [];
+        $segment     = Strings::removeDangerousChars($seg ['segment']);
+        $translation = Strings::removeDangerousChars($seg ['translation']);
+        $dataRefMap  = (isset($seg['data_ref_map']) && $seg['data_ref_map'] !== null) ? Strings::jsonToArray($seg['data_ref_map']) : [];
 
-        if ( is_null( $seg [ 'translation' ] ) || $seg [ 'translation' ] == '' ) {
+        if (is_null($seg ['translation']) || $seg ['translation'] == '') {
             $translation = $segment;
         } else {
-            if ( $this->callback ) {
-                if ( $this->callback->thereAreErrors( $seg[ 'sid' ], $segment, $translation, $dataRefMap ) ) {
+            if ($this->callback) {
+                if ($this->callback->thereAreErrors($seg['sid'], $segment, $translation, $dataRefMap)) {
                     $translation = '|||UNTRANSLATED_CONTENT_START|||' . $segment . '|||UNTRANSLATED_CONTENT_END|||';
                 }
             }
         }
 
         // for xliff v2 we ignore the marks on purpose
-        if ( $this->xliffVersion === 2 ) {
+        if ($this->xliffVersion === 2) {
             return $translation;
         }
 
-        if ( $seg[ 'mrk_id' ] !== null && $seg[ 'mrk_id' ] != '' ) {
-            if ( $this->targetLang === 'ja-JP' ) {
-                $seg[ 'mrk_succ_tags' ] = ltrim( $seg[ 'mrk_succ_tags' ] );
+        if ($seg['mrk_id'] !== null && $seg['mrk_id'] != '') {
+            if ($this->targetLang === 'ja-JP') {
+                $seg['mrk_succ_tags'] = ltrim($seg['mrk_succ_tags']);
             }
 
-            $translation = "<mrk mid=\"" . $seg[ 'mrk_id' ] . "\" mtype=\"seg\">" . $seg[ 'mrk_prev_tags' ] . $translation . $seg[ 'mrk_succ_tags' ] . "</mrk>";
+            $translation = "<mrk mid=\"" . $seg['mrk_id'] . "\" mtype=\"seg\">" . $seg['mrk_prev_tags'] . $translation . $seg['mrk_succ_tags'] . "</mrk>";
         }
 
-        $transUnitTranslation .= $seg[ 'prev_tags' ] . $translation . $endTags . $seg[ 'succ_tags' ];
+        $transUnitTranslation .= $seg['prev_tags'] . $translation . $endTags . $seg['succ_tags'];
 
         return $transUnitTranslation;
     }
@@ -663,15 +663,15 @@ 
      *
      * @return string
      */
-    private function buildTranslateTag( $targetLang, $stateProp, $translation, $rawWordCount, $eqWordCount ) {
-        switch ( $this->xliffVersion ) {
+    private function buildTranslateTag($targetLang, $stateProp, $translation, $rawWordCount, $eqWordCount) {
+        switch ($this->xliffVersion) {
             case 1:
             default:
                 $tag = "<target $targetLang $stateProp>$translation</target>";
 
                 // if it's a Trados file don't append count group
-                if ( get_class( $this ) !== SdlXliffSAXTranslationReplacer::class ) {
-                    $tag .= $this->getWordCountGroup( $rawWordCount, $eqWordCount );
+                if (get_class($this) !== SdlXliffSAXTranslationReplacer::class) {
+                    $tag .= $this->getWordCountGroup($rawWordCount, $eqWordCount);
                 }
 
                 return $tag;
@@ -687,7 +687,7 @@ 
      *
      * @return string
      */
-    private function getWordCountGroup( $raw_word_count, $eq_word_count ) {
+    private function getWordCountGroup($raw_word_count, $eq_word_count) {
         return "\n<count-group name=\"$this->currentTransUnitId\"><count count-type=\"x-matecat-raw\">$raw_word_count</count><count count-type=\"x-matecat-weighted\">$eq_word_count</count></count-group>";
     }
 
@@ -695,11 +695,11 @@ 
      * @return array
      */
     private function getCurrentSegment() {
-        if ( $this->currentTransUnitTranslate === 'yes' && isset( $this->transUnits[ $this->currentTransUnitId ] ) ) {
-            $index = $this->transUnits[ $this->currentTransUnitId ][ $this->segmentPositionInTu ];
+        if ($this->currentTransUnitTranslate === 'yes' && isset($this->transUnits[$this->currentTransUnitId])) {
+            $index = $this->transUnits[$this->currentTransUnitId][$this->segmentPositionInTu];
 
-            if ( isset( $this->segments[ $index ] ) ) {
-                return $this->segments[ $index ];
+            if (isset($this->segments[$index])) {
+                return $this->segments[$index];
             }
         }
 
@@ -714,10 +714,10 @@ 
      *
      * @return string
      */
-    private function createTargetTag( $translation, $stateProp ) {
+    private function createTargetTag($translation, $stateProp) {
         $targetLang = 'xml:lang="' . $this->targetLang . '"';
 
-        return $this->buildTranslateTag( $targetLang, $stateProp, $translation, $this->counts[ 'raw_word_count' ], $this->counts[ 'eq_word_count' ] );
+        return $this->buildTranslateTag($targetLang, $stateProp, $translation, $this->counts['raw_word_count'], $this->counts['eq_word_count']);
     }
 
     /**
@@ -727,11 +727,11 @@ 
      *
      * @return string
      */
-    private function getWordCountGroupForXliffV2( $raw_word_count, $eq_word_count, $withMetadataTag = true ) {
+    private function getWordCountGroupForXliffV2($raw_word_count, $eq_word_count, $withMetadataTag = true) {
         $this->mdaGroupCounter++;
         $id = 'word_count_tu_' . $this->mdaGroupCounter;
 
-        if ( $withMetadataTag === false ) {
+        if ($withMetadataTag === false) {
             return "    <mda:metaGroup id=\"" . $id . "\" category=\"row_xml_attribute\">
                                 <mda:meta type=\"x-matecat-raw\">$raw_word_count</mda:meta>
                                 <mda:meta type=\"x-matecat-weighted\">$eq_word_count</mda:meta>
@@ -754,19 +754,19 @@ 
      *
      * @return array
      */
-    private function setTransUnitState( $seg, $state_prop, $lastMrkState ) {
-        switch ( $seg[ 'status' ] ) {
+    private function setTransUnitState($seg, $state_prop, $lastMrkState) {
+        switch ($seg['status']) {
 
             case TranslationStatus::STATUS_FIXED:
             case TranslationStatus::STATUS_APPROVED:
-                if ( $lastMrkState == null || $lastMrkState == TranslationStatus::STATUS_APPROVED ) {
-                    $state_prop   = ( $this->xliffVersion === 2 ) ? "state=\"reviewed\"" : "state=\"signed-off\"";
+                if ($lastMrkState == null || $lastMrkState == TranslationStatus::STATUS_APPROVED) {
+                    $state_prop   = ($this->xliffVersion === 2) ? "state=\"reviewed\"" : "state=\"signed-off\"";
                     $lastMrkState = TranslationStatus::STATUS_APPROVED;
                 }
                 break;
 
             case TranslationStatus::STATUS_TRANSLATED:
-                if ( $lastMrkState == null || $lastMrkState == TranslationStatus::STATUS_TRANSLATED || $lastMrkState == TranslationStatus::STATUS_APPROVED ) {
+                if ($lastMrkState == null || $lastMrkState == TranslationStatus::STATUS_TRANSLATED || $lastMrkState == TranslationStatus::STATUS_APPROVED) {
                     $state_prop   = "state=\"translated\"";
                     $lastMrkState = TranslationStatus::STATUS_TRANSLATED;
                 }
@@ -774,15 +774,15 @@ 
 
             case TranslationStatus::STATUS_REJECTED:  // if there is a mark REJECTED and there is not a DRAFT, all the trans-unit is REJECTED. In V2 there is no way to mark
             case TranslationStatus::STATUS_REBUTTED:
-                if ( ( $lastMrkState == null ) || ( $lastMrkState != TranslationStatus::STATUS_NEW || $lastMrkState != TranslationStatus::STATUS_DRAFT ) ) {
-                    $state_prop   = ( $this->xliffVersion === 2 ) ? "state=\"initial\"" : "state=\"needs-review-translation\"";
+                if (($lastMrkState == null) || ($lastMrkState != TranslationStatus::STATUS_NEW || $lastMrkState != TranslationStatus::STATUS_DRAFT)) {
+                    $state_prop   = ($this->xliffVersion === 2) ? "state=\"initial\"" : "state=\"needs-review-translation\"";
                     $lastMrkState = TranslationStatus::STATUS_REJECTED;
                 }
                 break;
 
             case TranslationStatus::STATUS_NEW:
-                if ( ( $lastMrkState == null ) || $lastMrkState != TranslationStatus::STATUS_DRAFT ) {
-                    $state_prop   = ( $this->xliffVersion === 2 ) ? "state=\"initial\"" : "state=\"new\"";
+                if (($lastMrkState == null) || $lastMrkState != TranslationStatus::STATUS_DRAFT) {
+                    $state_prop   = ($this->xliffVersion === 2) ? "state=\"initial\"" : "state=\"new\"";
                     $lastMrkState = TranslationStatus::STATUS_NEW;
                 }
                 break;
@@ -795,7 +795,7 @@ 
                 // this is the case when a segment is not showed in cattool, so the row in
                 // segment_translations does not exists and
                 // ---> $seg[ 'status' ] is NULL
-                if ( $lastMrkState == null ) { //this is the first MRK ID
+                if ($lastMrkState == null) { //this is the first MRK ID
                     $state_prop   = "state=\"translated\"";
                     $lastMrkState = TranslationStatus::STATUS_TRANSLATED;
                 } else {
@@ -804,17 +804,17 @@ 
                 break;
         }
 
-        return [ $state_prop, $lastMrkState ];
+        return [$state_prop, $lastMrkState];
     }
 
     /**
      * @inheritDoc
      */
-    protected function characterData( $parser, $data ) {
+    protected function characterData($parser, $data) {
         // don't write <target> data
-        if ( !$this->inTarget && !$this->bufferIsActive ) {
-            $this->postProcAndFlush( $this->outputFP, $data );
-        } elseif ( $this->bufferIsActive ) {
+        if (!$this->inTarget && !$this->bufferIsActive) {
+            $this->postProcAndFlush($this->outputFP, $data);
+        } elseif ($this->bufferIsActive) {
             $this->CDATABuffer .= $data;
         }
     }

src/XliffParser.php

Spacing +25 added lines, -25 removed lines

@@ -64,22 +64,22 @@ 
      * @throws Exception\NotValidFileException
      * @throws Exception\XmlParsingException
      */
-    public function xliffToArray( $xliffContent, $collapseEmptyTags = false)
+    public function xliffToArray($xliffContent, $collapseEmptyTags = false)
     {
         $xliff = [];
         $xliffContent = self::forceUft8Encoding($xliffContent, $xliff);
         $xliffVersion = XliffVersionDetector::detect($xliffContent);
         $info = XliffProprietaryDetect::getInfoFromXliffContent($xliffContent);
 
-        if($xliffVersion === 1){
+        if ($xliffVersion === 1) {
             $xliffContent = self::removeInternalFileTagFromContent($xliffContent, $xliff);
         }
 
-        if($xliffVersion === 2){
+        if ($xliffVersion === 2) {
             $xliffContent = self::escapeDataInOriginalMap($xliffContent);
         }
 
-        if($collapseEmptyTags === false){
+        if ($collapseEmptyTags === false) {
             $xliffContent = self::insertPlaceholderInEmptyTags($xliffContent);
         }
 
@@ -104,8 +104,8 @@ 
         $enc = mb_detect_encoding($xliffContent);
 
         if ($enc !== 'UTF-8') {
-            $xliff[ 'parser-warnings' ][] = "Input identified as $enc ans converted UTF-8. May not be a problem if the content is English only";
-            $s = iconv( $enc, 'UTF-8', $xliffContent );
+            $xliff['parser-warnings'][] = "Input identified as $enc ans converted UTF-8. May not be a problem if the content is English only";
+            $s = iconv($enc, 'UTF-8', $xliffContent);
             $xliffContent = $s !== false ? $s : "";
         }
 
@@ -125,22 +125,22 @@ 
     private static function removeInternalFileTagFromContent($xliffContent, &$xliff)
     {
         $index = 1;
-        $a = Strings::preg_split( '|<internal-file[\s>]|si', $xliffContent );
+        $a = Strings::preg_split('|<internal-file[\s>]|si', $xliffContent);
 
         // no match, return original string
-        if(count($a) === 1){
+        if (count($a) === 1) {
             return $a[0];
         }
 
-        $b = Strings::preg_split( '|</internal-file>|si', $a[1] );
-        $strippedContent = $a[0].$b[1];
-        $xliff['files'][$index][ 'reference' ][] = self::extractBase64($b[0]);
+        $b = Strings::preg_split('|</internal-file>|si', $a[1]);
+        $strippedContent = $a[0] . $b[1];
+        $xliff['files'][$index]['reference'][] = self::extractBase64($b[0]);
         $index++;
 
-        if(isset($a[2])){
-            $c = Strings::preg_split( '|</internal-file[\s>]|si', $a[2] );
+        if (isset($a[2])) {
+            $c = Strings::preg_split('|</internal-file[\s>]|si', $a[2]);
             $strippedContent .= $c[1];
-            $xliff['files'][$index][ 'reference' ][] = self::extractBase64($c[0]);
+            $xliff['files'][$index]['reference'][] = self::extractBase64($c[0]);
         }
 
         return $strippedContent;
@@ -177,8 +177,8 @@ 
      */
     private static function escapeDataInOriginalMap($xliffContent)
     {
-        $xliffContent  = preg_replace_callback('/<data(.*?)>(.*?)<\/data>/iU', [XliffParser::class, 'replaceSpace' ], $xliffContent);
-        $xliffContent  = preg_replace_callback('/<data(.*?)>(.*?)<\/data>/iU', [XliffParser::class, 'replaceXliffTags' ], $xliffContent);
+        $xliffContent  = preg_replace_callback('/<data(.*?)>(.*?)<\/data>/iU', [XliffParser::class, 'replaceSpace'], $xliffContent);
+        $xliffContent  = preg_replace_callback('/<data(.*?)>(.*?)<\/data>/iU', [XliffParser::class, 'replaceXliffTags'], $xliffContent);
 
         return $xliffContent;
     }
@@ -201,11 +201,11 @@ 
     {
         preg_match_all('/<([a-zA-Z0-9._-]+)[^>]*><\/\1>/sm', $xliffContent, $emptyTagMatches);
 
-        if(!empty($emptyTagMatches[0])){
-            foreach ($emptyTagMatches[0] as $index => $emptyTagMatch){
+        if (!empty($emptyTagMatches[0])) {
+            foreach ($emptyTagMatches[0] as $index => $emptyTagMatch) {
                 $matchedTag = $emptyTagMatches[1][$index];
-                $subst = Placeholder::EMPTY_TAG_PLACEHOLDER.'</'.$matchedTag.'>';
-                $replacedTag = str_replace('</'.$matchedTag.'>', $subst, $emptyTagMatch);
+                $subst = Placeholder::EMPTY_TAG_PLACEHOLDER . '</' . $matchedTag . '>';
+                $replacedTag = str_replace('</' . $matchedTag . '>', $subst, $emptyTagMatch);
                 $xliffContent = str_replace($emptyTagMatch, $replacedTag, $xliffContent);
             }
         }
@@ -226,7 +226,7 @@ 
         $content = str_replace('\n', Placeholder::NEW_LINE_PLACEHOLDER, $content);
         $content = str_replace('\t', Placeholder::TAB_PLACEHOLDER, $content);
 
-        return '<data'.$matches[1].'>'.$content.'</data>';
+        return '<data' . $matches[1] . '>' . $content . '</data>';
     }
 
     /**
@@ -239,11 +239,11 @@ 
         $xliffTags = XliffTags::$tags;
         $content = $matches[2];
 
-        foreach ($xliffTags as $xliffTag){
-            $content = preg_replace( '|&lt;('.$xliffTag.'.*?)&gt;|si', Placeholder::LT_PLACEHOLDER . "$1" . Placeholder::GT_PLACEHOLDER, $content );
-            $content = preg_replace( '|&lt;(/'.$xliffTag.')&gt;|si', Placeholder::LT_PLACEHOLDER . "$1" . Placeholder::GT_PLACEHOLDER, $content );
+        foreach ($xliffTags as $xliffTag) {
+            $content = preg_replace('|&lt;(' . $xliffTag . '.*?)&gt;|si', Placeholder::LT_PLACEHOLDER . "$1" . Placeholder::GT_PLACEHOLDER, $content);
+            $content = preg_replace('|&lt;(/' . $xliffTag . ')&gt;|si', Placeholder::LT_PLACEHOLDER . "$1" . Placeholder::GT_PLACEHOLDER, $content);
         }
 
-        return '<data'.$matches[1].'>'.$content.'</data>';
+        return '<data' . $matches[1] . '>' . $content . '</data>';
     }
 }

src/XliffParser/AbstractXliffParser.php

Spacing +59 added lines, -59 removed lines

@@ -34,7 +34,7 @@ 
      * @param string|null          $xliffProprietary
      * @param LoggerInterface|null $logger
      */
-    public function __construct( $xliffVersion, $xliffProprietary = null, LoggerInterface $logger = null ) {
+    public function __construct($xliffVersion, $xliffProprietary = null, LoggerInterface $logger = null) {
         $this->xliffVersion     = $xliffVersion;
         $this->logger           = $logger;
         $this->xliffProprietary = $xliffProprietary;
@@ -44,7 +44,7 @@ 
      * @return string
      */
     protected function getTuTagName() {
-        return ( $this->xliffVersion === 1 ) ? 'trans-unit' : 'unit';
+        return ($this->xliffVersion === 1) ? 'trans-unit' : 'unit';
     }
 
     /**
@@ -52,7 +52,7 @@ 
      *
      * @return array
      */
-    abstract public function parse( DOMDocument $dom, $output = [] );
+    abstract public function parse(DOMDocument $dom, $output = []);
 
     /**
      * Extract trans-unit content from the current node
@@ -64,17 +64,17 @@ 
      * @param              $i
      * @param              $j
      */
-    protected function extractTuFromNode( $childNode, &$transUnitIdArrayForUniquenessCheck, DOMDocument $dom, &$output, &$i, &$j ) {
-        if ( $childNode->nodeName === 'group' ) {
-            foreach ( $childNode->childNodes as $nestedChildNode ) {
-                if ( $nestedChildNode->nodeName === 'group' ) {
-                    $this->extractTuFromNode( $nestedChildNode, $transUnitIdArrayForUniquenessCheck, $dom, $output, $i, $j );
-                } elseif ( $nestedChildNode->nodeName === $this->getTuTagName() ) {
-                    $this->extractTransUnit( $nestedChildNode, $transUnitIdArrayForUniquenessCheck, $dom, $output, $i, $j );
+    protected function extractTuFromNode($childNode, &$transUnitIdArrayForUniquenessCheck, DOMDocument $dom, &$output, &$i, &$j) {
+        if ($childNode->nodeName === 'group') {
+            foreach ($childNode->childNodes as $nestedChildNode) {
+                if ($nestedChildNode->nodeName === 'group') {
+                    $this->extractTuFromNode($nestedChildNode, $transUnitIdArrayForUniquenessCheck, $dom, $output, $i, $j);
+                } elseif ($nestedChildNode->nodeName === $this->getTuTagName()) {
+                    $this->extractTransUnit($nestedChildNode, $transUnitIdArrayForUniquenessCheck, $dom, $output, $i, $j);
                 }
             }
-        } elseif ( $childNode->nodeName === $this->getTuTagName() ) {
-            $this->extractTransUnit( $childNode, $transUnitIdArrayForUniquenessCheck, $dom, $output, $i, $j );
+        } elseif ($childNode->nodeName === $this->getTuTagName()) {
+            $this->extractTransUnit($childNode, $transUnitIdArrayForUniquenessCheck, $dom, $output, $i, $j);
         }
     }
 
@@ -90,7 +90,7 @@ 
      *
      * @return mixed
      */
-    abstract protected function extractTransUnit( $transUnit, &$transUnitIdArrayForUniquenessCheck, $dom, &$output, &$i, &$j );
+    abstract protected function extractTransUnit($transUnit, &$transUnitIdArrayForUniquenessCheck, $dom, &$output, &$i, &$j);
 
     /**
      * @param DOMDocument $dom
@@ -98,10 +98,10 @@ 
      *
      * @return array
      */
-    protected function extractContent( DOMDocument $dom, DOMNode $node ) {
+    protected function extractContent(DOMDocument $dom, DOMNode $node) {
         return [
-                'raw-content' => $this->extractTagContent( $dom, $node ),
-                'attr'        => $this->extractTagAttributes( $node )
+                'raw-content' => $this->extractTagContent($dom, $node),
+                'attr'        => $this->extractTagAttributes($node)
         ];
     }
 
@@ -118,12 +118,12 @@ 
      *
      * @return array
      */
-    protected function extractTagAttributes( DOMNode $element ) {
+    protected function extractTagAttributes(DOMNode $element) {
         $tagAttributes = [];
 
-        if ( $element->hasAttributes() ) {
-            foreach ( $element->attributes as $attr ) {
-                $tagAttributes[ $attr->nodeName ] = $attr->nodeValue;
+        if ($element->hasAttributes()) {
+            foreach ($element->attributes as $attr) {
+                $tagAttributes[$attr->nodeName] = $attr->nodeValue;
             }
         }
 
@@ -138,17 +138,17 @@ 
      *
      * @return string
      */
-    protected function extractTagContent( DOMDocument $dom, DOMNode $element ) {
+    protected function extractTagContent(DOMDocument $dom, DOMNode $element) {
         $childNodes       = $element->hasChildNodes();
         $extractedContent = '';
 
-        if ( !empty( $childNodes ) ) {
-            foreach ( $element->childNodes as $node ) {
-                $extractedContent .= Emoji::toEntity( Strings::fixNonWellFormedXml( $dom->saveXML( $node ) ) );
+        if (!empty($childNodes)) {
+            foreach ($element->childNodes as $node) {
+                $extractedContent .= Emoji::toEntity(Strings::fixNonWellFormedXml($dom->saveXML($node)));
             }
         }
 
-        return str_replace( Placeholder::EMPTY_TAG_PLACEHOLDER, '', $extractedContent );
+        return str_replace(Placeholder::EMPTY_TAG_PLACEHOLDER, '', $extractedContent);
     }
 
     /**
@@ -161,50 +161,50 @@ 
      *
      * @return array
      */
-    protected function extractContentWithMarksAndExtTags( DOMDocument $dom, DOMElement $childNode, $originalRawContent, array $originalData = [] ) {
+    protected function extractContentWithMarksAndExtTags(DOMDocument $dom, DOMElement $childNode, $originalRawContent, array $originalData = []) {
         $source = [];
 
         // example:
         // <g id="1"><mrk mid="0" mtype="seg">An English string with g tags</mrk></g>
-        $raw = $this->extractTagContent( $dom, $childNode );
+        $raw = $this->extractTagContent($dom, $childNode);
 
-        $markers = preg_split( '#<mrk\s#si', $raw, -1 );
+        $markers = preg_split('#<mrk\s#si', $raw, -1);
 
         $mi = 0;
-        while ( isset( $markers[ $mi + 1 ] ) ) {
-            unset( $mid );
+        while (isset($markers[$mi + 1])) {
+            unset($mid);
 
-            preg_match( '|mid\s?=\s?["\'](.*?)["\']|si', $markers[ $mi + 1 ], $mid );
+            preg_match('|mid\s?=\s?["\'](.*?)["\']|si', $markers[$mi + 1], $mid);
 
             // if it's a Trados file the trailing spaces after </mrk> are meaningful
             // so we add them to
             $trailingSpaces = '';
-            if ( $this->xliffProprietary === 'trados' ) {
-                preg_match_all( '/<\/mrk>[\s]+/iu', $markers[ $mi + 1 ], $trailingSpacesMatches );
+            if ($this->xliffProprietary === 'trados') {
+                preg_match_all('/<\/mrk>[\s]+/iu', $markers[$mi + 1], $trailingSpacesMatches);
 
-                if ( isset( $trailingSpacesMatches[ 0 ] ) && count( $trailingSpacesMatches[ 0 ] ) > 0 ) {
-                    foreach ( $trailingSpacesMatches[ 0 ] as $match ) {
-                        $trailingSpaces = str_replace( '</mrk>', '', $match );
+                if (isset($trailingSpacesMatches[0]) && count($trailingSpacesMatches[0]) > 0) {
+                    foreach ($trailingSpacesMatches[0] as $match) {
+                        $trailingSpaces = str_replace('</mrk>', '', $match);
                     }
                 }
             }
 
             //re-build the mrk tag after the split
-            $originalMark = trim( '<mrk ' . $markers[ $mi + 1 ] );
+            $originalMark = trim('<mrk ' . $markers[$mi + 1]);
 
-            $mark_string  = preg_replace( '#^<mrk\s[^>]+>(.*)#', '$1', $originalMark ); // at this point we have: ---> 'Test </mrk> </g>>'
-            $mark_content = preg_split( '#</mrk>#si', $mark_string );
+            $mark_string  = preg_replace('#^<mrk\s[^>]+>(.*)#', '$1', $originalMark); // at this point we have: ---> 'Test </mrk> </g>>'
+            $mark_content = preg_split('#</mrk>#si', $mark_string);
 
             $sourceArray = [
-                    'mid'           => ( isset( $mid[ 1 ] ) ) ? $mid[ 1 ] : $mi,
-                    'ext-prec-tags' => ( $mi == 0 ? $markers[ 0 ] : "" ),
-                    'raw-content'   => ( isset( $mark_content[ 0 ] ) ) ? $mark_content[ 0 ] . $trailingSpaces : '',
-                    'ext-succ-tags' => ( isset( $mark_content[ 1 ] ) ) ? $mark_content[ 1 ] : '',
+                    'mid'           => (isset($mid[1])) ? $mid[1] : $mi,
+                    'ext-prec-tags' => ($mi == 0 ? $markers[0] : ""),
+                    'raw-content'   => (isset($mark_content[0])) ? $mark_content[0] . $trailingSpaces : '',
+                    'ext-succ-tags' => (isset($mark_content[1])) ? $mark_content[1] : '',
             ];
 
-            if ( !empty( $originalData ) ) {
-                $dataRefMap                        = $this->getDataRefMap( $originalData );
-                $sourceArray[ 'replaced-content' ] = ( new DataRefReplacer( $dataRefMap ) )->replace( $mark_content[ 0 ] );
+            if (!empty($originalData)) {
+                $dataRefMap                        = $this->getDataRefMap($originalData);
+                $sourceArray['replaced-content'] = (new DataRefReplacer($dataRefMap))->replace($mark_content[0]);
             }
 
             $source[] = $sourceArray;
@@ -220,12 +220,12 @@ 
      *
      * @return array
      */
-    protected function getDataRefMap( $originalData ) {
+    protected function getDataRefMap($originalData) {
         // dataRef map
         $dataRefMap = [];
-        foreach ( $originalData as $datum ) {
-            if ( isset( $datum[ 'attr' ][ 'id' ] ) ) {
-                $dataRefMap[ $datum[ 'attr' ][ 'id' ] ] = $datum[ 'raw-content' ];
+        foreach ($originalData as $datum) {
+            if (isset($datum['attr']['id'])) {
+                $dataRefMap[$datum['attr']['id']] = $datum['raw-content'];
             }
         }
 
@@ -237,10 +237,10 @@ 
      *
      * @return bool
      */
-    protected function stringContainsMarks( $raw ) {
-        $markers = preg_split( '#<mrk\s#si', $raw, -1 );
+    protected function stringContainsMarks($raw) {
+        $markers = preg_split('#<mrk\s#si', $raw, -1);
 
-        return isset( $markers[ 1 ] );
+        return isset($markers[1]);
     }
 
     /**
@@ -250,7 +250,7 @@ 
      * @return array
      * @throws \Exception
      */
-    protected function JSONOrRawContentArray( $noteValue, $escapeStrings = true ) {
+    protected function JSONOrRawContentArray($noteValue, $escapeStrings = true) {
         //
         // convert double escaped entites
         //
@@ -260,17 +260,17 @@ 
         // &amp;amp; ---> &amp;
         // &amp;apos ---> &apos;
         //
-        if ( Strings::isADoubleEscapedEntity( $noteValue ) ) {
-            $noteValue = Strings::htmlspecialchars_decode( $noteValue, true );
+        if (Strings::isADoubleEscapedEntity($noteValue)) {
+            $noteValue = Strings::htmlspecialchars_decode($noteValue, true);
         } else {
             // for non escaped entities $escapeStrings is always true for security reasons
             $escapeStrings = true;
         }
 
-        if ( Strings::isJSON( $noteValue ) ) {
-            return [ 'json' => Strings::cleanCDATA( $noteValue ) ];
+        if (Strings::isJSON($noteValue)) {
+            return ['json' => Strings::cleanCDATA($noteValue)];
         }
 
-        return [ 'raw-content' => Strings::fixNonWellFormedXml( $noteValue, $escapeStrings ) ];
+        return ['raw-content' => Strings::fixNonWellFormedXml($noteValue, $escapeStrings)];
     }
 }

src/XliffParser/XliffParserV1.php

Spacing +88 added lines, -88 removed lines

@@ -15,33 +15,33 @@ 
      * @inheritDoc
      * @throws Exception
      */
-    public function parse( DOMDocument $dom, $output = [] ) {
+    public function parse(DOMDocument $dom, $output = []) {
         $i = 1;
         /** @var DOMElement $file */
-        foreach ( $dom->getElementsByTagName( 'file' ) as $file ) {
+        foreach ($dom->getElementsByTagName('file') as $file) {
 
             // metadata
-            $output[ 'files' ][ $i ][ 'attr' ] = $this->extractMetadata( $file );
+            $output['files'][$i]['attr'] = $this->extractMetadata($file);
 
             // reference
-            if ( !empty( $this->extractReference( $file ) ) ) {
-                $output[ 'files' ][ $i ][ 'reference' ] = $this->extractReference( $file );
+            if (!empty($this->extractReference($file))) {
+                $output['files'][$i]['reference'] = $this->extractReference($file);
             }
 
             // trans-units
             $transUnitIdArrayForUniquenessCheck = [];
             $j                                  = 1;
-            foreach ( $file->childNodes as $body ) {
-                if ( $body->nodeName === 'body' ) {
-                    foreach ( $body->childNodes as $childNode ) {
-                        $this->extractTuFromNode( $childNode, $transUnitIdArrayForUniquenessCheck, $dom, $output, $i, $j );
+            foreach ($file->childNodes as $body) {
+                if ($body->nodeName === 'body') {
+                    foreach ($body->childNodes as $childNode) {
+                        $this->extractTuFromNode($childNode, $transUnitIdArrayForUniquenessCheck, $dom, $output, $i, $j);
                     }
 
                     // trans-unit re-count check
-                    $totalTransUnitsId  = count( $transUnitIdArrayForUniquenessCheck );
-                    $transUnitsUniqueId = count( array_unique( $transUnitIdArrayForUniquenessCheck ) );
-                    if ( $totalTransUnitsId != $transUnitsUniqueId ) {
-                        throw new DuplicateTransUnitIdInXliff( "Invalid trans-unit id, duplicate found.", 400 );
+                    $totalTransUnitsId  = count($transUnitIdArrayForUniquenessCheck);
+                    $transUnitsUniqueId = count(array_unique($transUnitIdArrayForUniquenessCheck));
+                    if ($totalTransUnitsId != $transUnitsUniqueId) {
+                        throw new DuplicateTransUnitIdInXliff("Invalid trans-unit id, duplicate found.", 400);
                     }
 
                     $i++;
@@ -57,50 +57,50 @@ 
      *
      * @return array
      */
-    private function extractMetadata( DOMElement $file ) {
+    private function extractMetadata(DOMElement $file) {
         $metadata   = [];
         $customAttr = [];
 
         /** @var \DOMAttr $attribute */
-        foreach ( $file->attributes as $attribute ) {
-            switch ( $attribute->localName ) {
+        foreach ($file->attributes as $attribute) {
+            switch ($attribute->localName) {
                 // original
                 case 'original':
-                    $metadata[ 'original' ] = $attribute->value;
+                    $metadata['original'] = $attribute->value;
                     break;
 
                 // source-language
                 case 'source-language':
-                    $metadata[ 'source-language' ] = $attribute->value;
+                    $metadata['source-language'] = $attribute->value;
                     break;
 
                 // data-type
                 case 'datatype':
-                    $metadata[ 'data-type' ] = $attribute->value;
+                    $metadata['data-type'] = $attribute->value;
                     break;
 
                 // target-language
                 case 'target-language':
-                    $metadata[ 'target-language' ] = $attribute->value;
+                    $metadata['target-language'] = $attribute->value;
                     break;
             }
 
             // Custom MateCat x-Attribute
-            preg_match( '|x-(.*?)|si', $attribute->localName, $temp );
-            if ( isset( $temp[ 1 ] ) ) {
-                $customAttr[ $attribute->localName ] = $attribute->value;
+            preg_match('|x-(.*?)|si', $attribute->localName, $temp);
+            if (isset($temp[1])) {
+                $customAttr[$attribute->localName] = $attribute->value;
             }
-            unset( $temp );
+            unset($temp);
 
             // Custom MateCat namespace Attribute mtc:
-            preg_match( '|mtc:(.*?)|si', $attribute->nodeName, $temp );
-            if ( isset( $temp[ 1 ] ) ) {
-                $customAttr[ $attribute->nodeName ] = $attribute->value;
+            preg_match('|mtc:(.*?)|si', $attribute->nodeName, $temp);
+            if (isset($temp[1])) {
+                $customAttr[$attribute->nodeName] = $attribute->value;
             }
-            unset( $temp );
+            unset($temp);
 
-            if ( !empty( $customAttr ) ) {
-                $metadata[ 'custom' ] = $customAttr;
+            if (!empty($customAttr)) {
+                $metadata['custom'] = $customAttr;
             }
         }
 
@@ -112,16 +112,16 @@ 
      *
      * @return array
      */
-    private function extractReference( DOMElement $file ) {
+    private function extractReference(DOMElement $file) {
         $reference = [];
 
         $order = 0;
-        foreach ( $file->getElementsByTagName( 'reference' ) as $ref ) {
+        foreach ($file->getElementsByTagName('reference') as $ref) {
             /** @var DOMNode $childNode */
-            foreach ( $ref->childNodes as $childNode ) {
-                if ( $childNode->nodeName === 'internal-file' ) {
-                    $reference[ $order ][ 'form-type' ] = $childNode->attributes->getNamedItem( 'form' )->nodeValue;
-                    $reference[ $order ][ 'base64' ]    = trim( $childNode->nodeValue );
+            foreach ($ref->childNodes as $childNode) {
+                if ($childNode->nodeName === 'internal-file') {
+                    $reference[$order]['form-type'] = $childNode->attributes->getNamedItem('form')->nodeValue;
+                    $reference[$order]['base64']    = trim($childNode->nodeValue);
                     $order++;
                 }
             }
@@ -142,53 +142,53 @@ 
      *
      * @throws Exception
      */
-    protected function extractTransUnit( $transUnit, &$transUnitIdArrayForUniquenessCheck, $dom, &$output, &$i, &$j ) {
+    protected function extractTransUnit($transUnit, &$transUnitIdArrayForUniquenessCheck, $dom, &$output, &$i, &$j) {
         // metadata
-        $output[ 'files' ][ $i ][ 'trans-units' ][ $j ][ 'attr' ] = $this->extractTransUnitMetadata( $transUnit, $transUnitIdArrayForUniquenessCheck );
+        $output['files'][$i]['trans-units'][$j]['attr'] = $this->extractTransUnitMetadata($transUnit, $transUnitIdArrayForUniquenessCheck);
 
         // notes
-        $output[ 'files' ][ $i ][ 'trans-units' ][ $j ][ 'notes' ] = $this->extractTransUnitNotes( $dom, $transUnit );
+        $output['files'][$i]['trans-units'][$j]['notes'] = $this->extractTransUnitNotes($dom, $transUnit);
 
         // content
         /** @var DOMElement $childNode */
-        foreach ( $transUnit->childNodes as $childNode ) {
+        foreach ($transUnit->childNodes as $childNode) {
             // source
-            if ( $childNode->nodeName === 'source' ) {
-                $output[ 'files' ][ $i ][ 'trans-units' ][ $j ][ 'source' ] = $this->extractContent( $dom, $childNode );
+            if ($childNode->nodeName === 'source') {
+                $output['files'][$i]['trans-units'][$j]['source'] = $this->extractContent($dom, $childNode);
             }
 
             // seg-source
-            if ( $childNode->nodeName === 'seg-source' ) {
-                $rawSegment                                                     = $output[ 'files' ][ $i ][ 'trans-units' ][ $j ][ 'source' ][ 'raw-content' ];
-                $output[ 'files' ][ $i ][ 'trans-units' ][ $j ][ 'seg-source' ] = $this->extractContentWithMarksAndExtTags( $dom, $childNode, $rawSegment );
+            if ($childNode->nodeName === 'seg-source') {
+                $rawSegment                                                     = $output['files'][$i]['trans-units'][$j]['source']['raw-content'];
+                $output['files'][$i]['trans-units'][$j]['seg-source'] = $this->extractContentWithMarksAndExtTags($dom, $childNode, $rawSegment);
             }
 
             // target
-            if ( $childNode->nodeName === 'target' ) {
-                $output[ 'files' ][ $i ][ 'trans-units' ][ $j ][ 'target' ] = $this->extractContent( $dom, $childNode );
+            if ($childNode->nodeName === 'target') {
+                $output['files'][$i]['trans-units'][$j]['target'] = $this->extractContent($dom, $childNode);
 
                 // seg-target
-                $targetRawContent = @$output[ 'files' ][ $i ][ 'trans-units' ][ $j ][ 'target' ][ 'raw-content' ];
-                $segSource        = @$output[ 'files' ][ $i ][ 'trans-units' ][ $j ][ 'seg-source' ];
-                if ( isset( $targetRawContent ) && !empty( $targetRawContent ) && isset( $segSource ) && count( $segSource ) > 0 ) {
-                    $output[ 'files' ][ $i ][ 'trans-units' ][ $j ][ 'seg-target' ] = $this->extractContentWithMarksAndExtTags( $dom, $childNode, $targetRawContent );
+                $targetRawContent = @$output['files'][$i]['trans-units'][$j]['target']['raw-content'];
+                $segSource        = @$output['files'][$i]['trans-units'][$j]['seg-source'];
+                if (isset($targetRawContent) && !empty($targetRawContent) && isset($segSource) && count($segSource) > 0) {
+                    $output['files'][$i]['trans-units'][$j]['seg-target'] = $this->extractContentWithMarksAndExtTags($dom, $childNode, $targetRawContent);
                 }
             }
 
             // locked
-            if ( $childNode->nodeName === 'sdl:seg' ) {
-                $output[ 'files' ][ $i ][ 'trans-units' ][ $j ][ 'locked' ] = $this->extractLocked( $childNode );
+            if ($childNode->nodeName === 'sdl:seg') {
+                $output['files'][$i]['trans-units'][$j]['locked'] = $this->extractLocked($childNode);
             }
         }
 
         // context-group
-        foreach ( $transUnit->getElementsByTagName( 'context-group' ) as $contextGroup ) {
-            $output[ 'files' ][ $i ][ 'trans-units' ][ $j ][ 'context-group' ][] = $this->extractTransUnitContextGroup( $dom, $contextGroup );
+        foreach ($transUnit->getElementsByTagName('context-group') as $contextGroup) {
+            $output['files'][$i]['trans-units'][$j]['context-group'][] = $this->extractTransUnitContextGroup($dom, $contextGroup);
         }
 
         // alt-trans
-        foreach ( $transUnit->getElementsByTagName( 'alt-trans' ) as $altTrans ) {
-            $output[ 'files' ][ $i ][ 'trans-units' ][ $j ][ 'alt-trans' ][] = $this->extractTransUnitAltTrans( $altTrans );
+        foreach ($transUnit->getElementsByTagName('alt-trans') as $altTrans) {
+            $output['files'][$i]['trans-units'][$j]['alt-trans'][] = $this->extractTransUnitAltTrans($altTrans);
         }
 
         $j++;
@@ -201,32 +201,32 @@ 
      * @return array
      * @throws Exception
      */
-    private function extractTransUnitMetadata( DOMElement $transUnit, array &$transUnitIdArrayForUniquenessCheck ) {
+    private function extractTransUnitMetadata(DOMElement $transUnit, array &$transUnitIdArrayForUniquenessCheck) {
         $metadata = [];
 
         // id
-        if ( null === $transUnit->attributes->getNamedItem( 'id' ) ) {
-            throw new NotFoundIdInTransUnit( 'Invalid trans-unit id found. EMPTY value', 400 );
+        if (null === $transUnit->attributes->getNamedItem('id')) {
+            throw new NotFoundIdInTransUnit('Invalid trans-unit id found. EMPTY value', 400);
         }
 
-        $id = $transUnit->attributes->getNamedItem( 'id' )->nodeValue;
+        $id = $transUnit->attributes->getNamedItem('id')->nodeValue;
 
-        if ( strlen( $id ) > 100 ) {
-            throw new SegmentIdTooLongException( 'Segment-id too long. Max 100 characters allowed', 400 );
+        if (strlen($id) > 100) {
+            throw new SegmentIdTooLongException('Segment-id too long. Max 100 characters allowed', 400);
         }
 
         $transUnitIdArrayForUniquenessCheck[] = $id;
-        $metadata[ 'id' ]                     = $id;
+        $metadata['id']                     = $id;
 
         // translate
-        if ( null !== $transUnit->attributes->getNamedItem( 'translate' ) ) {
-            $metadata[ 'translate' ] = $transUnit->attributes->getNamedItem( 'translate' )->nodeValue;
+        if (null !== $transUnit->attributes->getNamedItem('translate')) {
+            $metadata['translate'] = $transUnit->attributes->getNamedItem('translate')->nodeValue;
         }
 
         // approved
         // http://docs.oasis-open.org/xliff/v1.2/os/xliff-core.html#approved
-        if ( null !== $transUnit->attributes->getNamedItem( 'approved' ) ) {
-            $metadata[ 'approved' ] = filter_var( $transUnit->attributes->getNamedItem( 'approved' )->nodeValue, FILTER_VALIDATE_BOOLEAN );
+        if (null !== $transUnit->attributes->getNamedItem('approved')) {
+            $metadata['approved'] = filter_var($transUnit->attributes->getNamedItem('approved')->nodeValue, FILTER_VALIDATE_BOOLEAN);
         }
 
         return $metadata;
@@ -238,19 +238,19 @@ 
      * @return array
      * @throws Exception
      */
-    private function extractTransUnitNotes( DOMDocument $dom, DOMElement $transUnit ) {
+    private function extractTransUnitNotes(DOMDocument $dom, DOMElement $transUnit) {
         $notes = [];
-        foreach ( $transUnit->getElementsByTagName( 'note' ) as $note ) {
+        foreach ($transUnit->getElementsByTagName('note') as $note) {
 
-            $noteValue = $this->extractTagContent( $dom, $note );
+            $noteValue = $this->extractTagContent($dom, $note);
 
-            if ( '' !== $noteValue ) {
+            if ('' !== $noteValue) {
 
-                $extractedNote = $this->JSONOrRawContentArray( $noteValue );
+                $extractedNote = $this->JSONOrRawContentArray($noteValue);
 
                 // extract all the attributes
-                foreach ( $note->attributes as $attribute ) {
-                    $extractedNote[ $attribute->name ] = $attribute->value;
+                foreach ($note->attributes as $attribute) {
+                    $extractedNote[$attribute->name] = $attribute->value;
                 }
 
                 $notes[] = $extractedNote;
@@ -265,14 +265,14 @@ 
      *
      * @return array
      */
-    private function extractTransUnitContextGroup( DOMDocument $dom, DOMElement $contextGroup ) {
+    private function extractTransUnitContextGroup(DOMDocument $dom, DOMElement $contextGroup) {
         $cg           = [];
-        $cg[ 'attr' ] = $this->extractTagAttributes( $contextGroup );
+        $cg['attr'] = $this->extractTagAttributes($contextGroup);
 
         /** @var DOMNode $context */
-        foreach ( $contextGroup->childNodes as $context ) {
-            if ( $context->nodeName === 'context' ) {
-                $cg[ 'contexts' ][] = $this->extractContent( $dom, $context );
+        foreach ($contextGroup->childNodes as $context) {
+            if ($context->nodeName === 'context') {
+                $cg['contexts'][] = $this->extractContent($dom, $context);
             }
         }
 
@@ -284,16 +284,16 @@ 
      *
      * @return array
      */
-    private function extractTransUnitAltTrans( DOMElement $altTrans ) {
+    private function extractTransUnitAltTrans(DOMElement $altTrans) {
         $at           = [];
-        $at[ 'attr' ] = $this->extractTagAttributes( $altTrans );
+        $at['attr'] = $this->extractTagAttributes($altTrans);
 
-        if ( $altTrans->getElementsByTagName( 'source' )->length > 0 ) {
-            $at[ 'source' ] = $altTrans->getElementsByTagName( 'source' )->item( 0 )->nodeValue;
+        if ($altTrans->getElementsByTagName('source')->length > 0) {
+            $at['source'] = $altTrans->getElementsByTagName('source')->item(0)->nodeValue;
         }
 
-        if ( $altTrans->getElementsByTagName( 'target' ) ) {
-            $at[ 'target' ] = $altTrans->getElementsByTagName( 'target' )->item( 0 )->nodeValue;
+        if ($altTrans->getElementsByTagName('target')) {
+            $at['target'] = $altTrans->getElementsByTagName('target')->item(0)->nodeValue;
         }
 
         return $at;
@@ -304,7 +304,7 @@ 
      *
      * @return bool
      */
-    private function extractLocked( DOMElement $locked ) {
-        return null !== $locked->getAttribute( 'locked' );
+    private function extractLocked(DOMElement $locked) {
+        return null !== $locked->getAttribute('locked');
     }
 }

src/XliffParser/XliffParserV2.php

Spacing +139 added lines, -139 removed lines

@@ -16,30 +16,30 @@ 
      * @inheritDoc
      * @throws \Exception
      */
-    public function parse( DOMDocument $dom, $output = [] ) {
+    public function parse(DOMDocument $dom, $output = []) {
         $i = 1;
         /** @var DOMElement $file */
-        foreach ( $dom->getElementsByTagName( 'file' ) as $file ) {
+        foreach ($dom->getElementsByTagName('file') as $file) {
 
             // metadata
-            $output[ 'files' ][ $i ][ 'attr' ] = $this->extractMetadata( $dom );
+            $output['files'][$i]['attr'] = $this->extractMetadata($dom);
 
             // notes
-            $output[ 'files' ][ $i ][ 'notes' ] = $this->extractNotes( $file );
+            $output['files'][$i]['notes'] = $this->extractNotes($file);
 
             // trans-units
             $transUnitIdArrayForUniquenessCheck = [];
             $j                                  = 1;
             /** @var DOMElement $transUnit */
-            foreach ( $file->childNodes as $childNode ) {
-                $this->extractTuFromNode( $childNode, $transUnitIdArrayForUniquenessCheck, $dom, $output, $i, $j );
+            foreach ($file->childNodes as $childNode) {
+                $this->extractTuFromNode($childNode, $transUnitIdArrayForUniquenessCheck, $dom, $output, $i, $j);
             }
 
             // trans-unit re-count check
-            $totalTransUnitsId  = count( $transUnitIdArrayForUniquenessCheck );
-            $transUnitsUniqueId = count( array_unique( $transUnitIdArrayForUniquenessCheck ) );
-            if ( $totalTransUnitsId != $transUnitsUniqueId ) {
-                throw new DuplicateTransUnitIdInXliff( "Invalid trans-unit id, duplicate found.", 400 );
+            $totalTransUnitsId  = count($transUnitIdArrayForUniquenessCheck);
+            $transUnitsUniqueId = count(array_unique($transUnitIdArrayForUniquenessCheck));
+            if ($totalTransUnitsId != $transUnitsUniqueId) {
+                throw new DuplicateTransUnitIdInXliff("Invalid trans-unit id, duplicate found.", 400);
             }
 
             $i++;
@@ -53,23 +53,23 @@ 
      *
      * @return array
      */
-    private function extractMetadata( DOMDocument $dom ) {
+    private function extractMetadata(DOMDocument $dom) {
         $metadata = [];
 
-        $xliffNode = $dom->getElementsByTagName( 'xliff' )->item( 0 );
-        $fileNode  = $dom->getElementsByTagName( 'file' )->item( 0 );
+        $xliffNode = $dom->getElementsByTagName('xliff')->item(0);
+        $fileNode  = $dom->getElementsByTagName('file')->item(0);
 
         // original
-        $metadata[ 'original' ] = ( null !== $fileNode->attributes->getNamedItem( 'original' ) ) ? $fileNode->attributes->getNamedItem( 'original' )->nodeValue : 'no-name';
+        $metadata['original'] = (null !== $fileNode->attributes->getNamedItem('original')) ? $fileNode->attributes->getNamedItem('original')->nodeValue : 'no-name';
 
         // source-language
-        $metadata[ 'source-language' ] = ( null !== $xliffNode->attributes->getNamedItem( 'srcLang' ) ) ? $xliffNode->attributes->getNamedItem( 'srcLang' )->nodeValue : 'en-US';
+        $metadata['source-language'] = (null !== $xliffNode->attributes->getNamedItem('srcLang')) ? $xliffNode->attributes->getNamedItem('srcLang')->nodeValue : 'en-US';
 
         // datatype
         // @TODO to be implemented
 
         // target-language
-        $metadata[ 'target-language' ] = ( null !== $xliffNode->attributes->getNamedItem( 'trgLang' ) ) ? $xliffNode->attributes->getNamedItem( 'trgLang' )->nodeValue : 'en-US';
+        $metadata['target-language'] = (null !== $xliffNode->attributes->getNamedItem('trgLang')) ? $xliffNode->attributes->getNamedItem('trgLang')->nodeValue : 'en-US';
 
         // custom MateCat x-attribute
         // @TODO to be implemented
@@ -83,16 +83,16 @@ 
      * @return array
      * @throws \Exception
      */
-    private function extractNotes( DOMElement $file ) {
+    private function extractNotes(DOMElement $file) {
         $notes = [];
 
         // loop <notes> to get nested <note> tag
-        foreach ( $file->childNodes as $childNode ) {
-            if ( $childNode->nodeName === 'notes' ) {
-                foreach ( $childNode->childNodes as $note ) {
-                    $noteValue = trim( $note->nodeValue );
-                    if ( '' !== $noteValue ) {
-                        $notes[] = $this->JSONOrRawContentArray( $noteValue );
+        foreach ($file->childNodes as $childNode) {
+            if ($childNode->nodeName === 'notes') {
+                foreach ($childNode->childNodes as $note) {
+                    $noteValue = trim($note->nodeValue);
+                    if ('' !== $noteValue) {
+                        $notes[] = $this->JSONOrRawContentArray($noteValue);
                     }
                 }
             }
@@ -113,33 +113,33 @@ 
      *
      * @throws \Exception
      */
-    protected function extractTransUnit( $transUnit, &$transUnitIdArrayForUniquenessCheck, $dom, &$output, &$i, &$j ) {
+    protected function extractTransUnit($transUnit, &$transUnitIdArrayForUniquenessCheck, $dom, &$output, &$i, &$j) {
         // metadata
-        $output[ 'files' ][ $i ][ 'trans-units' ][ $j ][ 'attr' ] = $this->extractTransUnitMetadata( $transUnit, $transUnitIdArrayForUniquenessCheck );
+        $output['files'][$i]['trans-units'][$j]['attr'] = $this->extractTransUnitMetadata($transUnit, $transUnitIdArrayForUniquenessCheck);
 
         // notes
         // merge <notes> with key and key-note contained in metadata <mda:metaGroup>
-        $output[ 'files' ][ $i ][ 'trans-units' ][ $j ][ 'notes' ] = $this->extractTransUnitNotes( $transUnit );
+        $output['files'][$i]['trans-units'][$j]['notes'] = $this->extractTransUnitNotes($transUnit);
 
         // uuid
-        foreach ( $output[ 'files' ][ $i ][ 'trans-units' ][ $j ][ 'notes' ] as $note ) {
-            if ( isset( $note[ 'raw-content' ] ) && Strings::isAValidUuid( $note[ 'raw-content' ] ) ) {
-                $output[ 'files' ][ $i ][ 'trans-units' ][ $j ][ 'attr' ][ 'uuid' ] = $note[ 'raw-content' ];
+        foreach ($output['files'][$i]['trans-units'][$j]['notes'] as $note) {
+            if (isset($note['raw-content']) && Strings::isAValidUuid($note['raw-content'])) {
+                $output['files'][$i]['trans-units'][$j]['attr']['uuid'] = $note['raw-content'];
             }
         }
 
         // original-data (exclusive for V2)
         // http://docs.oasis-open.org/xliff/xliff-core/v2.0/xliff-core-v2.0.html#originaldata
-        $originalData = $this->extractTransUnitOriginalData( $transUnit );
-        if ( !empty( $originalData ) ) {
-            $output[ 'files' ][ $i ][ 'trans-units' ][ $j ][ 'original-data' ] = $originalData;
-            $dataRefMap                                                        = $this->getDataRefMap( $originalData );
+        $originalData = $this->extractTransUnitOriginalData($transUnit);
+        if (!empty($originalData)) {
+            $output['files'][$i]['trans-units'][$j]['original-data'] = $originalData;
+            $dataRefMap                                                        = $this->getDataRefMap($originalData);
         }
 
         // additionalTagData (exclusive for V2)
-        $additionalTagData = $this->extractTransUnitAdditionalTagData( $transUnit );
-        if ( !empty( $additionalTagData ) ) {
-            $output[ 'files' ][ $i ][ 'trans-units' ][ $j ][ 'additional-tag-data' ] = $additionalTagData;
+        $additionalTagData = $this->extractTransUnitAdditionalTagData($transUnit);
+        if (!empty($additionalTagData)) {
+            $output['files'][$i]['trans-units'][$j]['additional-tag-data'] = $additionalTagData;
         }
 
         // content
@@ -159,62 +159,62 @@ 
 
         /** @var DOMElement $segment */
         $c = 0;
-        foreach ( $transUnit->childNodes as $segment ) {
-            if ( $segment->nodeName === 'segment' ) {
+        foreach ($transUnit->childNodes as $segment) {
+            if ($segment->nodeName === 'segment') {
 
                 // check segment id consistency
-                $attr = $output[ 'files' ][ $i ][ 'trans-units' ][ $j ][ 'attr' ];
-                $this->checkSegmentIdConsistency( $segment, $attr );
+                $attr = $output['files'][$i]['trans-units'][$j]['attr'];
+                $this->checkSegmentIdConsistency($segment, $attr);
 
                 // loop <segment> to get nested <source> and <target> tag
-                foreach ( $segment->childNodes as $childNode ) {
-                    if ( $childNode->nodeName === 'source' ) {
-                        $extractedSource               = $this->extractContent( $dom, $childNode );
-                        $source[ 'raw-content' ][ $c ] = $extractedSource[ 'raw-content' ];
+                foreach ($segment->childNodes as $childNode) {
+                    if ($childNode->nodeName === 'source') {
+                        $extractedSource               = $this->extractContent($dom, $childNode);
+                        $source['raw-content'][$c] = $extractedSource['raw-content'];
 
-                        if ( !empty( $originalData ) ) {
-                            $source[ 'replaced-content' ][ $c ] = ( new DataRefReplacer( $dataRefMap ) )->replace( $source[ 'raw-content' ][ $c ] );
+                        if (!empty($originalData)) {
+                            $source['replaced-content'][$c] = (new DataRefReplacer($dataRefMap))->replace($source['raw-content'][$c]);
                         }
 
-                        if ( !empty( $extractedSource[ 'attr' ] ) ) {
-                            $source[ 'attr' ][ $c ] = $extractedSource[ 'attr' ];
+                        if (!empty($extractedSource['attr'])) {
+                            $source['attr'][$c] = $extractedSource['attr'];
                         }
 
                         // append value to 'seg-source'
-                        if ( $this->stringContainsMarks( $extractedSource[ 'raw-content' ] ) ) {
-                            $segSource = $this->extractContentWithMarksAndExtTags( $dom, $childNode, $extractedSource[ 'raw-content' ], $originalData );
+                        if ($this->stringContainsMarks($extractedSource['raw-content'])) {
+                            $segSource = $this->extractContentWithMarksAndExtTags($dom, $childNode, $extractedSource['raw-content'], $originalData);
                         } else {
                             $segSource[] = [
-                                    'mid'              => count( $segSource ) > 0 ? count( $segSource ) : 0,
+                                    'mid'              => count($segSource) > 0 ? count($segSource) : 0,
                                     'ext-prec-tags'    => '',
-                                    'raw-content'      => $extractedSource[ 'raw-content' ],
-                                    'replaced-content' => ( !empty( $originalData ) ) ? ( new DataRefReplacer( $dataRefMap ) )->replace( $extractedSource[ 'raw-content' ] ) : null,
+                                    'raw-content'      => $extractedSource['raw-content'],
+                                    'replaced-content' => (!empty($originalData)) ? (new DataRefReplacer($dataRefMap))->replace($extractedSource['raw-content']) : null,
                                     'ext-succ-tags'    => '',
                             ];
                         }
                     }
 
-                    if ( $childNode->nodeName === 'target' ) {
-                        $extractedTarget               = $this->extractContent( $dom, $childNode );
-                        $target[ 'raw-content' ][ $c ] = $extractedTarget[ 'raw-content' ];
+                    if ($childNode->nodeName === 'target') {
+                        $extractedTarget               = $this->extractContent($dom, $childNode);
+                        $target['raw-content'][$c] = $extractedTarget['raw-content'];
 
-                        if ( !empty( $originalData ) ) {
-                            $target[ 'replaced-content' ][ $c ] = ( new DataRefReplacer( $dataRefMap ) )->replace( $target[ 'raw-content' ][ $c ] );
+                        if (!empty($originalData)) {
+                            $target['replaced-content'][$c] = (new DataRefReplacer($dataRefMap))->replace($target['raw-content'][$c]);
                         }
 
-                        if ( !empty( $extractedTarget[ 'attr' ] ) ) {
-                            $target[ 'attr' ][ $c ] = $extractedTarget[ 'attr' ];
+                        if (!empty($extractedTarget['attr'])) {
+                            $target['attr'][$c] = $extractedTarget['attr'];
                         }
 
                         // append value to 'seg-target'
-                        if ( $this->stringContainsMarks( $extractedTarget[ 'raw-content' ] ) ) {
-                            $segTarget = $this->extractContentWithMarksAndExtTags( $dom, $childNode, $extractedTarget[ 'raw-content' ], $originalData );
+                        if ($this->stringContainsMarks($extractedTarget['raw-content'])) {
+                            $segTarget = $this->extractContentWithMarksAndExtTags($dom, $childNode, $extractedTarget['raw-content'], $originalData);
                         } else {
                             $segTarget[] = [
-                                    'mid'              => count( $segTarget ) > 0 ? count( $segTarget ) : 0,
+                                    'mid'              => count($segTarget) > 0 ? count($segTarget) : 0,
                                     'ext-prec-tags'    => '',
-                                    'raw-content'      => $extractedTarget[ 'raw-content' ],
-                                    'replaced-content' => ( !empty( $originalData ) ) ? ( new DataRefReplacer( $dataRefMap ) )->replace( $extractedTarget[ 'raw-content' ] ) : null,
+                                    'raw-content'      => $extractedTarget['raw-content'],
+                                    'replaced-content' => (!empty($originalData)) ? (new DataRefReplacer($dataRefMap))->replace($extractedTarget['raw-content']) : null,
                                     'ext-succ-tags'    => '',
                             ];
                         }
@@ -225,10 +225,10 @@ 
             }
         }
 
-        $output[ 'files' ][ $i ][ 'trans-units' ][ $j ][ 'source' ]     = $source;
-        $output[ 'files' ][ $i ][ 'trans-units' ][ $j ][ 'target' ]     = $target;
-        $output[ 'files' ][ $i ][ 'trans-units' ][ $j ][ 'seg-source' ] = $segSource;
-        $output[ 'files' ][ $i ][ 'trans-units' ][ $j ][ 'seg-target' ] = $segTarget;
+        $output['files'][$i]['trans-units'][$j]['source']     = $source;
+        $output['files'][$i]['trans-units'][$j]['target']     = $target;
+        $output['files'][$i]['trans-units'][$j]['seg-source'] = $segSource;
+        $output['files'][$i]['trans-units'][$j]['seg-target'] = $segTarget;
 
         $j++;
     }
@@ -239,41 +239,41 @@ 
      *
      * @return array
      */
-    private function extractTransUnitMetadata( DOMElement $transUnit, &$transUnitIdArrayForUniquenessCheck ) {
+    private function extractTransUnitMetadata(DOMElement $transUnit, &$transUnitIdArrayForUniquenessCheck) {
         $metadata = [];
 
         // id
-        if ( null === $transUnit->attributes->getNamedItem( 'id' ) ) {
-            throw new NotFoundIdInTransUnit( 'Invalid trans-unit id found. EMPTY value', 400 );
+        if (null === $transUnit->attributes->getNamedItem('id')) {
+            throw new NotFoundIdInTransUnit('Invalid trans-unit id found. EMPTY value', 400);
         }
 
-        $id = $transUnit->attributes->getNamedItem( 'id' )->nodeValue;
+        $id = $transUnit->attributes->getNamedItem('id')->nodeValue;
 
-        if ( strlen( $id ) > 100 ) {
-            throw new SegmentIdTooLongException( 'Segment-id too long. Max 100 characters allowed', 400 );
+        if (strlen($id) > 100) {
+            throw new SegmentIdTooLongException('Segment-id too long. Max 100 characters allowed', 400);
         }
 
         $transUnitIdArrayForUniquenessCheck[] = $id;
-        $metadata[ 'id' ]                     = $id;
+        $metadata['id']                     = $id;
 
         // translate
-        if ( null !== $transUnit->attributes->getNamedItem( 'translate' ) ) {
-            $metadata[ 'translate' ] = $transUnit->attributes->getNamedItem( 'translate' )->nodeValue;
+        if (null !== $transUnit->attributes->getNamedItem('translate')) {
+            $metadata['translate'] = $transUnit->attributes->getNamedItem('translate')->nodeValue;
         }
 
         // tGroupBegin
-        if ( null !== $transUnit->attributes->getNamedItem( 'tGroupBegin' ) ) {
-            $metadata[ 'tGroupBegin' ] = $transUnit->attributes->getNamedItem( 'tGroupBegin' )->nodeValue;
+        if (null !== $transUnit->attributes->getNamedItem('tGroupBegin')) {
+            $metadata['tGroupBegin'] = $transUnit->attributes->getNamedItem('tGroupBegin')->nodeValue;
         }
 
         // tGroupEnd
-        if ( null !== $transUnit->attributes->getNamedItem( 'tGroupEnd' ) ) {
-            $metadata[ 'tGroupEnd' ] = $transUnit->attributes->getNamedItem( 'tGroupEnd' )->nodeValue;
+        if (null !== $transUnit->attributes->getNamedItem('tGroupEnd')) {
+            $metadata['tGroupEnd'] = $transUnit->attributes->getNamedItem('tGroupEnd')->nodeValue;
         }
 
         // sizeRestriction
-        if ( null !== $transUnit->attributes->getNamedItem( 'sizeRestriction' ) && '' !== $transUnit->attributes->getNamedItem( 'sizeRestriction' )->nodeValue ) {
-            $metadata[ 'sizeRestriction' ] = (int)$transUnit->attributes->getNamedItem( 'sizeRestriction' )->nodeValue;
+        if (null !== $transUnit->attributes->getNamedItem('sizeRestriction') && '' !== $transUnit->attributes->getNamedItem('sizeRestriction')->nodeValue) {
+            $metadata['sizeRestriction'] = (int)$transUnit->attributes->getNamedItem('sizeRestriction')->nodeValue;
         }
 
         return $metadata;
@@ -285,31 +285,31 @@ 
      * @return array
      * @throws \Exception
      */
-    private function extractTransUnitOriginalData( DOMElement $transUnit ) {
+    private function extractTransUnitOriginalData(DOMElement $transUnit) {
         $originalData = [];
 
         // loop <originalData> to get nested content
-        foreach ( $transUnit->childNodes as $childNode ) {
-            if ( $childNode->nodeName === 'originalData' ) {
-                foreach ( $childNode->childNodes as $data ) {
-                    if ( null !== $data->attributes && null !== $data->attributes->getNamedItem( 'id' ) ) {
-                        $dataId = $data->attributes->getNamedItem( 'id' )->nodeValue;
+        foreach ($transUnit->childNodes as $childNode) {
+            if ($childNode->nodeName === 'originalData') {
+                foreach ($childNode->childNodes as $data) {
+                    if (null !== $data->attributes && null !== $data->attributes->getNamedItem('id')) {
+                        $dataId = $data->attributes->getNamedItem('id')->nodeValue;
 
-                        $dataValue = str_replace( Placeholder::WHITE_SPACE_PLACEHOLDER, ' ', $data->nodeValue );
-                        $dataValue = str_replace( Placeholder::NEW_LINE_PLACEHOLDER, '\n', $dataValue );
-                        $dataValue = str_replace( Placeholder::TAB_PLACEHOLDER, '\t', $dataValue );
+                        $dataValue = str_replace(Placeholder::WHITE_SPACE_PLACEHOLDER, ' ', $data->nodeValue);
+                        $dataValue = str_replace(Placeholder::NEW_LINE_PLACEHOLDER, '\n', $dataValue);
+                        $dataValue = str_replace(Placeholder::TAB_PLACEHOLDER, '\t', $dataValue);
 
-                        if ( '' !== $dataValue ) {
+                        if ('' !== $dataValue) {
 
-                            $jsonOrRawContentArray = $this->JSONOrRawContentArray( $dataValue, false );
+                            $jsonOrRawContentArray = $this->JSONOrRawContentArray($dataValue, false);
 
                             // restore xliff tags
-                            if ( isset( $jsonOrRawContentArray[ 'json' ] ) ) {
-                                $jsonOrRawContentArray[ 'json' ] = str_replace( [ Placeholder::LT_PLACEHOLDER, Placeholder::GT_PLACEHOLDER ], [ '&lt;', '&gt;' ], $jsonOrRawContentArray[ 'json' ] );
+                            if (isset($jsonOrRawContentArray['json'])) {
+                                $jsonOrRawContentArray['json'] = str_replace([Placeholder::LT_PLACEHOLDER, Placeholder::GT_PLACEHOLDER], ['&lt;', '&gt;'], $jsonOrRawContentArray['json']);
                             }
 
-                            if ( isset( $jsonOrRawContentArray[ 'raw-content' ] ) ) {
-                                $jsonOrRawContentArray[ 'raw-content' ] = str_replace( [ Placeholder::LT_PLACEHOLDER, Placeholder::GT_PLACEHOLDER ], [ '&lt;', '&gt;' ], $jsonOrRawContentArray[ 'raw-content' ] );
+                            if (isset($jsonOrRawContentArray['raw-content'])) {
+                                $jsonOrRawContentArray['raw-content'] = str_replace([Placeholder::LT_PLACEHOLDER, Placeholder::GT_PLACEHOLDER], ['&lt;', '&gt;'], $jsonOrRawContentArray['raw-content']);
                             }
 
                             $originalData[] = array_merge(
@@ -334,41 +334,41 @@ 
      *
      * @return array
      */
-    private function extractTransUnitAdditionalTagData( DOMElement $transUnit ) {
+    private function extractTransUnitAdditionalTagData(DOMElement $transUnit) {
         $additionalTagData = [];
 
         // loop <originalData> to get nested content
-        foreach ( $transUnit->childNodes as $childNode ) {
-            if ( $childNode->nodeName === 'memsource:additionalTagData' ) {
-                foreach ( $childNode->childNodes as $data ) {
+        foreach ($transUnit->childNodes as $childNode) {
+            if ($childNode->nodeName === 'memsource:additionalTagData') {
+                foreach ($childNode->childNodes as $data) {
                     $dataArray = [];
 
                     // id
-                    if ( $data->nodeName === 'memsource:tag' ) {
-                        if ( null !== $data->attributes && null !== $data->attributes->getNamedItem( 'id' ) ) {
-                            $dataId                      = $data->attributes->getNamedItem( 'id' )->nodeValue;
-                            $dataArray[ 'attr' ][ 'id' ] = $dataId;
+                    if ($data->nodeName === 'memsource:tag') {
+                        if (null !== $data->attributes && null !== $data->attributes->getNamedItem('id')) {
+                            $dataId                      = $data->attributes->getNamedItem('id')->nodeValue;
+                            $dataArray['attr']['id'] = $dataId;
                         }
                     }
 
                     // in PHP 7.4 $data->childNodes is an empty DomNodeList, it is iterable with size 0
                     // PHP 5.6 check: in php 5.6 $data->childNodes can be null
-                    if ( $data->childNodes != null ) {
+                    if ($data->childNodes != null) {
 
                         // content
-                        foreach ( $data->childNodes as $datum ) {
-                            if ( $datum->nodeName === 'memsource:tagId' ) {
-                                $dataArray[ 'raw-content' ][ 'tagId' ] = $datum->nodeValue;
+                        foreach ($data->childNodes as $datum) {
+                            if ($datum->nodeName === 'memsource:tagId') {
+                                $dataArray['raw-content']['tagId'] = $datum->nodeValue;
                             }
 
-                            if ( $datum->nodeName === 'memsource:type' ) {
-                                $dataArray[ 'raw-content' ][ 'type' ] = $datum->nodeValue;
+                            if ($datum->nodeName === 'memsource:type') {
+                                $dataArray['raw-content']['type'] = $datum->nodeValue;
                             }
                         }
 
                     }
 
-                    if ( !empty( $dataArray ) ) {
+                    if (!empty($dataArray)) {
                         $additionalTagData[] = $dataArray;
                     }
                 }
@@ -384,15 +384,15 @@ 
      * @param DOMElement $segment
      * @param array      $attr
      */
-    private function checkSegmentIdConsistency( DOMElement $segment, array $attr ) {
-        if ( isset( $attr[ 'tGroupBegin' ] ) && isset( $attr[ 'tGroupEnd' ] ) && $segment->attributes->getNamedItem( 'id' ) ) {
-            $id  = $segment->attributes->getNamedItem( 'id' )->nodeValue;
-            $min = (int)$attr[ 'tGroupBegin' ];
-            $max = (int)$attr[ 'tGroupEnd' ];
-
-            if ( false === ( ( $min <= $id ) && ( $id <= $max ) ) ) {
-                if ( $this->logger ) {
-                    $this->logger->warning( 'Segment #' . $id . ' is not included within tGroupBegin and tGroupEnd' );
+    private function checkSegmentIdConsistency(DOMElement $segment, array $attr) {
+        if (isset($attr['tGroupBegin']) && isset($attr['tGroupEnd']) && $segment->attributes->getNamedItem('id')) {
+            $id  = $segment->attributes->getNamedItem('id')->nodeValue;
+            $min = (int)$attr['tGroupBegin'];
+            $max = (int)$attr['tGroupEnd'];
+
+            if (false === (($min <= $id) && ($id <= $max))) {
+                if ($this->logger) {
+                    $this->logger->warning('Segment #' . $id . ' is not included within tGroupBegin and tGroupEnd');
                 }
             }
         }
@@ -404,31 +404,31 @@ 
      * @return array
      * @throws \Exception
      */
-    private function extractTransUnitNotes( DOMElement $transUnit ) {
+    private function extractTransUnitNotes(DOMElement $transUnit) {
         $notes = [];
 
         // loop <notes> to get nested <note> tag
-        foreach ( $transUnit->childNodes as $childNode ) {
-            if ( $childNode->nodeName == 'notes' ) {
-                foreach ( $childNode->childNodes as $note ) {
-                    $noteValue = trim( $note->nodeValue );
-                    if ( '' !== $noteValue ) {
-                        $notes[] = $this->JSONOrRawContentArray( $noteValue );
+        foreach ($transUnit->childNodes as $childNode) {
+            if ($childNode->nodeName == 'notes') {
+                foreach ($childNode->childNodes as $note) {
+                    $noteValue = trim($note->nodeValue);
+                    if ('' !== $noteValue) {
+                        $notes[] = $this->JSONOrRawContentArray($noteValue);
                     }
                 }
             }
 
-            if ( $childNode->nodeName === 'mda:metadata' ) {
-                foreach ( $childNode->childNodes as $metadata ) {
-                    if ( $metadata->nodeName === 'mda:metaGroup' ) {
-                        foreach ( $metadata->childNodes as $meta ) {
-                            if ( null !== $meta->attributes && null !== $meta->attributes->getNamedItem( 'type' ) ) {
-                                $type      = $meta->attributes->getNamedItem( 'type' )->nodeValue;
-                                $metaValue = trim( $meta->nodeValue );
+            if ($childNode->nodeName === 'mda:metadata') {
+                foreach ($childNode->childNodes as $metadata) {
+                    if ($metadata->nodeName === 'mda:metaGroup') {
+                        foreach ($metadata->childNodes as $meta) {
+                            if (null !== $meta->attributes && null !== $meta->attributes->getNamedItem('type')) {
+                                $type      = $meta->attributes->getNamedItem('type')->nodeValue;
+                                $metaValue = trim($meta->nodeValue);
 
-                                if ( '' !== $metaValue ) {
+                                if ('' !== $metaValue) {
                                     $notes[] = array_merge(
-                                            $this->JSONOrRawContentArray( $metaValue ),
+                                            $this->JSONOrRawContentArray($metaValue),
                                             [
                                                     'attr' => [
                                                             'type' => $type