The Tracker class is infamous for its many methods that sneakily create their own DAOs and run DB queries. It's one of the reasons why it is often mocked. Among those methods are two: Tracker::getParentUserCanView() and Tracker::getParent(). Both sound reasonable enough, but in reality the Tracker object by itself does not know about tracker hierarchy (parents), does not know about permissions, and does not know how to create other Trackers (it's the job of TrackerFactory). Therefore, it has to create its own DAOs.

It's even worse than that: we "just" want to know the parent tracker, but to do that, we have to get the whole hierarchy from DB, and then only get the parent tracker ID.

We can do better, and faster. We should mark those methods as deprecated and introduce a dedicated class that will have all it needs passed by its constructor, making it clear what needs to be done to retrieve a parent tracker.