billing types in opensrf.xml are deprecated and should be removed from the opensrf.xml.example file

Bug #1160343 reported by Pasi Kallinen
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Evergreen
Fix Released
Medium
Unassigned

Bug Description

*EG master

See below for Bill Erickson's comment.

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

Hi Pasi, this bug should be changed (or a new one opened) to "billing types in opensrf.xml are deprecated and should be removed from the opensrf.xml.example file"

They have been replaced by the data in config.billing_type, which uses the standard in-db translation mechanism.

Pasi Kallinen (paxed)
summary: - billing_types in opensrf.xml should be picked up for translation
+ billing types in opensrf.xml are deprecated and should be removed from
+ the opensrf.xml.example file
description: updated
Revision history for this message
Galen Charlton (gmc) wrote :

The last remaining reference to the opensrf.xml billing_types list is the method open-ils.circ.billing_type.retrieve.all. There's no code in Evergreen that invokes that API call, so unless somebody knows of external clients of that method, it should be removed as well.

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

A patch that removes billing_types from opensrf.xml.example (and the unused circ_modifier list) is available at the tip of the user/gmcharlt/lp1160343_remove_old_btype_list branch in the working/Evergreen repository:

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

tags: added: pullrequest
Ben Shum (bshum)
Changed in evergreen:
milestone: none → 2.4.0-rc
status: New → Confirmed
importance: Undecided → Medium
Revision history for this message
Bill Erickson (berick) wrote :

Tested and works. Thanks for the patch, Galen. Merged.

Changed in evergreen:
status: Confirmed → Fix Committed
Ben Shum (bshum)
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.