narou 2.4.2 → 2.5.1

data/lib/web/views/novels/setting.haml

@@ -1,6 +1,6 @@
 -# このページ内専用のjavascriptでjQueryオブジェクトを参照したい場合この関数に記述
 :javascript
-  var local_initialize_function = function($) {
+  local_initialize_functions.push(function($) {
     var notification = new Narou.Notification();
     var stream_console = new Narou.Console(notification, {
       restore: false, buttons: false
@@ -27,27 +27,14 @@
       animation: false,
       container: "body",
     };
+    $("[data-toggle=tooltip]").tooltip(tooltip_opt);
 
-    // replace.txt 設定パネル
-    $(".btn-replace-remove").on("click", function(e) {
-      e.preventDefault();
-      if ($(".replace-pattern-tr").size() === 1) {
-        // 入力欄のtrが全部なくなると追加出来なくなるので、消す前に1個追加しておく
-        $("#btn-replace-add").trigger("click");
-      }
-      $(this).tooltip("destroy").closest("tr").remove();
-    });
-    $blank_tr = $(".replace-pattern-tr:last").clone(true);   // 読み込み直後の末尾は空なので流用
-    $("#btn-replace-add").on("click", function(e) {
-      e.preventDefault();
-      var $clone = $blank_tr.clone(true);
-      $clone.find("button").tooltip(tooltip_opt);
-      $(".replace-pattern-tr:last").after($clone);
-      $(this).tooltip("hide");
+    // セレクトボタン設定
+    $(".selectpicker").selectpicker({
+      container: "body",
+      width: "100%"
     });
-
-    $("[data-toggle=tooltip]").tooltip(tooltip_opt);   // tooltip要素がコピーされないようにclone後に
-  };
+  });
 
 %form(action="/novels/#{@id}/setting" method="POST")
   .navbar.navbar-default.navbar-fixed-top(role="navigation")
@@ -88,12 +75,12 @@
         ・環境設定で force.* 系設定が有効になっている場合、ここでの該当項目は無視されます<br>
         ・保存するとこの小説の setting.ini に書かれているコメントは削除されます
       .list-group
-        - @default_settings.each do |info|
-          - name = info[:name]
+        - @original_settings.each do |original|
+          - name = original[:name]
           - value = @novel_setting[name]
-          - help = info[:help].strip.gsub("\n", "<br>")
+          - help = original[:help].strip.gsub("\n", "<br>")
           - list_group_item_style = {}
-          - force_value = @force_settings["force.#{name}"]
+          - force_value = @force_settings[name]
           - if @error_list[name]
             - list_group_item_style = {class: "has-error", style: "background-color:#f2dede"}
           - elsif force_value.nil?.!
@@ -101,20 +88,44 @@
           .list-group-item.form-group{list_group_item_style}
             %h4.list-group-item-heading #{name}
             .list-group-item-text
-              - case info[:type]
+              - case original[:type]
               - when :boolean
-                %label(class="switch-light well")
-                  <input type="checkbox" name="#{name}" #{value ? "checked" : ""}>
-                  %span
-                    %span しない
-                    %span する
-                  %a(class="btn btn-primary")
+                .switch-toggle.well.switch-3
+                  <input id="#{name}-nil" type="radio" name="#{name}" value="nil" #{value.nil? ? "checked" : ""}>
+                  %label(for="#{name}-nil")
+                    未設定
+                  <input id="#{name}-false" type="radio" name="#{name}" value="off" #{value == false ? "checked" : ""}>
+                  %label(for="#{name}-false")
+                    しない
+                  <input id="#{name}-true" type="radio" name="#{name}" value="on" #{value ? "checked" : ""}>
+                  %label(for="#{name}-true")
+                    する
+                  %a.btn.btn-primary
+              - when :select
+                %div.input-style
+                  %select{name: name, class: "selectpicker show-tick"}
+                    %option(value="") 未設定
+                    - original[:select_keys].each_with_index do |key, index|
+                      %option{selected: value == key, value: key}
+                        = original[:select_summaries][index]
+              - when :multiple
+                %div.input-style
+                  %select{name: "#{name}[]", class: "selectpicker", multiple: true, title: "未設定"}
+                    - selected_items = value.to_s.split(",")
+                    - original[:select_keys].each_with_index do |key, index|
+                      %option{selected: selected_items.include?(key), value: key}
+                        = original[:select_summaries][index]
               - else
                 %input{type:"text", name:name, value:value, class:"form-control",
-                       placeholder:Helper.variable_type_to_description(info[:type]).strip }
+                       placeholder:Helper.variable_type_to_description(original[:type]).strip }
               %p #{help}
               .help-extra-messages
-                %p デフォルト：#{h value_to_msg(info[:value])}
+                %p
+                  未設定時：
+                  - if @default_settings.include?(name)
+                    = h value_to_msg(@default_settings[name])
+                  - else
+                    = h value_to_msg(original[:value])
                 - unless force_value.nil?
                   %p
                     %b force.#{name} が有効になっています（設定値：#{h value_to_msg(force_value)}）
@@ -126,39 +137,13 @@
     %p.panel-settings-link
       goto <a href="#setting-panel" data-move-to="#setting-panel">変換設定</a>
 
-    #replace-panel.panel.panel-info.panel-settings
-      .panel-heading <b>#{h @novel_title}</b> の置換設定
-      .panel-body
-        ・小説の文章を置換する設定を行うことが出来ます（replace.txt を書き換えます）<br>
-        ・変更を反映させるには再度変換を実行する必要があります<br>
-        ・保存するとこの小説の replace.txt に書かれているコメントは削除されます<br>
-      .list-group
-        .list-group-item.form-group
-          .list-group-item-text
-            %table#replace-txt-table
-              %thead
-                %tr
-                  %td 対象文字列
-                  %td &nbsp;
-                  %td 置き換える文字列
-                  %td &nbsp;
-              %tbody
-                -# 末尾に新規入力用の空要素を追加
-                - (@replace_pattern + [["",""]]).each do |pattern|
-                  %tr.replace-pattern-tr
-                    %td
-                      %input{type:"text", name:"replace_pattern[][left]", value:pattern[0], class:"form-control"}
-                    %td
-                      %span.glyphicon.glyphicon-chevron-right
-                    %td
-                      %input{type:"text", name:"replace_pattern[][right]", value:pattern[1], class:"form-control"}
-                    %td
-                      %button.btn-replace-remove.btn.btn-default(data-toggle="tooltip" title="削除" data-placement="right")
-                        %span.glyphicon.glyphicon-remove
-                %tr
-                  %td(colspan=4)
-                    %button#btn-replace-add.btn.btn-default(data-toggle="tooltip" title="１行追加" data-placement="right")
-                      %span.glyphicon.glyphicon-plus
+    - @replace_panel_var = { title: "<b>#{h @novel_title}</b> の置換設定",
+      panel_class: "panel-info",
+      body_class: "",
+      body: ["・この小説の文章を置換する設定を行うことが出来ます（replace.txt を書き換えます）",
+             "・変更を反映させるには再度変換を実行する必要があります",
+             "・保存するとこの小説の replace.txt に書かれているコメントは削除されます"] }
+    = haml :edit_replace_txt
 
     %input(type="submit" value="設定を保存" class="btn btn-info" style="float:none")
 

data/lib/web/views/settings.haml

@@ -1,11 +1,16 @@
 -# このページ内専用のjavascriptでjQueryオブジェクトを参照したい場合この関数に記述
 :javascript
-  var local_initialize_function = function($) {
+  local_initialize_functions.push(function($) {
     $("[data-toggle=tooltip]").tooltip({
       animation: false,
       container: "body",
     });
-  };
+    // セレクトボタン設定
+    $(".selectpicker").selectpicker({
+      container: "body",
+      width: "100%"
+    });
+  });
 
 %form(action="/settings" method="POST")
   %input(type="hidden" name="view_invisible" value="#{@view_invisible ? 1 : 0}")
@@ -35,12 +40,20 @@
   .container.container-main.theme-showcase(role="main")
     - @setting_variables.each do |scope, variables|
       - inventory = Inventory.load("#{scope}_setting", scope)
+      - unless @view_invisible
+        -# 隠しパラメータは配列の最初の方に集めておく
+        -# (リストの最後に隠しパラメータがあると、リストの最下部の表示が切れるため)
+        - variables = variables.select { |_, v| v[:invisible] }.merge(variables.reject { |_, v| v[:invisible] })
 
       %p.panel-settings-link
         - if scope == :local
           goto <a href="#global" data-move-to="#global">Global Variables</a>
+          ・
+          <a href="#replace-panel" data-move-to="#replace-panel">置換設定</a>
         - else
           goto <a href="#local" data-move-to="#local">Local Variables</a>
+          ・
+          <a href="#replace-panel" data-move-to="#replace-panel">置換設定</a>
 
       .panel.panel-primary.panel-settings
         .panel-heading{id: scope} #{scope.capitalize} Variables
@@ -48,34 +61,35 @@
           ・各コマンドの設定の変更が出来ます<br>
           - if scope == :local
             - if @view_invisible
-              ・force.* 系の設定は setting.ini の設定を無視して反映されるようになります<br>
+              ・default.* 系の設定は個別の変換設定で未設定の項目の挙動を指定することが出来ます<br>
+              ・force.* 系の設定は個別設定、default.* 等の設定を無視して反映されるようになります<br>
               ・default_args.* 系の設定は、各種コマンドのオプションを省略した場合に使用されるオプションを指定出来ます<br>
           - else
             ・Global な設定はユーザープロファイルに保存され、すべての narou コマンドで使われます<br>
           - if @view_invisible
-            ・隠しオプションを非表示にするには<a href="?"><u>ここをクリック</u></a><br>
+            ・隠しオプションを非表示にするには<a href="?##{scope}"><u>ここをクリック</u></a><br>
           - else
-            ・隠しオプションを表示するには<a href="?view_invisible=1"><u>ここをクリック</u></a><br>
+            ・隠しオプションを表示するには<a href="?view_invisible=1##{scope}"><u>ここをクリック</u></a><br>
         .list-group
           - variables.each do |name, value|
             - item = inventory[name]
             - unless @view_invisible
-              - if value[2]
-                - if value[0] == :boolean
+              - if value[:invisible]
+                - if value[:type] == :boolean
                   - hidden_value = convert_boolean_to_on_off(item)
                 - else
                   - hidden_value = item
                 %input{type:"hidden", name:name, value:hidden_value}
                 - next
-            - message = Narou::SETTING_VARIABLES_WEBUI_MESSAGES[name] || value[1]
+            - message = Narou::SETTING_VARIABLES_WEBUI_MESSAGES[name] || value[:help]
             - message = message.strip.gsub("\n", "<br>")
             - error_style = {class: "has-error", style: "background-color:#f2dede"}
             .list-group-item.form-group{@error_list[name] ? error_style : {}}
               %h4.list-group-item-heading #{name}
               .list-group-item-text
-                - case value[0]
+                - case value[:type]
                 - when :boolean
-                  - if name.start_with?("force.")
+                  - if name =~ /^(force|default)\./
                     .switch-toggle.well.switch-3
                       <input id="#{name}-nil" type="radio" name="#{name}" value="nil" #{item.nil? ? "checked" : ""}>
                       %label(for="#{name}-nil")
@@ -99,17 +113,40 @@
                           %span しない
                           %span する
                       %a.btn.btn-primary
+                - when :select
+                  %div.input-style
+                    %select{name: name, class: "selectpicker show-tick"}
+                      %option(value="") 未設定
+                      - value[:select_keys].each_with_index do |key, index|
+                        %option{selected: item == key, value: key}
+                          = value[:select_summaries][index]
+                - when :multiple
+                  %div.input-style
+                    %select{name: "#{name}[]", class: "selectpicker", multiple: true, title: "未設定"}
+                      - selected_items = item.to_s.split(",")
+                      - value[:select_keys].each_with_index do |key, index|
+                        %option{selected: selected_items.include?(key), value: key}
+                          = value[:select_summaries][index]
                 - else
                   %input{type:"text", name:name, value:item, class:"form-control",
-                         placeholder:Helper.variable_type_to_description(value[0]).strip }
+                         placeholder:Helper.variable_type_to_description(value[:type]).strip }
                 %p
                   = message != "" ? message : "&nbsp;"
                 - if @error_list[name]
                   %p.alert-danger
                     = @error_list[name]
                 .clear
-          .panel-footer
-            %input.btn.btn-info(type="submit" value="設定を保存" style="float:none")
-            %span(style="color:gray")
-              （Local、Globalどちらも保存されます）
 
+    %p.panel-settings-link
+      goto <a href="#local" data-move-to="#local">Local Variables</a>
+      ・
+      <a href="#global" data-move-to="#global">Global Variables</a>
+
+    - @replace_panel_var = { title: "全小説対象の置換設定",
+      panel_class: "panel-primary",
+      body_class: "list-group-item-info",
+      body: ["・全ての小説に対する置換設定を行うことが出来ます",
+             "・変更を反映させるには再度変換を実行する必要があります"] }
+    = haml :edit_replace_txt
+
+    %input(type="submit" value="設定を保存" class="btn btn-info" style="float:none")

data/lib/web/views/style.scss

@@ -486,6 +486,9 @@ input::-ms-clear {
   .modal-content {
     margin-top: 160px;
   }
+  .diff-list-modal .modal-content {
+    margin-top: inherit;
+  }
 }
 @media (max-width:767px) {
   .modal-content {
@@ -664,13 +667,12 @@ $console-color: white;
     width: 750px;
   }
 
-  input {
+  input, .input-style {
     @media (min-width: 501px) {
       float: right;
       width: 150px;
     }
     @media (max-width: 500px) {
-      float: left;
     }
   }
 
@@ -824,3 +826,43 @@ $console-color: white;
   display: none;
 }
 
+/* 差分表示用ダイアログ */
+.diff-list-container {
+  .title {
+    size: 2em;
+    font-weight: bold;
+    padding-bottom: 5px;
+  }
+  .list {
+    .item {
+      font-size: 90%;
+      cursor: pointer;
+      border-top: 1px solid #ddd;
+      padding-top: 3px;
+      padding-bottom: 3px;
+
+      ul {
+        margin-bottom: 0;
+      }
+
+      &:hover {
+        background-color: #eef;
+        color: mix(blue, black);
+      }
+
+      .date {
+        padding-left: 1em;
+        font-weight: bold;
+      }
+    }
+  }
+}
+
+/* テーブルのリロードボタン */
+#icon-refresh-table {
+  color: #aaa;
+  &:hover {
+    cursor: pointer;
+    color: #777;
+  }
+}

data/lib/web/views/widget.haml

@@ -15,13 +15,6 @@
       <script src="https://oss.maxcdn.com/html5shiv/3.7.2/html5shiv.min.js"></script>
       <script src="https://oss.maxcdn.com/respond/1.4.2/respond.min.js"></script>
   %body
-    :javascript
-      var local_initialize_function = function($) {
-        var notification = new Narou.Notification();
-        var stream_console = new Narou.Console(notification, {
-          restore: false, buttons: false
-        });
-      };
     :css
       html, body, .console.fullscreen {
         height: 100%;
@@ -35,5 +28,10 @@
     %script{:src => "/resources/perfect-scrollbar.min.js"}
     %script{:src => "/resources/narou.library.js"}
     :javascript
-      if (typeof local_initialize_function !== "undefined") local_initialize_function($);
+      $(document).ready(function($) {
+        var notification = new Narou.Notification();
+        var stream_console = new Narou.Console(notification, {
+          restore: false, buttons: false
+        });
+      });
 

data/narou.gemspec

@@ -32,14 +32,53 @@ Gem::Specification.new do |gem|
   install_message = <<-EOS
 #{"*" * 79}
 
-2.4.2 : 2015/06/07
+2.5.0 : 2015/07/10
 ------------------
+#### 追加機能
+- WEB UI の個別メニューから「差分を表示」を選択した際に、過去の差分を選択出来る
+  ようにしました
+- WEB UI の個別メニューに「タグを編集」を追加しました
+- 変換設定の各項目の共通設定を変更できる default.* 系設定を追加しました
+	+ この共通設定は setting.ini で未設定の項目がある場合に適用されます。また、
+	  force.* が設定されていた場合は setting.ini、共通設定ともに無視されます
+	+ 例えば `enable_illust` の共通設定を変更したい場合、
+	  `narou s default.enable_illust=false` とすることで変更出来ます
+	  (WEB UIからは環境設定のページで隠しオプションを表示)
+- `convert` コマンドに `--ignore-default` オプションを追加しました。変換時の
+  default.* 系設定を無視します
+- `convert.copy-to-grouping` オプションを追加しました
+	+ copy-to で指定したフォルダの中に更に device もしくは convert.multi-device
+	  で指定した端末毎に振り分けるように出来ます
+	+ 有効にするには `narou s convert.copy-to-grouping=true` を実行して下さい
+- 全小説を対象にした replace.txt による置換に対応しました
+	+ narou init を使用したフォルダに replace.txt を設置すると認識します。
+	  WEB UI においては環境設定にて設定できます
+- `update` コマンドにアップデートの順番を変更する `--sort-by` オプションが追加
+  されました（短縮 `-s`）
+	+ `narou u --sort-by general_update` のように直接コマンドのオプションとして
+	  指定するか、 `narou s update.sort-by=general_update` と設定として保存して
+	  下さい
+	+ 指定出来るのは id, last_update(更新日), title(タイトル), author(作者名),
+	  new_arrivals_date(新着日), general_lastup(最新話掲載日) です
+- WEB UI の設定画面において一部項目をセレクトボックスで選択出来るようにしました
+
+#### 仕様変更
+- default.* 系設定追加に伴い、新規DL時の setting.ini の全ての項目がコメント
+  アウトされた状態で生成されます。また、WEB UI の個別変換設定画面にて、各項目に
+  「未設定」が追加されます
+- `setting` コマンド(WEB UIにおける環境設定)の項目の順序・表示を変更しました
+- `setting` コマンドの一部項目で適切な値が入力されなかった場合にエラーが出る
+  ようになりました
+- WEB UI の個別メニュー内項目の並び順を変更しました
+- WEB UI サーバ起動中にコマンドラインで更新した場合でもサーバを再起動せずに変更
+  が反映されるようになりました
+
 #### Bug Fix
-- WEB UI のタグ検索結果がページリロードで解除されてしまう不具合を修正
-- WEB UI で最新話掲載日を更新したあとにリストを更新するように修正
-- 小説家になろうのルビ仕様に追随してない場合があったのを修正
-- ハーメルン、ムーンライトノベルズの小説をNコードでDL出来ない不具合を修正
-- v2.4.0以前にDLした短編が更新できなくなっていた不具合を修正
+- `enable_insert_word_separator` を有効時に `［` が正常に禁則処理されない問題を
+  修正
+- i文庫形式に変換する際に `enable_illust` の設定が無視されていたのを修正
+- WEB UI サーバ起動中に converter.rb を編集しても反映されなかったのを修正
+- 文章中のリンクが正しく飛べない場合があったのを修正
 
 #{"*" * 79}
   EOS

data/spec/convert_spec.rb

@@ -40,7 +40,7 @@ describe "convert" do
     dir = File.dirname(path)
     filename = File.basename(path)
     $stdout.silence do
-      CommandLine.run(["convert", path, "--no-epub", "--no-open", "--ignore-force"])
+      CommandLine.run(["convert", path, "--no-epub", "--no-open", "--ignore-force", "--ignore-default"])
     end
     output_file = File.join(dir, "[#{AUTHOR}] #{filename}")
     correct_file = File.join(dir, "correct_#{filename}")