Resolve #5087 "Meine Veranstaltungen verlinkt noch auf seminar_main.php"

Closes #5087

Merge request studip/studip!3802

app/controllers/course/go.php

@@ -50,7 +50,9 @@ class Course_GoController extends AuthenticatedController
             if (!is_internal_url($redirect_to)) {
                 throw new Exception('Invalid redirection');
             }
-
+            if (str_starts_with($redirect_to, '#')) {
+                $redirect_to = 'dispatch.php/course/go' . $redirect_to;
+            }
             $this->redirect(URLHelper::getURL($redirect_to, ['cid' => $course_id]));
             return;
         }

app/controllers/institute/overview.php

@@ -16,17 +16,20 @@ class Institute_OverviewController extends AuthenticatedController
 {
     protected $allow_nobody = true;
 
-    function before_filter(&$action, &$args) {
-
+    public function __construct(\Trails\Dispatcher $dispatcher)
+    {
+        if (Request::option('auswahl')) {
+            Request::set('cid', Request::option('auswahl'));
+        }
         //Check if anonymous access is really allowed:
         $config = Config::get();
         if (($config->ENABLE_FREE_ACCESS && ($config->ENABLE_FREE_ACCESS == 'courses_only'))) {
             $this->allow_nobody = false;
         }
+        parent::__construct($dispatcher);
+    }
 
-        if (Request::option('auswahl')) {
-            Request::set('cid', Request::option('auswahl'));
-        }
+    public function before_filter(&$action, &$args) {
 
         parent::before_filter($action, $args);
 

resources/vue/components/MyCoursesTables.vue

@@ -56,7 +56,7 @@
                         {{ course.number }}
                     </td>
                     <td :class="{'subcourse-indented': isChild(course)}">
-                        <a :href="urlFor('seminar_main.php', {to: course.id})">
+                        <a :href="urlFor('dispatch.php/course/go', {to: course.id})">
                             {{ getCourseName(course, getConfig('sem_number') && responsiveDisplay) }}
                             <span v-if="course.is_deputy">{{ $gettext('[Vertretung]') }}</span>
                         </a>

resources/vue/components/MyCoursesTiles.vue

@@ -24,7 +24,7 @@
                                     ></studip-action-menu>
                                 </span>
 
-                                <a :href="urlFor('seminar_main.php', {to: course.id})" class="tiles-grid-element-header-content" :title="getCourseName(course, getConfig('sem_number'))">
+                                <a :href="urlFor('dispatch.php/course/go', {to: course.id})" class="tiles-grid-element-header-content" :title="getCourseName(course, getConfig('sem_number'))">
                                     <span :style="{backgroundImage: `url(${course.avatar})`}" class="tiles-grid-element-header-image"></span>
                                     <span class="tiled-grid-element-header-title">
                                         {{ getCourseName(course, getConfig('sem_number')) }}

resources/vue/mixins/MyCoursesMixin.js

@@ -141,7 +141,7 @@ export default {
                         result.icon.role = 'clickable';
                     }
 
-                    result.url = this.urlFor('seminar_main.php', {
+                    result.url = this.urlFor('dispatch.php/course/go', {
                         to: course.id,
                         redirect_to: result.url,
                     });