Sidebar

[cece74]

I tried, but I can not. I might add the code please? So I learn. Thank You.

cece74-Sidebar.zip

[Sheer]

cece7420150405174209.rar

[c61]

Код: Выделить всё

$config->set('sidebar_text', htmlspecialchars_decode($request->variable('sidebar_text', '', true)));

Do not do it.

You have to inject a dependency into the listener (or whatever extension script) first: - @config_text as an argument in services.yml, \phpbb\config\db_text $config_text as a constructor argument in .php file, $this->config_text = $config_text; in the constructor body, and then do something like....

rxu, перейду-ка я на русский язык - в соответствии с правилами конференции, а также для того, чтобы нижесказанное было понятно владеющим только русским языком. Затем можете перевести нашему итальянскому другу. Итак:
1) не стоит сохранять в базе данных "чистый" html-код с непреобразованными спецсимволами (там ведь есть кавычки и прочие радости);
2) htmlspecialchars_decode следует использовать в момент присвоения html-кода переменной шаблона, которая будет использована для подстановки;
3) как я уже ранее говорил, в phpBB имеется неявный баг, связанный с set_array: когда Вы решите сохранить длинный текст в config_text, то обнаружите, что некоторые базы данных такого не переваривают, если текст не был изменён (баг с INSERT INTO); записывать следует только изменившиеся значения;
4) в ACP надо использовать 'validate' => 'text', 'type' => 'textarea с заданными размерчиками и класс no-auto-resize для обсуждаемого текстового поля.
Если не хотите собрать все "грабли", см. в качестве примера расширения "Empennage" и "Extender".

[rxu]

c61, я вообще не имел желания вдаваться в подробности кода, а лишь привел пример (do something like) работы с объектом config_text на основе уже существующего.
Остальное - дело непосредственного разработчика.

[cece74]

I'll make it. Thanks friends!

[Sheer]

я вообще не имел желания вдаваться в подробности кода

Я в общем-то тоже...

[cece74]

How you think? Tips?

cece74-Sidebar 0.0.4.zip

[rxu]

cece74, looks good, but as c61 has stated above (in Russian), probably there's no need in htmlspecialchars_decode() to store the values in config_text table, but just to display output.
When you store data via set_array(), the strings will be escaped by that method anyway.

[cece74]

But it works. No it does not take more than 255 characters.

[Sheer]

cece74, It was about the dangers of storing code in a database
listener.php
replace

Код: Выделить всё

            'SIDEBAR_TEXT'        => $sidebar_data['sidebar_text'], 

with

Код: Выделить всё

            'SIDEBAR_TEXT'        => htmlspecialchars_decode($sidebar_data['sidebar_text']), 

and in sidebar_module.php replace

Код: Выделить всё

            $config_text->set('sidebar_text', htmlspecialchars_decode($request->variable('sidebar_text', '', true))); 

with

Код: Выделить всё

            $config_text->set('sidebar_text', $request->variable('sidebar_text', '', true)); 

[cece74]

cece74-Sidebar 0.0.5.zip

I did it! If you have suggestions on how to improve. Tell me.
I can not understand how the upgrade version.

[Sheer]

cece7420150407234212.rar

[cece74]

Code shorter, I do not understand what you did.

[Sheer]

Some imrovement. Forgot change

Код: Выделить всё

        $sidebar_data = $config_text->get_array(array(
            'sidebar_text',
        ));

        $sidebar2_data = $config_text->get_array(array(
            'sidebar_urlpage',
        )); 

to

Код: Выделить всё

        $sidebar_data = $config_text->get_array(array(
            'sidebar_text',
            'sidebar_urlpage',
        )); 

and replace

Код: Выделить всё

            'SIDEBAR_URLPAGE'        => $sidebar2_data['sidebar_urlpage'], 

with

Код: Выделить всё

            'SIDEBAR_URLPAGE'        => $sidebar_data['sidebar_urlpage'], 

cece7420150408015048.rar

[cece74]

Thanks to our friends phpbbguru.net: https://www.phpbb.com/community/viewtop ... &t=2306856