[rt-users] Bug (Showstopper?) in 3.0.11rc2?

Jesse Vincent jesse at bestpractical.com
Sun May 23 22:07:33 EDT 2004



> >That is 100% correct. My testing confirmed this. I've just built new
> >tests into RT to make sure this doesn't happen again and altered the
> >query to behave apropriately. Regression tests are running now.
> After playing around this I end up with attached patch.

That isn't quite what I ended up with:

 in sub WithRight:


@@ -273,7 +312,7 @@
             $or_check_roles =
                 " OR ( ( (main.Domain = 'RT::Queue-Role' AND
main.Instance = " .
                 $args{'Object'}->Id . ") $or_check_ticket_roles ) " .
-                " AND main.Type = $acl.PrincipalType AND main.id = $acl.PrincipalId) ";
+                " AND main.Type = $acl.PrincipalType) ";
         }
 
        if ( $args{'IncludeSystemRights'} ) {


My next merge-up will have this included, as well as a testsuite for it.

	Jesse

-- 



More information about the rt-users mailing list