From: Ben Sturmfels <ben@sturm.com.au>
Date: Wed, 18 Sep 2019 05:30:32 +0000 (+1000)
Subject: Update metadata_display plugin for Python 3.
X-Git-Url: https://vcs.fsf.org/?a=commitdiff_plain;h=25bc7ea7f69597f14e39531ba6d6e2518af58d0a;p=mediagoblin.git

Update metadata_display plugin for Python 3.

Switching from iteritems() to items() is technically less memory efficient on
Python 2, but this appeared be a significantly cleaner approach than adding six
to the template context. It's also the right choice assuming the project will at
some point go Python 3 only.
---

diff --git a/mediagoblin/plugins/metadata_display/templates/mediagoblin/plugins/metadata_display/metadata_table.html b/mediagoblin/plugins/metadata_display/templates/mediagoblin/plugins/metadata_display/metadata_table.html
index 15ea1536..6fc46212 100644
--- a/mediagoblin/plugins/metadata_display/templates/mediagoblin/plugins/metadata_display/metadata_table.html
+++ b/mediagoblin/plugins/metadata_display/templates/mediagoblin/plugins/metadata_display/metadata_table.html
@@ -23,7 +23,7 @@
   {#- NOTE: In some smart future where the context is more extensible,
         we will need to add to the prefix here-#}
   <table class="metadata_info">
-    {%- for key, value in metadata.iteritems() if not key=='@context' %}
+    {%- for key, value in metadata.items() if key != '@context' %}
       {% if value -%}
         <tr>
           <th>{{ rdfa_to_readable(key) }}</th>