Description
opened by kouho at 2022-01-13T12:04:34; updated 2022-01-14T13:30:55
概要
戦争用の船が交易できてしまう問題の修正 #1458
URLの書き換えを利用すると戦争用の船が交易に出せる状態となっていましたので判定を追加して修正しました。
テスト方法、確認した内容
修正前の状態で交易の乗組員選択画面で以下のid xxxを戦争用の船のcard_holder_noに修正
http://127.0.0.1:20080/?M=Commerce&A=Detail&id=xxx
乗組員を選択して交易に出せるので、修正後の状態でエラーメッセージが出るか確認
TODO
~~実際に利用しているユーザーがいるかチェック~~ →いない(1人は野田くん)
確認できしだい反映予定
Comment
#16558 by kouho at 2022-01-13T13:34:53; updated 2022-01-13T13:34:53
@wada @iwamoto
お手隙で確認お願いいたします
kouho change_title at 2022-01-13T18:06:31: 戦争用の船を交易に出せなくする
【11/14】戦争用の船を交易に出せなくする
kouho change_title at 2022-01-13T18:33:39: 【11/14】戦争用の船を交易に出せなくする
【1/14】戦争用の船を交易に出せなくする
Comment
#16594 by wada at 2022-01-14T10:24:28; updated 2022-01-14T10:24:28
@kouho
戦争船の判定処理があるようなので、S_IsWarShip を使用してください。
Comment
#16597 by kouho at 2022-01-14T10:39:08; updated 2022-01-14T10:39:08
@wada
確認ありがとうございます。S_IsWarShipを利用するように修正しました。
Comment
#16600 by wada at 2022-01-14T11:02:30; updated 2022-01-14T11:02:30
wada add_label at 2022-01-14T11:02:33: 和田 LGTM
Comment
#16604 by iwamoto at 2022-01-14T11:07:54; updated 2022-01-14T11:07:54
@kouho
確認しました。こちらで問題ないと思います。
iwamoto add_label at 2022-01-14T11:07:58: 岩本 LGTM
kouho merge at 2022-01-14T11:26:59: 【1/14】戦争用の船を交易に出せなくする
kouho close at 2022-01-14T11:26:59: Close
kouho refer at 2022-01-14T13:30:55: 1466:【1/14】リリース