Description
opened by kouho at 2021-11-12T18:19:11; updated 2026-01-22T11:48:39
概要
現在デッキ用途を変更した際のログが残っておらずあるユーザーが特定の時間にどのデッキをどの用途で利用していたか分からないためログを追加しました。
## SQL
CREATE TABLE `log_deck_type_change` (
`id` int(11) NOT NULL AUTO_INCREMENT,
`user_no` bigint(20) NOT NULL,
`is_war_deck` tinyint(4) NOT NULL,
`deck_no` tinyint(4) NOT NULL,
`type` tinyint(4) NOT NULL,
`reg_date` timestamp NOT NULL DEFAULT CURRENT_TIMESTAMP,
PRIMARY KEY (`id`),
KEY `user_no_reg_date` (`user_no`,`reg_date`)
) ENGINE=InnoDB DEFAULT CHARSET=utf8mb4;```
## テスト方法、確認した内容
各デッキの設定変更時にログが残るか確認してください。
今回利用する定数をdefineで定義していますが他に推奨される方法があればお伝えください。
- メインデッキ
- 海賊バトルデッキ
- レイドバトルデッキ
- 戦争 1
- 戦争 2
## TODO
- 確認でき次第本番反映予定
- 管理画面でもデッキの用途変更ログを確認できるようにした方が便利そう。
kouho change_title at 2021-11-12T18:20:18: 【11/13】
【11/13】デッキ用途変更ログの追加
kouho add_label at 2021-11-12T18:21:17: レビュー待ち
kouho add_label at 2021-11-12T18:21:19: デベロップへマージ
kouho add_label at 2021-11-12T18:21:24: 改善
kouho delete_label at 2021-11-12T18:21:28: 改善
kouho assign at 2021-11-12T18:21:31: Not assigned:kouho
Comment
#15222 by kouho at 2021-11-12T18:22:02; updated 2021-11-12T18:22:02
@wada @iwamoto
確認よろしくお願いいたします。
kouho change_title at 2021-11-12T18:22:17: 【11/13】デッキ用途変更ログの追加
【11/15】デッキ用途変更ログの追加
Comment
#15243 by wada at 2021-11-15T14:50:23; updated 2021-11-15T14:50:23
@kouho
ログ出力は問題ないです。
デッキタイプは、define_y.incに定義すると他のデッキタイプにも影響にあるので、関数内で定義して欲しいです。
以下の関数で、誤って$get_deck_type = DECK_TYPE_MAINとする危険性
/***************************************************************/
// ユーザーの通常デッキ設定取得
// deck_type:1:レイドデッキA,2:レイドデッキB,3:パイコロデッキ,4:コンバットコンテスト
/***************************************************************/
function S_GetUserDeckSet($user_no, $get_deck_type = 0)
Comment
#15244 by kouho at 2021-11-15T15:11:03; updated 2021-11-15T15:11:03
@wada
確認ありがとうございます。定義の方法を変更しました。
Comment
#15246 by iwamoto at 2021-11-15T15:44:27; updated 2021-11-15T15:44:27
iwamoto add_label at 2021-11-15T15:44:31: 岩本 LGTM
wada add_label at 2021-11-15T16:36:23: 和田 LGTM
Comment
#15252 by wada at 2021-11-15T17:10:23; updated 2021-11-15T17:10:23
kouho refer at 2021-11-15T17:17:03: 1310:【11/15】デッキ用途変更ログリリース
kouho merge at 2021-11-15T17:21:55: 【11/15】デッキ用途変更ログの追加
kouho close at 2021-11-15T17:21:55: Close