From c1537e89f89d1460e3b0fb0dc87d2a310afa3cbc Mon Sep 17 00:00:00 2001 From: Jeremy Kemper Date: Thu, 8 Dec 2005 04:34:26 +0000 Subject: PostgreSQL: more robust sequence name discovery. References #3087. git-svn-id: http://svn-commit.rubyonrails.org/rails/trunk@3235 5ecf4fe2-1ee6-0310-87b1-e25e094e27de --- .../lib/active_record/connection_adapters/postgresql_adapter.rb | 7 ++++--- 1 file changed, 4 insertions(+), 3 deletions(-) (limited to 'activerecord/lib/active_record/connection_adapters/postgresql_adapter.rb') diff --git a/activerecord/lib/active_record/connection_adapters/postgresql_adapter.rb b/activerecord/lib/active_record/connection_adapters/postgresql_adapter.rb index 8619da8c5b..8f2ba6e6c4 100644 --- a/activerecord/lib/active_record/connection_adapters/postgresql_adapter.rb +++ b/activerecord/lib/active_record/connection_adapters/postgresql_adapter.rb @@ -251,7 +251,7 @@ module ActiveRecord # First try looking for a sequence with a dependency on the # given table's primary key. result = execute(<<-end_sql, 'PK and serial sequence')[0] - SELECT attr.attname, (name.nspname || '.' || seq.relname) + SELECT attr.attname, name.nspname, seq.relname FROM pg_class seq, pg_attribute attr, pg_depend dep, @@ -274,7 +274,7 @@ module ActiveRecord # the 8.1+ nextval('foo'::regclass). # TODO: assumes sequence is in same schema as table. result = execute(<<-end_sql, 'PK and custom sequence')[0] - SELECT attr.attname, (name.nspname || '.' || split_part(def.adsrc, '\\\'', 2)) + SELECT attr.attname, name.nspname, split_part(def.adsrc, '\\\'', 2) FROM pg_class t JOIN pg_namespace name ON (t.relnamespace = name.oid) JOIN pg_attribute attr ON (t.oid = attrelid) @@ -285,7 +285,8 @@ module ActiveRecord AND def.adsrc ~* 'nextval' end_sql end - result + # check for existence of . in sequence name as in public.foo_sequence. if it does not exist, join the current namespace + result.last['.'] ? [result.first, result.last] : [result.first, "#{result[1]}.#{result[2]}"] rescue nil end -- cgit v1.2.3