matecat / xliff-parser

src/XliffParser/XliffParserV2.php

Spacing +139 added lines, -139 removed lines

@@ -16,30 +16,30 @@ 
      * @inheritDoc
      * @throws \Exception
      */
-    public function parse( DOMDocument $dom, $output = [] ) {
+    public function parse(DOMDocument $dom, $output = []) {
         $i = 1;
         /** @var DOMElement $file */
-        foreach ( $dom->getElementsByTagName( 'file' ) as $file ) {
+        foreach ($dom->getElementsByTagName('file') as $file) {
 
             // metadata
-            $output[ 'files' ][ $i ][ 'attr' ] = $this->extractMetadata( $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 = [];
             $j                                  = 1;
             /** @var DOMElement $transUnit */
-            foreach ( $file->childNodes as $childNode ) {
-                $this->extractTuFromNode( $childNode, $transUnitIdArrayForUniquenessCheck, $dom, $output, $i, $j );
+            foreach ($file->childNodes as $childNode) {
+                $this->extractTuFromNode($childNode, $transUnitIdArrayForUniquenessCheck, $dom, $output, $i, $j);
             }
 
             // trans-unit re-count check
-            $totalTransUnitsId  = count( $transUnitIdArrayForUniquenessCheck );
-            $transUnitsUniqueId = count( array_unique( $transUnitIdArrayForUniquenessCheck ) );
-            if ( $totalTransUnitsId != $transUnitsUniqueId ) {
-                throw new DuplicateTransUnitIdInXliff( "Invalid trans-unit id, duplicate found.", 400 );
+            $totalTransUnitsId  = count($transUnitIdArrayForUniquenessCheck);
+            $transUnitsUniqueId = count(array_unique($transUnitIdArrayForUniquenessCheck));
+            if ($totalTransUnitsId != $transUnitsUniqueId) {
+                throw new DuplicateTransUnitIdInXliff("Invalid trans-unit id, duplicate found.", 400);
             }
 
             $i++;
@@ -53,23 +53,23 @@ 
      *
      * @return array
      */
-    private function extractMetadata( DOMDocument $dom ) {
+    private function extractMetadata(DOMDocument $dom) {
         $metadata = [];
 
-        $xliffNode = $dom->getElementsByTagName( 'xliff' )->item( 0 );
-        $fileNode  = $dom->getElementsByTagName( 'file' )->item( 0 );
+        $xliffNode = $dom->getElementsByTagName('xliff')->item(0);
+        $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
@@ -83,16 +83,16 @@ 
      * @return array
      * @throws \Exception
      */
-    private function extractNotes( DOMElement $file ) {
+    private function extractNotes(DOMElement $file) {
         $notes = [];
 
         // loop <notes> to get nested <note> tag
-        foreach ( $file->childNodes as $childNode ) {
-            if ( $childNode->nodeName === 'notes' ) {
-                foreach ( $childNode->childNodes as $note ) {
-                    $noteValue = trim( $note->nodeValue );
-                    if ( '' !== $noteValue ) {
-                        $notes[] = $this->JSONOrRawContentArray( $noteValue );
+        foreach ($file->childNodes as $childNode) {
+            if ($childNode->nodeName === 'notes') {
+                foreach ($childNode->childNodes as $note) {
+                    $noteValue = trim($note->nodeValue);
+                    if ('' !== $noteValue) {
+                        $notes[] = $this->JSONOrRawContentArray($noteValue);
                     }
                 }
             }
@@ -113,34 +113,34 @@ 
      *
      * @throws \Exception
      */
-    protected function extractTransUnit( $transUnit, &$transUnitIdArrayForUniquenessCheck, $dom, &$output, &$i, &$j ) {
+    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'];
             }
         }
 
         // original-data (exclusive for V2)
         // http://docs.oasis-open.org/xliff/xliff-core/v2.0/xliff-core-v2.0.html#originaldata
-        $originalData = $this->extractTransUnitOriginalData( $transUnit );
+        $originalData = $this->extractTransUnitOriginalData($transUnit);
         $dataRefMap   = null;
-        if ( !empty( $originalData ) ) {
-            $output[ 'files' ][ $i ][ 'trans-units' ][ $j ][ 'original-data' ] = $originalData;
-            $dataRefMap                                                        = $this->getDataRefMap( $originalData );
+        if (!empty($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;
+        $additionalTagData = $this->extractTransUnitAdditionalTagData($transUnit);
+        if (!empty($additionalTagData)) {
+            $output['files'][$i]['trans-units'][$j]['additional-tag-data'] = $additionalTagData;
         }
 
         // content
@@ -160,62 +160,62 @@ 
 
         /** @var DOMElement $segment */
         $c = 0;
-        foreach ( $transUnit->childNodes as $segment ) {
-            if ( $segment->nodeName === 'segment' ) {
+        foreach ($transUnit->childNodes as $segment) {
+            if ($segment->nodeName === 'segment') {
 
                 // check segment id consistency
-                $attr = $output[ 'files' ][ $i ][ 'trans-units' ][ $j ][ 'attr' ];
-                $this->checkSegmentIdConsistency( $segment, $attr );
+                $attr = $output['files'][$i]['trans-units'][$j]['attr'];
+                $this->checkSegmentIdConsistency($segment, $attr);
 
                 // loop <segment> to get nested <source> and <target> tag
-                foreach ( $segment->childNodes as $childNode ) {
-                    if ( $childNode->nodeName === 'source' ) {
-                        $extractedSource               = $this->extractContent( $dom, $childNode );
-                        $source[ 'raw-content' ][ $c ] = $extractedSource[ 'raw-content' ];
+                foreach ($segment->childNodes as $childNode) {
+                    if ($childNode->nodeName === 'source') {
+                        $extractedSource               = $this->extractContent($dom, $childNode);
+                        $source['raw-content'][$c] = $extractedSource['raw-content'];
 
-                        if ( !empty( $originalData ) ) {
-                            $source[ 'replaced-content' ][ $c ] = ( new DataRefReplacer( $dataRefMap ) )->replace( $source[ 'raw-content' ][ $c ] );
+                        if (!empty($originalData)) {
+                            $source['replaced-content'][$c] = (new DataRefReplacer($dataRefMap))->replace($source['raw-content'][$c]);
                         }
 
-                        if ( !empty( $extractedSource[ 'attr' ] ) ) {
-                            $source[ 'attr' ][ $c ] = $extractedSource[ 'attr' ];
+                        if (!empty($extractedSource['attr'])) {
+                            $source['attr'][$c] = $extractedSource['attr'];
                         }
 
                         // append value to 'seg-source'
-                        if ( $this->stringContainsMarks( $extractedSource[ 'raw-content' ] ) ) {
-                            $segSource = $this->extractContentWithMarksAndExtTags( $dom, $childNode, $extractedSource[ 'raw-content' ], $originalData );
+                        if ($this->stringContainsMarks($extractedSource['raw-content'])) {
+                            $segSource = $this->extractContentWithMarksAndExtTags($dom, $childNode, $extractedSource['raw-content'], $originalData);
                         } else {
                             $segSource[] = [
-                                    'mid'              => count( $segSource ) > 0 ? count( $segSource ) : 0,
+                                    '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,
+                                    'raw-content'      => $extractedSource['raw-content'],
+                                    'replaced-content' => (!empty($originalData)) ? (new DataRefReplacer($dataRefMap))->replace($extractedSource['raw-content']) : null,
                                     'ext-succ-tags'    => '',
                             ];
                         }
                     }
 
-                    if ( $childNode->nodeName === 'target' ) {
-                        $extractedTarget               = $this->extractContent( $dom, $childNode );
-                        $target[ 'raw-content' ][ $c ] = $extractedTarget[ 'raw-content' ];
+                    if ($childNode->nodeName === 'target') {
+                        $extractedTarget               = $this->extractContent($dom, $childNode);
+                        $target['raw-content'][$c] = $extractedTarget['raw-content'];
 
-                        if ( !empty( $originalData ) ) {
-                            $target[ 'replaced-content' ][ $c ] = ( new DataRefReplacer( $dataRefMap ) )->replace( $target[ 'raw-content' ][ $c ] );
+                        if (!empty($originalData)) {
+                            $target['replaced-content'][$c] = (new DataRefReplacer($dataRefMap))->replace($target['raw-content'][$c]);
                         }
 
-                        if ( !empty( $extractedTarget[ 'attr' ] ) ) {
-                            $target[ 'attr' ][ $c ] = $extractedTarget[ 'attr' ];
+                        if (!empty($extractedTarget['attr'])) {
+                            $target['attr'][$c] = $extractedTarget['attr'];
                         }
 
                         // append value to 'seg-target'
-                        if ( $this->stringContainsMarks( $extractedTarget[ 'raw-content' ] ) ) {
-                            $segTarget = $this->extractContentWithMarksAndExtTags( $dom, $childNode, $extractedTarget[ 'raw-content' ], $originalData );
+                        if ($this->stringContainsMarks($extractedTarget['raw-content'])) {
+                            $segTarget = $this->extractContentWithMarksAndExtTags($dom, $childNode, $extractedTarget['raw-content'], $originalData);
                         } else {
                             $segTarget[] = [
-                                    'mid'              => count( $segTarget ) > 0 ? count( $segTarget ) : 0,
+                                    '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,
+                                    'raw-content'      => $extractedTarget['raw-content'],
+                                    'replaced-content' => (!empty($originalData)) ? (new DataRefReplacer($dataRefMap))->replace($extractedTarget['raw-content']) : null,
                                     'ext-succ-tags'    => '',
                             ];
                         }
@@ -226,10 +226,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++;
     }
@@ -240,41 +240,41 @@ 
      *
      * @return array
      */
-    private function extractTransUnitMetadata( DOMElement $transUnit, &$transUnitIdArrayForUniquenessCheck ) {
+    private function extractTransUnitMetadata(DOMElement $transUnit, &$transUnitIdArrayForUniquenessCheck) {
         $metadata = [];
 
         // id
-        if ( null === $transUnit->attributes->getNamedItem( 'id' ) ) {
-            throw new NotFoundIdInTransUnit( 'Invalid trans-unit id found. EMPTY value', 400 );
+        if (null === $transUnit->attributes->getNamedItem('id')) {
+            throw new NotFoundIdInTransUnit('Invalid trans-unit id found. EMPTY value', 400);
         }
 
-        $id = $transUnit->attributes->getNamedItem( 'id' )->nodeValue;
+        $id = $transUnit->attributes->getNamedItem('id')->nodeValue;
 
-        if ( strlen( $id ) > 100 ) {
-            throw new SegmentIdTooLongException( 'Segment-id too long. Max 100 characters allowed', 400 );
+        if (strlen($id) > 100) {
+            throw new SegmentIdTooLongException('Segment-id too long. Max 100 characters allowed', 400);
         }
 
         $transUnitIdArrayForUniquenessCheck[] = $id;
-        $metadata[ 'id' ]                     = $id;
+        $metadata['id']                     = $id;
 
         // translate
-        if ( null !== $transUnit->attributes->getNamedItem( 'translate' ) ) {
-            $metadata[ 'translate' ] = $transUnit->attributes->getNamedItem( 'translate' )->nodeValue;
+        if (null !== $transUnit->attributes->getNamedItem('translate')) {
+            $metadata['translate'] = $transUnit->attributes->getNamedItem('translate')->nodeValue;
         }
 
         // tGroupBegin
-        if ( null !== $transUnit->attributes->getNamedItem( 'tGroupBegin' ) ) {
-            $metadata[ 'tGroupBegin' ] = $transUnit->attributes->getNamedItem( 'tGroupBegin' )->nodeValue;
+        if (null !== $transUnit->attributes->getNamedItem('tGroupBegin')) {
+            $metadata['tGroupBegin'] = $transUnit->attributes->getNamedItem('tGroupBegin')->nodeValue;
         }
 
         // tGroupEnd
-        if ( null !== $transUnit->attributes->getNamedItem( 'tGroupEnd' ) ) {
-            $metadata[ 'tGroupEnd' ] = $transUnit->attributes->getNamedItem( 'tGroupEnd' )->nodeValue;
+        if (null !== $transUnit->attributes->getNamedItem('tGroupEnd')) {
+            $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;
@@ -286,31 +286,31 @@ 
      * @return array
      * @throws \Exception
      */
-    private function extractTransUnitOriginalData( DOMElement $transUnit ) {
+    private function extractTransUnitOriginalData(DOMElement $transUnit) {
         $originalData = [];
 
         // loop <originalData> to get nested content
-        foreach ( $transUnit->childNodes as $childNode ) {
-            if ( $childNode->nodeName === 'originalData' ) {
-                foreach ( $childNode->childNodes as $data ) {
-                    if ( null !== $data->attributes && null !== $data->attributes->getNamedItem( 'id' ) ) {
-                        $dataId = $data->attributes->getNamedItem( 'id' )->nodeValue;
+        foreach ($transUnit->childNodes as $childNode) {
+            if ($childNode->nodeName === 'originalData') {
+                foreach ($childNode->childNodes as $data) {
+                    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::TAB_PLACEHOLDER, '\t', $dataValue );
+                        $dataValue = str_replace(Placeholder::WHITE_SPACE_PLACEHOLDER, ' ', $data->nodeValue);
+                        $dataValue = str_replace(Placeholder::NEW_LINE_PLACEHOLDER, '\n', $dataValue);
+                        $dataValue = str_replace(Placeholder::TAB_PLACEHOLDER, '\t', $dataValue);
 
-                        if ( '' !== $dataValue ) {
+                        if ('' !== $dataValue) {
 
-                            $jsonOrRawContentArray = $this->JSONOrRawContentArray( $dataValue, false );
+                            $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(
@@ -335,41 +335,41 @@ 
      *
      * @return array
      */
-    private function extractTransUnitAdditionalTagData( DOMElement $transUnit ) {
+    private function extractTransUnitAdditionalTagData(DOMElement $transUnit) {
         $additionalTagData = [];
 
         // loop <originalData> to get nested content
-        foreach ( $transUnit->childNodes as $childNode ) {
-            if ( $childNode->nodeName === 'memsource:additionalTagData' ) {
-                foreach ( $childNode->childNodes as $data ) {
+        foreach ($transUnit->childNodes as $childNode) {
+            if ($childNode->nodeName === 'memsource:additionalTagData') {
+                foreach ($childNode->childNodes as $data) {
                     $dataArray = [];
 
                     // id
-                    if ( $data->nodeName === 'memsource:tag' ) {
-                        if ( null !== $data->attributes && null !== $data->attributes->getNamedItem( 'id' ) ) {
-                            $dataId                      = $data->attributes->getNamedItem( 'id' )->nodeValue;
-                            $dataArray[ 'attr' ][ 'id' ] = $dataId;
+                    if ($data->nodeName === 'memsource:tag') {
+                        if (null !== $data->attributes && null !== $data->attributes->getNamedItem('id')) {
+                            $dataId                      = $data->attributes->getNamedItem('id')->nodeValue;
+                            $dataArray['attr']['id'] = $dataId;
                         }
                     }
 
                     // 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 ) {
-                            if ( $datum->nodeName === 'memsource:tagId' ) {
-                                $dataArray[ 'raw-content' ][ 'tagId' ] = $datum->nodeValue;
+                        foreach ($data->childNodes as $datum) {
+                            if ($datum->nodeName === 'memsource:tagId') {
+                                $dataArray['raw-content']['tagId'] = $datum->nodeValue;
                             }
 
-                            if ( $datum->nodeName === 'memsource:type' ) {
-                                $dataArray[ 'raw-content' ][ 'type' ] = $datum->nodeValue;
+                            if ($datum->nodeName === 'memsource:type') {
+                                $dataArray['raw-content']['type'] = $datum->nodeValue;
                             }
                         }
 
                     }
 
-                    if ( !empty( $dataArray ) ) {
+                    if (!empty($dataArray)) {
                         $additionalTagData[] = $dataArray;
                     }
                 }
@@ -385,15 +385,15 @@ 
      * @param DOMElement $segment
      * @param array      $attr
      */
-    private function checkSegmentIdConsistency( DOMElement $segment, array $attr ) {
-        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' ];
-
-            if ( false === ( ( $min <= $id ) && ( $id <= $max ) ) ) {
-                if ( $this->logger ) {
-                    $this->logger->warning( 'Segment #' . $id . ' is not included within tGroupBegin and tGroupEnd' );
+    private function checkSegmentIdConsistency(DOMElement $segment, array $attr) {
+        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'];
+
+            if (false === (($min <= $id) && ($id <= $max))) {
+                if ($this->logger) {
+                    $this->logger->warning('Segment #' . $id . ' is not included within tGroupBegin and tGroupEnd');
                 }
             }
         }
@@ -405,31 +405,31 @@ 
      * @return array
      * @throws \Exception
      */
-    private function extractTransUnitNotes( DOMElement $transUnit ) {
+    private function extractTransUnitNotes(DOMElement $transUnit) {
         $notes = [];
 
         // loop <notes> to get nested <note> tag
-        foreach ( $transUnit->childNodes as $childNode ) {
-            if ( $childNode->nodeName == 'notes' ) {
-                foreach ( $childNode->childNodes as $note ) {
-                    $noteValue = trim( $note->nodeValue );
-                    if ( '' !== $noteValue ) {
-                        $notes[] = $this->JSONOrRawContentArray( $noteValue );
+        foreach ($transUnit->childNodes as $childNode) {
+            if ($childNode->nodeName == 'notes') {
+                foreach ($childNode->childNodes as $note) {
+                    $noteValue = trim($note->nodeValue);
+                    if ('' !== $noteValue) {
+                        $notes[] = $this->JSONOrRawContentArray($noteValue);
                     }
                 }
             }
 
-            if ( $childNode->nodeName === 'mda:metadata' ) {
-                foreach ( $childNode->childNodes as $metadata ) {
-                    if ( $metadata->nodeName === 'mda:metaGroup' ) {
-                        foreach ( $metadata->childNodes as $meta ) {
-                            if ( null !== $meta->attributes && null !== $meta->attributes->getNamedItem( 'type' ) ) {
-                                $type      = $meta->attributes->getNamedItem( 'type' )->nodeValue;
-                                $metaValue = trim( $meta->nodeValue );
+            if ($childNode->nodeName === 'mda:metadata') {
+                foreach ($childNode->childNodes as $metadata) {
+                    if ($metadata->nodeName === 'mda:metaGroup') {
+                        foreach ($metadata->childNodes as $meta) {
+                            if (null !== $meta->attributes && null !== $meta->attributes->getNamedItem('type')) {
+                                $type      = $meta->attributes->getNamedItem('type')->nodeValue;
+                                $metaValue = trim($meta->nodeValue);
 
-                                if ( '' !== $metaValue ) {
+                                if ('' !== $metaValue) {
                                     $notes[] = array_merge(
-                                            $this->JSONOrRawContentArray( $metaValue ),
+                                            $this->JSONOrRawContentArray($metaValue),
                                             [
                                                     'attr' => [
                                                             'type' => $type

src/XliffReplacer/SdlXliffSAXTranslationReplacer.php

Spacing +47 added lines, -47 removed lines

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

src/Utils/HtmlParser.php

Spacing +92 added lines, -92 removed lines

@@ -15,17 +15,17 @@ 
      *
      * @return array
      */
-    public static function parse( $html ) {
-        $toBeEscaped = Strings::isAnEscapedHTML( $html );
+    public static function parse($html) {
+        $toBeEscaped = Strings::isAnEscapedHTML($html);
 
-        if ( $toBeEscaped ) {
-            $html = Strings::htmlspecialchars_decode( $html );
+        if ($toBeEscaped) {
+            $html = Strings::htmlspecialchars_decode($html);
         }
 
-        $html = self::protectNotClosedHtmlTags( $html );
-        $html = self::protectNotHtmlLessThanSymbols( $html );
+        $html = self::protectNotClosedHtmlTags($html);
+        $html = self::protectNotHtmlLessThanSymbols($html);
 
-        return self::extractHtmlNode( $html, $toBeEscaped );
+        return self::extractHtmlNode($html, $toBeEscaped);
     }
 
     /**
@@ -43,41 +43,41 @@ 
      *
      * @return string
      */
-    private static function protectNotHtmlLessThanSymbols( $html ) {
-        preg_match_all( '/<|>/iu', $html, $matches, PREG_OFFSET_CAPTURE );
+    private static function protectNotHtmlLessThanSymbols($html) {
+        preg_match_all('/<|>/iu', $html, $matches, PREG_OFFSET_CAPTURE);
 
         $delta          = 0;
         $realNextOffset = 0;
         $next           = null;
 
-        foreach ( $matches[ 0 ] as $key => $match ) {
+        foreach ($matches[0] as $key => $match) {
 
-            $current = $matches[ 0 ][ $key ][ 0 ];
+            $current = $matches[0][$key][0];
 
-            if ( isset( $matches[ 0 ][ $key + 1 ][ 0 ] ) ) {
-                $next           = $matches[ 0 ][ $key + 1 ][ 0 ];
-                $nextOffset     = $matches[ 0 ][ $key + 1 ][ 1 ];
-                $realNextOffset = ( $delta === 0 ) ? $nextOffset : ( $nextOffset + $delta );
+            if (isset($matches[0][$key + 1][0])) {
+                $next           = $matches[0][$key + 1][0];
+                $nextOffset     = $matches[0][$key + 1][1];
+                $realNextOffset = ($delta === 0) ? $nextOffset : ($nextOffset + $delta);
             }
 
-            $length     = strlen( $match[ 0 ] );
-            $offset     = $matches[ 0 ][ $key ][ 1 ];
-            $realOffset = ( $delta === 0 ) ? $offset : ( $offset + $delta );
+            $length     = strlen($match[0]);
+            $offset     = $matches[0][$key][1];
+            $realOffset = ($delta === 0) ? $offset : ($offset + $delta);
 
-            if ( $current === '<' && isset( $next ) ) {
+            if ($current === '<' && isset($next)) {
 
                 // 1. if next is > or
                 // 2. next is < and is not html tag (like < >)
-                $insideAngularTags = substr( $html, $realOffset, ( $realNextOffset - $realOffset + 1 ) );
+                $insideAngularTags = substr($html, $realOffset, ($realNextOffset - $realOffset + 1));
 
-                if ( $next !== '>' || !Strings::isHtmlString( $insideAngularTags ) ) {
-                    $html  = substr_replace( $html, self::LT_PLACEHOLDER, $realOffset, $length );
-                    $delta = $delta + strlen( self::LT_PLACEHOLDER ) - $length;
+                if ($next !== '>' || !Strings::isHtmlString($insideAngularTags)) {
+                    $html  = substr_replace($html, self::LT_PLACEHOLDER, $realOffset, $length);
+                    $delta = $delta + strlen(self::LT_PLACEHOLDER) - $length;
                 }
             }
         }
 
-        return !is_array( $html ) ? $html : implode( $html );
+        return !is_array($html) ? $html : implode($html);
     }
 
     /**
@@ -95,29 +95,29 @@ 
      *
      * @return string
      */
-    private static function protectNotClosedHtmlTags( $html ) {
-        preg_match_all( '/<|>/iu', $html, $matches, PREG_OFFSET_CAPTURE );
+    private static function protectNotClosedHtmlTags($html) {
+        preg_match_all('/<|>/iu', $html, $matches, PREG_OFFSET_CAPTURE);
 
         $tags            = [];
         $offsets         = [];
         $originalLengths = [];
 
         // 1. Map all tags
-        foreach ( $matches[ 0 ] as $key => $match ) {
-            $current       = $matches[ 0 ][ $key ][ 0 ];
-            $currentOffset = $matches[ 0 ][ $key ][ 1 ];
+        foreach ($matches[0] as $key => $match) {
+            $current       = $matches[0][$key][0];
+            $currentOffset = $matches[0][$key][1];
 
             // check every string inside angular brackets (< and >)
-            if ( $current === '<' && isset( $matches[ 0 ][ $key + 1 ][ 0 ] ) && $matches[ 0 ][ $key + 1 ][ 0 ] === '>' ) {
-                $nextOffset = $matches[ 0 ][ $key + 1 ][ 1 ];
-                $tag        = substr( $html, ( $currentOffset + 1 ), ( $nextOffset - $currentOffset - 1 ) );
-                $trimmedTag = trim( $tag );
+            if ($current === '<' && isset($matches[0][$key + 1][0]) && $matches[0][$key + 1][0] === '>') {
+                $nextOffset = $matches[0][$key + 1][1];
+                $tag        = substr($html, ($currentOffset + 1), ($nextOffset - $currentOffset - 1));
+                $trimmedTag = trim($tag);
 
                 // if the tag is self closed do nothing
-                if ( Strings::lastChar( $tag ) !== '/' ) {
+                if (Strings::lastChar($tag) !== '/') {
                     $tags[]            = $trimmedTag;
                     $offsets[]         = $currentOffset;
-                    $originalLengths[] = strlen( $tag ) + 2; // add 2 to length because there are < and >
+                    $originalLengths[] = strlen($tag) + 2; // add 2 to length because there are < and >
                 }
             }
         }
@@ -125,47 +125,47 @@ 
         // 2. Removing closed tags
         $indexes = [];
 
-        if ( count( $tags ) > 0 ) {
-            foreach ( $tags as $index => $tag ) {
+        if (count($tags) > 0) {
+            foreach ($tags as $index => $tag) {
 
-                if ( Strings::contains( '/', $tag ) ) {
+                if (Strings::contains('/', $tag)) {
                     $complementaryTag = $tag;
                 } else {
-                    $complementaryTag = '/' . explode( ' ', $tag )[ 0 ];
+                    $complementaryTag = '/' . explode(' ', $tag)[0];
                 }
 
-                $complementaryTagIndex = array_search( $complementaryTag, $tags );
+                $complementaryTagIndex = array_search($complementaryTag, $tags);
 
-                if ( false !== $complementaryTagIndex ) {
+                if (false !== $complementaryTagIndex) {
                     $indexes[] = $index;
                     $indexes[] = $complementaryTagIndex;
                 }
             }
         }
 
-        $indexes = array_unique( $indexes );
-        foreach ( $indexes as $index ) {
-            unset( $tags[ $index ] );
+        $indexes = array_unique($indexes);
+        foreach ($indexes as $index) {
+            unset($tags[$index]);
         }
 
         // 3. Loop not closed tags
         $delta = 0;
 
-        if ( count( $tags ) ) {
-            foreach ( $tags as $index => $tag ) {
+        if (count($tags)) {
+            foreach ($tags as $index => $tag) {
 
-                $length     = $originalLengths[ $index ];
-                $offset     = $offsets[ $index ];
-                $realOffset = ( $delta === 0 ) ? $offset : ( $offset + $delta );
+                $length     = $originalLengths[$index];
+                $offset     = $offsets[$index];
+                $realOffset = ($delta === 0) ? $offset : ($offset + $delta);
 
                 $replacement = self::LT_PLACEHOLDER . $tag . self::GT_PLACEHOLDER;
 
-                $html  = substr_replace( $html, $replacement, $realOffset, $length );
-                $delta = $delta + strlen( $replacement ) - $length;
+                $html  = substr_replace($html, $replacement, $realOffset, $length);
+                $delta = $delta + strlen($replacement) - $length;
             }
         }
 
-        return !is_array( $html ) ? $html : implode( $html );
+        return !is_array($html) ? $html : implode($html);
     }
 
     /**
@@ -174,55 +174,55 @@ 
      *
      * @return array
      */
-    private static function extractHtmlNode( $html, $toBeEscaped = false ) {
+    private static function extractHtmlNode($html, $toBeEscaped = false) {
         $pattern = "/<([a-zA-Z0-9._-]+)([^>]|[^<]*?)(([\s]*\/>)|" .
                 "(>((([^<]*?|<\!\-\-.*?\-\->)|(?R))*)<\/\\1[\s]*>))/sm";
-        preg_match_all( $pattern, $html, $matches, PREG_OFFSET_CAPTURE );
+        preg_match_all($pattern, $html, $matches, PREG_OFFSET_CAPTURE);
 
         $elements = [];
 
-        foreach ( $matches[ 0 ] as $key => $match ) {
+        foreach ($matches[0] as $key => $match) {
 
-            $attributes    = isset( $matches[ 2 ][ $key ][ 0 ] ) ? self::getAttributes( $matches[ 2 ][ $key ][ 0 ] ) : [];
-            $base64Decoded = ( isset( $attributes[ 'equiv-text' ] ) ) ? base64_decode( str_replace( "base64:", "", $attributes[ 'equiv-text' ] ) ) : null;
-            $tagName       = $matches[ 1 ][ $key ][ 0 ];
-            $text          = !empty( $matches[ 6 ][ $key ][ 0 ] ) ? $matches[ 6 ][ $key ][ 0 ] : "";
+            $attributes    = isset($matches[2][$key][0]) ? self::getAttributes($matches[2][$key][0]) : [];
+            $base64Decoded = (isset($attributes['equiv-text'])) ? base64_decode(str_replace("base64:", "", $attributes['equiv-text'])) : null;
+            $tagName       = $matches[1][$key][0];
+            $text          = !empty($matches[6][$key][0]) ? $matches[6][$key][0] : "";
             $originalText  = $text;
-            $strippedText  = strip_tags( $text );
+            $strippedText  = strip_tags($text);
 
             // get start and end tags
-            $explodedNode = explode( self::ORIGINAL_TEXT_PLACEHOLDER, str_replace( $originalText, self::ORIGINAL_TEXT_PLACEHOLDER, $match[ 0 ] ) );
+            $explodedNode = explode(self::ORIGINAL_TEXT_PLACEHOLDER, str_replace($originalText, self::ORIGINAL_TEXT_PLACEHOLDER, $match[0]));
 
-            $start = ( isset( $explodedNode[ 0 ] ) ) ? $explodedNode[ 0 ] : "";
-            $end   = ( isset( $explodedNode[ 1 ] ) ) ? $explodedNode[ 1 ] : "";
+            $start = (isset($explodedNode[0])) ? $explodedNode[0] : "";
+            $end   = (isset($explodedNode[1])) ? $explodedNode[1] : "";
 
             // inner_html
-            $inner_html = self::getInnerHtml( $matches, $key, $toBeEscaped );
+            $inner_html = self::getInnerHtml($matches, $key, $toBeEscaped);
 
             // node
-            $node = self::rebuildNode( $originalText, $toBeEscaped, $start, $end );
+            $node = self::rebuildNode($originalText, $toBeEscaped, $start, $end);
 
             // terminator
-            $terminator = ( $toBeEscaped ) ? '&gt;' : '>';
+            $terminator = ($toBeEscaped) ? '&gt;' : '>';
 
             // self closed
-            $selfClosed = Strings::contains( '/>', trim( $start ) );
+            $selfClosed = Strings::contains('/>', trim($start));
 
             $elements[] = (object)[
-                    'node'           => self::restoreLessThanAndGreaterThanSymbols( $node ),
-                    'start'          => self::restoreLessThanAndGreaterThanSymbols( $start ),
-                    'end'            => self::restoreLessThanAndGreaterThanSymbols( $end ),
+                    'node'           => self::restoreLessThanAndGreaterThanSymbols($node),
+                    'start'          => self::restoreLessThanAndGreaterThanSymbols($start),
+                    'end'            => self::restoreLessThanAndGreaterThanSymbols($end),
                     'terminator'     => $terminator,
-                    'offset'         => $match[ 1 ],
+                    'offset'         => $match[1],
                     'tagname'        => $tagName,
                     'attributes'     => $attributes,
                     'base64_decoded' => $base64Decoded,
                     'self_closed'    => $selfClosed,
-                    'omittag'        => ( $matches[ 4 ][ $key ][ 1 ] > -1 ), // boolean
+                    'omittag'        => ($matches[4][$key][1] > -1), // boolean
                     'inner_html'     => $inner_html,
-                    'has_children'   => is_array( $inner_html ),
-                    'original_text'  => ( $toBeEscaped ) ? self::restoreLessThanAndGreaterThanSymbols( Strings::escapeOnlyHTMLTags( $originalText ) ) : self::restoreLessThanAndGreaterThanSymbols( $originalText ),
-                    'stripped_text'  => self::restoreLessThanAndGreaterThanSymbols( $strippedText ),
+                    'has_children'   => is_array($inner_html),
+                    'original_text'  => ($toBeEscaped) ? self::restoreLessThanAndGreaterThanSymbols(Strings::escapeOnlyHTMLTags($originalText)) : self::restoreLessThanAndGreaterThanSymbols($originalText),
+                    'stripped_text'  => self::restoreLessThanAndGreaterThanSymbols($strippedText),
             ];
         }
 
@@ -234,8 +234,8 @@ 
      *
      * @return string|string[]
      */
-    private static function restoreLessThanAndGreaterThanSymbols( $text ) {
-        return str_replace( [ self::LT_PLACEHOLDER, self::GT_PLACEHOLDER ], [ '<', '>' ], $text );
+    private static function restoreLessThanAndGreaterThanSymbols($text) {
+        return str_replace([self::LT_PLACEHOLDER, self::GT_PLACEHOLDER], ['<', '>'], $text);
     }
 
     /**
@@ -246,17 +246,17 @@ 
      *
      * @return string
      */
-    private static function rebuildNode( $originalText, $toBeEscaped, $start = null, $end = null ) {
+    private static function rebuildNode($originalText, $toBeEscaped, $start = null, $end = null) {
         $node = '';
 
-        if ( !empty( $start ) ) {
-            $node .= ( $toBeEscaped ) ? Strings::escapeOnlyHTMLTags( $start ) : $start;
+        if (!empty($start)) {
+            $node .= ($toBeEscaped) ? Strings::escapeOnlyHTMLTags($start) : $start;
         }
 
-        $node .= ( $toBeEscaped ) ? Strings::escapeOnlyHTMLTags( $originalText ) : $originalText;
+        $node .= ($toBeEscaped) ? Strings::escapeOnlyHTMLTags($originalText) : $originalText;
 
-        if ( !empty( $end ) ) {
-            $node .= ( $toBeEscaped ) ? Strings::escapeOnlyHTMLTags( $end ) : $end;
+        if (!empty($end)) {
+            $node .= ($toBeEscaped) ? Strings::escapeOnlyHTMLTags($end) : $end;
         }
 
         return $node;
@@ -267,16 +267,16 @@ 
      *
      * @return mixed
      */
-    public static function getAttributes( $content ) {
+    public static function getAttributes($content) {
         $pattern = '/(.*?)=("|\'|\\\")(.*?)("|\'|\\\"|\\\')/';
 
-        preg_match_all( $pattern, $content, $matches, PREG_OFFSET_CAPTURE );
+        preg_match_all($pattern, $content, $matches, PREG_OFFSET_CAPTURE);
 
         $attributes = [];
 
-        if ( isset( $matches[ 1 ] ) && count( $matches[ 1 ] ) > 0 ) {
-            foreach ( $matches[ 1 ] as $key => $match ) {
-                $attributes[ trim( $match[ 0 ] ) ] = $matches[ 3 ][ $key ][ 0 ];
+        if (isset($matches[1]) && count($matches[1]) > 0) {
+            foreach ($matches[1] as $key => $match) {
+                $attributes[trim($match[0])] = $matches[3][$key][0];
             }
         }
 
@@ -291,11 +291,11 @@ 
      *
      * @return array|mixed|string
      */
-    private static function getInnerHtml( $matches, $key, $toBeEscaped = false ) {
-        if ( isset( $matches[ 6 ][ $key ][ 0 ] ) ) {
-            $node = self::extractHtmlNode( $matches[ 6 ][ $key ][ 0 ], $toBeEscaped );
+    private static function getInnerHtml($matches, $key, $toBeEscaped = false) {
+        if (isset($matches[6][$key][0])) {
+            $node = self::extractHtmlNode($matches[6][$key][0], $toBeEscaped);
 
-            return ( !empty( $node ) ) ? $node : $matches[ 6 ][ $key ][ 0 ];
+            return (!empty($node)) ? $node : $matches[6][$key][0];
         }
 
         return null;

src/XliffUtils/DataRefReplacer.php

Spacing +164 added lines, -164 removed lines

@@ -16,7 +16,7 @@ 
      *
      * @param array $map
      */
-    public function __construct( array $map = null ) {
+    public function __construct(array $map = null) {
         $this->map = $map;
     }
 
@@ -31,38 +31,38 @@ 
      *
      * @return string
      */
-    public function replace( $string ) {
+    public function replace($string) {
         // if map is empty
         // or the string has not a dataRef attribute
         // return string as is
-        if ( empty( $this->map ) || !$this->hasAnyDataRefAttribute( $string ) ) {
+        if (empty($this->map) || !$this->hasAnyDataRefAttribute($string)) {
             return $string;
         }
 
         // (recursively) clean string from equiv-text eventually present
-        $string = $this->cleanFromEquivText( $string );
+        $string = $this->cleanFromEquivText($string);
 
-        $html = HtmlParser::parse( $string );
+        $html = HtmlParser::parse($string);
 
         // 1. Replace <ph>|<sc>|<ec> tags
-        foreach ( $html as $node ) {
-            $string = $this->recursiveAddEquivTextToPhTag( $node, $string );
+        foreach ($html as $node) {
+            $string = $this->recursiveAddEquivTextToPhTag($node, $string);
         }
 
         // 2. Replace <pc> tags
-        $toBeEscaped = Strings::isAnEscapedHTML( $string );
+        $toBeEscaped = Strings::isAnEscapedHTML($string);
 
-        if ( $this->stringContainsPcTags( $string, $toBeEscaped ) ) {
+        if ($this->stringContainsPcTags($string, $toBeEscaped)) {
 
             // replace self-closed <pc />
-            $string = $this->replaceSelfClosedPcTags( $string, $toBeEscaped );
+            $string = $this->replaceSelfClosedPcTags($string, $toBeEscaped);
 
             // create a dataRefEnd map
             // (needed for correct handling of </pc> closing tags)
-            $dataRefEndMap = $this->buildDataRefEndMap( $html );
-            $string        = $this->replaceOpeningPcTags( $string, $toBeEscaped );
-            $string        = $this->replaceClosingPcTags( $string, $toBeEscaped, $dataRefEndMap );
-            $string        = ( $toBeEscaped ) ? Strings::escapeOnlyHTMLTags( $string ) : $string;
+            $dataRefEndMap = $this->buildDataRefEndMap($html);
+            $string        = $this->replaceOpeningPcTags($string, $toBeEscaped);
+            $string        = $this->replaceClosingPcTags($string, $toBeEscaped, $dataRefEndMap);
+            $string        = ($toBeEscaped) ? Strings::escapeOnlyHTMLTags($string) : $string;
         }
 
         return $string;
@@ -73,17 +73,17 @@ 
      *
      * @return bool
      */
-    private function hasAnyDataRefAttribute( $string ) {
+    private function hasAnyDataRefAttribute($string) {
         $dataRefTags = [
                 'dataRef',
                 'dataRefStart',
                 '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;
             }
         }
@@ -94,11 +94,11 @@ 
      *
      * @return string
      */
-    private function cleanFromEquivText( $string ) {
-        $html = HtmlParser::parse( $string );
+    private function cleanFromEquivText($string) {
+        $html = HtmlParser::parse($string);
 
-        foreach ( $html as $node ) {
-            $string = $this->recursiveCleanFromEquivText( $node, $string );
+        foreach ($html as $node) {
+            $string = $this->recursiveCleanFromEquivText($node, $string);
         }
 
         return $string;
@@ -117,57 +117,57 @@ 
      *
      * @return string
      */
-    private function recursiveAddEquivTextToPhTag( $node, $string ) {
-        if ( $node->has_children ) {
-            foreach ( $node->inner_html as $childNode ) {
-                $string = $this->recursiveAddEquivTextToPhTag( $childNode, $string );
+    private function recursiveAddEquivTextToPhTag($node, $string) {
+        if ($node->has_children) {
+            foreach ($node->inner_html as $childNode) {
+                $string = $this->recursiveAddEquivTextToPhTag($childNode, $string);
             }
         } else {
-            if ( $node->tagname === 'ph' || $node->tagname === 'sc' || $node->tagname === 'ec' ) {
-                if ( !isset( $node->attributes[ 'dataRef' ] ) ) {
+            if ($node->tagname === 'ph' || $node->tagname === 'sc' || $node->tagname === 'ec') {
+                if (!isset($node->attributes['dataRef'])) {
                     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 ] ) ) {
-                    $this->map[ $b ] = 'NULL';
+                if (is_null($this->map[$b])) {
+                    $this->map[$b] = 'NULL';
                 }
 
-                $value              = $this->map[ $b ];
-                $base64EncodedValue = base64_encode( $value );
+                $value              = $this->map[$b];
+                $base64EncodedValue = base64_encode($value);
 
-                if ( empty( $base64EncodedValue ) || $base64EncodedValue === '' ) {
+                if (empty($base64EncodedValue) || $base64EncodedValue === '') {
                     return $string;
                 }
 
                 // 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 );
-                    $d = trim( $d );
+                if ($this->isAEcOrScTag($node)) {
+                    $d = 'ph' . substr($d, 2);
+                    $d = trim($d);
                 }
 
-                return str_replace( $a, $d, $string );
+                return str_replace($a, $d, $string);
             }
         }
 
@@ -180,11 +180,11 @@ 
      *
      * @return bool
      */
-    private function stringContainsPcTags( $string, $toBeEscaped ) {
-        $regex = ( $toBeEscaped ) ? '/&lt;pc (.*?)&gt;/iu' : '/<pc (.*?)>/iu';
-        preg_match_all( $regex, $string, $openingPcMatches );
+    private function stringContainsPcTags($string, $toBeEscaped) {
+        $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);
     }
 
     /**
@@ -193,28 +193,28 @@ 
      *
      * @return mixed
      */
-    private function replaceSelfClosedPcTags( $string, $toBeEscaped ) {
-        if ( $toBeEscaped ) {
-            $string = str_replace( [ '&lt;', '&gt;' ], [ '<', '>' ], $string );
+    private function replaceSelfClosedPcTags($string, $toBeEscaped) {
+        if ($toBeEscaped) {
+            $string = str_replace(['&lt;', '&gt;'], ['<', '>'], $string);
         }
 
         $regex = '/<pc[^>]+?\/>/iu';
-        preg_match_all( $regex, $string, $selfClosedPcMatches );
+        preg_match_all($regex, $string, $selfClosedPcMatches);
 
-        foreach ( $selfClosedPcMatches[ 0 ] as $match ) {
+        foreach ($selfClosedPcMatches[0] as $match) {
 
-            $html       = HtmlParser::parse( $match );
-            $node       = $html[ 0 ];
+            $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' ] ] ) . '"/>';
-                $string      = str_replace( $match, $replacement, $string );
+            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;
@@ -228,12 +228,12 @@ 
      *
      * @return array
      */
-    private function buildDataRefEndMap( $html ) {
+    private function buildDataRefEndMap($html) {
         $dataRefEndMap = [];
 
-        foreach ( $html as $index => $node ) {
-            if ( $node->tagname === 'pc' ) {
-                $this->extractDataRefMapRecursively( $node, $dataRefEndMap );
+        foreach ($html as $index => $node) {
+            if ($node->tagname === 'pc') {
+                $this->extractDataRefMapRecursively($node, $dataRefEndMap);
             }
         }
 
@@ -246,25 +246,25 @@ 
      * @param object $node
      * @param        $dataRefEndMap
      */
-    private function extractDataRefMapRecursively( $node, &$dataRefEndMap ) {
-        if ( $this->nodeContainsNestedPcTags( $node ) ) {
-            foreach ( $node->inner_html as $nestedNode ) {
-                $this->extractDataRefMapRecursively( $nestedNode, $dataRefEndMap );
+    private function extractDataRefMapRecursively($node, &$dataRefEndMap) {
+        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' ] ) ) {
-                $dataRefEnd = $node->attributes[ 'dataRefEnd' ];
-            } elseif ( isset( $node->attributes[ 'dataRefStart' ] ) ) {
-                $dataRefEnd = $node->attributes[ 'dataRefStart' ];
+        if ($node->tagname === 'pc' && $node->self_closed === false) {
+            if (isset($node->attributes['dataRefEnd'])) {
+                $dataRefEnd = $node->attributes['dataRefEnd'];
+            } 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,
             ];
         }
@@ -276,15 +276,15 @@ 
      *
      * @return string|string[]
      */
-    private function recursiveCleanFromEquivText( $node, $string ) {
-        if ( $node->has_children ) {
-            foreach ( $node->inner_html as $childNode ) {
-                $string = $this->recursiveCleanFromEquivText( $childNode, $string );
+    private function recursiveCleanFromEquivText($node, $string) {
+        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 ) ) {
-                $cleaned = preg_replace( '/ equiv-text="(.*?)"/', '', $node->node );
-                $string  = str_replace( $node->node, $cleaned, $string );
+            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);
             }
         }
 
@@ -299,35 +299,35 @@ 
      *
      * @return string
      */
-    private function replaceOpeningPcTags( $string, $toBeEscaped ) {
-        $regex = ( $toBeEscaped ) ? '/&lt;pc (.*?)&gt;/iu' : '/<pc (.*?)>/iu';
-        preg_match_all( $regex, $string, $openingPcMatches );
+    private function replaceOpeningPcTags($string, $toBeEscaped) {
+        $regex = ($toBeEscaped) ? '/&lt;pc (.*?)&gt;/iu' : '/<pc (.*?)>/iu';
+        preg_match_all($regex, $string, $openingPcMatches);
 
-        foreach ( $openingPcMatches[ 0 ] as $index => $match ) {
-            $attr = HtmlParser::getAttributes( $openingPcMatches[ 1 ][ $index ] );
+        foreach ($openingPcMatches[0] as $index => $match) {
+            $attr = HtmlParser::getAttributes($openingPcMatches[1][$index]);
 
             // CASE 1 - Missing `dataRefStart`
-            if ( isset( $attr[ 'dataRefEnd' ] ) && !isset( $attr[ 'dataRefStart' ] ) ) {
-                $attr[ 'dataRefStart' ] = $attr[ 'dataRefEnd' ];
+            if (isset($attr['dataRefEnd']) && !isset($attr['dataRefStart'])) {
+                $attr['dataRefStart'] = $attr['dataRefEnd'];
             }
 
             // CASE 2 - Missing `dataRefEnd`
-            if ( isset( $attr[ 'dataRefStart' ] ) && !isset( $attr[ 'dataRefEnd' ] ) ) {
-                $attr[ 'dataRefEnd' ] = $attr[ 'dataRefStart' ];
+            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 );
+                $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:'
+                $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 );
+                $string = str_replace($startOriginalData, $openingPcConverted, $string);
             }
         }
 
@@ -344,34 +344,34 @@ 
      *
      * @return string
      */
-    private function replaceClosingPcTags( $string, $toBeEscaped, $dataRefEndMap = [] ) {
-        $regex = ( $toBeEscaped ) ? '/&lt;\/pc&gt;/iu' : '/<\/pc>/iu';
-        preg_match_all( $regex, $string, $closingPcMatches, PREG_OFFSET_CAPTURE );
+    private function replaceClosingPcTags($string, $toBeEscaped, $dataRefEndMap = []) {
+        $regex = ($toBeEscaped) ? '/&lt;\/pc&gt;/iu' : '/<\/pc>/iu';
+        preg_match_all($regex, $string, $closingPcMatches, PREG_OFFSET_CAPTURE);
         $delta = 0;
 
-        foreach ( $closingPcMatches[ 0 ] as $index => $match ) {
-            $offset = $match[ 1 ];
-            $length = strlen( $match[ 0 ] );
-            $attr   = $dataRefEndMap[ $index ];
+        foreach ($closingPcMatches[0] as $index => $match) {
+            $offset = $match[1];
+            $length = strlen($match[0]);
+            $attr   = $dataRefEndMap[$index];
 
-            if ( !empty( $attr ) && isset( $attr[ 'dataRefEnd' ] ) ) {
-                $endOriginalData       = $match[ 0 ]; // </pc>
-                $endValue              = $this->map[ $attr[ 'dataRefEnd' ] ] ?: 'NULL';
-                $base64EncodedEndValue = base64_encode( $endValue );
-                $base64EndOriginalData = base64_encode( $endOriginalData );
+            if (!empty($attr) && isset($attr['dataRefEnd'])) {
+                $endOriginalData       = $match[0]; // </pc>
+                $endValue              = $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 );
+                $realOffset = ($delta === 0) ? $offset : ($offset + $delta);
 
-                $string = substr_replace( $string, $closingPcConverted, $realOffset, $length );
-                $delta  = $delta + strlen( $closingPcConverted ) - $length;
+                $string = substr_replace($string, $closingPcConverted, $realOffset, $length);
+                $delta  = $delta + strlen($closingPcConverted) - $length;
             }
         }
 
-        return !is_array( $string ) ? $string : implode( $string );
+        return !is_array($string) ? $string : implode($string);
     }
 
     /**
@@ -379,13 +379,13 @@ 
      *
      * @return bool
      */
-    private function nodeContainsNestedPcTags( $node ) {
-        if ( !$node->has_children ) {
+    private function nodeContainsNestedPcTags($node) {
+        if (!$node->has_children) {
             return false;
         }
 
-        foreach ( $node->inner_html as $nestedNode ) {
-            if ( $nestedNode->tagname === 'pc' && ( isset( $node->attributes[ 'dataRefEnd' ] ) || isset( $node->attributes[ 'dataRefStart' ] ) ) ) {
+        foreach ($node->inner_html as $nestedNode) {
+            if ($nestedNode->tagname === 'pc' && (isset($node->attributes['dataRefEnd']) || isset($node->attributes['dataRefStart']))) {
                 return true;
             }
         }
@@ -398,18 +398,18 @@ 
      *
      * @return string
      */
-    public function restore( $string ) {
+    public function restore($string) {
         // if map is empty return string as is
-        if ( empty( $this->map ) ) {
+        if (empty($this->map)) {
             return $string;
         }
 
         // replace eventual empty equiv-text=""
-        $string = str_replace( ' equiv-text=""', '', $string );
-        $html   = HtmlParser::parse( $string );
+        $string = str_replace(' equiv-text=""', '', $string);
+        $html   = HtmlParser::parse($string);
 
-        foreach ( $html as $node ) {
-            $string = $this->recursiveRemoveOriginalData( $node, $string );
+        foreach ($html as $node) {
+            $string = $this->recursiveRemoveOriginalData($node, $string);
         }
 
         return $string;
@@ -421,76 +421,76 @@ 
      *
      * @return string|string[]
      */
-    private function recursiveRemoveOriginalData( $node, $string ) {
-        if ( $node->has_children ) {
-            foreach ( $node->inner_html as $childNode ) {
-                $string = $this->recursiveRemoveOriginalData( $childNode, $string );
+    private function recursiveRemoveOriginalData($node, $string) {
+        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"/>
-            $b = $node->attributes[ 'dataRef' ]; // map identifier. Eg: source1
-            $c = $node->terminator;            // terminator: Eg: >
+            $a = $node->node; // complete match. Eg:  <ph id="source1" dataRef="source1"/>
+            $b = $node->attributes['dataRef']; // map identifier. Eg: source1
+            $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 ] ) ) {
-                $this->map[ $b ] = 'NULL';
+            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 );
-                $d = trim( $d );
+            if ($this->wasAEcOrScTag($node)) {
+                $d = $node->attributes['dataType'] . substr($d, 3);
+                $d = trim($d);
             }
 
             // replace only content tag, no matter if the string is encoded or not
             // in this way we can handle string with mixed tags (encoded and not-encoded)
             // in the same string
-            $a = $this->purgeTags( $a );
-            $d = $this->purgeTags( $d );
+            $a = $this->purgeTags($a);
+            $d = $this->purgeTags($d);
 
-            $string = str_replace( $a, $d, $string );
+            $string = str_replace($a, $d, $string);
 
             // restoring <pc/> self-closed here
-            if ( Strings::contains( 'dataType="pcSelf"', $d ) ) {
-                preg_match( '/\s?originalData="(.*?)"\s?/', $d, $originalDataMatches );
+            if (Strings::contains('dataType="pcSelf"', $d)) {
+                preg_match('/\s?originalData="(.*?)"\s?/', $d, $originalDataMatches);
 
-                if ( isset( $originalDataMatches[ 1 ] ) ) {
-                    $originalData = base64_decode( $originalDataMatches[ 1 ] );
-                    $originalData = $this->purgeTags( $originalData );
-                    $string       = str_replace( $d, $originalData, $string );
+                if (isset($originalDataMatches[1])) {
+                    $originalData = base64_decode($originalDataMatches[1]);
+                    $originalData = $this->purgeTags($originalData);
+                    $string       = str_replace($d, $originalData, $string);
                 }
             }
 
             // restoring <pc> tags here
             // if <ph> tag has originalData and originalType is pcStart or pcEnd,
             // replace with original data
-            if ( Strings::contains( 'dataType="pcStart"', $d ) || Strings::contains( 'dataType="pcEnd"', $d ) ) {
-                preg_match( '/\s?originalData="(.*?)"\s?/', $d, $originalDataMatches );
+            if (Strings::contains('dataType="pcStart"', $d) || Strings::contains('dataType="pcEnd"', $d)) {
+                preg_match('/\s?originalData="(.*?)"\s?/', $d, $originalDataMatches);
 
-                if ( isset( $originalDataMatches[ 1 ] ) ) {
-                    $originalData = base64_decode( $originalDataMatches[ 1 ] );
-                    $originalData = $this->purgeTags( $originalData );
-                    $string       = str_replace( $d, $originalData, $string );
+                if (isset($originalDataMatches[1])) {
+                    $originalData = base64_decode($originalDataMatches[1]);
+                    $originalData = $this->purgeTags($originalData);
+                    $string       = str_replace($d, $originalData, $string);
                 }
             }
         }
@@ -503,8 +503,8 @@ 
      *
      * @return string
      */
-    private function purgeTags( $string ) {
-        return str_replace( [ '<', '>', '&lt;', '&gt;' ], '', $string );
+    private function purgeTags($string) {
+        return str_replace(['<', '>', '&lt;', '&gt;'], '', $string);
     }
 
     /**
@@ -514,8 +514,8 @@ 
      *
      * @return bool
      */
-    private function isAEcOrScTag( $node ) {
-        return ( $node->tagname === 'ec' || $node->tagname === 'sc' );
+    private function isAEcOrScTag($node) {
+        return ($node->tagname === 'ec' || $node->tagname === 'sc');
     }
 
     /**
@@ -526,7 +526,7 @@ 
      *
      * @return bool
      */
-    private function wasAEcOrScTag( $node ) {
-        return ( isset( $node->attributes[ 'dataType' ] ) && ( $node->attributes[ 'dataType' ] === 'ec' || $node->attributes[ 'dataType' ] === 'sc' ) );
+    private function wasAEcOrScTag($node) {
+        return (isset($node->attributes['dataType']) && ($node->attributes['dataType'] === 'ec' || $node->attributes['dataType'] === 'sc'));
     }
 }