Bug 1129937 - YaST Configuration Management module do not validate all field types
Summary: YaST Configuration Management module do not validate all field types
Status: CONFIRMED
Alias: None
Product: Beta SUSE Linux Enterprise Server 15 SP1
Classification: SUSE Linux Enterprise Server
Component: YaST2 (show other bugs)
Version: RC 1
Hardware: x86-64 SLES 15
: P3 - Medium : Normal
Target Milestone: ---
Assignee: YaST Team
QA Contact: Jiri Srain
URL: https://trello.com/c/BdE2MUbU/
Whiteboard:
Keywords:
Depends on: 1129936
Blocks: 1129938 1129940
  Show dependency treegraph
 
Reported: 2019-03-20 14:47 UTC by Joaquín Rivera
Modified: 2019-03-27 10:45 UTC (History)
2 users (show)

See Also:
Found By: ---
Services Priority:
Business Priority:
Blocker: ---
Marketing QA Status: ---
IT Deployment: ---


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Joaquín Rivera 2019-03-20 14:47:38 UTC
During testing of jsc#SLE-2995 and playing editing form.yml to create different visualizations, it was found that:

- Validation are missing for types: `number`, `url`
- For `date` even there is a visual validation, we can put cursor at the end and go backward writing '9999-99-99'. 
- Dialog with error validation only contains the first error found, instead of a list of them.

It is easy to check using a small portion of this example:
https://github.com/yast/yast-configuration-management/blob/master/test/fixtures/formulas-ng/test-formula/form.yml

Expectation:
An error is displayed if something that is not a number is introduced in that field, the same for url. Date field should avoid that specific trick and be validated. Regarding usability, give errors one by one perhaps not the best choice, I note that only just as an observation.
Comment 1 Imobach Gonzalez Sosa 2019-03-21 09:13:44 UTC
Moving the bug to our Trello board: https://trello.com/c/BdE2MUbU/