fix: add SetIdentityInsert method to handle manual ID insertion in Pa…
716 new problems found by Qodana for .NET
Qodana for .NET
716 new problems were found
| Inspection name | Severity | Problems |
|---|---|---|
Invalid property value |
🔴 Failure | 122 |
Unresolved custom property |
🔴 Failure | 8 |
Invalid pseudo-selector |
🔴 Failure | 5 |
Annotator |
🔴 Failure | 1 |
Redundant measure unit |
🔶 Warning | 36 |
Auto-property accessor is never used (non-private accessibility) |
🔶 Warning | 35 |
Redundant character escape |
🔶 Warning | 27 |
Unnecessary non-capturing group |
🔶 Warning | 15 |
Missing generic font family name |
🔶 Warning | 14 |
Overwritten property |
🔶 Warning | 8 |
Unknown property |
🔶 Warning | 8 |
Namespace does not correspond to file location |
🔶 Warning | 7 |
Dereference of a possibly null reference. |
🔶 Warning | 6 |
Possible null reference argument for a parameter. |
🔶 Warning | 5 |
Non-nullable member is uninitialized. |
🔶 Warning | 5 |
Expression is always 'true' or 'false' according to nullable reference types' annotations |
🔶 Warning | 5 |
Conditional access qualifier expression is not null according to nullable reference types' annotations |
🔶 Warning | 5 |
Possible performance issues caused by unlimited string length |
🔶 Warning | 5 |
Conditional access qualifier expression is known to be null or not null |
🔶 Warning | 3 |
Begin or end anchor in unexpected position |
🔶 Warning | 3 |
Unused parameter (private accessibility) |
🔶 Warning | 3 |
Variable can be declared as non-nullable |
🔶 Warning | 3 |
Async function without await expression |
🔶 Warning | 2 |
Unknown ID |
🔶 Warning | 2 |
Missing required 'lang' attribute |
🔶 Warning | 2 |
Invalid XML documentation comment |
🔶 Warning | 2 |
Non-accessed field (private accessibility) |
🔶 Warning | 2 |
Possible multiple enumeration |
🔶 Warning | 2 |
Annotator |
🔶 Warning | 1 |
Possible null reference return. |
🔶 Warning | 1 |
Expression is always 'true' or always 'false' |
🔶 Warning | 1 |
Unused selector |
🔶 Warning | 1 |
Empty statement is redundant |
🔶 Warning | 1 |
Unknown attribute |
🔶 Warning | 1 |
MVC (unknown view) |
🔶 Warning | 1 |
Non-accessed local variable only used to discard the 'out' parameter value |
🔶 Warning | 1 |
Not accessed primary constructor parameter |
🔶 Warning | 1 |
Redundant argument with default value |
🔶 Warning | 1 |
Duplicate character in character class |
🔶 Warning | 1 |
Type member is never used (private accessibility) |
🔶 Warning | 1 |
Unused local variable |
🔶 Warning | 1 |
Variable in local function hides variable from outer scope |
🔶 Warning | 1 |
Use preferred 'var' style (when type is simple) |
◽️ Notice | 106 |
Property can be made init-only (non-private accessibility) |
◽️ Notice | 45 |
Use preferred 'var' style (for built-in types) |
◽️ Notice | 39 |
Use preferred 'var' style (elsewhere) |
◽️ Notice | 34 |
Non-integer length in pixels |
◽️ Notice | 20 |
Replace with primary constructor parameter |
◽️ Notice | 18 |
RoslynAnalyzers Template should be a static expression |
◽️ Notice | 13 |
Invert 'if' statement to reduce nesting |
◽️ Notice | 11 |
Regular expression can be simplified |
◽️ Notice | 8 |
Type member is never used (non-private accessibility) |
◽️ Notice | 7 |
Redundant verbatim string prefix |
◽️ Notice | 6 |
Route parameter constraint can be added due to type of method argument |
◽️ Notice | 6 |
Use collection expression syntax |
◽️ Notice | 6 |
Convert constructor into primary constructor |
◽️ Notice | 4 |
Member can be made private (non-private accessibility) |
◽️ Notice | 4 |
RoslynAnalyzers Unused route parameter |
◽️ Notice | 3 |
Class is never instantiated (non-private accessibility) |
◽️ Notice | 3 |
Convert local variable or field into constant (private accessibility) |
◽️ Notice | 3 |
Not all route parameters are passed to method |
◽️ Notice | 3 |
Route parameter can be passed to method |
◽️ Notice | 3 |
'if' statement can be rewritten as '??=' assignment |
◽️ Notice | 2 |
Loop can be converted into LINQ-expression |
◽️ Notice | 2 |
RoslynAnalyzers Instantiate argument exceptions correctly |
◽️ Notice | 1 |
RoslynAnalyzers Named placeholders should not be numeric values |
◽️ Notice | 1 |
Class with virtual (overridable) members never inherited (non-private accessibility) |
◽️ Notice | 1 |
Convert 'if' statement into 'switch' |
◽️ Notice | 1 |
Convert into lambda expression |
◽️ Notice | 1 |
Inline temporary variable |
◽️ Notice | 1 |
Member can be made static (shared) (non-private accessibility) |
◽️ Notice | 1 |
Merge conditional ?: expression into conditional access |
◽️ Notice | 1 |
Merge null/pattern/value checks into 'or'/'and' patterns |
◽️ Notice | 1 |
Merge null/pattern checks into complex pattern |
◽️ Notice | 1 |
Parameter's type can be made stricter due to applied constraints |
◽️ Notice | 1 |
Local variable has too wide declaration scope |
◽️ Notice | 1 |
Type member is never accessed via base type (non-private accessibility) |
◽️ Notice | 1 |
Unused parameter (non-private accessibility) |
◽️ Notice | 1 |
Replace if statement with null-propagating code |
◽️ Notice | 1 |
☁️ View the detailed Qodana report
Contact Qodana team
Contact us at [email protected]
- Or via our issue tracker: https://jb.gg/qodana-issue
- Or share your feedback: https://jb.gg/qodana-discussions
Details
This result was published with Qodana GitHub Action
Annotations
Check notice on line 26 in Frontend/Controllers/HomeController.cs
github-actions / Qodana for .NET
RoslynAnalyzers Unused route parameter
Unused route parameter 'id'
Check notice on line 171 in Frontend/Controllers/PatientsController.cs
github-actions / Qodana for .NET
RoslynAnalyzers Unused route parameter
Unused route parameter 'id'
Check notice on line 161 in Frontend/Controllers/NotesController.cs
github-actions / Qodana for .NET
RoslynAnalyzers Unused route parameter
Unused route parameter 'id'
Check warning on line 1429 in Frontend/wwwroot/lib/jquery-validation/dist/additional-methods.js
github-actions / Qodana for .NET
Annotator
Redundant group nesting
Check failure on line 1346 in Frontend/wwwroot/lib/jquery/dist/jquery.js
github-actions / Qodana for .NET
Annotator
Unexpected token
Check notice on line 24 in ApiGateway/Program.cs
github-actions / Qodana for .NET
RoslynAnalyzers Instantiate argument exceptions correctly
Method $ passes 'secretKey' as the paramName argument to a ArgumentNullException constructor. Replace this argument with one of the method's parameter names. Note that the provided parameter name should have the exact casing as declared on the method.
Check notice on line 37 in Frontend/Controllers/HomeController.cs
github-actions / Qodana for .NET
RoslynAnalyzers Named placeholders should not be numeric values
Named placeholders in the logging message template should not be comprised of only numeric characters
github-actions / Qodana for .NET
RoslynAnalyzers Template should be a static expression
The logging message template should not vary between calls to 'LoggerExtensions.LogDebug(ILogger, string?, params object?\[\])'
Check notice on line 15 in Frontend/Services/ErrorHandling.cs
github-actions / Qodana for .NET
RoslynAnalyzers Template should be a static expression
The logging message template should not vary between calls to 'LoggerExtensions.LogError(ILogger, string?, params object?\[\])'
Check notice on line 89 in BackendNote/Data/DataSeeder.cs
github-actions / Qodana for .NET
RoslynAnalyzers Template should be a static expression
The logging message template should not vary between calls to 'LoggerExtensions.LogDebug(ILogger, string?, params object?\[\])'
Check notice on line 42 in Auth/Data/DataSeeder.cs
github-actions / Qodana for .NET
RoslynAnalyzers Template should be a static expression
The logging message template should not vary between calls to 'LoggerExtensions.LogInformation(ILogger, string?, params object?\[\])'
Check notice on line 105 in Auth/Data/DataSeeder.cs
github-actions / Qodana for .NET
RoslynAnalyzers Template should be a static expression
The logging message template should not vary between calls to 'LoggerExtensions.LogInformation(ILogger, string?, params object?\[\])'
Check notice on line 101 in Auth/Data/DataSeeder.cs
github-actions / Qodana for .NET
RoslynAnalyzers Template should be a static expression
The logging message template should not vary between calls to 'LoggerExtensions.LogInformation(ILogger, string?, params object?\[\])'
github-actions / Qodana for .NET
RoslynAnalyzers Template should be a static expression
The logging message template should not vary between calls to 'LoggerExtensions.LogInformation(ILogger, string?, params object?\[\])'
Check notice on line 27 in Auth/Controllers/AuthController.cs
github-actions / Qodana for .NET
RoslynAnalyzers Template should be a static expression
The logging message template should not vary between calls to 'LoggerExtensions.LogInformation(ILogger, string?, params object?\[\])'
Check notice on line 48 in Auth/Data/DataSeeder.cs
github-actions / Qodana for .NET
RoslynAnalyzers Template should be a static expression
The logging message template should not vary between calls to 'LoggerExtensions.LogInformation(ILogger, string?, params object?\[\])'
Check notice on line 68 in Auth/Data/DataSeeder.cs
github-actions / Qodana for .NET
RoslynAnalyzers Template should be a static expression
The logging message template should not vary between calls to 'LoggerExtensions.LogInformation(ILogger, string?, params object?\[\])'
Check notice on line 74 in Auth/Data/DataSeeder.cs
github-actions / Qodana for .NET
RoslynAnalyzers Template should be a static expression
The logging message template should not vary between calls to 'LoggerExtensions.LogInformation(ILogger, string?, params object?\[\])'
Check notice on line 93 in BackendNote/Data/DataSeeder.cs
github-actions / Qodana for .NET
RoslynAnalyzers Template should be a static expression
The logging message template should not vary between calls to 'LoggerExtensions.LogDebug(ILogger, string?, params object?\[\])'
Check notice on line 110 in Auth/Data/DataSeeder.cs
github-actions / Qodana for .NET
RoslynAnalyzers Template should be a static expression
The logging message template should not vary between calls to 'LoggerExtensions.LogError(ILogger, string?, params object?\[\])'
Check warning on line 62 in Frontend/Services/PatientService.cs
github-actions / Qodana for .NET
Async function without await expression
This async method lacks 'await' operators and will run synchronously. Consider using the 'await' operator to await non-blocking API calls, or 'await Task.Run(...)' to do CPU-bound work on a background thread.
Check warning on line 157 in Frontend/Controllers/AuthController.cs
github-actions / Qodana for .NET
Async function without await expression
This async method lacks 'await' operators and will run synchronously. Consider using the 'await' operator to await non-blocking API calls, or 'await Task.Run(...)' to do CPU-bound work on a background thread.
Check warning on line 32 in Auth/Services/JwtService.cs
github-actions / Qodana for .NET
Dereference of a possibly null reference.
Dereference of a possibly null reference
Check warning on line 105 in BackendDiabetesRiskPrediction/Services/ElasticsearchService.cs
github-actions / Qodana for .NET
Dereference of a possibly null reference.
Dereference of a possibly null reference
Check warning on line 39 in Frontend/Views/Patients/Details.cshtml
github-actions / Qodana for .NET
Dereference of a possibly null reference.
Dereference of a possibly null reference