2011-11-10 20 views
2

私は、住所と人が2回、「メイン」連絡先に1回、「請求書」連絡先に1回、InvoiceContactSameAsMainというブール値を持っています - 不器用な名前ですが説明的です。プロパティのgetterは、 "main"と "invoice"のAddressオブジェクトとContactオブジェクトが同じかどうかを調べ、一致する場合はtrueを返します。設定者は、値が真であるかどうかを調べ、そうであれば、主要人物を請求書担当者に、メインアドレスを請求書アドレスにコピーします。MVC3コントローラでモデルの再検証を強制する方法はありますか?

私のビューでは、ブール値は(期待どおりの)チェックボックスで表されます。これには小さなJS機能が付いています。このチェックボックスをオンにすると、請求書フィールドが非表示になり、データ値のHTML属性をfalseに設定して、クライアント側の検証を「オフ」にし、邪魔にならないように再解析しますフォーム内の検証属性。ボックスのチェックを外すと、フィールドが自然に表示され、検証が再びオンになります。

コントローラに到達するまで、これはすべて正常に動作します。

モデルが「有効」で、正しいフィールド(InvoiceContactSameAsMainセッターのおかげで)を含んでいるにもかかわらず、ModelState.IsValidは間違ったままであり、モデルの再検証を行う方法が見つからないようです。 ModelStateをクリアすると、すべてのエラーが消えます。 PersonとAddressオブジェクトはプロジェクト全体で使用され、ある時点で変更または拡張が必要な​​場合があるため、ModelStateのフィールドを名前で掘り下げることは避けています。

私がModelStateを再検証できるようにここで欠けていることがありますか? TryUpdateModelとTryValidateModelを試しましたが、どちらもキャッシュされたModelState値を使用しているようです。私は再帰的に "固定"モデルを渡し、私のアクションを再呼び出ししようとしました。私はほとんど働かなかったことに感謝しています。

これ以上の詳細や例が役立つかどうか教えてください。

編集:明らかに、これが問題に近づくのが間違っている場合は、私に知らせてください。

編集2:Ron Sijmの提案に従ってコードサンプルを追加しました。

モデルは次のとおりです。 public class Details { public int?ユーザーID {get;セット; }

public Company Company { get; set; } 

    public Address CompanyAddress { get; set; } 
    public Person MainPerson { get; set; } 

    public Address InvoiceAddress { get; set; } 
    public Person InvoiceContact { get; set; } 

    [Display(Name = "Promotional code")] 
    [StringLength(20, ErrorMessage = "Promotional code should not exceed 20 characters")] 
    public string PromotionalCode { get; set; } 

    [Display(Name = "Invoice contact same as main")] 
    public bool InvoiceContactSameasMain 
    { 
     get { return InvoiceContact.Equals(MainPerson); } 
     set 
     { 
      if (value) 
      { 
       InvoiceContact = MainPerson.Copy(); 
       InvoiceAddress = CompanyAddress.Copy(); 
      } 
     } 
    } 

    [_Common.MustAccept] 
    [Display(Name = "I agree with the Privacy Policy")] 
    public bool PrivacyFlag { get; set; } 

    [Display(Name = "Please subscribe to Sodexo News Letter")] 
    public bool MarketingOption { get; set; } 

    [Display(Name = "Contract number")] 
    public int? ContractNumber { get; set; } 

    public Details() 
    { 
     Company = new Company(); 
     CompanyAddress = new Address(); 
     MainPerson = new Person(); 
     InvoiceAddress = new Address(); 
     InvoiceContact = new Person(); 
    } 
} 

これは、ページに関与SelectListsの数があるとのViewModelに包まれて、次のよう

public class DetailsViewModel 
{ 
    public Details Details    { get; set; } 
    public SelectList MainContactTitles { get; set; } 
    public SelectList InvoiceContactTitles { get; set; } 
    public SelectList SICCodes    { get; set; } 
    public SelectList TypesOfBusiness  { get; set; } 
    public SelectList NumbersOfEmployees { get; set; } 

    public DetailsViewModel() 
    { 
    } 
} 

[コントローラの2つの関連アクションは次のとおりです。

public class DetailsController : _ClientController 
{ 
    [Authorize] 
    public ActionResult Index() 
    { 
     DetailsViewModel viewModel = new DetailsViewModel(); 
     if (Client == null) 
     { 
      viewModel.Details = DetailsFunctions.GetClient((int)UserId, null); 
     } 
     else 
     { 
      viewModel.Details = DetailsFunctions.GetClient((int)UserId, Client.ContractNumber); 
     } 
     viewModel.MainContactTitles = DetailsFunctions.GetTitles((int)UserId, viewModel.Details.MainPerson.title); 
     viewModel.InvoiceContactTitles = DetailsFunctions.GetTitles((int)UserId, viewModel.Details.InvoiceContact.title); 
     viewModel.SICCodes = DetailsFunctions.GetSICCodes(viewModel.Details.Company.sic_code); 
     viewModel.NumbersOfEmployees = DetailsFunctions.GetNumbersOfEmployees(viewModel.Details.Company.number_of_employees); 
     viewModel.TypesOfBusiness = DetailsFunctions.GetTypesOfBusiness(viewModel.Details.Company.public_private); 
     return View(viewModel); 
    } 

    [Authorize] 
    [HttpPost] 
    public ActionResult Index(DetailsViewModel ViewModel) 
    { 
     if (ModelState.IsValid) 
     { 
      //go to main page for now 
      DetailsFunctions.SetClient((int)UserId, ViewModel.Details); 
      return RedirectToAction("Index", "Home"); 
     } 
     else 
     { 
      ViewModel.MainContactTitles = DetailsFunctions.GetTitles((int)UserId, ViewModel.Details.MainPerson.title); 
      ViewModel.InvoiceContactTitles = DetailsFunctions.GetTitles((int)UserId, ViewModel.Details.InvoiceContact.title); 
      ViewModel.SICCodes = DetailsFunctions.GetSICCodes(ViewModel.Details.Company.sic_code); 
      ViewModel.NumbersOfEmployees = DetailsFunctions.GetNumbersOfEmployees(ViewModel.Details.Company.number_of_employees); 
      ViewModel.TypesOfBusiness = DetailsFunctions.GetTypesOfBusiness(ViewModel.Details.Company.public_private); 
      return View(ViewModel); 
     } 
    } 
} 

私は必要に応じてビューとJSを提供することができますが、Modelバインディングがすべて正常に機能しているので、どれだけの助けが得られるかわかりません。

+1

あなたが持っているコードサンプルが役に立ちそうです –

+0

デバッグ時に 'ModelState.IsValid'がなぜ偽であるのか分かります。いくつかのノードを展開すると、どのルールが失敗したかがわかります。 – Darcy

+0

@ダーシー:失敗したルールは、コピーされた請求書アドレスと個人のルールです。バリデーターはモデルで正しく記入されているとはいえ、空白と見なします。 –

答えて

3

それは適度がらくたハックですが、私はちょうどModelState.IsValidをチェックする前に、コントローラ内の関連する分野のためにModelStateエラーをクリアしてしまっている:

if(ViewModel.Details.InvoiceContactSameasMain) 
     { 
      //iterate all ModelState values, grabbing the keys we want to clear errors from 
      foreach (string Key in ModelState.Keys) 
      { 
       if (Key.StartsWith("Details.InvoiceContact") || Key.Startwith("Details.InvoiceAddress")) 
       { 
        ModelState[Key].Errors.Clear(); 
       } 
      } 
     } 

人かいる場合のみ、逆さまであり、アドレスオブジェクトが変更されると、このコードは変更する必要はありません。

+0

+1これは私が見つけた作業ソリューションに似ている唯一のものです非常に似た問題。良い方法があるはずだが、私はそれが何であるかわからない。 – Corin

関連する問題