Bumping version to 1.60

[dbsrgits/SQL-Translator.git] / lib / SQL / Translator / Schema / Procedure.pm

diff --git a/lib/SQL/Translator/Schema/Procedure.pm b/lib/SQL/Translator/Schema/Procedure.pm
index 15686e7..97e55cb 100644 (file)
--- a/lib/SQL/Translator/Schema/Procedure.pm
+++ b/lib/SQL/Translator/Schema/Procedure.pm
@@ -27,21 +27,15 @@ stored procedures (and possibly other pieces of nameable SQL code?).
 
 =cut
 
-use strict;
-use warnings;
-use SQL::Translator::Utils 'parse_list_arg';
+use Moo;
+use SQL::Translator::Utils qw(ex2err);
+use SQL::Translator::Role::ListAttr;
+use SQL::Translator::Types qw(schema_obj);
+use Sub::Quote qw(quote_sub);
 
-use base 'SQL::Translator::Schema::Object';
+extends 'SQL::Translator::Schema::Object';
 
-use vars qw($VERSION);
-
-$VERSION = '1.59';
-
-__PACKAGE__->_attributes( qw/
-    name sql parameters comments owner sql schema order
-/);
-
-=pod
+our $VERSION = '1.60';
 
 =head2 new
 
@@ -51,10 +45,6 @@ Object constructor.
 
 =cut
 
-sub parameters {
-
-=pod
-
 =head2 parameters
 
 Gets and set the parameters of the stored procedure.
@@ -69,26 +59,7 @@ Gets and set the parameters of the stored procedure.
 
 =cut
 
-    my $self   = shift;
-    my $parameters = parse_list_arg( @_ );
-
-    if ( @$parameters ) {
-        my ( %unique, @unique );
-        for my $p ( @$parameters ) {
-            next if $unique{ $p };
-            $unique{ $p } = 1;
-            push @unique, $p;
-        }
-
-        $self->{'parameters'} = \@unique;
-    }
-
-    return wantarray ? @{ $self->{'parameters'} || [] } : ($self->{'parameters'} || '');
-}
-
-sub name {
-
-=pod
+with ListAttr parameters => ( uniq => 1 );
 
 =head2 name
 
@@ -99,14 +70,7 @@ Get or set the procedure's name.
 
 =cut
 
-    my $self        = shift;
-    $self->{'name'} = shift if @_;
-    return $self->{'name'} || '';
-}
-
-sub sql {
-
-=pod
+has name => ( is => 'rw', default => quote_sub(q{ '' }) );
 
 =head2 sql
 
@@ -117,14 +81,7 @@ Get or set the procedure's SQL.
 
 =cut
 
-    my $self       = shift;
-    $self->{'sql'} = shift if @_;
-    return $self->{'sql'} || '';
-}
-
-sub order {
-
-=pod
+has sql => ( is => 'rw', default => quote_sub(q{ '' }) );
 
 =head2 order
 
@@ -135,14 +92,8 @@ Get or set the order of the procedure.
 
 =cut
 
-    my $self         = shift;
-    $self->{'order'} = shift if @_;
-    return $self->{'order'};
-}
-
-sub owner {
+has order => ( is => 'rw' );
 
-=pod
 
 =head2 owner
 
@@ -153,14 +104,7 @@ Get or set the owner of the procedure.
 
 =cut
 
-    my $self         = shift;
-    $self->{'owner'} = shift if @_;
-    return $self->{'owner'} || '';
-}
-
-sub comments {
-
-=pod
+has owner => ( is => 'rw', default => quote_sub(q{ '' }) );
 
 =head2 comments
 
@@ -172,26 +116,24 @@ Get or set the comments on a procedure.
 
 =cut
 
-    my $self = shift;
+has comments => (
+    is => 'rw',
+    coerce => quote_sub(q{ ref($_[0]) eq 'ARRAY' ? $_[0] : [$_[0]] }),
+    default => quote_sub(q{ [] }),
+);
 
-    for my $arg ( @_ ) {
-        $arg = $arg->[0] if ref $arg;
-        push @{ $self->{'comments'} }, $arg if $arg;
-    }
+around comments => sub {
+    my $orig     = shift;
+    my $self     = shift;
+    my @comments = ref $_[0] ? @{ $_[0] } : @_;
 
-    if ( @{ $self->{'comments'} || [] } ) {
-        return wantarray
-            ? @{ $self->{'comments'} || [] }
-            : join( "\n", @{ $self->{'comments'} || [] } );
-    }
-    else {
-        return wantarray ? () : '';
+    for my $arg ( @comments ) {
+        $arg = $arg->[0] if ref $arg;
+        push @{ $self->$orig }, $arg if defined $arg && $arg;
     }
-}
-
-sub schema {
 
-=pod
+    return wantarray ? @{ $self->$orig } : join( "\n", @{ $self->$orig } );
+};
 
 =head2 schema
 
@@ -202,19 +144,9 @@ Get or set the procedures's schema object.
 
 =cut
 
-    my $self = shift;
-    if ( my $arg = shift ) {
-        return $self->error('Not a schema object') unless
-            UNIVERSAL::isa( $arg, 'SQL::Translator::Schema' );
-        $self->{'schema'} = $arg;
-    }
+has schema => ( is => 'rw', isa => schema_obj('Schema'), weak_ref => 1 );
 
-    return $self->{'schema'};
-}
-
-sub equals {
-
-=pod
+around schema => \&ex2err;
 
 =head2 equals
 
@@ -224,12 +156,14 @@ Determines if this procedure is the same as another
 
 =cut
 
+around equals => sub {
+    my $orig = shift;
     my $self = shift;
     my $other = shift;
     my $case_insensitive = shift;
     my $ignore_sql = shift;
 
-    return 0 unless $self->SUPER::equals($other);
+    return 0 unless $self->$orig($other);
     return 0 unless $case_insensitive ? uc($self->name) eq uc($other->name) : $self->name eq $other->name;
 
     unless ($ignore_sql) {
@@ -249,12 +183,10 @@ Determines if this procedure is the same as another
 #    return 0 unless $case_insensitive ? uc($self->owner) eq uc($other->owner) : $self->owner eq $other->owner;
     return 0 unless $self->_compare_objects(scalar $self->extra, scalar $other->extra);
     return 1;
-}
+};
 
-sub DESTROY {
-    my $self = shift;
-    undef $self->{'schema'}; # destroy cyclical reference
-}
+# Must come after all 'has' declarations
+around new => \&ex2err;
 
 1;