The query filter 4 (continued)

Hi @malferov ??

It looks like we’ve been waiting for you to test a proposed release… Details in The query filter 3 (continued).

I don’t now, how i can do it.

I know this (each update i edits this fragment of code):

/* $wp_query->query(
                                array(
                                        'p'         => $reference_ID,
                                        'orderby'   => $orders[0],
                                        'order'     => $orders[1],
                                        'showposts' => $limit,
                                        'post_type' => $this->get_query_post_types( $reference_ID, $args ),
                                )
                        ); */

                        $wp_query->query(
                            apply_filters('yarpp_display_related_wp_query', [
                                'p'         => $reference_ID,
                                'orderby'   => $orders[0],
                                'order'     => $orders[1],
                                'showposts' => $limit,
                                // 'post_type' => isset($args['post_type']) ? $args['post_type'] : $this->get_post_types(),
                                'post_type' => $this->get_query_post_types($reference_ID, $args)
                            ], $args)
                        );

Is it necessary to test something else for such a change? I have been testing this change for more than three years, the flight is normal ??

@malferov

Our code is a bit different.

Download file:
https://drive.google.com/u/0/uc?id=1TZ6hsHlHPrr2zJ_I2Ww2SRHGbENktDc7&export=download

For instructions, see Manual Upload via WordPress Admin:
https://www.ads-software.com/support/article/managing-plugins/#manual-upload-via-wordpress-admin

I don’t want to test the code in production. I don’t have another testing site.

All I’m asking is to add a standard WordPress filter to be able to change the query. I don’t understand how this can harm the plugin.

@malferov that’s OK. YARPP is used on over 100,000 websites, including those operated by Fortune 500 companies (ex. Disney). Out of caution and best practice, we generally like to get confirmation from the user that requested a feature that the code change resolves their issue. Since our code change is different from yours, it would have been useful and would have expedited the process. But it’s OK. We’ll try to get this prioritized internally.