From 9eeb03c0dd148d992afed3ee9c241cd6ec01a1bd Mon Sep 17 00:00:00 2001 From: David Baker Date: Fri, 1 May 2015 14:21:03 +0100 Subject: [PATCH] Don't use self.execute: it's designed for fetching stuff --- synapse/storage/registration.py | 11 ++++++++--- 1 file changed, 8 insertions(+), 3 deletions(-) diff --git a/synapse/storage/registration.py b/synapse/storage/registration.py index ff0a2a9e8..530bbe6b4 100644 --- a/synapse/storage/registration.py +++ b/synapse/storage/registration.py @@ -112,10 +112,15 @@ class RegistrationStore(SQLBaseStore): @defer.inlineCallbacks def user_delete_access_tokens_apart_from(self, user_id, token_id): - yield self._execute( - "delete_access_tokens_apart_from", None, + yield self.runInteraction( + "user_delete_access_tokens_apart_from", + self._user_delete_access_tokens_apart_from, user_id, token_id + ) + + def _user_delete_access_tokens_apart_from(self, txn, user_id, token_id): + txn.execute( "DELETE FROM access_tokens WHERE user_id = ? AND id != ?", - user_id, token_id + (user_id, token_id) ) @defer.inlineCallbacks