Experimental catalog: baskets persist after logging out and logging in as a different user

Bug #1867834 reported by Jane Sandberg
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Evergreen
Fix Released
Medium
Unassigned
3.5
Fix Released
Medium
Unassigned

Bug Description

Steps to recreate:
1) Log into the Evergreen Staff Client using one set of credentials.
2) Go to the experimental catalog
3) Do a search, and add some items to your basket (by clicking the checkboxes next to the relevant entries in your search results).
4) Note the number of records in your basket.
5) Log out, and log in as a different user with the same workstation.
6) Go to the experimental catalog
7) Note that you still have those records in your basket

I'm pretty sure that this is not the behavior we want. But maybe it could be helpful for some workflows that I can't foresee? Adding the needsdiscussion tag just in case.

Revision history for this message
Bill Erickson (berick) wrote :

Confirmed and agreed the list should clear on logout. (That was the intention of the code, but it missed a piece). If staff need a list to persist, it can be saved to a bucket.

Changed in evergreen:
status: New → Confirmed
assignee: nobody → Bill Erickson (berick)
milestone: none → 3.4.3
Revision history for this message
Bill Erickson (berick) wrote :
tags: added: pullrequest
Changed in evergreen:
assignee: Bill Erickson (berick) → nobody
Changed in evergreen:
milestone: 3.4.3 → 3.4.4
tags: removed: needsdiscussion
Revision history for this message
Jane Sandberg (sandbergja) wrote :

Works well for me. Thanks, Bill. Signoff at user/sandbergja/lp1867834-staffcat-clear-basket-signoff

tags: added: signedoff
Revision history for this message
Galen Charlton (gmc) wrote :

I don't think the signoff branch made it to the working repository, so I went ahead and manually added Jane's signoff to the commit. Pushed to master, rel_3_5, and rel_3_4. Thanks, Bill and Jane!

Changed in evergreen:
importance: Undecided → Medium
Revision history for this message
Galen Charlton (gmc) wrote :

Ah, it had; there was just a typo in the branch name.

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.