From febed51b2cb3ac462dd6fc3a7e30e7d624e98445 Mon Sep 17 00:00:00 2001 From: modzeleg Date: Mon, 22 Mar 2021 16:14:46 +0100 Subject: [PATCH] Removed depracated code and added Javadoc --- .../java/com/release11/klaus/utilis/BusinessKey.java | 8 ++++++++ .../java/com/release11/klaus/utilis/ObjectConfig.java | 1 + .../java/com/release11/klaus/utilis/RedisAppender.java | 1 + .../com/release11/klaus/utilis/TrackingClient.java | 10 ++++++++++ 4 files changed, 20 insertions(+) diff --git a/src/main/java/com/release11/klaus/utilis/BusinessKey.java b/src/main/java/com/release11/klaus/utilis/BusinessKey.java index d647c75..6de7a14 100644 --- a/src/main/java/com/release11/klaus/utilis/BusinessKey.java +++ b/src/main/java/com/release11/klaus/utilis/BusinessKey.java @@ -2,6 +2,10 @@ package com.release11.klaus.utilis; import lombok.AllArgsConstructor; +/** + * Enum of keys for redis database. + * @author Rafał Żukowicz + */ @AllArgsConstructor public enum BusinessKey { INTERFACE_NAME("interfaceName"), @@ -10,6 +14,10 @@ public enum BusinessKey { private final String phrase; + /** + * Returns string value of given enum variant + * @return string value of enum + */ public String getReasonPhrase() { return this.phrase; } diff --git a/src/main/java/com/release11/klaus/utilis/ObjectConfig.java b/src/main/java/com/release11/klaus/utilis/ObjectConfig.java index 470708b..895456a 100644 --- a/src/main/java/com/release11/klaus/utilis/ObjectConfig.java +++ b/src/main/java/com/release11/klaus/utilis/ObjectConfig.java @@ -4,6 +4,7 @@ import com.release11.klaus.model.MockedMessageDto; import org.springframework.context.annotation.Bean; import org.springframework.context.annotation.Configuration; +//TODO: is it really necessary? kekw @Configuration public class ObjectConfig { @Bean diff --git a/src/main/java/com/release11/klaus/utilis/RedisAppender.java b/src/main/java/com/release11/klaus/utilis/RedisAppender.java index f8be5ae..a6dd5aa 100644 --- a/src/main/java/com/release11/klaus/utilis/RedisAppender.java +++ b/src/main/java/com/release11/klaus/utilis/RedisAppender.java @@ -15,6 +15,7 @@ import redis.clients.jedis.Jedis; import redis.clients.jedis.JedisPool; import redis.clients.jedis.Protocol; +//TODO: The check? Check if any of the following methods might be removed... public class RedisAppender extends UnsynchronizedAppenderBase { JedisPool pool; diff --git a/src/main/java/com/release11/klaus/utilis/TrackingClient.java b/src/main/java/com/release11/klaus/utilis/TrackingClient.java index c546e65..27fddbf 100644 --- a/src/main/java/com/release11/klaus/utilis/TrackingClient.java +++ b/src/main/java/com/release11/klaus/utilis/TrackingClient.java @@ -4,10 +4,20 @@ import org.slf4j.MDC; import java.util.Map; +//TODO: Check if constructor might be safely removed. It's static class tho +/** + * This static class has one purpose and one purpose only. It logs data about incomming requests. + * The data from logs is received via {@link com.release11.klaus.repository.EventRepositoryImpl} + * @author Rafał Żukowski + */ public final class TrackingClient { private TrackingClient(){} + /** + * Logs data inside the given map + * @param businessKeysMap map containing all the information about incomming request + */ public static void setBusinessKeys(Map businessKeysMap){ for (Map.Entry entry : businessKeysMap.entrySet()) { MDC.put(entry.getKey().getReasonPhrase(), entry.getValue());