I have around 250+ blog posts and currently in my custom theme I've implemented the functionality of showing the blogs with pagination option. But in all pages it is showing the same posts.
<?php
$data = getData();
if(!empty($_REQUEST['posts'])){
$args = array(
'post_type' => 'post',
'post_status' => 'publish',
'posts_per_page' => 15,
'category_name' => $_REQUEST['posts'],
);
}else{
$args = array(
'post_type' => 'post',
'post_status' => 'publish',
'posts_per_page' => 15,
);
}
$loop = new WP_Query($args);
?>
<style>
</style>
<div class="blog_main" data-aos="fade-up" data-aos-duration="500">
<div class="container">
<div class="filter-items">
<div class="items">
<div class="item <?php if(empty($_REQUEST['posts'])){echo 'active';} ?>">
<a href="<?php echo site_url('/blog')?>">Alla</a>
</div>
<div class="item <?php if($_REQUEST['posts'] == 'Blogg'){echo 'active';} ?>">
<a href="<?php echo site_url('/blog?posts=Blogg')?>">Blogg</a>
</div>
<div class="item <?php if($_REQUEST['posts'] == 'Nyheter'){echo 'active';} ?>">
<a href="<?php echo site_url('/blog?posts=Nyheter')?>">Nyheter</a>
</div>
</div>
</div>
<?php if ($loop->have_posts()) { ?>
<div class="blog-container">
<?php while ($loop->have_posts()): $loop->the_post(); ?>
<div class="single-blog">
<a class="feature-image" href="<?php the_permalink()?>">
<?php echo get_the_post_thumbnail($post->ID, 'full') ? get_the_post_thumbnail($post->ID, 'full') : '<img loading="lazy" width="582" height="280" src="'.site_url( 'wp-content/uploads/2023/04/anse_lazio_stranden.jpeg' ).'" >' ?>
</a>
<div class="tags">
<span class="entry-date"><?php echo get_the_date(); ?></span>
<?php
$categories = get_the_category();
if ( ! empty( $categories ) ) {
echo '<a href="' . esc_url( get_category_link( $categories[0]->term_id ) ) . '">' . esc_html( $categories[0]->name ) . '</a>';
}
?>
<!-- <a href="">Nyheter</a> -->
</div>
<div class="title-section">
<a href="<?php the_permalink()?>">
<h2><?php the_title()?></h2>
<svg width="24" height="24" viewBox="0 0 24 24" fill="none" xmlns="http://www.w3.org/2000/svg">
<path d="M7 17L17 7" stroke="#1D2635" stroke-width="2" stroke-linecap="round"
stroke-linejoin="round" />
<path d="M7 7H17V17" stroke="#1D2635" stroke-width="2" stroke-linecap="round"
stroke-linejoin="round" />
</svg>
</a>
</div>
</div>
<?php endwhile;?>
</div>
<div class="pagination">
<a href="<?php
if(max( 1, get_query_var( 'paged' ) ) == 1) {
echo "javascript:void(0)";
$active = "";
}else
{
echo site_url('/blog/page/'.(get_query_var( 'paged' )-1).'?posts='.get_query_var( 'posts' ));
$active = "active";
}
?>" class="preview <?php echo $active;?>">Tidigare</a>
<?php
echo paginate_links( array(
'base' => str_replace( 999999999, '%#%', esc_url( get_pagenum_link( 999999999 ) ) ),
'total' => $loop->max_num_pages,
'current' => max( 1, get_query_var( 'paged' ) ),
'format' => '?paged=%#%',
'show_all' => false,
'type' => 'list',
'end_size' => 2,
'mid_size' => 1,
'prev_next' => false,
'prev_text' => sprintf( '<i></i> %1$s', __( 'Tidigare', 'text-domain' ) ),
'next_text' => sprintf( '%1$s <i></i>', __( 'Nästa', 'text-domain' ) ),
'add_args' => false,
'add_fragment' => '',
) );
?>
<a href="<?php
if(max( 1, get_query_var( 'paged' ) ) > 1) {
echo "javascript:void(0)";
$active = "";
}else
{
echo site_url('/blog/page/'.(get_query_var( 'paged' )+1).'?posts='.get_query_var( 'posts' ));
$active = "active";
}
?>" <?php echo max( 1, get_query_var( 'paged' ) ); ?> class="next <?php echo $active;?>">Nästa</a>
</div>
<?php } ?>
</div>
</div>
How do I fix this so that each page shows items as per the page number and number of items.
WP_Query,pre_get_postswould be much faster and efficient, it's even possible this could be done entirely by appending?category_name=xyzto the URL instead of using?posts=, which is something WP already supports out of the boxthe_posts_pagination(). It didn't work @TomJNowellnew WP_Query, and the pagination functions are totally unaware of this. In addition, your new query has no concept of pagination, it's been told to use 15 posts per page, but at no point is it ever told which page it's on, thepagedparameter is completely missing. Unfortunately the secondary query method doubles the cost of the page.