PR

レイドアタックブースト修正 チームレイドにも適用 #2520

kouho opened this at 2023-01-06T17:20:11; closed

Description

opened by kouho at 2023-01-06T17:20:11; updated 2023-01-11T10:04:52

概要

問い合わせで気づきました。
以下の対応をした際にチームレイドへの対応が抜けてしまっていたため修正。
increment_bonus関数は使い回しだが、save()関数に同様の修正を適用する必要がありました。

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

Comment

#24862 by kouho at 2023-01-06T17:20:35; updated 2023-01-06T17:20:35

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

Comment

#24878 by wada at 2023-01-10T10:55:11; updated 2023-01-10T10:55:11

@kouho
save()は親クラスと同じようなので、Attack_bonus_vsクラスは親クラスのsave()を使用すれば、今後も対応漏れがなくなると思います。

Comment

#24891 by kouho at 2023-01-10T15:14:02; updated 2023-01-10T15:14:02

@wada
確認ありがとうございます。Attack_bonus_vsクラスのはsave関数は削除して親クラスのsave()を使用するように修正しました。

Comment

#24896 by wada at 2023-01-10T18:44:28; updated 2023-01-10T18:44:28

@kouho
確認しました。OKです。
Attack_bonus_vs クラスにS_error_logが残っていたので、こちら削除してほしいです。

Comment

#24903 by kouho at 2023-01-10T19:11:17; updated 2023-01-10T19:11:17

@wada
確認ありがとうございます。S_error_log削除しました。

kouho close at 2023-01-11T10:04:52: Close

Commit range

base4869e487f1f1e84b29e2fbe6279f32e1e337db37
headb96838d2dbff4972b02850c14750fa4509212fd3
base branchjolly/jr-web:master
head branchjolly/jr-web:fix_raid_atk_bonus_vs

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