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

Bug #1269884 reported by Bill Erickson on 2014-01-16
10
This bug affects 2 people
Affects Status Importance Assigned to Milestone
Evergreen
Undecided
Unassigned
2.5
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...

Bill Erickson (berick) wrote :
tags: added: pullrequest
Changed in evergreen:
assignee: Bill Erickson (erickson-esilibrary) → nobody
milestone: none → 2.6.0-alpha1
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  Edit
Everyone can see this information.

Other bug subscribers