Cdodge/shopping cart rewrite

General nit: "order_placed_by" is a terrible variable name. This represents a name, yes? To provide maximum context for translators, use variable names that represent what the variable holds in a way that is plain and clear, such as, "{purchaser_name}" ("{purchaser_name} placed an order...")