Как правильно поменять функцию preg_replace правильно на preg_replace_callback для PHP 7.2?

Question

[KBsts]

Перешел я тут наконец на PHP 7.2 ... И теперь ругается на функцию устаревшую!

$str = preg_replace('/"((?:""|[^"])*?)"/e', '"\\"".str_replace(";"," ","\\1")."\\""', $data);

И просит переделать её наконец в preg_replace_callback ... А каменный цветочек у меня не выходит что-то ((( Прощу помощи ...

Answer 1

[DanKud]

Вы немного не верно интерпретировали устаревание. Устарела не сама функция, а модификатор /e, который вы используете. В вашем примере, если я правильно понял, должно быть так:

$str = preg_replace_callback(
    '/"((?:""|[^"])*?)"',
    function ($m) {
        return '\\"'.str_replace(';',' ',$m[1]).'\\"';
    },
    $data
);

Answer 2

[Дмитрий Дарт]

Вы что-то путаете, эта функция не устарела php.net/manual/ru/function.preg-replace.php
Почему у вас в тексте замены другая функция?

Comments

[Melkij]

Речь о модификаторе e - eval

[Дмитрий Дарт]

Melkij, понял, прошу прощения

Answer 3

[dodo512]

$str = preg_replace_callback(
    '/"(?:""|[^"])*?"/',
    function ($m) {
        return str_replace(";"," ",$m[0]);
    },
    $data
);

Answer 4

[skv005]

Может и мне кто поможет. Часть старого кода который уже не работает на php7. Как его переделать, может кто привести пример?:

$text = preg_replace_callback('#\[url\]([\r\n]*)(http://|ftp://|https://|ftps://)([^\s\'\"]*?)([\r\n]*)\[/url\]#sie', "'<a href=\'".$pathto."'.base64_encode('\\2\\3').'\' rel=\'nofollow\' target=\'_blank\' title=\'\\2\\3\'>'.trimlink('\\2\\3', 20).(strlen('\\2\\3')>30?substr('\\2\\3', strlen('\\2\\3')-10, strlen('\\2\\3')):'').'</a>'", $text);

Comments

[dodo512]

$text = preg_replace_callback(
    '#\[url\]([\r\n]*)(http://|ftp://|https://|ftps://)([^\s\'\"]*?)([\r\n]*)\[/url\]#si', 
    function ($m) use($pathto) {
        $link = $m[2].$m[3];
        return "<a href='$pathto"
            .base64_encode($link)
            ."' rel='nofollow' target='_blank' title='$link'>"
            .trimlink($link, 20)
            .(strlen($link)>30?substr($link, strlen($link)-10, strlen($link)):'')
            .'</a>';
    },
    $text
);

[skv005]

dodo512, Большое Вам СПАСИБО! Теперь буду пытаться переделать всё остальное по этому образцу.

[skv005]

dodo512, - Можете ещё помочь. Если в коде присутствует 'http ://\\2').'\' как его прописывать в base64_encode?
Я пробовал так

....  function ($m) use($pathto) {$link = $m[2]; return "<a href='$pathto".base64_encode(http://$m[2])."' rel='....

Это не работает, пробовал и другие варианты методом тыка и всё я получаю сообщение об ошибке в $pathto.
Вот полная строка которую я не могу переделать:

$text = preg_replace('#\[url=([\r\n]*)([^\s\'\"]*?)\](.*?)([\r\n]*)\[/url\]#sie', "'<a href=\'".$pathto."'.base64_encode('http://\\2').'\' rel=\'nofollow\' target=\'_blank\' title=\'\\2\'>\\3</a>'", $text);

[dodo512]

base64_encode(http://$m[2])
skv005, тут нужны кавычки.
base64_encode("http://$m[2]")
А в остальном вроде норм.

[skv005]

dodo512, - Спасибо, это помогло. Теперь другая появилась ошибка:

unexpected '$m' (T_VARIABLE), expecting ';'

Я думаю что проблема у меня в этой части кода:

>\\3'", $text);

Я сделал так:

$text = preg_replace_callback('#\[url=([\r\n]*)([^\s\'\"]*?)\](.*?)([\r\n]*)\[/url\]#si', function ($m) use($pathto) {$link = $m[2]; return "<a href='$pathto".base64_encode(http://$m[2])."' rel='nofollow' target='_blank' title='$link'>"$m[3]</a>';}, $text);

Я просто вставил $m[3] и по идеи это должно работать... или неправильно?

[dodo512]

skv005,

{
        $link = $m[2];
        return "<a href='$pathto"
            .base64_encode("http://$m[2]")
            ."' rel='nofollow' target='_blank' title='$link'>$m[3]</a>";
        
    }

[skv005]

dodo512, Ох блин... ведь догадывался что дело в кавычках или в точке с запятой в конце строки. Извиняюсь за свою тупость. И ещё раз огромное вам СПАСИБО! В этой части кода теперь всё отлично.
Напоследок хотел спросить про одинарный обратный слэш. У меня он встречается в двух местах такой вариант http ://\2\ Мне так-же использовать $m[2] или для одинарного обратного слэша по другому?

[dodo512]

skv005, вероятно там такое же $m[2]