PR

S_DeleteDeckCard関数の中身をupdateに修正 #3646

kouho opened this at 2024-08-09T17:21:44; closed

Description

opened by kouho at 2024-08-09T17:21:44; updated 2026-04-14T09:30:00

概要

以下の問題への対応
中戸さんが開発で戦争デッキを選択 or 転生した際に以下のエラーが発生
!FireShot Capture 220 - - develop

 S_GetSubDeckがループしていた。
  ⇒ user_no 10003のuser_deck_stock_teamWarのデータが空になっていたことによるもの

 そもそもuser_deck_stock_teamWarがなぜ空になったか
  ⇒ デッキにセットされているカードのuser_card_holderのデータがDELETEされた後、S_GetSubDeckを通るとS_DeleteDeckCard関数によりuser_deck_stock_teamWarのデータがDELETEされる。
  ⇒ 今回は /?M=Debug&dell_card でカードを捨てるが使われていた
    (デバッグメニューの装備していないカードを捨てる)を利用したことによる。実際はデッキ装備判定が効いていない。

 DELETEではなく、UPDATEでcard_holder_no = 0を入れることでデータが空になることはないようにして対応

Comment

#36615 by wada at 2024-09-24T15:43:19; updated 2024-09-24T15:43:19

@kouho
ブランチがないようです。

wada add_label at 2024-09-24T15:43:21: コメントあり

Comment

#36640 by kouho at 2024-09-26T15:03:07; updated 2024-09-26T15:03:07

@wada
すみません、ブランチプッシュし直しました

Comment

#36659 by wada at 2024-09-27T15:53:53; updated 2024-09-27T15:53:53

@kouho
詳細がうまく理解できないので、月曜日に口頭で確認させてください。

kouho close at 2026-04-14T09:30:00: Close

Commit range

based7aad4cd70f7afc74c5bcf4a643dd704d7a7f325
heada4ae8e136388c59fe97e19a41512c731cff86caa
base branchjolly/jr-web:master
head branchjolly/jr-web:fix_delete_card

View commits

Commit details are loaded from Git when a dynamic archive server is running, or can be embedded with --include-commits.
Files changed1 filesView patch
statusfile
modifiedlib/SubFunc.inc
Patch body is stored as a separate file to keep this page lightweight.