From 293bb5d4ab60452c16be1ce219a850a160685e20 Mon Sep 17 00:00:00 2001
From: Sebastian Serth <Sebastian.Serth@hpi.de>
Date: Thu, 14 Dec 2023 00:05:05 +0100
Subject: [PATCH] Switch to proper, dynamic route for SortableJS elements

---
 app/assets/javascripts/exercise_collections.js.erb | 2 +-
 app/assets/javascripts/exercises.js.erb            | 2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/app/assets/javascripts/exercise_collections.js.erb b/app/assets/javascripts/exercise_collections.js.erb
index d0aad22e..154f82ff 100644
--- a/app/assets/javascripts/exercise_collections.js.erb
+++ b/app/assets/javascripts/exercise_collections.js.erb
@@ -174,7 +174,7 @@ $(document).on('turbolinks:load', function() {
           var template = '<tr data-id="' + exercise.id + '">' +
             '<td><span class="fa-solid fa-bars"></span></td>' +
             '<td>' + exercise.title + '</td>' +
-            '<td><a href="/exercises/' + exercise.id + '"><%= I18n.t('shared.show') %></td>' +
+            `<td><a href="${Routes.exercise_path(exercise.id)}"><%= I18n.t('shared.show') %></td>` +
             '<td><a class="remove-exercise" href="#"><%= I18n.t('shared.destroy') %></td></tr>';
           exerciseList.find('tbody').append(template);
           $('#exercise-list').find('option[value="' + exercise.id + '"]').prop('selected', true);
diff --git a/app/assets/javascripts/exercises.js.erb b/app/assets/javascripts/exercises.js.erb
index 0244cdbe..dee4a82e 100644
--- a/app/assets/javascripts/exercises.js.erb
+++ b/app/assets/javascripts/exercises.js.erb
@@ -236,7 +236,7 @@ $(document).on('turbolinks:load', function () {
             const template =
                 '<div class="list-group-item d-block" data-tip-id=' + tip.id + ' data-id="">' +
                 '<span class="fa-solid fa-bars me-3"></span>' + tip.title +
-                '<a class="fa-regular fa-eye ms-2" href="/tips/' + tip.id + '" target="_blank"></a>' +
+                `<a class="fa-regular fa-eye ms-2" href="${Routes.tip_path(tip.id)}" target="_blank"></a>` +
                 '<a class="fa-solid fa-xmark ms-2 remove-tip" href="#""></a>' +
                 '<div class="list-group nested-sortable-list"></div>' +
                 '</div>';