matecat / xliff-parser

src/Utils/FlatData.php

Spacing +1 added lines, -1 removed lines

@@ -12,7 +12,7 @@
     public static function flatArray(array $input = [], $separator_elements = ', ', $separator = ': ')
     {
         return implode($separator_elements, array_map(
-            function ($v, $k, $s) {
+            function($v, $k, $s) {
                 return sprintf("%s{$s}\"%s\"", $k, $v);
             },
             $input,

src/Utils/HtmlParser.php

Spacing +29 added lines, -29 removed lines

@@ -53,11 +53,11 @@ 
 
         foreach ($matches[0] as $key => $match) {
 
-            $current = $matches[ 0 ][ $key ][ 0 ];
+            $current = $matches[0][$key][0];
 
-            if(isset($matches[0][$key+1][0])){
-                $next = $matches[0][$key+1][0];
-                $nextOffset = $matches[0][$key+1][1];
+            if (isset($matches[0][$key + 1][0])) {
+                $next = $matches[0][$key + 1][0];
+                $nextOffset = $matches[0][$key + 1][1];
                 $realNextOffset = ($delta === 0) ? $nextOffset : ($nextOffset + $delta);
             }
 
@@ -65,13 +65,13 @@ 
             $offset = $matches[0][$key][1];
             $realOffset = ($delta === 0) ? $offset : ($offset + $delta);
 
-            if( $current === '<' and isset($next)){
+            if ($current === '<' and isset($next)) {
 
                 // 1. if next is > or
                 // 2. next is < and is not html tag (like < >)
-                $insideAngularTags = substr($html, $realOffset, ($realNextOffset-$realOffset+1));
+                $insideAngularTags = substr($html, $realOffset, ($realNextOffset - $realOffset + 1));
 
-                if($next !== '>' or !Strings::isHtmlString($insideAngularTags) ){
+                if ($next !== '>' or !Strings::isHtmlString($insideAngularTags)) {
                     $html = substr_replace($html, self::LT_PLACEHOLDER, $realOffset, $length);
                     $delta = $delta + strlen(self::LT_PLACEHOLDER) - $length;
                 }
@@ -96,7 +96,7 @@ 
      *
      * @return mixed
      */
-    private static function protectNotClosedHtmlTags( $html)
+    private static function protectNotClosedHtmlTags($html)
     {
         preg_match_all('/<|>/iu', $html, $matches, PREG_OFFSET_CAPTURE);
 
@@ -106,17 +106,17 @@ 
 
         // 1. Map all tags
         foreach ($matches[0] as $key => $match) {
-            $current       = $matches[ 0 ][ $key ][ 0 ];
-            $currentOffset = $matches[ 0 ][ $key ][ 1 ];
+            $current       = $matches[0][$key][0];
+            $currentOffset = $matches[0][$key][1];
 
             // check every string inside angular brackets (< and >)
-            if( $current === '<' and isset($matches[0][$key+1][0]) and $matches[0][$key+1][0] === '>' ){
-                $nextOffset = $matches[0][$key+1][1];
-                $tag = substr($html, ($currentOffset + 1), ( $nextOffset - $currentOffset - 1 ));
+            if ($current === '<' and isset($matches[0][$key + 1][0]) and $matches[0][$key + 1][0] === '>') {
+                $nextOffset = $matches[0][$key + 1][1];
+                $tag = substr($html, ($currentOffset + 1), ($nextOffset - $currentOffset - 1));
                 $trimmedTag = trim($tag);
 
                 // if the tag is self closed do nothing
-                if(Strings::lastChar($tag) !== '/'){
+                if (Strings::lastChar($tag) !== '/') {
                     $tags[] = $trimmedTag;
                     $offsets[] = $currentOffset;
                     $originalLengths[] = strlen($tag) + 2; // add 2 to length because there are < and >
@@ -127,18 +127,18 @@ 
         // 2. Removing closed tags
         $indexes = [];
 
-        if(count($tags) > 0){
-            foreach ($tags as $index => $tag){
+        if (count($tags) > 0) {
+            foreach ($tags as $index => $tag) {
 
-                if(Strings::contains('/', $tag)){
+                if (Strings::contains('/', $tag)) {
                     $complementaryTag = $tag;
                 } else {
-                    $complementaryTag = '/'.explode(' ', $tag)[0];
+                    $complementaryTag = '/' . explode(' ', $tag)[0];
                 }
 
                 $complementaryTagIndex = array_search($complementaryTag, $tags);
 
-                if(false !== $complementaryTagIndex){
+                if (false !== $complementaryTagIndex) {
                     $indexes[] = $index;
                     $indexes[] = $complementaryTagIndex;
                 }
@@ -146,15 +146,15 @@ 
         }
 
         $indexes = array_unique($indexes);
-        foreach ($indexes as $index){
+        foreach ($indexes as $index) {
             unset($tags[$index]);
         }
 
         // 3. Loop not closed tags
         $delta = 0;
 
-        if(count($tags)){
-            foreach ($tags as $index => $tag){
+        if (count($tags)) {
+            foreach ($tags as $index => $tag) {
 
                 $length = $originalLengths[$index];
                 $offset = $offsets[$index];
@@ -176,9 +176,9 @@ 
      *
      * @return array
      */
-    private static function extractHtmlNode( $html, $toBeEscaped = false)
+    private static function extractHtmlNode($html, $toBeEscaped = false)
     {
-        $pattern = "/<([a-zA-Z0-9._-]+)([^>]|[^<]*?)(([\s]*\/>)|".
+        $pattern = "/<([a-zA-Z0-9._-]+)([^>]|[^<]*?)(([\s]*\/>)|" .
                 "(>((([^<]*?|<\!\-\-.*?\-\->)|(?R))*)<\/\\1[\s]*>))/sm";
         preg_match_all($pattern, $html, $matches, PREG_OFFSET_CAPTURE);
 
@@ -187,7 +187,7 @@ 
         foreach ($matches[0] as $key => $match) {
 
             $attributes = isset($matches[2][$key][0]) ? self::getAttributes($matches[2][$key][0]) : [];
-            $base64Decoded = (isset($attributes['equiv-text'])) ? base64_decode(str_replace("base64:", "", $attributes['equiv-text'])): null;
+            $base64Decoded = (isset($attributes['equiv-text'])) ? base64_decode(str_replace("base64:", "", $attributes['equiv-text'])) : null;
             $tagName = $matches[1][$key][0];
             $text = (isset($matches[6][$key][0]) and '' !== $matches[6][$key][0]) ? $matches[6][$key][0] : null;
             $originalText = $text;
@@ -237,9 +237,9 @@ 
      *
      * @return string|string[]
      */
-    private static function restoreLessThanAndGreaterThanSymbols( $text)
+    private static function restoreLessThanAndGreaterThanSymbols($text)
     {
-        return str_replace([self::LT_PLACEHOLDER, self::GT_PLACEHOLDER], ['<','>'], $text);
+        return str_replace([self::LT_PLACEHOLDER, self::GT_PLACEHOLDER], ['<', '>'], $text);
     }
 
     /**
@@ -254,13 +254,13 @@ 
     {
         $node = '';
 
-        if($start !== null){
+        if ($start !== null) {
             $node .= ($toBeEscaped) ? Strings::escapeOnlyHTMLTags($start) : $start;
         }
 
         $node .= ($toBeEscaped) ? Strings::escapeOnlyHTMLTags($originalText) : $originalText;
 
-        if($end !== null){
+        if ($end !== null) {
             $node .= ($toBeEscaped) ? Strings::escapeOnlyHTMLTags($end) : $end;
         }
 

src/Utils/Emoji.php

Indentation +1 added lines, -1 removed lines

@@ -7404,7 +7404,7 @@
             '

Spacing +5 added lines, -5 removed lines

@@ -7402,7 +7402,7 @@
             '

src/Utils/Files.php

Spacing +3 added lines, -3 removed lines

@@ -46,7 +46,7 @@ 
             if (($options & $i) > 0) {
                 //variable substitution: $field can be one between 'dirname', 'basename', 'extension', 'filename'
                 // $$field gets the value of the variable named $field
-                $returnArray[ $field ] = $$field;
+                $returnArray[$field] = $$field;
             }
         }
 
@@ -70,7 +70,7 @@ 
             return false;
         }
 
-        return strtolower($pathInfo[ 'extension' ]);
+        return strtolower($pathInfo['extension']);
     }
 
     /**
@@ -112,7 +112,7 @@ 
             return false;
         }
 
-        switch (strtolower($pathInfo[ 'extension' ])) {
+        switch (strtolower($pathInfo['extension'])) {
             case 'tmx':
                 return 'tmx';
                 break;

src/XliffParser.php

Spacing +24 added lines, -24 removed lines

@@ -65,22 +65,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);
         }
 
@@ -105,7 +105,7 @@ 
         $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";
+            $xliff['parser-warnings'][] = "Input identified as $enc ans converted UTF-8. May not be a problem if the content is English only";
 
             return iconv($enc, 'UTF-8', $xliffContent);
         }
@@ -126,22 +126,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;
@@ -178,8 +178,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;
     }
@@ -202,11 +202,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);
             }
         }
@@ -227,7 +227,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>';
     }
 
     /**
@@ -240,11 +240,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/XliffUtils/XliffProprietaryDetect.php

Spacing +18 added lines, -18 removed lines

@@ -47,7 +47,7 @@ 
      *
      * @return array
      */
-    private static function getInfoFromTmp( $tmp)
+    private static function getInfoFromTmp($tmp)
     {
         try {
             self::checkVersion($tmp);
@@ -59,10 +59,10 @@ 
         // run CheckXliffProprietaryPipeline
         $pipeline = self::runPipeline($tmp);
 
-        self::$fileType['proprietary' ] = $pipeline['proprietary' ];
-        self::$fileType[ 'proprietary_name' ] = $pipeline['proprietary_name' ];
-        self::$fileType[ 'proprietary_short_name' ] = $pipeline['proprietary_short_name' ];
-        self::$fileType[ 'converter_version' ] = $pipeline['converter_version' ];
+        self::$fileType['proprietary'] = $pipeline['proprietary'];
+        self::$fileType['proprietary_name'] = $pipeline['proprietary_name'];
+        self::$fileType['proprietary_short_name'] = $pipeline['proprietary_short_name'];
+        self::$fileType['converter_version'] = $pipeline['converter_version'];
 
         return self::$fileType;
     }
@@ -125,7 +125,7 @@ 
         }
 
         if (!empty($stringData)) {
-            return array( $stringData );
+            return array($stringData);
         }
 
         return false;
@@ -139,8 +139,8 @@ 
      */
     protected static function checkVersion($tmp)
     {
-        if (isset($tmp[ 0 ])) {
-            self::$fileType[ 'version' ] = XliffVersionDetector::detect($tmp[ 0 ]);
+        if (isset($tmp[0])) {
+            self::$fileType['version'] = XliffVersionDetector::detect($tmp[0]);
         }
     }
 
@@ -162,10 +162,10 @@ 
         // run CheckXliffProprietaryPipeline
         $pipeline = self::runPipeline($tmp);
 
-        self::$fileType['proprietary' ] = $pipeline['proprietary' ];
-        self::$fileType[ 'proprietary_name' ] = $pipeline['proprietary_name' ];
-        self::$fileType[ 'proprietary_short_name' ] = $pipeline['proprietary_short_name' ];
-        self::$fileType[ 'converter_version' ] = $pipeline['converter_version' ];
+        self::$fileType['proprietary'] = $pipeline['proprietary'];
+        self::$fileType['proprietary_name'] = $pipeline['proprietary_name'];
+        self::$fileType['proprietary_short_name'] = $pipeline['proprietary_short_name'];
+        self::$fileType['converter_version'] = $pipeline['converter_version'];
 
         return self::$fileType;
     }
@@ -194,7 +194,7 @@ 
 
                     //if file is not proprietary AND Enforce is disabled
                     //we take it as is
-                    if (!$fileType[ 'proprietary' ] or $memoryFileType) {
+                    if (!$fileType['proprietary'] or $memoryFileType) {
                         $convert = false;
                         //ok don't convert a standard sdlxliff
                     }
@@ -202,16 +202,16 @@ 
                     //if conversion enforce is active
                     //we force all xliff files but not files produced by SDL Studio because we can handle them
                     if (
-                            $fileType[ 'proprietary_short_name' ] == 'matecat_converter'
-                            or $fileType[ 'proprietary_short_name' ] == 'trados'
-                            or $fileType[ 'proprietary_short_name' ] == 'xliff_v2'
+                            $fileType['proprietary_short_name'] == 'matecat_converter'
+                            or $fileType['proprietary_short_name'] == 'trados'
+                            or $fileType['proprietary_short_name'] == 'xliff_v2'
                             or $memoryFileType
                     ) {
                         $convert = false;
                         //ok don't convert a standard sdlxliff
                     }
                 }
-            } elseif ($fileType[ 'proprietary' ]) {
+            } elseif ($fileType['proprietary']) {
 
                 /**
                  * Application misconfiguration.
@@ -221,7 +221,7 @@ 
 
                 $convert = -1;
             //stop execution
-            } elseif (!$fileType[ 'proprietary' ]) {
+            } elseif (!$fileType['proprietary']) {
                 $convert = false;
                 //ok don't convert a standard sdlxliff
             }

src/XliffUtils/XliffVersionDetector.php

Spacing +1 added lines, -1 removed lines

@@ -28,7 +28,7 @@
     {
         preg_match('|<xliff.*?\sversion\s?=\s?["\'](.*?)["\']|si', substr($xliffContent, 0, 1000), $versionMatches);
 
-        if (empty($versionMatches) ) {
+        if (empty($versionMatches)) {
             throw new NotValidFileException('This is not a valid xliff file');
         }
 

src/XliffUtils/DataRefReplacer.php

Spacing +68 added lines, -68 removed lines

@@ -17,7 +17,7 @@ 
      *
      * @param array $map
      */
-    public function __construct( array $map)
+    public function __construct(array $map)
     {
         $this->map = $map;
     }
@@ -48,14 +48,14 @@ 
         $html = HtmlParser::parse($string);
 
         // 1. Replace <ph>|<sc>|<ec> tags
-        foreach ($html as $node){
+        foreach ($html as $node) {
             $string = $this->recursiveAddEquivTextToPhTag($node, $string);
         }
 
         // 2. Replace <pc> tags
         $toBeEscaped = Strings::isAnEscapedHTML($string);
 
-        if($this->stringContainsPcTags($string, $toBeEscaped)){
+        if ($this->stringContainsPcTags($string, $toBeEscaped)) {
 
             // replace self-closed <pc />
             $string = $this->replaceSelfClosedPcTags($string, $toBeEscaped);
@@ -84,10 +84,10 @@ 
             '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;
             }
         }
@@ -102,7 +102,7 @@ 
     {
         $html = HtmlParser::parse($string);
 
-        foreach ($html as $node){
+        foreach ($html as $node) {
             $string = $this->recursiveCleanFromEquivText($node, $string);
         }
 
@@ -122,10 +122,10 @@ 
      *
      * @return string
      */
-    private function recursiveAddEquivTextToPhTag( $node, $string)
+    private function recursiveAddEquivTextToPhTag($node, $string)
     {
-        if($node->has_children){
-            foreach ($node->inner_html as $childNode){
+        if ($node->has_children) {
+            foreach ($node->inner_html as $childNode) {
                 $string = $this->recursiveAddEquivTextToPhTag($childNode, $string);
             }
         } else {
@@ -134,18 +134,18 @@ 
                     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])){
+                if (is_null($this->map[$b])) {
                     $this->map[$b] = 'NULL';
                 }
 
@@ -157,19 +157,19 @@ 
                 }
 
                 // 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);
+                if ($this->isAEcOrScTag($node)) {
+                    $d = 'ph' . substr($d, 2);
                     $d = trim($d);
                 }
 
@@ -191,7 +191,7 @@ 
         $regex = ($toBeEscaped) ? '/&lt;pc (.*?)&gt;/iu' : '/<pc (.*?)>/iu';
         preg_match_all($regex, $string, $openingPcMatches);
 
-        return (isset($openingPcMatches[0]) and count($openingPcMatches[0])>0);
+        return (isset($openingPcMatches[0]) and count($openingPcMatches[0]) > 0);
     }
 
     /**
@@ -202,27 +202,27 @@ 
      */
     private function replaceSelfClosedPcTags($string, $toBeEscaped)
     {
-        if($toBeEscaped){
-            $string = str_replace(['&lt;','&gt;'],['<','>'],$string);
+        if ($toBeEscaped) {
+            $string = str_replace(['&lt;', '&gt;'], ['<', '>'], $string);
         }
 
         $regex = '/<pc[^>]+?\/>/iu';
         preg_match_all($regex, $string, $selfClosedPcMatches);
 
-        foreach ($selfClosedPcMatches[0] as $match){
+        foreach ($selfClosedPcMatches[0] as $match) {
 
             $html = HtmlParser::parse($match);
             $node = $html[0];
             $attributes = $node->attributes;
 
-            if(isset($attributes['dataRefStart']) and 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']]).'"/>';
+            if (isset($attributes['dataRefStart']) and 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;
@@ -241,7 +241,7 @@ 
         $dataRefEndMap = [];
 
         foreach ($html as $index => $node) {
-            if ( $node->tagname === 'pc' ) {
+            if ($node->tagname === 'pc') {
                 $this->extractDataRefMapRecursively($node, $dataRefEndMap);
             }
         }
@@ -255,26 +255,26 @@ 
      * @param object $node
      * @param $dataRefEndMap
      */
-    private function extractDataRefMapRecursively( $node, &$dataRefEndMap)
+    private function extractDataRefMapRecursively($node, &$dataRefEndMap)
     {
-        if($this->nodeContainsNestedPcTags($node)) {
-            foreach ( $node->inner_html as $nestedNode ) {
+        if ($this->nodeContainsNestedPcTags($node)) {
+            foreach ($node->inner_html as $nestedNode) {
                 $this->extractDataRefMapRecursively($nestedNode, $dataRefEndMap);
             }
         }
 
         // EXCLUDE self closed <pc/>
-        if($node->tagname === 'pc' and $node->self_closed === false){
-            if(isset($node->attributes['dataRefEnd'])){
+        if ($node->tagname === 'pc' and $node->self_closed === false) {
+            if (isset($node->attributes['dataRefEnd'])) {
                 $dataRefEnd = $node->attributes['dataRefEnd'];
-            } elseif(isset($node->attributes['dataRefStart'])) {
+            } 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,
             ];
         }
@@ -288,12 +288,12 @@ 
      */
     private function recursiveCleanFromEquivText($node, $string)
     {
-        if($node->has_children){
-            foreach ($node->inner_html as $childNode){
+        if ($node->has_children) {
+            foreach ($node->inner_html as $childNode) {
                 $string = $this->recursiveCleanFromEquivText($childNode, $string);
             }
         } else {
-            if(isset($node->attributes['dataRef']) and array_key_exists($node->attributes['dataRef'], $this->map)){
+            if (isset($node->attributes['dataRef']) and array_key_exists($node->attributes['dataRef'], $this->map)) {
                 $cleaned = preg_replace('/ equiv-text="(.*?)"/', '', $node->node);
                 $string = str_replace($node->node, $cleaned, $string);
             }
@@ -315,29 +315,29 @@ 
         $regex = ($toBeEscaped) ? '/&lt;pc (.*?)&gt;/iu' : '/<pc (.*?)>/iu';
         preg_match_all($regex, $string, $openingPcMatches);
 
-        foreach ($openingPcMatches[0] as $index => $match){
+        foreach ($openingPcMatches[0] as $index => $match) {
             $attr = HtmlParser::getAttributes($openingPcMatches[1][$index]);
 
             // CASE 1 - Missing `dataRefStart`
-            if( isset($attr['dataRefEnd']) and !isset($attr['dataRefStart'])  ){
+            if (isset($attr['dataRefEnd']) and !isset($attr['dataRefStart'])) {
                 $attr['dataRefStart'] = $attr['dataRefEnd'];
             }
 
             // CASE 2 - Missing `dataRefEnd`
-            if( isset($attr['dataRefStart']) and !isset($attr['dataRefEnd'])  ){
+            if (isset($attr['dataRefStart']) and !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);
 
                 // conversion for opening <pc> tag
-                $openingPcConverted  = '<ph '. ((isset($attr['id'])) ? 'id="'.$attr['id'].'_1"' : '') .' dataType="pcStart" originalData="'.$base64StartOriginalData.'" dataRef="'
-                        .$attr['dataRefStart'].'" equiv-text="base64:'
-                        .$base64EncodedStartValue.'"/>';
+                $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);
             }
@@ -362,20 +362,20 @@ 
         preg_match_all($regex, $string, $closingPcMatches, PREG_OFFSET_CAPTURE);
         $delta = 0;
 
-        foreach ( $closingPcMatches[ 0 ] as $index => $match ) {
+        foreach ($closingPcMatches[0] as $index => $match) {
             $offset = $match[1];
             $length = strlen($match[0]);
             $attr = $dataRefEndMap[$index];
 
-            if(!empty($attr) and isset($attr['dataRefEnd'])){
+            if (!empty($attr) and isset($attr['dataRefEnd'])) {
                 $endOriginalData = $match[0]; // </pc>
                 $endValue = $this->map[$attr['dataRefEnd']] ? $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);
 
@@ -394,12 +394,12 @@ 
      */
     private function nodeContainsNestedPcTags($node)
     {
-        if(!$node->has_children){
+        if (!$node->has_children) {
             return false;
         }
 
         foreach ($node->inner_html as $nestedNode) {
-            if($nestedNode->tagname === 'pc' and (isset($node->attributes['dataRefEnd']) or isset($node->attributes['dataRefStart']))){
+            if ($nestedNode->tagname === 'pc' and (isset($node->attributes['dataRefEnd']) or isset($node->attributes['dataRefStart']))) {
                 return true;
             }
         }
@@ -423,7 +423,7 @@ 
         $string = str_replace(' equiv-text=""', '', $string);
         $html = HtmlParser::parse($string);
 
-        foreach ($html as $node){
+        foreach ($html as $node) {
             $string = $this->recursiveRemoveOriginalData($node, $string);
         }
 
@@ -438,43 +438,43 @@ 
      */
     private function recursiveRemoveOriginalData($node, $string)
     {
-        if($node->has_children){
-            foreach ($node->inner_html as $childNode){
+        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"/>
+            $a = $node->node; // complete match. Eg:  <ph id="source1" dataRef="source1"/>
             $b = $node->attributes['dataRef']; // map identifier. Eg: source1
-            $c = $node->terminator;            // terminator: Eg: >
+            $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])){
+            if (is_null($this->map[$b])) {
                 $this->map[$b] = 'NULL';
             }
 
             // remove id?
-            $removeId = (isset($node->attributes['removeId']) and $node->attributes['removeId'] === "true") ? ' id="'.$b.'" removeId="true"' : '';
+            $removeId = (isset($node->attributes['removeId']) and $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);
+            if ($this->wasAEcOrScTag($node)) {
+                $d = $node->attributes['dataType'] . substr($d, 3);
                 $d = trim($d);
             }
 
@@ -487,7 +487,7 @@ 
             $string = str_replace($a, $d, $string);
 
             // restoring <pc/> self-closed here
-            if(Strings::contains('dataType="pcSelf"', $d)){
+            if (Strings::contains('dataType="pcSelf"', $d)) {
                 preg_match('/\s?originalData="(.*?)"\s?/', $d, $originalDataMatches);
 
                 if (isset($originalDataMatches[1])) {
@@ -546,6 +546,6 @@ 
      */
     private function wasAEcOrScTag($node)
     {
-        return (isset($node->attributes['dataType']) and ( $node->attributes['dataType'] === 'ec' or $node->attributes['dataType'] === 'sc'));
+        return (isset($node->attributes['dataType']) and ($node->attributes['dataType'] === 'ec' or $node->attributes['dataType'] === 'sc'));
     }
 }

src/XliffUtils/CheckPipeline/CheckXliffVersion2.php

Spacing +5 added lines, -5 removed lines

@@ -13,7 +13,7 @@ 
     {
         $fileType = [];
 
-        if (isset($tmp[ 0 ])) {
+        if (isset($tmp[0])) {
             preg_match('|<xliff.*?\sversion\s?=\s?["\'](.*?)["\']|si', substr($tmp[0], 0, 1000), $versionMatches);
             preg_match('|<xliff.*?\sxmlns\s?=\s?["\']urn:oasis:names:tc:xliff:document:(.*?)["\']|si', substr($tmp[0], 0, 1000), $xmlnsMatches);
 
@@ -22,10 +22,10 @@ 
                 $xmlns = $xmlnsMatches[1];
 
                 if ($version === $xmlns and $version >= 2) {
-                    $fileType[ 'proprietary' ]            = false;
-                    $fileType[ 'proprietary_name' ]       = 'Xliff v'.$version.' File';
-                    $fileType[ 'proprietary_short_name' ] = 'xliff_v2';
-                    $fileType[ 'converter_version' ]      = '2.0';
+                    $fileType['proprietary']            = false;
+                    $fileType['proprietary_name']       = 'Xliff v' . $version . ' File';
+                    $fileType['proprietary_short_name'] = 'xliff_v2';
+                    $fileType['converter_version']      = '2.0';
 
                     return $fileType;
                 }