PR

【2/24】SP建物から一括集金時の表示修正 #2627

kouho opened this at 2023-02-24T14:54:14; closed

Description

opened by kouho at 2023-02-24T14:54:14; updated 2023-02-24T15:41:43

概要

アイテム獲得可能なSP建物から一括集金してアイテムを獲得した際に表示がおかしくなる可能性がある問題がありましたのでその修正用PRです。
問い合わせにて発覚。

Main-ListGoldComplete.tplに渡っている$statusが建物毎の情報を取得できておりませんでしたので、
$gold_waitの最後の建物がとくぱんハウス大【SP】の時に$status['getItemNum'] = 2となり、
その一括集金で建物からアイテムを手に入れた個数分の活命酒が表示されてしまっておりました。

テスト方法、確認した内容

SubFunc.incの9989行目$rand = rand(1,81); にしてアイテムを手に入れやすく変更して確認してください。

SP効果期間中のとくぱんハウス【大】SPとアイテム獲得可能な建物を島に建てて一括集金した際に表示が問題なければOKです。

Comment

#26022 by kouho at 2023-02-24T14:54:31; updated 2023-02-24T14:54:31

@wada
レビューお願いいたします

Comment

#26023 by kouho at 2023-02-24T15:34:07; updated 2023-02-24T15:34:07

@wada
確認ありがとうございます。
SP建物からのアイテムは1種類だけで外に出そうかとしていましたが、妥当性は確かにないのでcollectDataにbonus_add_item_noを入れました。

Comment

#26024 by wada at 2023-02-24T15:41:04; updated 2023-02-24T15:41:04

@kouho
ありがとうございます。確認OKです。

wada add_label at 2023-02-24T15:41:07: 和田 LGTM
kouho merge at 2023-02-24T15:41:43: 【2/24】SP建物から一括集金時の表示修正
kouho close at 2023-02-24T15:41:43: Close

commit comment

#159 by wada at 2023-02-24T15:10:39; lib/public/Main-ListGoldComplete.inc / old:None new:114
680c8e6674c3700a7da4a9fec00dd2efaf109802

@kouho
変数$bonus_add_item_no$collectDatに入れない理由はなんですか?

Commit range

base87935d916041081e7fac581c49226d27a1e6ab4e
head68d4228a0acd56b092bd7e90a21eb69fa0d9dd03
base branchjolly/jr-web:develop
head branchjolly/jr-web:fix_gold_collect_result

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/Main-ListGoldComplete.inc
modifiedpublic_html/template/Main-ListGoldComplete.tpl
Patch body is stored as a separate file to keep this page lightweight.

Review comment

#159 by wada at 2023-02-24T15:10:39; lib/public/Main-ListGoldComplete.inc / old:None new:114
680c8e6674c3700a7da4a9fec00dd2efaf109802

@kouho
変数$bonus_add_item_no$collectDatに入れない理由はなんですか?