From acbcf3f606e0942cd1ba8a14a36cf7ea5ba2d31d Mon Sep 17 00:00:00 2001
From: Benjamin <bstadlbauer@posteo.de>
Date: Fri, 17 Feb 2023 16:39:00 +0100
Subject: [PATCH] fixup! Add forms for adding recipes and versions

Basic form for adding a version. Still needs:

* Check of slug uniqueness
* Choose between author user or author name
* slug auto-creation
* ingredients

TODO:

* Add recipe
---
 recipes/forms.py                   |  9 +++++++++
 recipes/templates/add-version.html | 11 +++++++++++
 recipes/templates/recipe.html      |  2 +-
 recipes/templates/version.html     |  2 +-
 recipes/urls.py                    |  1 +
 recipes/views.py                   | 21 +++++++++++++++++++++
 6 files changed, 44 insertions(+), 2 deletions(-)
 create mode 100644 recipes/forms.py
 create mode 100644 recipes/templates/add-version.html

diff --git a/recipes/forms.py b/recipes/forms.py
new file mode 100644
index 0000000..1a49779
--- /dev/null
+++ b/recipes/forms.py
@@ -0,0 +1,9 @@
+from django.forms import ModelForm, BooleanField
+from .models import Recipe, Version, Ingredient
+
+class VersionForm(ModelForm):
+	class Meta:
+		model = Version
+		fields = ['label', 'slug', 'body']
+
+	# use_author_user = BooleanField(initial=True)
diff --git a/recipes/templates/add-version.html b/recipes/templates/add-version.html
new file mode 100644
index 0000000..783c5ae
--- /dev/null
+++ b/recipes/templates/add-version.html
@@ -0,0 +1,11 @@
+{% extends "base_main.html" %}
+{% block title %}{{ recipe.title }}{% endblock %}
+{% block main %}
+<form action="" method="post">
+	{% csrf_token %}
+		<table>
+		{{ form.as_table }}
+	</table>
+	<input type="submit" value="Submit">
+</form>
+	{% endblock %}
diff --git a/recipes/templates/recipe.html b/recipes/templates/recipe.html
index a500538..0dfa759 100644
--- a/recipes/templates/recipe.html
+++ b/recipes/templates/recipe.html
@@ -3,7 +3,7 @@
 {% block main %}
 <h1>{{ recipe.title }}</h1>
 {% if perms.recipes.add_recipe %}
-<p><a href="#">Add version</a></p>
+<p><a href="{% url 'add-version' recipe.slug %}">Add version</a></p>
 {% endif %}
 	<ul>
 		{% for v in versions %}
diff --git a/recipes/templates/version.html b/recipes/templates/version.html
index 45190c7..dbf44b1 100644
--- a/recipes/templates/version.html
+++ b/recipes/templates/version.html
@@ -6,7 +6,7 @@
 <p><a href="{{ recipe.get_absolute_url }}">Show all versions</a></p>
 {% endif %}
 {% if perms.recipes.add_recipe %}
-<p><a href="#">Add version</a></p>
+<p><a href="{% url 'add-version' recipe.slug %}">Add version</a></p>
 {% endif %}
 	<ul>
 		{% for i in ingredients %}
diff --git a/recipes/urls.py b/recipes/urls.py
index a0c7f45..148c522 100644
--- a/recipes/urls.py
+++ b/recipes/urls.py
@@ -3,6 +3,7 @@ from . import views
 
 urlpatterns = [
 	path('', views.recipes, name='recipes'),
+	path('<slug:slug_recipe>/add-version/', views.add_version, name='add-version'),
 	path('<slug:slug_recipe>/<slug:slug_version>/', views.version, name='version'),
 	path('<slug:slug>/', views.recipe, name='recipe'),
 ]
diff --git a/recipes/views.py b/recipes/views.py
index 08588ec..f153c09 100644
--- a/recipes/views.py
+++ b/recipes/views.py
@@ -1,5 +1,6 @@
 from django.shortcuts import render, get_object_or_404
 from .models import Recipe, Version
+from .forms import VersionForm
 from django.contrib.auth.decorators import login_required, permission_required
 from django.urls import reverse
 from django.http.response import HttpResponseRedirect
@@ -34,3 +35,23 @@ def recipe(request, slug):
 			return version(request, slug, versions.get().slug)
 	else:
 		return render(request, 'recipe.html', context={'recipe': recipe, 'versions': versions})
+
+@login_required
+@permission_required('recipes.add_recipe')
+def add_version(request, slug_recipe):
+	recipe = Recipe.objects.get(slug=slug_recipe)
+
+	if request.method == 'POST':
+		form = VersionForm(request.POST)
+
+		if form.is_valid():
+			version = form.save(commit=False)
+			version.author_user = request.user
+			version.recipe = recipe
+			version.save()
+			return HttpResponseRedirect(reverse('version', kwargs={'slug_recipe': version.recipe.slug, 'slug_version': version.slug}))
+
+	else:
+		form = VersionForm()
+
+	return render(request, 'add-version.html', {'form': form})