Project

General

Profile

Actions

バグ #1369

closed

抽出タブの削除ができなくなっている

Added by toshi_a 初音 about 5 years ago. Updated about 5 years ago.

Status:
終了
Priority:
通常
Assignee:
-
Target version:
プラグイン名:
クラッシュする:
No

再現手順

  • 設定>抽出タブ を開く
  • 作成済みの抽出タブを選択する
  • リストボックス右の「削除」ボタンか、コンテキストメニューの「削除」をクリックする
Actions #1

Updated by Shibafu Midorino about 5 years ago

  • Status changed from 実装待ち to レビュー待ち
  • ブランチ set to topic/1369-fix-extract-tab-delete-2

topic/1369-fix-extract-tab-delete にて場当たり的な対応を試行していたのですが、そもそもslugがSymbolでなくなってしまう原因を突き止めたほうが良いと思い、調査を行いました。

結果的には、ExtractTabListの内部で保持されているslugを取り出す際、Stringになってしまうことが分かりました。
そのため、ExtractTabList内に限っては、slugを取り出す際にto_symを使うようなアプローチに変更しました。

また、問題の回避である f9bd652e はリバートしました。動かなくなった時はまたどこかが壊れているはずです。

Actions #2

Updated by toshi_a 初音 about 5 years ago

  • Status changed from レビュー待ち to マージ待ち

ちゃんと消せて神

Actions #3

Updated by toshi_a 初音 about 5 years ago

  • Status changed from マージ待ち to 終了

merged.

Actions

Also available in: Atom PDF