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

ок. понятно.

hcs, (Please log in or register to see this URL)

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

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

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

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

create_function() предлагают сделать устаревшей с PHP 7.2 (Please log in or register to see this URL)

hcs, а почему этот реквест (Please log in or register to see this URL) не принимается?

Судя по (Please log in or register to see this URL)

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

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

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

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

есть либа для фильтрования (Please log in or register to see this URL)

идея была - 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, разве он его давным давно не забросил? Это же он (Please log in or register to see this URL) ?

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

(Please log in or register to see this URL)

строку

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

?

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

(Please log in or register to see this URL)

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

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

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

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

(Please log in or register to see this URL)

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