Ruby on Rails | Screencasts | Download | Documentation | Weblog | Community | Source

Changeset 4719

Show
Ignore:
Timestamp:
08/08/06 06:57:59 (2 years ago)
Author:
bitsweat
Message:

Include called method in instance variable deprecation warning.

Files:

Legend:

Unmodified
Added
Removed
Modified
Copied
Moved
  • trunk/activesupport/lib/active_support/deprecation.rb

    r4698 r4719  
    5252        last = with_last_message_tracking_deprecation_behavior(&block) 
    5353        assert last, "Expected a deprecation warning within the block but received none" 
    54         match = Regexp.new(match) unless match.is_a?(Regexp) 
     54        match = Regexp.new(Regexp.escape(match)) unless match.is_a?(Regexp) 
    5555        assert_match match, last, "Deprecation warning didn't match #{match}: #{last}" 
    5656      end 
     
    8282 
    8383      private 
    84         def deprecation_warning(called, callstack) 
    85           ActiveSupport::Deprecation.warn("Using #{@var} directly is deprecated - call #{@method} instead.", callstack) 
    86         end 
    87  
    8884        def method_missing(called, *args, &block) 
    89           deprecation_warning called, caller 
     85          ActiveSupport::Deprecation.warn("#{@var} is deprecated! Call #{@method}.#{called} instead of #{@var}.#{called}. Args: #{args.inspect}", caller) 
    9086          @instance.__send__(@method).__send__(called, *args, &block) 
    9187        end 
  • trunk/activesupport/test/deprecation_test.rb

    r4698 r4719  
    6969    assert_not_deprecated { @dtc.request.size } 
    7070 
    71     warning = 'Using @request directly is deprecated - call request instead.' 
    72     assert_deprecated(warning) { assert_equal @dtc.request.size, @dtc.old_request.size } 
    73     assert_deprecated(warning) { assert_equal @dtc.request.to_s, @dtc.old_request.to_s } 
     71    assert_deprecated('@request.size') { assert_equal @dtc.request.size, @dtc.old_request.size } 
     72    assert_deprecated('@request.to_s') { assert_equal @dtc.request.to_s, @dtc.old_request.to_s } 
    7473  end 
    7574end