[FUN-1094] Resolve informational finding from audit

[agparadiso]

Description

This mr addresses the findings from the v1.0.0 contract audit more info here [FUN-1094](https://smartcontract-it.atlassian.net/browse/FUN-1094).

• Each of the 4 points have been addressed on separate commits in order to make review simpler.
• The only change that affects logic is the one that does not emits the event AddedAccess(recipient) event if the recipient already has access. In order to validate this works as expected, now the test validate that the log does not contains new events after calling the function.

ℹ️ I've marked the mr as draft until Im able to sign the commits, will rebase an sign once I get access

[github-actions]

I see that you haven't updated any README files. Would it make sense to do so?

[cl-sonarqube-production]

SonarQube Quality Gate

0 Bugs
0 Vulnerabilities
0 Security Hotspots
0 Code Smells

No Coverage information
No Duplication information