PR

【即時】戦争デッキ編集でカードを外した際に戦闘力が更新されない #2229

kouho opened this at 2022-08-26T16:21:57; closed

Description

opened by kouho at 2022-08-26T16:21:57; updated 2022-08-26T17:21:32

概要

デッキ編集でカードを外した際に戦闘力が更新されない
戦争デッキ特有の問題

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

戦争デッキ1のカードを外して海賊団ページの団員リストに表示される戦闘力が正しいか確認

TODO

確認でき次第本番反映

Comment

#21879 by kouho at 2022-08-26T16:22:07; updated 2022-08-26T16:22:07

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

Comment

#21880 by wada at 2022-08-26T16:49:22; updated 2022-08-26T16:49:22

@kouho
通常デッキの設定取得が、戦争デッキ更新時でも更新されるようになっていました。
害はないと思いますが、どう影響するかわからないので対応した方が良いかもしれません。

	//通常デッキの設定取得
	$deck_set_data = S_GetUserDeckSet($user_no);
	foreach($deck_set_data as $tmp_deck_type => $tmp_deck_data){
		if($deck == $tmp_deck_data['deck_no']){
			//デッキ設定がされていれば戦闘力を更新
			$tmp_deck_data['deck_power'] = S_GetSubDeckPower( $deck );
			S_SetUserDeckSet($user_no,$tmp_deck_type,$deck,$tmp_deck_data['deck_power']);
		}
	}

Comment

#21881 by kouho at 2022-08-26T16:55:38; updated 2022-08-26T16:55:38

@wada
確認ありがとうございます。
戦争デッキと通常デッキで処理を分けました。

Comment

#21882 by wada at 2022-08-26T17:03:54; updated 2022-08-26T17:03:54

@kouho
ありがとうございます。確認OKです。

wada add_label at 2022-08-26T17:03:57: 和田 LGTM
kouho merge at 2022-08-26T17:21:32: 【即時】戦争デッキ編集でカードを外した際に戦闘力が更新されない
kouho close at 2022-08-26T17:21:32: Close

Commit range

baseee1f77bc6af0caeffdc282cc7ce90c4b7ca27351
headae2f565cdc3f0c9d9b8514a70f92cfdf58d785a8
base branchjolly/jr-web:master
head branchjolly/jr-web:fix_war_deck_powerset

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