[Rt-commit] r4819 - in rtir/branches/1.9-EXPERIMENTAL: html/RTIR/Elements

[ruz at bestpractical.com]

Author: ruz
Date: Fri Mar 24 09:04:53 2006
New Revision: 4819

Modified:
   rtir/branches/1.9-EXPERIMENTAL/   (props changed)
   rtir/branches/1.9-EXPERIMENTAL/html/RTIR/Elements/ShowIncidents

Log:
 r1008 at cubic-pc:  cubic | 2006-03-24 17:09:30 +0300
 * don't change status on steal


Modified: rtir/branches/1.9-EXPERIMENTAL/html/RTIR/Elements/ShowIncidents
==============================================================================
--- rtir/branches/1.9-EXPERIMENTAL/html/RTIR/Elements/ShowIncidents	(original)
+++ rtir/branches/1.9-EXPERIMENTAL/html/RTIR/Elements/ShowIncidents	Fri Mar 24 09:04:53 2006
@@ -37,12 +37,15 @@
 %   if ( $Ticket->Owner == $session{'CurrentUser'}->id ) {
 <b><a href="<%RT->Config->Get('WebPath')%>/RTIR/Incident/LinkToIncident.html?id=<% $Ticket->Id %>">[Link]</a></b>
 <b><a href="<%RT->Config->Get('WebPath')%>/RTIR/Create.html?Child=<% $Ticket->Id %>&Queue=Incidents">[New]</a></b>
-%   } else {
-%       my $Action = $Ticket->Owner == $RT::Nobody->Id? 'Take' : 'Steal';
-%       if ( $Ticket->QueueObj->IsActiveStatus( $Ticket->Status ) ) {
-<a href="<% RT->Config->Get('WebPath') %>/RTIR/Display.html?Action=<% $Action %>&id=<% $Ticket->Id %>&Status=open">[<% loc( $Action ) %>]</a>
+%   } else { #XXX: looks too complex, refactor it
+%       if ( $Ticket->Owner == $RT::Nobody->Id ) {
+%           if ( $Ticket->QueueObj->IsActiveStatus( $Ticket->Status ) ) {
+<a href="<% RT->Config->Get('WebPath') %>/RTIR/Display.html?Action=Take&id=<% $Ticket->Id %>&Status=open">[<% loc('Take') %>]</a>
+%           } else {
+<a href="<% RT->Config->Get('WebPath') %>/RTIR/Display.html?Action=Take&id=<% $Ticket->Id %>">[<% loc('Take') %>]</a>
+%           }
 %       } else {
-<a href="<% RT->Config->Get('WebPath') %>/RTIR/Display.html?Action=<% $Action %>&id=<% $Ticket->Id %>">[<% loc( $Action ) %>]</a>
+<a href="<% RT->Config->Get('WebPath') %>/RTIR/Display.html?Action=Steal&id=<% $Ticket->Id %>">[<% loc('Steal') %>]</a>
 %       }
 %   }
 % }