- always call logout action as task (#1485919)

git-svn-id: https://svn.roundcube.net/trunk@2645 208e9e7b-5314-0410-a742-e7e81cd9613c

3 files changed, 3 insertions, 9 deletions

diff --git a/roundcubemail/index.php b/roundcubemail/index.php
index 5d37a93e0..e8111b113 100644
--- a/roundcubemail/index.php
+++ b/roundcubemail/index.php
@@ -121,7 +121,7 @@ if ($RCMAIL->action=='login' && $RCMAIL->task=='mail') {
 }
 
 // end session
-else if (($RCMAIL->task=='logout' || $RCMAIL->action=='logout') && isset($_SESSION['user_id'])) {
+else if ($RCMAIL->task=='logout' && isset($_SESSION['user_id'])) {
   $OUTPUT->show_message('loggedout');
   $RCMAIL->logout_actions();
   $RCMAIL->kill_session();
diff --git a/roundcubemail/program/include/rcube_template.php b/roundcubemail/program/include/rcube_template.php
index 3d0e7390a..455bbcfd2 100755
--- a/roundcubemail/program/include/rcube_template.php
+++ b/roundcubemail/program/include/rcube_template.php
@@ -969,7 +969,7 @@ class rcube_template extends rcube_html_page
         
         // save original url
         $url = get_input_value('_url', RCUBE_INPUT_POST);
-        if (empty($url) && !preg_match('/_action=logout/', $_SERVER['QUERY_STRING']))
+        if (empty($url) && !preg_match('/_(task|action)=logout/', $_SERVER['QUERY_STRING']))
             $url = $_SERVER['QUERY_STRING'];
 
         $input_user   = new html_inputfield(array('name' => '_user', 'id' => 'rcmloginuser', 'size' => 30) + $attrib);
diff --git a/roundcubemail/program/js/app.js b/roundcubemail/program/js/app.js
index 218351ca3..216bebc80 100644
--- a/roundcubemail/program/js/app.js
+++ b/roundcubemail/program/js/app.js
@@ -367,9 +367,6 @@ function rcube_webmail()
         break;
       }
 
-    // enable basic commands
-    this.enable_command('logout', true);
-
     // flag object as complete
     this.loaded = true;
 
@@ -551,14 +548,11 @@ function rcube_webmail()
           this.gui_objects.loginform.submit();
         break;
 
-      case 'logout':
-        this.goto_url('logout', '', true);
-        break;
-
       // commands to switch task
       case 'mail':
       case 'addressbook':
       case 'settings':
+      case 'logout':
         this.switch_task(command);
         break;