Automattic / WP-Job-Manager

Manage job listings from the WordPress admin panel, and allow users to post jobs directly to your site.
https://wpjobmanager.com
GNU General Public License v3.0
895 stars 366 forks source link

Featured first argument bug #2721

Closed Gnodesign closed 5 months ago

Gnodesign commented 6 months ago

Issue: https://github.com/Automattic/WP-Job-Manager/pull/2675

Describe the bug This feature performs as intended until the filters are deactivated. In such cases, the ordering reverts to its default behavior, disregarding the featured_first argument.

To Reproduce Steps to reproduce the behavior:

  1. Create a page with the shortcode [jobs orderby="ID" featured_first="true" order="DESC" show_filters="true"]
  2. Visit the page and you can see that it works as expected.
  3. Edit the jobs page and disable the filters: [jobs orderby="ID" featured_first="true" order="DESC" show_filters="false"]
  4. Visit the page, and you can see the featured job listings are not shown on top as expected.

Expected behavior The featured_first functionality should operate seamlessly in both scenarios, whether search filters are applied or not.

Isolating the problem (mark completed items with an [x]):

WordPress Environment

Gnodesign commented 6 months ago

@mikeyarce You may want to take a look at this.

mikeyarce commented 6 months ago

Thanks for the report, @Gnodesign !