1

Есть нюанс, может подскажите если есть возможность, как правильно сделать? А то уже голову сломал...

if ( ! is_user_logged_in() ) { 
if ( ! is_single() ) { 
echo '<div class="video"> 
    <iframe src="player.vimeo.com/video/<?php echo 
    carbon_get_post_meta($current_episode->ID, 'crb_vimeo_video_id'); ?>" 
    width="100%" height="100%" frameborder="0" webkitallowfullscreen="" 
    mozallowfullscreen="" allowfullscreen=""> 
    </iframe> 
    </div>'; 
 } 
}
7
  • что за код, откуда?
    – InDevX
    5 апр 2019 в 13:27
  • @InDevX а это имеет значение? вопрос то не об этом ) 5 апр 2019 в 13:29
  • А о чем вопрос? echo '...<iframe src="player.vimeo.com/video/' . carbon_get_post_meta($current_episode->ID, 'crb_vimeo_video_id') . '" width="100%" height="100%" frameborder="0"...'; - Об этом?
    – Manitikyl
    5 апр 2019 в 13:31
  • @Manitikyl Да, я не знаю как в фрейм html вставить в таком случае php код 5 апр 2019 в 13:33
  • 1
    @ТарасЛейтар я вам написал как правильно должно выглядеть, присмотритесь
    – Manitikyl
    5 апр 2019 в 13:34

3 ответа 3

2

Вот вам третий вариант... Тоже рабочий :) Я его предлагаю, потому что, он ближе к рекомендованному применению кода.

PHP очень удобен тем, что такую задачу как Ваша можно решить сразу тремя способами. Именно благодаря, такому "дружелюбному" подходу к разработчику PHP И стал одним из самых востребованных языков.

if ( ! is_user_logged_in() ) {
    if ( ! is_single() ) {
        $varembed = carbon_get_post_meta($current_episode->ID, 'crb_vimeo_video_id');

        echo "<div class='video'> 
                <iframe src='player.vimeo.com/video/{$varembed}' width='100%' height='100%' frameborder='0' webkitallowfullscreen='' mozallowfullscreen='' allowfullscreen=''> 
                </iframe> 
              </div>";
    }
}
7
  • Что такое "рекомендованное применение кода"?)) 6 апр 2019 в 5:09
  • И код не рабочий. Вы когда пишете что-то, тестируйте. 6 апр 2019 в 5:11
  • Вежливым кодом принято считать код без замеса html и php... Знакомы со стандартом PSR-1 PSR-2? 6 апр 2019 в 20:27
  • Код поправил. Виноват. Забыл про экранирование... Не использую сам уже давно такие конструкции. Вообще лучше примитивный MVC чем такое смешение. И дело даже не в стандартах каких либо, а просто в множестве возможностей которых вы себя лишите, используя такой подход. 6 апр 2019 в 20:38
  • 1
    Кроме того, раз уж мы заговорили о стандартах, и находимся под меткой WordPress, то надо следовать WPCS. Ни в одном из представленных здесь ответов, включая мой, код не соответствует WPCS, потому что вывод не экранирован. Но в вашем случае будет наиболее затруднительно применить функцию типа esc_. 7 апр 2019 в 7:24
0
if ( ! is_user_logged_in() ) { 
    if ( ! is_single() ) { 
        echo '<div class="video"> 
            <iframe src="player.vimeo.com/video/'.carbon_get_post_meta($current_episode->ID, "crb_vimeo_video_id").'" 
            width="100%" height="100%" frameborder="0" webkitallowfullscreen="" 
            mozallowfullscreen="" allowfullscreen=""> 
            </iframe> 
            </div>'; 
     } 
}
5
  • И так тоже можно. Но в моем ответе читаемость кода лучше. 5 апр 2019 в 13:40
  • @KAGG Design В обоих случаях 404 вылазит 5 апр 2019 в 13:41
  • 404 - это не к написанию кода вопрос, а к тому, что у вас в метаполе carbon. Посмотрите в консоли, увидите путь. Но к вашему вопросу это никак не относится. 5 апр 2019 в 13:43
  • @KAGG Design Спасибо! Разобрался, а как теперь сделать чтобы только на одну страницу вывод работал, или по Titile или ID поста, а то if ( ! is_single('Episode 1') ) { не работает 5 апр 2019 в 14:44
  • 1
    is_single('заголовок') работает только для постов. Если у вас страница, надо использовать is_page('заголовок') 5 апр 2019 в 15:06
0

Сделайте так:

<?php
if ( ! is_user_logged_in() ) {
    if ( ! is_single() ) {
        ?>
        <div class="video">
            <iframe
                    src="player.vimeo.com/video/<?php echo carbon_get_post_meta( $current_episode->ID, 'crb_vimeo_video_id' ); ?>"
                    width="100%" height="100%" frameborder="0" webkitallowfullscreen=""
                    mozallowfullscreen="" allowfullscreen="">
            </iframe>
        </div>
        <?php
    }
}

Ваш ответ

By clicking “Отправить ответ”, you agree to our terms of service and acknowledge you have read our privacy policy.

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.