Update metadata_display plugin for Python 3.
authorBen Sturmfels <ben@sturm.com.au>
Wed, 18 Sep 2019 05:30:32 +0000 (15:30 +1000)
committerBen Sturmfels <ben@sturm.com.au>
Wed, 18 Sep 2019 05:30:32 +0000 (15:30 +1000)
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.

mediagoblin/plugins/metadata_display/templates/mediagoblin/plugins/metadata_display/metadata_table.html

index 15ea15368785ba8f9e60c362ce96d42f2de66247..6fc4621289448425630410e53bfba95906518b65 100644 (file)
@@ -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>