Project

General

Profile

致命的 #740

抽出タブを編集しようとすると落ちる。

Added by Satoshi Okuno over 4 years ago. Updated almost 4 years ago.

Status:
終了
Priority:
通常
Target version:
Start date:
2014-12-26
Due date:
% Done:

0%

プラグイン名:
extract

Description

NoMethodError undefined method `path' for Skin:Class {MIKUTTER_DIR}/core/plugin/settings/builder.rb:29:in `instance_eval' {MIKUTTER_DIR}/core/plugin/settings/builder.rb:29:in `initialize' {MIKUTTER_DIR}/core/plugin/extract/edit_window.rb:112:in `new' {MIKUTTER_DIR}/core/plugin/extract/edit_window.rb:112:in `option_widget' {MIKUTTER_DIR}/core/plugin/extract/edit_window.rb:18:in `initialize'
・・・

多分Skinの改造の時に、どこからも使われてなさげと思ってpath()を削除したと思います。迂闊でした。
user_dir()が相当の処理ですので、メソッド名をpathに変更するか、エイリアスを貼ってもらえないでしょうか。


Files

skin.rb.diff (344 Bytes) skin.rb.diff テキトー path メソッド Izumi Tsutsui, 2014-12-31 05:23

Associated revisions

Revision 48df458f (diff)
Added by toshi_a 初音 over 4 years ago

抽出タブの編集画面を開くとクラッシュする refs #740

Revision 127937d2 (diff)
Added by toshi_a 初音 over 4 years ago

extract: データソースを参照していない抽出タブがあるとクラッシュすることがある refs #740

Revision 758cc27e (diff)
Added by toshi_a 初音 over 4 years ago

デフォルトスキンを選択ている時、抽出タブのアイコン選択ダイアログの最初のディレクトリをデフォルトスキンのディレクトリにしてみた refs #740

History

#1

Updated by Izumi Tsutsui over 4 years ago

user_dir() はスキンを指定していない時には nil を返すので path の alias にしてもダメっぽいです。
添付の skin.rb.diff のようにスキンを指定していない時は default_path を返す必要があるのでは。
(ruby っぽい書き方がよくわかってません)

#2

Updated by toshi_a 初音 over 4 years ago

  • Status changed from 新規 to 実装待ち
  • Assignee set to toshi_a 初音

再現出来ました。修正します。

#3

Updated by toshi_a 初音 over 4 years ago

  • Status changed from 実装待ち to 解決

Skin.path、配列で返していたんですね、なんとも奇妙なインターフェイスだったので、一度user_dirを使うように変更しました。
呼び出し側を変えることで、vanillaでもクラッシュはしません。

#4

Updated by toshi_a 初音 over 4 years ago

vanillaスキンの時に明示的にパスを与えるようにしました。チェック漏れです。これで問題ないですかねえ

#5

Updated by Izumi Tsutsui over 4 years ago

3.2.0 に リビジョン 758cc27e 相当の差分当てて
抽出タブ編集して問題なく操作できました。
いつもお世話になっております。

#6

Updated by toshi_a 初音 almost 4 years ago

  • Status changed from 解決 to 終了

Also available in: Atom PDF