Fix RT49301
[dbsrgits/SQL-Translator.git] / lib / SQL / Translator / Producer / PostgreSQL.pm
index d86ab4c..b89c0d0 100644 (file)
@@ -45,7 +45,7 @@ use SQL::Translator::Schema::Constants;
 use SQL::Translator::Utils qw(debug header_comment);
 use Data::Dumper;
 
-my %translate;
+my ( %translate, %index_name );
 my $max_id_length;
 
 BEGIN {
@@ -174,21 +174,17 @@ and table_constraint is:
 
 # -------------------------------------------------------------------
 sub produce {
-    my $translator     = shift;
-    local $DEBUG             = $translator->debug;
-    local $WARN              = $translator->show_warnings;
-    my $no_comments    = $translator->no_comments;
-    my $add_drop_table = $translator->add_drop_table;
-    my $schema         = $translator->schema;
-    my $pargs          = $translator->producer_args;
-    local %used_names  = ();
-
+    my $translator       = shift;
+    local $DEBUG         = $translator->debug;
+    local $WARN          = $translator->show_warnings;
+    my $no_comments      = $translator->no_comments;
+    my $add_drop_table   = $translator->add_drop_table;
+    my $schema           = $translator->schema;
+    my $pargs            = $translator->producer_args;
     my $postgres_version = $pargs->{postgres_version} || 0;
 
-    my $qt = '';
-    $qt = '"' if ($translator->quote_table_names);
-    my $qf = '';
-    $qf = '"' if ($translator->quote_field_names);
+    my $qt = $translator->quote_table_names ? q{"} : q{};
+    my $qf = $translator->quote_field_names ? q{"} : q{};
     
     my @output;
     push @output, header_comment unless ($no_comments);
@@ -196,15 +192,16 @@ sub produce {
     my (@table_defs, @fks);
     for my $table ( $schema->get_tables ) {
 
-        my ($table_def, $fks) = create_table($table, 
-                                             { quote_table_names => $qt,
-                                               quote_field_names => $qf,
-                                               no_comments => $no_comments,
-                                               postgres_version => $postgres_version,
-                                               add_drop_table => $add_drop_table,});
+        my ($table_def, $fks) = create_table($table, { 
+            quote_table_names => $qt,
+            quote_field_names => $qf,
+            no_comments       => $no_comments,
+            postgres_version  => $postgres_version,
+            add_drop_table    => $add_drop_table,
+        });
+
         push @table_defs, $table_def;
         push @fks, @$fks;
-
     }
 
     for my $view ( $schema->get_views ) {
@@ -216,23 +213,12 @@ sub produce {
       });
     }
 
-    my (@trigger_defs);
-    foreach my $trigger ( $schema->get_triggers ) {
-       push @trigger_defs, create_trigger($trigger);
-    }
-
-
     push @output, map { "$_;\n\n" } @table_defs;
     if ( @fks ) {
         push @output, "--\n-- Foreign Key Definitions\n--\n\n" unless $no_comments;
         push @output, map { "$_;\n\n" } @fks;
     }
 
-    if (@trigger_defs) {
-        push @output, "--\n-- Triggers \n--\n\n" unless $no_comments;
-        push @output, map { "$_;\n\n" } @trigger_defs;
-    }
-
     if ( $WARN ) {
         if ( %truncated ) {
             warn "Truncated " . keys( %truncated ) . " names:\n";
@@ -246,25 +232,6 @@ sub produce {
         }
     }
 
-    foreach my $procedure ( $schema->get_procedures ) {
-        my (@comments, $procedure_name);
-
-        $procedure_name = $procedure->name();
-        push @comments, 
-            "--\n-- Procedure: $procedure_name\n--" unless $no_comments;
-
-        # text of procedure  already has the 'create procedure' stuff
-        # so there is no need to do anything fancy. However, we should
-        # think about doing fancy stuff with granting permissions and
-        # so on.
-
-        push (@output, join("\n\n",
-                       @comments,
-                       $procedure->sql(),
-                       ), "\n");
-    }
-
-
     return wantarray
         ? @output
         : join ('', @output);
@@ -332,22 +299,22 @@ sub unreserve {
 
 # -------------------------------------------------------------------
 sub next_unused_name {
-    my $name = shift || '';
-    if ( !defined( $used_names{$name} ) ) {
-        $used_names{$name} = $name;
-        return $name;
-    }
+    my $orig_name = shift or return;
+    my $name      = $orig_name;
 
-    my $i = 2;
-    while ( defined( $used_names{ $name . $i } ) ) {
-        ++$i;
+    my $suffix_gen = sub {
+        my $suffix = 0;
+        return ++$suffix ? '' : $suffix;
+    };
+
+    for (;;) {
+        $name = $orig_name . $suffix_gen->();
+        last if $used_names{ $name }++;
     }
-    $name .= $i;
-    $used_names{$name} = $name;
+
     return $name;
 }
 
-
 sub create_table 
 {
     my ($table, $options) = @_;
@@ -365,7 +332,7 @@ sub create_table
         : unreserve($table_name);
     $table->name($table_name_ur);
 
-    # print STDERR "$table_name table_name\n";
+# print STDERR "$table_name table_name\n";
     my ( @comments, @field_defs, @sequence_defs, @constraint_defs, @type_defs, @type_drops, @fks );
 
     push @comments, "--\n-- Table: $table_name_ur\n--\n" unless $no_comments;
@@ -558,50 +525,55 @@ sub create_view {
     }
 }
 
-sub create_index {
+sub create_index
+{
     my ($index, $options) = @_;
 
     my $qt = $options->{quote_table_names} ||'';
     my $qf = $options->{quote_field_names} ||'';
     my $table_name = $index->table->name;
-    #        my $table_name_ur = $qt ? unreserve($table_name) : $table_name;
+#        my $table_name_ur = $qt ? unreserve($table_name) : $table_name;
 
     my ($index_def, @constraint_defs);
 
-    my $name = $index->name || '';
-    if ( $name ) {
-       $name = next_unused_name($name);
-    }
+    my $name = next_unused_name(
+        $index->name 
+        || join('_', $table_name, 'idx', ++$index_name{ $table_name })
+    );
 
     my $type = $index->type || NORMAL;
     my @fields     = 
-      map { $_ =~ s/\(.+\)//; $_ }
-        map { $qt ? $_ : unreserve($_, $table_name ) }
-         $index->fields;
+        map { $_ =~ s/\(.+\)//; $_ }
+    map { $qt ? $_ : unreserve($_, $table_name ) }
+    $index->fields;
     next unless @fields;
 
     my $def_start = qq[CONSTRAINT "$name" ];
     if ( $type eq PRIMARY_KEY ) {
-       push @constraint_defs, "${def_start}PRIMARY KEY ".
-         '(' .$qf . join( $qf. ', '.$qf, @fields ) . $qf . ')';
-    } elsif ( $type eq UNIQUE ) {
-       push @constraint_defs, "${def_start}UNIQUE " .
-         '(' . $qf . join( $qf. ', '.$qf, @fields ) . $qf.')';
-    } elsif ( $type eq NORMAL ) {
-       $index_def = 
-         "CREATE INDEX ${qf}${name}${qf} on ${qt}${table_name}${qt} (".
-           join( ', ', map { qq[$qf$_$qf] } @fields ).  
-             ')'
-                ; 
-    } else {
-       warn "Unknown index type ($type) on table $table_name.\n"
-         if $WARN;
+        push @constraint_defs, "${def_start}PRIMARY KEY ".
+            '(' .$qf . join( $qf. ', '.$qf, @fields ) . $qf . ')';
+    }
+    elsif ( $type eq UNIQUE ) {
+        push @constraint_defs, "${def_start}UNIQUE " .
+            '(' . $qf . join( $qf. ', '.$qf, @fields ) . $qf.')';
+    }
+    elsif ( $type eq NORMAL ) {
+        $index_def = 
+            "CREATE INDEX ${qf}${name}${qf} on ${qt}${table_name}${qt} (".
+            join( ', ', map { qq[$qf$_$qf] } @fields ).  
+            ')'
+            ; 
+    }
+    else {
+        warn "Unknown index type ($type) on table $table_name.\n"
+            if $WARN;
     }
 
     return $index_def, \@constraint_defs;
 }
 
-sub create_constraint {
+sub create_constraint
+{
     my ($c, $options) = @_;
 
     my $qf = $options->{quote_field_names} ||'';
@@ -611,79 +583,65 @@ sub create_constraint {
 
     my $name = $c->name || '';
     if ( $name ) {
-       $name = next_unused_name($name);
+        $name = next_unused_name($name);
     }
 
     my @fields     = 
-      map { $_ =~ s/\(.+\)//; $_ }
-        map { $qt ? $_ : unreserve( $_, $table_name )}
-         $c->fields;
+        map { $_ =~ s/\(.+\)//; $_ }
+    map { $qt ? $_ : unreserve( $_, $table_name )}
+    $c->fields;
 
     my @rfields     = 
-      map { $_ =~ s/\(.+\)//; $_ }
-        map { $qt ? $_ : unreserve( $_, $table_name )}
-         $c->reference_fields;
+        map { $_ =~ s/\(.+\)//; $_ }
+    map { $qt ? $_ : unreserve( $_, $table_name )}
+    $c->reference_fields;
 
     next if !@fields && $c->type ne CHECK_C;
     my $def_start = $name ? qq[CONSTRAINT "$name" ] : '';
     if ( $c->type eq PRIMARY_KEY ) {
-       push @constraint_defs, "${def_start}PRIMARY KEY ".
-         '('.$qf . join( $qf.', '.$qf, @fields ) . $qf.')';
-    } elsif ( $c->type eq UNIQUE ) {
-       $name = next_unused_name($name);
-       push @constraint_defs, "${def_start}UNIQUE " .
-         '('.$qf . join( $qf.', '.$qf, @fields ) . $qf.')';
-    } elsif ( $c->type eq CHECK_C ) {
-       my $expression = $c->expression;
-       push @constraint_defs, "${def_start}CHECK ($expression)";
-    } elsif ( $c->type eq FOREIGN_KEY ) {
-       my $def .= "ALTER TABLE ${qt}${table_name}${qt} ADD FOREIGN KEY (" . 
-         join( ', ', map { qq[$qf$_$qf] } @fields ) . ')' .
-           "\n  REFERENCES " . $qt . $c->reference_table . $qt;
-
-       if ( @rfields ) {
-           $def .= ' ('.$qf . join( $qf.', '.$qf, @rfields ) . $qf.')';
-       }
-
-       if ( $c->match_type ) {
-           $def .= ' MATCH ' . 
-             ( $c->match_type =~ /full/i ) ? 'FULL' : 'PARTIAL';
-       }
-
-       if ( $c->on_delete ) {
-           $def .= ' ON DELETE '.join( ' ', $c->on_delete );
-       }
-
-       if ( $c->on_update ) {
-           $def .= ' ON UPDATE '.join( ' ', $c->on_update );
-       }
-
-       if ( $c->deferrable ) {
-           $def .= ' DEFERRABLE';
-       }
-
-       push @fks, "$def";
+        push @constraint_defs, "${def_start}PRIMARY KEY ".
+            '('.$qf . join( $qf.', '.$qf, @fields ) . $qf.')';
     }
+    elsif ( $c->type eq UNIQUE ) {
+        $name = next_unused_name($name);
+        push @constraint_defs, "${def_start}UNIQUE " .
+            '('.$qf . join( $qf.', '.$qf, @fields ) . $qf.')';
+    }
+    elsif ( $c->type eq CHECK_C ) {
+        my $expression = $c->expression;
+        push @constraint_defs, "${def_start}CHECK ($expression)";
+    }
+    elsif ( $c->type eq FOREIGN_KEY ) {
+        my $def .= "ALTER TABLE ${qt}${table_name}${qt} ADD FOREIGN KEY (" . 
+            join( ', ', map { qq[$qf$_$qf] } @fields ) . ')' .
+            "\n  REFERENCES " . $qt . $c->reference_table . $qt;
 
-    return \@constraint_defs, \@fks;
-}
+        if ( @rfields ) {
+            $def .= ' ('.$qf . join( $qf.', '.$qf, @rfields ) . $qf.')';
+        }
 
+        if ( $c->match_type ) {
+            $def .= ' MATCH ' . 
+                ( $c->match_type =~ /full/i ) ? 'FULL' : 'PARTIAL';
+        }
 
-sub create_trigger {
-    my ($trigger) = @_;
-    # CREATE TRIGGER tree_change_trig BEFORE DELETE or INSERT or UPDATE ON type FOR EACH ROW EXECUTE PROCEDURE type_tree_change();
-    my $db_events = join ' or ', $trigger->database_events;
-    my $out = sprintf('CREATE TRIGGER %s %s %s ON %s %s',
-                      $trigger->name,
-                      $trigger->perform_action_when || 'AFTER',
-                      $db_events,
-                      $trigger->table->name,
-                      $trigger->action );
+        if ( $c->on_delete ) {
+            $def .= ' ON DELETE '.join( ' ', $c->on_delete );
+        }
 
-    return $out;
-}
+        if ( $c->on_update ) {
+            $def .= ' ON UPDATE '.join( ' ', $c->on_update );
+        }
+
+        if ( $c->deferrable ) {
+            $def .= ' DEFERRABLE';
+        }
 
+        push @fks, "$def";
+    }
 
+    return \@constraint_defs, \@fks;
+}
 
 sub convert_datatype
 {
@@ -741,23 +699,22 @@ sub convert_datatype
             $data_type = 'integer';
         }
     }
-    my @type_without_size = qw/bigint boolean box bytea cidr circle date inet
-                               integer smallint text line lseg macaddr money
-                               path point polygon real/;
-    foreach (@type_without_size) {
-        if ( $data_type =~ qr/$_/ ) {
-            undef @size; last;
-        }
+
+    my $type_with_size = join('|',
+        'bit', 'varbit', 'character', 'bit varying', 'character varying',
+        'time', 'timestamp', 'interval', 'numeric'
+    );
+
+    if ( $data_type !~ /$type_with_size/ ) {
+        @size = (); 
     }
 
     if (defined $size[0] && $size[0] > 0 && $data_type =~ /^time/i ) {
         $data_type =~ s/^(time.*?)( with.*)?$/$1($size[0])/;
         $data_type .= $2 if(defined $2);
     } elsif ( defined $size[0] && $size[0] > 0 ) {
-            $data_type .= '(' . join( ',', @size ) . ')';
+        $data_type .= '(' . join( ',', @size ) . ')';
     }
-    
-
 
     return $data_type;
 }
@@ -936,6 +893,6 @@ SQL::Translator, SQL::Translator::Producer::Oracle.
 
 =head1 AUTHOR
 
-Ken Y. Clark E<lt>kclark@cpan.orgE<gt>.
+Ken Youens-Clark E<lt>kclark@cpan.orgE<gt>.
 
 =cut