Early Bird Registration for DrupalCon Portland 2024 is open! Register by 23:59 PST on 31 March 2024, to get $100 off your ticket.
Would be nice to have the option to post directly on form submit without waiting for cron. Posting to eloqua at cron has some good usecases, but also has some drawbacks : it can be quite an heavy load for large batches, as we need to load the whole post from db and process it to map fields, ask for hooks, etc. This makes the cron run time quite long, and could be a pb with lots of submissions. Posting at submit could split the load.
Any thoughts ?
Comment | File | Size | Author |
---|---|---|---|
#5 | eloqua-cron-d7-1292302-5.patch | 89.78 KB | drupalninja99 |
#5 | eloqua-cron-only-d7-1292302.patch | 4.53 KB | drupalninja99 |
#5 | new setting | 34.55 KB | drupalninja99 |
Comments
Comment #1
bellesmanieres CreditAttribution: bellesmanieres commentedNo patch for now, first waiting for your opinion on it.
Also forgot to mention I'm the "colleague" greg.harvey is talking about in http://drupal.org/node/1288922
Comment #2
RichardLynch CreditAttribution: RichardLynch commentedWe are looking into using Eloqua for marketing lead generating sites.
Marketing being Marketing, waiting until the next cron run is NOT an option.
They want lead-flow in real-time to agents waiting to call back.
So put me down for +1 on real-time form processing.
For whatever that's worth. :-)
Comment #3
drupalninja99 CreditAttribution: drupalninja99 commentedI think I agree this should be an option. As long as the delay isn't too long.
I would maybe like to refactor the functions that say '_eloqua_cron' to say something more descriptive like '_eloqua_submit' or something like that and then allow the function to be run for just 1 submission (the current submission).
A workaround though is to have a cron job that runs once a minute and only runs the eloqua cron.
Comment #4
drupalninja99 CreditAttribution: drupalninja99 commentedI am going to create sub-branch based on the patch here: https://drupal.org/node/1392580#comment-7718491 for this ticket.
Comment #5
drupalninja99 CreditAttribution: drupalninja99 commented1. The first patch is rolled with [1392580].
2. The second is the new patch only.
3. The screenshot shows the new setting which let's you immediately submit to Eloqua.
This was a pretty easy change. I basically keep the same logic that was used in cron but add a way to only process the current submission so I did not have to do a lot of refactoring.
Comment #6
drupalninja99 CreditAttribution: drupalninja99 commentedComment #7
drupalninja99 CreditAttribution: drupalninja99 commentedI have tested and pushed this to 7.x-1.x.