You're right about the implementation.
However I'm using this form of setter (with `null`) in order to be able to make SiteAccess a synchronized service (hopefully later for 5.3).