PR

【管理画面】カードデータの編集 #3011

suzuki opened this at 2023-08-09T01:58:30; closed

Comment

#29640 by suzuki at 2023-08-09T02:25:07; updated 2023-08-09T02:25:07

@wada

現在、基本動作の実装が完了しました。
何点か確認事項があるため質問をさせてください。

1:"体力","攻撃力","速さ"はmaster_card_dataから該当するcard_noでlvが1の値を取得しているようですが、
  更新する際はlvが1のレコードのみで良いのでしょうか?
 →全lvの値がズレる?その場合はどのような規則で設定されている?

2:スキルの変更がされる場合はありますでしょうか?
 →master_cardテーブルのcard_skill1~4の値が"0"と"-1"ではどのような違いがある?

3:更新後に「更新完了しました」等の旨を表示する画面遷移を行う必要がありますか?
 →特別必要性を感じなかったため、現状実装なし。

Comment

#29651 by wada at 2023-08-09T12:02:20; updated 2023-08-09T12:02:20

@suzuki
確認ありがとうございます。以下のとおりです。
また、添付されている画像から想定されることとして、複数カードが選択されている場合の対応が必要かもしれません。

> 1:"体力","攻撃力","速さ"はmaster_card_dataから該当するcard_noでlvが1の値を取得しているようですが、

lv1時のパラメータ変更は、影響範囲が大きいため、今回はcard_skill1~4max_{param}のみ更新でお願いします。
依頼issueも修正しました。

> 2:スキルの変更がされる場合はありますでしょうか?

0と1で以下の違いがあります。
0: SKILL SLOT
1: スキルなし

> 3:更新後に「更新完了しました」等の旨を表示する画面遷移を行う必要がありますか?

更新後に変わってたことが分かればより良いです。

suzuki change_title at 2023-08-23T10:47:21: [WIP]【管理画面】カードデータの編集 【管理画面】カードデータの編集
suzuki add_label at 2023-08-23T10:47:25: レビュー待ち

Comment

#29863 by suzuki at 2023-08-23T10:57:25; updated 2023-08-23T10:57:25

@wada

上記を踏まえて修正しました。

Comment

#29873 by wada at 2023-08-23T14:43:24; updated 2023-08-23T14:43:24

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

wada add_label at 2023-08-23T14:43:27: 和田 LGTM
wada merge at 2023-08-25T10:40:13: 【管理画面】カードデータの編集
wada close at 2023-08-25T10:40:13: Close

Commit range

base9a349916f3e7334b13e3fac82d945558048eaea1
head97c78f4f5c19f0d843649450ef0d32d878b70d75
base branchjolly/jr-web:cpanel
head branchjolly/jr-web:modify_master_card0809

View commits

Commit details are loaded from Git when a dynamic archive server is running, or can be embedded with --include-commits.
Files changed7 filesView patch
statusfile
addedcpanel_html/template/m_card_data_edit.tpl
addedcpanel_html/template/m_card_data_edit_result.tpl
modifiedcpanel_html/template/m_card_search_result.tpl
modifiedcpanel_html/template/parts_card_search_result.tpl
addedlib/cpanel/m_card_data_edit.inc
addedlib/cpanel/m_card_data_edit_result.inc
modifiedlib/cpanel/m_card_search_result.inc
Patch body is stored as a separate file to keep this page lightweight.