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

[Seasoft]

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

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

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

---

2012/07/05 追記

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

[kim]

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