Propertygrid user input data validation issue
まだ誰も着手していません。
評価
調査の方向性
Start with the supplied WPF reproduction at XctkPropertyGrid and PropertyGrid_Loaded, using the Person and Person2 examples. Trace how PropertyGrid handles IDataErrorInfo and DataAnnotations validation during editing and focus loss. No repository file or test is identified; done means invalid input shows appropriate feedback and the error state clears when the value becomes valid.
索引モデルが issue の本文から書いたものです。
説明
question1: [Range()] causes the UI to have no user error prompts
//test example as follow:
using System;
using System.Collections.Generic;
using System.ComponentModel;
using System.ComponentModel.DataAnnotations;
using System.Linq;
using System.Runtime.CompilerServices;
namespace WpfApp1
{
public class Person: INotifyPropertyChanged, IDataErrorInfo
{
[Required(ErrorMessage = "Name is Required")]
[DisplayName("Name")]
public string Name { set; get; }
[Range(0,150,ErrorMessage = "Age= 0~150")]
[DisplayName("Age")]
public int Age { set; get; }
public event PropertyChangedEventHandler? PropertyChanged;
[Browsable(false)]
public string Error { get; }
public string this[string propertyName]
{
get
{
if (string.IsNullOrEmpty(propertyName))
{
throw new ArgumentException("Invalid property name", propertyName);
}
string error = string.Empty;
var val = GetType().GetProperty(propertyName).GetValue(this, null);
var results = new List<System.ComponentModel.DataAnnotations.ValidationResult>(1);
var result = Validator.TryValidateProperty(val, new ValidationContext(this, null, null)
{ MemberName = propertyName }, results);
if (!result)
{
var validationResult = results.First();
error = validationResult.ErrorMessage;
}
return error;
}
}
protected void RaisePropertyChanged([CallerMemberName] string propertyName = null) => this.OnPropertyChanged(new PropertyChangedEventArgs(propertyName));
protected virtual void OnPropertyChanged(PropertyChangedEventArgs args)
{
PropertyChangedEventHandler propertyChanged = this.PropertyChanged;
if (propertyChanged == null)
return;
propertyChanged((object)this, args);
}
}
}
<xctk:PropertyGrid Name="XctkPropertyGrid" Loaded="PropertyGrid_Loaded"></xctk:PropertyGrid>
private void PropertyGrid_Loaded(object sender, RoutedEventArgs e)
{
Person p = new Person(){Age = 20,Name = "wala"};
this.XctkPropertyGrid.SelectedObject = p;
}
[Range (0150, ErrorMessage="Age=0-150")] can prevent users from entering illegal values, but there is no red box mark or error prompt. Debugging found that this [string propertyName] indexer has not been called because it prevents users from entering illegal values.
question2:use person2 insteadof person1,When the user inputs an incorrect value, a red boder will prompt, but when the user leaves the attribute input box, the value in the red boder will return to the legal value, but the red boder will not disappear.
public class Person2 : INotifyPropertyChanged, IDataErrorInfo
{
[DisplayName("Name")]
public string Name { set; get; }
private int _age;
[DisplayName("Age")]
public int Age {
set
{
if (!ValidataProperty(nameof(Age), value))
return;
_age = value;
RaisePropertyChanged(nameof(Age));
}
get
{
return _age;
}}
public event PropertyChangedEventHandler? PropertyChanged;
[Browsable(false)]
public string Error { get; }
protected bool ValidataProperty(string propertyName, object val)
{
if (propertyName.Equals(nameof(Age)))
{
int ageVal = (int)val;
if (ageVal < 0 || ageVal > 150)
{
ErrorProperty = propertyName;
ErrorMsg = "Age =0~150";
return false;
}
}
ErrorMsg = string.Empty;
ErrorProperty = string.Empty;
return true;
}
private string ErrorProperty;
private string ErrorMsg;
public string this[string propertyName]
{
get
{
if (ErrorProperty.Equals(propertyName))
{
return ErrorMsg;
}
return string.Empty;
}
}
protected void RaisePropertyChanged([CallerMemberName] string propertyName = null) => this.OnPropertyChanged(new PropertyChangedEventArgs(propertyName));
protected virtual void OnPropertyChanged(PropertyChangedEventArgs args)
{
PropertyChangedEventHandler propertyChanged = this.PropertyChanged;
if (propertyChanged == null)
return;
propertyChanged((object)this, args);
}
}
- 主要言語
- C#
- スター
- 4.2k
- フォーク
- 912
- PR マージ指標
- 30日以内にマージされた PR はありません
コントリビューションガイド
このリポジトリのコントリビューションガイドは索引されていません
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
xceedsoftware/wpftoolkit のほかの issue
-
難易度 4/5 3〜5日 初心者へのやさしさ 35/100
xceedsoftware/wpftoolkit#1808 · コメント 1 件 ·
-
難易度 3/5 1〜2日 初心者へのやさしさ 48/100
xceedsoftware/wpftoolkit#1807 · コメント 2 件 ·
-
難易度 3/5 1〜2日 初心者へのやさしさ 35/100
xceedsoftware/wpftoolkit#1806 · コメント 1 件 ·
-
難易度 1/5 1時間未満 初心者へのやさしさ 45/100
xceedsoftware/wpftoolkit#1804 · コメント 1 件 ·
-
難易度 3/5 1〜2日 初心者へのやさしさ 45/100
xceedsoftware/wpftoolkit#1802 · コメント 3 件 ·
xceedsoftware/wpftoolkit の issue をすべて見る
似ている issue
-
untriaged
難易度 1/5 1時間未満 初心者へのやさしさ 88/100
dotnet/dotnet-api-docs#13095 ·
-
area-deployment area-integrations triage:bot-seen
難易度 2/5 半日 初心者へのやさしさ 86/100
-
type/automation type/tech-debt
難易度 2/5 1〜3時間 初心者へのやさしさ 84/100
-
bug
難易度 1/5 1時間未満 初心者へのやさしさ 90/100
newrelic/newrelic-dotnet-agent#3850 · コメント 1 件 ·
-
難易度 2/5 1〜3時間 初心者へのやさしさ 88/100
LuckyPennySoftware/AutoMapper#4660 ·