Changeset 55427 for trunk/src/wp-content/themes/twentytwenty/classes/class-twentytwenty-non-latin-languages.php

Timestamp:

02/27/2023 05:30:07 PM (2 years ago)

Author:

flixos90

Message:

Twenty Twenty: Avoid PHP warnings in 8.1 due to incorrect usage of wp_add_inline_style().

This changeset adjusts the logic to use a more defensive approach, to avoid passing potentially invalid values to wp_add_inline_style().

Props flixos90, mukesh27, costdev, swissspidy, poena, sabernhardt.
Fixes #57777.

File:

• trunk/src/wp-content/themes/twentytwenty/classes/class-twentytwenty-non-latin-languages.php (modified) (3 diffs)

trunk/src/wp-content/themes/twentytwenty/classes/class-twentytwenty-non-latin-languages.php

@@ -26 +26 @@
          *
          * @param string $type Whether to return CSS for the "front-end", "block-editor", or "classic-editor".
-         * @return void
+         * @return string|null Custom CSS, or null if not applicable.
          */
         public static function get_non_latin_css( $type = 'front-end' ) {
@@ -106 +106 @@
             // Return if the selected language has no fallback fonts.
             if ( empty( $font_family[ $locale ] ) ) {
-                return;
+                return null;
             }
 
@@ -127 +127 @@
             // Return if the specified type doesn't exist.
             if ( empty( $elements[ $type ] ) ) {
-                return;
+                return null;
             }