Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Featured first argument bug #2721

Closed
3 tasks done
Gnodesign opened this issue Jan 30, 2024 · 2 comments · Fixed by #2725
Closed
3 tasks done

Featured first argument bug #2721

Gnodesign opened this issue Jan 30, 2024 · 2 comments · Fixed by #2725
Assignees

Comments

@Gnodesign
Copy link
Collaborator

Issue: #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]):

  • I have deactivated other plugins and confirmed this bug occurs when only WP Job Manager plugin is active.
  • This bug happens with a default WordPress theme active.
  • I can reproduce this bug consistently using the steps above.

WordPress Environment

  • WordPress Version: 6.4.2
  • WP Job Manager Version: 2.2.0
  • PHP Version: 8.0.30
@Gnodesign
Copy link
Collaborator Author

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

@mikeyarce mikeyarce self-assigned this Jan 30, 2024
@mikeyarce
Copy link
Member

Thanks for the report, @Gnodesign !

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants