matecat / xliff-parser

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

src/XliffReplacer/XliffReplacerCallbackInterface.php

Spacing +1 added lines, -1 removed lines

@@ -11,5 +11,5 @@
      * @param null $error
      * @return bool
      */
-    public function thereAreErrors( $segmentId, $segment, $translation, array $dataRefMap = [], $error = null );
+    public function thereAreErrors($segmentId, $segment, $translation, array $dataRefMap = [], $error = null);
 }

src/XliffReplacer/SdlXliffSAXTranslationReplacer.php

Spacing +47 added lines, -47 removed lines

@@ -10,22 +10,22 @@ 
     /**
      * @inheritDoc
      */
-    protected function tagOpen( $parser, $name, $attr ) {
+    protected function tagOpen($parser, $name, $attr) {
         // check if we are entering into a <trans-unit> (xliff v1.*) or <unit> (xliff v2.*)
-        if ( $this->tuTagName === $name ) {
+        if ($this->tuTagName === $name) {
             $this->inTU = true;
 
             // get id
             // trim to first 100 characters because this is the limit on Matecat's DB
-            $this->currentTransUnitId = substr( $attr[ 'id' ], 0, 100 );
+            $this->currentTransUnitId = substr($attr['id'], 0, 100);
 
             // current 'translate' attribute of the current trans-unit
-            $this->currentTransUnitTranslate = isset( $attr[ 'translate' ] ) ? $attr[ 'translate' ] : 'yes';
+            $this->currentTransUnitTranslate = isset($attr['translate']) ? $attr['translate'] : 'yes';
         }
 
         // check if we are entering into a <target>
-        if ( 'target' == $name ) {
-            if ( $this->currentTransUnitTranslate === 'no' ) {
+        if ('target' == $name) {
+            if ($this->currentTransUnitTranslate === 'no') {
                 $this->inTarget = false;
             } else {
                 $this->inTarget = true;
@@ -33,13 +33,13 @@ 
         }
 
         // reset Marker positions
-        if ( 'sdl:seg-defs' == $name ) {
+        if ('sdl:seg-defs' == $name) {
             $this->markerPos = 0;
         }
 
         // 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 ";
@@ -49,23 +49,23 @@ 
             // so the check on it's name is not enough
             $_sdlStatus_confWritten = false;
 
-            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\" ";
 
-                    if ( null !== $this->logger ) {
-                        $this->logger->debug( $k . " => " . $this->targetLang );
+                    if (null !== $this->logger) {
+                        $this->logger->debug($k . " => " . $this->targetLang);
                     }
-                } elseif ( 'sdl:seg' == $name ) {
+                } elseif ('sdl:seg' == $name) {
 
                     // write the confidence level for this segment ( Translated, Draft, etc. )
-                    if ( isset( $this->segments[ 'matecat|' . $this->currentTransUnitId ] ) && $_sdlStatus_confWritten === false ) {
+                    if (isset($this->segments['matecat|' . $this->currentTransUnitId]) && $_sdlStatus_confWritten === false) {
 
                         // append definition attribute
-                        $tag .= $this->prepareTargetStatuses( $this->lastTransUnit[ $this->markerPos ] );
+                        $tag .= $this->prepareTargetStatuses($this->lastTransUnit[$this->markerPos]);
 
                         //prepare for an eventual next cycle
                         $this->markerPos++;
@@ -73,7 +73,7 @@ 
                     }
 
                     // 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\" ";
                     }
@@ -86,29 +86,29 @@ 
             // this logic helps detecting empty tags
             // get current position of SAX pointer in all the stream of data is has read so far:
             // it points at the end of current tag
-            $idx = xml_get_current_byte_index( $parser );
+            $idx = xml_get_current_byte_index($parser);
 
             // check whether the bounds of current tag are entirely in current buffer || the end of the current tag
             // is outside current buffer (in the latter case, it's in next buffer to be read by the while loop);
             // this check is necessary because we may have truncated a tag in half with current read,
             // and the other half may be encountered in the next buffer it will be passed
-            if ( isset( $this->currentBuffer[ $idx - $this->offset ] ) ) {
+            if (isset($this->currentBuffer[$idx - $this->offset])) {
                 // if this tag entire lenght fitted in the buffer, the last char must be the last
                 // symbol before the '>'; if it's an empty tag, it is assumed that it's a '/'
                 $tmp_offset = $idx - $this->offset;
-                $lastChar   = $this->currentBuffer[ $tmp_offset ];
+                $lastChar   = $this->currentBuffer[$tmp_offset];
             } else {
                 //if it's out, simple use the last character of the chunk
                 $tmp_offset = $this->len - 1;
-                $lastChar   = $this->currentBuffer[ $tmp_offset ];
+                $lastChar   = $this->currentBuffer[$tmp_offset];
             }
 
             // trim last space
-            $tag = rtrim( $tag );
+            $tag = rtrim($tag);
 
             // detect empty tag
-            $this->isEmpty = ( $lastChar == '/' || $name == 'x' );
-            if ( $this->isEmpty ) {
+            $this->isEmpty = ($lastChar == '/' || $name == 'x');
+            if ($this->isEmpty) {
                 $tag .= '/';
             }
 
@@ -116,7 +116,7 @@ 
             $tag .= ">";
 
             // set a a Buffer for the segSource Source tag
-            if ( 'source' == $name
+            if ('source' == $name
                     || 'seg-source' === $name
                     || $this->bufferIsActive
                     || 'value' === $name
@@ -125,19 +125,19 @@ 
                     || 'ph' === $name
                     || 'st' === $name
                     || 'note' === $name
-                    || 'context' === $name ) { // we are opening a critical CDATA section
+                    || 'context' === $name) { // we are opening a critical CDATA section
 
                 // WARNING BECAUSE SOURCE AND SEG-SOURCE TAGS CAN BE EMPTY IN SOME CASES!!!!!
                 // so check for isEmpty also in conjunction with name
-                if ( $this->isEmpty && ( 'source' == $name || 'seg-source' == $name ) ) {
-                    $this->postProcAndFlush( $this->outputFP, $tag );
+                if ($this->isEmpty && ('source' == $name || 'seg-source' == $name)) {
+                    $this->postProcAndFlush($this->outputFP, $tag);
                 } else {
                     //these are NOT source/seg-source/value empty tags, THERE IS A CONTENT, write it in buffer
                     $this->bufferIsActive = true;
-                    $this->CDATABuffer    .= $tag;
+                    $this->CDATABuffer .= $tag;
                 }
             } else {
-                $this->postProcAndFlush( $this->outputFP, $tag );
+                $this->postProcAndFlush($this->outputFP, $tag);
             }
         }
     }
@@ -150,19 +150,19 @@ 
      *
      * @return string
      */
-    protected function prepareTranslation( $seg, $transUnitTranslation = "" ) {
+    protected function prepareTranslation($seg, $transUnitTranslation = "") {
         $endTags = "";
 
-        $segment     = Strings::removeDangerousChars( $seg [ 'segment' ] );
-        $translation = Strings::removeDangerousChars( $seg [ 'translation' ] );
-        $dataRefMap  = ( isset( $seg[ 'data_ref_map' ] ) && $seg[ 'data_ref_map' ] !== null ) ? Strings::jsonToArray( $seg[ 'data_ref_map' ] ) : [];
+        $segment     = Strings::removeDangerousChars($seg ['segment']);
+        $translation = Strings::removeDangerousChars($seg ['translation']);
+        $dataRefMap  = (isset($seg['data_ref_map']) && $seg['data_ref_map'] !== null) ? Strings::jsonToArray($seg['data_ref_map']) : [];
 
-        if ( is_null( $seg [ 'translation' ] ) || $seg [ 'translation' ] == '' ) {
+        if (is_null($seg ['translation']) || $seg ['translation'] == '') {
             $translation = $segment;
         } else {
-            if ( $this->callback instanceof XliffReplacerCallbackInterface ) {
+            if ($this->callback instanceof XliffReplacerCallbackInterface) {
                 $error = (isset($seg['error'])) ? $seg['error'] : null;
-                if ( $this->callback->thereAreErrors( $seg[ 'sid' ], $segment, $translation, $dataRefMap, $error ) ) {
+                if ($this->callback->thereAreErrors($seg['sid'], $segment, $translation, $dataRefMap, $error)) {
                     $translation = '|||UNTRANSLATED_CONTENT_START|||' . $segment . '|||UNTRANSLATED_CONTENT_END|||';
                 }
             }
@@ -171,20 +171,20 @@ 
         // for Trados the trailing spaces after </mrk> are meaningful
         // so we trim the translation from Matecat DB and add them after </mrk>
         $trailingSpaces = '';
-        for ( $s = 0; $s < Strings::getTheNumberOfTrailingSpaces( $translation ); $s++ ) {
+        for ($s = 0; $s < Strings::getTheNumberOfTrailingSpaces($translation); $s++) {
             $trailingSpaces .= ' ';
         }
 
-        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;
         }
 
         // we need to trim succ_tags here because we already added the trailing spaces after </mrk>
-        $transUnitTranslation .= $seg[ 'prev_tags' ] . $translation . $endTags . ltrim( $seg[ 'succ_tags' ] );
+        $transUnitTranslation .= $seg['prev_tags'] . $translation . $endTags . ltrim($seg['succ_tags']);
 
         return $transUnitTranslation;
     }
@@ -194,7 +194,7 @@ 
      *
      * @return string
      */
-    protected function prepareTargetStatuses( $segment ) {
+    protected function prepareTargetStatuses($segment) {
         $statusMap = [
                 'NEW'        => '',
                 'DRAFT'      => 'Draft',
@@ -203,7 +203,7 @@ 
                 'REJECTED'   => 'RejectedTranslation',
         ];
 
-        return "conf=\"{$statusMap[ $segment[ 'status' ] ]}\" ";
+        return "conf=\"{$statusMap[$segment['status']]}\" ";
     }
 
     /**
@@ -213,8 +213,8 @@ 
      *
      * @return array
      */
-    protected function setTransUnitState( $seg, $state_prop, $lastMrkState ) {
-        return [ null, null ];
+    protected function setTransUnitState($seg, $state_prop, $lastMrkState) {
+        return [null, null];
     }
 
     /**
@@ -223,7 +223,7 @@ 
      *
      * @return string
      */
-    protected function getWordCountGroup( $raw_word_count, $eq_word_count ) {
+    protected function getWordCountGroup($raw_word_count, $eq_word_count) {
         return '';
     }
 }

src/Utils/Strings.php

Spacing +70 added lines, -70 removed lines

@@ -17,8 +17,8 @@ 
      * @return string
      * @throws Exception
      */
-    public static function cleanCDATA( $testString ) {
-        $cleanXMLContent = new SimpleXMLElement( '<rootNoteNode>' . $testString . '</rootNoteNode>', LIBXML_NOCDATA );
+    public static function cleanCDATA($testString) {
+        $cleanXMLContent = new SimpleXMLElement('<rootNoteNode>' . $testString . '</rootNoteNode>', LIBXML_NOCDATA);
 
         return $cleanXMLContent->__toString();
     }
@@ -28,28 +28,28 @@ 
      *
      * @return bool
      */
-    public static function isJSON( $string ) {
-        if ( is_numeric( $string ) ) {
+    public static function isJSON($string) {
+        if (is_numeric($string)) {
             return false;
         }
 
         try {
-            $string = Strings::cleanCDATA( $string );
-        } catch ( Exception $e ) {
+            $string = Strings::cleanCDATA($string);
+        } catch (Exception $e) {
             return false;
         }
 
-        $string = trim( $string );
-        if ( empty( $string ) ) {
+        $string = trim($string);
+        if (empty($string)) {
             return false;
         }
 
         // String representation in json is "quoted", but we want to accept only object or arrays.
         // exclude strings and numbers and other primitive types
-        if ( in_array( $string [ 0 ], [ "{", "[" ] ) ) {
-            json_decode( $string );
+        if (in_array($string [0], ["{", "["])) {
+            json_decode($string);
 
-            return empty( self::getLastJsonError()[ 0 ] );
+            return empty(self::getLastJsonError()[0]);
         } else {
             return false; // Not accepted: string or primitive types.
         }
@@ -61,10 +61,10 @@ 
      *
      * @return array
      */
-    public static function jsonToArray( $string ) {
-        $decodedJSON = json_decode( $string, true );
+    public static function jsonToArray($string) {
+        $decodedJSON = json_decode($string, true);
 
-        return ( is_array( $decodedJSON ) ) ? $decodedJSON : [];
+        return (is_array($decodedJSON)) ? $decodedJSON : [];
     }
 
     /**
@@ -75,10 +75,10 @@ 
      */
     private static function raiseLastJsonException() {
 
-        list( $msg, $error ) = self::getLastJsonError();
+        list($msg, $error) = self::getLastJsonError();
 
-        if ( $error != JSON_ERROR_NONE ) {
-            throw new NotValidJSONException( $msg, $error );
+        if ($error != JSON_ERROR_NONE) {
+            throw new NotValidJSONException($msg, $error);
         }
 
     }
@@ -88,11 +88,11 @@ 
      */
     private static function getLastJsonError() {
 
-        if ( function_exists( "json_last_error" ) ) {
+        if (function_exists("json_last_error")) {
 
             $error = json_last_error();
 
-            switch ( $error ) {
+            switch ($error) {
                 case JSON_ERROR_NONE:
                     $msg = null; # - No errors
                     break;
@@ -116,10 +116,10 @@ 
                     break;
             }
 
-            return [ $msg, $error ];
+            return [$msg, $error];
         }
 
-        return [ null, JSON_ERROR_NONE ];
+        return [null, JSON_ERROR_NONE];
 
     }
 
@@ -142,11 +142,11 @@ 
      *
      * @return string
      */
-    public static function fixNonWellFormedXml( $content, $escapeStrings = true ) {
-        if ( self::$find_xliff_tags_reg === null ) {
+    public static function fixNonWellFormedXml($content, $escapeStrings = true) {
+        if (self::$find_xliff_tags_reg === null) {
             // Convert the list of tags in a regexp list, for example "g|x|bx|ex"
             $xliffTags           = XliffTags::$tags;
-            $xliff_tags_reg_list = implode( '|', $xliffTags );
+            $xliff_tags_reg_list = implode('|', $xliffTags);
             // Regexp to find all the XLIFF tags:
             //   </?               -> matches the tag start, for both opening and
             //                        closure tags (see the optional slash)
@@ -167,30 +167,30 @@ 
         }
 
         // Find all the XLIFF tags
-        preg_match_all( self::$find_xliff_tags_reg, $content, $matches );
-        $tags = (array)$matches[ 0 ];
+        preg_match_all(self::$find_xliff_tags_reg, $content, $matches);
+        $tags = (array)$matches[0];
 
         // Prepare placeholders
         $tags_placeholders = [];
-        $tagsNum           = count( $tags );
-        for ( $i = 0; $i < $tagsNum; $i++ ) {
-            $tag                       = $tags[ $i ];
-            $tags_placeholders[ $tag ] = "#@!XLIFF-TAG-$i!@#";
+        $tagsNum           = count($tags);
+        for ($i = 0; $i < $tagsNum; $i++) {
+            $tag                       = $tags[$i];
+            $tags_placeholders[$tag] = "#@!XLIFF-TAG-$i!@#";
         }
 
         // Replace all XLIFF tags with placeholders that will not be escaped
-        foreach ( $tags_placeholders as $tag => $placeholder ) {
-            $content = str_replace( $tag, $placeholder, $content );
+        foreach ($tags_placeholders as $tag => $placeholder) {
+            $content = str_replace($tag, $placeholder, $content);
         }
 
         // Escape the string with the remaining non-XLIFF tags
-        if ( $escapeStrings ) {
-            $content = htmlspecialchars( $content, ENT_NOQUOTES, 'UTF-8', false );
+        if ($escapeStrings) {
+            $content = htmlspecialchars($content, ENT_NOQUOTES, 'UTF-8', false);
         }
 
         // Put again in place the original XLIFF tags replacing placeholders
-        foreach ( $tags_placeholders as $tag => $placeholder ) {
-            $content = str_replace( $placeholder, $tag, $content );
+        foreach ($tags_placeholders as $tag => $placeholder) {
+            $content = str_replace($placeholder, $tag, $content);
         }
 
         return $content;
@@ -201,17 +201,17 @@ 
      *
      * @return string
      */
-    public static function removeDangerousChars( $string ) {
+    public static function removeDangerousChars($string) {
         // clean invalid xml entities ( characters with ascii < 32 and different from 0A, 0D and 09
         $regexpEntity = '/&#x(0[0-8BCEF]|1[\dA-F]|7F);/u';
 
         // remove binary chars in some xliff files
         $regexpAscii = '/[\x{00}-\x{08}\x{0B}\x{0C}\x{0E}-\x{1F}\x{7F}]/u';
 
-        $string = preg_replace( $regexpAscii, '', $string );
-        $string = preg_replace( $regexpEntity, '', $string );
+        $string = preg_replace($regexpAscii, '', $string);
+        $string = preg_replace($regexpEntity, '', $string);
 
-        return !empty( $string ) || strlen( $string ) > 0 ? $string : "";
+        return !empty($string) || strlen($string) > 0 ? $string : "";
     }
 
     /**
@@ -220,8 +220,8 @@ 
      *
      * @return bool
      */
-    public static function contains( $needle, $haystack ) {
-        return mb_strpos( $haystack, $needle ) !== false;
+    public static function contains($needle, $haystack) {
+        return mb_strpos($haystack, $needle) !== false;
     }
 
     /**
@@ -229,8 +229,8 @@ 
      *
      * @return string
      */
-    public static function htmlentities( $string ) {
-        return htmlentities( $string, ENT_NOQUOTES );
+    public static function htmlentities($string) {
+        return htmlentities($string, ENT_NOQUOTES);
     }
 
     /**
@@ -239,15 +239,15 @@ 
      *
      * @return string
      */
-    public static function htmlspecialchars_decode( $string, $onlyEscapedEntities = false ) {
-        if ( false === $onlyEscapedEntities ) {
-            return htmlspecialchars_decode( $string, ENT_NOQUOTES );
+    public static function htmlspecialchars_decode($string, $onlyEscapedEntities = false) {
+        if (false === $onlyEscapedEntities) {
+            return htmlspecialchars_decode($string, ENT_NOQUOTES);
         }
 
-        return preg_replace_callback( self::$htmlEntityRegex,
-                function ( $match ) {
-                    return self::htmlspecialchars_decode( $match[ 0 ] );
-                }, $string );
+        return preg_replace_callback(self::$htmlEntityRegex,
+                function($match) {
+                    return self::htmlspecialchars_decode($match[0]);
+                }, $string);
     }
 
     /**
@@ -262,8 +262,8 @@ 
      *
      * @return bool
      */
-    public static function isADoubleEscapedEntity( $str ) {
-        return preg_match( self::$htmlEntityRegex, $str ) != 0;
+    public static function isADoubleEscapedEntity($str) {
+        return preg_match(self::$htmlEntityRegex, $str) != 0;
     }
 
     /**
@@ -271,8 +271,8 @@ 
      *
      * @return bool
      */
-    public static function isAnEscapedHTML( $str ) {
-        return preg_match( '#/[a-z]*&gt;#i', $str ) != 0;
+    public static function isAnEscapedHTML($str) {
+        return preg_match('#/[a-z]*&gt;#i', $str) != 0;
     }
 
     /**
@@ -280,8 +280,8 @@ 
      *
      * @return bool
      */
-    public static function isAValidUuid( $uuid ) {
-        return preg_match( '/^[\da-f]{8}-[\da-f]{4}-4[\da-f]{3}-[89ab][\da-f]{3}-[\da-f]{12}$/', $uuid ) === 1;
+    public static function isAValidUuid($uuid) {
+        return preg_match('/^[\da-f]{8}-[\da-f]{4}-4[\da-f]{3}-[89ab][\da-f]{3}-[\da-f]{12}$/', $uuid) === 1;
     }
 
     /**
@@ -290,8 +290,8 @@ 
      *
      * @return array|false|string[]
      */
-    public static function preg_split( $pattern, $subject ) {
-        return preg_split( $pattern, $subject, -1, PREG_SPLIT_NO_EMPTY );
+    public static function preg_split($pattern, $subject) {
+        return preg_split($pattern, $subject, -1, PREG_SPLIT_NO_EMPTY);
     }
 
     /**
@@ -309,8 +309,8 @@ 
      *
      * @return string
      */
-    public static function escapeOnlyHTMLTags( $string ) {
-        return preg_replace( '/<(.*?)>/iu', '&lt;$1&gt;', $string );
+    public static function escapeOnlyHTMLTags($string) {
+        return preg_replace('/<(.*?)>/iu', '&lt;$1&gt;', $string);
     }
 
     /**
@@ -320,8 +320,8 @@ 
      *
      * @return string
      */
-    public static function lastChar( $string ) {
-        return mb_substr( $string, -1 );
+    public static function lastChar($string) {
+        return mb_substr($string, -1);
     }
 
     /**
@@ -329,8 +329,8 @@ 
      *
      * @return int
      */
-    public static function getTheNumberOfTrailingSpaces( $segment ) {
-        return mb_strlen( $segment ) - mb_strlen( rtrim( $segment, ' ' ) );
+    public static function getTheNumberOfTrailingSpaces($segment) {
+        return mb_strlen($segment) - mb_strlen(rtrim($segment, ' '));
     }
 
     /**
@@ -340,15 +340,15 @@ 
      *
      * @return bool
      */
-    public static function isHtmlString( $string ) {
-        $string = stripslashes( $string );
+    public static function isHtmlString($string) {
+        $string = stripslashes($string);
 
-        if ( $string === '<>' ) {
+        if ($string === '<>') {
             return false;
         }
 
-        preg_match( "#</?[a-zA-Z1-6-]+((\s+[a-zA-Z1-6-]+(\s*=\s*(?:\".*?\"|'.*?'|[^'\">\s]+))?)+\s*|\s*)/?>#", $string, $matches );
+        preg_match("#</?[a-zA-Z1-6-]+((\s+[a-zA-Z1-6-]+(\s*=\s*(?:\".*?\"|'.*?'|[^'\">\s]+))?)+\s*|\s*)/?>#", $string, $matches);
 
-        return count( $matches ) !== 0;
+        return count($matches) !== 0;
     }
 }

src/XliffParser.php

Spacing +56 added lines, -56 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( $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
         }
     }
@@ -67,26 +67,26 @@ 
      * @throws InvalidXmlException
      * @throws 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' ] ) ? $info[ 'proprietary_short_name' ] : null;
-        $parser           = XliffParserFactory::getInstance( $xliffVersion, $xliffProprietary, $this->logger );
+        $xliffProprietary = isset($info['proprietary_short_name']) ? $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 ) {
-        $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 : "";
         }
 
@@ -131,24 +131,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>]|i', $xliffContent );
+        $a     = Strings::preg_split('|<internal-file[\s>]|i', $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>|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++;
 
-        if ( isset( $a[ 2 ] ) ) {
-            $c                                           = Strings::preg_split( '|</internal-file[\s>]|i', $a[ 2 ] );
-            $strippedContent                             .= $c[ 1 ];
-            $xliff[ 'files' ][ $index ][ 'reference' ][] = self::extractBase64( $c[ 0 ] );
+        if (isset($a[2])) {
+            $c                                           = Strings::preg_split('|</internal-file[\s>]|i', $a[2]);
+            $strippedContent .= $c[1];
+            $xliff['files'][$index]['reference'][] = self::extractBase64($c[0]);
         }
 
         return $strippedContent;
@@ -159,10 +159,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)),
         ];
     }
 
@@ -182,9 +182,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;
     }
@@ -203,15 +203,15 @@ 
      *
      * @return string
      */
-    private static function insertPlaceholderInEmptyTags( $xliffContent ) {
-        preg_match_all( '|<([a-zA-Z0-9._-]+)[^>]*></\1>|m', $xliffContent, $emptyTagMatches );
+    private static function insertPlaceholderInEmptyTags($xliffContent) {
+        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);
             }
         }
 
@@ -225,12 +225,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>';
     }
 
     /**
@@ -238,15 +238,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/XliffParser/XliffParserV1.php

Spacing +95 added lines, -95 removed lines

@@ -16,33 +16,33 @@ 
      * @inheritDoc
      * @throws Exception
      */
-    public function parse( DOMDocument $dom, $output = [] ) {
+    public function parse(DOMDocument $dom, $output = []) {
         $i = 1;
         /** @var DOMElement $file */
-        foreach ( $dom->getElementsByTagName( 'file' ) as $file ) {
+        foreach ($dom->getElementsByTagName('file') as $file) {
 
             // metadata
-            $output[ 'files' ][ $i ][ 'attr' ] = $this->extractMetadata( $file );
+            $output['files'][$i]['attr'] = $this->extractMetadata($file);
 
             // reference
-            if ( !empty( $this->extractReference( $file ) ) ) {
-                $output[ 'files' ][ $i ][ 'reference' ] = $this->extractReference( $file );
+            if (!empty($this->extractReference($file))) {
+                $output['files'][$i]['reference'] = $this->extractReference($file);
             }
 
             // trans-units
             $transUnitIdArrayForUniquenessCheck = [];
             $j                                  = 1;
-            foreach ( $file->childNodes as $body ) {
-                if ( $body->nodeName === 'body' ) {
-                    foreach ( $body->childNodes as $childNode ) {
-                        $this->extractTuFromNode( $childNode, $transUnitIdArrayForUniquenessCheck, $dom, $output, $i, $j );
+            foreach ($file->childNodes as $body) {
+                if ($body->nodeName === 'body') {
+                    foreach ($body->childNodes as $childNode) {
+                        $this->extractTuFromNode($childNode, $transUnitIdArrayForUniquenessCheck, $dom, $output, $i, $j);
                     }
 
                     // trans-unit re-count check
-                    $totalTransUnitsId  = count( $transUnitIdArrayForUniquenessCheck );
-                    $transUnitsUniqueId = count( array_unique( $transUnitIdArrayForUniquenessCheck ) );
-                    if ( $totalTransUnitsId != $transUnitsUniqueId ) {
-                        throw new DuplicateTransUnitIdInXliff( "Invalid trans-unit id, duplicate found.", 400 );
+                    $totalTransUnitsId  = count($transUnitIdArrayForUniquenessCheck);
+                    $transUnitsUniqueId = count(array_unique($transUnitIdArrayForUniquenessCheck));
+                    if ($totalTransUnitsId != $transUnitsUniqueId) {
+                        throw new DuplicateTransUnitIdInXliff("Invalid trans-unit id, duplicate found.", 400);
                     }
 
                     $i++;
@@ -58,50 +58,50 @@ 
      *
      * @return array
      */
-    private function extractMetadata( DOMElement $file ) {
+    private function extractMetadata(DOMElement $file) {
         $metadata   = [];
         $customAttr = [];
 
         /** @var DOMAttr $attribute */
-        foreach ( $file->attributes as $attribute ) {
-            switch ( $attribute->localName ) {
+        foreach ($file->attributes as $attribute) {
+            switch ($attribute->localName) {
                 // original
                 case 'original':
-                    $metadata[ 'original' ] = $attribute->value;
+                    $metadata['original'] = $attribute->value;
                     break;
 
                 // source-language
                 case 'source-language':
-                    $metadata[ 'source-language' ] = $attribute->value;
+                    $metadata['source-language'] = $attribute->value;
                     break;
 
                 // data-type
                 case 'datatype':
-                    $metadata[ 'data-type' ] = $attribute->value;
+                    $metadata['data-type'] = $attribute->value;
                     break;
 
                 // target-language
                 case 'target-language':
-                    $metadata[ 'target-language' ] = $attribute->value;
+                    $metadata['target-language'] = $attribute->value;
                     break;
             }
 
             // Custom MateCat x-Attribute
-            preg_match( '|x-(.*?)|si', $attribute->localName, $temp );
-            if ( isset( $temp[ 1 ] ) ) {
-                $customAttr[ $attribute->localName ] = $attribute->value;
+            preg_match('|x-(.*?)|si', $attribute->localName, $temp);
+            if (isset($temp[1])) {
+                $customAttr[$attribute->localName] = $attribute->value;
             }
-            unset( $temp );
+            unset($temp);
 
             // Custom MateCat namespace Attribute mtc:
-            preg_match( '|mtc:(.*?)|si', $attribute->nodeName, $temp );
-            if ( isset( $temp[ 1 ] ) ) {
-                $customAttr[ $attribute->nodeName ] = $attribute->value;
+            preg_match('|mtc:(.*?)|si', $attribute->nodeName, $temp);
+            if (isset($temp[1])) {
+                $customAttr[$attribute->nodeName] = $attribute->value;
             }
-            unset( $temp );
+            unset($temp);
 
-            if ( !empty( $customAttr ) ) {
-                $metadata[ 'custom' ] = $customAttr;
+            if (!empty($customAttr)) {
+                $metadata['custom'] = $customAttr;
             }
         }
 
@@ -113,16 +113,16 @@ 
      *
      * @return array
      */
-    private function extractReference( DOMElement $file ) {
+    private function extractReference(DOMElement $file) {
         $reference = [];
 
         $order = 0;
-        foreach ( $file->getElementsByTagName( 'reference' ) as $ref ) {
+        foreach ($file->getElementsByTagName('reference') as $ref) {
             /** @var DOMNode $childNode */
-            foreach ( $ref->childNodes as $childNode ) {
-                if ( $childNode->nodeName === 'internal-file' ) {
-                    $reference[ $order ][ 'form-type' ] = $childNode->attributes->getNamedItem( 'form' )->nodeValue;
-                    $reference[ $order ][ 'base64' ]    = trim( $childNode->nodeValue );
+            foreach ($ref->childNodes as $childNode) {
+                if ($childNode->nodeName === 'internal-file') {
+                    $reference[$order]['form-type'] = $childNode->attributes->getNamedItem('form')->nodeValue;
+                    $reference[$order]['base64']    = trim($childNode->nodeValue);
                     $order++;
                 }
             }
@@ -144,60 +144,60 @@ 
      *
      * @throws Exception
      */
-    protected function extractTransUnit( DOMElement $transUnit, &$transUnitIdArrayForUniquenessCheck, $dom, &$output, &$i, &$j, $contextGroups = [] ) {
+    protected function extractTransUnit(DOMElement $transUnit, &$transUnitIdArrayForUniquenessCheck, $dom, &$output, &$i, &$j, $contextGroups = []) {
         // metadata
-        $output[ 'files' ][ $i ][ 'trans-units' ][ $j ][ 'attr' ] = $this->extractTransUnitMetadata( $transUnit, $transUnitIdArrayForUniquenessCheck );
+        $output['files'][$i]['trans-units'][$j]['attr'] = $this->extractTransUnitMetadata($transUnit, $transUnitIdArrayForUniquenessCheck);
 
         // notes
-        $output[ 'files' ][ $i ][ 'trans-units' ][ $j ][ 'notes' ] = $this->extractTransUnitNotes( $dom, $transUnit );
+        $output['files'][$i]['trans-units'][$j]['notes'] = $this->extractTransUnitNotes($dom, $transUnit);
 
         // content
         /** @var DOMElement $childNode */
-        foreach ( $transUnit->childNodes as $childNode ) {
+        foreach ($transUnit->childNodes as $childNode) {
             // source
-            if ( $childNode->nodeName === 'source' ) {
-                $output[ 'files' ][ $i ][ 'trans-units' ][ $j ][ 'source' ] = $this->extractContent( $dom, $childNode );
+            if ($childNode->nodeName === 'source') {
+                $output['files'][$i]['trans-units'][$j]['source'] = $this->extractContent($dom, $childNode);
             }
 
             // seg-source
-            if ( $childNode->nodeName === 'seg-source' ) {
-                $rawSegment                                                     = $output[ 'files' ][ $i ][ 'trans-units' ][ $j ][ 'source' ][ 'raw-content' ];
-                $output[ 'files' ][ $i ][ 'trans-units' ][ $j ][ 'seg-source' ] = $this->extractContentWithMarksAndExtTags( $dom, $childNode, $rawSegment );
+            if ($childNode->nodeName === 'seg-source') {
+                $rawSegment                                                     = $output['files'][$i]['trans-units'][$j]['source']['raw-content'];
+                $output['files'][$i]['trans-units'][$j]['seg-source'] = $this->extractContentWithMarksAndExtTags($dom, $childNode, $rawSegment);
             }
 
             // target
-            if ( $childNode->nodeName === 'target' ) {
-                $output[ 'files' ][ $i ][ 'trans-units' ][ $j ][ 'target' ] = $this->extractContent( $dom, $childNode );
+            if ($childNode->nodeName === 'target') {
+                $output['files'][$i]['trans-units'][$j]['target'] = $this->extractContent($dom, $childNode);
 
                 // seg-target
-                $targetRawContent = @$output[ 'files' ][ $i ][ 'trans-units' ][ $j ][ 'target' ][ 'raw-content' ];
-                $segSource        = @$output[ 'files' ][ $i ][ 'trans-units' ][ $j ][ 'seg-source' ];
-                if ( isset( $targetRawContent ) && !empty( $targetRawContent ) && isset( $segSource ) && count( $segSource ) > 0 ) {
-                    $output[ 'files' ][ $i ][ 'trans-units' ][ $j ][ 'seg-target' ] = $this->extractContentWithMarksAndExtTags( $dom, $childNode );
-                    $output[ 'files' ][ $i ][ 'trans-units' ][ $j ][ 'seg-target' ][ 0 ]['attr'] = $this->extractTagAttributes($childNode);
+                $targetRawContent = @$output['files'][$i]['trans-units'][$j]['target']['raw-content'];
+                $segSource        = @$output['files'][$i]['trans-units'][$j]['seg-source'];
+                if (isset($targetRawContent) && !empty($targetRawContent) && isset($segSource) && count($segSource) > 0) {
+                    $output['files'][$i]['trans-units'][$j]['seg-target'] = $this->extractContentWithMarksAndExtTags($dom, $childNode);
+                    $output['files'][$i]['trans-units'][$j]['seg-target'][0]['attr'] = $this->extractTagAttributes($childNode);
                 }
             }
 
             // locked
-            if ( $childNode->nodeName === 'sdl:seg' ) {
-                $output[ 'files' ][ $i ][ 'trans-units' ][ $j ][ 'locked' ] = $this->extractLocked( $childNode );
+            if ($childNode->nodeName === 'sdl:seg') {
+                $output['files'][$i]['trans-units'][$j]['locked'] = $this->extractLocked($childNode);
             }
         }
 
         // context-group
-        if(!empty($contextGroups)){
-            foreach ($contextGroups as $contextGroup){
-                $output[ 'files' ][ $i ][ 'trans-units' ][ $j ][ 'context-group' ][] = $this->extractTransUnitContextGroup( $dom, $contextGroup );
+        if (!empty($contextGroups)) {
+            foreach ($contextGroups as $contextGroup) {
+                $output['files'][$i]['trans-units'][$j]['context-group'][] = $this->extractTransUnitContextGroup($dom, $contextGroup);
             }
         }
 
-        foreach ( $transUnit->getElementsByTagName( 'context-group' ) as $contextGroup ) {
-            $output[ 'files' ][ $i ][ 'trans-units' ][ $j ][ 'context-group' ][] = $this->extractTransUnitContextGroup( $dom, $contextGroup );
+        foreach ($transUnit->getElementsByTagName('context-group') as $contextGroup) {
+            $output['files'][$i]['trans-units'][$j]['context-group'][] = $this->extractTransUnitContextGroup($dom, $contextGroup);
         }
 
         // alt-trans
-        foreach ( $transUnit->getElementsByTagName( 'alt-trans' ) as $altTrans ) {
-            $output[ 'files' ][ $i ][ 'trans-units' ][ $j ][ 'alt-trans' ][] = $this->extractTransUnitAltTrans( $altTrans );
+        foreach ($transUnit->getElementsByTagName('alt-trans') as $altTrans) {
+            $output['files'][$i]['trans-units'][$j]['alt-trans'][] = $this->extractTransUnitAltTrans($altTrans);
         }
 
         $j++;
@@ -210,41 +210,41 @@ 
      * @return array
      * @throws Exception
      */
-    private function extractTransUnitMetadata( DOMElement $transUnit, array &$transUnitIdArrayForUniquenessCheck ) {
+    private function extractTransUnitMetadata(DOMElement $transUnit, array &$transUnitIdArrayForUniquenessCheck) {
         $metadata = [];
 
         // id MUST NOT be null
-        if ( null === $transUnit->attributes->getNamedItem( 'id' ) ) {
-            throw new NotFoundIdInTransUnit( 'Invalid trans-unit id found. EMPTY value', 400 );
+        if (null === $transUnit->attributes->getNamedItem('id')) {
+            throw new NotFoundIdInTransUnit('Invalid trans-unit id found. EMPTY value', 400);
         }
 
         /**
          * @var DOMAttr $element
          */
-        foreach ( $transUnit->attributes as $element ) {
+        foreach ($transUnit->attributes as $element) {
 
-            if ( $element->nodeName === "id" ) {
+            if ($element->nodeName === "id") {
 
                 $id = $element->nodeValue;
 
-                if ( strlen( $id ) > 100 ) {
-                    throw new SegmentIdTooLongException( 'Segment-id too long. Max 100 characters allowed', 400 );
+                if (strlen($id) > 100) {
+                    throw new SegmentIdTooLongException('Segment-id too long. Max 100 characters allowed', 400);
                 }
 
                 $transUnitIdArrayForUniquenessCheck[] = $id;
-                $metadata[ 'id' ]                     = $id;
+                $metadata['id']                     = $id;
 
-            } elseif ( $element->nodeName === "approved" ) {
+            } elseif ($element->nodeName === "approved") {
                 // approved as BOOLEAN
                 // http://docs.oasis-open.org/xliff/v1.2/os/xliff-core.html#approved
-                $metadata[ $element->nodeName ] = filter_var( $element->nodeValue, FILTER_VALIDATE_BOOLEAN );
-            } elseif ( $element->nodeName === "maxwidth" ) {
+                $metadata[$element->nodeName] = filter_var($element->nodeValue, FILTER_VALIDATE_BOOLEAN);
+            } elseif ($element->nodeName === "maxwidth") {
                 // we ignore ( but we get ) the attribute size-unit="char" assuming that a restriction is everytime done by character
                 // we duplicate the info to allow Xliff V1 and V2 to work the same
-                $metadata[ 'sizeRestriction' ]  = filter_var( $element->nodeValue, FILTER_SANITIZE_NUMBER_INT );
-                $metadata[ $element->nodeName ] = filter_var( $element->nodeValue, FILTER_SANITIZE_NUMBER_INT );
+                $metadata['sizeRestriction']  = filter_var($element->nodeValue, FILTER_SANITIZE_NUMBER_INT);
+                $metadata[$element->nodeName] = filter_var($element->nodeValue, FILTER_SANITIZE_NUMBER_INT);
             } else {
-                $metadata[ $element->nodeName ] = $element->nodeValue;
+                $metadata[$element->nodeName] = $element->nodeValue;
             }
 
         }
@@ -258,19 +258,19 @@ 
      * @return array
      * @throws Exception
      */
-    private function extractTransUnitNotes( DOMDocument $dom, DOMElement $transUnit ) {
+    private function extractTransUnitNotes(DOMDocument $dom, DOMElement $transUnit) {
         $notes = [];
-        foreach ( $transUnit->getElementsByTagName( 'note' ) as $note ) {
+        foreach ($transUnit->getElementsByTagName('note') as $note) {
 
-            $noteValue = $this->extractTagContent( $dom, $note );
+            $noteValue = $this->extractTagContent($dom, $note);
 
-            if ( '' !== $noteValue ) {
+            if ('' !== $noteValue) {
 
-                $extractedNote = $this->JSONOrRawContentArray( $noteValue );
+                $extractedNote = $this->JSONOrRawContentArray($noteValue);
 
                 // extract all the attributes
-                foreach ( $note->attributes as $attribute ) {
-                    $extractedNote[ $attribute->name ] = $attribute->value;
+                foreach ($note->attributes as $attribute) {
+                    $extractedNote[$attribute->name] = $attribute->value;
                 }
 
                 $notes[] = $extractedNote;
@@ -285,14 +285,14 @@ 
      *
      * @return array
      */
-    private function extractTransUnitContextGroup( DOMDocument $dom, DOMElement $contextGroup ) {
+    private function extractTransUnitContextGroup(DOMDocument $dom, DOMElement $contextGroup) {
         $cg           = [];
-        $cg[ 'attr' ] = $this->extractTagAttributes( $contextGroup );
+        $cg['attr'] = $this->extractTagAttributes($contextGroup);
 
         /** @var DOMNode $context */
-        foreach ( $contextGroup->childNodes as $context ) {
-            if ( $context->nodeName === 'context' ) {
-                $cg[ 'contexts' ][] = $this->extractContent( $dom, $context );
+        foreach ($contextGroup->childNodes as $context) {
+            if ($context->nodeName === 'context') {
+                $cg['contexts'][] = $this->extractContent($dom, $context);
             }
         }
 
@@ -304,16 +304,16 @@ 
      *
      * @return array
      */
-    private function extractTransUnitAltTrans( DOMElement $altTrans ) {
+    private function extractTransUnitAltTrans(DOMElement $altTrans) {
         $at           = [];
-        $at[ 'attr' ] = $this->extractTagAttributes( $altTrans );
+        $at['attr'] = $this->extractTagAttributes($altTrans);
 
-        if ( $altTrans->getElementsByTagName( 'source' )->length > 0 ) {
-            $at[ 'source' ] = $altTrans->getElementsByTagName( 'source' )->item( 0 )->nodeValue;
+        if ($altTrans->getElementsByTagName('source')->length > 0) {
+            $at['source'] = $altTrans->getElementsByTagName('source')->item(0)->nodeValue;
         }
 
-        if ( $altTrans->getElementsByTagName( 'target' ) ) {
-            $at[ 'target' ] = $altTrans->getElementsByTagName( 'target' )->item( 0 )->nodeValue;
+        if ($altTrans->getElementsByTagName('target')) {
+            $at['target'] = $altTrans->getElementsByTagName('target')->item(0)->nodeValue;
         }
 
         return $at;
@@ -324,7 +324,7 @@ 
      *
      * @return bool
      */
-    private function extractLocked( DOMElement $locked ) {
-        return null !== $locked->getAttribute( 'locked' );
+    private function extractLocked(DOMElement $locked) {
+        return null !== $locked->getAttribute('locked');
     }
 }