Ticket #711 (closed バグ指摘: 修正済)
配送業者設定のバリデーションエラーが不適切
Reported by: | Seasoft | Owned by: | pingpong |
---|---|---|---|
Priority: | 低 | Milestone: | EC-CUBE2.12.0 |
Component: | 管理画面 | Version: | コミュニティ (eccube-comu) |
Keywords: | Cc: | ||
修正済み: | yes |
Description
新規登録時「名称」が他と重複した際のエラーが、「配送業者名」のエラーのように表示された。
また、更新時にはエラー検出されない模様。
このエラーチェックが必要なのか自体が懐疑的な気も。
Change History
comment:2 Changed 12 years ago by nanasess
- Milestone changed from EC-CUBE2.5.0beta to EC-CUBE2.5.1(仮)
comment:4 Changed 12 years ago by kotani
- 修正済み unset
- Milestone changed from EC-CUBE2.11.1 to EC-CUBE2.12.0(仮)
comment:5 Changed 11 years ago by pingpong
- Owner changed from somebody to pingpong
- Status changed from new to assigned
comment:6 Changed 11 years ago by pingpong
- 修正済み set
data/class/pages/admin/basis/LC_Page_Admin_Basis_DeliveryInput.php のエラーチェックを変更しました。
行っているエラーは同じ名称の有無であり、配送業者自体はチェック内容とは無関係でしたので、エラー内容は配送業業者のエラーではなく名称のエラーに入力するように変更しました。
また、アップデート時(deliv_idがある場合)は自身以外に同じ名称登録があるかチェックするようにしました。
そもそも、同一配送業者で同じ名称の場合のチェックのほうが意味があるとは思われますが、いかがでしょうか。
※異なる配送業者の場合でも同じ名称だとエラーにする必要性はありますか?
とりあえずチケット内容のエラーは修正したので修正済みとさせていただきます。
comment:7 Changed 11 years ago by kajiwara
- Status changed from assigned to closed
- Resolution set to 修正済
Seasoftさん、pingpongさん
本件、ご報告ならびに修正いただきありがとうございます。
r21531 にて対応済。
たしかに、エラーチェック自体必要かという疑問もなくはないのですが、商品購入者目線で言うと、名称が同じで配送業者が異なる等々の分かりづらい可能性があり、そのことを排除するという意味では、こちらの通り「名称」に対するエラーチェックのみという対応で良いと思います。
本件、こちらにてクローズとさせていただきます。