CMS MADE SIMPLE FORGE

Frontend Users

 

[#10664] {$url_changesettings} in logout form redirect to logout form

avatar
Created By: jean-christophe ghio (jissey)
Date Submitted: Tue Sep 29 10:05:26 -0400 2015

Assigned To: Robert Campbell (calguy1000)
Version: 1.30.5
CMSMS Version: 1.12
Severity: Minor
Resolution: Fixed
State: Open
Summary:
{$url_changesettings} in logout form redirect to logout form
Detailed Description:
Hi Robert,
when I click on changesettings url in the logout form, the page come back to the
logout form.
In FrontEndUsers.module.php, around the line 560, the action parameter is
initialized to 'logout' :
        if( $action == 'default' ) {
            $form = \cge_param::get_string($params,'form','login');
            if( $uid = $this->LoggedInId() ) $form = 'logout';
            switch( $form ) {
            case 'login':
            case 'forgotpw':
            case 'silent':
            case 'changesettings':
            case 'lostusername':
                $action = $form;
                break;

            case 'logout':
                $action = 'logoutform';
                break;

It seems that it comes from this line :
 if( $uid = $this->LoggedInId() ) $form = 'logout';

regards


History

Comments
avatar
Date: 2016-07-30 12:49
Posted By: Robert Campbell (calguy1000)

Should be fixed now.
      
avatar
Date: 2016-10-07 02:58
Posted By: Raymond Juillerat (raymond)

Hi Robert,

This last corrections had no effect on the behaviour for a front-end user
asking the  "changesetings" page from the logout page in the cmsms site
<cnf-fribourg.ch>.
I found a good idea to correct this in the French forum of cmsms.

For version 2.1.1 of FrontEndUsers the page action.logoutform.php, I changed the
line 85 in that manner:
$url = $this->create_url($id,'changesettings',$cspage,$parms);
instead of
$url = $this->create_url($id,'default',$cspage,$parms);

Page in the French Forum
<http://www.cmsmadesimple.fr/forum/viewtopic.php?id=6109>

regards 

  
      
Updates

Updated: 2016-07-30 12:49
resolution_id: => 7