Great add-on
@tenants. Please add the hint "XenForo_ControllerPublic_Register" to the load_class_controller listener.
Thanks
Why do you need the
Event Hint, there will be no significance performance increase, it will just break backwards compatibility
FBHP works pre XF 1.2, hints are post XF 1.2.
In this case, I'm also using one controller to extend multiple classes (so one hint wont do)
I would actually have to create 12/13 separate load_class_controller listeners instead of one (that would probably have a negative impact on performance)
I don't see why you might think a hint is needed in this case, I think this is more like one of the sort of cases where you
should not use a hint
1) I want to keep backward compatibility
2) The listener extends many areas
Not only that, but
3) using the edit hint will always run after a general purpose listener
3 is something we absolutely do not want. As soon as a bot is detected as known bot for that area, we 401 and exit (or redirect) with 0 queries and very low bandwidth. Obviously we want to do this 1st before anything else, and we don't want anything to run after doing it (to keep the 0 query). If we add the hint, we are likely to run all of the other add-on queries first.
I also believe the hints are better used when your listener only covers one area (or very few)
This hint can be used to limit your listener to only being executed when it's needed, increasing performance.
In our case, in all most all public areas, the listener is needed (to prevent spam bots taking up resources to public facing functions)