Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Closes #142 #143

Open
wants to merge 14 commits into
base: main
Choose a base branch
from
Open

Closes #142 #143

Changes from all commits
Commits
Show all changes
14 commits
Select commit Hold shift + click to select a range
10ddaee
Better validators scan
GioviQ Sep 14, 2020
9a02a79
Considering lazy loading
GioviQ Sep 21, 2020
34b0e52
Merge branch 'main' into main
chrissainty Sep 21, 2020
647e925
FluentValidation upgrade to 9.2.2
GioviQ Nov 10, 2020
2196e3e
Merge branch 'main' of https://github.com/GioviQ/FluentValidation int…
GioviQ Nov 10, 2020
58e494c
now FluentValidation RuleSet are all included in validation
GioviQ Nov 10, 2020
336fc65
Add partial validation support by RuleSets useful for EditForm fields…
GioviQ Nov 10, 2020
f9a43d7
Merge branch 'main' into main
GioviQ Nov 10, 2020
5353660
Async rule from FirstName to EmailAddress
GioviQ Nov 10, 2020
521b0e8
Merge branch 'main' of https://github.com/GioviQ/FluentValidation int…
GioviQ Nov 10, 2020
8ef6eb4
fix rare ObjectDisposedException
GioviQ Dec 11, 2020
50768a9
PersonValidator refactor
GioviQ Feb 19, 2021
a7160df
merge
GioviQ Jun 10, 2022
9cf94a5
closes #142
GioviQ Jun 10, 2022
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
12 changes: 6 additions & 6 deletions src/Blazored.FluentValidation/EditContextFluentValidationExtensions.cs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -76,7 +76,7 @@ private static async Task ValidateField(EditContext editContext,
{
var properties = new[] { fieldIdentifier.FieldName };
var context = new ValidationContext<object>(fieldIdentifier.Model, new PropertyChain(), new MemberNameValidatorSelector(properties));

validator ??= GetValidatorForModel(serviceProvider, fieldIdentifier.Model, disableAssemblyScanning);

if (validator is not null)
Expand Down Expand Up @@ -148,7 +148,7 @@ private static FieldIdentifier ToFieldIdentifier(in EditContext editContext, in

var obj = editContext.Model;
var nextTokenEnd = propertyPath.IndexOfAny(Separators);

// Optimize for a scenario when parsing isn't needed.
if (nextTokenEnd < 0)
{
Expand All @@ -168,15 +168,15 @@ private static FieldIdentifier ToFieldIdentifier(in EditContext editContext, in
// It's an indexer
// This code assumes C# conventions (one indexer named Item with one param)
nextToken = nextToken.Slice(0, nextToken.Length - 1);
var prop = obj.GetType().GetProperty("Item");
var prop = obj.GetType().GetProperties().SingleOrDefault(p => p.Name == "Item" && p.CanWrite);

if (prop is not null)
{
// we've got an Item property
var indexerType = prop.GetIndexParameters()[0].ParameterType;
var indexerValue = Convert.ChangeType(nextToken.ToString(), indexerType);
newObj = prop.GetValue(obj, new [] { indexerValue });

newObj = prop.GetValue(obj, new[] { indexerValue });
}
else
{
Expand Down Expand Up @@ -211,7 +211,7 @@ private static FieldIdentifier ToFieldIdentifier(in EditContext editContext, in
}

obj = newObj;

nextTokenEnd = propertyPathAsSpan.IndexOfAny(Separators);
if (nextTokenEnd < 0)
{
Expand Down