From 680c2953cbe64ce20b5a19d7a55960428b132337 Mon Sep 17 00:00:00 2001 From: Kate Yang <150059882+kateyang1998@users.noreply.github.com> Date: Fri, 13 Dec 2024 16:00:56 -0500 Subject: [PATCH 1/4] fix: fixed receivePayment compilation error and CopyFavourites2 syntax error --- src/main/webapp/billing/CA/BC/receivePayment.jsp | 2 +- src/main/webapp/oscarRx/CopyFavorites2.jsp | 3 +-- 2 files changed, 2 insertions(+), 3 deletions(-) diff --git a/src/main/webapp/billing/CA/BC/receivePayment.jsp b/src/main/webapp/billing/CA/BC/receivePayment.jsp index fd30d8e8c7e..ed8534b467a 100644 --- a/src/main/webapp/billing/CA/BC/receivePayment.jsp +++ b/src/main/webapp/billing/CA/BC/receivePayment.jsp @@ -32,7 +32,7 @@
<% - oscar.oscarBilling.ca.bc.pageUtil.ReceivePaymentActionForm frm = (oscar.oscarBilling.ca.bc.pageUtil.ReceivePaymentActionForm) request.getAttribute("receivePaymentActionForm"); + oscar.oscarBilling.ca.bc.pageUtil.ReceivePayment2Action frm = (oscar.oscarBilling.ca.bc.pageUtil.ReceivePayment2Action) request.getAttribute("receivePaymentActionForm"); %> <%=java.text.NumberFormat.getCurrencyInstance().format(new Double(frm.getAmountReceived()))%>     diff --git a/src/main/webapp/oscarRx/CopyFavorites2.jsp b/src/main/webapp/oscarRx/CopyFavorites2.jsp index 3d5b6bbaa9a..b02481b8b63 100644 --- a/src/main/webapp/oscarRx/CopyFavorites2.jsp +++ b/src/main/webapp/oscarRx/CopyFavorites2.jsp @@ -62,10 +62,10 @@ <% - oscar.oscarRx.data.RxCodesData.FrequencyCode[] freq = new oscar.oscarRx.data.RxCodesData().getFrequencyCodes(); int i, j; + %> @@ -79,7 +79,6 @@ - %> diff --git a/src/main/webapp/admin/billingSettings.jsp b/src/main/webapp/admin/billingSettings.jsp index dff2fa5cac9..5dd6a15050e 100644 --- a/src/main/webapp/admin/billingSettings.jsp +++ b/src/main/webapp/admin/billingSettings.jsp @@ -43,7 +43,7 @@ <%@ taglib uri="/WEB-INF/oscar-tag.tld" prefix="oscar" %> -<%@ taglib prefix="c" uri="http://java.sun.com/jstl/core" %> +<%@ taglib prefix="c" uri="http://java.sun.com/jsp/jstl/core" %> <%@ page import="org.oscarehr.util.SpringUtils" %> <%@ page import="org.owasp.encoder.Encode" %> <%@ page import="org.oscarehr.common.dao.PropertyDao" %> diff --git a/src/main/webapp/casemgmt/IssueSearch.jsp b/src/main/webapp/casemgmt/IssueSearch.jsp index 786a9103808..655b5f3c074 100644 --- a/src/main/webapp/casemgmt/IssueSearch.jsp +++ b/src/main/webapp/casemgmt/IssueSearch.jsp @@ -108,39 +108,27 @@ <%-- table contents --%> - - " align="center"> - - - - Local - - - + + + + + + + + Local + + Community - - - - - - - + - - - - - - - - - - - - + ${newIssueCheckList.issue.code} + + ${newIssueCheckList.issue.description} + + ${newIssueCheckList.issue.role} + + -
diff --git a/src/main/webapp/casemgmt/navigation.jsp b/src/main/webapp/casemgmt/navigation.jsp index b4b0f6c1e4c..e6df8030f10 100644 --- a/src/main/webapp/casemgmt/navigation.jsp +++ b/src/main/webapp/casemgmt/navigation.jsp @@ -29,6 +29,7 @@ <%@page import="org.oscarehr.util.LoggedInInfo" %> <% long loadPage = System.currentTimeMillis(); %> <%@ include file="/casemgmt/taglibs.jsp" %> +<%@ taglib uri="http://java.sun.com/jsp/jstl/functions" prefix="fn" %> <%@ taglib uri="/WEB-INF/caisi-tag.tld" prefix="caisi" %> <%@ page import="org.apache.commons.lang.StringEscapeUtils" %> <%@ page import="oscar.oscarEncounter.oscarMeasurements.MeasurementFlowSheet" %> @@ -335,24 +336,18 @@ @@ -364,17 +359,13 @@ @@ -431,7 +422,7 @@ onchange="javascript:popUpMsg(600,900,this.options[this.selectedIndex].value)"> - + diff --git a/src/main/webapp/casemgmt/noteIssueList.jsp b/src/main/webapp/casemgmt/noteIssueList.jsp index ece8f64437c..f14c5754fe1 100644 --- a/src/main/webapp/casemgmt/noteIssueList.jsp +++ b/src/main/webapp/casemgmt/noteIssueList.jsp @@ -23,16 +23,14 @@ Ontario, Canada --%> - <%@page import="org.oscarehr.util.LoggedInInfo" %> -<%@ page - import="org.oscarehr.casemgmt.web.formbeans.CaseManagementEntryFormBean, org.oscarehr.common.model.Facility" %> +<%@page import="org.oscarehr.casemgmt.web.formbeans.CaseManagementEntryFormBean, org.oscarehr.common.model.Facility" %> <%@page import="org.apache.commons.lang.StringEscapeUtils" %> <%@page import="org.oscarehr.PMmodule.model.Program" %> <%@page import="org.oscarehr.PMmodule.dao.ProgramDao" %> <%@page import="org.oscarehr.util.SpringUtils" %> <%@ include file="/casemgmt/taglibs.jsp" %> - +<%@ taglib prefix="fn" uri="http://java.sun.com/jsp/jstl/functions" %> <% String roleName$ = (String) session.getAttribute("userrole") + "," + (String) session.getAttribute("user"); boolean authed = true; @@ -46,10 +44,7 @@ return; } %> - - - + <% LoggedInInfo loggedInInfo73557 = LoggedInInfo.getLoggedInInfoFromSession(request); String noteIndex = ""; @@ -62,87 +57,85 @@ request.setAttribute("caseManagementEntryForm", frm); } %> - - - <% - noteIndex = request.getParameter("newNoteIdx"); - %> -
"> -
" - style="float: right; margin-right: 3px;"> - - - <% - noteIndex = "0"; - %> -
-
- - - - <% - noteIndex = String.valueOf(frm.getCaseNote().getId()); - %> -
"> -
" - style="float: right; margin-right: 3px;"> - + + + + + +
+
+
+ + + +
+
+
+ + + + + +
+
+
+
+
+
- - "> -   - ', event);"> - - - " - id="observationDate_cal" - alt="calendar"> "> - rev', event);"> - + + + +   + + +   + + + + ${caseManagementEntryForm.caseNote.revision} + + + + +   + calendar  + "> + rev + + ${caseManagementEntryForm.caseNote.revision} + + +
: - + +
    - - - -
  • ; + + +
  • ${editor.formattedName}; - - -
  • + + ${editor.formattedName} -
    - + +
-
+ +
 
-
+ +
<% @@ -156,24 +149,28 @@ if (currentFacility.isEnableEncounterTime() || (currentProgram != null && currentProgram.getEnableEncounterTime())) { %>
- - :  - "> - - : - "> - - - - - :  - "> :  - "> - + + + + :  + + ${caseManagementEntryForm.caseNote.hourOfEncounterTime} + : + + ${caseManagementEntryForm.caseNote.minuteOfEncounterTime} + + + + + :  +  :  + + +
<%}%> @@ -181,25 +178,29 @@ if (currentFacility.isEnableEncounterTransportationTime() || (currentProgram != null && currentProgram.isEnableEncounterTransportationTime())) { %>
- - :  - "> - - : - "> - - - - - :  - "> :  - "> - + + + + :  + + ${caseManagementEntryForm.caseNote.hourOfEncTransportationTime} + : + + ${caseManagementEntryForm.caseNote.minuteOfEncTransportationTime} + + + + + :  +  :  + + + +
<%}%> @@ -211,56 +212,77 @@
- - - - "" - + + + + + + """" + +
- -<%-- -
 
-
--%> - -
- + + + +
+ + + +
    - +
  • -
    +

- - -

+ + + +
+
- +
+
@@ -268,112 +290,77 @@ <% int countResolvedIssue = -1; %> - - - <% - String winame = "window" + issueCheckList.getIssueDisplay().getDescription(); - winame = winame.replaceAll("\\s|\\/|\\*", "_"); - winame = winame.replaceAll("'", ""); - winame = StringEscapeUtils.escapeJavaScript(winame); - countResolvedIssue++; - if (countResolvedIssue % 2 == 0) { - %> - - <% - } - %> - - - - - <% - if (countResolvedIssue % 2 != 0) { - %> - - <% - } - %> - - + + + + + + + + + + +
- <% - String submitString = "this.form.method.value='issueChange';"; - submitString = submitString + "this.form.lineId.value=" + "'" + ind.intValue() + "'; return ajaxUpdateIssues('issueChange', $('noteIssues').up().id);"; - String id = "noteIssue" + ind; - org.oscarehr.casemgmt.web.CheckBoxBean cbb = (org.oscarehr.casemgmt.web.CheckBoxBean) pageContext.getAttribute("issueCheckList"); - boolean writeAccess = cbb.getIssueDisplay().isWriteAccess(); - boolean disabled = !writeAccess; - %> - - - - - - - - - <% - String submitDelete = "removeIssue('" + winame + "');document.forms['caseManagementEntryForm'].deleteId.value=" + "'" + ind.intValue() + "';return ajaxUpdateIssues('issueDelete', $('noteIssues').up().id);"; - %> -   - Delete -   - - - - <% - String submitChange = "return changeDiagnosisResolved('" + ind.intValue() + "');"; - %> -   - Change - - - - -
+ + + + + + + + + + + + + + + + + + + +   + Delete  + + + + +   + Change + + +
@@ -384,101 +371,82 @@ - - - - - - <% - String winame = "window" + issueCheckList.getIssueDisplay().getDescription(); - winame = winame.replaceAll("\\s|\\/|\\*", "_"); - winame = winame.replaceAll("'", ""); - winame = StringEscapeUtils.escapeJavaScript(winame); - countUnresolvedIssue++; - if (countUnresolvedIssue % 2 == 0) { - %> - <% - } - %> - - <% if (countUnresolvedIssue % 2 != 0) { %> - - <% } %> - - + + + + + + + + + + + + + + + + + + + + + +
- <% - String submitString = "this.form.method.value='issueChange';"; - submitString = submitString + "this.form.lineId.value=" + "'" + ind.intValue() + "'; return ajaxUpdateIssues('issueChange', $('noteIssues').up().id);"; - String id = "noteIssue" + ind; - org.oscarehr.casemgmt.web.CheckBoxBean cbb = (org.oscarehr.casemgmt.web.CheckBoxBean) pageContext.getAttribute("issueCheckList"); - boolean writeAccess = cbb.getIssueDisplay().isWriteAccess(); - boolean disabled = !writeAccess; - %> - - - - - - - <% - String submitDelete = "removeIssue('" + winame + "');document.forms['caseManagementEntryForm'].deleteId.value=" + "'" + ind.intValue() + "';return ajaxUpdateIssues('issueDelete', $('noteIssues').up().id);"; - %> -   - Delete -   - - - <% - String submitChange = "return changeDiagnosisUnresolved('" + ind.intValue() + "');"; - %> -   - Change - -
+ + + + + + + + + + ${issueCheckList.issueDisplay.description} + + + + +   + Delete +   + + +   + Change + + +
@@ -489,34 +457,37 @@ //check to see if we need to update div containers to most recent note id //this happens only when we're called thru ajaxsave - - var origId = ""; - var newId = ""; - var oldDiv; - var newDiv; - var prequel = ["n", "sig", "signed", "full", "bgColour", "print", "editWarn"]; - - for (var idx = 0; idx < prequel.length; ++idx) { - oldDiv = prequel[idx] + origId; - newDiv = prequel[idx] + newId; - if ($(oldDiv) != null) - $(oldDiv).id = newDiv; - } - updatedNoteId = newId; - - <%//CaseManagementEntryFormBean form = (CaseManagementEntryFormBean)request.getAttribute("caseManagementEntryForm"); - String noteTxt = frm.getCaseNote().getNote(); - noteTxt = org.apache.commons.lang.StringEscapeUtils.escapeJavaScript(noteTxt);%> - completeChangeToView("<%=noteTxt%>", newId); - if (origId.substr(0, 1) == "0") { - $("nc" + origId).id = "nc" + numNotes; - ++numNotes; - } + + + - - alert(""); - - var c = "bgColour" + "<%=noteIndex%>"; var txtStyles = $F(c).split(";"); @@ -542,7 +513,7 @@ } //do we have a custom encounter type? if so add an option to the encounter type select - var encounterType = ''; + var encounterType = '${caseManagementEntryForm.caseNote.encounter_type}'; var selectEnc = "<%=encSelect%>"; if ($(selectEnc) != null) { diff --git a/src/main/webapp/casemgmt/showHistory.jsp b/src/main/webapp/casemgmt/showHistory.jsp index 61b44134120..d49629eed6f 100644 --- a/src/main/webapp/casemgmt/showHistory.jsp +++ b/src/main/webapp/casemgmt/showHistory.jsp @@ -23,7 +23,7 @@ Ontario, Canada --%> - +<%@ taglib uri="http://java.sun.com/jsp/jstl/functions" prefix="fn" %> <%@ include file="/casemgmt/taglibs.jsp" %> <%@page import="org.oscarehr.casemgmt.model.CaseManagementNote" %> <% @@ -58,7 +58,7 @@

- +

diff --git a/src/main/webapp/casemgmt/viewNotes.jsp b/src/main/webapp/casemgmt/viewNotes.jsp index 2170f8e4887..b3e25668244 100644 --- a/src/main/webapp/casemgmt/viewNotes.jsp +++ b/src/main/webapp/casemgmt/viewNotes.jsp @@ -23,7 +23,7 @@ Ontario, Canada --%> - +<%@ taglib uri="http://java.sun.com/jsp/jstl/functions" prefix="fn" %> <%@page import="org.apache.commons.lang.StringEscapeUtils" %> <%@page import="oscar.util.ConversionUtils" %> <%@page import="org.oscarehr.casemgmt.web.NoteDisplay" %> @@ -58,9 +58,8 @@ String titleMsg = bundle.getString(titleParam); %> - - + +