matecat / subfiltering

src/Enum/FiltersTagsEnum.php

Spacing +27 added lines, -27 removed lines

@@ -53,18 +53,18 @@ 
     ];
 
     protected const reverseTagMap = [
-            self::tagsMap[ self::xml ]                  => self::xml,
-            self::tagsMap[ self::percent_double_curly ] => self::percent_double_curly,
-            self::tagsMap[ self::twig ]                 => self::twig,
-            self::tagsMap[ self::ruby_on_rails ]        => self::ruby_on_rails,
-            self::tagsMap[ self::double_snail ]         => self::double_snail,
-            self::tagsMap[ self::double_square ]        => self::double_square,
-            self::tagsMap[ self::dollar_curly ]         => self::dollar_curly,
-            self::tagsMap[ self::single_curly ]         => self::single_curly,
-            self::tagsMap[ self::objective_c_ns ]       => self::objective_c_ns,
-            self::tagsMap[ self::double_percent ]       => self::double_percent,
-            self::tagsMap[ self::square_sprintf ]       => self::square_sprintf,
-            self::tagsMap[ self::sprintf ]              => self::sprintf,
+            self::tagsMap[self::xml]                  => self::xml,
+            self::tagsMap[self::percent_double_curly] => self::percent_double_curly,
+            self::tagsMap[self::twig]                 => self::twig,
+            self::tagsMap[self::ruby_on_rails]        => self::ruby_on_rails,
+            self::tagsMap[self::double_snail]         => self::double_snail,
+            self::tagsMap[self::double_square]        => self::double_square,
+            self::tagsMap[self::dollar_curly]         => self::dollar_curly,
+            self::tagsMap[self::single_curly]         => self::single_curly,
+            self::tagsMap[self::objective_c_ns]       => self::objective_c_ns,
+            self::tagsMap[self::double_percent]       => self::double_percent,
+            self::tagsMap[self::square_sprintf]       => self::square_sprintf,
+            self::tagsMap[self::sprintf]              => self::sprintf,
     ];
 
     /**
@@ -72,8 +72,8 @@ 
      *
      * @return string|null
      */
-    public static function classForTagName( ?string $name ): ?string {
-        return self::tagsMap[ $name ] ?? null;
+    public static function classForTagName(?string $name): ?string {
+        return self::tagsMap[$name] ?? null;
     }
 
     /**
@@ -85,18 +85,18 @@ 
      * @return array|null An array of handler class names corresponding to the provided tag names,
      *                    or null if the input is null.
      */
-    public static function classesForArrayTagNames( ?array $tagNames = [] ): ?array {
+    public static function classesForArrayTagNames(?array $tagNames = []): ?array {
 
-        if ( $tagNames === null ) {
+        if ($tagNames === null) {
             return null;
         }
 
         // Map tags to handler class names using the enum
         return array_values(
                 array_filter(
-                        array_map( static function ( string $tag ) {
-                            return self::classForTagName( $tag );
-                        }, $tagNames )
+                        array_map(static function(string $tag) {
+                            return self::classForTagName($tag);
+                        }, $tagNames)
                 )
         );
     }
@@ -107,7 +107,7 @@ 
      * @return array An array of tag keys.
      */
     public static function getTags(): array {
-        return array_keys( self::tagsMap );
+        return array_keys(self::tagsMap);
     }
 
 
@@ -118,8 +118,8 @@ 
      *
      * @return string|null The class name associated with the provided tag, or null if no association exists.
      */
-    public static function tagForClassName( string $tag ): ?string {
-        return self::reverseTagMap[ $tag ] ?? null;
+    public static function tagForClassName(string $tag): ?string {
+        return self::reverseTagMap[$tag] ?? null;
     }
 
     /**
@@ -130,17 +130,17 @@ 
      *
      * @return array|null An array of tag names or null if the input is null.
      */
-    public static function tagNamesForArrayClasses( ?array $classNames = [] ): ?array {
+    public static function tagNamesForArrayClasses(?array $classNames = []): ?array {
 
-        if ( $classNames === null ) {
+        if ($classNames === null) {
             return null;
         }
 
         return array_values(
                 array_filter(
-                        array_map( static function ( string $className ) {
-                            return self::tagForClassName( $className );
-                        }, $classNames )
+                        array_map(static function(string $className) {
+                            return self::tagForClassName($className);
+                        }, $classNames)
                 )
         );
     }

src/HandlersSorter.php

Spacing +21 added lines, -21 removed lines

@@ -40,18 +40,18 @@ 
      * @var array[] A map of handler class names to their integer priority and whether they are enabled by default.
      */
     protected const injectableHandlersOrder = [
-            XmlToPh::class                    => [ 'position' => 0, 'default_enabled' => true ],
-            PercentDoubleCurlyBrackets::class => [ 'position' => 1, 'default_enabled' => true ],
-            TwigToPh::class                   => [ 'position' => 2, 'default_enabled' => true ],
-            RubyOnRailsI18n::class            => [ 'position' => 3, 'default_enabled' => true ],
-            Snails::class                     => [ 'position' => 4, 'default_enabled' => true ],
-            DoubleSquareBrackets::class       => [ 'position' => 5, 'default_enabled' => true ],
-            DollarCurlyBrackets::class        => [ 'position' => 6, 'default_enabled' => true ],
-            SingleCurlyBracketsToPh::class    => [ 'position' => 7, 'default_enabled' => false ], // Disabled by default because it may conflict with other curly braces handlers
-            ObjectiveCNSString::class         => [ 'position' => 8, 'default_enabled' => true ],
-            DoublePercentages::class          => [ 'position' => 9, 'default_enabled' => true ],
-            SquareSprintf::class              => [ 'position' => 10, 'default_enabled' => true ],
-            SprintfToPH::class                => [ 'position' => 11, 'default_enabled' => true ],
+            XmlToPh::class                    => ['position' => 0, 'default_enabled' => true],
+            PercentDoubleCurlyBrackets::class => ['position' => 1, 'default_enabled' => true],
+            TwigToPh::class                   => ['position' => 2, 'default_enabled' => true],
+            RubyOnRailsI18n::class            => ['position' => 3, 'default_enabled' => true],
+            Snails::class                     => ['position' => 4, 'default_enabled' => true],
+            DoubleSquareBrackets::class       => ['position' => 5, 'default_enabled' => true],
+            DollarCurlyBrackets::class        => ['position' => 6, 'default_enabled' => true],
+            SingleCurlyBracketsToPh::class    => ['position' => 7, 'default_enabled' => false], // Disabled by default because it may conflict with other curly braces handlers
+            ObjectiveCNSString::class         => ['position' => 8, 'default_enabled' => true],
+            DoublePercentages::class          => ['position' => 9, 'default_enabled' => true],
+            SquareSprintf::class              => ['position' => 10, 'default_enabled' => true],
+            SprintfToPH::class                => ['position' => 11, 'default_enabled' => true],
     ];
 
     /**
@@ -60,8 +60,8 @@ 
      * @return array The array of handlers that are enabled by default.
      */
     public static function getDefaultInjectedHandlers(): array {
-        return array_filter( self::injectableHandlersOrder, function ( $settings ) {
-            return $settings[ 'default_enabled' ];
+        return array_filter(self::injectableHandlersOrder, function($settings) {
+            return $settings['default_enabled'];
         } );
     }
 
@@ -84,13 +84,13 @@ 
      *
      * @param class-string[] $injectedHandlers An array of handler class names to be sorted.
      */
-    public function __construct( array $injectedHandlers = [] ) {
+    public function __construct(array $injectedHandlers = []) {
 
         // Start with the default order of handlers.
         $this->defaultInjectedHandlers = self::injectableHandlersOrder;
 
         // Sort the final list of handlers according to their predefined execution order.
-        $this->injectedHandlers = $this->quickSort( $injectedHandlers );
+        $this->injectedHandlers = $this->quickSort($injectedHandlers);
 
     }
 
@@ -105,16 +105,16 @@ 
      *
      * @return string[] The sorted list of handler class names based on their priorities.
      */
-    private function quickSort( array $handlersList ): array {
+    private function quickSort(array $handlersList): array {
         // Filter the list to include only valid handlers.
-        $filteredHandlers = array_filter( $handlersList, function ( $handler ) {
-            return array_key_exists( $handler, $this->defaultInjectedHandlers );
+        $filteredHandlers = array_filter($handlersList, function($handler) {
+            return array_key_exists($handler, $this->defaultInjectedHandlers);
         } );
 
         // Sort the handlers based on their priority using a custom comparison function.
-        usort( $filteredHandlers, function ( $a, $b ) {
+        usort($filteredHandlers, function($a, $b) {
             // The spaceship operator (<=>) returns -1, 0, or 1, which is what usort expects.
-            return $this->defaultInjectedHandlers[ $a ] <=> $this->defaultInjectedHandlers[ $b ];
+            return $this->defaultInjectedHandlers[$a] <=> $this->defaultInjectedHandlers[$b];
         } );
 
         return $filteredHandlers;

src/AbstractFilter.php

Spacing +28 added lines, -28 removed lines

@@ -90,7 +90,7 @@ 
      *
      * @return T The configured instance of the filter.
      */
-    public static function getInstance( FeatureSetInterface $featureSet, ?string $source = null, ?string $target = null, ?array $dataRefMap = [], ?array $handlerTagNamesForLayer0ToLayer1Transition = [] ): ?AbstractFilter {
+    public static function getInstance(FeatureSetInterface $featureSet, ?string $source = null, ?string $target = null, ?array $dataRefMap = [], ?array $handlerTagNamesForLayer0ToLayer1Transition = []): ?AbstractFilter {
         // Create a new instance of the specific filter class (e.g., MateCatFilter).
         $newInstance = new static();
 
@@ -101,20 +101,20 @@ 
         // Use the null coalescing operator to default to an empty array if $dataRefMap is null.
         $newInstance->dataRefMap = $dataRefMap ?? [];
 
-        $handlerClassNamesForLayer0ToLayer1Transition = FiltersTagsEnum::classesForArrayTagNames( $handlerTagNamesForLayer0ToLayer1Transition );
+        $handlerClassNamesForLayer0ToLayer1Transition = FiltersTagsEnum::classesForArrayTagNames($handlerTagNamesForLayer0ToLayer1Transition);
 
         // Determine which handlers to use for the Layer 0 to Layer 1 transition.
-        if ( is_array( $handlerClassNamesForLayer0ToLayer1Transition ) && empty( $handlerClassNamesForLayer0ToLayer1Transition ) ) {
+        if (is_array($handlerClassNamesForLayer0ToLayer1Transition) && empty($handlerClassNamesForLayer0ToLayer1Transition)) {
             // If an empty array is passed, load the default set of handlers from the sorter.
-            $handlerClassNamesForLayer0ToLayer1Transition = array_keys( HandlersSorter::getDefaultInjectedHandlers() );
-        } elseif ( is_null( $handlerClassNamesForLayer0ToLayer1Transition ) ) {
+            $handlerClassNamesForLayer0ToLayer1Transition = array_keys(HandlersSorter::getDefaultInjectedHandlers());
+        } elseif (is_null($handlerClassNamesForLayer0ToLayer1Transition)) {
             // If null is passed, use no handlers.
             $handlerClassNamesForLayer0ToLayer1Transition = [];
         }
         // Otherwise, use the custom list of handlers provided.
 
         // Sort the dynamic feature-based handlers.
-        $sorter                                                  = new HandlersSorter( $handlerClassNamesForLayer0ToLayer1Transition );
+        $sorter                                                  = new HandlersSorter($handlerClassNamesForLayer0ToLayer1Transition);
         $newInstance->orderedHandlersForLayer0ToLayer1Transition = $sorter->getOrderedHandlersClassNames();
 
         // Return the fully configured filter instance.
@@ -132,26 +132,26 @@ 
      * @return string The transformed segment in Layer 0 format.
      * @throws Exception If any handler in the pipeline fails.
      */
-    public function fromLayer1ToLayer0( string $segment ): string {
+    public function fromLayer1ToLayer0(string $segment): string {
         // Initialize a new pipeline for this transformation.
-        $channel = new Pipeline( $this->source, $this->target, $this->dataRefMap );
+        $channel = new Pipeline($this->source, $this->target, $this->dataRefMap);
 
         // Add handlers to reverse the sub-filtering process.
-        $channel->addLast( MateCatCustomPHToOriginalValue::class ); // Restore original PH values
-        $channel->addLast( PlaceHoldXliffTags::class );             // Isolate XLIFF tags
-        $channel->addLast( EncodeToRawXML::class );                 // Encode for raw XML storage
-        $channel->addLast( LtGtEncode::class );                     // Encode '<' and '>'
-        $channel->addLast( RestoreXliffTagsContent::class );        // Restore original XLIFF content
-        $channel->addLast( RestorePlaceHoldersToXLIFFLtGt::class ); // Restore placeholders for '<' and '>'
-        $channel->addLast( SplitPlaceholder::class );               // Handle split placeholders
-        $channel->addLast( RestoreEquivText::class );               // Restore equiv-text content
+        $channel->addLast(MateCatCustomPHToOriginalValue::class); // Restore original PH values
+        $channel->addLast(PlaceHoldXliffTags::class); // Isolate XLIFF tags
+        $channel->addLast(EncodeToRawXML::class); // Encode for raw XML storage
+        $channel->addLast(LtGtEncode::class); // Encode '<' and '>'
+        $channel->addLast(RestoreXliffTagsContent::class); // Restore original XLIFF content
+        $channel->addLast(RestorePlaceHoldersToXLIFFLtGt::class); // Restore placeholders for '<' and '>'
+        $channel->addLast(SplitPlaceholder::class); // Handle split placeholders
+        $channel->addLast(RestoreEquivText::class); // Restore equiv-text content
 
         // Allow the current feature set to modify the pipeline (e.g., add or remove handlers).
         /** @var $channel Pipeline */
-        $channel = $this->featureSet->filter( 'fromLayer1ToLayer0', $channel );
+        $channel = $this->featureSet->filter('fromLayer1ToLayer0', $channel);
 
         // Process the segment through the pipeline and return the result.
-        return $channel->transform( $segment );
+        return $channel->transform($segment);
     }
 
 
@@ -167,7 +167,7 @@ 
      *
      * @return string The transformed segment after processing from Layer 0 to Layer 1.
      */
-    public abstract function fromLayer0ToLayer1( string $segment, ?string $cid = null ): string;
+    public abstract function fromLayer0ToLayer1(string $segment, ?string $cid = null): string;
 
     /**
      * Configures the pipeline for transforming content from Layer 0 to Layer 1.
@@ -182,22 +182,22 @@ 
      * @param Pipeline    $channel
      * @param string|null $cid
      */
-    protected function configureFromLayer0ToLayer1Pipeline( Pipeline $channel, ?string $cid = null ): void {
+    protected function configureFromLayer0ToLayer1Pipeline(Pipeline $channel, ?string $cid = null): void {
 
         // Add initial handlers for standard XLIFF and placeholder normalization.
-        $channel->addLast( StandardPHToMateCatCustomPH::class );
-        $channel->addLast( StandardXEquivTextToMateCatCustomPH::class );
-        $channel->addLast( PlaceHoldXliffTags::class );
+        $channel->addLast(StandardPHToMateCatCustomPH::class);
+        $channel->addLast(StandardXEquivTextToMateCatCustomPH::class);
+        $channel->addLast(PlaceHoldXliffTags::class);
 
         // Add the dynamic feature-based handlers.
-        foreach ( $this->orderedHandlersForLayer0ToLayer1Transition as $handler ) {
-            $channel->addLast( $handler );
+        foreach ($this->orderedHandlersForLayer0ToLayer1Transition as $handler) {
+            $channel->addLast($handler);
         }
 
         // Add final handlers to restore XLIFF content and encode for the target layer.
-        $channel->addLast( RestoreXliffTagsContent::class );
-        $channel->addLast( RestorePlaceHoldersToXLIFFLtGt::class );
-        $channel->addLast( EquivTextToBase64::class );
+        $channel->addLast(RestoreXliffTagsContent::class);
+        $channel->addLast(RestorePlaceHoldersToXLIFFLtGt::class);
+        $channel->addLast(EquivTextToBase64::class);
 
     }
 

src/MyMemoryFilter.php

Spacing +14 added lines, -14 removed lines

@@ -36,14 +36,14 @@ 
      *
      * @return string The transformed segment in Layer 1 format.
      */
-    public function fromLayer0ToLayer1( string $segment, ?string $cid = null ): string {
+    public function fromLayer0ToLayer1(string $segment, ?string $cid = null): string {
 
-        $channel = new Pipeline( $this->source, $this->target, $this->dataRefMap );
+        $channel = new Pipeline($this->source, $this->target, $this->dataRefMap);
 
-        $this->configureFromLayer0ToLayer1Pipeline( $channel, $cid );
+        $this->configureFromLayer0ToLayer1Pipeline($channel, $cid);
 
         // Process the segment and return the result.
-        return $channel->transform( $segment );
+        return $channel->transform($segment);
 
     }
 
@@ -61,33 +61,33 @@ 
      *                             - 'familysearch': Removes `TwigToPh` and adds `SingleCurlyBracketsToPh`.
      *
      */
-    protected function configureFromLayer0ToLayer1Pipeline( Pipeline $channel, ?string $cid = null ): void {
+    protected function configureFromLayer0ToLayer1Pipeline(Pipeline $channel, ?string $cid = null): void {
 
-        parent::configureFromLayer0ToLayer1Pipeline( $channel );
+        parent::configureFromLayer0ToLayer1Pipeline($channel);
 
-        switch ( $cid ) {
+        switch ($cid) {
             case 'airbnb':
                 // For Airbnb, add the SmartCounts handler to process specific variable syntax
                 // that looks like `%{smart_count}`.
-                if ( !$channel->contains( SmartCounts::class ) ) {
-                    $channel->addAfter( PercentDoubleCurlyBrackets::class, SmartCounts::class );
+                if (!$channel->contains(SmartCounts::class)) {
+                    $channel->addAfter(PercentDoubleCurlyBrackets::class, SmartCounts::class);
                 }
                 break;
 
             case 'roblox':
                 // For Roblox, add a handler to convert single curly bracket placeholders
                 // (e.g., `{placeholder}`) into standard <ph> tags.
-                if ( !$channel->contains( SingleCurlyBracketsToPh::class ) ) {
-                    $channel->addAfter( DollarCurlyBrackets::class, SingleCurlyBracketsToPh::class );
+                if (!$channel->contains(SingleCurlyBracketsToPh::class)) {
+                    $channel->addAfter(DollarCurlyBrackets::class, SingleCurlyBracketsToPh::class);
                 }
                 break;
 
             case 'familysearch':
                 // For FamilySearch, customize the pipeline by removing Twig support and adding
                 // support for single curly bracket placeholders.
-                if ( !$channel->contains( SingleCurlyBracketsToPh::class ) ) {
-                    $channel->remove( TwigToPh::class );
-                    $channel->addAfter( DollarCurlyBrackets::class, SingleCurlyBracketsToPh::class );
+                if (!$channel->contains(SingleCurlyBracketsToPh::class)) {
+                    $channel->remove(TwigToPh::class);
+                    $channel->addAfter(DollarCurlyBrackets::class, SingleCurlyBracketsToPh::class);
                 }
                 break;
         }