replaced db->execute with prepared statements
Part of #757 (closed)
What does this MR do?
Replaces some of the db->execute
calls with prepared statements. This are only those that could be replaced without changes in the Database
class.
How confident are you it won't break things if deployed?
Quite sure, the tests work
Links to related issues
Checklist
-
added a test, or explain why one is not needed/possible... using the same tests as before -
no unrelated changes -
asked someone for a code review -
joined #foodsharing-beta channel at https://slackin.yunity.org -
added an entry to CHANGELOG.md (description, merge request link, username(s)) -
Once your MR has been merged, you are responsible to update the #foodsharing-beta Slack channel about what has been changed here. They will test your work in different browsers, roles or other settings