Changing logout URL

config/routing.yml

@@ -1,9 +1,11 @@
-admin_login_check:
-    pattern:   /admin/login_check
+secured_login_check:
+    pattern:   /secured/login_check
 
-admin_logout:
-    pattern:   /admin/logout
+secured_logout:
+    pattern:   /secured/logout
 
+logout:
+    pattern:   /secured/logout
 home:
     path:      /home
     defaults:  { _controller: index.controller:indexAction, culture: en }

main/exercice/exercise.class.php

@@ -5904,7 +5904,7 @@ class Exercise
                 break;
             case '2':
                 global $app;
-                $url = $app['url_generator']->generate('admin_logout');
+                $url = $app['url_generator']->generate('logout');
                 $html = Display::url(get_lang('Logout'), $url, array('class' => 'btn btn-large'));
                 break;
         }

main/template/default/auth/login.tpl

@@ -22,7 +22,7 @@
         </script>
     {% endif %}
 
-    <form class="form-signin" action="{{ url('admin_login_check') }}" method="post">
+    <form class="form-signin" action="{{ url('secured_login_check') }}" method="post">
         <h2 class="form-signin-heading">{{ 'SignIn' | get_lang }}</h2>
         {% if error %}
             <div class="alert">

main/template/default/layout/menu.tpl

@@ -48,7 +48,7 @@
                         </li>
                     {% endif %}
                     <li>
-                        <a id="logout_button" class="logout" title="{{ "Logout"|get_lang }}" href="{{ url('admin_logout') }}" >
+                        <a id="logout_button" class="logout" title="{{ "Logout"|get_lang }}" href="{{ url('logout') }}" >
                             <img src="{{ "exit.png"|icon(22) }}">
                         </a>
                     </li>

main/template/default/layout/page_body.tpl

@@ -1,4 +1,4 @@
-{#  Actions  #}
+{# Actions  #}
 {% if actions != '' %}
     <div class="actions">
         {{ actions }}
@@ -9,21 +9,21 @@
     {{ knp_menu_render('actions_menu', { 'currentClass': 'active'}) }}
 {% endif %}
 
-{#  Page header #}
+{# Page header #}
 {% if header != '' %}
     <div class="page-header">
         <h1>{{ header }}</h1>
     </div>
 {% endif %}
 
-
-{% if is_granted('ROLE_PREVIOUS_ADMIN') %}
+{# Check if security exists #}
+{% if app.security.token and is_granted('ROLE_PREVIOUS_ADMIN') %}
     <div class="alert">
         <a href="{{ path('index', {'_switch_user': '_exit'}) }}">{{ 'Exit impersonation' | trans }}</a>
     </div>
 {% endif %}
 
-{#  Show messages #}
+{# Show messages #}
 {% if message != '' %}
     <section id="messages">
         {{ message}}
@@ -32,7 +32,7 @@
 
 {% include app.template_style ~ "/layout/messages.tpl" %}
 
-{#  Welcome to course block  #}
+{# Welcome to course block  #}
 {% if welcome_to_course_block %}
     <section id="welcome_to_course">
         {% include app.template_style ~ "/layout/welcome_to_course.tpl" %}

src/ChamiloLMS/Controller/IndexController.php

@@ -236,7 +236,7 @@ class IndexController extends CommonController
           ->getForm();
           return $app['template']->assign('form', $form->createView());
          */
-        $form = new \FormValidator('formLogin', 'POST', $app['url_generator']->generate('admin_login_check'), null, array('class' => 'form-vertical'));
+        $form = new \FormValidator('formLogin', 'POST', $app['url_generator']->generate('secured_login_check'), null, array('class' => 'form-vertical'));
         $form->addElement(
             'text',
             'username',