リリース済み強化アイテムの判定を修正 所持状況を考慮 #2579

概要

以下のissueへの対応用PR
https://git.dev.d-es.co.jp/gitbucket/jolly/jr-cs-issue/issues/247

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

氷河のロロノアで確認

対象ページ

• カードアルバム(Card-AlbumDetail)
• デッキ編集のカード選択した画面(JrControllerDeck)
• カード詳細(Card-Detail)

対象ページで未確認のページ

• 海域イベントデッキ編集(JrControllerOceanArea)

確認内容

• 強化アイテムを所持していたら表示される
• 強化アイテムを所持していなくてアルバム登録されていたら表示
• 強化アイテムを所持していなくてアルバム登録されていなければ非表示

TODO

懸念点・備考

Card-Detailでは関数外でアイテムの所持状況を取得しているためアイテム所持状況取得処理は被りますが、
良き対応方法が思いつかないためそのままにしてみています。

統一性が必要だと思いますので関数を新たに定義する訳ではなく、既存関数と引数を修正する形で対応しました。

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

@kouho
SubFunc.incのS_GetCardAvailableItemで、85695~85715行目（S_GetUserItem,S_GetItemMasterの箇所）がif($search_item_no_list){の外に出た理由はなんでしょうか？

search_item_no_listが空の時に、アイテムを全件取得しています。

@wada
確認ありがとうございます。

実装中に何かを考えていたのかもですが、再度確認したところ、
$search_item_no_listを外に出す理由がありませんでしたので判定の中に入れるようにしました。

@kouho
ありがとうございます。

85697行目のS_GetUserItem戻り値がnullだとforeachでエラーが出るので、こちらの対応もお願いします。

user_itemがあると、アイテムの表示順がおかしくなるので、対応できるならお願いしたいです。

@wada
確認ありがとうございます。以下対応しました。
> 85697行目のS_GetUserItem戻り値がnullだとforeachでエラーが出るので、こちらの対応もお願いします。
> user_itemがあると、アイテムの表示順がおかしくなるので、対応できるならお願いしたいです。

アイテムの表示順については良い方法が思い浮かばず数強化アイテム数が増えるわけでもないと想定して、
loopの中で並び替え処理を入れてます。

@kouho
対応ありがとうございます。確認OKです。