diff options
| author | Neeraj Singh <neerajdotname@gmail.com> | 2013-04-09 08:45:02 -0400 |
|---|---|---|
| committer | Neeraj Singh <neerajdotname@gmail.com> | 2013-04-09 08:53:10 -0400 |
| commit | d87966c1c4119dbfa0808d7bd477b59a6b5f232c (patch) | |
| tree | d017667b23964d94b6778142794416b5ffc26aa8 /activemodel/test | |
| parent | d20c3204bf99d7828be101519d6711dc5f26d2ce (diff) | |
| download | rails-d87966c1c4119dbfa0808d7bd477b59a6b5f232c.tar.gz rails-d87966c1c4119dbfa0808d7bd477b59a6b5f232c.tar.bz2 rails-d87966c1c4119dbfa0808d7bd477b59a6b5f232c.zip | |
changed variable name active_record => base_klass
Current code stores the klass name in active_record and this
is used throughout. While reviewing the code time and again
I had the mental picture of active_record being an instance of
a klass. However here the actual klass is being stored in
@active_record.
Secondly at two different places while referring to @active_record
the comment refers to it as base klass. All this points to
active_record being not the best variable name.
So I thought it is better to replace active_record with base_klass.
This change is confined to JoinDependency, JoinBase, JoinPart and
JoinAssociation - all joining related work.
Diffstat (limited to 'activemodel/test')
0 files changed, 0 insertions, 0 deletions