review 3.2.0 → 5.1.0

data/lib/review/version.rb

@@ -1,3 +1,3 @@
 module ReVIEW
-  VERSION = '3.2.0'.freeze
+  VERSION = '5.1.0'.freeze
 end

data/lib/review/volumeprinter.rb

@@ -0,0 +1,97 @@
+#
+# Copyright (c) 2014-2021 Minero Aoki, Kenshi Muto
+#               2003-2014 Minero Aoki
+#
+# This program is free software.
+# You can distribute or modify this program under the terms of
+# the GNU LGPL, Lesser General Public License version 2.1.
+# For details of the GNU LGPL, see the file "COPYING".
+#
+require 'optparse'
+require 'review'
+require 'review/i18n'
+require 'review/plaintextbuilder'
+
+include ReVIEW::TextUtils
+
+module ReVIEW
+  class VolumePrinter
+    def self.execute(*args)
+      new.execute(*args)
+    end
+
+    def initialize
+      @logger = ReVIEW.logger
+      @yamlfile = 'config.yml'
+    end
+
+    def execute(*args)
+      parse_options(args)
+      @config = ReVIEW::Configure.create(yamlfile: @yamlfile)
+      @book = ReVIEW::Book::Base.new('.', config: @config)
+      unless File.readable?(@yamlfile)
+        @logger.error("No such fiile or can't open #{@yamlfile}.")
+        exit 1
+      end
+      I18n.setup(@book.config['language'])
+
+      begin
+        @book.each_part do |part|
+          if part.number
+            print_chapter_volume(part)
+          end
+          part.each_chapter do |chap|
+            print_chapter_volume(chap)
+          end
+        end
+      rescue ReVIEW::FileNotFound, ReVIEW::CompileError, ReVIEW::ApplicationError => e
+        @logger.error e.message
+        exit 1
+      end
+      puts '============================='
+      print_volume(@book.volume)
+    end
+
+    def parse_options(args)
+      opts = OptionParser.new
+      opts.version = ReVIEW::VERSION
+      opts.on('--yaml=YAML', 'Read configurations from YAML file.') { |yaml| @yamlfile = yaml }
+      opts.on('--help', 'Print this message and quit') do
+        puts opts.help
+        exit 0
+      end
+      begin
+        opts.parse!(args)
+      rescue OptionParser::ParseError => e
+        @logger.error e.message
+        $stderr.puts opts.help
+        exit 1
+      end
+    end
+
+    def print_chapter_volume(chap)
+      builder = ReVIEW::PLAINTEXTBuilder.new
+      builder.bind(ReVIEW::Compiler.new(builder), chap, nil)
+
+      vol = chap.volume
+      title = chap.format_number
+      unless title.empty?
+        title += '  '
+      end
+      begin
+        title += builder.compile_inline(chap.title)
+      rescue ReVIEW::ApplicationError => e
+        @logger.warn "#{chap.name} : #{e.message.sub(/.+error: /, '')}"
+      end
+
+      printf("%3dKB %6dC %5dL %3dP  %s %-s\n",
+             vol.kbytes, vol.chars, vol.lines, vol.page,
+             "#{chap.name} ".ljust(15, '.'), title)
+    end
+
+    def print_volume(vol)
+      # total
+      printf("%3dKB %6dC %5dL %3dP\n", vol.kbytes, vol.chars, vol.lines, vol.page)
+    end
+  end
+end

data/lib/review/webmaker.rb

@@ -1,4 +1,4 @@
-# Copyright (c) 2016-2018 Masayoshi Takahashi, Masanori Kado, Kenshi Muto
+# Copyright (c) 2016-2021 Masayoshi Takahashi, Masanori Kado, Kenshi Muto
 #
 # This program is free software.
 # You can distribute or modify this program under the terms of
@@ -19,8 +19,8 @@ require 'review/yamlloader'
 require 'review/template'
 require 'review/tocprinter'
 require 'review/version'
-require 'erb'
 require 'review/makerhelper'
+require 'review/img_math'
 
 module ReVIEW
   class WEBMaker
@@ -32,15 +32,16 @@ module ReVIEW
     def initialize
       @basedir = nil
       @logger = ReVIEW.logger
+      @img_math = nil
     end
 
     def error(msg)
-      @logger.error "#{File.basename($PROGRAM_NAME, '.*')}: #{msg}"
+      @logger.error msg
       exit 1
     end
 
     def warn(msg)
-      @logger.warn "#{File.basename($PROGRAM_NAME, '.*')}: #{msg}"
+      @logger.warn msg
     end
 
     def self.execute(*args)
@@ -50,6 +51,7 @@ module ReVIEW
     def parse_opts(args)
       cmd_config = {}
       opts = OptionParser.new
+      @buildonly = nil
 
       opts.banner = 'Usage: review-webmaker [option] configfile'
       opts.version = ReVIEW::VERSION
@@ -58,6 +60,7 @@ module ReVIEW
         exit 0
       end
       opts.on('--ignore-errors', 'Ignore review-compile errors.') { cmd_config['ignore-errors'] = true }
+      opts.on('-y', '--only file1,file2,...', 'Build only specified files.') { |v| @buildonly = v.split(/\s*,\s*/).map { |m| m.strip.sub(/\.re\Z/, '') } }
 
       opts.parse!(args)
       if args.size != 1
@@ -73,30 +76,28 @@ module ReVIEW
     end
 
     def remove_old_files(path)
-      cleanup_mathimg
+      @img_math.cleanup_mathimg
       FileUtils.rm_rf(path)
     end
 
     def execute(*args)
-      @config = ReVIEW::Configure.values
-      @config.maker = 'webmaker'
       cmd_config, yamlfile = parse_opts(args)
       error "#{yamlfile} not found." unless File.exist?(yamlfile)
 
-      begin
-        loader = ReVIEW::YAMLLoader.new
-        @config.deep_merge!(loader.load_file(yamlfile))
-      rescue => e
-        error "yaml error #{e.message}"
-      end
-      # YAML configs will be overridden by command line options.
-      @config.deep_merge!(cmd_config)
+      @config = ReVIEW::Configure.create(maker: 'webmaker',
+                                         yamlfile: yamlfile,
+                                         config: cmd_config)
+
       @config['htmlext'] = 'html'
+      @img_math = ReVIEW::ImgMath.new(@config)
+
       I18n.setup(@config['language'])
       begin
         generate_html_files(yamlfile)
+        @logger.success("built #{build_path}")
       rescue ApplicationError => e
         raise if @config['debug']
+
         error(e.message)
       end
     end
@@ -107,17 +108,16 @@ module ReVIEW
       remove_old_files(@path)
       Dir.mkdir(@path)
 
-      @book = ReVIEW::Book.load(@basedir)
-      @book.config = @config
+      @book = ReVIEW::Book::Base.new(@basedir, config: @config)
+      @converter = ReVIEW::Converter.new(@book, ReVIEW::HTMLBuilder.new(img_math: @img_math))
 
       copy_stylesheet(@path)
       copy_frontmatter(@path)
       build_body(@path, yamlfile)
       copy_backmatter(@path)
 
-      math_dir = "./#{@config['imagedir']}/_review_math"
-      if @config['imgmath'] && File.exist?("#{math_dir}/__IMGMATH_BODY__.tex")
-        make_math_images(math_dir)
+      if @config['math_format'] == 'imgmath'
+        @img_math.make_math_images
       end
 
       copy_images(@config['imagedir'], "#{@path}/#{@config['imagedir']}")
@@ -127,16 +127,8 @@ module ReVIEW
       copy_resources(@config['fontdir'], "#{@path}/fonts", @config['font_ext'])
     end
 
-    def clean_mathdir
-      if @config['imgmath'] && File.exist?("#{@config['imagedir']}/_review_math")
-        FileUtils.rm_rf("#{@config['imagedir']}/_review_math")
-      end
-    end
-
     def build_body(basetmpdir, _yamlfile)
       base_path = Pathname.new(@basedir)
-      builder = ReVIEW::HTMLBuilder.new
-      @converter = ReVIEW::Converter.new(@book, builder)
       @book.parts.each do |part|
         if part.name.present?
           if part.file?
@@ -154,6 +146,7 @@ module ReVIEW
     end
 
     def build_part(part, basetmpdir, htmlfile)
+      @title = h("#{ReVIEW::I18n.t('part', part.number)} #{part.name.strip}")
       File.open("#{basetmpdir}/#{htmlfile}", 'w') do |f|
         @body = ''
         @body << %Q(<div class="part">\n)
@@ -163,8 +156,7 @@ module ReVIEW
 
         @language = @config['language']
         @stylesheets = @config['stylesheet']
-        tmplfile = File.expand_path(template_name, ReVIEW::Template::TEMPLATE_DIR)
-        f.write ReVIEW::Template.load(tmplfile).result(binding)
+        f.write ReVIEW::Template.generate(path: template_name, binding: binding)
       end
     end
 
@@ -186,6 +178,11 @@ module ReVIEW
       end
       id = File.basename(filename).sub(/\.re\Z/, '')
 
+      if @buildonly && !@buildonly.include?(id)
+        warn "skip #{id}.re"
+        return
+      end
+
       htmlfile = "#{id}.#{@config['htmlext']}"
 
       if @config['params'].present?
@@ -202,6 +199,7 @@ module ReVIEW
 
     def copy_images(resdir, destdir)
       return nil unless File.exist?(resdir)
+
       allow_exts = @config['image_ext']
       FileUtils.mkdir_p(destdir)
       recursive_copy_files(resdir, destdir, allow_exts)
@@ -209,6 +207,7 @@ module ReVIEW
 
     def copy_resources(resdir, destdir, allow_exts = nil)
       return nil if !resdir || !File.exist?(resdir)
+
       allow_exts ||= @config['image_ext']
       FileUtils.mkdir_p(destdir)
       recursive_copy_files(resdir, destdir, allow_exts)
@@ -218,6 +217,7 @@ module ReVIEW
       Dir.open(resdir) do |dir|
         dir.each do |fname|
           next if fname.start_with?('.')
+
           if FileTest.directory?("#{resdir}/#{fname}")
             recursive_copy_files("#{resdir}/#{fname}", "#{destdir}/#{fname}", allow_exts)
           elsif fname =~ /\.(#{allow_exts.join('|')})\Z/i
@@ -252,6 +252,7 @@ module ReVIEW
     end
 
     def build_indexpage(basetmpdir)
+      @title = h('index')
       File.open("#{basetmpdir}/index.html", 'w') do |f|
         if @config['coverimage']
           file = File.join(@config['imagedir'], @config['coverimage'])
@@ -268,16 +269,16 @@ module ReVIEW
         @toc = ReVIEW::WEBTOCPrinter.book_to_string(@book)
         @next = @book.chapters[0]
         @next_title = @next ? @next.title : ''
-        tmplfile = File.expand_path(template_name, ReVIEW::Template::TEMPLATE_DIR)
-        f.write ReVIEW::Template.load(tmplfile).result(binding)
+        f.write ReVIEW::Template.generate(path: template_name, binding: binding)
       end
     end
 
     def build_titlepage(basetmpdir, htmlfile)
+      @title = h('titlepage')
       File.open("#{basetmpdir}/#{htmlfile}", 'w') do |f|
         @body = ''
         @body << %Q(<div class="titlepage">)
-        @body << %Q(<h1 class="tp-title">#{CGI.escapeHTML(@config.name_of('booktitle'))}</h1>)
+        @body << %Q(<h1 class="tp-title">#{h(@config.name_of('booktitle'))}</h1>)
         if @config['aut']
           @body << %Q(<h2 class="tp-author">#{join_with_separator(@config.names_of('aut'), ReVIEW::I18n.t('names_splitter'))}</h2>)
         end
@@ -288,8 +289,7 @@ module ReVIEW
 
         @language = @config['language']
         @stylesheets = @config['stylesheet']
-        tmplfile = File.expand_path(template_name, ReVIEW::Template::TEMPLATE_DIR)
-        f.write ReVIEW::Template.load(tmplfile).result(binding)
+        f.write ReVIEW::Template.generate(path: template_name, binding: binding)
       end
     end
 
@@ -304,6 +304,7 @@ module ReVIEW
 
     def copy_file_with_param(name, target_file = nil)
       return if @config[name].nil? || !File.exist?(@config[name])
+
       target_file ||= File.basename(@config[name])
       FileUtils.cp(@config[name], File.join(@path, target_file))
     end

data/lib/review/webtocprinter.rb

@@ -1,54 +1,58 @@
 require 'review/tocprinter'
 require 'review/i18n'
+require 'review/htmlutils'
 
 module ReVIEW
   class WEBTOCPrinter < TOCPrinter
     include HTMLUtils
 
     def self.book_to_string(book)
-      io = StringIO.new
-      ReVIEW::WEBTOCPrinter.new(1, {}, io).print_book(book)
-      io.seek(0)
-      io.read
+      ReVIEW::WEBTOCPrinter.new.print_book(book)
     end
 
     def print_book(book)
-      @out.puts '<ul class="book-toc">'
-      @out.puts %Q(<li><a href="index.html">TOP</a></li>\n)
-      book.each_part do |part|
-        print_part(part)
-      end
-      @out.puts '</ul>'
+      @book = book
+      @indent = nil
+      @upper = 1 # only part and chapter
+      print_result(build_result_array)
     end
 
-    def print_part(part)
-      if part.number
-        if part.file?
-          ext = part.book.config['htmlext'] || 'html'
-          path = File.basename(part.path.sub(/\.re/, '.' + ext))
-          @out.puts %Q(<li><a href="#{path}">#{h(I18n.t('part_short', part.number) + ' ' + part.title)}</a>\n<ul>\n)
+    def print_result(result_array)
+      content = <<EOT
+<ul class="book-toc">
+<li><a href="index.html">TOP</a></li>
+EOT
+
+      path = ''
+      result_array.each do |result|
+        unless result.headline
+          result.headline = '-'
+        end
+
+        if result.name
+          path = "#{result.name}.#{@book.config['htmlext']}"
+          next
+        end
+
+        if result.part
+          if result.part == 'end'
+            content << "</ul></li>\n"
+          end
+          next
+        end
+
+        if path.start_with?('.')
+          content << "<li>#{escape(result.headline)}"
         else
-          @out.puts %Q(<li>#{h(I18n.t('part_short', part.number) + ' ' + part.title)}\n<ul>\n)
+          content << %Q(<li><a href="#{path}">#{escape(result.headline)}</a>)
+        end
+        if result.level == 0
+          content << "\n<ul>" # part
+        else
+          content << "</li>\n"
         end
       end
-      part.each_chapter do |chap|
-        print_chapter(chap)
-      end
-      if part.number
-        @out.puts "</ul>\n</li>\n"
-      end
-    end
-
-    def print_chapter(chap)
-      chap_node = TOCParser.chapter_node(chap)
-      ext = chap.book.config['htmlext'] || 'html'
-      path = File.basename(chap.path.sub(/\.re/, '.' + ext))
-      label = if chap_node.number && chap.on_chaps?
-                "#{I18n.t('chapter_short', chap.number)} #{chap.title}"
-              else
-                chap.title
-              end
-      @out.puts %Q(<li><a href="#{path}">#{h(label)}</a></li>\n)
+      content << "</ul>\n"
     end
   end
 end

data/lib/review/yamlloader.rb

@@ -18,6 +18,10 @@ module ReVIEW
       while file_queue.present?
         current_file = file_queue.shift
         current_yaml = YAML.load_file(current_file)
+        if current_yaml.instance_of?(FalseClass)
+          raise "#{File.basename(current_file)} is malformed."
+        end
+
         yaml = current_yaml.deep_merge(yaml)
 
         if yaml.key?('inherit')

data/review.gemspec

@@ -1,4 +1,4 @@
-$LOAD_PATH.push File.expand_path('lib', __dir__)
+$LOAD_PATH.push(File.expand_path('lib', __dir__))
 require 'review/version'
 
 Gem::Specification.new do |gem|
@@ -7,12 +7,11 @@ Gem::Specification.new do |gem|
   gem.platform    = Gem::Platform::RUBY
   gem.license     = 'LGPL'
   gem.authors     = %w[kmuto takahashim]
-  gem.email       = 'kmuto@debian.org'
+  gem.email       = 'kmuto@kmuto.jp'
   gem.homepage    = 'http://github.com/kmuto/review'
   gem.summary     = 'Re:VIEW: a easy-to-use digital publishing system'
   gem.description = 'Re:VIEW is a digital publishing system for books and ebooks. It supports InDesign, EPUB and LaTeX.'
-  gem.required_rubygems_version = Gem::Requirement.new('>= 0') if gem.respond_to? :required_rubygems_version=
-  gem.date = '2019-06-29'
+  gem.required_rubygems_version = Gem::Requirement.new('>= 0') if gem.respond_to?(:required_rubygems_version=)
 
   gem.files         = `git ls-files`.split("\n")
   gem.test_files    = `git ls-files -- {test,spec,features}/*`.split("\n")
@@ -23,10 +22,14 @@ Gem::Specification.new do |gem|
   gem.add_dependency('image_size')
   gem.add_dependency('rouge')
   gem.add_dependency('rubyzip')
+  gem.add_dependency('tty-logger')
+  gem.add_development_dependency('mini_magick')
   gem.add_development_dependency('pygments.rb')
   gem.add_development_dependency('rake')
-  gem.add_development_dependency('rubocop', '~> 0.72.0')
+  gem.add_development_dependency('rubocop', '~> 1.10.0')
   gem.add_development_dependency('rubocop-performance')
+  gem.add_development_dependency('rubocop-rake')
   gem.add_development_dependency('simplecov')
   gem.add_development_dependency('test-unit')
+  gem.add_development_dependency('unicode-eaw')
 end