activeadmin / activeadmin / 5021

79%

Pull #4615

travis-ci

Change the way of assigning template variables

ActionView::Base#assign method removes all previous view variable assignments if there are any.

For example:

    template.assigns                                                         # => {page_title: "Awesome Title"}
    template.assigns(has_many_block: true)     # this will remove all previous assignments
    template.assignments                                             # => {has_many_block: true}

So, it should not be called when there is a need to add a variable to a template.

This pull request changes this:

    template.assign(foo: 'bar')

to this:

    template.assigns[:foo] = 'bar'

3 of 3 new or added lines in 2 files covered. (100.0%)

3072 of 3773 relevant lines covered (81.42%)

29.97 hits per line