https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18064

--- Comment #24 from Thibaud Guillot <[email protected]> ---
(In reply to Kyle M Hall from comment #21)
> Comment on attachment 136213 [details] [review]
> Bug 18064 : Add syspref to launch longOverdue script with calendar support
> 
> Review of attachment 136213 [details] [review]:
> -----------------------------------------------------------------
> 
> ::: misc/cronjobs/longoverdue.pl
> @@ +492,4 @@
> >      }
> >  }
> >  
> > +if (!( C4::Context->preference( 'LongOverdueNoticeCalendar' ) )) {
> 
> Why is this here? I don't understand the utility of suppressing the summary
> if using the calendar.

Yes you're right. I worked on this script again recently and realized that I
had misunderstood this part. We need this output to get the summary, it's my
mistake.

-- 
You are receiving this mail because:
You are watching all bug changes.
_______________________________________________
Koha-bugs mailing list
[email protected]
https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/

Reply via email to