Changeset 44395 for trunk/src/wp-includes/class.wp-scripts.php

Timestamp:

01/04/2019 09:11:01 PM (7 years ago)

Author:

ocean90

Message:

I18N: Make domain argument optional in wp_set_script_translations() / WP_Scripts::set_translations().

Props swissspidy.
Fixes #45489.

File:

• trunk/src/wp-includes/class.wp-scripts.php (modified) (2 diffs)

trunk/src/wp-includes/class.wp-scripts.php

@@ -500 +500 @@
      *
      * @since 5.0.0
+     * @since 5.1.0 The `$domain` parameter was made optional.
      *
      * @param string $handle Name of the script to register a translation domain to.
-     * @param string $domain The textdomain.
+     * @param string $domain Optional. Text domain. Default 'default'.
      * @param string $path   Optional. The full file path to the directory containing translation files.
-     *
-     * @return bool True if the textdomain was registered, false if not.
-     */
-    public function set_translations( $handle, $domain, $path = null ) {
+     * @return bool True if the text domain was registered, false if not.
+     */
+    public function set_translations( $handle, $domain = 'default', $path = null ) {
         if ( ! isset( $this->registered[ $handle ] ) ) {
             return false;
@@ -518 +518 @@
             $obj->deps[] = 'wp-i18n';
         }
+
         return $obj->set_translations( $domain, $path );
     }