[yocto] [PATCH] views.py: Added validation in recipe detail

mariano.lopez at linux.intel.com mariano.lopez at linux.intel.com
Mon Jun 29 14:53:21 PDT 2015


From: Mariano Lopez <mariano.lopez at linux.intel.com>

Added the validation for the maintainer in the
recipe detail view. Also added a check if the
recipe exists.

[YOCTO #7904]

Signed-off-by: Mariano Lopez <mariano.lopez at linux.intel.com>
---
 rrs/views.py | 12 ++++++++----
 1 file changed, 8 insertions(+), 4 deletions(-)

diff --git a/rrs/views.py b/rrs/views.py
index ad741f6..e2aa1ce 100644
--- a/rrs/views.py
+++ b/rrs/views.py
@@ -329,6 +329,8 @@ class RecipeDetailView(DetailView):
     def get_context_data(self, **kwargs):
         context = super(RecipeDetailView, self).get_context_data(**kwargs)
         recipe = self.get_object()
+        if not recipe:
+            raise django.http.Http404
 
         release = Release.get_current()
         context['release_name'] = release.name
@@ -357,10 +359,12 @@ class RecipeDetailView(DetailView):
 
         self.recipe_maintainer_history = RecipeMaintainerHistory.get_last()
         recipe_maintainer = RecipeMaintainer.objects.filter(recipe = recipe,
-                history = self.recipe_maintainer_history)[0]
-        maintainer = recipe_maintainer.maintainer
-
-        context['maintainer_name'] = maintainer.name
+                history = self.recipe_maintainer_history)
+        if recipe_maintainer:
+            maintainer = recipe_maintainer[0].maintainer
+            context['maintainer_name'] = maintainer.name
+        else:
+            context['maintainer_name'] = 'No maintainer'
 
         context['recipe_upgrade_details'] = []
         for ru in RecipeUpgrade.objects.filter(recipe =
-- 
1.9.1




More information about the yocto mailing list