Предложение к будущей версии :) (Страница 6 из 7)

ок. понятно.

hcs, https://github.com/punbb/punbb/pull/137

кстати, расширениям было бы очень неплохо иметь возможность показывать свою картинку, чтобы в списке расширений было легче ориентироваться

Добавлено спустя 1 минуту 16 секунд:

Visman,  я правильно понимаю, что для quote preg_replace_callback не нужен?

Твой парсер - твой пулл-реквест?

create_function() предлагают сделать устаревшей с PHP 7.2 https://wiki.php.net/rfc/deprecations_php_7_2

hcs, а почему этот реквест https://github.com/punbb/punbb/pull/128 не принимается?

Судя по http://php.net/manual/ru/reserved.variables.server.php

логика верная.

если переделывать парсер - может на xml или html?

типа заменяем квадратные скобки на угловые фильтруем лишние теги и атрибуты...

Добавлено спустя 1 минуту:

есть либа для фильтрования http://htmlpurifier.org/

идея была - bbcode parser делать на базе xml.

Добавлено спустя 7 минут 14 секунд:

нужен механизм для переопределения функций punbb

добавляем в essentials.php

if (is_readable(FORUM_ROOT.'include/user/functions.php')) {
 require FORUM_ROOT.'include/user/functions.php'
}
require FORUM_ROOT.'include/functions.php';

пример для get_hook в include/user/functions.php

function get_hook($hook_id)
{
    global $forum_hooks;

// тут чтото делаем
    return !defined('FORUM_DISABLE_HOOKS') && isset($forum_hooks[$hook_id]) ? implode("\n", $forum_hooks[$hook_id]) : false;
}

конечно для каждой функции в include/functions.php придется добавить проверку
if (!function_exists("get_hook")) {
//...
}

но это можно делать постепенно

hcs,
что думаешь?

Добавлено спустя 8 минут 2 секунды:

примеш такой пулреквест?

hcs, разве он его давным давно не забросил? Это же он https://github.com/samclarke/SBBCodeParser ?

hcs, криво работает

$parser = new \SBBCodeParser\Node_Container_Document();
echo $parser->parse('[quote="[b]имя[/b]"]цитата[/quote]')
    ->get_html();

Результат html

[quote=&quot;<strong>имя</strong>&quot;]цитата[/quote]

Один из вариантов своего парсера делал по аналогии, то есть теги не в массиве, а как дерево классов. Удобно конечно, но медленнее шевелится.

это не для расширений. нужно по быстрому менять код некоторых функций без изменения кода движка.
т.е. через расширения такие системные функции никак не изменить.
такая возможность реализована в темах Wordpress.
ладно - код вроде не такой сложный. можно и сразу в файле поменять.

Добавлено спустя 2 минуты 51 секунду:

xml парсер работает быстрее и имеет валидацию структуры кода.
легко применить для внедрения новых html снипетов.
что как мне каждется намного упрощает реализацию bbcode парсера.

Добавлено спустя 21 минуту 13 секунд:

hcs, а тогда как насчет добавления в функцию get_hook

https://github.com/punbb/punbb/blob/mas … s.php#L787

строку

($hook = get_hook('get_hook')) ? eval($hook) : null;

?

хочу запилить в расширении вывод хуков страницы
для дебага полезная фича - но пока не осуществима т.к. в данной функции нельзя применить хуки

https://github.com/rivetweb/punbb-ext-h … v/issues/3

Добавлено спустя 23 минуты 34 секунды:

написал бред - там получится рекурсивный вывод

Добавлено спустя 29 минут 29 секунд:

может флаг тогда сделать и в коде прописать дебаг инструкцию?

https://github.com/punbb/punbb/blob/mas … s.php#L787

if (defined("FORUM_DEBUG")) {
  echo "<script>console.log(HOOK: " . $hook_id . ");</script>"; // выводить отладочную инфу
}