aboutsummaryrefslogtreecommitdiffstats
path: root/actionpack/lib
diff options
context:
space:
mode:
authorJoshua Peek <josh@joshpeek.com>2008-08-28 10:37:46 -0500
committerJoshua Peek <josh@joshpeek.com>2008-08-28 10:37:46 -0500
commitacbf2b74aa3001fb6064bba96cd0033495774357 (patch)
tree54bd7e783a063ddf359721eefd2a5566d12a9a0f /actionpack/lib
parent8b6870cfae8d50a2ffd4f024d33d51aadaa6a6f7 (diff)
downloadrails-acbf2b74aa3001fb6064bba96cd0033495774357.tar.gz
rails-acbf2b74aa3001fb6064bba96cd0033495774357.tar.bz2
rails-acbf2b74aa3001fb6064bba96cd0033495774357.zip
Deprecated implicit local assignments when rendering partials
Diffstat (limited to 'actionpack/lib')
-rw-r--r--actionpack/lib/action_view/renderable_partial.rb10
1 files changed, 8 insertions, 2 deletions
diff --git a/actionpack/lib/action_view/renderable_partial.rb b/actionpack/lib/action_view/renderable_partial.rb
index 5203e57ead..123a9aebbc 100644
--- a/actionpack/lib/action_view/renderable_partial.rb
+++ b/actionpack/lib/action_view/renderable_partial.rb
@@ -27,8 +27,14 @@ module ActionView
def render_partial(view, object = nil, local_assigns = {}, as = nil)
object ||= local_assigns[:object] ||
- local_assigns[variable_name] ||
- view.controller.instance_variable_get("@#{variable_name}") if view.respond_to?(:controller)
+ local_assigns[variable_name]
+
+ if view.respond_to?(:controller)
+ object ||= ActiveSupport::Deprecation::DeprecatedObjectProxy.new(
+ view.controller.instance_variable_get("@#{variable_name}"),
+ "@#{variable_name} will no longer be implicitly assigned to #{variable_name}"
+ )
+ end
# Ensure correct object is reassigned to other accessors
local_assigns[:object] = local_assigns[variable_name] = object