Improvement: isRequired about class-validator HOT 13 CLOSED

wouldn't @NotEmpty decorator work?

from class-validator.

I am using skipmissingproperties and want to ensure that if a attribute is set, but not required will be validated and some attributes should be required and for example type of string.
I realized that if skipmissingproperties is set the validators are not executed at all. So also a NotEmpty will not work.... Do you have a solution for this kind of issue?

I need this to ensure that only validated data is saved in my database.

from class-validator.

I realized that if skipmissingproperties is set the validators are not executed at all.

if skipmissingproperties is set then validation are not being executed only on missing properties

So, you want to use skipmissingproperties, but at same time if some property is missing, but its totally required, you want skipmissingproperties to be ignore that that property. Do I get you right?

from class-validator.

@pleerock Exactly. I know which fields have which type and i know which fields are required. Some fields i know the datatype are not required but i need to ensure their data type if they have data.

from class-validator.

okay, got you. I'll think how do add option to decorators, so it will possible and let you know

from class-validator.

Okay, thank you very much. That sounds great!

from class-validator.

I think I can do NotEmpty decorator the only decorator that will ignore skipmissingproperties option. I don't see any benefit of doing it for any other decorator, and I don't see any benifit of ignoring NotEmpty in the cases when skipmissingproperties is set to true

from class-validator.

That would be great!

I am implementing a ODM layer and i have fields which are required and some (most) fields are not required.

A typical constellation will for example be @isnotempty @ISINT.

To ensure that its not empty and if filled it should be a integer.

I am pretty sure it only makes sense for IsNotEmpty.

from class-validator.

I changed my mind and decided to introduce a new decorator, as you originally suggested, but I named it @IsDefined(). This will be available in upcoming release soon d25ce59

from class-validator.

added it in new version, you can try it by npm i class-validator@beta

from class-validator.

Wow. Thanks for this great new feature. Class-validator will be a huge part of my upcoming mongodb elasticsearch ORM - layer.

from class-validator.

thats great!

from class-validator.

fixed in 0.4.0

from class-validator.