sourcing from #1744 it came up that the macro $NOTIFICATIONRECIPIENTS$ is populated wrong.

everytime, add_notification is called, a new entry is also added to that macro. so even when adding a contact to the notification list and not notifying afterwards due to the viability checks left the entry in there.

by moving the viability checks before even adding a contact to the notification list, that macro is not populated too and remains clean.