aboutsummaryrefslogtreecommitdiffstats
path: root/actionmailer
diff options
context:
space:
mode:
authorclaudiob <claudiob@gmail.com>2014-09-29 00:22:36 -0700
committerclaudiob <claudiob@gmail.com>2014-10-15 10:15:45 -0700
commitdee00df11c34c43de65d36288f211a5229987ab2 (patch)
tree651db02244946391042c213ae2a39764d2065d72 /actionmailer
parentc86957b884f50013696dd614c78f5e5275d5ecd7 (diff)
downloadrails-dee00df11c34c43de65d36288f211a5229987ab2.tar.gz
rails-dee00df11c34c43de65d36288f211a5229987ab2.tar.bz2
rails-dee00df11c34c43de65d36288f211a5229987ab2.zip
Fix how file_ and password_field_tag edit options
This commit fixes the behavior of `file_field_tag` and `password_field_tag` when invoked with a hash of options. These two helpers are different from all the other ones in that they modify the options hash passed as a parameter, whereas all the other helpers duplicate it before updating it. The result is that *bad things* can happen if the user re-uses the same hash. For instance, users who write the following code to display a file field followed by a text field (both with the same class): ```rhtml <% options = {class: 'important'} %> <%= file_field_tag 'Upload', options %> <%= text_field_tag 'Name', options %> ``` would instead see **two file fields!** ```html <input class="important" id="Upload" name="Upload" type="file"> <input class="important" id="Name" name="Name" type="file" value="value"> ``` This PR replaces `update` with `merge` in the code of the two helpers, fixing the issue above. The included test verifies the change, since it passes after this PR, but fails before with the following error: ``` Expected: <input type="text" name="title" id="title" value="Hello!" class="important" /> Actual: <input type="password" name="title" id="title" value="Hello!" class="important" /> ```
Diffstat (limited to 'actionmailer')
0 files changed, 0 insertions, 0 deletions