It seems when you select 'URL' as type when scheduling a report from the staff client, that the resulting email is blank. To test, 1) Navigate to: Home › Reports › Guided reports wizard › Saved reports 2) Select for any saved report: Actions › Schedule 3) Scheduled a report to run 'soon' 4) Select 'URL' as the 'Output format' 5) Enter your email address and wait for the resulting email The resulting email will be 'bodyless', i.e. empty
Looking at the runreports.pl script that this instantiates, it looks like the URL type handling was commented out some time ago. I'm not sure if this feature is ever used/likely to be used. Perhaps the solution is to just tidy up a bit.. remove the commented code in runreport.pl and drop the URL option in the staff client... thoughts?
Offering options/features that don't work is never a good thing - removing it from the GUI would be good in my opinion. I can imagine maybe the reason was not to send out patron information by email, but offer a link to access it instead?
I agree strongly with Katrin that it's a bad idea to leave broken features in the UI. This doesn't work as advertised and should be removed. I'm perfectly willing to write a patch and removes the <select> from the scheduler.tt template, replacing it with a hidden input, and the corresponding help text in corresponding template as well if people are agreed that that's the best course of action. While this may be a useful feature I think it's better to remove the broken appendage of it first and add back if/when it's fully complete. It's also already possible to construct a link that goes to the results of a particular report. One could create a calendar reminder in some external application that handles this use case easily.
Created attachment 73804 [details] [review] patch
I wrote a patch that removes "URL" and adds CSV & HTML options to the "format" list. I tested and both of the new format options work just fine even though they weren't listed previously; I found them by looking at misc/cronjobs/runreport.pl which is the script invoked by the task scheduler.
(In reply to Eric Phetteplace from comment #5) > I wrote a patch that removes "URL" and adds CSV & HTML options to the > "format" list. I tested and both of the new format options work just fine > even though they weren't listed previously; I found them by looking at > misc/cronjobs/runreport.pl which is the script invoked by the task scheduler. I have confirmed there are 4 valid values, but tsv and text are identical in the runreport.pl script. It really isn't removing URL as much as it is correcting the name to HTML, I believe. This looks good. Just about to test...
Created attachment 73962 [details] [review] Bug 13445 - Scheduled Report, Type=URL fails Remove the non-functional 'URL' format Offer the (functional) CSV & HTML formats testing plan: - visit the task scheduler tool at cgi-bin/koha/tools/scheduler.pl - 'Output format' select list should offer Text (TSV), CSV, & HTML - scheduled reports for each of the above types should run & send emails correctly Signed-off-by: Mark Tompsett <mtompset@hotmail.com>
Created attachment 74177 [details] [review] Bug 13445 - Scheduled Report, Type=URL fails Remove the non-functional 'URL' format Offer the (functional) CSV & HTML formats testing plan: - visit the task scheduler tool at cgi-bin/koha/tools/scheduler.pl - 'Output format' select list should offer Text (TSV), CSV, & HTML - scheduled reports for each of the above types should run & send emails correctly Signed-off-by: Mark Tompsett <mtompset@hotmail.com> Signed-off-by: Maksim Sen <maksim.sen@inlibro.com>
Created attachment 74693 [details] [review] Bug 13445: Remove URL option from scheduled reports, add Text option Remove the non-functional 'URL' format Offer the (functional) CSV & HTML formats testing plan: - visit the task scheduler tool at cgi-bin/koha/tools/scheduler.pl - 'Output format' select list should offer Text (TSV), CSV, & HTML - scheduled reports for each of the above types should run & send emails correctly Signed-off-by: Mark Tompsett <mtompset@hotmail.com> Signed-off-by: Maksim Sen <maksim.sen@inlibro.com> Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Eric, I notice that your author line in the patch reads: phette23 <phette23@gmail.com> Do you want to fix it to have your name? It will be listed in the release notes then. I can do it for you if you want to.
Pushed to master for 18.05, thanks to everybody involved!
(In reply to Katrin Fischer from comment #10) > Eric, I notice that your author line in the patch reads: > phette23 <phette23@gmail.com> > > Do you want to fix it to have your name? It will be listed in the release > notes then. I can do it for you if you want to. I actually prefer my phette23 handle, but thank you for offering.
That's good, because pushed :)