diff --git a/smash/web/models.py b/smash/web/models.py index 52ee25f30fb07a09461e45ab32f99cf2cf341f17..ee07ad6b5598ca4ccc2807d9cb804a394ffec1da 100644 --- a/smash/web/models.py +++ b/smash/web/models.py @@ -53,11 +53,11 @@ class Subject(models.Model): ) def mark_as_dead(self): - self.status = STATUS_CHOICES_DEAD + self.status = self.STATUS_CHOICES_DEAD self.save() def mark_as_rejected(self): - self.status = 'REJ' + self.status = self.STATUS_CHOICES_REJECTED self.save() sex = models.CharField(max_length=1, @@ -150,10 +150,15 @@ class Visit(models.Model): datetime_end = models.DateTimeField( verbose_name='Visit ends at' ) # Deadline before which all appointments need to be scheduled + + TYPE_CHOICES_LEVEL_A_VISIT = 'A' + TYPE_CHOICES_LEVEL_B_VISIT = 'B' + TYPE_CHOICES_OTHER = 'O' + TYPE_CHOICES = ( - ('A', 'Level A visit'), - ('B', 'Level B visit'), - ('O', 'Other') + (TYPE_CHOICES_LEVEL_A_VISIT, 'Level A visit'), + (TYPE_CHOICES_LEVEL_B_VISIT, 'Level B visit'), + (TYPE_CHOICES_OTHER, 'Other') ) visit_type = models.CharField(max_length=1, choices=TYPE_CHOICES, verbose_name='Visit type', diff --git a/smash/web/templates/_base.html b/smash/web/templates/_base.html index 197e7717e12de2ad36218dbf4bba071f58df2174..1262ad83813554c047e3da6042303642b1a911b8 100644 --- a/smash/web/templates/_base.html +++ b/smash/web/templates/_base.html @@ -445,7 +445,7 @@ desired effect <script src="{% static 'AdminLTE/plugins/jQuery/jquery-2.2.3.min.js' %}"></script> <!-- Bootstrap 3.3.6 --> <script src="{% static 'AdminLTE/js/bootstrap.min.js' %}"></script> - <!-- AdminLTE App --> + <!-- AdminLTE Template Helpers (for example- left side bar) --> <script src="{% static 'AdminLTE/js/app.min.js' %}"></script> <script> diff --git a/smash/web/templates/subjects/edit.html b/smash/web/templates/subjects/edit.html index eb705a01804513b5b4f4839e8b2bf627834c9d1e..0824827ee76668b17b33b96a9cdb8eab82321a88 100644 --- a/smash/web/templates/subjects/edit.html +++ b/smash/web/templates/subjects/edit.html @@ -54,8 +54,8 @@ {% endfor %} <div class="col-md-6"> - <a href="{% url 'web.views.subject_mark' id 0 %}" class="btn btn-warning btn-block">Mark as rejected</a> - <a href="{% url 'web.views.subject_mark' id 1 %}" class="btn btn-danger btn-block">Mark as dead</a><br /> + <a href="{% url 'web.views.subject_mark' id 'rejected' %}" class="btn btn-warning btn-block">Mark as rejected</a> + <a href="{% url 'web.views.subject_mark' id 'dead' %}" class="btn btn-danger btn-block">Mark as dead</a><br /> </div> </div> diff --git a/smash/web/urls.py b/smash/web/urls.py index a5ba2717f52b67b9aa6ff7d3072e367a5937e969..4a71af8d219c17bfc38fc7546edd2bc090122305 100644 --- a/smash/web/urls.py +++ b/smash/web/urls.py @@ -33,7 +33,7 @@ urlpatterns = [ url(r'subjects/subject_visit_details/(?P<id>\d+)$', views.subject_visit_details, name='web.views.subject_visit_details'), url(r'subjects/edit/(?P<id>\d+)$', views.subject_edit, name='web.views.subject_edit'), url(r'subjects/delete/(?P<id>\d+)$', views.subject_delete, name='web.views.subject_delete'), - url(r'subjects/mark/(?P<id>\d+)/(?P<as_what>\d+)$', views.subject_mark, name='web.views.subject_mark'), + url(r'subjects/mark/(?P<id>\d+)/(?P<as_what>[A-z]+)$', views.subject_mark, name='web.views.subject_mark'), url(r'doctors$', views.doctors, name='web.views.doctors'), url(r'doctors/add$', views.doctor_add, name='web.views.doctor_add'), diff --git a/smash/web/views.py b/smash/web/views.py index bb88a1d41f1f2789e46d165af37feb13e730a6f2..0b280f556f58fa3143e17919959e92620921fbff 100644 --- a/smash/web/views.py +++ b/smash/web/views.py @@ -139,9 +139,9 @@ def subject_delete(request, id): def subject_mark(request, id, as_what): who = Subject.objects.get(id=id) - if as_what == '1': + if as_what == 'dead': who.mark_as_dead() - elif as_what == '0': + elif as_what == 'rejected': who.mark_as_rejected() return redirect(subject_edit, id=id)