プロジェクト

全般

プロフィール

致命的 #740

完了

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

Satoshi Okuno さんが9年以上前に追加. ほぼ9年前に更新.

ステータス:
終了
優先度:
通常
担当者:
対象バージョン:
開始日:
2014-12-26
期日:
進捗率:

0%

プラグイン名:
extract

説明

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に変更するか、エイリアスを貼ってもらえないでしょうか。


ファイル

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

Izumi Tsutsui さんが9年以上前に更新

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

toshi_a 初音 さんが9年以上前に更新

  • ステータス新規 から 実装待ち に変更
  • 担当者toshi_a 初音 にセット

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

toshi_a 初音 さんが9年以上前に更新

  • ステータス実装待ち から 解決 に変更

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

toshi_a 初音 さんが9年以上前に更新

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

Izumi Tsutsui さんが9年以上前に更新

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

toshi_a 初音 さんがほぼ9年前に更新

  • ステータス解決 から 終了 に変更

他の形式にエクスポート: Atom PDF