Closed (fixed)
Project:
Elysia Cron
Version:
7.x-2.x-dev
Component:
Code
Priority:
Major
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
13 Nov 2012 at 01:20 UTC
Updated:
7 May 2016 at 09:14 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #1
chaps2 commentedI second this. 64 should be enough...
Comment #2
gotheric commentedInteresting.
I should put a check before the save to database and crop larger strings. This is to avoid the error always (even with strings > 64 chars)
Probably i'll update the column to 64 chars too.
I'll commit something soon, thanks for the issue.
Comment #3
regilero commentedHere is a 7.2.x-dev patch
Comment #4
regilero commentedAnd this one should work with 7.x-2.1 last stable delivered version
Comment #5
xaris.tsimpouris commentedShouldn't be 120 characters, as "name" column has and mostly includes function names?
Comment #6
adammaloneI've rerolled and refactored the above patch to:
Comment #7
adammaloneSetting this issue to major as it has the potential to stop cron from running on sites and it's a simple enough fix to get into the module. Also hiding patches that no longer apply to latest 7.x-2.x.
Comment #8
randell commentedPatch at #6 works as expected.
Comment #9
randell commentedComment #10
wylbur commentedApplied this patch to our site, which was crashing because of a function name that was too long. This resolved our issue immediately. This is ready for deployment.
Comment #11
wylbur commentedJust to confirm, I applied the patch in comment #6.
Comment #13
kala4ekCommitted to latest dev.
Comment #14
kala4ek