Make WordPress Core

Opened 8 years ago

Closed 8 years ago

#45973 closed enhancement (duplicate)

Wrap global functions like __() in 'function_exists' condition

Reported by: jonasemde Owned by:
Priority: normal Milestone:
Component: I18N Version:
Severity: trivial Keywords:
Cc: Focuses:

Description

The () function is used in multiple frameworks and other open source libraries.

When both systems (e.g. WordPress and Laravel or WordPress and Magento) are required in one composer project the 'function already exists' Exception got thrown.

To override the function behaviour and/or be compatible with other frameworks like Laravel the functions should be wrapped within a 'function_exists' condition like so:

<?php
if (! function_exists('__')) {
    function __( $text, $domain = 'default' ) {
        return translate( $text, $domain );
    }
}

Especially the Laravel core does the same: https://github.com/laravel/framework/blob/5.7/src/Illuminate/Foundation/helpers.php

Attachments (2)

l10n.php (46.9 KB ) - added by jonasemde 8 years ago.
patch.diff (9.4 KB ) - added by jonasemde 8 years ago.
Patch

Download all attachments as: .zip

Change History (3)

@jonasemde
8 years ago

@jonasemde
8 years ago

Patch

#1 @swissspidy
8 years ago

  • Component GeneralI18N
  • Milestone Awaiting Review
  • Resolutionduplicate
  • Status newclosed

Hi there and welcome to WordPress Trac!

We've already gotten the same request in #41498, thus I am going to close this one as a duplicate.

Note: See TracTickets for help on using tickets.