Lighthouse has a new layout. Prefer the old one? Return to the old layout, and switch back any time from the link at the top of each page.

This project is archived and is in readonly mode.

[PATCH] Warning about method redefined in active_model

#5072

When runnign test suite there are warnings:


/Users/strzalek/projects/ruby/rails_source/activemodel/lib/active_model/observing.rb:161: warning: method redefined; discarding old observed_classes
/Users/strzalek/projects/ruby/rails_source/activemodel/lib/active_model/observing.rb:161: warning: previous definition of observed_classes was here
/Users/strzalek/projects/ruby/rails_source/activemodel/lib/active_model/observing.rb:161: warning: method redefined; discarding old observed_classes
/Users/strzalek/projects/ruby/rails_source/activemodel/lib/active_model/observing.rb:161: warning: previous definition of observed_classes was here
/Users/strzalek/projects/ruby/rails_source/activemodel/lib/active_model/observing.rb:161: warning: method redefined; discarding old observed_classes
/Users/strzalek/projects/ruby/rails_source/activemodel/lib/active_model/observing.rb:161: warning: previous definition of observed_classes was here
/Users/strzalek/projects/ruby/rails_source/activemodel/lib/active_model/observing.rb:161: warning: method redefined; discarding old observed_classes
/Users/strzalek/projects/ruby/rails_source/activemodel/lib/active_model/observing.rb:161: warning: previous definition of observed_classes was here

There are 2 ways to avoid this warning. First is to alias old method, second is to remove it. As it isn't no longer used I think it can be removed.

Patch attached.

Reported by Łukasz Strzałkowski · July 8th, 2010 @ 08:39 PM

State: resolved
Milestone: none
Assigned to: nobody
Importance: none

Activity

  1. Repository
    Repository
    • State changed from new to resolved

    July 8th, 2010 @ 10:03 PM