[Rt-commit] rt branch, 3.8-trunk, updated. rt-3.8.8-146-ga76de24

[Emannuel Lacour]

The branch, 3.8-trunk has been updated
       via  a76de24d2d62e0a063ff672dc8a132e423d32797 (commit)
      from  79f29ac88e030084e52808f841a8f9ad645988ea (commit)

Summary of changes:
 share/html/SelfService/Elements/Tabs |    6 +++---
 1 files changed, 3 insertions(+), 3 deletions(-)

- Log -----------------------------------------------------------------
commit a76de24d2d62e0a063ff672dc8a132e423d32797
Author: Emmanuel Lacour <elacour at easter-eggs.com>
Date:   Mon Sep 6 14:47:25 2010 +0200

    Reverse createticket logic in SelfService:
    
    * if user has CreateTicket right on only one queue, then go directly to this queue
    * else, see the list of available queues so the user choose the one he needs
    
    this fixes the case when there is CreateTicket right on several queues, but
    ShowTicket right on none (create ticket only be email). Before this patch, user
    can always create a ticket in an arbitrary queue with id=1.

diff --git a/share/html/SelfService/Elements/Tabs b/share/html/SelfService/Elements/Tabs
index 36ac3bf..3017a5d 100755
--- a/share/html/SelfService/Elements/Tabs
+++ b/share/html/SelfService/Elements/Tabs
@@ -83,13 +83,13 @@ my $tabs = { A  => { title => loc('Open tickets'),
                        },
            };
 
-if ($queue_count > 1) {
+if ($queue_count == 1) {
         $tabs->{C} = { title => loc('New ticket'),
-                       path => 'SelfService/CreateTicketInQueue.html'
+                       path => 'SelfService/Create.html?Queue=' . $queue_id
                        };
 } else {
         $tabs->{C} = { title => loc('New ticket'),
-                       path => 'SelfService/Create.html?Queue=' . $queue_id
+                       path => 'SelfService/CreateTicketInQueue.html'
                        };
 }
 

-----------------------------------------------------------------------