Project

General

Profile

bug #8214

WeaklyRelatedEntityCombobox is not validated

Added by Andreas Kohlbecker 7 months ago. Updated 6 months ago.

Status:
Closed
Priority:
Highest
Category:
cdm-vaadin
Target version:
Start date:
03/29/2019
Due date:
% Done:

100%

Severity:
major
Found in Version:
Tags:

Description

WeaklyRelatedEntityComboboxes set to required are not validated on form submission.
If affects the TaxonNamePopupEditor


Related issues

Copied to Edit - bug #8249: WeaklyRelatedEntityCombobox: validation of isRequired only after user interaction New 03/29/2019

Associated revisions

Revision ddb31e64 (diff)
Added by Andreas Kohlbecker 7 months ago

fix #8214 fixing required validation for WeaklyRelatedEntityCombobox and centralizing related messages

Revision ee53ccf7 (diff)
Added by Andreas Kohlbecker 7 months ago

fix #8214 fixing required validation for WeaklyRelatedEntityCombobox and centralizing related messages

Revision acd5f27e (diff)
Added by Andreas Kohlbecker 6 months ago

ref #8214 minor cleanup and improving a TODO comment

History

#1 Updated by Andreas Kohlbecker 7 months ago

  • Status changed from New to Resolved
  • % Done changed from 0 to 50

#2 Updated by Andreas Kohlbecker 7 months ago

  • Assignee changed from Andreas Kohlbecker to Wolf-Henning Kusber

please review

#3 Updated by Andreas Kohlbecker 7 months ago

  • Status changed from Resolved to In Progress
  • Assignee changed from Wolf-Henning Kusber to Andreas Kohlbecker
  • % Done changed from 50 to 40

the field should only be set on error if the user tries to save the editor or when the focus leaved the field. Otherwise it is not very user friendly to show validation errors initially.

#4 Updated by Andreas Kohlbecker 6 months ago

  • Target version changed from Release 5.6 to Release 5.7

#5 Updated by Andreas Kohlbecker 6 months ago

  • Copied to bug #8249: WeaklyRelatedEntityCombobox: validation of isRequired only after user interaction added

#6 Updated by Andreas Kohlbecker 6 months ago

  • Status changed from In Progress to Closed
  • Target version changed from Release 5.7 to Release 5.6
  • % Done changed from 40 to 100

closing issue after copying the additional requirement to a new ticket #8249. The solution for this additional requirement seems to be a bit time consuming, therefore I am postponing it this way.

Also available in: Atom PDF

Add picture from clipboard (Maximum size: 40 MB)