You are using pre_get_posts incorrectly. You should not be returning anything at all And you should be manipulating the query object that is passed into the filter and not modifying the globals at all. For example: function modify_onthisday_cat( $query ) { if ($query->is_category(7) && $query->is_main_query() ) { global $post; // not going to be set … Read more
I just ran into this with WooCommerce. I ended up passing the IDs I needed to a variable for safe keeping, and then hooking a function with extremely low priority. That worked fine. I figure woocommerce or what of my cleint’s 8000000 plugins was overwriting post__in. add_filter(‘pre_get_posts’, ‘assign_ids’, 1000);
I used tax_query instead of tag__not_in and it works now. if ( is_post_type_archive( ‘event’ ) ) { $taxquery = array( array( ‘taxonomy’ => ‘event-tag’, ‘field’ => ‘id’, ‘terms’ => array( 53 ), //the ID of the event tag ‘operator’=> ‘NOT IN’ ) ); $query->set(‘tax_query’, $taxquery); return; }
You’ve got a typo there, should be: is_admin(). You should enable debugging by setting define(‘WP_DEBUG’, true); in wp-config.php to see such errors. Also there’s no need of returning anything in action hook. add_action( ‘pre_get_posts’,’filter_archive’, 1 ); function filter_archive( $query ) { if ( ! is_admin() && is_category(’76’) && $query->is_main_query() ) $query->set(‘posts_per_page’, ’12’); } Sidenote: personally … Read more
This is not how pre_get_posts works. The pre_get_posts action gives developers access to the $query object by reference (any changes you make to $query are made directly to the original object – no return value is necessary). https://codex.wordpress.org/Plugin_API/Action_Reference/pre_get_posts What you are doing is simply wrong. You don’t “return” another query via pre_get_posts, you alter the … Read more
I am not 100% sure what you are trying to accomplish, but you can “search for both strings” simply by concatenating them: function checkForAa($query){ if($query->is_search){ $search_string = $query->get( ‘s’ ); if( stripos( $search_string , ‘aa’ ) !== false){ $new_string = str_ireplace( ‘aa’ , ‘xxx’ , $search_string ); }elseif( stripos( $search_string , ‘å’ ) !== false){ … Read more
The WP_Query Object does not support retrieving posts by author_meta so neither will the pre_get_posts hook. The next best thing would be to run a quick WP_User_Query and get all users by the specific author_meta then limit our query to only pull posts from those authors. Here’s what that would look like: /** * Modify … Read more
$screen may not be available yet when pre_get_posts fires. Try this instead: public function my_post_type_default_sort( $query ){ if(!isset($_GET[‘orderby’]) && is_admin()) { $orderby = ‘0’; } if($query->is_main_query() && is_admin() && is_post_type_archive(‘my_post_type’) && $orderby=0) { $query->set(‘orderby’,’scr_date’); $query->set(‘order’, ‘ASC’); } }
pre_get_posts is not a filter hook, it is an action. Also you should not set orderby to menu_order for posts as they are not hierarchical. Set orderby to date instead. function modify_query_amount_shown( $query ) { if ( $query->is_category ) { $query->set( ‘posts_per_page’, 30 ); $query->set( ‘orderby’, ‘date’ ); } } add_action( ‘pre_get_posts’, ‘modify_query_amount_shown’ ); Now … Read more
Here’s one way with the following tax query: ‘tax_query’ => [ [ ‘taxonomy’ => ‘category’, ‘field’ => ‘slug’, ‘terms’ => [ ‘foo’ ], ‘operator’ => ‘NOT IN’ ], ], where the foo term slug is excluded.