PR

ピックアップガチャのチケットをGOLD変換しない #2388

wada opened this at 2022-11-07T19:21:33; closed

Description

opened by wada at 2022-11-07T19:21:33; updated 2022-11-18T16:16:39

概要

ピックアップガチャの限定チケット(おまけ)はGOLD変換しないので、誤って変換しないようにする

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

user_limited_gacha_ticket.ticket_no=9item_countを1以上にして、以下のスクリプトを実行
item_countが0になっていなければ良い

cd /home/web/jr-web/lib/exec/
ENVIRONMENT=develop PF=yamada php tool_ticket_change_gold.php

Comment

#23508 by wada at 2022-11-07T19:21:42; updated 2022-11-07T19:21:42

@kouho
レビューお願いします。

Comment

#23790 by kouho at 2022-11-16T16:15:27; updated 2022-11-16T16:15:27

@wada
今回の更新については問題ないのですが、tool_ticket_change_gold.phpのスクリプト内で気になったことが2点ありますので確認お願いいたします。

1. SELECTクエリについて
クエリ内の g.end_time < CURRENT_DATE(); のCURRENT_DATE() の返り値が Y-m-dで時刻まで返ってきてませんでした。特に日付のみ取得する理由がなければ g.end_time <NOW(); にしておいていただけないでしょうか?

2. 対象者がいない場合の対応について
$array_user_noが空の場合に直後のセレクトクエリが以下のようになりエラーでスクリプトが終了しておりました。

		SELECT
			user_no, gold
		FROM
			user_info
		WHERE
			user_no IN ()

できれば対象者がいない場合についても対応をいれておいていただきたいです。

Comment

#23813 by wada at 2022-11-17T12:12:35; updated 2022-11-17T12:12:35

@kouho
確認ありがとうございます。1,2対応入れました。

Comment

#23817 by kouho at 2022-11-17T13:43:25; updated 2022-11-17T13:43:25

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

kouho add_label at 2022-11-17T13:43:28: 皇甫 LGTM
wada merge at 2022-11-17T13:48:52: ピックアップガチャのチケットをGOLD変換しない
wada close at 2022-11-17T13:48:52: Close
wada refer at 2022-11-17T14:56:29: 2417:修正issue本番リリース
wada delete_branch at 2022-11-18T16:16:39: update_ticket_gold

Commit range

basea45a7d2b55b33d8a0781c2f62f7a8df60f0e72ea
headeee354e69ae6082a47d0cb3730aa546a0961f7f1
base branchjolly/jr-web:develop
head branchjolly/jr-web:update_ticket_gold

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/tool_ticket_change_gold.php
Patch body is stored as a separate file to keep this page lightweight.