reporter: action.all_circulations_combined_types should break out non-cat in-house-use

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

Bug Description

Found at a PINES library that records some non-cataloged item circulations as in-house use. We have been using the action.all_circulation_combined_types source for annual reports for the last few years, but it became clear this year that the source combines non-cataloged circulations and non-cataloged in-house use circulations as non-cataloged circs. Our annual reports require our libraries to exclude in-house use when reporting total circulation.

Branch on the way.

Evergreen 3.0.2+
OpenSRF 3.0.1
PostgreSQL 9.5
Ubuntu 16.04

Revision history for this message
Chris Sharp (chrissharp123) wrote :
tags: added: pullrequest
Changed in evergreen:
milestone: none → 3.2.2
milestone: 3.2.2 → 3.next
no longer affects: evergreen/3.1
no longer affects: evergreen/3.2
Dan Wells (dbw2)
Changed in evergreen:
milestone: 3.next → 3.3-beta1
Dan Wells (dbw2)
Changed in evergreen:
importance: Undecided → Wishlist
Changed in evergreen:
milestone: 3.3-beta1 → 3.3-rc
Changed in evergreen:
milestone: 3.3-rc → 3.next
Changed in evergreen:
assignee: nobody → Rogan Hamby (rogan-hamby)
Changed in evergreen:
assignee: Rogan Hamby (rogan-hamby) → nobody
Revision history for this message
Jane Sandberg (sandbergja) wrote :

Chris, since this is a wishlist bug, could you please add some brief release notes?

tags: added: needsreleasenote
Revision history for this message
Blake GH (bmagic) wrote :

Not sure why this is wishlist. It's a bug with existing code. Maybe it was flagged wishlist before the original code was committed to master?

Galen Charlton (gmc)
Changed in evergreen:
importance: Wishlist → Medium
status: New → Confirmed
Revision history for this message
Rogan Hamby (rogan-hamby) wrote :

signoff at user/rogan/lp1788260_non_cat_in_house_use_aacct

tags: added: signedoff
Revision history for this message
Rogan Hamby (rogan-hamby) wrote :

I'm going to take off the needs-release notes tag. I know this was tagged as wishlist but I suspect that was a misclick. This was a bug since it was the mislabeling and not addition of a new feature.

tags: removed: needsreleasenote
Galen Charlton (gmc)
Changed in evergreen:
milestone: 3.next → 3.5.2
assignee: nobody → Galen Charlton (gmc)
Revision history for this message
Galen Charlton (gmc) wrote :

Agreed, this is a bugfix, although it probably is worth a special announcement in the 3.4.x and 3.5.x release notes. Pushed to master, rel_3_5, and rel_3_4. Thanks, Chris and Rogan!

Changed in evergreen:
assignee: Galen Charlton (gmc) → nobody
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.