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

BUG: Indexing parameters for object IDs only work for posts #3654

Open
1 task done
JakePT opened this issue Sep 25, 2023 · 0 comments
Open
1 task done

BUG: Indexing parameters for object IDs only work for posts #3654

JakePT opened this issue Sep 25, 2023 · 0 comments
Labels
bug Something isn't working low priority module:sync Issues related to the Sync functionality
Milestone

Comments

@JakePT
Copy link
Contributor

JakePT commented Sep 25, 2023

Describe the bug

When indexing using the parameters for syncing specific object IDs, such as include, lower_limit_object_id, and upper_limit_object_id, the parameters will only work as expected when indexing Posts. This issue is present when syncing via IndexHelper in PHP, WP CLI, or the REST API.

Steps to Reproduce

  1. In WP CLI run wp elasticpress sync --indexables=comment --include=30,32, with real comment IDs.
  2. All comments will be indexed, not just the included IDs.
  3. Repeaet, with posts: wp elasticpress sync --indexables=post --include=30,32, with real post IDs.
  4. Only the included posts will be synced.

Screenshots, screen recording, code snippet

No response

Environment information

No response

WordPress and ElasticPress information

No response

Code of Conduct

  • I agree to follow this project's Code of Conduct
@JakePT JakePT added bug Something isn't working module:sync Issues related to the Sync functionality labels Sep 25, 2023
@JakePT JakePT added this to the 5.0.0 milestone Sep 25, 2023
@felipeelia felipeelia modified the milestones: 5.0.0, 5.1.0 Oct 24, 2023
@felipeelia felipeelia removed their assignment Feb 6, 2024
@felipeelia felipeelia modified the milestones: 5.1.0, 5.2.0 Feb 28, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working low priority module:sync Issues related to the Sync functionality
Projects
None yet
Development

No branches or pull requests

2 participants