Juan Hernandez has posted comments on this change.
Change subject: packaging: fixed redirect to reports to not show resend dialog
......................................................................
Patch Set 1: Verified+1 Code-Review+1
(1 comment)
I did this same change manually and it worked correctly.
....................................................
File server-customizations/WEB-INF/jsp/modules/viewReport/ViewReportState.jsp
Line 22: <%@ page import="org.apache.commons.lang.StringEscapeUtils" %>
Line 23:
Line 24: <c:if test="${needPageRefresh}">
Line 25: <script type="text/javascript">
Line 26: <%-- HTTP redirect can't be applied here because it adds
jsessionid parameter --%>
I would suggest you to add a comment here (or somewhere else) to clearly
indicate what was changed from the original, and why. Otherwise it will be
difficult in the future to understand why this file is needed.
Line 27: </script>
Line 28: </c:if>
Line 29:
Line 30:
--
To view, visit http://gerrit.ovirt.org/21632
To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: Idd6f1523100d29d18eb2c8cd10356e779070702a
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-reports
Gerrit-Branch: master
Gerrit-Owner: Yaniv Dary <[email protected]>
Gerrit-Reviewer: Juan Hernandez <[email protected]>
Gerrit-HasComments: Yes
_______________________________________________
Engine-patches mailing list
[email protected]
http://lists.ovirt.org/mailman/listinfo/engine-patches
