Render a single day event's year once

Author: jefftriplett

apps/events/templates/events/includes/time_tag.html

@@ -4,10 +4,6 @@
             {{ next_time.dt_start|date:"Y" }}
         </span>
 
-        <span id="start-{{ object.id }}"{% if scheduled_end_this_year %} class="say-no-more"{% endif %}>
-            {{ next_time.dt_start|date:"Y" }}
-        </span>
-
         {% if not next_time.all_day %}
             {{ next_time.dt_start|date:"fA"|lower }} {{ next_time.dt_start|date:"e" }}
                 {% if next_time.valid_dt_end %} – {{ next_time.dt_end|date:"fA"|lower }}

apps/events/tests/test_views.py

@@ -233,6 +233,13 @@ def test_event_ending_future_year_displays_relevant_year(self):
         response = self.client.get(url)
         self.assertIn(f'<span id="end-{event.id}">', response.content.decode())
 
+    def test_single_day_event_renders_its_year_once(self):
+        """A single day event rendered the same start year twice. See #2626."""
+        event = self.event_single_day
+        response = self.client.get(reverse("events:events"))
+
+        self.assertEqual(response.content.decode().count(f'id="start-{event.id}"'), 1)
+
     def test_events_scheduled_current_year_does_not_display_current_year(self):
         event = self.event_single_day
         url = reverse("events:events")