Hi,

with rev. 434334 I've split up the CheckOutEvents.finalizeOrderEntry(...) method into two ones:

1) CheckOutEvents.finalizeOrderEntry(...) where all the actions are taken
2) (new) CheckOutEvents.determineNextFinalizeStep(...) where the next step is determined: I've simply moved the logic at the end of the original finalizeOrderEntry method into this new one to make things a bit more clear (hopefully)

The next step should be that using these new methods every time we need to perform an order checkout (both in order manager and ecommerce); right now it seems to me that we have a lot of duplicated methods that performs similar tasks in slightly different ways... and this is a bit scaring.

For example: CheckOutEvents.setCheckOutPages(...) (and related methods) are very similar to the CheckOutEvents.finalizeOrderEntry(...)

I'd like to get your feedback about this.

Thanks,

Jacopo

Reply via email to