Problem/Motivation

The sensor log shows: Execution time: "[execution_time]ms"

The value is not replaced. It is though correctly stored in the database.

Proposed resolution

Make the placeholder replace.

Remaining tasks

User interface changes

API changes

CommentFileSizeAuthor
#2 monitoring_2482145_log_time.patch1.32 KBmiro_dietiker
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Berdir’s picture

That's because views placeholders are now using twig syntax => {{ execution_time }}ms

miro_dietiker’s picture

Status: Active » Needs review
FileSize
1.32 KB

Status: Needs review » Needs work

The last submitted patch, 2: monitoring_2482145_log_time.patch, failed testing.

Status: Needs work » Needs review
LKS90’s picture

The patch fixes the issue. Installed without it and saw the [execution_time]ms in the log of a sensor detail page, reinstalled with this patch applied and the log shows the execution time. Any other place to look? When adding execution time in a view, it isn't set up with overwrite etc., maybe we can add this with this issue?

Status: Needs review » Needs work

The last submitted patch, 2: monitoring_2482145_log_time.patch, failed testing.

  • Berdir committed c1696e7 on 8.x-1.x authored by miro_dietiker
    Issue #2482145 by miro_dietiker: Fixed Execution time in log does not...
Berdir’s picture

Status: Needs work » Fixed

Committed.

Status: Fixed » Closed (fixed)

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