I messed around with the code some more, and my particular issue actually ended up relating to the original if statement parameters. Changing the condition to check the home page for queries to if ( $query->is_home() && ! $query->is_main_query() ) { return; } and else { //This is the first page. Just use the offset… … Read more
The pre_get_posts filter is the right place to go. You can check if you are on a search query, and alter it the way you want. Here’s an example: function search_filter($query) { if ( !is_admin() && $query->is_main_query() ) { // Check if we are on a search page if ($query->is_search) { // Get the search … Read more
This code should work OK, but… There are some problems with it. Let’s break it apart and add some comments. function searchcategory($query) { if ($query->is_search) { // Checking only for is_search is very risky. It will be set to true // whenever param “s” is set. So your function will modify any wp_query with s, … Read more
There are a couple of problems with your condition: A single query cannot match all of your conditions – it can’t be a tag as well as search, for example. You need to use || (or) and possibly nest – so for example, if(!is_admin() && ($query->is_home() || $query->is_search())) { This says, “if we’re not in … Read more
In your code ( $query->is_home() && $query->is_date() && $query->is_main_query() ) always returns false because $query->is_home() and $query->is_date() cannot be true on same page. Try this, function add_custom_post_type_to_query( $query ) { if ( ! is_admin() && $query->is_main_query() ) { if ( $query->is_date() || $query->is_home() ) { $query->set( ‘post_type’, array(‘post’, ‘pictures’) ); } } } add_action( ‘pre_get_posts’, … Read more
The reason for that is pretty straight forward… In you code (at least in the part you’ve showed) the only condition you check is: if ( is_post_type_archive( ‘people’ ) ) So when you’re on archive for people, this condition will be true for every WP_Query on that page. Let’s say you have some “Latest news” … Read more
Looks like you can define the AND logical operator with the URL parameters, specifically adding the “+” between your terms like this: url?taxonomy=term1+term2 This will ensure that only posts containing all terms listed are returned. Example: https://example.com/blog?food=sauce+cheese [tax_query] => WP_Tax_Query Object ( [queries] => Array ( [0] => Array ( [taxonomy] => food [terms] => … Read more
Since I just wrote a pre_get_posts answer for someone else, I will modify it slightly and see if it works for you too. When using pre_get_posts the $query variable is passed “by reference” which means we can directly manipulate the $query object with the shortcut functions such as set_query_var. Remove the query code from your … Read more
Add the filter where you need it and remove it afterwards. add_action(‘pre_get_posts’,’blog_filter’); // your query; presumably WP_Query // your first loop remove_action(‘pre_get_posts’,’blog_filter’); // another query; presumably WP_Query // your second loop You can also make your action self-removing: function blog_filter($query) { remove_action(‘pre_get_posts’,’blog_filter’); if ( !is_admin() && $query->is_main_query() ) { if ($query->is_post_type_archive(‘rg_blog’)) { $query->set(‘post_type’, ‘post’); } … Read more
If we look in source, we’ll see under what conditions is_archive is true: if ( $this->is_post_type_archive || $this->is_date || $this->is_author || $this->is_category || $this->is_tag || $this->is_tax ) $this->is_archive = true; The issue is that is_archive is triggering your filter on your post type archive. If you want to exclude those, either make sure it’s not … Read more