Passing multiple variables to partial

Question

I get error in partial line #1: undefined local variable or method 'level'

Code in my view:

<div id="comments">
    <% level = 0%>
    <% @comments.each do |comment| %>
      <%=render partial: 'comments/single_comment', locals: {level: level, comment: comment} %>
    <% end %>
    <% if 0 < level %>
    <% (level).times do %>
    </div>
    <% end %>
    <% end %>
  </div>
</div>

And partial first lines:

<% if comment.level < level %>
  <% (level - comment.level).times do %>
  </div>
  <% end %>
<% end %>

Any idea what's wrong here?

Funny thing is `comment` variable in partial works, `level` doesn't. Is this because I set it before the block? — Marcin Doliwa, Feb 12 '13 at 18:37
yep, 100% sure, just double checked it. But it's not `comment.level` problem but `level` variable. — Marcin Doliwa, Feb 12 '13 at 18:39
try to call `comment.level` in your view. I believe the problem is not related to the variable... — gabrielhilal, Feb 12 '13 at 18:43
Looks like it is, I changed `level` to `@level` and left only `locals: {comment: comment}`, in partial I changed local variable `level` to `@level` as well. All works fine this way. — Marcin Doliwa, Feb 12 '13 at 18:47
Try `..., :locals => {:level => level, :comment => comment}` — AlexQueue, Feb 12 '13 at 19:39
@AlexQueue The new hash syntax is fine. He's just passing the `level` variable as a parameter and needs to reference it accordingly. Namely, `params[:level]` — Huy, Feb 12 '13 at 20:02

score 2 · Answer 1 · answered Feb 12 '13 at 20:03

2

Looks like this code should be work(it's no so good, but it should be work) I think problem that you use your partial (comments/single_comment) elsewhere in some part of code what we didn't see without 'level local' :)

answered Feb 12 '13 at 20:03

Dmitry Dedov

1,106
7
15

Unfortunately not :(, just run grep to double check it. – Marcin Doliwa Feb 12 '13 at 21:33
can I see "Application Trace" here?) – Dmitry Dedov Feb 12 '13 at 22:40

score 1 · Answer 2 · answered Feb 12 '13 at 20:30

1

seems like the code is right .. try to check if you have any partial view that use single_comment filename ... and also you can use collection instead doing a loop

<%= render partial: 'comments/single_comment', collections: @comments, locals: { level: level } %>

answered Feb 12 '13 at 20:30

Paul Lopez

11
1
1

just run `grep -r 'single_comment' .` in `views` directory and it's the only place I use it (thanks for collections tip) – Marcin Doliwa Feb 12 '13 at 21:31

score 0 · Answer 3 · answered Mar 28 '16 at 09:14

You are passing arguments to the partial in a wrong way. update it to following.

<%=render partial: 'comments/single_comment', locals: => {:level => level, :comment => comment} %>

now you can access your objects in partials as follows

<% if locals[:level] < level %>
  <% (level - locals[:level]).times do %>
  </div>
  <% end %>
<% end %>

Passing multiple variables to partial

3 Answers3