Changeset 57518 for trunk/src/wp-includes/l10n/class-wp-translation-file.php

Timestamp:

02/01/2024 08:57:03 PM (2 years ago)

Author:

swissspidy

Message:

I18N: Fix plural forms parsing in WP_Translation_File.

Ensures the plural expression from the translation file header is correctly parsed.
Prevents silent failures in the attempt to create the plural form function.

Adds additional tests.

Props Chouby.
See #59656.

File:

• trunk/src/wp-includes/l10n/class-wp-translation-file.php (modified) (5 diffs)

trunk/src/wp-includes/l10n/class-wp-translation-file.php

@@ -208 +208 @@
 
     /**
-     * Returns the plural form for a count.
+     * Returns the plural form for a given number.
      *
      * @since 6.5.0
@@ -220 +220 @@
         }
 
-        // In case a plural form is specified as a header, but no function included, build one.
         if ( null === $this->plural_forms && isset( $this->headers['plural-forms'] ) ) {
-            $this->plural_forms = $this->make_plural_form_function( $this->headers['plural-forms'] );
+            $expression         = $this->get_plural_expression_from_header( $this->headers['plural-forms'] );
+            $this->plural_forms = $this->make_plural_form_function( $expression );
         }
 
@@ -232 +232 @@
              */
             $result = call_user_func( $this->plural_forms, $number );
+
             return $result;
         }
@@ -240 +241 @@
 
     /**
+     * Returns the plural forms expression as a tuple.
+     *
+     * @since 6.5.0
+     *
+     * @param string $header Plural-Forms header string.
+     * @return string Plural forms expression.
+     */
+    protected function get_plural_expression_from_header( $header ) {
+        if ( preg_match( '/^\s*nplurals\s*=\s*(\d+)\s*;\s+plural\s*=\s*(.+)$/', $header, $matches ) ) {
+            return trim( $matches[2] );
+        }
+
+        return 'n != 1';
+    }
+
+    /**
      * Makes a function, which will return the right translation index, according to the
      * plural forms header.
@@ -248 +265 @@
      * @return callable(int $num): int Plural forms function.
      */
-    public function make_plural_form_function( string $expression ): callable {
+    protected function make_plural_form_function( string $expression ): callable {
         try {
             $handler = new Plural_Forms( rtrim( $expression, ';' ) );