1.3.1 - fix command bot creation

FunixG · FunixG · commit 3e73b7996c93 · 2024-02-05T00:32:28.000+01:00
diff --git a/Dockerfile b/Dockerfile
@@ -4,7 +4,7 @@ ARG service_name
 ARG service_base_dir
 ENV SERVICE_NAME=${service_name}
 ENV SERVICE_BASE_DIR=${service_base_dir}
-ENV APP_VERSION=1.3.0.10
+ENV APP_VERSION=1.3.1
 
 WORKDIR /container/java
 
diff --git a/modules/core/client/pom.xml b/modules/core/client/pom.xml
@@ -6,7 +6,7 @@
     <parent>
         <groupId>fr.funixgaming.api.core</groupId>
         <artifactId>funixgaming-core</artifactId>
-        <version>1.3.0.10</version>
+        <version>1.3.1</version>
     </parent>
 
     <groupId>fr.funixgaming.api.core.client</groupId>
diff --git a/modules/core/pom.xml b/modules/core/pom.xml
@@ -6,7 +6,7 @@
     <parent>
         <groupId>fr.funixgaming.api</groupId>
         <artifactId>funixgaming-api</artifactId>
-        <version>1.3.0.10</version>
+        <version>1.3.1</version>
         <relativePath>../../pom.xml</relativePath>
     </parent>
 
diff --git a/modules/core/service/pom.xml b/modules/core/service/pom.xml
@@ -6,7 +6,7 @@
     <parent>
         <groupId>fr.funixgaming.api.core</groupId>
         <artifactId>funixgaming-core</artifactId>
-        <version>1.3.0.10</version>
+        <version>1.3.1</version>
     </parent>
 
     <groupId>fr.funixgaming.api.core.service</groupId>
diff --git a/modules/core/test/pom.xml b/modules/core/test/pom.xml
@@ -6,7 +6,7 @@
     <parent>
         <groupId>fr.funixgaming.api.core</groupId>
         <artifactId>funixgaming-core</artifactId>
-        <version>1.3.0.10</version>
+        <version>1.3.1</version>
     </parent>
 
     <groupId>fr.funixgaming.api.core.test</groupId>
diff --git a/modules/funixbot/client/pom.xml b/modules/funixbot/client/pom.xml
@@ -6,7 +6,7 @@
     <parent>
         <groupId>fr.funixgaming.api.funixbot</groupId>
         <artifactId>funixgaming-funixbot</artifactId>
-        <version>1.3.0.10</version>
+        <version>1.3.1</version>
     </parent>
 
     <groupId>fr.funixgaming.api.funixbot.client</groupId>
diff --git a/modules/funixbot/client/src/main/java/fr/funixgaming/api/funixbot/client/dtos/FunixBotAutoMessageDTO.java b/modules/funixbot/client/src/main/java/fr/funixgaming/api/funixbot/client/dtos/FunixBotAutoMessageDTO.java
@@ -3,6 +3,7 @@
 import com.funixproductions.core.crud.dtos.ApiDTO;
 import jakarta.validation.constraints.NotBlank;
 import jakarta.validation.constraints.NotNull;
+import jakarta.validation.constraints.Size;
 import lombok.AllArgsConstructor;
 import lombok.Getter;
 import lombok.NoArgsConstructor;
@@ -20,13 +21,15 @@ public class FunixBotAutoMessageDTO extends ApiDTO {
      * The message to send.
      */
     @NotBlank(message = "Le message ne peut pas être vide.")
+    @Size(max = 500, message = "Le message ne peut pas dépasser 500 caractères.")
     private String message;
 
     /**
      * The game name to send the message to.
      * Null if you can send it anytime
      */
     @Nullable
+    @Size(max = 100, message = "Le nom du jeu ne peut pas dépasser 100 caractères.")
     private String gameName;
 
     /**
diff --git a/modules/funixbot/client/src/main/java/fr/funixgaming/api/funixbot/client/dtos/FunixBotCommandDTO.java b/modules/funixbot/client/src/main/java/fr/funixgaming/api/funixbot/client/dtos/FunixBotCommandDTO.java
@@ -4,18 +4,23 @@
 import fr.funixgaming.api.funixbot.client.enums.FunixBotCommandType;
 import jakarta.validation.constraints.NotBlank;
 import jakarta.validation.constraints.NotNull;
+import jakarta.validation.constraints.Pattern;
+import jakarta.validation.constraints.Size;
 import lombok.Getter;
 import lombok.Setter;
 
 @Getter
 @Setter
 public class FunixBotCommandDTO extends ApiDTO {
-    @NotBlank
+    @NotBlank(message = "La commande ne peut pas être vide.")
+    @Size(max = 30, message = "La commande ne peut pas dépasser 30 caractères.")
+    @Pattern(regexp = "^[a-zA-Z0-9]+$", message = "La commande ne peut contenir que des caractères alphanumériques.")
     private String command;
 
-    @NotNull
+    @NotNull(message = "Le type de commande ne peut pas être vide.")
     private FunixBotCommandType type;
 
-    @NotBlank
+    @NotBlank(message = "Le message ne peut pas être vide.")
+    @Size(max = 500, message = "Le message ne peut pas dépasser 500 caractères.")
     private String message;
 }
diff --git a/modules/funixbot/pom.xml b/modules/funixbot/pom.xml
@@ -6,7 +6,7 @@
     <parent>
         <groupId>fr.funixgaming.api</groupId>
         <artifactId>funixgaming-api</artifactId>
-        <version>1.3.0.10</version>
+        <version>1.3.1</version>
         <relativePath>../../pom.xml</relativePath>
     </parent>
 
diff --git a/modules/funixbot/service/pom.xml b/modules/funixbot/service/pom.xml
@@ -6,7 +6,7 @@
     <parent>
         <groupId>fr.funixgaming.api.funixbot</groupId>
         <artifactId>funixgaming-funixbot</artifactId>
-        <version>1.3.0.10</version>
+        <version>1.3.1</version>
     </parent>
 
     <groupId>fr.funixgaming.api.funixbot.service</groupId>
diff --git a/modules/funixbot/service/src/main/java/fr/funixgaming/api/funixbot/service/services/FunixBotAutoMessagesService.java b/modules/funixbot/service/src/main/java/fr/funixgaming/api/funixbot/service/services/FunixBotAutoMessagesService.java
@@ -12,24 +12,8 @@
 @Service
 public class FunixBotAutoMessagesService extends ApiService<FunixBotAutoMessageDTO, FunixBotAutoMessage, FunixBotAutoMessagesMapper, FunixBotAutoMessagesRepository> {
 
-    private static final int MAX_LEN_MESSAGE = 500;
-    private static final int MAX_LEN_GAME_NAME = 100;
-    private static final String ERROR_MESSAGE_GAME_NAME_MAX_LEN = String.format("Le nom du jeu ne peut pas dépasser %d caractères.", MAX_LEN_GAME_NAME);
-    private static final String ERROR_MESSAGE_MAX_LEN = String.format("Le message ne peut pas dépasser %d caractères.", MAX_LEN_MESSAGE);
-
     public FunixBotAutoMessagesService(FunixBotAutoMessagesRepository repository, FunixBotAutoMessagesMapper mapper) {
         super(repository, mapper);
     }
 
-    @Override
-    public void beforeMappingToEntity(@NonNull Iterable<FunixBotAutoMessageDTO> request) {
-        for (FunixBotAutoMessageDTO message : request) {
-            if (message.getMessage().length() > MAX_LEN_MESSAGE) {
-                throw new ApiBadRequestException(ERROR_MESSAGE_MAX_LEN);
-            }
-            if (message.getGameName() != null && message.getGameName().length() > MAX_LEN_GAME_NAME) {
-                throw new ApiBadRequestException(ERROR_MESSAGE_GAME_NAME_MAX_LEN);
-            }
-        }
-    }
 }
diff --git a/modules/funixbot/service/src/main/java/fr/funixgaming/api/funixbot/service/services/FunixBotCommandsService.java b/modules/funixbot/service/src/main/java/fr/funixgaming/api/funixbot/service/services/FunixBotCommandsService.java
@@ -15,13 +15,6 @@
 @Service
 public class FunixBotCommandsService extends ApiService<FunixBotCommandDTO, FunixBotCommand, FunixBotCommandMapper, FunixBotCommandRepository> {
 
-    private static final int MAX_LEN_MESSAGE = 500;
-    private static final int MAX_LEN_COMMAND = 30;
-    private static final String ERROR_MESSAGE_COMMAND_MAX_LEN = String.format("La commande ne peut pas dépasser %d caractères.", MAX_LEN_COMMAND);
-    private static final String ERROR_MESSAGE_MAX_LEN = String.format("Le message de la commande ne peut pas dépasser %d caractères.", MAX_LEN_MESSAGE);
-    private static final String ERROR_MESSAGE_NOT_ALPHANUMERIC = "La commande ne peut contenir que des caractères alphanumériques.";
-    private final Pattern pattern = Pattern.compile("^[a-zA-Z0-9]+$");
-
     public FunixBotCommandsService(FunixBotCommandRepository repository,
                                    FunixBotCommandMapper mapper) {
         super(repository, mapper);
@@ -30,32 +23,11 @@ public FunixBotCommandsService(FunixBotCommandRepository repository,
     @Override
     public void beforeMappingToEntity(@NonNull Iterable<FunixBotCommandDTO> request) {
         for (FunixBotCommandDTO command : request) {
-            if (command.getMessage().length() > MAX_LEN_MESSAGE) {
-                throw new ApiBadRequestException(ERROR_MESSAGE_MAX_LEN);
-            }
-
-            if (command.getCommand().startsWith("!")) {
-                command.setCommand(command.getCommand().substring(1));
-            }
-            if (command.getCommand().length() > MAX_LEN_COMMAND) {
-                throw new ApiBadRequestException(ERROR_MESSAGE_COMMAND_MAX_LEN);
-            }
-
-            if (!isAlphanumeric(command.getCommand())) {
-                throw new ApiBadRequestException(ERROR_MESSAGE_NOT_ALPHANUMERIC);
-            }
-
             if (command.getId() == null && super.getRepository().existsFunixBotCommandByCommandContainsIgnoreCase(command.getCommand())) {
                 throw new ApiBadRequestException(String.format("La commande '%s' existe déjà.", command.getCommand()));
             }
-
             command.setCommand(command.getCommand().toLowerCase());
         }
     }
 
-    private boolean isAlphanumeric(String input) {
-        Matcher matcher = pattern.matcher(input);
-        return matcher.matches();
-    }
-
 }
diff --git a/modules/funixbot/service/src/test/java/fr/funixgaming/api/funixbot/service/resources/FunixBotCommandResourceTest.java b/modules/funixbot/service/src/test/java/fr/funixgaming/api/funixbot/service/resources/FunixBotCommandResourceTest.java
@@ -76,7 +76,7 @@ void testCreateAndEditCommand() throws Exception {
 
         final String commandName = "tEst1";
         final FunixBotCommandDTO commandDTO = new FunixBotCommandDTO();
-        commandDTO.setCommand("!" + commandName);
+        commandDTO.setCommand(commandName);
         commandDTO.setMessage("testMessage");
         commandDTO.setType(FunixBotCommandType.FUN);
 
diff --git a/modules/twitch/client/pom.xml b/modules/twitch/client/pom.xml
@@ -6,7 +6,7 @@
     <parent>
         <groupId>fr.funixgaming.api.twitch</groupId>
         <artifactId>funixgaming-twitch</artifactId>
-        <version>1.3.0.10</version>
+        <version>1.3.1</version>
     </parent>
 
     <groupId>fr.funixgaming.api.twitch.client</groupId>
diff --git a/modules/twitch/pom.xml b/modules/twitch/pom.xml
@@ -6,7 +6,7 @@
     <parent>
         <groupId>fr.funixgaming.api</groupId>
         <artifactId>funixgaming-api</artifactId>
-        <version>1.3.0.10</version>
+        <version>1.3.1</version>
         <relativePath>../../pom.xml</relativePath>
     </parent>
 
diff --git a/modules/twitch/service/pom.xml b/modules/twitch/service/pom.xml
@@ -6,7 +6,7 @@
     <parent>
         <groupId>fr.funixgaming.api.twitch</groupId>
         <artifactId>funixgaming-twitch</artifactId>
-        <version>1.3.0.10</version>
+        <version>1.3.1</version>
     </parent>
 
     <groupId>fr.funixgaming.api.twitch.service</groupId>
diff --git a/pom.xml b/pom.xml
@@ -6,7 +6,7 @@
 
     <groupId>fr.funixgaming.api</groupId>
     <artifactId>funixgaming-api</artifactId>
-    <version>1.3.0.10</version>
+    <version>1.3.1</version>
     <name>FunixGamingAPI</name>
     <description>Project used in Funix projects</description>
     <url>https://github.com/FunixProductions/FunixGaming-API</url>
@@ -15,7 +15,7 @@
     <parent>
         <groupId>com.funixproductions.api</groupId>
         <artifactId>funixproductions-api</artifactId>
-        <version>1.2.2.5</version>
+        <version>1.3.0</version>
     </parent>
 
     <modules>
@@ -47,7 +47,7 @@
     </scm>
 
     <properties>
-        <fr.funixgaming.api.version>1.3.0.10</fr.funixgaming.api.version>
+        <fr.funixgaming.api.version>1.3.1</fr.funixgaming.api.version>
 
         <maven.compiler.source>${java.version}</maven.compiler.source>
         <maven.compiler.target>${java.version}</maven.compiler.target>
