PR

【確認次第】転生ガチャIDによるフィルターを保持 #1838

wada opened this at 2022-05-19T11:21:22; closed

Description

opened by wada at 2022-05-19T11:21:22; updated 2022-05-20T13:48:21

概要

【5/17】転生機能 #1725

対象ガチャの転生可能カード一覧が、フィルターやソートすると全転生カードが表示される。
フィルターやソートしても、ガチャIDの条件を保持したままにする。
戦闘デモの戻り先もフィルターやソートが適応されたリストに戻す。

Comment

#19158 by wada at 2022-05-19T11:21:32; updated 2022-05-19T11:22:06

@kouho
レビューお願いします。
デバッグのためdevelopには先にマージしています。

Comment

#19160 by kouho at 2022-05-19T12:01:52; updated 2022-05-19T12:01:52

@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);
}

Comment

#19163 by wada at 2022-05-19T12:13:38; updated 2022-05-19T12:13:38

@kouho
> 念の為LimitBreak-List.incにエラー制御を入れておいていただけないでしょうか?

どのような時に、そのような処理が必要になりますか?

Comment

#19164 by kouho at 2022-05-19T12:32:46; updated 2022-05-19T12:32:46

@wada
$gacha_item = S_GetGachaItemMaster($gacha_id);
こちらはgacha_idのガチャデータが見つからなかった時

$demo_id_no_array = S_GetMasterBattleDemoFromCardNo($demo_card_no_array);
こちらはガチャの転生カード一覧なのに転生対象カードがガチャ内にいなかった時です。

いずれも通常操作や実装では起きないですが何かのURLのリンクミスやURL書き換えでエラーになる可能性があるかと思い、
念のためエラー制御をお願いいたしました。

Comment

#19169 by wada at 2022-05-19T15:16:28; updated 2022-05-19T15:16:28

@kouho
不正な$gacha_idの対応をしました。
細かいエラー対応も実施しています。

Comment

#19170 by kouho at 2022-05-19T15:31:49; updated 2022-05-19T15:31:49

@wada
対応ありがとうございます。
S_GetMasterBattleDemoFromCardNoでタイピングミスをしていましたので確認お願いいたします。
if (empty($cad_no)) {

Comment

#19172 by wada at 2022-05-19T16:01:03; updated 2022-05-19T16:01:03

@kouho
> S_GetMasterBattleDemoFromCardNoでタイピングミスをしていましたので確認お願いいたします。

修正しました。

Comment

#19176 by kouho at 2022-05-19T16:06:40; updated 2022-05-19T16:06:40

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

kouho add_label at 2022-05-19T16:06:44: 皇甫 LGTM
wada merge at 2022-05-19T17:15:53: 【確認次第】転生ガチャIDによるフィルターを保持
wada close at 2022-05-19T17:15:53: Close
wada refer at 2022-05-20T13:48:21: 1845:【5/23】ガチャ転生対応

Commit range

base3f947ca52f44da01c3153b47d61f1d1f69b370c0
head2f63941991270d1e7816ce092b5e093df4f1a8f7
base branchjolly/jr-web:gacha_limit_break
head branchjolly/jr-web:hotfix_gacha_limit_break

View commits

Commit details are loaded from Git when a dynamic archive server is running, or can be embedded with --include-commits.
Files changed5 filesView patch
statusfile
modifiedlib/SubFunc.inc
modifiedlib/public/Demonstration-Battle.inc
modifiedlib/public/LimitBreak-List.inc
modifiedpublic_html/js/JOLLYROGER.js
modifiedpublic_html/template/LimitBreak-List.tpl
Patch body is stored as a separate file to keep this page lightweight.