ありがとうございます。
無理やり答えさせたみたいで申し訳ないです。

この件とは直接関係あるわけではないのですが、よろしければ下記をご覧ください。
※既にご存知でしたら申し訳ありません。

・カスタマイズと移行の手引（v2.0）
・EC-CUBEのフォーク版

フォーク版の記事のほうに当時の私もズケズケと書いておりますが、2系の形はnanasessさんが作ったもので、そのメインは自動更新の部分と複数カテゴリだったと思います。
1系のページ周りや関数をガッツリとクラス分けしたのもこのときです。

で、カスタマイズと移行手順のほうに書かれていますが
引用：
という分け方で、当時はとりあえず分けて、動くところまで持っていって、後から見直そう的なムードでした。
※私がそう思っていただけかも知れませんが。

結局のところ今でもそのまんまなのですが、見直すチャンスでもあるのかなぁと。

seasoftさんの考えは現状をなるべくいじらず、簡単に実装することだと思います。
※違っていたらごめんなさい

私が思ったのは、その処理はどこでやるべきかみたいな。
場合によっては「ステータスをリターンするコード」がいいのではないかと。
深く考えているわけではないので本当に申し訳なないです...

なんだか大事を書いちゃってますが、そういう意見を他の人からも聞いてみたいなぁと思っています。