Web Staff Client - Group Members Don't Display

[Terran McCanna]

In the 2.11 web client:

When I'm viewing a patron account that is in a group and click on Other > Group Member Details, the members do not display.

(I confirmed that they do display in the 2.11 web client, so they are grouping properly both when Cloned and when added to the group manually.)

[Andrea Neiman (aneiman)]

2.12 web client

Noting that Group Members will display correctly after a manual refresh or an action (like adding a group member) that causes a refresh.

[Terran McCanna (tmccanna)]

Manual refresh does work. Adding a group member to trigger a refresh doesn't work because the action menu is greyed out when there is no list.

Related issue: Actions button is greyed out unless at least one person on the group list is checked. That is non-intuitive for the "Move another patron to this group" option.

[Kathy Lussier (klussier)]

For the last related issue, take a look at bug 1670457.

[Blake GH (bmagic)]

This bug seems like it could be pretty easy to fix. Especially with the added clue that the grid DOES show the member details when the web page is refreshed from the group member details screen. Looking at the browser development tools console I see that it's complaining about:

grid.setQuery is not a function

This code is executed in ui/default/staff/circ/patron/app.js in the section about $scope.initTab('other'

here it's calling grid.setQuery which apparently is not instantiated if the page was loaded on a different section of the patron account before toggled to "Group Member Details". Thoughts on this?

[Blake GH (bmagic)]

This seems to be solving the issue for me
http://git.evergreen-ils.org/?p=working/Evergreen.git;a=shortlog;h=refs/heads/user/blake/lp1642036_Web_Staff_Client_Group_Members_Dont_Display

[Galen Charlton (gmc)]

I tested the patch, and I was still sometimes able to make the race condition between the completion of initialization of the group tab's egGrid (by which point grid.setQuery is guaranteed to exist) and the post-initTab handler occur. What did work was wrapping that call to grid.setQuery() in a $timeout.

So, we're close, but not for 3.0.0. Setting milestone to 3.0.1.

[Kyle Huckins (khuckins)]

Hey Galen, I was wondering what steps you were able to take to make that race condition occur? I'm looking at tacking this unless Blake's got any objections, but having trouble replicating it with Blake's patch.

[Blake GH (bmagic)]

Kyle,

The $timeout approach is going to be the full-proof approach. I didn't think of that when I was working on this one. My patch just delays the execution arbitrarily via "more code".

[Jennifer Pringle (jpringle-u)]

Confirming that this is still an issue in 3.1

[Cesar V (cesardv)]

3.1 is still suffering from this apparently...

Here's a branch that includes the above mentioned $timeout fix:

http://git.evergreen-ils.org/?p=working/Evergreen.git;a=shortlog;h=refs/heads/user/cesardv/lp1642036-Patron_Group_Grid_bug

[Dawn Dale (ddale)]

I am testing this bug patch.

[Dawn Dale (ddale)]

I tested this and I still have to refresh the screen to see group member details.

[Terran McCanna (tmccanna)]

Actually, we may have been seeing a problem with our test server when we tested this, we will try again.

[Dawn Dale (ddale)]

I tested this one and the screen still has to be refreshed to see the group details. Also "Actions" is still grayed out unless at least one person is checked.

[Scott Thomas (scott-thomas-9)]

This is a major nuisance for us since upgrading to 3.1.

Scott

[Meg Stroup (mstroup)]

Still occurring in 3.1.5.

[Courtney Brown (courtney.brown)]

Still occurring 3.1.6

[Terran McCanna (tmccanna)]

This has been an even bigger problem since our move to 3.2.2.

[Chris Sharp (chrissharp123)]

Current status of our testing this: We are not seeing any change after applying Cesar's branch alone. We are seeing Blake's branch work consistently. Was Cesar's branch meant to fix the issue Galen mentions in Comment #6? We have not tested them together.

[Meg Stroup (mstroup)]

Remains an issue of great concern/significant workflow hinderance to SCLENDS circulation workgroup, per last workgroup meeting. We're 3.1.8/Chrome.

[Terran McCanna (tmccanna)]

Update at PINES: We have Blake's and Cesar's fixes on our production server, and it helps some people, but a lot of people are still needing to refresh their browser window every time.

[Bill Erickson (berick)]

Here's another patch:

https://git.evergreen-ils.org/?p=working/Evergreen.git;a=shortlog;h=refs/heads/user/berick/lp1642036-grp-member-details

This one addresses how we apply the grid.setQuery function specifically to resolve the "grid.setQuery is not a function" error. I am now able to load details, but I have not confirmed (w/ limited test data) that it always loads all members every time.

[Blake GH (bmagic)]

Thanks Bill!

Signed off

https://git.evergreen-ils.org/?p=working/Evergreen.git;a=shortlog;h=refs/heads/user/blake/LP1642036_Repair_group_member_details_display_error-signoff

[Terran McCanna (tmccanna)]

Wishing I could add a gif to this so I could show Kermit the Frog waving his hands in the air with delight. Thanks Bill and Blake!

[John Amundson (jamundson)]

Jason Stephenson loaded this patch on a system to test, and the group members are indeed loading initially now. However, if I reload the page, the group members fail to display. Does anyone else see this, too?

I'm also seeing inconsistent behavior with certain actions. For example, I selected the action to add my user to another group. The list reloaded but was blank. I had to click another tab in the patron's account and come back to the group member details to see the member appear.

[Jason Stephenson (jstephenson)]

I have added my signoff in a branch here: user/dyrcona/LP1642036_Repair_group_member_details_display_error-signoff

https://git.evergreen-ils.org/?p=working/Evergreen.git;a=shortlog;h=refs/heads/user/dyrcona/LP1642036_Repair_group_member_details_display_error-signoff

I'm waiting on some feedback from staff at CW MARS, but will push the fix this evening because it is working for me in my testing, too.

Thanks, Bill and Blake!

[Bill Erickson (berick)]

Here's another one.

https://git.evergreen-ils.org/?p=working/Evergreen.git;a=shortlog;h=refs/heads/user/berick/lp1642036-grp-member-details-v2

It improves and ultimately replaces my previous commit. Now members display during reloads and tab navigation.

[John Amundson (jamundson)]

Thanks, Bill. This new patch is working for me.

I have tested this code and consent to signing off on it with my name, John Amundson and my email address, <email address hidden>.

[Jason Stephenson (jstephenson)]

Works for me, too!

Pushed to master, rel_3_1, rel_3_2, and rel_3_3.

You still have to reload after changing group membership, but that's a different bug in my opinion.

Thanks to everyone who commented, shared code, and tested!