【10/14】デッキページでundefinedエラーが多かったため修正 #2347

概要

デッキ編集の船を変更・カードを変更のページにアクセスすると大量のエラーログが流れたため修正するPRです。

# 特に問題の箇所 SubFunc.inc S_GetSkillName関数
$skill_data[$sqlResult['skill_no']] = preg_replace('/名前/',$skill_name[$fellow_array[$card_no]],$sqlResult['skill_name']);

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

修正前確認

• ローカルのエラーphp-fpmのエラーログを削除
• デッキ編集のカードを変更ページにアクセスしphp-fpmのエラーログの容量を確認

修正後確認

• ローカルのエラーphp-fpmのエラーログを削除
• デッキ編集のカードを変更ページにアクセスしphp-fpmのエラーログの容量を確認

エラーログの容量が少なくなっていればエラー削減の確認はOK

TODO

確認出来次第反映予定

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

@kouho
それぞれのエラーが発生した理由はわかりますか？

@kouho
以下の箇所は、デバッグ期間延長による$_df['battle_event']['ship_bonus_list']不足が原因だと思われますので、config側の対応の方が良いです。
（_configureTestEnv内で、延長分の$this->_config['ship_bonus_list']を追加するなど）

//船のボーナスがループしない場合
if( $time >= $_df['battle_event']['start'] && $time <= $_df['battle_event']['end']){
	if(isset($ship_type_array[$day_count])){
		$return = $ship_type_array[$day_count];
	}else{
		$return = [];
}

> デバッグ期間延長による$_df['battle_event']['ship_bonus_list']不足が原因だと思われますので、config側の対応の方が良いです。

S_CheckBattleEventShipScheduleの戻り値が必ず返ってくることを前提として実装されているため

@wada
> それぞれのエラーが発生した理由はわかりますか？

和田さんが書いている通り、賞金首のデバッグ期間延長によるものと、

S_GetFellowSkill関数で仲間スキルかどうかというところが正確に判定されておらず、
skill_noでおおまかな判定となっており$fellow_array[$card_no]の存在判定がされていないからです。

@kouho
ありがとうございます。
全てのエラーが発生している状態で確認したかったので聞いてみました。

@wada
賞金首イベントの船ボーナスの関数修正はもとに戻しました

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