@mkurzeja opened this Pull Request on June 9th 2014 Contributor

The ReportRenderer can be overridden using ScheduledReports.getRendererInstance event. This allows to render non-standard html/pdf/csv reports. If such report contains several graphs then it is not possible to send it as e-mail as the attachments are generated in ScheduledReports/API. This seems to be inconsistent. If a ReportRenderer can decide about the content of a report it should also be able to decide about the attachments.

I 've added small changes to the code which allow to override the default attachment creation process.

@mattab commented on June 9th 2014 Member

:+1: for fixing this inconsistency with a little refactoring that improves the core platform!

This Pull Request was closed on June 9th 2014
Powered by GitHub Issue Mirror