When running the cronjob in verbose mode, such warnings are generated: sending print message to patron: 1008537 at /home/kivilahtio/koha/kohaclone/C4/Letters.pm line 754. sending print message to patron: 1049202 at /home/kivilahtio/koha/kohaclone/C4/Letters.pm line 754. sending print message to patron: 1015993 at /home/kivilahtio/koha/kohaclone/C4/Letters.pm line 754. sending print message to patron: 1048403 at /home/kivilahtio/koha/kohaclone/C4/Letters.pm line 754. When collecting cronjob output to log files, this irks me off because it actually doesn't do anything. As a convenience, a patch is provided to simply warn when something is actually done, or tried to do.
Created attachment 31234 [details] [review] pätch Made the C4::Letters::SendQueuedMessages() notify all but 'print' messages in verbose mode. This way verbose mode provides a bit cleaner output.
So, test plan?
I don't think this is relevant, please detail the need.