[ic] Proposed Route patch

[Josh Lavin]

It seems that the long-standing comment about the 'main' Route in
catalog.cfg is wrong:

    ## This route emails the order to you unless email is set to "", and
    ## failsafe-logs the order report a couple of places

If you follow those instructions, then you see an error:

    > Empty order routing main_entry (and not explicitly empty).
    > Either attach or email are required in the route setting.

Kevin's docs are correct in this area:
"Note that either attach, email or empty must be set for a route."
http://interchange.rtfm.info/icdocs/Order_routing.html#email

If you blank out 'email', but also set either 'attach' or 'empty' to 1,
then customer orders never show a receipt page, nor clear the cart, yet
an order is accepted.

I followed the rabbit, and this ocurrs as $status is returned by
route_order() and is necessary to show the receipt and clear the cart.
Yet, $status is only set if emails are sent.

The following patch fixes it, and corrects the documentation:

https://github.com/jdigory/interchange/commit/928a93b11c8862f38f201c0d1fdfb94d777cee69

Does anyone know if setting $status to true when no emails are sent is a
problem? Routing is an odd beast, but this seems to work fine in my
tests.

Thanks,
Josh 
-- 
Josh Lavin
End Point Corporation