set table/schema when added to table/schema, rather then on object construction

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

diff --git a/lib/SQL/Translator/Object/Table.pm b/lib/SQL/Translator/Object/Table.pm
index 785963c..46a4219 100644 (file)
--- a/lib/SQL/Translator/Object/Table.pm
+++ b/lib/SQL/Translator/Object/Table.pm
@@ -1,9 +1,16 @@
 use MooseX::Declare;
-class SQL::Translator::Object::Table extends SQL::Translator::Object {
+class SQL::Translator::Object::Table extends SQL::Translator::Object is dirty {
     use MooseX::Types::Moose qw(Any Bool HashRef Str);
     use MooseX::MultiMethods;
     use SQL::Translator::Types qw(Column Constraint Index Schema Sequence);
-    
+    clean;
+
+    use overload
+        '""'     => sub { shift->name },
+        'bool'   => sub { $_[0]->name || $_[0] },
+        fallback => 1,
+    ;
+
     has 'name' => (
         is => 'rw',
         isa => Str,
@@ -20,9 +27,11 @@ class SQL::Translator::Object::Table extends SQL::Translator::Object {
             get_columns   => 'values',
             get_column    => 'get',
             add_column    => 'set',
+            remove_column => 'delete',
 
             ## compat
             get_fields    => 'values',
+            get_field     => 'get',
             fields        => 'keys',
         },
         default => sub { my %hash = (); tie %hash, 'Tie::IxHash'; return \%hash },
@@ -74,7 +83,6 @@ class SQL::Translator::Object::Table extends SQL::Translator::Object {
         is => 'rw',
         isa => Schema,
         weak_ref => 1,
-        required => 1,
     );
 
     has 'temporary' => (
@@ -83,7 +91,14 @@ class SQL::Translator::Object::Table extends SQL::Translator::Object {
         default => 0
     );
 
-    around add_column(Column $column) { $self->$orig($column->name, $column) }
+    method add_field(Column $column does coerce) { $self->add_column($column) }
+
+    around add_column(Column $column does coerce) {
+        die "Can't use column name " . $column->name if $self->exists_column($column->name) || $column->name eq '';
+        $column->table($self);
+        return $self->$orig($column->name, $column);
+    }
+
     around add_constraint(Constraint $constraint) {
         my $name = $constraint->name;
         if ($name eq '') {
@@ -91,21 +106,39 @@ class SQL::Translator::Object::Table extends SQL::Translator::Object {
             while ($self->exists_constraint('ANON' . $idx)) { $idx++ }
             $name = 'ANON' . $idx;
         }
+        $constraint->table($self);
         $self->$orig($name, $constraint)
     }
-    around add_index(Index $index) {
+
+    around add_index(Index $index does coerce) {
         my $name = $index->name;
         if ($name eq '') {
             my $idx = 0;
             while ($self->exists_index('ANON' . $idx)) { $idx++ }
             $name = 'ANON' . $idx;
         }
+        $index->table($self);
         $self->$orig($name, $index)
     }
+
     around add_sequence(Sequence $sequence) { $self->$orig($sequence->name, $sequence) }
 
     multi method primary_key(Any $) { grep /^PRIMARY KEY$/, $_->type for $self->get_constraints }
     multi method primary_key(Str $column) { $self->get_column($column)->is_primary_key(1) }
 
+    method is_valid { return $self->get_columns ? 1 : undef }
     method order { }
+
+    before name($name?) { die "Can't use table name $name, table already exists" if $name && $self->schema->exists_table($name) && $name ne $self->name }
+
+    multi method drop_column(Column $column, Int :$cascade = 0) {
+        die "Can't drop non-existant table " . $column->name unless $self->exists_column($column->name);
+        $self->remove_column($column->name);
+
+    }
+
+    multi method drop_column(Str $column, Int :$cascade = 0) {
+        die "Can't drop non-existant table " . $column unless $self->exists_column($column);
+        $self->remove_column($column);
+    }
 }