Patch for RT::Shredder

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.

Regards,
Kai Storbeck

signature.asc (841 Bytes)

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?