Bug 1389065

Summary: UI: Analysis Profile - Edit of existing Registry Item is not working
Product: Red Hat CloudForms Management Engine Reporter: Harpreet Kataria <hkataria>
Component: UI - OPSAssignee: Eric Winchell <ewinchel>
Status: CLOSED CURRENTRELEASE QA Contact: Satyajit Bulage <sbulage>
Severity: medium Docs Contact:
Priority: high    
Version: 5.7.0CC: cpelland, dajohnso, hkataria, jhardy, mmojzis, mpovolny, obarenbo, simaishi
Target Milestone: GAKeywords: TestOnly
Target Release: 5.8.0   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard: smartstate:windows
Fixed In Version: 5.8.0.0 Doc Type: If docs needed, set a value
Doc Text:
Story Points: ---
Clone Of:
: 1391674 1440409 (view as bug list) Environment:
Last Closed: 2017-06-12 17:48:04 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:
Embargoed:
Bug Depends On:    
Bug Blocks: 1391674, 1440409    

Description Harpreet Kataria 2016-10-26 18:40:58 UTC
Description of problem: UI: Analysis Profile - Edit of existing Registry Item is not working


Version-Release number of selected component (if applicable):
5.7

How reproducible: 100%


Steps to Reproduce:
1. Under Configuration/Configure, on Settings accordion, go to Analysis Profiles node
2. Edit an existing Profile
3. When trying to edit an existing Registry Item, Clicking on Update button to the left to Registry Item row does not do anything.

Actual results: Clicking on update button does not do anything.

Expected results: It should update an existing entry


Additional info: this issue was reported on upstream under https://github.com/ManageIQ/manageiq/issues/12004

Upstream Fix: https://github.com/ManageIQ/manageiq/pull/12200

Comment 2 CFME Bot 2016-11-01 15:21:02 UTC
New commit detected on ManageIQ/manageiq/euwe:
https://github.com/ManageIQ/manageiq/commit/38313da9058f4713557bc1ac3fc003ec5db19940

commit 38313da9058f4713557bc1ac3fc003ec5db19940
Author:     Harpreet Kataria <hkataria>
AuthorDate: Wed Oct 26 14:46:36 2016 -0400
Commit:     Oleg Barenboim <chessbyte>
CommitDate: Tue Nov 1 11:18:34 2016 -0400

    Merge pull request #12200 from epwinchell/action_menu_for_ap
    
    Replace images with PF Action Menu on Analysis Profile Screens
    (cherry picked from commit 809ab09495e30db774f2d2784567314d571ec3c6)
    
    https://bugzilla.redhat.com/show_bug.cgi?id=1389065

 app/views/ops/_ap_form_file.html.haml       | 49 +++++++++++-----------
 app/views/ops/_ap_form_nteventlog.html.haml | 65 ++++++++++++++++++-----------
 app/views/ops/_ap_form_registry.html.haml   | 62 ++++++++++++++++-----------
 3 files changed, 104 insertions(+), 72 deletions(-)

Comment 5 Satyajit Bulage 2017-03-08 15:46:56 UTC
Able to update existing Registry Item in analysis profiles.

Verified Version: 5.8.0.4.20170307183144_745a3d9

Comment 7 Satoe Imaishi 2017-04-25 13:14:14 UTC
*** Bug 1423381 has been marked as a duplicate of this bug. ***