Closed (fixed)
Project:
Redirect 403 to User Login
Version:
5.x-1.x-dev
Component:
Code
Priority:
Normal
Category:
Support request
Assigned:
Unassigned
Reporter:
Created:
1 Feb 2008 at 15:29 UTC
Updated:
17 Oct 2008 at 20:23 UTC
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
Comment #1
jbjaaz commentedActually, 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.
Comment #2
deekayen commentedp and theming fixed in CVS revision 1.2
Comment #3
Anonymous (not verified) commentedAutomatically closed -- issue fixed for two weeks with no activity.