Ticket #1550 (closed 改善提案: 修正済)

Opened 12 years ago

Last modified 12 years ago

例外エラーと扱うべき状況では、例外エラーを発生させる

Reported by: Seasoft Owned by: Seasoft
Priority: Milestone: EC-CUBE2.12.0
Component: その他 Version: 2.11.4
Keywords: Cc:
修正済み: yes

Description (last modified by Seasoft) (diff)

後続の処理でエラーが発生し、実際の問題点が分かりにくくなるのを防ぐ意図。

r21331, r21833

Change History

comment:1 follow-up: ↓ 3 Changed 12 years ago by kotani

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

comment:2 follow-up: ↓ 4 Changed 12 years ago by h_yoshimoto

ご報告いただきました内容について、具体的に対応が想定される箇所をお教えいただけますでしょうか。

comment:3 in reply to: ↑ 1 Changed 12 years ago by Seasoft

kotani への返信

version-2_11-dev ブランチは r21408 で revert しました。

comment:4 in reply to: ↑ 2 ; follow-up: ↓ 5 Changed 12 years ago by Seasoft

  • Description modified (diff)

h_yoshimoto への返信

ご報告いただきました内容について、具体的に対応が想定される箇所をお教えいただけますでしょうか。

現状では受注登録で対応が必要と判明しています。他にもあると思いますが、現状調査できていません。

comment:5 in reply to: ↑ 4 ; follow-up: ↓ 7 Changed 12 years ago by Seasoft

現状では受注登録で対応が必要と判明しています。他にもあると思いますが、現状調査できていません。

「受注登録」の部分につきまして、分かった範囲で記載しておきます。

商品規格1のみの商品を、受注に追加した場合に例外エラーを検出するようです。product_select.tpl の JavaScript? 関数「fnSetClassCategories」の中で「if (sele2) {」を評価していますが、恐らく else の処理も必要です。この件は、本チケットではなく、別のチケットで扱うべきと考えます。

comment:6 Changed 12 years ago by Seasoft

  • Owner changed from somebody to Seasoft
  • 修正済み set
  • Status changed from new to assigned
  • Description modified (diff)

2.12.0 向けは、これ以上の実装は難しそうなので終わります。

comment:7 in reply to: ↑ 5 ; follow-up: ↓ 8 Changed 12 years ago by Seasoft

別チケットを登録しました。#1815

comment:8 in reply to: ↑ 7 Changed 12 years ago by h_yoshimoto

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

Seasoft さまへの返信

別チケットを登録しました。#1815

ありがとうございます。 こちら閉じます。

Note: See TracTickets for help on using tickets.