From bea48c514818b351bfdaa07cdc0827b4911bc910 Mon Sep 17 00:00:00 2001
From: John Carroll <jcarroll@Mac.fios-router.home>
Date: Wed, 25 Mar 2026 11:22:13 -0400
Subject: [PATCH 1/3] filter to remove redundancy btwn doc desc and doc lbl

---
 fec/data/templatetags/filters.py     | 21 +++++++++++++++++++++
 fec/legal/templates/rulemaking.jinja | 26 +++++++++++++++-----------
 2 files changed, 36 insertions(+), 11 deletions(-)

diff --git a/fec/data/templatetags/filters.py b/fec/data/templatetags/filters.py
index db7d9be048..4a5c9546af 100644
--- a/fec/data/templatetags/filters.py
+++ b/fec/data/templatetags/filters.py
@@ -4,6 +4,7 @@
 import re
 
 from dateutil.parser import parse as parse_date
+from difflib import SequenceMatcher
 
 from django.conf import settings
 from django_jinja import library
@@ -164,6 +165,26 @@ def filesize(value):
     return '%d %s' % (value, units[unit])
 
 
+@library.filter
+def compare(string, string_1):
+    """Compares two strings to determine if they are nearly identical or semantically similar. 
+    Returns False if `string` is >= than 50% similar to `string_1`. Else returns True.
+    Uses difflib.SequenceMatcher: https://docs.python.org/3/library/difflib.html
+    """
+
+    # Calculate the similarity ratio
+    similarity_ratio = SequenceMatcher(None, string, string_1).ratio()
+
+    # Define a threshold for redundancy (e.g., 50% similar)
+    threshold = .5
+
+    if similarity_ratio >= threshold:
+        return False
+        
+    else:
+        return True
+
+
 @library.global_function
 def path_for_css(key):
     """Looks up the hashed asset key in rev-manifest-css.json
diff --git a/fec/legal/templates/rulemaking.jinja b/fec/legal/templates/rulemaking.jinja
index c0ad863c57..c6b1f494bd 100644
--- a/fec/legal/templates/rulemaking.jinja
+++ b/fec/legal/templates/rulemaking.jinja
@@ -71,16 +71,15 @@
       <ul>
         <li>
           {%- if docL1.url -%}
-          <a class="embed-pdf" href="{{ CANONICAL_BASE }}{{ docL1.url }}" data-doc_id="{{ docL1.doc_id }}" target="_blank">{{ docL1.label }}</a>
+           <a class="embed-pdf" href="{{ CANONICAL_BASE }}{{ docL1.url }}" data-doc_id="{{ docL1.doc_id }}" target="_blank">{{ docL1.label }}</a>
           {%- else -%}
-          {{ docL1.label }}
+           {{ docL1.label }}
           {%- endif -%}
-          {%- if docL1.doc_description and 'pdf' not in docL1.doc_description.lower() -%}
-            <br>{{ docL1.doc_description }}
+          {%- if docL1.doc_description and 'pdf' not in docL1.doc_description.lower() and docL1.doc_description | compare(docL1.label) -%}
+           <br>{{ docL1.doc_description }}
           {%- endif -%}
         </li>
           {%- if docL1.doc_entities -%}
-
                 {%- for entity in docL1.doc_entities -%}
                   <li class="indent">{{ entity.name }}, {{ entity.role }}</li>
                 {%- endfor -%}
@@ -105,7 +104,7 @@
           <ul>
             {% if (doc.label == 'Comments' or doc.label == 'Hearing Schedule') and doc.doc_entities %}
               {% if doc.label == 'Hearing Schedule' %}
-                <li><a class="embed-pdf" href="{{ CANONICAL_BASE }}{{ doc.url }}" data-doc_id="{{ doc.doc_id }}" target="_blank">{{ doc.label }}</a></li>
+               <li><a class="embed-pdf" href="{{ CANONICAL_BASE }}{{ doc.url }}" data-doc_id="{{ doc.doc_id }}" target="_blank">{{ doc.label }}</a></li>
                 {% for entity in doc.doc_entities %}
                   <li class="indent">{{ entity.name }}, {{ entity.role }}</li>
                 {% endfor %}
@@ -114,8 +113,10 @@
                 {% if doc.commenters|length == 1 %}
                   {# Single commenter: show commenter name #}
                   <li>
-                  <a class="embed-pdf" href="{{ CANONICAL_BASE }}{{ doc.url }}" data-doc_id="{{ doc.doc_id }}" target="_blank">{{ doc.commenters[0].name }} | {{ doc.commenters[0].role }}</a>
-                  {% if doc.doc_description and 'pdf' not in doc.doc_description.lower() %}<br>{{ doc.doc_description }}{% endif %}
+                    <a class="embed-pdf" href="{{ CANONICAL_BASE }}{{ doc.url }}" data-doc_id="{{ doc.doc_id }}" target="_blank">{{ doc.commenters[0].name }} | {{ doc.commenters[0].role }}</a>
+                    {% if doc.doc_description and 'pdf' not in doc.doc_description.lower() and doc.doc_description | compare(doc.label)  %}
+                     <br>{{ doc.doc_description }} 
+                    {% endif %}
                   </li>
                   {% for entity in doc.doc_entities %}
                     <li class="indent">{{ entity.name }}, {{ entity.role }}</li>
@@ -123,7 +124,10 @@
                 {% else %}
                   {# Multiple commenters: show "Joint comment" #}
                   <li><a class="embed-pdf" href="{{ CANONICAL_BASE }}{{ doc.url }}" data-doc_id="{{ doc.doc_id }}" target="_blank">Joint comment</a>
-                  {% if doc.doc_description and 'pdf' not in doc.doc_description.lower() %}<br>{{ doc.doc_description }}{% endif %}</li>
+                    {% if doc.doc_description and 'pdf' not in doc.doc_description.lower() and doc.doc_description | compare(doc.label) %}
+                      <br>{{ doc.doc_description }} 
+                    {% endif %}
+                  </li>
                   {% for entity in doc.doc_entities %}
                     <li class="indent">{{ entity.name }}, {{ entity.role }}</li>
                   {% endfor %}
@@ -131,8 +135,8 @@
               {% endif %}
             {% else %}
               <a class="embed-pdf" href="{{ CANONICAL_BASE }}{{ doc.url }}" data-doc_id="{{ doc.doc_id }}" target="_blank">{{ doc.label }}</a>
-              {% if doc.doc_description and 'pdf' not in doc.doc_description.lower() %}
-                <br>{{ doc.doc_description }}
+              {% if doc.doc_description and 'pdf' not in doc.doc_description.lower() and doc.doc_description | compare(doc.label)  %}
+               <br>{{ doc.doc_description }} 
               {% endif %}
             {% endif %}
           </ul>

From 9b8129a9670340e5e6c46c90a7d8986fa38244ae Mon Sep 17 00:00:00 2001
From: John Carroll <jcarroll@Mac.fios-router.home>
Date: Thu, 26 Mar 2026 11:15:28 -0400
Subject: [PATCH 2/3] handle redudancy in commenter names

---
 fec/data/templatetags/filters.py     | 15 +++++++++++++++
 fec/legal/templates/rulemaking.jinja |  4 ++--
 2 files changed, 17 insertions(+), 2 deletions(-)

diff --git a/fec/data/templatetags/filters.py b/fec/data/templatetags/filters.py
index 4a5c9546af..ea9de83cfc 100644
--- a/fec/data/templatetags/filters.py
+++ b/fec/data/templatetags/filters.py
@@ -185,6 +185,21 @@ def compare(string, string_1):
         return True
 
 
+@library.filter
+def compare_commenter(string, string_1):
+    """Convert `string_1` from "LastName, FirstName" format to "FirstName LastName" 
+    Returns False if `converted_string_1` is in `string`. Else returns True.
+    """
+
+    # Convert name from "LastName, FirstName" format to "FirstName LastName"
+    converted_string_1 = ' '.join(reversed(string_1.split(', ')))
+
+    if converted_string_1 in string:
+        return False
+    else:
+        return True
+
+
 @library.global_function
 def path_for_css(key):
     """Looks up the hashed asset key in rev-manifest-css.json
diff --git a/fec/legal/templates/rulemaking.jinja b/fec/legal/templates/rulemaking.jinja
index c6b1f494bd..cdea9f1c6d 100644
--- a/fec/legal/templates/rulemaking.jinja
+++ b/fec/legal/templates/rulemaking.jinja
@@ -114,7 +114,7 @@
                   {# Single commenter: show commenter name #}
                   <li>
                     <a class="embed-pdf" href="{{ CANONICAL_BASE }}{{ doc.url }}" data-doc_id="{{ doc.doc_id }}" target="_blank">{{ doc.commenters[0].name }} | {{ doc.commenters[0].role }}</a>
-                    {% if doc.doc_description and 'pdf' not in doc.doc_description.lower() and doc.doc_description | compare(doc.label)  %}
+                    {% if doc.doc_description and 'pdf' not in doc.doc_description.lower() and doc.doc_description | compare_commenter(doc.commenters[0].name)  %}
                      <br>{{ doc.doc_description }} 
                     {% endif %}
                   </li>
@@ -124,7 +124,7 @@
                 {% else %}
                   {# Multiple commenters: show "Joint comment" #}
                   <li><a class="embed-pdf" href="{{ CANONICAL_BASE }}{{ doc.url }}" data-doc_id="{{ doc.doc_id }}" target="_blank">Joint comment</a>
-                    {% if doc.doc_description and 'pdf' not in doc.doc_description.lower() and doc.doc_description | compare(doc.label) %}
+                    {% if doc.doc_description and 'pdf' not in doc.doc_description.lower() %}
                       <br>{{ doc.doc_description }} 
                     {% endif %}
                   </li>

From 57fa8ddd1b48a65d985e27aa7f812ea4191c6582 Mon Sep 17 00:00:00 2001
From: John Carroll <jcarroll@Mac.fios-router.home>
Date: Fri, 27 Mar 2026 02:20:32 -0400
Subject: [PATCH 3/3] handle third repetition of commenter name and rm doc desc
 when its just the word Comment

---
 fec/data/templatetags/filters.py     | 6 ++++--
 fec/legal/templates/rulemaking.jinja | 2 +-
 2 files changed, 5 insertions(+), 3 deletions(-)

diff --git a/fec/data/templatetags/filters.py b/fec/data/templatetags/filters.py
index ea9de83cfc..eb05abd615 100644
--- a/fec/data/templatetags/filters.py
+++ b/fec/data/templatetags/filters.py
@@ -188,13 +188,15 @@ def compare(string, string_1):
 @library.filter
 def compare_commenter(string, string_1):
     """Convert `string_1` from "LastName, FirstName" format to "FirstName LastName" 
-    Returns False if `converted_string_1` is in `string`. Else returns True.
+    Returns False if `converted_string_1` is in `string`.
+    Returns False if `string` == 'Comment'.
+    Else returns True.
     """
 
     # Convert name from "LastName, FirstName" format to "FirstName LastName"
     converted_string_1 = ' '.join(reversed(string_1.split(', ')))
 
-    if converted_string_1 in string:
+    if converted_string_1 in string or string == 'Comment':
         return False
     else:
         return True
diff --git a/fec/legal/templates/rulemaking.jinja b/fec/legal/templates/rulemaking.jinja
index cdea9f1c6d..a62577b424 100644
--- a/fec/legal/templates/rulemaking.jinja
+++ b/fec/legal/templates/rulemaking.jinja
@@ -118,7 +118,7 @@
                      <br>{{ doc.doc_description }} 
                     {% endif %}
                   </li>
-                  {% for entity in doc.doc_entities %}
+                  {% for entity in doc.doc_entities if entity.role != 'Commenter' %}
                     <li class="indent">{{ entity.name }}, {{ entity.role }}</li>
                   {% endfor %}
                 {% else %}