Malcolm Young commented on merge request !185 at project / smart_date

Malcolm Young
2025-09-25T13:47:52Z

there's no point in throwing and catching within the same class - just use an if

    if ($this->checkAllElementsAreNull($min) || $this->checkAllElementsAreNull($max)) {
      $this->messenger->addError('Please enter valid dates);
      return [$min_value, $max_value];
    }

although the text should go through t() - I guess that would also need to be injected?