PR

【11/15】デッキ用途変更ログの追加 #1306

kouho opened this at 2021-11-12T18:19:11; closed

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

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

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
確認OKです

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

Commit range

baseff9515b38bee08a7cfe9516abc39e23bc27378b8
head071e2aff61575aaec21e57d557ae7e64fffae12d
base branchjolly/jr-web:develop
head branchjolly/jr-web:add_decK_change_log

View commits

Commit details are loaded from Git when a dynamic archive server is running, or can be embedded with --include-commits.
Files changed5 filesView patch
statusfile
modifiedlib/SubFunc.inc
modifiedlib/public/Deck-DefaultSaveBattleDeck.inc
modifiedlib/public/Deck-DefaultSaveDeck.inc
modifiedlib/public/Deck-DefaultSaveRaidDeck.inc
modifiedlib/public/Deck-DefaultSaveWarDeck.inc
Patch body is stored as a separate file to keep this page lightweight.