Hold Matrix weights admin interface can't save

Bug #985009 reported by Bob Wicksall
14
This bug affects 2 people
Affects Status Importance Assigned to Milestone
Evergreen
Status tracked in Master
2.1
Fix Released
Medium
Unassigned
2.2
Fix Released
Medium
Unassigned
Master
Fix Released
Medium
Unassigned

Bug Description

The Hold Matrix Weights admin screen can't save new weight sets. It appears to be a mismatch between the editor and the actual columns in the table.

Evergreen 2.1.1and 2.2 Beta 1

open-ils.pcrud 2012-04-18 10:24:59 [ERR :1762:oils_sql.c:2265:13344855041826080] open-ils.pcrud ERROR inserting config::hold_matrix_weights object using query [INSERT INTO config.hold_matrix_weights (id,name,user_home_ou,request_ou,pickup_ou,item_owning_ou,item_circ_ou,usr_grp,requestor_grp,circ_modifier,marc_type,marc_form,marc_vr_format,ref_flag,item_age) VALUES (DEFAULT,'Bob_Test','5','1','3','4','3','6','4','1','5','4','4','2','2');]: 0 ERROR: null value in column "marc_bib_level" violates not-null constraint

Tags: pullrequest
Changed in evergreen:
assignee: nobody → Thomas Berezansky (tsbere)
Revision history for this message
Thomas Berezansky (tsbere) wrote :

The branch below adds the missing fields to the IDL. It cherry-picks cleanly back to 2.1, and should be applied there.

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

tags: added: pullrequest
Revision history for this message
Lebbeous Fogle-Weekley (lebbeous) wrote :

Merged to master, rel_2_2 and rel_2_1.

Took a moment to realize how this interface works. It's a bit unusual in that virtually every field must be filled in on new rows, and for other reasons. Opened a wishlist bug #998080 about that.

Thanks Bob and Thomas.

To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Duplicates of this bug

Other bug subscribers