From 99f3a2a37d9b54b7f41eacec269d117a1182f434 Mon Sep 17 00:00:00 2001
From: girst <tobi@isticktoit.net>
Date: Wed, 19 Aug 2020 19:22:26 +0200
Subject: [PATCH] add final / to /channel/ urls in templates

TECHDEBT: this should just use url_for!
---
 app/dangerous/templates/playlist.html.j2           | 2 +-
 app/templates/macros.imp.j2                        | 4 ++--
 app/youtube/templates/subscription_manager.html.j2 | 2 +-
 app/youtube/templates/watch.html.j2                | 2 +-
 4 files changed, 5 insertions(+), 5 deletions(-)

diff --git a/app/dangerous/templates/playlist.html.j2 b/app/dangerous/templates/playlist.html.j2
index 8c8c0e7..82083a1 100644
--- a/app/dangerous/templates/playlist.html.j2
+++ b/app/dangerous/templates/playlist.html.j2
@@ -4,7 +4,7 @@
 
 {% block content %}
 <h1>{{ title | e }}<br>
-<small><a href="/channel/{{ channel_id }}">{{ author | e }}</a></small></h1>
+<small><a href="/channel/{{ channel_id }}/">{{ author | e }}</a></small></h1>
 <div class="cards">
         {# TODO: copy pasted from dangerous/search.j2 #}
         {% for card in rows %}
diff --git a/app/templates/macros.imp.j2 b/app/templates/macros.imp.j2
index 98b3590..b56087c 100644
--- a/app/templates/macros.imp.j2
+++ b/app/templates/macros.imp.j2
@@ -45,7 +45,7 @@
                         {{ infobar_subscriptions(c.video_id, c.channel_id, c.author) }}
                 {% endcall %}
         {% elif params.type == 'CHANNEL' %}
-                {% call card_generic("/channel/"~c.channel_id, c.icons[c.icons.largest] if 'icons' in c else '', c.title) %}
+                {% call card_generic("/channel/"~c.channel_id~"/", c.icons[c.icons.largest] if 'icons' in c else '', c.title) %}
                         <span class=channel>Channel</span>
                         <span class=advanced>{{ c.subscribers }}</span>
                 {% endcall %}
@@ -69,7 +69,7 @@
 
 {% macro infobar_subscriptions(video_id, channel_id, author) -%}
 	{% if channel_id is defined and channel_id is not none %}
-	<a class="info-main" href="/channel/{{ channel_id }}">{{ author | e }}</a>
+	<a class="info-main" href="/channel/{{ channel_id }}/">{{ author | e }}</a>
 	{% else %}
 	<span class="info-main">{{ author | e }}</span>
 	{% endif %}
diff --git a/app/youtube/templates/subscription_manager.html.j2 b/app/youtube/templates/subscription_manager.html.j2
index 642faaa..7e8c7d9 100644
--- a/app/youtube/templates/subscription_manager.html.j2
+++ b/app/youtube/templates/subscription_manager.html.j2
@@ -16,7 +16,7 @@
 	{% for row in rows %}
 	<li>{{ macros.emoji_button("subscribe", row.channel_id, True) }}
 		{% if row.type == "channel" %}
-		<a href="/channel/{{ row.channel_id }}">{{ row.author | e }}</a>
+		<a href="/channel/{{ row.channel_id }}/">{{ row.author | e }}</a>
 		{% elif row.type == "playlist" %}
 		<a href="/playlist?list={{ row.channel_id }}">{{ row.author | e }}</a> <small>(playlist)</small>
 		{% endif %}
diff --git a/app/youtube/templates/watch.html.j2 b/app/youtube/templates/watch.html.j2
index 359351d..d317cec 100644
--- a/app/youtube/templates/watch.html.j2
+++ b/app/youtube/templates/watch.html.j2
@@ -53,7 +53,7 @@
 {% endif %}
 
 <h1>{{ title | e }}<br>
-<small><a href="/channel/{{ channel_id }}">{{ author | e }}</a></small></h1>
+<small><a href="/channel/{{ channel_id }}/">{{ author | e }}</a></small></h1>
 
 <details><summary>Description</summary>
 	<p style="white-space:pre-wrap">{{ description | e }}
-- 
2.39.3