Use functions rather than direct queries

Howdy

further looking through your code there are a number of places you are querying the database directly rather than using functions that exist to do this

eg getting an activity by its id. Is there a reason for this? I ask as generally it’s better to use functions are you can benefit from caching where appropriate