Ruby on Rails

You're absolutely right Colin, thanks. I'm not that familiar with that simplified syntax of if/else statements, so I learned quite a bit from this exchange! 

On Fri, Feb 20, 2015 at 3:57 AM, Colin Law <clanlaw@gmail.com> wrote:

On 20 February 2015 at 00:01, TTambe <tambe257@gmail.com> wrote:
> I actually ended up doing this which worked:
>
>     <% last_artist ||= nil %>
>
>   <table id="artist">
>     <tr>
>       <th>Artist</th>
>       <th></th>
>       <th>Album</th>
>    </tr>
> <% @pin_albums.each do |pin| %>
>   <% if last_artist != pin.artist %>
>     <tr>
>       <td><%= pin.artist %></td>
>       <td><%= link_to image_tag(pin.image), pin %></td>
>       <td><%= link_to pin.album, copy_pin_path(pin) %></td>
>     </tr>
>   <% else %>
>     <tr>
>       <td></td>
>       <td><%= link_to image_tag(pin.image), pin %></td>
>       <td><%= link_to pin.album, copy_pin_path(pin) %></td>
>     </tr>
>   <% end %>
>   <% last_artist = pin.artist %>
> <% end %>

You are right in that the key was to initialise last_artist at an
appropriate scope.
There are an awful lot of repeated lines there, Suppose you wanted to
change the detail of the contents of the last cell in each row, you
would have to change it in two places.  You should put the if
statement round just the first cell not round the whole row.  so for
that cell use something like
<td>
  <%= (last_artist == pin.artist)  ?  ''   :   pin.artist %>
</td>
Or use an if statement inside the <td> </td> if you prefer.  That is
two single quotes in the line above of course.

Colin

--
You received this message because you are subscribed to the Google Groups "Ruby on Rails: Talk" group.
To unsubscribe from this group and stop receiving emails from it, send an email to rubyonrails-talk+unsubscribe@googlegroups.com.
To post to this group, send email to rubyonrails-talk@googlegroups.com.
To view this discussion on the web visit https://groups.google.com/d/msgid/rubyonrails-talk/CAL%3D0gLuBuJUBego%3Dq_5jDGaBRCA0rVHRwHdZDwjKrg665Hbqiw%40mail.gmail.com.

For more options, visit https://groups.google.com/d/optout.