Good work on this module. It's surprising that this behaviour is not in drupal core.

If I may make a couple of suggestions;

drupal_set_message('Access denied. You must login to view this page.', 'error');
Wrap 'Access denied. You must login to view this page.' in the t function

$return = t('You are not authorized to access this page.');
Change it to
$return = '<p>'. t('You are not authorized to access this page.') .'</p>';
It just makes the page a little more semantic.

Let me know if you want me to provide a patch.

Thanks again.

Comments

jbjaaz’s picture

Actually, it might be cool to turn the r4032login function into a theme function so sites can customize the messages if they don't like the default wording.

deekayen’s picture

Status: Active » Fixed

p and theming fixed in CVS revision 1.2

Anonymous’s picture

Status: Fixed » Closed (fixed)

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