From 99dcbdfe3f9dd6adcd837a3f8b05ba7d3474db71 Mon Sep 17 00:00:00 2001
From: "piotr.atyjaszyk" <piotrmk1@gmail.com>
Date: Wed, 18 Jan 2017 16:14:07 +0100
Subject: [PATCH] Removed empty id field from doctor addition
---
smash/web/forms.py | 2 +-
smash/web/templates/doctors/add.html | 2 --
smash/web/views.py | 19 ++++++++++---------
3 files changed, 11 insertions(+), 12 deletions(-)
diff --git a/smash/web/forms.py b/smash/web/forms.py
index 91cc3c37..145ea50f 100644
--- a/smash/web/forms.py
+++ b/smash/web/forms.py
@@ -9,7 +9,7 @@ Redundancy? Maybe yes, but if need arises, contents of forms can be more easily
class SubjectAddForm(ModelForm):
class Meta:
model = Subject
- exclude = ['title']
+ fields = '__all__'
class SubjectDetailForm(ModelForm):
diff --git a/smash/web/templates/doctors/add.html b/smash/web/templates/doctors/add.html
index e51dee18..ae0fe52b 100644
--- a/smash/web/templates/doctors/add.html
+++ b/smash/web/templates/doctors/add.html
@@ -32,7 +32,6 @@
<form method="post" action="" class="form-horizontal">
{% csrf_token %}
- {% for form in formset %}
<div class="box-body">
{% for field in form %}
<div class="form-group">
@@ -52,7 +51,6 @@
</div>
{% endfor %}
</div><!-- /.box-body -->
- {% endfor %}
<div class="box-footer">
<div class="col-sm-6">
diff --git a/smash/web/views.py b/smash/web/views.py
index 0ebc72c7..51054775 100644
--- a/smash/web/views.py
+++ b/smash/web/views.py
@@ -73,7 +73,7 @@ def subjectadd(request):
return redirect(subjects)
else:
form = SubjectAddForm()
-
+
return wrap_response(request, 'subjects/add.html', {'form': form})
@@ -113,15 +113,16 @@ def doctors(request):
def doctoradd(request):
- WorkerFormSet = modelformset_factory(Worker , fields='__all__')
- if request.method == 'POST':
- formset = WorkerFormSet(request.POST, request.FILES)
- if formset.is_valid():
- formset.save()
+ if request.method == 'POST':
+ form = WorkerAddForm(request.POST, request.FILES)
+ if form.is_valid():
+ form.save()
return redirect(doctors)
- else:
- formset = WorkerFormSet(queryset=Worker.objects.none())
- return wrap_response(request, 'doctors/add.html', {'formset': formset})
+ else:
+ form = WorkerAddForm()
+
+ return wrap_response(request, 'doctors/add.html', {'form': form})
+
def doctoredit(request, ids):
--
GitLab