Email Notification Links

[aewatkins]

Links to a Sugar record within email notifications (new contact, lead, account notifications, etc.) will take me to the login screen of our Sugar instance; however, after logging in, I get a "Page cannot be displayed". I have checked the "site_url" in the config.php and it is correct. If I am already logged into Sugar, I will be taken directly to the record. So the link is correct. I found this thread about by-passing the login using SOAP - but I do want a login screen if the user is not already logged into Sugar.

Any ideas on what the issue could be? Thank you!

Sugar Version: Version 5.0.0f (Build 3338)
Sugar Edition: SugarPro
Operating System (server): Windows 2000 Server
PHP Version: PHP 5.2.6
Database: MySQL 5
Web Server: IIS 5

[eNick]

Could you post some examples of the url embedded in the email links?

It might help show up any obvious errors.

[aewatkins]

Sure thing....

<http://www.mydomain.com/mysugarcrm/index.php?module=Tasks&action=DetailView&record=a8 420282-0c58-f42e-c7c2-48888356c7cc>

If I am logged into Sugar, this link goes right to the task. Otherwise, I get the "page cannot be displayed" after I am prompted to log in.

[eNick]

Well what d'ya know - same thing happens here!

I'll see if I can find out why.

It seems it forgets to pass through the module because if I ask for...

index.php?module=Accounts&action=DetailView&record =7bd3d92c-99e1-db92-6d93-48621912a310

the line recorded in my log file is...

GET /sugar/index.php&action=DetailView&record=7bd3d92c-99e1-db92-6d93-48621912a310 HTTP/1.1

This definitely looks like a bug in Sugar

[eNick]

Ok, I have your answer...


In modules/Users/Authenticate.php, line 48 onwards change...

if(isset($_SESSION['authenticated_user_id'])) {
	global $record;
	$GLOBALS['module'] = !empty($_REQUEST['login_module']) ? '?module='.$_REQUEST['login_module'] : '?module=Home';
	$GLOBALS['action'] = !empty($_REQUEST['login_action']) ? '&action='.$_REQUEST['login_action'] : '&action=index';
	$GLOBALS['record']= !empty($_REQUEST['login_record']) ? '&record='.$_REQUEST['login_record'] : '';

to...

if(isset($_SESSION['authenticated_user_id'])) {
	global $record;
	global $module;
	$GLOBALS['module'] = !empty($_REQUEST['login_module']) ? '?module='.$_REQUEST['login_module'] : '?module=Home';
	$GLOBALS['action'] = !empty($_REQUEST['login_action']) ? '&action='.$_REQUEST['login_action'] : '&action=index';
	$GLOBALS['record']= !empty($_REQUEST['login_record']) ? '&record='.$_REQUEST['login_record'] : '';

ie. insert the declaration "global $module;" after "global $record;".

Basically, when you're not yet logged in $module hasn't yet been declared as a global variable, so when it is assigned in the statement $GLOBALS['module'] = !empty($_REQUEST['login_module']) ? '?module='.$_REQUEST['login_module'] : '?module=Home'; nothing will happen.

This will depend, to some extent, on which version of PHP you're running as older versions allowed globals to be defined without explicit declarations.

You should raise this as a bug with Sugar but the above change will fix your problem.

[aewatkins]

Thank you! That fixed it! I will report it to Sugar.

[dcurrie]

Thanks Nick! This worked great. Its so nice when people take the time to post solutions. I really appreciate it.

don

[eNick]

You're welcome! Glad its helping people.