Description
概要
https://docs.google.com/spreadsheets/d/13wuBm5UjVd8nISkB3mWBAK554SJX_Ks3pL7KOCiwaBI/edit?gid=549398672#gid=549398672
https://docs.google.com/spreadsheets/d/13wuBm5UjVd8nISkB3mWBAK554SJX_Ks3pL7KOCiwaBI/edit?gid=549398672#gid=549398672
@wada
レビューお願いいたします
@kouho
config内の$_df['PAID_COIN_ONLY_SALE_ENABLE']呼び出し箇所にglobal $_df;記載お願いします。
もう使用していないので該当処理(if (!$_df['PAID_COIN_ONLY_SALE_ENABLE']) {...})削除でも良いです。
@wada
確認ありがとうございます。該当処理の削除で対応しました。
> config内の$_df['PAID_COIN_ONLY_SALE_ENABLE']呼び出し箇所にglobal $_df;記載お願いします。
もう使用していないので該当処理(if (!$_df['PAID_COIN_ONLY_SALE_ENABLE']) {...})削除でも良いです。
@kouho
言葉足らずですいません、if文内も含めてすべて削除してください。
これが残っているとデバッグユーザが本来の設定ではなくなるため、問題が発生した際に気付きにくくなります。
> 確認ありがとうございます。該当処理の削除で対応しました。
@wada
デバッグユーザー判定も削除しました
> 言葉足らずですいません、if文内も含めてすべて削除してください。
これが残っているとデバッグユーザが本来の設定ではなくなるため、問題が発生した際に気付きにくくなります。
@kouho $psを使わなくなったため、以下の処理も削除で大丈夫です。
global $user_info;
$user_pf = S_GetUserPlatformByUserNo($user_info['user_no']);
$ps = YamadaPlatformService::getInstance();@wada
すみません、以下も対応しました
> $psを使わなくなったため、以下の処理も削除で大丈夫です。
@kouho
確認OKです。
| base | 614b1faa37ca3e3cbc31522d0f474c3fa59d459e |
|---|---|
| head | d0f937e4d9f5b7849254ff6fff8940d2ac82d882 |
| base branch | jolly/jr-web:master |
| head branch | jolly/jr-web:item_pack_0807 |
| status | file |
|---|---|
| modified | lib/SubFunc/SubFunc_yamada.inc |
| modified | lib/conf/class/ShopConfig.class.php |