right now this gets called when you edit the location and want to update the supply point too. so really what it's checking is "did any of the properties in the location that i care about" change? i think the last check (`location_id`) is not possible in any current codepath.