web client: user perm editor grantable flag display issue

Bug #1655399 reported by Jason Etheridge
10
This bug affects 2 people
Affects Status Importance Assigned to Milestone
Evergreen
Fix Released
Low
Unassigned

Bug Description

This is only for the HTML user perm editor that was resurrected for the web client, and not the XUL-based user perm editor.

On a concerto-based test system, retrieve the admin (or any user) in the web client, and select Other -> User Permission Editor, or alternately, pull up the interface via Administration -> User Permission Editor.

Give the user the ABORT_TRANSIT permission with a depth of System and leave Grantable unchecked.

Save the changes. When the interface refreshes itself, it will incorrectly show the ABORT_TRANSIT permission as being Grantable. In the database, the user perm map will have Grantable = false for that perm.

tags: added: webstaffclient
Bill Erickson (berick)
Changed in evergreen:
status: New → Confirmed
Revision history for this message
Bill Erickson (berick) wrote :

Fix pushed:

http://git.evergreen-ils.org/?p=working/Evergreen.git;a=shortlog;h=refs/heads/user/berick/lp1655399-user-perms-bool-fix

This was a case of Javascript "0"=true, where 0=false was expected.

tags: added: pullrequest
Changed in evergreen:
milestone: none → 2.next
Galen Charlton (gmc)
Changed in evergreen:
assignee: nobody → Galen Charlton (gmc)
Revision history for this message
Galen Charlton (gmc) wrote :

Tested and pushed to master. Thanks, Bill!

Changed in evergreen:
importance: Undecided → Low
assignee: Galen Charlton (gmc) → nobody
status: Confirmed → Fix Committed
Changed in evergreen:
milestone: 2.next → 2.12-beta
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.