PR

【確認出来次第】カウンター・蘇生・連撃処理が合わさったときのフリーズを修正 #3891

kouho opened this at 2025-01-30T13:55:58; closed

Description

opened by kouho at 2025-01-30T13:55:58; updated 2025-01-30T15:12:36

概要

開発デバッグ時に中戸さんが発見した不具合の修正。本家の修正を移植。

本家への報告内容

攻撃系スキル(連撃)とハードカウンター、クロスカウンターと慈愛の光を組み合わせた場合にバトル演出がフリーズいたしました。
再現方法をお伝えいたしますのでご確認いただけないでしょうか。

◼︎再現方法
①バトルシミュレーターで以下のデッキを設定
上デッキ
・前列に連撃系攻撃スキル持ちカード(カードA)を配置(例2連撃)

下デッキ
・前列に(ハードカウンターまたはクロスカウンター持ち【女】カード(カードB)を配置)
・後列に慈愛の光持ちカードを配置

②フリーズまでの流れ
カードAが2連撃でカードBを攻撃→カードBの体力が0になるが慈愛の光で復活→カードBがハードカウンターでカードAの体力が0に→2連撃のスキル表示と「慈愛の光とクロスカウンター」の表示が出たままフリーズ

本家からの返信内容

こちら確認いたしました。
スキルの演出停止不具合につきまして、こちらでも現象が再現しjsの調整を入れました。
参考までにご確認ください。

https://github.com/rzstaffgit/jrce2_github/pull/13038
修正内容としては、蘇生とダメージの演出両方を行うためにbattle_statusを書き換えている部分で
battle_statusが27~38に収まらない可能性がありましたのでその部分を修正いたしました。

本家修正PRリンク

https://github.com/rzstaffgit/jrce2_github/pull/13038

Comment

#38653 by kouho at 2025-01-30T13:56:11; updated 2025-01-30T13:56:11

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

Comment

#38657 by wada at 2025-01-30T15:03:35; updated 2025-01-30T15:03:35

@kouho
確認OKです。

wada add_label at 2025-01-30T15:03:39: 和田 LGTM
kouho merge at 2025-01-30T15:12:36: 【確認出来次第】カウンター・蘇生・連撃処理が合わさったときのフリーズを修正
kouho close at 2025-01-30T15:12:36: Close

Commit range

based8a3dbff4e4ef44c468eace1282b725aba7bb40f
head6d550a53751b5f4a2b662ad59cc7405c97af82fb
base branchjolly/jr-web:master
head branchjolly/jr-web:fix_battle_counter

View commits

Commit details are loaded from Git when a dynamic archive server is running, or can be embedded with --include-commits.
Files changed3 filesView patch
statusfile
modifiedpublic_html/js/JOLLYROGER_battle.js
modifiedpublic_html/js/jr_battle.min.js
modifiedpublic_html/template/header_sp.tpl
Patch body is stored as a separate file to keep this page lightweight.