Ticket #1887 (new 改善提案) — at Version 1
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 をコールしているのが蛇足といった状況かも。悩ましい・・・
Note: See
TracTickets for help on using
tickets.