PR

【確認出来次第】イベントアイテムGOLD変換スクリプト修正 #3735

kouho opened this at 2024-10-21T13:29:34; closed

Description

opened by kouho at 2024-10-21T13:29:34; updated 2024-10-21T17:21:03

概要

イベントアイテムGOLD変換スクリプトで変換に失敗していたことがある。

$resultGold = $resultGoldOther[$target_event['item_no']] ?? $resultGold;

原因箇所が上記なので、上記をなくし、check_event_end関数内でgoldを入れてreturnするように修正
念の為にreturnされた値にgoldが含まれない場合はスクリプトを終了するようにしています。

Comment

#37030 by kouho at 2024-10-21T13:30:14; updated 2024-10-21T13:30:14

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

Comment

#37045 by wada at 2024-10-21T16:43:28; updated 2024-10-21T16:43:28

@kouho
$resultGold = $resultGoldOther[$target_event['gold']]; でエラーが出ています。

wada add_label at 2024-10-21T16:43:38: コメントあり

Comment

#37047 by kouho at 2024-10-21T16:45:57; updated 2024-10-21T16:45:57

@wada
すみません、修正コミットしました。

Comment

#37051 by wada at 2024-10-21T17:15:05; updated 2024-10-21T17:15:05

@kouho
確認OKです。

wada delete_label at 2024-10-21T17:15:08: コメントあり
wada add_label at 2024-10-21T17:15:09: 和田 LGTM
kouho merge at 2024-10-21T17:21:03: 【確認出来次第】イベントアイテムGOLD変換スクリプト修正
kouho close at 2024-10-21T17:21:03: Close

Commit range

basecd61baccfb7171e8bc9c75bb03e6a9b6b1466df1
head181e59830cc98a703b4f1ab066727c59afc0da18
base branchjolly/jr-web:master
head branchjolly/jr-web:fix_gold_item_change

View commits

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