[Rt-commit] r5227 - rt/branches/3.5-TESTING/lib/RT

[jesse at bestpractical.com]

Author: jesse
Date: Mon May 15 14:02:23 2006
New Revision: 5227

Modified:
   rt/branches/3.5-TESTING/lib/RT/Ticket_Overlay.pm

Log:
* The change to AddLink ACLs was a bit premature. It was actually a functionality change relative to 3.4


Modified: rt/branches/3.5-TESTING/lib/RT/Ticket_Overlay.pm
==============================================================================
--- rt/branches/3.5-TESTING/lib/RT/Ticket_Overlay.pm	(original)
+++ rt/branches/3.5-TESTING/lib/RT/Ticket_Overlay.pm	Mon May 15 14:02:23 2006
@@ -2593,20 +2593,18 @@
         return( 0, $msg );
     }
 
-    # By popular demand, adding a ticket link doesn't check ACLs on the second ticket 
-    #
-    #    if ( $other_ticket_uri->Resolver->Scheme eq 'fsck.com-rt') {
-    #        my $object = $other_ticket_uri->Resolver->Object;
-    #
-    #        if (   UNIVERSAL::isa( $object, 'RT::Ticket' )
-    #            && $object->id
-    #            && !$object->CurrentUserHasRight('ModifyTicket') )
-    #        {
-    #            return ( 0, $self->loc("Permission Denied") );
-    #        }
-    #
-    #    }
-    #
+    if ( $other_ticket_uri->Resolver->Scheme eq 'fsck.com-rt') {
+        my $object = $other_ticket_uri->Resolver->Object;
+
+        if (   UNIVERSAL::isa( $object, 'RT::Ticket' )
+            && $object->id
+            && !$object->CurrentUserHasRight('ModifyTicket') )
+        {
+            return ( 0, $self->loc("Permission Denied") );
+        }
+
+    }
+
     # }}}
 
     my ($val, $Msg) = $self->SUPER::_AddLink(%args);