PR

【12/15】新大陸レアモンスター判定 被りが入った場合にも判定が問題ないように #2473

kouho opened this at 2022-12-14T19:36:13; closed

Description

opened by kouho at 2022-12-14T19:36:13; updated 2022-12-15T11:40:16

概要

以下の問い合わせの対応用PR
記載中の補填方針は検討中。
https://git.dev.d-es.co.jp/gitbucket/jolly/jr-cs-issue/issues/236

team_event_new_island_mode_limitに発見者の違う期間被りのデータがあり。
この場合、期間が被ったデータが存在しているとその被ったデータ上のモード終了時間までレアモンスターモードに入ることができない不具合があったため判定を修正

他に過去に同様のデータは1件あり。

id team_no mode_start_time mode_end_time user_no
174114	249	1670937421	1670944621	3031
174104	249	1670936921	1670944121	4093
151663	4290	1663165674	1663172874	33544
151649	4290	1663164582	1663171782	30723
151629	4290	1663162907	1663170107	30598

確認方法

できれば修正前の挙動も確認してください。

[修正前]
フックモンスターを通常の方法で討伐
team_event_new_island_mode_limitに入ったデータをコピー
レアモンスターモードに入ることができずフックモンスターを再度討伐しても変わらない。

[修正後]
フックモンスターを通常の方法で討伐
team_event_new_island_mode_limitに入ったデータをコピー
レアモンスターモードに入ることができ、レアモンスターが出現するようになる。

TODO

確認できしだい本番に反映予定です。

Comment

#24418 by kouho at 2022-12-14T19:39:15; updated 2022-12-14T19:39:15

@wada
レビューお願いいたします

Comment

#24424 by wada at 2022-12-15T10:58:21; updated 2022-12-15T10:58:21

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

wada add_label at 2022-12-15T10:58:26: 和田 LGTM
kouho merge at 2022-12-15T11:40:16: 【12/15】新大陸レアモンスター判定 被りが入った場合にも判定が問題ないように
kouho close at 2022-12-15T11:40:16: Close

Commit range

base42a235b2c1546cf3405904c0b2e01e7980d09c97
headd73554af45bb15d6f49e7688aaea3b57f7cf9fff
base branchjolly/jr-web:master
head branchjolly/jr-web:fix_newisland_rare_limit_check

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