WFNEWS-2203 : Use ROF API field instead of comments for visible flames (

#1957)
bcgov · Jun 13, 2024 · ccabeb0 · ccabeb0
1 parent 79c8a91
commit ccabeb0
Show file tree

Hide file tree

Showing 4 changed files with 14 additions and 26 deletions.
diff --git a/server/wfnews-api/wfnews-service-api/pom.xml b/server/wfnews-api/wfnews-service-api/pom.xml
@@ -29,7 +29,7 @@
     <dependency>
       <groupId>ca.bc.gov.nrs.wfim</groupId>
       <artifactId>wfim-incidents-api-rest-common</artifactId>
-      <version>1.5.0-SNAPSHOT</version>
+      <version>1.9.5-SNAPSHOT</version>
     </dependency>
   	 <dependency>
 			<groupId>org.postgresql</groupId>

diff --git a/server/wfone-notifications-api/wfone-notifications-quartz-scheduler/pom.xml b/server/wfone-notifications-api/wfone-notifications-quartz-scheduler/pom.xml
@@ -62,7 +62,7 @@
     <dependency>
       <groupId>ca.bc.gov.nrs.wfim</groupId>
       <artifactId>wfim-incidents-api-rest-common</artifactId>
-      <version>1.6.1</version>
+      <version>1.9.5-SNAPSHOT</version>
     </dependency>
 
 	</dependencies>

diff --git a/server/wfone-notifications-api/wfone-notifications-service-api/pom.xml b/server/wfone-notifications-api/wfone-notifications-service-api/pom.xml
@@ -22,7 +22,7 @@
 		<dependency>
 			<groupId>ca.bc.gov.nrs.wfim</groupId>
 			<artifactId>wfim-incidents-api-rest-common</artifactId>
-			<version>1.6.1</version>
+			<version>1.9.5-SNAPSHOT</version>
 		</dependency>
 
 		<dependency>

diff --git a/...rvice-api/src/main/java/ca/bc/gov/nrs/wfone/service/api/v1/impl/RecordRoFServiceImpl.java b/...rvice-api/src/main/java/ca/bc/gov/nrs/wfone/service/api/v1/impl/RecordRoFServiceImpl.java
@@ -116,11 +116,6 @@ public class RecordRoFServiceImpl implements RecordRoFService {
 	EmailNotificationService emailNotificationService;
 
 	private Date codeTableRefresh;
-	private String visibleFlameString = "Visible Flames = Yes; ";
-	private String noVisibleFlameString = "Visible Flames = No; ";
-	private String noCommentVisibleFlameString = "Visible Flames = N/C; ";
-	private String noCommentString = "No comments";
-
 	private Properties applicationProperties;
 	private RoFFormDao rofFormDao;
 	private RoFImageDao rofImageDao;
@@ -575,25 +570,18 @@ private String prepareRoF(JSONObject rofFormDataJson, ObjectMapper mapper)
 			rof.setSubmittedTimestamp(Date.from(submittedDateTime.atZone(ZoneId.systemDefault()).toInstant()));
 		}
 
-		// set default visible flame string as No Comment
-		String visibleFlame = noCommentVisibleFlameString;
-
-		if (rofFormDataJson.has("visibleFlame") && rofFormDataJson.optJSONArray("visibleFlame") != null) {
-			if (rofFormDataJson.optJSONArray("visibleFlame").isEmpty()) {
-				visibleFlame = noCommentVisibleFlameString;
-			} else if (rofFormDataJson.getJSONArray("visibleFlame").toString().replace("[", "").replace("]", "")
-					.replace("\"", "").equalsIgnoreCase("YES")) {
-				visibleFlame = visibleFlameString;
-			} else if (rofFormDataJson.getJSONArray("visibleFlame").toString().replace("[", "").replace("]", "")
-					.replace("\"", "").equalsIgnoreCase("NO")) {
-				visibleFlame = noVisibleFlameString;
-			}
-		}
+		Boolean visibleFlame = rofFormDataJson.has("visibleFlame")
+				&& rofFormDataJson.optJSONArray("visibleFlame") != null
+				&& rofFormDataJson.getJSONArray("visibleFlame").toString()
+					.replace("[", "").replace("]", "")
+					.replace("\"", "")
+				.equalsIgnoreCase("YES");
+		rof.setFlamesVisibleInd(visibleFlame);
 
-		if (rofFormDataJson.has("otherInfo") && !rofFormDataJson.optString("otherInfo", "").equals(""))
-			rof.setCallerReportDetails(visibleFlame + rofFormDataJson.optString("otherInfo"));
-		else if (rofFormDataJson.has("otherInfo") && rofFormDataJson.optString("otherInfo", "").equals(""))
-			rof.setCallerReportDetails(visibleFlame + noCommentString);
+
+		if (rofFormDataJson.has("otherInfo") && !rofFormDataJson.optString("otherInfo", "").isEmpty()) {
+			rof.setCallerReportDetails(rofFormDataJson.optString("otherInfo"));
+		}
 
 		logger.info(" ### START POST - Serializing and executing RoF POST...");
 		mapper.activateDefaultTyping(mapper.getPolymorphicTypeValidator(), ObjectMapper.DefaultTyping.JAVA_LANG_OBJECT,