PR

【4/15】log_team_raid_monster_per_damageにgetCurrentTime()利用 #1738

kouho opened this at 2022-04-15T16:04:31; closed

Description

opened by kouho at 2022-04-15T16:04:31; updated 2022-04-15T16:28:44

概要

コミットコメント間違えました。
log_team_raid_monster_per_damageにgetCurrentTime()利用させて
log_team_raid_monster_per_damageと戦況履歴の時間を統一させたいPRです

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

攻撃して時間が合うか確認。→現状合わない。→修正済み

Raid_battle_vs_model->save_damage()関数の$damage_buf['time']と
log_team_raid_monster_per_damage->create_log()関数の$timeは一致していることを確認。

TODO

Comment

#18480 by kouho at 2022-04-15T16:04:39; updated 2022-04-15T16:04:39

@wada
確認お願いいたします

Comment

#18481 by wada at 2022-04-15T16:06:31; updated 2022-04-15T16:06:31

@kouho
実際にどのようなsqlが実行されましたか?

Comment

#18482 by kouho at 2022-04-15T16:10:03; updated 2022-04-15T16:10:03

@wada
sqlは以下の内容です

			INSERT INTO
				`log_team_raid_monster_per_damage` (
					`user_no`,
					`event_no`,
					`raid_monster_id`,
					`damage`,
					`raid_after_hp`,
					`attack_count`,
					`max_attack_count`,
					`reg_date`
				)
			VALUES
				(10040, 32, 187, 1631, 2984532, 1, 1, '2022-04-15 16:08:26')

戦況履歴はこうなっていました
!スクリーンショット 2022-04-15 16

Comment

#18483 by kouho at 2022-04-15T16:19:56; updated 2022-04-15T16:19:56

@wada
すみません、コード間違えてました

Comment

#18485 by kouho at 2022-04-15T16:22:31; updated 2022-04-15T16:22:31

@wada
修正分をコミットしましたので確認お願いいたします。

Comment

#18486 by wada at 2022-04-15T16:28:26; updated 2022-04-15T16:28:26

@kouho
確認しました。okです

wada add_label at 2022-04-15T16:28:30: 和田 LGTM
kouho merge at 2022-04-15T16:28:44: 【4/15】log_team_raid_monster_per_damageにgetCurrentTime()利用
kouho close at 2022-04-15T16:28:44: Close

Commit range

basee6238ba80a243a952189ca330d49acbaa96a8c59
head318d7e526720280de59bcc3fb66457a5c0c3f2c4
base branchjolly/jr-web:develop
head branchjolly/jr-web:fix_attack_bonus_actual

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