Description
opened by kouho at 2021-10-27T16:34:28; updated 2025-12-01T11:57:21
概要
user_infoのwar_deckと戦争デッキ1に設定しているデッキの不整合を修正するPRです。
戦争の一騎討ちの際にwar_deck=0が入っているとメインデッキが参照されるため修正と
合わせて不整合が起きている場合に修正をするようにしています。
- デッキ編集画面での整合性チェック
- 不整合を修正するスクリプト
スクリプトは既存スクリプトを修正しましたが、戦争デッキ1にデッキを設定していない場合の対応を削除しています。
テスト方法、確認した内容
デッキ編集画面での確認(戦争デッキ1に現在設定しているデッキ番号が入ればOK)
- war_deckに0を入れての確認
- war_deckに戦争デッキ1に現在設定しているデッキ番号と違う番号を入れての確認
スクリプトの実行確認(戦争デッキ1に現在設定しているデッキ番号が入ればOK)
ENVIRONMENT=develop PF=yamada HTTP_HOST=127.0.0.1 IS_DOCKER=true php tool_TeamWar_UserInfoWarDeck_Update.php
- war_deckに0を入れての確認
- war_deckに戦争デッキ1に現在設定しているデッキ番号と違う番号を入れての確認
TODO
本番反映後にスクリプト実行予定
kouho assign at 2021-10-27T16:34:32: Not assigned:kouho
kouho add_label at 2021-10-27T16:34:38: バグ
kouho add_label at 2021-10-27T16:34:40: レビュー待ち
Comment
#14709 by kouho at 2021-10-27T16:36:45; updated 2021-10-27T16:36:45
@wada @iwamoto
確認お願いいたします。
Comment
#14717 by kouho at 2021-10-27T18:16:05; updated 2021-10-27T18:16:05
@wada
$sub_decks[1]には戦争デッキ1に登録しているデッキ番号を入れる想定でしたがif文の判定が誤っていたため修正しました
Comment
#14724 by wada at 2021-10-28T10:41:13; updated 2021-10-28T10:41:13
wada add_label at 2021-10-28T10:41:17: 和田 LGTM
Comment
#14735 by iwamoto at 2021-10-28T11:23:44; updated 2021-10-28T11:23:44
@kouho
確認しました、問題ないように思います。
iwamoto add_label at 2021-10-28T11:23:48: 岩本 LGTM
kouho merge at 2021-10-28T11:28:25: 【10/28】user_infoのwar_deckと戦争デッキ1に設定しているデッキの不整合を修正
kouho close at 2021-10-28T11:28:25: Close
kouho refer at 2021-10-28T11:50:24: 1261:【10/28】イベント・戦争デッキ修正リリース
commit comment
#95 by wada at 2021-10-27T17:59:41; lib/class/controller/JrControllerDeck.class.php / old:None new:80
b2dc8e870564a278502a59178555eb8f8408618b
@kouho
$sub_decks[1] は、どのような値を想定していますか？
確認した範囲では、$sub_decks[1] == $this->_user_info->war_deck になるケースがありませんでした。
正確には、$this->_user_info->war_deck != 0 かつ $sub_decks[1] == $this->_user_info->war_deck で、このif文に入らなくなるケースがないという意味です。