Fixed the case of the key for the internal feelings text and removed the debug logging.

Author: ocielliottc

server/src/main/java/com/objectcomputing/checkins/services/pulseresponse/PulseResponseController.java

@@ -25,9 +25,6 @@
 import jakarta.validation.Valid;
 import jakarta.validation.constraints.NotNull;
 
-import org.slf4j.Logger;
-import org.slf4j.LoggerFactory;
-
 import java.net.URI;
 import java.time.LocalDate;
 import java.util.Set;
@@ -39,8 +36,6 @@
 @ExecuteOn(TaskExecutors.BLOCKING)
 @Tag(name = "pulse-responses")
 public class PulseResponseController {
-    private static final Logger LOG = LoggerFactory.getLogger(PulseResponseController.class);
-
     private final PulseResponseService pulseResponseServices;
     private final MemberProfileServices memberProfileServices;
     private final SlackSignatureVerifier slackSignatureVerifier;
@@ -153,15 +148,9 @@ public HttpResponse externalPulseResponse(
                @Header("X-Slack-Request-Timestamp") String timestamp,
                @Body String requestBody,
                HttpRequest<?> request) {
-        // DEBUG Only
-        LOG.info(requestBody);
-
         // Validate the request
         if (slackSignatureVerifier.verifyRequest(signature,
                                                  timestamp, requestBody)) {
-            // DEBUG Only
-            LOG.info("Request has been verified");
-
             // Convert the request body to a map of values.
             FormUrlEncodedDecoder formUrlEncodedDecoder =
                 new FormUrlEncodedDecoder();
@@ -174,16 +163,14 @@ public HttpResponse externalPulseResponse(
                 PulseResponseCreateDTO pulseResponseDTO =
                     slackPulseResponseConverter.get(memberProfileServices,
                                                     (String)body.get(key));
+
                 // If we receive a null DTO, that means that this is not the
                 // actual submission of the form.  We can just return 200 so
                 // that Slack knows to continue without error.
                 if (pulseResponseDTO == null) {
                     return HttpResponse.ok();
                 }
 
-                // DEBUG Only
-                LOG.info("Request has been converted");
-
                 // Create the pulse response
                 PulseResponse pulseResponse =
                     pulseResponseServices.unsecureSave(

server/src/main/java/com/objectcomputing/checkins/services/pulseresponse/SlackPulseResponseConverter.java

@@ -45,8 +45,6 @@ public PulseResponseCreateDTO get(
                 final Map<String, Object> values =
                         (Map<String, Object>)state.get("values");
 
-                dumpMap(values, "");
-
                 // Create the pulse DTO and fill in the values.
                 PulseResponseCreateDTO response = new PulseResponseCreateDTO();
                 response.setTeamMemberId(lookupUser(memberProfileServices, map));
@@ -59,7 +57,7 @@ public PulseResponseCreateDTO get(
                     internalBlock, "internalScore", "selected_option", true)));
                 // Internal Feelings
                 response.setInternalFeelings(getMappedValue(
-                    values, "internaltext", "internalFeelings", false));
+                    values, "internalText", "internalFeelings", false));
 
                 // External Score
                 Map<String, Object> externalBlock =
@@ -126,15 +124,4 @@ private UUID lookupUser(MemberProfileServices memberProfileServices,
         MemberProfile member = memberProfileServices.findByWorkEmail(email);
         return member.getId();
     }
-
-    // DEBUG Only
-    private void dumpMap(Map<?, ?> map, String indent) {
-        for (Map.Entry<?, ?> entry : map.entrySet()) {
-            LOG.info(indent + entry.getKey() + " : " + entry.getValue());
-
-            if (entry.getValue() instanceof Map) {
-                dumpMap((Map<?, ?>) entry.getValue(), indent + "  ");
-            }
-        }
-    }
 }