Needs review
Project:
SuperCron
Version:
6.x-1.3
Component:
Code
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
12 Jan 2011 at 22:50 UTC
Updated:
17 Mar 2011 at 15:58 UTC
Jump to comment: Most recent file
The current recommended version of supercron (6.x-1.3) will generate tons of php notices because PHP constants are defined like this:
define(SUPERCRON_INVOKE_ALL, 'admin/reports/status/run-supercron');
not this
define('SUPERCRON_INVOKE_ALL', 'admin/reports/status/run-supercron');
Patch attached.
N.b. this appears to be fixed in 6.x-2.x-dev
P.s. Thanks for the excellent module
| Comment | File | Size | Author |
|---|---|---|---|
| #3 | supercron-n1024868-3.patch | 1.17 KB | jwilson3 |
| #1 | 1024868_supercron_defines.patch | 1.46 KB | alexpott |
| supercron_defines.patch | 1.46 KB | alexpott |
Comments
Comment #1
alexpottDoh! Misplaced quote in patch... new patch attached.
Comment #2
Jochus commented!subscribe
Comment #3
jwilson3My version of supercron 6.x-1.3 did not contain the constant: SUPERCRON_PHP_BINARY_PATH_PLACEHOLDER So the patch didnt apply cleanly. Here is my version of the patch for 6.x-1.3.