Project

General

Profile

バグ #1369

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

Added by toshi_a 初音 about 2 months ago. Updated 10 days ago.

Status:
終了
Priority:
通常
Assignee:
-
Target version:
プラグイン名:
ブランチ:
topic/1369-fix-extract-tab-delete-2
クラッシュする:
No

再現手順

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

Associated revisions

Revision 798f37f4 (diff)
Added by Shibafu Midorino 13 days ago

抽出タブの削除ができない問題を修正 refs #1369

Revision 48d0b3b7 (diff)
Added by Shibafu Midorino 13 days ago

抽出タブの削除後、ListViewに反映されない問題を修正 refs #1369

Revision c57627f7 (diff)
Added by Shibafu Midorino 13 days ago

それslugじゃないよ refs #1369

Revision d6e19fec (diff)
Added by Shibafu Midorino 13 days ago

TreeIterからslugを取り出す際はto_symが必要 refs #1369

Revision 1f63d00c (diff)
Added by Shibafu Midorino 13 days ago

Revert "抽出タブ設定からタブを編集しようとするとクラッシュしていた"

This reverts commit f9bd652e930ff8dedcfb9f9e4976538800e55d14.

refs #1369

Revision ab44738f (diff)
Added by Shibafu Midorino 13 days ago

TreeIterからslugを取り出す際はto_symが必要 refs #1369

Revision 3e6ed639 (diff)
Added by Shibafu Midorino 13 days ago

Revert "抽出タブ設定からタブを編集しようとするとクラッシュしていた"

This reverts commit f9bd652e930ff8dedcfb9f9e4976538800e55d14.

refs #1369

Revision 7b746105 (diff)
Added by toshi_a 初音 12 days ago

コメント修正 refs #1369

History

#1

Updated by Shibafu Midorino 13 days 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 はリバートしました。動かなくなった時はまたどこかが壊れているはずです。

#2

Updated by toshi_a 初音 12 days ago

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

ちゃんと消せて神

#3

Updated by toshi_a 初音 10 days ago

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

merged.

Also available in: Atom PDF