Redmine

I've discovered I need to change the .collect{...} syntax to do what I need here, effectively moving the <li> tags inside the loop that collect uses for each array element. However, this is as far as I got.

The best I could come up with, after reading through docs and tutorials, is the following:

<% if @subprojects.any? %>
    <li class="subproj-head"><%=l(:label_subproject_plural)%>:</li>
    <%= @subprojects.collect{|p| "<li>"+ link_to p +"</li>", project_path(p)} %>
<% end %>

As per my understanding of collect, this should simply wrap the generated link in list item tags, but apparently not.

Nobody here able to help? Perhaps at least point me in the right direction?

Hello Andre,
Replace:

<% if @subprojects.any? %>
  <li><%=l(:label_subproject_plural)%>:
    <%= @subprojects.collect{|p| link_to p, project_path(p)}.join(", ").html_safe %></li>
<% end %>

by:

<% if @subprojects.any? %>
<li><%=l(:label_subproject_plural)%>:
  <ul>
    <% @subprojects.each do |p| %>
      <li><%= link_to p, project_path(p) %></li>
    <% end %>
  </ul>
</li>
<% end %>

If it's helpful, you're welcome to endorse me on CoderWall, it could help me out ;)

Cheers,

Thank you so much Martin, you're a legend.. I wasn't sure if the .collect{...} function (?) was required for this part of the template or not, my lack of Ruby knowledge really got me stumped by this.

Your example is simple enough for me to make sense of and it achieves precisely what I needed. :D

Martin Denizet wrote:

If it's helpful, you're welcome to endorse me on CoderWall, it could help me out ;)

I don't have a CoderWall account, but I'll most definitely keep that in mind if I do join the site. :)

Glad it helped ;)
Collect is used to run a block on each element of an array.

For example to get(collect) the first names of all users you would do something like:

users = User.all
firstnames = users.collect { |user| user.firstname } #Array of all the users' first names!

Also can be used to retreive a modified copy of an array as stated in the documentation :

a = [ "a", "b", "c", "d" ]
a.collect { |x| x + "!" }        #=> ["a!", "b!", "c!", "d!"]

I suggest you to play (Not in production!) with the Rails console or IRB to experiment on this like that if you get stuck.

Cheers!