Thanks, that's neater when done before the update_attributes.  Branch updated + rebased.