review 2.3.0 → 2.4.0

data/lib/epubmaker/epubv2.rb

@@ -1,5 +1,3 @@
-# encoding: utf-8
-
 # = epubv2.rb -- EPUB version 2 producer.
 #
 # Copyright (c) 2010-2017 Kenshi Muto and Masayoshi Takahashi
@@ -15,10 +13,9 @@ require 'cgi'
 require 'epubmaker/zip_exporter'
 
 module EPUBMaker
-
   # EPUBv2 is EPUB version 2 producer.
   class EPUBv2 < EPUBCommon
-    # Construct object with parameter hash +params+ and message resource hash +res+.
+    # Construct object with parameter hash +config+ and message resource hash +res+.
     def initialize(producer)
       super
     end
@@ -32,43 +29,41 @@ module EPUBMaker
 
       tmplfile = File.expand_path('./opf/epubv2.opf.erb', ReVIEW::Template::TEMPLATE_DIR)
       tmpl = ReVIEW::Template.load(tmplfile)
-      return tmpl.result(binding)
+      tmpl.result(binding)
     end
 
     def opf_metainfo
-      s = ""
+      s = ''
       %w[title language date type format source description relation coverage subject rights].each do |item|
-        next unless @producer.params[item]
-        if @producer.params[item].kind_of?(Array)
-          s << @producer.params.names_of(item).map {|i| %Q[    <dc:#{item}>#{CGI.escapeHTML(i)}</dc:#{item}>\n]}.join
+        next unless @producer.config[item]
+        if @producer.config[item].is_a?(Array)
+          s << @producer.config.names_of(item).map { |i| %Q(    <dc:#{item}>#{CGI.escapeHTML(i)}</dc:#{item}>\n) }.join
         else
-          s << %Q[    <dc:#{item}>#{CGI.escapeHTML(@producer.params.name_of(item))}</dc:#{item}>\n]
+          s << %Q(    <dc:#{item}>#{CGI.escapeHTML(@producer.config.name_of(item))}</dc:#{item}>\n)
         end
       end
 
       # ID
-      if @producer.params["isbn"].nil?
-        s << %Q[    <dc:identifier id="BookId">#{@producer.params["urnid"]}</dc:identifier>\n]
+      if @producer.config['isbn'].nil?
+        s << %Q(    <dc:identifier id="BookId">#{@producer.config['urnid']}</dc:identifier>\n)
       else
-        s << %Q[    <dc:identifier id="BookId" opf:scheme="ISBN">#{@producer.params["isbn"]}</dc:identifier>\n]
+        s << %Q(    <dc:identifier id="BookId" opf:scheme="ISBN">#{@producer.config['isbn']}</dc:identifier>\n)
       end
 
       # creator (should be array)
       %w[aut a-adp a-ann a-arr a-art a-asn a-aqt a-aft a-aui a-ant a-bkp a-clb a-cmm a-dsr a-edt a-ill a-lyr a-mdc a-mus a-nrt a-oth a-pht a-prt a-red a-rev a-spn a-ths a-trc a-trl].each do |role|
-        next unless @producer.params[role]
-        @producer.params.names_of(role).each do |v|
-          s << %Q[    <dc:creator opf:role="#{role.sub('a-', '')}">#{CGI.escapeHTML(v)}</dc:creator>\n]
+        next unless @producer.config[role]
+        @producer.config.names_of(role).each do |v|
+          s << %Q(    <dc:creator opf:role="#{role.sub('a-', '')}">#{CGI.escapeHTML(v)}</dc:creator>\n)
         end
       end
 
       # contributor (should be array)
       %w[adp ann arr art asn aqt aft aui ant bkp clb cmm dsr edt ill lyr mdc mus nrt oth pht prt red rev spn ths trc trl].each do |role|
-        next unless @producer.params[role]
-        @producer.params.names_of(role).each do |v|
-          s << %Q[    <dc:contributor opf:role="#{role}">#{CGI.escapeHTML(v)}</dc:contributor>\n]
-          if role == "prt"
-            s << %Q[    <dc:publisher>#{v}</dc:publisher>\n]
-          end
+        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'
         end
       end
 
@@ -76,40 +71,40 @@ module EPUBMaker
     end
 
     def opf_manifest
-      s = ""
+      s = ''
       s << <<EOT
   <manifest>
-    <item id="ncx" href="#{@producer.params["bookname"]}.ncx" media-type="application/x-dtbncx+xml"/>
-    <item id="#{@producer.params["bookname"]}" href="#{@producer.params["cover"]}" media-type="application/xhtml+xml"/>
+    <item id="ncx" href="#{@producer.config['bookname']}.ncx" media-type="application/x-dtbncx+xml"/>
+    <item id="#{@producer.config['bookname']}" href="#{@producer.config['cover']}" media-type="application/xhtml+xml"/>
 EOT
 
-      s << %Q[    <item id="toc" href="#{@producer.params["bookname"]}-toc.#{@producer.params["htmlext"]}" media-type="application/xhtml+xml"/>\n] if @producer.params["toc"] && @producer.params["mytoc"]
+      s << %Q(    <item id="toc" href="#{@producer.config['bookname']}-toc.#{@producer.config['htmlext']}" media-type="application/xhtml+xml"/>\n) if @producer.config['toc'] && @producer.config['mytoc']
 
       @producer.contents.each do |item|
         next if item.file =~ /#/ # skip subgroup
-        s << %Q[    <item id="#{item.id}" href="#{item.file}" media-type="#{item.media}"/>\n]
+        s << %Q(    <item id="#{item.id}" href="#{item.file}" media-type="#{item.media}"/>\n)
       end
-      s << %Q[  </manifest>\n]
+      s << %Q(  </manifest>\n)
       s
     end
 
     def opf_tocx
-      if @producer.params["epubmaker"]["cover_linear"] && @producer.params["epubmaker"]["cover_linear"] != "no"
-        cover_linear = "yes"
-      else
-        cover_linear = "no"
-      end
+      cover_linear = if @producer.config['epubmaker']['cover_linear'] && @producer.config['epubmaker']['cover_linear'] != 'no'
+                       'yes'
+                     else
+                       'no'
+                     end
 
-      s = ""
-      s << %Q[  <spine toc="ncx">\n]
-      s << %Q[    <itemref idref="#{@producer.params["bookname"]}" linear="#{cover_linear}"/>\n]
-      s << %Q[    <itemref idref="toc" />\n] unless @producer.params["mytoc"].nil?
+      s = ''
+      s << %Q(  <spine toc="ncx">\n)
+      s << %Q(    <itemref idref="#{@producer.config['bookname']}" linear="#{cover_linear}"/>\n)
+      s << %Q(    <itemref idref="toc" />\n) unless @producer.config['mytoc'].nil?
 
       @producer.contents.each do |item|
         next if item.media !~ /xhtml\+xml/ # skip non XHTML
-        s << %Q[    <itemref idref="#{item.id}"/>\n]
+        s << %Q(    <itemref idref="#{item.id}"/>\n)
       end
-      s << %Q[  </spine>\n]
+      s << %Q(  </spine>\n)
       s
     end
 
@@ -120,8 +115,7 @@ EOT
       @ncx_navmap = ncx_navmap(indentarray)
 
       tmplfile = File.expand_path('./ncx/epubv2.ncx.erb', ReVIEW::Template::TEMPLATE_DIR)
-      tmpl = ReVIEW::Template.load(tmplfile)
-      return tmpl.result(binding)
+      ReVIEW::Template.load(tmplfile).result(binding)
     end
 
     # Produce EPUB file +epubfile+.
@@ -130,13 +124,12 @@ EOT
     def produce(epubfile, basedir, tmpdir)
       produce_write_common(basedir, tmpdir)
 
-      File.open("#{tmpdir}/OEBPS/#{@producer.params["bookname"]}.ncx", "w") {|f| @producer.ncx(f, @producer.params["epubmaker"]["ncxindent"]) }
-      File.open("#{tmpdir}/OEBPS/#{@producer.params["bookname"]}-toc.#{@producer.params["htmlext"]}", "w") {|f| @producer.mytoc(f) } unless @producer.params["mytoc"].nil?
+      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']
 
-      @producer.call_hook(@producer.params["epubmaker"]["hook_prepack"], tmpdir)
-      expoter = EPUBMaker::ZipExporter.new(tmpdir, @producer.params)
+      @producer.call_hook(@producer.config['epubmaker']['hook_prepack'], tmpdir)
+      expoter = EPUBMaker::ZipExporter.new(tmpdir, @producer.config)
       expoter.export_zip(epubfile)
     end
-
   end
 end

data/lib/epubmaker/epubv3.rb

@@ -1,5 +1,3 @@
-# encoding: utf-8
-
 # = epubv3.rb -- EPUB version 3 producer.
 #
 # Copyright (c) 2010-2017 Kenshi Muto
@@ -14,18 +12,13 @@ require 'epubmaker/epubcommon'
 require 'epubmaker/zip_exporter'
 
 module EPUBMaker
-
   # EPUBv3 is EPUB version 3 producer.
   class EPUBv3 < EPUBCommon
-    # Construct object with parameter hash +params+ and message resource hash +res+.
+    # Construct object with parameter hash +config+ and message resource hash +res+.
     def initialize(producer)
       super
       @opf_prefix = {}
-      if @producer.params["opf_prefix"].present?
-        @producer.params["opf_prefix"].each do |k, v|
-          @opf_prefix[k] = v
-        end
-      end
+      @producer.config['opf_prefix'].each { |k, v| @opf_prefix[k] = v } if @producer.config['opf_prefix'].present?
     end
 
     # Return opf file content.
@@ -33,189 +26,186 @@ module EPUBMaker
       @opf_metainfo = opf_metainfo
       @opf_manifest = opf_manifest
       @opf_toc = opf_tocx
-      @package_attrs = ""
+      @package_attrs = ''
 
-      if !@opf_prefix.empty?
-        prefixes_str = @opf_prefix.map{|k,v| %Q|#{k}: #{v}| }.join(" ")
-        @package_attrs << " prefix=\"#{prefixes_str}\""
+      if @opf_prefix && @opf_prefix.size > 0
+        prefixes_str = @opf_prefix.map { |k, v| %Q(#{k}: #{v}) }.join(' ')
+        @package_attrs << %Q( prefix="#{prefixes_str}")
       end
 
       tmplfile = File.expand_path('./opf/epubv3.opf.erb', ReVIEW::Template::TEMPLATE_DIR)
-      tmpl = ReVIEW::Template.load(tmplfile)
-      return tmpl.result(binding)
+      ReVIEW::Template.load(tmplfile).result(binding)
     end
 
     def opf_metainfo
-      s = ""
+      s = ''
       %w[title language date type format source description relation coverage subject rights].each do |item|
-        next unless @producer.params[item]
-        if @producer.params[item].kind_of?(Array)
-          @producer.params[item].each_with_index do |v, i|
-            if v.kind_of?(Hash)
-              s << %Q[    <dc:#{item} id="#{item}-#{i}">#{CGI.escapeHTML(v["name"])}</dc:#{item}>\n]
+        next unless @producer.config[item]
+        if @producer.config[item].is_a?(Array)
+          @producer.config[item].each_with_index do |v, i|
+            if v.is_a?(Hash)
+              s << %Q(    <dc:#{item} id="#{item}-#{i}">#{CGI.escapeHTML(v['name'])}</dc:#{item}>\n)
               v.each_pair do |name, val|
-                next if name == "name"
-                s << %Q[    <meta refines="##{item}-#{i}" property="#{name}">#{CGI.escapeHTML(val)}</meta>\n]
+                next if name == 'name'
+                s << %Q(    <meta refines="##{item}-#{i}" property="#{name}">#{CGI.escapeHTML(val)}</meta>\n)
               end
             else
-              s << %Q[    <dc:#{item} id="#{item}-#{i}">#{CGI.escapeHTML(v.to_s)}</dc:#{item}>\n]
+              s << %Q(    <dc:#{item} id="#{item}-#{i}">#{CGI.escapeHTML(v.to_s)}</dc:#{item}>\n)
             end
           end
-        elsif @producer.params[item].kind_of?(Hash)
-          s << %Q[    <dc:#{item} id="#{item}">#{CGI.escapeHTML(@producer.params[item]["name"])}</dc:#{item}>\n]
-          @producer.params[item].each_pair do |name, val|
-            next if name == "name"
-            s << %Q[    <meta refines="##{item}" property="#{name}">#{CGI.escapeHTML(val)}</meta>\n]
+        elsif @producer.config[item].is_a?(Hash)
+          s << %Q(    <dc:#{item} id="#{item}">#{CGI.escapeHTML(@producer.config[item]['name'])}</dc:#{item}>\n)
+          @producer.config[item].each_pair do |name, val|
+            next if name == 'name'
+            s << %Q(    <meta refines="##{item}" property="#{name}">#{CGI.escapeHTML(val)}</meta>\n)
           end
         else
-          s << %Q[    <dc:#{item} id="#{item}">#{CGI.escapeHTML(@producer.params[item].to_s)}</dc:#{item}>\n]
+          s << %Q(    <dc:#{item} id="#{item}">#{CGI.escapeHTML(@producer.config[item].to_s)}</dc:#{item}>\n)
         end
       end
 
-      s << %Q[    <meta property="dcterms:modified">#{@producer.params["modified"]}</meta>\n]
+      s << %Q(    <meta property="dcterms:modified">#{@producer.config['modified']}</meta>\n)
 
       # ID
-      if @producer.params["isbn"].nil?
-        s << %Q[    <dc:identifier id="BookId">#{@producer.params["urnid"]}</dc:identifier>\n]
+      if @producer.config['isbn'].nil?
+        s << %Q(    <dc:identifier id="BookId">#{@producer.config['urnid']}</dc:identifier>\n)
       else
-        s << %Q[    <dc:identifier id="BookId">#{@producer.params["isbn"]}</dc:identifier>\n]
+        s << %Q(    <dc:identifier id="BookId">#{@producer.config['isbn']}</dc:identifier>\n)
       end
 
       # creator (should be array)
       %w[a-adp a-ann a-arr a-art a-asn a-aqt a-aft a-aui a-ant a-bkp a-clb a-cmm a-csl a-dsr a-edt a-ill a-lyr a-mdc a-mus a-nrt a-oth a-pht a-prt a-red a-rev a-spn a-ths a-trc a-trl aut].each do |role|
-        next unless @producer.params[role]
-        @producer.params[role].each_with_index do |v, i|
-          if v.kind_of?(Hash)
-            s << %Q[    <dc:creator id="#{role}-#{i}">#{CGI.escapeHTML(v["name"])}</dc:creator>\n]
-            s << %Q[    <meta refines="##{role}-#{i}" property="role" scheme="marc:relators">#{role.sub('a-', '')}</meta>\n]
+        next unless @producer.config[role]
+        @producer.config[role].each_with_index do |v, i|
+          if v.is_a?(Hash)
+            s << %Q(    <dc:creator id="#{role}-#{i}">#{CGI.escapeHTML(v['name'])}</dc:creator>\n)
+            s << %Q(    <meta refines="##{role}-#{i}" property="role" scheme="marc:relators">#{role.sub('a-', '')}</meta>\n)
             v.each_pair do |name, val|
-              next if name == "name"
-              s << %Q[    <meta refines="##{role.sub('a-', '')}-#{i}" property="#{name}">#{CGI.escapeHTML(val)}</meta>\n]
+              next if name == 'name'
+              s << %Q(    <meta refines="##{role.sub('a-', '')}-#{i}" property="#{name}">#{CGI.escapeHTML(val)}</meta>\n)
             end
           else
-            s << %Q[    <dc:creator id="#{role}-#{i}">#{CGI.escapeHTML(v)}</dc:creator>\n]
-            s << %Q[    <meta refines="##{role}-#{i}" property="role" scheme="marc:relators">#{role.sub('a-', '')}</meta>\n]
+            s << %Q(    <dc:creator id="#{role}-#{i}">#{CGI.escapeHTML(v)}</dc:creator>\n)
+            s << %Q(    <meta refines="##{role}-#{i}" property="role" scheme="marc:relators">#{role.sub('a-', '')}</meta>\n)
           end
         end
       end
 
       # contributor (should be array)
       %w[adp ann arr art asn aqt aft aui ant bkp clb cmm csl dsr edt ill lyr mdc mus nrt oth pbd pbl pht prt red rev spn ths trc trl].each do |role|
-        next unless @producer.params[role]
-        @producer.params[role].each_with_index do |v, i|
-          if v.kind_of?(Hash)
-            s << %Q[    <dc:contributor id="#{role}-#{i}">#{CGI.escapeHTML(v["name"])}</dc:contributor>\n]
-            s << %Q[    <meta refines="##{role}-#{i}" property="role" scheme="marc:relators">#{role}</meta>\n]
+        next unless @producer.config[role]
+        @producer.config[role].each_with_index do |v, i|
+          if v.is_a?(Hash)
+            s << %Q(    <dc:contributor id="#{role}-#{i}">#{CGI.escapeHTML(v['name'])}</dc:contributor>\n)
+            s << %Q(    <meta refines="##{role}-#{i}" property="role" scheme="marc:relators">#{role}</meta>\n)
             v.each_pair do |name, val|
-              next if name == "name"
-              s << %Q[    <meta refines="##{role}-#{i}" property="#{name}">#{CGI.escapeHTML(val)}</meta>\n]
+              next if name == 'name'
+              s << %Q(    <meta refines="##{role}-#{i}" property="#{name}">#{CGI.escapeHTML(val)}</meta>\n)
             end
           else
-            s << %Q[    <dc:contributor id="#{role}-#{i}">#{CGI.escapeHTML(v)}</dc:contributor>\n]
-            s << %Q[    <meta refines="##{role}-#{i}" property="role" scheme="marc:relators">#{role}</meta>\n]
+            s << %Q(    <dc:contributor id="#{role}-#{i}">#{CGI.escapeHTML(v)}</dc:contributor>\n)
+            s << %Q(    <meta refines="##{role}-#{i}" property="role" scheme="marc:relators">#{role}</meta>\n)
           end
 
-          if role == "prt" || role == "pbl"
-            if v.kind_of?(Hash)
-              s << %Q[    <dc:publisher id="pub-#{role}-#{i}">#{CGI.escapeHTML(v["name"])}</dc:publisher>\n]
-              s << %Q[    <meta refines="#pub-#{role}-#{i}" property="role" scheme="marc:relators">#{role}</meta>\n]
+          if %w[prt pbl].include?(role)
+            if v.is_a?(Hash)
+              s << %Q(    <dc:publisher id="pub-#{role}-#{i}">#{CGI.escapeHTML(v['name'])}</dc:publisher>\n)
+              s << %Q(    <meta refines="#pub-#{role}-#{i}" property="role" scheme="marc:relators">#{role}</meta>\n)
               v.each_pair do |name, val|
-                next if name == "name"
-                s << %Q[    <meta refines="#pub-#{role}-#{i}" property="#{name}">#{CGI.escapeHTML(val)}</meta>\n]
+                next if name == 'name'
+                s << %Q(    <meta refines="#pub-#{role}-#{i}" property="#{name}">#{CGI.escapeHTML(val)}</meta>\n)
               end
             else
-              s << %Q[    <dc:publisher id="pub-#{role}-#{i}">#{CGI.escapeHTML(v)}</dc:publisher>\n]
-              s << %Q[    <meta refines="#pub-#{role}-#{i}" property="role" scheme="marc:relators">prt</meta>\n]
+              s << %Q(    <dc:publisher id="pub-#{role}-#{i}">#{CGI.escapeHTML(v)}</dc:publisher>\n)
+              s << %Q(    <meta refines="#pub-#{role}-#{i}" property="role" scheme="marc:relators">prt</meta>\n)
             end
           end
         end
       end
 
       ## add custom <meta> element
-      if @producer.params["opf_meta"].present?
-        @producer.params["opf_meta"].each do |k, v|
-          s << %Q[    <meta property="#{k}">#{CGI.escapeHTML(v)}</meta>\n]
-        end
-      end
+      @producer.config['opf_meta'].each { |k, v| s << %Q(    <meta property="#{k}">#{CGI.escapeHTML(v)}</meta>\n) } if @producer.config['opf_meta'].present?
 
       s
     end
 
     def opf_manifest
-      s = ""
+      s = ''
       s << <<EOT
   <manifest>
-    <item properties="nav" id="#{@producer.params["bookname"]}-toc.#{@producer.params["htmlext"]}" href="#{@producer.params["bookname"]}-toc.#{@producer.params["htmlext"]}" media-type="application/xhtml+xml"/>
-    <item id="#{@producer.params["bookname"]}" href="#{@producer.params["cover"]}" media-type="application/xhtml+xml"/>
+    <item properties="nav" id="#{@producer.config['bookname']}-toc.#{@producer.config['htmlext']}" href="#{@producer.config['bookname']}-toc.#{@producer.config['htmlext']}" media-type="application/xhtml+xml"/>
+    <item id="#{@producer.config['bookname']}" href="#{@producer.config['cover']}" media-type="application/xhtml+xml"/>
 EOT
 
-      if @producer.params["coverimage"]
+      if @producer.config['coverimage']
         @producer.contents.each do |item|
-          if item.media.start_with?('image') && File.basename(item.file) == @producer.params["coverimage"]
-            s << %Q[    <item properties="cover-image" id="cover-#{item.id}" href="#{item.file}" media-type="#{item.media}"/>\n]
-            item.id = nil
-            break
-          end
+          next if !item.media.start_with?('image') || File.basename(item.file) != @producer.config['coverimage']
+          s << %Q(    <item properties="cover-image" id="cover-#{item.id}" href="#{item.file}" media-type="#{item.media}"/>\n)
+          item.id = nil
+          break
         end
       end
 
       @producer.contents.each do |item|
         next if item.file =~ /#/ || item.id.nil? # skip subgroup, or id=nil (for cover)
-        propstr = ""
+        propstr = ''
         if item.properties.size > 0
-          propstr = %Q[ properties="#{item.properties.sort.uniq.join(" ")}"]
+          propstr = %Q( properties="#{item.properties.sort.uniq.join(' ')}")
         end
-        s << %Q[    <item id="#{item.id}" href="#{item.file}" media-type="#{item.media}"#{propstr}/>\n]
+        s << %Q(    <item id="#{item.id}" href="#{item.file}" media-type="#{item.media}"#{propstr}/>\n)
       end
-      s << %Q[  </manifest>\n]
+      s << %Q(  </manifest>\n)
 
       s
     end
 
     def opf_tocx
-      if @producer.params["epubmaker"]["cover_linear"] && @producer.params["epubmaker"]["cover_linear"] != "no"
-        cover_linear = "yes"
+      if @producer.config['epubmaker']['cover_linear'] && @producer.config['epubmaker']['cover_linear'] != 'no'
+        cover_linear = 'yes'
       else
-        cover_linear = "no"
+        cover_linear = 'no'
       end
 
-      s = ""
-      if @producer.params["direction"]
-        s << %Q[  <spine page-progression-direction="#{@producer.params["direction"]}">\n]
+      s = ''
+      if @producer.config['direction']
+        s << %Q(  <spine page-progression-direction="#{@producer.config['direction']}">\n)
       else
-        s << %Q[  <spine>\n]
+        s << %Q(  <spine>\n)
       end
-      s << %Q[    <itemref idref="#{@producer.params["bookname"]}" linear="#{cover_linear}"/>\n]
-      s << %Q[    <itemref idref="#{@producer.params["bookname"]}-toc.#{@producer.params["htmlext"]}" />\n] if @producer.params["toc"]
+      s << %Q(    <itemref idref="#{@producer.config['bookname']}" linear="#{cover_linear}"/>\n)
 
+      toc = nil
       @producer.contents.each do |item|
         next if item.media !~ /xhtml\+xml/ # skip non XHTML
-        s << %Q[    <itemref idref="#{item.id}"/>\n]
+        if toc.nil? && item.chaptype != 'pre'
+          s << %Q(    <itemref idref="#{@producer.config['bookname']}-toc.#{@producer.config['htmlext']}" />\n) if @producer.config['toc']
+          toc = true
+        end
+        s << %Q(    <itemref idref="#{item.id}"/>\n)
       end
-      s << %Q[  </spine>\n]
+      s << %Q(  </spine>\n)
 
       s
     end
 
     def ncx(indentarray)
-      if @producer.params["epubmaker"]["flattoc"].nil?
-        ncx_main = hierarchy_ncx("ol")
-      else
-        ncx_main = flat_ncx("ol", @producer.params["epubmaker"]["flattocindent"])
-      end
+      ncx_main = if @producer.config['epubmaker']['flattoc'].nil?
+                   hierarchy_ncx('ol')
+                 else
+                   flat_ncx('ol', @producer.config['epubmaker']['flattocindent'])
+                 end
 
       @body = <<EOT
   <nav xmlns:epub="http://www.idpf.org/2007/ops" epub:type="toc" id="toc">
-  <h1 class="toc-title">#{CGI.escapeHTML(@producer.res.v("toctitle"))}</h1>
+  <h1 class="toc-title">#{CGI.escapeHTML(@producer.res.v('toctitle'))}</h1>
 #{ncx_main}  </nav>
 EOT
 
-      @title = CGI.escapeHTML(@producer.res.v("toctitle"))
-      @language = @producer.params['language']
-      @stylesheets = @producer.params["stylesheet"]
+      @title = CGI.escapeHTML(@producer.res.v('toctitle'))
+      @language = @producer.config['language']
+      @stylesheets = @producer.config['stylesheet']
       tmplfile = File.expand_path('./html/layout-html5.html.erb', ReVIEW::Template::TEMPLATE_DIR)
-      tmpl = ReVIEW::Template.load(tmplfile)
-      return tmpl.result(binding)
+      ReVIEW::Template.load(tmplfile).result(binding)
     end
 
     # Produce EPUB file +epubfile+.
@@ -224,10 +214,10 @@ EOT
     def produce(epubfile, basedir, tmpdir)
       produce_write_common(basedir, tmpdir)
 
-      File.open("#{tmpdir}/OEBPS/#{@producer.params["bookname"]}-toc.#{@producer.params["htmlext"]}", "w") {|f| @producer.ncx(f, @producer.params["epubmaker"]["ncxindent"]) }
+      File.open("#{tmpdir}/OEBPS/#{@producer.config['bookname']}-toc.#{@producer.config['htmlext']}", 'w') { |f| @producer.ncx(f, @producer.config['epubmaker']['ncxindent']) }
 
-      @producer.call_hook(@producer.params["epubmaker"]["hook_prepack"], tmpdir)
-      expoter = EPUBMaker::ZipExporter.new(tmpdir, @producer.params)
+      @producer.call_hook(@producer.config['epubmaker']['hook_prepack'], tmpdir)
+      expoter = EPUBMaker::ZipExporter.new(tmpdir, @producer.config)
       expoter.export_zip(epubfile)
     end
   end