Web Client unable to edit users with ident_type2 field set

Bug #1724725 reported by Jason Boyer
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Evergreen
Fix Released
Medium
Unassigned
2.12
Fix Released
Medium
Unassigned

Bug Description

Eg 3.0

The ident_type2 field isn't handled in the webstaff user editor so while you can set an ident_value2 you can't say what it is and because of the way this is handled an account with a non-null ident_type2 field can't be edited. When you click Save it just sits and nothing happens (unless you're watching the error console).

Because this field isn't handled in the same way as ident_type1 the cstore backend that tries to save it dies quietly and there's no indication of what happened except a line about "drone_blah_blah IS NOT CONNECTED TO THE NETWORK!" shortly thereafter. This is because the ident_type1 field is passed as a number but the ident_type2 is sent as an object. So the au object passed to the cstore drone looks like this: ...,3,{"id":3,"name":"Other"},"value1","value2"... and it just dies.

Patch incoming to handle ident_type2 the same way as ident_type1, though it won't make cstore any more resilient against this sort of type mismatch.

Tags: pullrequest
Revision history for this message
Jason Boyer (jboyer) wrote :
tags: added: pullrequest
Galen Charlton (gmc)
Changed in evergreen:
status: New → Confirmed
importance: Undecided → Medium
assignee: nobody → Galen Charlton (gmc)
milestone: none → 3.0.2
Revision history for this message
Galen Charlton (gmc) wrote :

Pushed to master, rel_3_0, and rel_2_12. Thanks, Cesar!

Changed in evergreen:
assignee: Galen Charlton (gmc) → nobody
status: Confirmed → Fix Committed
Changed in evergreen:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.