Archive / jolly / jr-web
PR

デッキカード選択画面でcard_pos指定が想定外の場合の対応 #1971

kouho opened this at 2022-06-17T15:03:48; closed

Description

opened by kouho at 2022-06-17T15:03:48; updated 2022-06-20T11:34:17

概要

URL直接書き換えでデッキデータがおかしくなる問題への対応用PRです　#1839

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

修正前確認

デッキ編集
→ デッキのカードを変更
→ URL `M=Deck&A=SelectChangeCard&deck_no=6&is_war_deck=&pos=2` の pos=以下に適当な値を入れる
→ カードを選択するとpos1~9以外にカードデータをセットされることを確認

↓
修正後確認

上記と同手順でメインページにリダイレクトされることを確認

TODO

確認できしだい本家に共有して反映予定。

Comment

#20116 by kouho at 2022-06-17T15:03:56; updated 2022-06-17T15:03:56

@wada
確認お願いいたします

kouho assign at 2022-06-17T16:54:50: Not assigned:kouho

Comment

#20133 by wada at 2022-06-17T17:49:03; updated 2022-06-17T17:49:03

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

wada add_label at 2022-06-17T17:49:05: 和田 LGTM
kouho merge at 2022-06-20T10:17:08: デッキカード選択画面でcard_pos指定が想定外の場合の対応
kouho close at 2022-06-20T10:17:08: Close
kouho refer at 2022-06-20T11:34:17: 1974:【6/20】リリース・デッキ編集修正

Commit range

base67575f45f7394350d0dac019e794bc2941a6aff3
head892508bb1f94d05e6ae0ae1a75221d61f828ab27
base branchjolly/jr-web:develop
head branchjolly/jr-web:fix_deck_over

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
modifiedlib/public/Deck-SelectCard.inc
Patch body is stored as a separate file to keep this page lightweight.