search

igor-alexandrov / wisepdf

Wkhtmltopdf wrapper done right
MIT License
281 stars 20 forks source link

nill error in render_to_string #6

Closed fwoeck closed 12 years ago

fwoeck commented 12 years ago

Hi, when I invoke the library with

   render_to_string pdf: 'Meeting-Report', template: 'meetings/generate_report', layout: 'meeting_report'

I get a

TypeError: can't convert nil into String
    from /opt/rvm/gems/ruby-1.9.3-p125-perf@livesein_ng/gems/wisepdf-1.2.0/lib/wisepdf/render.rb:47:in `join'
    from /opt/rvm/gems/ruby-1.9.3-p125-perf@livesein_ng/gems/wisepdf-1.2.0/lib/wisepdf/render.rb:47:in `default_pdf_render_options'
    from /opt/rvm/gems/ruby-1.9.3-p125-perf@livesein_ng/gems/wisepdf-1.2.0/lib/wisepdf/render.rb:26:in `render_to_string'

replacing line 47 with this:

    def default_pdf_render_options
      Wisepdf::Configuration.options.merge({
        :wkhtmltopdf => nil,
        :layout => false,
        :template => File.join(controller_path || "", action_name || ""),
        :disposition => "inline"
      })
    end

fixes it for me. --Frank

igor-alexandrov commented 12 years ago

Thanks for reporting. Fixed in 1.2.1.