This is an automated email from the ASF dual-hosted git repository.
kentontaylor pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/allura.git
The following commit(s) were added to refs/heads/master by this push:
new d0da40cb7 remove some "with context" from template imports
d0da40cb7 is described below
commit d0da40cb7f0919e9f71a14aea723c4a3c244b255
Author: Dave Brondsema <[email protected]>
AuthorDate: Mon Aug 8 13:28:02 2022 -0400
remove some "with context" from template imports
---
Allura/allura/config/app_cfg.py | 7 ++++++-
Allura/allura/templates/jinja_master/lib.html | 8 ++------
Allura/allura/templates/widgets/attachment_list.html | 2 +-
Allura/allura/templates/widgets/edit_post.html | 2 +-
Allura/allura/templates/widgets/moderate_post.html | 2 +-
Allura/allura/templates/widgets/moderate_posts.html | 2 +-
Allura/allura/templates/widgets/new_topic_post.html | 2 +-
Allura/allura/templates/widgets/post_widget.html | 2 +-
Allura/allura/templates/widgets/subscription_form.html | 2 +-
Allura/allura/templates/widgets/thread_widget.html | 2 +-
Allura/allura/templates/widgets/threads_table.html | 2 +-
ForgeBlog/forgeblog/templates/blog_widgets/view_post.html | 2 +-
.../templates/discussion_widgets/thread_header.html | 2 +-
ForgeTracker/forgetracker/templates/tracker/ticket.html | 2 +-
ForgeWiki/forgewiki/templates/wiki/page_view.html | 2 +-
15 files changed, 21 insertions(+), 20 deletions(-)
diff --git a/Allura/allura/config/app_cfg.py b/Allura/allura/config/app_cfg.py
index 6bde340ef..9d2257952 100644
--- a/Allura/allura/config/app_cfg.py
+++ b/Allura/allura/config/app_cfg.py
@@ -110,7 +110,12 @@ class AlluraJinjaRenderer(JinjaRenderer):
jinja2_env.filters['filter'] = lambda s, t=None: list(filter(t and
jinja2_env.tests[t], s))
jinja2_env.filters['nl2br'] = helpers.nl2br_jinja_filter
jinja2_env.filters['subrender'] = helpers.subrender_jinja_filter
- jinja2_env.globals.update({'hasattr': hasattr})
+ jinja2_env.globals.update({
+ 'hasattr': hasattr,
+ 'h': helpers,
+ 'g': app_globals,
+ 'request': tg.request,
+ })
config['tg.app_globals'].jinja2_env = jinja2_env # TG doesn't need
this, but we use g.jinja2_env a lot
return {'jinja': cls(jinja2_env)}
diff --git a/Allura/allura/templates/jinja_master/lib.html
b/Allura/allura/templates/jinja_master/lib.html
index b6177a16a..91b4d3e80 100644
--- a/Allura/allura/templates/jinja_master/lib.html
+++ b/Allura/allura/templates/jinja_master/lib.html
@@ -18,19 +18,15 @@
-#}
{% macro csrf() -%}
- {% if request is defined -%}
{{ request.cookies['_session_id'] or request.environ['_session_id'] }}
- {%- endif %}
{%- endmacro %}
{% macro csrf_token() -%}
- {% if request is defined %}
<input name="_session_id" type="hidden" value="{{csrf()}}">
- {% endif %}
{%- endmacro %}
-{% macro related_artifacts(artifact) -%}
- {% set related_artifacts = artifact.related_artifacts(user=c.user) %}
+{% macro related_artifacts(artifact, user) -%}
+ {% set related_artifacts = artifact.related_artifacts(user=user) %}
{% if related_artifacts %}
<h4>Related</h4>
<p>
diff --git a/Allura/allura/templates/widgets/attachment_list.html
b/Allura/allura/templates/widgets/attachment_list.html
index 1bedec0cd..c992659fb 100644
--- a/Allura/allura/templates/widgets/attachment_list.html
+++ b/Allura/allura/templates/widgets/attachment_list.html
@@ -16,7 +16,7 @@
specific language governing permissions and limitations
under the License.
-#}
-{% import 'allura:templates/jinja_master/lib.html' as lib with context %}
+{% import 'allura:templates/jinja_master/lib.html' as lib %}
<div class="attachments">
{% if attachments %}
{% set attachments = attachments|list %}
diff --git a/Allura/allura/templates/widgets/edit_post.html
b/Allura/allura/templates/widgets/edit_post.html
index d38f9a9bb..ce8263c60 100644
--- a/Allura/allura/templates/widgets/edit_post.html
+++ b/Allura/allura/templates/widgets/edit_post.html
@@ -16,7 +16,7 @@
specific language governing permissions and limitations
under the License.
-#}
-{% import 'allura:templates/jinja_master/lib.html' as lib with context %}
+{% import 'allura:templates/jinja_master/lib.html' as lib %}
<div class="edit_markdown_form">
<form id="edit_{{ action[-4:] }}" method="post" action="{{action}}"
enctype="multipart/form-data">
diff --git a/Allura/allura/templates/widgets/moderate_post.html
b/Allura/allura/templates/widgets/moderate_post.html
index f57648a34..a0960800b 100644
--- a/Allura/allura/templates/widgets/moderate_post.html
+++ b/Allura/allura/templates/widgets/moderate_post.html
@@ -16,7 +16,7 @@
specific language governing permissions and limitations
under the License.
-#}
-{% import 'allura:templates/jinja_master/lib.html' as lib with context %}
+{% import 'allura:templates/jinja_master/lib.html' as lib %}
<form method="POST" action="{{action}}">
<input type="hidden" name="delete" value="True"/>
<a href="" title="Delete" class="moderate_post delete little_link icon btn
ui-button ui-widget ui-state-default ui-corner-all
ui-button-text-only"><span><i class="fa fa-trash-o fa-lg"></i></span></a>
diff --git a/Allura/allura/templates/widgets/moderate_posts.html
b/Allura/allura/templates/widgets/moderate_posts.html
index de0132a3c..09d523b4d 100644
--- a/Allura/allura/templates/widgets/moderate_posts.html
+++ b/Allura/allura/templates/widgets/moderate_posts.html
@@ -16,7 +16,7 @@
specific language governing permissions and limitations
under the License.
-#}
-{% import 'allura:templates/jinja_master/lib.html' as lib with context %}
+{% import 'allura:templates/jinja_master/lib.html' as lib %}
<form {{widget.j2_attrs({'name':name, 'id':id, 'method':method,
'action':action, 'enctype':enctype})}}
{{attrs|default({}, true)|xmlattr}}>
<fieldset class="grid-19">
diff --git a/Allura/allura/templates/widgets/new_topic_post.html
b/Allura/allura/templates/widgets/new_topic_post.html
index e43f40f21..23516c643 100644
--- a/Allura/allura/templates/widgets/new_topic_post.html
+++ b/Allura/allura/templates/widgets/new_topic_post.html
@@ -16,7 +16,7 @@
specific language governing permissions and limitations
under the License.
-#}
-{% import 'allura:templates/jinja_master/lib.html' as lib with context %}
+{% import 'allura:templates/jinja_master/lib.html' as lib %}
<form id="create_new_topic" method="post" action="{{action}}"
enctype='multipart/form-data'>
{% if show_subject %}
<div class="grid-19"> </div>
diff --git a/Allura/allura/templates/widgets/post_widget.html
b/Allura/allura/templates/widgets/post_widget.html
index 53262aa09..d986d5f5d 100644
--- a/Allura/allura/templates/widgets/post_widget.html
+++ b/Allura/allura/templates/widgets/post_widget.html
@@ -16,7 +16,7 @@
specific language governing permissions and limitations
under the License.
-#}
-{% import 'allura:templates/jinja_master/lib.html' as lib with context %}
+{% import 'allura:templates/jinja_master/lib.html' as lib %}
{% set page = request.params.get('page','0') %}
{% set limit = request.params.get('limit','0') %}
{% set params_dict = {} %}
diff --git a/Allura/allura/templates/widgets/subscription_form.html
b/Allura/allura/templates/widgets/subscription_form.html
index f86e3b3e4..782176ac3 100644
--- a/Allura/allura/templates/widgets/subscription_form.html
+++ b/Allura/allura/templates/widgets/subscription_form.html
@@ -16,7 +16,7 @@
specific language governing permissions and limitations
under the License.
-#}
-{% import 'allura:templates/jinja_master/lib.html' as lib with context %}
+{% import 'allura:templates/jinja_master/lib.html' as lib %}
<div {{attrs|default({}, true)|xmlattr}} class="discussion_subscription_form">
<div class="clear"></div>
{% set pager_widget_output = widget.fields['page_list'].display(limit=limit,
page=page, count=count) %}
diff --git a/Allura/allura/templates/widgets/thread_widget.html
b/Allura/allura/templates/widgets/thread_widget.html
index 46df6eee2..7099a6fb5 100644
--- a/Allura/allura/templates/widgets/thread_widget.html
+++ b/Allura/allura/templates/widgets/thread_widget.html
@@ -16,7 +16,7 @@
specific language governing permissions and limitations
under the License.
-#}
-{% import 'allura:templates/jinja_master/lib.html' as lib with context %}
+{% import 'allura:templates/jinja_master/lib.html' as lib %}
{% set primary_artifact = value.primary() %}
<div>
<div class="comment-row">
diff --git a/Allura/allura/templates/widgets/threads_table.html
b/Allura/allura/templates/widgets/threads_table.html
index e259446cb..fc9db05a9 100644
--- a/Allura/allura/templates/widgets/threads_table.html
+++ b/Allura/allura/templates/widgets/threads_table.html
@@ -16,7 +16,7 @@
specific language governing permissions and limitations
under the License.
-#}
-{% import 'allura:templates/jinja_master/lib.html' as lib with context %}
+{% import 'allura:templates/jinja_master/lib.html' as lib %}
{% set forum_subscribed = c.subscribed %} {# set in controller #}
{% set tool_subscribed = c.tool_subscribed %}
diff --git a/ForgeBlog/forgeblog/templates/blog_widgets/view_post.html
b/ForgeBlog/forgeblog/templates/blog_widgets/view_post.html
index 57cee5a84..16e281bfb 100644
--- a/ForgeBlog/forgeblog/templates/blog_widgets/view_post.html
+++ b/ForgeBlog/forgeblog/templates/blog_widgets/view_post.html
@@ -19,7 +19,7 @@
{% import 'allura:templates/jinja_master/lib.html' as lib with context %}
<div{% if h.has_access(value, 'edit') %} class="active-md"
data-markdownlink="" {% endif %}>{{value.html_text|safe}}</div>
-{{lib.related_artifacts(value)}}
+{{lib.related_artifacts(value, c.user)}}
<em class="grid-19">
Posted by {{lib.user_link(base_post.author(), avatar=True)}}
{{lib.abbr_date(value.timestamp)}}
{% if value.state != 'published' %}
diff --git
a/ForgeDiscussion/forgediscussion/templates/discussion_widgets/thread_header.html
b/ForgeDiscussion/forgediscussion/templates/discussion_widgets/thread_header.html
index 1c71c85d0..0779ef7e6 100644
---
a/ForgeDiscussion/forgediscussion/templates/discussion_widgets/thread_header.html
+++
b/ForgeDiscussion/forgediscussion/templates/discussion_widgets/thread_header.html
@@ -16,7 +16,7 @@
specific language governing permissions and limitations
under the License.
-#}
-{% import 'allura:templates/jinja_master/lib.html' as lib with context %}
+{% import 'allura:templates/jinja_master/lib.html' as lib %}
<div class="grid-5"><label>Forum:</label> <a
href="{{value.discussion.url()}}">{{value.discussion.name}}</a></div>
{% if value.first_post %}
diff --git a/ForgeTracker/forgetracker/templates/tracker/ticket.html
b/ForgeTracker/forgetracker/templates/tracker/ticket.html
index 2688cc969..a3e69b392 100644
--- a/ForgeTracker/forgetracker/templates/tracker/ticket.html
+++ b/ForgeTracker/forgetracker/templates/tracker/ticket.html
@@ -170,7 +170,7 @@
{% endblock %}
{% block after_content %}
- {{lib.related_artifacts(ticket)}}
+ {{lib.related_artifacts(ticket, c.user)}}
<div id="discussion_holder">
<h2>Discussion</h2>
{% set thread=ticket.discussion_thread %}
diff --git a/ForgeWiki/forgewiki/templates/wiki/page_view.html
b/ForgeWiki/forgewiki/templates/wiki/page_view.html
index 07b1249b4..681ee4f11 100644
--- a/ForgeWiki/forgewiki/templates/wiki/page_view.html
+++ b/ForgeWiki/forgewiki/templates/wiki/page_view.html
@@ -117,7 +117,7 @@
{% block after_content %}
<hr style="margin-top: 1em; margin-bottom: 2em; clear:both;">
- {{lib.related_artifacts(page)}}
+ {{lib.related_artifacts(page, c.user)}}
{% if page.discussion_thread and c.app.show_discussion %}
{% if h.has_access(page, 'post')() or page.discussion_thread.num_replies
> 0 %}
<div id="discussion_holder"><h2>Discussion</h2>