query_posts→pre get posts　その後・・・

お世話になります。
以前query_postsからpre get postsに書き換える質問をさせて頂いたものです。
その後サンプルページを教えていただき自分なりに書いてみました。
記述に何かアドバイスがありましたらご指示下さいませ。
一応表示はできました。
よろしくお願い申し上げます。

/*—————-
category.php
—————*/
<?php if(have_posts()) : while (have_posts($query)) : the_post(); ?>

<p>“><?php the_time(‘Y.n/j’); ?> <?php if(mb_strlen($post->post_title)>20) { $title= mb_substr($post->post_title,0,20) ; echo $title. “…” ;
} else {echo $post->post_title;}?></p>

<?php endwhile; endif; ?>

/*—————-
functions.php
—————*/
<?php
function news_post( $query ) {
if ( is_admin() || ! $query->is_main_query() )
return;

if ( $query->is_category(‘2’) ) {
$query->set( ‘posts_per_page’, 5 );
return;
}
}
add_action( ‘pre_get_posts’, ‘news_post’ );
?>