From f9176330daa9e482699d89aec46d3f5d6f0feeff Mon Sep 17 00:00:00 2001
From: Diane Trout <diane@ghic.org>
Date: Wed, 11 Jan 2017 15:26:20 -0800
Subject: [PATCH] Update flowcell chage_form override

The old version had hardcode url names which have changed some.
this version uses the named url views to point to started email page,

Additionally I copied and repasted the chunk of object-tools from the
source template.
---
 experiments/urls.py                             |  4 ++--
 .../admin/experiments/flowcell/change_form.html | 17 ++++++++---------
 2 files changed, 10 insertions(+), 11 deletions(-)

diff --git a/experiments/urls.py b/experiments/urls.py
index f0fbbf4..18169e8 100644
--- a/experiments/urls.py
+++ b/experiments/urls.py
@@ -16,6 +16,6 @@ urlpatterns = [
     url(r'^config/(?P<fc_id>.+)/json$', flowcell_json),
     url(r'^lanes_for/(?P<username>.+)/json$', lanes_for_json),
     url(r'^file/(?P<key>.+)/?$', read_result_file, name="read_result_file"),
-    url(r'^started/(?P<pk>.+)/$', startedEmail),
-    url(r'^finished/(?P<pk>.+)/$', finishedEmail),
+    url(r'^started/(?P<pk>.+)/$', startedEmail, name='started_email'),
+    url(r'^finished/(?P<pk>.+)/$', finishedEmail, name='finished_email'),
 ]
diff --git a/htsworkflow/templates/admin/experiments/flowcell/change_form.html b/htsworkflow/templates/admin/experiments/flowcell/change_form.html
index 068fbfe..24fea57 100644
--- a/htsworkflow/templates/admin/experiments/flowcell/change_form.html
+++ b/htsworkflow/templates/admin/experiments/flowcell/change_form.html
@@ -1,11 +1,10 @@
 {% extends "admin/change_form.html" %}
-{% load i18n %}
-{% block object-tools %}
-{% if change %}{% if not is_popup %}
-  <ul class="object-tools">
-    <li><a href="../../../../{{ app_label }}/started/{{ object_id }}/">{% trans "Started Email" %}</a></li>
-    <li><a href="history/" class="historylink">{% trans "History" %}</a></li>
-  {% if has_absolute_url %}<li><a href="../../../r/{{ content_type_id }}/{{ object_id }}/" class="viewsitelink">{% trans "View on site" %}</a></li>{% endif%}
-  </ul>
-{% endif %}{% endif %}
+{% load i18n admin_urls %}
+{% block object-tools-items %}
+    <li><a href="{% url 'started_email' pk=object_id %}">{% trans "Started Email" %}</a></li>
+    <li>
+        {% url opts|admin_urlname:'history' original.pk|admin_urlquote as history_url %}
+        <a href="{% add_preserved_filters history_url %}" class="historylink">{% trans "History" %}</a>
+    </li>
+    {% if has_absolute_url %}<li><a href="{{ absolute_url }}" class="viewsitelink">{% trans "View on site" %}</a></li>{% endif %}
 {% endblock %}
-- 
2.30.2