Ok, thanks for all. Sorry if i was a bit persistent. It was my first contribution.
I hope that i could help on other tickets on rails core. Subject closed. On Sep 14, 1:33 pm, Jeremy Kemper <[email protected]> wrote: > Mateo suggests a plain Ruby shorthand since you don't like the > obvious, concise Ruby conditional. Not liking this shorthand is not > cause for some even more obscure render option. > > The ticket was set to "wontfix" already. If you like the feature, > please pursue it as a plugin. > > Best, > jeremy > > On Mon, Sep 14, 2009 at 3:35 AM, rouffj <[email protected]> wrote: > > > Mateo Murphy i think that your code proposal is a little bit less > > clear and more difficult to read. > > > It's for that i created a patch. > > > *For those who would like this to see this patch integrated to rails > > could you update the lighthouse ticket with +1.* > > > lighthouse ticket > > :https://rails.lighthouseapp.com/projects/8994-ruby-on-rails/tickets/3187 > > > Thanks for your feedbacks. > > > On Sep 13, 11:12 pm, Mateo Murphy <[email protected]> wrote: > >> You can already do this in one line: > > >> <%= render :partial => "post", :collection => @posts or > >> render :partial => "no_post" %> > > >> And soon you'll be able to do: > > >> <%= render @posts or render 'no_post' %> > > >> Which is about as concise as one could want! > > >> On 12-Sep-09, at 6:12 AM, Jeremy Kemper wrote: > > >> > On Fri, Sep 11, 2009 at 2:20 PM, rouffj <[email protected]> wrote: > > >> >> Hi, > > >> >> Until now with Rails when we would handle empty collections in apps, > >> >> we had to code it like that : > > >> >> <%# app/views/posts/index.erb %> > >> >> <% if @posts.empty? %> > >> >> <p>Sorry, there is no posts yet !</p> > >> >> <% else %> > >> >> <% render :partial => "post", :collection => @posts %> > > >> >> <% end % > > >> >> Now with my modest contribution we can handle it in one line in > >> >> view : > > >> >> <%# app/views/posts/index.erb %> > >> >> <%= render :partial => "post", :collection => > >> >> @posts, :default_template => "no_post" %> > > >> >> <%# app/views/posts/_no_post.erb %> > > >> >> <p>Sorry, there is no posts yet !</p > > >> >> So we can implement Getting real notion called "The blank slate" more > >> >> easily. > > >> >> One of you can review my patch (to see if i forget something) or > >> >> comment it ? > > >> >> Thanks. > > >> >> --- > >> >> lighthouse ticket > >> >> :https://rails.lighthouseapp.com/projects/8994-ruby-on-rails/tickets/3187 > >> >> patch > >> >> :https://rails.lighthouseapp.com/projects/8994/tickets/3187/a/266544/e... > > >> > Hi rouffj, > > >> > Nice patch, but I think the option will obfuscate rather than clarify. > > >> > Simply using a conditional here is appropriate and clear, not > >> > verbose or hard. > > >> > Best, > >> > jeremy --~--~---------~--~----~------------~-------~--~----~ You received this message because you are subscribed to the Google Groups "Ruby on Rails: Core" group. To post to this group, send email to [email protected] To unsubscribe from this group, send email to [email protected] For more options, visit this group at http://groups.google.com/group/rubyonrails-core?hl=en -~----------~----~----~----~------~----~------~--~---
