matecat / xliff-parser

src/XliffParser/XliffParserV2.php

Spacing +46 added lines, -46 removed lines

@@ -17,17 +17,17 @@ 
      * @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) {
 
             // 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 = [];
@@ -55,7 +55,7 @@ 
      *
      * @return array
      */
-    private function extractMetadata( DOMDocument $dom)
+    private function extractMetadata(DOMDocument $dom)
     {
         $metadata = [];
 
@@ -63,16 +63,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
@@ -86,7 +86,7 @@ 
      * @return array
      * @throws \Exception
      */
-    private function extractNotes( DOMElement $file)
+    private function extractNotes(DOMElement $file)
     {
         $notes = [];
 
@@ -120,16 +120,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']) && 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']) && Strings::isAValidUuid($note['raw-content'])) {
+                $output['files'][$i]['trans-units'][$j]['attr']['uuid'] = $note['raw-content'];
             }
         }
 
@@ -137,14 +137,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
@@ -168,7 +168,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
@@ -193,7 +193,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' => '',
                             ];
                         }
@@ -219,7 +219,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' => '',
                             ];
                         }
@@ -230,10 +230,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++;
     }
@@ -244,7 +244,7 @@ 
      *
      * @return array
      */
-    private function extractTransUnitMetadata( DOMElement $transUnit, &$transUnitIdArrayForUniquenessCheck)
+    private function extractTransUnitMetadata(DOMElement $transUnit, &$transUnitIdArrayForUniquenessCheck)
     {
         $metadata = [];
 
@@ -255,31 +255,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') &&  '' !== $transUnit->attributes->getNamedItem('sizeRestriction')->nodeValue ) {
-            $metadata[ 'sizeRestriction' ] = (int)$transUnit->attributes->getNamedItem('sizeRestriction')->nodeValue;
+        if (null !== $transUnit->attributes->getNamedItem('sizeRestriction') && '' !== $transUnit->attributes->getNamedItem('sizeRestriction')->nodeValue) {
+            $metadata['sizeRestriction'] = (int)$transUnit->attributes->getNamedItem('sizeRestriction')->nodeValue;
         }
 
         return $metadata;
@@ -291,7 +291,7 @@ 
      * @return array
      * @throws \Exception
      */
-    private function extractTransUnitOriginalData( DOMElement $transUnit)
+    private function extractTransUnitOriginalData(DOMElement $transUnit)
     {
         $originalData = [];
 
@@ -299,11 +299,11 @@ 
         foreach ($transUnit->childNodes as $childNode) {
             if ($childNode->nodeName === 'originalData') {
                 foreach ($childNode->childNodes as $data) {
-                    if (null!== $data->attributes && null !== $data->attributes->getNamedItem('id')) {
+                    if (null !== $data->attributes && 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) {
@@ -311,12 +311,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(
@@ -341,7 +341,7 @@ 
      *
      * @return array
      */
-    private function extractTransUnitAdditionalTagData( DOMElement $transUnit)
+    private function extractTransUnitAdditionalTagData(DOMElement $transUnit)
     {
         $additionalTagData = [];
 
@@ -353,7 +353,7 @@ 
 
                     // id
                     if ($data->nodeName === 'memsource:tag') {
-                        if (null!== $data->attributes && null !== $data->attributes->getNamedItem('id')) {
+                        if (null !== $data->attributes && null !== $data->attributes->getNamedItem('id')) {
                             $dataId = $data->attributes->getNamedItem('id')->nodeValue;
                             $dataArray['attr']['id'] = $dataId;
                         }
@@ -361,7 +361,7 @@ 
 
                     // in PHP 7.4 $data->childNodes is an empty DomNodeList, it is iterable with size 0
                     // PHP 5.6 check: in php 5.6 $data->childNodes can be null
-                    if( $data->childNodes != null ){
+                    if ($data->childNodes != null) {
 
                         // content
                         foreach ($data->childNodes as $datum) {
@@ -392,12 +392,12 @@ 
      * @param DOMElement $segment
      * @param array $attr
      */
-    private function checkSegmentIdConsistency( DOMElement $segment, array $attr)
+    private function checkSegmentIdConsistency(DOMElement $segment, array $attr)
     {
-        if (isset($attr[ 'tGroupBegin' ]) && isset($attr[ 'tGroupEnd' ]) && $segment->attributes->getNamedItem('id')) {
+        if (isset($attr['tGroupBegin']) && isset($attr['tGroupEnd']) && $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) && ($id <= $max))) {
                 if ($this->logger) {
@@ -413,7 +413,7 @@ 
      * @return array
      * @throws \Exception
      */
-    private function extractTransUnitNotes( DOMElement $transUnit)
+    private function extractTransUnitNotes(DOMElement $transUnit)
     {
         $notes = [];
 
@@ -432,7 +432,7 @@ 
                 foreach ($childNode->childNodes as $metadata) {
                     if ($metadata->nodeName === 'mda:metaGroup') {
                         foreach ($metadata->childNodes as $meta) {
-                            if (null!== $meta->attributes && null !== $meta->attributes->getNamedItem('type')) {
+                            if (null !== $meta->attributes && null !== $meta->attributes->getNamedItem('type')) {
                                 $type = $meta->attributes->getNamedItem('type')->nodeValue;
                                 $metaValue = trim($meta->nodeValue);
 

src/XliffReplacer/XliffSAXTranslationReplacer.php

Spacing +104 added lines, -104 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' ]) && $attr[ 'translate' ] === 'no'){
-                $attr[ 'translate' ] = 'no';
+            if (isset($attr['translate']) && $attr['translate'] === 'no') {
+                $attr['translate'] = 'no';
             } else {
-                $attr[ 'translate' ] = 'yes';
+                $attr['translate'] = 'yes';
             }
 
             // current 'translate' attribute of the current trans-unit
-            $this->currentTransUnitTranslate = $attr[ 'translate' ];
+            $this->currentTransUnitTranslate = $attr['translate'];
         }
 
-        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 && ( $name === 'notes' || $name === 'originalData' || $name === 'segment' || $name === 'ignorable') && $this->unitContainsMda === false ){
-                if (isset($this->transUnits[ $this->currentTransUnitId ]) && !empty($this->transUnits[ $this->currentTransUnitId ]) && !$this->unitContainsMda && !$this->hasWrittenCounts) {
+            if ($this->xliffVersion === 2 && ($name === 'notes' || $name === 'originalData' || $name === 'segment' || $name === 'ignorable') && $this->unitContainsMda === false) {
+                if (isset($this->transUnits[$this->currentTransUnitId]) && !empty($this->transUnits[$this->currentTransUnitId]) && !$this->unitContainsMda && !$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 && strpos($tag, 'help-id') === false){
+                        if ($this->xliffVersion === 1 && strpos($tag, 'help-id') === false) {
                             if (!empty($sid)) {
                                 $tag .= "help-id=\"$sid\" ";
                             }
-                        } elseif($this->xliffVersion === 2 && strpos($tag, 'mtc:segment-id') === false) {
+                        } elseif ($this->xliffVersion === 2 && strpos($tag, 'mtc:segment-id') === false) {
                             if (!empty($sid)) {
                                 $tag .= "mtc:segment-id=\"$sid\" ";
                             }
                         }
 
                     } elseif ('segment' === $name && $this->xliffVersion === 2) { // add state to segment in Xliff v2
-                        [$stateProp, $lastMrkState] = $this->setTransUnitState($this->segments[ $pos ], $stateProp, $lastMrkState);
+                        [$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,13 +250,13 @@ 
             //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 '/'
-                $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
-                $lastChar   = $this->currentBuffer[ $this->len - 1 ];
+                $lastChar   = $this->currentBuffer[$this->len - 1];
             }
 
             //trim last space
@@ -281,7 +281,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);
@@ -290,11 +290,11 @@ 
 
         // update segmentPositionInTu
 
-        if($this->xliffVersion === 1 && $this->inTU && $name === 'source'){
+        if ($this->xliffVersion === 1 && $this->inTU && $name === 'source') {
             $this->segmentPositionInTu++;
         }
 
-        if($this->xliffVersion === 2 && $this->inTU && $name === 'segment'){
+        if ($this->xliffVersion === 2 && $this->inTU && $name === 'segment') {
             $this->segmentPositionInTu++;
         }
     }
@@ -320,17 +320,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);
                     }
 
@@ -349,13 +349,13 @@ 
 
                     $warning       = false;
                     $last_value    = null;
-                    $segmentsCount = count( $listOfSegmentsIds );
+                    $segmentsCount = count($listOfSegmentsIds);
                     for ($i = 0; $i < $segmentsCount; $i++) {
-                        if (isset($listOfSegmentsIds[ $i ])) {
-                            $id = $listOfSegmentsIds[ $i ];
-                            if (isset($this->segments[ $id ]) && ($i == 0 || $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]) && ($i == 0 || $last_value + 1 == $listOfSegmentsIds[$i])) {
+                                $last_value            = $listOfSegmentsIds[$i];
+                                $this->lastTransUnit[] = $this->segments[$id];
                             }
                         } else {
                             $warning = true;
@@ -364,31 +364,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 && !empty($seg)){
+                        if (!$this->hasWrittenCounts && !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();
                         }
 
@@ -410,8 +410,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 && $this->segments[ $id ][ "mrk_id" ] !== null) {
-                                $this->segments[ $id ][ "mrk_id" ] = 0;
+                            if ((int)$this->segments[$id]["mrk_id"] < 0 && $this->segments[$id]["mrk_id"] !== null) {
+                                $this->segments[$id]["mrk_id"] = 0;
                             }
 
                             /*
@@ -420,22 +420,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();
                             }
 
@@ -444,9 +444,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"];
 
                             [$stateProp, $lastMrkState] = $this->setTransUnitState($seg, $stateProp, $lastMrkState);
                         }
@@ -455,10 +455,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
@@ -471,21 +471,21 @@ 
 
                 // only for Xliff 2.*
                 // write here <mda:metaGroup> and <mda:meta> if already present in the <unit>
-                if('mda:metadata' === $name && $this->unitContainsMda && $this->xliffVersion === 2 && !$this->hasWrittenCounts){
+                if ('mda:metadata' === $name && $this->unitContainsMda && $this->xliffVersion === 2 && !$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);
@@ -493,7 +493,7 @@ 
 
                 // only for Xliff 2.*
                 // if segment has no <target> add it BEFORE </segment>
-                if($this->xliffVersion === 2 && !$this->targetWasWritten){
+                if ($this->xliffVersion === 2 && !$this->targetWasWritten) {
 
                     $seg = $this->getCurrentSegment();
 
@@ -503,7 +503,7 @@ 
                         $tag .= " $k=\"$v\"";
                     }
 
-                    $tag .= '>'.$seg['translation'].'</target></segment>';
+                    $tag .= '>' . $seg['translation'] . '</target></segment>';
                 }
 
                 $this->postProcAndFlush($this->outputFP, $tag);
@@ -511,18 +511,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'])) {
                         [$stateProp, $lastMrkState] = $this->setTransUnitState($seg, $stateProp, $lastMrkState);
                         $tag .= $this->createTargetTag($seg['translation'], $stateProp);
                     }
@@ -576,7 +576,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 = [
@@ -593,23 +593,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);
                 }
             }
         }
@@ -622,14 +622,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;
     }
 
     /**
@@ -644,11 +644,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']) && $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) {
@@ -663,15 +663,15 @@ 
             return $translation;
         }
 
-        if ($seg[ 'mrk_id' ] !== null && $seg[ 'mrk_id' ] != '') {
+        if ($seg['mrk_id'] !== null && $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;
     }
@@ -693,7 +693,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);
                 }
 
@@ -720,10 +720,10 @@ 
      */
     private function getCurrentSegment()
     {
-        if($this->currentTransUnitTranslate === 'yes' && isset($this->transUnits[$this->currentTransUnitId])){
+        if ($this->currentTransUnitTranslate === 'yes' && 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];
             }
         }
@@ -741,9 +741,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']);
     }
 
     /**
@@ -756,10 +756,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>
@@ -767,7 +767,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>
@@ -783,12 +783,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 || $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;
@@ -831,7 +831,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 ]) && $_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++;
@@ -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']) && $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) {
@@ -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 && $seg[ 'mrk_id' ] != '') {
+        if ($seg['mrk_id'] !== null && $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/XliffUtils/DataRefReplacer.php

Spacing +68 added lines, -68 removed lines

@@ -17,7 +17,7 @@ 
      *
      * @param array $map
      */
-    public function __construct( array $map)
+    public function __construct(array $map)
     {
         $this->map = $map;
     }
@@ -48,14 +48,14 @@ 
         $html = HtmlParser::parse($string);
 
         // 1. Replace <ph>|<sc>|<ec> tags
-        foreach ($html as $node){
+        foreach ($html as $node) {
             $string = $this->recursiveAddEquivTextToPhTag($node, $string);
         }
 
         // 2. Replace <pc> tags
         $toBeEscaped = Strings::isAnEscapedHTML($string);
 
-        if($this->stringContainsPcTags($string, $toBeEscaped)){
+        if ($this->stringContainsPcTags($string, $toBeEscaped)) {
 
             // replace self-closed <pc />
             $string = $this->replaceSelfClosedPcTags($string, $toBeEscaped);
@@ -84,10 +84,10 @@ 
             'dataRefEnd',
         ];
 
-        foreach ($dataRefTags as $tag){
-            preg_match('/ '.$tag.'=[\\\\"](.*?)[\\\\"]/', $string, $matches);
+        foreach ($dataRefTags as $tag) {
+            preg_match('/ ' . $tag . '=[\\\\"](.*?)[\\\\"]/', $string, $matches);
 
-            if(count($matches) > 0){
+            if (count($matches) > 0) {
                 return true;
             }
         }
@@ -102,7 +102,7 @@ 
     {
         $html = HtmlParser::parse($string);
 
-        foreach ($html as $node){
+        foreach ($html as $node) {
             $string = $this->recursiveCleanFromEquivText($node, $string);
         }
 
@@ -122,10 +122,10 @@ 
      *
      * @return string
      */
-    private function recursiveAddEquivTextToPhTag( $node, $string)
+    private function recursiveAddEquivTextToPhTag($node, $string)
     {
-        if($node->has_children){
-            foreach ($node->inner_html as $childNode){
+        if ($node->has_children) {
+            foreach ($node->inner_html as $childNode) {
                 $string = $this->recursiveAddEquivTextToPhTag($childNode, $string);
             }
         } else {
@@ -134,18 +134,18 @@ 
                     return $string;
                 }
 
-                $a = $node->node;  // complete match. Eg:  <ph id="source1" dataRef="source1"/>
-                $b = $node->attributes['dataRef'];   // map identifier. Eg: source1
+                $a = $node->node; // complete match. Eg:  <ph id="source1" dataRef="source1"/>
+                $b = $node->attributes['dataRef']; // map identifier. Eg: source1
 
 
                 // if isset a value in the map calculate base64 encoded value
                 // otherwise skip
-                if(!in_array($b, array_keys($this->map))){
+                if (!in_array($b, array_keys($this->map))) {
                     return $string;
                 }
 
                 // check if is null, in this case convert it to NULL string
-                if(is_null($this->map[$b])){
+                if (is_null($this->map[$b])) {
                     $this->map[$b] = 'NULL';
                 }
 
@@ -157,19 +157,19 @@ 
                 }
 
                 // if there is no id copy it from dataRef
-                $id = (!isset($node->attributes['id'])) ? ' id="'.$b.'" removeId="true"': '';
+                $id = (!isset($node->attributes['id'])) ? ' id="' . $b . '" removeId="true"' : '';
 
                 // introduce dataType for <ec>/<sc> tag handling
-                $dataType = ($this->isAEcOrScTag($node)) ? ' dataType="'.$node->tagname.'"' : '';
+                $dataType = ($this->isAEcOrScTag($node)) ? ' dataType="' . $node->tagname . '"' : '';
 
                 // replacement
-                $d = str_replace('/', $id.$dataType. ' equiv-text="base64:'.$base64EncodedValue.'"/', $a);
-                $a = str_replace(['<','>','&gt;', '&lt;'], '', $a);
-                $d = str_replace(['<','>','&gt;', '&lt;'], '', $d);
+                $d = str_replace('/', $id . $dataType . ' equiv-text="base64:' . $base64EncodedValue . '"/', $a);
+                $a = str_replace(['<', '>', '&gt;', '&lt;'], '', $a);
+                $d = str_replace(['<', '>', '&gt;', '&lt;'], '', $d);
 
                 // convert <ec>/<sc> into <ph>
-                if($this->isAEcOrScTag($node)){
-                    $d = 'ph'.substr($d, 2);
+                if ($this->isAEcOrScTag($node)) {
+                    $d = 'ph' . substr($d, 2);
                     $d = trim($d);
                 }
 
@@ -191,7 +191,7 @@ 
         $regex = ($toBeEscaped) ? '/&lt;pc (.*?)&gt;/iu' : '/<pc (.*?)>/iu';
         preg_match_all($regex, $string, $openingPcMatches);
 
-        return (isset($openingPcMatches[0]) && count($openingPcMatches[0])>0);
+        return (isset($openingPcMatches[0]) && count($openingPcMatches[0]) > 0);
     }
 
     /**
@@ -202,27 +202,27 @@ 
      */
     private function replaceSelfClosedPcTags($string, $toBeEscaped)
     {
-        if($toBeEscaped){
-            $string = str_replace(['&lt;','&gt;'],['<','>'],$string);
+        if ($toBeEscaped) {
+            $string = str_replace(['&lt;', '&gt;'], ['<', '>'], $string);
         }
 
         $regex = '/<pc[^>]+?\/>/iu';
         preg_match_all($regex, $string, $selfClosedPcMatches);
 
-        foreach ($selfClosedPcMatches[0] as $match){
+        foreach ($selfClosedPcMatches[0] as $match) {
 
             $html = HtmlParser::parse($match);
             $node = $html[0];
             $attributes = $node->attributes;
 
-            if(isset($attributes['dataRefStart']) && array_key_exists($node->attributes['dataRefStart'], $this->map)){
-                $replacement = '<ph id="'.$attributes['id'].'" dataType="pcSelf" originalData="'.base64_encode($match).'" dataRef="'.$attributes['dataRefStart'].'" equiv-text="base64:'.base64_encode($this->map[$node->attributes['dataRefStart']]).'"/>';
+            if (isset($attributes['dataRefStart']) && array_key_exists($node->attributes['dataRefStart'], $this->map)) {
+                $replacement = '<ph id="' . $attributes['id'] . '" dataType="pcSelf" originalData="' . base64_encode($match) . '" dataRef="' . $attributes['dataRefStart'] . '" equiv-text="base64:' . base64_encode($this->map[$node->attributes['dataRefStart']]) . '"/>';
                 $string = str_replace($match, $replacement, $string);
             }
         }
 
-        if($toBeEscaped){
-            $string = str_replace(['<','>'],['&lt;','&gt;'],$string);
+        if ($toBeEscaped) {
+            $string = str_replace(['<', '>'], ['&lt;', '&gt;'], $string);
         }
 
         return $string;
@@ -241,7 +241,7 @@ 
         $dataRefEndMap = [];
 
         foreach ($html as $index => $node) {
-            if ( $node->tagname === 'pc' ) {
+            if ($node->tagname === 'pc') {
                 $this->extractDataRefMapRecursively($node, $dataRefEndMap);
             }
         }
@@ -255,26 +255,26 @@ 
      * @param object $node
      * @param $dataRefEndMap
      */
-    private function extractDataRefMapRecursively( $node, &$dataRefEndMap)
+    private function extractDataRefMapRecursively($node, &$dataRefEndMap)
     {
-        if($this->nodeContainsNestedPcTags($node)) {
-            foreach ( $node->inner_html as $nestedNode ) {
+        if ($this->nodeContainsNestedPcTags($node)) {
+            foreach ($node->inner_html as $nestedNode) {
                 $this->extractDataRefMapRecursively($nestedNode, $dataRefEndMap);
             }
         }
 
         // EXCLUDE self closed <pc/>
-        if($node->tagname === 'pc' && $node->self_closed === false){
-            if(isset($node->attributes['dataRefEnd'])){
+        if ($node->tagname === 'pc' && $node->self_closed === false) {
+            if (isset($node->attributes['dataRefEnd'])) {
                 $dataRefEnd = $node->attributes['dataRefEnd'];
-            } elseif(isset($node->attributes['dataRefStart'])) {
+            } elseif (isset($node->attributes['dataRefStart'])) {
                 $dataRefEnd = $node->attributes['dataRefStart'];
             } else {
                 $dataRefEnd = null;
             }
 
             $dataRefEndMap[] = [
-                    'id' => isset($node->attributes['id'] ) ? $node->attributes['id'] : null,
+                    'id' => isset($node->attributes['id']) ? $node->attributes['id'] : null,
                     'dataRefEnd' => $dataRefEnd,
             ];
         }
@@ -288,12 +288,12 @@ 
      */
     private function recursiveCleanFromEquivText($node, $string)
     {
-        if($node->has_children){
-            foreach ($node->inner_html as $childNode){
+        if ($node->has_children) {
+            foreach ($node->inner_html as $childNode) {
                 $string = $this->recursiveCleanFromEquivText($childNode, $string);
             }
         } else {
-            if(isset($node->attributes['dataRef']) && array_key_exists($node->attributes['dataRef'], $this->map)){
+            if (isset($node->attributes['dataRef']) && array_key_exists($node->attributes['dataRef'], $this->map)) {
                 $cleaned = preg_replace('/ equiv-text="(.*?)"/', '', $node->node);
                 $string = str_replace($node->node, $cleaned, $string);
             }
@@ -315,29 +315,29 @@ 
         $regex = ($toBeEscaped) ? '/&lt;pc (.*?)&gt;/iu' : '/<pc (.*?)>/iu';
         preg_match_all($regex, $string, $openingPcMatches);
 
-        foreach ($openingPcMatches[0] as $index => $match){
+        foreach ($openingPcMatches[0] as $index => $match) {
             $attr = HtmlParser::getAttributes($openingPcMatches[1][$index]);
 
             // CASE 1 - Missing `dataRefStart`
-            if( isset($attr['dataRefEnd']) && !isset($attr['dataRefStart'])  ){
+            if (isset($attr['dataRefEnd']) && !isset($attr['dataRefStart'])) {
                 $attr['dataRefStart'] = $attr['dataRefEnd'];
             }
 
             // CASE 2 - Missing `dataRefEnd`
-            if( isset($attr['dataRefStart']) && !isset($attr['dataRefEnd'])  ){
+            if (isset($attr['dataRefStart']) && !isset($attr['dataRefEnd'])) {
                 $attr['dataRefEnd'] = $attr['dataRefStart'];
             }
 
-            if(isset($attr['dataRefStart'])){
+            if (isset($attr['dataRefStart'])) {
                 $startOriginalData = $match; // opening <pc>
                 $startValue = $this->map[$attr['dataRefStart']] ? $this->map[$attr['dataRefStart']] : 'NULL'; //handling null values in original data map
                 $base64EncodedStartValue = base64_encode($startValue);
                 $base64StartOriginalData = base64_encode($startOriginalData);
 
                 // conversion for opening <pc> tag
-                $openingPcConverted  = '<ph '. ((isset($attr['id'])) ? 'id="'.$attr['id'].'_1"' : '') .' dataType="pcStart" originalData="'.$base64StartOriginalData.'" dataRef="'
-                        .$attr['dataRefStart'].'" equiv-text="base64:'
-                        .$base64EncodedStartValue.'"/>';
+                $openingPcConverted = '<ph ' . ((isset($attr['id'])) ? 'id="' . $attr['id'] . '_1"' : '') . ' dataType="pcStart" originalData="' . $base64StartOriginalData . '" dataRef="'
+                        .$attr['dataRefStart'] . '" equiv-text="base64:'
+                        .$base64EncodedStartValue . '"/>';
 
                 $string = str_replace($startOriginalData, $openingPcConverted, $string);
             }
@@ -362,20 +362,20 @@ 
         preg_match_all($regex, $string, $closingPcMatches, PREG_OFFSET_CAPTURE);
         $delta = 0;
 
-        foreach ( $closingPcMatches[ 0 ] as $index => $match ) {
+        foreach ($closingPcMatches[0] as $index => $match) {
             $offset = $match[1];
             $length = strlen($match[0]);
             $attr = $dataRefEndMap[$index];
 
-            if(!empty($attr) && isset($attr['dataRefEnd'])){
+            if (!empty($attr) && isset($attr['dataRefEnd'])) {
                 $endOriginalData = $match[0]; // </pc>
                 $endValue = $this->map[$attr['dataRefEnd']] ? $this->map[$attr['dataRefEnd']] : 'NULL';
                 $base64EncodedEndValue = base64_encode($endValue);
                 $base64EndOriginalData = base64_encode($endOriginalData);
 
                 // conversion for closing <pc> tag
-                $closingPcConverted = '<ph '. ((isset($attr['id'])) ? 'id="'.$attr['id'].'_2"': '') .' dataType="pcEnd" originalData="'.$base64EndOriginalData.'" dataRef="'
-                        .$attr['dataRefEnd'].'" equiv-text="base64:' .$base64EncodedEndValue.'"/>';
+                $closingPcConverted = '<ph ' . ((isset($attr['id'])) ? 'id="' . $attr['id'] . '_2"' : '') . ' dataType="pcEnd" originalData="' . $base64EndOriginalData . '" dataRef="'
+                        .$attr['dataRefEnd'] . '" equiv-text="base64:' . $base64EncodedEndValue . '"/>';
 
                 $realOffset = ($delta === 0) ? $offset : ($offset + $delta);
 
@@ -394,12 +394,12 @@ 
      */
     private function nodeContainsNestedPcTags($node)
     {
-        if(!$node->has_children){
+        if (!$node->has_children) {
             return false;
         }
 
         foreach ($node->inner_html as $nestedNode) {
-            if($nestedNode->tagname === 'pc' && (isset($node->attributes['dataRefEnd']) || isset($node->attributes['dataRefStart']))){
+            if ($nestedNode->tagname === 'pc' && (isset($node->attributes['dataRefEnd']) || isset($node->attributes['dataRefStart']))) {
                 return true;
             }
         }
@@ -423,7 +423,7 @@ 
         $string = str_replace(' equiv-text=""', '', $string);
         $html = HtmlParser::parse($string);
 
-        foreach ($html as $node){
+        foreach ($html as $node) {
             $string = $this->recursiveRemoveOriginalData($node, $string);
         }
 
@@ -438,43 +438,43 @@ 
      */
     private function recursiveRemoveOriginalData($node, $string)
     {
-        if($node->has_children){
-            foreach ($node->inner_html as $childNode){
+        if ($node->has_children) {
+            foreach ($node->inner_html as $childNode) {
                 $string = $this->recursiveRemoveOriginalData($childNode, $string);
             }
         } else {
 
-            if(!isset($node->attributes['dataRef'])){
+            if (!isset($node->attributes['dataRef'])) {
                 return $string;
             }
 
-            $a = $node->node;                  // complete match. Eg:  <ph id="source1" dataRef="source1"/>
+            $a = $node->node; // complete match. Eg:  <ph id="source1" dataRef="source1"/>
             $b = $node->attributes['dataRef']; // map identifier. Eg: source1
-            $c = $node->terminator;            // terminator: Eg: >
+            $c = $node->terminator; // terminator: Eg: >
 
             // if isset a value in the map calculate base64 encoded value
             // or it is an empty string
             // otherwise skip
-            if(!in_array($b, array_keys($this->map))  ){
+            if (!in_array($b, array_keys($this->map))) {
                 return $string;
             }
 
             // check if is null, in this case convert it to NULL string
-            if(is_null($this->map[$b])){
+            if (is_null($this->map[$b])) {
                 $this->map[$b] = 'NULL';
             }
 
             // remove id?
-            $removeId = (isset($node->attributes['removeId']) && $node->attributes['removeId'] === "true") ? ' id="'.$b.'" removeId="true"' : '';
+            $removeId = (isset($node->attributes['removeId']) && $node->attributes['removeId'] === "true") ? ' id="' . $b . '" removeId="true"' : '';
 
             // grab dataType attribute for <ec>/<sc> tag handling
-            $dataType = ($this->wasAEcOrScTag($node)) ? ' dataType="'.$node->attributes['dataType'].'"' : '';
+            $dataType = ($this->wasAEcOrScTag($node)) ? ' dataType="' . $node->attributes['dataType'] . '"' : '';
 
-            $d = str_replace($removeId.$dataType.' equiv-text="base64:'.base64_encode($this->map[$b]).'"/'.$c, '/'.$c, $a);
+            $d = str_replace($removeId . $dataType . ' equiv-text="base64:' . base64_encode($this->map[$b]) . '"/' . $c, '/' . $c, $a);
 
             // replace original <ec>/<sc> tag
-            if($this->wasAEcOrScTag($node)){
-                $d = $node->attributes['dataType'].substr($d, 3);
+            if ($this->wasAEcOrScTag($node)) {
+                $d = $node->attributes['dataType'] . substr($d, 3);
                 $d = trim($d);
             }
 
@@ -487,7 +487,7 @@ 
             $string = str_replace($a, $d, $string);
 
             // restoring <pc/> self-closed here
-            if(Strings::contains('dataType="pcSelf"', $d)){
+            if (Strings::contains('dataType="pcSelf"', $d)) {
                 preg_match('/\s?originalData="(.*?)"\s?/', $d, $originalDataMatches);
 
                 if (isset($originalDataMatches[1])) {
@@ -546,6 +546,6 @@ 
      */
     private function wasAEcOrScTag($node)
     {
-        return (isset($node->attributes['dataType']) && ( $node->attributes['dataType'] === 'ec' || $node->attributes['dataType'] === 'sc'));
+        return (isset($node->attributes['dataType']) && ($node->attributes['dataType'] === 'ec' || $node->attributes['dataType'] === 'sc'));
     }
 }