From 09c49d88261c2c2979f13a793c604448df01b6ab Mon Sep 17 00:00:00 2001
From: Nik Nyby <nikolas@gnu.org>
Date: Wed, 1 Jul 2015 15:53:54 -0400
Subject: [PATCH 1/2] Use 'pageblock' instead of 'block' as django template var

Because 'block' has the potential to conflict with django's 'block'
templatetag.

Closes github issue #29
---
 pagetree/templates/pagetree/edit_page.html | 38 +++++++++++-----------
 pagetree/templates/pagetree/page.html      | 16 ++++-----
 pagetree/templates/pagetree/test_page.html | 14 ++++----
 pagetree/templates/pagetree/testblock.html |  2 +-
 4 files changed, 35 insertions(+), 35 deletions(-)

diff --git a/pagetree/templates/pagetree/edit_page.html b/pagetree/templates/pagetree/edit_page.html
index 557b51c..ecff2a3 100644
--- a/pagetree/templates/pagetree/edit_page.html
+++ b/pagetree/templates/pagetree/edit_page.html
@@ -179,39 +179,39 @@ <h3 style="margin-top: 0;"><a href="{{root.get_edit_url}}">{{section.hierarchy.n
 
     <p>Drag the <span class="glyphicon glyphicon-resize-vertical"></span> arrows to reorder pageblocks:</p>
 
-    {% for block in section.pageblock_set.all %}
-    <div class="row block-dragger" id="pageblock-{{block.id}}">
+    {% for pageblock in section.pageblock_set.all %}
+    <div class="row block-dragger" id="pageblock-{{pageblock.id}}">
 
         <div class="col-md-1 draghandle">
             <span class="glyphicon glyphicon-resize-vertical"></span>
         </div>
 
         <div class="col-md-2">
-            <b><a data-toggle="modal" href="#edit-pageblock-{{block.id}}">{{block.block.display_name}}</a></b>
+            <b><a data-toggle="modal" href="#edit-pageblock-{{pageblock.id}}">{{pageblock.block.display_name}}</a></b>
         </div>
 
         <div class="col-md-6">
-            <a data-toggle="modal" href="#edit-pageblock-{{block.id}}">{{block.label}}</a>
-            {% rendersummary block %}
+            <a data-toggle="modal" href="#edit-pageblock-{{pageblock.id}}">{{pageblock.label}}</a>
+            {% rendersummary pageblock %}
         </div>
 
         <div class="col-md-3">
             <div class="btn-group">
                 <a data-toggle="modal" class="btn btn-default btn-sm"
-                   href="#edit-pageblock-{{block.id}}" title="Edit">
+                   href="#edit-pageblock-{{pageblock.id}}" title="Edit">
                     <span class="glyphicon glyphicon-edit"></span> edit</a>
 
-                <a href="{% url 'delete-pageblock' block.id %}"
+                <a href="{% url 'delete-pageblock' pageblock.id %}"
                    class="btn btn-danger btn-sm" title="Delete">
                     <span class="glyphicon glyphicon-trash"></span></a>
             </div>
 
             <div class="btn-group">
-                {% if block.block.exportable %}
+                {% if pageblock.block.exportable %}
                 <a class="btn btn-sm"
                    rel="tooltip" title="Download JSON dump of this block"
-                   id="export-{{block.id}}"
-                   href="{% url 'export-pageblock-json' block.id %}">
+                   id="export-{{pageblock.id}}"
+                   href="{% url 'export-pageblock-json' pageblock.id %}">
                     <span class="glyphicon glyphicon-download"></span> export</a>
                 {% endif %}
             </div>
@@ -222,29 +222,29 @@ <h3 style="margin-top: 0;"><a href="{{root.get_edit_url}}">{{section.hierarchy.n
     {% endfor %}
 
 
-    {% for block in section.pageblock_set.all %}
-    <div class="modal fade block-edit" id="edit-pageblock-{{block.id}}" role="dialog">
+    {% for pageblock in section.pageblock_set.all %}
+    <div class="modal fade block-edit" id="edit-pageblock-{{pageblock.id}}" role="dialog">
         <div class="modal-dialog">
             <div class="modal-content">
                 <div class="modal-header">
                     <button type="button" class="close" data-dismiss="modal">×</button>
-                    <h3>Edit {{block.block.display_name}}</h3>
+                    <h3>Edit {{pageblock.block.display_name}}</h3>
                 </div>
                 <div class="modal-body">
 
-                    {% if block.block.importable %}
-                    <a href="{% url 'import-pageblock-json' block.id %}">import json</a>
+                    {% if pageblock.block.importable %}
+                    <a href="{% url 'import-pageblock-json' pageblock.id %}">import json</a>
                     {% endif %}
 
-                    <form action="{% url 'edit-pageblock' block.id %}" method="post"
+                    <form action="{% url 'edit-pageblock' pageblock.id %}" method="post"
                           class=""
-                          {% if block.edit_form.is_multipart %}
+                          {% if pageblock.edit_form.is_multipart %}
                           enctype="multipart/form-data"
                           {% endif %}>
                         {% csrf_token %}
 
-                        {{ block.default_edit_form|bootstrap }}
-                        {% with block.edit_form as ef %}
+                        {{ pageblock.default_edit_form|bootstrap }}
+                        {% with pageblock.edit_form as ef %}
                         {{ ef|bootstrap }}
                         {% if ef.alt_text %}
                         <div>{{ ef.alt_text|safe }}</div>
diff --git a/pagetree/templates/pagetree/page.html b/pagetree/templates/pagetree/page.html
index fa088b9..9eb7714 100644
--- a/pagetree/templates/pagetree/page.html
+++ b/pagetree/templates/pagetree/page.html
@@ -2,14 +2,14 @@
 {% load render %}
 
 {% block js %}
-{% for block in section.pageblock_set.all %}
-{% renderjs block %}
+{% for pageblock in section.pageblock_set.all %}
+{% renderjs pageblock %}
 {% endfor %}
 {% endblock %}
 
 {% block css %}
-{% for block in section.pageblock_set.all %}
-{% rendercss block %}
+{% for pageblock in section.pageblock_set.all %}
+{% rendercss pageblock %}
 {% endfor %}
 {% endblock %}
 
@@ -58,10 +58,10 @@ <h3>Sections</h3>
 {% endif %}
 
 
-{% for block in section.pageblock_set.all %}
-<div class="pageblock{% if block.css_extra %} {{block.css_extra}}{% endif %}">
-{% if block.label %}<h3>{{block.label}}</h3>{% endif %}
-{% render block %}
+{% for pageblock in section.pageblock_set.all %}
+<div class="pageblock{% if pageblock.css_extra %} {{pageblock.css_extra}}{% endif %}">
+{% if pageblock.label %}<h3>{{pageblock.label}}</h3>{% endif %}
+{% render pageblock %}
 </div>
 {% endfor %}
 
diff --git a/pagetree/templates/pagetree/test_page.html b/pagetree/templates/pagetree/test_page.html
index dab2e60..29ff3cb 100644
--- a/pagetree/templates/pagetree/test_page.html
+++ b/pagetree/templates/pagetree/test_page.html
@@ -1,14 +1,14 @@
 {% load render %}
 
 {% block js %}
-{% for block in section.pageblock_set.all %}
-{% renderjs block %}
+{% for pageblock in section.pageblock_set.all %}
+{% renderjs pageblock %}
 {% endfor %}
 {% endblock %}
 
 {% block css %}
-{% for block in section.pageblock_set.all %}
-{% rendercss block %}
+{% for pageblock in section.pageblock_set.all %}
+{% rendercss pageblock %}
 {% endfor %}
 {% endblock %}
 
@@ -55,9 +55,9 @@ <h3>Sections</h3>
 {% endif %}
 
 
-{% for block in section.pageblock_set.all %}
-<div class="pageblock{% if block.css_extra %} {{block.css_extra}}{% endif %}">
-{% if block.label %}<h3>{{block.label}}</h3>{% endif %}
+{% for pageblock in section.pageblock_set.all %}
+<div class="pageblock{% if pageblock.css_extra %} {{pageblock.css_extra}}{% endif %}">
+{% if pageblock.label %}<h3>{{pageblock.label}}</h3>{% endif %}
 {% render block %}
 </div>
 {% endfor %}
diff --git a/pagetree/templates/pagetree/testblock.html b/pagetree/templates/pagetree/testblock.html
index 0185b9e..0893953 100644
--- a/pagetree/templates/pagetree/testblock.html
+++ b/pagetree/templates/pagetree/testblock.html
@@ -1,2 +1,2 @@
 {% load markup %}
-{{block.body|markdown}}
+{{pageblock.body|markdown}}

From 141c42f8ca0aa562c5354dbb74d186a0df3ae841 Mon Sep 17 00:00:00 2001
From: Nik Nyby <nikolas@gnu.org>
Date: Thu, 2 Jul 2015 15:28:57 -0400
Subject: [PATCH 2/2] backwards compatibility for block/pageblock

---
 pagetree/models.py                         | 3 ++-
 pagetree/templates/pagetree/edit_page.html | 4 ++++
 pagetree/templates/pagetree/page.html      | 6 ++++++
 pagetree/templates/pagetree/test_page.html | 6 ++++++
 4 files changed, 18 insertions(+), 1 deletion(-)

diff --git a/pagetree/models.py b/pagetree/models.py
index 94a28e7..c38087a 100644
--- a/pagetree/models.py
+++ b/pagetree/models.py
@@ -701,7 +701,8 @@ def render(self, **kwargs):
         if hasattr(self.content_object, "template_file"):
             t = get_template(getattr(self.content_object, "template_file"))
             d = kwargs
-            d['block'] = self.content_object
+            d['pageblock'] = self.content_object
+            d['block'] = d['pageblock']
             c = Context(d)
             return t.render(c)
         else:
diff --git a/pagetree/templates/pagetree/edit_page.html b/pagetree/templates/pagetree/edit_page.html
index ecff2a3..0603f35 100644
--- a/pagetree/templates/pagetree/edit_page.html
+++ b/pagetree/templates/pagetree/edit_page.html
@@ -180,6 +180,7 @@ <h3 style="margin-top: 0;"><a href="{{root.get_edit_url}}">{{section.hierarchy.n
     <p>Drag the <span class="glyphicon glyphicon-resize-vertical"></span> arrows to reorder pageblocks:</p>
 
     {% for pageblock in section.pageblock_set.all %}
+    {% with block=pageblock %}
     <div class="row block-dragger" id="pageblock-{{pageblock.id}}">
 
         <div class="col-md-1 draghandle">
@@ -219,10 +220,12 @@ <h3 style="margin-top: 0;"><a href="{{root.get_edit_url}}">{{section.hierarchy.n
 
 
     </div>
+    {% endwith %}
     {% endfor %}
 
 
     {% for pageblock in section.pageblock_set.all %}
+    {% with block=pageblock %}
     <div class="modal fade block-edit" id="edit-pageblock-{{pageblock.id}}" role="dialog">
         <div class="modal-dialog">
             <div class="modal-content">
@@ -261,6 +264,7 @@ <h3>Edit {{pageblock.block.display_name}}</h3>
                 </form>
         </div></div>
     </div>
+    {% endwith %}
     {% endfor %}
 
     {% else %}
diff --git a/pagetree/templates/pagetree/page.html b/pagetree/templates/pagetree/page.html
index 9eb7714..81d3233 100644
--- a/pagetree/templates/pagetree/page.html
+++ b/pagetree/templates/pagetree/page.html
@@ -3,13 +3,17 @@
 
 {% block js %}
 {% for pageblock in section.pageblock_set.all %}
+{% with block=pageblock %}
 {% renderjs pageblock %}
+{% endwith %}
 {% endfor %}
 {% endblock %}
 
 {% block css %}
 {% for pageblock in section.pageblock_set.all %}
+{% with block=pageblock %}
 {% rendercss pageblock %}
+{% endwith %}
 {% endfor %}
 {% endblock %}
 
@@ -59,10 +63,12 @@ <h3>Sections</h3>
 
 
 {% for pageblock in section.pageblock_set.all %}
+{% with block=pageblock %}
 <div class="pageblock{% if pageblock.css_extra %} {{pageblock.css_extra}}{% endif %}">
 {% if pageblock.label %}<h3>{{pageblock.label}}</h3>{% endif %}
 {% render pageblock %}
 </div>
+{% endwith %}
 {% endfor %}
 
 
diff --git a/pagetree/templates/pagetree/test_page.html b/pagetree/templates/pagetree/test_page.html
index 29ff3cb..14ffda2 100644
--- a/pagetree/templates/pagetree/test_page.html
+++ b/pagetree/templates/pagetree/test_page.html
@@ -2,13 +2,17 @@
 
 {% block js %}
 {% for pageblock in section.pageblock_set.all %}
+{% with block=pageblock %}
 {% renderjs pageblock %}
+{% endwith %}
 {% endfor %}
 {% endblock %}
 
 {% block css %}
 {% for pageblock in section.pageblock_set.all %}
+{% with block=pageblock %}
 {% rendercss pageblock %}
+{% endwith %}
 {% endfor %}
 {% endblock %}
 
@@ -56,10 +60,12 @@ <h3>Sections</h3>
 
 
 {% for pageblock in section.pageblock_set.all %}
+{% with block=pageblock %}
 <div class="pageblock{% if pageblock.css_extra %} {{pageblock.css_extra}}{% endif %}">
 {% if pageblock.label %}<h3>{{pageblock.label}}</h3>{% endif %}
 {% render block %}
 </div>
+{% endwith %}
 {% endfor %}