WordPress.org
Get WordPress

Make WordPress Core

Ticket #37082: 0001-Remove-two-unnecessary-uses-of-create_function-in-PO.patch

File 0001-Remove-two-unnecessary-uses-of-create_function-in-PO.patch, 2.0 KB (added by sgolemon, 8 years ago)

0001-Remove-two-unnecessary-uses-of-create_function-in-PO.patch

  • wp-includes/pomo/po.php 

    diff --git wp-includes/pomo/po.php wp-includes/pomo/po.php
index 65d65ff..6f06148 100644
    class PO extends Gettext_Translations { 
    280280        }
    281281
    282282        /**
     283         * Helper function for read_entry
     284         * @param string $context
     285         * @return bool
     286         */
     287        private static function isMsgStr($context) {
     288                return ($context === 'msgstr') || ($context === 'msgstr_plural');
     289        }
     290
     291        /**
    283292         * @param resource $f
    284293         * @param int      $lineno
    285294         * @return null|false|array
    … … class PO extends Gettext_Translations { 
    290299                // can be: comment, msgctxt, msgid, msgid_plural, msgstr, msgstr_plural
    291300                $context = '';
    292301                $msgstr_index = 0;
    293                 $is_final = create_function('$context', 'return $context == "msgstr" || $context == "msgstr_plural";');
    294302                while (true) {
    295303                        $lineno++;
    296304                        $line = PO::read_line($f);
    297305                        if (!$line)  {
    298306                                if (feof($f)) {
    299                                         if ($is_final($context))
     307                                        if (self::isMsgStr($context))
    300308                                                break;
    301309                                        elseif (!$context) // we haven't read a line and eof came
    302310                                                return null;
    … … class PO extends Gettext_Translations { 
    310318                        $line = trim($line);
    311319                        if (preg_match('/^#/', $line, $m)) {
    312320                                // the comment is the start of a new entry
    313                                 if ($is_final($context)) {
     321                                if (self::isMsgStr($context)) {
    314322                                        PO::read_line($f, 'put-back');
    315323                                        $lineno--;
    316324                                        break;
    … … class PO extends Gettext_Translations { 
    322330                                // add comment
    323331                                $this->add_comment_to_entry($entry, $line);
    324332                        } elseif (preg_match('/^msgctxt\s+(".*")/', $line, $m)) {
    325                                 if ($is_final($context)) {
     333                                if (self::isMsgStr($context)) {
    326334                                        PO::read_line($f, 'put-back');
    327335                                        $lineno--;
    328336                                        break;
    … … class PO extends Gettext_Translations { 
    333341                                $context = 'msgctxt';
    334342                                $entry->context .= PO::unpoify($m[1]);
    335343                        } elseif (preg_match('/^msgid\s+(".*")/', $line, $m)) {
    336                                 if ($is_final($context)) {
     344                                if (self::isMsgStr($context)) {
    337345                                        PO::read_line($f, 'put-back');
    338346                                        $lineno--;
    339347                                        break;