No need to reassign line_item

The same object is passed around
Make PromotionItemHandler have pluggable behaviour