Early Bird Registration for DrupalCon Portland 2024 is open! Register by 23:59 PST on 31 March 2024, to get $100 off your ticket.
Some content was cut from security-cron.txt and moved into security-cron-concept.txt.
There is no line break at 80 chars (yet)
Comments
Comment #2
dalinComment #3
stingrayfuse CreditAttribution: stingrayfuse commentedComment #4
stingrayfuse CreditAttribution: stingrayfuse commentedComment #5
stingrayfuse CreditAttribution: stingrayfuse commentedComment #6
stingrayfuse CreditAttribution: stingrayfuse commentedHi guys, I'm _brand_ new but this seemed to make sense to me (even as a non-developer) so I put it through. Hope that sits well with you all!
Comment #7
jhodgdonHm....
I don't think this is quite ready to add to the Guide:
a)
For future reference... We actually do *not* want to completely remove empty sections from the templates, so this piece (and the similar one below) do not belong in this patch.
I don't think "Cron" should be capitalized at the end of this line.
Actually it should be in italics as a new term, but only the one at the beginning of the line.
_Cron_ is a task scheduler...
To me, the term "cron job" would refer to the whole cron process, not an individual action within the process.
So maybe we could say "cron job" is the whole job, and "cron maintenance task" is one task within it?
This last sentence is not a sentence.
maybe say:
Examples of cron tasks include: ...
Comment #8
batigolixHere is new version. Instead of addressing your feedback point by point, I reused the cron description we are using in the hook help of system.module (slightly adapted to be more like a user guide topic). It has been reviewed pretty well before (I hope) ;)
Comment #9
eojthebraveThe text copied from hook_help() looks good to me. Good idea. I think we need to provide an attribution for this text though.
Comment #10
jhodgdonYeah... and the other problem is that technically, anything in hook_help() is in drupal Core and is therefore GPL, whereas our guide is CC-by-SA. So we cannot really copy that word for word. :(
Comment #11
batigolixI remember that now ;)
So here is a rewording without touching the main concepts:
Original from system_help():
User guide proposed text (see patch):
Comment #12
batigolixFixing a couple of small errors
Comment #14
jhodgdonThat looks good. I added batigolix to attributions as well as dalin. Reformatted slightly as well, see #2667978: Review/fix AsciiDoc formatting.