diff options
author | Xavier Noria <fxn@hashref.com> | 2010-06-14 18:36:04 +0200 |
---|---|---|
committer | Xavier Noria <fxn@hashref.com> | 2010-06-14 18:36:04 +0200 |
commit | 4a8c8804ff267db8fb4fcf147b8a4eeed78db209 (patch) | |
tree | 7de229746c35aa2f0e5ae2e30e9af98196a50062 /activerecord/lib/active_record | |
parent | 8d82bef58a002b4441d86d0b08e7a5fd493c7e39 (diff) | |
download | rails-4a8c8804ff267db8fb4fcf147b8a4eeed78db209.tar.gz rails-4a8c8804ff267db8fb4fcf147b8a4eeed78db209.tar.bz2 rails-4a8c8804ff267db8fb4fcf147b8a4eeed78db209.zip |
refactors AR::Base#reset_table_name
Diffstat (limited to 'activerecord/lib/active_record')
-rwxr-xr-x | activerecord/lib/active_record/base.rb | 39 |
1 files changed, 20 insertions, 19 deletions
diff --git a/activerecord/lib/active_record/base.rb b/activerecord/lib/active_record/base.rb index 110f131e6c..3b6ffa46f2 100755 --- a/activerecord/lib/active_record/base.rb +++ b/activerecord/lib/active_record/base.rb @@ -14,6 +14,7 @@ require 'active_support/core_ext/hash/slice' require 'active_support/core_ext/string/behavior' require 'active_support/core_ext/kernel/singleton_class' require 'active_support/core_ext/module/delegation' +require 'active_support/core_ext/module/introspection' require 'active_support/core_ext/object/duplicable' require 'active_support/core_ext/object/blank' require 'arel' @@ -652,26 +653,9 @@ module ActiveRecord #:nodoc: @quoted_table_name ||= connection.quote_table_name(table_name) end - # Computes the table name, resets it internally, and returns it. + # Computes the table name, (re)sets it internally, and returns it. def reset_table_name #:nodoc: - base = base_class - - name = - # STI subclasses always use their superclass' table. - unless self == base - base.table_name - else - # Nested classes are prefixed with singular parent table name. - if parent < ActiveRecord::Base && !parent.abstract_class? - contained = parent.table_name - contained = contained.singularize if parent.pluralize_table_names - contained << '_' - end - name = "#{full_table_name_prefix}#{contained}#{undecorated_table_name(base.name)}#{table_name_suffix}" - end - - set_table_name(name) - name + self.table_name = compute_table_name end def full_table_name_prefix #:nodoc: @@ -1003,6 +987,23 @@ module ActiveRecord #:nodoc: table_name end + # Computes and returns a table name according to default conventions. + def compute_table_name + base = base_class + if self == base + # Nested classes are prefixed with singular parent table name. + if parent < ActiveRecord::Base && !parent.abstract_class? + contained = parent.table_name + contained = contained.singularize if parent.pluralize_table_names + contained << '_' + end + "#{full_table_name_prefix}#{contained}#{undecorated_table_name(name)}#{table_name_suffix}" + else + # STI subclasses always use their superclass' table. + base.table_name + end + end + # Enables dynamic finders like <tt>find_by_user_name(user_name)</tt> and <tt>find_by_user_name_and_password(user_name, password)</tt> # that are turned into <tt>where(:user_name => user_name).first</tt> and <tt>where(:user_name => user_name, :password => :password).first</tt> # respectively. Also works for <tt>all</tt> by using <tt>find_all_by_amount(50)</tt> that is turned into <tt>where(:amount => 50).all</tt>. |