[rt-users] [PATCH REQ for 4.2.2] callback for Modify.html

Arkadiusz Miśkiewicz arekm at maven.pl
Tue Nov 26 12:18:39 EST 2013


On Tuesday 26 of November 2013, Kevin Falcone wrote:
> On Tue, Nov 26, 2013 at 01:58:47PM +0100, Arkadiusz Miśkiewicz wrote:
> > Was there any reason for not adding BeforeModify callback in Modify.html?
> > It's logical to have the same sets of callbacks for similar actions IMO.
> > 
> > This patch below works (hand made, line wrapped, so likely won't apply
> > directly). Please add such modification to 4.2.2. Thanks.
> 
> Your patch adds a callback right before another callback which passes
> more useful information and can be used to accomplish the same goals.
> The existing callback is even visible in your patch.
> 
> What does your new callback do that the Default callback doesn't?

That's a good question + #rt irc conversation explained few things for me. 
It's not needed at all. "Default" will be enough.

-- 
Arkadiusz Miśkiewicz, arekm / maven.pl



More information about the rt-users mailing list