PR

【8/11】戦争デッキ用の表示修正 #1034

iwamoto opened this at 2021-08-10T16:18:10; closed

Description

opened by iwamoto at 2021-08-10T16:18:10; updated 2025-03-26T15:10:03

概要

デッキ編集:戦争デッキ #1023
`戦争デッキに設定しているデッキ編集時、
「セットできるカードのみ表示」がONのときは
他の戦争デッキにセットしているカードをリストに表示しないでほしい`

本番へ反映は**8/11中**を予定

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

  • デッキ編集(戦争ドック)で、戦争デッキ1でカード変更時の表示を変更

「セットできるカードのみ表示」が**ON**の場合、戦争デッキ2にセットしたカードは**非表示**
「セットできるカードのみ表示」が**OFF**の場合、戦争デッキ2にセットしたカードは**表示**
(戦争デッキ2を選択した場合は、戦争デッキ1のカードが非表示の対象)

  • デッキ編集(戦争ドック)で、戦争デッキに設定していないデッキでカード変更時の表示(変更なし)

「セットできるカードのみ表示」がON/OFFに関わらず、戦争デッキ(1or2)にセットしたカードは表示

  • 戦争デッキ(1or2)に設定されているカードを表示した際の文言を変更

変更前「別のデッキに編成中」
変更後「戦争デッキ1に編成中」or「戦争デッキ2に編成中」
また、表示方法を「詳細」「リスト」「サムネイル」へ変更しても同様の表示

TODO

現在のところなし

iwamoto change_title at 2021-08-10T16:20:13: 戦争デッキ用の表示修正 【8/11】戦争デッキ用の表示修正

Comment

#12852 by iwamoto at 2021-08-10T16:25:20; updated 2021-08-10T16:25:20

@wada @kouho
コードレビューをお願い致します。
依頼内容通りの動作になっているかご確認お願いします。

Comment

#12866 by wada at 2021-08-10T18:15:45; updated 2021-08-10T18:15:45

@iwamoto
確認しました。OKです。

Comment

#12873 by kouho at 2021-08-10T19:21:20; updated 2021-08-10T19:21:20

@iwamoto
確認しました。こちら動作問題ないと思います。

kouho add_label at 2021-08-10T19:25:59: 皇甫 LGTM
wada add_label at 2021-08-11T09:43:28: 和田 LGTM
iwamoto merge at 2021-08-11T12:46:38: 【8/11】戦争デッキ用の表示修正
iwamoto close at 2021-08-11T12:46:38: Close

Commit range

base88a59b6299196c44c59b735b08f689c18dded21b
headf46101d8cf55eb92b10cc13ac7894e4f60f8ea55
base branchjolly/jr-web:develop
head branchjolly/jr-web:fix_deck_teamwar

View commits

Commit details are loaded from Git when a dynamic archive server is running, or can be embedded with --include-commits.
Files changed2 filesView patch
statusfile
modifiedlib/class/controller/JrControllerDeck.class.php
modifiedpublic_html/template/Deck-SelectChangeCard.tpl
Patch body is stored as a separate file to keep this page lightweight.