Description
デッキ編集リニューアル #903
概要
・デッキ編集
-> TOP
-> カード詳細
-> カード変更Q&A 追加登録
既存のものでは不要なものが多数記載されているのでq_idを新しく追加登録する。
新規 q_id = 224Comment
@wada @kouho
お手すきの際にレビュー確認お願いします。
デッキ編集はガラッと変更しています
・ヤマダ版では未実装の機能についての表示&説明がないか
・既存機能は問題なく使用できるか
・カード詳細ページの検索条件
このあたりを特にみていただければと思います。
iwamoto add_label at 2021-06-30T18:06:18: レビュー待ち
iwamoto change_title at 2021-06-30T18:06:37: [WIP]デッキ編集のリニューアル
デッキ編集のリニューアル
Comment
ヤバすぎるw
Comment
@iwamoto
レイドページのデッキ編集ページなどの古いリンクで、デッキ番号が機能しなくなるので、
リンクの修正かController内で古いgetパラメータの制御をお願いします。
./?M=Deck&deck={$_temp['use_deck_no']}errorパラメータの制御って今回のマージに入っていますか？
（なくて良いです。見つからなかったのでただの確認です）
//交易中の船だった
S_redirect("./?M=Deck&A=Home&deck_no={$set_deck_no}&is_war_deck=1&error=2");Comment
@iwamoto
S_CheckBonusFollowerCardについて、Followerクラスのメソッド呼び出しにしてほしいと書きましたが、今後のマージでも同様にS_CheckBonusFollowerCard関数を使用している箇所があるかもしれないので、
SubFunc.incにS_CheckBonusFollowerCard関数を追加し、関数内でFollowerクラスのメソッドを呼び出してください。
Comment
@wada
下記対応しまいた
・GiantKillerConfigのボーナスの閾値を修正しました
・S_CheckBonusFollowerCard関数を追加 & 修正しました
・JrControllerDeckの$deck_set_dataは削除しました
Comment
@iwamoto
対応確認できました。
JrControllerDeck内で、S_CheckBonusFollowerCard関数と同じくクラスメソッドになっていた関数があったので同じ対応をお願いします。
L460: S_GetNeedBreakThroughItem => Breakthrough.php
L486: S_GetResurrectionItemData => Resurrection.ph
L511: S_GetResurrectionData => Resurrection.ph
L560: S_GetMasterLimitBreakItem => Limit_break.php
L599: S_GetLimitBreakMaster => Limit_break.phpSubFunc.incから逃がそうとして諦めたため、めんどくさいことになってごめんなさい。
Comment
@wada
同一コードの削除を行いました。
if (in_array($card_holder["card_holder_no"], $ignore_follower_holder_nos)) {また、下記対応も行いました。
L460: S_GetNeedBreakThroughItem => Breakthrough.php
L486: S_GetResurrectionItemData => Resurrection.ph
L511: S_GetResurrectionData => Resurrection.ph
L560: S_GetMasterLimitBreakItem => Limit_break.php
L599: S_GetLimitBreakMaster => Limit_break.phpComment
@iwamoto
対応確認しました。
Comment
@iwamoto
デッキのカードを選択した画面ですが、
こちらの未リリースの強化アイテムと専用ドリンクは非表示にしている方が良いと思います。
Comment
@iwamoto
!スクリーンショット 2021-07-06 16
こっちは本家
!スクリーンショット 2021-07-06 16
本家はカードの詳細モーダルから「アルバム詳細へ」のリンクがあるようなので、可能であれば付けていただければと思います。
Comment
@iwamoto
!スクリーンショット 2021-07-06 16
本家も同様でしたが、船を変更する際はタイトルを「変更したい海賊を選ぼう」から「変更したい船を選ぼう」にしている方が良いかと思います。
元々に合わせて「変更したいカードを選ぼう」に統一でもOKです。
wada refer at 2021-07-06T17:59:14: 903:デッキ編集リニューアル
Comment
@iwamoto
SubFunc.incの戦争船チェックが数値でのチェックなので、define.incの$_df['WAR_SHIP_NO']を参照してください。
//戦争用の船かチェック
function S_IsWarShip($ship_card_no)
{
return ($ship_card_no >= 10021 && $ship_card_no <= 10032) || ($ship_card_no >= 10041 && $ship_card_no <= 10044);
}Comment
@wada
失礼しました、こちら対応しました。
Comment
@wada
船強化のリンク先直しました
Comment
@iwamoto
修正確認しました。
ソースの確認は完了しました。
簡単な動作確認も完了したので、あとはデバッグに投げてしまいたい。
wada add_label at 2021-07-07T17:47:15: 和田 LGTM
kouho add_label at 2021-07-07T17:51:25: 皇甫 LGTM
iwamoto merge at 2021-07-09T14:31:18: Merged by user
iwamoto close at 2021-07-09T14:31:18: Close
commit comment
ボーナスの閾値がRaidConfigの数値と一致しないため、今後のマージで仕様とのズレが発生し得ます。
RaidConfigの数値を参照するようにしてください。
commit comment
S_CheckBonusFollowerCard関数は、Followerクラスに取り込んでしまったので、クラスメソッドの呼び出しに変更してください。
commit comment
$deck_set_dataが使われてなさそうなので、問題なければ削除お願いします。
commit comment
同じチェックが連続で設定されています。
// デッキに設定されているカードの親分は設定できない
if (in_array($card_holder["card_holder_no"], $ignore_follower_holder_nos)) {commit comment
表記が重いときは？の項目は、まだエフェクトが表示される機能が実装されていないので、非表示で大丈夫です。
commit comment
酒はリリース済みなので、リンクは機能させて、遷移先のQ&Aの内容を調整お願いします。（管理画面で）
commit comment
falseだと見逃しやすいので、変数などで対応お願いします。
commit comment
このリンクが機能していません。