The status report page shows the link in the message from this module with raw HTML.

Furthermore, the text isn't correctly created -- sentence text should not be split into t() calls:

          'Some jobs are behind their schedule. Please check if @link is running properly.', array(
          '@link' => \Drupal::l(t('Cron'), Url::fromRoute('system.cron', ['key' => \Drupal::state()->get('system.cron_key')])),
        ));

The A element should be within the original string, and the url inserted with a placeholder, not the link.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

joachim created an issue. See original summary.

id.aleks’s picture

Assigned: Unassigned » id.aleks
Issue tags: +LutskGSW18
id.aleks’s picture

Assigned: id.aleks » Unassigned
Status: Active » Needs review
FileSize
1.07 KB

Changed cron_jobs status description to markup.

Status: Needs review » Needs work

The last submitted patch, 3: raw-html-link-2910840-3.patch, failed testing. View results

id.aleks’s picture

Status: Needs work » Needs review
FileSize
1.07 KB

Updated patch.
Just one note, as for me the link should go to admin/config/system/cron/jobs. Right now it goes to "run cron from outside the site" page.

Status: Needs review » Needs work

The last submitted patch, 5: raw-html-link-2910840-5.patch, failed testing. View results
- codesniffer_fixes.patch Interdiff of automated coding standards fixes only.

id.aleks’s picture

Status: Needs work » Needs review
FileSize
1.98 KB

Fixed CronJobInstallTest test =)

joachim’s picture

Status: Needs review » Reviewed & tested by the community

Looks good to me!

  • Berdir committed b4fc1f7 on 8.x-2.x authored by id.aleks
    Issue #2910840 by id.aleks: raw HTML link in status report page
    
Berdir’s picture

Status: Reviewed & tested by the community » Fixed

Committed.

Status: Fixed » Closed (fixed)

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