Re: PATCH: add_trigger()

[prev] [thread] [next] [Date index for 2004/09/30]

On Thu, Sep 30, 2004 at 03:59:43PM -0400, Drew Taylor wrote:
> It would be very nice to have checking in add_trigger() which makes
> sure you're using a valid trigger. Version 0.96 doesn't have this
> functionality, so a stab at it is below. I don't have a handy recent
> perl available to test the patch so I apologize if I've done something
> stupid. I did update the docs but not t/11-triggers.t (but I will if
> Tony is interested).

I believe we can do this just by passing the valid trigger names to  
Class::Trigger at import:

use Class::Trigger qw/before_create after_create before_update
  after_update before_delete after_delete select create/;

I'm not sure this plays out if you want to add other trigger points
of your own, though. But I'm sure Miyagawa will be open to patches for
that :)

Tony