PR

スキル良星が発動しないことがある不具合の修正 #3330

kouho opened this at 2024-01-19T18:09:07; closed

Description

opened by kouho at 2024-01-19T18:09:07; updated 2024-01-19T18:47:48

概要

霊妙の占星術師ドルアークが石化することがある問題の修正用プルリク
・問い合わせが複数
・霊妙の占星術師ドルアーの潜在能力系スキル「良星」を無視して石化されてしまうことがあった
・SKILL SLOTに防御系技玉をつけている場合が問題

原因
・ディーエス側追加スキルの問題
・S_CheckActiveSkillNo関数内で本来 if(is_array($card_data[$num]['attack_skill'])) の判定に入るべきところが $set_skill_no > 10000の判定でdefense_skill側の判定に入った

対応
・潜在能力系スキルはattack_skill側の判定に入れるように修正

懸念
・潜在能力系スキル以外でS_CheckActiveSkillNo関数内でdefense_skill側の判定に入るとまずい新規スキルを作成することがあるかどうか

メモ

問題が現在発生しているため暫定での対応
根本解決は別途対応し、この対応を入れることで他の問題が発生しないかを確認いただきたいです。

Comment

#32947 by kouho at 2024-01-19T18:09:18; updated 2024-01-19T18:09:18

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

Comment

#32948 by wada at 2024-01-19T18:46:32; updated 2024-01-19T18:46:32

@kouho
スキルIDが10000を超える各スキルで以下の通り判定されることを確認しました。

100502: attack側チェック
110502: defense側チェック
110501: defense側チェック
wada add_label at 2024-01-19T18:46:57: 和田 LGTM
kouho merge at 2024-01-19T18:47:48: スキル良星が発動しないことがある不具合の修正
kouho close at 2024-01-19T18:47:48: Close

Commit range

basee86ff8ddd20368dcb4aea33d1416fb79c2ad6648
head302192453bc2b66280000570202953a95b5328eb
base branchjolly/jr-web:master
head branchjolly/jr-web:fix_skill_100502

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