Removed decodeURI call.

[h-kataria]

decodeURI was causing an issue rendering html on screen when '%' character is present in html output. Removed decodeURI call as html is getting rendered correctly.

https://bugzilla.redhat.com/show_bug.cgi?id=1444853

@bzwei can you please test.

[miq-bot]

Checked commit h-kataria@9e0cffa with ruby 2.2.6, rubocop 0.47.1, and haml-lint 0.20.0
0 files checked, 0 offenses detected
Everything looks fine. 👍

[dclarizio]

@himdel just need a quick glance :)

[bzwei]

Seems a simple fix. Tested good 👍

[himdel]

Looks good, URI decoding is done by the browser, so no need to do it twice 👍.

(That said, I'm not 100% sure the js_function('#{html.gsub(..).gsub(..)}') approach is safe in all the cases, but that's a separate issue (but if you do look into it, it may make sense to merge with the miq-sanitize directive).)

[simaishi]

Fine backport details:

$ git log -1
commit 08f4b0af78c3889ced8acddbdad7eb40a1503a02
Author: Martin Hradil <himdel@seznam.cz>
Date:   Thu Apr 27 09:23:00 2017 +0000

    Merge pull request #1167 from h-kataria/std_output_fix
    
    Removed decodeURI call.
    (cherry picked from commit 48b51dd9a41e6d79143e9d90edf299ab881c8394)
    
    https://bugzilla.redhat.com/show_bug.cgi?id=1446245