Log Noise - retrieve config.hard_due_date called with no ID...

Bug #1667115 reported by Josh Stompro
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Evergreen
Fix Released
Medium
Unassigned
2.11
Fix Released
Medium
Unassigned

Bug Description

EG 2.10.6

For every check out on our system, we get the following notice in the logs.

2017-02-22 15:13:14 virt-egapp1 open-ils.circ: [WARN:15197:CStoreEditor.pm:139:1487797961154853] editor[1|112246] retrieve config.hard_due_date called with no ID...

I'll try and track down where an extra check needs to go to avoid that message.

Josh

Tags: pullrequest
Revision history for this message
Josh Stompro (u-launchpad-stompro-org) wrote :

It might be at:
http://git.evergreen-ils.org/?p=Evergreen.git;a=blob;f=Open-ILS/src/perlmods/lib/OpenILS/Application/Circ/Circulate.pm;h=7dd361175588eeb44c2145c0d8093b4211fddbbb;hb=HEAD#l1057

Just needs to be wrapped with an if defined?

if(defined($results->[0]->{hard_due_date})) { $self->circ_matrix_matchpoint->hard_due_date($self->editor->retrieve_config_hard_due_date($results->[0]->{hard_due_date}));
}

I'll try that out.
Josh

Revision history for this message
Josh Stompro (u-launchpad-stompro-org) wrote :

That seems to take care of it. The warning message is gone, and setting a hard due date still works.

Working branch at: user/stompro/lp1667115_log_noise_hard_due_date
http://git.evergreen-ils.org/?p=working/Evergreen.git;a=shortlog;h=refs/heads/user/stompro/lp1667115_log_noise_hard_due_date

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

Josh, you are my hero. This has been super tedious in our logs forever, and now it will be gone!

Picked to master and backported as a bug fix for rel_2_12 and rel_2_11.

Changed in evergreen:
milestone: none → 2.12.1
status: New → Fix Committed
importance: Undecided → Medium
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.