Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

review-initで作成されたテンプレートでPDFの作成エラー #652

Closed
kazken3 opened this issue Jun 18, 2016 · 5 comments
Closed

Comments

@kazken3
Copy link
Contributor

kazken3 commented Jun 18, 2016

review-initで作成されるconfig.ymlには"rights"が含まれていない(コメントアウト)ため、
デフォルトのtemplates/latex/layout.tex.erbを利用してPDF作成処理を進めた際、
escape_latexで以下のエラーが発生します。

review-pdfmaker config.yml
compiling test.tex
/home/x/work/review/lib/review/latexutils.rb:72:in escape_latex': undefined methodgsub' for nil:NilClass (NoMethodError)
from (erb):343:in block in get_template' from (erb):343:inmap'
from (erb):343:in get_template' from /usr/lib/ruby/2.3.0/erb.rb:864:ineval'
from /usr/lib/ruby/2.3.0/erb.rb:864:in result' from /home/x/work/review/lib/review/template.rb:21:inresult'
from /home/x/work/review/lib/review/pdfmaker.rb:371:in get_template' from /home/x/work/review/lib/review/pdfmaker.rb:175:ingenerate_pdf'
from /home/x/work/review/lib/review/pdfmaker.rb:130:in execute' from /home/x/work/review/lib/review/pdfmaker.rb:85:inexecute'
from /home/x/work/review/bin/review-pdfmaker:18:in `

'

つまり、layout.tex.erb内にて"rights"の存在チェックを行わず、
escape_latexに文字列を引き渡している状況であるためのエラーと考えます。

@kazken3
Copy link
Contributor Author

kazken3 commented Jun 18, 2016

すみません、バージョン情報が抜けてました...
review-init 2.0.0
review-pdfmaker 2.0.0
いずれも2.x系のHEADです

@kazken3
Copy link
Contributor Author

kazken3 commented Jun 18, 2016

#653 にてlayout.tex.erbの修正案を出してみました。

@takahashim
Copy link
Collaborator

報告ありがとうございます。これは以前からある現象のようです。#653 で対応してみます

@takahashim
Copy link
Collaborator

#653 をマージしました

@kazken3
Copy link
Contributor Author

kazken3 commented Jun 19, 2016

HEADで確認できましたのでcloseします。

@kazken3 kazken3 closed this as completed Jun 19, 2016
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

No branches or pull requests

2 participants