From 3d2f97b50fd2d2611f921b684333f12e7a090284 Mon Sep 17 00:00:00 2001
From: Benjamin <bstadlbauer@posteo.de>
Date: Mon, 6 Mar 2023 17:53:01 +0100
Subject: [PATCH] remove explicit permission checks

---
 recipes/templates/recipe.html  |  2 --
 recipes/templates/recipes.html | 14 +++++---------
 recipes/templates/version.html |  2 --
 recipes/views.py               |  7 +------
 templates/base_main.html       |  2 +-
 templates/index.html           |  4 +---
 6 files changed, 8 insertions(+), 23 deletions(-)

diff --git a/recipes/templates/recipe.html b/recipes/templates/recipe.html
index 6a932f0..08d20d9 100644
--- a/recipes/templates/recipe.html
+++ b/recipes/templates/recipe.html
@@ -2,10 +2,8 @@
 {% block title %}{{ recipe.title }}{% endblock %}
 {% block main %}
 <h1>{{ recipe.title }}</h1>
-{% if perms.recipes.add_recipe %}
 <p><a href="{% url 'edit-recipe' recipe.slug %}">Edit recipe name</a></p>
 <p><a href="{% url 'add-version' recipe.slug %}">Add version</a></p>
-{% endif %}
 	<ul>
 		{% for v in versions %}
 		<li><a href="{{ v.get_absolute_url }}">{{ v.label }}</a></li>
diff --git a/recipes/templates/recipes.html b/recipes/templates/recipes.html
index 33273b0..faee634 100644
--- a/recipes/templates/recipes.html
+++ b/recipes/templates/recipes.html
@@ -2,14 +2,10 @@
 {% block title %}Recipes{% endblock %}
 {% block main %}
 <h1>Recipes</h1>
-{% if perms.recipes.add_recipe %}
 <p><a href="{% url 'add-recipe' %}">Add recipe</a></p>
-{% endif %}
-	{% if perms.recipes.view_recipe %}
-		<ul>
-			{% for recipe in recipes %}
-			<li><a href="{{ recipe.get_absolute_url }}">{{ recipe.title }}</a></li>
-			{% endfor %}
-		</ul>
-	{% endif %}
+	<ul>
+		{% for recipe in recipes %}
+		<li><a href="{{ recipe.get_absolute_url }}">{{ recipe.title }}</a></li>
+		{% endfor %}
+	</ul>
 {% endblock %}
diff --git a/recipes/templates/version.html b/recipes/templates/version.html
index c1490fe..00ad0ff 100644
--- a/recipes/templates/version.html
+++ b/recipes/templates/version.html
@@ -5,11 +5,9 @@
 {% if has_multiple_versions %}
 <p><a href="{{ recipe.get_absolute_url }}">Show all versions</a></p>
 {% endif %}
-{% if perms.recipes.add_recipe %}
 <p><a href="{% url 'edit-recipe' recipe.slug %}">Edit recipe name</a></p>
 <p><a href="{% url 'add-version' recipe.slug %}">Add version</a></p>
 <p><a href="{% url 'edit-version' recipe.slug version.slug %}">Edit Version</a></p>
-{% endif %}
 	<ul>
 		{% for i in ingredients %}
 			<li>{{ i.text }}</li>
diff --git a/recipes/views.py b/recipes/views.py
index a64ff82..f2797ed 100644
--- a/recipes/views.py
+++ b/recipes/views.py
@@ -1,7 +1,7 @@
 from django.shortcuts import render, get_object_or_404
 from .models import Recipe, Version, Ingredient
 from .forms import RecipeForm, VersionForm, IngredientFormSet
-from django.contrib.auth.decorators import login_required, permission_required
+from django.contrib.auth.decorators import login_required
 from django.urls import reverse
 from django.http.response import HttpResponseRedirect
 from django.conf import settings
@@ -10,13 +10,11 @@ def get_name_of_user(user):
 	return user.first_name if user.first_name else user.username
 
 @login_required
-@permission_required('recipes.view_recipe')
 def recipes(request):
 	recipes = Recipe.objects.all()
 	return render(request, 'recipes.html', context={'recipes': recipes})
 
 @login_required
-@permission_required('recipes.view_recipe')
 def version(request, slug_recipe, slug_version):
 	recipe = get_object_or_404(Recipe, slug=slug_recipe)
 	has_multiple_versions = recipe.versions.all().count() > 1 # type: ignore
@@ -26,7 +24,6 @@ def version(request, slug_recipe, slug_version):
 	return render(request, 'version.html', context={'recipe': recipe, 'has_multiple_versions': has_multiple_versions, 'version': version, 'ingredients': ingredients})
 
 @login_required
-@permission_required('recipes.view_recipe')
 def recipe(request, slug):
 	recipe = get_object_or_404(Recipe, slug=slug)
 	versions = recipe.versions.all() # type: ignore
@@ -86,7 +83,6 @@ def edit_recipe(request, slug):
 	return render(request, 'edit-recipe.html', {'form': form})
 
 @login_required
-@permission_required('recipes.add_recipe')
 def add_version(request, slug):
 	recipe = get_object_or_404(Recipe, slug=slug)
 	if (recipe.versions.all().count() > 0): # type: ignore
@@ -118,7 +114,6 @@ def add_version(request, slug):
 	return render(request, 'add-version.html', {'version_form': version_form, 'ingredients_formset': ingredients_formset})
 
 @login_required
-@permission_required('recipes.add_recipe')
 def edit_version(request, slug_recipe, slug_version):
 	recipe = get_object_or_404(Recipe, slug=slug_recipe)
 	version = get_object_or_404(Version, recipe=recipe, slug=slug_version)
diff --git a/templates/base_main.html b/templates/base_main.html
index 42750a6..de264fd 100644
--- a/templates/base_main.html
+++ b/templates/base_main.html
@@ -18,7 +18,7 @@
 		{% endif %}
 	</ul>
 	<ul>
-		{% if perms.recipes %}<li><a href="{% url 'recipes' %}">Recipes</a></li>{% endif %}
+		<li><a href="{% url 'recipes' %}">Recipes</a></li>
 	</ul>
 </nav>
 <main>
diff --git a/templates/index.html b/templates/index.html
index 202c06b..a36b4e1 100644
--- a/templates/index.html
+++ b/templates/index.html
@@ -3,8 +3,6 @@
 {% block main %}
 <h1>Barn</h1>
 <ul>
-	{% if perms.recipes %}
-		<li><h2><a href="{% url 'recipes' %}">Recipes</a></h2></li>
-	{% endif %}
+	<li><h2><a href="{% url 'recipes' %}">Recipes</a></h2></li>
 </ul>
 {% endblock %}