PR

【11/10】イベントスケジュール管理テーブル #1293

wada opened this at 2021-11-08T18:39:36; closed

Description

opened by wada at 2021-11-08T18:39:36; updated 2026-01-08T13:49:22

概要

イベントスケジュールのテーブル化 #1291

24時間おきのバッチ実行によるデータ登録を想定して、直近24時間以内に開始したイベント・キャンペーンをテーブルに登録
すでに登録してあるなどで期間が被ったら登録しない
ソース上に記述されてあり、リリース実績があるイベント・キャンペーンが対象(今後増える可能性あり)

テーブル

CREATE TABLE `event_schedule` (
  `id` int(11) unsigned NOT NULL AUTO_INCREMENT,
  `event_type` int(11) unsigned NOT NULL,
  `event_id` int(10) unsigned NOT NULL DEFAULT '0',
  `event_name` varchar(200) NOT NULL DEFAULT '',
  `start_time` int(11) unsigned NOT NULL,
  `end_time` int(11) unsigned NOT NULL,
  PRIMARY KEY (`id`)
) ENGINE=InnoDB DEFAULT CHARSET=utf8mb4;

2021/10までのイベントデータ

event_schedule.sql
11月以降のイベント・キャンペーンはリリース日確定後に対応

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

開始日時を24時間以内にしてスクリプト実行するとテーブルに登録される
テスト環境だと、start_timeが調整されていることがあるので注意

cd /home/web/jr-web/lib/exec
ENVIRONMENT=develop PF=yamada HTTP_HOST=127.0.0.1 IS_DOCKER=true php tool_event_schedule_insertion.inc

TODO

管理画面に一覧、登録を実装

wada change_title at 2021-11-08T18:54:17: [WIP]イベントスケジュール管理テーブル [WIP]【11/10】イベントスケジュール管理テーブル
wada change_title at 2021-11-08T18:54:59: [WIP]【11/10】イベントスケジュール管理テーブル 【11/10】イベントスケジュール管理テーブル

Comment

#15073 by wada at 2021-11-08T18:57:35; updated 2021-11-08T18:57:35

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

wada add_label at 2021-11-08T18:58:00: レビュー待ち

Comment

#15116 by kouho at 2021-11-09T17:16:13; updated 2021-11-09T17:36:05

@wada
運用上は問題ないかもしれませんが、
EVENT_TYPE_MULTI_LOGIN_BONUSについてevent_scheduleテーブルに入るevent_noがスクリプト実行のたびにカウントアップされてインサートされていきます。
原因がよく分からず確認していただけないでしょうか?
!スクリーンショット 2021-11-09 17

他のイベントタイプのものでは同様の現象は起きませんでした。

Comment

#15117 by wada at 2021-11-09T17:59:32; updated 2021-11-09T17:59:32

@kouho
おそらくstart_time > end_timeとなっているからですね。
event_idが定義されている場合、その数値のまま登録するように変更しました。

Comment

#15121 by kouho at 2021-11-09T19:11:34; updated 2021-11-09T19:11:34

@wada
対応ありがとうございます。修正確認しました。問題ないと思います。

kouho add_label at 2021-11-09T19:11:39: 皇甫 LGTM

Comment

#15133 by iwamoto at 2021-11-10T11:35:46; updated 2021-11-10T11:35:46

@wada
確認しました。問題ないと思います。

iwamoto add_label at 2021-11-10T11:35:52: 岩本 LGTM
wada merge at 2021-11-10T15:11:01: 【11/10】イベントスケジュール管理テーブル
wada close at 2021-11-10T15:11:01: Close
wada refer at 2021-11-12T10:56:25: 1301:【11/12 12:00】リリース

Commit range

base72e3d2e85614bdaec751e34c11c0fcb0f41880e5
headb6d2a9ab1f35fa7e5d354145a219474901577154
base branchjolly/jr-web:develop
head branchjolly/jr-web:event_schedule

View commits

Commit details are loaded from Git when a dynamic archive server is running, or can be embedded with --include-commits.
Files changed4 filesView patch
statusfile
addedlib/conf/class/EventScheduleConfig.class.php
addedlib/exec/tool_event_schedule_insertion.inc
addedlib/exec/tool_event_schedule_insertion.sh
addedlib/models/dao/event_schedule.php
Patch body is stored as a separate file to keep this page lightweight.