PR

【即日】集計時にガチャIDを切り替える #984

wada opened this at 2021-07-21T17:04:35; closed

Description

opened by wada at 2021-07-21T17:04:35; updated 2025-02-03T10:36:41

issue #983
サマコレガチャのgacha_no:167が過去のガチャで使用されていたので、集計時に数値が混ざってしまう
集計スクリプトに対応を入れることで、上記の問題を回避する

対応

log_yamada_paymentから課金データを取得した際に、
条件(type:[4,5],item_id:167)に一致するレコードのitem_idを変更(167→168)し、集計処理を実行

対象スクリプト

cpanel_payment_log.php
cpanel_payment_gacha.php

集計テーブル

cpanel_month_sum_payment_log
cpanel_daily_payment_log
cpanel_daily_payment_gacha
cpanel_daily_sum_payment_gacha

更新テーブル

管理画面でのガチャデータ参照用のレコードを設定
間違ってゲーム内で表示されないように、開始日と終了日を同時刻に設定

master_gacha

Comment

#12369 by wada at 2021-07-21T17:12:08; updated 2021-07-21T17:12:08

@iwamoto @kouho
レビューお願いします。
当日の課金ログデータ:log_yamada_payment.sql

テスト用の課金ログデータを挿入した後に以下のスクリプトを実行してください

ENVIRONMENT=develop PF=yamada HTTP_HOST=127.0.0.1 IS_DOCKER=true php cpanel_payment_log.php "2021-07-21 00:00:00"
#スクリプト内の$yesterdayを当日の0時に修正してください
ENVIRONMENT=develop PF=yamada HTTP_HOST=127.0.0.1 IS_DOCKER=true php cpanel_payment_gacha.php

また、define_exec.incの$_df['DB_CONF_GEMUTOMO'][99]がjrye_logになっていたので、jrye_logに対象のテーブルがない場合、jrye_cpanelに変更してください。
それぞれ4つのテーブルにid:168のデータがあれば成功です。

データ集計後、cpanelブランチに切り替えて、ガチャ別売り上げのイベントガチャに集計したデータが表示されるかも確認お願いします。(id:168のデータがあれば成功)

<img src="https://git.dev.d-es.co.jp/gitbucket/jolly/jr-web/_attached/1626855065093ahkvbIp9EM" width="400">

Comment

#12371 by kouho at 2021-07-21T18:17:29; updated 2021-07-21T18:17:29

@wada
当日分のデータが修正されることを確認しました。
一点気になったのですが、元々入っているgacha_id167:バレンタインガチャの売り上げは確認できるでしょうか?

Comment

#12372 by wada at 2021-07-21T18:20:27; updated 2021-07-21T18:20:27

@kouho
ありがとうございます。
集計は当月のログのみ集計されるので、2月に開催されたデータが上書きされることはありません。
167は管理画面上ではサマコレ表記になっているため、サマコレ終了後に変更します。

Comment

#12373 by kouho at 2021-07-21T18:26:32; updated 2021-07-21T18:26:32

@wada
かしこまりました。でしたら問題ないと思います。

wada merge at 2021-07-21T19:04:13: 【即日】集計時にガチャIDを切り替える
wada close at 2021-07-21T19:04:13: Close

Comment

#12377 by iwamoto at 2021-07-21T19:04:28; updated 2021-07-21T19:04:28

@wada
確認しました。大丈夫だと思います。

Commit range

basee59beee0db5c5f8ae95234f9b024184ec8189e49
head1432c9a817f045201f388e73fc6661f54ecb7a76
base branchjolly/jr-web:master
head branchjolly/jr-web:fix_gacha_id

View commits

Commit details are loaded from Git when a dynamic archive server is running, or can be embedded with --include-commits.
Files changed12 filesView patch
statusfile
modifiedlib/exec/cpanel_payment_gacha.php
modifiedlib/exec/cpanel_payment_log.php
deletedpublic_html/img/gacha/168/banner_gacha168.jpg
deletedpublic_html/img/gacha/168/banner_gacha168_2.jpg
deletedpublic_html/img/gacha/168/gacha168.jpg
deletedpublic_html/img/gacha/168/gacha168_2.jpg
deletedpublic_html/img/gacha/168/gacha168_card_info508.jpg
deletedpublic_html/img/gacha/168/gacha168_card_info509.jpg
deletedpublic_html/img/gacha/168/gacha168_info.jpg
deletedpublic_html/img/gacha/168/gacha168_info2.jpg
deletedpublic_html/img/gacha/168/gacha168_special.jpg
deletedpublic_html/img/gacha/168/gacha168_title.jpg
Patch body is stored as a separate file to keep this page lightweight.