- Issue created by @thomwilhelm
Logic in WatchdogPruneSettings relating to dblog_row_limit needing to be set to 0 could be better handled by checking the actual setting in the system, currently it just appears hardcoded.
    $form['core_fs']['dblog_row_limit'] = [
      '#type' => 'select',
      '#title' => $this->t('[From Drupal Core:] Database log messages to keep'),
      '#options' => ['0' => 'All'],
      '#default_value' => 0,
      '#description' => $this->t('For this module to function, we must keep this Drupal Core setting set to <strong>All</strong>.  This setting is provided here simply as a reminder of where this setting is coming from.'),
    ];
Refactor WatchdogPruneSettings form, this might be a good time to actually refactor other some other settings, add better validation etc.
Active
3.0
Code