DateOnly? and DateTime? return validation errors when null or empty #9736

10 Answer(s)

• 

  0

  AI-Bot created one month ago

  Support Team ABP Agent

  • In ABP Framework, input parameters for application service methods and controllers are automatically validated using data annotation attributes and custom validation logic. By default, nullable types like DateOnly? or DateTime? should not trigger a validation error when their value is null, unless you have applied a [Required] attribute or custom validation that enforces a value.

  • If you are seeing a validation error such as "inspectionDate is null!" when passing null for a DateOnly? or DateTime? parameter, it is likely that:

    1. The parameter or its DTO property is decorated with the [Required] attribute, or
    2. There is a custom validation rule (e.g., IValidatableObject, custom validator, or manual check) that throws an error when the value is null.

  • To resolve this:

    • Remove the [Required] attribute from the parameter/property if you want to allow null values.
    • Check for any custom validation logic in your DTOs, controllers, or services that may be enforcing non-null values.
    • If you want to allow nulls, ensure your method signature uses DateOnly? or DateTime? (nullable) and that no validation rule requires a value.

  Example:

  public async Task<IActionResult> GetAsync(string plateId, DateOnly? inspectionDate, string vin, string chassisNumber)
  {
      // inspectionDate can be null here
  }
  

  If you have a DTO:

  public class MyDto
  {
      // Do NOT add [Required] if you want to allow nulls
      public DateOnly? InspectionDate { get; set; }
  }
  

  • If you are not using [Required] and still see this error, check for custom validation logic in your codebase that may be enforcing this rule.

  Sources:

  • ABP Application Service Validation
  • ABP Validation Fundamentals

  ---

  This response was generated by AI. If it didn’t solve your problem, feel free to post a new answer — our support team will jump in to help you soon.

  Save Cancel
• 

  0

  auxo-devsu created one month ago

  There's no [Required] attribute or custom logic enforcing it.

  Save Cancel
• 

  0

  maliming created one month ago

  Support Team Fullstack Developer

  hi

  Could you please share the code that reproduces the issue?

  Thanks

  Save Cancel
• 

  0

  auxo-devsu created one month ago

  I provided an example on my request, but if you have any API projects on ABP, as I assume you do, please do the following:

  public async Task<IActionResult> GetSomethingByDateAsync([FromQuery] DateOnly? date)
  {
      yourAppService.GetSomethingByDateAsync(date); // Before it reaches the AppService the ABP Validations will throw an exception when date is null despite being a nullable DateOnly
  }
  

  Save Cancel
• 

  0

  maliming created one month ago

  Support Team Fullstack Developer

  hi

  It will best if you can share a template project.

  

  Thanks.

  Save Cancel
• 

  0

  auxo-devsu created one month ago

  This makes me feel that you either didn't understand what I said or that you don't want to try by yourself. For most of your customers, we have real IP in our projects, and a lot of them would be large enough not to be shared here.It's a job in itself to create a new project just to share it here.

  public async Task<IActionResult> GetSomethingByDateAsync([FromQuery] DateOnly? date)
  {
      yourAppService.GetSomethingByDateAsync(date); // Before it reaches the AppService the ABP Validations will throw an exception when date is null despite being a nullable DateOnly
  }
  

  If you look at your code, there's no App Service dependency accepting the DateOnly. Instead of having a Redirect create an AppService that accepts DateOnly? and you will see what I described above.

  Save Cancel
• 

  0

  maliming created one month ago

  Support Team Fullstack Developer

  hi

  Try to add this to your project, and I will fix it in the framework.

  Thanks

  using System.Reflection;
  using System.Threading.Tasks;
  using Volo.Abp.DependencyInjection;
  using Volo.Abp.Reflection;
  using Volo.Abp.Validation;
  
  namespace MyCompanyName.MyProjectName.Web;
  
  [Dependency(ReplaceServices = true)]
  [ExposeServices(typeof(MethodInvocationValidator), typeof(IMethodInvocationValidator))]
  public class MyMethodInvocationValidator : MethodInvocationValidator
  {
      private readonly IObjectValidator _objectValidator;
  
      public MyMethodInvocationValidator(IObjectValidator objectValidator)
          : base(objectValidator)
      {
          _objectValidator = objectValidator;
      }
      
      protected override async Task AddMethodParameterValidationErrorsAsync(IAbpValidationResult context, ParameterInfo parameterInfo, object parameterValue)
      {
          var allowNulls = parameterInfo.IsOptional ||
                           parameterInfo.HasDefaultValue ||
                           parameterInfo.IsOut ||
                           TypeHelper.IsNullable(parameterInfo.ParameterType) ||
                           TypeHelper.IsPrimitiveExtended(parameterInfo.ParameterType, includeEnums: true);
  
          context.Errors.AddRange(
               await _objectValidator.GetErrorsAsync(
                  parameterValue,
                  parameterInfo.Name,
                  allowNulls
              )
          );
      }
  }
  
  

  Save Cancel
• 

  0

  alper created one month ago

  Support Team Director

  we cannot help you if you don't cooperate with us for your problem. In this post, he's showing that he can successfully use DateOnly?

  Save Cancel
• 

  0

  auxo-devsu created one month ago

  @alper I have been a customer for years, and I'm sure if anything, I'm cooperating with the framework by sharing my challenge here. There are multiple ways of getting around that, and bringing this to attention is the right call. If you have read my message and the response I provided, you would see nothing but cooperation.

  @mailiming Thanks for the acknowledgement and for having a closer look at the problem I described. That fix should resolve the issue for me and any other ABP users looking to do that in the future. Much appreciated!

  Save Cancel
• 

  0

  maliming created one month ago

  Support Team Fullstack Developer

  Thanks @auxo-devsu

  Save Cancel