次のコードを読む必要があります。現在のカスタム投稿タイプ(bbp_forum)が表示されているものである場合は、 class 'current'をそれぞれの<li>
タグに割り当てます。しかし、何らかの理由で 'current' (現在のbbp_forum
リンクを強調表示するため)クラスがすべての<li>
タグに表示されています。
<body <?php body_class(); ?>>
<div id="wrapper" class="hfeed">
<div id="header">
<div id="masthead">
<div id="branding" role="banner">
<h1><a href="<?php echo home_url( '/' ); ?>" title="<?php echo esc_attr( get_bloginfo( 'name', 'display' ) ); ?>" rel="home"><?php bloginfo( 'name' ); ?></a></h1>
</div><!-- #branding -->
<div id="access" role="navigation">
<?php wp_nav_menu( array( 'container_class' => 'menu-header', 'theme_location' => 'primary' ) ); ?>
</div><!-- #access -->
</div><!-- #masthead -->
<ul id="forums">
<?php global $post; $cat_posts = get_posts('post_type=bbp_forum');
foreach($cat_posts as $post) : ?>
<li <?php if($post->ID == get_the_ID()){ ?>class="current" <?php } ?>>
<a href="<?php the_permalink(); ?>" title="<?php printf( esc_attr__( 'Permalink to %s', 'twentyten' ), the_title_attribute( 'echo=0' ) ); ?>" rel="bookmark"><?php the_title(); ?></a>
</li>
<?php endforeach; ?>
</ul><!-- #access -->
</div><!-- #header -->
<div id="main">
助言がありますか?
式は常に真になります。 get_the_ID()
を見てください。
function get_the_ID() {
global $post;
return $post->ID;
}
だからあなたのコードは効果的に実行されています。
if ( $post->ID == $post->ID ) // always true!
代わりに、メインの投稿のIDを変数にキャッシュしてから、代わりにそれを比較してください。
<?php
global $post;
/**
* @var int Current post ID.
*/
$the_post_ID = $post->ID;
/**
* @var array All posts for bbp_forum.
*/
$cat_posts = get_posts('post_type=bbp_forum');
?>
<?php foreach ( $cat_posts as $post ) : ?>
<li<?php if ( $post->ID == $the_post_ID ) echo ' class="current"'; ?>>
<a href="<?php the_permalink(); ?>" title="<?php printf( esc_attr__( 'Permalink to %s', 'twentyten' ), the_title_attribute( 'echo=0' ) ); ?>" rel="bookmark"><?php the_title(); ?></a>
</li>
<?php endforeach; ?>