matecat / subfiltering

src/Contracts/FeatureSetInterface.php

Spacing +1 added lines, -1 removed lines

@@ -15,5 +15,5 @@
      *
      * @throws Exception
      */
-    public function filter( $method, $filterable );
+    public function filter($method, $filterable);
 }
\ No newline at end of file

src/Filters/ObjectiveCNSString.php

Spacing +5 added lines, -5 removed lines

@@ -35,7 +35,7 @@ 
      *
      * @inheritDoc
      */
-    public function transform( string $segment ): string {
+    public function transform(string $segment): string {
 
 //        $sprintfLocker = new SprintfLocker( $this->pipeline->getSource(), $this->pipeline->getTarget() );
 
@@ -43,12 +43,12 @@ 
 //        $segment = $sprintfLocker->lock( $segment );
 
 
-        preg_match_all( '/%\d+\$@|%@/', $segment, $html, PREG_SET_ORDER );
-        foreach ( $html as $percentNumberSnailVariable ) {
+        preg_match_all('/%\d+\$@|%@/', $segment, $html, PREG_SET_ORDER);
+        foreach ($html as $percentNumberSnailVariable) {
 
             $segment = preg_replace(
-                    '/' . preg_quote( $percentNumberSnailVariable[ 0 ], '/' ) . '/',
-                    '<ph id="' . $this->getPipeline()->getNextId() . '" ctype="' . CTypeEnum::OBJECTIVE_C_NSSTRING . '" equiv-text="base64:' . base64_encode( $percentNumberSnailVariable[ 0 ] ) . '"/>',
+                    '/'.preg_quote($percentNumberSnailVariable[0], '/').'/',
+                    '<ph id="'.$this->getPipeline()->getNextId().'" ctype="'.CTypeEnum::OBJECTIVE_C_NSSTRING.'" equiv-text="base64:'.base64_encode($percentNumberSnailVariable[0]).'"/>',
                     $segment,
                     1
             );

src/Filters/PlaceHoldXliffTags.php

Spacing +29 added lines, -29 removed lines

@@ -14,39 +14,39 @@
 
 class PlaceHoldXliffTags extends AbstractHandler {
 
-    public function transform( string $segment ): string {
+    public function transform(string $segment): string {
 
         // input : <g id="43">bang &amp; &lt; 3 olufsen </g>; <x id="33"/>
 
         //remove not existent </x> tags
-        $segment = preg_replace( '|(</x>)|si', "", $segment );
-
-        $segment = preg_replace( '|<(g\s*.*?)>|si', ConstantEnum::LTPLACEHOLDER . "$1" . ConstantEnum::GTPLACEHOLDER, $segment );
-        $segment = preg_replace( '|<(/g)>|si', ConstantEnum::LTPLACEHOLDER . "$1" . ConstantEnum::GTPLACEHOLDER, $segment );
-
-        $segment = preg_replace( '|<(x .*?/?)>|si', ConstantEnum::LTPLACEHOLDER . "$1" . ConstantEnum::GTPLACEHOLDER, $segment );
-        $segment = preg_replace( '#<(bx */?|bx .*?/?)>#si', ConstantEnum::LTPLACEHOLDER . "$1" . ConstantEnum::GTPLACEHOLDER, $segment );
-        $segment = preg_replace( '#<(ex */?|ex .*?/?)>#si', ConstantEnum::LTPLACEHOLDER . "$1" . ConstantEnum::GTPLACEHOLDER, $segment );
-        $segment = preg_replace( '|<(bpt\s*.*?)>|si', ConstantEnum::LTPLACEHOLDER . "$1" . ConstantEnum::GTPLACEHOLDER, $segment );
-        $segment = preg_replace( '|<(/bpt)>|si', ConstantEnum::LTPLACEHOLDER . "$1" . ConstantEnum::GTPLACEHOLDER, $segment );
-        $segment = preg_replace( '|<(ept\s*.*?)>|si', ConstantEnum::LTPLACEHOLDER . "$1" . ConstantEnum::GTPLACEHOLDER, $segment );
-        $segment = preg_replace( '|<(/ept)>|si', ConstantEnum::LTPLACEHOLDER . "$1" . ConstantEnum::GTPLACEHOLDER, $segment );
-        $segment = preg_replace( '|<(ph .*?)>|si', ConstantEnum::LTPLACEHOLDER . "$1" . ConstantEnum::GTPLACEHOLDER, $segment );
-        $segment = preg_replace( '|<(/ph)>|si', ConstantEnum::LTPLACEHOLDER . "$1" . ConstantEnum::GTPLACEHOLDER, $segment );
-        $segment = preg_replace( '|<(ec .*?)>|si', ConstantEnum::LTPLACEHOLDER . "$1" . ConstantEnum::GTPLACEHOLDER, $segment );
-        $segment = preg_replace( '|<(/ec)>|si', ConstantEnum::LTPLACEHOLDER . "$1" . ConstantEnum::GTPLACEHOLDER, $segment );
-        $segment = preg_replace( '|<(sc .*?)>|si', ConstantEnum::LTPLACEHOLDER . "$1" . ConstantEnum::GTPLACEHOLDER, $segment );
-        $segment = preg_replace( '|<(/sc)>|si', ConstantEnum::LTPLACEHOLDER . "$1" . ConstantEnum::GTPLACEHOLDER, $segment );
-        $segment = preg_replace( '|<(pc .*?)>|si', ConstantEnum::LTPLACEHOLDER . "$1" . ConstantEnum::GTPLACEHOLDER, $segment );
-        $segment = preg_replace( '|<(/pc)>|si', ConstantEnum::LTPLACEHOLDER . "$1" . ConstantEnum::GTPLACEHOLDER, $segment );
-        $segment = preg_replace( '|<(it .*?)>|si', ConstantEnum::LTPLACEHOLDER . "$1" . ConstantEnum::GTPLACEHOLDER, $segment );
-        $segment = preg_replace( '|<(/it)>|si', ConstantEnum::LTPLACEHOLDER . "$1" . ConstantEnum::GTPLACEHOLDER, $segment );
-        $segment = preg_replace( '|<(mrk\s*.*?)>|si', ConstantEnum::LTPLACEHOLDER . "$1" . ConstantEnum::GTPLACEHOLDER, $segment );
-        $segment = preg_replace( '|<(/mrk)>|si', ConstantEnum::LTPLACEHOLDER . "$1" . ConstantEnum::GTPLACEHOLDER, $segment );
-
-        return preg_replace_callback( '/' . ConstantEnum::LTPLACEHOLDER . '(.*?)' . ConstantEnum::GTPLACEHOLDER . '/u',
-                function ( $matches ) {
-                    return ConstantEnum::LTPLACEHOLDER . base64_encode( $matches[ 1 ] ) . ConstantEnum::GTPLACEHOLDER;
+        $segment = preg_replace('|(</x>)|si', "", $segment);
+
+        $segment = preg_replace('|<(g\s*.*?)>|si', ConstantEnum::LTPLACEHOLDER."$1".ConstantEnum::GTPLACEHOLDER, $segment);
+        $segment = preg_replace('|<(/g)>|si', ConstantEnum::LTPLACEHOLDER."$1".ConstantEnum::GTPLACEHOLDER, $segment);
+
+        $segment = preg_replace('|<(x .*?/?)>|si', ConstantEnum::LTPLACEHOLDER."$1".ConstantEnum::GTPLACEHOLDER, $segment);
+        $segment = preg_replace('#<(bx */?|bx .*?/?)>#si', ConstantEnum::LTPLACEHOLDER."$1".ConstantEnum::GTPLACEHOLDER, $segment);
+        $segment = preg_replace('#<(ex */?|ex .*?/?)>#si', ConstantEnum::LTPLACEHOLDER."$1".ConstantEnum::GTPLACEHOLDER, $segment);
+        $segment = preg_replace('|<(bpt\s*.*?)>|si', ConstantEnum::LTPLACEHOLDER."$1".ConstantEnum::GTPLACEHOLDER, $segment);
+        $segment = preg_replace('|<(/bpt)>|si', ConstantEnum::LTPLACEHOLDER."$1".ConstantEnum::GTPLACEHOLDER, $segment);
+        $segment = preg_replace('|<(ept\s*.*?)>|si', ConstantEnum::LTPLACEHOLDER."$1".ConstantEnum::GTPLACEHOLDER, $segment);
+        $segment = preg_replace('|<(/ept)>|si', ConstantEnum::LTPLACEHOLDER."$1".ConstantEnum::GTPLACEHOLDER, $segment);
+        $segment = preg_replace('|<(ph .*?)>|si', ConstantEnum::LTPLACEHOLDER."$1".ConstantEnum::GTPLACEHOLDER, $segment);
+        $segment = preg_replace('|<(/ph)>|si', ConstantEnum::LTPLACEHOLDER."$1".ConstantEnum::GTPLACEHOLDER, $segment);
+        $segment = preg_replace('|<(ec .*?)>|si', ConstantEnum::LTPLACEHOLDER."$1".ConstantEnum::GTPLACEHOLDER, $segment);
+        $segment = preg_replace('|<(/ec)>|si', ConstantEnum::LTPLACEHOLDER."$1".ConstantEnum::GTPLACEHOLDER, $segment);
+        $segment = preg_replace('|<(sc .*?)>|si', ConstantEnum::LTPLACEHOLDER."$1".ConstantEnum::GTPLACEHOLDER, $segment);
+        $segment = preg_replace('|<(/sc)>|si', ConstantEnum::LTPLACEHOLDER."$1".ConstantEnum::GTPLACEHOLDER, $segment);
+        $segment = preg_replace('|<(pc .*?)>|si', ConstantEnum::LTPLACEHOLDER."$1".ConstantEnum::GTPLACEHOLDER, $segment);
+        $segment = preg_replace('|<(/pc)>|si', ConstantEnum::LTPLACEHOLDER."$1".ConstantEnum::GTPLACEHOLDER, $segment);
+        $segment = preg_replace('|<(it .*?)>|si', ConstantEnum::LTPLACEHOLDER."$1".ConstantEnum::GTPLACEHOLDER, $segment);
+        $segment = preg_replace('|<(/it)>|si', ConstantEnum::LTPLACEHOLDER."$1".ConstantEnum::GTPLACEHOLDER, $segment);
+        $segment = preg_replace('|<(mrk\s*.*?)>|si', ConstantEnum::LTPLACEHOLDER."$1".ConstantEnum::GTPLACEHOLDER, $segment);
+        $segment = preg_replace('|<(/mrk)>|si', ConstantEnum::LTPLACEHOLDER."$1".ConstantEnum::GTPLACEHOLDER, $segment);
+
+        return preg_replace_callback('/'.ConstantEnum::LTPLACEHOLDER.'(.*?)'.ConstantEnum::GTPLACEHOLDER.'/u',
+                function($matches) {
+                    return ConstantEnum::LTPLACEHOLDER.base64_encode($matches[1]).ConstantEnum::GTPLACEHOLDER;
                 }, $segment
         ); //base64 of the tag content to avoid unwanted manipulation
     }

src/Filters/SprintfToPH.php

Indentation +1 added lines, -1 removed lines

@@ -42,7 +42,7 @@
      */
     public function transform( string $segment ): string {
 
-          // disabled for now, we want to check if this is really needed
+            // disabled for now, we want to check if this is really needed
 //        $sprintfLocker = new SprintfLocker( $this->pipeline->getSource(), $this->pipeline->getTarget() );
 
         // placeholding

Spacing +5 added lines, -5 removed lines

@@ -40,7 +40,7 @@ 
      * - https://www.w3resource.com/c-programming/stdio/c_library_method_sprintf.php
      *
      */
-    public function transform( string $segment ): string {
+    public function transform(string $segment): string {
 
           // disabled for now, we want to check if this is really needed
 //        $sprintfLocker = new SprintfLocker( $this->pipeline->getSource(), $this->pipeline->getTarget() );
@@ -52,13 +52,13 @@ 
         $regex = '/(?:\x25\x25)|(\x25(?:(?:[1-9]\d*)\$|\((?:[^)]+)\))?(?:\+)?(?:0|[+-]?\'[^$])?(?:-)?(?:\d+)?(?:\.(?:\d+))?((?:[hjlqtzL]{0,2}[a-iopsuxAC-HOSUX])))/';
 
 
-        preg_match_all( $regex, $segment, $vars, PREG_SET_ORDER );
-        foreach ( $vars as $variable ) {
+        preg_match_all($regex, $segment, $vars, PREG_SET_ORDER);
+        foreach ($vars as $variable) {
 
             //replace subsequent elements excluding already encoded
             $segment = preg_replace(
-                    '/' . preg_quote( $variable[ 0 ], '/' ) . '/',
-                    '<ph id="' . $this->getPipeline()->getNextId() . '" ctype="' . CTypeEnum::SPRINTF . '" equiv-text="base64:' . base64_encode( $variable[ 0 ] ) . '"/>',
+                    '/'.preg_quote($variable[0], '/').'/',
+                    '<ph id="'.$this->getPipeline()->getNextId().'" ctype="'.CTypeEnum::SPRINTF.'" equiv-text="base64:'.base64_encode($variable[0]).'"/>',
                     $segment,
                     1
             );