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

Fix mistake in db query to delete a user config value #33386

Merged
merged 1 commit into from
Jul 27, 2022

Conversation

julien-nc
Copy link
Member

The key value was deleted for all users and all apps. 💥 😁

Signed-off-by: Julien Veyssier <eneiluj@posteo.net>
@julien-nc julien-nc added this to the Nextcloud 25 milestone Jul 27, 2022
@juliusknorr
Copy link
Member

juliusknorr commented Jul 27, 2022

I'm wondering if that is actually something we could safeguard in the where() method of the query builder 🙈

@juliusknorr juliusknorr merged commit 365df89 into master Jul 27, 2022
@juliusknorr juliusknorr deleted the fix/noid/delete-user-value branch July 27, 2022 16:31
@skjnldsv skjnldsv mentioned this pull request Aug 11, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants