| 2017-03-24 19:07:53 |
Bill Erickson |
description |
Evergreen 2.12 only.
/openils/bin/hold_targeter_v2.pl contains a thinko in the command line arguments configuration, causing any interval value passed to --retarget-interval to result in a value of "1". The field was erroneously configured as a boolean field, instead of a field that accepts a string value.
This field is used to override the default retarget interval configured in the database. As it stands, if this field is used, all targetable holds will be retargeted regardless of their previous check time.
To confirm, run the targeter script with any value for --retarget-interval, grep the logs for DESC (to get the holds-to-target-query) and note that the query contains:
"-or":[{"prev_check_time":null},{"prev_check_time":{"<=":"<RIGHT NOW>"}}]
Once patched, <RIGHT NOW> will contain <RIGHT NOW minus RETARGET INTERVAL>
Wee patch en route. |
Evergreen 2.12.
/openils/bin/hold_targeter_v2.pl contains a thinko in the command line arguments configuration, causing any interval value passed to --retarget-interval to result in a value of "1". The field was erroneously configured as a boolean field, instead of a field that accepts a string value.
This field is used to override the default retarget interval configured in the database. As it stands, if this field is used, all targetable holds will be retargeted regardless of their previous check time.
To confirm, run the targeter script with any value for --retarget-interval, grep the logs for DESC (to get the holds-to-target-query) and note that the query contains:
"-or":[{"prev_check_time":null},{"prev_check_time":{"<=":"<RIGHT NOW>"}}]
Once patched, <RIGHT NOW> will contain <RIGHT NOW minus RETARGET INTERVAL>
Wee patch en route. |
|
