DataAnnotations のエラーメッセージの複合書式指定 ({0} {1} と書くやつ) の対応について

[onodera-sf]

お世話になります、おのでらです。

実装されているかどうかが調べても分からなかったので要望として書かさせていただきます。(すでに実装されているのであればすみません)

ViewModel のプロパティを ReactiveProperty で実装しているのですが、値の検証に DataAnnotations を使用しています。その際、以下のようにエラーメッセージに名称部分を任意に設定したいと思っております。

[Required(ErrorMessage = "{0} は必須")]
[Display(Name = "Child Name")]
public ReactiveProperty<string> Name { get; set; }

ASP.NET MVC の場合は {0} の部分にプロパティ名、または Display 属性を指定していればその名称がメッセージに反映されるのですが、ReactiveProperty では「Value」と表示されてしまいます。

また、上記は文字列を直接入力していますが、Resx を使用したいので以下のように設定した場合でも複合書式指定を反映させたいです。

[Required(ErrorMessageResourceType=typeof(MyApp.Resources), ErrorMessageResourceName="Required")]
[Display(Name = "Name", ResourceType = typeof(MyApp.Resources))]
public ReactiveProperty<string> Name { get; set; }

ViewModel のコンストラクタで

NameValidate = Name.ObserveErrorChanged
  .Select(x => x?.Cast<string>()?.FirstOrDefault())
  .ToReadOnlyReactiveProperty();

の箇所をうまく調整すればいけるのではと思ったのですがどう記述すればスマートなのか分からなかったので公式の見解をいただきたいと思います。
よろしくお願いします。

[runceel]

今の状態だと恐らく対応してないと思います。
DataAnnotations によるバリデーションは、ReactivePropertyのValueプロパティに対して ValidationAttribute 型のものだけを抜き出してバリデーションを実行しています。

コード的には以下の部分が該当します。

https://github.com/runceel/ReactiveProperty/blob/master/Source/ReactiveProperty.NETStandard/ReactivePropertyExtensions.cs#L22

ReactiveProperty として対応するとしたら、いじるのはここらへんになると思うのですが、どうやるのがいいかはちょっとぱっとは出てきません。

[soi013]

横からすいません、ValidationContextのDisplayNameを変更するというのはどうでしょうか？

public static ReactiveProperty<T> SetValidateAttribute<T>(this ReactiveProperty<T> self, Expression<Func<ReactiveProperty<T>>> selfSelector)
{
    var memberExpression = (MemberExpression)selfSelector.Body;
    var propertyInfo = (PropertyInfo)memberExpression.Member;
    var attrs = propertyInfo.GetCustomAttributes<ValidationAttribute>().ToArray();

    //Add
    var dispAttr = propertyInfo.GetCustomAttributes<DisplayAttribute>().FirstOrDefault();

    var context = new ValidationContext(self)
    {
        //Add
        DisplayName = dispAttr?.Name ?? propertyInfo.Name,
        MemberName = nameof(ReactiveProperty<T>.Value)
    };

    if (attrs.Length != 0)
    {
        self.SetValidateNotifyError(x =>
        {
            var validationResults = new List<ValidationResult>();
            if (Validator.TryValidateValue(x, context, validationResults, attrs))
            {
                return null;
            }

            return validationResults[0].ErrorMessage;
        });
    }

    return self;
}

ただ、Annotationはあまり詳しくないので自信はないです。

[onodera-sf]

かずきさん回答ありがとうございます。
現時点では対応されていないこと承知しました。

.NET Framework のコードを見る限りでは各属性の継承元である「ValidationAttribute」クラスの「FormatErrorMessage」メソッドの第１引数に name が指定できるようになっており、そこに文字列を渡すと {0} の部分が置き換わるみたいですね。

soi013 さんのコメントの通り FormatErrorMessage メソッドには DisplayName を渡している箇所もあるようなので反映される可能性はありますね。(動作は未確認です)

https://github.com/microsoft/referencesource/tree/master/System.ComponentModel.DataAnnotations/DataAnnotations

※この issue は close していただいて構いません

[runceel]

v6.2.0 で対応してリリースしました。

[onodera-sf]

対応ありがとうございます。
nuget から v6.2.0 にアップデートして動作を確認できました。
Resource 指定からの Name 参照も問題なく動作しています。

ありがとうございました！