[Rt-commit] r6773 - in rt/branches/3.7-EXPERIMENTAL: .

[ruz at bestpractical.com]

Author: ruz
Date: Thu Jan 18 10:56:05 2007
New Revision: 6773

Modified:
   rt/branches/3.7-EXPERIMENTAL/   (props changed)
   rt/branches/3.7-EXPERIMENTAL/html/Ticket/Elements/EditWatchers

Log:
 r4393 at cubic-pc:  cubic | 2007-01-18 11:05:58 +0300
 * refactor code


Modified: rt/branches/3.7-EXPERIMENTAL/html/Ticket/Elements/EditWatchers
==============================================================================
--- rt/branches/3.7-EXPERIMENTAL/html/Ticket/Elements/EditWatchers	(original)
+++ rt/branches/3.7-EXPERIMENTAL/html/Ticket/Elements/EditWatchers	Thu Jan 18 10:56:05 2007
@@ -45,21 +45,22 @@
 %# END BPS TAGGED BLOCK }}}
 <ul>
 %# Print out a placeholder if there are none.
-%if ($Members->Count == 0 ) {
+% unless ( $Members->Count ) {
 <li><i><&|/l&>none</&></i></li>
 % }
 
 
-%while (my $watcher=$Members->Next) {
+% while ( my $watcher = $Members->Next ) {
+% my $member = $watcher->MemberObj->Object;
 <li>
-<input type="checkbox" class="checkbox" name="Ticket-DeleteWatcher-Type-<%$Watchers->Type%>-Principal-<%$watcher->MemberId%>" value="1" unchecked />
-%if ($watcher->MemberObj->IsUser) { 
-<a href="<%RT->Config->Get('WebPath')%>/Admin/Users/Modify.html?id=<%$watcher->MemberObj->Object->id%>">
-<%$watcher->MemberObj->Object->Name%></a>
-%} else {
-<a href="<%RT->Config->Get('WebPath')%>/Admin/Groups/Modify.html?id=<%$watcher->MemberObj->Object->id%>">
-<%$watcher->MemberObj->Object->Name%></a>
-%}
+<input type="checkbox" class="checkbox" name="Ticket-DeleteWatcher-Type-<% $Watchers->Type %>-Principal-<% $watcher->MemberId %>" value="1" unchecked />
+% if ( $member->isa( 'RT::User' ) ) { 
+<a href="<% RT->Config->Get('WebPath') %>/Admin/Users/Modify.html?id=<% $watcher->MemberId %>">
+<% $member->Name %></a>
+% } else {
+<a href="<% RT->Config->Get('WebPath') %>/Admin/Groups/Modify.html?id=<% $watcher->MemberId %>">
+<% $member->Name %></a>
+% }
 </li>
 % }
 </ul>
@@ -70,6 +71,3 @@
 $TicketObj => undef
 $Watchers => undef
 </%ARGS>
-
-
-