スキルチェンジ修正本家マージ #1957

概要

スキルチェンジ不具合について本家対応の確認と必要箇所のマージ #1924
の対応用PR

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

TODO

@kouho
以下の認識で間違っていませんか？

強化アイテムを使用してスキルグループが変わっている場合、スキルグループの重複チェックは、強化アイテム使用前のグループをチェックする。

強化アイテム使用後

• 強化後スキルと同じグループ

装備できる

• 強化前スキルと同じグループ

装備できない

ラゼストprのコメントの抜粋

> ■強化アイテムのタイプが「強化」の場合
・強化前後でスキルグループが変わらない：そのスキルグループはＮＧ
→分かる（もともと付けられないし）
・強化前後でスキルグループが変わる：強化後のスキルグループの技玉はＮＧ
→分からない
（むしろもともと付けられないはずの強化前のスキルグループをＮＧにするべきでは？強化後のスキルと同じものを技玉でつけられるのを防いでもいるけど）

@wada
確認ありがとうございます。
和田さんがまとめてくれた認識で間違いないかと思います。

@kouho
直感的には現在の仕様（強化アイテム使用後スキルで重複チェック）の方が正しく感じるんですが、これ実装して大丈夫なんでしょうか？

@wada
確かにそのまま入れると仕様が変わってしまうので相談した方が良さそうですね。。
MTGで聞いてみることにします。

@kouho
Item-ChangeSkillCheck.incに追加された処理で、$_temp['skill_group']が作成されていますが、使用している箇所が見当たりません。
pushされていない更新はないでしょうか？

@wada
対応が遅れてすみません。本家からのマージ漏れの箇所をマージしました。

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