Bug 1277077

Summary: HTML character codes present in tool tip of "Edit My Company Tags for selected tenant"
Product: Red Hat CloudForms Management Engine Reporter: Aziza Karol <akarol>
Component: UI - OPSAssignee: Martin Povolny <mpovolny>
Status: CLOSED ERRATA QA Contact: Aziza Karol <akarol>
Severity: medium Docs Contact:
Priority: medium    
Version: 5.5.0CC: cpelland, dclarizi, hkataria, jhardy, mpovolny, obarenbo, simaishi
Target Milestone: GA   
Target Release: 5.5.0   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: 5.5.0.10 Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2015-12-08 13:42:55 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:
Attachments:
Description Flags
htmltags none

Description Aziza Karol 2015-11-02 09:36:26 UTC
Created attachment 1088498 [details]
htmltags

Description of problem:


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

How reproducible:
100%

Steps to Reproduce:
1.Navigate to configure->configuration->access control->Tenants->select "my company"
2.Hover on "Edit my company tags for selected tenants" in Policy
3.

Actual results:
html tags displayed when you hover on it. see attached screenshot

Expected results:
No html tags

Additional info:

Comment 3 Martin Povolny 2015-11-02 18:15:23 UTC
PR: https://github.com/ManageIQ/manageiq/pull/5231

Comment 4 CFME Bot 2015-11-02 19:50:15 UTC
New commit detected on ManageIQ/manageiq/master:
https://github.com/ManageIQ/manageiq/commit/4ff5aa74d344c9a3064567bbef55caa0bbc0c45e

commit 4ff5aa74d344c9a3064567bbef55caa0bbc0c45e
Author:     Martin Povolny <mpovolny>
AuthorDate: Mon Nov 2 19:07:09 2015 +0100
Commit:     Martin Povolny <mpovolny>
CommitDate: Mon Nov 2 19:07:09 2015 +0100

    Toolbars: prevent double-escape of toolbar tooltips.
    
    Remove CGI.escapeHTML to prevent double-escape of toolbar tooltips.
    
    https://bugzilla.redhat.com/show_bug.cgi?id=1277077

 app/helpers/application_helper/toolbar_builder.rb | 18 ++++++++----------
 1 file changed, 8 insertions(+), 10 deletions(-)

Comment 6 CFME Bot 2015-11-05 14:49:43 UTC
New commit detected on cfme/5.5.z:
https://code.engineering.redhat.com/gerrit/gitweb?p=cfme.git;a=commitdiff;h=ed815495571637973ca790207297352e5e04c756

commit ed815495571637973ca790207297352e5e04c756
Author:     Martin Povolny <mpovolny>
AuthorDate: Mon Nov 2 19:07:09 2015 +0100
Commit:     Martin Povolny <mpovolny>
CommitDate: Thu Nov 5 11:10:18 2015 +0100

    Toolbars: prevent double-escape of toolbar tooltips.
    
    Remove CGI.escapeHTML to prevent double-escape of toolbar tooltips.
    
    https://bugzilla.redhat.com/show_bug.cgi?id=1277077

 app/helpers/application_helper/toolbar_builder.rb | 18 ++++++++----------
 1 file changed, 8 insertions(+), 10 deletions(-)

Comment 7 Aziza Karol 2015-11-25 11:01:28 UTC
No html tags displayed.

Verified:
5.5.0.12-rc2.20151124135609_653c0d4

Comment 9 errata-xmlrpc 2015-12-08 13:42:55 UTC
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/RHSA-2015:2551