| Summary: | Catalog Item : Catalog tab fields goes missing when "Display in Catalog" is checked | ||||||||||||
|---|---|---|---|---|---|---|---|---|---|---|---|---|---|
| Product: | Red Hat CloudForms Management Engine | Reporter: | Shveta <sshveta> | ||||||||||
| Component: | UI - OPS | Assignee: | Harpreet Kataria <hkataria> | ||||||||||
| Status: | CLOSED ERRATA | QA Contact: | Shveta <sshveta> | ||||||||||
| Severity: | medium | Docs Contact: | |||||||||||
| Priority: | high | ||||||||||||
| Version: | 5.5.0 | CC: | cpelland, dajohnso, hkataria, jhardy, jprause, mpovolny, obarenbo, simaishi, sshveta | ||||||||||
| Target Milestone: | GA | Keywords: | ZStream | ||||||||||
| Target Release: | 5.6.0 | ||||||||||||
| Hardware: | Unspecified | ||||||||||||
| OS: | Unspecified | ||||||||||||
| Whiteboard: | service:catalog | ||||||||||||
| Fixed In Version: | 5.6.0.1 | Doc Type: | Bug Fix | ||||||||||
| Doc Text: | Story Points: | --- | |||||||||||
| Clone Of: | |||||||||||||
| : | 1322834 (view as bug list) | Environment: | |||||||||||
| Last Closed: | 2016-06-29 15:45:13 UTC | Type: | Bug | ||||||||||
| Regression: | --- | Mount Type: | --- | ||||||||||
| Documentation: | --- | CRM: | |||||||||||
| Verified Versions: | Category: | --- | |||||||||||
| oVirt Team: | --- | RHEL 7.3 requirements from Atomic Host: | |||||||||||
| Cloudforms Team: | --- | Target Upstream Version: | |||||||||||
| Bug Depends On: | |||||||||||||
| Bug Blocks: | 1322834 | ||||||||||||
| Attachments: |
|
||||||||||||
Created attachment 1141947 [details]
checkbox checked
Created attachment 1141948 [details]
with display in catalog unchecked
Created attachment 1141949 [details]
logs
Shveta, is this new with 5.5.3, is it the same on 5.5.2? Dave , this is new in 5.5.3 This does impact some test automation, we came up with an ugly workaround but better to fix sooner than later. New commit detected on ManageIQ/manageiq/master: https://github.com/ManageIQ/manageiq/commit/85272bd43ff416b63aeffd22ba05dd08df4923f9 commit 85272bd43ff416b63aeffd22ba05dd08df4923f9 Author: Harpreet Kataria <hkataria> AuthorDate: Mon Apr 4 12:06:29 2016 -0400 Commit: Harpreet Kataria <hkataria> CommitDate: Mon Apr 4 12:42:53 2016 -0400 form_div should only be replaced for certain type of catalog items. Only replace form_div for generic type catalog items because there are no tabs on the screen, if display in catalog is set need to redraw the screen to draw the tabs and show/hide them. Replacing form_div for the types that already have tabs on screen was causing instance variables to be reset in provisioning dialogs. https://bugzilla.redhat.com/show_bug.cgi?id=1321631 https://bugzilla.redhat.com/show_bug.cgi?id=1322834 app/controllers/catalog_controller.rb | 3 ++- spec/controllers/catalog_controller_spec.rb | 29 +++++++++++++++++++++++++++++ 2 files changed, 31 insertions(+), 1 deletion(-) New commit detected on cfme/5.5.z: https://code.engineering.redhat.com/gerrit/gitweb?p=cfme.git;a=commitdiff;h=e09fad9332da50bab9596582361007957cd6e983 commit e09fad9332da50bab9596582361007957cd6e983 Author: Harpreet Kataria <hkataria> AuthorDate: Mon Apr 4 12:06:29 2016 -0400 Commit: Harpreet Kataria <hkataria> CommitDate: Fri Apr 15 12:19:04 2016 -0400 form_div should only be replaced for certain type of catalog items. - Only replace form_div for generic type catalog items because there are no tabs on the screen, if display in catalog is set need to redraw the screen to draw the tabs and show/hide them. Replacing form_div for the types that already have tabs on screen was causing instance variables to be reset in provisioning dialogs. - Adjusted spec test to pass on 5.5.z https://bugzilla.redhat.com/show_bug.cgi?id=1321631 https://bugzilla.redhat.com/show_bug.cgi?id=1322834 (cherry picked from commit 85272bd) app/controllers/catalog_controller.rb | 3 ++- spec/controllers/catalog_controller_spec.rb | 29 +++++++++++++++++++++++++++++ 2 files changed, 31 insertions(+), 1 deletion(-) New commit detected on cfme/5.5.z: https://code.engineering.redhat.com/gerrit/gitweb?p=cfme.git;a=commitdiff;h=a6e7de373d30c2cf32d083d2c60ade0c7a8c2e6d commit a6e7de373d30c2cf32d083d2c60ade0c7a8c2e6d Merge: 32af3c0 e09fad9 Author: Dan Clarizio <dclarizi> AuthorDate: Thu Apr 21 13:29:56 2016 -0400 Commit: Dan Clarizio <dclarizi> CommitDate: Thu Apr 21 13:29:56 2016 -0400 Merge branch '55z_pr_7693' into '5.5.z' form_div should only be replaced for certain type of catalog items. Only replace form_div for generic type catalog items because there are no tabs on the screen, if display in catalog is set need to redraw the screen to draw the tabs and show/hide them. Replacing form_div for the types that already have tabs on screen was causing instance variables to be reset in provisioning dialogs. https://bugzilla.redhat.com/show_bug.cgi?id=1321631 https://bugzilla.redhat.com/show_bug.cgi?id=1322834 (cherry picked from commit 85272bd) @dclarizi please review, clean cherry-pick from https://github.com/ManageIQ/manageiq/pull/7693 See merge request !891 app/controllers/catalog_controller.rb | 3 ++- spec/controllers/catalog_controller_spec.rb | 29 +++++++++++++++++++++++++++++ 2 files changed, 31 insertions(+), 1 deletion(-) Fixed. Verified in 5.6.0.5-beta2.4.20160503153816_1fb554f Since the problem described in this bug report should be resolved in a recent advisory, it has been closed with a resolution of ERRATA. For information on the advisory, and where to find the updated files, follow the link below. If the solution does not work for you, open a new bug report. https://access.redhat.com/errata/RHBA-2016:1348 |
Created attachment 1140937 [details] err Description of problem: Version-Release number of selected component (if applicable): How reproducible: Steps to Reproduce: 1. Create a catalog item of type Vmware. 2. Select "display in catalog" . 3. Under Request Info tab , catalog tab doesn't show host and datastore . Actual results:Workaround : click on environment tab and then again on catalog tab . Fields are displayed now Expected results: Additional info: 5.5.3.2.20160318152106_92d2d67