The value '' is invalid

Cześć,

Korzystam z Razor Pages w swoim projekcie. Mam jednak problem ze zwalidowaniem modelu, który przekazuję z widoku do kontrolera:

@page
@model MealApp.Pages.Meals.AddModel
@{
    <h1>Dodaj posiłek</h1>

    <form method="post" class="mt-3">
        <input type="hidden" asp-for="Meal.Id" />
        <div class="form-group row">
            <label asp-for="Meal.MealName" class="col-sm-2 col-form-label">
            </label>
            <div class="col-sm-10">
                <input asp-for="Meal.MealName" class="form-control" placeholder="Nazwa posiłku">
            </div>
        </div>
        <div class="form-group row">
            <label asp-for="Meal.Type" class="col-sm-2 col-form-label">
            </label>
            <div class="col-sm-10">
                @Html.DropDownListFor(p => p.Meal.Type.Id, new SelectList(Model.Types, "Id", "TypeName"), "--Wybierz typ posiłku--", new { @class = "form-control" })
            </div>
        </div>
        <div class="form-group row">
            <label asp-for="Meal.Description" class="col-sm-2 col-form-label">
            </label>
            <div class="col-sm-10">
                <input asp-for="Meal.Description" class="form-control" placeholder="Opis">
            </div>
        </div>
        <div id="myTable" class="datatable" data-hover="true" data-border-color="dark ">
            <table class="table datatable-table">
                <thead class="datatable-header">
                    <tr>
                        <th>Produkt</th>
                        <th>Ilość</th>
                    </tr>
                </thead>
                <tbody class="datatable-body">
                    <tr>
                        <td>
                            @Html.DropDownListFor(p => p.Meal.MealProducts[0].ProductId, new SelectList(Model.Products, "Id", "ProductName"), "--Wybierz produkt--", new { @class = "form-control" })
                        </td>
                        <td>
                            <input asp-for="Meal.MealProducts[0].Quantity" class="form-control" placeholder="Ilość" />
                        </td>
                    </tr>
                </tbody>
            </table>
        </div>
        <button type="button" id="add-row" class="btn btn-primary">Dodaj produkt</button>
        <div class="form-group row">
            <div class="col-sm-10">
                <button type="submit" class="btn btn-primary">Dodaj</button>
                <a asp-page="/Products/Index" class="btn btn-danger">Cancel</a>
            </div>
        </div>
    </form>
}

Sprawa rozchodzi się o parametr "meal.id", który jest ukryty. Przy debugowaniu zauważyłem, że do kontrolera wpada wartość meal.Id=0, ale co ciekawe chwilę później przy walidacji modelu otrzymuję komunikat jak w tytule i na poniższym screenie:

W MVC nie miałem z tym problemu, a przy Razor Pages jakby czegoś brakowało lub coś zostało po mojej stronie źle skonfigurowane. W czym może być problem?
Dorzucam jeszcze model:

public class Meal
    {
        public int Id { get; set; }
        public TypeOfMeal Type { get; set; }
        public string MealName { get; set; }
        public string Description { get; set; }
        public List<MealProduct> MealProducts { get; set; }
    }

Zrzut ekranu sugeruje, że formularz nie wysłał wartości meal.id. Jak to pole zostało wygenerowane w przeglądarce? Ma nazwę? Co dostajesz potem od przeglądarki w żądaniu?

W przeglądarce wygląda tak:

<input type="hidden" data-val="true" data-val-required="The Id field is required." id="Meal_Id" name="Meal.Id" value="">

Po wysłaniu żądania POST, otrzymuję meal.Id = 0 w modelu.

Krispekowy napisał(a):

Po wysłaniu żądania POST, otrzymuję meal.Id = 0 w modelu.

To nie jest to, o co pytam. Co leci po sieci? Jeżeli Id jest intem, to zawsze będzie tam zero, gdy wartość z przeglądarki przyjdzie pusta.

No i po stronie przeglądarki wartość jest pusta (value = ""). Co tam powinno być według Ciebie? Co wysyłasz ze swojego modelu?

@Afish: jest to formatka dodawania nowego obiektu, więc w modelu teoretycznie id może być puste, nada się ono w momencie dodawania do bazy danych (przynajmniej tak to wyglądało w moim innym projekcie w MVC)

Krispekowy napisał(a):

w modelu teoretycznie id może być puste

Jeżeli id jest liczbą, to nie może być puste, więc tu jest pierwsza niezgodność, że na widoku nie ma wartości. Potem przeglądarka odsyła pusty string i mechanizm sprawdzający poprawność danych słusznie krzyczy, że pusty tekst nie jest poprawną liczbą. Jak dla mnie albo razor pages działa zupełnie inaczej od poprzednich rozwiązań (nie znam razor pages, więc nie mogę powiedzieć na pewno), albo coś skopałeś przy obsłudze modeli.

Pokaż klasę AddModel i kod kontrolerów (zarówno geta wyświetlającego formatkę jak i tego obsługującego post,).

@Afish:

    public class AddModel : PageModel
    {
        private readonly IMealRepository mealRepository;
        private readonly IProductRepository productRepository;
        private readonly ITypeOfMealRepository typeOfMealRepository;

        public Meal Meal { get; set; }
        public IEnumerable<Product> Products { get; set; }
        public IEnumerable<TypeOfMeal> Types { get; set; }
        public AddModel(IMealRepository mealRepository,
            IProductRepository productRepository,
            ITypeOfMealRepository typeOfMealRepository)
        {
            this.mealRepository = mealRepository;
            this.productRepository = productRepository;
            this.typeOfMealRepository = typeOfMealRepository;
        }
        public IActionResult OnGet()
        {
            Types = typeOfMealRepository.GetAllTypes();
            Products = productRepository.GetAllProducts();
            return Page();
        }
        public IActionResult OnPost(Meal meal)
        {
            Meal = meal;
            if (ModelState.IsValid)
            {
                mealRepository.AddMeal(Meal);
                return RedirectToPage("Index");
            }
            else
            {
                return RedirectToPage("Error");
            }
        }
    }

Na oko meal jest nullem, więc wszystko ma sens. Na widok leci brak wartości, przeglądarka odsyła pusty tekst, validator krzyczy, że pusty tekst nie jest liczbą. Albo zrób tak, żeby w widoku była wartość, albo zmień validator, żeby pusty string był akceptowany. Póki co chyba działa as designed.

W public IActionResult OnGet() zwracasz widok. Do widoku zostaje przekazany AddModel. Z racji tego, że właściwość public Meal Meal { get; set; } w tym momencie jest równa NULL, to też <input type="hidden" asp-for="Meal.Id" /> ma nieprawidłową wartość. Są 2 rozwiązania tej sytuacji:

1. W akcji public IActionResult OnGet() definiujesz instancję dla public Meal Meal { get; set; } określając z góry parametr Id.
2. Usuwasz z widoku <input type="hidden" asp-for="Meal.Id" />, a następnie w akcji public IActionResult OnPost(Meal meal) określasz nową wartość dla meal.Id

@Grzegorz Świdwa: Dzięki! Usunąłem z widoku ten input z id i hula elegancko :)