Bug do PHP 5.5 - Funcionalidade preterida: preg_replace ()

16

Após a atualização para o PHP 5.5, obtemos o seguinte erro ao adicionar um site, loja ou exibição de loja. Este bug ainda está presente no Magento 1.9.0.1

Exception message: Deprecated functionality: preg_replace(): The /e modifier is deprecated, use preg_replace_callback instead in app/code/core/Mage/Core/Helper/Abstract.php on line 238
Trace: #0 [internal function]: mageCoreErrorHandler(8192, 'preg_replace():...', 'app...', 238, Array)
#1 app/code/core/Mage/Core/Helper/Abstract.php(238): preg_replace('# <(?![/a-z]) |...', 'htmlentities('$...', 'New Store Name')
#2 app/code/core/Mage/Adminhtml/controllers/System/StoreController.php(175): Mage_Core_Helper_Abstract->removeTags('New Store Name')
#3 app/code/core/Mage/Core/Controller/Varien/Action.php(418): Mage_Adminhtml_System_StoreController->saveAction()
#4 app/code/core/Mage/Core/Controller/Varien/Router/Standard.php(250): Mage_Core_Controller_Varien_Action->dispatch('save')
#5 app/code/core/Mage/Core/Controller/Varien/Front.php(172): Mage_Core_Controller_Varien_Router_Standard->match(Object(Mage_Core_Controller_Request_Http))
#6 app/code/core/Mage/Core/Model/App.php(354): Mage_Core_Controller_Varien_Front->dispatch()
#7 app/Mage.php(686): Mage_Core_Model_App->run(Array)
#8 index.php(87): Mage::run('', 'store')
#9 {main}

Este é o código que produz o erro

O código pode ser encontrado em Mage_Core_Helper_Abstract

/**
 * Remove html tags, but leave "<" and ">" signs
 *
 * @param   string $html
 * @return  string
 */
public function removeTags($html)
{
    $html = preg_replace("# <(?![/a-z]) | (?<=\s)>(?![a-z]) #exi", "htmlentities('$0')", $html);
    $html =  strip_tags($html);
    return htmlspecialchars_decode($html);
}

Este é, na minha opinião, o patch mais fácil para o método:

/**
 * Remove html tags, but leave "<" and ">" signs
 *
 * @param   string $html
 * @return  string
 */
public function removeTags($html)
{
    $html = preg_replace_callback("# <(?![/a-z]) | (?<=\s)>(?![a-z]) #xi",
        create_function('$matches', 'return htmlentities($matches);'),
        $html
    );
    $html =  strip_tags($html);
    return htmlspecialchars_decode($html);
}

O método é usado apenas pelo Mage_Adminhtml_System_StoreController::storeAction().

Existem três lugares possíveis para corrigi-lo:

  1. Mage_Core_Helper_Abstract => é onde o método está localizado, mas é péssimo porque toca em um arquivo principal.
  2. Reescreva Mage_Core_Helper_Abstract => é uma classe abstrata, portanto não deve / não deve ser reescrita.
  3. Reescreva Mage_Adminhtml_Helper_Data e adicione o método lá. => Eu acho que este é o caminho a percorrer.

O que é que vocês acham?

  1. A opção 3 é a maneira correta de corrigir o problema.
  2. O código do meu patch está correto?
RobM84
fonte
O problema ainda existe no 1.9.1 CE e no 1.14.1 EE

Respostas:

13

Sim você está certo. Corrija o adminhtml helper. Este é o diff para a correção que eu uso:

--- app/code/core/Mage/Core/Helper/Abstract.php.orig 2014-09-25 15:32:56.000000000 +0200
+++ app/code/core/Mage/Core/Helper/Abstract.php 2014-09-25 15:34:42.000000000 +0200
@@ -235,7 +235,9 @@
  */
 public function removeTags($html)
 {
-        $html = preg_replace("# <(?![/a-z]) | (?<=\s)>(?![a-z]) #exi", "htmlentities('$0')", $html);
+        $html = preg_replace_callback("# <(?![/a-z]) | (?<=\s)>(?![a-z]) #xi", function($matches) {
+            return htmlentities($matches[0]);
+        }, $html);
         $html =  strip_tags($html);
         return htmlspecialchars_decode($html);
 }

Este é um teste para confirmar que o comportamento é o mesmo do php 5.4:

<?php

namespace Vinai\Kopp\Magento\Tests;

class MageAdminhtmlHelperDataTest extends \PHPUnit_Framework_TestCase
{
    /**
     * @var \Mage_Adminhtml_Helper_Data
     */
    private $helper;

    static public function setUpBeforeClass()
    {
        ini_set('display_errors', 1);
        umask(0);
        error_reporting(E_ALL);
        require_once 'app/Mage.php';
        \Mage::setIsDeveloperMode(true);
    }

    public function setUp()
    {
        $this->helper = new \Mage_Adminhtml_Helper_Data();
    }

    /**
     * @covers \Mage_Core_Helper_Abstract::removeTags
     * @dataProvider removeTagsDataProvider
     */
    public function testRemoveTags($inputHtml, $expected)
    {
        $result = $this->helper->removeTags($inputHtml);
        $this->assertEquals($expected, $result);
    }

    public function removeTagsDataProvider()
    {
        return array(
            array('<b>', ''),
            array('<b> >', ' >'),
            array('<b> <', ' <'),
            array('<b/> </', ' '),
            array('< <b/>', '< '),
            array('> <b/>', '> '),
            array('</ <b/>', ''),
            array('x />', 'x />'),
            array('> <', '> <'),
            array('>>', '>>'),
            array('<<', '<<'),
            array('<>', '<>'),
        );
    }
} 
Vinai
fonte
4

Agora isso foi corrigido no Magento EE 1.14.1 e 1.9.1. Incompatibilidade adicional são as alterações pack () / unpack () que afetam o Backup / Rollback e algumas extensões durante a instalação - qualquer coisa que toque em arquivos tar. Suponho que qualquer um que esteja executando o Magento em produção não esteja usando esses.

Piotr Kaminski
fonte
Quando o patch está sendo lançado para versões anteriores;)
Ben Lessani - Sonassi
ainda não sei se / quando
Piotr Kaminski 14/11
3

Resposta curta: Magento não é compatível com PHP 5.5, não atualize seu servidor da Web para 5.5.

Resposta mais longa: eu diria que o Magento corrige esse bug com a próxima versão, então eu faria um hack principal e esperaria o melhor. Não sei se o código está correto, desculpe.

Fabian Blechschmidt
fonte
Oi Fabian, nós rodamos todos os nossos servidores no PHP 5.5 por um bom tempo. Este é o primeiro problema que eu já encontrei. Quais outras incompatibilidades conhecidas existem ou de onde vêm essas informações?
21414 Robl84 #
1
tbh não faço ideia. Você pode apenas verificar o changelog php.net/manual/en/migration54.php e grep para os métodos e configurações ini
Fabian Blechschmidt
1
na verdade, esta é a única PHP 5.5 problema em magento CE, que não acertou outro no último semestre em execução
Flyingmana
2
também é um péssimo conselho, já que o 5.3 está muito desatualizado, o php 5.4 nunca atinge um estado estável real, pois a maioria das pessoas o usa com o APC, o 5.5 é atualmente a única versão mínima suportada do PHP disponível, sem mencionar todas as correções de segurança incluído em versões mais antigas do PHP #
Flyingmana 14/10