Closed (fixed)
Project:
Panels Pane Report
Component:
Code
Priority:
Minor
Category:
Task
Assigned:
Reporter:
Created:
4 Nov 2016 at 05:29 UTC
Updated:
7 Nov 2016 at 20:45 UTC
Jump to comment: Most recent, Most recent file
We need the ability to schedule the report to run on cron (if the user wants it).
| Comment | File | Size | Author |
|---|---|---|---|
| #7 | panels-pane-report-2824809-setup-cron-7.patch | 4.2 KB | WidgetsBurritos |
Comments
Comment #2
WidgetsBurritos commentedComment #3
WidgetsBurritos commentedThis patch adds an admin page and the ability to turn cron on/off and adjust the interval.
Comment #4
pobster commentedI'd maybe suggest using a select for the time period as core module tend to, this is lifted from aggregator;
Also it makes a lot of sense to slim the module files themselves as much as possible as they're always loaded into memory. Every little helps so would recommend moving out the admin functions to a separate file, e.g. "panels_pane_report.admin.inc" with a reference in hook_menu.
Comment #5
pobster commentedComment #6
WidgetsBurritos commented@pobster Okay how about now?
Comment #7
WidgetsBurritos commentedNow with fewer variables.
Comment #8
pobster commentedLooks great, thanks! +1
Comment #10
WidgetsBurritos commentedComment #11
WidgetsBurritos commented