[EF / C#] Required Attribute wordt genegeerd

[PdeBie]

Zie onderstaande code:

C#:

[Required]
[StringLength(13, ErrorMessageResourceType = typeof (ValidationMessages), ErrorMessageResourceName = "MinimumStringLength", MinimumLength = 1)]
public String ArticleCode { get; set; }

Wanneer ik het StringLength attribuut weglaat en ik schiet een object in zonder ArticleCode, krijg ik keurig een foutmelding dat ArticleCode verplicht is.

Zet ik het StringLengthAttribuut terug en schiet hetzelfde object in, wordt mijn modelstate als valid beschouwd.

Schiet mij maar lek.

Jullie een idee?

[Feanathiel]

Maak je gebruik van ValidateObject(instance, validationContext, validateAllProperties: true)? Zo niet, dan wordt alleen het eerste attribuut gevalideerd.

Via: http://stackoverflow.com/...doesnt-seem-to-be-working

[PdeBie]

Ehm nee. Mijn HttpPost methode ziet er als volgt uit. Het gaat om een Web API overigens.
En hier geeft ModelState.IsValid dus aan dat de waarde valide is. Terwijl dit dus niet het geval is.

C#:

[HttpPost]
public IHttpActionResult PostArticle(Article paramArticle)
{
    // Validate modelstate
    if (!ModelState.IsValid)
    {
        return BadRequest(ModelState);
    }

    //TODO PB: validate for under and overposting
    //TODO PB: validate values. If not valid, return result with error messages. Set validation rules in Article class

    // Store article in datacontext
    var result = Articles.DataAccessLayer.ArticleDal.AddArticle(paramArticle);

    // Log message
    result.LogApiMessages();

    // Return result
    return Ok(result);
}

--edit--

de post zoals ik deze in Fiddler doe:

Request Headers:
User-Agent: Fiddler
Host: localhost:49998
Content-Type: application/json; charset=utf-8

Request Body:
{"ArticleCode":""}

[ Voor 12% gewijzigd door PdeBie op 07-07-2014 10:38 ]

[NickThissen]

Wat gebeurd er als je het Required attribuut weglaat? Je hebt al een minlength van 1 dus het required attribuut lijkt me eigenlijk helemaal niet nodig, of wel?

[PdeBie]

Net even geprobeerd, maar ook dan zegt hij dat de modelstate valid is.

NickThissen schreef op maandag 07 juli 2014 @ 11:44:
Je hebt al een minlength van 1 dus het required attribuut lijkt me eigenlijk helemaal niet nodig, of wel?

Ik weet alleen niet of die validatie ook af gaat op het moment dat je geen ArticleCode mee geeft (under posting). Daarom dat ik die required heb toegevoegd.

[ Voor 60% gewijzigd door PdeBie op 07-07-2014 11:54 ]

[D-Raven]

Dit is standaard gedrag van de WebApi.

Wat er gebeurd is dat de var paramArticle null is, waardoor de validations op het object Article niet getriggerd worden. Daarom is je ModelState valid.

Dit is een scenario wat je zelf moet afhandelen. Of schrijf een actionfilter welke dit voor je afhandeld.

Geen idee of dit in MVC ook van dit Under Posting scenario "last" heeft overigens. Nooit tegenaan gelopen..

[ Voor 15% gewijzigd door D-Raven op 07-07-2014 12:00 ]

[PdeBie]

D-Raven schreef op maandag 07 juli 2014 @ 11:56:
Dit is standaard gedrag van de WebApi.

Wat er gebeurd is dat de var paramArticle null is, waardoor de validations op het object Article niet getriggerd worden. Daarom is je ModelState valid.

Dit is een scenario wat je zelf moet afhandelen. Of schrijf een actionfilter welke dit voor je afhandeld.

Maar paramArticle is helemaal niet null.


Ik heb overigens (puur ter test) de POST eens aangepast en in plaats van ArticleCode doe ik nu dezelfde test met de property Name (ook van het type String). Ik verwachte geen verschil en gelukkig is dit er ook niet. Maar mijn voorbeelden zijn vanaf nu dus even met de property Name, dat jullie niet denken van 'huh? was toch ArticleCode?'

Maar ik zie wel wat anders. Ik heb de validatie op de property aangepast.

Van

C#:

[Required]
[StringLength(50, ErrorMessageResourceType = typeof (ValidationMessages), ErrorMessageResourceName = "MinimumStringLength", MinimumLength = 1)]
public String Name { get; set; }

Naar

C#:

//[Required]
//[StringLength(50, ErrorMessageResourceType = typeof (ValidationMessages),
//    ErrorMessageResourceName = "MinimumStringLength", MinimumLength = 1)]
[StringLength(50, ErrorMessage = "Test foutmelding", MinimumLength = 1)]
public String Name { get; set; }

Het verschil zit hem dus in het required attribuut en in de foutmelding. De foutmelding kwam eerst uit een resource en nu is het gewoon een String. En wat denk je!?







Zet ik het required attribuut terug is de modelstate nog steeds invalid. En krijg ik keurig twee foutmeldingen in m'n response.


Zet ik de foutmelding terug naar de tekst uit de resourcefiles, dan is hij wel valid!

[ Voor 7% gewijzigd door PdeBie op 07-07-2014 12:15 ]

[D-Raven]

Gebruik je misschien een custom modelbinder ?

Verder is het "vage shit".

[ Voor 26% gewijzigd door D-Raven op 07-07-2014 13:43 ]

[PdeBie]

Nope, dit is echt recht-toe recht-aan werk.

Ik heb gewoon een klasse 'Article' met daarin wat properties en geen vage DataAnnotations of iets dergelijks.


Eerste dingetje ook wat ik maak in Web API overigens.

--edit @ vage shit regel--
Behoorlijk!

[ Voor 48% gewijzigd door PdeBie op 07-07-2014 13:45 ]

[PdeBie]

Geen idee waarom, maar na het cleanen van mijn solution en het opnieuw toevoegen van de DefaultModelBinder() aan de Application_Start() in de Global.asax werkt het ineens wel. Validatie gaat nu keurig af.

Alleen nu haalt hij de tekst nog niet uit mijn resource file, maar dat is een nieuwe uitdaging.

Bedankt voor het meedenken allemaal!