matecat / subfiltering

src/Utils/Map.php

Spacing +79 added lines, -79 removed lines

@@ -25,7 +25,7 @@ 
      */
     private $map;
 
-    public function __construct( array $map ) {
+    public function __construct(array $map) {
         $this->map = $map;
     }
 
@@ -33,7 +33,7 @@ 
      * @return ArrayIterator
      */
     public function getIterator() {
-        return new ArrayIterator( $this->map );
+        return new ArrayIterator($this->map);
     }
 
     /**
@@ -41,8 +41,8 @@ 
      *
      * @return bool
      */
-    public function offsetExists( $offset ) {
-        return array_key_exists( $offset, $this->map );
+    public function offsetExists($offset) {
+        return array_key_exists($offset, $this->map);
     }
 
     /**
@@ -50,8 +50,8 @@ 
      *
      * @return mixed|null
      */
-    public function offsetGet( $offset ) {
-        return $this->get( $offset );
+    public function offsetGet($offset) {
+        return $this->get($offset);
     }
 
     /**
@@ -60,8 +60,8 @@ 
      *
      * @return void
      */
-    public function offsetSet( $offset, $value ) {
-        $this->put( $offset, $value );
+    public function offsetSet($offset, $value) {
+        $this->put($offset, $value);
     }
 
     /**
@@ -69,8 +69,8 @@ 
      *
      * @return void
      */
-    public function offsetUnset( $offset ) {
-        $this->remove( $offset );
+    public function offsetUnset($offset) {
+        $this->remove($offset);
     }
 
 
@@ -78,13 +78,13 @@ 
      * @return Generator
      */
     public function entrySet() {
-        foreach ( $this->map as $k => $v ) {
-            yield [ $k, $v ];
+        foreach ($this->map as $k => $v) {
+            yield [$k, $v];
         }
     }
 
-    public static function instance( array $map = [] ) {
-        return new static( $map );
+    public static function instance(array $map = []) {
+        return new static($map);
     }
 
     /**
@@ -94,8 +94,8 @@ 
      *
      * @return mixed|null
      */
-    public function get( $needle ) {
-        return $this->getOrDefault( $needle, null );
+    public function get($needle) {
+        return $this->getOrDefault($needle, null);
     }
 
     /**
@@ -104,8 +104,8 @@ 
      *
      * @return mixed
      */
-    public function getOrDefault( $needle, $default ) {
-        return array_key_exists( $needle, $this->map ) ? $this->map [ $needle ] : $default;
+    public function getOrDefault($needle, $default) {
+        return array_key_exists($needle, $this->map) ? $this->map [$needle] : $default;
     }
 
     /**
@@ -121,11 +121,11 @@ 
      * @return $this
      */
     public function __clone() {
-        return new static( $this->map );
+        return new static($this->map);
     }
 
-    public function containsKey( $key ) {
-        return array_key_exists( $key, $this->map );
+    public function containsKey($key) {
+        return array_key_exists($key, $this->map);
     }
 
     /**
@@ -136,8 +136,8 @@ 
      *
      * @return false|string
      */
-    public function containsValue( $value ) {
-        return array_search( $value, $this->map, true );
+    public function containsValue($value) {
+        return array_search($value, $this->map, true);
     }
 
     /**
@@ -148,9 +148,9 @@ 
      *
      * @return Generator
      */
-    public function foreEach( callable $callable ) {
-        foreach ( $this->map as $k => $v ) {
-            yield $callable( $k, $v );
+    public function foreEach(callable $callable) {
+        foreach ($this->map as $k => $v) {
+            yield $callable($k, $v);
         }
     }
 
@@ -160,14 +160,14 @@ 
      * @return bool
      */
     public function isEmpty() {
-        return empty( $this->map );
+        return empty($this->map);
     }
 
     /**
      * @return Map
      */
     public function keySet() {
-        return static::instance( array_keys( $this->map ) );
+        return static::instance(array_keys($this->map));
     }
 
     /**
@@ -178,9 +178,9 @@ 
      *
      * @return mixed|null the previous value associated with `key`, or null if there was no mapping for `key`.
      */
-    public function put( $key, $value ) {
-        $previousValue     = $this->get( $key );
-        $this->map[ $key ] = $value;
+    public function put($key, $value) {
+        $previousValue     = $this->get($key);
+        $this->map[$key] = $value;
 
         return $previousValue;
     }
@@ -190,9 +190,9 @@ 
      *
      * @return void
      */
-    public function putAll( $map ) {
-        foreach ( $map as $k => $v ) {
-            $this->map[ $k ] = $v;
+    public function putAll($map) {
+        foreach ($map as $k => $v) {
+            $this->map[$k] = $v;
         }
     }
 
@@ -205,10 +205,10 @@ 
      * @return mixed|null the previous value associated with the specified key, or null if there was no mapping for the key.
      *               A null return can also indicate that the map previously associated null with the `key`
      */
-    public function putIfAbsent( $key, $value ) {
-        $previousValue = $this->get( $key );
-        if ( $previousValue === null ) {
-            $this->map[ $key ] = $value;
+    public function putIfAbsent($key, $value) {
+        $previousValue = $this->get($key);
+        if ($previousValue === null) {
+            $this->map[$key] = $value;
         }
 
         return $previousValue;
@@ -221,10 +221,10 @@ 
      *
      * @return bool true if the value was removed
      */
-    public function remove( $key ) {
-        $exists = array_key_exists( $key, $this->map );
-        if ( $exists ) {
-            unset( $this->map[ $key ] );
+    public function remove($key) {
+        $exists = array_key_exists($key, $this->map);
+        if ($exists) {
+            unset($this->map[$key]);
         }
 
         return $exists;
@@ -236,11 +236,11 @@ 
      * @return mixed|null the previous value associated with the specified key, or null if there was no mapping for the key.
      *                      A null return can also indicate that the map previously associated null with the key.
      */
-    public function replace( $key, $value ) {
-        $exists        = array_key_exists( $key, $this->map );
-        $previousValue = $this->get( $key );
-        if ( $exists ) {
-            $this->map[ $key ] = $value;
+    public function replace($key, $value) {
+        $exists        = array_key_exists($key, $this->map);
+        $previousValue = $this->get($key);
+        if ($exists) {
+            $this->map[$key] = $value;
         }
 
         return $previousValue;
@@ -255,11 +255,11 @@ 
      *
      * @return boolean true if the value was replaced
      */
-    public function replaceIfEquals( $key, $newValue, $oldValue ) {
-        $exists        = array_key_exists( $key, $this->map );
-        $previousValue = $this->get( $key );
-        if ( $exists && $previousValue === $oldValue ) {
-            $this->map[ $key ] = $newValue;
+    public function replaceIfEquals($key, $newValue, $oldValue) {
+        $exists        = array_key_exists($key, $this->map);
+        $previousValue = $this->get($key);
+        if ($exists && $previousValue === $oldValue) {
+            $this->map[$key] = $newValue;
 
             return true;
         }
@@ -276,9 +276,9 @@ 
      *
      * @return void
      */
-    public function replaceAll( callable $callable ) {
-        foreach ( $this->map as $key => $value ) {
-            $this->map[ $key ] = $callable( $key, $value );
+    public function replaceAll(callable $callable) {
+        foreach ($this->map as $key => $value) {
+            $this->map[$key] = $callable($key, $value);
         }
     }
 
@@ -286,14 +286,14 @@ 
      * @return int
      */
     public function size() {
-        return sizeof( $this->map );
+        return sizeof($this->map);
     }
 
     /**
      * @return Map
      */
     public function values() {
-        return new static( array_values( $this->map ) );
+        return new static(array_values($this->map));
     }
 
     /**
@@ -312,14 +312,14 @@ 
      *
      * @return mixed|null the new value associated with the specified key, or null if none
      */
-    public function compute( $key, callable $callable ) {
-        $exists = array_key_exists( $key, $this->map );
-        if ( $exists ) {
-            $res = $callable( $key, $this->get( $key ) );
-            if ( $res == null ) {
-                unset( $this->map[ $key ] );
+    public function compute($key, callable $callable) {
+        $exists = array_key_exists($key, $this->map);
+        if ($exists) {
+            $res = $callable($key, $this->get($key));
+            if ($res == null) {
+                unset($this->map[$key]);
             } else {
-                $this->map[ $key ] = $callable( $key, $this->get( $key ) );
+                $this->map[$key] = $callable($key, $this->get($key));
             }
         }
 
@@ -340,14 +340,14 @@ 
      *
      * @return mixed|null the current (existing or computed) value associated with the specified key, or null if the computed value is null
      */
-    public function computeIfAbsent( $key, callable $callable ) {
-        $exists        = array_key_exists( $key, $this->map );
-        $previousValue = $this->get( $key );
+    public function computeIfAbsent($key, callable $callable) {
+        $exists        = array_key_exists($key, $this->map);
+        $previousValue = $this->get($key);
         $res           = null;
-        if ( !$exists || $previousValue !== null ) {
-            $res = $callable( $key, $this->get( $key ) );
-            if ( $res != null ) {
-                $this->map[ $key ] = $res;
+        if (!$exists || $previousValue !== null) {
+            $res = $callable($key, $this->get($key));
+            if ($res != null) {
+                $this->map[$key] = $res;
             }
 
         }
@@ -367,16 +367,16 @@ 
      *
      * @return mixed|null the new value associated with the specified key, or null if none
      */
-    public function computeIfPresent( $key, callable $callable ) {
-        $exists        = array_key_exists( $key, $this->map );
-        $previousValue = $this->get( $key );
+    public function computeIfPresent($key, callable $callable) {
+        $exists        = array_key_exists($key, $this->map);
+        $previousValue = $this->get($key);
         $res           = null;
-        if ( $exists && $previousValue !== null ) {
-            $res = $callable( $key, $this->map[ $key ] );
-            if ( $res == null ) {
-                unset( $this->map[ $key ] );
+        if ($exists && $previousValue !== null) {
+            $res = $callable($key, $this->map[$key]);
+            if ($res == null) {
+                unset($this->map[$key]);
             } else {
-                $this->map[ $key ] = $res;
+                $this->map[$key] = $res;
             }
         }
 

src/Utils/DataRefReplacer.php

Spacing +90 added lines, -90 removed lines

@@ -27,8 +27,8 @@ 
      *
      * @param array $map
      */
-    public function __construct( array $map = [] ) {
-        $this->map = Map::instance( $this->sanitizeMap( $map ) );
+    public function __construct(array $map = []) {
+        $this->map = Map::instance($this->sanitizeMap($map));
     }
 
     /**
@@ -42,42 +42,42 @@ 
      *
      * @return string
      */
-    public function replace( $string ) {
+    public function replace($string) {
 
         // if the map is empty
         // or the string has not a dataRef attribute
         // return string as is
-        if ( $this->map->isEmpty() || !$this->hasAnyDataRefAttribute( $string ) ) {
+        if ($this->map->isEmpty() || !$this->hasAnyDataRefAttribute($string)) {
             return $string;
         }
 
         // try not to throw exception for wrong segments with opening tags and no closing
         try {
 
-            $html = XmlParser::parse( $string, true );
+            $html = XmlParser::parse($string, true);
 
             $dataRefEndMap = new ArrayList();
 
-            foreach ( $html as $node ) {
+            foreach ($html as $node) {
 
                 // 1. Replace <ph>|<sc>|<ec> tags
-                $string = $this->recursiveTransformDataRefToPhTag( $node, $string );
+                $string = $this->recursiveTransformDataRefToPhTag($node, $string);
 
                 // 2. Replace self-closed <pc dataRefStart="xyz" /> tags
-                $string = $this->recursiveReplaceSelfClosedPcTags( $node, $string );
+                $string = $this->recursiveReplaceSelfClosedPcTags($node, $string);
 
                 // 3. Build the DataRefEndMap needed by replaceClosingPcTags function
                 // (needed for correct handling of </pc> closing tags)
                 // make this inline with one foreach cycle
-                $this->extractDataRefMapRecursively( $node, $dataRefEndMap );
+                $this->extractDataRefMapRecursively($node, $dataRefEndMap);
 
             }
 
             // 4. replace pc tags
-            $string = $this->replaceOpeningPcTags( $string );
-            $string = $this->replaceClosingPcTags( $string, $dataRefEndMap );
+            $string = $this->replaceOpeningPcTags($string);
+            $string = $this->replaceClosingPcTags($string, $dataRefEndMap);
 
-        } catch ( Exception $ignore ) {
+        } catch (Exception $ignore) {
             // if something fails here, do not throw exception and return the original string instead
 //            var_dump( $ignore );
         } finally {
@@ -91,8 +91,8 @@ 
      *
      * @return bool
      */
-    private function hasAnyDataRefAttribute( $string ) {
-        return (bool)preg_match( '/(dataRef|dataRefStart|dataRefEnd)=[\'"].*?[\'"]/', $string );
+    private function hasAnyDataRefAttribute($string) {
+        return (bool) preg_match('/(dataRef|dataRefStart|dataRefEnd)=[\'"].*?[\'"]/', $string);
     }
 
     /**
@@ -108,18 +108,18 @@ 
      *
      * @return string
      */
-    private function recursiveTransformDataRefToPhTag( $node, $string ) {
+    private function recursiveTransformDataRefToPhTag($node, $string) {
 
-        if ( $node->has_children ) {
+        if ($node->has_children) {
 
-            foreach ( $node->inner_html as $childNode ) {
-                $string = $this->recursiveTransformDataRefToPhTag( $childNode, $string );
+            foreach ($node->inner_html as $childNode) {
+                $string = $this->recursiveTransformDataRefToPhTag($childNode, $string);
             }
 
         } else {
 
             // accept only those tags
-            switch ( $node->tagName ) {
+            switch ($node->tagName) {
                 case 'ph':
                     $ctype = CTypeEnum::PH_DATA_REF;
                     break;
@@ -134,17 +134,17 @@ 
             }
 
             // if isset a value in the map proceed with conversion otherwise skip
-            $attributesMap = Map::instance( $node->attributes );
-            if ( !$this->map->get( $attributesMap->get( 'dataRef' ) ) ) {
+            $attributesMap = Map::instance($node->attributes);
+            if (!$this->map->get($attributesMap->get('dataRef'))) {
                 return $string;
             }
 
-            $dataRefName = $node->attributes[ 'dataRef' ];   // map identifier. Eg: source1
+            $dataRefName = $node->attributes['dataRef']; // map identifier. Eg: source1
 
             return $this->replaceNewTagString(
                     $node->node,
-                    $attributesMap->getOrDefault( 'id', $dataRefName ),
-                    $this->map->getOrDefault( $node->attributes[ 'dataRef' ], 'NULL' ),
+                    $attributesMap->getOrDefault('id', $dataRefName),
+                    $this->map->getOrDefault($node->attributes['dataRef'], 'NULL'),
                     $ctype,
                     $string,
                     null
@@ -162,11 +162,11 @@ 
      *
      * @return array
      */
-    private function sanitizeMap( $map ) {
+    private function sanitizeMap($map) {
 
-        foreach ( $map as $name => $value ) {
-            if ( is_null( $value ) || $value === '' ) {
-                $map[ $name ] = 'NULL';
+        foreach ($map as $name => $value) {
+            if (is_null($value) || $value === '') {
+                $map[$name] = 'NULL';
             }
         }
 
@@ -182,23 +182,23 @@ 
      * @throws InvalidXmlException
      * @throws XmlParsingException
      */
-    private function recursiveReplaceSelfClosedPcTags( $node, $string ) {
+    private function recursiveReplaceSelfClosedPcTags($node, $string) {
 
-        if ( $node->has_children ) {
+        if ($node->has_children) {
 
-            foreach ( $node->inner_html as $childNode ) {
-                $string = $this->recursiveReplaceSelfClosedPcTags( $childNode, $string );
+            foreach ($node->inner_html as $childNode) {
+                $string = $this->recursiveReplaceSelfClosedPcTags($childNode, $string);
             }
 
-        } elseif ( $node->tagName == 'pc' && $node->self_closed === true ) {
+        } elseif ($node->tagName == 'pc' && $node->self_closed === true) {
 
-            $attributesMap = Map::instance( $node->attributes );
+            $attributesMap = Map::instance($node->attributes);
 
-            if ( $dataRefStartValue = $this->map->get( $node->attributes[ 'dataRefStart' ] ) ) {
+            if ($dataRefStartValue = $this->map->get($node->attributes['dataRefStart'])) {
 
                 $string = $this->replaceNewTagString(
                         $node->node,
-                        $attributesMap->get( 'id' ),
+                        $attributesMap->get('id'),
                         $dataRefStartValue,
                         CTypeEnum::PC_SELF_CLOSE_DATA_REF,
                         $string
@@ -218,23 +218,23 @@ 
      * @param object    $node
      * @param ArrayList $dataRefEndMap
      */
-    private function extractDataRefMapRecursively( $node, ArrayList $dataRefEndMap ) {
+    private function extractDataRefMapRecursively($node, ArrayList $dataRefEndMap) {
 
         // we have to build the map for the closing pc tag, so get the children first
-        if ( $node->has_children ) {
-            foreach ( $node->inner_html as $nestedNode ) {
-                $this->extractDataRefMapRecursively( $nestedNode, $dataRefEndMap );
+        if ($node->has_children) {
+            foreach ($node->inner_html as $nestedNode) {
+                $this->extractDataRefMapRecursively($nestedNode, $dataRefEndMap);
             }
         }
 
         // EXCLUDE self closed <pc/>
-        if ( $node->tagName === 'pc' && $node->self_closed === false ) {
+        if ($node->tagName === 'pc' && $node->self_closed === false) {
 
-            $attributesMap = Map::instance( $node->attributes );
-            $dataRefEnd    = $attributesMap->getOrDefault( 'dataRefEnd', $attributesMap->get( 'dataRefStart' ) );
+            $attributesMap = Map::instance($node->attributes);
+            $dataRefEnd    = $attributesMap->getOrDefault('dataRefEnd', $attributesMap->get('dataRefStart'));
 
             $dataRefEndMap[] = [
-                    'id'         => $attributesMap->get( 'id' ),
+                    'id'         => $attributesMap->get('id'),
                     'dataRefEnd' => $dataRefEnd,
             ];
 
@@ -252,31 +252,31 @@ 
      * @throws InvalidXmlException
      * @throws XmlParsingException
      */
-    private function replaceOpeningPcTags( $string ) {
+    private function replaceOpeningPcTags($string) {
 
-        preg_match_all( '|<pc ([^>/]+?)>|iu', $string, $openingPcMatches );
+        preg_match_all('|<pc ([^>/]+?)>|iu', $string, $openingPcMatches);
 
-        foreach ( $openingPcMatches[ 0 ] as $match ) {
+        foreach ($openingPcMatches[0] as $match) {
 
-            $node = XmlParser::parse( $match . '</pc>', true )[ 0 ]; // add a closing tag to not break xml integrity
+            $node = XmlParser::parse($match.'</pc>', true)[0]; // add a closing tag to not break xml integrity
 
             // CASE 1 - Missing `dataRefStart`
-            if ( isset( $node->attributes[ 'dataRefEnd' ] ) && !isset( $node->attributes[ 'dataRefStart' ] ) ) {
-                $node->attributes[ 'dataRefStart' ] = $node->attributes[ 'dataRefEnd' ];
+            if (isset($node->attributes['dataRefEnd']) && !isset($node->attributes['dataRefStart'])) {
+                $node->attributes['dataRefStart'] = $node->attributes['dataRefEnd'];
             }
 
             // CASE 2 - Missing `dataRefEnd`
-            if ( isset( $node->attributes[ 'dataRefStart' ] ) && !isset( $node->attributes[ 'dataRefEnd' ] ) ) {
-                $node->attributes[ 'dataRefEnd' ] = $node->attributes[ 'dataRefStart' ];
+            if (isset($node->attributes['dataRefStart']) && !isset($node->attributes['dataRefEnd'])) {
+                $node->attributes['dataRefEnd'] = $node->attributes['dataRefStart'];
             }
 
-            if ( isset( $node->attributes[ 'dataRefStart' ] ) ) {
+            if (isset($node->attributes['dataRefStart'])) {
 
-                $attributesMap = Map::instance( $node->attributes );
+                $attributesMap = Map::instance($node->attributes);
                 $string        = $this->replaceNewTagString(
                         $match,
-                        $attributesMap->get( 'id' ),
-                        $this->map->getOrDefault( $node->attributes[ 'dataRefStart' ], 'NULL' ),
+                        $attributesMap->get('id'),
+                        $this->map->getOrDefault($node->attributes['dataRefStart'], 'NULL'),
                         CTypeEnum::PC_OPEN_DATA_REF,
                         $string
                 );
@@ -296,31 +296,31 @@ 
      *
      * @return string
      */
-    private function replaceClosingPcTags( $string, ArrayList $dataRefEndMap ) {
+    private function replaceClosingPcTags($string, ArrayList $dataRefEndMap) {
 
-        preg_match_all( '|</pc>|iu', $string, $closingPcMatches, PREG_OFFSET_CAPTURE );
+        preg_match_all('|</pc>|iu', $string, $closingPcMatches, PREG_OFFSET_CAPTURE);
         $delta = 0;
 
-        foreach ( $closingPcMatches[ 0 ] as $index => $match ) {
+        foreach ($closingPcMatches[0] as $index => $match) {
 
-            $offset = $match[ 1 ];
+            $offset = $match[1];
             $length = 5; // strlen of '</pc>'
 
-            $attr = $dataRefEndMap->get( $index );
-            if ( !empty( $attr ) && isset( $attr[ 'dataRefEnd' ] ) ) {
+            $attr = $dataRefEndMap->get($index);
+            if (!empty($attr) && isset($attr['dataRefEnd'])) {
 
                 // conversion for opening <pc> tag
                 $completeTag = $this->getNewTagString(
                         '</pc>',
-                        $attr[ 'id' ],
-                        $this->map->getOrDefault( $attr[ 'dataRefEnd' ], 'NULL' ),
+                        $attr['id'],
+                        $this->map->getOrDefault($attr['dataRefEnd'], 'NULL'),
                         CTypeEnum::PC_CLOSE_DATA_REF,
                         '_2'
                 );
 
-                $realOffset = ( $delta === 0 ) ? $offset : ( $offset + $delta );
-                $string     = substr_replace( $string, $completeTag, $realOffset, $length );
-                $delta      = $delta + strlen( $completeTag ) - $length;
+                $realOffset = ($delta === 0) ? $offset : ($offset + $delta);
+                $string     = substr_replace($string, $completeTag, $realOffset, $length);
+                $delta      = $delta + strlen($completeTag) - $length;
 
             }
 
@@ -338,17 +338,17 @@ 
      * @throws InvalidXmlException
      * @throws XmlParsingException
      */
-    public function restore( $string ) {
+    public function restore($string) {
 
         // if the map is empty return string as is
-        if ( empty( $this->map ) ) {
+        if (empty($this->map)) {
             return $string;
         }
 
-        $html = XmlParser::parse( $string, true );
+        $html = XmlParser::parse($string, true);
 
-        foreach ( $html as $node ) {
-            $string = $this->recursiveRestoreOriginalTags( $node, $string );
+        foreach ($html as $node) {
+            $string = $this->recursiveRestoreOriginalTags($node, $string);
         }
 
         return $string;
@@ -360,21 +360,21 @@ 
      *
      * @return string
      */
-    private function recursiveRestoreOriginalTags( $node, $string ) {
+    private function recursiveRestoreOriginalTags($node, $string) {
 
-        if ( $node->has_children ) {
+        if ($node->has_children) {
 
-            foreach ( $node->inner_html as $childNode ) {
-                $string = $this->recursiveRestoreOriginalTags( $childNode, $string );
+            foreach ($node->inner_html as $childNode) {
+                $string = $this->recursiveRestoreOriginalTags($childNode, $string);
             }
 
         } else {
 
-            $nodeAttributesMap = Map::instance( $node->attributes );
-            $cType             = $nodeAttributesMap->get( 'ctype' );
+            $nodeAttributesMap = Map::instance($node->attributes);
+            $cType             = $nodeAttributesMap->get('ctype');
 
-            if ( CTypeEnum::isLayer2Constant( $cType ) ) {
-                return preg_replace( '/' . preg_quote( $node->node, '/' ) . '/', base64_decode( $node->attributes[ 'x-orig' ] ), $string, 1 );
+            if (CTypeEnum::isLayer2Constant($cType)) {
+                return preg_replace('/'.preg_quote($node->node, '/').'/', base64_decode($node->attributes['x-orig']), $string, 1);
             }
 
         }
@@ -392,24 +392,24 @@ 
      *
      * @return string
      */
-    private function getNewTagString( $actualNodeString, $id, $dataRefValue, $ctype, $upCountIdValue = null ) {
+    private function getNewTagString($actualNodeString, $id, $dataRefValue, $ctype, $upCountIdValue = null) {
 
-        $newTag = [ '<ph' ];
+        $newTag = ['<ph'];
 
-        if ( isset( $id ) ) {
-            $newTag[] = 'id="' . $id . $upCountIdValue . '"';
+        if (isset($id)) {
+            $newTag[] = 'id="'.$id.$upCountIdValue.'"';
         }
 
-        $newTag[] = 'ctype="' . $ctype . '"';
-        $newTag[] = 'equiv-text="base64:' . base64_encode( $dataRefValue ) . '"';
-        $newTag[] = 'x-orig="' . base64_encode( $actualNodeString ) . '"';
+        $newTag[] = 'ctype="'.$ctype.'"';
+        $newTag[] = 'equiv-text="base64:'.base64_encode($dataRefValue).'"';
+        $newTag[] = 'x-orig="'.base64_encode($actualNodeString).'"';
 
-        return implode( " ", $newTag ) . '/>';
+        return implode(" ", $newTag).'/>';
 
     }
 
-    private function replaceNewTagString( $actualNodeString, $id, $dataRefValue, $ctype, $originalString, $upCountIdValue = '_1' ) {
-        return str_replace( $actualNodeString, $this->getNewTagString( $actualNodeString, $id, $dataRefValue, $ctype, $upCountIdValue ), $originalString );
+    private function replaceNewTagString($actualNodeString, $id, $dataRefValue, $ctype, $originalString, $upCountIdValue = '_1') {
+        return str_replace($actualNodeString, $this->getNewTagString($actualNodeString, $id, $dataRefValue, $ctype, $upCountIdValue), $originalString);
     }
 
 }
\ No newline at end of file

src/Utils/ArrayList.php

Spacing +9 added lines, -9 removed lines

@@ -16,12 +16,12 @@ 
     /**
      * @param array $list
      */
-    public function __construct( array $list = [] ) {
-        parent::__construct( $list );
+    public function __construct(array $list = []) {
+        parent::__construct($list);
     }
 
-    public static function instance( array $list = [] ) {
-        return new static( $list );
+    public static function instance(array $list = []) {
+        return new static($list);
     }
 
     /**
@@ -29,9 +29,9 @@ 
      *
      * @return false|mixed|null
      */
-    public function offsetGet( $key ) {
-        if ( $this->offsetExists( $key ) ) {
-            return parent::offsetGet( $key );
+    public function offsetGet($key) {
+        if ($this->offsetExists($key)) {
+            return parent::offsetGet($key);
         }
 
         return null;
@@ -44,8 +44,8 @@ 
      *
      * @return false|mixed|null the element at the specified position in this list
      */
-    public function get( $key ) {
-        return $this->offsetGet( $key );
+    public function get($key) {
+        return $this->offsetGet($key);
     }
 
 

src/Filters/DataRefRestore.php

Spacing +20 added lines, -20 removed lines

@@ -23,23 +23,23 @@ 
     /**
      * @inheritDoc
      */
-    public function transform( $segment ) {
+    public function transform($segment) {
 
-        if ( empty( $this->dataRefMap ) ) {
+        if (empty($this->dataRefMap)) {
             $this->dataRefMap = $this->pipeline->getDataRefMap();
         }
 
-        if ( empty( $this->dataRefMap ) ) {
-            $segment = $this->restoreXliffPhTagsFromMatecatPhTags( $segment );
+        if (empty($this->dataRefMap)) {
+            $segment = $this->restoreXliffPhTagsFromMatecatPhTags($segment);
 
-            return $this->restoreXliffPcTagsFromMatecatPhTags( $segment );
+            return $this->restoreXliffPcTagsFromMatecatPhTags($segment);
         }
 
-        $dataRefReplacer = new DataRefReplacer( $this->dataRefMap );
-        $segment         = $dataRefReplacer->restore( $segment );
-        $segment         = $this->restoreXliffPhTagsFromMatecatPhTags( $segment );
+        $dataRefReplacer = new DataRefReplacer($this->dataRefMap);
+        $segment         = $dataRefReplacer->restore($segment);
+        $segment         = $this->restoreXliffPhTagsFromMatecatPhTags($segment);
 
-        return $this->restoreXliffPcTagsFromMatecatPhTags( $segment );
+        return $this->restoreXliffPcTagsFromMatecatPhTags($segment);
     }
 
     /**
@@ -58,15 +58,15 @@ 
      *
      * @return string
      */
-    private function restoreXliffPhTagsFromMatecatPhTags( $segment ) {
-        preg_match_all( '|<ph id="mtc_[0-9]+" ctype="' . CTypeEnum::ORIGINAL_PH_OR_NOT_DATA_REF . '" equiv-text="base64:(.*?)"/>|iu', $segment, $matches );
+    private function restoreXliffPhTagsFromMatecatPhTags($segment) {
+        preg_match_all('|<ph id="mtc_[0-9]+" ctype="'.CTypeEnum::ORIGINAL_PH_OR_NOT_DATA_REF.'" equiv-text="base64:(.*?)"/>|iu', $segment, $matches);
 
-        if ( empty( $matches[ 0 ] ) ) {
+        if (empty($matches[0])) {
             return $segment;
         }
 
-        foreach ( $matches[ 0 ] as $index => $match ) {
-            $segment = str_replace( $match, base64_decode( $matches[ 1 ][ $index ] ), $segment );
+        foreach ($matches[0] as $index => $match) {
+            $segment = str_replace($match, base64_decode($matches[1][$index]), $segment);
         }
 
         return $segment;
@@ -88,16 +88,16 @@ 
      *
      * @return string
      */
-    private function restoreXliffPcTagsFromMatecatPhTags( $segment ) {
-        preg_match_all( '|<ph.+? ctype="' . CTypeEnum::ORIGINAL_PC_OPEN_NO_DATA_REF . '" equiv-text="base64:(.*?)"/>|iu', $segment, $matches );
-        preg_match_all( '|<ph.+? ctype="' . CTypeEnum::ORIGINAL_PC_CLOSE_NO_DATA_REF . '" equiv-text="base64:(.*?)"/>|iu', $segment, $matches );
+    private function restoreXliffPcTagsFromMatecatPhTags($segment) {
+        preg_match_all('|<ph.+? ctype="'.CTypeEnum::ORIGINAL_PC_OPEN_NO_DATA_REF.'" equiv-text="base64:(.*?)"/>|iu', $segment, $matches);
+        preg_match_all('|<ph.+? ctype="'.CTypeEnum::ORIGINAL_PC_CLOSE_NO_DATA_REF.'" equiv-text="base64:(.*?)"/>|iu', $segment, $matches);
 
-        if ( empty( $matches[ 0 ] ) ) {
+        if (empty($matches[0])) {
             return $segment;
         }
 
-        foreach ( $matches[ 0 ] as $index => $match ) {
-            $segment = str_replace( $match, base64_decode( $matches[ 1 ][ $index ] ), $segment );
+        foreach ($matches[0] as $index => $match) {
+            $segment = str_replace($match, base64_decode($matches[1][$index]), $segment);
         }
 
         return $segment;

src/Filters/FromLayer2ToRawXML.php

Spacing +3 added lines, -3 removed lines

@@ -23,11 +23,11 @@ 
 
     private $brokenHTML = false;
 
-    public function transform( $segment ) {
+    public function transform($segment) {
 
         //normal control characters must be converted to entities
         $segment = str_replace(
-                [ "\r\n", "\r", "\n", "\t", "", ],
+                ["\r\n", "\r", "\n", "\t", "", ],
                 [
                         '
',
                         '
',
@@ -37,7 +37,7 @@ 
                 ], $segment );
 
         // now convert the real  
-        return str_replace( ConstantEnum::nbspPlaceholder, Utils::unicode2chr( 0Xa0 ), $segment );
+        return str_replace(ConstantEnum::nbspPlaceholder, Utils::unicode2chr(0Xa0), $segment);
 
     }
 

src/Filters/EncodeToRawXML.php

Spacing +12 added lines, -12 removed lines

@@ -6,42 +6,42 @@
 use Matecat\SubFiltering\Utils\Utils;
 
 class EncodeToRawXML extends AbstractHandler {
-    public function transform( $segment ) {
+    public function transform($segment) {
 
         // handling 
 (line feed)
         // prevent to convert it to \n
-        $segment = preg_replace( '/&(#10;|#x0A;)|\n/', '##_ent_0A_##', $segment );
+        $segment = preg_replace('/&(#10;|#x0A;)|\n/', '##_ent_0A_##', $segment);
 
         // handling 
 (carriage return)
         // prevent to convert it to \r
-        $segment = preg_replace( '/&(#13;|#x0D;)|\r/', '##_ent_0D_##', $segment );
+        $segment = preg_replace('/&(#13;|#x0D;)|\r/', '##_ent_0D_##', $segment);
 
         // handling 	 (tab)
         // prevent to convert it to \t
-        $segment = preg_replace( '/	|\t/', '##_ent_09_##', $segment );
+        $segment = preg_replace('/	|\t/', '##_ent_09_##', $segment);
 
         //Substitute 4(+)-byte characters from a UTF-8 string to htmlentities
-        $segment = preg_replace_callback( '/([\xF0-\xF7]...)/s', [ Utils::class, 'htmlentitiesFromUnicode' ], $segment );
+        $segment = preg_replace_callback('/([\xF0-\xF7]...)/s', [Utils::class, 'htmlentitiesFromUnicode'], $segment);
 
         // handling 

-        if ( strpos( $segment, '##_ent_0D_##' ) !== false ) {
-            $segment = str_replace( '##_ent_0D_##', '
', $segment );
+        if (strpos($segment, '##_ent_0D_##') !== false) {
+            $segment = str_replace('##_ent_0D_##', '
', $segment);
         }
 
         // handling 
-        if ( strpos( $segment, '##_ent_0A_##' ) !== false ) {
-            $segment = str_replace( '##_ent_0A_##', '
', $segment );
+        if (strpos($segment, '##_ent_0A_##') !== false) {
+            $segment = str_replace('##_ent_0A_##', '
', $segment);
         }
 
         // handling 	 (tab)
         // prevent to convert it to \t
-        if ( strpos( $segment, '##_ent_09_##' ) !== false ) {
-            $segment = str_replace( '##_ent_09_##', '	', $segment );
+        if (strpos($segment, '##_ent_09_##') !== false) {
+            $segment = str_replace('##_ent_09_##', '	', $segment);
         }
 
 
         //encode all not valid XML entities
-        $segment = preg_replace( '/&(?!lt;|gt;|amp;|quot;|apos;|#[x]{0,1}[0-9A-F]{1,7};)/', '&', $segment );
+        $segment = preg_replace('/&(?!lt;|gt;|amp;|quot;|apos;|#[x]{0,1}[0-9A-F]{1,7};)/', '&', $segment);
 
         return $segment;
     }

src/Filters/DataRefReplace.php

Spacing +40 added lines, -40 removed lines

@@ -25,24 +25,24 @@ 
     /**
      * @inheritDoc
      */
-    public function transform( $segment ) {
+    public function transform($segment) {
 
-        if ( empty( $this->dataRefMap ) ) {
+        if (empty($this->dataRefMap)) {
             $this->dataRefMap = $this->pipeline->getDataRefMap();
         }
 
         // dataRefMap is present only in xliff 2.0 files
-        if ( empty( $this->dataRefMap ) ) {
-            $segment = $this->replace_Ph_TagsWithoutDataRefCorrespondenceToMatecatPhTags( $segment );
+        if (empty($this->dataRefMap)) {
+            $segment = $this->replace_Ph_TagsWithoutDataRefCorrespondenceToMatecatPhTags($segment);
 
-            return $this->replace_Pc_TagsWithoutDataRefCorrespondenceToMatecatPhTags( $segment );
+            return $this->replace_Pc_TagsWithoutDataRefCorrespondenceToMatecatPhTags($segment);
         }
 
-        $dataRefReplacer = new DataRefReplacer( $this->dataRefMap );
-        $segment         = $dataRefReplacer->replace( $segment );
-        $segment         = $this->replace_Ph_TagsWithoutDataRefCorrespondenceToMatecatPhTags( $segment );
+        $dataRefReplacer = new DataRefReplacer($this->dataRefMap);
+        $segment         = $dataRefReplacer->replace($segment);
+        $segment         = $this->replace_Ph_TagsWithoutDataRefCorrespondenceToMatecatPhTags($segment);
 
-        return $this->replace_Pc_TagsWithoutDataRefCorrespondenceToMatecatPhTags( $segment );
+        return $this->replace_Pc_TagsWithoutDataRefCorrespondenceToMatecatPhTags($segment);
     }
 
     /**
@@ -61,23 +61,23 @@ 
      *
      * @return string
      */
-    private function replace_Ph_TagsWithoutDataRefCorrespondenceToMatecatPhTags( $segment ) {
+    private function replace_Ph_TagsWithoutDataRefCorrespondenceToMatecatPhTags($segment) {
 
-        preg_match_all( '/<(ph .*?)>/iu', $segment, $phTags );
+        preg_match_all('/<(ph .*?)>/iu', $segment, $phTags);
 
-        if ( count( $phTags[ 0 ] ) === 0 ) {
+        if (count($phTags[0]) === 0) {
             return $segment;
         }
 
-        foreach ( $phTags[ 0 ] as $phTag ) {
+        foreach ($phTags[0] as $phTag) {
 
             // check if phTag has not any correspondence on dataRef map
-            if ( $this->isAValidPhTag( $phTag ) ) {
+            if ($this->isAValidPhTag($phTag)) {
                 $segment = preg_replace(
-                        '/' . preg_quote( $phTag, '/' ) . '/',
-                        '<ph id="' . $this->getPipeline()->getNextId() .
-                        '" ctype="' . CTypeEnum::ORIGINAL_PH_OR_NOT_DATA_REF .
-                        '" equiv-text="base64:' . base64_encode( $phTag ) .
+                        '/'.preg_quote($phTag, '/').'/',
+                        '<ph id="'.$this->getPipeline()->getNextId().
+                        '" ctype="'.CTypeEnum::ORIGINAL_PH_OR_NOT_DATA_REF.
+                        '" equiv-text="base64:'.base64_encode($phTag).
                         '"/>',
                         $segment,
                         1 // replace ONLY ONE occurrence
@@ -97,28 +97,28 @@ 
      *
      * @return bool
      */
-    private function isAValidPhTag( $phTag ) {
+    private function isAValidPhTag($phTag) {
 
         // try not to throw exception for wrong segments with opening tags and no closing
         try {
-            $parsed = XmlParser::parse( $phTag, true );
-        } catch ( Exception $e ){
+            $parsed = XmlParser::parse($phTag, true);
+        } catch (Exception $e) {
             return false;
         }
 
         // check for matecat ctype
-        $cType = isset( $parsed[ 0 ]->attributes[ 'ctype' ] ) ? $parsed[ 0 ]->attributes[ 'ctype' ] : null;
-        if ( CTypeEnum::isMatecatCType( $cType ) ) {
+        $cType = isset($parsed[0]->attributes['ctype']) ? $parsed[0]->attributes['ctype'] : null;
+        if (CTypeEnum::isMatecatCType($cType)) {
             return false;
         }
 
         // if has equiv-text don't touch
-        if ( isset( $parsed[ 0 ]->attributes[ 'equiv-text' ] ) ) {
+        if (isset($parsed[0]->attributes['equiv-text'])) {
             return false;
         }
 
-        if ( isset( $parsed[ 0 ]->attributes[ 'dataRef' ] ) ) {
-            return !array_key_exists( $parsed[ 0 ]->attributes[ 'dataRef' ], $this->dataRefMap );
+        if (isset($parsed[0]->attributes['dataRef'])) {
+            return !array_key_exists($parsed[0]->attributes['dataRef'], $this->dataRefMap);
         }
 
         return true;
@@ -141,33 +141,33 @@ 
      *
      * @return string
      */
-    private function replace_Pc_TagsWithoutDataRefCorrespondenceToMatecatPhTags( $segment ) {
+    private function replace_Pc_TagsWithoutDataRefCorrespondenceToMatecatPhTags($segment) {
 
-        preg_match_all( '/<(pc .*?)>/iu', $segment, $openingPcTags );
-        preg_match_all( '|<(/pc)>|iu', $segment, $closingPcTags );
+        preg_match_all('/<(pc .*?)>/iu', $segment, $openingPcTags);
+        preg_match_all('|<(/pc)>|iu', $segment, $closingPcTags);
 
-        if ( count( $openingPcTags[ 0 ] ) === 0 ) {
+        if (count($openingPcTags[0]) === 0) {
             return $segment;
         }
 
-        foreach ( $openingPcTags[ 0 ] as $openingPcTag ) {
+        foreach ($openingPcTags[0] as $openingPcTag) {
             $segment = preg_replace(
-                    '/' . preg_quote( $openingPcTag, '/' ) . '/',
-                    '<ph id="' . $this->getPipeline()->getNextId() .
-                    '" ctype="' . CTypeEnum::ORIGINAL_PC_OPEN_NO_DATA_REF .
-                    '" equiv-text="base64:' . base64_encode( $openingPcTag ) .
+                    '/'.preg_quote($openingPcTag, '/').'/',
+                    '<ph id="'.$this->getPipeline()->getNextId().
+                    '" ctype="'.CTypeEnum::ORIGINAL_PC_OPEN_NO_DATA_REF.
+                    '" equiv-text="base64:'.base64_encode($openingPcTag).
                     '"/>',
                     $segment,
                     1
             );
         }
 
-        foreach ( $closingPcTags[ 0 ] as $closingPcTag ) {
+        foreach ($closingPcTags[0] as $closingPcTag) {
             $segment = preg_replace(
-                    '/' . preg_quote( $closingPcTag, '/' ) . '/',
-                    '<ph id="' . $this->getPipeline()->getNextId() .
-                    '" ctype="' . CTypeEnum::ORIGINAL_PC_CLOSE_NO_DATA_REF .
-                    '" equiv-text="base64:' . base64_encode( $closingPcTag ) .
+                    '/'.preg_quote($closingPcTag, '/').'/',
+                    '<ph id="'.$this->getPipeline()->getNextId().
+                    '" ctype="'.CTypeEnum::ORIGINAL_PC_CLOSE_NO_DATA_REF.
+                    '" equiv-text="base64:'.base64_encode($closingPcTag).
                     '"/>',
                     $segment,
                     1

src/Enum/CTypeEnum.php

Spacing +11 added lines, -11 removed lines

@@ -44,17 +44,17 @@ 
      * @return array
      */
     protected static function getAllConstantValuesMap() {
-        if ( empty( static::$allConstantValues ) ) {
-            $reflectedProperty            = ( new ReflectionClass( static::class ) )->getConstants();
-            static::$allConstantValues    = array_flip( $reflectedProperty );
+        if (empty(static::$allConstantValues)) {
+            $reflectedProperty            = (new ReflectionClass(static::class))->getConstants();
+            static::$allConstantValues    = array_flip($reflectedProperty);
             static::$layer2ConstantValues = array_flip(
-                    array_filter( $reflectedProperty, function ( $key ) {
-                        return Utils::contains( 'DATA_REF', $key );
-                    }, ARRAY_FILTER_USE_KEY )
+                    array_filter($reflectedProperty, function($key) {
+                        return Utils::contains('DATA_REF', $key);
+                    }, ARRAY_FILTER_USE_KEY)
             );
         }
 
-        return [ 'all' => static::$allConstantValues, 'layer2' => static::$layer2ConstantValues ];
+        return ['all' => static::$allConstantValues, 'layer2' => static::$layer2ConstantValues];
     }
 
     /**
@@ -62,8 +62,8 @@ 
      *
      * @return bool
      */
-    public static function isMatecatCType( $ctype ) {
-        return array_key_exists( $ctype, static::getAllConstantValuesMap()[ 'all' ] );
+    public static function isMatecatCType($ctype) {
+        return array_key_exists($ctype, static::getAllConstantValuesMap()['all']);
     }
 
     /**
@@ -71,8 +71,8 @@ 
      *
      * @return bool
      */
-    public static function isLayer2Constant( $ctype ) {
-        return array_key_exists( $ctype, static::getAllConstantValuesMap()[ 'layer2' ] );
+    public static function isLayer2Constant($ctype) {
+        return array_key_exists($ctype, static::getAllConstantValuesMap()['layer2']);
     }
 
 }
\ No newline at end of file