[rt-users] BUG: shredder noticket option ignores deleted messages
The shredder users module will delete users actively assigned to tickets which are Deleted. I suspect that the tickets should be "Wiped" before these users are removed. --- Users.pm_orig 2009-06-02 12:28:24.0 -0700 +++ Users.pm2009-06-02 15:34:56.0 -0700 @@ -219,7 +219,8 @@ sub _WithoutTickets { my ($self, $user) = @_; my $tickets = RT::Tickets->new( $RT::SystemUser ); -$tickets->FromSQL( 'Watcher.id = '. $user->id ); + $tickets->{'allow_deleted_search'} = 1; $tickets->FromSQL( 'Watcher.id = '. $user->id ); # HACK: we may use Count method which counts all records # that match condtion, but we really want to know only that # at least one record exist, so we fetch first row only -- Jo Rhett Net Consonance : consonant endings by net philanthropy, open source and other randomness ___ http://lists.bestpractical.com/cgi-bin/mailman/listinfo/rt-users Community help: http://wiki.bestpractical.com Commercial support: sa...@bestpractical.com Discover RT's hidden secrets with RT Essentials from O'Reilly Media. Buy a copy at http://rtbook.bestpractical.com
Re: [rt-users] BUG: shredder noticket option ignores deleted messages
Sorry, revised patch is: --- Users.pm_orig 2009-06-02 12:28:24.0 -0700 +++ Users.pm2009-06-02 15:34:56.0 -0700 @@ -219,7 +219,8 @@ sub _WithoutTickets { my ($self, $user) = @_; my $tickets = RT::Tickets->new( $RT::SystemUser ); + $tickets->{'allow_deleted_search'} = 1; $tickets->FromSQL( 'Watcher.id = '. $user->id ); # HACK: we may use Count method which counts all records # that match condtion, but we really want to know only that # at least one record exist, so we fetch first row only -- Jo Rhett Net Consonance : consonant endings by net philanthropy, open source and other randomness ___ http://lists.bestpractical.com/cgi-bin/mailman/listinfo/rt-users Community help: http://wiki.bestpractical.com Commercial support: sa...@bestpractical.com Discover RT's hidden secrets with RT Essentials from O'Reilly Media. Buy a copy at http://rtbook.bestpractical.com