Created attachment 1118827 [details] add storage number fields Description of problem: There is several fields where integer number should be. For that reason there is special functionality which allows to increase or decrease the number. However it's possible to: 1. put there something which is not number at all 2. put there float number - e.g. 2.5 as Pools to create 3. put there minus number or 0 4. use the decrease functionality to go to zero or bellow Version-Release number of selected component (if applicable): rhscon-ceph-0.0.5-0.1.alpha1.el7.x86_64 rhscon-ui-0.0.6-0.1.alpha1.el7.noarch rhscon-core-0.0.7-0.1.alpha1.el7.x86_64 How reproducible: 100% Steps to Reproduce: 1. Go to Storage 2. Click on Add Storage, choose some cluster and click on 'Next' button 3. see for fields with numbers, put there some values Actual results: No error message appears. All fields looks like they are accepted and in some cases they even are accepted and storage was created even with bad value. e.g. float number as number of pools or zero as number of replicas etc. Expected results: Some validation should be done before/after user click on 'Next' button. Moreover 'decrease' should not allow to go bellow 1. Additional info: see screen-shots in attachment
Comment on attachment 1118827 [details] add storage number fields not all related fields are highlighted
Tested on: rhscon-core-selinux-0.0.29-1.el7scon.noarch rhscon-ui-0.0.43-1.el7scon.noarch rhscon-core-0.0.29-1.el7scon.x86_64 rhscon-ceph-0.0.29-1.el7scon.x86_64 Most bad inputs are checked correctly, however there are still some issues. 1. A float number instead of integer number could be filled 2. Very long number could be used as a value