[Bug 31101] New: Allow other separation options for digesting circulation alerts (checkin, checkout, renewal)
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=31101 Bug ID: 31101 Summary: Allow other separation options for digesting circulation alerts (checkin, checkout, renewal) Change sponsored?: --- Product: Koha Version: master Hardware: All OS: All Status: NEW Severity: enhancement Priority: P5 - low Component: Notices Assignee: koha-bugs@lists.koha-community.org Reporter: m.de.rooy@rijksmuseum.nl QA Contact: testopia@bugs.koha-community.org In Circulation::SendCirculationAlert we find the only use of the C4::Message module and one of the last TRUNCATE calls btw in the same context: my $message = C4::Message->find_last_message($borrower, $type, $mtt); unless ( $message ) { C4::Context->dbh->do(q|UNLOCK TABLES|) unless $do_not_lock; C4::Message->enqueue($letter, $borrower, $mtt); } else { $message->append($letter); $message->update; The append method digests multiple notices by looking for a ---- separator: if ($letter->{content} =~ /----/) { my ($header, $body, $footer) = split(/----\r?\n?/, $letter->{content}); In the first place I would not mind sending these notices separately. But anyway. If we do combine them (within the timeframe of the message queue processing interval), it would be convenient to allow separating too on an html tag like <item> as used elsewhere in notices. I looked a bit at the notice digesting and actually only find this feature working for advance notices. The overdue digest flag is a bit weird: the job already digests by default? -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=31101 --- Comment #1 from Katrin Fischer <katrin.fischer@bsz-bw.de> --- What do you mean by ovedue digest flag? -- You are receiving this mail because: You are watching all bug changes. You are the assignee for the bug.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=31101 --- Comment #2 from Marcel de Rooy <m.de.rooy@rijksmuseum.nl> --- (In reply to Katrin Fischer from comment #1)
What do you mean by ovedue digest flag?
The digests only box on Messaging preferences. -- You are receiving this mail because: You are watching all bug changes. You are the assignee for the bug.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=31101 --- Comment #3 from Katrin Fischer <katrin.fischer@bsz-bw.de> --- I think the 'overdue' confused me, as we don't have the overdues in messaging preferences. Overdues are digested by branch/overdue level or you can include all overdue items with a flag. -- You are receiving this mail because: You are watching all bug changes. You are the assignee for the bug.
participants (1)
-
bugzilla-daemon@bugs.koha-community.org