deleting flavors now works, as well as displays a proper flash message

django-openstack/django_openstack/syspanel/views/flavors.py

@@ -39,11 +39,12 @@ class DeleteFlavor(forms.SelfHandlingForm):
     flavorid = forms.CharField(required=True)
 
     def handle(self, request, data):
-        flavor_id = data.get('flavorid', None)
         try:
-            api.flavor_delete(flavor_id, True)
+            flavor_id = data['flavorid']
+            flavor = api.flavor_get(request, flavor_id)
+            api.flavor_delete(request, flavor_id, True)
             messages.info(request, 'Successfully deleted flavor: %s' %
-                          flavor_id)
+                          flavor.name)
         except api_exceptions.ApiException, e:
             messages.error(request, 'Unable to delete flavor: %s' %
                                      e.message)
@@ -56,8 +57,6 @@ def index(request):
         if handled:
             return handled
 
-    # We don't have any way of showing errors for these, so don't bother
-    # trying to reuse the forms from above
     delete_form = DeleteFlavor()
 
     flavors = []