Почему не работает? Как правильно вставить php в html?

Question

[Василий Петров]

Имеется код:

$rt = "<li class='attachment-".$attach_id."'>"
            . rcl_button_fast_delete_post($attach_id)
            . "<label>"
            . rcl_get_insert_image($attach_id,$mime[0]);
            if($mime[0]=='image') 
                $rt .=    '<span class="rcl-field-input type-checkbox-input">'
                        . '<span class="rcl-checkbox-box">'
                        . '<input class="thumb-foto" id="thumb-'.$attach_id.'" type="checkbox" '.checked(get_post_thumbnail_id( $editpost ),$attach_id,false).' name="thumb['.$attach_id.']" value="1"     (//ЭТО ЗДЕСЬ>>>>)   '.if (is_page(291)) {echo "checked"}.'>'
                        . '<label for="thumb-'.$attach_id.'" class="block-label"> - '.__('Сделать изображением статьи','wp-recall').'</label>'
                        . '</span>'
                        . '</span>';
            $rt .= "</label>
    </li>";
    return $rt;

if (is_page(291)) {echo "checked"} пытаюсь вставить вот это, но не выходит, не пойму в чем ошибка, как сделать правильно?

Answer 1

[v3shin]

Проблема в несоблюдении синтаксиса php.
if (is_page(291)) {echo "checked"}
Замените тернарным оператором
(is_page(291) ? "checked" : "")

Comments

[Василий Петров]

Это получается: Страница 291 ? Тогда выводим checked : (двоеточие это иначе?) выводим что-то другое. Так ведь?

[v3shin]

Василий Петров: верно.
php.net/manual/ru/language.operators.comparison.ph...

[Василий Петров]

v3shin: А почему checked появляется только после обновления страницы?

[v3shin]

Василий Петров: потому что php, как и любой серверный ЯП, работает до загрузки страницы в браузере. Он не умеет реагировать на действия пользователя.
https://ru.wikipedia.org/wiki/%D0%92%D0%B5%D0%B1-%...
Либо у вас страница закешировалась.

[Василий Петров]

v3shin: Понятно, спасибо. А что делают в этом случае? ("работает до загрузки страницы")

[v3shin]

Василий Петров: используют javascript.

Answer 2

[alex stephen]

Че-то у вас какая-то жесть с кучей ошибок в синтаксисе вообще.
Напишите в 1 переменную шаблон, а ля

$template = "<li class='attachment-%s'>%s<label>..............";

А $rt получите с помощью sprintf. И не забывайте про скобки в if

Comments

[Василий Петров]

Не мой код — из плагина. И это только кусочек

[Василий Петров]

А что такое %s ?

[alex stephen]

Василий Петров: %s - строка.
Грубо говоря, вы указываете в шаблоне, где должны быть строки, после чего подставляете в них данные.
В документации по ссылке выше все подробно описано

[Василий Петров]

alex stephen: Спасибо

Answer 3

[lxfr]

Вы пытаетесь вызвать echo посреди слияния строки посредством точки.
Т.е. вы пишите $x = 'test' . echo 123 . ' test'....
Уберите echo, оно вообще то выводит на экран.

Answer 4

[Анатолий Медведев]

Ваша проблема уже решена но я не могу молчать. Что за &%(*$#&%@()*&% вы делаете?
Шаблоны с помощью конкатенации? Себя пожалейте.

Сделайте хотя бы так:

в отдельный файл, например template1.php, поместите примерно такой код:

<li class='attachment-<?= $attach_id ?>'>
  <?= rcl_button_fast_delete_post($attach_id) ?>
  <label>
    <?= rcl_get_insert_image($attach_id,$mime[0]) ?>
    <?php if($mime[0]=='image'): ?>
      <span class="rcl-field-input type-checkbox-input">
        <span class="rcl-checkbox-box">
          <input class="thumb-foto" 
                 id="thumb- <?= $attach_id ?>" 
                 type="checkbox" <?= checked(get_post_thumbnail_id( $editpost ),$attach_id,false) ?> 
                 name="thumb[<?= $attach_id ?>]" 
                 value="1"
                 <?= is_page(291) ? 'checked' : ''?>
          >
        </span>
      </span>
    <?php endif; ?>
  </label>
</li>

Выглядит по приятнее? но как этим добром воспользоваться?
Очень просто, с помощью require подключаете файл:
$rt = require(__DIR__ . '/template1.php');
всё, в $rt результат. Это просто пример, он ДАЛЕКО НЕ ИДЕАЛЕН и в нем ПОЧТИ ВСЕ ПЛОХО, но это в разы лучше того ужаса что вы делаете.

Comments

[Василий Петров]

Это не я писал... Плагин. А что конкретно здесь не так? Не удобно читать? Синтаксис?

[Анатолий Медведев]

Василий Петров: если коротко то всё, форменный говнокод.

[Вова Дружаев]

Анатолий Медведев - ты не обижайся - но так как ты написал - не меньший говнокод. И в разы нечитаемее

+ ты даешь совет на 4 блока заводить свой шаблон. Это нечто с чем-то...

[Анатолий Медведев]

Владимир Дружаев: Увы, вы не правы. Если вы отроете это в нормальном редакторе это будет в разы лучше выглядеть чем каша ТСа из конкатенаций. И на любой более менее локальный блок должен быть отдельный файл шаблоне. Если вы этого не понимаете то этого говорит лишь о том что вы не имеете опыта работы с реальными проектами крупнее бложика, либо мазохист.

[Вова Дружаев]

Анатолий Медведев я вас услышал. Что спорить то. Про код тс я комментарии не давал. Только про ваш "вариант"