From 564e32b71dfdf9b39ed243b25d9a3f090294ce9d Mon Sep 17 00:00:00 2001 From: Aaron Patterson Date: Sat, 19 May 2012 11:46:13 -0700 Subject: Merge pull request #6397 from kennyj/fix_translate_exception Fix a problem of translate_exception method in a Japanese (non English) environment. --- activerecord/CHANGELOG.md | 5 +++++ .../active_record/connection_adapters/postgresql_adapter.rb | 10 +++++++--- 2 files changed, 12 insertions(+), 3 deletions(-) diff --git a/activerecord/CHANGELOG.md b/activerecord/CHANGELOG.md index 4a445529b5..a263c26b3a 100644 --- a/activerecord/CHANGELOG.md +++ b/activerecord/CHANGELOG.md @@ -1,5 +1,10 @@ ## Rails 3.2.10 (unreleased) +* Fix a problem with `translate_exception` method in a non English environment. + Backport of #6397. + + *kennyj* + * Fix dirty attribute checks for TimeZoneConversion with nil and blank datetime attributes. Setting a nil datetime to a blank string should not result in a change being flagged. Fix #8310 [Backport #8311] diff --git a/activerecord/lib/active_record/connection_adapters/postgresql_adapter.rb b/activerecord/lib/active_record/connection_adapters/postgresql_adapter.rb index afc363c9b2..08ef704752 100644 --- a/activerecord/lib/active_record/connection_adapters/postgresql_adapter.rb +++ b/activerecord/lib/active_record/connection_adapters/postgresql_adapter.rb @@ -1142,11 +1142,15 @@ module ActiveRecord @connection.server_version end + # See http://www.postgresql.org/docs/9.1/static/errcodes-appendix.html + FOREIGN_KEY_VIOLATION = "23503" + UNIQUE_VIOLATION = "23505" + def translate_exception(exception, message) - case exception.message - when /duplicate key value violates unique constraint/ + case exception.result.error_field(PGresult::PG_DIAG_SQLSTATE) + when UNIQUE_VIOLATION RecordNotUnique.new(message, exception) - when /violates foreign key constraint/ + when FOREIGN_KEY_VIOLATION InvalidForeignKey.new(message, exception) else super -- cgit v1.2.3