matecat / subfiltering

src/Commons/EmptyFeatureSet.php

Spacing +1 added lines, -1 removed lines

@@ -19,7 +19,7 @@
     /**
      * @inheritDoc
      */
-    public function filter( $method, $filterable ) {
+    public function filter($method, $filterable) {
         return $filterable;
     }
 }
\ No newline at end of file

src/Enum/CTypeEnum.php

Spacing +11 added lines, -11 removed lines

@@ -45,17 +45,17 @@ 
      * @return array
      */
     protected static function getAllConstantValuesMap(): array {
-        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];
     }
 
     /**
@@ -63,8 +63,8 @@ 
      *
      * @return bool
      */
-    public static function isMatecatCType( string $ctype ): bool {
-        return array_key_exists( $ctype, static::getAllConstantValuesMap()[ 'all' ] );
+    public static function isMatecatCType(string $ctype): bool {
+        return array_key_exists($ctype, static::getAllConstantValuesMap()['all']);
     }
 
     /**
@@ -72,8 +72,8 @@ 
      *
      * @return bool
      */
-    public static function isLayer2Constant( string $ctype ): bool {
-        return array_key_exists( $ctype, static::getAllConstantValuesMap()[ 'layer2' ] );
+    public static function isLayer2Constant(string $ctype): bool {
+        return array_key_exists($ctype, static::getAllConstantValuesMap()['layer2']);
     }
 
 }
\ No newline at end of file

src/Contracts/FeatureSetInterface.php

Spacing +1 added lines, -1 removed lines

@@ -17,5 +17,5 @@
      * modified in cascade to the next function in the queue.
      * @throws Exception
      */
-    public function filter( $method, $filterable );
+    public function filter($method, $filterable);
 }
\ No newline at end of file

src/Filters/Sprintf/SprintfLocker.php

Spacing +24 added lines, -24 removed lines

@@ -36,7 +36,7 @@ 
      * @param string|null $source
      * @param string|null $target
      */
-    public function __construct( ?string $source = null, ?string $target = null ) {
+    public function __construct(?string $source = null, ?string $target = null) {
         $this->source        = $source;
         $this->target        = $target;
         $this->notAllowedMap = $this->createNotAllowedMap();
@@ -48,17 +48,17 @@ 
     private function createNotAllowedMap(): array {
         $map = [];
 
-        $all = include __DIR__ . "/language/all/not_allowed.php";
-        $map = array_merge( $map, $all );
+        $all = include __DIR__."/language/all/not_allowed.php";
+        $map = array_merge($map, $all);
 
-        if ( $this->source and file_exists( __DIR__ . "/language/" . $this->source . "/not_allowed.php" ) ) {
-            $source = include __DIR__ . "/language/" . $this->source . "/not_allowed.php";
-            $map    = array_merge( $map, $source );
+        if ($this->source and file_exists(__DIR__."/language/".$this->source."/not_allowed.php")) {
+            $source = include __DIR__."/language/".$this->source."/not_allowed.php";
+            $map    = array_merge($map, $source);
         }
 
-        if ( $this->target and file_exists( __DIR__ . "/language/" . $this->target . "/not_allowed.php" ) ) {
-            $target = include __DIR__ . "/language/" . $this->target . "/not_allowed.php";
-            $map    = array_merge( $map, $target );
+        if ($this->target and file_exists(__DIR__."/language/".$this->target."/not_allowed.php")) {
+            $target = include __DIR__."/language/".$this->target."/not_allowed.php";
+            $map    = array_merge($map, $target);
         }
 
         return $map;
@@ -69,11 +69,11 @@ 
      *
      * @return string
      */
-    public function lock( string $segment ): string {
-        $replacementMap       = $this->createReplacementMap( $segment );
+    public function lock(string $segment): string {
+        $replacementMap       = $this->createReplacementMap($segment);
         $this->replacementMap = $replacementMap;
 
-        return str_replace( array_keys( $replacementMap ), array_values( $replacementMap ), $segment );
+        return str_replace(array_keys($replacementMap), array_values($replacementMap), $segment);
     }
 
     /**
@@ -81,10 +81,10 @@ 
      *
      * @return string
      */
-    public function unlock(string $segment ): string {
+    public function unlock(string $segment): string {
         $replacementMap = $this->replacementMap;
 
-        return str_replace( array_values( $replacementMap ), array_keys( $replacementMap ), $segment );
+        return str_replace(array_values($replacementMap), array_keys($replacementMap), $segment);
     }
 
     /**
@@ -94,23 +94,23 @@ 
      *
      * @return array
      */
-    private function createReplacementMap( string $segment ): array {
+    private function createReplacementMap(string $segment): array {
         $replacementMap = [];
 
-        foreach ( $this->notAllowedMap as $item => $details ) {
+        foreach ($this->notAllowedMap as $item => $details) {
 
-            $type = $details[ 'type' ];
+            $type = $details['type'];
 
-            switch ( $type ) {
+            switch ($type) {
                 case 'exact':
-                    $replacementMap[ $item ] = self::PRE_LOCK_TAG . $this->maskString( $item ) . self::POST_LOCK_TAG;
+                    $replacementMap[$item] = self::PRE_LOCK_TAG.$this->maskString($item).self::POST_LOCK_TAG;
                     break;
 
                 case 'regex':
-                    preg_match_all( '/' . $item . '/', $segment, $matches );
+                    preg_match_all('/'.$item.'/', $segment, $matches);
 
-                    foreach ( $matches[ 0 ] as $match ) {
-                        $replacementMap[ $match ] = self::PRE_LOCK_TAG . $this->maskString( $match ) . self::POST_LOCK_TAG;
+                    foreach ($matches[0] as $match) {
+                        $replacementMap[$match] = self::PRE_LOCK_TAG.$this->maskString($match).self::POST_LOCK_TAG;
                     }
                     break;
             }
@@ -124,8 +124,8 @@ 
      *
      * @return string
      */
-    private function maskString( string $string ): string {
-        return str_replace( [ '%', '-', '_' ], '', $string );
+    private function maskString(string $string): string {
+        return str_replace(['%', '-', '_'], '', $string);
     }
 
 }
\ No newline at end of file

src/Filters/RestoreEquivText.php

Spacing +8 added lines, -8 removed lines

@@ -13,18 +13,18 @@
 
 class RestoreEquivText extends AbstractHandler {
 
-    public function transform( string $segment ): string {
+    public function transform(string $segment): string {
 
         // extract equiv-text attribute
-        preg_match_all( '/equiv-text=\"(.*?)\"/', $segment, $equiv_tags );
+        preg_match_all('/equiv-text=\"(.*?)\"/', $segment, $equiv_tags);
 
-        if ( !empty( $equiv_tags[ 0 ] ) ) {
-            foreach ( $equiv_tags[ 0 ] as $index => $equiv_tag ) {
-                $tag = $equiv_tags[ 1 ][ $index ];
+        if (!empty($equiv_tags[0])) {
+            foreach ($equiv_tags[0] as $index => $equiv_tag) {
+                $tag = $equiv_tags[1][$index];
 
-                if ( strpos( $tag, "base64:" ) !== false ) {
-                    $b       = base64_decode( str_replace( "base64:", "", $tag ) );
-                    $segment = str_replace( $equiv_tags[ $index ], 'equiv-text="' . $b . '"', $segment );
+                if (strpos($tag, "base64:") !== false) {
+                    $b       = base64_decode(str_replace("base64:", "", $tag));
+                    $segment = str_replace($equiv_tags[$index], 'equiv-text="'.$b.'"', $segment);
                 }
             }
         }

src/Filters/EncodeToRawXML.php

Spacing +12 added lines, -12 removed lines

@@ -6,41 +6,41 @@
 use Matecat\SubFiltering\Utils\Utils;
 
 class EncodeToRawXML extends AbstractHandler {
-    public function transform( string $segment ): string {
+    public function transform(string $segment): string {
 
         // 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
-        return preg_replace( '/&(?!lt;|gt;|amp;|quot;|apos;|#[x]{0,1}[0-9A-F]{1,7};)/', '&', $segment );
+        return preg_replace('/&(?!lt;|gt;|amp;|quot;|apos;|#[x]{0,1}[0-9A-F]{1,7};)/', '&', $segment);
     }
 }
\ No newline at end of file

src/Filters/EmojiToEntity.php

Spacing +2 added lines, -2 removed lines

@@ -14,8 +14,8 @@
 
 class EmojiToEntity extends AbstractHandler {
 
-    public function transform( string $segment ): string {
-        return Emoji::toEntity( $segment );
+    public function transform(string $segment): string {
+        return Emoji::toEntity($segment);
     }
 
 }
\ No newline at end of file

src/Filters/SquareSprintf.php

Spacing +6 added lines, -6 removed lines

@@ -9,7 +9,7 @@ 
     /**
      * @inheritDoc
      */
-    public function transform( string $segment ): string {
+    public function transform(string $segment): string {
 
         $tags = [
                 '\[%s\]',
@@ -29,15 +29,15 @@ 
                 '\[%[a-z_]+:\d+%\]',
         ];
 
-        $regex = '/' . implode( "|", $tags ) . '/iu';
+        $regex = '/'.implode("|", $tags).'/iu';
 
-        preg_match_all( $regex, $segment, $html, PREG_SET_ORDER );
+        preg_match_all($regex, $segment, $html, PREG_SET_ORDER);
 
-        foreach ( $html as $pos => $percentSnailVariable ) {
+        foreach ($html as $pos => $percentSnailVariable) {
 
             $segment = preg_replace(
-                    '/' . preg_quote( $percentSnailVariable[ 0 ], '/' ) . '/',
-                    '<ph id="' . $this->getPipeline()->getNextId() . '" ctype="' . CTypeEnum::SQUARE_SPRINTF . '" equiv-text="base64:' . base64_encode( $percentSnailVariable[ 0 ] ) . '"/>',
+                    '/'.preg_quote($percentSnailVariable[0], '/').'/',
+                    '<ph id="'.$this->getPipeline()->getNextId().'" ctype="'.CTypeEnum::SQUARE_SPRINTF.'" equiv-text="base64:'.base64_encode($percentSnailVariable[0]).'"/>',
                     $segment,
                     1
             );

src/Filters/DataRefReplace.php

Spacing +40 added lines, -40 removed lines

@@ -25,24 +25,24 @@ 
     /**
      * @inheritDoc
      */
-    public function transform( string $segment ): string {
+    public function transform(string $segment): string {
 
-        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( string $segment ): string {
+    private function replace_Ph_TagsWithoutDataRefCorrespondenceToMatecatPhTags(string $segment): string {
 
-        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( string $phTag ): bool {
+    private function isAValidPhTag(string $phTag): bool {
 
         // 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' ] : 'not-found';
-        if ( CTypeEnum::isMatecatCType( $cType ) ) {
+        $cType = isset($parsed[0]->attributes['ctype']) ? $parsed[0]->attributes['ctype'] : 'not-found';
+        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( string $segment ): string {
+    private function replace_Pc_TagsWithoutDataRefCorrespondenceToMatecatPhTags(string $segment): string {
 
-        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