- Issue created by @wangshy
Closing it. Just found second/minute is converted in the setting form so not really a bug though could be a little confusing when reading the code.
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);
}
Keep them consistent. Might be better to use second considering since it's what the code is.
Active
3.0
Code
Closing it. Just found second/minute is converted in the setting form so not really a bug though could be a little confusing when reading the code.