matecat / xliff-parser

src/Utils/HtmlParser.php

Spacing +92 added lines, -92 removed lines

@@ -15,17 +15,17 @@ 
      *
      * @return array
      */
-    public static function parse( $html ) {
-        $toBeEscaped = Strings::isAnEscapedHTML( $html );
+    public static function parse($html) {
+        $toBeEscaped = Strings::isAnEscapedHTML($html);
 
-        if ( $toBeEscaped ) {
-            $html = Strings::htmlspecialchars_decode( $html );
+        if ($toBeEscaped) {
+            $html = Strings::htmlspecialchars_decode($html);
         }
 
-        $html = self::protectNotClosedHtmlTags( $html );
-        $html = self::protectNotHtmlLessThanSymbols( $html );
+        $html = self::protectNotClosedHtmlTags($html);
+        $html = self::protectNotHtmlLessThanSymbols($html);
 
-        return self::extractHtmlNode( $html, $toBeEscaped );
+        return self::extractHtmlNode($html, $toBeEscaped);
     }
 
     /**
@@ -43,41 +43,41 @@ 
      *
      * @return string
      */
-    private static function protectNotHtmlLessThanSymbols( $html ) {
-        preg_match_all( '/<|>/iu', $html, $matches, PREG_OFFSET_CAPTURE );
+    private static function protectNotHtmlLessThanSymbols($html) {
+        preg_match_all('/<|>/iu', $html, $matches, PREG_OFFSET_CAPTURE);
 
         $delta          = 0;
         $realNextOffset = 0;
         $next           = null;
 
-        foreach ( $matches[ 0 ] as $key => $match ) {
+        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 ];
-                $realNextOffset = ( $delta === 0 ) ? $nextOffset : ( $nextOffset + $delta );
+            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);
             }
 
-            $length     = strlen( $match[ 0 ] );
-            $offset     = $matches[ 0 ][ $key ][ 1 ];
-            $realOffset = ( $delta === 0 ) ? $offset : ( $offset + $delta );
+            $length     = strlen($match[0]);
+            $offset     = $matches[0][$key][1];
+            $realOffset = ($delta === 0) ? $offset : ($offset + $delta);
 
-            if ( $current === '<' && isset( $next ) ) {
+            if ($current === '<' && 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 !== '>' || !Strings::isHtmlString( $insideAngularTags ) ) {
-                    $html  = substr_replace( $html, self::LT_PLACEHOLDER, $realOffset, $length );
-                    $delta = $delta + strlen( self::LT_PLACEHOLDER ) - $length;
+                if ($next !== '>' || !Strings::isHtmlString($insideAngularTags)) {
+                    $html  = substr_replace($html, self::LT_PLACEHOLDER, $realOffset, $length);
+                    $delta = $delta + strlen(self::LT_PLACEHOLDER) - $length;
                 }
             }
         }
 
-        return !is_array( $html ) ? $html : implode( $html );
+        return !is_array($html) ? $html : implode($html);
     }
 
     /**
@@ -95,29 +95,29 @@ 
      *
      * @return string
      */
-    private static function protectNotClosedHtmlTags( $html ) {
-        preg_match_all( '/<|>/iu', $html, $matches, PREG_OFFSET_CAPTURE );
+    private static function protectNotClosedHtmlTags($html) {
+        preg_match_all('/<|>/iu', $html, $matches, PREG_OFFSET_CAPTURE);
 
         $tags            = [];
         $offsets         = [];
         $originalLengths = [];
 
         // 1. Map all tags
-        foreach ( $matches[ 0 ] as $key => $match ) {
-            $current       = $matches[ 0 ][ $key ][ 0 ];
-            $currentOffset = $matches[ 0 ][ $key ][ 1 ];
+        foreach ($matches[0] as $key => $match) {
+            $current       = $matches[0][$key][0];
+            $currentOffset = $matches[0][$key][1];
 
             // check every string inside angular brackets (< and >)
-            if ( $current === '<' && isset( $matches[ 0 ][ $key + 1 ][ 0 ] ) && $matches[ 0 ][ $key + 1 ][ 0 ] === '>' ) {
-                $nextOffset = $matches[ 0 ][ $key + 1 ][ 1 ];
-                $tag        = substr( $html, ( $currentOffset + 1 ), ( $nextOffset - $currentOffset - 1 ) );
-                $trimmedTag = trim( $tag );
+            if ($current === '<' && isset($matches[0][$key + 1][0]) && $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 >
+                    $originalLengths[] = strlen($tag) + 2; // add 2 to length because there are < and >
                 }
             }
         }
@@ -125,47 +125,47 @@ 
         // 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 );
+                $complementaryTagIndex = array_search($complementaryTag, $tags);
 
-                if ( false !== $complementaryTagIndex ) {
+                if (false !== $complementaryTagIndex) {
                     $indexes[] = $index;
                     $indexes[] = $complementaryTagIndex;
                 }
             }
         }
 
-        $indexes = array_unique( $indexes );
-        foreach ( $indexes as $index ) {
-            unset( $tags[ $index ] );
+        $indexes = array_unique($indexes);
+        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 ];
-                $realOffset = ( $delta === 0 ) ? $offset : ( $offset + $delta );
+                $length     = $originalLengths[$index];
+                $offset     = $offsets[$index];
+                $realOffset = ($delta === 0) ? $offset : ($offset + $delta);
 
                 $replacement = self::LT_PLACEHOLDER . $tag . self::GT_PLACEHOLDER;
 
-                $html  = substr_replace( $html, $replacement, $realOffset, $length );
-                $delta = $delta + strlen( $replacement ) - $length;
+                $html  = substr_replace($html, $replacement, $realOffset, $length);
+                $delta = $delta + strlen($replacement) - $length;
             }
         }
 
-        return !is_array( $html ) ? $html : implode( $html );
+        return !is_array($html) ? $html : implode($html);
     }
 
     /**
@@ -174,55 +174,55 @@ 
      *
      * @return array
      */
-    private static function extractHtmlNode( $html, $toBeEscaped = false ) {
+    private static function extractHtmlNode($html, $toBeEscaped = false) {
         $pattern = "/<([a-zA-Z0-9._-]+)([^>]|[^<]*?)(([\s]*\/>)|" .
                 "(>((([^<]*?|<\!\-\-.*?\-\->)|(?R))*)<\/\\1[\s]*>))/sm";
-        preg_match_all( $pattern, $html, $matches, PREG_OFFSET_CAPTURE );
+        preg_match_all($pattern, $html, $matches, PREG_OFFSET_CAPTURE);
 
         $elements = [];
 
-        foreach ( $matches[ 0 ] as $key => $match ) {
+        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;
-            $tagName       = $matches[ 1 ][ $key ][ 0 ];
-            $text          = !empty( $matches[ 6 ][ $key ][ 0 ] ) ? $matches[ 6 ][ $key ][ 0 ] : "";
+            $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;
+            $tagName       = $matches[1][$key][0];
+            $text          = !empty($matches[6][$key][0]) ? $matches[6][$key][0] : "";
             $originalText  = $text;
-            $strippedText  = strip_tags( $text );
+            $strippedText  = strip_tags($text);
 
             // get start and end tags
-            $explodedNode = explode( self::ORIGINAL_TEXT_PLACEHOLDER, str_replace( $originalText, self::ORIGINAL_TEXT_PLACEHOLDER, $match[ 0 ] ) );
+            $explodedNode = explode(self::ORIGINAL_TEXT_PLACEHOLDER, str_replace($originalText, self::ORIGINAL_TEXT_PLACEHOLDER, $match[0]));
 
-            $start = ( isset( $explodedNode[ 0 ] ) ) ? $explodedNode[ 0 ] : "";
-            $end   = ( isset( $explodedNode[ 1 ] ) ) ? $explodedNode[ 1 ] : "";
+            $start = (isset($explodedNode[0])) ? $explodedNode[0] : "";
+            $end   = (isset($explodedNode[1])) ? $explodedNode[1] : "";
 
             // inner_html
-            $inner_html = self::getInnerHtml( $matches, $key, $toBeEscaped );
+            $inner_html = self::getInnerHtml($matches, $key, $toBeEscaped);
 
             // node
-            $node = self::rebuildNode( $originalText, $toBeEscaped, $start, $end );
+            $node = self::rebuildNode($originalText, $toBeEscaped, $start, $end);
 
             // terminator
-            $terminator = ( $toBeEscaped ) ? '&gt;' : '>';
+            $terminator = ($toBeEscaped) ? '&gt;' : '>';
 
             // self closed
-            $selfClosed = Strings::contains( '/>', trim( $start ) );
+            $selfClosed = Strings::contains('/>', trim($start));
 
             $elements[] = (object)[
-                    'node'           => self::restoreLessThanAndGreaterThanSymbols( $node ),
-                    'start'          => self::restoreLessThanAndGreaterThanSymbols( $start ),
-                    'end'            => self::restoreLessThanAndGreaterThanSymbols( $end ),
+                    'node'           => self::restoreLessThanAndGreaterThanSymbols($node),
+                    'start'          => self::restoreLessThanAndGreaterThanSymbols($start),
+                    'end'            => self::restoreLessThanAndGreaterThanSymbols($end),
                     'terminator'     => $terminator,
-                    'offset'         => $match[ 1 ],
+                    'offset'         => $match[1],
                     'tagname'        => $tagName,
                     'attributes'     => $attributes,
                     'base64_decoded' => $base64Decoded,
                     'self_closed'    => $selfClosed,
-                    'omittag'        => ( $matches[ 4 ][ $key ][ 1 ] > -1 ), // boolean
+                    'omittag'        => ($matches[4][$key][1] > -1), // boolean
                     'inner_html'     => $inner_html,
-                    'has_children'   => is_array( $inner_html ),
-                    'original_text'  => ( $toBeEscaped ) ? self::restoreLessThanAndGreaterThanSymbols( Strings::escapeOnlyHTMLTags( $originalText ) ) : self::restoreLessThanAndGreaterThanSymbols( $originalText ),
-                    'stripped_text'  => self::restoreLessThanAndGreaterThanSymbols( $strippedText ),
+                    'has_children'   => is_array($inner_html),
+                    'original_text'  => ($toBeEscaped) ? self::restoreLessThanAndGreaterThanSymbols(Strings::escapeOnlyHTMLTags($originalText)) : self::restoreLessThanAndGreaterThanSymbols($originalText),
+                    'stripped_text'  => self::restoreLessThanAndGreaterThanSymbols($strippedText),
             ];
         }
 
@@ -234,8 +234,8 @@ 
      *
      * @return string|string[]
      */
-    private static function restoreLessThanAndGreaterThanSymbols( $text ) {
-        return str_replace( [ self::LT_PLACEHOLDER, self::GT_PLACEHOLDER ], [ '<', '>' ], $text );
+    private static function restoreLessThanAndGreaterThanSymbols($text) {
+        return str_replace([self::LT_PLACEHOLDER, self::GT_PLACEHOLDER], ['<', '>'], $text);
     }
 
     /**
@@ -246,17 +246,17 @@ 
      *
      * @return string
      */
-    private static function rebuildNode( $originalText, $toBeEscaped, $start = null, $end = null ) {
+    private static function rebuildNode($originalText, $toBeEscaped, $start = null, $end = null) {
         $node = '';
 
-        if ( !empty( $start ) ) {
-            $node .= ( $toBeEscaped ) ? Strings::escapeOnlyHTMLTags( $start ) : $start;
+        if (!empty($start)) {
+            $node .= ($toBeEscaped) ? Strings::escapeOnlyHTMLTags($start) : $start;
         }
 
-        $node .= ( $toBeEscaped ) ? Strings::escapeOnlyHTMLTags( $originalText ) : $originalText;
+        $node .= ($toBeEscaped) ? Strings::escapeOnlyHTMLTags($originalText) : $originalText;
 
-        if ( !empty( $end ) ) {
-            $node .= ( $toBeEscaped ) ? Strings::escapeOnlyHTMLTags( $end ) : $end;
+        if (!empty($end)) {
+            $node .= ($toBeEscaped) ? Strings::escapeOnlyHTMLTags($end) : $end;
         }
 
         return $node;
@@ -267,16 +267,16 @@ 
      *
      * @return mixed
      */
-    public static function getAttributes( $content ) {
+    public static function getAttributes($content) {
         $pattern = '/(.*?)=("|\'|\\\")(.*?)("|\'|\\\"|\\\')/';
 
-        preg_match_all( $pattern, $content, $matches, PREG_OFFSET_CAPTURE );
+        preg_match_all($pattern, $content, $matches, PREG_OFFSET_CAPTURE);
 
         $attributes = [];
 
-        if ( isset( $matches[ 1 ] ) && count( $matches[ 1 ] ) > 0 ) {
-            foreach ( $matches[ 1 ] as $key => $match ) {
-                $attributes[ trim( $match[ 0 ] ) ] = $matches[ 3 ][ $key ][ 0 ];
+        if (isset($matches[1]) && count($matches[1]) > 0) {
+            foreach ($matches[1] as $key => $match) {
+                $attributes[trim($match[0])] = $matches[3][$key][0];
             }
         }
 
@@ -291,11 +291,11 @@ 
      *
      * @return array|mixed|string
      */
-    private static function getInnerHtml( $matches, $key, $toBeEscaped = false ) {
-        if ( isset( $matches[ 6 ][ $key ][ 0 ] ) ) {
-            $node = self::extractHtmlNode( $matches[ 6 ][ $key ][ 0 ], $toBeEscaped );
+    private static function getInnerHtml($matches, $key, $toBeEscaped = false) {
+        if (isset($matches[6][$key][0])) {
+            $node = self::extractHtmlNode($matches[6][$key][0], $toBeEscaped);
 
-            return ( !empty( $node ) ) ? $node : $matches[ 6 ][ $key ][ 0 ];
+            return (!empty($node)) ? $node : $matches[6][$key][0];
         }
 
         return null;

src/XliffUtils/DataRefReplacer.php

Spacing +164 added lines, -164 removed lines

@@ -16,7 +16,7 @@ 
      *
      * @param array $map
      */
-    public function __construct( array $map = null ) {
+    public function __construct(array $map = null) {
         $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/CheckXliffProprietaryPipeline.php

Spacing +8 added lines, -8 removed lines

@@ -20,7 +20,7 @@ 
      *
      * @param $tmp
      */
-    public function __construct( $tmp ) {
+    public function __construct($tmp) {
         $this->tmp   = $tmp;
         $this->steps = [];
     }
@@ -28,7 +28,7 @@ 
     /**
      * @param CheckInterface $step
      */
-    public function addCheck( CheckInterface $step ) {
+    public function addCheck(CheckInterface $step) {
         $this->steps[] = $step;
     }
 
@@ -39,13 +39,13 @@ 
         $fileType = [];
 
         /** @var CheckInterface $step */
-        foreach ( $this->steps as $step ) {
-            if ( null !== $step->check( $this->tmp ) ) {
-                $fileType = $step->check( $this->tmp );
+        foreach ($this->steps as $step) {
+            if (null !== $step->check($this->tmp)) {
+                $fileType = $step->check($this->tmp);
             }
         }
 
-        if ( !empty( $fileType ) && $this->isValid( $fileType ) ) {
+        if (!empty($fileType) && $this->isValid($fileType)) {
             return $fileType;
         }
 
@@ -62,7 +62,7 @@ 
      *
      * @return bool
      */
-    private function isValid( $fileType ) {
+    private function isValid($fileType) {
         $mandatoryKeys = [
                 'proprietary',
                 'proprietary_name',
@@ -70,6 +70,6 @@ 
                 'converter_version',
         ];
 
-        return array_keys( $fileType ) === $mandatoryKeys;
+        return array_keys($fileType) === $mandatoryKeys;
     }
 }

src/XliffUtils/XliffProprietaryDetect.php

Spacing +58 added lines, -58 removed lines

@@ -22,11 +22,11 @@ 
      *
      * @return array
      */
-    public static function getInfoFromXliffContent( $xliffContent ) {
+    public static function getInfoFromXliffContent($xliffContent) {
         self::reset();
-        $tmp = self::getFirst1024CharsFromXliff( $xliffContent, null );
+        $tmp = self::getFirst1024CharsFromXliff($xliffContent, null);
 
-        return self::getInfoFromTmp( $tmp );
+        return self::getInfoFromTmp($tmp);
     }
 
     /**
@@ -34,12 +34,12 @@ 
      *
      * @return array
      */
-    public static function getInfo( $fullPathToFile ) {
+    public static function getInfo($fullPathToFile) {
         self::reset();
-        $tmp                      = self::getFirst1024CharsFromXliff( null, $fullPathToFile );
-        self::$fileType[ 'info' ] = Files::pathInfo( $fullPathToFile );
+        $tmp                      = self::getFirst1024CharsFromXliff(null, $fullPathToFile);
+        self::$fileType['info'] = Files::pathInfo($fullPathToFile);
 
-        return self::getInfoFromTmp( $tmp );
+        return self::getInfoFromTmp($tmp);
     }
 
     /**
@@ -47,21 +47,21 @@ 
      *
      * @return array
      */
-    private static function getInfoFromTmp( $tmp ) {
+    private static function getInfoFromTmp($tmp) {
         try {
-            self::checkVersion( $tmp );
-        } catch ( Exception $ignore ) {
+            self::checkVersion($tmp);
+        } catch (Exception $ignore) {
             // do nothing
             // self::$fileType[ 'version' ] is left empty
         }
 
         // run CheckXliffProprietaryPipeline
-        $pipeline = self::runPipeline( $tmp );
+        $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;
     }
@@ -71,12 +71,12 @@ 
      *
      * @return array
      */
-    private static function runPipeline( $tmp ) {
-        $pipeline = new CheckXliffProprietaryPipeline( $tmp );
-        $pipeline->addCheck( new CheckSDL() );
-        $pipeline->addCheck( new CheckGlobalSight() );
-        $pipeline->addCheck( new CheckMateCATConverter() );
-        $pipeline->addCheck( new CheckXliffVersion2() );
+    private static function runPipeline($tmp) {
+        $pipeline = new CheckXliffProprietaryPipeline($tmp);
+        $pipeline->addCheck(new CheckSDL());
+        $pipeline->addCheck(new CheckGlobalSight());
+        $pipeline->addCheck(new CheckMateCATConverter());
+        $pipeline->addCheck(new CheckXliffVersion2());
 
         return $pipeline->run();
     }
@@ -99,29 +99,29 @@ 
      *
      * @return array|false
      */
-    private static function getFirst1024CharsFromXliff( $stringData = null, $fullPathToFile = null ) {
-        if ( !empty( $stringData ) && empty( $fullPathToFile ) ) {
+    private static function getFirst1024CharsFromXliff($stringData = null, $fullPathToFile = null) {
+        if (!empty($stringData) && empty($fullPathToFile)) {
             $pathInfo   = [];
-            $stringData = substr( $stringData, 0, 1024 );
-        } elseif ( empty( $stringData ) && !empty( $fullPathToFile ) ) {
-            $pathInfo = Files::pathInfo( $fullPathToFile );
+            $stringData = substr($stringData, 0, 1024);
+        } elseif (empty($stringData) && !empty($fullPathToFile)) {
+            $pathInfo = Files::pathInfo($fullPathToFile);
 
-            if ( is_file( $fullPathToFile ) ) {
-                $file_pointer = fopen( "$fullPathToFile", 'r' );
+            if (is_file($fullPathToFile)) {
+                $file_pointer = fopen("$fullPathToFile", 'r');
                 // Checking Requirements (By specs, I know that xliff version is in the first 1KB)
-                $stringData = fread( $file_pointer, 1024 );
-                fclose( $file_pointer );
+                $stringData = fread($file_pointer, 1024);
+                fclose($file_pointer);
             }
-        } elseif ( !empty( $stringData ) && !empty( $fullPathToFile ) ) {
-            $pathInfo = Files::pathInfo( $fullPathToFile );
+        } elseif (!empty($stringData) && !empty($fullPathToFile)) {
+            $pathInfo = Files::pathInfo($fullPathToFile);
         }
 
-        if ( !empty( $pathInfo ) && !Files::isXliff( $fullPathToFile ) ) {
+        if (!empty($pathInfo) && !Files::isXliff($fullPathToFile)) {
             return false;
         }
 
-        if ( !empty( $stringData ) ) {
-            return [ $stringData ];
+        if (!empty($stringData)) {
+            return [$stringData];
         }
 
         return false;
@@ -133,9 +133,9 @@ 
      * @throws NotSupportedVersionException
      * @throws NotValidFileException
      */
-    protected static function checkVersion( $tmp ) {
-        if ( isset( $tmp[ 0 ] ) ) {
-            self::$fileType[ 'version' ] = XliffVersionDetector::detect( $tmp[ 0 ] );
+    protected static function checkVersion($tmp) {
+        if (isset($tmp[0])) {
+            self::$fileType['version'] = XliffVersionDetector::detect($tmp[0]);
         }
     }
 
@@ -146,20 +146,20 @@ 
      * @throws NotSupportedVersionException
      * @throws NotValidFileException
      */
-    public static function getInfoByStringData( $stringData ) {
+    public static function getInfoByStringData($stringData) {
         self::reset();
 
-        $tmp                      = self::getFirst1024CharsFromXliff( $stringData );
-        self::$fileType[ 'info' ] = [];
-        self::checkVersion( $tmp );
+        $tmp                      = self::getFirst1024CharsFromXliff($stringData);
+        self::$fileType['info'] = [];
+        self::checkVersion($tmp);
 
         // run CheckXliffProprietaryPipeline
-        $pipeline = self::runPipeline( $tmp );
+        $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;
     }
@@ -171,21 +171,21 @@ 
      *
      * @return bool|int
      */
-    public static function fileMustBeConverted( $fullPath, $enforceOnXliff = false, $filterAddress = null ) {
+    public static function fileMustBeConverted($fullPath, $enforceOnXliff = false, $filterAddress = null) {
         $convert = true;
 
-        $fileType       = self::getInfo( $fullPath );
-        $memoryFileType = Files::getMemoryFileType( $fullPath );
+        $fileType       = self::getInfo($fullPath);
+        $memoryFileType = Files::getMemoryFileType($fullPath);
 
-        if ( Files::isXliff( $fullPath ) || $memoryFileType ) {
-            if ( !empty( $filterAddress ) ) {
+        if (Files::isXliff($fullPath) || $memoryFileType) {
+            if (!empty($filterAddress)) {
 
                 //conversion enforce
-                if ( !$enforceOnXliff ) {
+                if (!$enforceOnXliff) {
 
                     //if file is not proprietary AND Enforce is disabled
                     //we take it as is
-                    if ( !$fileType[ 'proprietary' ] || $memoryFileType ) {
+                    if (!$fileType['proprietary'] || $memoryFileType) {
                         $convert = false;
                         //ok don't convert a standard sdlxliff
                     }
@@ -193,16 +193,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'
-                            || $fileType[ 'proprietary_short_name' ] == 'trados'
-                            || $fileType[ 'proprietary_short_name' ] == 'xliff_v2'
+                            $fileType['proprietary_short_name'] == 'matecat_converter'
+                            || $fileType['proprietary_short_name'] == 'trados'
+                            || $fileType['proprietary_short_name'] == 'xliff_v2'
                             || $memoryFileType
                     ) {
                         $convert = false;
                         //ok don't convert a standard sdlxliff
                     }
                 }
-            } elseif ( $fileType[ 'proprietary' ] ) {
+            } elseif ($fileType['proprietary']) {
 
                 /**
                  * Application misconfiguration.

src/XliffUtils/CheckPipeline/CheckGlobalSight.php

Spacing +7 added lines, -7 removed lines

@@ -8,15 +8,15 @@
      *
      * @return array|null
      */
-    public function check( $tmp ) {
+    public function check($tmp) {
         $fileType = [];
 
-        if ( isset( $tmp[ 0 ] ) ) {
-            if ( stripos( $tmp[ 0 ], 'globalsight' ) !== false ) {
-                $fileType[ 'proprietary' ]            = true;
-                $fileType[ 'proprietary_name' ]       = 'GlobalSight Download File';
-                $fileType[ 'proprietary_short_name' ] = 'globalsight';
-                $fileType[ 'converter_version' ]      = 'legacy';
+        if (isset($tmp[0])) {
+            if (stripos($tmp[0], 'globalsight') !== false) {
+                $fileType['proprietary']            = true;
+                $fileType['proprietary_name']       = 'GlobalSight Download File';
+                $fileType['proprietary_short_name'] = 'globalsight';
+                $fileType['converter_version']      = 'legacy';
 
                 return $fileType;
             }

src/XliffUtils/CheckPipeline/CheckXliffVersion2.php

Spacing +12 added lines, -12 removed lines

@@ -8,22 +8,22 @@
      *
      * @return array|void|null
      */
-    public function check( $tmp ) {
+    public function check($tmp) {
         $fileType = [];
 
-        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 );
+        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);
 
-            if ( !empty( $versionMatches ) && !empty( $xmlnsMatches ) ) {
-                $version = $versionMatches[ 1 ];
-                $xmlns   = $xmlnsMatches[ 1 ];
+            if (!empty($versionMatches) && !empty($xmlnsMatches)) {
+                $version = $versionMatches[1];
+                $xmlns   = $xmlnsMatches[1];
 
-                if ( $version === $xmlns && $version >= 2 ) {
-                    $fileType[ 'proprietary' ]            = false;
-                    $fileType[ 'proprietary_name' ]       = 'Xliff v' . $version . ' File';
-                    $fileType[ 'proprietary_short_name' ] = 'xliff_v2';
-                    $fileType[ 'converter_version' ]      = '2.0';
+                if ($version === $xmlns && $version >= 2) {
+                    $fileType['proprietary']            = false;
+                    $fileType['proprietary_name']       = 'Xliff v' . $version . ' File';
+                    $fileType['proprietary_short_name'] = 'xliff_v2';
+                    $fileType['converter_version']      = '2.0';
 
                     return $fileType;
                 }

src/XliffUtils/CheckPipeline/CheckSDL.php

Spacing +7 added lines, -7 removed lines

@@ -3,16 +3,16 @@
 namespace Matecat\XliffParser\XliffUtils\CheckPipeline;
 
 class CheckSDL implements CheckInterface {
-    public function check( $tmp ) {
+    public function check($tmp) {
         $fileType = [];
 
-        if ( isset( $tmp[ 0 ] ) ) {
-            if ( stripos( $tmp[ 0 ], 'sdl:version' ) !== false ) {
+        if (isset($tmp[0])) {
+            if (stripos($tmp[0], 'sdl:version') !== false) {
                 //little trick, we consider not proprietary Sdlxliff files because we can handle them
-                $fileType[ 'proprietary' ]            = false;
-                $fileType[ 'proprietary_name' ]       = 'SDL Studio ';
-                $fileType[ 'proprietary_short_name' ] = 'trados';
-                $fileType[ 'converter_version' ]      = 'legacy';
+                $fileType['proprietary']            = false;
+                $fileType['proprietary_name']       = 'SDL Studio ';
+                $fileType['proprietary_short_name'] = 'trados';
+                $fileType['converter_version']      = 'legacy';
 
                 return $fileType;
             }

src/XliffUtils/CheckPipeline/CheckMateCATConverter.php

Spacing +10 added lines, -10 removed lines

@@ -8,20 +8,20 @@
      *
      * @return array|null
      */
-    public function check( $tmp ) {
+    public function check($tmp) {
         $fileType = [];
 
-        if ( isset( $tmp[ 0 ] ) ) {
-            preg_match( '#tool-id\s*=\s*"matecat-converter(\s+([^"]+))?"#i', $tmp[ 0 ], $matches );
-            if ( !empty( $matches ) ) {
-                $fileType[ 'proprietary' ]            = false;
-                $fileType[ 'proprietary_name' ]       = 'MateCAT Converter';
-                $fileType[ 'proprietary_short_name' ] = 'matecat_converter';
-                if ( isset( $matches[ 2 ] ) ) {
-                    $fileType[ 'converter_version' ] = $matches[ 2 ];
+        if (isset($tmp[0])) {
+            preg_match('#tool-id\s*=\s*"matecat-converter(\s+([^"]+))?"#i', $tmp[0], $matches);
+            if (!empty($matches)) {
+                $fileType['proprietary']            = false;
+                $fileType['proprietary_name']       = 'MateCAT Converter';
+                $fileType['proprietary_short_name'] = 'matecat_converter';
+                if (isset($matches[2])) {
+                    $fileType['converter_version'] = $matches[2];
                 } else {
                     // First converter release didn't specify version
-                    $fileType[ 'converter_version' ] = '1.0';
+                    $fileType['converter_version'] = '1.0';
                 }
 
                 return $fileType;

src/XliffUtils/CheckPipeline/CheckInterface.php

Spacing +1 added lines, -1 removed lines

@@ -8,5 +8,5 @@
      *
      * @return array|null
      */
-    public function check( $tmp );
+    public function check($tmp);
 }