From 271a40b03f94eab1be0cf5caf027871284c745b6 Mon Sep 17 00:00:00 2001
From: ThrawnCA <shell_layer-github@yahoo.com.au>
Date: Wed, 4 Dec 2024 14:03:44 +1000
Subject: [PATCH] [QOLDEV-1011] drop Pylons 'c' object from data request
 handling

---
 ckanext/data_qld/controller_functions.py             | 8 ++++----
 ckanext/data_qld/templates/datarequests/close.html   | 8 ++++++--
 ckanext/data_qld/templates/datarequests/comment.html | 5 +++--
 ckanext/data_qld/templates/datarequests/edit.html    | 8 ++++++--
 ckanext/data_qld/templates/datarequests/new.html     | 5 ++++-
 ckanext/data_qld/templates/datarequests/show.html    | 9 +++++----
 6 files changed, 28 insertions(+), 15 deletions(-)

diff --git a/ckanext/data_qld/controller_functions.py b/ckanext/data_qld/controller_functions.py
index 63d3bcae..25d98524 100644
--- a/ckanext/data_qld/controller_functions.py
+++ b/ckanext/data_qld/controller_functions.py
@@ -38,17 +38,17 @@ def open_datarequest(id):
     context = _get_context()
 
     # Basic initialization
-    c.datarequest = {}
+    datarequest = {}
     try:
         check_access(OPEN_DATAREQUEST, context, data_dict)
-        c.datarequest = get_action(SHOW_DATAREQUEST)(context, data_dict)
+        datarequest = get_action(SHOW_DATAREQUEST)(context, data_dict)
 
-        if c.datarequest.get('closed', False) is False:
+        if datarequest.get('closed', False) is False:
             return abort(403, _('This data request is already open'))
         else:
             data_dict = {}
             data_dict['id'] = id
-            data_dict['organization_id'] = c.datarequest.get('organization_id')
+            data_dict['organization_id'] = datarequest.get('organization_id')
 
             get_action(OPEN_DATAREQUEST)(context, data_dict)
             return redirect_to(url_for('datarequest.show', id=data_dict['id']))
diff --git a/ckanext/data_qld/templates/datarequests/close.html b/ckanext/data_qld/templates/datarequests/close.html
index 130799dd..77605fcf 100644
--- a/ckanext/data_qld/templates/datarequests/close.html
+++ b/ckanext/data_qld/templates/datarequests/close.html
@@ -1,16 +1,20 @@
 {% ckan_extends %}
+{% set datasets = datasets or c.datasets %}
+{% set datarequest = datarequest or c.datarequest %}
+{% set errors = errors or c.errors %}
+{% set errors_summary = errors_summary or c.errors_summary %}
 
 {% block subtitle %}{{ _('Close data request') }}{% endblock %}
 
 {% block breadcrumb_content %}
   <li>{% link_for _('Data requests'), named_route='datarequest.index' %}</li>
-  <li>{% link_for c.datarequest.get('title')|truncate(30), named_route='datarequest.show', id=c.datarequest.get('id') %}</li>
+  <li>{% link_for datarequest.get('title')|truncate(30), named_route='datarequest.show', id=datarequest.get('id') %}</li>
   <li class="active"><a href="">{{ _('Close data request') }}</a></li>
 {% endblock %}
 
 {% block primary_content_inner %}
   <h1 class="{% block page_heading_class %}page-heading{% endblock %}">{% block page_heading %}{{ _('Close data request') }}{% endblock %}</h1>
-  {% snippet "datarequests/snippets/close_datarequest_form.html", datarequest=c.datarequest, datasets=c.datasets, errors=c.errors, errors_summary=c.errors_summary  %}
+  {% snippet "datarequests/snippets/close_datarequest_form.html", datarequest=datarequest, datasets=datasets, errors=errors, errors_summary=errors_summary  %}
 {% endblock %}
 
 {% block page_header %}{% endblock %}
diff --git a/ckanext/data_qld/templates/datarequests/comment.html b/ckanext/data_qld/templates/datarequests/comment.html
index 58a4dc62..f230b969 100644
--- a/ckanext/data_qld/templates/datarequests/comment.html
+++ b/ckanext/data_qld/templates/datarequests/comment.html
@@ -8,7 +8,8 @@
 
 
 {% block primary_content_inner %}
-
-  {% snippet "package/comment_list.html", content_type='datarequest', pkg_id=c.datarequest.id, pkg_name=c.datarequest.id, userobj=c.userobj %}
+  {% set datarequest = datarequest or c.datarequest %}
+  {% set userobj = userobj or c.userobj %}
+  {% snippet "package/comment_list.html", content_type='datarequest', pkg_id=datarequest.id, pkg_name=datarequest.id, userobj=userobj %}
 
 {% endblock %}
diff --git a/ckanext/data_qld/templates/datarequests/edit.html b/ckanext/data_qld/templates/datarequests/edit.html
index 6a78af2d..aefec1fd 100644
--- a/ckanext/data_qld/templates/datarequests/edit.html
+++ b/ckanext/data_qld/templates/datarequests/edit.html
@@ -1,16 +1,20 @@
 {% ckan_extends %}
+{% set datarequest = datarequest or c.datarequest %}
+{% set errors = errors or c.errors %}
+{% set errors_summary = errors_summary or c.errors_summary %}
+{% set original_title = original_title or c.original_title %}
 
 {% block subtitle %}{{ _('Edit data request') }}{% endblock %}
 
 {% block breadcrumb_content %}
   <li>{% link_for _('Data requests'), named_route='datarequest.index' %}</li>
-  <li>{% link_for c.original_title|truncate(30), named_route='datarequest.show', id=c.datarequest.get('id') %}</li>
+  <li>{% link_for original_title|truncate(30), named_route='datarequest.show', id=datarequest.get('id') %}</li>
   <li class="active"><a href="">{{ _('Edit data request') }}</a></li>
 {% endblock %}
 
 {% block primary_content_inner %}
   <h1 class="{% block page_heading_class %}page-heading{% endblock %}">{% block page_heading %}{{ _('Edit data request') }}{% endblock %}</h1>
-  {% snippet "datarequests/snippets/edit_datarequest_form.html", data=c.datarequest, errors=c.errors, errors_summary=c.errors_summary, offering=c.offering %}
+  {% snippet "datarequests/snippets/edit_datarequest_form.html", data=datarequest, errors=errors, errors_summary=errors_summary %}
 {% endblock %}
 
 {% block page_header %}{% endblock %}
diff --git a/ckanext/data_qld/templates/datarequests/new.html b/ckanext/data_qld/templates/datarequests/new.html
index cdb7e28c..acc3ea05 100644
--- a/ckanext/data_qld/templates/datarequests/new.html
+++ b/ckanext/data_qld/templates/datarequests/new.html
@@ -1,4 +1,7 @@
 {% ckan_extends %}
+{% set datarequest = datarequest or c.datarequest %}
+{% set errors = errors or c.errors %}
+{% set errors_summary = errors_summary or c.errors_summary %}
 
 {% block subtitle %}{{ _('Create data request') }}{% endblock %}
 
@@ -9,7 +12,7 @@
 
 {% block primary_content_inner %}
   <h1 class="{% block page_heading_class %}page-heading{% endblock %}">{% block page_heading %}{{ _('Create data request') }}{% endblock %}</h1>
-  {% snippet "datarequests/snippets/new_datarequest_form.html", data=c.datarequest, errors=c.errors, errors_summary=c.errors_summary, offering=c.offering %}
+  {% snippet "datarequests/snippets/new_datarequest_form.html", data=datarequest, errors=errors, errors_summary=errors_summary %}
 {% endblock %}
 
 {% block secondary_content_additional_info %}
diff --git a/ckanext/data_qld/templates/datarequests/show.html b/ckanext/data_qld/templates/datarequests/show.html
index 611afa87..69a15e0d 100644
--- a/ckanext/data_qld/templates/datarequests/show.html
+++ b/ckanext/data_qld/templates/datarequests/show.html
@@ -1,10 +1,11 @@
 {% ckan_extends %}
+{% set datarequest = datarequest or c.datarequest %}
 
-{% block title %}Data request {{c.datarequest.get('title', '')}}{% endblock %}
+{% block title %}Data request {{datarequest.get('title', '')}}{% endblock %}
 
 {% block breadcrumb_content %}
   <li>{% link_for _('Data requests'), named_route='datarequest.index' %}</li>
-  <li>{% link_for c.datarequest.get('title'), named_route='datarequest.show', id=datarequest_id %}</li>
+  <li>{% link_for datarequest.get('title'), named_route='datarequest.show', id=datarequest_id %}</li>
 {% endblock %}
 
 {% block content_action %}
@@ -13,11 +14,11 @@
     {% link_for _('Manage'), named_route='datarequest.update', id=datarequest_id, class_='btn btn-default', icon='wrench' %}
   {% endif %}
 
-  {% if h.check_access('close_datarequest', {'id':datarequest_id }) and not c.datarequest.closed %}
+  {% if h.check_access('close_datarequest', {'id':datarequest_id }) and not datarequest.closed %}
     {% link_for _('Close'), named_route='datarequest.close', id=datarequest_id, class_='btn btn-danger', icon='lock' %}
   {% endif %}
 
-  {% if h.check_access('open_datarequest', {'id':datarequest_id }) and c.datarequest.closed %}
+  {% if h.check_access('open_datarequest', {'id':datarequest_id }) and datarequest.closed %}
     {% link_for _('Re-open'), named_route='data_qld.open_datarequest', id=datarequest_id, class_='btn btn-success', icon='unlock' %}
   {% endif %}