partials.rb

# stolen from http://github.com/cschneid/irclogger/blob/master/lib/partials.rb
#   and made a lot more robust by me
# this implementation uses erb by default. if you want to use any other template mechanism
#   then replace `erb` on line 13 and line 17 with `haml` or whatever 
module Sinatra::Partials
  def partial(template, *args)
    template_array = template.to_s.split('/')
    template = template_array[0..-2].join('/') + "/_#{template_array[-1]}"
    options = args.last.is_a?(Hash) ? args.pop : {}
    options.merge!(:layout => false)
    locals = options[:locals] || {}
    if collection = options.delete(:collection) then
      collection.inject([]) do |buffer, member|
        buffer << erb(:"#{template}", options.merge(:layout =>
        false, :locals => {template_array[-1].to_sym => member}.merge(locals)))
      end.join("\n")
    else
      erb(:"#{template}", options)
    end
  end
end

So I'm not sure about including this amendment, but people can use it if they want to.

I have various issues with it, first of all being the additional complexity it adds to the code, and second being the approach of .method(:name).call(*args) above .send(:name, *args) (yes, not sure why i prefer the second, does anyone have any well-founded opinions?)

Iain Barnett (@yb66) made a gem of this code - https://github.com/yb66/Sinatra-Partial - with some improvements . I might submit a pull request later this week with some of my ideas about how we can get over the selection of template renderers.

You could also call Sinatra's render method directly, some things like inline markaby or auto-detecting content-types (irrelevant for partials) just won't work.

@rkh that's actually what i was planning on doing, though I didn't know a render method existed, so I was going to write my own. Time to dive into the sinatra source

@lenary https://github.com/sinatra/sinatra/blob/v1.3.1/lib/sinatra/base.rb#L604-605

I spent some time studying base.rb before making my implementation, and I considered using render directly, but it I personally did not like the idea of bypassing the built-in template rendering methods.

I went with your approach (above using send) in the end when I sent a pull request to the Sinatra-Partial gem maintainer. The request has been merged, so it won't be long until there's another gem release i hope. I recommend that you swap over to using the gem. It's unobtrusive and I'll make an effort to support any changes etc. First up on my list: a test suite and travis-ci. And yes, I agree `render` felt a little low-level for my needs too. Soon I'll find an approach I prefer, but a priority is tests.

…

On 10 Apr 2012, at 03:23 PM, Carlos ***@***.*** wrote: I spent some time studying base.rb before making my implementation, and I considered using render directly, but it I personally did not like the idea of bypassing the built-in template rendering methods. --- Reply to this email directly or view it on GitHub: https://gist.github.com/119874

Excellent! I will be switching my current project over to using the gem. Thanks for all of the great work!