xmlrpc: remove a redundant try/except block

The try block just raises the exception it catches, so just get rid of
it.

Signed-off-by: Daniel Axtens <dja@axtens.net>
Reviewed-by: Stephen Finucane <stephenfinucane@hotmail.com>

diff --git a/patchwork/views/xmlrpc.py b/patchwork/views/xmlrpc.py
index c5e7b9078f1563ec314f360c65fc3006375e5a67..65471e7f9e847955e605cb11ffe7cbf5cd71272d 100644 (file)
--- a/patchwork/views/xmlrpc.py
+++ b/patchwork/views/xmlrpc.py
@@ -747,30 +747,26 @@ def patch_set(user, patch_id, params):
             patch
         Patch.DoesNotExist: The patch did not exist.
     """
-    try:
-        ok_params = ['state', 'commit_ref', 'archived']
-
-        patch = Patch.objects.get(id=patch_id)
+    ok_params = ['state', 'commit_ref', 'archived']
 
-        if not patch.is_editable(user):
-            raise Exception('No permissions to edit this patch')
+    patch = Patch.objects.get(id=patch_id)
 
-        for (k, v) in params.items():
-            if k not in ok_params:
-                continue
+    if not patch.is_editable(user):
+        raise Exception('No permissions to edit this patch')
 
-            if k == 'state':
-                patch.state = State.objects.get(id=v)
+    for (k, v) in params.items():
+        if k not in ok_params:
+            continue
 
-            else:
-                setattr(patch, k, v)
+        if k == 'state':
+            patch.state = State.objects.get(id=v)
 
-        patch.save()
+        else:
+            setattr(patch, k, v)
 
-        return True
+    patch.save()
 
-    except Patch.DoesNotExist:
-        raise
+    return True
 
 
 @xmlrpc_method()