魂の器の表示追加 #3278

仕様書：
https://docs.google.com/spreadsheets/d/1EiBvQxHF3e_y783KOsAQ3ZuGhR4zhqpVEFXUqpzzSG8/edit#gid=0

【１】活性カード
!スクリーンショット 2023-12-14 13
【２】活性カード：遷移１
!スクリーンショット 2023-12-14 14
【３】活性カード：遷移２
!スクリーンショット 2023-12-14 14

【４】非活性カード
!スクリーンショット 2023-12-14 14
【５】非活性カード：遷移１
!スクリーンショット 2023-12-14 14

@wada

実装完了しました。
レビューお願い致します。

@suzuki
ソース内のインデントはタブ１つでお願いします。

@suzuki
$_parts['private_soul_items_list']の初期化が、$_temp['item_list']のforeach文内で実装されているため、$_temp['item_list']が空の場合、表示されなくなります。

foreach文の外で初期化してください。

@wada
指摘修正、完了しました。

また、`` onclick="enchangeItem() `と`hiddenオブジェクト``に関しては不要と判断して削除しました。

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

lib/public/Card-ExpItem.inc

// 魂の器取得
$private_soul_no = S_LoadModelClass('Powerup_item')->get_private_soul_item($card_data[$card]['card_no']);
$private_soul_no_int = !empty($private_soul_no) ? (int)$private_soul_no['item_no'] : '';

if (!empty($private_soul_no_int)) {
	$private_soul_items = S_GetUserItem($user_info['user_no'], $private_soul_no_int);

	$_temp['private_soul_items'] = $private_soul_items;
} else {
	$_temp['private_soul_items'] = []; // $private_soul_no_int が空の場合
}

返り値$private_soul_noに実質2度empty()で空チェックを実施しているのため、冗長な処理になっています。
!empty($private_soul_no)と!empty($private_soul_no_int)を統合して、empty()の空チェックを1度だけにしてください。

public_html/template/Card-ExpItem.tpl

if (!empty($_temp['private_soul_items'])) {のif文が、まだif(is_array($_temp['item_list'])){のif文にふくまれています。
if(is_array($_temp['item_list'])){の結果をfalseにした状態で、ゲーム画面を確認し、魂の器が表示されていればOKです。
また、対応時にインデントがズレた場合は、インデントを整え直してください。

foreach ($_temp['private_soul_items'] as $item_no => $private_item) {内の変数$item_type、$img_noについて、処理の変更により変数に格納する意味がなくなったので、変数に代入せず使用してください。

@wada

修正完了しました。

また、``if(is_array($_temp['item_list'])){``の結果をfalseにした状態で、
魂の器が表示されていることも確認済みです。
!スクリーンショット 2023-12-21 14

@suzuki
確認しました。

アイテムリストを非表示にする処理が残っているので削除をお願いします。
$_temp['item_list'] = null;

@wada
修正完了しました

@suzuki
確認OKです。

@suzuki
すでに使用していない変数ですので削除をお願いします。

@suzuki
同一の処理が直後のif文内にもあります。どちらか削除お願いします。

@suzuki
このメソッドの返り値は以下の形式なので、$private_soul_no_int = (int)$private_soul_no['item_no'];でアイテムIDを取得できます。

array('item_no' => 100, 'card_no' => 100);

@suzuki
魂の器はカードにつき1つしか存在しないので、
$private_soul_itemsを$_temp['private_soul_items']に代入するだけで表示に必要な情報は取得できます。

@suzuki
このonclick="enchangeItem()"はどのイベントですか？

@suzuki
このhiddenオブジェクトはどのような用途で使用しますか？