Simplify ReceivingOrderItemStep

Since it doesn't use any SellableItemStep's functionality, there's no point in
inheriting from the last.

Also, clean up some old code that wasn't used (and was broken too).

Change-Id: I8b9b76d355a2237786919d19a36cce8845c80f72