Ticket #810 (closed バグ指摘: 修正済)
商品に割り当てている規格分類を削除できてしまう
Reported by: | Seasoft | Owned by: | undertree |
---|---|---|---|
Priority: | 低 | Milestone: | EC-CUBE2.13.0 |
Component: | 管理画面 | Version: | 2.4.4 |
Keywords: | Cc: | ||
修正済み: | yes |
Description
フォーラムにて kumakuma39 様からご報告いただきました。
http://xoops.ec-cube.net/modules/newbb/viewtopic.php?topic_id=6756&forum=8
Change History
comment:2 Changed 13 years ago by nanasess
- Milestone changed from EC-CUBE2.5.0beta to EC-CUBE2.5.1(仮)
comment:4 Changed 13 years ago by kajiwara
- 修正済み unset
- Milestone changed from EC-CUBE2.11.1 to EC-CUBE2.11.2
引き続きの対応とします。
comment:5 follow-up: ↓ 18 Changed 13 years ago by kajiwara
規格の削除ができないというのは仕様上も問題と思われるので、
「規格を削除する前にメッセージを表示する」
という案を採用さいたいと思います。
どなたか、修正いただければ取り込みたいと思います。
よろしくお願いします!
comment:7 Changed 13 years ago by 468
- Owner changed from somebody to 468
- Status changed from new to assigned
comment:10 Changed 12 years ago by kajiwara
- Owner changed from 468 to somebody
- Status changed from assigned to new
本件、担当者が割り当てられてから時間が経過しておりますので、一旦担当者を戻させていただきます。
comment:12 Changed 12 years ago by h_yoshimoto
- Milestone changed from EC-CUBE2.12.2 to EC-CUBE 2.12.3
comment:14 Changed 11 years ago by kim
- Milestone changed from EC-CUBE2.12.3 to EC-CUBE2.12.4
dkazuto様
本件アサイン後だいぶ時間が経過されております。 よろしければ、2.12.4までにご対応いただけますでしょうか。 ご対応いただけない場合は、担当をsomebodyに更新いただけますと幸いです。
comment:15 Changed 11 years ago by undertree
- 修正済み set
対応が遅くなって大変申しわけございません。 r22303 としてコミットいたしました。
削除時に下記メッセージのポップアップ(window.confirm())を追加して、確認を行うように変更しております。
分類名を削除すると、その分類を利用している商品規格が無効になります。 整合性の問題を把握し、バックアップを行ってから削除することを推奨致します。
comment:18 in reply to: ↑ 5 ; follow-up: ↓ 19 Changed 11 years ago by nanasess
kajiwara への返信
規格の削除ができないというのは仕様上も問題と思われるので、
「規格を削除する前にメッセージを表示する」
という案を採用さいたいと思います。
遅い反応ですみませんが、そもそも削除してしまうと、その規格が使用できなくなるので、削除できない方が正しいのではないでしょうか。。。
comment:19 in reply to: ↑ 18 Changed 11 years ago by undertree
nanasess への返信
kajiwara への返信
規格の削除ができないというのは仕様上も問題と思われるので、
「規格を削除する前にメッセージを表示する」
という案を採用さいたいと思います。遅い反応ですみませんが、そもそも削除してしまうと、その規格が使用できなくなるので、削除できない方が正しいのではないでしょうか。。。
規格の削除は現在は行えないようになっています。(削除ボタンがない) 今回の修正は「分類名」に関する部分です。
いずれにせよ、DBの不整合が発生しうるには違いが無いので、どのような仕様にするべきか決める必要はあるかと思います。
・警告が出るが削除できる(<-いまここ) ・削除は出来ない ・使用されている場合は削除できない ・使用されている場合は削除できない+どの商品が利用しているのか表示する
下に行くほど修正の難度は高くなりますが、望ましい動作ではあると思います。
チケットの経緯から、ひとまず警告追加で対応いたしましたが、3番目くらいまでなら来週対応できると思います。
如何致しましょうか?
comment:20 Changed 11 years ago by h_yoshimoto
- Status changed from assigned to closed
- Resolution set to 修正済
#2366 別チケットを発行させて頂きます。 本チケットとしては一旦クローズとさせて頂きます。