EditorValidator multiple attributes or ability to output error

[RomanZhu]

[Validate("Object Id conflict or empty (in scene) or not empty (in prefab)", nameof(IsObjectIdInvalid))]
public string ObjectId;

I often face this issue, where I need to check for multiple cases in different contexts (as an asset or in a hierarchy).
What do you imagine is a good approach for this?

I'm unsure what is better:

• the ability to have multiple [Validate] attributes
• output custom error text from inside the validation logic

[v0lt13]

I don't think you can put multiple validate attributes on one field, I can try and test something like that to see if it works well.

You could try using the Void struct to hold another Validate attribute.

[Validate(...))]
public Void validateHolder;

[RomanZhu]

I guess something like this would be more descriptive, than separate checks (that would also do the same checks)

public ValidationResult ValidatePrefabObjectId() {
            if (DirtyHelper.IsPersistent(gameObject)) {
                if (PrefabObjectId.IsNullOrWhitespace()) {
                    return ValidationResult.Error("Prefab ObjectId can't be empty");
                }
                
                if (SaveObjectPrefabs.HasConflict(this, out conflictData)) {
                    return ValidationResult.Error($"Prefab ObjectId is not unique. {conflictData}");
                }
            }
            else {
                if (PrefabObjectId.IsNullOrWhitespace())
                    return default;
                
                if (!SaveObjectPrefabs.ExistsId(PrefabObjectId)) {
                    return ValidationResult.Error("Prefab ObjectId doesn't exist");
                }
            }
            
            return  ValidationResult.Pass(); //   or     return default;
        }

but using Void to hold validators sounds like a good workaround

[v0lt13]

Yeah this is actually a good idea. I'm going to look into it.

[v0lt13]

Added it in v2.6.0

https://editorattributesdocs.readthedocs.io/en/latest/Attributes/MiscellaneousAttributes/validate.html#:~:text=function%20returning%20a-,ValidationCheck,-%3A