PR

【1/14】戦争用の船を交易に出せなくする #1461

kouho opened this at 2022-01-13T12:04:34; closed

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

@kouho
確認しました。okです

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】リリース

Commit range

base9723282d18df0be0ac88b4cd21791b58602712ba
head908570ad679d2fc150955d492d1f914724e72e16
base branchjolly/jr-web:develop
head branchjolly/jr-web:fix_commerce

View commits

Commit details are loaded from Git when a dynamic archive server is running, or can be embedded with --include-commits.
Files changed2 filesView patch
statusfile
modifiedlib/public/Commerce-Detail.inc
modifiedlib/public/Commerce-Send.inc
Patch body is stored as a separate file to keep this page lightweight.