action_prob_pre_action_dice_roll() checking action possibility for wrong player

• 報告者: cazfi
• 担当者: kvilhaugsvik
• チケットの種類: バグ
• 状況: 完了
• コンポーネント: General
• マイルストーン: 3.1.0 (完了済み)
• 優先度: 5 - 中
• 重要度: 5 - 中
• 解決法: 修正済み
• ファイル: 1

詳細 返信

It seems wrong to me that action_prob_pre_action_dice_roll() checks "is_effect_val_known(EFT_ACTION_ODDS_PCT, act_player, tgt_player, tgt_player, ...". If I understand the functions right, the first player argument 'act_player' is who gets the information. Second and third ones are the the parties of the incident. So the call is like 'tgt_player' attacking 'tgt_player'. Should be 'act_player' attacking 'tgt_player', I think.

チケットの履歴 (5 件中 3 件表示)

2021-11-06 22:05 更新者: cazfi

• 新しいチケット "action_prob_pre_action_dice_roll() checking action possibility for wrong player" が作成されました

2021-11-07 00:06 更新者: kvilhaugsvik

コメント 返信

I agree that it looks that way. I have sent the author of that code an email to check if we have missed something.

(編集済, 2021-11-07 00:07 更新者: kvilhaugsvik)

2021-11-08 15:39 更新者: kvilhaugsvik

• 添付ファイル 0001-Check-effect-value-known-for-the-correct-player.patch (File ID: 7909) が付加されました

2021-11-08 15:41 更新者: kvilhaugsvik

• 担当者 が (未割り当て) から kvilhaugsvik に更新されました
• 解決法 が なし から 受領 に更新されました
• マイルストーン が (未割り当て) から 3.1.0 (完了済み) に更新されました

コメント 返信

Reply To kvilhaugsvik

I have sent the author of that code an email to check if we have missed something.

We didn't miss something.

2021-11-12 18:34 更新者: kvilhaugsvik

• 状況 が オープン から 完了 に更新されました
• 解決法 が 受領 から 修正済み に更新されました