Problem/Motivation

In CronJobInstallTest we have an url as absolute (/cron/..), and it assumes that Drupal is installed in the top level directory which is true for the local machine but not for Testbot.

Proposed resolution

There it is /checkout/cron.. so it needs to build the url string dynamically, with Url::fromRoute()

Remaining tasks

User interface changes

API changes

Data model changes

CommentFileSizeAuthor
#2 ultimate_cron-change_absolute_url_to_dynamic-2630852-2.patch1.22 KBtduong
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

tduong created an issue. See original summary.

tduong’s picture

tduong CreditAttribution: tduong at MD Systems GmbH commented
FileSize
ultimate_cron-change_absolute_url_to_dynamic-2630852-2.patch1.22 KB

Change url to dynamic.

tduong’s picture

tduong CreditAttribution: tduong at MD Systems GmbH commented
Status: Active » Needs review
Berdir’s picture

Berdir
Primary language German
Location Switzerland
CreditAttribution: Berdir at MD Systems GmbH commented
Status: Needs review » Fixed

Thanks, committed.

  • Berdir committed 606380c on 8.x-2.x authored by tduong 
    Issue #2630852 by tduong: Change absolute url to dynamic in requirements...

Status: Fixed » Closed (fixed)

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