[rt-devel] Patch for RT::Shredder

[Thomas Sibley]

On 01/18/2013 07:37 AM, Kai wrote:
> Hi Guys, (Girls),
> 
> I was running into a problem where a removed user was still referenced from
> within old Transactions in existing tickets.
> 
> The removed user was a Requestor to some tickets, but was removed from that role
> in the past. Upon removal the 'OldValue' in the Transactions table was still
> referencing the (old) principalid.
> 
> I've written a patch with some tests to verify correct working on
>  https://github.com/giganteous/rt/commit/0d0a1e60
> 
> I'm new to contributing to RT, so I might be failing on coding style, or other
> things. Comments are welcome.

Submit it as a pull request against the bestpractical repo?