API Giftaid.updatedeclarations overwrites existing declaration address with current address

Giftaid.updatedeclarations overwrites an existing declaration address with the current address of the contact. The documentation says it updates missing addresses so I assumed that it would fill in missing addresses but not change existing ones on declarations.

Do you agree it should only be updating missing addresses?

Separately, but in the same function - why does it specifically disable logging before making these updates? I'd prefer to be able to track all changes in the log.