matecat / xliff-parser

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);
 }

src/XliffUtils/XliffVersionDetector.php

Spacing +12 added lines, -12 removed lines

@@ -9,12 +9,12 @@ 
     /**
      * @var array
      */
-    private static $versions_1 = [ '1.0', '1.1', '1.2' ];
+    private static $versions_1 = ['1.0', '1.1', '1.2'];
 
     /**
      * @var array
      */
-    private static $versions_2 = [ '2.0', '2.1' ];
+    private static $versions_2 = ['2.0', '2.1'];
 
     /**
      * @param string $xliffContent
@@ -23,16 +23,16 @@ 
      * @throws NotSupportedVersionException
      * @throws NotValidFileException
      */
-    public static function detect( $xliffContent ) {
-        preg_match( '|<xliff.*?\sversion\s?=\s?["\'](.*?)["\']|si', substr( $xliffContent, 0, 1000 ), $versionMatches );
+    public static function detect($xliffContent) {
+        preg_match('|<xliff.*?\sversion\s?=\s?["\'](.*?)["\']|si', substr($xliffContent, 0, 1000), $versionMatches);
 
-        if ( empty( $versionMatches ) ) {
-            throw new NotValidFileException( 'This is not a valid xliff file' );
+        if (empty($versionMatches)) {
+            throw new NotValidFileException('This is not a valid xliff file');
         }
 
-        $version = $versionMatches[ 1 ];
+        $version = $versionMatches[1];
 
-        return self::resolveVersion( $version );
+        return self::resolveVersion($version);
     }
 
     /**
@@ -41,15 +41,15 @@ 
      * @return int
      * @throws NotSupportedVersionException
      */
-    private static function resolveVersion( $version ) {
-        if ( in_array( $version, self::$versions_1 ) ) {
+    private static function resolveVersion($version) {
+        if (in_array($version, self::$versions_1)) {
             return 1;
         }
 
-        if ( in_array( $version, self::$versions_2 ) ) {
+        if (in_array($version, self::$versions_2)) {
             return 2;
         }
 
-        throw new NotSupportedVersionException( 'Not supported version' );
+        throw new NotSupportedVersionException('Not supported version');
     }
 }

src/XliffReplacer/XliffReplacerFactory.php

Spacing +5 added lines, -5 removed lines

@@ -18,13 +18,13 @@
      *
      * @return SdlXliffSAXTranslationReplacer|XliffSAXTranslationReplacer
      */
-    public static function getInstance( $originalXliffPath, &$data, &$transUnits, $targetLang, $outputFilePath, $setSourceInTarget, LoggerInterface $logger = null, XliffReplacerCallbackInterface $callback = null ) {
-        $info = XliffProprietaryDetect::getInfo( $originalXliffPath );
+    public static function getInstance($originalXliffPath, &$data, &$transUnits, $targetLang, $outputFilePath, $setSourceInTarget, LoggerInterface $logger = null, XliffReplacerCallbackInterface $callback = null) {
+        $info = XliffProprietaryDetect::getInfo($originalXliffPath);
 
-        if ( $info[ 'proprietary_short_name' ] !== 'trados' ) {
-            return new XliffSAXTranslationReplacer( $originalXliffPath, $info[ 'version' ], $data, $transUnits, $targetLang, $outputFilePath, $setSourceInTarget, $logger, $callback );
+        if ($info['proprietary_short_name'] !== 'trados') {
+            return new XliffSAXTranslationReplacer($originalXliffPath, $info['version'], $data, $transUnits, $targetLang, $outputFilePath, $setSourceInTarget, $logger, $callback);
         }
 
-        return new SdlXliffSAXTranslationReplacer( $originalXliffPath, $info[ 'version' ], $data, $transUnits, $targetLang, $outputFilePath, $setSourceInTarget, $logger, $callback );
+        return new SdlXliffSAXTranslationReplacer($originalXliffPath, $info['version'], $data, $transUnits, $targetLang, $outputFilePath, $setSourceInTarget, $logger, $callback);
     }
 }

src/XliffParser.php

Spacing +57 added lines, -57 removed lines

@@ -24,7 +24,7 @@ 
      *
      * @param LoggerInterface $logger
      */
-    public function __construct( LoggerInterface $logger = null ) {
+    public function __construct(LoggerInterface $logger = null) {
         $this->logger = $logger;
     }
 
@@ -39,11 +39,11 @@ 
      * @param bool                                $setSourceInTarget
      * @param XliffReplacerCallbackInterface|null $callback
      */
-    public function replaceTranslation( $originalXliffPath, &$data, &$transUnits, $targetLang, $outputFile, $setSourceInTarget = false, XliffReplacerCallbackInterface $callback = null ) {
+    public function replaceTranslation($originalXliffPath, &$data, &$transUnits, $targetLang, $outputFile, $setSourceInTarget = false, XliffReplacerCallbackInterface $callback = null) {
         try {
-            $parser = XliffReplacerFactory::getInstance( $originalXliffPath, $data, $transUnits, $targetLang, $outputFile, $setSourceInTarget, $this->logger, $callback );
+            $parser = XliffReplacerFactory::getInstance($originalXliffPath, $data, $transUnits, $targetLang, $outputFile, $setSourceInTarget, $this->logger, $callback);
             $parser->replaceTranslation();
-        } catch ( \Exception $exception ) {
+        } catch (\Exception $exception) {
             // do nothing
         }
     }
@@ -61,29 +61,29 @@ 
      * @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 );
+        $xliffContent = self::forceUft8Encoding($xliffContent, $xliff);
+        $xliffVersion = XliffVersionDetector::detect($xliffContent);
+        $info         = XliffProprietaryDetect::getInfoFromXliffContent($xliffContent);
 
-        if ( $xliffVersion === 1 ) {
-            $xliffContent = self::removeInternalFileTagFromContent( $xliffContent, $xliff );
+        if ($xliffVersion === 1) {
+            $xliffContent = self::removeInternalFileTagFromContent($xliffContent, $xliff);
         }
 
-        if ( $xliffVersion === 2 ) {
-            $xliffContent = self::escapeDataInOriginalMap( $xliffContent );
+        if ($xliffVersion === 2) {
+            $xliffContent = self::escapeDataInOriginalMap($xliffContent);
         }
 
-        if ( $collapseEmptyTags === false ) {
-            $xliffContent = self::insertPlaceholderInEmptyTags( $xliffContent );
+        if ($collapseEmptyTags === false) {
+            $xliffContent = self::insertPlaceholderInEmptyTags($xliffContent);
         }
 
-        $xliffProprietary = ( isset( $info[ 'proprietary_short_name' ] ) && null !== $info[ 'proprietary_short_name' ] ) ? $info[ 'proprietary_short_name' ] : null;
-        $parser           = XliffParserFactory::getInstance( $xliffVersion, $xliffProprietary, $this->logger );
-        $dom              = XmlParser::parse( $xliffContent );
+        $xliffProprietary = (isset($info['proprietary_short_name']) && null !== $info['proprietary_short_name']) ? $info['proprietary_short_name'] : null;
+        $parser           = XliffParserFactory::getInstance($xliffVersion, $xliffProprietary, $this->logger);
+        $dom              = XmlParser::parse($xliffContent);
 
-        return $parser->parse( $dom, $xliff );
+        return $parser->parse($dom, $xliff);
     }
 
     /**
@@ -95,12 +95,12 @@ 
      *
      * @return string
      */
-    private static function forceUft8Encoding( $xliffContent, &$xliff ) {
-        $enc = mb_detect_encoding( $xliffContent );
+    private static function forceUft8Encoding($xliffContent, &$xliff) {
+        $enc = mb_detect_encoding($xliffContent);
 
-        if ( $enc !== 'UTF-8' ) {
-            $xliff[ 'parser-warnings' ][] = "Input identified as $enc ans converted UTF-8. May not be a problem if the content is English only";
-            $s                            = iconv( $enc, 'UTF-8', $xliffContent );
+        if ($enc !== 'UTF-8') {
+            $xliff['parser-warnings'][] = "Input identified as $enc ans converted UTF-8. May not be a problem if the content is English only";
+            $s                            = iconv($enc, 'UTF-8', $xliffContent);
             $xliffContent                 = $s !== false ? $s : "";
         }
 
@@ -117,24 +117,24 @@ 
      *
      * @return mixed|string
      */
-    private static function removeInternalFileTagFromContent( $xliffContent, &$xliff ) {
+    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 ) {
-            return $a[ 0 ];
+        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 ] );
-            $strippedContent                             .= $c[ 1 ];
-            $xliff[ 'files' ][ $index ][ 'reference' ][] = self::extractBase64( $c[ 0 ] );
+        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]);
         }
 
         return $strippedContent;
@@ -145,10 +145,10 @@ 
      *
      * @return array
      */
-    private static function extractBase64( $base64 ) {
+    private static function extractBase64($base64) {
         return [
                 'form-type' => 'base64',
-                'base64'    => trim( str_replace( 'form="base64">', '', $base64 ) ),
+                'base64'    => trim(str_replace('form="base64">', '', $base64)),
         ];
     }
 
@@ -168,9 +168,9 @@ 
      *
      * @return string
      */
-    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 );
+    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);
 
         return $xliffContent;
     }
@@ -189,15 +189,15 @@ 
      *
      * @return string
      */
-    private static function insertPlaceholderInEmptyTags( $xliffContent ) {
-        preg_match_all( '/<([a-zA-Z0-9._-]+)[^>]*><\/\1>/sm', $xliffContent, $emptyTagMatches );
+    private static function insertPlaceholderInEmptyTags($xliffContent) {
+        preg_match_all('/<([a-zA-Z0-9._-]+)[^>]*><\/\1>/sm', $xliffContent, $emptyTagMatches);
 
-        if ( !empty( $emptyTagMatches[ 0 ] ) ) {
-            foreach ( $emptyTagMatches[ 0 ] as $index => $emptyTagMatch ) {
-                $matchedTag   = $emptyTagMatches[ 1 ][ $index ];
+        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 );
-                $xliffContent = str_replace( $emptyTagMatch, $replacedTag, $xliffContent );
+                $replacedTag  = str_replace('</' . $matchedTag . '>', $subst, $emptyTagMatch);
+                $xliffContent = str_replace($emptyTagMatch, $replacedTag, $xliffContent);
             }
         }
 
@@ -211,12 +211,12 @@ 
      *
      * @return string
      */
-    private static function replaceSpace( $matches ) {
-        $content = str_replace( ' ', Placeholder::WHITE_SPACE_PLACEHOLDER, $matches[ 2 ] );
-        $content = str_replace( '\n', Placeholder::NEW_LINE_PLACEHOLDER, $content );
-        $content = str_replace( '\t', Placeholder::TAB_PLACEHOLDER, $content );
+    private static function replaceSpace($matches) {
+        $content = str_replace(' ', Placeholder::WHITE_SPACE_PLACEHOLDER, $matches[2]);
+        $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>';
     }
 
     /**
@@ -224,15 +224,15 @@ 
      *
      * @return string
      */
-    private static function replaceXliffTags( $matches ) {
+    private static function replaceXliffTags($matches) {
         $xliffTags = XliffTags::$tags;
-        $content   = $matches[ 2 ];
+        $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/Utils/FlatData.php

Spacing +7 added lines, -7 removed lines

@@ -8,14 +8,14 @@
  * https://stackoverflow.com/questions/11427398/how-to-implode-array-with-key-and-value-without-foreach-in-php
  */
 class FlatData {
-    public static function flatArray( array $input = [], $separator_elements = ', ', $separator = ': ' ) {
-        return implode( $separator_elements, array_map(
-                function ( $v, $k, $s ) {
-                    return sprintf( "%s{$s}\"%s\"", $k, $v );
+    public static function flatArray(array $input = [], $separator_elements = ', ', $separator = ': ') {
+        return implode($separator_elements, array_map(
+                function($v, $k, $s) {
+                    return sprintf("%s{$s}\"%s\"", $k, $v);
                 },
                 $input,
-                array_keys( $input ),
-                array_fill( 0, count( $input ), $separator )
-        ) );
+                array_keys($input),
+                array_fill(0, count($input), $separator)
+        ));
     }
 }

src/XliffParser/XliffParserFactory.php

Spacing +2 added lines, -2 removed lines

@@ -12,10 +12,10 @@
      *
      * @return AbstractXliffParser
      */
-    public static function getInstance( $xliffVersion, $xliffProprietary = null, LoggerInterface $logger = null ) {
+    public static function getInstance($xliffVersion, $xliffProprietary = null, LoggerInterface $logger = null) {
         $parserClass = 'Matecat\\XliffParser\\XliffParser\\XliffParserV' . $xliffVersion;
 
         /** @var AbstractXliffParser $parser */
-        return new $parserClass( $xliffVersion, $xliffProprietary, $logger );
+        return new $parserClass($xliffVersion, $xliffProprietary, $logger);
     }
 }