Lightning と WordPress コーディング規約 と バグ

翻訳者向けのコメントがない

→ _x() 関数の第2引数などの事ですか？

翻訳者向けのコメントがないで私が子テーマを作成する際にで遭遇したのは

/**
 *  Category and Tax Data
 */
$args  = array(
	'template' => __( '<dl><dt>%s</dt><dd>%l</dd></dl>', 'lightning' ),
	'term_template' => '<a href="%1$s">%2$s</a>',
);

の部分のみです。

エスケープ漏れ

→ エスケープ漏れのないようにとは思っていますが、エスケープ漏れに気付きましたらGitHub（Lighhhtningのリポジトリあるいはhttps://github.com/vektor-inc/vektor-wp-libraries ）などでプルリクエストいただけると助かります。

Githubを本格的に使ったことがないのと（興味はある）、
チームの一員でもない一ユーザーが勝手にブランチを作ってプルリクエストをするのは
なんとなくはばかられたので今回はここで書かせていただきます。

とりあえず、気になったのはindex.phpの現状が下記のようになっている部分で、

<div class="col-md-8 mainSection" id="main" role="main">

	<?php
	/*-------------------------------------------*/
	/*  Archive title
	/*-------------------------------------------*/
	$page_for_posts = lightning_get_page_for_posts();
	// Use post top page（ Archive title wrap to div ）
	if ( $page_for_posts['post_top_use'] || get_post_type() != 'post' ) {
		if ( is_year() || is_month() || is_day() || is_tag() || is_author() || is_tax() || is_category() ) {
			$archiveTitle      = get_the_archive_title();
			$archiveTitle_html = '<header class="archive-header"><h1>' . $archiveTitle . '</h1></header>';
			echo apply_filters( 'lightning_mainSection_archiveTitle', $archiveTitle_html );
		}
	}

	/*-------------------------------------------*/
	/*  Archive description
	/*-------------------------------------------*/
	if ( is_category() || is_tax() || is_tag() ) {
		$category_description = term_description();
		$page                 = get_query_var( 'paged', 0 );
		if ( ! empty( $category_description ) && $page == 0 ) {
			$archiveDescription_html = '<div class="archive-meta">' . $category_description . '</div>';
			echo apply_filters( 'lightning_mainSection_archiveDescription', $archiveDescription_html );
		}
	}

	$postType = lightning_get_post_type();

	do_action( 'lightning_loop_before' );
?>

<div class="postList">

上記を下記のように変更してみると良いかもしれません。

<div class="col-md-8 mainSection" id="main" role="main">

	<?php
	// Archive title.
	$page_for_posts = lightning_get_page_for_posts();
	// Use post top page（ Archive title wrap to div ）.
	if ( $page_for_posts['post_top_use'] || get_post_type() !== 'post' ) {
		if ( is_year() || is_month() || is_day() || is_tag() || is_author() || is_tax() || is_category() ) {
			$archiveTitle      = get_the_archive_title();
			$archiveTitle_html = '<header class="archive-header"><h1>' . $archiveTitle . '</h1></header>';
		}
	}
	echo wp_kses_post( apply_filters( 'lightning_mainSection_archiveTitle', $archiveTitle_html ) );

	// Archive description.
	if ( is_category() || is_tax() || is_tag() ) {
		$category_description = term_description();
		$page                 = get_query_var( 'paged', 0 );
		if ( ! empty( $category_description ) && $page_number == 0 ) {
			$archiveDescription_html = '<div class="archive-meta">' . $category_description . '</div>';
		}
	}
	echo wp_kses_post( apply_filters( 'lightning_mainSection_archiveDescription', $archiveDescription_html ) );

	$postType = lightning_get_post_type();

	do_action( 'lightning_loop_before' );
	?>

	<div class="postList">

<?php if ( have_posts() ) : ?>

変更点は以下のとおりです。

• apply_filters()の外側にwp_kses_post()をつけることでエスケープ
• apply_filters()の部分をif文の外側に出すことで$archiveTitle_htmlや$archiveDescription_htmlを外から上書き可能に

特に、２番目はLightning Skin Variety において
アーカイブタイトル・説明の扱いをおかしくせざるをえなかった原因を修正できます。