Commit: a6d20c4a49f3381ec26ec3b89c1d2d7826c2bd17 - allura (git)

allura

Commit MetaInfo

• Revision: a6d20c4a49f3381ec26ec3b89c1d2d7826c2bd17 (tree)
• Time: 2010-09-23 00:49:24
• Author: Rick Copeland <rcopeland@geek...>
• Commiter: Rick Copeland

Log Message

[#959] - fix the missing whereclause delete problem for hosted apps

Change Summary

• modified: ForgeClassic/sfx/model/hosted_apps_model.py

Incremental Difference

--- a/ForgeClassic/sfx/model/hosted_apps_model.py
+++ b/ForgeClassic/sfx/model/hosted_apps_model.py
@@ -100,11 +100,12 @@ class HostedApp(object):
         if user is None: user = c.user
         if project is None: project = c.project
         self._queue_ha_operation('hostedapp_disable', user, project)
-        stmt = T.feature_optin.delete()
-        stmt.execute(
-            feature_type=self.feature_type,
-            owner_table='project',
-            owner_pk=project.get_tool_data('sfx', 'group_id'))
+        stmt = T.feature_optin.delete(
+            whereclause=sa.and_(
+                T.feature_optin.c.feature_type==self.feature_type,
+                T.feature_optin.c.owner_table=='project',
+                T.feature_optin.c.owner_pk==project.get_tool_data('sfx', 'group_id')))
+        stmt.execute()
 
     def addperm(self, user=None, project=None):
         if user is None: user = c.user