Ticket #810 (closed バグ指摘: 修正済)

Opened 10 years ago

Last modified 7 years ago

商品に割り当てている規格分類を削除できてしまう

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:1 Changed 10 years ago by Seasoft

  • Keywords 2.5.1 added

comment:2 Changed 10 years ago by nanasess

  • Milestone changed from EC-CUBE2.5.0beta to EC-CUBE2.5.1(仮)

comment:3 Changed 10 years ago by Seasoft

  • Keywords 2.5.1 removed

comment:4 Changed 9 years ago by kajiwara

  • 修正済み unset
  • Milestone changed from EC-CUBE2.11.1 to EC-CUBE2.11.2

引き続きの対応とします。

comment:5 follow-up: ↓ 18 Changed 9 years ago by kajiwara

規格の削除ができないというのは仕様上も問題と思われるので、
「規格を削除する前にメッセージを表示する」
という案を採用さいたいと思います。

どなたか、修正いただければ取り込みたいと思います。

よろしくお願いします!

comment:6 Changed 9 years ago by kajiwara

  • Milestone changed from EC-CUBE2.11.2 to EC-CUBE2.11.3(仮)

comment:7 Changed 9 years ago by 468

  • Owner changed from somebody to 468
  • Status changed from new to assigned

comment:8 Changed 9 years ago by kotani

  • Milestone changed from EC-CUBE2.11.3 to EC-CUBE2.11.4(仮)

comment:9 Changed 9 years ago by kotani

  • Milestone changed from EC-CUBE2.11.5 to EC-CUBE2.12.0alpha

comment:10 Changed 8 years ago by kajiwara

  • Owner changed from 468 to somebody
  • Status changed from assigned to new

本件、担当者が割り当てられてから時間が経過しておりますので、一旦担当者を戻させていただきます。

comment:11 Changed 8 years ago by pingpong

  • Owner changed from somebody to pingpong

comment:12 Changed 8 years ago by h_yoshimoto

  • Milestone changed from EC-CUBE2.12.2 to EC-CUBE 2.12.3

comment:13 Changed 8 years ago by pingpong

  • Owner changed from pingpong to dkazuto

comment:14 Changed 8 years ago by kim

  • Milestone changed from EC-CUBE2.12.3 to EC-CUBE2.12.4

dkazuto様

本件アサイン後だいぶ時間が経過されております。 よろしければ、2.12.4までにご対応いただけますでしょうか。 ご対応いただけない場合は、担当をsomebodyに更新いただけますと幸いです。

comment:15 Changed 8 years ago by undertree

  • 修正済み set

対応が遅くなって大変申しわけございません。 r22303 としてコミットいたしました。

削除時に下記メッセージのポップアップ(window.confirm())を追加して、確認を行うように変更しております。

分類名を削除すると、その分類を利用している商品規格が無効になります。
整合性の問題を把握し、バックアップを行ってから削除することを推奨致します。

comment:16 Changed 8 years ago by dkazuto

  • Owner changed from dkazuto to undertree

comment:17 Changed 8 years ago by undertree

  • Status changed from new to assigned

comment:18 in reply to: ↑ 5 ; follow-up: ↓ 19 Changed 8 years ago by nanasess

kajiwara への返信

規格の削除ができないというのは仕様上も問題と思われるので、
「規格を削除する前にメッセージを表示する」
という案を採用さいたいと思います。

遅い反応ですみませんが、そもそも削除してしまうと、その規格が使用できなくなるので、削除できない方が正しいのではないでしょうか。。。

comment:19 in reply to: ↑ 18 Changed 8 years ago by undertree

nanasess への返信

kajiwara への返信

規格の削除ができないというのは仕様上も問題と思われるので、
「規格を削除する前にメッセージを表示する」
という案を採用さいたいと思います。

遅い反応ですみませんが、そもそも削除してしまうと、その規格が使用できなくなるので、削除できない方が正しいのではないでしょうか。。。

規格の削除は現在は行えないようになっています。(削除ボタンがない) 今回の修正は「分類名」に関する部分です。

いずれにせよ、DBの不整合が発生しうるには違いが無いので、どのような仕様にするべきか決める必要はあるかと思います。

・警告が出るが削除できる(<-いまここ) ・削除は出来ない ・使用されている場合は削除できない ・使用されている場合は削除できない+どの商品が利用しているのか表示する

下に行くほど修正の難度は高くなりますが、望ましい動作ではあると思います。

チケットの経緯から、ひとまず警告追加で対応いたしましたが、3番目くらいまでなら来週対応できると思います。

如何致しましょうか?

comment:20 Changed 7 years ago by h_yoshimoto

  • Status changed from assigned to closed
  • Resolution set to 修正済

#2366 別チケットを発行させて頂きます。 本チケットとしては一旦クローズとさせて頂きます。

Note: See TracTickets for help on using tickets.