Project

General

Profile

Actions

致命的 #740

closed

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

Added by Satoshi Okuno over 9 years ago. Updated over 9 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
Actions #1

Updated by Izumi Tsutsui over 9 years ago

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

Actions #2

Updated by toshi_a 初音 over 9 years ago

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

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

Actions #3

Updated by toshi_a 初音 over 9 years ago

  • Status changed from 実装待ち to 解決

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

Actions #4

Updated by toshi_a 初音 over 9 years ago

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

Actions #5

Updated by Izumi Tsutsui over 9 years ago

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

Actions #6

Updated by toshi_a 初音 over 9 years ago

  • Status changed from 解決 to 終了
Actions

Also available in: Atom PDF