diff options
| author | Xavier Noria <fxn@hashref.com> | 2016-08-08 01:05:28 +0200 |
|---|---|---|
| committer | Xavier Noria <fxn@hashref.com> | 2016-08-08 01:12:38 +0200 |
| commit | a9dc45459abcd9437085f4dd0aa3c9d0e64e062f (patch) | |
| tree | fe160bcd744ad3dffbe901a4b44df2c2343130de /activerecord/test/models | |
| parent | b45c9ca9b6571108242c1dfc3d3e160f56baf025 (diff) | |
| download | rails-a9dc45459abcd9437085f4dd0aa3c9d0e64e062f.tar.gz rails-a9dc45459abcd9437085f4dd0aa3c9d0e64e062f.tar.bz2 rails-a9dc45459abcd9437085f4dd0aa3c9d0e64e062f.zip | |
code gardening: removes redundant selfs
A few have been left for aesthetic reasons, but have made a pass
and removed most of them.
Note that if the method `foo` returns an array, `foo << 1`
is a regular push, nothing to do with assignments, so
no self required.
Diffstat (limited to 'activerecord/test/models')
| -rw-r--r-- | activerecord/test/models/subject.rb | 2 | ||||
| -rw-r--r-- | activerecord/test/models/topic.rb | 2 |
2 files changed, 2 insertions, 2 deletions
diff --git a/activerecord/test/models/subject.rb b/activerecord/test/models/subject.rb index f31a33ec7e..29e290825e 100644 --- a/activerecord/test/models/subject.rb +++ b/activerecord/test/models/subject.rb @@ -7,7 +7,7 @@ class Subject < ActiveRecord::Base protected def set_email_address - unless self.persisted? + unless persisted? self.author_email_address = "test@test.com" end end diff --git a/activerecord/test/models/topic.rb b/activerecord/test/models/topic.rb index 00f89a5364..db04735d01 100644 --- a/activerecord/test/models/topic.rb +++ b/activerecord/test/models/topic.rb @@ -84,7 +84,7 @@ class Topic < ActiveRecord::Base end def set_email_address - unless self.persisted? + unless persisted? self.author_email_address = "test@test.com" end end |