no more wantarray

[dbsrgits/SQL-Translator-2.0-ish.git] / lib / SQL / Translator / Object / Trigger.pm

diff --git a/lib/SQL/Translator/Object/Trigger.pm b/lib/SQL/Translator/Object/Trigger.pm
index 2f8f442..bbea4dc 100644 (file)
--- a/lib/SQL/Translator/Object/Trigger.pm
+++ b/lib/SQL/Translator/Object/Trigger.pm
@@ -1,10 +1,21 @@
 use MooseX::Declare;
-class SQL::Translator::Object::Trigger {
-    use MooseX::Types::Moose qw(ArrayRef HashRef Str);
-    use MooseX::AttributeHelpers;
-    use SQL::Translator::Types qw(Column);
-    extends 'SQL::Translator::Object';
-    
+class SQL::Translator::Object::Trigger extends SQL::Translator::Object {
+    use MooseX::Types::Moose qw(Any ArrayRef HashRef Str);
+    use MooseX::MultiMethods;
+    use SQL::Translator::Types qw(Column Schema Table);
+
+    has 'schema' => (
+        is => 'rw',
+        isa => Schema,
+        weak_ref => 1,
+    );
+
+    has 'table' => (
+        is => 'rw',
+        isa => Table,
+        weak_ref => 1,
+    );
+
     has 'name' => (
         is => 'ro',
         isa => Str,
@@ -12,22 +23,16 @@ class SQL::Translator::Object::Trigger {
     );
 
     has 'columns' => (
-        metaclass => 'Collection::Hash',
+        traits => ['Hash'],
         is => 'rw',
         isa => HashRef[Column],
-        provides => {
-            exists => 'exists_column',
-            keys   => 'column_ids',
-            values => 'get_columns',
-            get    => 'get_column',
-        },
-        curries => {
-            set => {
-                add_column => sub {
-                    my ($self, $body, $column) = @_;
-                    $self->$body($column->name, $column);
-                }
-            }
+        handles => {
+            exists_column => 'exists',
+            column_ids    => 'keys',
+            get_columns   => 'values',
+            get_column    => 'get',
+            add_column    => 'set',
+            clear_columns => 'clear',
         },
         default => sub { my %hash = (); tie %hash, 'Tie::IxHash'; return \%hash },
     );
@@ -35,12 +40,13 @@ class SQL::Translator::Object::Trigger {
     has 'on_table' => (
         is => 'rw', 
         isa => Str,
-        required => 1
+        required => 1,
+#        trigger => sub { my ($self, $new, $old) = @_; $self->table($self->schema->get_table($new)) },
     );
 
     has 'action' => (
         is => 'rw',
-        isa => Str
+        isa => Any
     );
 
     has 'perform_action_when' => (
@@ -49,12 +55,21 @@ class SQL::Translator::Object::Trigger {
         required => 1
     );
 
-    has 'database_events' => (
-        is => 'rw',
+    has '_database_events' => (
+        traits => ['Array'],
         isa => ArrayRef,
-        required => 1
+        handles => {
+            _database_events            => 'elements',
+            add_database_event          => 'push',
+            remove_last_database_option => 'pop',
+        },
+        default => sub { [] },
+        required => 1,
     );
 
-    method get_fields { return $self->get_columns }
-    method fields { return $self->column_ids }
+    around add_column(Column $column) { $self->$orig($column->name, $column) }
+
+    multi method database_events(Str $database_event) { $self->add_database_event($database_event); $self->database_events }
+    multi method database_events(ArrayRef $database_events) { $self->add_database_event($_) for @$database_events; $self->database_events }
+    multi method database_events { $self->_database_events }
 }