Skip to content

fix: clear bindings after DELETE and COUNT query execution - #10693

Open
michalsn wants to merge 2 commits into
codeigniter4:4.8from
michalsn:fix/executed-query-bindings-48
Open

fix: clear bindings after DELETE and COUNT query execution#10693
michalsn wants to merge 2 commits into
codeigniter4:4.8from
michalsn:fix/executed-query-bindings-48

Conversation

@michalsn

@michalsn michalsn commented Oct 31, 2026

Copy link
Copy Markdown
Member

Description
This PR fixes binding accumulation when the same Query Builder is reused for delete() or countAllResults() with the default reset enabled.

This is a follow-up to #10692.

Checklist:

  • Securely signed commits
  • Component(s) with PHPDoc blocks, only if necessary or adds value (without duplication)
  • Unit testing, with >80% coverage
  • User guide updated
  • Conforms to style guide

@michalsn michalsn added bug Verified issues on the current code behavior or pull requests that will fix them 4.8 PRs that target the `4.8` branch. labels Oct 31, 2026

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

4.8 PRs that target the `4.8` branch. bug Verified issues on the current code behavior or pull requests that will fix them

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@michalsn