matecat / xliff-parser

src/XliffUtils/CheckPipeline/CheckInterface.php

Spacing +1 added lines, -1 removed lines

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

src/XliffUtils/XliffVersionDetector.php

Spacing +12 added lines, -12 removed lines

@@ -9,12 +9,12 @@ 
     /**
      * @var array
      */
-    private static array $versions_1 = [ '1.0', '1.1', '1.2' ];
+    private static array $versions_1 = ['1.0', '1.1', '1.2'];
 
     /**
      * @var array
      */
-    private static array $versions_2 = [ '2.0', '2.1' ];
+    private static array $versions_2 = ['2.0', '2.1'];
 
     /**
      * @param string $xliffContent
@@ -23,16 +23,16 @@ 
      * @throws NotSupportedVersionException
      * @throws NotValidFileException
      */
-    public static function detect( string $xliffContent ): int {
-        preg_match( '|<xliff.*?\sversion\s?=\s?["\'](.*?)["\']|si', substr( $xliffContent, 0, 1000 ), $versionMatches );
+    public static function detect(string $xliffContent): int {
+        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( string $version ): int {
-        if ( in_array( $version, self::$versions_1 ) ) {
+    private static function resolveVersion(string $version): int {
+        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 +6 added lines, -6 removed lines

@@ -28,14 +28,14 @@
             ?LoggerInterface                $logger = null,
             ?XliffReplacerCallbackInterface $callback = null
     ): AbstractXliffReplacer {
-        $info = XliffProprietaryDetect::getInfo( $originalXliffPath );
+        $info = XliffProprietaryDetect::getInfo($originalXliffPath);
 
-        if ( $info[ 'version' ] == 1 && $info[ 'proprietary_short_name' ] !== 'trados' ) {
-            return new Xliff12( $originalXliffPath, $info[ 'version' ], $data, $transUnits, $targetLang, $outputFilePath, $setSourceInTarget, $logger, $callback );
-        } elseif ( $info[ 'version' ] == 2 ) {
-            return new Xliff20( $originalXliffPath, $info[ 'version' ], $data, $transUnits, $targetLang, $outputFilePath, $setSourceInTarget, $logger, $callback );
+        if ($info['version'] == 1 && $info['proprietary_short_name'] !== 'trados') {
+            return new Xliff12($originalXliffPath, $info['version'], $data, $transUnits, $targetLang, $outputFilePath, $setSourceInTarget, $logger, $callback);
+        } elseif ($info['version'] == 2) {
+            return new Xliff20($originalXliffPath, $info['version'], $data, $transUnits, $targetLang, $outputFilePath, $setSourceInTarget, $logger, $callback);
         }
 
-        return new XliffSdl( $originalXliffPath, $info[ 'version' ], $data, $transUnits, $targetLang, $outputFilePath, $setSourceInTarget, $logger, $callback );
+        return new XliffSdl($originalXliffPath, $info['version'], $data, $transUnits, $targetLang, $outputFilePath, $setSourceInTarget, $logger, $callback);
     }
 }

src/XliffReplacer/XliffReplacerCallbackInterface.php

Spacing +1 added lines, -1 removed lines

@@ -12,5 +12,5 @@
      *
      * @return bool
      */
-    public function thereAreErrors( int $segmentId, string $segment, string $translation, ?array $dataRefMap = [], ?string $error = null ): bool;
+    public function thereAreErrors(int $segmentId, string $segment, string $translation, ?array $dataRefMap = [], ?string $error = null): bool;
 }

src/XliffUtils/XliffProprietaryDetect.php

Spacing +55 added lines, -55 removed lines

@@ -22,11 +22,11 @@ 
      *
      * @return array
      */
-    public static function getInfoFromXliffContent( string $xliffContent ): array {
+    public static function getInfoFromXliffContent(string $xliffContent): array {
         self::reset();
-        $tmp = self::getFirst1024CharsFromXliff( $xliffContent );
+        $tmp = self::getFirst1024CharsFromXliff($xliffContent);
 
-        return self::getInfoFromTmp( $tmp );
+        return self::getInfoFromTmp($tmp);
     }
 
     /**
@@ -34,12 +34,12 @@ 
      *
      * @return array
      */
-    public static function getInfo( string $fullPathToFile ): array {
+    public static function getInfo(string $fullPathToFile): array {
         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( array $tmp ): array {
+    private static function getInfoFromTmp(array $tmp): array {
         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( ?array $tmp = [] ): array {
-        $pipeline = new CheckXliffProprietaryPipeline( $tmp );
-        $pipeline->addCheck( new CheckSDL() );
-        $pipeline->addCheck( new CheckGlobalSight() );
-        $pipeline->addCheck( new CheckMateCATConverter() );
-        $pipeline->addCheck( new CheckXliffVersion2() );
+    private static function runPipeline(?array $tmp = []): array {
+        $pipeline = new CheckXliffProprietaryPipeline($tmp);
+        $pipeline->addCheck(new CheckSDL());
+        $pipeline->addCheck(new CheckGlobalSight());
+        $pipeline->addCheck(new CheckMateCATConverter());
+        $pipeline->addCheck(new CheckXliffVersion2());
 
         return $pipeline->run();
     }
@@ -98,9 +98,9 @@ 
      *
      * @return string
      */
-    private static function getFirst1024CharsFromString( ?string $stringData ): string {
-        if ( !empty( $stringData ) ) {
-            return substr( $stringData, 0, 1024 );
+    private static function getFirst1024CharsFromString(?string $stringData): string {
+        if (!empty($stringData)) {
+            return substr($stringData, 0, 1024);
         }
 
         return '';
@@ -111,13 +111,13 @@ 
      *
      * @return string
      */
-    private static function getFirst1024CharsFromFile( ?string $fullPathToFile ): string {
+    private static function getFirst1024CharsFromFile(?string $fullPathToFile): string {
         $stringData = '';
-        if ( !empty( $fullPathToFile ) && is_file( $fullPathToFile ) ) {
-            $file_pointer = fopen( "$fullPathToFile", 'r' );
+        if (!empty($fullPathToFile) && 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);
         }
 
         return $stringData;
@@ -130,13 +130,13 @@ 
      *
      * @return string[]
      */
-    private static function getFirst1024CharsFromXliff( ?string $stringData = null, string $fullPathToFile = null ): ?array {
-        $stringData = static::getFirst1024CharsFromString( $stringData );
-        if ( empty( $stringData ) ) {
-            $stringData = static::getFirst1024CharsFromFile( $fullPathToFile );
+    private static function getFirst1024CharsFromXliff(?string $stringData = null, string $fullPathToFile = null): ?array {
+        $stringData = static::getFirst1024CharsFromString($stringData);
+        if (empty($stringData)) {
+            $stringData = static::getFirst1024CharsFromFile($fullPathToFile);
         }
 
-        return !empty( $stringData ) ? [ $stringData ] : [];
+        return !empty($stringData) ? [$stringData] : [];
     }
 
     /**
@@ -145,9 +145,9 @@ 
      * @throws NotSupportedVersionException
      * @throws NotValidFileException
      */
-    protected static function checkVersion( array $tmp ) {
-        if ( isset( $tmp[ 0 ] ) ) {
-            self::$fileType[ 'version' ] = XliffVersionDetector::detect( $tmp[ 0 ] );
+    protected static function checkVersion(array $tmp) {
+        if (isset($tmp[0])) {
+            self::$fileType['version'] = XliffVersionDetector::detect($tmp[0]);
         }
     }
 
@@ -158,20 +158,20 @@ 
      * @throws NotSupportedVersionException
      * @throws NotValidFileException
      */
-    public static function getInfoByStringData( string $stringData ): array {
+    public static function getInfoByStringData(string $stringData): array {
         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;
     }
@@ -183,33 +183,33 @@ 
      *
      * @return bool|int
      */
-    public static function fileMustBeConverted( string $fullPath, bool $enforceOnXliff = false, ?string $filterAddress = null ) {
+    public static function fileMustBeConverted(string $fullPath, bool $enforceOnXliff = false, ?string $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
                     }
                 } else {
                     //if conversion enforce is active
                     //we force all xliff files but not files produced by SDL Studio because we can handle them
-                    if ( in_array( $fileType[ 'proprietary_short_name' ], [ 'matecat_converter', 'trados', 'xliff_v2' ] ) || $memoryFileType ) {
+                    if (in_array($fileType['proprietary_short_name'], ['matecat_converter', 'trados', 'xliff_v2']) || $memoryFileType) {
                         $convert = false;
                         //ok don't convert a standard sdlxliff
                     }
                 }
-            } elseif ( $fileType[ 'proprietary' ] ) {
+            } elseif ($fileType['proprietary']) {
 
                 /**
                  * Application misconfiguration.

src/Utils/Files.php

Spacing +28 added lines, -28 removed lines

@@ -18,15 +18,15 @@ 
      *
      * @return array|mixed
      */
-    public static function pathInfo( string $path, ?int $options = 15 ) {
-        $rawPath = explode( DIRECTORY_SEPARATOR, $path );
+    public static function pathInfo(string $path, ?int $options = 15) {
+        $rawPath = explode(DIRECTORY_SEPARATOR, $path);
 
-        $basename = array_pop( $rawPath );
-        $dirname  = implode( DIRECTORY_SEPARATOR, $rawPath );
+        $basename = array_pop($rawPath);
+        $dirname  = implode(DIRECTORY_SEPARATOR, $rawPath);
 
-        $explodedFileName = explode( ".", $basename );
-        $extension        = strtolower( array_pop( $explodedFileName ) );
-        $filename         = implode( ".", $explodedFileName );
+        $explodedFileName = explode(".", $basename);
+        $extension        = strtolower(array_pop($explodedFileName));
+        $filename         = implode(".", $explodedFileName);
 
         $returnArray = [];
 
@@ -39,17 +39,17 @@ 
 
         // foreach flag, add in $return_array the corresponding field,
         // obtained by variable name correspondence
-        foreach ( $flagMap as $field => $i ) {
+        foreach ($flagMap as $field => $i) {
             //binary AND
-            if ( ( $options & $i ) > 0 ) {
+            if (($options & $i) > 0) {
                 //variable substitution: $field can be one between 'dirname', 'basename', 'extension', 'filename'
                 // $$field gets the value of the variable named $field
-                $returnArray[ $field ] = $$field;
+                $returnArray[$field] = $$field;
             }
         }
 
-        if ( count( $returnArray ) == 1 ) {
-            $returnArray = array_pop( $returnArray );
+        if (count($returnArray) == 1) {
+            $returnArray = array_pop($returnArray);
         }
 
         return $returnArray;
@@ -60,14 +60,14 @@ 
      *
      * @return ?string
      */
-    public static function getExtension( $path ): ?string {
-        $pathInfo = self::pathInfo( $path );
+    public static function getExtension($path): ?string {
+        $pathInfo = self::pathInfo($path);
 
-        if ( empty( $pathInfo ) ) {
+        if (empty($pathInfo)) {
             return null;
         }
 
-        return strtolower( $pathInfo[ 'extension' ] );
+        return strtolower($pathInfo['extension']);
     }
 
     /**
@@ -75,14 +75,14 @@ 
      *
      * @return bool
      */
-    public static function isXliff( ?string $path ): bool {
-        $extension = self::getExtension( $path );
+    public static function isXliff(?string $path): bool {
+        $extension = self::getExtension($path);
 
-        if ( !$extension ) {
+        if (!$extension) {
             return false;
         }
 
-        switch ( $extension ) {
+        switch ($extension) {
             case 'xliff':
             case 'sdlxliff':
             case 'tmx':
@@ -98,14 +98,14 @@ 
      *
      * @return bool|string
      */
-    public static function getMemoryFileType( string $path ) {
-        $pathInfo = self::pathInfo( $path );
+    public static function getMemoryFileType(string $path) {
+        $pathInfo = self::pathInfo($path);
 
-        if ( empty( $pathInfo ) ) {
+        if (empty($pathInfo)) {
             return false;
         }
 
-        switch ( strtolower( $pathInfo[ 'extension' ] ) ) {
+        switch (strtolower($pathInfo['extension'])) {
             case 'tmx':
                 return 'tmx';
             default:
@@ -118,8 +118,8 @@ 
      *
      * @return bool
      */
-    public static function isTMXFile( $path ): bool {
-        return self::getMemoryFileType( $path ) === 'tmx';
+    public static function isTMXFile($path): bool {
+        return self::getMemoryFileType($path) === 'tmx';
     }
 
     /**
@@ -127,8 +127,8 @@ 
      *
      * @return bool
      */
-    public static function isGlossaryFile( $path ): bool {
-        return self::getMemoryFileType( $path ) === 'glossary'; // return false
+    public static function isGlossaryFile($path): bool {
+        return self::getMemoryFileType($path) === 'glossary'; // return false
     }
 
 }

src/XliffReplacer/StatusToStateAttribute.php

Spacing +12 added lines, -12 removed lines

@@ -26,7 +26,7 @@ 
             ?string $lastMrkState = null
     ): array {
 
-        $status = empty( $status ) ? TranslationStatus::STATUS_APPROVED2 : $status;
+        $status = empty($status) ? TranslationStatus::STATUS_APPROVED2 : $status;
 
         $stateLevelsMap = [
                 TranslationStatus::STATUS_APPROVED2  => 100,
@@ -37,26 +37,26 @@ 
                 TranslationStatus::STATUS_NEW        => 50
         ];
 
-        $orderedValues = array_flip( $stateLevelsMap );
+        $orderedValues = array_flip($stateLevelsMap);
 
         // Define state mappings for different statuses
         $stateMap = [
-                TranslationStatus::STATUS_APPROVED2  => [ "state=\"final\"", TranslationStatus::STATUS_APPROVED2 ],
+                TranslationStatus::STATUS_APPROVED2  => ["state=\"final\"", TranslationStatus::STATUS_APPROVED2],
                 TranslationStatus::STATUS_APPROVED   => [
-                        ( $xliffVersion === 2 ) ? "state=\"reviewed\"" : "state=\"signed-off\"",
+                        ($xliffVersion === 2) ? "state=\"reviewed\"" : "state=\"signed-off\"",
                         TranslationStatus::STATUS_APPROVED
                 ],
-                TranslationStatus::STATUS_TRANSLATED => [ "state=\"translated\"", TranslationStatus::STATUS_TRANSLATED ],
+                TranslationStatus::STATUS_TRANSLATED => ["state=\"translated\"", TranslationStatus::STATUS_TRANSLATED],
                 TranslationStatus::STATUS_REJECTED   => [
-                        ( $xliffVersion === 2 ) ? "state=\"initial\"" : "state=\"needs-review-translation\"",
+                        ($xliffVersion === 2) ? "state=\"initial\"" : "state=\"needs-review-translation\"",
                         TranslationStatus::STATUS_REJECTED
                 ],
                 TranslationStatus::STATUS_NEW        => [
-                        ( $xliffVersion === 2 ) ? "state=\"initial\"" : "state=\"new\"",
+                        ($xliffVersion === 2) ? "state=\"initial\"" : "state=\"new\"",
                         TranslationStatus::STATUS_NEW
                 ],
                 TranslationStatus::STATUS_DRAFT      => [
-                        ( $xliffVersion === 2 ) ? "state=\"initial\"" : "state=\"new\"",
+                        ($xliffVersion === 2) ? "state=\"initial\"" : "state=\"new\"",
                         TranslationStatus::STATUS_DRAFT
                 ],
         ];
@@ -68,14 +68,14 @@ 
         // ---> $seg[ 'status' ] is NULL
         // If lastMrkState is empty
         $minStatus = min(
-                $stateLevelsMap[ $status ],
-                ( $stateLevelsMap[ $lastMrkState ] ?? $stateLevelsMap[ TranslationStatus::STATUS_NEW ] )
+                $stateLevelsMap[$status],
+                ($stateLevelsMap[$lastMrkState] ?? $stateLevelsMap[TranslationStatus::STATUS_NEW])
         );
 
         // If the last mark state is set, get the minimum value, otherwise get the current state
-        [ $state_prop, $lastMrkState ] = empty( $lastMrkState ) ? $stateMap[ $status ] : $stateMap[ $orderedValues[ $minStatus ] ];
+        [$state_prop, $lastMrkState] = empty($lastMrkState) ? $stateMap[$status] : $stateMap[$orderedValues[$minStatus]];
 
-        return [ $state_prop, $lastMrkState ];
+        return [$state_prop, $lastMrkState];
 
     }
 

src/XliffReplacer/XliffSdl.php

Spacing +21 added lines, -21 removed lines

@@ -28,38 +28,38 @@ 
     /**
      * @inheritDoc
      */
-    protected function tagOpen( $parser, string $name, array $attr ) {
+    protected function tagOpen($parser, string $name, array $attr) {
 
-        $this->handleOpenUnit( $name, $attr );
+        $this->handleOpenUnit($name, $attr);
 
         // check if we are entering into a <target>
-        $this->checkSetInTarget( $name );
+        $this->checkSetInTarget($name);
 
         // reset Marker positions
-        if ( 'sdl:seg-defs' == $name ) {
+        if ('sdl:seg-defs' == $name) {
             $this->segmentInUnitPosition = 0;
         }
 
         // open buffer
-        $this->setInBuffer( $name );
+        $this->setInBuffer($name);
 
         // check if we are inside a <target>, obviously this happen only if there are targets inside the trans-unit
         // <target> must be stripped to be replaced, so this check avoids <target> reconstruction
-        if ( !$this->inTarget ) {
+        if (!$this->inTarget) {
 
             // costruct tag
             $tag = "<$name ";
 
-            foreach ( $attr as $k => $v ) {
+            foreach ($attr as $k => $v) {
 
                 // if tag name is file, we must replace the target-language attribute
-                if ( $name == 'file' && $k == 'target-language' && !empty( $this->targetLang ) ) {
+                if ($name == 'file' && $k == 'target-language' && !empty($this->targetLang)) {
                     //replace Target language with job language provided from constructor
                     $tag .= "$k=\"$this->targetLang\" ";
                 } else {
                     //normal tag flux, put attributes in it
                     // Warning, this is NOT an elseif
-                    if ( $k != 'conf' ) {
+                    if ($k != 'conf') {
                         //put also the current attribute in it if it is not a "conf" attribute
                         $tag .= "$k=\"$v\" ";
                     }
@@ -68,11 +68,11 @@ 
 
             $seg = $this->getCurrentSegment();
 
-            if ( 'sdl:seg' == $name && !empty( $seg ) && isset( $seg[ 'sid' ] ) ) {
-                $tag .= $this->prepareTargetStatuses( $seg );
+            if ('sdl:seg' == $name && !empty($seg) && isset($seg['sid'])) {
+                $tag .= $this->prepareTargetStatuses($seg);
             }
 
-            $this->checkForSelfClosedTagAndFlush( $parser, $tag );
+            $this->checkForSelfClosedTagAndFlush($parser, $tag);
 
         }
 
@@ -83,7 +83,7 @@ 
      *
      * @return string
      */
-    protected function prepareTargetStatuses( $segment ): string {
+    protected function prepareTargetStatuses($segment): string {
         $statusMap = [
                 'NEW'        => '',
                 'DRAFT'      => 'Draft',
@@ -93,19 +93,19 @@ 
                 'REJECTED'   => 'RejectedTranslation',
         ];
 
-        return "conf=\"{$statusMap[ $segment[ 'status' ] ]}\" ";
+        return "conf=\"{$statusMap[$segment['status']]}\" ";
     }
 
-    protected function rebuildMarks( array $seg, string $translation ): string {
+    protected function rebuildMarks(array $seg, string $translation): string {
 
-        $trailingSpaces = str_repeat( ' ', Strings::getTheNumberOfTrailingSpaces( $translation ) );
+        $trailingSpaces = str_repeat(' ', Strings::getTheNumberOfTrailingSpaces($translation));
 
-        if ( $seg[ 'mrk_id' ] !== null && $seg[ 'mrk_id' ] != '' ) {
-            if ( $this->targetLang === 'ja-JP' ) {
-                $seg[ 'mrk_succ_tags' ] = ltrim( $seg[ 'mrk_succ_tags' ] );
+        if ($seg['mrk_id'] !== null && $seg['mrk_id'] != '') {
+            if ($this->targetLang === 'ja-JP') {
+                $seg['mrk_succ_tags'] = ltrim($seg['mrk_succ_tags']);
             }
 
-            $translation = "<mrk mid=\"" . $seg[ 'mrk_id' ] . "\" mtype=\"seg\">" . $seg[ 'mrk_prev_tags' ] . rtrim( $translation ) . $seg[ 'mrk_succ_tags' ] . "</mrk>" . $trailingSpaces;
+            $translation = "<mrk mid=\"" . $seg['mrk_id'] . "\" mtype=\"seg\">" . $seg['mrk_prev_tags'] . rtrim($translation) . $seg['mrk_succ_tags'] . "</mrk>" . $trailingSpaces;
         }
 
         return $translation;
@@ -120,7 +120,7 @@ 
      *
      * @return string
      */
-    protected function createTargetTag( string $translation, string $stateProp ): string {
+    protected function createTargetTag(string $translation, string $stateProp): string {
         $targetLang = ' xml:lang="' . $this->targetLang . '"';
 
         return "<target $targetLang $stateProp>$translation</target>";

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