Person CRUD working

RIGS/models.py

@@ -34,6 +34,6 @@ class Person(models.Model):
 
     def __unicode__(self):
         string = self.name
-        if self.comments:
+        if self.comments.count() > 0:
             string += "*"
         return string

RIGS/templates/RIGS/person_detail.html

@@ -0,0 +1,30 @@
+{% extends request.is_ajax|yesno:"base_ajax.html,base.html" %}
+{% load widget_tweaks %}
+
+{% block title %}{{ object.name }}{% endblock %}
+
+{% block content %}
+    <div class="col-sm-10 col-sm-offset-1">
+        <h3>{{ object.name }}</h3>
+        <div class="pull-right">
+            <a href="{% url 'person_update' object.pk %}" class="btn btn-primary">Edit <span class="glyphicon glyphicon-pencil"></span></a>
+        </div>
+        <div class="row col-sm-12">
+            <div class="col-sm-6">
+                <dl class="dl-horizontal">
+                    <dt>Name</dt>
+                    <dd>{{ object.name }}</dd>
+
+                    <dt>Phone</dt>
+                    <dd><a href="tel:{{ object.phone }}">{{ object.phone }}</a></dd>
+
+                    <dt>Email</dt>
+                    <dd><a href="mailto:{{ object.email }}">{{ object.email }}</a></dd>
+
+                    <dt>Address</dt>
+                    <dd>{{ object.address|linebreaksbr }}</dd>
+                </dl>
+            </div>
+        </div>
+    </div>
+{% endblock %}

RIGS/templates/RIGS/person_form.html

@@ -1,7 +1,7 @@
 {% extends request.is_ajax|yesno:'base_ajax.html,base.html' %}
 {% load widget_tweaks %}
 
-{% block title %}Add Person{% endblock %}
+{% block title %}{% if object.pk %}Edit {{ object.name }}{% else %}Add Person{% endif %}{% endblock %}
 
 {% block content %}
     <div class="col-sm-offset-1 col-sm-10">

RIGS/urls.py

@@ -13,6 +13,9 @@ urlpatterns = patterns('',
 
     # People
     url(r'^people/$', permission_required_with_403('RIGS.view_person')(views.PersonIndex.as_view()), name='person_list'),
-    url(r'^people/add$', permission_required_with_403('RIGS.add_person')(views.PersonCreate.as_view()), name='person_add'),
+    url(r'^people/(?P<pk>\d+)/$', permission_required_with_403('RIGS.view_person')(views.PersonDetail.as_view()),
+        name='person_detail'),
+    url(r'^people/add/$', permission_required_with_403('RIGS.add_person')(views.PersonCreate.as_view()), name='person_add'),
+    url(r'^people/(?P<pk>\d+)/edit/$', permission_required_with_403('RIGS.change_person')(views.PersonUpdate.as_view()), name='person_update'),
 )
 

RIGS/views.py

@@ -15,6 +15,21 @@ def login(request, **kwargs):
 class PersonIndex(generic.ListView):
     model = models.Person
 
+class PersonDetail(generic.DetailView):
+    model = models.Person
+
 class PersonCreate(generic.CreateView):
     model = models.Person
-    success_url = reverse_lazy('person_list')
+
+    def get_success_url(self):
+        return reverse_lazy('person_detail', kwargs={
+            'pk': self.object.pk,
+        })
+
+class PersonUpdate(generic.UpdateView):
+    model = models.Person
+
+    def get_success_url(self):
+        return reverse_lazy('person_detail', kwargs={
+            'pk': self.object.pk,
+        })