[Rt-commit] r13196 - in rt/branches/3.8-TESTING: . share/html/Elements

[kyoki at bestpractical.com]

Author: kyoki
Date: Thu Jun 12 00:36:51 2008
New Revision: 13196

Modified:
   rt/branches/3.8-TESTING/   (props changed)
   rt/branches/3.8-TESTING/share/html/Elements/ShowLinks
   rt/branches/3.8-TESTING/share/html/NoAuth/css/web2/ticket.css

Log:
 r22785 at nyx:  kyoki | 2008-06-12 00:34:42 -0400
 Resolved tickets in the Depends On list are now grey and show up at the end of the list.


Modified: rt/branches/3.8-TESTING/share/html/Elements/ShowLinks
==============================================================================
--- rt/branches/3.8-TESTING/share/html/Elements/ShowLinks	(original)
+++ rt/branches/3.8-TESTING/share/html/Elements/ShowLinks	Thu Jun 12 00:36:51 2008
@@ -49,9 +49,29 @@
   <tr>
     <td class="labeltop"><&|/l&>Depends on</&>: (<a href="<%$clone->{'DependsOn-new'}%>"><% loc('Create') %></a>)</td>
     <td class="value">
+
+<%PERL>
+my ( @active, @inactive, @not_tickets );
+for my $link ( @{ $Ticket->DependsOn->ItemsArrayRef } ) {
+    my $target = $link->TargetObj;
+    if ( $target && $target->isa('RT::Ticket') ) {
+        if ( $target->QueueObj->IsInactiveStatus( $target->Status ) ) {
+            push( @inactive, $link->TargetURI );
+        }
+        else {
+            push( @active, $link->TargetURI );
+        }
+    }
+    else {
+        push( @not_tickets, $link->TargetURI );
+    }
+}
+</%PERL>
+
+
 <ul>
-% while (my $Link = $Ticket->DependsOn->Next) {
-<li><& ShowLink, URI => $Link->TargetURI &></li>
+% for my $Link (@not_tickets, @active, @inactive) {
+<li><& ShowLink, URI => $Link &></li>
 % }
 </ul>
     </td>

Modified: rt/branches/3.8-TESTING/share/html/NoAuth/css/web2/ticket.css
==============================================================================
--- rt/branches/3.8-TESTING/share/html/NoAuth/css/web2/ticket.css	(original)
+++ rt/branches/3.8-TESTING/share/html/NoAuth/css/web2/ticket.css	Thu Jun 12 00:36:51 2008
@@ -206,5 +206,6 @@
 
 .ticket-inactive {
   text-decoration: line-through;
+  color: #666
 }