review 2.5.0 → 3.0.0.preview1

data/bin/review-init

@@ -7,165 +7,13 @@
 # the GNU LGPL, Lesser General Public License version 2.1.
 # For details of the GNU LGPL, see the file "COPYING".
 
-require 'fileutils'
-require 'optparse'
-
 require 'pathname'
 
 bindir = Pathname.new(__FILE__).realpath.dirname
 $LOAD_PATH.unshift((bindir + '../lib').realpath)
 
-require 'review'
-
-def main
-  @logger = ReVIEW.logger
-  opts = OptionParser.new
-  opts.version = ReVIEW::VERSION
-  opts.banner = "Usage: #{File.basename($PROGRAM_NAME)} [option] dirname"
-  opts.on('-h', '--help', 'print this message and quit.') do
-    puts opts.help
-    exit 0
-  end
-  opts.on('-f', '--force', 'generate files (except *.re) if directory has already existed.') { @force = true }
-  opts.on('-l', '--locale', 'generate locale.yml file.') { @locale = true }
-  opts.on('', '--epub-version VERSION', 'define EPUB version') { |version| @epub_version = version }
-  opts.on('', '--without-doc', "don't generate doc files") { @without_doc = true }
-  begin
-    opts.parse!
-  rescue OptionParser::ParseError => err
-    @logger.error err.message
-    $stderr.puts opts.help
-    exit 1
-  end
-
-  if ARGV.empty?
-    $stderr.puts opts.help
-    exit 1
-  end
-
-  initdir = File.expand_path(ARGV.shift)
-  @review_dir = File.dirname(File.expand_path(__FILE__ + './../'))
-
-  generate_dir(initdir) do |dir|
-    generate_catalog_file(dir)
-    generate_sample(dir)
-    generate_images_dir(dir)
-    generate_cover_image(dir)
-    generate_layout(dir)
-    generate_style(dir)
-    generate_texmacro(dir)
-    generate_config(dir)
-    generate_locale(dir) if @locale
-    generate_rakefile(dir)
-    generate_gemfile(dir)
-    generate_doc(dir) unless @without_doc
-  end
-end
-
-def generate_dir(dir)
-  if File.exist?(dir) && !@force
-    @logger.error "#{dir} already exists."
-    exit
-  end
-  FileUtils.mkdir_p dir
-  yield dir
-end
-
-def generate_sample(dir)
-  File.open("#{dir}/#{File.basename(dir)}.re", 'w') { |file| file.write('= ') } unless @force
-end
-
-def generate_layout(dir)
-  FileUtils.mkdir_p dir + '/layouts'
-end
-
-def generate_catalog_file(dir)
-  File.open(dir + '/catalog.yml', 'w') do |file|
-    file.write <<-EOS
-PREDEF:
-
-CHAPS:
-  - #{File.basename(dir)}.re
-
-APPENDIX:
-
-POSTDEF:
-
-EOS
-  end
-end
-
-def generate_images_dir(dir)
-  FileUtils.mkdir_p dir + '/images'
-end
+require 'review/init'
 
-def generate_cover_image(dir)
-  FileUtils.cp @review_dir + '/test/sample-book/src/images/cover.jpg', dir + '/images/'
+if File.basename($PROGRAM_NAME) == File.basename(__FILE__)
+  ReVIEW::Init.execute(*ARGV)
 end
-
-def generate_config(dir)
-  today = Time.now.strftime('%Y-%m-%d')
-  content = File.read(@review_dir + '/doc/config.yml.sample', encoding: 'utf-8')
-  content.gsub!(/^#\s*coverimage:.*$/, 'coverimage: cover.jpg')
-  content.gsub!(/^#\s*date:.*$/, "date: #{today}")
-  content.gsub!(/^#\s*history:.*$/, %Q(history: [["#{today}"]]))
-  content.gsub!(/^#\s*texstyle:.*$/, 'texstyle: reviewmacro')
-  content.gsub!(/^(#\s*)?stylesheet:.*$/, %Q(stylesheet: ["style.css"]))
-  if @epub_version.to_i == 2
-    content.gsub!(/^#.*epubversion:.*$/, 'epubversion: 2')
-    content.gsub!(/^#.*htmlversion:.*$/, 'htmlversion: 4')
-  end
-  File.open(File.join(dir, 'config.yml'), 'w') { |f| f.write(content) }
-end
-
-def generate_style(dir)
-  FileUtils.cp @review_dir + '/test/sample-book/src/style.css', dir
-end
-
-def generate_texmacro(dir)
-  texmacrodir = dir + '/sty'
-  FileUtils.mkdir_p texmacrodir
-  FileUtils.cp [
-    @review_dir + '/test/sample-book/src/sty/reviewmacro.sty',
-    @review_dir + '/test/sample-book/src/sty/jumoline.sty'
-  ], texmacrodir
-end
-
-def generate_rakefile(dir)
-  FileUtils.mkdir_p dir + '/lib/tasks'
-
-  File.open(dir + '/Rakefile', 'w') do |file|
-    file.write <<-EOS
-Dir.glob('lib/tasks/*.rake').each do |file|
-  load(file)
-end
-EOS
-  end
-
-  FileUtils.cp @review_dir + '/test/sample-book/src/Rakefile',
-               dir + '/lib/tasks/review.rake'
-end
-
-def generate_locale(dir)
-  FileUtils.cp @review_dir + '/lib/review/i18n.yml', dir + '/locale.yml'
-end
-
-def generate_gemfile(dir)
-  File.open(dir + '/Gemfile', 'w') do |file|
-    file.write <<-EOS
-source 'https://rubygems.org'
-
-gem 'rake'
-gem 'review', '#{ReVIEW::VERSION}'
-EOS
-  end
-end
-
-def generate_doc(dir)
-  docdir = dir + '/doc'
-  FileUtils.mkdir_p docdir
-  md_files = Dir.glob(@review_dir + '/doc/*.md').map.to_a
-  FileUtils.cp md_files, docdir
-end
-
-main if File.basename($PROGRAM_NAME) == File.basename(__FILE__)

data/bin/review-preproc

@@ -15,7 +15,6 @@ bindir = Pathname.new(__FILE__).realpath.dirname
 $LOAD_PATH.unshift((bindir + '../lib').realpath)
 
 require 'review/preprocessor'
-require 'review/unfold'
 require 'review/version'
 require 'review/extentions'
 require 'review/logger'
@@ -54,7 +53,6 @@ def main
   opts.on('-d', '--diff', 'Show diff from current file.') { mode = :diff }
   opts.on('--replace', 'Replace file by preprocessed one.') { mode = :replace }
   opts.on('-s', '--strip', 'Strip preprocessor tags.') { mode = :strip }
-  opts.on('--final', 'Unfold text and strip preprocessor tags. (deprecated)') { mode = :final }
   opts.on('--tabwidth=WIDTH', "Replace tabs with space characters. (0: don't replace)") { |width| param['tabwidth'] = width.to_i }
   opts.on('--help', 'Print this message and quit.') do
     puts opts.help
@@ -94,9 +92,6 @@ def main
       File.open(path) do |f|
         ReVIEW::Preprocessor::Strip.new(f).each { |line| puts line }
       end
-    when :final
-      u = ReVIEW::Unfold.new
-      File.open(path) { |f| u.unfold ReVIEW::Preprocessor::Strip.new(f), $stdout }
     else
       raise "must not happen: #{mode}"
     end

data/bin/review-validate

@@ -26,28 +26,42 @@ ARGF.each do |line|
   if line =~ %r{\A//([a-z]+)\{\s*\Z} || line =~ %r{\A//([a-z]+)\[.+?\{\s*\Z}
     # block
     new_block = $1
-    @logger.warn "#{ln}: block #{new_block} started, but previous block #{block} didn't close yet." if block
+    if block
+      @logger.warn "#{ln}: block #{new_block} started, but previous block #{block} didn't close yet."
+    end
     block = new_block
   elsif line =~ %r{\A//\}}
-    @logger.warn "#{ln}: block ended, but not opened." if block.nil?
+    if block.nil?
+      @logger.warn "#{ln}: block ended, but not opened."
+    end
     block = nil
     maxcolcount = 0
     colcount = 0
   elsif line =~ /\A(\d+\.)\s+/
     # number
-    @logger.warn "#{ln}: found '#{$1}' without the head space. Is it correct?" unless %w[list emlist listnum emlistnum cmd image table].include?(block)
+    unless %w[list emlist listnum emlistnum cmd image table].include?(block)
+      @logger.warn "#{ln}: found '#{$1}' without the head space. Is it correct?"
+    end
   elsif line =~ /\A\*\s+/
     # itemize
-    @logger.warn "#{ln}: found '*' without the head space. Is it correct?" unless %w[list emlist listnum emlistnum cmd image table].include?(block)
+    unless %w[list emlist listnum emlistnum cmd image table].include?(block)
+      @logger.warn "#{ln}: found '*' without the head space. Is it correct?"
+    end
   elsif line =~ /\A\s+(\d+\.)\s+/ && line =~ /\A\s+\*\s+/
-    @logger.warn "#{ln}: found itemized list or numbered list in #{block}. Is it correct?" unless Tw[list emlist listnum emlistnum cmd image table].include?(block)
+    unless Tw[list emlist listnum emlistnum cmd image table].include?(block)
+      @logger.warn "#{ln}: found itemized list or numbered list in #{block}. Is it correct?"
+    end
   elsif block == 'table'
     next if line.start_with?('#@')
     if line !~ /\A\-\-\-\-\-/
       # table
       colcount = line.split("\t").size
-      maxcolcount = colcount if maxcolcount == 0
-      @logger.warn "#{ln}: the number of table columns seems mismatch. (#{maxcolcount} != #{colcount})" if colcount != maxcolcount
+      if maxcolcount == 0
+        maxcolcount = colcount
+      end
+      if colcount != maxcolcount
+        @logger.warn "#{ln}: the number of table columns seems mismatch. (#{maxcolcount} != #{colcount})"
+      end
     end
   end
 end

data/bin/review-vol

@@ -25,7 +25,7 @@ def main
 
   part_sensitive = false
   basedir = nil
-  yamlfile = nil
+  yamlfile = 'config.yml'
   opts = OptionParser.new
   opts.version = ReVIEW::VERSION
   opts.on('--yaml=YAML', 'Read configurations from YAML file.') { |yaml| yamlfile = yaml }
@@ -45,20 +45,28 @@ def main
 
   book = basedir ? ReVIEW::Book.load(basedir) : ReVIEW::Book::Base.load
   book.config = @config
-  book.load_config(yamlfile) if yamlfile
+  if yamlfile
+    book.load_config(yamlfile)
+  end
 
   if part_sensitive
     sep = ''
     book.each_part do |part|
       print sep
       sep = "\n"
-      puts "Part #{part.number} #{part.name}" if part.number
-      part.each_chapter { |chap| print_chapter_volume chap }
+      if part.number
+        puts "Part #{part.number} #{part.name}"
+      end
+      part.each_chapter do |chap|
+        print_chapter_volume chap
+      end
       puts '    --------------------'
       print_volume part.volume
     end
   else
-    book.each_chapter { |chap| print_chapter_volume chap }
+    book.each_chapter do |chap|
+      print_chapter_volume chap
+    end
   end
   puts '============================='
   print_volume book.volume # puts "Total #{book.volume}"

data/doc/config.yml.sample

@@ -5,8 +5,7 @@
 # このファイルはUTF-8エンコーディングで記述してください。
 
 # この設定ファイルでサポートするRe:VIEWのバージョン番号。
-# major versionが違うときにはエラーを出す。
-review_version: 2.0
+review_version: 3.0
 
 # ほかの設定ファイルの継承を指定できる。同じパラメータに異なる値がある場合は、
 # 呼び出し元の値が優先される。
@@ -128,7 +127,7 @@ secnolevel: 2
 # ePUB3においてはこの設定によらず必ず作成される
 # mytoc: true
 
-# 表紙にするHTMLファイル。ファイル名を指定すると表紙として入る
+# 表紙にするファイル。ファイル名を指定すると表紙として入る (PDFMaker向けにはLaTeXソース断片、EPUBMaker向けにはHTMLファイル)
 # cover: null
 #
 # 表紙に配置し、書籍の影絵にも利用する画像ファイル。省略した場合はnull (画像を使わない)。画像ディレクトリ内に置いてもディレクトリ名は不要(例: cover.jpg)
@@ -140,7 +139,7 @@ secnolevel: 2
 # 自動生成される大扉ページを上書きするファイル。ファイル名を指定すると大扉として入る (PDFMaker向けにはLaTeXソース断片、EPUBMaker向けにはHTMLファイル)
 # titlefile: null
 #
-# 原書大扉ページにするHTMLファイル。ファイル名を指定すると原書大扉として入る (PDFMaker向けにはLaTeXソース断片、EPUBMaker向けにはHTMLファイル)
+# 原書大扉ページにするファイル。ファイル名を指定すると原書大扉として入る (PDFMaker向けにはLaTeXソース断片、EPUBMaker向けにはHTMLファイル)
 # originaltitlefile: null
 #
 # 権利表記ページファイル。ファイル名を指定すると権利表記として入る (PDFMaker向けにはLaTeXソース断片、EPUBMaker向けにはHTMLファイル)
@@ -180,6 +179,12 @@ secnolevel: 2
 # カタログファイル名を指定する
 # catalogfile: catalog.yml
 
+# reファイルを格納するディレクトリ。省略した場合は以下 (. はカレントディレクトリを示す)
+# contentdir: .
+
+# @<w>命令で使用する単語ファイルのパス
+# words_file: words.csv
+
 # 1ページの行数文字数と1kbごとのページ数を用紙サイズで指定する(A5 or B5)。
 # page_metric: A5
 #
@@ -285,13 +290,13 @@ epubmaker:
 # texcommand: "uplatex"
 #
 # LaTeXのコマンドに渡すオプションを指定する
-# texoptions: null
+# texoptions: "-interaction=nonstopmode -file-line-error"
 #
 # LaTeX用のdvi変換コマンドを指定する(dvipdfmx)
 # dvicommand: "dvipdfmx"
 #
-# LaTeX用のdvi変換コマンドのオプションを指定する
-# dvioptions: "-d 5"
+# LaTeX用のdvi変換コマンドのオプションを指定する。変換が遅い場合は`-d 5 -z 3`等にする
+# dvioptions: "-d 5 -z 9"
 
 # 以下のパラメータを有効にするときには、
 # pdfmaker:

data/doc/config.yml.sample-simple

@@ -1,5 +1,5 @@
 # simple sample of config.yml
-review_version: 2.0
+review_version: 3.0
 
 debug: true
 

data/doc/format.ja.md

@@ -2,7 +2,7 @@
 
 Re:VIEW フォーマットの文法について解説します。Re:VIEW フォーマットはアスキー社（現カドカワ）の EWB を基本としながら、一部に RD や各種 Wiki の文法を取り入れて簡素化しています。
 
-このドキュメントは、Re:VIEW 2.0 に基づいています。
+このドキュメントは、Re:VIEW 3.0 に基づいています。
 
 ## 段落
 
@@ -372,12 +372,18 @@ V1 --> V6 --|
 例: gnuplotの使用
 
 ```
-//graph[sin_x][gnuplot]{
+//graph[sin_x][gnuplot][Gnuplotの使用]{
 plot sin(x)
 //}
 ```
 
-コマンド名には、「`graphviz`」「`gnuplot`」「`blockdiag`」「`aafigure`」のいずれかを指定できます。ツールはそれぞれ別途インストールする必要があります。
+コマンド名には、「`graphviz`」「`gnuplot`」「`blockdiag`」「`aafigure`」「`plantuml`」のいずれかを指定できます。ツールはそれぞれ別途インストールし、インストール先のフォルダ名を指定することなく実行できる (パスを通す) 必要があります。
+
+* Graphviz ( https://www.graphviz.org/ ) : `dot` コマンドへのパスを OS に設定すること
+* Gnuplot ( http://www.gnuplot.info/ ) : `gnuplot` コマンドへのパスを OS に設定すること
+* Blockdiag ( http://blockdiag.com/ ) : `blockdiag` コマンドへのパスを OS に設定すること。PDF を生成する場合は ReportLab もインストールすること
+* aafigure ( https://launchpad.net/aafigure ) : `aafigure` コマンドへのパスを OS に設定すること
+* PlantUML ( http://plantuml.com/ ) : `java` コマンドへのパスを OS に設定し、`plantuml.jar` が作業フォルダにあること
 
 ## 表
 
@@ -657,6 +663,33 @@ Web ハイパーリンクを記述するには、リンクに `@<href>`、アン
 //label[point1]
 ```
 
+## 単語ファイルの展開
+
+キーと値のペアを持つ単語ファイルを用意しておき、キーが指定されたら対応するその値を展開するようにできます。`@<w>{キー}` あるいは `@<wb>{キー}` インライン命令を使います。後者は太字にします。
+
+現在、単語ファイルは CSV（コンマ区切り）形式で拡張子 .csv のものに限定されます。1列目にキー、2列目に値を指定して列挙します。
+
+```
+"LGPL","Lesser General Public License"
+"i18n","""i""nternationalizatio""n"""
+```
+
+単語ファイルのファイルパスは、`config.yml` に `words_file: ファイルパス` で指定します。
+
+例:
+
+```review
+@<w>{LGPL}, @<wb>{i18n}
+```
+
+テキストビルダを使用している場合:
+
+```
+Lesser General Public License, ★"i"nternationalizatio"n"☆
+```
+
+展開されるときには、ビルダごとに決められたエスケープが行われます。インライン命令を含めるといったことはできません。
+
 ## コメント
 
 最終結果に出力されないコメントを記述するには、「`#@#`」を使います。行末までがコメントとして無視されます。
@@ -791,10 +824,13 @@ LaTeXビルダを使用している場合:
 * `@<href>{URL}`, `@<href>{URL, 文字表現}` : ハイパーリンクを作成します（後述）。
 * `@<icon>{識別子}` : インラインの画像を出力します。
 * `@<m>{数式}` : インラインの数式を出力します。
+* `@<w>{キー}` : キー文字列に対応する、単語ファイル内の値文字列を展開します。
+* `@<wb>{キー}` : キー文字列に対応する、単語ファイル内の値文字列を展開し、太字にします。
 * `@<raw>{|ビルダ|〜}` : 生の文字列を出力します。`\}`は`}`に、`\\`は`\`に、`\n`は改行に置き換えられます。
 * `@<embed>{|ビルダ|〜}` : 生の文字列を埋め込みます。`\}`は`}`に、`\\`は`\`に置き換えられます（`\n`はそのままです）。
 * `@<idx>{文字列}` : 文字列を出力するとともに、索引として登録します。索引の使い方については、makeindex.ja.md を参照してください。
 * `@<hidx>{文字列}` : 索引として登録します (idx と異なり、紙面内に出力はしません)。`親索引文字列<<>>子索引文字列` のように親子関係にある索引も定義できます。
+* `@<balloon>{〜}` : コードブロック (emlist など) 内などでのいわゆる吹き出しを作成します。たとえば「`@<balloon>{ABC}`」とすると、「`←ABC`」となります。デフォルトの挙動および表現は簡素なので、より装飾されたものにするにはスタイルシートを改変するか、`review-ext.rb` を使って挙動を書き換える必要があります。
 
 ## 著者用タグ（プリプロセッサ命令）
 

data/doc/format.md

@@ -4,7 +4,7 @@ The document is a brief guide for Re:VIEW markup syntax.
 
 Re:VIEW is based on EWB of ASCII (now KADOKAWA), influenced RD and other Wiki system's syntax.
 
-This document explains about the format of Re:VIEW 2.0.
+This document explains about the format of Re:VIEW 3.0.
 
 
 ## Paragraph
@@ -416,8 +416,14 @@ plot sin(x)
 //}
 ```
 
-You can use `graphviz`, `gnuplot`, `blockdiag`, `aafigure` as the command name.
-Before using these tools, you should installed them.
+You can use `graphviz`, `gnuplot`, `blockdiag`, `aafigure`, and `plantuml` as the command name.
+Before using these tools, you should installed them and configured path appropriately.
+
+* Graphviz ( https://www.graphviz.org/ ) : set path to `dot` command
+* Gnuplot ( http://www.gnuplot.info/ ) : set path to `gnuplot` command
+* Blockdiag ( http://blockdiag.com/ ) : set path to `blockdiag` command. Install ReportLab also to make a PDF
+* aafigure ( https://launchpad.net/aafigure ) : set path to `aafigure` command
+* PlantUML ( http://plantuml.com/ ) : set path to `java` command. place `plantuml.jar` on working folder
 
 ## Tables
 
@@ -712,6 +718,33 @@ Usage:
 //label[point1]
 ```
 
+## Words file
+
+By creating a word file with key / value pair, `@<w>{key}` or `@<wb>{key}` will be expanded the key to the corresponding value. `@<wb>` means bold style.
+
+This word file is a CSV file with extension .csv. This first columns is the key, the second row is the value.
+
+```
+"LGPL","Lesser General Public License"
+"i18n","""i""nternationalizatio""n"""
+```
+
+Specify the word file path in `words_file` parameter of `config.yml`.
+
+Usage:
+
+```review
+@<w>{LGPL}, @<wb>{i18n}
+```
+
+(In HTML:)
+
+```
+Lesser General Public License, ★"i"nternationalizatio"n"☆
+```
+
+Values are escaped by the builder. It is not possible to include inline commands in the value.
+
 ## Comments
 
 If you want to write some comments that do not output in the document, you can use comment notation `#@#`.
@@ -844,10 +877,13 @@ this is a special line.
 @<href>{http://www.google.com/, google}:: hyper link(URL)
 @<icon>{samplephoto}:: inline image
 @<m>{a + \alpha}:: TeX inline equation
+@<w>{key}:: expand the value corresponding to the key.
+@<wb>{key}:: expand the value corresponding to the key with bold style.
 @<raw>{|html|<span>ABC</span>}:: inline raw data inline. `\}` is `}`, `\\` is `\`, and `\n` is newline.
 @<embed>{|html|<span>ABC</span>}:: inline raw data inline. `\}` is `}` and `\\` is `\`.
 @<idx>{string}:: output a string and register it as an index. See makeindex.md.
 @<hidx>{string}:: register a string as an index. A leveled index is expressed like `parent<<>>child`
+@<balloon>{abc}:: inline balloon in code block. For example, `@<balloon>{ABC}` produces `←ABC`. This may seem too simple. To decorate it, modify the style sheet file or override a function by `review-ext.rb`
 ```
 
 ## Commands for Authors (pre-processor commands)

data/doc/format_idg.ja.md

@@ -37,10 +37,6 @@ Re:VIEW フォーマットから、Adobe 社の DTP ソフトウェア「InDesig
 
 `@<keytop>{キー文字}` でキーボードマークを出力します（実際のフォントとの割り当て処理はフィルタや review-ext.rb で調整します）。
 
-## 吹き出し
-
-`@<balloon>{〜}` でコード内などでの吹き出しを作成します。通常、吹き出しは右に寄せられ、記入した箇所から吹き出しまで線が引かれます。
-
 ## DTP 命令指定
 
 `@<dtp>{ 〜 }` で「`<? dtp 〜 ?>`」という XML インストラクションを埋め込みます。処理系に渡す任意の文字列を指定できますが、次のような文字列を特殊文字指定できます。

data/doc/pdfmaker.ja.md

@@ -21,7 +21,7 @@ LaTeX コンパイラコマンドおよびオプションについて、Re:VIEW
 
 ```yaml
 texcommand: uplatex
-texoptions: null
+texoptions: "-interaction=nonstopmode -file-line-error"
 texdocumentclass: ["jsbook", "uplatex,oneside"]
 dvicommand: dvipdfmx
 dvioptions: "-d 5"

data/doc/pdfmaker.md

@@ -21,7 +21,7 @@ Default settings of Re:VIEW is below:
 
 ```yaml
 texcommand: uplatex
-texoptions: null
+texoptions: "-interaction=nonstopmode -file-line-error"
 texdocumentclass: ["jsbook", "uplatex,oneside"]
 dvicommand: dvipdfmx
 dvioptions: "-d 5"

data/doc/sample.css

@@ -75,6 +75,13 @@ span.kw {
     font-weight: bold;
 }
 
+span.balloon {
+    font-size: 0.9em;
+}
+span.balloon:before {
+    content: "←";
+}
+
 div.column {
     padding: 0.5em;
     background: #ffd0d0;

data/lib/epubmaker/content.rb

@@ -72,20 +72,39 @@ module EPUBMaker
 
     # Complement other parameters by using file parameter.
     def complement
-      @id = @file.gsub(%r{[\\/\. ]}, '-') if @id.nil?
-      @id = "rv-#{@id}" if @id =~ /\A[^a-z]/i
-      @media = @file.sub(/.+\./, '').downcase if !@file.nil? && @media.nil?
+      if @id.nil?
+        @id = @file.gsub(%r{[\\/\. ]}, '-')
+      end
+      if @id =~ /\A[^a-z]/i
+        @id = "rv-#{@id}"
+      end
+
+      if !@file.nil? && @media.nil?
+        @media = @file.sub(/.+\./, '').downcase
+      end
 
-      @media = 'application/xhtml+xml' if @media == 'xhtml' || @media == 'xml' || @media == 'html'
-      @media = 'text/css' if @media == 'css'
-      @media = 'image/jpeg' if @media == 'jpg' || @media == 'jpeg' || @media == 'image/jpg'
-      @media = 'image/png' if @media == 'png'
-      @media = 'image/gif' if @media == 'gif'
-      @media = 'image/svg+xml' if @media == 'svg' || @media == 'image/svg'
-      @media = 'application/vnd.ms-opentype' if @media == 'ttf' || @media == 'otf'
-      @media = 'application/font-woff' if @media == 'woff'
+      case @media
+      when 'xhtml', 'xml', 'html'
+        @media = 'application/xhtml+xml'
+      when 'css'
+        @media = 'text/css'
+      when 'jpg', 'jpeg', 'image/jpg'
+        @media = 'image/jpeg'
+      when 'png'
+        @media = 'image/png'
+      when 'gif'
+        @media = 'image/gif'
+      when 'svg', 'image/svg'
+        @media = 'image/svg+xml'
+      when 'ttf', 'otf'
+        @media = 'application/vnd.ms-opentype'
+      when 'woff'
+        @media = 'application/font-woff'
+      end
 
-      raise "Type error: #{id}, #{file}, #{media}, #{title}, #{notoc}" if @id.nil? || @file.nil? || @media.nil?
+      if @id.nil? || @file.nil? || @media.nil?
+        raise "Type error: #{id}, #{file}, #{media}, #{title}, #{notoc}"
+      end
     end
   end
 end

data/lib/epubmaker/epubcommon.rb

@@ -36,13 +36,18 @@ module EPUBMaker
       if @producer.config['coverimage']
         file = nil
         @producer.contents.each do |item|
-          next if !item.media.start_with?('image') || item.file !~ /#{@producer.config["coverimage"]}\Z/
+          if !item.media.start_with?('image') || item.file !~ /#{@producer.config["coverimage"]}\Z/
+            next
+          end
 
           s << %Q(    <meta name="cover" content="#{item.id}"/>\n)
           file = item.file
           break
         end
-        raise "coverimage #{@producer.config['coverimage']} not found. Abort." if file.nil?
+
+        if file.nil?
+          raise "coverimage #{@producer.config['coverimage']} not found. Abort."
+        end
       end
       s
     end
@@ -233,7 +238,9 @@ EOT
 
       @body << %Q(      <tr><th>ISBN</th><td>#{@producer.isbn_hyphen}</td></tr>\n) if @producer.isbn_hyphen
       @body << %Q(    </table>\n)
-      @body << %Q(    <p class="copyright">#{join_with_separator(@producer.config.names_of('rights').map { |m| CGI.escapeHTML(m) }, '<br />')}</p>\n) if @producer.config['rights'] && !@producer.config['rights'].empty?
+      if @producer.config['rights'] && !@producer.config['rights'].empty?
+        @body << %Q(    <p class="copyright">#{join_with_separator(@producer.config.names_of('rights').map { |m| CGI.escapeHTML(m) }, '<br />')}</p>\n)
+      end
       @body << %Q(  </div>\n)
 
       @language = @producer.config['language']

data/lib/epubmaker/epubv2.rb

@@ -63,7 +63,9 @@ module EPUBMaker
         next unless @producer.config[role]
         @producer.config.names_of(role).each do |v|
           s << %Q(    <dc:contributor opf:role="#{role}">#{CGI.escapeHTML(v)}</dc:contributor>\n)
-          s << %Q(    <dc:publisher>#{v}</dc:publisher>\n) if role == 'prt'
+          if role == 'prt'
+            s << %Q(    <dc:publisher>#{v}</dc:publisher>\n)
+          end
         end
       end
 
@@ -124,8 +126,14 @@ EOT
     def produce(epubfile, basedir, tmpdir)
       produce_write_common(basedir, tmpdir)
 
-      File.open("#{tmpdir}/OEBPS/#{@producer.config['bookname']}.ncx", 'w') { |f| @producer.ncx(f, @producer.config['epubmaker']['ncxindent']) }
-      File.open("#{tmpdir}/OEBPS/#{@producer.config['bookname']}-toc.#{@producer.config['htmlext']}", 'w') { |f| @producer.mytoc(f) } if @producer.config['mytoc']
+      File.open("#{tmpdir}/OEBPS/#{@producer.config['bookname']}.ncx", 'w') do |f|
+        @producer.ncx(f, @producer.config['epubmaker']['ncxindent'])
+      end
+      if @producer.config['mytoc']
+        File.open("#{tmpdir}/OEBPS/#{@producer.config['bookname']}-toc.#{@producer.config['htmlext']}", 'w') do |f|
+          @producer.mytoc(f)
+        end
+      end
 
       @producer.call_hook(@producer.config['epubmaker']['hook_prepack'], tmpdir)
       expoter = EPUBMaker::ZipExporter.new(tmpdir, @producer.config)