PR

【8/5】冒険応援セール #4198

kouho opened this at 2025-08-04T10:48:25; closed

Description

opened by kouho at 2025-08-04T10:48:25; updated 2025-08-06T17:46:03

概要

https://docs.google.com/spreadsheets/d/13wuBm5UjVd8nISkB3mWBAK554SJX_Ks3pL7KOCiwaBI/edit?gid=549398672#gid=549398672

Comment

#41758 by kouho at 2025-08-04T10:48:34; updated 2025-08-04T10:48:34

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

Comment

#41763 by wada at 2025-08-04T11:55:11; updated 2025-08-04T11:55:11

@kouho
config内の$_df['PAID_COIN_ONLY_SALE_ENABLE']呼び出し箇所にglobal $_df;記載お願いします。
もう使用していないので該当処理(if (!$_df['PAID_COIN_ONLY_SALE_ENABLE']) {...})削除でも良いです。

wada add_label at 2025-08-04T11:55:15: 和田 LGTM
wada delete_label at 2025-08-04T11:55:19: 和田 LGTM
wada add_label at 2025-08-04T11:55:20: コメントあり

Comment

#41796 by kouho at 2025-08-05T11:12:24; updated 2025-08-05T11:12:24

@wada
確認ありがとうございます。該当処理の削除で対応しました。
> config内の$_df['PAID_COIN_ONLY_SALE_ENABLE']呼び出し箇所にglobal $_df;記載お願いします。
もう使用していないので該当処理(if (!$_df['PAID_COIN_ONLY_SALE_ENABLE']) {...})削除でも良いです。

Comment

#41816 by wada at 2025-08-06T09:33:15; updated 2025-08-06T09:39:22

@kouho
言葉足らずですいません、if文内も含めてすべて削除してください。
これが残っているとデバッグユーザが本来の設定ではなくなるため、問題が発生した際に気付きにくくなります。

> 確認ありがとうございます。該当処理の削除で対応しました。

Comment

#41818 by kouho at 2025-08-06T10:30:28; updated 2025-08-06T10:30:28

@wada
デバッグユーザー判定も削除しました
> 言葉足らずですいません、if文内も含めてすべて削除してください。
これが残っているとデバッグユーザが本来の設定ではなくなるため、問題が発生した際に気付きにくくなります。

Comment

#41820 by wada at 2025-08-06T11:50:36; updated 2025-08-06T11:50:36

@kouho
$psを使わなくなったため、以下の処理も削除で大丈夫です。

global $user_info;
$user_pf = S_GetUserPlatformByUserNo($user_info['user_no']);
$ps = YamadaPlatformService::getInstance();

Comment

#41823 by kouho at 2025-08-06T14:08:11; updated 2025-08-06T14:08:11

@wada
すみません、以下も対応しました
> $psを使わなくなったため、以下の処理も削除で大丈夫です。

Comment

#41828 by wada at 2025-08-06T14:41:11; updated 2025-08-06T14:41:11

@kouho
確認OKです。

wada delete_label at 2025-08-06T14:41:14: コメントあり
wada add_label at 2025-08-06T14:41:15: 和田 LGTM
kouho merge at 2025-08-06T17:46:03: 【8/5】冒険応援セール
kouho close at 2025-08-06T17:46:03: Close

Commit range

base614b1faa37ca3e3cbc31522d0f474c3fa59d459e
headd0f937e4d9f5b7849254ff6fff8940d2ac82d882
base branchjolly/jr-web:master
head branchjolly/jr-web:item_pack_0807

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/SubFunc/SubFunc_yamada.inc
modifiedlib/conf/class/ShopConfig.class.php
Patch body is stored as a separate file to keep this page lightweight.