From a29ebc0eb30d47ccbfad10f9833ec1895c320999 Mon Sep 17 00:00:00 2001
From: Christophe Boulanger <christophe.boulanger@imio.be>
Date: Thu, 20 Apr 2017 13:33:56 +0200
Subject: [PATCH] Add Motivations and Destinations update function (template,
 view, urls)

---
 .../imiots1datasources_detail.html            |  4 ++--
 passerelle_imio_ts1_datasources/urls.py       | 10 ++++++--
 passerelle_imio_ts1_datasources/views.py      | 24 ++++++++++++++++++-
 3 files changed, 33 insertions(+), 5 deletions(-)

diff --git a/passerelle_imio_ts1_datasources/templates/passerelle_imio_ts1_datasources/imiots1datasources_detail.html b/passerelle_imio_ts1_datasources/templates/passerelle_imio_ts1_datasources/imiots1datasources_detail.html
index 9b0272a..bf30e24 100644
--- a/passerelle_imio_ts1_datasources/templates/passerelle_imio_ts1_datasources/imiots1datasources_detail.html
+++ b/passerelle_imio_ts1_datasources/templates/passerelle_imio_ts1_datasources/imiots1datasources_detail.html
@@ -10,7 +10,7 @@
 </thead>
 <tbody>
 {% for destination in object.get_destination_terms %}
-<tr><td>{{destination.text}}</td><td>{{destination.price}}</td><td>{{destination.description}}</td>
+<tr><td><a href="{% url 'destinationterm-update' pk=destination.id connector_slug=object.slug %}">{{destination.text}}</a></td><td>{{destination.price}}</td><td>{{destination.description}}</td>
   <td>{{destination.paymentrequired}}</td>
     <td><a rel="popup" href="{% url 'destinationterm-delete' pk=destination.id connector_slug=object.slug %}" class="icon-remove-sign"></a></td>
 </tr>
@@ -29,7 +29,7 @@
 </thead>
 <tbody>
 {% for motivation in object.get_motivation_terms %}
-<tr><td>{{motivation.text}}</td><td>{{motivation.price}}</td><td>{{motivation.description}}</td>
+<tr><td><a href="{% url 'motivationterm-update' pk=motivation.id connector_slug=object.slug %}">{{motivation.text}}</a></td><td>{{motivation.price}}</td><td>{{motivation.description}}</td>
     <td><a rel="popup" href="{% url 'motivationterm-delete' pk=motivation.id connector_slug=object.slug %}" class="icon-remove-sign"></a></td>
 </tr>
 {% endfor %}
diff --git a/passerelle_imio_ts1_datasources/urls.py b/passerelle_imio_ts1_datasources/urls.py
index 82113cc..1641610 100644
--- a/passerelle_imio_ts1_datasources/urls.py
+++ b/passerelle_imio_ts1_datasources/urls.py
@@ -2,8 +2,10 @@ from django.conf.urls import patterns, include, url
 from .views import (DatasourcesView,
 		    MotivationtermAddView,
 		    MotivationtermDeleteView,
+		    MotivationtermUpdateView,
 		    DestinationtermAddView,
-		    DestinationtermDeleteView)
+		    DestinationtermDeleteView,
+		    DestinationtermUpdateView)
 
 
 urlpatterns = patterns('',
@@ -15,8 +17,12 @@ management_urlpatterns = patterns('',
 	MotivationtermAddView.as_view(), name='motivationterm-add'),
     url(r'^(?P<connector_slug>[\w,-]+)/motivationterm/(?P<pk>[\w,-]+)/delete/',
 	MotivationtermDeleteView.as_view(), name='motivationterm-delete'),
+    url(r'^(?P<connector_slug>[\w,-]+)/motivationterm/(?P<pk>[\w,-]+)/update/',
+	MotivationtermUpdateView.as_view(), name='motivationterm-update'),
     url(r'^(?P<connector_slug>[\w,-]+)/destinationterm/add/',
 	DestinationtermAddView.as_view(), name='destinationterm-add'),
     url(r'^(?P<connector_slug>[\w,-]+)/destinationterm/(?P<pk>[\w,-]+)/delete/',
-	DestinationtermDeleteView.as_view(), name='destinationterm-delete')
+	DestinationtermDeleteView.as_view(), name='destinationterm-delete'),
+    url(r'^(?P<connector_slug>[\w,-]+)/destinationterm/(?P<pk>[\w,-]+)/update/',
+	DestinationtermUpdateView.as_view(), name='destinationterm-update')	
 )
diff --git a/passerelle_imio_ts1_datasources/views.py b/passerelle_imio_ts1_datasources/views.py
index a0bb64a..36a754e 100644
--- a/passerelle_imio_ts1_datasources/views.py
+++ b/passerelle_imio_ts1_datasources/views.py
@@ -1,4 +1,4 @@
-from django.views.generic import View, CreateView, DeleteView
+from django.views.generic import View, CreateView, DeleteView, UpdateView
 from django.views.generic.detail import SingleObjectMixin
 from django.core.urlresolvers import reverse
 
@@ -31,6 +31,17 @@ class MotivationtermDeleteView(DeleteView):
                         'slug': connector.slug})
 
 
+class MotivationtermUpdateView(UpdateView):
+    model = MotivationTerm
+    fields = '__all__'
+
+    def get_success_url(self):
+        connector = ImioTs1Datasources.objects.get(slug=self.kwargs['connector_slug'])
+        return reverse('view-connector',
+                kwargs={'connector': connector.get_connector_slug(),
+                        'slug': connector.slug})
+
+
 class DestinationtermAddView(CreateView):
     model = DestinationTerm
     fields = '__all__'
@@ -51,3 +62,14 @@ class DestinationtermDeleteView(DeleteView):
         return reverse('view-connector',
                 kwargs={'connector': connector.get_connector_slug(),
                         'slug': connector.slug})
+
+
+class DestinationtermUpdateView(UpdateView):
+    model = DestinationTerm
+    fields = '__all__'
+
+    def get_success_url(self):
+        connector = ImioTs1Datasources.objects.get(slug=self.kwargs['connector_slug'])
+        return reverse('view-connector',
+                kwargs={'connector': connector.get_connector_slug(),
+                        'slug': connector.slug})