[PATCH ] Update JMC-5419 : Links in rules do not work

Miro Wengner miro.wengner at gmail.com
Tue Sep 11 21:17:21 UTC 2018


Hi Everyone,
  Marcus notified me about an issue with current patch https://bugs.openjdk.java.net/browse/JMC-5419
I’ve not tested it on windows machine (currently working on to have windows system)

 I’ve  tested the anchor behavior  on my old windows machine but I still don’t have full JMC there, if some with windows machine can test it.

Thank you for notifying me!
Kind Regards,
Miro

Here is the fix to the issue.

PATCH:

diff -r 31576d187ec3 application/org.openjdk.jmc.flightrecorder.ui/src/main/java/org/openjdk/jmc/flightrecorder/ui/overview/ResultReportUi.java
--- a/application/org.openjdk.jmc.flightrecorder.ui/src/main/java/org/openjdk/jmc/flightrecorder/ui/overview/ResultReportUi.java        Fri Sep 07 18:39:12 2018 +0200
+++ b/application/org.openjdk.jmc.flightrecorder.ui/src/main/java/org/openjdk/jmc/flightrecorder/ui/overview/ResultReportUi.java        Tue Sep 11 21:17:27 2018 +0200
@@ -293,7 +293,7 @@
        private static String openWindowMethod(String url, String name){
         return new StringBuilder().append("#\" onclick=\"").append(OPEN_BROWSER_WINDOW).append("(").append("\u0027")
                 .append(url).append("\u0027").append(',').append("\u0027")
-                .append(name).append("\u0027").append(")").toString();
+                .append(name).append("\u0027").append(")").append(";return false;").toString();
     }

        public ResultReportUi(boolean isSinglePage) {


More information about the jmc-dev mailing list