Fixed #76 to print license text once in the attribrition output.

Author: chinyeungli

about_code_tool/about.py

@@ -929,6 +929,16 @@ def notice_text(self):
 
         return ""  # Returns empty string if the notice file does not exist
 
+    def dje_license(self):
+        """
+        Return the dje_license value if the dje_license field exists
+        """
+        try:
+            return self.parsed['dje_license']
+        except Exception as e:
+            pass
+
+        return ""  # Returns empty string if the dje_license key does not exist
 
 class AboutCollector(object):
     """
@@ -1071,15 +1081,20 @@ def generate_attribution(self, template_path='templates/default.html',
         validated_fields = []
         license_text = []
         notice_text = []
+        unique_license = []
         for about_object in self:
             if not limit_to or about_object.about_resource_path in limit_to:
                 validated_fields.append(about_object.validated_fields)
-                license_text.append(about_object.license_text())
                 notice_text.append(about_object.notice_text())
+                if not about_object.dje_license() in unique_license \
+                    and not about_object.dje_license() == None:
+                    unique_license.append(about_object.dje_license())
+                    license_text.append(about_object.license_text())
 
         return template.render(about_objects=validated_fields,
                                license_texts=license_text,
-                               notice_texts=notice_text)
+                               notice_texts=notice_text,
+                               unique_licenses=unique_license)
 
 
 USAGE_SYNTAX = """\

about_code_tool/templates/default.html

@@ -55,7 +55,7 @@ <h3 class="component-name">{{ about_object.name }}{% if about_object.version %}
                 {% endif %}
                 {% if about_object.dje_license %} 
                     <p>Full text of
-                        <a class="{{ about_object.dje_license }}" href="#component-license-{{ loop.index0 }}">
+                        <a class="{{ about_object.dje_license }}" href="#component-license-{{ about_object.dje_license }}">
                          {{ about_object.dje_license }}
                          </a>
                             is available at the end of this document.</p>
@@ -65,12 +65,10 @@ <h3 class="component-name">{{ about_object.name }}{% if about_object.version %}
         <hr>
 
         <h3>Licenses Used in This Product</h3>
-            {% for index in range(about_objects | count) %}
-                {% if about_objects[index].dje_license %}
-                <h3 id="component-license-{{ index }}">{{ about_objects[index].dje_license }}</h3>
-                <pre>{{ license_texts[index] }}</pre>
-                {% endif %}
-            {% endfor %}
+        {% for index in range(unique_licenses | count) %}
+            <h3 id="component-license-{{ unique_licenses[index] }}">{{ unique_licenses[index] }}</h3>
+            <pre>{{ license_texts[index] }}</pre>
+        {% endfor %}
         <h3><a id="End">End</a></h3>
     </body>
 </html>