Hide Forgot
This is the Web UI portion of the RFE: Description of problem: Need default validation for data type on TextBox fields when submitting Dialog Version-Release number of selected component (if applicable): How reproducible: Steps to Reproduce: 1. Create Dialog with dialog field TextBox and set data type as integer for this field 2. Create a Catalog Item and assign the Dialog created in step 1 3. Go Service Catalog and order a service, when submitting the dialog, it should do default validation on data type for Textbox fields. Note that the Provisioning Portion of the RFE is: https://bugzilla.redhat.com/show_bug.cgi?id=1278170
https://github.com/ManageIQ/manageiq/pull/5281
New commit detected on ManageIQ/manageiq/master: https://github.com/ManageIQ/manageiq/commit/3c3eb12e07994e20c1a31fa25b272292efe3d110 commit 3c3eb12e07994e20c1a31fa25b272292efe3d110 Author: Harpreet Kataria <hkataria> AuthorDate: Wed Nov 4 11:56:50 2015 -0500 Commit: Harpreet Kataria <hkataria> CommitDate: Mon Mar 21 10:08:53 2016 -0400 Added data_type support for textbox field in Dialog editor. https://bugzilla.redhat.com/show_bug.cgi?id=1319781 .../miq_ae_customization_controller/dialogs.rb | 3 + .../_dialog_field_form.html.haml | 11 +++ .../dialogs_spec.rb | 91 +++------------------- 3 files changed, 25 insertions(+), 80 deletions(-)
https://github.com/ManageIQ/manageiq/pull/7408
Created attachment 1148011 [details] value validator Created a dialog with value type Integer . while ordering the service dialog no validation message is shown . Not fixed . Checked in 5.6.0.1-beta2.20160413141124_e25ac0e
Currently the textbox for integer has some problems reported in https://bugzilla.redhat.com/show_bug.cgi?id=1328226 Please retest after those are fixed.
Will check in next build.
Shveta, Can you re-test this issue, please let me know if this has been fixed with the fix for https://bugzilla.redhat.com/show_bug.cgi?id=1328226 Thanks, ~Harpreet
Checked in latest build 5.6.0.13.20160624114606_13a9153 Created a service dialog with type Integer for Textbox. While ordering service gave a string value but No validation message shown.
https://github.com/ManageIQ/manageiq/pull/9674
New commit detected on ManageIQ/manageiq/master: https://github.com/ManageIQ/manageiq/commit/b0894e1e50c1d45256d032e890af6ead7dffe4db commit b0894e1e50c1d45256d032e890af6ead7dffe4db Author: Harpreet Kataria <hkataria> AuthorDate: Thu Jul 7 17:13:48 2016 -0400 Commit: Harpreet Kataria <hkataria> CommitDate: Thu Jul 7 17:20:53 2016 -0400 Initialize data type for TextBox field to be string by default. Data type for TextBox field was being set to to nil by default, this was causing validation issues when dialog was submitted. https://bugzilla.redhat.com/show_bug.cgi?id=1319781 https://bugzilla.redhat.com/show_bug.cgi?id=1348638 app/controllers/miq_ae_customization_controller/dialogs.rb | 5 +++++ 1 file changed, 5 insertions(+)
Created attachment 1202620 [details] Integer type Still not fixed.Attached screenshot shows dialog with type Integer and while ordering no validation message was shown when I entered string value. Same goes for Value type = String . While ordering I gave integer values and no validation was shown . Build : 5.7.0.0.20160906172503_12f65cb
Assigning it to Harpreet again . Please check appliance https://10.8.197.183 . Created a dialog with type TextBox and value Integer. While ordering the catalog item OUI.. gave string value , no validation message was shown
https://github.com/ManageIQ/manageiq/pull/13235
New commit detected on ManageIQ/manageiq/master: https://github.com/ManageIQ/manageiq/commit/4a008b986a70b78e0af83366463d1287333e6dbf commit 4a008b986a70b78e0af83366463d1287333e6dbf Author: Harpreet Kataria <hkataria> AuthorDate: Sun Dec 18 15:16:37 2016 -0500 Commit: Harpreet Kataria <hkataria> CommitDate: Sun Dec 18 15:16:37 2016 -0500 Fixed params[:field_typ] value check. https://bugzilla.redhat.com/show_bug.cgi?id=1319781 .../miq_ae_customization_controller/dialogs.rb | 2 +- .../dialogs_spec.rb | 32 ++++++++++++++++++++++ 2 files changed, 33 insertions(+), 1 deletion(-)
Fixed in 5.8.0.2-nightly.20170228141623_8710eed