PR

【確認でき次第】神格 魔格したカードの専用技玉をスキルチェンジで選べるように #3763

kouho opened this at 2024-11-05T18:23:12; closed

Description

opened by kouho at 2024-11-05T18:23:12; updated 2024-11-06T18:02:29

概要

神格 魔格したカードがスキルチェンジで専用技玉を選択できなくなっていたため修正。

ニックネーム:        OGURIN
SAPキー:        21014080190e1b15de38a61b5ba9c38f2a3330d95efee1dd151f586b6c83a4cd
ID:	132756
登録日時:	2024/11/05 01:19
タイトル:	スキルチェンジ出来ない
メッセージ:	魔格後のひばりにスキルチェンジで専用技玉の疾風迅雷を付けようとしても出来ません。改善お願いします。

ユーザーのカードのSQL
card_hodler_noとuser_noは適宜変更

INSERT INTO `user_card_holder` (`card_holder_no`, `user_no`, `card_no`, `lv`, `growth`, `exp`, `max_lv`, `bonus_stamina`, `bonus_attack`, `bonus_speed`, `deck_set`, `skill_no1`, `skill_no2`, `skill_no3`, `skill_no4`, `block_flg`, `test_flg`, `reg_date`)
VALUES
	('30840343', '1', '1216', '160', '2', '66598', '160', '8051', '8428', '8386', '0', '2133', '710', '10523', '765', '0', '0', '2024-11-05 17:14:10');

Comment

#37314 by kouho at 2024-11-05T18:23:36; updated 2024-11-05T18:23:36

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

Comment

#37318 by wada at 2024-11-06T10:20:30; updated 2024-11-06T10:20:30

@kouho
ここのif文はすべてANDで良いですか?
before_card_noについては分けたほうが無難かと思います。

if( $card_data['card_no'] != $item_data['card_character'] && isset($card_data['before_card_no']) && $card_data['before_card_no'] != $item_data['card_character'] ){
wada add_label at 2024-11-06T10:20:33: コメントあり

Comment

#37324 by kouho at 2024-11-06T14:51:10; updated 2024-11-06T14:51:10

@wada
if文を分けてみました。
神魔降臨しているかの判定をisset($card_data['before_card_no'])で入れています。

Comment

#37326 by wada at 2024-11-06T17:35:43; updated 2024-11-06T17:35:43

@kouho
確認OKです。

wada add_label at 2024-11-06T17:35:48: 和田 LGTM
wada delete_label at 2024-11-06T17:35:49: コメントあり
kouho merge at 2024-11-06T18:02:29: 【確認でき次第】神格 魔格したカードの専用技玉をスキルチェンジで選べるように
kouho close at 2024-11-06T18:02:29: Close

Commit range

based8e9eccc52180f4c55b3826596eed81f23acdcfd
head89a08b15df7dd562f40033dfb11e38de56fffa79
base branchjolly/jr-web:master
head branchjolly/jr-web:skill_change_fix

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
modifiedlib/SubFunc.inc
modifiedlib/public/Item-ChangeSkillCheck.inc
modifiedlib/public/Item-ChangeSkillResult.inc
Patch body is stored as a separate file to keep this page lightweight.