The value '' is invalid

The value '' is invalid
Krispekowy
  • Rejestracja:ponad 6 lat
  • Ostatnio:prawie 4 lata
  • Postów:184
0

Cześć,

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

Kopiuj
@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:
modelstate.PNG

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:

Kopiuj
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; }
    }
AF
  • Rejestracja:prawie 18 lat
  • Ostatnio:około miesiąc
0

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?

Krispekowy
  • Rejestracja:ponad 6 lat
  • Ostatnio:prawie 4 lata
  • Postów:184
0

W przeglądarce wygląda tak:

Kopiuj
<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.

AF
  • Rejestracja:prawie 18 lat
  • Ostatnio:około miesiąc
0
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?

Krispekowy
  • Rejestracja:ponad 6 lat
  • Ostatnio:prawie 4 lata
  • Postów:184
0

@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)

AF
  • Rejestracja:prawie 18 lat
  • Ostatnio:około miesiąc
0
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,).

Krispekowy
  • Rejestracja:ponad 6 lat
  • Ostatnio:prawie 4 lata
  • Postów:184
0

@Afish:

Kopiuj
    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");
            }
        }
    }
edytowany 1x, ostatnio: Krispekowy
AF
  • Rejestracja:prawie 18 lat
  • Ostatnio:około miesiąc
1

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.

Grzegorz Świdwa
Grzegorz Świdwa
  • Rejestracja:ponad 5 lat
  • Ostatnio:około 4 lata
  • Postów:385
2

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
Krispekowy
  • Rejestracja:ponad 6 lat
  • Ostatnio:prawie 4 lata
  • Postów:184
0

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

Zarejestruj się i dołącz do największej społeczności programistów w Polsce.

Otrzymaj wsparcie, dziel się wiedzą i rozwijaj swoje umiejętności z najlepszymi.