When anonymous users do not have permission to signup, login and register links are suppose to show up. I am unsure at what stage this was broken but I tried 2.1, 2.2, and 2.x-dev. All of which have the same problem.

I tracked down the problem to a break statement that broke out before the html output was assigned back to the theme variables. This seems to have also caused the issue of not being able to show signups to anon users if given the permission.

I have attached a patch to fix this. As their many ways to fix it I am not sure if my is up to par. But I chose to let the code run its course by removing the break and adding another condition on the next if/else to be an if/elseif so that anon users would not be able to cancel.

CommentFileSizeAuthor
signup-anon-login-link-fix.patch1.36 KBdragonwize

Comments

dww’s picture

Status: Needs review » Fixed

Cool, thanks. I reviewed, tested, and then committed a slightly modified version of your patch to HEAD, DRUPAL-5 and DRUPAL-4-7 (the bug was present in all versions).

Anonymous’s picture

Status: Fixed » Closed (fixed)