Misleading inconsistent timeout unit

Created on 5 August 2025, about 1 month ago

Problem/Motivation

The configuration form message for timeout is in minute:

The maximum time in minutes that each lock may be kept.

But the code appears to be in second:
$query->condition('timestamp', $this->time->getCurrentTime() - $timeout, '>=');

  if ($installed) {
    $config->set('timeout', ((int) \Drupal::config('content_lock_timeout.settings')->get('content_lock_timeout_minutes')) * 60);
  }

Proposed resolution

Keep them consistent. Might be better to use second considering since it's what the code is.

🐛 Bug report
Status

Active

Version

3.0

Component

Code

Created by

Live updates comments and jobs are added and updated live.
Sign in to follow issues

Comments & Activities

Production build 0.71.5 2024