Circulation->Retrieve Recent Patrons can have duplicate entries

Bug #2009281 reported by Dan Briem
12
This bug affects 2 people
Affects Status Importance Assigned to Milestone
Evergreen
Fix Committed
Low
Unassigned
3.10
Fix Committed
Low
Unassigned

Bug Description

Observed on 3.8 and master

The Retrieve Recent Patrons can have duplicate entries depending on how the accounts were opened. This only affects the AngularJS interface.

To test:

1. Set the "Number of Retrievable Recent Patrons" setting to 2.
2. Open a patron record from Patron Search.
3. Go to Circulation->Retrieve Last Patron
4. Go to Circulation->Retrieve Recent Patrons and notice the user is duplicated.

Dan Briem (dbriem)
tags: added: circulaton patron
Revision history for this message
Dan Briem (dbriem) wrote :
tags: added: pullrequest
tags: added: circulation
removed: circulaton
Galen Charlton (gmc)
Changed in evergreen:
status: New → Confirmed
importance: Undecided → Low
Revision history for this message
Susan Morrison (smorrison425) wrote :

Testing for bug squashing. With the "Number of Retrievable Recent Patrons" set to 2, I accessed two accounts and then clicked "Retrieve Last Patron." Where the previous behavior was that the patron would show up twice, now the entry is not duplicated, but the results only show the one patron, so it appears to still be counting both instances of opening the same account, instead of what I think is the desired behavior of showing both accounts that were accessed.

Revision history for this message
Joan Kranich (jkranich) wrote :

Testing on https://terran-master.gapines.org/

1. Set the "Number of Retrievable Recent Patrons" setting to 2.
2. Open a patron record from Patron Search
3. Open a second patron record from Patron Search
4. Go to Circulation->Retrieve Last Patron
5. Go to Circulation->Retrieve Recent Patrons and only the last patron displays.

Next test I searched and retrieved 4 patron records.
For this test I returned to the home screen.
I did Retrieve Last Patron which correctly retrieved my last patron.
I did Retrieve Recent Patrons and only the last patron displayed.

Revision history for this message
Terran McCanna (tmccanna) wrote :

Removing pullrequest due to testing comments 2 & 3. (Note that for testing this was installed along with https://bugs.launchpad.net/evergreen/+bug/1901072 which has since been committed.)

tags: added: needswork
removed: pullrequest
Revision history for this message
Dan Briem (dbriem) wrote :

Susan and Joan seem to have found another problem that applies to both the AngularJS and Angular interfaces.

The issue: user is added to the front of the list (ex. [A, A, B]), then the list is trimmed by 1 relative to the max setting ([A, A]), then 1 duplicate is removed ([A]).

This is related enough that I don't think it needs another bug. Branch for testing: https://git.evergreen-ils.org/?p=working/Evergreen.git;a=shortlog;h=refs/heads/user/dbriem/lp2009281_recent_patrons_duplicates

tags: added: pullrequest
removed: needswork
Revision history for this message
Susan Morrison (smorrison425) wrote :

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

tags: added: signedoff
Revision history for this message
Jane Sandberg (sandbergja) wrote :

Thanks, Dan and Susan! Pushed to rel_3_10 and above.

Changed in evergreen:
milestone: none → 3.11.2
status: Confirmed → Fix Committed
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.