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.

Sign up for GitHub

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

Jump to bottom

Deep filtering doesn't ignore draft entries #8331

Closed
alexandrebodin opened this issue Oct 14, 2020 · 0 comments · Fixed by #8356
Closed

Deep filtering doesn't ignore draft entries #8331

alexandrebodin opened this issue Oct 14, 2020 · 0 comments · Fixed by #8356
Assignees
@Convly
Labels
issue: bug Issue reporting a bug severity: medium If it breaks the basic use of the product but can be worked around source: core:database Source is core/database package status: confirmed Confirmed by a Strapi Team member or multiple community members

Comments

@alexandrebodin
Copy link
Member

Describe the bug

WHen using deep filtering the query match against draft content even if publicationState is live.

Steps to reproduce the behavior

  1. Create an article CT with a one way relation to a category CT
  2. Create draft & publish contents and relations
  3. Test a deep filtering query against the articles that would match a draft category:
/articles?categories.name="draft-cat"

Expected behavior
should not match the draft content in deep filtering joins

System

  • Node.js version: v12
  • Strapi version: 3.2.3
  • Database: sqlite
  • Operating system: macOS
@alexandrebodin alexandrebodin added severity: medium If it breaks the basic use of the product but can be worked around source: core:database Source is core/database package labels Oct 14, 2020
@derrickmehaffy derrickmehaffy added status: confirmed Confirmed by a Strapi Team member or multiple community members issue: bug Issue reporting a bug labels Oct 14, 2020
@derrickmehaffy derrickmehaffy mentioned this issue Oct 14, 2020
Merged
@Convly Convly mentioned this issue Oct 16, 2020
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@Convly Convly

Labels
issue: bug Issue reporting a bug severity: medium If it breaks the basic use of the product but can be worked around source: core:database Source is core/database package status: confirmed Confirmed by a Strapi Team member or multiple community members
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Fix deepFiltering with draft & publish strapi/strapi
3 participants
@alexandrebodin @derrickmehaffy @Convly