narou 2.4.2 → 2.5.1

data/lib/command/tag.rb

@@ -171,21 +171,21 @@ module Command
     end
 
     def self.get_color(tagname)
-      @@tag_colors ||= Inventory.load("tag_colors", :local)
-      color = @@tag_colors[tagname]
+      tag_colors = Inventory.load("tag_colors", :local)
+      color = tag_colors[tagname]
       return color if color
-      last_color = @@tag_colors.values.last || COLORS.last
+      last_color = tag_colors.values.last || COLORS.last
       index = (COLORS.index(last_color) + 1) % COLORS.size
       color = COLORS[index]
-      @@tag_colors[tagname] = color
-      @@tag_colors.save
+      tag_colors[tagname] = color
+      tag_colors.save
       color
     end
 
     def set_color(tagname, color)
-      @@tag_colors ||= Inventory.load("tag_colors", :local)
-      @@tag_colors[tagname] = color
-      @@tag_colors.save
+      tag_colors = Inventory.load("tag_colors", :local)
+      tag_colors[tagname] = color
+      tag_colors.save
     end
   end
 end

data/lib/command/update.rb

@@ -3,11 +3,14 @@
 # Copyright 2013 whiteleaf. All rights reserved.
 #
 
+require "memoist"
 require_relative "../database"
 require_relative "../downloader"
 
 module Command
   class Update < CommandBase
+    extend Memoist
+
     LOG_DIR_NAME = "log"
     LOG_NUM_LIMIT = 30   # ログの保存する上限数
     LOG_FILENAME_FORMAT = "update_log_%s.txt"
@@ -53,6 +56,47 @@ module Command
         update_general_lastup
         exit 0
       }
+      @opt.on("-s", "--sort-by KEY", "アップデートする順番を変更する\n#{Narou.update_sort_key_summaries}") { |key|
+        @options["sort-by"] = key
+      }
+    end
+
+    def get_data_value(target, key)
+      data = Downloader.get_data_by_target(target) or return nil
+      value = data[key]
+      value ? value : Time.new(0)
+    end
+    memoize :get_data_value
+
+    #
+    # 項目名でアップデート対象をソートする
+    #
+    # key に偽を渡した場合はソートしない
+    #
+    def sort_by_key(key, list)
+      return list unless key
+      list.sort { |a, b|
+        value_a, value_b = [a, b].map { |target|
+          get_data_value(target, key)
+        }
+        if value_a.nil? && !value_b.nil?
+          next 1
+        elsif !value_a.nil? && value_b.nil?
+          next -1
+        elsif value_a.nil? && value_b.nil?
+          next 0
+        end
+        # 日付系は降順にする
+        if value_a.class == Time
+          value_b <=> value_a
+        else
+          value_a <=> value_b
+        end
+      }
+    end
+
+    def valid_sort_key?(key)
+      Narou::UPDATE_SORT_KEYS.keys.include?(key)
     end
 
     def execute(argv)
@@ -67,8 +111,20 @@ module Command
         no_open = true
       end
       tagname_to_ids(update_target_list)
+
+      sort_key = @options["sort-by"]
+      if sort_key
+        sort_key.downcase!
+        unless valid_sort_key?(sort_key)
+          error "#{sort_key} は正しいキーではありません。次の中から選択して下さい\n " \
+                "#{Narou.update_sort_key_summaries(17)}"
+          exit Narou::EXIT_ERROR_CODE
+        end
+      end
+      flush_cache    # memoist のキャッシュ削除
+
       update_log = $stdout.capture(quiet: false) do
-        update_target_list.each_with_index do |target, i|
+        sort_by_key(sort_key, update_target_list).each_with_index do |target, i|
           display_message = nil
           data = Downloader.get_data_by_target(target)
           if !data

data/lib/commandbase.rb

@@ -48,6 +48,9 @@ module Command
     rescue OptionParser::MissingArgument => e
       error "オプションの引数が指定されていないか正しくありません(#{e})"
       exit Narou::EXIT_ERROR_CODE
+    rescue OptionParser::AmbiguousOption => e
+      error "曖昧な省略オプションです(#{e})"
+      exit Narou::EXIT_ERROR_CODE
     end
 
     def load_local_settings

data/lib/converterbase.rb

@@ -952,7 +952,7 @@ class ConverterBase
   def rebuild_url(data)
     @url_list.each_with_index do |url, id|
       data.sub!("［＃ＵＲＬ＝#{convert_numbers(id.to_s)}］",
-                "<a href=\"#{Helper.ampersand_to_entity(url)}\">#{url}</a>")
+                "<a href=\"#{url}\">#{url}</a>")
     end
   end
 
@@ -1171,11 +1171,13 @@ class ConverterBase
       when "｜"
         ss.scan(/.+?》/)
       when "［"
+        buffer << char
         if ss.scan(/^＃.+?］/)
-          buffer << "［#{ss.matched}"
-          next
+          buffer << "#{ss.matched}"
+        else
+          before_symbol = false
         end
-        symbol = true
+        next
       when "<"
         if ss.scan(/.+?>/)
           buffer << "<#{ss.matched}"
@@ -1189,7 +1191,7 @@ class ConverterBase
       when /[ァ-ヶ]/
         ss.scan(/[ァ-ヶー・]+/)
       when /[Ａ-Ｚａ-ｚA-Za-z]/
-        ss.scan(/[Ａ-Ｚａ-ｚA-Za-z]+/)
+        ss.scan(/[Ａ-Ｚａ-ｚA-Za-z ]+/)
       when /[一-龥朗-鶴]/
         ss.scan(/[一-龥朗-鶴]+/)
       when /[〔「『\(（【〈《≪〝]/
@@ -1225,17 +1227,23 @@ class ConverterBase
       when "｜"
         ss.scan(/.+?》/)
       when "［"
+        buffer << char
         if ss.scan(/^＃.+?］/)
-          buffer << "［#{ss.matched}"
-          next
+          buffer << "#{ss.matched}"
+        else
+          before_symbol = false
         end
-        symbol = true
+        next
       when "<"
         if ss.scan(/.+?>/)
           buffer << "<#{ss.matched}"
           next
         end
         symbol = true
+      when /[〔「『\(（【〈《≪〝]/
+        buffer << char
+        before_symbol = false
+        next
       when /[―…!?！？※]/
         symbol = true
       end
@@ -1362,7 +1370,7 @@ class ConverterBase
   #
   def replace_by_replace_txt(text)
     result = text.dup
-    @setting.replace_pattern.each do |pattern|
+    (@setting.replace_pattern + Narou.global_replace_pattern).each do |pattern|
       src, dst = pattern
       result.gsub!(src, dst)
     end

data/lib/database.rb

@@ -48,6 +48,10 @@ class Database
   end
 
   def initialize
+    refresh
+  end
+
+  def refresh
     @database = Inventory.load(DATABASE_NAME, :local)
   end
 

data/lib/device/epub.rb

@@ -15,6 +15,6 @@ module Device::Epub
   DISPLAY_NAME = "EPUB"
 
   RELATED_VARIABLES = {
-    "force.enable_half_indent_bracket" => false,
+    "default.enable_half_indent_bracket" => false,
   }
 end

data/lib/device/ibooks.rb

@@ -14,7 +14,7 @@ module Device::Ibooks
   IBOOKS_CONTAINER_DIR = "~/Library/Containers/com.apple.BKAgentService/Data/Documents/iBooks/Books"
 
   RELATED_VARIABLES = {
-    "force.enable_half_indent_bracket" => false,
+    "default.enable_half_indent_bracket" => false,
   }
 
   def hook_change_settings(&original_func)

data/lib/device/ibunko.rb

@@ -12,8 +12,8 @@ module Device::Ibunko
   DISPLAY_NAME = "i文庫"
 
   RELATED_VARIABLES = {
-    "force.enable_half_indent_bracket" => false,
-    "force.enable_dakuten_font" => false
+    "default.enable_half_indent_bracket" => false,
+    "default.enable_dakuten_font" => false
   }
 
   #
@@ -23,17 +23,24 @@ module Device::Ibunko
     return false if @options["no-zip"]
     require "zip"
     Zip.unicode_names = true
+    # TODO: テキストファイル変換時もsettingを取れるようにする
+    setting = {}
+    if @novel_data
+      setting = NovelSetting.load(@novel_data["id"], @options["ignore-force"], @options["ignore-default"])
+    end
     dirpath = File.dirname(@converted_txt_path)
     translate_illust_chuki_to_img_tag
     zipfile_path = @converted_txt_path.sub(/.txt$/, @device.ebook_file_ext)
     File.delete(zipfile_path) if File.exist?(zipfile_path)
     Zip::File.open(zipfile_path, Zip::File::CREATE) do |zip|
       zip.add(File.basename(@converted_txt_path), @converted_txt_path)
-      illust_dirpath = File.join(dirpath, Illustration::ILLUST_DIR)
       # 挿絵
-      if File.exist?(illust_dirpath)
-        Dir.glob(File.join(illust_dirpath, "*")) do |img_path|
-          zip.add(File.join(Illustration::ILLUST_DIR, File.basename(img_path)), img_path)
+      if setting["enable_illust"]
+        illust_dirpath = File.join(dirpath, Illustration::ILLUST_DIR)
+        if File.exist?(illust_dirpath)
+          Dir.glob(File.join(illust_dirpath, "*")) do |img_path|
+            zip.add(File.join(Illustration::ILLUST_DIR, File.basename(img_path)), img_path)
+          end
         end
       end
       # 表紙画像

data/lib/device/kindle.rb

@@ -14,7 +14,7 @@ module Device::Kindle
   DISPLAY_NAME = "Kindle"
 
   RELATED_VARIABLES = {
-    "force.enable_half_indent_bracket" => true,
+    "default.enable_half_indent_bracket" => true,
   }
 
   include Device::BackupBookmarkUtility

data/lib/device/kobo.rb

@@ -12,6 +12,6 @@ module Device::Kobo
   DISPLAY_NAME = "Kobo"
 
   RELATED_VARIABLES = {
-    "force.enable_half_indent_bracket" => false,
+    "default.enable_half_indent_bracket" => false,
   }
 end

data/lib/device/reader.rb

@@ -12,6 +12,6 @@ module Device::Reader
   DISPLAY_NAME = "SonyReader"
 
   RELATED_VARIABLES = {
-    "force.enable_half_indent_bracket" => false,
+    "default.enable_half_indent_bracket" => false,
   }
 end

data/lib/downloader.rb

@@ -729,14 +729,14 @@ class Downloader
       info = NovelInfo.load(@setting)
     end
     if info
-      raise DownloaderHTTP404Error unless info["title"]
+      raise DownloaderNotFoundError unless info["title"]
       @setting["title"] = info["title"]
       @setting["author"] = info["writer"]
       @setting["story"] = info["story"]
     else
       # 小説情報ページがないサイトの場合は目次ページから取得する
       @setting.multi_match(toc_source, "title", "author", "story")
-      raise DownloaderHTTP404Error unless @setting.matched?("title")
+      raise DownloaderNotFoundError unless @setting.matched?("title")
       @setting["story"] = HTML.new(@setting["story"]).to_aozora
     end
     @setting["info"] = info
@@ -1193,10 +1193,15 @@ class Downloader
     novel_dir_path = get_novel_data_dir
     file_title = File.basename(novel_dir_path)
     FileUtils.mkdir_p(novel_dir_path) unless File.exist?(novel_dir_path)
-    default_settings = NovelSetting::DEFAULT_SETTINGS
+    original_settings = NovelSetting::ORIGINAL_SETTINGS
     special_preset_dir = File.join(Narou.get_preset_dir, @setting["domain"], @setting["ncode"])
     exists_special_preset_dir = File.exist?(special_preset_dir)
-    [NovelSetting::INI_NAME, "converter.rb", NovelSetting::REPLACE_NAME].each do |filename|
+    templates = [
+      [NovelSetting::INI_NAME, 1.1],
+      ["converter.rb", 1.0],
+      [NovelSetting::REPLACE_NAME, 1.0]
+    ]
+    templates.each do |(filename, binary_version)|
       if exists_special_preset_dir
         preset_file_path = File.join(special_preset_dir, filename)
         if File.exist?(preset_file_path)
@@ -1206,7 +1211,7 @@ class Downloader
           next
         end
       end
-      Template.write(filename, novel_dir_path, binding)
+      Template.write(filename, novel_dir_path, binding, binary_version)
     end
   end
 end

data/lib/helper.rb

@@ -4,6 +4,7 @@
 #
 
 require "open3"
+require "time"
 
 #
 # 雑多なお助けメソッド群
@@ -174,7 +175,7 @@ module Helper
       "整数        "
     when :float
       "小数点数    "
-    when :string
+    when :string, :select, :multiple
       "文字列      "
     when :directory
       "フォルダパス"
@@ -218,7 +219,7 @@ module Helper
       else
         raise InvalidVariableType, type
       end
-    when :string
+    when :string, :select, :multiple
       result = value
     else
       raise UnknownVariableType, type
@@ -261,7 +262,26 @@ module Helper
   # 日付形式の文字列をTime型に変換する
   #
   def date_string_to_time(date)
-    date ? Time.parse(date.sub(/[\(（].+?[\)）]/, "").tr("年月日時分秒", "///:::")) : nil
+    date ? Time.parse(date.sub(/[\(（].+?[\)）]/, "").tr("年月日時分秒@;", "///::: :")) : nil
+  end
+
+  #
+  # 指定のファイルが前回のチェック時より新しいかどうか
+  #
+  # 初回チェック時は無条件で新しいと判定
+  #
+  def file_latest?(path)
+    @@file_mtime_list ||= {}
+    fullpath = File.expand_path(path)
+    last_mtime = @@file_mtime_list[fullpath]
+    mtime = File.mtime(fullpath)
+    if mtime == last_mtime
+      result = false
+    else
+      result = true
+      @@file_mtime_list[fullpath] = mtime
+    end
+    result
   end
 
   #
@@ -302,5 +322,96 @@ module Helper
       }
     end
   end
+
+  #
+  # 更新時刻を考慮したファイルのローダー
+  #
+  module CacheLoader
+    module_function
+
+    @@mutex = Mutex.new
+    @@caches = {}
+    @@result_caches = {}
+
+    DEFAULT_OPTIONS = { mode: "r:BOM|UTF-8" }
+
+    #
+    # ファイルの更新時刻を考慮してファイルのデータを取得する。
+    # 前回取得した時からファイルが変更されていない場合は、キャッシュを返す
+    #
+    # options にはファイルを読み込む時に File.read に渡すオプションを指定できる
+    #
+    def load(path, options = DEFAULT_OPTIONS)
+      @@mutex.synchronize do
+        fullpath = File.expand_path(path)
+        cache_data = @@caches[fullpath]
+        if Helper.file_latest?(fullpath) || !cache_data
+          body = File.read(fullpath, options)
+          @@caches[fullpath] = body
+          return body
+        else
+          return cache_data
+        end
+      end
+    end
+
+    #
+    # ファイルを処理するブロックの結果をキャッシュ化する
+    #
+    # CacheLoader.load がファイルの中身だけをキャッシュ化するのに対して
+    # これはブロックの結果をキャッシュする。ファイルが更新されない限り、
+    # ブロックの結果は変わらない
+    #
+    # ex.)
+    # Helper::CacheLoader.memo("filepath") do |data|
+    #   # data に関する処理
+    #   result  # ここで nil を返すと次回も再度読み込まれる
+    # end
+    #
+    def memo(path, options = DEFAULT_OPTIONS, &block)
+      @@mutex.synchronize do
+        fail ArgumentError, "need a block" unless block
+        fullpath = File.expand_path(path)
+        key = generate_key(fullpath, block)
+        cache = @@result_caches[key]
+        if Helper.file_latest?(fullpath) || !cache
+          data = File.read(fullpath, options)
+          @@result_caches[key] = result = block.call(data)
+          return result
+        else
+          return cache
+        end
+      end
+    end
+
+    #
+    # キャッシュを格納する際に必要なキーを生成する
+    #
+    # ブロックはその場所が実行されるたびに違うprocオブジェクトが生成されるため、
+    # 同一性判定のために「どのソース」の「何行目」かで判定を行う
+    #
+    def generate_key(fullpath, block)
+      src, line = block.source_location
+      "#{fullpath}:#{src}:#{line}"
+    end
+
+    #
+    # 指定したファイルのキャッシュを削除する
+    #
+    # path を指定しなかった場合、全てのキャッシュを削除する
+    #
+    def clear(path = nil)
+      @@mutex.synchronize do
+        if path
+          fullpath = File.expand_path(path)
+          @@cache.delete(fullpath)
+          @@result_caches.delete(fullpath)
+        else
+          @@cache.clear
+          @@result_caches.clear
+        end
+      end
+    end
+  end
 end