[#666] user.unix_status modified from admin/useredit.php regardless if they are added to a project group or not



Detailed description

The user.unixstatus field cannot be modified directly in admin/useredit.php. Per the note displayed on that page: "Unix status updated mirroring web status, unless it has value 'No unix account (N)'". By default, new accounts have unixstatus set to 'N'. I noticed that if you click on the "Update" button at the bottom of the page (below the project listing), the status is always changed to 'A', regardless if you selected any projects for them to be added to or not. Clicking on the "Update" button in the middle of the page (just below Account Information), the status is not changed.

This has been handy while debugging another issue, but it seems like it may not be desired behavior. It appears that both Update buttons submit to the same form target, so the split behavior seems to be an error.

On a related note, I see why a user probably doesn't need shell access if they are not a member of a project, but it might be nice to be able to manually toggle this field value from the web UI.

General Information
Submitted by:
Seth Galitzer
Date Submitted: 2014-04-29 21:23
Last Modified by: Nobody
Last Modified: 2017-11-02 20:00
Date Closed: 2014-05-15 14:02
Permalink: https://fusionforge.org/tracker/a_follow.php/666
Internal Fields
Data Type: Bugs
Assigned to: Franck Villaume (nerville)
State: Closed
Priority: 3
Extra Fields
Target Release:
Follow-up tabs
Message  ↓
Date: 2014-05-15 14:02
Sender: Franck Villaume

fix in Branch53: c7c0d12f0f72458ce960297bf4812d3c9b8d5294

Date: 2014-05-15 12:52
Sender: Franck Villaume

after some tests, here are some facts. 1) you create an account, unixstatus is set to N (No unix account, default value for unixstatus column in the users table) 2) you click on the "update" button in the middle of the page, unixstatus is set to D (check setUnixStatus function with status = N) 3) you click again on the "update" button in the middle of the page, unixstatus is set to A. For this case, the easy fix is to check getGroups before calling setUnixStatus.

Still, it is not a complete fix.

No attached documents

No related commits.

Field Old Value Date By
status_idOpen2014-05-15 14:02
Franck Villaume
close_dateNone2014-05-15 14:02
Franck Villaume
assigned_tonone2014-05-15 14:02
Franck Villaume
Target ReleaseNone2014-05-15 14:02
Franck Villaume
ResolutionNone2014-05-15 14:02
Franck Villaume

No relations found.