つけられる技玉ページ修正 #1809

概要

つけられる技玉ページの修正 #1777

テスト方法、確認した内容

反射スキル持ちかつスキルスロットを持ったカードのつけられる技玉のページで防御系の技玉が表示されないことを確認

TODO

@wada
確認お願いいたします

@kouho
確認しました。OKです

この更新自体は問題ないですが、判定が10000以上なので、ヤマダ側で攻撃スキル追加するとカウンター以外も条件に引っかかってしまいますね。

tagを取得する関数を作成

@wada
カウンター判定について既存関数のS_GetMasterSkill関数を使いタグを元に判定するように修正しましたので再度確認お願いいたします。

@kouho
条件からskill_groupを外しても大丈夫ですか？
条件判定が同一なので関数化して欲しいです。

@wada
判定にskill_groupを入れて関数化しました

@kouho
S_IsCounterSkill内で呼んでいるS_GetMasterSkillの戻り値が配列になるので対応が必要です。
（$skill_dataのデフォルト引数を無くして、S_GetMasterSkillを使用しない対応でもOK）

Item-ChangeSkillBallSelect.incでも同じような条件式を見つけました。
こちらも対応をおねがいします。

if($skill_data_val['skill_group'] == 1 && $key > 10000){
	//所持スキルの中にカウンターのスキルがある
	$_temp['counter_flg'] = 1;
}

メモ
・和田さん指摘箇所の修正
・スキルチェンジでURL直打ちで防御スキルを覚えることができる
・アイテム欄の技玉ページの修正

@wada
以下対応を行いました。

• S_IsCounterSkill関数のデフォルト引数をなくしてS_GetMasterSkillを使用しない対応。
• Item-ChangeSkillBallSelect.incの条件式も修正。
• Card-ItemSkillCardList.inc (アイテム欄の技玉タブからのリンク先)にも条件式を追加。

TODO
メモに書いているURL直打ちについては未対応

@kouho
function S_IsCounterSkill( $skill_data = null ){の=nullが意味のないコードになっているので削除をお願いします。
他はOKです。

@wada
= nullを削除しました

@kouho
ありがとうございます。確認しました。