69306c43f9
Fixes #10264 [Review URL](https://review.docs.microsoft.com/en-us/aspnet/core/mvc/models/validation?view=aspnetcore-2.2&branch=pr-en-us-10390) * Not too keen on "top-level node." It isn't documented anywhere, and Google Fu wasn't much help either. I've *guessed* from the API what it means (and I could be *wrong*). * Side note: `Required` is *required* as a model property attribute to get client-side required validation to work AFAICT. Adding `Required` or `BindRequired` to the top-level node (parameter) doesn't make it work client-side. Not sure if that's an issue or not ... just a side behavior that I noticed when testing. * `jQuery.validate()` doesn't seem to be a thing any longer. JS complains that it can't find the function with the most recent jQuery validation scripts (from the current templates). Everything seems to work without it, so I nixed it. * The sample was *almost* updated for 2.2. This should take it across the finish line. 🏁 * In the OP, the dev said that they were missing `AddDataAnnotations`, but we don't doc that level of detail. If they go `AddMvcCore`, they're kind'a on their own to make sure that they have all of the bits in place. |
||
|---|---|---|
| .. | ||
| Views | ||
| ClassicMovieAttribute.cs | ||
| Genre.cs | ||
| IUserRepository.cs | ||
| MVCMovieContext.cs | ||
| Movie.cs | ||
| MovieIValidatable.cs | ||
| MoviesController.cs | ||
| Program.cs | ||
| Startup.cs | ||
| User.cs | ||
| UserRepository.cs | ||
| UsersController.cs | ||
| ValidationSample.csproj | ||