One DB query for each `registerProvider` call? Why not put it into a static variable? Don't mind, that static stuff is evil, it is even a static method - and once this is refactored to a class instance it could easily be made non-static. 