Merge pull request #117 from mboersma/fix-exception-handlers

fix(api): delete new_release only if it exists

Author: mboersma

rootfs/api/models.py

@@ -738,7 +738,8 @@ def create(self, user, *args, **kwargs):
             self.app.deploy(user, new_release)
             return new_release
         except RuntimeError:
-            new_release.delete()
+            if 'new_release' in locals():
+                new_release.delete()
             raise
 
     def save(self, **kwargs):
@@ -895,7 +896,8 @@ def rollback(self, user, version):
             self.app.deploy(user, new_release)
             return new_release
         except RuntimeError:
-            new_release.delete()
+            if 'new_release' in locals():
+                new_release.delete()
             raise
 
     def save(self, *args, **kwargs):  # noqa

rootfs/api/views.py

@@ -363,7 +363,8 @@ def rollback(self, request, **kwargs):
         except EnvironmentError as e:
             return Response({'detail': str(e)}, status=status.HTTP_400_BAD_REQUEST)
         except RuntimeError:
-            new_release.delete()
+            if 'new_release' in locals():
+                new_release.delete()
             raise