Стандарты кодирования сэндвичей

8

Недавно я обнаружил код для wp-lunch.phpфайла, предназначенного для шаблона WordPress, как бы это выглядело, если бы соблюдали надлежащие стандарты кодирования WordPress?

сор-lunch.php

<?php if ( current_user_can('has_sandwich') ): ?>

    <?php get_sandwich_header(); ?>

        <?php while( has_filling() ): the_filling(); ?>

            <?php get_sandwich_part( 'thick_layer',
    get_filling() ); ?>

        <?php endwhile; ?>

    <?php get_sandwich_footer(); ?>

<?php endif; ?>

coding-standards

— Том Дж Новелл
источник

Отличный вопрос, пусть начнутся игры!

— Адам

Я думаю, что идеальный вопрос, чтобы спросить здесь, как должен the_filling()выглядеть

— Питер Гусен

Что делают все эти разработчики WP по выходным? ;)

— Николай

5

Что произойдет, если пользователь не сможет съесть бутерброд? WSOF?

Если бы я хотел следовать шаблонам тем по умолчанию, я бы пошел на

// eat-sandwich.php (as @Rarst said avoid wp-lunch.php as it's not part of WP core)

get_header( 'sandwich' );

if ( current_user_can( 'eat_sandwich' ) ) {

  get_template_part( 'eat-sandwich', 'content' );

} else { // user can't eat sandwich. An apple?

  $alternative = apply_filters( 'alternative_to_sandwich', 'apple' );

  if ( 'sandwich' == $alternative ) {
     // No sandwich allowed!
     $alternative = 'apple';
  }

  get_template_part( "eat-$alternative", 'content' );

}

get_footer( 'sandwich' );

А потом

// eat-sandwich-content.php

$fillings = get_fillings_query(); // in functions.php

if ( $fillings->have_posts() ) : while ( $fillings->have_posts() ) :

   get_template_part( 'filling', get_filling_type() );

endwhile;

wp_reset_postdata();

else :

  _e( 'Sorry, no fillings found. Eating an apple may help to stop hunger.', 'txtdomain');

endif;

— Gmazzap
источник

Этот запрос сломался бы, если бы не было хлеба, так как хлеб не считается начинкой и неотъемлемой частью бутерброда. Я рекомендую добавить get_ingredients();вместо get_fillings_query();которого хлеб и начинка являются частью. Также начинка должна иметь интерфейс JSON API на передней панели;)

— Wyck

5

<!-- file shouldn't be named wp-lunch.php as it's not part of WP core -->

<?php if ( current_user_can( 'eat_sandwich' ) ): // more specific verb makes more sense to me ?>

    <?php get_header( 'sandwich' ); // native function accepts type argument ?>

    <?php while ( have_fillings() ): the_filling(); // maybe native API, but feels acceptable wrapper for semantics ?>

        <?php get_template_part( 'filling', get_filling_type() ); // native API, what would be `thick_layer` base? ?>

    <?php endwhile; wp_reset_postdata(); // reset $post global ?>

    <?php get_footer( 'sandwich' ); // native function accepts type argument ?>

<?php endif; ?>

Интервал с учетом стиля кодирования и т. Д.

Сэндвич-шаблон с кусочком веточки, съеденный на лугу, будет выглядеть примерно так:

{% if ( current_user_can( 'eat_sandwich' ) ) %}

    {% include 'header-sandwich.twig' %}

    {% loop fillings %}

        {% include 'filling-' ~ get_filling_type() ~ '.twig' ignore missing %}

    {% endloop %}

    {% include 'footer-sandwich.twig' %}

{% endif %}

— Rarst
источник

3

Все это <?phpдает мне озноб.

— gmazzap

7

@GM Я думал о введении шаблона Усы, но кто хочет усы в их бутерброд?

— Первый день

4

Если бы я нашел веточку в своем сэндвиче, я бы справился с этим, но не с усами.

— Адам

1

Нет необходимости во всех открывающих и закрывающих разделителях или чистых пробелах, если отступ уже:

<?php
if ( current_user_can( 'has_sandwich' ) ) {
    get_sandwich_header();
    while ( has_filling() ) {
        the_filling();
        get_sandwich_part( 'thick_layer', get_filling() );
    }
    get_sandwich_footer();
}

Вероятно, после этого должны быть сброшены данные заполнения ...

— GaryJ
источник