Following from #1023414: Add tabs to admin scheduler settings page I had intended to add the Lightweight Cron help text using hook_help() but instead you have added it directly into the form. I think it is preferable to separate the wordy text from the operation of the form, so here is a patch which makes this change. The benefit of having a hook_help is that you automatically get a 'more help' link which includes a link to the permissions. Additional help can be added in future, but it is good to have the function, as this is a basic drupal hook which all modules should have.
| Comment | File | Size | Author |
|---|---|---|---|
| #4 | hook_help central help.jpg | 370.57 KB | jonathan1055 |
| #4 | hook_help main with readme.txt.jpg | 213.04 KB | jonathan1055 |
| #4 | hook_help settings page.jpg | 347.98 KB | jonathan1055 |
| #4 | _scheduler.1027302.implement_hook_help.4.D7.patch | 2.35 KB | jonathan1055 |
| #1 | _1027302.scheduler.implement_hook_help-D6.patch | 3.16 KB | jonathan1055 |
Comments
Comment #1
jonathan1055 commentedHere's the patch, against the latest dev of 14th Jan.
Comment #2
eric-alexander schaefer commentedOh, I didn't know that. Thanks a lot.
http://drupal.org/cvs?commit=483594
Comment #3
jonathan1055 commentedRequires a D7 version.
Comment #4
jonathan1055 commentedHere is the patch for D7. In the main settings page I have expanded the help to include a link to the content type admin page. Also for the main help section I have added a link to the readme.txt file - the attached screen grabs show these, and I have hovered over the link to show where it is poiting to.
Comment #5
eric-alexander schaefer commentedWorks.
http://drupal.org/cvs?commit=487144