Skip to content

Prawn PDF bug fix & some features - #10

Open
AknEp wants to merge 8 commits into
dpickett:masterfrom
AknEp:master
Open

Prawn PDF bug fix & some features#10
AknEp wants to merge 8 commits into
dpickett:masterfrom
AknEp:master

Conversation

@AknEp

@AknEp AknEp commented Apr 16, 2013

Copy link
Copy Markdown

Hi.

I found some bugs when using has_barcode with Prawn & Prawn::Rails.

I wrote model code below.

has_barcode :barcode, 
  :outputter => :pdf, 
  :type => :code_39,
  :value => Proc.new{|p| p.random_string_of_numbers}

or

has_barcode :barcode, 
  :outputter => :annotate_pdf, 
  :type => :code_39,
  :value => Proc.new{|p| p.random_string_of_numbers}

First, I fixed require bug. ( #de47b49f , #a68637b9 )

Second, I adopted it with annotate_pdf outputter.
I think prawn_rails need to get Prawn::Document instance. This method seems to be good for it.

It's my first time to do "Pull Request", and I'm not good at English.
Please read my RSpec & business code. :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant