Summary: | Overdue Rules not processed for patrons with checkouts more than 90 days overdue. | ||
---|---|---|---|
Product: | Koha | Reporter: | Chris Cormack <chris> |
Component: | Notices | Assignee: | Bugs List <koha-bugs> |
Status: | CLOSED WORKSFORME | QA Contact: | |
Severity: | normal | ||
Priority: | P5 - low | CC: | george, mathsabypro |
Version: | Main | ||
Hardware: | All | ||
OS: | All | ||
Change sponsored?: | --- | Patch complexity: | --- |
Documentation contact: | Documentation submission: | ||
Text to go in the release notes: | Version(s) released in: | ||
Circulation function: |
Description
Chris Cormack
2010-05-21 01:09:39 UTC
I am not sure if this is still valid: These appear to be the relevant lines of code: 102 =item B<-max> 103 104 Items older than max days are assumed to be handled somewhere else, 105 probably the F<longoverdues.pl> script. They are therefore ignored by 106 this program. No notices are sent for them, and they are not added to 107 any CSV files. Defaults to 90 to match F<longoverdues.pl>. 499 my $maxdays = ( 500 $overdue_rules->{ "delay" . ( $i + 1 ) } 501 ? $overdue_rules->{ "delay" . ( $i + 1 ) } - 1 502 : ($MAX) 503 ); So I think if you use triggered and have a delay defined... it would not use MAX? (In reply to Katrin Fischer from comment #1) > I am not sure if this is still valid: > > These appear to be the relevant lines of code: > > 102 =item B<-max> > 103 > 104 Items older than max days are assumed to be handled somewhere else, > 105 probably the F<longoverdues.pl> script. They are therefore ignored by > 106 this program. No notices are sent for them, and they are not added to > 107 any CSV files. Defaults to 90 to match F<longoverdues.pl>. > > 499 my $maxdays = ( > 500 $overdue_rules->{ "delay" . ( $i + 1 ) } > 501 ? $overdue_rules->{ "delay" . ( $i + 1 ) } - 1 > 502 : ($MAX) > 503 ); > > So I think if you use triggered and have a delay defined... it would not use > MAX? I think MAX is still used with --list-all, but you can use the -max option to overwrite. Closing, please re-open if you disagree. |