Ask your WordPress questions! Pay money and get answers fast!

Custom Taxonomies with Pagination getting 404, Page Not Found WordPress

SOLVED

For several custom taxonomies I am using taxonomy.php to show a list of posts. It works fine without pagination. And I can show the first page of the pagination but page 2 gets at 404, page not found.
In other words http://www.haringkids.com/lesson_plans/?curriculum=art works but http://www.haringkids.com/lesson_plans/?curriculum=art&paged=2 goes to "page not found". (If you go to these pages, you will see some var_dumps, just ignore them.)

Some of the code related to paging comes from:
http://wpquestions.com/question/show/id/701

It seems to me the problem is not exactly with the pagination but once "paged=2" is added to the url, WP is not using taxonomy.php

Anyway. I am stymied. Any help would be much appreciated. Thanks

Here is my code:

<?php /** * The template for displaying Category Archive pages. * * @package WordPress * @subpackage Twenty_Ten * @since Twenty Ten 1.0 */ ?> <?php get_header(); $term = get_term_by( 'slug', get_query_var( 'term' ), get_query_var( 'taxonomy' ) ); $taxonomy = get_taxonomy($term->taxonomy); ?> <div id="container"> <div id="content" role="main"> <h1 class="page-title"><?php printf( __( '%s', 'twentyten' ), '<a href="/lesson_plans/' . $taxonomy->query_var . '">' . $taxonomy->labels->name . ':</a> ' . $term->name); ?></h1> <?php $category_description = category_description(); if ( ! empty( $category_description ) ) echo '<div class="archive-meta">' . $category_description . '</div>'; ?> <?php global $paged, $wp_query, $wp; $args = wp_parse_args($wp->matched_query); if ( !empty ( $args['paged'] ) && 0 == $paged ) { $wp_query->set('paged', $args['paged']); $paged = $args['paged']; } $temp = $wp_query; $wp_query= null; $wp_query = new WP_Query(); $args_2 = array( 'is_paged => true', 'paged' => $paged, 'posts_per_page' => 12, $taxonomy->query_var => $term->slug ); $wp_query->query($args_2); ?> <?php while ( $wp_query->have_posts() ) : $wp_query->the_post(); ?> <div id="post-<?php the_ID(); ?>" <?php post_class() ?>> <h2 class="toc_title"><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></h2> <div class="entry-summary"> <?php if(has_post_thumbnail()): ?> <a href="<?php the_permalink(); ?>"> <?php the_post_thumbnail('small-thumbnail', array('class' => 'alignleft', 'title' => trim(strip_tags($post->post_title)), 'alt' => trim(strip_tags($post->post_title)))); ?> </a> <?php endif; ?> <?php if(get_post_meta($post->ID, "_dwdescription_value", $single = true) != "") : ?> <p><?php echo get_post_meta($post->ID, "_dwdescription_value", $single = true);?></p> <?php endif; ?> </div> <div class="entry-utility"> </div> </div> <?php endwhile; ?> <div class="navigation"> <div class="alignleft"><?php previous_posts_link('« Previous') ?></div> <div class="alignright"><?php next_posts_link('More »') ?></div> </div> <?php $wp_query = null; $wp_query = $temp; ?> </div> </div> <?php get_sidebar(); ?> <?php get_footer(); ?>

2010-08-05 by Daniel Wiener

Answers (5)

2010-08-05

wjm answers:

Daniel,
first we need to debug your request. And that is to understand what rewrite rule is being applied.

1) please save this file in your theme folder (i.e. the same folder as your functions.php is located)
http://wordpress.pastebin.com/gvMDP8sN
name it "debug.php"

2) add this line at the end of your "functions.php" file
require( get_stylesheet_directory() . '/debug.php' );

3) reload the page. View its source code and paste here or to pastebin.com what you get around the line with "<!-- DEBUG INFO:"

looking forward to hearing from you.}
-wjm

wjm comments:

do you page wp_pagenavi installed?

wjm comments:

if you havent uploaded the file, use this version of debug.php instead
http://wordpress.pastebin.com/xdE2wiEF

Daniel Wiener comments:

wjm comments:

could you update debug.php to
http://wordpress.pastebin.com/xdE2wiEF
and paste the results to pastebin.com
tahnks,

i am looking into your output

wjm comments:

Well. tracking down your issue.
1st, your link should be
http://www.haringkids.com/lesson_plans/curriculum/art/page/2
and not
http://www.haringkids.com/lesson_plans/?curriculum=art&paged=2

I see the debug results for
http://www.haringkids.com/lesson_plans/curriculum/art/page/2
 <!-- matched_rule: curriculum/([^/]+)/page/?([0-9]{1,})/?$ matched_query: index.php?curriculum=$matches[1]&paged=$matches[2] matched_query (evaluated): curriculum=art&paged=2

there custom post type or taxonomy rewrite rule is matched,
but for some reason is it not being matched.

also, i think this coudl be the isseu
i see that your posts_per_page is set to 999
can how many posts do you have there? set it to 2 to test, this may all be the issue.

if this doesnt work, please update your debug.php
thanks

wjm comments:

to update the posts_per_page value you should go to /wp-admin/options-reading.php

and update
"Blog pages show at most"...

wjm comments:

just to let you know that your issue so far has nothing to do with pagination but with rewrite rules.
if you see the template tha is being called. it is 404.php
so you request does not even call taxonomy.php

Daniel Wiener comments:

I have pasted the most recent debug info into
http://wordpress.pastebin.com/1JxCTT4F

I think I may have figured out the problem. In order to get a page that lists the terms for each taxonomy I made a custom page template, then created Pages with the same name as the taxonomy.
e.g. http://www.haringkids.com/lesson_plans/subject/
So that may be causing the confusion. I am going to test it now. And explain more if that does anything.

wjm comments:

i have found the issue, it is the posts_per_page

reading the output for http://www.haringkids.com/lesson_plans/curriculum/art/page/2
you are requiring 999 posts per page. and page number 2 doe not exist
your request ends with
DESC LIMIT 999, 999

which obviusly will give you only one result in case you have 999 or more posts.

you aresetting posts per page to 12 but that is in taxonomy.php
and that is not even called

if you want to change that youneed to modify the post_per_page value at init time.

i will rewrite the code that will let you do taht.

wjm comments:

Ok, the soultion is to either update the options for posts_per_page win wp-admin/
or add this filter.

add this code to functions.php

add_action( 'init', 'childtheme_modify_posts_per_page', 0); function childtheme_modify_posts_per_page() { add_filter( 'option_posts_per_page', 'childtheme_option_posts_per_page' ); } function childtheme_option_posts_per_page( $value ) { return 12; }

you make create a conditional inside childtheme_modify_posts_per_page() to change the default posts per page for a specific situation, lets say for taxonomy curriculum and term art for example

let me know if it works, as i am sure it will.

Daniel Wiener comments:

WJM,
Thanks. No need to write code. I just set posts per page to 12 on the WP dashboard. And now it all works.

So. I am looking looking looking for the complicated programming solution and all it needs is a simple change the admin. Sometimes the "smart" way is the stupid way.

Thanks.

Daniel

Daniel Wiener comments:

wjm comments:

good!
so in case you need to change the posts per page for a specific page, that needs to be done at init time.
actually at <strong>init time priority 0</strong>

2010-08-05

Utkarsh Kukreti answers:

Wouldn't

$args_2 = array( 'is_paged => true',

be
$args_2 = array( 'is_paged' => true,

?

Daniel Wiener comments:

Yes. You are right. I changed my typo. But still have the same problem. It would have been soooooooo nice if that was the solution.

2010-08-05

Lew Ayotte answers:

I would do it this way:

<?php /** * The template for displaying Category Archive pages. * * @package WordPress * @subpackage Twenty_Ten * @since Twenty Ten 1.0 */ ?> <?php get_header(); $term = get_term_by( 'slug', get_query_var( 'term' ), get_query_var( 'taxonomy' ) ); $taxonomy = get_taxonomy($term->taxonomy); ?> <div id="container"> <div id="content" role="main"> <h1 class="page-title"><?php printf( __( '%s', 'twentyten' ), '<a href="/lesson_plans/' . $taxonomy->query_var . '">' . $taxonomy->labels->name . ':</a> ' . $term->name); ?></h1> <?php $category_description = category_description(); if ( ! empty( $category_description ) ) echo '<div class="archive-meta">' . $category_description . '</div>'; ?> <?php $paged = (get_query_var('paged')) ? get_query_var('paged') : 1; $args_2 = array( 'is_paged => true', 'paged' => $paged, 'posts_per_page' => 12, $taxonomy->query_var => $term->slug ); $query = new WP_Query($args_2); ?> <?php while ( $query->have_posts() ) : $query->the_post(); ?> <div id="post-<?php the_ID(); ?>" <?php post_class() ?>> <h2 class="toc_title"><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></h2> <div class="entry-summary"> <?php if(has_post_thumbnail()): ?> <a href="<?php the_permalink(); ?>"> <?php the_post_thumbnail('small-thumbnail', array('class' => 'alignleft', 'title' => trim(strip_tags($post->post_title)), 'alt' => trim(strip_tags($post->post_title)))); ?> </a> <?php endif; ?> <?php if(get_post_meta($post->ID, "_dwdescription_value", $single = true) != "") : ?> <p><?php echo get_post_meta($post->ID, "_dwdescription_value", $single = true);?></p> <?php endif; ?> </div> <div class="entry-utility"> </div> </div> <?php endwhile; ?> <div class="navigation"> <div class="alignleft"><?php previous_posts_link('« Previous') ?></div> <div class="alignright"><?php next_posts_link('More »') ?></div> </div> <?php $wp_query = null; $wp_query = $temp; ?> </div> </div> <?php get_sidebar(); ?> <?php get_footer(); ?>

Lew Ayotte comments:

note:
I'm not overwriting $wp_query, because the whole point of WP_Query() is so you can create a custom query without killing the WP default query.

I also changed how you get your paged variable.

Daniel Wiener comments:

thanks. I had tried this. Your solution makes sense but it does not work.

And the results are:
There is no navigation at the bottom of the page for the next/previous link.

And if I add page=2 manually I still get a "page not found".

I did a var_dump for $paged to see what is happening and it is 1. Using my original code it was 0.

I am not sure the problem is with the "paging" but that the url is not "finding" taxonomy.php when "page=2" is added.

Lew Ayotte comments:

hrm, the first page should be page 1, not page 0...

http://www.haringkids.com/lesson_plans/curriculum/art/page/1

http://www.haringkids.com/lesson_plans/curriculum/art/page/2

also, I accidentally copied in the error w/ 'is_paged => true', - as Utkarsh pointed out.

2010-08-05

Michael Fields answers:

I installed your template locally and it worked perfectly for me. How many posts have been associated with the "art" term of the "curriculum" taxonomy? You may be experiencing a 404 because there are not enough posts to create a second page. When I visit http://www.haringkids.com/lesson_plans/curriculum/art no navigation is generated.

Daniel Wiener comments:

Yes. The template works for the first 12 entries but not for the subsequent "paged" entries. I have at least 50 entries for the term "art".

There is no navigation now because I tried Lew's solution (see above). I am going to re-upload my original code. Now.

Thanks

Michael Fields comments:

Oh, That makes sense, I thought they were there a few minutes ago :) I really don't know what to tell you, your template works perfectly. There must be conflicting code elsewhere in your site. Do you use $query->set() a lot throughout functions.php. If so, this method can create very confusing results. My advice would be to check for this as I can really find no wrong-doing in your template. Secondly, have you tried using a good, ole' fashioned, WordPress loop? Or is there a specific need to define a custom query. WordPress can handle custom taxonomy queries rather well in my experience.
Best wishes,
-Mike

2010-08-05

Guillermo Sornoza answers:

HI. In your case, i let you WP handles everything. I can't see why you have to write your own query.

In you case, i should remove the code:
<?php global $paged, $wp_query, $wp; $args = wp_parse_args($wp->matched_query); if ( !empty ( $args['paged'] ) && 0 == $paged ) { $wp_query->set('paged', $args['paged']); $paged = $args['paged']; } $temp = $wp_query; $wp_query= null; $wp_query = new WP_Query(); $args_2 = array( 'is_paged => true', 'paged' => $paged, 'posts_per_page' => 12, $taxonomy->query_var => $term->slug ); $wp_query->query($args_2); ?>

and write the loop without the $wp_query variable.

Let me know if i this result helpful.