bib source should be included in the 901 field

Bug #1307553 reported by Galen Charlton
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Evergreen
Fix Released
Wishlist
Unassigned

Bug Description

It might be handy of the maintain_901() trigger included the bib source code in a subfield in the 901. This would allow:

- more directly expressing bib source as a match point for import
- the option of easily indexing the bib source as a searchable attribute
- providing a hook for additional record-handing logic (e.g., for managing electronic resource records)

Note that a related issue is bug 1178377, which proposes to add the bib source to unapi.bre output for use in TPAC styling. This is orthogonal to this wishlist.

Original idea by Mike Rylander

Evergreen master

Tags: pullrequest
Galen Charlton (gmc)
Changed in evergreen:
importance: Undecided → Wishlist
Revision history for this message
Doug Kyle (dkyle) wrote :

I'm wondering what subfield would be best, and for all schemas or just biblio?

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

Only bibs have source at the moment. Subfield $s seems fine for the purpose.

<dkyle> Drycona: guess I should ask in lp. was wondering what orgs might be using what 901 subfields already for their purposes.

Unless they've a local customization in place (which we can't officially care about), nobody is using subfields that aren't managed by the maintain_901() trigger function, as it completely replaces any existing 901 fields whenever a record gets updated.

For the record, here's the current 901 subfield list:

biblio
------
$a = TCN value
$b = TCN source
$c = bre.id
$t = 'biblio'
$o = owner
$d = share depth

authority
-----------
$c = are.id
$t = 'authority'

serial
------
$c = sre.id
$o = owning library
$r = serial.record_entry.record, i.e., the ID of the bib that the MFHD is linked to
$t = 'serial'

Revision history for this message
Doug Kyle (dkyle) wrote :

Here is a patch

Doug Kyle (dkyle)
tags: added: pullrequest
Revision history for this message
Ben Shum (bshum) wrote :

There's code here, adding initial review target.

Changed in evergreen:
milestone: none → 2.next
status: New → Triaged
Kathy Lussier (klussier)
tags: added: needsreleasenote
Revision history for this message
Galen Charlton (gmc) wrote :

I've pushed a patch to the tip of the user/gmcharlt/lp1307553_bib_source_in_901 branch in the working/Evergreen repository:

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

Unlike Doug's patch, the source name is copied to 901$s rather than the numeric ID. This is consistent with how unAPI returns the bib source. Also, release notes are included.

Changed in evergreen:
status: Triaged → Confirmed
tags: removed: needsreleasenote
Revision history for this message
Mike Rylander (mrylander) wrote :

Pulled into master. Thanks, Galen!

Changed in evergreen:
status: Confirmed → Fix Committed
milestone: 2.next → 2.11-beta
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.