animal.insert_animallocation is writing extra records to the log

[bobintetley]

Bug in the new code that writes exit locations to animallocation in animal.insert_animallocation

It's writing the same messages to the log that it writes to animallocation, which shouldn't be an issue, but if you save an animal record that is already dead or has an exit movement, it's writing a log message every time.

[bobintetley]

It's repeating the logs for both the exit movement and the return if a return date is set.

Given that there are tests against the movement ID being present in animallocation to stop it doing this, something is not quite right.