matecat / xliff-parser

src/XliffParser/AbstractXliffParser.php

Spacing +14 added lines, -14 removed lines

@@ -32,7 +32,7 @@ 
      * @param string|null          $xliffProprietary
      * @param LoggerInterface|null $logger
      */
-    public function __construct( $xliffVersion, $xliffProprietary = null, LoggerInterface $logger = null)
+    public function __construct($xliffVersion, $xliffProprietary = null, LoggerInterface $logger = null)
     {
         $this->xliffVersion = $xliffVersion;
         $this->logger       = $logger;
@@ -126,7 +126,7 @@ 
 
         if ($element->hasAttributes()) {
             foreach ($element->attributes as $attr) {
-                $tagAttributes[ $attr->nodeName ] = $attr->nodeValue;
+                $tagAttributes[$attr->nodeName] = $attr->nodeValue;
             }
         }
 
@@ -175,40 +175,40 @@ 
         $markers = preg_split('#<mrk\s#si', $raw, -1);
 
         $mi = 0;
-        while (isset($markers[ $mi + 1 ])) {
+        while (isset($markers[$mi + 1])) {
             unset($mid);
 
-            preg_match('|mid\s?=\s?["\'](.*?)["\']|si', $markers[ $mi + 1 ], $mid);
+            preg_match('|mid\s?=\s?["\'](.*?)["\']|si', $markers[$mi + 1], $mid);
 
             // if it's a Trados file the trailing spaces after </mrk> are meaningful
             // so we add them to
             $trailingSpaces = '';
-            if($this->xliffProprietary === 'trados'){
-                preg_match_all('/<\/mrk>[\s]+/iu', $markers[ $mi + 1 ], $trailingSpacesMatches);
+            if ($this->xliffProprietary === 'trados') {
+                preg_match_all('/<\/mrk>[\s]+/iu', $markers[$mi + 1], $trailingSpacesMatches);
 
-                if(isset($trailingSpacesMatches[0]) and count($trailingSpacesMatches[0]) > 0){
-                    foreach ($trailingSpacesMatches[0] as $match){
+                if (isset($trailingSpacesMatches[0]) and count($trailingSpacesMatches[0]) > 0) {
+                    foreach ($trailingSpacesMatches[0] as $match) {
                         $trailingSpaces = str_replace('</mrk>', '', $match);
                     }
                 }
             }
 
             //re-build the mrk tag after the split
-            $originalMark = trim('<mrk ' . $markers[ $mi + 1 ]);
+            $originalMark = trim('<mrk ' . $markers[$mi + 1]);
 
             $mark_string  = preg_replace('#^<mrk\s[^>]+>(.*)#', '$1', $originalMark); // at this point we have: ---> 'Test </mrk> </g>>'
             $mark_content = preg_split('#</mrk>#si', $mark_string);
 
             $sourceArray = [
-                'mid' => (isset($mid[ 1 ])) ? $mid[ 1 ] : $mi,
-                'ext-prec-tags' => ($mi == 0 ? $markers[ 0 ] : ""),
-                'raw-content' => (isset($mark_content[ 0 ])) ? $mark_content[ 0 ].$trailingSpaces : '',
-                'ext-succ-tags' => (isset($mark_content[ 1 ])) ? $mark_content[ 1 ] : '',
+                'mid' => (isset($mid[1])) ? $mid[1] : $mi,
+                'ext-prec-tags' => ($mi == 0 ? $markers[0] : ""),
+                'raw-content' => (isset($mark_content[0])) ? $mark_content[0] . $trailingSpaces : '',
+                'ext-succ-tags' => (isset($mark_content[1])) ? $mark_content[1] : '',
             ];
 
             if (!empty($originalData)) {
                 $dataRefMap = $this->getDataRefMap($originalData);
-                $sourceArray['replaced-content'] = (new DataRefReplacer($dataRefMap))->replace($mark_content[ 0 ]);
+                $sourceArray['replaced-content'] = (new DataRefReplacer($dataRefMap))->replace($mark_content[0]);
             }
 
             $source[] = $sourceArray;

src/XliffParser/XliffParserV2.php

Spacing +37 added lines, -37 removed lines

@@ -22,10 +22,10 @@ 
         foreach ($dom->getElementsByTagName('file') as $file) {
 
             // metadata
-            $output[ 'files' ][ $i ][ 'attr' ] = $this->extractMetadata($dom);
+            $output['files'][$i]['attr'] = $this->extractMetadata($dom);
 
             // notes
-            $output[ 'files' ][ $i ]['notes'] = $this->extractNotes($file);
+            $output['files'][$i]['notes'] = $this->extractNotes($file);
 
             // trans-units
             $transUnitIdArrayForUniquenessCheck = [];
@@ -61,16 +61,16 @@ 
         $fileNode = $dom->getElementsByTagName('file')->item(0);
 
         // original
-        $metadata[ 'original' ] = (null !== $fileNode->attributes->getNamedItem('original')) ? $fileNode->attributes->getNamedItem('original')->nodeValue : 'no-name';
+        $metadata['original'] = (null !== $fileNode->attributes->getNamedItem('original')) ? $fileNode->attributes->getNamedItem('original')->nodeValue : 'no-name';
 
         // source-language
-        $metadata[ 'source-language' ] = (null !== $xliffNode->attributes->getNamedItem('srcLang')) ? $xliffNode->attributes->getNamedItem('srcLang')->nodeValue : 'en-US';
+        $metadata['source-language'] = (null !== $xliffNode->attributes->getNamedItem('srcLang')) ? $xliffNode->attributes->getNamedItem('srcLang')->nodeValue : 'en-US';
 
         // datatype
         // @TODO to be implemented
 
         // target-language
-        $metadata[ 'target-language' ] = (null !== $xliffNode->attributes->getNamedItem('trgLang')) ? $xliffNode->attributes->getNamedItem('trgLang')->nodeValue : 'en-US';
+        $metadata['target-language'] = (null !== $xliffNode->attributes->getNamedItem('trgLang')) ? $xliffNode->attributes->getNamedItem('trgLang')->nodeValue : 'en-US';
 
         // custom MateCat x-attribute
         // @TODO to be implemented
@@ -118,16 +118,16 @@ 
     protected function extractTransUnit($transUnit, &$transUnitIdArrayForUniquenessCheck, $dom, &$output, &$i, &$j)
     {
         // metadata
-        $output[ 'files' ][ $i ][ 'trans-units' ][ $j ][ 'attr' ] = $this->extractTransUnitMetadata($transUnit, $transUnitIdArrayForUniquenessCheck);
+        $output['files'][$i]['trans-units'][$j]['attr'] = $this->extractTransUnitMetadata($transUnit, $transUnitIdArrayForUniquenessCheck);
 
         // notes
         // merge <notes> with key and key-note contained in metadata <mda:metaGroup>
-        $output[ 'files' ][ $i ][ 'trans-units' ][ $j ][ 'notes' ] = $this->extractTransUnitNotes($transUnit);
+        $output['files'][$i]['trans-units'][$j]['notes'] = $this->extractTransUnitNotes($transUnit);
 
         // uuid
-        foreach ($output[ 'files' ][ $i ][ 'trans-units' ][ $j ][ 'notes' ] as $note){
-            if(isset($note['raw-content']) and Strings::isAValidUuid($note['raw-content'])){
-                $output[ 'files' ][ $i ][ 'trans-units' ][ $j ][ 'attr' ]['uuid'] = $note['raw-content'];
+        foreach ($output['files'][$i]['trans-units'][$j]['notes'] as $note) {
+            if (isset($note['raw-content']) and Strings::isAValidUuid($note['raw-content'])) {
+                $output['files'][$i]['trans-units'][$j]['attr']['uuid'] = $note['raw-content'];
             }
         }
 
@@ -135,14 +135,14 @@ 
         // http://docs.oasis-open.org/xliff/xliff-core/v2.0/xliff-core-v2.0.html#originaldata
         $originalData = $this->extractTransUnitOriginalData($transUnit);
         if (!empty($originalData)) {
-            $output[ 'files' ][ $i ][ 'trans-units' ][ $j ][ 'original-data' ] = $originalData;
+            $output['files'][$i]['trans-units'][$j]['original-data'] = $originalData;
             $dataRefMap = $this->getDataRefMap($originalData);
         }
 
         // additionalTagData (exclusive for V2)
         $additionalTagData = $this->extractTransUnitAdditionalTagData($transUnit);
         if (!empty($additionalTagData)) {
-            $output[ 'files' ][ $i ][ 'trans-units' ][ $j ][ 'additional-tag-data' ] = $additionalTagData;
+            $output['files'][$i]['trans-units'][$j]['additional-tag-data'] = $additionalTagData;
         }
 
         // content
@@ -166,7 +166,7 @@ 
             if ($segment->nodeName === 'segment') {
 
                 // check segment id consistency
-                $attr = $output[ 'files' ][ $i ][ 'trans-units' ][ $j ][ 'attr' ];
+                $attr = $output['files'][$i]['trans-units'][$j]['attr'];
                 $this->checkSegmentIdConsistency($segment, $attr);
 
                 // loop <segment> to get nested <source> and <target> tag
@@ -191,7 +191,7 @@ 
                                 'mid'           => count($segSource) > 0 ? count($segSource) : 0,
                                 'ext-prec-tags' => '',
                                 'raw-content'   => $extractedSource['raw-content'],
-                                'replaced-content'   => (!empty($originalData)) ?  (new DataRefReplacer($dataRefMap))->replace($extractedSource['raw-content']) : null,
+                                'replaced-content'   => (!empty($originalData)) ? (new DataRefReplacer($dataRefMap))->replace($extractedSource['raw-content']) : null,
                                 'ext-succ-tags' => '',
                             ];
                         }
@@ -217,7 +217,7 @@ 
                                 'mid'           => count($segTarget) > 0 ? count($segTarget) : 0,
                                 'ext-prec-tags' => '',
                                 'raw-content'   => $extractedTarget['raw-content'],
-                                'replaced-content' => (!empty($originalData)) ?  (new DataRefReplacer($dataRefMap))->replace($extractedTarget['raw-content']) : null,
+                                'replaced-content' => (!empty($originalData)) ? (new DataRefReplacer($dataRefMap))->replace($extractedTarget['raw-content']) : null,
                                 'ext-succ-tags' => '',
                             ];
                         }
@@ -228,10 +228,10 @@ 
             }
         }
 
-        $output[ 'files' ][ $i ][ 'trans-units' ][ $j ][ 'source' ] = $source;
-        $output[ 'files' ][ $i ][ 'trans-units' ][ $j ][ 'target' ] = $target;
-        $output[ 'files' ][ $i ][ 'trans-units' ][ $j ][ 'seg-source' ] = $segSource;
-        $output[ 'files' ][ $i ][ 'trans-units' ][ $j ][ 'seg-target' ] = $segTarget;
+        $output['files'][$i]['trans-units'][$j]['source'] = $source;
+        $output['files'][$i]['trans-units'][$j]['target'] = $target;
+        $output['files'][$i]['trans-units'][$j]['seg-source'] = $segSource;
+        $output['files'][$i]['trans-units'][$j]['seg-target'] = $segTarget;
 
         $j++;
     }
@@ -253,31 +253,31 @@ 
 
         $id = $transUnit->attributes->getNamedItem('id')->nodeValue;
 
-        if(strlen($id) > 100){
+        if (strlen($id) > 100) {
             throw new SegmentIdTooLongException('Segment-id too long. Max 100 characters allowed', 400);
         }
 
         $transUnitIdArrayForUniquenessCheck[] = $id;
-        $metadata[ 'id' ] = $id;
+        $metadata['id'] = $id;
 
         // translate
         if (null !== $transUnit->attributes->getNamedItem('translate')) {
-            $metadata[ 'translate' ] = $transUnit->attributes->getNamedItem('translate')->nodeValue;
+            $metadata['translate'] = $transUnit->attributes->getNamedItem('translate')->nodeValue;
         }
 
         // tGroupBegin
         if (null !== $transUnit->attributes->getNamedItem('tGroupBegin')) {
-            $metadata[ 'tGroupBegin' ] = $transUnit->attributes->getNamedItem('tGroupBegin')->nodeValue;
+            $metadata['tGroupBegin'] = $transUnit->attributes->getNamedItem('tGroupBegin')->nodeValue;
         }
 
         // tGroupEnd
         if (null !== $transUnit->attributes->getNamedItem('tGroupEnd')) {
-            $metadata[ 'tGroupEnd' ] = $transUnit->attributes->getNamedItem('tGroupEnd')->nodeValue;
+            $metadata['tGroupEnd'] = $transUnit->attributes->getNamedItem('tGroupEnd')->nodeValue;
         }
 
         // sizeRestriction
-        if (null !== $transUnit->attributes->getNamedItem('sizeRestriction') and  '' !== $transUnit->attributes->getNamedItem('sizeRestriction')->nodeValue ) {
-            $metadata[ 'sizeRestriction' ] = (int)$transUnit->attributes->getNamedItem('sizeRestriction')->nodeValue;
+        if (null !== $transUnit->attributes->getNamedItem('sizeRestriction') and '' !== $transUnit->attributes->getNamedItem('sizeRestriction')->nodeValue) {
+            $metadata['sizeRestriction'] = (int)$transUnit->attributes->getNamedItem('sizeRestriction')->nodeValue;
         }
 
         return $metadata;
@@ -297,11 +297,11 @@ 
         foreach ($transUnit->childNodes as $childNode) {
             if ($childNode->nodeName === 'originalData') {
                 foreach ($childNode->childNodes as $data) {
-                    if (null!== $data->attributes and null !== $data->attributes->getNamedItem('id')) {
+                    if (null !== $data->attributes and null !== $data->attributes->getNamedItem('id')) {
                         $dataId = $data->attributes->getNamedItem('id')->nodeValue;
 
                         $dataValue = str_replace(Placeholder::WHITE_SPACE_PLACEHOLDER, ' ', $data->nodeValue);
-                        $dataValue = str_replace(Placeholder::NEW_LINE_PLACEHOLDER,'\n', $dataValue);
+                        $dataValue = str_replace(Placeholder::NEW_LINE_PLACEHOLDER, '\n', $dataValue);
                         $dataValue = str_replace(Placeholder::TAB_PLACEHOLDER, '\t', $dataValue);
 
                         if ('' !== $dataValue) {
@@ -309,12 +309,12 @@ 
                             $jsonOrRawContentArray = $this->JSONOrRawContentArray($dataValue, false);
 
                             // restore xliff tags
-                            if (isset($jsonOrRawContentArray['json'])){
-                                $jsonOrRawContentArray['json'] = str_replace([Placeholder::LT_PLACEHOLDER, Placeholder::GT_PLACEHOLDER], ['&lt;','&gt;'], $jsonOrRawContentArray['json']);
+                            if (isset($jsonOrRawContentArray['json'])) {
+                                $jsonOrRawContentArray['json'] = str_replace([Placeholder::LT_PLACEHOLDER, Placeholder::GT_PLACEHOLDER], ['&lt;', '&gt;'], $jsonOrRawContentArray['json']);
                             }
 
-                            if (isset($jsonOrRawContentArray['raw-content'])){
-                                $jsonOrRawContentArray['raw-content'] = str_replace([Placeholder::LT_PLACEHOLDER, Placeholder::GT_PLACEHOLDER], ['&lt;','&gt;'], $jsonOrRawContentArray['raw-content']);
+                            if (isset($jsonOrRawContentArray['raw-content'])) {
+                                $jsonOrRawContentArray['raw-content'] = str_replace([Placeholder::LT_PLACEHOLDER, Placeholder::GT_PLACEHOLDER], ['&lt;', '&gt;'], $jsonOrRawContentArray['raw-content']);
                             }
 
                             $originalData[] = array_merge(
@@ -351,7 +351,7 @@ 
 
                     // id
                     if ($data->nodeName === 'memsource:tag') {
-                        if (null!== $data->attributes and null !== $data->attributes->getNamedItem('id')) {
+                        if (null !== $data->attributes and null !== $data->attributes->getNamedItem('id')) {
                             $dataId = $data->attributes->getNamedItem('id')->nodeValue;
                             $dataArray['attr']['id'] = $dataId;
                         }
@@ -386,10 +386,10 @@ 
      */
     private function checkSegmentIdConsistency(\DOMElement $segment, array $attr)
     {
-        if (isset($attr[ 'tGroupBegin' ]) and isset($attr[ 'tGroupEnd' ]) and $segment->attributes->getNamedItem('id')) {
+        if (isset($attr['tGroupBegin']) and isset($attr['tGroupEnd']) and $segment->attributes->getNamedItem('id')) {
             $id = $segment->attributes->getNamedItem('id')->nodeValue;
-            $min = (int)$attr[ 'tGroupBegin' ];
-            $max = (int)$attr[ 'tGroupEnd' ];
+            $min = (int)$attr['tGroupBegin'];
+            $max = (int)$attr['tGroupEnd'];
 
             if (false === (($min <= $id) and ($id <= $max))) {
                 if ($this->logger) {
@@ -424,7 +424,7 @@ 
                 foreach ($childNode->childNodes as $metadata) {
                     if ($metadata->nodeName === 'mda:metaGroup') {
                         foreach ($metadata->childNodes as $meta) {
-                            if (null!== $meta->attributes and null !== $meta->attributes->getNamedItem('type')) {
+                            if (null !== $meta->attributes and null !== $meta->attributes->getNamedItem('type')) {
                                 $type = $meta->attributes->getNamedItem('type')->nodeValue;
                                 $metaValue = trim($meta->nodeValue);
 

src/XliffParser/XliffParserV1.php

Spacing +29 added lines, -29 removed lines

@@ -19,11 +19,11 @@ 
         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);
+                $output['files'][$i]['reference'] = $this->extractReference($file);
             }
 
             // trans-units
@@ -65,36 +65,36 @@ 
             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;
+            if (isset($temp[1])) {
+                $customAttr[$attribute->localName] = $attribute->value;
             }
             unset($temp);
 
             // Custom MateCat namespace Attribute mtc:
             preg_match('|mtc:(.*?)|si', $attribute->nodeName, $temp);
-            if (isset($temp[ 1 ])) {
-                $customAttr[ $attribute->nodeName ] = $attribute->value;
+            if (isset($temp[1])) {
+                $customAttr[$attribute->nodeName] = $attribute->value;
             }
             unset($temp);
 
@@ -120,8 +120,8 @@ 
             /** @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);
+                    $reference[$order]['form-type'] = $childNode->attributes->getNamedItem('form')->nodeValue;
+                    $reference[$order]['base64']    = trim($childNode->nodeValue);
                     $order++;
                 }
             }
@@ -145,51 +145,51 @@ 
     protected function extractTransUnit($transUnit, &$transUnitIdArrayForUniquenessCheck, $dom, &$output, &$i, &$j)
     {
         // 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) {
             // source
             if ($childNode->nodeName === 'source') {
-                $output[ 'files' ][ $i ][ 'trans-units' ][ $j ][ 'source' ] = $this->extractContent($dom, $childNode);
+                $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);
+                $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);
+                $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'];
+                $targetRawContent = @$output['files'][$i]['trans-units'][$j]['target']['raw-content'];
+                $segSource = @$output['files'][$i]['trans-units'][$j]['seg-source'];
                 if (isset($targetRawContent) and !empty($targetRawContent) and isset($segSource) and count($segSource) > 0) {
-                    $output[ 'files' ][ $i ][ 'trans-units' ][ $j ]['seg-target'] = $this->extractContentWithMarksAndExtTags($dom, $childNode, $targetRawContent);
+                    $output['files'][$i]['trans-units'][$j]['seg-target'] = $this->extractContentWithMarksAndExtTags($dom, $childNode, $targetRawContent);
                 }
             }
 
             // locked
             if ($childNode->nodeName === 'sdl:seg') {
-                $output[ 'files' ][ $i ][ 'trans-units' ][ $j ][ 'locked' ] = $this->extractLocked($childNode);
+                $output['files'][$i]['trans-units'][$j]['locked'] = $this->extractLocked($childNode);
             }
         }
 
         // context-group
         foreach ($transUnit->getElementsByTagName('context-group') as $contextGroup) {
-            $output[ 'files' ][ $i ][ 'trans-units' ][ $j ][ 'context-group' ][] = $this->extractTransUnitContextGroup($dom, $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);
+            $output['files'][$i]['trans-units'][$j]['alt-trans'][] = $this->extractTransUnitAltTrans($altTrans);
         }
 
         $j++;
@@ -213,22 +213,22 @@ 
 
         $id = $transUnit->attributes->getNamedItem('id')->nodeValue;
 
-        if(strlen($id) > 100){
+        if (strlen($id) > 100) {
             throw new SegmentIdTooLongException('Segment-id too long. Max 100 characters allowed', 400);
         }
 
         $transUnitIdArrayForUniquenessCheck[] = $id;
-        $metadata[ 'id' ] = $id;
+        $metadata['id'] = $id;
 
         // translate
         if (null !== $transUnit->attributes->getNamedItem('translate')) {
-            $metadata[ 'translate' ] = $transUnit->attributes->getNamedItem('translate')->nodeValue;
+            $metadata['translate'] = $transUnit->attributes->getNamedItem('translate')->nodeValue;
         }
 
         // approved
         // http://docs.oasis-open.org/xliff/v1.2/os/xliff-core.html#approved
         if (null !== $transUnit->attributes->getNamedItem('approved')) {
-            $metadata[ 'approved' ] = filter_var($transUnit->attributes->getNamedItem('approved')->nodeValue, FILTER_VALIDATE_BOOLEAN);
+            $metadata['approved'] = filter_var($transUnit->attributes->getNamedItem('approved')->nodeValue, FILTER_VALIDATE_BOOLEAN);
         }
 
         return $metadata;
@@ -252,7 +252,7 @@ 
                 $extractedNote = $this->JSONOrRawContentArray($noteValue);
 
                 // extract all the attributes
-                foreach ($note->attributes as $attribute){
+                foreach ($note->attributes as $attribute) {
                     $extractedNote[$attribute->name] = $attribute->value;
                 }
 

src/XliffReplacer/XliffSAXTranslationReplacer.php

Spacing +103 added lines, -103 removed lines

@@ -17,16 +17,16 @@ 
      */
     private $nodesToCopy = [
         'source',
-        'mda:metadata' ,
-        'memsource:additionalTagData' ,
-        'originalData' ,
-        'seg-source' ,
-        'value' ,
-        'bpt' ,
-        'ept' ,
-        'ph' ,
-        'st' ,
-        'note' ,
+        'mda:metadata',
+        'memsource:additionalTagData',
+        'originalData',
+        'seg-source',
+        'value',
+        'bpt',
+        'ept',
+        'ph',
+        'st',
+        'note',
         'context',
     ];
 
@@ -44,7 +44,7 @@ 
             // obfuscate entities because sax automatically does html_entity_decode
             $temporary_check_buffer = preg_replace("/&(.*?);/", self::$INTERNAL_TAG_PLACEHOLDER . '$1' . self::$INTERNAL_TAG_PLACEHOLDER, $this->currentBuffer);
 
-            $lastByte = $temporary_check_buffer[ strlen($temporary_check_buffer) - 1 ];
+            $lastByte = $temporary_check_buffer[strlen($temporary_check_buffer) - 1];
 
             //avoid cutting entities in half:
             //the last fread could have truncated an entity (say, '<' in '&l'), thus invalidating the escaping
@@ -66,7 +66,7 @@ 
                 }
 
                 //if an entity is still present, fetch some more and repeat the escaping
-                $this->currentBuffer    .= fread($this->originalFP, 9);
+                $this->currentBuffer .= fread($this->originalFP, 9);
                 $temporary_check_buffer = preg_replace("/&(.*?);/", self::$INTERNAL_TAG_PLACEHOLDER . '$1' . self::$INTERNAL_TAG_PLACEHOLDER, $this->currentBuffer);
             }
 
@@ -110,31 +110,31 @@ 
 
             // 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);
 
             // `translate` attribute can be only yes or no
-            if(isset($attr[ 'translate' ]) and $attr[ 'translate' ] === 'no'){
-                $attr[ 'translate' ] = 'no';
+            if (isset($attr['translate']) and $attr['translate'] === 'no') {
+                $attr['translate'] = 'no';
             } else {
-                $attr[ 'translate' ] = 'yes';
+                $attr['translate'] = 'yes';
             }
 
             // current 'translate' attribute of the current trans-unit
-            $this->currentTransUnitTranslate = isset($attr[ 'translate' ]) ? $attr[ 'translate' ] : 'yes';
+            $this->currentTransUnitTranslate = isset($attr['translate']) ? $attr['translate'] : 'yes';
         }
 
-        if('source' === $name){
+        if ('source' === $name) {
             $this->sourceAttributes = $attr;
         }
 
-        if('mda:metadata' === $name){
+        if ('mda:metadata' === $name) {
             $this->unitContainsMda = true;
         }
 
         // check if we are entering into a <target>
         if ('target' === $name) {
 
-            if($this->currentTransUnitTranslate === 'no'){
+            if ($this->currentTransUnitTranslate === 'no') {
                 $this->inTarget = false;
             } else {
                 $this->inTarget = true;
@@ -165,14 +165,14 @@ 
             //
             // http://docs.oasis-open.org/xliff/xliff-core/v2.0/os/xliff-core-v2.0-os.html#unit
             //
-            if($this->xliffVersion === 2 and ( $name === 'notes' or $name === 'originalData' or $name === 'segment' or $name === 'ignorable') and $this->unitContainsMda === false ){
-                if (isset($this->transUnits[ $this->currentTransUnitId ]) and !empty($this->transUnits[ $this->currentTransUnitId ]) and !$this->unitContainsMda and !$this->hasWrittenCounts) {
+            if ($this->xliffVersion === 2 and ($name === 'notes' or $name === 'originalData' or $name === 'segment' or $name === 'ignorable') and $this->unitContainsMda === false) {
+                if (isset($this->transUnits[$this->currentTransUnitId]) and !empty($this->transUnits[$this->currentTransUnitId]) and !$this->unitContainsMda and !$this->hasWrittenCounts) {
 
                     // we need to update counts here
                     $this->updateCounts();
                     $this->hasWrittenCounts = true;
 
-                    $tag .= $this->getWordCountGroupForXliffV2($this->counts[ 'raw_word_count' ], $this->counts[ 'eq_word_count' ]);
+                    $tag .= $this->getWordCountGroupForXliffV2($this->counts['raw_word_count'], $this->counts['eq_word_count']);
                     $this->unitContainsMda = true;
                 }
             }
@@ -181,7 +181,7 @@ 
             $tag .= "<$name ";
 
             $lastMrkState = null;
-            $stateProp   = '';
+            $stateProp = '';
 
             foreach ($attr as $k => $v) {
 
@@ -192,27 +192,27 @@ 
                 } else {
                     $pos = 0;
                     if ($this->currentTransUnitId) {
-                        $pos = current($this->transUnits[ $this->currentTransUnitId ]);
+                        $pos = current($this->transUnits[$this->currentTransUnitId]);
                     }
 
                     if ($name === $this->tuTagName) {
 
-                        $sid = $this->segments[ $pos ][ 'sid' ];
+                        $sid = $this->segments[$pos]['sid'];
 
                         // add `help-id` to xliff v.1*
                         // add `mtc:segment-id` to xliff v.2*
-                        if($this->xliffVersion === 1 and strpos($tag, 'help-id') === false){
+                        if ($this->xliffVersion === 1 and strpos($tag, 'help-id') === false) {
                             if (!empty($sid)) {
                                 $tag .= "help-id=\"$sid\" ";
                             }
-                        } elseif($this->xliffVersion === 2 and strpos($tag, 'mtc:segment-id') === false) {
+                        } elseif ($this->xliffVersion === 2 and strpos($tag, 'mtc:segment-id') === false) {
                             if (!empty($sid)) {
                                 $tag .= "mtc:segment-id=\"$sid\" ";
                             }
                         }
 
                     } elseif ('segment' === $name and $this->xliffVersion === 2) { // add state to segment in Xliff v2
-                        list($stateProp, $lastMrkState) = $this->setTransUnitState($this->segments[ $pos ], $stateProp, $lastMrkState);
+                        list($stateProp, $lastMrkState) = $this->setTransUnitState($this->segments[$pos], $stateProp, $lastMrkState);
                     }
 
                     //normal tag flux, put attributes in it
@@ -238,7 +238,7 @@ 
 
             // trgLang
             if ($name === 'xliff') {
-                $tag = preg_replace('/trgLang="(.*?)"/', 'trgLang="'.$this->targetLang.'"', $tag);
+                $tag = preg_replace('/trgLang="(.*?)"/', 'trgLang="' . $this->targetLang . '"', $tag);
             }
 
             //this logic helps detecting empty tags
@@ -250,15 +250,15 @@ 
             //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[ $idx - $this->offset ];
+                $lastChar   = $this->currentBuffer[$idx - $this->offset];
             } else {
                 //if it's out, simple use the last character of the chunk
                 $tmp_offset = $this->len - 1;
-                $lastChar   = $this->currentBuffer[ $this->len - 1 ];
+                $lastChar   = $this->currentBuffer[$this->len - 1];
             }
 
             //trim last space
@@ -283,7 +283,7 @@ 
                 } 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);
@@ -292,11 +292,11 @@ 
 
         // update segmentPositionInTu
 
-        if($this->xliffVersion === 1 and $this->inTU and $name === 'source'){
+        if ($this->xliffVersion === 1 and $this->inTU and $name === 'source') {
             $this->segmentPositionInTu++;
         }
 
-        if($this->xliffVersion === 2 and $this->inTU and $name === 'segment'){
+        if ($this->xliffVersion === 2 and $this->inTU and $name === 'segment') {
             $this->segmentPositionInTu++;
         }
     }
@@ -322,17 +322,17 @@ 
 
             if ('target' == $name) {
 
-                if($this->currentTransUnitTranslate === 'no') {
+                if ($this->currentTransUnitTranslate === 'no') {
                     // do nothing
-                } elseif (isset($this->transUnits[ $this->currentTransUnitId ])) {
+                } elseif (isset($this->transUnits[$this->currentTransUnitId])) {
 
                     // get translation of current segment, by indirect indexing: id -> positional index -> segment
                     // actually there may be more that one segment to that ID if there are two mrk of the same source segment
 
-                    $listOfSegmentsIds = $this->transUnits[ $this->currentTransUnitId ];
+                    $listOfSegmentsIds = $this->transUnits[$this->currentTransUnitId];
 
                     // $currentSegmentId
-                    if(!empty($listOfSegmentsIds)){
+                    if (!empty($listOfSegmentsIds)) {
                         $this->setCurrentSegmentArray($listOfSegmentsIds);
                     }
 
@@ -352,11 +352,11 @@ 
                     $warning    = false;
                     $last_value = null;
                     for ($i = 0; $i < count($listOfSegmentsIds); $i++) {
-                        if (isset($listOfSegmentsIds[ $i ])) {
-                            $id = $listOfSegmentsIds[ $i ];
-                            if (isset($this->segments[ $id ]) and ($i == 0 or $last_value + 1 == $listOfSegmentsIds[ $i ])) {
-                                $last_value            = $listOfSegmentsIds[ $i ];
-                                $this->lastTransUnit[] = $this->segments[ $id ];
+                        if (isset($listOfSegmentsIds[$i])) {
+                            $id = $listOfSegmentsIds[$i];
+                            if (isset($this->segments[$id]) and ($i == 0 or $last_value + 1 == $listOfSegmentsIds[$i])) {
+                                $last_value            = $listOfSegmentsIds[$i];
+                                $this->lastTransUnit[] = $this->segments[$id];
                             }
                         } else {
                             $warning = true;
@@ -365,31 +365,31 @@ 
 
                     if ($warning) {
                         if (null !== $this->logger) {
-                            $this->logger->warning("WARNING: PHP Notice polling. CurrentId: '" . $this->currentTransUnitId . "' - Filename: '" . $this->segments[ 0 ][ 'filename' ] . "' - First Segment: '" . $this->segments[
-                                    0 ][ 'sid' ] . "'");
+                            $this->logger->warning("WARNING: PHP Notice polling. CurrentId: '" . $this->currentTransUnitId . "' - Filename: '" . $this->segments[0]['filename'] . "' - First Segment: '" . $this->segments[
+                                    0]['sid'] . "'");
                         }
                     }
 
                     // init translation and state
                     $translation  = '';
                     $lastMrkState = null;
-                    $stateProp   = '';
+                    $stateProp = '';
 
                     // we must reset the lastMrkId found because this is a new segment.
-                    $lastMrkId      = -1;
+                    $lastMrkId = -1;
 
                     if ($this->xliffVersion === 2) {
-                        $seg = $this->segments[ $this->currentSegmentArray['sid'] ];
+                        $seg = $this->segments[$this->currentSegmentArray['sid']];
 
                         // update counts
-                        if(!$this->hasWrittenCounts and !empty($seg)){
+                        if (!$this->hasWrittenCounts and !empty($seg)) {
                             $this->updateSegmentCounts($seg);
                         }
 
                         // delete translations so the prepareSegment
                         // will put source content in target tag
                         if ($this->sourceInTarget) {
-                            $seg[ 'translation' ] = '';
+                            $seg['translation'] = '';
                             $this->resetCounts();
                         }
 
@@ -411,8 +411,8 @@ 
                              * pre-assign zero to the new mrk if this is the first one ( in this segment )
                              * If it is null leave it NULL
                              */
-                            if ((int)$this->segments[ $id ][ "mrk_id" ] < 0 and $this->segments[ $id ][ "mrk_id" ] !== null) {
-                                $this->segments[ $id ][ "mrk_id" ] = 0;
+                            if ((int)$this->segments[$id]["mrk_id"] < 0 and $this->segments[$id]["mrk_id"] !== null) {
+                                $this->segments[$id]["mrk_id"] = 0;
                             }
 
                             /*
@@ -421,22 +421,22 @@ 
                              * ( null <= -1 ) === true
                              * so, cast to int
                              */
-                            if ((int)$this->segments[ $id ][ "mrk_id" ] <= $lastMrkId) {
+                            if ((int)$this->segments[$id]["mrk_id"] <= $lastMrkId) {
                                 break;
                             }
 
                             // set $this->currentSegment
-                            $seg = $this->segments[ $id ];
+                            $seg = $this->segments[$id];
 
                             // update counts
-                            if(!empty($seg)){
+                            if (!empty($seg)) {
                                 $this->updateSegmentCounts($seg);
                             }
 
                             // delete translations so the prepareSegment
                             // will put source content in target tag
                             if ($this->sourceInTarget) {
-                                $seg[ 'translation' ] = '';
+                                $seg['translation'] = '';
                                 $this->resetCounts();
                             }
 
@@ -445,9 +445,9 @@ 
 
                             // for xliff 2 we need $this->transUnits[ $this->currentId ] [ $pos ] for populating metadata
 
-                            unset($this->transUnits[ $this->currentTransUnitId ] [ $pos ]);
+                            unset($this->transUnits[$this->currentTransUnitId] [$pos]);
 
-                            $lastMrkId = $this->segments[ $id ][ "mrk_id" ];
+                            $lastMrkId = $this->segments[$id]["mrk_id"];
 
                             list($stateProp, $lastMrkState) = $this->setTransUnitState($seg, $stateProp, $lastMrkState);
                         }
@@ -456,10 +456,10 @@ 
                     //append translation
                     $targetLang = '';
                     if ($this->xliffVersion === 1) {
-                        $targetLang = ' xml:lang="'.$this->targetLang.'"';
+                        $targetLang = ' xml:lang="' . $this->targetLang . '"';
                     }
 
-                    $tag = $this->buildTranslateTag($targetLang, $stateProp, $translation, $this->counts[ 'raw_word_count' ], $this->counts[ 'eq_word_count' ]);
+                    $tag = $this->buildTranslateTag($targetLang, $stateProp, $translation, $this->counts['raw_word_count'], $this->counts['eq_word_count']);
                 }
 
                 // signal we are leaving a target
@@ -472,21 +472,21 @@ 
 
                 // only for Xliff 2.*
                 // write here <mda:metaGroup> and <mda:meta> if already present in the <unit>
-                if('mda:metadata' === $name and $this->unitContainsMda and $this->xliffVersion === 2 and !$this->hasWrittenCounts){
+                if ('mda:metadata' === $name and $this->unitContainsMda and $this->xliffVersion === 2 and !$this->hasWrittenCounts) {
 
                     // we need to update counts here
                     $this->updateCounts();
                     $this->hasWrittenCounts = true;
 
                     $tag = $this->CDATABuffer;
-                    $tag .= $this->getWordCountGroupForXliffV2($this->counts[ 'raw_word_count' ], $this->counts[ 'eq_word_count' ], false);
+                    $tag .= $this->getWordCountGroupForXliffV2($this->counts['raw_word_count'], $this->counts['eq_word_count'], false);
                     $tag .= "    </mda:metadata>";
 
                 } else {
-                    $tag = $this->CDATABuffer."</$name>";
+                    $tag = $this->CDATABuffer . "</$name>";
                 }
 
-                $this->CDATABuffer    = "";
+                $this->CDATABuffer = "";
 
                 //flush to pointer
                 $this->postProcAndFlush($this->outputFP, $tag);
@@ -494,7 +494,7 @@ 
 
                 // only for Xliff 2.*
                 // if segment has no <target> add it BEFORE </segment>
-                if($this->xliffVersion === 2 and !$this->targetWasWritten){
+                if ($this->xliffVersion === 2 and !$this->targetWasWritten) {
 
                     $seg = $this->getCurrentSegment();
 
@@ -504,7 +504,7 @@ 
                         $tag .= " $k=\"$v\"";
                     }
 
-                    $tag .= '>'.$seg['translation'].'</target></segment>';
+                    $tag .= '>' . $seg['translation'] . '</target></segment>';
                 }
 
                 $this->postProcAndFlush($this->outputFP, $tag);
@@ -512,18 +512,18 @@ 
                 // we are leaving <segment>, reset $segmentHasTarget
                 $this->targetWasWritten = false;
 
-            } elseif($name === 'trans-unit') {
+            } elseif ($name === 'trans-unit') {
 
                 // only for Xliff 1.*
                 // handling </trans-unit> closure
-                if(!$this->targetWasWritten){
+                if (!$this->targetWasWritten) {
                     $seg = $this->getCurrentSegment();
                     $lastMrkState = null;
-                    $stateProp   = '';
+                    $stateProp = '';
                     $tag = '';
 
                     // if there is translation available insert <target> BEFORE </trans-unit>
-                    if(isset($seg['translation'])){
+                    if (isset($seg['translation'])) {
                         list($stateProp, $lastMrkState) = $this->setTransUnitState($seg, $stateProp, $lastMrkState);
                         $tag .= $this->createTargetTag($seg['translation'], $stateProp);
                     }
@@ -577,7 +577,7 @@ 
         } else {
             if ($this->currentSegmentArray['tid'] === $this->currentTransUnitId) {
                 $key = array_search($this->currentSegmentArray['sid'], $listOfSegmentsIds);
-                $this->currentSegmentArray['sid'] = $listOfSegmentsIds[$key+1];
+                $this->currentSegmentArray['sid'] = $listOfSegmentsIds[$key + 1];
                 $this->currentSegmentArray['tid'] = $this->currentTransUnitId;
             } else {
                 $this->currentSegmentArray = [
@@ -594,23 +594,23 @@ 
     private function updateCounts()
     {
         // populate counts
-        $listOfSegmentsIds = $this->transUnits[ $this->currentTransUnitId ];
+        $listOfSegmentsIds = $this->transUnits[$this->currentTransUnitId];
 
         // $currentSegmentId
-        if(!empty($listOfSegmentsIds)){
+        if (!empty($listOfSegmentsIds)) {
             $this->setCurrentSegmentArray($listOfSegmentsIds);
         }
 
         if ($this->xliffVersion === 2) {
-            $seg = $this->segments[ $this->currentSegmentArray[ 'sid' ] ];
-            if(!empty($seg)){
+            $seg = $this->segments[$this->currentSegmentArray['sid']];
+            if (!empty($seg)) {
                 $this->updateSegmentCounts($seg);
             }
         } else {
             foreach ($listOfSegmentsIds as $pos => $id) {
-                $seg = $this->segments[ $id ];
-                if(!empty($seg)){
-                    $this->updateSegmentCounts( $seg );
+                $seg = $this->segments[$id];
+                if (!empty($seg)) {
+                    $this->updateSegmentCounts($seg);
                 }
             }
         }
@@ -623,14 +623,14 @@ 
      */
     private function updateSegmentCounts(array $seg = [])
     {
-        $this->counts[ 'raw_word_count' ] += $seg['raw_word_count'];
-        $this->counts[ 'eq_word_count' ] += (floor($seg[ 'eq_word_count' ] * 100) / 100);
+        $this->counts['raw_word_count'] += $seg['raw_word_count'];
+        $this->counts['eq_word_count'] += (floor($seg['eq_word_count'] * 100) / 100);
     }
 
     private function resetCounts()
     {
-        $this->counts[ 'raw_word_count' ] = 0;
-        $this->counts[ 'eq_word_count' ] = 0;
+        $this->counts['raw_word_count'] = 0;
+        $this->counts['eq_word_count'] = 0;
     }
 
     /**
@@ -645,11 +645,11 @@ 
     {
         $endTags = "";
 
-        $segment     = Strings::removeDangerousChars($seg [ 'segment' ]);
-        $translation = Strings::removeDangerousChars($seg [ 'translation' ]);
+        $segment     = Strings::removeDangerousChars($seg ['segment']);
+        $translation = Strings::removeDangerousChars($seg ['translation']);
         $dataRefMap  = (isset($seg['data_ref_map']) and $seg['data_ref_map'] !== null) ? Strings::jsonToArray($seg['data_ref_map']) : [];
 
-        if (is_null($seg [ 'translation' ]) or $seg [ 'translation' ] == '') {
+        if (is_null($seg ['translation']) or $seg ['translation'] == '') {
             $translation = $segment;
         } else {
             if ($this->callback) {
@@ -664,15 +664,15 @@ 
             return $translation;
         }
 
-        if ($seg[ 'mrk_id' ] !== null and $seg[ 'mrk_id' ] != '') {
+        if ($seg['mrk_id'] !== null and $seg['mrk_id'] != '') {
             if ($this->targetLang === 'ja-JP') {
-                $seg[ 'mrk_succ_tags' ] = ltrim($seg[ 'mrk_succ_tags' ]);
+                $seg['mrk_succ_tags'] = ltrim($seg['mrk_succ_tags']);
             }
 
-            $translation = "<mrk mid=\"" . $seg[ 'mrk_id' ] . "\" mtype=\"seg\">" . $seg[ 'mrk_prev_tags' ] . $translation . $seg[ 'mrk_succ_tags' ] . "</mrk>";
+            $translation = "<mrk mid=\"" . $seg['mrk_id'] . "\" mtype=\"seg\">" . $seg['mrk_prev_tags'] . $translation . $seg['mrk_succ_tags'] . "</mrk>";
         }
 
-        $transUnitTranslation .= $seg[ 'prev_tags' ] . $translation . $endTags . $seg[ 'succ_tags' ];
+        $transUnitTranslation .= $seg['prev_tags'] . $translation . $endTags . $seg['succ_tags'];
 
         return $transUnitTranslation;
     }
@@ -694,7 +694,7 @@ 
                 $tag = "<target $targetLang $stateProp>$translation</target>";
 
                 // if it's a Trados file don't append count group
-                if(get_class($this) !== SdlXliffSAXTranslationReplacer::class){
+                if (get_class($this) !== SdlXliffSAXTranslationReplacer::class) {
                     $tag .= $this->getWordCountGroup($rawWordCount, $eqWordCount);
                 }
 
@@ -721,10 +721,10 @@ 
      */
     private function getCurrentSegment()
     {
-        if($this->currentTransUnitTranslate === 'yes' and isset($this->transUnits[$this->currentTransUnitId])){
+        if ($this->currentTransUnitTranslate === 'yes' and isset($this->transUnits[$this->currentTransUnitId])) {
             $index = $this->transUnits[$this->currentTransUnitId][$this->segmentPositionInTu];
 
-            if(isset($this->segments[$index])){
+            if (isset($this->segments[$index])) {
                 return $this->segments[$index];
             }
         }
@@ -742,9 +742,9 @@ 
      */
     private function createTargetTag($translation, $stateProp)
     {
-        $targetLang = 'xml:lang="'.$this->targetLang.'"';
+        $targetLang = 'xml:lang="' . $this->targetLang . '"';
 
-        return $this->buildTranslateTag($targetLang, $stateProp, $translation, $this->counts[ 'raw_word_count' ], $this->counts[ 'eq_word_count' ]);
+        return $this->buildTranslateTag($targetLang, $stateProp, $translation, $this->counts['raw_word_count'], $this->counts['eq_word_count']);
     }
 
     /**
@@ -757,10 +757,10 @@ 
     private function getWordCountGroupForXliffV2($raw_word_count, $eq_word_count, $withMetadataTag = true)
     {
         $this->mdaGroupCounter++;
-        $id = 'word_count_tu_'. $this->mdaGroupCounter;
+        $id = 'word_count_tu_' . $this->mdaGroupCounter;
 
-        if($withMetadataTag === false){
-            return "    <mda:metaGroup id=\"".$id."\" category=\"row_xml_attribute\">
+        if ($withMetadataTag === false) {
+            return "    <mda:metaGroup id=\"" . $id . "\" category=\"row_xml_attribute\">
                                 <mda:meta type=\"x-matecat-raw\">$raw_word_count</mda:meta>
                                 <mda:meta type=\"x-matecat-weighted\">$eq_word_count</mda:meta>
                             </mda:metaGroup>
@@ -768,7 +768,7 @@ 
         }
 
         return "<mda:metadata>
-                <mda:metaGroup id=\"".$id."\" category=\"row_xml_attribute\">
+                <mda:metaGroup id=\"".$id . "\" category=\"row_xml_attribute\">
                     <mda:meta type=\"x-matecat-raw\">$raw_word_count</mda:meta>
                     <mda:meta type=\"x-matecat-weighted\">$eq_word_count</mda:meta>
                 </mda:metaGroup>
@@ -784,12 +784,12 @@ 
      */
     private function setTransUnitState($seg, $state_prop, $lastMrkState)
     {
-        switch ($seg[ 'status' ]) {
+        switch ($seg['status']) {
 
             case TranslationStatus::STATUS_FIXED:
             case TranslationStatus::STATUS_APPROVED:
                 if ($lastMrkState == null or $lastMrkState == TranslationStatus::STATUS_APPROVED) {
-                    $state_prop   = ($this->xliffVersion === 2) ? "state=\"reviewed\"": "state=\"signed-off\"";
+                    $state_prop   = ($this->xliffVersion === 2) ? "state=\"reviewed\"" : "state=\"signed-off\"";
                     $lastMrkState = TranslationStatus::STATUS_APPROVED;
                 }
                 break;
@@ -832,7 +832,7 @@ 
                 break;
         }
 
-        return [ $state_prop, $lastMrkState ];
+        return [$state_prop, $lastMrkState];
     }
 
     /**

src/XliffReplacer/SdlXliffSAXTranslationReplacer.php

Spacing +19 added lines, -19 removed lines

@@ -19,15 +19,15 @@ 
 
             // 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 ($this->currentTransUnitTranslate === 'no') {
                 $this->inTarget = false;
             } else {
                 $this->inTarget = true;
@@ -64,10 +64,10 @@ 
                 } elseif ('sdl:seg' == $name) {
 
                     // write the confidence level for this segment ( Translated, Draft, etc. )
-                    if (isset($this->segments[ 'matecat|' . $this->currentTransUnitId ]) and $_sdlStatus_confWritten == false) {
+                    if (isset($this->segments['matecat|' . $this->currentTransUnitId]) and $_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++;
@@ -94,15 +94,15 @@ 
             // 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[ $idx - $this->offset ];
+                $lastChar   = $this->currentBuffer[$idx - $this->offset];
             } else {
                 //if it's out, simple use the last character of the chunk
                 $tmp_offset = $this->len - 1;
-                $lastChar   = $this->currentBuffer[ $this->len - 1 ];
+                $lastChar   = $this->currentBuffer[$this->len - 1];
             }
 
             // trim last space
@@ -136,7 +136,7 @@ 
                 } 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);
@@ -156,11 +156,11 @@ 
     {
         $endTags = "";
 
-        $segment     = Strings::removeDangerousChars($seg [ 'segment' ]);
-        $translation = Strings::removeDangerousChars($seg [ 'translation' ]);
+        $segment     = Strings::removeDangerousChars($seg ['segment']);
+        $translation = Strings::removeDangerousChars($seg ['translation']);
         $dataRefMap  = (isset($seg['data_ref_map']) and $seg['data_ref_map'] !== null) ? Strings::jsonToArray($seg['data_ref_map']) : [];
 
-        if (is_null($seg [ 'translation' ]) or $seg [ 'translation' ] == '') {
+        if (is_null($seg ['translation']) or $seg ['translation'] == '') {
             $translation = $segment;
         } else {
             if ($this->callback) {
@@ -173,20 +173,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 and $seg[ 'mrk_id' ] != '') {
+        if ($seg['mrk_id'] !== null and $seg['mrk_id'] != '') {
             if ($this->targetLang === 'ja-JP') {
-                $seg[ 'mrk_succ_tags' ] = ltrim($seg[ 'mrk_succ_tags' ]);
+                $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;
     }
@@ -206,7 +206,7 @@ 
             'REJECTED'   => 'RejectedTranslation',
         ];
 
-        return "conf=\"{$statusMap[ $segment[ 'status' ] ]}\" ";
+        return "conf=\"{$statusMap[$segment['status']]}\" ";
     }
 
     /**
@@ -218,7 +218,7 @@ 
      */
     protected function setTransUnitState($seg, $state_prop, $lastMrkState)
     {
-        return [ null, null ];
+        return [null, null];
     }
 
     /**

src/XliffReplacer/XliffReplacerFactory.php

Spacing +1 added lines, -1 removed lines

@@ -26,7 +26,7 @@
     {
         $info = XliffProprietaryDetect::getInfo($originalXliffPath);
 
-        if ($info[ 'proprietary_short_name' ] !== 'trados') {
+        if ($info['proprietary_short_name'] !== 'trados') {
             return new XliffSAXTranslationReplacer($originalXliffPath, $info['version'], $data, $transUnits, $targetLang, $outputFilePath, $setSourceInTarget, $logger, $callback);
         }
 

src/XliffReplacer/AbstractXliffReplacer.php

Spacing +22 added lines, -22 removed lines

@@ -8,28 +8,28 @@ 
 {
     protected $originalFP;
 
-    protected $tuTagName;                 // <trans-unit> (forXliff v 1.*) or <unit> (forXliff v 2.*)
-    protected $inTU                = false;  // flag to check whether we are in a <trans-unit>
-    protected $inTarget            = false;  // flag to check whether we are in a <target>, to ignore everything
-    protected $isEmpty             = false;  // flag to check whether we are in an empty tag (<tag/>)
-    protected $targetWasWritten    = false;  // flag to check is <target> was written in the current unit
-    protected $segmentPositionInTu = -1;  // the current position of segment in the current <unit> (forXliff v 2.*)
-
-    protected $CDATABuffer    = "";       // buffer for special tag
-    protected $bufferIsActive = false;    // buffer for special tag
-
-    protected $offset        = 0;         // offset for SAX pointer
-    protected $outputFP;                  // output stream pointer
-    protected $currentBuffer;             // the current piece of text it's been parsed
-    protected $len;                       // length of the currentBuffer
-    protected $segments;                  // array of translations
+    protected $tuTagName; // <trans-unit> (forXliff v 1.*) or <unit> (forXliff v 2.*)
+    protected $inTU                = false; // flag to check whether we are in a <trans-unit>
+    protected $inTarget            = false; // flag to check whether we are in a <target>, to ignore everything
+    protected $isEmpty             = false; // flag to check whether we are in an empty tag (<tag/>)
+    protected $targetWasWritten    = false; // flag to check is <target> was written in the current unit
+    protected $segmentPositionInTu = -1; // the current position of segment in the current <unit> (forXliff v 2.*)
+
+    protected $CDATABuffer    = ""; // buffer for special tag
+    protected $bufferIsActive = false; // buffer for special tag
+
+    protected $offset = 0; // offset for SAX pointer
+    protected $outputFP; // output stream pointer
+    protected $currentBuffer; // the current piece of text it's been parsed
+    protected $len; // length of the currentBuffer
+    protected $segments; // array of translations
     protected $lastTransUnit = [];
-    protected $currentTransUnitId;        // id of current <trans-unit>
+    protected $currentTransUnitId; // id of current <trans-unit>
     protected $currentTransUnitTranslate; // 'translate' attribute of current <trans-unit>
-    protected $currentSegmentArray = [];  // id of current <segment> (forXliff v 2.*)
-    protected $unitContainsMda = false;   // check if <unit> already contains a <mda:metadata> (forXliff v 2.*)
-    protected $hasWrittenCounts = false;  // check if <unit> already wrote segment counts (forXliff v 2.*)
-    protected $sourceAttributes = [];     // current <source> attributes (needed when handling xliff files without <target>)
+    protected $currentSegmentArray = []; // id of current <segment> (forXliff v 2.*)
+    protected $unitContainsMda = false; // check if <unit> already contains a <mda:metadata> (forXliff v 2.*)
+    protected $hasWrittenCounts = false; // check if <unit> already wrote segment counts (forXliff v 2.*)
+    protected $sourceAttributes = []; // current <source> attributes (needed when handling xliff files without <target>)
 
     protected $targetLang;
 
@@ -95,7 +95,7 @@ 
         return "§" .
                 substr(
                     str_replace(
-                        [ '+', '/' ],
+                        ['+', '/'],
                         '',
                         base64_encode(openssl_random_pseudo_bytes(10, $_crypto_strong))
                     ),
@@ -134,7 +134,7 @@ 
      */
     private function setTuTagName()
     {
-        $this->tuTagName = ($this->xliffVersion === 2) ? 'unit': 'trans-unit';
+        $this->tuTagName = ($this->xliffVersion === 2) ? 'unit' : 'trans-unit';
     }
 
     /**

src/Constants/TranslationStatus.php

Spacing +1 added lines, -1 removed lines

@@ -24,7 +24,7 @@
             self::STATUS_REJECTED   => 5,
             self::STATUS_FIXED      => 6,
             self::STATUS_REBUTTED   => 7
-    ] ;
+    ];
 
     public static $STATUSES = array(
             self::STATUS_NEW,

src/Constants/XliffTags.php

Spacing +1 added lines, -1 removed lines

@@ -7,5 +7,5 @@
     /**
      * List of allowed Xliff tags
      */
-    public static $tags = [ 'g', 'x', 'bx', 'ex', 'bpt', 'ept', 'ph', 'pc', 'ec', 'sc', 'it', 'mrk' ];
+    public static $tags = ['g', 'x', 'bx', 'ex', 'bpt', 'ept', 'ph', 'pc', 'ec', 'sc', 'it', 'mrk'];
 }
\ No newline at end of file