Instead of using the global variables directly, here’s a way to use the content_pagination filter to add custom HTML to the last content page: /** * Append HTML to the last content page, if it’s paginated */ add_filter( ‘content_pagination’, function( $pages ) { if( count( $pages ) > 1 ) $pages[count($pages)-1] .= ‘<div>MY HTML HERE!</div>’; … Read more
As said in comments, it’s not a good idea editing WP core files. You can easily modify the query that displays your media posts by adding the following code to a plugin or your theme’s functions.php add_filter( ‘pre_get_posts’, ‘_wp_media_pre_get_posts’ ); function _wp_media_pre_get_posts( $wp_query ) { global $pagenow; if( ! in_array( $pagenow, array( ‘upload.php’, ‘admin-ajax.php’ ) … Read more
The function paginate_links() can return “plain”, “list” and “array” (http://codex.wordpress.org/Function_Reference/paginate_links). Just define the type as array then you’ll be to display it as you want: <?php global $wp_query; $big = 999999999; // need an unlikely integer $paginate_links = paginate_links( array( ‘base’ => str_replace( $big, ‘%#%’, get_pagenum_link( $big ) ), ‘format’ => ‘?paged=%#%’, ‘current’ => max( … Read more
If the rest of your function works you just need to add a call to wp_safe_redirect and hook the whole thing into WordPress. I think that the first hook that will have a populated $wp_query is wp. So alter the last part of your function… if($max_page < $paged){ wp_safe_redirect(get_bloginof(‘url’),’301′); } And then add the following … Read more
You could add an else to both ifs and get the first post/latest post: <?php $nextPost = get_next_post(); if($nextPost) { $nextPostID = $nextPost->ID; ?> <a class=”prev-post” href=”https://wordpress.stackexchange.com/questions/270728/<?php echo get_permalink( $nextPostID ); ?>”> <?php echo $nextPost->post_title; ?> </a> <?php } else { $first_post = get_posts( array( ‘posts_per_page’ => 1, ‘order’ => ‘ASC’ ) ); ?> <a … Read more
Since 4.1.0, WordPress introduce the_posts_pagination to handle number paginate link. I use it all the time, and it just works. With any custom post type. You’ll want to use that function after the while loop. See: https://developer.wordpress.org/reference/functions/the_posts_pagination/
pre_get_posts does fire for REST requests. After copying your function and stepping through it, your code is actually working — the first time. However the WP REST controller has the following bit: if ( $total_posts < 1 ) { // Out-of-bounds, run the query again without LIMIT for total count. unset( $query_args[‘paged’] ); $count_query = … Read more
You’ve specified a priority of 1 for your pre_get_posts function here: add_action( ‘pre_get_posts’, ‘mytheme_portfolio_archive_pages’, 1 ); Jetpack doesn’t specify a priority: add_action( ‘pre_get_posts’, array( $this, ‘posts_per_page_query’ ) ); So that gets added with the default priority for add_action, which is 10: $priority (int) (optional) Used to specify the order in which the functions associated with … Read more
The following code line: $paged = ( get_query_var(‘paged’) ) ? get_query_var(‘paged’) : 1; is using the ternary operator ?: It’s a shorthand notation for if( get_query_var( ‘paged’ ) ) $paged = get_query_var(‘paged’ ); else $paged = 1; This can in fact be simplified further, by using the the second input parameter for the default value, … Read more
You use your custom WP_Query to query posts, but then in pagination you use global $wp_query variable, which contains different query. You should use your custom query variable $works in your pagination, or change global $wp_query variable before your pagination function call. Since you use universal function to output pagination, second method will be better/easier. … Read more