matecat / xliff-parser

src/XliffReplacer/Xliff12.php

Spacing +67 added lines, -67 removed lines

@@ -36,34 +36,34 @@ 
     /**
      * @var string
      */
-    protected string $namespace = "mtc";       // Custom namespace
+    protected string $namespace = "mtc"; // Custom namespace
 
     /**
      * @inheritDoc
      */
-    protected function tagOpen( $parser, string $name, array $attr ) {
+    protected function tagOpen($parser, string $name, array $attr) {
 
-        $this->handleOpenUnit( $name, $attr );
+        $this->handleOpenUnit($name, $attr);
 
-        $this->trySetAltTrans( $name );;
-        $this->checkSetInTarget( $name );
+        $this->trySetAltTrans($name); ;
+        $this->checkSetInTarget($name);
 
         // open buffer
-        $this->setInBuffer( $name );
+        $this->setInBuffer($name);
 
         // check if we are inside a <target>, obviously this happen only if there are targets inside the trans-unit
         // <target> must be stripped to be replaced, so this check avoids <target> reconstruction
-        if ( !$this->inTarget ) {
+        if (!$this->inTarget) {
 
             $tag = '';
 
             // construct tag
             $tag .= "<$name ";
 
-            foreach ( $attr as $k => $v ) {
+            foreach ($attr as $k => $v) {
 
                 //if tag name is file, we must replace the target-language attribute
-                if ( $name === 'file' && $k === 'target-language' && !empty( $this->targetLang ) ) {
+                if ($name === 'file' && $k === 'target-language' && !empty($this->targetLang)) {
                     //replace Target language with job language provided from constructor
                     $tag .= "$k=\"$this->targetLang\" ";
                 } else {
@@ -74,20 +74,20 @@ 
 
             $seg = $this->getCurrentSegment();
 
-            if ( $name === $this->tuTagName && !empty( $seg ) && isset( $seg[ 'sid' ] ) ) {
+            if ($name === $this->tuTagName && !empty($seg) && isset($seg['sid'])) {
 
                 // add `help-id` to xliff v.1*
-                if ( strpos( $tag, 'help-id' ) === false ) {
-                    if ( !empty( $seg[ 'sid' ] ) ) {
-                        $tag .= "help-id=\"{$seg[ 'sid' ]}\" ";
+                if (strpos($tag, 'help-id') === false) {
+                    if (!empty($seg['sid'])) {
+                        $tag .= "help-id=\"{$seg['sid']}\" ";
                     }
                 }
 
             }
 
-            $tag = $this->handleOpenXliffTag( $name, $attr, $tag );
+            $tag = $this->handleOpenXliffTag($name, $attr, $tag);
 
-            $this->checkForSelfClosedTagAndFlush( $parser, $tag );
+            $this->checkForSelfClosedTagAndFlush($parser, $tag);
 
         }
 
@@ -97,7 +97,7 @@ 
     /**
      * @inheritDoc
      */
-    protected function tagClose( $parser, string $name ) {
+    protected function tagClose($parser, string $name) {
         $tag = '';
 
         /**
@@ -106,23 +106,23 @@ 
          *
          * self::tagOpen method
          */
-        if ( !$this->isEmpty ) {
+        if (!$this->isEmpty) {
 
             // write closing tag if is not a target
             // EXCLUDE the target nodes with currentTransUnitIsTranslatable = 'NO'
-            if ( !$this->inTarget and $this->currentTransUnitIsTranslatable !== 'no' ) {
+            if (!$this->inTarget and $this->currentTransUnitIsTranslatable !== 'no') {
                 $tag = "</$name>";
             }
 
-            if ( 'target' == $name && !$this->inAltTrans ) {
+            if ('target' == $name && !$this->inAltTrans) {
 
-                if ( isset( $this->transUnits[ $this->currentTransUnitId ] ) ) {
+                if (isset($this->transUnits[$this->currentTransUnitId])) {
 
                     // get translation of current segment, by indirect indexing: id -> positional index -> segment
                     // actually there may be more than one segment to that ID if there are two mrk of the same source segment
                     $tag = $this->rebuildTarget();
 
-                } elseif( !empty($this->CDATABuffer) and $this->currentTransUnitIsTranslatable === 'no' ) {
+                } elseif (!empty($this->CDATABuffer) and $this->currentTransUnitIsTranslatable === 'no') {
 
                     // These are target nodes with currentTransUnitIsTranslatable = 'NO'
                     $this->bufferIsActive = false;
@@ -133,45 +133,45 @@ 
                 $this->targetWasWritten = true;
                 // signal we are leaving a target
                 $this->inTarget = false;
-                $this->postProcAndFlush( $this->outputFP, $tag, true );
+                $this->postProcAndFlush($this->outputFP, $tag, true);
 
-            } elseif ( in_array( $name, $this->nodesToBuffer ) ) { // we are closing a critical CDATA section
+            } elseif (in_array($name, $this->nodesToBuffer)) { // we are closing a critical CDATA section
 
                 $this->bufferIsActive = false;
                 $tag                  = $this->CDATABuffer . "</$name>";
                 $this->CDATABuffer    = "";
 
                 //flush to the pointer
-                $this->postProcAndFlush( $this->outputFP, $tag, false, $name === 'seg-source' );
+                $this->postProcAndFlush($this->outputFP, $tag, false, $name === 'seg-source');
 
-            } elseif ( $name === $this->tuTagName ) {
+            } elseif ($name === $this->tuTagName) {
 
                 $tag = "";
 
                 // handling </trans-unit> closure
-                if ( !$this->targetWasWritten ) {
+                if (!$this->targetWasWritten) {
 
-                    if ( isset( $this->transUnits[ $this->currentTransUnitId ] ) ) {
+                    if (isset($this->transUnits[$this->currentTransUnitId])) {
                         $tag = $this->rebuildTarget();
                     } else {
-                        $tag = $this->createTargetTag( "", "" );
+                        $tag = $this->createTargetTag("", "");
                     }
 
                 }
 
-                $tag                    .= "</$this->tuTagName>";
+                $tag .= "</$this->tuTagName>";
                 $this->targetWasWritten = false;
-                $this->postProcAndFlush( $this->outputFP, $tag );
+                $this->postProcAndFlush($this->outputFP, $tag);
 
-            } elseif ( $this->bufferIsActive ) { // this is a tag ( <g | <mrk ) inside a seg or seg-source tag
+            } elseif ($this->bufferIsActive) { // this is a tag ( <g | <mrk ) inside a seg or seg-source tag
                 $this->CDATABuffer .= "</$name>";
                 // Do NOT Flush
             } else { //generic tag closure do Nothing
                 // flush to pointer
-                $this->postProcAndFlush( $this->outputFP, $tag );
+                $this->postProcAndFlush($this->outputFP, $tag);
             }
 
-        } elseif ( in_array( $name, $this->nodesToBuffer ) ) {
+        } elseif (in_array($name, $this->nodesToBuffer)) {
 
             $this->isEmpty        = false;
             $this->bufferIsActive = false;
@@ -179,7 +179,7 @@ 
             $this->CDATABuffer    = "";
 
             //flush to the pointer
-            $this->postProcAndFlush( $this->outputFP, $tag, false, $name === 'seg-source'  );
+            $this->postProcAndFlush($this->outputFP, $tag, false, $name === 'seg-source');
 
         } else {
             //ok, nothing to be done; reset flag for next coming tag
@@ -187,10 +187,10 @@ 
         }
 
         // try to signal that we are leaving a target
-        $this->tryUnsetAltTrans( $name );
+        $this->tryUnsetAltTrans($name);
 
         // check if we are leaving a <trans-unit> (xliff v1.*) or <unit> (xliff v2.*)
-        if ( $this->tuTagName === $name ) {
+        if ($this->tuTagName === $name) {
             $this->currentTransUnitIsTranslatable = null;
             $this->inTU                           = false;
             $this->hasWrittenCounts               = false;
@@ -207,23 +207,23 @@ 
      *
      * @return string
      */
-    protected function prepareTranslation( array $seg, string $transUnitTranslation = "" ): string {
+    protected function prepareTranslation(array $seg, string $transUnitTranslation = ""): string {
 
-        $segment     = Strings::removeDangerousChars( $seg [ 'segment' ] );
-        $translation = Strings::removeDangerousChars( $seg [ 'translation' ] );
+        $segment     = Strings::removeDangerousChars($seg ['segment']);
+        $translation = Strings::removeDangerousChars($seg ['translation']);
 
-        if ( $seg [ 'translation' ] == '' ) {
+        if ($seg ['translation'] == '') {
             $translation = $segment;
         } else {
-            if ( $this->callback instanceof XliffReplacerCallbackInterface ) {
-                $error = ( !empty( $seg[ 'error' ] ) ) ? $seg[ 'error' ] : null;
-                if ( $this->callback->thereAreErrors( $seg[ 'sid' ], $segment, $translation, [], $error ) ) {
+            if ($this->callback instanceof XliffReplacerCallbackInterface) {
+                $error = (!empty($seg['error'])) ? $seg['error'] : null;
+                if ($this->callback->thereAreErrors($seg['sid'], $segment, $translation, [], $error)) {
                     $translation = '|||UNTRANSLATED_CONTENT_START|||' . $segment . '|||UNTRANSLATED_CONTENT_END|||';
                 }
             }
         }
 
-        $transUnitTranslation .= $seg[ 'prev_tags' ] . $this->rebuildMarks( $seg, $translation ) . ltrim( $seg[ 'succ_tags' ] );
+        $transUnitTranslation .= $seg['prev_tags'] . $this->rebuildMarks($seg, $translation) . ltrim($seg['succ_tags']);
 
         return $transUnitTranslation;
     }
@@ -233,17 +233,17 @@ 
      * @param string $translation
      * @return string
      */
-    protected function rebuildMarks( array $seg, string $translation ): string {
+    protected function rebuildMarks(array $seg, string $translation): string {
 
-        if ( $seg[ 'mrk_id' ] !== null && $seg[ 'mrk_id' ] != '' ) {
-            $translation = "<mrk mid=\"" . $seg[ 'mrk_id' ] . "\" mtype=\"seg\">" . $seg[ 'mrk_prev_tags' ] . $translation . $seg[ 'mrk_succ_tags' ] . "</mrk>";
+        if ($seg['mrk_id'] !== null && $seg['mrk_id'] != '') {
+            $translation = "<mrk mid=\"" . $seg['mrk_id'] . "\" mtype=\"seg\">" . $seg['mrk_prev_tags'] . $translation . $seg['mrk_succ_tags'] . "</mrk>";
 
             // check if there is a trailing space in the map of this trans unit
-            if(
+            if (
                 isset($this->mrkTagsMap[$this->currentTransUnitId]) and
-                isset($this->mrkTagsMap[$this->currentTransUnitId][$seg[ 'mrk_id' ]])
-            ){
-                $translation .= $this->mrkTagsMap[$this->currentTransUnitId][$seg[ 'mrk_id' ]];
+                isset($this->mrkTagsMap[$this->currentTransUnitId][$seg['mrk_id']])
+            ) {
+                $translation .= $this->mrkTagsMap[$this->currentTransUnitId][$seg['mrk_id']];
             }
         }
 
@@ -259,10 +259,10 @@ 
      *
      * @return string
      */
-    protected function createTargetTag( string $translation, string $stateProp ): string {
+    protected function createTargetTag(string $translation, string $stateProp): string {
         $targetLang = ' xml:lang="' . $this->targetLang . '"';
         $tag        = "<target $targetLang $stateProp>$translation</target>";
-        $tag        .= "\n<count-group name=\"$this->currentTransUnitId\"><count count-type=\"x-matecat-raw\">" . $this->counts[ 'raw_word_count' ] . "</count><count count-type=\"x-matecat-weighted\">" . $this->counts[ 'eq_word_count' ] . '</count></count-group>';
+        $tag .= "\n<count-group name=\"$this->currentTransUnitId\"><count count-type=\"x-matecat-raw\">" . $this->counts['raw_word_count'] . "</count><count count-type=\"x-matecat-weighted\">" . $this->counts['eq_word_count'] . '</count></count-group>';
 
         return $tag;
 
@@ -278,7 +278,7 @@ 
         // we must reset the lastMrkId found because this is a new segment.
         $lastMrkId = -1;
 
-        foreach ( $this->lastTransUnit as $pos => $seg ) {
+        foreach ($this->lastTransUnit as $pos => $seg) {
 
             /*
              * This routine works to respect the positional orders of markers.
@@ -291,8 +291,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)$seg[ "mrk_id" ] < 0 && $seg[ "mrk_id" ] !== null ) {
-                $seg[ "mrk_id" ] = 0;
+            if ((int)$seg["mrk_id"] < 0 && $seg["mrk_id"] !== null) {
+                $seg["mrk_id"] = 0;
             }
 
             /*
@@ -301,39 +301,39 @@ 
              * ( null <= -1 ) === true
              * so, cast to int
              */
-            if ( (int)$seg[ "mrk_id" ] <= $lastMrkId ) {
+            if ((int)$seg["mrk_id"] <= $lastMrkId) {
                 break;
             }
 
             // update counts
-            if ( !empty( $seg ) ) {
-                $this->updateSegmentCounts( $seg );
+            if (!empty($seg)) {
+                $this->updateSegmentCounts($seg);
             }
 
             // delete translations so the prepareSegment
             // will put source content in target tag
-            if ( $this->sourceInTarget ) {
-                $seg[ 'translation' ] = '';
+            if ($this->sourceInTarget) {
+                $seg['translation'] = '';
                 $this->resetCounts();
             }
 
             // append $translation
-            $translation = $this->prepareTranslation( $seg, $translation );
+            $translation = $this->prepareTranslation($seg, $translation);
 
-            $lastMrkId = $seg[ "mrk_id" ];
+            $lastMrkId = $seg["mrk_id"];
 
-            [ $stateProp, $lastMrkState ] = StatusToStateAttribute::getState( $this->xliffVersion, $seg[ 'status' ], $lastMrkState );
+            [$stateProp, $lastMrkState] = StatusToStateAttribute::getState($this->xliffVersion, $seg['status'], $lastMrkState);
 
         }
 
         //append translation
-        return $this->createTargetTag( $translation, $stateProp );
+        return $this->createTargetTag($translation, $stateProp);
 
     }
 
     protected function getCurrentSegment(): array {
-        if ( $this->currentTransUnitIsTranslatable !== 'no' && isset( $this->transUnits[ $this->currentTransUnitId ] ) ) {
-            return $this->segments[ $this->transUnits[ $this->currentTransUnitId ][ 0 ] ]; // TODO try to understand why here is needed to override the method and set 0 index hardcoded
+        if ($this->currentTransUnitIsTranslatable !== 'no' && isset($this->transUnits[$this->currentTransUnitId])) {
+            return $this->segments[$this->transUnits[$this->currentTransUnitId][0]]; // TODO try to understand why here is needed to override the method and set 0 index hardcoded
         }
 
         return [];

src/XliffReplacer/Xliff20.php

Spacing +68 added lines, -68 removed lines

@@ -20,7 +20,7 @@ 
     /**
      * @var bool
      */
-    protected bool $unitContainsMda = false;   // check if <unit> already contains a <mda:metadata> (forXliff v 2.*)
+    protected bool $unitContainsMda = false; // check if <unit> already contains a <mda:metadata> (forXliff v 2.*)
 
     /**
      * @var string
@@ -35,7 +35,7 @@ 
     /**
      * @var string
      */
-    protected string $namespace = "matecat";       // Custom namespace
+    protected string $namespace = "matecat"; // Custom namespace
 
     /**
      * @var array
@@ -51,23 +51,23 @@ 
     /**
      * @inheritDoc
      */
-    protected function tagOpen( $parser, string $name, array $attr ) {
+    protected function tagOpen($parser, string $name, array $attr) {
 
-        $this->handleOpenUnit( $name, $attr );
+        $this->handleOpenUnit($name, $attr);
 
-        if ( 'mda:metadata' === $name ) {
+        if ('mda:metadata' === $name) {
             $this->unitContainsMda = true;
         }
 
-        $this->trySetAltTrans( $name );;
-        $this->checkSetInTarget( $name );
+        $this->trySetAltTrans($name); ;
+        $this->checkSetInTarget($name);
 
         // open buffer
-        $this->setInBuffer( $name );
+        $this->setInBuffer($name);
 
         // check if we are inside a <target>, obviously this happen only if there are targets inside the trans-unit
         // <target> must be stripped to be replaced, so this check avoids <target> reconstruction
-        if ( !$this->inTarget ) {
+        if (!$this->inTarget) {
 
             $tag = '';
 
@@ -89,48 +89,48 @@ 
             //
             // http://docs.oasis-open.org/xliff/xliff-core/v2.0/os/xliff-core-v2.0-os.html#unit
             //
-            if ( in_array( $name, [ 'notes', 'originalData', 'segment', 'ignorable' ] ) &&
+            if (in_array($name, ['notes', 'originalData', 'segment', 'ignorable']) &&
                     $this->unitContainsMda === false &&
-                    !empty( $this->transUnits[ $this->currentTransUnitId ] ) &&
+                    !empty($this->transUnits[$this->currentTransUnitId]) &&
                     !$this->hasWrittenCounts
             ) {
                 // we need to update counts here
                 $this->updateCounts();
                 $this->hasWrittenCounts = true;
-                $tag                    .= $this->getWordCountGroupForXliffV2();
+                $tag .= $this->getWordCountGroupForXliffV2();
                 $this->unitContainsMda  = true;
             }
 
             // construct tag
             $tag .= "<$name ";
 
-            foreach ( $attr as $k => $v ) {
+            foreach ($attr as $k => $v) {
                 //normal tag flux, put attributes in it but skip for translation state and set the right value for the attribute
-                if ( $k != 'state' ) {
+                if ($k != 'state') {
                     $tag .= "$k=\"$v\" ";
                 }
             }
 
             $seg = $this->getCurrentSegment();
 
-            if ( $name === $this->tuTagName && !empty( $seg ) && isset( $seg[ 'sid' ] ) ) {
+            if ($name === $this->tuTagName && !empty($seg) && isset($seg['sid'])) {
 
                 // add `matecat:segment-id` to xliff v.2*
-                if ( strpos( $tag, 'matecat:segment-id' ) === false ) {
-                    $tag .= "matecat:segment-id=\"{$seg[ 'sid' ]}\" ";
+                if (strpos($tag, 'matecat:segment-id') === false) {
+                    $tag .= "matecat:segment-id=\"{$seg['sid']}\" ";
                 }
 
             }
 
             // replace state for xliff v2
-            if ( 'segment' === $name ) { // add state to segment in Xliff v2
-                [ $stateProp, ] = StatusToStateAttribute::getState( $this->xliffVersion, $seg[ 'status' ] );
+            if ('segment' === $name) { // add state to segment in Xliff v2
+                [$stateProp, ] = StatusToStateAttribute::getState($this->xliffVersion, $seg['status']);
                 $tag .= $stateProp;
             }
 
-            $tag = $this->handleOpenXliffTag( $name, $attr, $tag );
+            $tag = $this->handleOpenXliffTag($name, $attr, $tag);
 
-            $this->checkForSelfClosedTagAndFlush( $parser, $tag );
+            $this->checkForSelfClosedTagAndFlush($parser, $tag);
 
         }
 
@@ -143,10 +143,10 @@ 
      *
      * @return string
      */
-    protected function handleOpenXliffTag( string $name, array $attr, string $tag ): string {
-        $tag = parent::handleOpenXliffTag( $name, $attr, $tag );
+    protected function handleOpenXliffTag(string $name, array $attr, string $tag): string {
+        $tag = parent::handleOpenXliffTag($name, $attr, $tag);
         // add oasis xliff 20 namespace
-        if ( $name === 'xliff' && !array_key_exists( 'xmlns:mda', $attr ) ) {
+        if ($name === 'xliff' && !array_key_exists('xmlns:mda', $attr)) {
             $tag .= 'xmlns:mda="urn:oasis:names:tc:xliff:metadata:2.0"';
         }
 
@@ -156,7 +156,7 @@ 
     /**
      * @inheritDoc
      */
-    protected function tagClose( $parser, string $name ) {
+    protected function tagClose($parser, string $name) {
         $tag = '';
 
         /**
@@ -165,39 +165,39 @@ 
          *
          * self::tagOpen method
          */
-        if ( !$this->isEmpty ) {
+        if (!$this->isEmpty) {
 
             // write closing tag if is not a target
             // EXCLUDE the target nodes with currentTransUnitIsTranslatable = 'NO'
-            if ( !$this->inTarget and $this->currentTransUnitIsTranslatable !== 'no' ) {
+            if (!$this->inTarget and $this->currentTransUnitIsTranslatable !== 'no') {
                 $tag = "</$name>";
             }
 
-            if ( 'target' == $name && !$this->inAltTrans ) {
+            if ('target' == $name && !$this->inAltTrans) {
 
-                if ( isset( $this->transUnits[ $this->currentTransUnitId ] ) ) {
+                if (isset($this->transUnits[$this->currentTransUnitId])) {
 
                     $seg = $this->getCurrentSegment();
 
                     // update counts
-                    if ( !$this->hasWrittenCounts && !empty( $seg ) ) {
-                        $this->updateSegmentCounts( $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' ] = '';
+                    if ($this->sourceInTarget) {
+                        $seg['translation'] = '';
                         $this->resetCounts();
                     }
 
                     // append $translation
-                    $translation = $this->prepareTranslation( $seg );
+                    $translation = $this->prepareTranslation($seg);
 
                     //append translation
                     $tag = "<target>$translation</target>";
 
-                } elseif( !empty($this->CDATABuffer) and $this->currentTransUnitIsTranslatable === 'no' ) {
+                } elseif (!empty($this->CDATABuffer) and $this->currentTransUnitIsTranslatable === 'no') {
 
                     // These are target nodes with currentTransUnitIsTranslatable = 'NO'
                     $this->bufferIsActive = false;
@@ -208,22 +208,22 @@ 
                 // signal we are leaving a target
                 $this->targetWasWritten = true;
                 $this->inTarget         = false;
-                $this->postProcAndFlush( $this->outputFP, $tag, true );
+                $this->postProcAndFlush($this->outputFP, $tag, true);
 
-            } elseif ( in_array( $name, $this->nodesToBuffer ) ) { // we are closing a critical CDATA section
+            } elseif (in_array($name, $this->nodesToBuffer)) { // we are closing a critical CDATA section
 
                 $this->bufferIsActive = false;
 
                 // only for Xliff 2.*
                 // write here <mda:metaGroup> and <mda:meta> if already present in the <unit>
-                if ( 'mda:metadata' === $name && $this->unitContainsMda && !$this->hasWrittenCounts ) {
+                if ('mda:metadata' === $name && $this->unitContainsMda && !$this->hasWrittenCounts) {
 
                     // we need to update counts here
                     $this->updateCounts();
                     $this->hasWrittenCounts = true;
 
                     $tag = $this->CDATABuffer;
-                    $tag .= $this->getWordCountGroupForXliffV2( false );
+                    $tag .= $this->getWordCountGroupForXliffV2(false);
                     $tag .= "    </mda:metadata>";
 
                 } else {
@@ -233,19 +233,19 @@ 
                 $this->CDATABuffer = "";
 
                 //flush to the pointer
-                $this->postProcAndFlush( $this->outputFP, $tag, false, $name === 'source' );
+                $this->postProcAndFlush($this->outputFP, $tag, false, $name === 'source');
 
-            } elseif ( 'segment' === $name ) {
+            } elseif ('segment' === $name) {
 
                 // only for Xliff 2.*
                 // if segment has no <target> add it BEFORE </segment>
-                if ( !$this->targetWasWritten ) {
+                if (!$this->targetWasWritten) {
 
                     $seg = $this->getCurrentSegment();
 
-                    if ( isset( $seg[ 'translation' ] ) ) {
+                    if (isset($seg['translation'])) {
 
-                        $translation = $this->prepareTranslation( $seg );
+                        $translation = $this->prepareTranslation($seg);
                         // replace the tag
                         $tag = "<target>$translation</target>";
 
@@ -258,19 +258,19 @@ 
                 // update segmentPositionInTu
                 $this->segmentInUnitPosition++;
 
-                $this->postProcAndFlush( $this->outputFP, $tag );
+                $this->postProcAndFlush($this->outputFP, $tag);
 
                 // we are leaving <segment>, reset $segmentHasTarget
                 $this->targetWasWritten = false;
 
-            } elseif ( $this->bufferIsActive ) { // this is a tag ( <g | <mrk ) inside a seg or seg-source tag
+            } elseif ($this->bufferIsActive) { // this is a tag ( <g | <mrk ) inside a seg or seg-source tag
                 $this->CDATABuffer .= "</$name>";
                 // Do NOT Flush
             } else { //generic tag closure do Nothing
                 // flush to pointer
-                $this->postProcAndFlush( $this->outputFP, $tag );
+                $this->postProcAndFlush($this->outputFP, $tag);
             }
-        } elseif ( in_array( $name, $this->nodesToBuffer ) ) {
+        } elseif (in_array($name, $this->nodesToBuffer)) {
 
             $this->isEmpty        = false;
             $this->bufferIsActive = false;
@@ -278,7 +278,7 @@ 
             $this->CDATABuffer    = "";
 
             //flush to the pointer
-            $this->postProcAndFlush( $this->outputFP, $tag, false, $name === 'source' );
+            $this->postProcAndFlush($this->outputFP, $tag, false, $name === 'source');
 
         } else {
             //ok, nothing to be done; reset flag for next coming tag
@@ -286,10 +286,10 @@ 
         }
 
         // try to signal that we are leaving a target
-        $this->tryUnsetAltTrans( $name );
+        $this->tryUnsetAltTrans($name);
 
         // check if we are leaving a <trans-unit> (xliff v1.*) or <unit> (xliff v2.*)
-        if ( $this->tuTagName === $name ) {
+        if ($this->tuTagName === $name) {
             $this->currentTransUnitIsTranslatable = null;
             $this->inTU                           = false;
             $this->unitContainsMda                = false;
@@ -305,8 +305,8 @@ 
     private function updateCounts() {
 
         $seg = $this->getCurrentSegment();
-        if ( !empty( $seg ) ) {
-            $this->updateSegmentCounts( $seg );
+        if (!empty($seg)) {
+            $this->updateSegmentCounts($seg);
         }
 
     }
@@ -316,30 +316,30 @@ 
      *
      * @return string
      */
-    private function getWordCountGroupForXliffV2( bool $withMetadataTag = true ): string {
+    private function getWordCountGroupForXliffV2(bool $withMetadataTag = true): string {
 
         $this->mdaGroupCounter++;
-        $segments_count_array = $this->counts[ 'segments_count_array' ];
+        $segments_count_array = $this->counts['segments_count_array'];
 
         $tag = '';
 
-        if ( $withMetadataTag === true ) {
+        if ($withMetadataTag === true) {
             $tag .= '<mda:metadata>';
         }
 
         $index = 0;
-        foreach ( $segments_count_array as $segments_count_item ) {
+        foreach ($segments_count_array as $segments_count_item) {
 
             $id = 'word_count_tu.' . $this->currentTransUnitId . '.' . $index;
             $index++;
 
             $tag .= "    <mda:metaGroup id=\"" . $id . "\" category=\"row_xml_attribute\">
-                                <mda:meta type=\"x-matecat-raw\">" . $segments_count_item[ 'raw_word_count' ] . "</mda:meta>
-                                <mda:meta type=\"x-matecat-weighted\">" . $segments_count_item[ 'eq_word_count' ] . "</mda:meta>
+                                <mda:meta type=\"x-matecat-raw\">" . $segments_count_item['raw_word_count'] . "</mda:meta>
+                                <mda:meta type=\"x-matecat-weighted\">" . $segments_count_item['eq_word_count'] . "</mda:meta>
                             </mda:metaGroup>";
         }
 
-        if ( $withMetadataTag === true ) {
+        if ($withMetadataTag === true) {
             $tag .= '</mda:metadata>';
         }
 
@@ -354,18 +354,18 @@ 
      *
      * @return string
      */
-    protected function prepareTranslation( array $seg ): string {
+    protected function prepareTranslation(array $seg): string {
 
-        $segment     = Strings::removeDangerousChars( $seg [ 'segment' ] );
-        $translation = Strings::removeDangerousChars( $seg [ 'translation' ] );
-        $dataRefMap  = ( isset( $seg[ 'data_ref_map' ] ) ) ? Strings::jsonToArray( $seg[ 'data_ref_map' ] ) : [];
+        $segment     = Strings::removeDangerousChars($seg ['segment']);
+        $translation = Strings::removeDangerousChars($seg ['translation']);
+        $dataRefMap  = (isset($seg['data_ref_map'])) ? Strings::jsonToArray($seg['data_ref_map']) : [];
 
-        if ( $seg [ 'translation' ] == '' ) {
+        if ($seg ['translation'] == '') {
             $translation = $segment;
         } else {
-            if ( $this->callback instanceof XliffReplacerCallbackInterface ) {
-                $error = ( !empty( $seg[ 'error' ] ) ) ? $seg[ 'error' ] : null;
-                if ( $this->callback->thereAreErrors( $seg[ 'sid' ], $segment, $translation, $dataRefMap, $error ) ) {
+            if ($this->callback instanceof XliffReplacerCallbackInterface) {
+                $error = (!empty($seg['error'])) ? $seg['error'] : null;
+                if ($this->callback->thereAreErrors($seg['sid'], $segment, $translation, $dataRefMap, $error)) {
                     $translation = '|||UNTRANSLATED_CONTENT_START|||' . $segment . '|||UNTRANSLATED_CONTENT_END|||';
                 }
             }

src/XliffReplacer/AbstractXliffReplacer.php

Spacing +124 added lines, -124 removed lines

@@ -7,29 +7,29 @@ 
 
 abstract class AbstractXliffReplacer {
     protected $originalFP;
-    protected $outputFP;                  // output stream pointer
+    protected $outputFP; // output stream pointer
 
-    protected string $tuTagName;                 // <trans-unit> (forXliff v 1.*) or <unit> (forXliff v 2.*)
-    protected bool   $inTU                  = false;  // flag to check whether we are in a <trans-unit>
-    protected bool   $inTarget              = false;  // flag to check whether we are in a <target>, to ignore everything
-    protected bool   $inAltTrans            = false;  // flag to check whether we are in an <alt-trans> (xliff 1.2) or <mtc:matches> (xliff 2.0)
+    protected string $tuTagName; // <trans-unit> (forXliff v 1.*) or <unit> (forXliff v 2.*)
+    protected bool   $inTU                  = false; // flag to check whether we are in a <trans-unit>
+    protected bool   $inTarget              = false; // flag to check whether we are in a <target>, to ignore everything
+    protected bool   $inAltTrans            = false; // flag to check whether we are in an <alt-trans> (xliff 1.2) or <mtc:matches> (xliff 2.0)
     protected string $alternativeMatchesTag = ""; // polymorphic tag name for xliff 1.2 and 2.0
-    protected bool   $isEmpty               = false;  // flag to check whether we are in an empty tag (<tag/>)
-    protected bool   $targetWasWritten      = false;  // flag to check is <target> was written in the current unit
-    protected string $CDATABuffer           = "";       // buffer for special tag
-    protected string $namespace             = "";       // Custom namespace
-    protected bool   $bufferIsActive        = false;    // flag for buffeting
+    protected bool   $isEmpty               = false; // flag to check whether we are in an empty tag (<tag/>)
+    protected bool   $targetWasWritten      = false; // flag to check is <target> was written in the current unit
+    protected string $CDATABuffer           = ""; // buffer for special tag
+    protected string $namespace             = ""; // Custom namespace
+    protected bool   $bufferIsActive        = false; // flag for buffeting
 
-    protected int $offset = 0;         // offset for SAX pointer
+    protected int $offset = 0; // offset for SAX pointer
 
-    protected string  $currentBuffer;             // the current piece of text it's been parsed
-    protected int     $len;                       // length of the currentBuffer
-    protected array   $segments;                  // array of translations
+    protected string  $currentBuffer; // the current piece of text it's been parsed
+    protected int     $len; // length of the currentBuffer
+    protected array   $segments; // array of translations
     protected array   $lastTransUnit                  = [];
     protected int     $segmentInUnitPosition          = 0;
-    protected ?string $currentTransUnitId             = null;        // id of current <trans-unit>
+    protected ?string $currentTransUnitId             = null; // id of current <trans-unit>
     protected ?string $currentTransUnitIsTranslatable = null; // 'translate' attribute of current <trans-unit>
-    protected bool    $hasWrittenCounts               = false;  // check if <unit> already wrote segment counts (forXliff v 2.*)
+    protected bool    $hasWrittenCounts               = false; // check if <unit> already wrote segment counts (forXliff v 2.*)
     protected string  $targetLang;
     protected bool    $sourceInTarget                 = false;
 
@@ -81,8 +81,8 @@ 
             XliffReplacerCallbackInterface $callback = null
     ) {
         self::$INTERNAL_TAG_PLACEHOLDER = $this->getInternalTagPlaceholder();
-        $this->createOutputFileIfDoesNotExist( $outputFilePath );
-        $this->setFileDescriptors( $originalXliffPath, $outputFilePath );
+        $this->createOutputFileIfDoesNotExist($outputFilePath);
+        $this->setFileDescriptors($originalXliffPath, $outputFilePath);
         $this->xliffVersion   = $xliffVersion;
         $this->segments       = $segments;
         $this->targetLang     = $trgLang;
@@ -93,17 +93,17 @@ 
     }
 
     public function replaceTranslation() {
-        fwrite( $this->outputFP, '<?xml version="1.0" encoding="UTF-8"?>' );
+        fwrite($this->outputFP, '<?xml version="1.0" encoding="UTF-8"?>');
 
         //create Sax parser
         $xmlParser = $this->initSaxParser();
 
-        while ( $this->currentBuffer = fread( $this->originalFP, 4096 ) ) {
+        while ($this->currentBuffer = fread($this->originalFP, 4096)) {
             /*
                preprocess file
              */
             // 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 );
+            $temporary_check_buffer = preg_replace("/&(.*?);/", self::$INTERNAL_TAG_PLACEHOLDER . '$1' . self::$INTERNAL_TAG_PLACEHOLDER, $this->currentBuffer);
 
             //avoid cutting entities in half:
             //the last fread could have truncated an entity (say, '&lt;' in '&l'), thus invalidating the escaping
@@ -111,27 +111,27 @@ 
             // 9 is the max length of an entity. So, suppose that the & is at the end of buffer,
             // add 9 Bytes and substitute the entities, if the & is present, and it is not at the end
             //it can't be an entity, exit the loop
-            while ( true ) {
-                $_ampPos = strpos( $temporary_check_buffer, '&' );
+            while (true) {
+                $_ampPos = strpos($temporary_check_buffer, '&');
 
                 //check for real entity or escape it to safely exit from the loop!!!
-                if ( $_ampPos === false || strlen( substr( $temporary_check_buffer, $_ampPos ) ) > 9 ) {
+                if ($_ampPos === false || strlen(substr($temporary_check_buffer, $_ampPos)) > 9) {
                     break;
                 }
 
                 //if an entity is still present, fetch some more and repeat the escaping
-                $this->currentBuffer    .= fread( $this->originalFP, 9 );
-                $temporary_check_buffer = preg_replace( "/&(.*?);/", self::$INTERNAL_TAG_PLACEHOLDER . '$1' . self::$INTERNAL_TAG_PLACEHOLDER, $this->currentBuffer );
+                $this->currentBuffer .= fread($this->originalFP, 9);
+                $temporary_check_buffer = preg_replace("/&(.*?);/", self::$INTERNAL_TAG_PLACEHOLDER . '$1' . self::$INTERNAL_TAG_PLACEHOLDER, $this->currentBuffer);
             }
 
             //free stuff outside the loop
-            unset( $temporary_check_buffer );
+            unset($temporary_check_buffer);
 
-            $this->currentBuffer = preg_replace( "/&(.*?);/", self::$INTERNAL_TAG_PLACEHOLDER . '$1' . self::$INTERNAL_TAG_PLACEHOLDER, $this->currentBuffer );
-            $this->currentBuffer = str_replace( "&", self::$INTERNAL_TAG_PLACEHOLDER . 'amp' . self::$INTERNAL_TAG_PLACEHOLDER, $this->currentBuffer );
+            $this->currentBuffer = preg_replace("/&(.*?);/", self::$INTERNAL_TAG_PLACEHOLDER . '$1' . self::$INTERNAL_TAG_PLACEHOLDER, $this->currentBuffer);
+            $this->currentBuffer = str_replace("&", self::$INTERNAL_TAG_PLACEHOLDER . 'amp' . self::$INTERNAL_TAG_PLACEHOLDER, $this->currentBuffer);
 
             //get length of chunk
-            $this->len = strlen( $this->currentBuffer );
+            $this->len = strlen($this->currentBuffer);
 
             /*
             * Get the accumulated this->offset in the document:
@@ -141,12 +141,12 @@ 
             $this->offset += $this->len;
 
             //parse chunk of text
-            $this->runParser( $xmlParser );
+            $this->runParser($xmlParser);
 
         }
 
         // close Sax parser
-        $this->closeSaxParser( $xmlParser );
+        $this->closeSaxParser($xmlParser);
 
     }
 
@@ -155,15 +155,15 @@ 
      *
      * @return void
      */
-    protected function runParser( $xmlParser ) {
+    protected function runParser($xmlParser) {
         //parse chunk of text
-        if ( !xml_parse( $xmlParser, $this->currentBuffer, feof( $this->originalFP ) ) ) {
+        if (!xml_parse($xmlParser, $this->currentBuffer, feof($this->originalFP))) {
             //if unable, raise an exception
-            throw new RuntimeException( sprintf(
+            throw new RuntimeException(sprintf(
                     "XML error: %s at line %d",
-                    xml_error_string( xml_get_error_code( $xmlParser ) ),
-                    xml_get_current_line_number( $xmlParser )
-            ) );
+                    xml_error_string(xml_get_error_code($xmlParser)),
+                    xml_get_current_line_number($xmlParser)
+            ));
         }
     }
 
@@ -172,18 +172,18 @@ 
      *
      * @return string
      */
-    protected function getLastCharacter( $parser ): string {
+    protected function getLastCharacter($parser): string {
 
         //this logic helps detecting empty tags
         //get current position of SAX pointer in all the stream of data is has read so far:
         //it points at the end of current tag
-        $idx = xml_get_current_byte_index( $parser );
+        $idx = xml_get_current_byte_index($parser);
 
         //check whether the bounds of current tag are entirely in current buffer or the end of the current tag
         //is outside current buffer (in the latter case, it's in next buffer to be read by the while loop);
         //this check is necessary because we may have truncated a tag in half with current read,
         //and the other half may be encountered in the next buffer it will be passed
-        return $this->currentBuffer[ $idx - $this->offset ] ?? $this->currentBuffer[ $this->len - 1 ];
+        return $this->currentBuffer[$idx - $this->offset] ?? $this->currentBuffer[$this->len - 1];
 
     }
 
@@ -194,19 +194,19 @@ 
         return "§" .
                 substr(
                         str_replace(
-                                [ '+', '/' ],
+                                ['+', '/'],
                                 '',
-                                base64_encode( openssl_random_pseudo_bytes( 10, $_crypto_strong ) )
+                                base64_encode(openssl_random_pseudo_bytes(10, $_crypto_strong))
                         ),
                         0,
                         4
                 );
     }
 
-    private function createOutputFileIfDoesNotExist( string $outputFilePath ) {
+    private function createOutputFileIfDoesNotExist(string $outputFilePath) {
         // create output file
-        if ( !file_exists( $outputFilePath ) ) {
-            touch( $outputFilePath );
+        if (!file_exists($outputFilePath)) {
+            touch($outputFilePath);
         }
     }
 
@@ -214,13 +214,13 @@ 
      * @param string $originalXliffPath
      * @param string $outputFilePath
      */
-    private function setFileDescriptors( string $originalXliffPath, string $outputFilePath ) {
-        $this->outputFP = fopen( $outputFilePath, 'w+' );
+    private function setFileDescriptors(string $originalXliffPath, string $outputFilePath) {
+        $this->outputFP = fopen($outputFilePath, 'w+');
 
         $streamArgs = null;
 
-        if ( !( $this->originalFP = fopen( $originalXliffPath, "r", false, stream_context_create( $streamArgs ) ) ) ) {
-            throw new RuntimeException( "could not open XML input" );
+        if (!($this->originalFP = fopen($originalXliffPath, "r", false, stream_context_create($streamArgs)))) {
+            throw new RuntimeException("could not open XML input");
         }
     }
 
@@ -230,12 +230,12 @@ 
     public function __destruct() {
         //this stream can be closed outside the class
         //to permit multiple concurrent downloads, so suppress warnings
-        if ( is_resource( $this->originalFP ) ) {
-            fclose( $this->originalFP );
+        if (is_resource($this->originalFP)) {
+            fclose($this->originalFP);
         }
 
-        if ( is_resource( $this->outputFP ) ) {
-            fclose( $this->outputFP );
+        if (is_resource($this->outputFP)) {
+            fclose($this->outputFP);
         }
 
     }
@@ -246,11 +246,11 @@ 
      * @return resource
      */
     protected function initSaxParser() {
-        $xmlSaxParser = xml_parser_create( 'UTF-8' );
-        xml_set_object( $xmlSaxParser, $this );
-        xml_parser_set_option( $xmlSaxParser, XML_OPTION_CASE_FOLDING, false );
-        xml_set_element_handler( $xmlSaxParser, 'tagOpen', 'tagClose' );
-        xml_set_character_data_handler( $xmlSaxParser, 'characterData' );
+        $xmlSaxParser = xml_parser_create('UTF-8');
+        xml_set_object($xmlSaxParser, $this);
+        xml_parser_set_option($xmlSaxParser, XML_OPTION_CASE_FOLDING, false);
+        xml_set_element_handler($xmlSaxParser, 'tagOpen', 'tagClose');
+        xml_set_character_data_handler($xmlSaxParser, 'characterData');
 
         return $xmlSaxParser;
     }
@@ -258,8 +258,8 @@ 
     /**
      * @param resource $xmlSaxParser
      */
-    protected function closeSaxParser( $xmlSaxParser ) {
-        xml_parser_free( $xmlSaxParser );
+    protected function closeSaxParser($xmlSaxParser) {
+        xml_parser_free($xmlSaxParser);
     }
 
     /**
@@ -269,7 +269,7 @@ 
      *
      * @return mixed
      */
-    abstract protected function tagOpen( $parser, string $name, array $attr );
+    abstract protected function tagOpen($parser, string $name, array $attr);
 
     /**
      * @param resource $parser
@@ -277,7 +277,7 @@ 
      *
      * @return mixed
      */
-    abstract protected function tagClose( $parser, string $name );
+    abstract protected function tagClose($parser, string $name);
 
     /**
      * @param resource $parser
@@ -285,11 +285,11 @@ 
      *
      * @return mixed
      */
-    protected function characterData( $parser, string $data ): void {
+    protected function characterData($parser, string $data): void {
         // don't write <target> data
-        if ( !$this->inTarget && !$this->bufferIsActive ) {
-            $this->postProcAndFlush( $this->outputFP, $data );
-        } elseif ( $this->bufferIsActive ) {
+        if (!$this->inTarget && !$this->bufferIsActive) {
+            $this->postProcAndFlush($this->outputFP, $data);
+        } elseif ($this->bufferIsActive) {
             $this->CDATABuffer .= $data;
         }
     }
@@ -302,33 +302,33 @@ 
      * @param bool $treatAsCDATA
      * @param bool $parseMarks
      */
-    protected function postProcAndFlush($fp, string $data, bool $treatAsCDATA = false, $parseMarks = false ) {
+    protected function postProcAndFlush($fp, string $data, bool $treatAsCDATA = false, $parseMarks = false) {
         //postprocess string
-        $data = preg_replace( "/" . self::$INTERNAL_TAG_PLACEHOLDER . '(.*?)' . self::$INTERNAL_TAG_PLACEHOLDER . "/", '&$1;', $data );
-        $data = str_replace( '&nbsp;', ' ', $data );
+        $data = preg_replace("/" . self::$INTERNAL_TAG_PLACEHOLDER . '(.*?)' . self::$INTERNAL_TAG_PLACEHOLDER . "/", '&$1;', $data);
+        $data = str_replace('&nbsp;', ' ', $data);
 
         // extract <mrk> map only for <seg-source> tag
-        if($parseMarks){
+        if ($parseMarks) {
             // check if there are spaces between <mrk> tags
             preg_match_all('/<mrk \b[^>]*>(.*?)<\/mrk>(\s+)/', $data, $spacesBetweenMrkCheck);
 
-            if(!empty($spacesBetweenMrkCheck[0])){
+            if (!empty($spacesBetweenMrkCheck[0])) {
 
                 // $spacesBetweenMrkCheck[0] // holds the complete tags
                 // $spacesBetweenMrkCheck[1] // holds the text
                 // $spacesBetweenMrkCheck[2] // holds the spaces
 
-                foreach ($spacesBetweenMrkCheck[0] as $index => $mrk){
+                foreach ($spacesBetweenMrkCheck[0] as $index => $mrk) {
 
-                    if($this instanceof Xliff20){
+                    if ($this instanceof Xliff20) {
                         preg_match('/id="(\d+)"/', $mrk, $markMatch);
                     } else {
                         preg_match('/mid="(\d+)"/', $mrk, $markMatch);
                     }
 
-                    if(isset($markMatch[1])){
+                    if (isset($markMatch[1])) {
 
-                        if(!isset($this->mrkTagsMap[$this->currentTransUnitId])){
+                        if (!isset($this->mrkTagsMap[$this->currentTransUnitId])) {
                             $this->mrkTagsMap[$this->currentTransUnitId] = [];
                         }
 
@@ -338,15 +338,15 @@ 
             }
         }
 
-        if ( !$treatAsCDATA ) {
+        if (!$treatAsCDATA) {
             //unix2dos
-            $data = str_replace( "\r\n", "\r", $data );
-            $data = str_replace( "\n", "\r", $data );
-            $data = str_replace( "\r", "\r\n", $data );
+            $data = str_replace("\r\n", "\r", $data);
+            $data = str_replace("\n", "\r", $data);
+            $data = str_replace("\r", "\r\n", $data);
         }
 
         //flush to disk
-        fwrite( $fp, $data );
+        fwrite($fp, $data);
     }
 
     /**
@@ -355,19 +355,19 @@ 
      *
      * @return void
      */
-    protected function handleOpenUnit( string $name, array $attr ) {
+    protected function handleOpenUnit(string $name, array $attr) {
 
         // check if we are entering into a <trans-unit> (xliff v1.*) or <unit> (xliff v2.*)
-        if ( $this->tuTagName === $name ) {
+        if ($this->tuTagName === $name) {
             $this->inTU = true;
 
             // get id
             // trim to first 100 characters because this is the limit on Matecat's DB
-            $this->currentTransUnitId = substr( $attr[ 'id' ], 0, 100 );
+            $this->currentTransUnitId = substr($attr['id'], 0, 100);
 
             // `translate` attribute can be only yes or no
             // current 'translate' attribute of the current trans-unit
-            $this->currentTransUnitIsTranslatable = empty( $attr[ 'translate' ] ) ? 'yes' : $attr[ 'translate' ];
+            $this->currentTransUnitIsTranslatable = empty($attr['translate']) ? 'yes' : $attr['translate'];
 
             $this->setLastTransUnitSegments();
 
@@ -381,15 +381,15 @@ 
      *
      * @return string
      */
-    protected function handleOpenXliffTag( string $name, array $attr, string $tag ): string {
+    protected function handleOpenXliffTag(string $name, array $attr, string $tag): string {
 
         // Add MateCat specific namespace.
         // Add trgLang
-        if ( $name === 'xliff' ) {
-            if ( !array_key_exists( 'xmlns:' . $this->namespace, $attr ) ) {
+        if ($name === 'xliff') {
+            if (!array_key_exists('xmlns:' . $this->namespace, $attr)) {
                 $tag .= ' xmlns:' . $this->namespace . '="https://www.matecat.com" ';
             }
-            $tag = preg_replace( '/trgLang="(.*?)"/', 'trgLang="' . $this->targetLang . '"', $tag );
+            $tag = preg_replace('/trgLang="(.*?)"/', 'trgLang="' . $this->targetLang . '"', $tag);
         }
 
         return $tag;
@@ -401,11 +401,11 @@ 
      *
      * @return void
      */
-    protected function checkSetInTarget( string $name ) {
+    protected function checkSetInTarget(string $name) {
 
         // check if we are entering into a <target>
-        if ( 'target' == $name && !$this->inAltTrans ) {
-            if ( $this->currentTransUnitIsTranslatable === 'no' ) {
+        if ('target' == $name && !$this->inAltTrans) {
+            if ($this->currentTransUnitIsTranslatable === 'no') {
                 $this->inTarget = false;
             } else {
                 $this->inTarget = true;
@@ -414,12 +414,12 @@ 
 
     }
 
-    protected function trySetAltTrans( string $name ) {
+    protected function trySetAltTrans(string $name) {
         $this->inAltTrans = $this->inAltTrans || $this->alternativeMatchesTag == $name;
     }
 
-    protected function tryUnsetAltTrans( string $name ) {
-        if ( $this->alternativeMatchesTag == $name ) {
+    protected function tryUnsetAltTrans(string $name) {
+        if ($this->alternativeMatchesTag == $name) {
             $this->inAltTrans = false;
         }
     }
@@ -429,15 +429,15 @@ 
      *
      * @return void
      */
-    protected function setInBuffer( string $name ) {
-        if ( in_array( $name, $this->nodesToBuffer ) ) {
+    protected function setInBuffer(string $name) {
+        if (in_array($name, $this->nodesToBuffer)) {
             $this->bufferIsActive = true;
         }
 
         // We need bufferIsActive for <target> nodes with currentTransUnitIsTranslatable = 'NO'
         // because in the other case, the target can be chunked into pieces by xml_set_character_data_handler()
         // and this can potentially lead to a wrong string rebuild by postProcAndFlush function if the internal placeholders are split
-        if($name === 'target' and $this->currentTransUnitIsTranslatable === 'no'){
+        if ($name === 'target' and $this->currentTransUnitIsTranslatable === 'no') {
             $this->bufferIsActive = true;
         }
     }
@@ -445,24 +445,24 @@ 
     /**
      * @param array $seg
      */
-    protected function updateSegmentCounts( array $seg = [] ) {
+    protected function updateSegmentCounts(array $seg = []) {
 
-        $raw_word_count = $seg[ 'raw_word_count' ];
-        $eq_word_count  = ( floor( $seg[ 'eq_word_count' ] * 100 ) / 100 );
+        $raw_word_count = $seg['raw_word_count'];
+        $eq_word_count  = (floor($seg['eq_word_count'] * 100) / 100);
 
-        $this->counts[ 'segments_count_array' ][ $seg[ 'sid' ] ] = [
+        $this->counts['segments_count_array'][$seg['sid']] = [
                 'raw_word_count' => $raw_word_count,
                 'eq_word_count'  => $eq_word_count,
         ];
 
-        $this->counts[ 'raw_word_count' ] += $raw_word_count;
-        $this->counts[ 'eq_word_count' ]  += $eq_word_count;
+        $this->counts['raw_word_count'] += $raw_word_count;
+        $this->counts['eq_word_count']  += $eq_word_count;
     }
 
     protected function resetCounts() {
-        $this->counts[ 'segments_count_array' ] = [];
-        $this->counts[ 'raw_word_count' ]       = 0;
-        $this->counts[ 'eq_word_count' ]        = 0;
+        $this->counts['segments_count_array'] = [];
+        $this->counts['raw_word_count']       = 0;
+        $this->counts['eq_word_count']        = 0;
     }
 
     /**
@@ -471,16 +471,16 @@ 
      *
      * @return void
      */
-    protected function checkForSelfClosedTagAndFlush( $parser, string $tag ) {
+    protected function checkForSelfClosedTagAndFlush($parser, string $tag) {
 
-        $lastChar = $this->getLastCharacter( $parser );
+        $lastChar = $this->getLastCharacter($parser);
 
         //trim last space
-        $tag = rtrim( $tag );
+        $tag = rtrim($tag);
 
         //detect empty tag
         $this->isEmpty = $lastChar == '/';
-        if ( $this->isEmpty ) {
+        if ($this->isEmpty) {
             $tag .= $lastChar;
         }
 
@@ -488,11 +488,11 @@ 
         $tag .= ">";
 
         //set a Buffer for the segSource Source tag
-        if ( $this->bufferIsActive ) { // we are opening a critical CDATA section
+        if ($this->bufferIsActive) { // we are opening a critical CDATA section
             //these are NOT source/seg-source/value empty tags, THERE IS A CONTENT, write it in buffer
             $this->CDATABuffer .= $tag;
         } else {
-            $this->postProcAndFlush( $this->outputFP, $tag );
+            $this->postProcAndFlush($this->outputFP, $tag);
         }
 
     }
@@ -517,18 +517,18 @@ 
          */
         $this->lastTransUnit = [];
 
-        if ( !isset( $this->transUnits[ $this->currentTransUnitId ] ) ) {
+        if (!isset($this->transUnits[$this->currentTransUnitId])) {
             return;
         }
 
-        $listOfSegmentsIds = $this->transUnits[ $this->currentTransUnitId ];
+        $listOfSegmentsIds = $this->transUnits[$this->currentTransUnitId];
         $last_value        = null;
-        $segmentsCount     = count( $listOfSegmentsIds );
-        for ( $i = 0; $i < $segmentsCount; $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 ];
+        $segmentsCount     = count($listOfSegmentsIds);
+        for ($i = 0; $i < $segmentsCount; $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];
             }
         }
     }
@@ -539,10 +539,10 @@ 
     protected function getCurrentSegment(): array {
         if (
             $this->currentTransUnitIsTranslatable !== 'no' &&
-            isset( $this->transUnits[ $this->currentTransUnitId ] ) &&
-            isset( $this->segments[ $this->segmentInUnitPosition ] )
+            isset($this->transUnits[$this->currentTransUnitId]) &&
+            isset($this->segments[$this->segmentInUnitPosition])
         ) {
-            return $this->segments[ $this->segmentInUnitPosition ];
+            return $this->segments[$this->segmentInUnitPosition];
         }
 
         return [];