武器匠で武器を比較したあと、一部コマンドで武器しか選択できない

• Reporter: iks
• Owner: iks
• Type: Bugs
• Status: Closed
• Component: (None)
• MileStone: 変愚蛮怒3.0.0alphaリリース (closed)
• Priority: 5 - Medium
• Severity: 5 - Medium
• Resolution: Fixed
• File: None

Details

武器匠で、武器の比較サービスを選んで武器を選択したあと、建物から出てアイテムを見る(I)コマンドや落とす(d)コマンド等で武器しか選べなくなる。
一度キャンセルしたあと、もう一度同じコマンドを入力すると、他のアイテムも選べるようになる。
(#40327から分離)

Ticket History (3/4 Histories)

2020-11-13 03:21 Updated by: iks

• New Ticket "武器匠で武器を比較したあと、一部コマンドで武器しか選択できない" created

2020-11-13 03:34 Updated by: iks

• Resolution Update from None to Fixed
• Status Update from Open to Closed

Comment

情報表示及び第2武器選択のためのwhileループの先頭でitem_tester_hookが指定されていて、かつそれをリセットすることなくループから抜けるので、発生していた模様。
choose_object(の内部のget_item)関数でitem_tester_hookはNULLにリセットされるため、選択の直前でitem_tester_hookを指定するように変更。

ついでに、目に見えない形で合計コストが増えて行くので、比較表示中の武器は選べないようにした。

2020-11-24 08:19 Updated by: deskull

Comment

item_tester_hook周りのバグは主に私のリファクタリングで、雑に整理した感が否めんのでその副作用の一つのようです。 いい加減グローバル変数を関数の引数代わりにするのは控えたいので、備忘としてチケット化だけしておきます。

2021-01-03 00:03 Updated by: deskull

Comment

#41005にチケット化したので本件は改めて3.0.0alpha完了としておきます。