Ticket #1887 (new 改善提案)

Opened 8 years ago

Last modified 7 years ago

SC_Helper_Purchase#sfUpdateOrderStatus の呼び出し元で第5引数のセットが不自然

Reported by: Seasoft Owned by: somebody
Priority: Milestone: バックログ
Component: その他 Version: 2.12.1
Keywords: Cc:
修正済み: no

Description (last modified by Seasoft) (diff)

サブルーチン内で UPDATE しているので、その値を受取る必要は無いはず。

むしろ、サブルーチンで更新している値は呼び出し元で破棄するのが妥当だと感じる。その面では、SC_Helper_Purchase#registerOrder の INSERT 側の処理が妥当にも思える (そもそも第5引数は意味を成していないようだが。)。

とりあえず、同じ値を上書きするだけなので、実害はなさそうだが、改訂やカスタマイズで混乱を招きそうにも感じた。


2012/07/05 追記

実装を分析したところ、むしろ UPDATE 時に呼び出し元で UPDATE をコールしているのが蛇足といった状況かも。悩ましい・・・

Change History

comment:1 Changed 8 years ago by Seasoft

  • Description modified (diff)

comment:2 Changed 8 years ago by adachi

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

comment:3 Changed 8 years ago by kim

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

comment:4 Changed 7 years ago by kim

  • Milestone changed from EC-CUBE2.13.0 to バックログ

2.13.0の対応内容を固めていくため、バックログに移動します。

Note: See TracTickets for help on using tickets.