Description
概要
【5/17】転生機能 #1725
対象ガチャの転生可能カード一覧が、フィルターやソートすると全転生カードが表示される。
フィルターやソートしても、ガチャIDの条件を保持したままにする。
戦闘デモの戻り先もフィルターやソートが適応されたリストに戻す。
【5/17】転生機能 #1725
対象ガチャの転生可能カード一覧が、フィルターやソートすると全転生カードが表示される。
フィルターやソートしても、ガチャIDの条件を保持したままにする。
戦闘デモの戻り先もフィルターやソートが適応されたリストに戻す。
@kouho
レビューお願いします。
デバッグのためdevelopには先にマージしています。
@wada
フィルター保持は問題ないことを確認しましたが
念の為LimitBreak-List.incにエラー制御を入れておいていただけないでしょうか?
$gacha_item = S_GetGachaItemMaster($gacha_id);↓
$gacha_item = S_GetGachaItemMaster($gacha_id);
if(empty($gacha_item)){
S_redirect("?M=LimitBreak&A=List");
}$demo_id_no_array = S_GetMasterBattleDemoFromCardNo($demo_card_no_array);↓
$demo_id_no_array = [];
if(!empty($demo_card_no_array)){
$demo_id_no_array = S_GetMasterBattleDemoFromCardNo($demo_card_no_array);
}@kouho
> 念の為LimitBreak-List.incにエラー制御を入れておいていただけないでしょうか?
どのような時に、そのような処理が必要になりますか?
@wada
$gacha_item = S_GetGachaItemMaster($gacha_id);
こちらはgacha_idのガチャデータが見つからなかった時
$demo_id_no_array = S_GetMasterBattleDemoFromCardNo($demo_card_no_array);
こちらはガチャの転生カード一覧なのに転生対象カードがガチャ内にいなかった時です。
いずれも通常操作や実装では起きないですが何かのURLのリンクミスやURL書き換えでエラーになる可能性があるかと思い、
念のためエラー制御をお願いいたしました。
@kouho
不正な$gacha_idの対応をしました。
細かいエラー対応も実施しています。
@wada
対応ありがとうございます。
S_GetMasterBattleDemoFromCardNoでタイピングミスをしていましたので確認お願いいたします。if (empty($cad_no)) {
@kouho
> S_GetMasterBattleDemoFromCardNoでタイピングミスをしていましたので確認お願いいたします。
修正しました。
@wada
対応ありがとうございます。確認OKです。
| base | 3f947ca52f44da01c3153b47d61f1d1f69b370c0 |
|---|---|
| head | 2f63941991270d1e7816ce092b5e093df4f1a8f7 |
| base branch | jolly/jr-web:gacha_limit_break |
| head branch | jolly/jr-web:hotfix_gacha_limit_break |
| status | file |
|---|---|
| modified | lib/SubFunc.inc |
| modified | lib/public/Demonstration-Battle.inc |
| modified | lib/public/LimitBreak-List.inc |
| modified | public_html/js/JOLLYROGER.js |
| modified | public_html/template/LimitBreak-List.tpl |