matecat / xliff-parser

src/XliffParser/XliffParserV1.php

Spacing +88 added lines, -88 removed lines

@@ -15,33 +15,33 @@ 
      * @inheritDoc
      * @throws Exception
      */
-    public function parse( DOMDocument $dom, $output = [] ) {
+    public function parse(DOMDocument $dom, $output = []) {
         $i = 1;
         /** @var DOMElement $file */
-        foreach ( $dom->getElementsByTagName( 'file' ) as $file ) {
+        foreach ($dom->getElementsByTagName('file') as $file) {
 
             // metadata
-            $output[ 'files' ][ $i ][ 'attr' ] = $this->extractMetadata( $file );
+            $output['files'][$i]['attr'] = $this->extractMetadata($file);
 
             // reference
-            if ( !empty( $this->extractReference( $file ) ) ) {
-                $output[ 'files' ][ $i ][ 'reference' ] = $this->extractReference( $file );
+            if (!empty($this->extractReference($file))) {
+                $output['files'][$i]['reference'] = $this->extractReference($file);
             }
 
             // trans-units
             $transUnitIdArrayForUniquenessCheck = [];
             $j                                  = 1;
-            foreach ( $file->childNodes as $body ) {
-                if ( $body->nodeName === 'body' ) {
-                    foreach ( $body->childNodes as $childNode ) {
-                        $this->extractTuFromNode( $childNode, $transUnitIdArrayForUniquenessCheck, $dom, $output, $i, $j );
+            foreach ($file->childNodes as $body) {
+                if ($body->nodeName === 'body') {
+                    foreach ($body->childNodes as $childNode) {
+                        $this->extractTuFromNode($childNode, $transUnitIdArrayForUniquenessCheck, $dom, $output, $i, $j);
                     }
 
                     // trans-unit re-count check
-                    $totalTransUnitsId  = count( $transUnitIdArrayForUniquenessCheck );
-                    $transUnitsUniqueId = count( array_unique( $transUnitIdArrayForUniquenessCheck ) );
-                    if ( $totalTransUnitsId != $transUnitsUniqueId ) {
-                        throw new DuplicateTransUnitIdInXliff( "Invalid trans-unit id, duplicate found.", 400 );
+                    $totalTransUnitsId  = count($transUnitIdArrayForUniquenessCheck);
+                    $transUnitsUniqueId = count(array_unique($transUnitIdArrayForUniquenessCheck));
+                    if ($totalTransUnitsId != $transUnitsUniqueId) {
+                        throw new DuplicateTransUnitIdInXliff("Invalid trans-unit id, duplicate found.", 400);
                     }
 
                     $i++;
@@ -57,50 +57,50 @@ 
      *
      * @return array
      */
-    private function extractMetadata( DOMElement $file ) {
+    private function extractMetadata(DOMElement $file) {
         $metadata   = [];
         $customAttr = [];
 
         /** @var \DOMAttr $attribute */
-        foreach ( $file->attributes as $attribute ) {
-            switch ( $attribute->localName ) {
+        foreach ($file->attributes as $attribute) {
+            switch ($attribute->localName) {
                 // original
                 case 'original':
-                    $metadata[ 'original' ] = $attribute->value;
+                    $metadata['original'] = $attribute->value;
                     break;
 
                 // source-language
                 case 'source-language':
-                    $metadata[ 'source-language' ] = $attribute->value;
+                    $metadata['source-language'] = $attribute->value;
                     break;
 
                 // data-type
                 case 'datatype':
-                    $metadata[ 'data-type' ] = $attribute->value;
+                    $metadata['data-type'] = $attribute->value;
                     break;
 
                 // target-language
                 case 'target-language':
-                    $metadata[ 'target-language' ] = $attribute->value;
+                    $metadata['target-language'] = $attribute->value;
                     break;
             }
 
             // Custom MateCat x-Attribute
-            preg_match( '|x-(.*?)|si', $attribute->localName, $temp );
-            if ( isset( $temp[ 1 ] ) ) {
-                $customAttr[ $attribute->localName ] = $attribute->value;
+            preg_match('|x-(.*?)|si', $attribute->localName, $temp);
+            if (isset($temp[1])) {
+                $customAttr[$attribute->localName] = $attribute->value;
             }
-            unset( $temp );
+            unset($temp);
 
             // Custom MateCat namespace Attribute mtc:
-            preg_match( '|mtc:(.*?)|si', $attribute->nodeName, $temp );
-            if ( isset( $temp[ 1 ] ) ) {
-                $customAttr[ $attribute->nodeName ] = $attribute->value;
+            preg_match('|mtc:(.*?)|si', $attribute->nodeName, $temp);
+            if (isset($temp[1])) {
+                $customAttr[$attribute->nodeName] = $attribute->value;
             }
-            unset( $temp );
+            unset($temp);
 
-            if ( !empty( $customAttr ) ) {
-                $metadata[ 'custom' ] = $customAttr;
+            if (!empty($customAttr)) {
+                $metadata['custom'] = $customAttr;
             }
         }
 
@@ -112,16 +112,16 @@ 
      *
      * @return array
      */
-    private function extractReference( DOMElement $file ) {
+    private function extractReference(DOMElement $file) {
         $reference = [];
 
         $order = 0;
-        foreach ( $file->getElementsByTagName( 'reference' ) as $ref ) {
+        foreach ($file->getElementsByTagName('reference') as $ref) {
             /** @var DOMNode $childNode */
-            foreach ( $ref->childNodes as $childNode ) {
-                if ( $childNode->nodeName === 'internal-file' ) {
-                    $reference[ $order ][ 'form-type' ] = $childNode->attributes->getNamedItem( 'form' )->nodeValue;
-                    $reference[ $order ][ 'base64' ]    = trim( $childNode->nodeValue );
+            foreach ($ref->childNodes as $childNode) {
+                if ($childNode->nodeName === 'internal-file') {
+                    $reference[$order]['form-type'] = $childNode->attributes->getNamedItem('form')->nodeValue;
+                    $reference[$order]['base64']    = trim($childNode->nodeValue);
                     $order++;
                 }
             }
@@ -142,53 +142,53 @@ 
      *
      * @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
-        $output[ 'files' ][ $i ][ 'trans-units' ][ $j ][ 'notes' ] = $this->extractTransUnitNotes( $dom, $transUnit );
+        $output['files'][$i]['trans-units'][$j]['notes'] = $this->extractTransUnitNotes($dom, $transUnit);
 
         // content
         /** @var DOMElement $childNode */
-        foreach ( $transUnit->childNodes as $childNode ) {
+        foreach ($transUnit->childNodes as $childNode) {
             // source
-            if ( $childNode->nodeName === 'source' ) {
-                $output[ 'files' ][ $i ][ 'trans-units' ][ $j ][ 'source' ] = $this->extractContent( $dom, $childNode );
+            if ($childNode->nodeName === 'source') {
+                $output['files'][$i]['trans-units'][$j]['source'] = $this->extractContent($dom, $childNode);
             }
 
             // seg-source
-            if ( $childNode->nodeName === 'seg-source' ) {
-                $rawSegment                                                     = $output[ 'files' ][ $i ][ 'trans-units' ][ $j ][ 'source' ][ 'raw-content' ];
-                $output[ 'files' ][ $i ][ 'trans-units' ][ $j ][ 'seg-source' ] = $this->extractContentWithMarksAndExtTags( $dom, $childNode, $rawSegment );
+            if ($childNode->nodeName === 'seg-source') {
+                $rawSegment                                                     = $output['files'][$i]['trans-units'][$j]['source']['raw-content'];
+                $output['files'][$i]['trans-units'][$j]['seg-source'] = $this->extractContentWithMarksAndExtTags($dom, $childNode, $rawSegment);
             }
 
             // target
-            if ( $childNode->nodeName === 'target' ) {
-                $output[ 'files' ][ $i ][ 'trans-units' ][ $j ][ 'target' ] = $this->extractContent( $dom, $childNode );
+            if ($childNode->nodeName === 'target') {
+                $output['files'][$i]['trans-units'][$j]['target'] = $this->extractContent($dom, $childNode);
 
                 // seg-target
-                $targetRawContent = @$output[ 'files' ][ $i ][ 'trans-units' ][ $j ][ 'target' ][ 'raw-content' ];
-                $segSource        = @$output[ 'files' ][ $i ][ 'trans-units' ][ $j ][ 'seg-source' ];
-                if ( isset( $targetRawContent ) && !empty( $targetRawContent ) && isset( $segSource ) && count( $segSource ) > 0 ) {
-                    $output[ 'files' ][ $i ][ 'trans-units' ][ $j ][ 'seg-target' ] = $this->extractContentWithMarksAndExtTags( $dom, $childNode, $targetRawContent );
+                $targetRawContent = @$output['files'][$i]['trans-units'][$j]['target']['raw-content'];
+                $segSource        = @$output['files'][$i]['trans-units'][$j]['seg-source'];
+                if (isset($targetRawContent) && !empty($targetRawContent) && isset($segSource) && count($segSource) > 0) {
+                    $output['files'][$i]['trans-units'][$j]['seg-target'] = $this->extractContentWithMarksAndExtTags($dom, $childNode, $targetRawContent);
                 }
             }
 
             // locked
-            if ( $childNode->nodeName === 'sdl:seg' ) {
-                $output[ 'files' ][ $i ][ 'trans-units' ][ $j ][ 'locked' ] = $this->extractLocked( $childNode );
+            if ($childNode->nodeName === 'sdl:seg') {
+                $output['files'][$i]['trans-units'][$j]['locked'] = $this->extractLocked($childNode);
             }
         }
 
         // context-group
-        foreach ( $transUnit->getElementsByTagName( 'context-group' ) as $contextGroup ) {
-            $output[ 'files' ][ $i ][ 'trans-units' ][ $j ][ 'context-group' ][] = $this->extractTransUnitContextGroup( $dom, $contextGroup );
+        foreach ($transUnit->getElementsByTagName('context-group') as $contextGroup) {
+            $output['files'][$i]['trans-units'][$j]['context-group'][] = $this->extractTransUnitContextGroup($dom, $contextGroup);
         }
 
         // alt-trans
-        foreach ( $transUnit->getElementsByTagName( 'alt-trans' ) as $altTrans ) {
-            $output[ 'files' ][ $i ][ 'trans-units' ][ $j ][ 'alt-trans' ][] = $this->extractTransUnitAltTrans( $altTrans );
+        foreach ($transUnit->getElementsByTagName('alt-trans') as $altTrans) {
+            $output['files'][$i]['trans-units'][$j]['alt-trans'][] = $this->extractTransUnitAltTrans($altTrans);
         }
 
         $j++;
@@ -201,32 +201,32 @@ 
      * @return array
      * @throws Exception
      */
-    private function extractTransUnitMetadata( DOMElement $transUnit, array &$transUnitIdArrayForUniquenessCheck ) {
+    private function extractTransUnitMetadata(DOMElement $transUnit, array &$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;
         }
 
         // approved
         // http://docs.oasis-open.org/xliff/v1.2/os/xliff-core.html#approved
-        if ( null !== $transUnit->attributes->getNamedItem( 'approved' ) ) {
-            $metadata[ 'approved' ] = filter_var( $transUnit->attributes->getNamedItem( 'approved' )->nodeValue, FILTER_VALIDATE_BOOLEAN );
+        if (null !== $transUnit->attributes->getNamedItem('approved')) {
+            $metadata['approved'] = filter_var($transUnit->attributes->getNamedItem('approved')->nodeValue, FILTER_VALIDATE_BOOLEAN);
         }
 
         return $metadata;
@@ -238,19 +238,19 @@ 
      * @return array
      * @throws Exception
      */
-    private function extractTransUnitNotes( DOMDocument $dom, DOMElement $transUnit ) {
+    private function extractTransUnitNotes(DOMDocument $dom, DOMElement $transUnit) {
         $notes = [];
-        foreach ( $transUnit->getElementsByTagName( 'note' ) as $note ) {
+        foreach ($transUnit->getElementsByTagName('note') as $note) {
 
-            $noteValue = $this->extractTagContent( $dom, $note );
+            $noteValue = $this->extractTagContent($dom, $note);
 
-            if ( '' !== $noteValue ) {
+            if ('' !== $noteValue) {
 
-                $extractedNote = $this->JSONOrRawContentArray( $noteValue );
+                $extractedNote = $this->JSONOrRawContentArray($noteValue);
 
                 // extract all the attributes
-                foreach ( $note->attributes as $attribute ) {
-                    $extractedNote[ $attribute->name ] = $attribute->value;
+                foreach ($note->attributes as $attribute) {
+                    $extractedNote[$attribute->name] = $attribute->value;
                 }
 
                 $notes[] = $extractedNote;
@@ -265,14 +265,14 @@ 
      *
      * @return array
      */
-    private function extractTransUnitContextGroup( DOMDocument $dom, DOMElement $contextGroup ) {
+    private function extractTransUnitContextGroup(DOMDocument $dom, DOMElement $contextGroup) {
         $cg           = [];
-        $cg[ 'attr' ] = $this->extractTagAttributes( $contextGroup );
+        $cg['attr'] = $this->extractTagAttributes($contextGroup);
 
         /** @var DOMNode $context */
-        foreach ( $contextGroup->childNodes as $context ) {
-            if ( $context->nodeName === 'context' ) {
-                $cg[ 'contexts' ][] = $this->extractContent( $dom, $context );
+        foreach ($contextGroup->childNodes as $context) {
+            if ($context->nodeName === 'context') {
+                $cg['contexts'][] = $this->extractContent($dom, $context);
             }
         }
 
@@ -284,16 +284,16 @@ 
      *
      * @return array
      */
-    private function extractTransUnitAltTrans( DOMElement $altTrans ) {
+    private function extractTransUnitAltTrans(DOMElement $altTrans) {
         $at           = [];
-        $at[ 'attr' ] = $this->extractTagAttributes( $altTrans );
+        $at['attr'] = $this->extractTagAttributes($altTrans);
 
-        if ( $altTrans->getElementsByTagName( 'source' )->length > 0 ) {
-            $at[ 'source' ] = $altTrans->getElementsByTagName( 'source' )->item( 0 )->nodeValue;
+        if ($altTrans->getElementsByTagName('source')->length > 0) {
+            $at['source'] = $altTrans->getElementsByTagName('source')->item(0)->nodeValue;
         }
 
-        if ( $altTrans->getElementsByTagName( 'target' ) ) {
-            $at[ 'target' ] = $altTrans->getElementsByTagName( 'target' )->item( 0 )->nodeValue;
+        if ($altTrans->getElementsByTagName('target')) {
+            $at['target'] = $altTrans->getElementsByTagName('target')->item(0)->nodeValue;
         }
 
         return $at;
@@ -304,7 +304,7 @@ 
      *
      * @return bool
      */
-    private function extractLocked( DOMElement $locked ) {
-        return null !== $locked->getAttribute( 'locked' );
+    private function extractLocked(DOMElement $locked) {
+        return null !== $locked->getAttribute('locked');
     }
 }

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,33 +113,33 @@ 
      *
      * @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 );
-        if ( !empty( $originalData ) ) {
-            $output[ 'files' ][ $i ][ 'trans-units' ][ $j ][ 'original-data' ] = $originalData;
-            $dataRefMap                                                        = $this->getDataRefMap( $originalData );
+        $originalData = $this->extractTransUnitOriginalData($transUnit);
+        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
@@ -159,62 +159,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'    => '',
                             ];
                         }
@@ -225,10 +225,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++;
     }
@@ -239,41 +239,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;
@@ -285,31 +285,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(
@@ -334,41 +334,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;
                     }
                 }
@@ -384,15 +384,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');
                 }
             }
         }
@@ -404,31 +404,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/Utils/HtmlParser.php

Spacing +29 added lines, -29 removed lines

@@ -55,11 +55,11 @@ 
 
         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];
+            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);
             }
 
@@ -67,13 +67,13 @@ 
             $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) ){
+                if ($next !== '>' || !Strings::isHtmlString($insideAngularTags)) {
                     $html = substr_replace($html, self::LT_PLACEHOLDER, $realOffset, $length);
                     $delta = $delta + strlen(self::LT_PLACEHOLDER) - $length;
                 }
@@ -98,7 +98,7 @@ 
      *
      * @return string
      */
-    private static function protectNotClosedHtmlTags( $html)
+    private static function protectNotClosedHtmlTags($html)
     {
         preg_match_all('/<|>/iu', $html, $matches, PREG_OFFSET_CAPTURE);
 
@@ -108,17 +108,17 @@ 
 
         // 1. Map all tags
         foreach ($matches[0] as $key => $match) {
-            $current       = $matches[ 0 ][ $key ][ 0 ];
-            $currentOffset = $matches[ 0 ][ $key ][ 1 ];
+            $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 ));
+            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 >
@@ -129,18 +129,18 @@ 
         // 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);
 
-                if(false !== $complementaryTagIndex){
+                if (false !== $complementaryTagIndex) {
                     $indexes[] = $index;
                     $indexes[] = $complementaryTagIndex;
                 }
@@ -148,15 +148,15 @@ 
         }
 
         $indexes = array_unique($indexes);
-        foreach ($indexes as $index){
+        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];
@@ -178,9 +178,9 @@ 
      *
      * @return array
      */
-    private static function extractHtmlNode( $html, $toBeEscaped = false)
+    private static function extractHtmlNode($html, $toBeEscaped = false)
     {
-        $pattern = "/<([a-zA-Z0-9._-]+)([^>]|[^<]*?)(([\s]*\/>)|".
+        $pattern = "/<([a-zA-Z0-9._-]+)([^>]|[^<]*?)(([\s]*\/>)|" .
                 "(>((([^<]*?|<\!\-\-.*?\-\->)|(?R))*)<\/\\1[\s]*>))/sm";
         preg_match_all($pattern, $html, $matches, PREG_OFFSET_CAPTURE);
 
@@ -189,7 +189,7 @@ 
         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;
+            $base64Decoded = (isset($attributes['equiv-text'])) ? base64_decode(str_replace("base64:", "", $attributes['equiv-text'])) : null;
             $tagName = $matches[1][$key][0];
             $text = (isset($matches[6][$key][0]) && '' !== $matches[6][$key][0]) ? $matches[6][$key][0] : null;
             $originalText = $text;
@@ -239,9 +239,9 @@ 
      *
      * @return string|string[]
      */
-    private static function restoreLessThanAndGreaterThanSymbols( $text)
+    private static function restoreLessThanAndGreaterThanSymbols($text)
     {
-        return str_replace([self::LT_PLACEHOLDER, self::GT_PLACEHOLDER], ['<','>'], $text);
+        return str_replace([self::LT_PLACEHOLDER, self::GT_PLACEHOLDER], ['<', '>'], $text);
     }
 
     /**
@@ -256,13 +256,13 @@ 
     {
         $node = '';
 
-        if($start !== null){
+        if ($start !== null) {
             $node .= ($toBeEscaped) ? Strings::escapeOnlyHTMLTags($start) : $start;
         }
 
         $node .= ($toBeEscaped) ? Strings::escapeOnlyHTMLTags($originalText) : $originalText;
 
-        if($end !== null){
+        if ($end !== null) {
             $node .= ($toBeEscaped) ? Strings::escapeOnlyHTMLTags($end) : $end;
         }
 

src/Utils/Strings.php

Spacing +69 added lines, -69 removed lines

@@ -17,8 +17,8 @@ 
      * @return string
      * @throws Exception
      */
-    public static function cleanCDATA( $testString ) {
-        $cleanXMLContent = new SimpleXMLElement( '<rootNoteNode>' . $testString . '</rootNoteNode>', LIBXML_NOCDATA );
+    public static function cleanCDATA($testString) {
+        $cleanXMLContent = new SimpleXMLElement('<rootNoteNode>' . $testString . '</rootNoteNode>', LIBXML_NOCDATA);
 
         return $cleanXMLContent->__toString();
     }
@@ -29,21 +29,21 @@ 
      * @return bool
      * @throws Exception
      */
-    public static function isJSON( $string ) {
-        if ( is_numeric( $string ) ) {
+    public static function isJSON($string) {
+        if (is_numeric($string)) {
             return false;
         }
 
         try {
-            $string = Strings::cleanCDATA( $string );
+            $string = Strings::cleanCDATA($string);
 
-            if ( empty( $string ) ) {
+            if (empty($string)) {
                 throw new Exception();
             }
 
-            json_decode( $string );
+            json_decode($string);
             self::raiseJsonExceptionError();
-        } catch ( Exception $exception ) {
+        } catch (Exception $exception) {
             return false;
         }
 
@@ -55,8 +55,8 @@ 
      *
      * @return array
      */
-    public static function jsonToArray( $string ) {
-        return ( is_array( json_decode( $string, true ) ) ) ? json_decode( $string, true ) : [];
+    public static function jsonToArray($string) {
+        return (is_array(json_decode($string, true))) ? json_decode($string, true) : [];
     }
 
     /**
@@ -65,11 +65,11 @@ 
      * @return string|null
      * @throws NotValidJSONException
      */
-    private static function raiseJsonExceptionError( $raise = true ) {
-        if ( function_exists( "json_last_error" ) ) {
+    private static function raiseJsonExceptionError($raise = true) {
+        if (function_exists("json_last_error")) {
             $error = json_last_error();
 
-            switch ( $error ) {
+            switch ($error) {
                 case JSON_ERROR_NONE:
                     $msg = null; # - No errors
                     break;
@@ -93,9 +93,9 @@ 
                     break;
             }
 
-            if ( $raise && $error != JSON_ERROR_NONE ) {
-                throw new NotValidJSONException( $msg, $error );
-            } elseif ( $error != JSON_ERROR_NONE ) {
+            if ($raise && $error != JSON_ERROR_NONE) {
+                throw new NotValidJSONException($msg, $error);
+            } elseif ($error != JSON_ERROR_NONE) {
                 return $msg;
             }
         }
@@ -120,11 +120,11 @@ 
      *
      * @return string
      */
-    public static function fixNonWellFormedXml( $content, $escapeStrings = true ) {
-        if ( self::$find_xliff_tags_reg === null ) {
+    public static function fixNonWellFormedXml($content, $escapeStrings = true) {
+        if (self::$find_xliff_tags_reg === null) {
             // Convert the list of tags in a regexp list, for example "g|x|bx|ex"
             $xliffTags           = XliffTags::$tags;
-            $xliff_tags_reg_list = implode( '|', $xliffTags );
+            $xliff_tags_reg_list = implode('|', $xliffTags);
             // Regexp to find all the XLIFF tags:
             //   </?               -> matches the tag start, for both opening and
             //                        closure tags (see the optional slash)
@@ -145,30 +145,30 @@ 
         }
 
         // Find all the XLIFF tags
-        preg_match_all( self::$find_xliff_tags_reg, $content, $matches );
-        $tags = (array)$matches[ 0 ];
+        preg_match_all(self::$find_xliff_tags_reg, $content, $matches);
+        $tags = (array)$matches[0];
 
         // Prepare placeholders
         $tags_placeholders = [];
-        $tagsNum           = count( $tags );
-        for ( $i = 0; $i < $tagsNum; $i++ ) {
-            $tag                       = $tags[ $i ];
-            $tags_placeholders[ $tag ] = "#@!XLIFF-TAG-$i!@#";
+        $tagsNum           = count($tags);
+        for ($i = 0; $i < $tagsNum; $i++) {
+            $tag                       = $tags[$i];
+            $tags_placeholders[$tag] = "#@!XLIFF-TAG-$i!@#";
         }
 
         // Replace all XLIFF tags with placeholders that will not be escaped
-        foreach ( $tags_placeholders as $tag => $placeholder ) {
-            $content = str_replace( $tag, $placeholder, $content );
+        foreach ($tags_placeholders as $tag => $placeholder) {
+            $content = str_replace($tag, $placeholder, $content);
         }
 
         // Escape the string with the remaining non-XLIFF tags
-        if ( $escapeStrings ) {
-            $content = htmlspecialchars( $content, ENT_NOQUOTES, 'UTF-8', false );
+        if ($escapeStrings) {
+            $content = htmlspecialchars($content, ENT_NOQUOTES, 'UTF-8', false);
         }
 
         // Put again in place the original XLIFF tags replacing placeholders
-        foreach ( $tags_placeholders as $tag => $placeholder ) {
-            $content = str_replace( $placeholder, $tag, $content );
+        foreach ($tags_placeholders as $tag => $placeholder) {
+            $content = str_replace($placeholder, $tag, $content);
         }
 
         return $content;
@@ -179,17 +179,17 @@ 
      *
      * @return string
      */
-    public static function removeDangerousChars( $string ) {
+    public static function removeDangerousChars($string) {
         // clean invalid xml entities ( characters with ascii < 32 and different from 0A, 0D and 09
         $regexpEntity = '/&#x(0[0-8BCEF]|1[0-9A-F]|7F);/u';
 
         // remove binary chars in some xliff files
         $regexpAscii = '/[\x{00}-\x{08}\x{0B}\x{0C}\x{0E}-\x{1F}\x{7F}]/u';
 
-        $string = preg_replace( $regexpAscii, '', $string );
-        $string = preg_replace( $regexpEntity, '', $string );
+        $string = preg_replace($regexpAscii, '', $string);
+        $string = preg_replace($regexpEntity, '', $string);
 
-        return !empty( $string ) ? $string : "";
+        return !empty($string) ? $string : "";
     }
 
     /**
@@ -198,8 +198,8 @@ 
      *
      * @return bool
      */
-    public static function contains( $needle, $haystack ) {
-        return strpos( $haystack, $needle ) !== false;
+    public static function contains($needle, $haystack) {
+        return strpos($haystack, $needle) !== false;
     }
 
     /**
@@ -207,8 +207,8 @@ 
      *
      * @return string
      */
-    public static function htmlentities( $string ) {
-        return htmlentities( $string, ENT_NOQUOTES );
+    public static function htmlentities($string) {
+        return htmlentities($string, ENT_NOQUOTES);
     }
 
     /**
@@ -217,15 +217,15 @@ 
      *
      * @return string
      */
-    public static function htmlspecialchars_decode( $string, $onlyEscapedEntities = false ) {
-        if ( false === $onlyEscapedEntities ) {
-            return htmlspecialchars_decode( $string, ENT_NOQUOTES );
+    public static function htmlspecialchars_decode($string, $onlyEscapedEntities = false) {
+        if (false === $onlyEscapedEntities) {
+            return htmlspecialchars_decode($string, ENT_NOQUOTES);
         }
 
-        return preg_replace_callback( self::$htmlEntityRegex,
-                function ( $match ) {
-                    return self::htmlspecialchars_decode( $match[ 0 ] );
-                }, $string );
+        return preg_replace_callback(self::$htmlEntityRegex,
+                function($match) {
+                    return self::htmlspecialchars_decode($match[0]);
+                }, $string);
     }
 
     /**
@@ -240,8 +240,8 @@ 
      *
      * @return bool
      */
-    public static function isADoubleEscapedEntity( $str ) {
-        return preg_match( self::$htmlEntityRegex, $str ) != 0;
+    public static function isADoubleEscapedEntity($str) {
+        return preg_match(self::$htmlEntityRegex, $str) != 0;
     }
 
     /**
@@ -249,8 +249,8 @@ 
      *
      * @return bool
      */
-    public static function isAnEscapedHTML( $str ) {
-        return preg_match( "/\/[a-z]*&gt;/i", $str ) != 0;
+    public static function isAnEscapedHTML($str) {
+        return preg_match("/\/[a-z]*&gt;/i", $str) != 0;
     }
 
     /**
@@ -258,8 +258,8 @@ 
      *
      * @return bool
      */
-    public static function isAValidUuid( $uuid ) {
-        return preg_match( '/^[0-9a-f]{8}-[0-9a-f]{4}-4[0-9a-f]{3}-[89ab][0-9a-f]{3}-[0-9a-f]{12}$/', $uuid ) === 1;
+    public static function isAValidUuid($uuid) {
+        return preg_match('/^[0-9a-f]{8}-[0-9a-f]{4}-4[0-9a-f]{3}-[89ab][0-9a-f]{3}-[0-9a-f]{12}$/', $uuid) === 1;
     }
 
     /**
@@ -268,8 +268,8 @@ 
      *
      * @return array|false|string[]
      */
-    public static function preg_split( $pattern, $subject ) {
-        return preg_split( $pattern, $subject, -1, PREG_SPLIT_NO_EMPTY );
+    public static function preg_split($pattern, $subject) {
+        return preg_split($pattern, $subject, -1, PREG_SPLIT_NO_EMPTY);
     }
 
     /**
@@ -287,8 +287,8 @@ 
      *
      * @return string
      */
-    public static function escapeOnlyHTMLTags( $string ) {
-        return preg_replace( '/<(.*?)>/iu', '&lt;$1&gt;', $string );
+    public static function escapeOnlyHTMLTags($string) {
+        return preg_replace('/<(.*?)>/iu', '&lt;$1&gt;', $string);
     }
 
     /**
@@ -298,8 +298,8 @@ 
      *
      * @return false|string
      */
-    public static function lastChar( $string ) {
-        return substr( $string, -1 );
+    public static function lastChar($string) {
+        return substr($string, -1);
     }
 
     /**
@@ -307,10 +307,10 @@ 
      *
      * @return int
      */
-    public static function getTheNumberOfTrailingSpaces( $segment ) {
+    public static function getTheNumberOfTrailingSpaces($segment) {
         $number = 0;
 
-        return self::recursiveIncrementNumberOfTrailingSpaces( $segment, $number );
+        return self::recursiveIncrementNumberOfTrailingSpaces($segment, $number);
     }
 
     /**
@@ -319,12 +319,12 @@ 
      *
      * @return int
      */
-    private static function recursiveIncrementNumberOfTrailingSpaces( $segment, &$number ) {
-        if ( self::lastChar( $segment ) === ' ' ) {
+    private static function recursiveIncrementNumberOfTrailingSpaces($segment, &$number) {
+        if (self::lastChar($segment) === ' ') {
             $number++;
-            $segment = substr( $segment, 0, -1 );
+            $segment = substr($segment, 0, -1);
 
-            return self::recursiveIncrementNumberOfTrailingSpaces( $segment, $number );
+            return self::recursiveIncrementNumberOfTrailingSpaces($segment, $number);
         }
 
         return $number;
@@ -337,15 +337,15 @@ 
      *
      * @return bool
      */
-    public static function isHtmlString( $string ) {
-        $string = stripslashes( $string );
+    public static function isHtmlString($string) {
+        $string = stripslashes($string);
 
-        if ( $string === '<>' ) {
+        if ($string === '<>') {
             return false;
         }
 
-        preg_match( "/<\/?[a-zA-Z1-6-]+((\s+[a-zA-Z1-6-]+(\s*=\s*(?:\".*?\"|'.*?'|[^'\">\s]+))?)+\s*|\s*)\/?>/", $string, $matches );
+        preg_match("/<\/?[a-zA-Z1-6-]+((\s+[a-zA-Z1-6-]+(\s*=\s*(?:\".*?\"|'.*?'|[^'\">\s]+))?)+\s*|\s*)\/?>/", $string, $matches);
 
-        return count( $matches ) !== 0;
+        return count($matches) !== 0;
     }
 }

src/Utils/Files.php

Spacing +28 added lines, -28 removed lines

@@ -18,15 +18,15 @@ 
      *
      * @return array|mixed
      */
-    public static function pathInfo( $path, $options = 15 ) {
-        $rawPath = explode( DIRECTORY_SEPARATOR, $path );
+    public static function pathInfo($path, $options = 15) {
+        $rawPath = explode(DIRECTORY_SEPARATOR, $path);
 
-        $basename = array_pop( $rawPath );
-        $dirname  = implode( DIRECTORY_SEPARATOR, $rawPath );
+        $basename = array_pop($rawPath);
+        $dirname  = implode(DIRECTORY_SEPARATOR, $rawPath);
 
-        $explodedFileName = explode( ".", $basename );
-        $extension        = strtolower( array_pop( $explodedFileName ) );
-        $filename         = implode( ".", $explodedFileName );
+        $explodedFileName = explode(".", $basename);
+        $extension        = strtolower(array_pop($explodedFileName));
+        $filename         = implode(".", $explodedFileName);
 
         $returnArray = [];
 
@@ -39,17 +39,17 @@ 
 
         // foreach flag, add in $return_array the corresponding field,
         // obtained by variable name correspondence
-        foreach ( $flagMap as $field => $i ) {
+        foreach ($flagMap as $field => $i) {
             //binary AND
-            if ( ( $options & $i ) > 0 ) {
+            if (($options & $i) > 0) {
                 //variable substitution: $field can be one between 'dirname', 'basename', 'extension', 'filename'
                 // $$field gets the value of the variable named $field
-                $returnArray[ $field ] = $$field;
+                $returnArray[$field] = $$field;
             }
         }
 
-        if ( count( $returnArray ) == 1 ) {
-            $returnArray = array_pop( $returnArray );
+        if (count($returnArray) == 1) {
+            $returnArray = array_pop($returnArray);
         }
 
         return $returnArray;
@@ -60,14 +60,14 @@ 
      *
      * @return false|string
      */
-    public static function getExtension( $path ) {
-        $pathInfo = self::pathInfo( $path );
+    public static function getExtension($path) {
+        $pathInfo = self::pathInfo($path);
 
-        if ( empty( $pathInfo ) ) {
+        if (empty($pathInfo)) {
             return false;
         }
 
-        return strtolower( $pathInfo[ 'extension' ] );
+        return strtolower($pathInfo['extension']);
     }
 
     /**
@@ -75,14 +75,14 @@ 
      *
      * @return bool
      */
-    public static function isXliff( $path ) {
-        $extension = self::getExtension( $path );
+    public static function isXliff($path) {
+        $extension = self::getExtension($path);
 
-        if ( !$extension ) {
+        if (!$extension) {
             return false;
         }
 
-        switch ( $extension ) {
+        switch ($extension) {
             case 'xliff':
             case 'sdlxliff':
             case 'tmx':
@@ -98,14 +98,14 @@ 
      *
      * @return bool|string
      */
-    public static function getMemoryFileType( $path ) {
-        $pathInfo = self::pathInfo( $path );
+    public static function getMemoryFileType($path) {
+        $pathInfo = self::pathInfo($path);
 
-        if ( empty( $pathInfo ) ) {
+        if (empty($pathInfo)) {
             return false;
         }
 
-        switch ( strtolower( $pathInfo[ 'extension' ] ) ) {
+        switch (strtolower($pathInfo['extension'])) {
             case 'tmx':
                 return 'tmx';
             case 'g':
@@ -120,8 +120,8 @@ 
      *
      * @return bool
      */
-    public static function isTMXFile( $path ) {
-        return self::getMemoryFileType( $path ) === 'tmx';
+    public static function isTMXFile($path) {
+        return self::getMemoryFileType($path) === 'tmx';
     }
 
     /**
@@ -129,7 +129,7 @@ 
      *
      * @return bool
      */
-    public static function isGlossaryFile( $path ) {
-        return self::getMemoryFileType( $path ) === 'glossary';
+    public static function isGlossaryFile($path) {
+        return self::getMemoryFileType($path) === 'glossary';
     }
 }