just cleaned up code a bit and made names more intuitive

codeforamerica · Oct 11, 2023 · e05d290 · e05d290
1 parent 3aeb0af
commit e05d290
Show file tree

Hide file tree

Showing 5 changed files with 15 additions and 16 deletions.
diff --git a/src/main/java/org/ladocuploader/app/inputs/LaDigitalAssister.java b/src/main/java/org/ladocuploader/app/inputs/LaDigitalAssister.java
@@ -73,14 +73,14 @@ public class LaDigitalAssister extends FlowInputs {
 
 //  Sensitive Questions
   private String householdHasPersonalSituations;
-  private String uuidsByPersonalSituations;
+  private String personalSituationsHouseholdUUID;
   private List<String> personalSituationsListed;
   private String householdHasDomesticViolenceSituation;
   private String householdHasCriminalJusticeSituation;
-  private String homelessSwitch;
-  private String otherIssue;
-  private String strugglingDrugsAlcohol;
-  private String domesticViolenceFlag;
-  private String personaIssueDisability;
+  private String personalSituationHomelessSwitch;
+  private String personalSituationOtherIssue;
+  private String personalSituationStrugglingDrugsAlcohol;
+  private String personalSituationDomesticViolenceFlag;
+  private String personalSituationDisability;
 }
 
diff --git a/...ns/ReformatsUserDataBeforeDisplaying.java → ...ns/ReformatPersonalSituationUserData.java b/...ns/ReformatsUserDataBeforeDisplaying.java → ...ns/ReformatPersonalSituationUserData.java
@@ -3,7 +3,6 @@
 import formflow.library.config.submission.Action;
 import formflow.library.data.Submission;
 import java.util.ArrayList;
-import java.util.HashMap;
 import java.util.LinkedHashMap;
 import java.util.Map;
 import lombok.extern.slf4j.Slf4j;
@@ -14,7 +13,7 @@
  */
 @Component
 @Slf4j
-public class ReformatsUserDataBeforeDisplaying implements Action {
+public class ReformatPersonalSituationUserData implements Action {
 
   @Override
   public void run(Submission submission) {
@@ -25,7 +24,7 @@ public void run(Submission submission) {
 
   private ArrayList<LinkedHashMap> reformatPersonalSituations(Map<String, Object> inputData) {
     ArrayList<LinkedHashMap> householdMembers = (ArrayList) inputData.get("household");
-    ArrayList<String> hasPersonalSituations = (ArrayList) inputData.get("uuidsByPersonalSituations[]");
+    ArrayList<String> hasPersonalSituations = (ArrayList) inputData.get("personalSituationsHouseholdUUID[]");
     ArrayList<LinkedHashMap> personalSituationsObject = new ArrayList<>();
 
     hasPersonalSituations.forEach((String id) -> {

diff --git a/src/main/resources/flows-config.yaml b/src/main/resources/flows-config.yaml
@@ -173,7 +173,7 @@ flow:
         condition: HouseholdHasPersonalSituations
       - name: householdVictimOfDomesticViolence
   householdPersonalSituationsWho:
-    beforeSaveAction: ReformatsUserDataBeforeDisplaying
+    beforeSaveAction: ReformatPersonalSituationUserData
     nextScreens:
       - name: householdWhichPersonalSituations
   householdWhichPersonalSituations:

diff --git a/src/main/resources/templates/laDigitalAssister/householdPersonalSituationsWho.html b/src/main/resources/templates/laDigitalAssister/householdPersonalSituationsWho.html
@@ -15,7 +15,7 @@
           <th:block th:ref="formContents">
             <div class="form-card__content">
               <th:block th:replace="~{'fragments/inputs/householdCheckboxFieldset' ::
-                    householdCheckboxFieldset(inputName='uuidsByPersonalSituations')}"/>
+                    householdCheckboxFieldset(inputName='personalSituationsHouseholdUUID')}"/>
             </div>
             <div class="form-card__footer">
               <th:block th:replace="~{fragments/inputs/submitButton :: submitButton(

diff --git a/src/main/resources/templates/laDigitalAssister/householdWhichPersonalSituations.html b/src/main/resources/templates/laDigitalAssister/householdWhichPersonalSituations.html
@@ -22,15 +22,15 @@
                   <h2 th:text="#{personal-situations-which.subheader(${individual.firstName})}"></h2>
                   <p th:text="#{general.check-all-that-apply}"></p>
                   <th:block
-                      th:replace="~{'fragments/inputs/checkboxInSet' :: checkboxInSet(inputName='homelessSwitch',value=${individual.uuid}, label=#{personal-situations.which.option1})}"/>
+                      th:replace="~{'fragments/inputs/checkboxInSet' :: checkboxInSet(inputName='personalSituationHomelessSwitch',value=${individual.uuid}, label=#{personal-situations.which.option1})}"/>
                   <th:block
-                      th:replace="~{'fragments/inputs/checkboxInSet' :: checkboxInSet(inputName='strugglingDrugsAlcohol',value=${individual.uuid}, label=#{personal-situations.which.option2})}"/>
+                      th:replace="~{'fragments/inputs/checkboxInSet' :: checkboxInSet(inputName='personalSituationStrugglingDrugsAlcohol',value=${individual.uuid}, label=#{personal-situations.which.option2})}"/>
                   <th:block
-                      th:replace="~{'fragments/inputs/checkboxInSet' :: checkboxInSet(inputName='domesticViolenceFlag',value=${individual.uuid}, label=#{personal-situations.which.option3})}"/>
+                      th:replace="~{'fragments/inputs/checkboxInSet' :: checkboxInSet(inputName='personalSituationDomesticViolenceFlag',value=${individual.uuid}, label=#{personal-situations.which.option3})}"/>
                   <th:block
-                      th:replace="~{'fragments/inputs/checkboxInSet' :: checkboxInSet(inputName='personaIssueDisability',value=${individual.uuid}, label=#{personal-situations.which.option4})}"/>
+                      th:replace="~{'fragments/inputs/checkboxInSet' :: checkboxInSet(inputName='personalSituationDisability',value=${individual.uuid}, label=#{personal-situations.which.option4})}"/>
                   <th:block
-                      th:replace="~{'fragments/inputs/checkboxInSet' :: checkboxInSet(inputName='otherIssue',value=${individual.uuid}, label=#{personal-situations.which.option5}, helptext=#{personal-situations.other.helptext})}"/>
+                      th:replace="~{'fragments/inputs/checkboxInSet' :: checkboxInSet(inputName='personalSituationOtherIssue',value=${individual.uuid}, label=#{personal-situations.which.option5}, helptext=#{personal-situations.other.helptext})}"/>
                 </th:block>
               </th:block>
             </div>