Angular Perm Group UI Mapping Modal Fails to load on null perm description

Bug #1851831 reported by Chris Sharp
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Evergreen
Fix Released
High
Unassigned

Bug Description

When creating a new perm/group mapping in Administration -> Server Administration -> Permission Group, upon clicking on a Permission Group, then the Group Permissions tab, then Add New Mapping, then clicking within the New Permission box, the perm list fails to load with the following error:

ERROR TypeError: "l.description(...) is null"

PINES had 20 perms without descriptions. Adding descriptions to these solved the problem. Ideally, some defensive coding would be added to accommodate this possibility, but the most straightforward thing to do is to add the perm descriptions, which I will create a branch to do.

Evergreen 3.4.1
OpenSRF 3.1
PostgreSQL 10
Ubuntu 16.04

Changed in evergreen:
importance: Undecided → High
Revision history for this message
Chris Sharp (chrissharp123) wrote :
Changed in evergreen:
assignee: Chris Sharp (chrissharp123) → nobody
tags: added: pullrequest
Changed in evergreen:
milestone: none → 3.4.2
Revision history for this message
Bill Erickson (berick) wrote :

Issue also occurs in the permissions filter input. Branch to add defensive coding underway.

Changed in evergreen:
assignee: nobody → Bill Erickson (berick)
status: New → Confirmed
Revision history for this message
Bill Erickson (berick) wrote :

Fixes pushed:

https://git.evergreen-ils.org/?p=working/Evergreen.git;a=shortlog;h=refs/heads/user/berick/lp1851831-perm-grp-null-desc

This repairs 3 cases where a description was assumed to exist. Two caused JS errors and one resulted in empty title attributes in the permission list.

Changed in evergreen:
assignee: Bill Erickson (berick) → nobody
Revision history for this message
Chris Sharp (chrissharp123) wrote :

Bill's fix is confirmed working. Pushed to master and rel_3_4. Thanks for the quick fix, Bill!

Changed in evergreen:
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.