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

chore(db): Apply query prepared statements #48765

Merged
merged 1 commit into from
Oct 18, 2024
Merged

Conversation

solracsf
Copy link
Member

@solracsf solracsf commented Oct 17, 2024

Summary

Apply correct execute type on each query.

This is th 2nd batch PR, others will follow, to not have one single PR with a gazillion changes to be reviewed.

TODO

  • CI

Checklist

@solracsf solracsf added the 2. developing Work in progress label Oct 17, 2024
@solracsf solracsf changed the title chore(db): Apply query prepared statements [WiP] chore(db): Apply query prepared statements Oct 17, 2024
Fix: psalm

fix: bad file

fix: bug

chore: add batch

chore: add batch

chore: add batch

fix: psalm
@solracsf solracsf marked this pull request as ready for review October 17, 2024 18:31
@solracsf solracsf changed the title [WiP] chore(db): Apply query prepared statements chore(db): Apply query prepared statements Oct 17, 2024
@solracsf solracsf added 3. to review Waiting for reviews feature: database Database related DB and removed 2. developing Work in progress labels Oct 17, 2024
@solracsf solracsf added this to the Nextcloud 31 milestone Oct 17, 2024
@solracsf solracsf self-assigned this Oct 18, 2024
@provokateurin
Copy link
Member

refactor would be the correct conventional commit type btw.

@provokateurin provokateurin merged commit d062564 into master Oct 18, 2024
179 of 181 checks passed
@provokateurin provokateurin deleted the dbQueriesExecStmt2 branch October 18, 2024 09:45
@solracsf
Copy link
Member Author

@provokateurin noted 👍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
3. to review Waiting for reviews feature: database Database related DB
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants