From 89edfad4a52a2924dbf2d67d5609870e1200f7d3 Mon Sep 17 00:00:00 2001 From: "mkanat%bugzilla.org" <> Date: Wed, 1 Nov 2006 07:27:31 +0000 Subject: Bug 358956: [PostgreSQL] Sequences need to be renamed when their field is renamed Patch By Max Kanat-Alexander (module owner) a=myk --- Bugzilla/DB/Pg.pm | 19 +++++++++++++++++++ 1 file changed, 19 insertions(+) (limited to 'Bugzilla/DB/Pg.pm') diff --git a/Bugzilla/DB/Pg.pm b/Bugzilla/DB/Pg.pm index 98bfc5903..ff596ad0d 100644 --- a/Bugzilla/DB/Pg.pm +++ b/Bugzilla/DB/Pg.pm @@ -207,6 +207,15 @@ sub bz_unlock_tables { } } +# Tell us whether or not a particular sequence exists in the DB. +sub bz_sequence_exists { + my ($self, $seq_name) = @_; + my $exists = $self->selectrow_array( + 'SELECT 1 FROM pg_statio_user_sequences WHERE relname = ?', + undef, $seq_name); + return $exists || 0; +} + ##################################################################### # Custom Database Setup ##################################################################### @@ -236,6 +245,16 @@ sub bz_setup_database { _fix_case_differences('products', 'name'); $self->bz_add_index('products', 'products_name_lower_idx', {FIELDS => ['LOWER(name)'], TYPE => 'UNIQUE'}); + + # bz_rename_column didn't correctly rename the sequence. + if ($self->bz_column_info('fielddefs', 'id') + && $self->bz_sequence_exists('fielddefs_fieldid_seq')) + { + print "Fixing fielddefs_fieldid_seq sequence...\n"; + $self->do("ALTER TABLE fielddefs_fieldid_seq RENAME TO fielddefs_id_seq"); + $self->do("ALTER TABLE fielddefs ALTER COLUMN id + SET DEFAULT NEXTVAL('fielddefs_id_seq')"); + } } # Renames things that differ only in case. -- cgit v1.2.3-24-g4f1b