CMS MADE SIMPLE FORGE

Frontend Users

 

[#12190] feu_allow_changeusername does not work on user changing form

avatar
Created By: Virgo Pärna (virgo)
Date Submitted: Fri Nov 15 04:54:29 -0500 2019

Assigned To:
Version: 3.1.5
CMSMS Version: 2.2.12
Severity: Major
Resolution: Fixed
State: Open
Summary:
feu_allow_changeusername does not work on user changing form
Detailed Description:
Even if feu_allow_changeusername  is set to false user can change username when
changing settings.
$onerow->readonly = $this->get_settings()->allow_changeusername;
needs to be changed to 
$onerow->readonly = !$this->get_settings()->allow_changeusername;
in action.changesettings.php


History

Comments
avatar
Date: 2019-11-15 04:55
Posted By: Virgo Pärna (virgo)

Forgot to set version info
      
avatar
Date: 2019-12-13 05:27
Posted By: Mathieu Muths (airelibre) (airelibre)

Confirmed, and fix is OK
      
avatar
Date: 2019-12-13 05:31
Posted By: Virgo Pärna (virgo)

Updated original description. For some reason I had written true instead of
false.
      
avatar
Date: 2019-12-13 12:46
Posted By: Robert Campbell (calguy1000)

fixed in SVN, thanks
      
Updates

Updated: 2019-12-13 12:46
resolution_id: 5 => 7

Updated: 2019-12-13 05:31
description: Even if feu_allow_changeusername is set to true user can change username when changing settings. $onerow->readonly = $this->get_settings()->allow_changeusername; needs to be changed to $onerow->readonly = !$this->get_settings()->allow_changeusername; => Even if feu_allow_changeusername is set to false user can change username when changing settings. $onerow->readonly = $this->get_settings()->allow_changeusername; needs to be changed to $onerow->readonly = !$this->get_settings()->allow_changeusername

Updated: 2019-11-15 04:55
version_id: -1 => 31672
resolution_id: => 5
severity_id: 12 => 2
cmsms_version_id: -1 => 31647