expect possible null in HttpContextExtension method

src/JsonApiDotNetCore/Extensions/HttpContextExtensions.cs

@@ -15,11 +15,11 @@ internal static void SetJsonApiRequest(this HttpContext httpContext)
  httpContext.Items["IsJsonApiRequest"] = bool.TrueString;
  }
 
- internal static void DisableValidator(this HttpContext httpContext, string model, string name)
+ internal static void DisableValidator(this HttpContext httpContext, string propertyName, string model = null)
  {
  if (httpContext == null) return;
- var itemKey = $"JsonApiDotNetCore_DisableValidation_{model}_{name}";
- if (!httpContext.Items.ContainsKey(itemKey))
+ var itemKey = $"JsonApiDotNetCore_DisableValidation_{model}_{propertyName}";
+ if (!httpContext.Items.ContainsKey(itemKey) && model != null)
  {
  httpContext.Items.Add(itemKey, true);
  }

src/JsonApiDotNetCore/Serialization/Common/BaseDocumentParser.cs

@@ -78,13 +78,10 @@ protected IIdentifiable SetAttributes(IIdentifiable entity, Dictionary<string, o
  {
  foreach (var attr in attributes)
  {
+ var disableValidator = false;
  if (attributeValues == null || attributeValues.Count == 0)
  {
- if (_httpContextAccessor?.HttpContext?.Request.Method != "PATCH") continue; 
- if (attr.PropertyInfo.GetCustomAttribute<IsRequiredAttribute>() != null)
- {
- _httpContextAccessor.HttpContext.DisableValidator(attr.PropertyInfo.ReflectedType?.Name, attr.PropertyInfo.Name);
- }
+ disableValidator = true;
  }
  else
  {
@@ -96,13 +93,16 @@ protected IIdentifiable SetAttributes(IIdentifiable entity, Dictionary<string, o
  }
  else
  {
- if (_httpContextAccessor?.HttpContext?.Request.Method != "PATCH") continue;
- if (attr.PropertyInfo.GetCustomAttribute<IsRequiredAttribute>() != null)
- {
- _httpContextAccessor.HttpContext.DisableValidator(attr.PropertyInfo.ReflectedType?.Name, attr.PropertyInfo.Name);
- }
+ disableValidator = true;
  }
  }
+
+ if (!disableValidator) continue;
+ if (_httpContextAccessor?.HttpContext?.Request.Method != "PATCH") continue;
+ if (attr.PropertyInfo.GetCustomAttribute<IsRequiredAttribute>() != null)
+ {
+ _httpContextAccessor.HttpContext.DisableValidator(attr.PropertyInfo.Name, attr.PropertyInfo.ReflectedType?.Name);
+ }
  }
 
  return entity;
@@ -123,7 +123,7 @@ protected IIdentifiable SetRelationships(IIdentifiable entity, Dictionary<string
  var entityProperties = entity.GetType().GetProperties();
  foreach (var attr in relationshipAttributes)
  {
- _httpContextAccessor?.HttpContext?.DisableValidator(attr.PropertyInfo.Name, "Relation");
+ _httpContextAccessor?.HttpContext?.DisableValidator( "Relation", attr.PropertyInfo.Name);
 
  if (!relationshipsValues.TryGetValue(attr.PublicRelationshipName, out RelationshipEntry relationshipData) || !relationshipData.IsPopulated)
  continue;