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

Opened 13 years ago

Last modified 11 years ago

配送業者設定のバリデーションエラーが不適切

Reported by: Seasoft Owned by: pingpong
Priority: Milestone: EC-CUBE2.12.0
Component: 管理画面 Version: コミュニティ (eccube-comu)
Keywords: Cc:
修正済み: yes

Description

新規登録時「名称」が他と重複した際のエラーが、「配送業者名」のエラーのように表示された。

また、更新時にはエラー検出されない模様。

このエラーチェックが必要なのか自体が懐疑的な気も。

Change History

comment:1 Changed 12 years ago by Seasoft

  • Keywords 2.5.1 added

comment:2 Changed 12 years ago by nanasess

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

comment:3 Changed 12 years ago by Seasoft

  • Keywords 2.5.1 removed

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 にて対応済。

たしかに、エラーチェック自体必要かという疑問もなくはないのですが、商品購入者目線で言うと、名称が同じで配送業者が異なる等々の分かりづらい可能性があり、そのことを排除するという意味では、こちらの通り「名称」に対するエラーチェックのみという対応で良いと思います。

本件、こちらにてクローズとさせていただきます。

Note: See TracTickets for help on using tickets.