user_auth_help_links currently loops over module_list() and then uses if(module_hook('auth')) to filter this list.

Is there a specific reason why the code does not loop over module_implements('auth') directly, which would eliminate the if condition and possibly save (minimal) overhead?

Otherwise, patch attached.

CommentFileSizeAuthor
user_module_auth-203727.patch825 bytescburschka

Comments

drumm’s picture

Status: Needs review » Fixed

Committed to 5.x.

This section of code does not exist in 6.x, so it does not need to be forward-ported.

Anonymous’s picture

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for two weeks with no activity.