narou 2.9.5 → 3.0.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: aec577773e278bf063bc2d4312e5b0756b6fed17
-  data.tar.gz: 2dd3643848b6f6c62c17a184d0eab3069f8aa9d1
+  metadata.gz: 1cabce3f355f2c95df4744fc220712459beead27
+  data.tar.gz: 7e9747fd610ebfacdc1902983cc15652030c1316
 SHA512:
-  metadata.gz: 564a5a1adda1be0b07e3cc80caca7d71479098c7d4ae6e85c85fce047421b57341d82746b8cdc7e43beeeae3bf6749ac68c802223fdc00bed10ffaa03dcb4691
-  data.tar.gz: 52d40b770819b24b2564bce2567c28b487e2b27d3eefce6c9857b8c69b21078d8b4456cff8cf74cd769011655ba704e5695120b8f23f2d52a83d079e31ab0411
+  metadata.gz: 5c9e20d2a3607ca480deb607b4d85725616049a8df00f93bc34ca1c1ea0136c033bebcffc3d3ea9b6383a9e9e94fe7e078a5823fec6f1ecf903b803825a97d67
+  data.tar.gz: b06844ba3d860dc992732420bc503fe961ef92337c5c9329a5e26cb8e9705f9e3ee363952e239a9a892a2a7b6210019ae5de445cd8b7e4b7f4d15a84b5879542

data/.gitignore

@@ -3,6 +3,7 @@
 #
 .backup/
 小説データ/
+小説データ/
 kindlestrip.py
 kindlegen.exe
 t.rb
@@ -11,6 +12,7 @@ AozoraEpub3*
 TODO.txt
 MEMO.txt
 .narou/
+.narousetting/
 *.zip
 debug
 .DS_Store
@@ -33,6 +35,11 @@ tags
 archive
 commitversion
 
+.test_dot_narou/
+.test_novel_data/
+
+mail_setting.yaml
+
 #
 # for gem
 #

data/.reek

@@ -0,0 +1,34 @@
+# vim: set ft=yaml :
+---
+# クラスコメントはなくてもいい
+IrresponsibleModule:
+  enabled: false
+
+# メソッドの複雑さは rubocop でチェックする
+TooManyStatements:
+  enabled: false
+
+DuplicateMethodCall:
+  max_calls: 2
+
+NestedIterators:
+  max_allowed_nesting: 3
+
+TooManyMethods:
+  enabled: false
+
+# Narou.rb ではクラス変数は使用許可
+ClassVariable:
+  enabled: false
+
+UtilityFunction:
+  enabled: false
+
+FeatureEnvy:
+  enabled: false
+
+UncommunicativeVariableName:
+  accept:
+    - _
+    - e
+    - i

data/.rspec

@@ -0,0 +1 @@
+ -cr spec_helper

data/.rubocop.yml

@@ -5,7 +5,9 @@ AllCops:
   Exclude:
     - "小説データ/**/*"
     - "trash/**/*"
+    - "narou.gemspec"
   DisplayCopNames: true
+  TargetRubyVersion: 2.1
 
 Metrics/ClassLength:
   CountComments: false
@@ -22,7 +24,7 @@ Metrics/BlockNesting:
   Max: 3
 
 Metrics/CyclomaticComplexity:
-  Max: 10
+  Max: 8
 
 Metrics/LineLength:
   Max: 140
@@ -208,3 +210,9 @@ Lint/UnderscorePrefixedVariableName:
 
 Style/EmptyLinesAroundClassBody:
   Enabled: false
+
+Style/FrozenStringLiteralComment:
+  Enabled: false
+
+Style/MultilineOperationIndentation:
+  Enabled: false

data/ChangeLog.md

@@ -2,6 +2,96 @@
 更新履歴 - ChangeLog
 ====================
 
+3.0.0 : 2016/11/25
+-------------------
+#### 追加機能
+- update コマンドに --ignore-all オプションを追加しました
+	+ 更新対象小説を指定しない場合の全件アップデート処理を無効にします
+	+ `narou l | narou u --ignore-all` のように PIPE で繋いだ時、処理件数が偶然
+	  ０件になった場合に意図せず全件更新にならない為に使用します
+- コマンド使用時に `^tag:NAME` でタグの除外指定が出来るようになりました。
+  先に実装された WEB UI での除外検索は `-tag:NAME` ですが、コマンドラインでは
+  -tag とするとオプションと誤認識されてしまう問題があり、^ に変更しました。
+	+ これに合わせて、WEB UI でも ^tag: を使えるようになります（従来の -tag も
+	  使えます）
+- WEB UI 関連
+	- Update ボタンに「タグを指定して更新」を追加しました
+	- Update ボタンの「最新話更新日を確認」を刷新しました。なろうAPIを使用して
+	  一括で更新されたかを確認することた出来るようになります
+		+ コマンドライン版の `narou update --gl` も同様の機能に変わっています。
+		  詳細は narou u -h を参照
+		+ > “実際に小説のデータが修正されてからなろうAPIに反映されるまで平均5分
+		  程度(最大2時間)の誤差があります。”
+
+		  （[公式サイト](http://dev.syosetu.com/man/api/)から引用）
+		+ ダイアログ内の各チェックリストの項目を独立させたものが、「な」「他」
+		  「(更新アイコン)」ボタンに独立させてあります。詳細はマウスホーバーの説明
+		  を参照
+		+ 最新話掲載日を確認した結果、新着がありそうな小説には「最新話掲載日」の
+		  日付の左上あたりに目印が表示されます
+	- 小説リストに「話数」を追加しました
+	- 小説リストに「更新チェック日」を追加しました
+	- Linux でも Eject ボタンに対応しました
+		+ デスクトップにログインした状態での udisks-daemon または udisksd による
+		  自動マウントにのみ対応しています
+		+ 使用中のデスクトップの外部で `narou web` を動かしている場合(例えばOS
+		  起動時に自動起動させた場合やログインし直した場合)、権限の問題により
+		  Eject が効かない場合があります
+- 設定に `normalize-filename` を追加しました。ファイル名をNFCで正規化します。
+  Dropbox 使用時などに合成文字列関係で問題がある場合などに使用して下さい。
+  正規化後のファイル名が以前の物と変わった場合、別ファイルと認識されるので理解
+  の上で有効にして下さい
+- 調査ログ保存時にエラー・警告・INFOの個数を表示するようにしました。
+  （`convert.inspect` が false の場合）
+- setting.ini の title_date_format に、小説のタグを埋め込める書式 $ntag を追加
+  しました。小説にタグがついている場合、全てのタグをカンマ区切りで埋め込めます
+
+#### 仕様変更
+- WEB UI の小説リストの初期表示状態を項目数増加に伴い整理しました
+	+ 初期ソートを「最新話掲載日」に変更
+	+ 種別、DL、更新チェック日、話数は初期状態では非表示
+	+ 項目追加に伴い表示のカスタマイズがリセットされるので再設定をお願いします
+- 現在ユーザーフォルダに保存されている .narousetting フォルダ(グローバル設定用)
+  を .narou フォルダと同じ場所に移動／コピーした場合、そちらを優先して読みに
+  行くようになりました
+	+ 全てのデータが一つのフォルダにまとめることが出来るようになるので、別PCへの
+	  データ移行がしやすくなります
+	+ また、現在はサーバー設定がグローバル設定で保存されているため、小説フォルダ
+	  を分けても WEB UI サーバのアドレスが同じになってしまっていましたが、別々の
+	  アドレス／ポートにすることが可能になります
+- 複数のタグを同時に指定した場合、同じ小説が複数回対象になっていたのを一度のみ
+  対象になるように変更しました
+	+ 例) A と B というタグをもつ小説があったとして、`narou u A B` で二度更新され
+	  てしまっていたのが、一度のみ更新されるようになる
+- send --without-freeze オプションを指定時、対象小説を指定した場合でも凍結時は
+  送信されなくなりました。（send.without-freeze を設定していても同じ）
+	+ WEB UI ではこれまでと同様に一括時のみ凍結済みは送信されず、個別指定時は凍結
+	  済みでも送信出来ます
+- send --without-freeze オプションの短縮形として -w を設定しました
+- send で送信時、対象小説を指定した／しないに関わらずタイムスタンプを必ず比較
+  して新しければ送信するように変更しました
+	+ 必ず送信したい場合は --force オプションを指定して下さい
+- 栞のバックアップ・レストア時にタイムスタンプを見て新しい場合のみコピーする
+  ように変更しました
+
+#### 修正内容
+- カクヨムの情報が一部取得出来なくなっていた
+- mail コマンド送信時に添付ファイルに @ が付いている場合、全角の＠に置換する様
+  にしました
+- WEB UI でタグ検索時にフィルター末尾に自動で半角空白を挿入するようにしました
+- list コマンドで検索やタグによる絞り込みと＜limit＞の指定による表示個数を制限
+  を併用した場合に、＜limit＞で指定した分のみしか絞り込み対象になっていなかった
+- list --filter series を指定時に一つも連載小説が表示されていなかった
+- Linux 環境で send コマンドが送信出来ない場合があった #119
+- 小説IDが一桁の場合に、download.use-subdirectory で作られるサブフォルダに半角
+  スペースが含まれてしまっていた
+- WEB UI のフィルターのおいて、本来表示されない内部データにまでマッチングして
+  しまう不具合
+- update.logging オプションが有効な場合に、アップデートをしない時でもログを
+  保存してしまう場合がある不具合
+- その他動作の安定性を向上などの細かい修正
+
+
 2.9.5 : 2016/07/01
 ------------------
 #### 仕様変更

data/Gemfile

@@ -7,6 +7,10 @@ gemspec
 
 gem "rspec"
 gem "rspec-core"
+gem "rspec-retry"
+gem "timecop"
 gem "pry"
+gem "pry-byebug"
 gem "better_errors"
 gem "binding_of_caller"
+gem "awesome_print"

data/README.md

@@ -33,17 +33,94 @@ Narou.rb ― 「小説家になろう」「小説を読もう！」ダウンロ
 更新履歴 - ChangeLog
 --------------------
 
-2.9.5 : 2016/07/01
-------------------
+3.0.0 : 2016/11/25
+-------------------
+#### 追加機能
+- update コマンドに --ignore-all オプションを追加しました
+	+ 更新対象小説を指定しない場合の全件アップデート処理を無効にします
+	+ `narou l | narou u --ignore-all` のように PIPE で繋いだ時、処理件数が偶然
+	  ０件になった場合に意図せず全件更新にならない為に使用します
+- コマンド使用時に `^tag:NAME` でタグの除外指定が出来るようになりました。
+  先に実装された WEB UI での除外検索は `-tag:NAME` ですが、コマンドラインでは
+  -tag とするとオプションと誤認識されてしまう問題があり、^ に変更しました。
+	+ これに合わせて、WEB UI でも ^tag: を使えるようになります（従来の -tag も
+	  使えます）
+- WEB UI 関連
+	- Update ボタンに「タグを指定して更新」を追加しました
+	- Update ボタンの「最新話更新日を確認」を刷新しました。なろうAPIを使用して
+	  一括で更新されたかを確認することた出来るようになります
+		+ コマンドライン版の `narou update --gl` も同様の機能に変わっています。
+		  詳細は narou u -h を参照
+		+ > “実際に小説のデータが修正されてからなろうAPIに反映されるまで平均5分
+		  程度(最大2時間)の誤差があります。”
+
+		  （[公式サイト](http://dev.syosetu.com/man/api/)から引用）
+		+ ダイアログ内の各チェックリストの項目を独立させたものが、「な」「他」
+		  「(更新アイコン)」ボタンに独立させてあります。詳細はマウスホーバーの説明
+		  を参照
+		+ 最新話掲載日を確認した結果、新着がありそうな小説には「最新話掲載日」の
+		  日付の左上あたりに目印が表示されます
+	- 小説リストに「話数」を追加しました
+	- 小説リストに「更新チェック日」を追加しました
+	- Linux でも Eject ボタンに対応しました
+		+ デスクトップにログインした状態での udisks-daemon または udisksd による
+		  自動マウントにのみ対応しています
+		+ 使用中のデスクトップの外部で `narou web` を動かしている場合(例えばOS
+		  起動時に自動起動させた場合やログインし直した場合)、権限の問題により
+		  Eject が効かない場合があります
+- 設定に `normalize-filename` を追加しました。ファイル名をNFCで正規化します。
+  Dropbox 使用時などに合成文字列関係で問題がある場合などに使用して下さい。
+  正規化後のファイル名が以前の物と変わった場合、別ファイルと認識されるので理解
+  の上で有効にして下さい
+- 調査ログ保存時にエラー・警告・INFOの個数を表示するようにしました。
+  （`convert.inspect` が false の場合）
+- setting.ini の title_date_format に、小説のタグを埋め込める書式 $ntag を追加
+  しました。小説にタグがついている場合、全てのタグをカンマ区切りで埋め込めます
+
 #### 仕様変更
-- WEB UI の Eject ボタンを押した際に、即時取り出しではなく一旦キューに積む様に
-  変更しました。これにより変換が終了したら取り外すということが可能になります
-	+ すぐに取り外したい場合は Eject ボタン横のドロップダウンメニューから可能です
-- Windows でも Eject ボタンに対応しました
+- WEB UI の小説リストの初期表示状態を項目数増加に伴い整理しました
+	+ 初期ソートを「最新話掲載日」に変更
+	+ 種別、DL、更新チェック日、話数は初期状態では非表示
+	+ 項目追加に伴い表示のカスタマイズがリセットされるので再設定をお願いします
+- 現在ユーザーフォルダに保存されている .narousetting フォルダ(グローバル設定用)
+  を .narou フォルダと同じ場所に移動／コピーした場合、そちらを優先して読みに
+  行くようになりました
+	+ 全てのデータが一つのフォルダにまとめることが出来るようになるので、別PCへの
+	  データ移行がしやすくなります
+	+ また、現在はサーバー設定がグローバル設定で保存されているため、小説フォルダ
+	  を分けても WEB UI サーバのアドレスが同じになってしまっていましたが、別々の
+	  アドレス／ポートにすることが可能になります
+- 複数のタグを同時に指定した場合、同じ小説が複数回対象になっていたのを一度のみ
+  対象になるように変更しました
+	+ 例) A と B というタグをもつ小説があったとして、`narou u A B` で二度更新され
+	  てしまっていたのが、一度のみ更新されるようになる
+- send --without-freeze オプションを指定時、対象小説を指定した場合でも凍結時は
+  送信されなくなりました。（send.without-freeze を設定していても同じ）
+	+ WEB UI ではこれまでと同様に一括時のみ凍結済みは送信されず、個別指定時は凍結
+	  済みでも送信出来ます
+- send --without-freeze オプションの短縮形として -w を設定しました
+- send で送信時、対象小説を指定した／しないに関わらずタイムスタンプを必ず比較
+  して新しければ送信するように変更しました
+	+ 必ず送信したい場合は --force オプションを指定して下さい
+- 栞のバックアップ・レストア時にタイムスタンプを見て新しい場合のみコピーする
+  ように変更しました
 
 #### 修正内容
-- Windows 環境において、kindlegen を実行時に何も実行されず mobi ファイルが作成
-  出来ない場合がある
+- カクヨムの情報が一部取得出来なくなっていた
+- mail コマンド送信時に添付ファイルに @ が付いている場合、全角の＠に置換する様
+  にしました
+- WEB UI でタグ検索時にフィルター末尾に自動で半角空白を挿入するようにしました
+- list コマンドで検索やタグによる絞り込みと＜limit＞の指定による表示個数を制限
+  を併用した場合に、＜limit＞で指定した分のみしか絞り込み対象になっていなかった
+- list --filter series を指定時に一つも連載小説が表示されていなかった
+- Linux 環境で send コマンドが送信出来ない場合があった #119
+- 小説IDが一桁の場合に、download.use-subdirectory で作られるサブフォルダに半角
+  スペースが含まれてしまっていた
+- WEB UI のフィルターのおいて、本来表示されない内部データにまでマッチングして
+  しまう不具合
+- update.logging オプションが有効な場合に、アップデートをしない時でもログを
+  保存してしまう場合がある不具合
+- その他動作の安定性を向上などの細かい修正
 
 ----
 

data/lib/command/convert.rb

@@ -103,7 +103,7 @@ module Command
       @opt.separator <<-EOS
 
   Configuration:
-    --no-epub, --no-mobi, --no-strip, --no-open , --inspect は narou setting コマンドで恒常的な設定にすることが可能です。
+    --no-epub, --no-mobi, --no-strip, --no-zip, --no-open , --inspect は narou setting コマンドで恒常的な設定にすることが可能です。
     convert.copy-to を設定すれば変換したEPUB/MOBIを指定のフォルダに自動でコピー出来ます。
     device で設定した端末が接続されていた場合、対応するデータを自動送信します。
     詳しくは narou setting --help を参照して下さい。

data/lib/command/list.rb

@@ -4,10 +4,12 @@
 #
 
 require_relative "../database"
+require_relative "list/novel_decorator"
 
 module Command
   class List < CommandBase
-    ANNOTATION_COLOR_TIME_LIMIT = 6 * 60 * 60   # 更新してから何秒まで色を変更するか
+    # 更新してから何秒まで色を変更するか
+    ANNOTATION_COLOR_TIME_LIMIT = 6 * 60 * 60
 
     # MEMO: 0 は昔の小説を凍結したままな場合、novel_type が設定されていないので、
     #       nil.to_i → 0 という互換性維持のため
@@ -15,10 +17,13 @@ module Command
 
     FILTERS_TYPE = Hash[*%w(series 連載 ss 短編 frozen 凍結 nonfrozen 非凍結)]
 
+    attr_reader :options
+
     def self.oneline_help
       "現在管理している小説の一覧を表示します"
     end
 
+    # rubocop:disable Metrics/MethodLength
     def initialize
       super("[<limit>] [options]")
       @opt.separator <<-EOS
@@ -81,7 +86,7 @@ module Command
       }
       @opt.on("-f", "--filter VAL", String,
               "表示を絞るためのフィルター。スペース区切りで複数可\n" \
-              "#{' '*25}#{filter_type_help}") { |filter|
+              "#{' ' * 25}#{filter_type_help}") { |filter|
         @options["filters"] = filter.downcase.split
       }
       @opt.on("-g", "--grep VAL", String,
@@ -119,7 +124,7 @@ module Command
       apply_sum = filters.inject(0) do |sum, item|
         apply = case item
                 when "series"
-                  novel_type == 0 && novel_type == 1
+                  novel_type == 0 || novel_type == 1
                 when "ss"
                   novel_type == 2
                 when "frozen"
@@ -138,26 +143,33 @@ module Command
 
     def header
       [
-         " ID ",
-         @options["general-lastup"] ? " 掲載日 " : " 更新日 ",
-         @options["kind"] ? "種別" : nil,
-         @options["author"] ? "作者名" : nil,
-         @options["site"] ? "サイト名" : nil,
-         "     タイトル"
+        " ID ",
+        @options["general-lastup"] ? " 掲載日 " : " 更新日 ",
+        @options["kind"] ? "種別" : nil,
+        @options["author"] ? "作者名" : nil,
+        @options["site"] ? "サイト名" : nil,
+        "     タイトル"
       ].compact.join(" | ")
     end
 
     def view_date_type
-      @options["general-lastup"] ? "general_lastup" : "last_update"
+      @_view_data_type ||= @options["general-lastup"] ? "general_lastup" : "last_update"
+    end
+
+    def now
+      @now ||= Time.now
+    end
+
+    def output_list(novels, limit)
+      took_lines = Hash[decorate_lines(novels).take(limit)]
+      output_lines(took_lines)
     end
 
-    def output_list(novels)
-      now = Time.now
-      today = now.strftime("%y/%m/%d")
+    def decorate_lines(novels)
       filters = @options["filters"] || []
       selected_lines = {}
-      view_date_type_key = view_date_type
       novels.each do |novel|
+        novel_decorator = NovelDecorator.new(novel, self)
         novel_type = novel["novel_type"].to_i
         id = novel["id"]
         frozen = Narou.novel_frozen?(id)
@@ -169,80 +181,54 @@ module Command
         if @options["tags"]
           next unless valid_tags?(novel, @options["tags"])
         end
-        disp_id = ((frozen ? "*" : "") + id.to_s).rjust(4)
-        disp_id = disp_id.sub("*", "<bold><cyan>*</cyan></bold>") if frozen
-        tags = novel["tags"] || []
-        selected_lines[id] = [
-          disp_id,
-          novel[view_date_type_key].strftime("%y/%m/%d").tap { |s|
-            new_arrivals_date = novel["new_arrivals_date"]
-            last_update = novel["last_update"]
-            if new_arrivals_date && new_arrivals_date >= last_update \
-               && new_arrivals_date + ANNOTATION_COLOR_TIME_LIMIT >= now
-              # 新着表示色
-              s.replace "<bold><magenta>#{s}</magenta></bold>"
-            elsif last_update + ANNOTATION_COLOR_TIME_LIMIT >= now
-              # 更新だけあった色
-              s.replace "<bold><green>#{s}</green></bold>"
-            end
-          },
-          @options["kind"] ? NOVEL_TYPE_LABEL[novel_type] : nil,
-          @options["author"] ? novel["author"].escape : nil,
-          @options["site"] ? novel["sitename"].escape : nil,
-          novel["title"].escape + (!@options["kind"] && novel_type == 2 ?
-                           "  <bold><black>(#{NOVEL_TYPE_LABEL[novel_type]})</black></bold>" :
-                           "") +
-                           (tags.include?("end") ? " <bold><black>(完結)</black></bold>" : "") +
-                           (tags.include?("404") ? " <bold><black>(削除)</black></bold>" : ""),
-          @options["url"] ? novel["toc_url"].escape : nil,
-          @options["tags"] || @options["all-tags"] ?
-              tags.empty? ? nil : tags.map{ |tag|
-                color = Tag.get_color(tag)
-                "<bold><#{color}>#{tag.escape}</#{color}></bold>"
-              }.join(",") : nil,
-        ].compact.join(" | ")
+        selected_lines[id] = novel_decorator.decorate_line
       end
-      if @options["grep"]
-        @options["grep"].each do |search_word|
-          selected_lines.keep_if { |_, line|
-            if search_word =~ /^-(.+)/
-              # NOT検索
-              !line.include?($1)
-            else
-              line.include?(search_word)
-            end
-          }
-        end
+      grep(selected_lines)
+    end
+
+    def grep(selected_lines)
+      return selected_lines unless @options["grep"]
+      @options["grep"].each do |search_word|
+        selected_lines.keep_if { |_, line|
+          if search_word =~ /^-(.+)/
+            # NOT検索
+            !line.include?($1)
+          else
+            line.include?(search_word)
+          end
+        }
       end
+      selected_lines
+    end
+
+    def output_lines(took_lines)
       if STDOUT.tty?
         puts header
-        puts selected_lines.values.join("\n").termcolor
+        puts took_lines.values.join("\n").termcolor
+      elsif @options["echo"]
+        # pipeにそのまま出力するときはansicolorコードが邪魔なので削除
+        puts header
+        puts TermColorLight.strip_tag(took_lines.values.join("\n"))
       else
-        if @options["echo"]
-          # pipeにそのまま出力するときはansicolorコードが邪魔なので削除
-          puts header
-          puts TermColorLight.strip_tag(selected_lines.values.join("\n"))
-        else
-          # pipeに接続するときはIDを渡す
-          puts selected_lines.keys.join(" ")
-        end
+        # pipeに接続するときはIDを渡す
+        puts took_lines.keys.join(" ")
       end
     end
 
     def execute(argv)
       super
       database_values = Database.instance.get_object.values
-      if !argv.empty? && argv.first =~ /^\d+$/
-        num = argv.first.to_i
-      else
-        num = database_values.size
-      end
+      limit =
+        if !argv.empty? && argv.first =~ /^\d+$/
+          argv.first.to_i
+        else
+          database_values.size
+        end
       if @options["latest"]
         database_values = Database.instance.sort_by(view_date_type)
       end
       database_values.reverse! if @options["reverse"]
-      novels = database_values[0, num]
-      output_list(novels)
+      output_list(database_values, limit)
     end
   end
 end