PR

【即日】スキルディアレストウィッシュ対象修正 #3787

kouho opened this at 2024-11-22T15:43:15; closed

Description

opened by kouho at 2024-11-22T15:43:15; updated 2024-11-22T16:56:07

概要

スキルディアレストウィッシュで発動者が1番の位置にいるときに8番の位置を対象として取得できていなかった問題の修正

本家の以下の修正プルリクの移植
https://github.com/rzstaffgit/jrce2_github/pull/12731

本家様からの共有内容

調査したところ、発動者が1番の位置にいるときに8番の位置を対象として取得できていなかったようです。
//同じ縦列か横列にいる
$check_pos_array = array();
$skill_pos = $atk_team*10+$value;//発動者の位置
switch ($num%2) {
	case 0://(自分が)列の真ん中にいる場合
		//同じ列の位置情報
		---発動者の次の番号を取ろうとしてる---
		if($skill_pos%10 == 8){ ←発動者が8番なら
			$check_pos_array[] = $atk_team*10 + 1; ←1番
		}else{
			$check_pos_array[] = $skill_pos + 1; ←発動者が1番の位置なら2番
		}
		---発動者の前の番号を取ろうとしてる---
		$check_pos_array[] = $skill_pos - 1; ←発動者が1番の時に0番が入る(これが原因で8番が取れていない)

		//船を挟んで向かいの位置情報
		if($skill_pos%10 >= 6){
			$check_pos_array[] = $skill_pos - 4;
		}else{
			$check_pos_array[] = $skill_pos + 4;
		}
		break;
}
こちらの修正に関しまして、switch ($num%2) {}自体もあまり意味が無いように見受けられたので、味方の位置に関係なくディアレストウィッシュの発動者の位置から見た同じ縦列横列にいる対象者を取得するように修正をかけました。
下記の修正プルリクを作成しましたので参考にしてください。
https://github.com/rzstaffgit/jrce2_github/pull/12731

Comment

#37556 by kouho at 2024-11-22T15:43:30; updated 2024-11-22T15:43:30

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

Comment

#37563 by wada at 2024-11-22T16:40:04; updated 2024-11-22T16:40:04

@kouho
確認OKです。

wada add_label at 2024-11-22T16:40:11: 和田 LGTM
kouho merge at 2024-11-22T16:56:07: 【即日】スキルディアレストウィッシュ対象修正
kouho close at 2024-11-22T16:56:07: Close

Commit range

basef59bfc27cb59a2c261382845aecbf52c4fde1948
headbea8f4bf8a0635a117fec07076c6e2bff72b99dd
base branchjolly/jr-web:master
head branchjolly/jr-web:fix_skill_1240

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.