Skip to content

Commit ed2839d

Browse files
Merge pull request rails#49836 from skipkayhil/hm-assign-attribute-respond-to
Remove respond_to? in assign_attribute happy path
2 parents a561dd4 + 7a9a537 commit ed2839d

File tree

1 file changed

+3
-1
lines changed

1 file changed

+3
-1
lines changed

activemodel/lib/active_model/attribute_assignment.rb

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -45,8 +45,10 @@ def _assign_attributes(attributes)
4545

4646
def _assign_attribute(k, v)
4747
setter = :"#{k}="
48+
public_send(setter, v)
49+
rescue NoMethodError
4850
if respond_to?(setter)
49-
public_send(setter, v)
51+
raise
5052
else
5153
raise UnknownAttributeError.new(self, k.to_s)
5254
end

0 commit comments

Comments
 (0)