Где ошибка в do_action?

Question

[Анатолий]

Версия Wordpress: 5.3.2
Версия PHP: 7.4.2

Подскажите, что не так:
Функция:

add_action( 'fff', 'do_fff' );
function do_fff(){echo '123';}

И хук:

<?php echo '<div id="fff">'. do_action("fff") .'</div>'; ?>

Если вызываю так, то работает:
<?php do_action("fff"); ?>

Что не так при вставке в тег div, пробовал echo и return, все равно не вставляется.

Comments

[Модератор]

Укажи версию PHP в вопросе, пожалуйста

[Анатолий]

Модератор, fixed

[Владимир Брумер]

echo '<div id="fff">';do_action("fff");echo'</div>';

у вас в функции echo... т.е. вы выводите грубо говоря echo в нутри echo:
echo '<div id="fff">'. echo '123' . '</div>';

[Анатолий]

Владимир Брумер, в вопросе написано про return, echo я просто тестировал.

[Владимир Брумер]

Анатолий, ну дак по поводу echo я вроди по-русски Вам написал - Вы выводите echo внутри echo (глядя на код выше в вопросе).

function one() { echo 1; }
function two() { echo 2; }
function three() { echo 3; }

add_action( 'foo', 'one' );
add_action( 'foo', 'two' );
add_action( 'foo', 'three' );
echo '<div id="fff">';
do_action( 'foo' );
echo '</div>';

А по поводу return Вам Анатолий написал

Если сделаете через apply_filters все будет корректно

...

function plus_foo($value){
    return $value;
}
add_filter( 'foo', 'plus_foo' );

echo '<div id="fff">'.apply_filters('plus_foo',123).'</div>';

у Вас он возвращает null

[Владимир Брумер]

Анатолий,

в вопросе написано про return

не вижу.

Где ошибка в do_action?

Что не так при вставке в тег div, пробовал echo и return, все равно не вставляется.

Answer 1

[godsplane]

Попробуйте так
<div id="fff"><? do_action("fff") ?></div>

Comments

[Анатолий]

Да, так работает, но запись в таком виде, я конечно использовать не буду.
<?php echo '';?><?php do_action('fff'); ?><?php echo''; ?>

Answer 2

[Пычев Анатолий]

Код синтаксически верен.
проверил, на wp 5.3.2 , php (7.0.33, 7.2.25, 7.3.12, 7.4.0). Работает на ура.

С другой стороны, семантически это не верно. Правильнее будет вызвать apply_filters(...).

Comments

[Анатолий]

Эффект тот же.
Если вставляю через return:
<?php echo '<div>'. do_action("fff") .'</div>'; ?> то получаю <div></div>

Если вставляю через echo, получаю: 123<div></div>

[Пычев Анатолий]

Так все верно, так и должно быть судя по Вашему коду.
Я сначала понял, что у Вас вообще ничего не выводится.

Вспоминайте, как работает функция echo и стек вызовов

Если сделаете через apply_filters все будет корректно.

[Анатолий]

Пычев Анатолий, я вот думаю, а зачем тут использовать apply_filters? Про подмену вроде речь не идет.
Можно тогда просто сделать:
'<div>'.test().'</div'

и дальше вставить, что-то как:
function test(){ return 'Welcome';}

[Анатолий]

Решил по другому, тему закрываю.

[Пычев Анатолий]

Ну просто Вы изначально использовали do_action(). Это подразумевает, что Вы хотите уведомлять сторонние плагины о выполнении Вами некоего действия. Поэтому я и продложил использовать apply_filter() что является семантически более верным в блоке вывода информации.

к тому же через apply_filters() Ваш код выдал бы корректную верстку.