As more and more folks migrate to Drupal 8, we'll see D8 CiviCRM sites popping-up and some will require the integration to Drupal's cron provided by this module.

Let's get this thing upgraded!

CommentFileSizeAuthor
#2 civicrm_cron-d8-port-2914206-2.patch18.53 KBkerasai

Comments

kerasai created an issue. See original summary.

kerasai’s picture

kerasai commented
Assigned: kerasai » Unassigned
Status: Active » Needs review
StatusFileSize
new civicrm_cron-d8-port-2914206-2.patch18.53 KB

Attached is a patch applicable to the 7.x-2.x branch which should be suitable for an alpha (or beta) D8 release.

It's essentially a straight port of the functionality, however there are a couple of noteworthy changes:

  1. The HTTP request made to the passtrough endpoint returns a specific header which callers may analyze to determine if the cron run was successful, versus success checking for an empty payload.
  2. The admin config form had some "crafty" bits in it to allow usage of system_setting_form(). This has been removed and replaced with standard form build/validate/submit methods.
kerasai’s picture

kerasai commented
Status: Needs review » Fixed

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.