Partially revert changes for CA1873 (Evaluation of this argument may be expensive and unnecessary if logging is disabled), now that it's not so rigid anymore since .NET 10 SDK v10.0.102 (#1904)

Author: bkoelman

src/JsonApiDotNetCore/QueryStrings/QueryStringReader.cs

@@ -47,11 +47,7 @@ public void ReadAll(DisableQueryStringAttribute? disableQueryStringAttribute)
 
             if (reader != null)
             {
-                if (_logger.IsEnabled(LogLevel.Debug))
-                {
-                    string readerType = reader.GetType().Name;
-                    LogParameterAccepted(parameterName, parameterValue, readerType);
-                }
+                LogParameterAccepted(parameterName, parameterValue, reader.GetType().Name);
 
                 if (!reader.AllowEmptyValue && string.IsNullOrEmpty(parameterValue))
                 {
@@ -78,8 +74,7 @@ public void ReadAll(DisableQueryStringAttribute? disableQueryStringAttribute)
         }
     }
 
-    [LoggerMessage(Level = LogLevel.Debug, SkipEnabledCheck = true,
-        Message = "Query string parameter '{ParameterName}' with value '{ParameterValue}' was accepted by {ReaderType}.")]
+    [LoggerMessage(Level = LogLevel.Debug, Message = "Query string parameter '{ParameterName}' with value '{ParameterValue}' was accepted by {ReaderType}.")]
     private partial void LogParameterAccepted(string parameterName, StringValues parameterValue, string readerType);
 
     [LoggerMessage(Level = LogLevel.Debug, Message = "Query string parameter '{ParameterName}' was successfully read.")]