PR

【12/2】最多攻撃回数に最初に到達した人がアタックブースト2倍になるように修正 #2445

kouho opened this at 2022-12-01T15:06:27; closed

Description

opened by kouho at 2022-12-01T15:06:27; updated 2023-01-06T17:20:11

概要

https://git.dev.d-es.co.jp/gitbucket/jolly/jr-cs-issue/issues/204
アタックブーストの2倍の対象者について
本来の仕様では
最大攻撃回数に最初に到達した人が2倍 に だったところが
現在は
一回目の攻撃が早い人が2倍に
なってしまっていたため修正用PR

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

1. ユーザーAとユーザーBを同じ海賊団に
2. ユーザーAでレイドに最初に攻撃
3. ユーザーBでレイドに10回攻撃
4. ユーザーAでレイドに9回攻撃
2倍の対象がユーザーBになっていればOK

備考

レイド開催中に修正を反映させることも踏まえて既存のデータ構造は変えないようにしています。

TODO

開発に修正を反映後本番に反映予定です。
お知らせは掲載しません。

kouho change_title at 2022-12-01T15:08:03: 最多攻撃回数に最初に到達した人がアタックブースト2倍になるように修正 [WIP]最多攻撃回数に最初に到達した人がアタックブースト2倍になるように修正
kouho change_title at 2022-12-01T15:20:22: [WIP]最多攻撃回数に最初に到達した人がアタックブースト2倍になるように修正 最多攻撃回数に最初に到達した人がアタックブースト2倍になるように修正

Comment

#24140 by kouho at 2022-12-01T15:20:31; updated 2022-12-01T15:20:31

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

kouho add_label at 2022-12-02T10:10:03: バグ
kouho change_title at 2022-12-02T15:47:18: 最多攻撃回数に最初に到達した人がアタックブースト2倍になるように修正 【12/2】最多攻撃回数に最初に到達した人がアタックブースト2倍になるように修正

Comment

#24197 by wada at 2022-12-02T18:17:59; updated 2022-12-02T18:17:59

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

レイド終了後で良いので、冗長になってしまっている条件文の修正をお願いしたいです。

wada add_label at 2022-12-02T18:18:02: 和田 LGTM

Comment

#24199 by kouho at 2022-12-02T18:19:25; updated 2022-12-02T18:19:25

@wada
確認ありがとうございます。

> レイド終了後で良いので、冗長になってしまっている条件文の修正をお願いしたいです。

かしこまりました。

kouho merge at 2022-12-02T18:20:42: 【12/2】最多攻撃回数に最初に到達した人がアタックブースト2倍になるように修正
kouho close at 2022-12-02T18:20:42: Close
kouho refer at 2023-01-06T17:20:11: 2520:レイドアタックブースト修正 チームレイドにも適用

Commit range

base2d672936bb6f5c6c91944cdd1ad6464eda0c2280
headb7a79f64f47e4836250538efd7d78833b556ae12
base branchjolly/jr-web:develop
head branchjolly/jr-web:fix_attack_bonus_booster

View commits

Commit details are loaded from Git when a dynamic archive server is running, or can be embedded with --include-commits.
Files changed2 filesView patch
statusfile
modifiedlib/conf/class/RaidMonsterConfig.class.php
modifiedlib/models/raid/Attack_bonus.php
Patch body is stored as a separate file to keep this page lightweight.