Selfcheck checkout dies on invalid function name: record.id()

Bug #1269884 reported by Bill Erickson
10
This bug affects 2 people
Affects Status Importance Assigned to Milestone
Evergreen
Fix Released
Undecided
Unassigned
2.5
Fix Released
Undecided
Unassigned

Bug Description

Evergreen 2.5+

From commit b69ffb6a8377282d9e1978bb732c439c28cdec1a

- if(record.isbn()) {
- this.byName(row, 'jacket').setAttribute('src', '/opac/extras/ac/jacket/small/' + record.isbn());
+ if(record.id()) {
+ this.byName(row, 'jacket').setAttribute('src', '/opac/extras/ac/jacket/small/r/' + record.doc_id());
     }

The "record" in question is an MVR, not a BRE, so the value we need is .doc_id(). Pushing fix...

Revision history for this message
Bill Erickson (berick) wrote :
tags: added: pullrequest
Changed in evergreen:
assignee: Bill Erickson (erickson-esilibrary) → nobody
milestone: none → 2.6.0-alpha1
Revision history for this message
Ben Shum (bshum) wrote :

Whoops, confirmed problem and tested out this good fix. Thanks Bill!

Picked to master and rel_2_5.

Changed in evergreen:
status: New → 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.