Test Failed
Push — master ( e8c8e2...a44def )
by Mauro
01:15 queued 14s
created
src/XliffParser/XliffParserV1.php 1 patch
Spacing   +93 added lines, -93 removed lines patch added patch discarded remove patch
@@ -16,33 +16,33 @@  discard block
 block discarded – undo
16 16
      * @inheritDoc
17 17
      * @throws Exception
18 18
      */
19
-    public function parse( DOMDocument $dom, $output = [] ) {
19
+    public function parse(DOMDocument $dom, $output = []) {
20 20
         $i = 1;
21 21
         /** @var DOMElement $file */
22
-        foreach ( $dom->getElementsByTagName( 'file' ) as $file ) {
22
+        foreach ($dom->getElementsByTagName('file') as $file) {
23 23
 
24 24
             // metadata
25
-            $output[ 'files' ][ $i ][ 'attr' ] = $this->extractMetadata( $file );
25
+            $output['files'][$i]['attr'] = $this->extractMetadata($file);
26 26
 
27 27
             // reference
28
-            if ( !empty( $this->extractReference( $file ) ) ) {
29
-                $output[ 'files' ][ $i ][ 'reference' ] = $this->extractReference( $file );
28
+            if (!empty($this->extractReference($file))) {
29
+                $output['files'][$i]['reference'] = $this->extractReference($file);
30 30
             }
31 31
 
32 32
             // trans-units
33 33
             $transUnitIdArrayForUniquenessCheck = [];
34 34
             $j                                  = 1;
35
-            foreach ( $file->childNodes as $body ) {
36
-                if ( $body->nodeName === 'body' ) {
37
-                    foreach ( $body->childNodes as $childNode ) {
38
-                        $this->extractTuFromNode( $childNode, $transUnitIdArrayForUniquenessCheck, $dom, $output, $i, $j );
35
+            foreach ($file->childNodes as $body) {
36
+                if ($body->nodeName === 'body') {
37
+                    foreach ($body->childNodes as $childNode) {
38
+                        $this->extractTuFromNode($childNode, $transUnitIdArrayForUniquenessCheck, $dom, $output, $i, $j);
39 39
                     }
40 40
 
41 41
                     // trans-unit re-count check
42
-                    $totalTransUnitsId  = count( $transUnitIdArrayForUniquenessCheck );
43
-                    $transUnitsUniqueId = count( array_unique( $transUnitIdArrayForUniquenessCheck ) );
44
-                    if ( $totalTransUnitsId != $transUnitsUniqueId ) {
45
-                        throw new DuplicateTransUnitIdInXliff( "Invalid trans-unit id, duplicate found.", 400 );
42
+                    $totalTransUnitsId  = count($transUnitIdArrayForUniquenessCheck);
43
+                    $transUnitsUniqueId = count(array_unique($transUnitIdArrayForUniquenessCheck));
44
+                    if ($totalTransUnitsId != $transUnitsUniqueId) {
45
+                        throw new DuplicateTransUnitIdInXliff("Invalid trans-unit id, duplicate found.", 400);
46 46
                     }
47 47
 
48 48
                     $i++;
@@ -58,50 +58,50 @@  discard block
 block discarded – undo
58 58
      *
59 59
      * @return array
60 60
      */
61
-    private function extractMetadata( DOMElement $file ) {
61
+    private function extractMetadata(DOMElement $file) {
62 62
         $metadata   = [];
63 63
         $customAttr = [];
64 64
 
65 65
         /** @var DOMAttr $attribute */
66
-        foreach ( $file->attributes as $attribute ) {
67
-            switch ( $attribute->localName ) {
66
+        foreach ($file->attributes as $attribute) {
67
+            switch ($attribute->localName) {
68 68
                 // original
69 69
                 case 'original':
70
-                    $metadata[ 'original' ] = $attribute->value;
70
+                    $metadata['original'] = $attribute->value;
71 71
                     break;
72 72
 
73 73
                 // source-language
74 74
                 case 'source-language':
75
-                    $metadata[ 'source-language' ] = $attribute->value;
75
+                    $metadata['source-language'] = $attribute->value;
76 76
                     break;
77 77
 
78 78
                 // data-type
79 79
                 case 'datatype':
80
-                    $metadata[ 'data-type' ] = $attribute->value;
80
+                    $metadata['data-type'] = $attribute->value;
81 81
                     break;
82 82
 
83 83
                 // target-language
84 84
                 case 'target-language':
85
-                    $metadata[ 'target-language' ] = $attribute->value;
85
+                    $metadata['target-language'] = $attribute->value;
86 86
                     break;
87 87
             }
88 88
 
89 89
             // Custom MateCat x-Attribute
90
-            preg_match( '|x-(.*?)|si', $attribute->localName, $temp );
91
-            if ( isset( $temp[ 1 ] ) ) {
92
-                $customAttr[ $attribute->localName ] = $attribute->value;
90
+            preg_match('|x-(.*?)|si', $attribute->localName, $temp);
91
+            if (isset($temp[1])) {
92
+                $customAttr[$attribute->localName] = $attribute->value;
93 93
             }
94
-            unset( $temp );
94
+            unset($temp);
95 95
 
96 96
             // Custom MateCat namespace Attribute mtc:
97
-            preg_match( '|mtc:(.*?)|si', $attribute->nodeName, $temp );
98
-            if ( isset( $temp[ 1 ] ) ) {
99
-                $customAttr[ $attribute->nodeName ] = $attribute->value;
97
+            preg_match('|mtc:(.*?)|si', $attribute->nodeName, $temp);
98
+            if (isset($temp[1])) {
99
+                $customAttr[$attribute->nodeName] = $attribute->value;
100 100
             }
101
-            unset( $temp );
101
+            unset($temp);
102 102
 
103
-            if ( !empty( $customAttr ) ) {
104
-                $metadata[ 'custom' ] = $customAttr;
103
+            if (!empty($customAttr)) {
104
+                $metadata['custom'] = $customAttr;
105 105
             }
106 106
         }
107 107
 
@@ -113,16 +113,16 @@  discard block
 block discarded – undo
113 113
      *
114 114
      * @return array
115 115
      */
116
-    private function extractReference( DOMElement $file ) {
116
+    private function extractReference(DOMElement $file) {
117 117
         $reference = [];
118 118
 
119 119
         $order = 0;
120
-        foreach ( $file->getElementsByTagName( 'reference' ) as $ref ) {
120
+        foreach ($file->getElementsByTagName('reference') as $ref) {
121 121
             /** @var DOMNode $childNode */
122
-            foreach ( $ref->childNodes as $childNode ) {
123
-                if ( $childNode->nodeName === 'internal-file' ) {
124
-                    $reference[ $order ][ 'form-type' ] = $childNode->attributes->getNamedItem( 'form' )->nodeValue;
125
-                    $reference[ $order ][ 'base64' ]    = trim( $childNode->nodeValue );
122
+            foreach ($ref->childNodes as $childNode) {
123
+                if ($childNode->nodeName === 'internal-file') {
124
+                    $reference[$order]['form-type'] = $childNode->attributes->getNamedItem('form')->nodeValue;
125
+                    $reference[$order]['base64']    = trim($childNode->nodeValue);
126 126
                     $order++;
127 127
                 }
128 128
             }
@@ -143,55 +143,55 @@  discard block
 block discarded – undo
143 143
      *
144 144
      * @throws Exception
145 145
      */
146
-    protected function extractTransUnit( $transUnit, &$transUnitIdArrayForUniquenessCheck, $dom, &$output, &$i, &$j ) {
146
+    protected function extractTransUnit($transUnit, &$transUnitIdArrayForUniquenessCheck, $dom, &$output, &$i, &$j) {
147 147
         // metadata
148
-        $output[ 'files' ][ $i ][ 'trans-units' ][ $j ][ 'attr' ] = $this->extractTransUnitMetadata( $transUnit, $transUnitIdArrayForUniquenessCheck );
148
+        $output['files'][$i]['trans-units'][$j]['attr'] = $this->extractTransUnitMetadata($transUnit, $transUnitIdArrayForUniquenessCheck);
149 149
 
150 150
         // notes
151
-        $output[ 'files' ][ $i ][ 'trans-units' ][ $j ][ 'notes' ] = $this->extractTransUnitNotes( $dom, $transUnit );
151
+        $output['files'][$i]['trans-units'][$j]['notes'] = $this->extractTransUnitNotes($dom, $transUnit);
152 152
 
153 153
         // content
154 154
         /** @var DOMElement $childNode */
155
-        foreach ( $transUnit->childNodes as $childNode ) {
155
+        foreach ($transUnit->childNodes as $childNode) {
156 156
             // source
157
-            if ( $childNode->nodeName === 'source' ) {
158
-                $output[ 'files' ][ $i ][ 'trans-units' ][ $j ][ 'source' ] = $this->extractContent( $dom, $childNode );
157
+            if ($childNode->nodeName === 'source') {
158
+                $output['files'][$i]['trans-units'][$j]['source'] = $this->extractContent($dom, $childNode);
159 159
             }
160 160
 
161 161
             // seg-source
162
-            if ( $childNode->nodeName === 'seg-source' ) {
163
-                $rawSegment                                                     = $output[ 'files' ][ $i ][ 'trans-units' ][ $j ][ 'source' ][ 'raw-content' ];
164
-                $output[ 'files' ][ $i ][ 'trans-units' ][ $j ][ 'seg-source' ] = $this->extractContentWithMarksAndExtTags( $dom, $childNode, $rawSegment );
165
-                $output[ 'files' ][ $i ][ 'trans-units' ][ $j ][ 'seg-source' ][ 0 ]['attr'] = $this->extractTagAttributes($transUnit);
162
+            if ($childNode->nodeName === 'seg-source') {
163
+                $rawSegment                                                     = $output['files'][$i]['trans-units'][$j]['source']['raw-content'];
164
+                $output['files'][$i]['trans-units'][$j]['seg-source'] = $this->extractContentWithMarksAndExtTags($dom, $childNode, $rawSegment);
165
+                $output['files'][$i]['trans-units'][$j]['seg-source'][0]['attr'] = $this->extractTagAttributes($transUnit);
166 166
             }
167 167
 
168 168
             // target
169
-            if ( $childNode->nodeName === 'target' ) {
170
-                $output[ 'files' ][ $i ][ 'trans-units' ][ $j ][ 'target' ] = $this->extractContent( $dom, $childNode );
169
+            if ($childNode->nodeName === 'target') {
170
+                $output['files'][$i]['trans-units'][$j]['target'] = $this->extractContent($dom, $childNode);
171 171
 
172 172
                 // seg-target
173
-                $targetRawContent = @$output[ 'files' ][ $i ][ 'trans-units' ][ $j ][ 'target' ][ 'raw-content' ];
174
-                $segSource        = @$output[ 'files' ][ $i ][ 'trans-units' ][ $j ][ 'seg-source' ];
175
-                if ( isset( $targetRawContent ) && !empty( $targetRawContent ) && isset( $segSource ) && count( $segSource ) > 0 ) {
176
-                    $output[ 'files' ][ $i ][ 'trans-units' ][ $j ][ 'seg-target' ] = $this->extractContentWithMarksAndExtTags( $dom, $childNode, $targetRawContent );
177
-                    $output[ 'files' ][ $i ][ 'trans-units' ][ $j ][ 'seg-target' ][ 0 ]['attr'] = $this->extractTagAttributes($transUnit);
173
+                $targetRawContent = @$output['files'][$i]['trans-units'][$j]['target']['raw-content'];
174
+                $segSource        = @$output['files'][$i]['trans-units'][$j]['seg-source'];
175
+                if (isset($targetRawContent) && !empty($targetRawContent) && isset($segSource) && count($segSource) > 0) {
176
+                    $output['files'][$i]['trans-units'][$j]['seg-target'] = $this->extractContentWithMarksAndExtTags($dom, $childNode, $targetRawContent);
177
+                    $output['files'][$i]['trans-units'][$j]['seg-target'][0]['attr'] = $this->extractTagAttributes($transUnit);
178 178
                 }
179 179
             }
180 180
 
181 181
             // locked
182
-            if ( $childNode->nodeName === 'sdl:seg' ) {
183
-                $output[ 'files' ][ $i ][ 'trans-units' ][ $j ][ 'locked' ] = $this->extractLocked( $childNode );
182
+            if ($childNode->nodeName === 'sdl:seg') {
183
+                $output['files'][$i]['trans-units'][$j]['locked'] = $this->extractLocked($childNode);
184 184
             }
185 185
         }
186 186
 
187 187
         // context-group
188
-        foreach ( $transUnit->getElementsByTagName( 'context-group' ) as $contextGroup ) {
189
-            $output[ 'files' ][ $i ][ 'trans-units' ][ $j ][ 'context-group' ][] = $this->extractTransUnitContextGroup( $dom, $contextGroup );
188
+        foreach ($transUnit->getElementsByTagName('context-group') as $contextGroup) {
189
+            $output['files'][$i]['trans-units'][$j]['context-group'][] = $this->extractTransUnitContextGroup($dom, $contextGroup);
190 190
         }
191 191
 
192 192
         // alt-trans
193
-        foreach ( $transUnit->getElementsByTagName( 'alt-trans' ) as $altTrans ) {
194
-            $output[ 'files' ][ $i ][ 'trans-units' ][ $j ][ 'alt-trans' ][] = $this->extractTransUnitAltTrans( $altTrans );
193
+        foreach ($transUnit->getElementsByTagName('alt-trans') as $altTrans) {
194
+            $output['files'][$i]['trans-units'][$j]['alt-trans'][] = $this->extractTransUnitAltTrans($altTrans);
195 195
         }
196 196
 
197 197
         $j++;
@@ -204,41 +204,41 @@  discard block
 block discarded – undo
204 204
      * @return array
205 205
      * @throws Exception
206 206
      */
207
-    private function extractTransUnitMetadata( DOMElement $transUnit, array &$transUnitIdArrayForUniquenessCheck ) {
207
+    private function extractTransUnitMetadata(DOMElement $transUnit, array &$transUnitIdArrayForUniquenessCheck) {
208 208
         $metadata = [];
209 209
 
210 210
         // id MUST NOT be null
211
-        if ( null === $transUnit->attributes->getNamedItem( 'id' ) ) {
212
-            throw new NotFoundIdInTransUnit( 'Invalid trans-unit id found. EMPTY value', 400 );
211
+        if (null === $transUnit->attributes->getNamedItem('id')) {
212
+            throw new NotFoundIdInTransUnit('Invalid trans-unit id found. EMPTY value', 400);
213 213
         }
214 214
 
215 215
         /**
216 216
          * @var DOMAttr $element
217 217
          */
218
-        foreach ( $transUnit->attributes as $element ) {
218
+        foreach ($transUnit->attributes as $element) {
219 219
 
220
-            if ( $element->nodeName === "id" ) {
220
+            if ($element->nodeName === "id") {
221 221
 
222 222
                 $id = $element->nodeValue;
223 223
 
224
-                if ( strlen( $id ) > 100 ) {
225
-                    throw new SegmentIdTooLongException( 'Segment-id too long. Max 100 characters allowed', 400 );
224
+                if (strlen($id) > 100) {
225
+                    throw new SegmentIdTooLongException('Segment-id too long. Max 100 characters allowed', 400);
226 226
                 }
227 227
 
228 228
                 $transUnitIdArrayForUniquenessCheck[] = $id;
229
-                $metadata[ 'id' ]                     = $id;
229
+                $metadata['id']                     = $id;
230 230
 
231
-            } elseif ( $element->nodeName === "approved" ) {
231
+            } elseif ($element->nodeName === "approved") {
232 232
                 // approved as BOOLEAN
233 233
                 // http://docs.oasis-open.org/xliff/v1.2/os/xliff-core.html#approved
234
-                $metadata[ $element->nodeName ] = filter_var( $element->nodeValue, FILTER_VALIDATE_BOOLEAN );
235
-            } elseif ( $element->nodeName === "maxwidth" ) {
234
+                $metadata[$element->nodeName] = filter_var($element->nodeValue, FILTER_VALIDATE_BOOLEAN);
235
+            } elseif ($element->nodeName === "maxwidth") {
236 236
                 // we ignore ( but we get ) the attribute size-unit="char" assuming that a restriction is everytime done by character
237 237
                 // we duplicate the info to allow Xliff V1 and V2 to work the same
238
-                $metadata[ 'sizeRestriction' ]  = filter_var( $element->nodeValue, FILTER_SANITIZE_NUMBER_INT );
239
-                $metadata[ $element->nodeName ] = filter_var( $element->nodeValue, FILTER_SANITIZE_NUMBER_INT );
238
+                $metadata['sizeRestriction']  = filter_var($element->nodeValue, FILTER_SANITIZE_NUMBER_INT);
239
+                $metadata[$element->nodeName] = filter_var($element->nodeValue, FILTER_SANITIZE_NUMBER_INT);
240 240
             } else {
241
-                $metadata[ $element->nodeName ] = $element->nodeValue;
241
+                $metadata[$element->nodeName] = $element->nodeValue;
242 242
             }
243 243
 
244 244
         }
@@ -252,19 +252,19 @@  discard block
 block discarded – undo
252 252
      * @return array
253 253
      * @throws Exception
254 254
      */
255
-    private function extractTransUnitNotes( DOMDocument $dom, DOMElement $transUnit ) {
255
+    private function extractTransUnitNotes(DOMDocument $dom, DOMElement $transUnit) {
256 256
         $notes = [];
257
-        foreach ( $transUnit->getElementsByTagName( 'note' ) as $note ) {
257
+        foreach ($transUnit->getElementsByTagName('note') as $note) {
258 258
 
259
-            $noteValue = $this->extractTagContent( $dom, $note );
259
+            $noteValue = $this->extractTagContent($dom, $note);
260 260
 
261
-            if ( '' !== $noteValue ) {
261
+            if ('' !== $noteValue) {
262 262
 
263
-                $extractedNote = $this->JSONOrRawContentArray( $noteValue );
263
+                $extractedNote = $this->JSONOrRawContentArray($noteValue);
264 264
 
265 265
                 // extract all the attributes
266
-                foreach ( $note->attributes as $attribute ) {
267
-                    $extractedNote[ $attribute->name ] = $attribute->value;
266
+                foreach ($note->attributes as $attribute) {
267
+                    $extractedNote[$attribute->name] = $attribute->value;
268 268
                 }
269 269
 
270 270
                 $notes[] = $extractedNote;
@@ -279,14 +279,14 @@  discard block
 block discarded – undo
279 279
      *
280 280
      * @return array
281 281
      */
282
-    private function extractTransUnitContextGroup( DOMDocument $dom, DOMElement $contextGroup ) {
282
+    private function extractTransUnitContextGroup(DOMDocument $dom, DOMElement $contextGroup) {
283 283
         $cg           = [];
284
-        $cg[ 'attr' ] = $this->extractTagAttributes( $contextGroup );
284
+        $cg['attr'] = $this->extractTagAttributes($contextGroup);
285 285
 
286 286
         /** @var DOMNode $context */
287
-        foreach ( $contextGroup->childNodes as $context ) {
288
-            if ( $context->nodeName === 'context' ) {
289
-                $cg[ 'contexts' ][] = $this->extractContent( $dom, $context );
287
+        foreach ($contextGroup->childNodes as $context) {
288
+            if ($context->nodeName === 'context') {
289
+                $cg['contexts'][] = $this->extractContent($dom, $context);
290 290
             }
291 291
         }
292 292
 
@@ -298,16 +298,16 @@  discard block
 block discarded – undo
298 298
      *
299 299
      * @return array
300 300
      */
301
-    private function extractTransUnitAltTrans( DOMElement $altTrans ) {
301
+    private function extractTransUnitAltTrans(DOMElement $altTrans) {
302 302
         $at           = [];
303
-        $at[ 'attr' ] = $this->extractTagAttributes( $altTrans );
303
+        $at['attr'] = $this->extractTagAttributes($altTrans);
304 304
 
305
-        if ( $altTrans->getElementsByTagName( 'source' )->length > 0 ) {
306
-            $at[ 'source' ] = $altTrans->getElementsByTagName( 'source' )->item( 0 )->nodeValue;
305
+        if ($altTrans->getElementsByTagName('source')->length > 0) {
306
+            $at['source'] = $altTrans->getElementsByTagName('source')->item(0)->nodeValue;
307 307
         }
308 308
 
309
-        if ( $altTrans->getElementsByTagName( 'target' ) ) {
310
-            $at[ 'target' ] = $altTrans->getElementsByTagName( 'target' )->item( 0 )->nodeValue;
309
+        if ($altTrans->getElementsByTagName('target')) {
310
+            $at['target'] = $altTrans->getElementsByTagName('target')->item(0)->nodeValue;
311 311
         }
312 312
 
313 313
         return $at;
@@ -318,7 +318,7 @@  discard block
 block discarded – undo
318 318
      *
319 319
      * @return bool
320 320
      */
321
-    private function extractLocked( DOMElement $locked ) {
322
-        return null !== $locked->getAttribute( 'locked' );
321
+    private function extractLocked(DOMElement $locked) {
322
+        return null !== $locked->getAttribute('locked');
323 323
     }
324 324
 }
Please login to merge, or discard this patch.