This is a really great module. The only issue I've found is in testing. It's hard to get my hooks to fire at their various intervals without setting my system clock so I was thinking it would be nice to have a "testing" checkbox in the configuration section so that when checked, all hook implementations fire every time cron.php is run. Currently, I have to run "delete from variable where name like '%cron%'" to get it to run. Just a thought..

CommentFileSizeAuthor
#8 cronplus-add-on-demand-363809-8.patch11.28 KBNROTC_Webmaster
#7 cronplus.module.txt12.8 KBNROTC_Webmaster

Comments

syscrusher’s picture

syscrusher commented
Assigned: Unassigned » syscrusher

That's a wonderful suggestion, and very easy to implement. I'll try go get this into the code soon.

Scott

syscrusher’s picture

syscrusher commented

BTW, thanks for the kind words about the module, and for the test report.

jwbuzz’s picture

jwbuzz commented

Sounds great. Looking forward to it.

syscrusher’s picture

syscrusher commented

I have just added this feature and committed a development release, pending beta testing.

I'm working with some new developer tools at my end, and I think I did everything right for the commit, but please be patient if not. I can't check the new release until drupal.org's next batch run.

syscrusher’s picture

syscrusher commented
Version: 6.x-1.0-beta4 » 6.x-1.x-dev
sydneyshan’s picture

sydneyshan commented

Hi syscrusher,

I can't see this functionality in any release avaliable on Drupal.org - a testing facility to trigger any of the registered crong functions would be great to have!

Kind regards,
Shannon

NROTC_Webmaster’s picture

NROTC_Webmaster commented
StatusFileSize
new cronplus.module.txt12.8 KB

I also had a need to run cronplus on demand. Since I'm not very good with PHP this is probably not the best way to do it but it works for what I needed it to do. I simply added an on demand call and then added that call in the module I needed it for.

Great work on the cronplus module. I don't know how to make a patch yet but I figured I would go ahead and post my file incase anyone else is interested in this ability.

NROTC_Webmaster’s picture

NROTC_Webmaster commented
Status: Active » Needs review
StatusFileSize
new cronplus-add-on-demand-363809-8.patch11.28 KB

Added an on demand option for testing cronplus along with some other minor tweaks including the capitalization of the menu items.

izmeez’s picture

izmeez commented

The patch in #8 appears to work. It only applies to the latest 6.x-1.x-dev release.