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

Filter and Advanced Search Results different #318

Closed
mateuswetah opened this issue Oct 16, 2019 · 3 comments
Assignees
Labels
bug

Comments

@mateuswetah
Copy link
Contributor

@mateuswetah mateuswetah commented Oct 16, 2019

We have a situation on this installation: https://mhn.tainacan.org/. The following two requests to the https://mhn.tainacan.org/wp-json/tainacan/v2/collection/233012/items endpoint with this parameters:

context=edit
perpage=12
paged=1
order=DESC
orderby=date
admin_view_mode=table
fetch_only=thumbnail
fetch_only_meta=280706,233016
taxquery[0][taxonomy]=tnc_tax_232911
taxquery[0][operator]=LIKE
taxquery[0][originalMeta]=232953-tnc_tax_232911-term
taxquery[0][taxonomy_id]=232911
taxquery[0][isFetching]=0
taxquery[0][terms]=kleuber
advancedSearch=true

and

context=edit
fromBreadcrumb=true
perpage=12
paged=1
order=DESC
orderby=date
admin_view_mode=table
fetch_only=thumbnail
fetch_only_meta=280706,233016
taxquery[0][taxonomy]=tnc_tax_232911
taxquery[0][terms][0]=7372
taxquery[0][compare]=IN

are returning different values. The first one shows an empty list and the second displays some items. I am not sure what could be causing the error so far, need to investigate...

@mateuswetah mateuswetah added the bug label Oct 16, 2019
@leogermani leogermani added this to To Do in tainacan/tainacan Oct 17, 2019
@vnmedeiros vnmedeiros moved this from To Do to In Progress in tainacan/tainacan Oct 22, 2019
@vnmedeiros

This comment has been minimized.

Copy link
Contributor

@vnmedeiros vnmedeiros commented Oct 22, 2019

@mateuswetah, the index on arguments array tax_query don't mounted correctly by the front-end, but this is not a critical problem.

@mateuswetah

This comment has been minimized.

Copy link
Contributor Author

@mateuswetah mateuswetah commented Oct 22, 2019

@vnmedeiros I'll be opening a separate issue for that if is not high priority : #321.

@vnmedeiros vnmedeiros moved this from In Progress to Validation in tainacan/tainacan Oct 22, 2019
leogermani added a commit that referenced this issue Oct 23, 2019
vnmedeiros added a commit that referenced this issue Oct 24, 2019
@leogermani

This comment has been minimized.

Copy link
Contributor

@leogermani leogermani commented Oct 24, 2019

Apparently, the problem was that WordPress was not properly updating terms count because its default behavior is to count only when related posts have the publish status. We changed that

@leogermani leogermani closed this Oct 24, 2019
tainacan/tainacan automation moved this from Validation to Done Oct 24, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
3 participants
You can’t perform that action at this time.