Skip to content

Commit

Permalink
Remove two more articles|sort constructs.
Browse files Browse the repository at this point in the history
This is the issue getpelican#483 which was
supposed to be fixed by getpelican#506 (and commit
f1a4bba), but two occurences of
the problem were missed.

Basically, as
getpelican/pelican#1591 (comment)
says:

> For future reference: Don't use sort in templates on articles.
  • Loading branch information
mcepl committed Sep 24, 2018
1 parent 5d3f1ee commit 22bb2cf
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 3 deletions.
2 changes: 1 addition & 1 deletion uikit/templates/articles.html
Original file line number Diff line number Diff line change
Expand Up @@ -12,7 +12,7 @@
<h1>{{'all the articles'|capitalize}}</h1>

<ul>
{% for article in articles|sort %}
{% for article in articles %}
<li><a class="{% if CAPITALIZE_HEADINGS %}capitalize{% endif %}" href="/{{ article.url }}">{{ article.title }}</a></li>
{% endfor %}
</ul>
Expand Down
4 changes: 2 additions & 2 deletions zurb-F5-basic/templates/tags.html
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@
{% block content %}

<ul class="circle">
{% for tag, articles in tags|sort %}
{% for tag, articles in tags %}
<li><a href="{{ SITEURL }}/{{ tag.url }}">{{ tag }}</a></li>
<ul class="square">
{% for article in articles %}
Expand All @@ -12,4 +12,4 @@
{% endfor %}
</ul>

{% endblock %}
{% endblock %}

0 comments on commit 22bb2cf

Please sign in to comment.