matecat / xliff-parser

src/XliffParser.php

Spacing +58 added lines, -58 removed lines

@@ -30,7 +30,7 @@ 
      *
      * @param ?LoggerInterface $logger
      */
-    public function __construct( ?LoggerInterface $logger = null ) {
+    public function __construct(?LoggerInterface $logger = null) {
         $this->logger = $logger;
     }
 
@@ -45,11 +45,11 @@ 
      * @param bool                                $setSourceInTarget
      * @param XliffReplacerCallbackInterface|null $callback
      */
-    public function replaceTranslation( string $originalXliffPath, array $data, array $transUnits, string $targetLang, string $outputFile, bool $setSourceInTarget = false, ?XliffReplacerCallbackInterface $callback = null ) {
+    public function replaceTranslation(string $originalXliffPath, array $data, array $transUnits, string $targetLang, string $outputFile, bool $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
         }
     }
@@ -67,26 +67,26 @@ 
      * @throws InvalidXmlException
      * @throws XmlParsingException
      */
-    public function xliffToArray( string $xliffContent, ?bool $collapseEmptyTags = false ): array {
+    public function xliffToArray(string $xliffContent, ?bool $collapseEmptyTags = false): array {
         $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 = $info[ 'proprietary_short_name' ] ?? null;
-        $parser           = XliffParserFactory::getInstance( $xliffVersion, $xliffProprietary, $this->logger );
+        $xliffProprietary = $info['proprietary_short_name'] ?? null;
+        $parser           = XliffParserFactory::getInstance($xliffVersion, $xliffProprietary, $this->logger);
 
         $dom = XmlDomLoader::load(
                 $xliffContent,
@@ -97,7 +97,7 @@ 
                 )
         );
 
-        return $parser->parse( $dom, $xliff );
+        return $parser->parse($dom, $xliff);
     }
 
     /**
@@ -109,12 +109,12 @@ 
      *
      * @return string
      */
-    private static function forceUft8Encoding( $xliffContent, &$xliff ): string {
-        $enc = mb_detect_encoding( $xliffContent );
+    private static function forceUft8Encoding($xliffContent, &$xliff): string {
+        $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 : "";
         }
 
@@ -131,28 +131,28 @@ 
      *
      * @return mixed|string
      */
-    private static function removeInternalFileTagFromContent( $xliffContent, &$xliff ) {
+    private static function removeInternalFileTagFromContent($xliffContent, &$xliff) {
         $index       = 1;
-        $a           = Strings::preg_split( '|<internal-file[\s>]|i', $xliffContent );
-        $tagMatches  = count( $a );
+        $a           = Strings::preg_split('|<internal-file[\s>]|i', $xliffContent);
+        $tagMatches  = count($a);
 
         // no match, return original string
-        if ( $tagMatches === 1 ) {
-            return $a[ 0 ];
+        if ($tagMatches === 1) {
+            return $a[0];
         }
 
-        $b                                           = Strings::preg_split( '|</internal-file>|i', $a[ 1 ] );
-        $strippedContent                             = $a[ 0 ] . $b[ 1 ];
-        $xliff[ 'files' ][ $index ][ 'reference' ][] = self::extractBase64( $b[ 0 ] );
+        $b                                           = Strings::preg_split('|</internal-file>|i', $a[1]);
+        $strippedContent                             = $a[0] . $b[1];
+        $xliff['files'][$index]['reference'][] = self::extractBase64($b[0]);
         $index++;
 
         // Sometimes, sdlxliff files can contain more than 2 <internal-file> nodes.
         // In this case loop and extract any other extra <internal-file> node
-        for($i=2; $i < $tagMatches; $i++){
-            if ( isset( $a[ $i ] ) ) {
-                $c                                           = Strings::preg_split( '|</internal-file[\s>]|i', $a[ $i ] );
-                $strippedContent                             .= $c[ 1 ];
-                $xliff[ 'files' ][ $index ][ 'reference' ][] = self::extractBase64( $c[ 0 ] );
+        for ($i = 2; $i < $tagMatches; $i++) {
+            if (isset($a[$i])) {
+                $c                                           = Strings::preg_split('|</internal-file[\s>]|i', $a[$i]);
+                $strippedContent .= $c[1];
+                $xliff['files'][$index]['reference'][] = self::extractBase64($c[0]);
             }
         }
 
@@ -164,10 +164,10 @@ 
      *
      * @return array
      */
-    private static function extractBase64( $base64 ): array {
+    private static function extractBase64($base64): array {
         return [
                 'form-type' => 'base64',
-                'base64'    => trim( str_replace( 'form="base64">', '', $base64 ) ),
+                'base64'    => trim(str_replace('form="base64">', '', $base64)),
         ];
     }
 
@@ -187,9 +187,9 @@ 
      *
      * @return string
      */
-    private static function escapeDataInOriginalMap( string $xliffContent ): string {
-        $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(string $xliffContent): string {
+        $xliffContent = preg_replace_callback('|<data(.*?)>(.*?)</data>|iU', [XliffParser::class, 'replaceSpace'], $xliffContent);
+        $xliffContent = preg_replace_callback('|<data(.*?)>(.*?)</data>|iU', [XliffParser::class, 'replaceXliffTags'], $xliffContent);
 
         return $xliffContent;
     }
@@ -208,15 +208,15 @@ 
      *
      * @return string
      */
-    private static function insertPlaceholderInEmptyTags( $xliffContent ): string {
-        preg_match_all( '|<([a-zA-Z0-9._-]+)[^>]*></\1>|m', $xliffContent, $emptyTagMatches );
+    private static function insertPlaceholderInEmptyTags($xliffContent): string {
+        preg_match_all('|<([a-zA-Z0-9._-]+)[^>]*></\1>|m', $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);
             }
         }
 
@@ -230,12 +230,12 @@ 
      *
      * @return string
      */
-    private static function replaceSpace( array $matches ): string {
-        $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(array $matches): string {
+        $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>';
     }
 
     /**
@@ -243,15 +243,15 @@ 
      *
      * @return string
      */
-    private static function replaceXliffTags( array $matches ): string {
+    private static function replaceXliffTags(array $matches): string {
         $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>';
     }
 }