Skip to content

DefaultBodyModelValidator extensibility #9

Closed
@Eilon

Description

@Eilon

Original PR: https://aspnetwebstack.codeplex.com/SourceControl/network/forks/abatishchev/modelvalidation2/contribution/8644


Opening DefaultBodyModelValidator for changes via inheritance and overriding of its private (now protected virtual) method.

As described in the following issue for Fluent Validation Web API integration package FluentValidation/FluentValidation#178 (comment), FV makes small changes in DFMV but has to copy-paste the whole class as-is because it's barely extendable having all method as private.

This pull request changes them to protected virtual with minimal related changes (few nested private methods should be now protected, one internal interface - public).

Please review!

MPSIT Engineering extensively uses ASP.NET Web API for current and new projects, and relies on Fluent Validation to validate input. But unfortunately we're unable to change some of the behaviors without copy-pasting huge pieces of code, what would be perfectly avoidable with the proposed change.

Thanks,
Alex

Metadata

Metadata

Assignees

Type

No type

Projects

No projects

Relationships

None yet

Development

No branches or pull requests

Issue actions