Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
in system.install the code seems to be invalid:
$form['cron_url'] = array(
'#markup' => '<p>' . t('To run cron from outside the site, go to <a href="!cron">!cron</a>', array('!cron' => $this->url('system.cron', array('key' => $this->state->get('system.cron_key')), array('absolute' => TRUE)))) . '</p>',
);
To run cron from outside the site, go to <a href="/drupal8/cron/[KEY]?0%5Babsolute%5D=1">/drupal8/cron/[KEY]?0%5Babsolute%5D=1</a>
Comment | File | Size | Author |
---|---|---|---|
#25 | 2563757-25.patch | 995 bytes | jofitz |
#25 | interdiff-2563757-23-25.txt | 847 bytes | jofitz |
#23 | 2563757-23.patch | 983 bytes | jofitz |
#12 | external_cron_url_on-2563757-12.patch | 1.61 KB | cilefen |
#12 | interdiff-2563757-12.txt | 775 bytes | cilefen |
Comments
Comment #2
hass CreditAttribution: hass commentedComment #3
hass CreditAttribution: hass commentedComment #4
hass CreditAttribution: hass commented.
Comment #5
hass CreditAttribution: hass commentedComment #6
hass CreditAttribution: hass commentedComment #7
cilefen CreditAttribution: cilefen commentedIs there a simpletest assert to test if a URL is absolute? I think this feature is important enough to regression test if we can.
Comment #8
hass CreditAttribution: hass commentedI have not found any assert message tests for the reports page. Do you know where these are located?
Comment #9
cilefen CreditAttribution: cilefen commentedI just reinstalled from HEAD and the page source shows an absolute URL.
<p>To run cron from outside the site, go to <a href="http://localhost/drupal8x/cron/1euTIONtUYMmjtMgPUFRqIZoWE-RBRJb414pnpUmACNjpsj4PbaCjTQWyA_plmD1iMaPoeuhwQ">http://localhost/drupal8x/cron/1euTIONtUYMmjtMgPUFRqIZoWE-RBRJb414pnpUmACNjpsj4PbaCjTQWyA_plmD1iMaPoeuhwQ</a></p>
Comment #10
cilefen CreditAttribution: cilefen commentedOh, wait, #9 was on admin/config/system/cron. This is a different path. Ignore #9.
Comment #11
cilefen CreditAttribution: cilefen commentedDrupal\system\Tests\System\StatusTest
Comment #12
cilefen CreditAttribution: cilefen commentedComment #17
dagmarThe patch doesn't apply anymore.
Comment #22
penyaskitoComment #23
jofitz CreditAttribution: jofitz at ComputerMinds commentedThe change has since been implemented, but the test has not, so here it is.
Comment #25
jofitz CreditAttribution: jofitz at ComputerMinds commentedUse a string not the url object.
Comment #34
smustgrave CreditAttribution: smustgrave at Mobomo commentedClosing as outdated since the message update has since been implemented. And the current tests seem to cover that. If you still feel this is an issue please reopen