spree/spree — pull request #4775 by magnusvk (2014-06-03 18:18:54+00:00)
Title: Make sure we always choose the best promotion when recalculating

Before this change, we could end up with multiple eligible promotions on
the same order. This happened because changing the order (for example,
by adding line items) rechecked the eligibility of promotions,
potentially making multiple ones eligible, but forgot to re-choose the
one best promotion.

This change adds a call to the logic that chooses the best promotion any
time the ```OrderUpdater``` recalculates adjustments.

I believe this fixes #4675