🔨 Add endpoint

build.gradle

@@ -4,6 +4,7 @@ plugins {
     id 'java'
     id 'groovy'
     id 'org.liquibase.gradle' version '2.0.4'
+    id "nebula.lint" version "16.9.0"
 }
 
 
@@ -35,41 +36,33 @@ dependencies {
     implementation("org.springframework.boot:spring-boot-starter-web")
     implementation("org.springframework.boot:spring-boot-starter-security")
     implementation("org.springframework.boot:spring-boot-starter-data-jpa")
-    implementation("org.springframework.boot:spring-boot-starter-thymeleaf")
     implementation("org.springframework.boot:spring-boot-starter-log4j2")
     implementation("org.springframework.boot:spring-boot-starter-oauth2-client")
 
-    implementation("org.springdoc:springdoc-openapi-ui:1.6.8")
-    implementation("org.springdoc:springdoc-openapi-security:1.6.8")
+    implementation('org.springdoc:springdoc-openapi-ui:1.6.9')
+    implementation('org.springdoc:springdoc-openapi-security:1.6.9')
 
     implementation('org.liquibase:liquibase-core')
 
     implementation('io.jsonwebtoken:jjwt-api:0.11.5')
     implementation('io.jsonwebtoken:jjwt-impl:0.11.5')
-    implementation('io.jsonwebtoken:jjwt-gson:0.11.5')
-
-    implementation 'org.codehaus.groovy:groovy-all:3.0.8'
+    implementation('io.jsonwebtoken:jjwt-jackson:0.11.5')
 
     implementation 'com.sedmelluq:lavaplayer:1.3.77'
     implementation 'net.dv8tion:JDA:4.4.0_350'
-    implementation group: 'org.json', name: 'json', version: '20210307'
-//     JPA Data (We are going to use Repositories, Entities, Hibernate, etc...)
 
     implementation(platform("org.apache.logging.log4j:log4j-bom:2.17.1"))
     implementation group: 'org.hibernate', name: 'hibernate-validator', version: '7.0.4.Final'
 
 //     Use MySQL Connector-J
-    implementation 'mysql:mysql-connector-java'
-    implementation 'org.reflections:reflections:0.9.12'
+    implementation 'mysql:mysql-connector-java:8.0.29'
+    implementation 'org.reflections:reflections:0.10.2'
     implementation 'org.apache.commons:commons-lang3:3.12.0'
-    implementation 'com.google.api-client:google-api-client:1.31.5'
-    implementation 'com.google.apis:google-api-services-youtube:v3-rev20210410-1.31.0'
-    implementation group: 'org.jsoup', name: 'jsoup', version: '1.13.1'
 
 
-    liquibaseRuntime 'org.liquibase:liquibase-core:4.2.2'
-    liquibaseRuntime 'org.liquibase:liquibase-groovy-dsl:2.1.1'
-    liquibaseRuntime 'mysql:mysql-connector-java:5.1.34'
+    liquibaseRuntime 'org.liquibase:liquibase-core:4.8.0'
+    liquibaseRuntime 'org.liquibase:liquibase-groovy-dsl:3.0.2'
+    liquibaseRuntime 'mysql:mysql-connector-java:8.0.29'
     liquibaseRuntime group: 'javax.xml.bind', name: 'jaxb-api', version: '2.3.1'
     liquibaseRuntime group: 'org.liquibase.ext', name: 'liquibase-hibernate5', version: '4.10.0'
     liquibaseRuntime 'org.springframework.boot:spring-boot-starter-data-jpa'

src/main/java/net/Broken/Api/Controllers/AudioController.java

@@ -1,5 +1,6 @@
 package net.Broken.Api.Controllers;
 
+import net.Broken.Api.Data.Music.Add;
 import net.Broken.Api.Data.Music.Connect;
 import net.Broken.Api.Data.Music.Status;
 import net.Broken.Api.Security.Data.JwtPrincipal;
@@ -9,6 +10,8 @@ import org.springframework.security.access.prepost.PreAuthorize;
 import org.springframework.security.core.Authentication;
 import org.springframework.web.bind.annotation.*;
 
+import java.util.concurrent.ExecutionException;
+
 @RestController
 @RequestMapping("/api/v2/audio")
 @CrossOrigin(origins = "*", maxAge = 3600)
@@ -23,7 +26,7 @@ public class AudioController {
 
     @GetMapping("/{guildId}/status")
     @PreAuthorize("isInGuild(#guildId)")
-    public Status getMusicStatus(@PathVariable String guildId, Authentication authentication){
+    public Status getMusicStatus(@PathVariable String guildId, Authentication authentication) {
         JwtPrincipal principal = (JwtPrincipal) authentication.getPrincipal();
         return audioService.getGuildAudioStatus(guildId, principal.user().getDiscordId());
     }
@@ -31,7 +34,50 @@ public class AudioController {
 
     @PostMapping("/{guildId}/connect")
     @PreAuthorize("isInGuild(#guildId) && canInteractWithVoiceChannel(#guildId, #body)")
-    public ResponseEntity<String> connect(@PathVariable String guildId, @RequestBody Connect body){
-        return audioService.connect(guildId, body);
+    public ResponseEntity<Status> connect(@PathVariable String guildId, @RequestBody Connect body, Authentication authentication) {
+        JwtPrincipal principal = (JwtPrincipal) authentication.getPrincipal();
+        return audioService.connect(guildId, body, principal.user().getDiscordId());
+    }
+
+    @PostMapping("/{guildId}/disconnect")
+    @PreAuthorize("isInGuild(#guildId) && canInteractWithVoiceChannel(#guildId)")
+    public ResponseEntity<Status> disconnect(@PathVariable String guildId, Authentication authentication) {
+        JwtPrincipal principal = (JwtPrincipal) authentication.getPrincipal();
+        return audioService.disconnect(guildId, principal.user().getDiscordId());
+    }
+
+    @PostMapping("/{guildId}/resume")
+    @PreAuthorize("isInGuild(#guildId) && canInteractWithVoiceChannel(#guildId)")
+    public ResponseEntity<Status> resume(@PathVariable String guildId, Authentication authentication) {
+        JwtPrincipal principal = (JwtPrincipal) authentication.getPrincipal();
+        return audioService.resume(guildId, principal.user().getDiscordId());
+    }
+
+    @PostMapping("/{guildId}/pause")
+    @PreAuthorize("isInGuild(#guildId) && canInteractWithVoiceChannel(#guildId)")
+    public ResponseEntity<Status> pause(@PathVariable String guildId, Authentication authentication) {
+        JwtPrincipal principal = (JwtPrincipal) authentication.getPrincipal();
+        return audioService.pause(guildId, principal.user().getDiscordId());
+    }
+
+    @PostMapping("/{guildId}/skip")
+    @PreAuthorize("isInGuild(#guildId) && canInteractWithVoiceChannel(#guildId)")
+    public ResponseEntity<Status> skip(@PathVariable String guildId, Authentication authentication) {
+        JwtPrincipal principal = (JwtPrincipal) authentication.getPrincipal();
+        return audioService.skip(guildId, principal.user().getDiscordId());
+    }
+
+    @PostMapping("/{guildId}/stop")
+    @PreAuthorize("isInGuild(#guildId) && canInteractWithVoiceChannel(#guildId)")
+    public ResponseEntity<Status> stop(@PathVariable String guildId, Authentication authentication) {
+        JwtPrincipal principal = (JwtPrincipal) authentication.getPrincipal();
+        return audioService.stop(guildId, principal.user().getDiscordId());
+    }
+
+    @PostMapping("/{guildId}/add")
+    @PreAuthorize("isInGuild(#guildId) && canInteractWithVoiceChannel(#guildId)")
+    public ResponseEntity<Status> add(@PathVariable String guildId, @RequestBody Add body, Authentication authentication) throws ExecutionException, InterruptedException {
+        JwtPrincipal principal = (JwtPrincipal) authentication.getPrincipal();
+        return audioService.add(guildId, principal.user().getDiscordId(), body);
     }
 }

src/main/java/net/Broken/Api/Controllers/AuthController.java

@@ -11,8 +11,6 @@ import org.springframework.security.core.Authentication;
 import org.springframework.validation.annotation.Validated;
 import org.springframework.web.bind.annotation.*;
 
-import io.swagger.v3.oas.annotations.security.SecurityRequirements;
-
 @RestController
 @RequestMapping("/api/v2/auth")
 @CrossOrigin(origins = "*", maxAge = 3600)
@@ -27,7 +25,6 @@ public class AuthController {
     }
 
     @PostMapping("/discord")
-    @SecurityRequirements(value = {})
     public JwtResponse loginDiscord(@Validated @RequestBody Login login) {
         Authentication authentication = authenticationManager.authenticate(
                 new UsernamePasswordAuthenticationToken(login.redirectUri(), login.code())

src/main/java/net/Broken/Api/Controllers/CrossOptionController.java

@@ -1,5 +1,6 @@
 package net.Broken.Api.Controllers;
 
+import io.swagger.v3.oas.annotations.Hidden;
 import org.springframework.http.HttpStatus;
 import org.springframework.http.ResponseEntity;
 import org.springframework.web.bind.annotation.CrossOrigin;
@@ -7,8 +8,6 @@ import org.springframework.web.bind.annotation.RequestMapping;
 import org.springframework.web.bind.annotation.RequestMethod;
 import org.springframework.web.bind.annotation.RestController;
 
-import io.swagger.v3.oas.annotations.Hidden;
-
 
 @RestController
 @RequestMapping("/api/v2")
@@ -23,7 +22,7 @@ public class CrossOptionController {
             value = "/**",
             method = RequestMethod.OPTIONS
     )
-    public ResponseEntity handle() {
-        return new ResponseEntity(HttpStatus.OK);
+    public ResponseEntity<String> handle() {
+        return new ResponseEntity<>("",HttpStatus.OK);
     }
 }

src/main/java/net/Broken/Api/Controllers/GuildController.java

@@ -1,9 +1,9 @@
 package net.Broken.Api.Controllers;
 
+import net.Broken.Api.Data.Guild.Channel;
 import net.Broken.Api.Data.Guild.Guild;
 import net.Broken.Api.Data.Guild.Role;
 import net.Broken.Api.Data.InviteLink;
-import net.Broken.Api.Data.Guild.Channel;
 import net.Broken.Api.Security.Data.JwtPrincipal;
 import net.Broken.Api.Services.GuildService;
 import net.Broken.MainBot;

src/main/java/net/Broken/Api/Controllers/HelloController.java

@@ -1,11 +1,6 @@
 package net.Broken.Api.Controllers;
 
-import io.swagger.v3.oas.annotations.Operation;
-import io.swagger.v3.oas.annotations.security.SecurityRequirement;
-import io.swagger.v3.oas.annotations.security.SecurityRequirements;
 import net.Broken.Api.Security.Data.JwtPrincipal;
-import net.Broken.DB.Entity.UserEntity;
-import org.springframework.security.access.prepost.PreAuthorize;
 import org.springframework.security.core.Authentication;
 import org.springframework.web.bind.annotation.CrossOrigin;
 import org.springframework.web.bind.annotation.GetMapping;

src/main/java/net/Broken/Api/Data/Music/Add.java

@@ -0,0 +1,4 @@
+package net.Broken.Api.Data.Music;
+
+public record Add(String url) {
+}

src/main/java/net/Broken/Api/Data/Music/Status.java

@@ -2,10 +2,12 @@ package net.Broken.Api.Data.Music;
 
 import com.fasterxml.jackson.annotation.JsonInclude;
 import net.Broken.Api.Data.Guild.Channel;
+import net.dv8tion.jda.api.audio.hooks.ConnectionStatus;
 
 @JsonInclude(JsonInclude.Include.NON_NULL)
 public record Status(
         Boolean connected,
+        ConnectionStatus connectionStatus,
         Channel channel,
         Boolean canView,
         Boolean canInteract,

src/main/java/net/Broken/Api/Data/Music/TrackInfo.java

@@ -2,13 +2,9 @@ package net.Broken.Api.Data.Music;
 
 import com.sedmelluq.discord.lavaplayer.track.AudioTrackInfo;
 import net.Broken.Api.Data.UserInfo;
-import net.Broken.audio.UserAudioTrack;
+import net.Broken.Audio.UserAudioTrack;
 
 public record TrackInfo(UserInfo submitter, AudioTrackInfo detail) {
-    public TrackInfo(UserInfo submitter, AudioTrackInfo detail) {
-        this.submitter = submitter;
-        this.detail = detail;
-    }
 
     public TrackInfo(UserAudioTrack userAudioTrack) {
         this(new UserInfo(userAudioTrack.getSubmittedUser().getId(), userAudioTrack.getSubmittedUser().getName(), userAudioTrack.getSubmittedUser().getAvatarUrl()),

src/main/java/net/Broken/Api/OpenApi/OpenApiConfig.java

@@ -5,7 +5,6 @@ import io.swagger.v3.oas.models.OpenAPI;
 import io.swagger.v3.oas.models.security.SecurityRequirement;
 import io.swagger.v3.oas.models.security.SecurityScheme;
 import io.swagger.v3.oas.models.servers.Server;
-import org.springframework.beans.factory.annotation.Value;
 import org.springframework.context.annotation.Bean;
 import org.springframework.context.annotation.Configuration;
 

src/main/java/net/Broken/Api/Security/Components/UnauthorizedHandler.java

@@ -8,7 +8,6 @@ import org.springframework.security.core.AuthenticationException;
 import org.springframework.security.web.AuthenticationEntryPoint;
 import org.springframework.stereotype.Component;
 
-import javax.servlet.ServletException;
 import javax.servlet.http.HttpServletRequest;
 import javax.servlet.http.HttpServletResponse;
 import java.io.IOException;
@@ -20,7 +19,7 @@ public class UnauthorizedHandler implements AuthenticationEntryPoint {
     private final Logger logger = LogManager.getLogger();
 
     @Override
-    public void commence(HttpServletRequest request, HttpServletResponse response, AuthenticationException authException) throws IOException, ServletException {
+    public void commence(HttpServletRequest request, HttpServletResponse response, AuthenticationException authException) throws IOException {
         logger.error("[API] Unauthorized error: {}", authException.getMessage());
 
         response.setContentType(MediaType.APPLICATION_JSON_VALUE);

src/main/java/net/Broken/Api/Security/Data/AccessTokenResponse.java

@@ -1,7 +1,5 @@
 package net.Broken.Api.Security.Data;
 
-import com.fasterxml.jackson.annotation.JsonIgnoreProperties;
-
 public record AccessTokenResponse(
         String access_token,
         String token_type,

src/main/java/net/Broken/Api/Security/Data/JwtResponse.java

@@ -1,7 +1,5 @@
 package net.Broken.Api.Security.Data;
 
-import com.fasterxml.jackson.annotation.JsonIgnoreProperties;
-
 
 public record JwtResponse(String token) {
 }

src/main/java/net/Broken/Api/Security/Expression/CustomMethodSecurityExpressionHandler.java

@@ -1,6 +1,5 @@
 package net.Broken.Api.Security.Expression;
 
-import net.Broken.Api.Security.Expression.CustomMethodSecurityExpressionRoot;
 import org.aopalliance.intercept.MethodInvocation;
 import org.springframework.security.access.expression.method.DefaultMethodSecurityExpressionHandler;
 import org.springframework.security.access.expression.method.MethodSecurityExpressionOperations;
@@ -10,7 +9,7 @@ import org.springframework.security.core.Authentication;
 
 public class CustomMethodSecurityExpressionHandler
         extends DefaultMethodSecurityExpressionHandler {
-    private AuthenticationTrustResolver trustResolver =
+    private final AuthenticationTrustResolver trustResolver =
             new AuthenticationTrustResolverImpl();
 
     @Override

src/main/java/net/Broken/Api/Security/Expression/CustomMethodSecurityExpressionRoot.java

@@ -2,13 +2,13 @@ package net.Broken.Api.Security.Expression;
 
 import net.Broken.Api.Data.Music.Connect;
 import net.Broken.Api.Security.Data.JwtPrincipal;
+import net.Broken.Audio.GuildAudioBotService;
 import net.Broken.MainBot;
 import net.Broken.Tools.CacheTools;
 import net.dv8tion.jda.api.Permission;
 import net.dv8tion.jda.api.entities.Guild;
 import net.dv8tion.jda.api.entities.Member;
 import net.dv8tion.jda.api.entities.VoiceChannel;
-import okhttp3.Cache;
 import org.springframework.security.access.expression.SecurityExpressionRoot;
 import org.springframework.security.access.expression.method.MethodSecurityExpressionOperations;
 import org.springframework.security.core.Authentication;
@@ -58,6 +58,23 @@ public class CustomMethodSecurityExpressionRoot
                 && member.hasPermission(channel, Permission.VOICE_SPEAK);
     }
 
+    public boolean canInteractWithVoiceChannel(String guildId) {
+        JwtPrincipal jwtPrincipal = (JwtPrincipal) authentication.getPrincipal();
+        Guild guild = MainBot.jda.getGuildById(guildId);
+        GuildAudioBotService guildAudioBotService = GuildAudioBotService.getInstance(guild);
+        VoiceChannel channel = guild.getAudioManager().getConnectedChannel();
+
+        if (channel == null) {
+            return false;
+        }
+
+        Member member = guild.getMemberById(jwtPrincipal.user().getDiscordId());
+        return (member.hasPermission(channel, Permission.VOICE_CONNECT)
+                || member.getVoiceState() != null
+                && member.getVoiceState().getChannel() == channel)
+                && member.hasPermission(channel, Permission.VOICE_SPEAK);
+    }
+
     @Override
     public void setFilterObject(Object filterObject) {
         this.filterObject = filterObject;

src/main/java/net/Broken/Api/Security/SecurityConfig.java

@@ -33,9 +33,6 @@ public class SecurityConfig extends WebSecurityConfigurerAdapter {
 
         http.addFilterBefore(jwtFilter(), UsernamePasswordAuthenticationFilter.class);
 
-//        http.exceptionHandling().authenticationEntryPoint((request, response, authException) -> {
-//            response.sendError(HttpServletResponse.SC_UNAUTHORIZED);
-//        });
     }
 
     @Bean

src/main/java/net/Broken/Api/Security/Services/DiscordOauthService.java

@@ -12,7 +12,6 @@ import org.springframework.beans.factory.annotation.Value;
 import org.springframework.stereotype.Service;
 
 import java.io.IOException;
-import java.io.UnsupportedEncodingException;
 import java.net.URI;
 import java.net.URLEncoder;
 import java.net.http.HttpClient;
@@ -143,7 +142,7 @@ public class DiscordOauthService {
     }
 
 
-    private String getFormString(HashMap<String, String> params) throws UnsupportedEncodingException {
+    private String getFormString(HashMap<String, String> params) {
         StringBuilder result = new StringBuilder();
         boolean first = true;
         for (Map.Entry<String, String> entry : params.entrySet()) {

src/main/java/net/Broken/Api/Security/Services/JwtService.java

@@ -1,7 +1,10 @@
 package net.Broken.Api.Security.Services;
 
 
-import io.jsonwebtoken.*;
+import io.jsonwebtoken.Claims;
+import io.jsonwebtoken.Jws;
+import io.jsonwebtoken.Jwts;
+import io.jsonwebtoken.SignatureAlgorithm;
 import io.jsonwebtoken.security.Keys;
 import net.Broken.DB.Entity.UserEntity;
 import net.Broken.DB.Repository.UserRepository;

src/main/java/net/Broken/Api/Services/AudioService.java

@@ -1,81 +1,131 @@
 package net.Broken.Api.Services;
 
 import net.Broken.Api.Data.Guild.Channel;
-import net.Broken.Api.Data.Music.Connect;
-import net.Broken.Api.Data.Music.PlayBackInfo;
-import net.Broken.Api.Data.Music.Status;
-import net.Broken.Api.Data.Music.TrackInfo;
+import net.Broken.Api.Data.Music.*;
+import net.Broken.Audio.GuildAudioBotService;
+import net.Broken.Audio.UserAudioTrack;
 import net.Broken.MainBot;
-import net.Broken.audio.AudioM;
-import net.Broken.audio.UserAudioTrack;
 import net.dv8tion.jda.api.Permission;
+import net.dv8tion.jda.api.audio.hooks.ConnectionStatus;
 import net.dv8tion.jda.api.entities.Guild;
-import net.dv8tion.jda.api.entities.GuildVoiceState;
 import net.dv8tion.jda.api.entities.Member;
 import net.dv8tion.jda.api.entities.VoiceChannel;
+import org.apache.logging.log4j.LogManager;
+import org.apache.logging.log4j.Logger;
 import org.springframework.http.HttpStatus;
 import org.springframework.http.ResponseEntity;
 import org.springframework.stereotype.Service;
 
+import java.util.concurrent.ExecutionException;
+
 @Service
 public class AudioService {
+
+    final Logger logger = LogManager.getLogger();
+
     public Status getGuildAudioStatus(String guildId, String userId) {
         Guild guild = MainBot.jda.getGuildById(guildId);
         Member member = guild.getMemberById(userId);
-        GuildVoiceState voiceState = guild.getSelfMember().getVoiceState();
 
-        if (voiceState != null) {
-            VoiceChannel channel = voiceState.getChannel();
-            if (channel != null) {
+
+        VoiceChannel channel = guild.getAudioManager().getConnectedChannel();
+        ConnectionStatus status = guild.getAudioManager().getConnectionStatus();
+        if (channel != null) {
 //              The user can view the audio status if:
 //                  -> He can view the voice channel
 //                  -> OR He can *not* view the voice channel, but he is connected to this voice channel
-                boolean canView = member.hasPermission(channel, Permission.VIEW_CHANNEL)
-                        || (member.getVoiceState() != null
-                        && member.getVoiceState().getChannel() == channel);
-                if (canView) {
+            boolean canView = member.hasPermission(channel, Permission.VIEW_CHANNEL)
+                    || (member.getVoiceState() != null
+                    && member.getVoiceState().getChannel() == channel);
+            GuildAudioBotService guildAudioBotService = GuildAudioBotService.getInstance(guild);
+
+            if (canView) {
 //                  The user can interact with the audio if:
 //                      -> He can connect to this voice channel
 //                          -> OR he is connected to this voice channel
 //                      -> AND He can speak in this voice channel
-                    boolean canInteract = (member.hasPermission(channel, Permission.VOICE_CONNECT)
-                            || member.getVoiceState() != null
-                            && member.getVoiceState().getChannel() == channel)
-                            && member.hasPermission(channel, Permission.VOICE_SPEAK);
+                boolean canInteract = (member.hasPermission(channel, Permission.VOICE_CONNECT)
+                        || member.getVoiceState() != null
+                        && member.getVoiceState().getChannel() == channel)
+                        && member.hasPermission(channel, Permission.VOICE_SPEAK);
 
-                    AudioM audioM = AudioM.getInstance(guild);
-                    boolean stopped = audioM.getGuildAudioPlayer().player.getPlayingTrack() == null;
-                    PlayBackInfo playBackInfo;
-                    if (!stopped) {
-                        boolean paused = audioM.getGuildAudioPlayer().player.isPaused();
-                        long position = audioM.getGuildAudioPlayer().player.getPlayingTrack().getPosition();
-                        UserAudioTrack userAudioTrack = audioM.getGuildAudioPlayer().scheduler.getCurrentPlayingTrack();
 
-                        playBackInfo = new PlayBackInfo(paused, false, position, new TrackInfo(userAudioTrack));
+                boolean stopped = guildAudioBotService.getGuidAudioManager().player.getPlayingTrack() == null;
+                PlayBackInfo playBackInfo;
+                if (!stopped) {
+                    boolean paused = guildAudioBotService.getGuidAudioManager().player.isPaused();
+                    long position = guildAudioBotService.getGuidAudioManager().player.getPlayingTrack().getPosition();
+                    UserAudioTrack userAudioTrack = guildAudioBotService.getGuidAudioManager().scheduler.getCurrentPlayingTrack();
 
-                    } else {
-                        playBackInfo = new PlayBackInfo(false, true, null, null);
-                    }
-                    Channel channelApi = new Channel(channel.getId(), channel.getName());
-                    return new Status(true, channelApi, true, canInteract, playBackInfo);
+                    playBackInfo = new PlayBackInfo(paused, false, position, new TrackInfo(userAudioTrack));
 
                 } else {
-                    return new Status(true, null, false, false, null);
+                    playBackInfo = new PlayBackInfo(false, true, null, null);
                 }
+                Channel channelApi = new Channel(channel.getId(), channel.getName());
+                return new Status(true, status, channelApi, true, canInteract, playBackInfo);
 
-
+            } else {
+                return new Status(true, status, null, false, false, null);
             }
         }
-        return new Status(false, null, null, null, null);
+        return new Status(false, status, null, null, null, null);
     }
-    
-    public ResponseEntity<String> connect(String guildId, Connect body){
+
+    public ResponseEntity<Status> connect(String guildId, Connect body, String userId) {
         Guild guild = MainBot.jda.getGuildById(guildId);
-        AudioM audioM = AudioM.getInstance(guild);
         VoiceChannel voiceChannel = guild.getVoiceChannelById(body.channelId());
-        audioM.getGuildAudioPlayer();
-        guild.getAudioManager().openAudioConnection(voiceChannel);
-        audioM.setPlayedChanel(voiceChannel);
-        return new ResponseEntity<>("OK", HttpStatus.OK);
+        GuildAudioBotService.getInstance(guild).connect(voiceChannel);
+        Status status = getGuildAudioStatus(guildId, userId);
+        return new ResponseEntity<>(status, HttpStatus.OK);
+    }
+
+    public ResponseEntity<Status> disconnect(String guildId, String userId) {
+        Guild guild = MainBot.jda.getGuildById(guildId);
+        GuildAudioBotService guildAudioBotService = GuildAudioBotService.getInstance(guild);
+        guildAudioBotService.disconnect();
+        Status status = getGuildAudioStatus(guildId, userId);
+        return new ResponseEntity<>(status, HttpStatus.OK);
+    }
+
+
+    public ResponseEntity<Status> pause(String guildId, String userId) {
+        Guild guild = MainBot.jda.getGuildById(guildId);
+        GuildAudioBotService.getInstance(guild).pause();
+        Status status = getGuildAudioStatus(guildId, userId);
+        return new ResponseEntity<>(status, HttpStatus.OK);
+    }
+
+    public ResponseEntity<Status> resume(String guildId, String userId) {
+        Guild guild = MainBot.jda.getGuildById(guildId);
+        GuildAudioBotService.getInstance(guild).resume();
+        Status status = getGuildAudioStatus(guildId, userId);
+        return new ResponseEntity<>(status, HttpStatus.OK);
+    }
+
+    public ResponseEntity<Status> skip(String guildId, String userId) {
+        Guild guild = MainBot.jda.getGuildById(guildId);
+        GuildAudioBotService.getInstance(guild).skipTrack();
+        Status status = getGuildAudioStatus(guildId, userId);
+        return new ResponseEntity<>(status, HttpStatus.OK);
+    }
+
+    public ResponseEntity<Status> stop(String guildId, String userId) {
+        Guild guild = MainBot.jda.getGuildById(guildId);
+        GuildAudioBotService.getInstance(guild).stop();
+        Status status = getGuildAudioStatus(guildId, userId);
+        return new ResponseEntity<>(status, HttpStatus.OK);
+    }
+
+    public ResponseEntity<Status> add(String guildId, String userId, Add body) throws ExecutionException, InterruptedException {
+        Guild guild = MainBot.jda.getGuildById(guildId);
+        boolean success = GuildAudioBotService.getInstance(guild).loadAndPlaySync(body.url(), userId);
+        if (success) {
+            Status status = getGuildAudioStatus(guildId, userId);
+            return new ResponseEntity<>(status, HttpStatus.OK);
+        } else {
+            return new ResponseEntity<>(null, HttpStatus.BAD_REQUEST);
+        }
+
     }
 }

src/main/java/net/Broken/Api/Services/GuildService.java

@@ -1,11 +1,10 @@
 package net.Broken.Api.Services;
 
-import net.Broken.Api.Data.Guild.Guild;
 import net.Broken.Api.Data.Guild.Channel;
+import net.Broken.Api.Data.Guild.Guild;
 import net.Broken.Api.Data.Guild.Role;
 import net.Broken.DB.Entity.UserEntity;
 import net.Broken.MainBot;
-import net.Broken.Tools.CacheTools;
 import net.dv8tion.jda.api.Permission;
 import net.dv8tion.jda.api.entities.Member;
 import net.dv8tion.jda.api.entities.User;

src/main/java/net/Broken/Api/Services/SettingService.java

@@ -1,7 +1,5 @@
 package net.Broken.Api.Services;
 
-import liquibase.pro.packaged.V;
-import net.Broken.Api.Data.Settings.SettingDescriber;
 import net.Broken.Api.Data.Settings.SettingGroup;
 import net.Broken.Api.Data.Settings.Value;
 import net.Broken.DB.Entity.GuildPreferenceEntity;
@@ -13,7 +11,6 @@ import org.apache.logging.log4j.LogManager;
 import org.apache.logging.log4j.Logger;
 import org.springframework.stereotype.Service;
 
-import java.util.ArrayList;
 import java.util.List;
 
 @Service

src/main/java/net/Broken/Audio/AudioPlayerSendHandler.java

@@ -1,4 +1,4 @@
-package net.Broken.audio;
+package net.Broken.Audio;
 
 import com.sedmelluq.discord.lavaplayer.player.AudioPlayer;
 import com.sedmelluq.discord.lavaplayer.track.playback.MutableAudioFrame;
@@ -33,7 +33,7 @@ public class AudioPlayerSendHandler implements AudioSendHandler {
 
     @Override
     public ByteBuffer provide20MsAudio() {
-        return (ByteBuffer) buffer.flip();
+        return buffer.flip();
     }
 
     @Override

src/main/java/net/Broken/Audio/GuildAudioBotService.java

@@ -1,4 +1,4 @@
-package net.Broken.audio;
+package net.Broken.Audio;
 
 import com.sedmelluq.discord.lavaplayer.player.AudioLoadResultHandler;
 import com.sedmelluq.discord.lavaplayer.player.AudioPlayerManager;
@@ -9,7 +9,6 @@ import com.sedmelluq.discord.lavaplayer.track.AudioPlaylist;
 import com.sedmelluq.discord.lavaplayer.track.AudioTrack;
 import com.sedmelluq.discord.lavaplayer.track.AudioTrackInfo;
 import net.Broken.MainBot;
-import net.Broken.RestApi.Data.UserAudioTrackData;
 import net.Broken.Tools.EmbedMessageUtils;
 import net.dv8tion.jda.api.EmbedBuilder;
 import net.dv8tion.jda.api.MessageBuilder;
@@ -18,76 +17,61 @@ import net.dv8tion.jda.api.events.interaction.GenericInteractionCreateEvent;
 import net.dv8tion.jda.api.events.interaction.SlashCommandEvent;
 import net.dv8tion.jda.api.interactions.components.ActionRow;
 import net.dv8tion.jda.api.interactions.components.Button;
-import net.dv8tion.jda.api.interactions.components.ComponentLayout;
 import org.apache.logging.log4j.LogManager;
 import org.apache.logging.log4j.Logger;
 
 import java.awt.*;
-import java.util.ArrayList;
-import java.util.Collections;
-import java.util.HashMap;
+import java.util.*;
 import java.util.List;
+import java.util.concurrent.ExecutionException;
+import java.util.concurrent.Future;
 
-public class AudioM {
+public class GuildAudioBotService {
+
+    private static final HashMap<Guild, GuildAudioBotService> INSTANCES = new HashMap<>();
+
+    private final GuildAudioManager guildAudioManager;
+
+    private final AudioPlayerManager audioPlayerManager;
 
-    private static HashMap<Guild, AudioM> INSTANCES = new HashMap<>();
-    /**
-     * Music manager for this guild
-     */
-    private GuildMusicManager musicManager;
-    /**
-     * Audio player manager for this guild
-     */
-    private AudioPlayerManager playerManager;
-    /**
-     * Current voice chanel (null if not connected)
-     */
-    private VoiceChannel playedChanel;
-    /**
-     * Time out for list message
-     */
-    private int listTimeOut = 30;
     /**
      * Extrem limit for playlist
      */
-    private int listExtremLimit = 300;
-    /**
-     * Current guild
-     */
-    private Guild guild;
-    private Logger logger = LogManager.getLogger();
+    private final int listExtremLimit = 300;
+
+    private final Guild guild;
+    private final Logger logger = LogManager.getLogger();
+
+    private final Map<String, Boolean> addStatus = new HashMap<>();
 
     private Message lastMessageWithButton;
 
-    private AudioM(Guild guild) {
-        this.playerManager = new DefaultAudioPlayerManager();
-        AudioSourceManagers.registerRemoteSources(playerManager);
-        AudioSourceManagers.registerLocalSource(playerManager);
+    private GuildAudioBotService(Guild guild) {
+        this.audioPlayerManager = new DefaultAudioPlayerManager();
+        AudioSourceManagers.registerRemoteSources(audioPlayerManager);
+        AudioSourceManagers.registerLocalSource(audioPlayerManager);
+        this.guildAudioManager = new GuildAudioManager(audioPlayerManager, guild);
+        guild.getAudioManager().setSendingHandler(guildAudioManager.getSendHandler());
         this.guild = guild;
     }
 
-    public static AudioM getInstance(Guild guild) {
+    public static GuildAudioBotService getInstance(Guild guild) {
         if (!INSTANCES.containsKey(guild)) {
-            INSTANCES.put(guild, new AudioM(guild));
+            INSTANCES.put(guild, new GuildAudioBotService(guild));
         }
-
         return INSTANCES.get(guild);
     }
 
     /**
      * Load audio track from url, connect to chanel if not connected
      *
-     * @param event
      * @param voiceChannel  Voice channel to connect if no connected
      * @param trackUrl      Audio track url
      * @param playlistLimit Limit of playlist
      * @param onHead        True for adding audio track on top of playlist
      */
     public void loadAndPlay(SlashCommandEvent event, VoiceChannel voiceChannel, final String trackUrl, int playlistLimit, boolean onHead) {
-        GuildMusicManager musicManager = getGuildAudioPlayer();
-        playedChanel = voiceChannel;
-
-        playerManager.loadItemOrdered(musicManager, trackUrl, new AudioLoadResultHandler() {
+        audioPlayerManager.loadItemOrdered(guildAudioManager, trackUrl, new AudioLoadResultHandler() {
             @Override
             public void trackLoaded(AudioTrack track) {
                 logger.info("[" + guild + "] Single Track detected!");
@@ -97,7 +81,7 @@ public class AudioM {
                         .build();
                 clearLastButton();
                 lastMessageWithButton = event.getHook().sendMessage(message).addActionRow(getActionButton()).complete();
-                play(guild, voiceChannel, musicManager, uat, onHead);
+                play(guild, voiceChannel, guildAudioManager, uat, onHead);
             }
 
             @Override
@@ -130,34 +114,43 @@ public class AudioM {
         });
     }
 
-    public void loadAndPlayAuto(String trackUrl) {
-        playerManager.loadItemOrdered(musicManager, trackUrl, new AudioLoadResultHandler() {
+    public boolean loadAndPlaySync(String trackUrl, String userId) throws ExecutionException, InterruptedException {
+        Member member = guild.getMemberById(userId);
+        VoiceChannel playedChanel = guild.getAudioManager().getConnectedChannel();
+        final String uuid = UUID.randomUUID().toString();
+        Future<Void> future = audioPlayerManager.loadItemOrdered(guildAudioManager, trackUrl, new AudioLoadResultHandler() {
             @Override
             public void trackLoaded(AudioTrack track) {
                 logger.info("[" + guild + "] Auto add " + track.getInfo().title + " to playlist.");
-                UserAudioTrack userAudioTrack = new UserAudioTrack(MainBot.jda.getSelfUser(), track);
-                play(guild, playedChanel, musicManager, userAudioTrack, true);
+                UserAudioTrack userAudioTrack = new UserAudioTrack(member.getUser(), track);
+                play(guild, playedChanel, guildAudioManager, userAudioTrack, true);
+                addStatus.put(uuid, true);
             }
 
             @Override
             public void playlistLoaded(AudioPlaylist playlist) {
                 AudioTrack track = playlist.getTracks().get(0);
                 logger.info("[" + guild + "] Auto add " + track.getInfo().title + " to playlist.");
-                UserAudioTrack userAudioTrack = new UserAudioTrack(MainBot.jda.getSelfUser(), track);
-                play(guild, playedChanel, musicManager, userAudioTrack, true);
+                UserAudioTrack userAudioTrack = new UserAudioTrack(member.getUser(), track);
+                play(guild, playedChanel, guildAudioManager, userAudioTrack, true);
+                addStatus.put(uuid, true);
             }
 
             @Override
             public void noMatches() {
                 logger.warn("[" + guild + "] Track not found: " + trackUrl);
+                addStatus.put(uuid, false);
             }
 
             @Override
             public void loadFailed(FriendlyException exception) {
                 logger.error("[" + guild + "] Cant load media!");
                 logger.error(exception.getMessage());
+                addStatus.put(uuid, false);
             }
         });
+        future.get();
+        return addStatus.remove(uuid);
     }
 
 
@@ -170,31 +163,23 @@ public class AudioM {
      * @param onHead        True for adding audio track on top of playlist
      */
     public void playListLoader(AudioPlaylist playlist, int playlistLimit, User user, boolean onHead) {
-        int i = 0;
+
+        VoiceChannel playedChanel = guild.getAudioManager().getConnectedChannel();
         List<AudioTrack> tracks = playlist.getTracks();
         if (onHead)
             Collections.reverse(tracks);
 
+        int i = 0;
         for (AudioTrack track : playlist.getTracks()) {
             UserAudioTrack uat = new UserAudioTrack(user, track);
-            play(guild, playedChanel, musicManager, uat, onHead);
-            i++;
-            if ((i >= playlistLimit && i != -1) || i > listExtremLimit)
+            play(guild, playedChanel, guildAudioManager, uat, onHead);
+            if ((playlistLimit != -1 && i >= playlistLimit) || i > listExtremLimit)
                 break;
+            i++;
         }
     }
 
 
-    public GuildMusicManager getGuildAudioPlayer() {
-        if (musicManager == null) {
-            musicManager = new GuildMusicManager(playerManager, guild);
-        }
-
-        guild.getAudioManager().setSendingHandler(musicManager.getSendHandler());
-
-        return musicManager;
-    }
-
     /**
      * Add single track to playlist, auto-connect if not connected to vocal chanel
      *
@@ -204,7 +189,7 @@ public class AudioM {
      * @param track        Track to add to playlist
      * @param onHead       True for adding audio track on top of playlist
      */
-    public void play(Guild guild, VoiceChannel channel, GuildMusicManager musicManager, UserAudioTrack track, boolean onHead) {
+    public void play(Guild guild, VoiceChannel channel, GuildAudioManager musicManager, UserAudioTrack track, boolean onHead) {
         if (!guild.getAudioManager().isConnected())
             guild.getAudioManager().openAudioConnection(channel);
         if (!onHead)
@@ -213,32 +198,21 @@ public class AudioM {
             musicManager.scheduler.addNext(track);
     }
 
-    /**
-     * Skip current track
-     *
-     * @param event
-     */
-    public void skipTrack(GenericInteractionCreateEvent event) {
-        GuildMusicManager musicManager = getGuildAudioPlayer();
-        musicManager.scheduler.nextTrack();
-        Message message = new MessageBuilder().setEmbeds(
-                EmbedMessageUtils.buildStandar(
-                        new EmbedBuilder()
-                                .setTitle(":track_next:  Next Track")
-                                .setColor(Color.green)
-                )).build();
-        clearLastButton();
-        lastMessageWithButton = event.getHook().sendMessage(message).addActionRow(getActionButton()).complete();
+    public void add(SlashCommandEvent event, String url, int playListLimit, boolean onHead) {
+        if (guild.getAudioManager().isConnected()) {
+            loadAndPlay(event, guild.getAudioManager().getConnectedChannel(), url, playListLimit, onHead);
+        } else {
+            Message message = new MessageBuilder().setEmbeds(EmbedMessageUtils.getMusicError("Not connected to vocal chanel !")).build();
+            event.getHook().setEphemeral(true).sendMessage(message).queue();
+        }
+    }
+
+    public void connect(VoiceChannel voiceChannel) {
+        guild.getAudioManager().openAudioConnection(voiceChannel);
     }
 
-    /**
-     * Pause current track
-     *
-     * @param event
-     */
     public void pause(GenericInteractionCreateEvent event) {
-        GuildMusicManager musicManager = getGuildAudioPlayer();
-        musicManager.scheduler.pause();
+        pause();
         Message message = new MessageBuilder().setEmbeds(
                 EmbedMessageUtils.buildStandar(
                         new EmbedBuilder()
@@ -247,27 +221,23 @@ public class AudioM {
                 )).build();
         clearLastButton();
         lastMessageWithButton = event.getHook().sendMessage(message).addActionRow(getActionButton()).complete();
-
-
     }
 
-    /**
-     * Resume paused track
-     *
-     * @param event
-     */
+    public void pause() {
+        guildAudioManager.scheduler.pause();
+    }
+
     public void resume(GenericInteractionCreateEvent event) {
-        GuildMusicManager musicManager = getGuildAudioPlayer();
         Message message;
-        if(musicManager.player.getPlayingTrack() == null){
+        if (guildAudioManager.player.getPlayingTrack() == null) {
             message = new MessageBuilder().setEmbeds(
                     EmbedMessageUtils.buildStandar(
                             new EmbedBuilder()
                                     .setTitle(":warning:  Nothing to play, playlist is empty !")
                                     .setColor(Color.green)
                     )).build();
-        }else{
-            musicManager.scheduler.resume();
+        } else {
+            resume();
             message = new MessageBuilder().setEmbeds(
                     EmbedMessageUtils.buildStandar(
                             new EmbedBuilder()
@@ -279,23 +249,73 @@ public class AudioM {
         lastMessageWithButton = event.getHook().sendMessage(message).addActionRow(getActionButton()).complete();
     }
 
-    /**
-     * Print current played track info
-     *
-     * @param event
-     */
+    public void resume() {
+        guildAudioManager.scheduler.resume();
+    }
+
+    public void skipTrack(GenericInteractionCreateEvent event) {
+        skipTrack();
+        Message message = new MessageBuilder().setEmbeds(
+                EmbedMessageUtils.buildStandar(
+                        new EmbedBuilder()
+                                .setTitle(":track_next:  Next Track")
+                                .setColor(Color.green)
+                )).build();
+        clearLastButton();
+        lastMessageWithButton = event.getHook().sendMessage(message).addActionRow(getActionButton()).complete();
+    }
+
+    public void skipTrack() {
+        guildAudioManager.scheduler.nextTrack();
+    }
+
+    public void stop(GenericInteractionCreateEvent event) {
+        stop();
+        Message message = new MessageBuilder().setEmbeds(
+                EmbedMessageUtils.buildStandar(
+                        new EmbedBuilder()
+                                .setTitle(":stop_button:  Playback stopped")
+                                .setColor(Color.green)
+                )).build();
+        clearLastButton();
+        lastMessageWithButton = event.getHook().sendMessage(message).addActionRow(getActionButton()).complete();
+
+    }
+
+    public void stop() {
+        guildAudioManager.scheduler.stop();
+        guildAudioManager.scheduler.flush();
+        clearLastButton();
+    }
+
+    public void disconnect(GenericInteractionCreateEvent event) {
+        disconnect();
+        Message message = new MessageBuilder().setEmbeds(
+                EmbedMessageUtils.buildStandar(
+                        new EmbedBuilder()
+                                .setTitle(":eject:  Disconnected")
+                                .setColor(Color.green)
+                )).build();
+        clearLastButton();
+        event.getHook().sendMessage(message).queue();
+    }
+
+    public void disconnect() {
+        guildAudioManager.scheduler.stop();
+        guildAudioManager.scheduler.flush();
+        guild.getAudioManager().closeAudioConnection();
+        clearLastButton();
+    }
     public void info(GenericInteractionCreateEvent event) {
-        GuildMusicManager musicManager = getGuildAudioPlayer();
-        AudioTrackInfo info = musicManager.scheduler.getInfo();
-        UserAudioTrack userAudioTrack = musicManager.scheduler.getCurrentPlayingTrack();
+        AudioTrackInfo info = guildAudioManager.scheduler.getInfo();
+        UserAudioTrack userAudioTrack = guildAudioManager.scheduler.getCurrentPlayingTrack();
         Message message = new MessageBuilder().setEmbeds(EmbedMessageUtils.getMusicInfo(info, userAudioTrack)).build();
         clearLastButton();
         lastMessageWithButton = event.getHook().sendMessage(message).addActionRow(getActionButton()).complete();
     }
 
     public void flush(GenericInteractionCreateEvent event) {
-        GuildMusicManager musicManager = getGuildAudioPlayer();
-        musicManager.scheduler.flush();
+        guildAudioManager.scheduler.flush();
         Message message = new MessageBuilder().setEmbeds(
                 EmbedMessageUtils.buildStandar(
                         new EmbedBuilder()
@@ -312,8 +332,7 @@ public class AudioM {
      * @param event
      */
     public void list(GenericInteractionCreateEvent event) {
-        GuildMusicManager musicManager = getGuildAudioPlayer();
-        List<UserAudioTrackData> list = musicManager.scheduler.getList();
+        List<UserAudioTrack> list = guildAudioManager.scheduler.getList();
 
         if (list.size() == 0) {
             Message message = new MessageBuilder().setEmbeds(
@@ -327,11 +346,11 @@ public class AudioM {
         } else {
             StringBuilder resp = new StringBuilder();
             int i = 0;
-            for (UserAudioTrackData trackInfo : list) {
+            for (UserAudioTrack trackInfo : list) {
                 resp.append(":arrow_right:  ");
-                resp.append(trackInfo.getAudioTrackInfo().title);
+                resp.append(trackInfo.getAudioTrack().getInfo().title);
                 resp.append(" - ");
-                resp.append(trackInfo.getAudioTrackInfo().author);
+                resp.append(trackInfo.getAudioTrack().getInfo().author);
                 resp.append("\n\n");
                 if (i >= 5) {
                     resp.append(":arrow_forward: And ");
@@ -353,123 +372,41 @@ public class AudioM {
 
     }
 
-    /**
-     * Called by //add, only if already connected
-     *
-     * @param event
-     * @param url           Audio track url
-     * @param playListLimit Limit of playlist
-     * @param onHead        True for adding audio track on top of playlist
-     */
-    public void add(SlashCommandEvent event, String url, int playListLimit, boolean onHead) {
-        if (playedChanel != null) {
-            loadAndPlay(event, playedChanel, url, playListLimit, onHead);
-        } else {
-            Message message = new MessageBuilder().setEmbeds(EmbedMessageUtils.getMusicError("Not connected to vocal chanel !")).build();
-            event.getHook().setEphemeral(true).sendMessage(message).queue();
-        }
-    }
-
-    /**
-     * Stop current playing track and flush playlist
-     *
-     * @param event
-     */
-    public void stop(GenericInteractionCreateEvent event) {
-        musicManager.scheduler.stop();
-        musicManager.scheduler.flush();
-
-        if (event != null) {
-            Message message = new MessageBuilder().setEmbeds(
-                    EmbedMessageUtils.buildStandar(
-                            new EmbedBuilder()
-                                    .setTitle(":stop_button:  Playback stopped")
-                                    .setColor(Color.green)
-                    )).build();
-            clearLastButton();
-            lastMessageWithButton = event.getHook().sendMessage(message).addActionRow(getActionButton()).complete();
-        }
-    }
-
-    public void disconect(GenericInteractionCreateEvent event){
-        GuildMusicManager musicManager = getGuildAudioPlayer();
-        musicManager.scheduler.stop();
-        musicManager.scheduler.flush();
-        playedChanel = null;
-        guild.getAudioManager().closeAudioConnection();
-        clearLastButton();
-        Message message = new MessageBuilder().setEmbeds(
-                EmbedMessageUtils.buildStandar(
-                        new EmbedBuilder()
-                                .setTitle(":eject:  Disconnected")
-                                .setColor(Color.green)
-                )).build();
-        clearLastButton();
-        event.getHook().sendMessage(message).queue();
-    }
-
-    /**
-     * Stop current playing track and flush playlist (no confirmation message)
-     */
-    public void stop() {
-
-        GuildMusicManager musicManager = getGuildAudioPlayer();
-        musicManager.scheduler.stop();
-        musicManager.scheduler.flush();
-        playedChanel = null;
-        guild.getAudioManager().closeAudioConnection();
-        clearLastButton();
-    }
-
-    public GuildMusicManager getGuildMusicManager() {
-        if (musicManager == null)
-            musicManager = getGuildAudioPlayer();
-        return musicManager;
-
-    }
-
     public Guild getGuild() {
         return guild;
     }
 
-    public AudioPlayerManager getPlayerManager() {
-        return playerManager;
+    public GuildAudioManager getGuidAudioManager() {
+        return guildAudioManager;
     }
 
-    public VoiceChannel getPlayedChanel() {
-        return playedChanel;
-    }
-
-    public void setPlayedChanel(VoiceChannel playedChanel) {
-        this.playedChanel = playedChanel;
-    }
 
     public void clearLastButton() {
-        if (lastMessageWithButton != null){
+        if (lastMessageWithButton != null) {
             this.lastMessageWithButton.editMessageComponents(new ArrayList<>()).queue();
             this.lastMessageWithButton = null;
         }
 
     }
-    public void updateLastButton(){
+
+    public void updateLastButton() {
         if (lastMessageWithButton != null)
             lastMessageWithButton = lastMessageWithButton.editMessageComponents(ActionRow.of(getActionButton())).complete();
     }
 
 
-    private List<Button> getActionButton(){
+    private List<Button> getActionButton() {
         ArrayList<Button> buttonArrayList = new ArrayList<>();
-        if(musicManager.player.getPlayingTrack() == null){
+        if (guildAudioManager.player.getPlayingTrack() == null) {
             buttonArrayList.add(Button.success("play", Emoji.fromUnicode("▶️")).withDisabled(true));
             buttonArrayList.add(Button.primary("next", Emoji.fromUnicode("⏭️")).withDisabled(true));
             buttonArrayList.add(Button.primary("stop", Emoji.fromUnicode("⏹️")).withDisabled(true));
             buttonArrayList.add(Button.danger("disconnect", Emoji.fromUnicode("⏏️")));
             return buttonArrayList;
         }
-        if(musicManager.player.isPaused()){
+        if (guildAudioManager.player.isPaused()) {
             buttonArrayList.add(Button.success("play", Emoji.fromUnicode("▶️")));
-        }
-        else{
+        } else {
             buttonArrayList.add(Button.success("pause", Emoji.fromUnicode("⏸️")));
         }
         buttonArrayList.add(Button.primary("next", Emoji.fromUnicode("⏭️")));

src/main/java/net/Broken/Audio/GuildAudioManager.java

@@ -1,4 +1,4 @@
-package net.Broken.audio;
+package net.Broken.Audio;
 
 import com.sedmelluq.discord.lavaplayer.player.AudioPlayer;
 import com.sedmelluq.discord.lavaplayer.player.AudioPlayerManager;
@@ -7,7 +7,7 @@ import net.dv8tion.jda.api.entities.Guild;
 /**
  * Holder for both the player and a track scheduler for one guild.
  */
-public class GuildMusicManager {
+public class GuildAudioManager {
     /**
      * Audio player for the guild.
      */
@@ -24,7 +24,7 @@ public class GuildMusicManager {
      *
      * @param manager Audio player manager to use for creating the player.
      */
-    public GuildMusicManager(AudioPlayerManager manager, Guild guild) {
+    public GuildAudioManager(AudioPlayerManager manager, Guild guild) {
         player = manager.createPlayer();
         scheduler = new TrackScheduler(player, guild);
         player.addListener(scheduler);

src/main/java/net/Broken/Audio/TrackScheduler.java

@@ -1,6 +1,5 @@
-package net.Broken.audio;
+package net.Broken.Audio;
 
-import com.google.api.client.googleapis.json.GoogleJsonResponseException;
 import com.sedmelluq.discord.lavaplayer.player.AudioPlayer;
 import com.sedmelluq.discord.lavaplayer.player.event.AudioEventAdapter;
 import com.sedmelluq.discord.lavaplayer.tools.FriendlyException;
@@ -8,15 +7,10 @@ import com.sedmelluq.discord.lavaplayer.track.AudioTrack;
 import com.sedmelluq.discord.lavaplayer.track.AudioTrackEndReason;
 import com.sedmelluq.discord.lavaplayer.track.AudioTrackInfo;
 import net.Broken.MainBot;
-import net.Broken.RestApi.Data.UserAudioTrackData;
-import net.Broken.audio.Youtube.RelatedIdNotFound;
-import net.Broken.audio.Youtube.YoutubeSearchRework;
-import net.Broken.audio.Youtube.YoutubeTools;
 import net.dv8tion.jda.api.entities.Guild;
 import org.apache.logging.log4j.LogManager;
 import org.apache.logging.log4j.Logger;
 
-import java.io.IOException;
 import java.util.ArrayList;
 import java.util.List;
 import java.util.concurrent.BlockingDeque;
@@ -31,9 +25,7 @@ public class TrackScheduler extends AudioEventAdapter {
     private final Guild guild;
 
     private UserAudioTrack currentPlayingTrack;
-    private boolean autoFlow = false;
-    private ArrayList<String> history = new ArrayList<>();
-    private Logger logger = LogManager.getLogger();
+    private final Logger logger = LogManager.getLogger();
 
     /**
      * @param player The audio player this scheduler uses
@@ -57,20 +49,13 @@ public class TrackScheduler extends AudioEventAdapter {
         // track goes to the queue instead.
         if (track.getSubmittedUser() != MainBot.jda.getSelfUser()) {
             logger.debug("[" + guild + "] Flush history");
-            history = new ArrayList<>();
         }
 
-        history.add(track.getAudioTrack().getIdentifier());
         if (!player.startTrack(track.getAudioTrack(), true)) {
             queue.offer(track);
         } else {
             currentPlayingTrack = track;
         }
-        if (track.getSubmittedUser() != MainBot.jda.getSelfUser()) {
-            needAutoPlay();
-        }
-
-
     }
 
     /**
@@ -82,22 +67,11 @@ public class TrackScheduler extends AudioEventAdapter {
         // Calling startTrack with the noInterrupt set to true will start the track only if nothing is currently playing. If
         // something is playing, it returns false and does nothing. In that case the player was already playing so this
         // track goes to the queue instead.
-        if (track.getSubmittedUser() != MainBot.jda.getSelfUser()) {
-            logger.debug("Flush history");
-            history = new ArrayList<>();
-        }
-
-        history.add(track.getAudioTrack().getIdentifier());
         if (!player.startTrack(track.getAudioTrack(), true)) {
             queue.addFirst(track);
         } else {
             currentPlayingTrack = track;
         }
-        if (track.getSubmittedUser() != MainBot.jda.getSelfUser()) {
-
-            needAutoPlay();
-        } else
-            logger.debug("[" + guild + "] Bot add, ignore autoFlow");
     }
 
     public void pause() {
@@ -119,14 +93,12 @@ public class TrackScheduler extends AudioEventAdapter {
         queue.clear();
     }
 
-    public List<UserAudioTrackData> getList() {
-//        AudioTrack[] test = (AudioTrack[]) queue.toArray();
-
-        List<UserAudioTrackData> temp = new ArrayList<>();
+    public List<UserAudioTrack> getList() {
+        List<UserAudioTrack> temp = new ArrayList<>();
         Object[] test = queue.toArray();
         for (Object track : test) {
             UserAudioTrack casted = (UserAudioTrack) track;
-            temp.add(new UserAudioTrackData(casted.getSubmittedUser().getName(), casted.getAudioTrack().getInfo()));
+            temp.add(casted);
         }
         return temp;
     }
@@ -147,7 +119,6 @@ public class TrackScheduler extends AudioEventAdapter {
                     return false;
                 } else {
                     logger.info("[" + guild + "] Delete successful");
-                    needAutoPlay();
                     return true;
                 }
             }
@@ -168,8 +139,6 @@ public class TrackScheduler extends AudioEventAdapter {
             this.currentPlayingTrack = track;
             player.startTrack(track.getAudioTrack(), false);
         }
-
-        needAutoPlay();
     }
 
     @Override
@@ -178,7 +147,7 @@ public class TrackScheduler extends AudioEventAdapter {
         if (endReason.mayStartNext) {
             if(queue.isEmpty()){
                 logger.debug("[" + guild.getName() + "] End of track, Playlist empty.");
-                AudioM.getInstance(guild).updateLastButton();
+                GuildAudioBotService.getInstance(guild).updateLastButton();
             }else{
                 logger.debug("[" + guild.getName() + "] End of track, start next.");
                 nextTrack();
@@ -191,65 +160,30 @@ public class TrackScheduler extends AudioEventAdapter {
     @Override
     public void onTrackStart(AudioPlayer player, AudioTrack track) {
         super.onTrackStart(player, track);
-        AudioM.getInstance(guild).updateLastButton();
+        GuildAudioBotService.getInstance(guild).updateLastButton();
     }
 
     @Override
     public void onPlayerPause(AudioPlayer player) {
         super.onPlayerPause(player);
-        AudioM.getInstance(guild).updateLastButton();
+        GuildAudioBotService.getInstance(guild).updateLastButton();
     }
 
     @Override
     public void onPlayerResume(AudioPlayer player) {
         super.onPlayerResume(player);
-        AudioM.getInstance(guild).updateLastButton();
+        GuildAudioBotService.getInstance(guild).updateLastButton();
     }
 
     @Override
     public void onTrackException(AudioPlayer player, AudioTrack track, FriendlyException exception) {
         super.onTrackException(player, track, exception);
-        AudioM.getInstance(guild).updateLastButton();
+        GuildAudioBotService.getInstance(guild).updateLastButton();
     }
 
     @Override
     public void onTrackStuck(AudioPlayer player, AudioTrack track, long thresholdMs) {
         super.onTrackStuck(player, track, thresholdMs);
-        AudioM.getInstance(guild).updateLastButton();
-    }
-
-    private void needAutoPlay() {
-        if ((queue.size() < 1) && autoFlow && currentPlayingTrack != null) {
-            logger.debug("[" + guild.getName() + "] Auto add needed!");
-            AudioM audioM = AudioM.getInstance(guild);
-            YoutubeSearchRework youtubeSearchRework = YoutubeSearchRework.getInstance();
-            try {
-                String id = youtubeSearchRework.getRelatedVideo(currentPlayingTrack.getAudioTrack().getInfo().identifier);
-                logger.debug("[" + guild.getName() + "] Related id: " + id);
-                audioM.loadAndPlayAuto(id);
-            } catch (IOException | RelatedIdNotFound ex) {
-                logger.debug("[" + guild.getName() + "] Can't find related id, try API...");
-                YoutubeTools youtubeTools = YoutubeTools.getInstance();
-                try {
-                    String id = youtubeTools.getRelatedVideo(currentPlayingTrack.getAudioTrack().getInfo().identifier, history);
-                    logger.debug("[" + guild.getName() + "] Related id: " + id);
-                    audioM.loadAndPlayAuto(id);
-
-                } catch (GoogleJsonResponseException e) {
-                    logger.error("[" + guild.getName() + "] There was a service error: " + e.getDetails().getCode() + " : " + e.getDetails().getMessage());
-                } catch (IOException t) {
-                    logger.catching(t);
-                }
-            }
-        }
-    }
-
-    public boolean isAutoFlow() {
-        return autoFlow;
-    }
-
-    public void setAutoFlow(boolean autoFlow) {
-        this.autoFlow = autoFlow;
-        needAutoPlay();
+        GuildAudioBotService.getInstance(guild).updateLastButton();
     }
 }

src/main/java/net/Broken/Audio/UserAudioTrack.java

@@ -1,4 +1,4 @@
-package net.Broken.audio;
+package net.Broken.Audio;
 
 import com.sedmelluq.discord.lavaplayer.track.AudioTrack;
 import net.dv8tion.jda.api.entities.User;
@@ -7,8 +7,8 @@ import net.dv8tion.jda.api.entities.User;
  * Container that link AudioTrack to who submit it (User)
  */
 public class UserAudioTrack {
-    private User user;
-    private AudioTrack audioTrack;
+    private final User user;
+    private final AudioTrack audioTrack;
 
     public UserAudioTrack(User user, AudioTrack audioTrack) {
         this.user = user;

src/main/java/net/Broken/BotConfigLoader.java

@@ -2,7 +2,7 @@ package net.Broken;
 
 import org.springframework.boot.context.properties.ConfigurationProperties;
 import org.springframework.boot.context.properties.ConstructorBinding;
-import org.springframework.context.annotation.Configuration;
+
 @ConfigurationProperties(prefix = "discord.bot")
 @ConstructorBinding
 public record BotConfigLoader (

src/main/java/net/Broken/BotListener.java

@@ -1,11 +1,11 @@
 package net.Broken;
 
+import net.Broken.Audio.GuildAudioBotService;
 import net.Broken.DB.Entity.GuildPreferenceEntity;
 import net.Broken.DB.Repository.GuildPreferenceRepository;
 import net.Broken.Tools.AutoVoiceChannel;
 import net.Broken.Tools.EmbedMessageUtils;
 import net.Broken.Tools.UserManager.Stats.UserStatsUtils;
-import net.Broken.audio.AudioM;
 import net.dv8tion.jda.api.EmbedBuilder;
 import net.dv8tion.jda.api.MessageBuilder;
 import net.dv8tion.jda.api.entities.*;
@@ -119,10 +119,10 @@ public class BotListener extends ListenerAdapter {
 
             if (event.getGuild().getAudioManager().getConnectedChannel().getMembers().size() == 1) {
                 logger.debug("I'm alone, close audio connection.");
-                AudioM.getInstance(event.getGuild()).stop();
+                GuildAudioBotService.getInstance(event.getGuild()).stop();
             }
         } else if (event.getMember().getUser() == MainBot.jda.getSelfUser()) {
-            AudioM.getInstance(event.getGuild()).clearLastButton();
+            GuildAudioBotService.getInstance(event.getGuild()).clearLastButton();
         }
         AutoVoiceChannel autoVoiceChannel = AutoVoiceChannel.getInstance(event.getGuild());
         autoVoiceChannel.leave(event.getChannelLeft());
@@ -147,13 +147,13 @@ public class BotListener extends ListenerAdapter {
     public void onButtonClick(@NotNull ButtonClickEvent event) {
         super.onButtonClick(event);
         event.deferReply().queue();
-        AudioM audioM = AudioM.getInstance(event.getGuild());
+        GuildAudioBotService guildAudioBotService = GuildAudioBotService.getInstance(event.getGuild());
         switch (event.getComponentId()) {
-            case "pause" -> audioM.pause(event);
-            case "play" -> audioM.resume(event);
-            case "next" -> audioM.skipTrack(event);
-            case "stop" -> audioM.stop(event);
-            case "disconnect" -> audioM.disconect(event);
+            case "pause" -> guildAudioBotService.pause(event);
+            case "play" -> guildAudioBotService.resume(event);
+            case "next" -> guildAudioBotService.skipTrack(event);
+            case "stop" -> guildAudioBotService.stop(event);
+            case "disconnect" -> guildAudioBotService.disconnect(event);
         }
     }
 
@@ -193,9 +193,7 @@ public class BotListener extends ListenerAdapter {
         logger.info("Join new guild! (" + event.getGuild().getName() + " " + event.getGuild().getMembers().size() + " Members)");
         super.onGuildJoin(event);
         getPreference(event.getGuild());
-        event.getGuild().loadMembers().onSuccess((members -> {
-            logger.debug("[" + event.getGuild().getName() + "] Members loaded");
-        }));
+        event.getGuild().loadMembers().onSuccess((members -> logger.debug("[" + event.getGuild().getName() + "] Members loaded")));
         EmbedBuilder eb = new EmbedBuilder().setColor(Color.GREEN)
                 .setTitle("Hello there !")
                 .setDescription("Allow me to introduce myself -- I am a CL4P-TP the discord bot, but my friends call me Claptrap ! Or they would, if any of them were real...\n" +

src/main/java/net/Broken/Commands/ChannelsReview.java

@@ -17,7 +17,7 @@ import java.util.HashMap;
 import java.util.Locale;
 
 public class ChannelsReview implements Commande {
-    Logger logger = LogManager.getLogger();
+    final Logger logger = LogManager.getLogger();
 
     @Override
     public void action(String[] args, MessageReceivedEvent event) {
@@ -45,7 +45,7 @@ public class ChannelsReview implements Commande {
                 try {
                     Message lastMessage = textChannel.retrieveMessageById(lastMessageId).complete();
                     if (beforeDate.compareTo(format.parse(lastMessage.getTimeCreated().format(formatter))) > 0) {
-                        logger.debug("Last message in channel " + textChannel.toString() + " is " + lastMessageId);
+                        logger.debug("Last message in channel " + textChannel + " is " + lastMessageId);
                         String date = lastMessage.getTimeCreated().format(formatter);
                         charCtl += textChannel.getName().length() + date.length();
                         result.put(textChannel.getName(), date);
@@ -77,7 +77,7 @@ public class ChannelsReview implements Commande {
         for (TextChannel textChannel : event.getGuild().getTextChannels()) {
             if (textChannel.hasLatestMessage()) {
                 String lastMessageId = textChannel.getLatestMessageId();
-                logger.debug("Last message in channel " + textChannel.toString() + " is " + lastMessageId);
+                logger.debug("Last message in channel " + textChannel + " is " + lastMessageId);
 
                 try {
                     Message lastMessage = textChannel.retrieveMessageById(lastMessageId).complete();

src/main/java/net/Broken/Commands/ReportUsers.java

@@ -6,6 +6,8 @@ import net.dv8tion.jda.api.entities.Member;
 import net.dv8tion.jda.api.events.message.MessageReceivedEvent;
 import net.dv8tion.jda.api.requests.RestAction;
 
+import java.util.Objects;
+
 
 public class ReportUsers implements Commande {
 
@@ -18,11 +20,8 @@ public class ReportUsers implements Commande {
                 for (Member member : members) {
                     if (member.getRoles().size() == 1) { //check if the member has a role
                         if (member.getRoles().contains(event.getMessage().getMentionedRoles().get(0))) { //check if the mentioned role is the same as the member's role
-                            if (restAction == null) {
-                                restAction = event.getTextChannel().sendMessage("List des membres : ").and(event.getTextChannel().sendMessage(member.getEffectiveName()));
-                            } else {
-                                restAction = restAction.and(event.getTextChannel().sendMessage(member.getEffectiveName()));
-                            }
+                            restAction = Objects.requireNonNullElseGet(restAction, () -> event.getTextChannel().sendMessage("List des membres : "))
+                                    .and(event.getTextChannel().sendMessage(member.getEffectiveName()));
                         }
                     }
                 }

src/main/java/net/Broken/DB/Entity/GuildPreferenceEntity.java

@@ -1,11 +1,6 @@
 package net.Broken.DB.Entity;
 
-import net.Broken.Api.Data.Settings.Value;
-import net.dv8tion.jda.api.entities.Guild;
-
 import javax.persistence.*;
-import java.util.ArrayList;
-import java.util.List;
 
 @Entity
 public class GuildPreferenceEntity {

src/main/java/net/Broken/DB/Entity/PlaylistEntity.java

@@ -1,79 +0,0 @@
-package net.Broken.DB.Entity;
-
-
-import com.fasterxml.jackson.annotation.JsonIgnore;
-
-import javax.persistence.*;
-import java.util.ArrayList;
-import java.util.Arrays;
-import java.util.List;
-
-@Entity
-public class PlaylistEntity {
-
-    @Id
-    @GeneratedValue(strategy = GenerationType.IDENTITY)
-    private Integer id;
-
-    private String name;
-
-    @JsonIgnore
-    @ManyToOne
-    @JoinColumn(name = "userEntity_id", nullable = false)
-    private UserEntity user;
-
-
-    @OneToMany(mappedBy = "playlist")
-    private List<TrackEntity> tracks;
-
-
-    public PlaylistEntity() {
-    }
-
-    public PlaylistEntity(String name, UserEntity user) {
-        this.name = name;
-        this.user = user;
-    }
-
-    public Integer getId() {
-        return id;
-    }
-
-    public void setId(Integer id) {
-        this.id = id;
-    }
-
-
-    public String getName() {
-        return name;
-    }
-
-    public void setName(String name) {
-        this.name = name;
-    }
-
-    public UserEntity getUser() {
-        return user;
-    }
-
-    public void setUser(UserEntity user) {
-        this.user = user;
-    }
-
-    public List<TrackEntity> getTracks() {
-        return tracks;
-    }
-
-    public void setTracks(List<TrackEntity> tracks) {
-        this.tracks = tracks;
-    }
-
-    public void addTracks(TrackEntity... tracks) {
-        if (this.tracks == null)
-            this.tracks = new ArrayList<>();
-
-        this.tracks.addAll(Arrays.asList(tracks));
-    }
-
-
-}

src/main/java/net/Broken/DB/Entity/TrackEntity.java

@@ -1,94 +0,0 @@
-package net.Broken.DB.Entity;
-
-import com.fasterxml.jackson.annotation.JsonIgnore;
-import com.sedmelluq.discord.lavaplayer.track.AudioTrackInfo;
-
-import javax.persistence.*;
-
-@Entity
-public class TrackEntity {
-
-    @Id
-    @GeneratedValue(strategy = GenerationType.IDENTITY)
-    private Integer id;
-
-    private String title;
-
-    private String url;
-
-    private String identifier;
-
-    private Integer pos;
-
-    @JsonIgnore
-    @ManyToOne
-    @JoinColumn(name = "playlistEntity_id", nullable = false)
-    private PlaylistEntity playlist;
-
-    public TrackEntity() {
-    }
-
-    public TrackEntity(AudioTrackInfo trackInfo, int pos, PlaylistEntity playlist) {
-        this.title = trackInfo.title;
-        this.url = trackInfo.uri;
-        this.identifier = trackInfo.identifier;
-        this.playlist = playlist;
-        this.pos = pos;
-    }
-
-    public TrackEntity(TrackEntity trackEntity) {
-        this.title = trackEntity.title;
-        this.url = trackEntity.url;
-        this.identifier = trackEntity.identifier;
-        this.pos = trackEntity.pos;
-        this.playlist = trackEntity.playlist;
-    }
-
-    public Integer getId() {
-        return id;
-    }
-
-    public void setId(Integer id) {
-        this.id = id;
-    }
-
-    public PlaylistEntity getPlaylist() {
-        return playlist;
-    }
-
-    public void setPlaylist(PlaylistEntity playlist) {
-        this.playlist = playlist;
-    }
-
-    public String getTitle() {
-        return title;
-    }
-
-    public void setTitle(String title) {
-        this.title = title;
-    }
-
-    public String getUrl() {
-        return url;
-    }
-
-    public void setUrl(String url) {
-        this.url = url;
-    }
-
-    public String getIdentifier() {
-        return identifier;
-    }
-
-    public void setIdentifier(String identifier) {
-        this.identifier = identifier;
-    }
-
-    public Integer getPos() {
-        return pos;
-    }
-
-    public void setPos(Integer pos) {
-        this.pos = pos;
-    }
-}

src/main/java/net/Broken/DB/Entity/UserEntity.java

@@ -5,8 +5,6 @@ import net.Broken.Api.Security.Data.DiscordOauthUserInfo;
 import net.dv8tion.jda.api.entities.User;
 
 import javax.persistence.*;
-import java.util.ArrayList;
-import java.util.Arrays;
 import java.util.List;
 
 /**
@@ -22,7 +20,7 @@ public class UserEntity {
 
     private String discriminator;
 
-    @Column(unique=true)
+    @Column(unique = true)
     private String discordId;
 
     private String avatar;
@@ -33,10 +31,6 @@ public class UserEntity {
     @OneToMany(fetch = FetchType.EAGER, mappedBy = "user")
     private List<UserStats> userStats;
 
-    @OneToMany(mappedBy = "user")
-    private List<PlaylistEntity> playlists;
-
-
     public UserEntity() {
     }
 
@@ -50,7 +44,7 @@ public class UserEntity {
         this.discordId = id;
     }
 
-    public UserEntity(DiscordOauthUserInfo discordOauthUserInfo){
+    public UserEntity(DiscordOauthUserInfo discordOauthUserInfo) {
         this.username = discordOauthUserInfo.username();
         this.discriminator = discordOauthUserInfo.discriminator();
         this.discordId = discordOauthUserInfo.id();
@@ -82,21 +76,6 @@ public class UserEntity {
         this.discordId = discordId;
     }
 
-    public List<PlaylistEntity> getPlaylists() {
-        return playlists;
-    }
-
-    public void setPlaylists(List<PlaylistEntity> playlists) {
-        this.playlists = playlists;
-    }
-
-    public void addPlaylist(PlaylistEntity... playlists) {
-        if (this.playlists == null)
-            this.playlists = new ArrayList<>();
-
-        this.playlists.addAll(Arrays.asList(playlists));
-    }
-
     public List<UserStats> getUserStats() {
         return userStats;
     }

src/main/java/net/Broken/DB/Repository/PlaylistRepository.java

@@ -1,8 +0,0 @@
-package net.Broken.DB.Repository;
-
-import net.Broken.DB.Entity.PlaylistEntity;
-import org.springframework.data.repository.CrudRepository;
-
-public interface PlaylistRepository extends CrudRepository<PlaylistEntity, Integer> {
-    PlaylistEntity findById(int id);
-}

src/main/java/net/Broken/DB/Repository/TrackRepository.java

@@ -1,13 +0,0 @@
-package net.Broken.DB.Repository;
-
-import net.Broken.DB.Entity.PlaylistEntity;
-import net.Broken.DB.Entity.TrackEntity;
-import org.springframework.data.repository.CrudRepository;
-
-import java.util.List;
-
-public interface TrackRepository extends CrudRepository<TrackEntity, Integer> {
-    List<TrackEntity> findDistinctByPlaylistOrderByPos(PlaylistEntity playlistEntity);
-
-    TrackEntity findById(int id);
-}

src/main/java/net/Broken/Init.java

@@ -4,7 +4,6 @@ import net.Broken.DB.Entity.GuildPreferenceEntity;
 import net.Broken.DB.Entity.UserEntity;
 import net.Broken.DB.Repository.GuildPreferenceRepository;
 import net.Broken.DB.Repository.UserRepository;
-import net.Broken.RestApi.ApiCommandLoader;
 import net.Broken.Tools.Command.SlashCommandLoader;
 import net.Broken.Tools.DayListener.DayListener;
 import net.Broken.Tools.DayListener.Listeners.DailyMadame;
@@ -19,7 +18,6 @@ import net.dv8tion.jda.api.utils.MemberCachePolicy;
 import org.apache.logging.log4j.LogManager;
 import org.apache.logging.log4j.Logger;
 import org.springframework.context.ApplicationContext;
-import org.thymeleaf.spring5.processor.SpringOptionFieldTagProcessor;
 
 import javax.security.auth.login.LoginException;
 import java.util.List;
@@ -68,7 +66,6 @@ public class Init {
         logger.info("Loading commands");
         SlashCommandLoader.load(config);
         SlashCommandLoader.registerSlashCommands(jda.updateCommands());
-        ApiCommandLoader.load();
         DayListener dayListener = DayListener.getInstance();
         dayListener.addListener(new DailyMadame());
         dayListener.start();

src/main/java/net/Broken/MainBot.java

@@ -3,7 +3,6 @@ package net.Broken;
 import net.dv8tion.jda.api.JDA;
 import org.apache.logging.log4j.LogManager;
 import org.apache.logging.log4j.Logger;
-import org.springframework.boot.ExitCodeGenerator;
 import org.springframework.boot.SpringApplication;
 import org.springframework.boot.autoconfigure.SpringBootApplication;
 import org.springframework.boot.context.properties.ConfigurationPropertiesScan;
@@ -18,10 +17,8 @@ import java.util.HashMap;
 @ConfigurationPropertiesScan
 public class MainBot {
 
-    public static HashMap<String, Commande> commandes = new HashMap<>();
-    public static HashMap<String, SlashCommand> slashCommands = new HashMap<>();
+    public static final HashMap<String, SlashCommand> slashCommands = new HashMap<>();
     public static HashMap<String, Integer> mutualGuildCount = new HashMap<>();
-    public static boolean roleFlag = false;
     public static JDA jda;
     public static boolean ready = false;
 
@@ -42,12 +39,11 @@ public class MainBot {
 
         jda = Init.initJda(config);
         if (jda == null) {
-            System.exit(SpringApplication.exit(ctx, (ExitCodeGenerator) () -> {
+            System.exit(SpringApplication.exit(ctx, () -> {
                 logger.fatal("Init error! Close application!");
                 return 1;
             }));
         }
-
         Init.polish(jda, config);
         ready = true;
     }

src/main/java/net/Broken/RestApi/ApiCommandLoader.java

@@ -1,44 +0,0 @@
-package net.Broken.RestApi;
-
-import org.apache.logging.log4j.LogManager;
-import org.apache.logging.log4j.Logger;
-import org.reflections.Reflections;
-import org.reflections.util.ClasspathHelper;
-import org.reflections.util.ConfigurationBuilder;
-
-import java.util.HashMap;
-import java.util.Set;
-
-public class ApiCommandLoader {
-    public static HashMap<String, CommandInterface> apiCommands = new HashMap<>();
-    private static Logger logger = LogManager.getLogger();
-
-    public static void load() {
-        logger.info("Loading Api Command...");
-//        Reflections reflections = new Reflections("net.Broken.RestApi.Command");
-        Reflections reflections = new Reflections(new ConfigurationBuilder().setUrls(ClasspathHelper.forPackage(
-                "net.Broken.RestApi.Commands",
-                ClasspathHelper.contextClassLoader(),
-                ClasspathHelper.staticClassLoader()))
-        );
-        Set<Class<? extends CommandInterface>> modules =
-                reflections.getSubTypesOf(CommandInterface.class);
-        logger.info("Find " + modules.size() + " Command:");
-        for (Class<? extends CommandInterface> apiClass : modules) {
-
-            String reference = apiClass.getName();
-            String[] splited = reference.split("\\.");
-            String name = splited[splited.length - 1].toUpperCase();
-
-            logger.info("..." + name);
-
-            try {
-                apiCommands.put(name, apiClass.newInstance());
-            } catch (InstantiationException | IllegalAccessException e) {
-                logger.error("Failed to load " + name + "!");
-            }
-
-        }
-    }
-
-}

src/main/java/net/Broken/RestApi/CommandInterface.java

@@ -1,24 +0,0 @@
-package net.Broken.RestApi;
-
-import net.Broken.RestApi.Data.CommandPostData;
-import net.Broken.RestApi.Data.CommandResponseData;
-import net.dv8tion.jda.api.entities.Guild;
-import net.dv8tion.jda.api.entities.User;
-import org.springframework.http.ResponseEntity;
-
-
-/**
- * Represent RestApi command
- */
-public interface CommandInterface {
-    /**
-     * Main action
-     *
-     * @param musicCommande Current guild music command
-     * @param data          Received data
-     * @param user          User who submit RestApi command
-     * @param guild
-     * @return HTTP Response
-     */
-    ResponseEntity<CommandResponseData> action(CommandPostData data, User user, Guild guild);
-}

src/main/java/net/Broken/RestApi/Commands/Add.java

@@ -1,19 +0,0 @@
-package net.Broken.RestApi.Commands;
-
-import net.Broken.RestApi.CommandInterface;
-import net.Broken.RestApi.Data.CommandPostData;
-import net.Broken.RestApi.Data.CommandResponseData;
-import net.Broken.audio.WebLoadUtils;
-import net.dv8tion.jda.api.entities.Guild;
-import net.dv8tion.jda.api.entities.User;
-import org.springframework.http.ResponseEntity;
-
-/**
- * Add track RestApi
- */
-public class Add implements CommandInterface {
-    @Override
-    public ResponseEntity<CommandResponseData> action(CommandPostData data, User user, Guild guild) {
-        return new WebLoadUtils(data, user, guild, true).getResponse();
-    }
-}

src/main/java/net/Broken/RestApi/Commands/AutoFlowOff.java

@@ -1,22 +0,0 @@
-package net.Broken.RestApi.Commands;
-
-import net.Broken.RestApi.CommandInterface;
-import net.Broken.RestApi.Data.CommandPostData;
-import net.Broken.RestApi.Data.CommandResponseData;
-import net.Broken.audio.AudioM;
-import net.Broken.audio.TrackScheduler;
-import net.dv8tion.jda.api.entities.Guild;
-import net.dv8tion.jda.api.entities.User;
-import org.springframework.http.HttpStatus;
-import org.springframework.http.ResponseEntity;
-
-public class AutoFlowOff implements CommandInterface {
-
-    @Override
-    public ResponseEntity<CommandResponseData> action(CommandPostData data, User user, Guild guild) {
-        AudioM audioM = AudioM.getInstance(guild);
-        TrackScheduler scheduler = audioM.getGuildMusicManager().scheduler;
-        scheduler.setAutoFlow(false);
-        return new ResponseEntity<>(new CommandResponseData(data.command, "ok"), HttpStatus.OK);
-    }
-}

src/main/java/net/Broken/RestApi/Commands/AutoFlowOn.java

@@ -1,23 +0,0 @@
-package net.Broken.RestApi.Commands;
-
-import net.Broken.RestApi.CommandInterface;
-import net.Broken.RestApi.Data.CommandPostData;
-import net.Broken.RestApi.Data.CommandResponseData;
-import net.Broken.audio.AudioM;
-import net.Broken.audio.TrackScheduler;
-import net.dv8tion.jda.api.entities.Guild;
-import net.dv8tion.jda.api.entities.User;
-import org.springframework.http.HttpStatus;
-import org.springframework.http.ResponseEntity;
-
-public class AutoFlowOn implements CommandInterface {
-
-    @Override
-    public ResponseEntity<CommandResponseData> action(CommandPostData data, User user, Guild guild) {
-        AudioM audioM = AudioM.getInstance(guild);
-        TrackScheduler scheduler = audioM.getGuildMusicManager().scheduler;
-        scheduler.setAutoFlow(true);
-        return new ResponseEntity<>(new CommandResponseData(data.command, "ok"), HttpStatus.OK);
-
-    }
-}

src/main/java/net/Broken/RestApi/Commands/Connect.java

@@ -1,37 +0,0 @@
-package net.Broken.RestApi.Commands;
-
-import net.Broken.RestApi.CommandInterface;
-import net.Broken.RestApi.Data.CommandPostData;
-import net.Broken.RestApi.Data.CommandResponseData;
-import net.Broken.audio.AudioM;
-import net.dv8tion.jda.api.entities.Guild;
-import net.dv8tion.jda.api.entities.User;
-import net.dv8tion.jda.api.entities.VoiceChannel;
-import org.springframework.http.HttpStatus;
-import org.springframework.http.ResponseEntity;
-
-/**
- * Connect to vocal channel RestApi command
- */
-public class Connect implements CommandInterface {
-    @Override
-    public ResponseEntity<CommandResponseData> action(CommandPostData data, User user, Guild guild) {
-        AudioM audioM = AudioM.getInstance(guild);
-        if (data.chanelId == null)
-            return new ResponseEntity<>(new CommandResponseData(data.command, "Missing chanelId"), HttpStatus.BAD_REQUEST);
-        VoiceChannel voiceChannel = null;
-        try {
-            voiceChannel = guild.getVoiceChannelById(data.chanelId);
-        } catch (NumberFormatException ignored) {
-        }
-
-        if (voiceChannel == null) {
-            return new ResponseEntity<>(new CommandResponseData(data.command, "Channel Not found"), HttpStatus.BAD_REQUEST);
-        }
-
-        audioM.getGuildAudioPlayer();
-        guild.getAudioManager().openAudioConnection(guild.getVoiceChannelById(data.chanelId));
-        audioM.setPlayedChanel(voiceChannel);
-        return new ResponseEntity<>(new CommandResponseData(data.command, "Accepted"), HttpStatus.OK);
-    }
-}

src/main/java/net/Broken/RestApi/Commands/Dell.java

@@ -1,27 +0,0 @@
-package net.Broken.RestApi.Commands;
-
-import net.Broken.RestApi.CommandInterface;
-import net.Broken.RestApi.Data.CommandPostData;
-import net.Broken.RestApi.Data.CommandResponseData;
-import net.Broken.audio.AudioM;
-import net.dv8tion.jda.api.entities.Guild;
-import net.dv8tion.jda.api.entities.User;
-import org.springframework.http.HttpStatus;
-import org.springframework.http.ResponseEntity;
-
-/**
- * Delete track RestApi command
- */
-public class Dell implements CommandInterface {
-    @Override
-    public ResponseEntity<CommandResponseData> action(CommandPostData data, User user, Guild guild) {
-        if (data.url != null) {
-            if (AudioM.getInstance(guild).getGuildMusicManager().scheduler.remove(data.url)) {
-                return new ResponseEntity<>(new CommandResponseData(data.command, "Accepted"), HttpStatus.OK);
-            } else
-                return new ResponseEntity<>(new CommandResponseData(data.command, "URL not found"), HttpStatus.NOT_FOUND);
-        }
-        return new ResponseEntity<>(new CommandResponseData(data.command, "Missing URL"), HttpStatus.NOT_ACCEPTABLE);
-
-    }
-}

src/main/java/net/Broken/RestApi/Commands/Disconnect.java

@@ -1,22 +0,0 @@
-package net.Broken.RestApi.Commands;
-
-import net.Broken.RestApi.CommandInterface;
-import net.Broken.RestApi.Data.CommandPostData;
-import net.Broken.RestApi.Data.CommandResponseData;
-import net.Broken.audio.AudioM;
-import net.dv8tion.jda.api.entities.Guild;
-import net.dv8tion.jda.api.entities.User;
-import org.springframework.http.HttpStatus;
-import org.springframework.http.ResponseEntity;
-
-/**
- * Disconnect from vocal chanel RestApi Command
- */
-public class Disconnect implements CommandInterface {
-    @Override
-    public ResponseEntity<CommandResponseData> action(CommandPostData data, User user, Guild guild) {
-        AudioM.getInstance(guild).stop();
-        return new ResponseEntity<>(new CommandResponseData(data.command, "Ok"), HttpStatus.OK);
-    }
-
-}

src/main/java/net/Broken/RestApi/Commands/Flush.java

@@ -1,21 +0,0 @@
-package net.Broken.RestApi.Commands;
-
-import net.Broken.RestApi.CommandInterface;
-import net.Broken.RestApi.Data.CommandPostData;
-import net.Broken.RestApi.Data.CommandResponseData;
-import net.Broken.audio.AudioM;
-import net.dv8tion.jda.api.entities.Guild;
-import net.dv8tion.jda.api.entities.User;
-import org.springframework.http.HttpStatus;
-import org.springframework.http.ResponseEntity;
-
-/**
- * Flush playlist RestApi Command
- */
-public class Flush implements CommandInterface {
-    @Override
-    public ResponseEntity<CommandResponseData> action(CommandPostData data, User user, Guild guild) {
-        AudioM.getInstance(guild).getGuildMusicManager().scheduler.flush();
-        return new ResponseEntity<>(new CommandResponseData(data.command, "Accepted"), HttpStatus.OK);
-    }
-}

src/main/java/net/Broken/RestApi/Commands/Next.java

@@ -1,21 +0,0 @@
-package net.Broken.RestApi.Commands;
-
-import net.Broken.RestApi.CommandInterface;
-import net.Broken.RestApi.Data.CommandPostData;
-import net.Broken.RestApi.Data.CommandResponseData;
-import net.Broken.audio.AudioM;
-import net.dv8tion.jda.api.entities.Guild;
-import net.dv8tion.jda.api.entities.User;
-import org.springframework.http.HttpStatus;
-import org.springframework.http.ResponseEntity;
-
-/**
- * Next Track RestApi command
- */
-public class Next implements CommandInterface {
-    @Override
-    public ResponseEntity<CommandResponseData> action(CommandPostData data, User user, Guild guild) {
-        AudioM.getInstance(guild).getGuildMusicManager().scheduler.nextTrack();
-        return new ResponseEntity<>(new CommandResponseData(data.command, "Accepted"), HttpStatus.OK);
-    }
-}

src/main/java/net/Broken/RestApi/Commands/Pause.java

@@ -1,21 +0,0 @@
-package net.Broken.RestApi.Commands;
-
-import net.Broken.RestApi.CommandInterface;
-import net.Broken.RestApi.Data.CommandPostData;
-import net.Broken.RestApi.Data.CommandResponseData;
-import net.Broken.audio.AudioM;
-import net.dv8tion.jda.api.entities.Guild;
-import net.dv8tion.jda.api.entities.User;
-import org.springframework.http.HttpStatus;
-import org.springframework.http.ResponseEntity;
-
-/**
- * Pause track RestApi command
- */
-public class Pause implements CommandInterface {
-    @Override
-    public ResponseEntity<CommandResponseData> action(CommandPostData data, User user, Guild guild) {
-        AudioM.getInstance(guild).getGuildMusicManager().scheduler.pause();
-        return new ResponseEntity<>(new CommandResponseData(data.command, "Accepted"), HttpStatus.OK);
-    }
-}

src/main/java/net/Broken/RestApi/Commands/Play.java

@@ -1,21 +0,0 @@
-package net.Broken.RestApi.Commands;
-
-import net.Broken.RestApi.CommandInterface;
-import net.Broken.RestApi.Data.CommandPostData;
-import net.Broken.RestApi.Data.CommandResponseData;
-import net.Broken.audio.AudioM;
-import net.dv8tion.jda.api.entities.Guild;
-import net.dv8tion.jda.api.entities.User;
-import org.springframework.http.HttpStatus;
-import org.springframework.http.ResponseEntity;
-
-/**
- * Resume (play button) RestApi command
- */
-public class Play implements CommandInterface {
-    @Override
-    public ResponseEntity<CommandResponseData> action(CommandPostData data, User user, Guild guild) {
-        AudioM.getInstance(guild).getGuildMusicManager().scheduler.resume();
-        return new ResponseEntity<>(new CommandResponseData(data.command, "Accepted"), HttpStatus.OK);
-    }
-}

src/main/java/net/Broken/RestApi/Commands/Stop.java

@@ -1,22 +0,0 @@
-package net.Broken.RestApi.Commands;
-
-import net.Broken.RestApi.CommandInterface;
-import net.Broken.RestApi.Data.CommandPostData;
-import net.Broken.RestApi.Data.CommandResponseData;
-import net.Broken.audio.AudioM;
-import net.dv8tion.jda.api.entities.Guild;
-import net.dv8tion.jda.api.entities.User;
-import org.springframework.http.HttpStatus;
-import org.springframework.http.ResponseEntity;
-
-/**
- * Stop RestApi Command
- */
-public class Stop implements CommandInterface {
-    @Override
-    public ResponseEntity<CommandResponseData> action(CommandPostData data, User user, Guild guild) {
-        AudioM.getInstance(guild).stop(null);
-        return new ResponseEntity<>(new CommandResponseData(data.command, "Accepted"), HttpStatus.OK);
-
-    }
-}

src/main/java/net/Broken/RestApi/Data/AllMusicInfoData.java

@@ -1,11 +0,0 @@
-package net.Broken.RestApi.Data;
-
-public class AllMusicInfoData {
-    public CurrentMusicData currentMusic;
-    public PlaylistData playlist;
-
-    public AllMusicInfoData(CurrentMusicData currentMusic, PlaylistData playlist) {
-        this.currentMusic = currentMusic;
-        this.playlist = playlist;
-    }
-}

src/main/java/net/Broken/RestApi/Data/ChanelData.java

@@ -1,17 +0,0 @@
-package net.Broken.RestApi.Data;
-
-/**
- * Data for JSON Parsing
- */
-public class ChanelData {
-    public String name;
-    public String id;
-    public int pos;
-
-    public ChanelData(String name, String id, int pos) {
-        this.name = name;
-        this.id = id;
-        this.pos = pos;
-
-    }
-}

src/main/java/net/Broken/RestApi/Data/CommandPostData.java

@@ -1,13 +0,0 @@
-package net.Broken.RestApi.Data;
-
-/**
- * Data for JSON Parsing
- */
-public class CommandPostData {
-    public String command;
-    public boolean onHead;
-    public String url;
-    public int playlistLimit;
-    public String chanelId;
-    public String name;
-}

src/main/java/net/Broken/RestApi/Data/CommandResponseData.java

@@ -1,24 +0,0 @@
-package net.Broken.RestApi.Data;
-
-import com.fasterxml.jackson.annotation.JsonInclude;
-
-/**
- * Data for JSON Parsing
- */
-@JsonInclude(JsonInclude.Include.NON_NULL)
-public class CommandResponseData {
-    public String Commande;
-    public String Message;
-    public String error;
-
-    public CommandResponseData(String commande, String message) {
-        Commande = commande;
-        Message = message;
-    }
-
-    public CommandResponseData(String commande, String message, String error) {
-        Commande = commande;
-        Message = message;
-        this.error = error;
-    }
-}

src/main/java/net/Broken/RestApi/Data/CurrentMusicData.java

@@ -1,43 +0,0 @@
-package net.Broken.RestApi.Data;
-
-import com.fasterxml.jackson.annotation.JsonInclude;
-
-/**
- * Data for JSON Parsing
- */
-@JsonInclude(JsonInclude.Include.NON_NULL)
-public class CurrentMusicData {
-    private final UserAudioTrackData info;
-    private final long currentPos;
-    private final String state;
-    private final boolean pause;
-    private final boolean autoflow;
-
-
-    public CurrentMusicData(UserAudioTrackData info, long currentPos, String state, boolean pause, boolean autoflow) {
-        this.info = info;
-        this.currentPos = currentPos;
-        this.state = state;
-        this.pause = pause;
-        this.autoflow = autoflow;
-    }
-
-    public UserAudioTrackData getInfo() {
-        return info;
-    }
-
-    public long getCurrentPos() {
-        return currentPos;
-    }
-
-    public String getState() {
-        if (pause)
-            return "PAUSE";
-        else
-            return state;
-    }
-
-    public boolean isAutoflow() {
-        return autoflow;
-    }
-}

src/main/java/net/Broken/RestApi/Data/Playlist/AddToPlaylistData.java

@@ -1,11 +0,0 @@
-package net.Broken.RestApi.Data.Playlist;
-
-import net.Broken.RestApi.Data.CommandPostData;
-
-public class AddToPlaylistData extends CommandPostData {
-
-    public int playlistId;
-
-
-    public int pos;
-}

src/main/java/net/Broken/RestApi/Data/Playlist/CreatePlaylistData.java

@@ -1,5 +0,0 @@
-package net.Broken.RestApi.Data.Playlist;
-
-public class CreatePlaylistData {
-    public String name;
-}

src/main/java/net/Broken/RestApi/Data/Playlist/DeleteTrackData.java

@@ -1,6 +0,0 @@
-package net.Broken.RestApi.Data.Playlist;
-
-public class DeleteTrackData {
-    public int id;
-    public int playlistId;
-}

src/main/java/net/Broken/RestApi/Data/Playlist/MoveTrackData.java

@@ -1,10 +0,0 @@
-package net.Broken.RestApi.Data.Playlist;
-
-public class MoveTrackData {
-
-    public int playlistId;
-
-    public int id;
-
-    public int newPos;
-}

src/main/java/net/Broken/RestApi/Data/Playlist/PlaylistResponseData.java

@@ -1,21 +0,0 @@
-package net.Broken.RestApi.Data.Playlist;
-
-import com.fasterxml.jackson.annotation.JsonInclude;
-import net.Broken.DB.Entity.PlaylistEntity;
-
-@JsonInclude(JsonInclude.Include.NON_NULL)
-public class PlaylistResponseData {
-    public String message;
-    public String error;
-    public PlaylistEntity playlist;
-
-    public PlaylistResponseData(String message, PlaylistEntity playlist) {
-        this.message = message;
-        this.playlist = playlist;
-    }
-
-    public PlaylistResponseData(String message, String error) {
-        this.message = message;
-        this.error = error;
-    }
-}

src/main/java/net/Broken/RestApi/Data/PlaylistData.java

@@ -1,19 +0,0 @@
-package net.Broken.RestApi.Data;
-
-import java.util.List;
-
-/**
- * Data for JSON Parsing
- */
-public class PlaylistData {
-
-    private List<UserAudioTrackData> list;
-
-    public PlaylistData(List<UserAudioTrackData> list) {
-        this.list = list;
-    }
-
-    public List<UserAudioTrackData> getList() {
-        return list;
-    }
-}

src/main/java/net/Broken/RestApi/Data/Settings/GetSettingsData.java

@@ -1,32 +0,0 @@
-package net.Broken.RestApi.Data.Settings;
-
-import com.fasterxml.jackson.annotation.JsonInclude;
-
-import java.util.List;
-
-@JsonInclude(JsonInclude.Include.NON_NULL)
-public class GetSettingsData {
-    public String description;
-    public String name;
-    public String id;
-    public TYPE type;
-    public List<Value> values;
-    public String current;
-
-    public GetSettingsData() {
-    }
-
-    public GetSettingsData(String name, String description, String id, TYPE type, List<Value> values, String current) {
-        this.name = name;
-        this.description = description;
-        this.id = id;
-        this.type = type;
-        this.values = values;
-        this.current = current;
-    }
-
-    public enum TYPE {
-        BOOL, LIST, STRING, SELECT_LIST
-    }
-}
-

src/main/java/net/Broken/RestApi/Data/Settings/ListPostSetting.java

@@ -1,7 +0,0 @@
-package net.Broken.RestApi.Data.Settings;
-
-import java.util.List;
-
-public class ListPostSetting {
-    public List<PostSetSettings> settings;
-}

src/main/java/net/Broken/RestApi/Data/Settings/PostSetSettings.java

@@ -1,9 +0,0 @@
-package net.Broken.RestApi.Data.Settings;
-
-import java.util.List;
-
-public class PostSetSettings {
-    public String id;
-    public String val;
-    public List<String> vals;
-}

src/main/java/net/Broken/RestApi/Data/Settings/Value.java

@@ -1,21 +0,0 @@
-package net.Broken.RestApi.Data.Settings;
-
-public class Value {
-    public String name;
-    public String id;
-    public boolean selected;
-
-    public Value() {
-    }
-
-    public Value(String name, String id) {
-        this.name = name;
-        this.id = id;
-    }
-
-    public Value(String name, String id, boolean selected) {
-        this.name = name;
-        this.id = id;
-        this.selected = selected;
-    }
-}

src/main/java/net/Broken/RestApi/Data/UserAudioTrackData.java

@@ -1,38 +0,0 @@
-package net.Broken.RestApi.Data;
-
-import com.sedmelluq.discord.lavaplayer.track.AudioTrackInfo;
-import net.Broken.audio.UserAudioTrack;
-
-/**
- * Data for JSON Parsing
- */
-public class UserAudioTrackData {
-    private String user;
-    private AudioTrackInfo audioTrackInfo;
-
-    public UserAudioTrackData(String user, AudioTrackInfo audioTrackInfo) {
-        this.user = user;
-        this.audioTrackInfo = audioTrackInfo;
-    }
-
-    public UserAudioTrackData(UserAudioTrack userAudioTrack) {
-        this.audioTrackInfo = userAudioTrack.getAudioTrack().getInfo();
-        this.user = userAudioTrack.getSubmittedUser().getName();
-    }
-
-    public String getUser() {
-        return user;
-    }
-
-    public void setUser(String user) {
-        this.user = user;
-    }
-
-    public AudioTrackInfo getAudioTrackInfo() {
-        return audioTrackInfo;
-    }
-
-    public void setAudioTrackInfo(AudioTrackInfo audioTrackInfo) {
-        this.audioTrackInfo = audioTrackInfo;
-    }
-}

src/main/java/net/Broken/RestApi/Data/UserManager/CheckResposeData.java

@@ -1,18 +0,0 @@
-package net.Broken.RestApi.Data.UserManager;
-
-/**
- * Data for JSON Parsing
- */
-public class CheckResposeData {
-    public boolean accepted;
-    public String name;
-    public String message;
-    public String id;
-
-    public CheckResposeData(boolean accepted, String name, String message, String id) {
-        this.accepted = accepted;
-        this.name = name;
-        this.message = message;
-        this.id = id;
-    }
-}

src/main/java/net/Broken/RestApi/Data/UserManager/ConfirmData.java

@@ -1,10 +0,0 @@
-package net.Broken.RestApi.Data.UserManager;
-
-/**
- * Data for JSON Parsing
- */
-public class ConfirmData {
-    public String id;
-    public String checkToken;
-
-}

src/main/java/net/Broken/RestApi/Data/UserManager/GuildInfo.java

@@ -1,16 +0,0 @@
-package net.Broken.RestApi.Data.UserManager;
-
-public class GuildInfo {
-    public String name;
-    public String id;
-    public boolean isAdmin;
-    public String imageUrl;
-
-    public GuildInfo(String name, String id, boolean isAdmin, String imageUrl) {
-        this.name = name;
-        this.id = id;
-        this.isAdmin = isAdmin;
-        this.imageUrl = imageUrl;
-
-    }
-}

src/main/java/net/Broken/RestApi/Data/UserManager/UserConnectionData.java

@@ -1,28 +0,0 @@
-package net.Broken.RestApi.Data.UserManager;
-
-/**
- * Data for JSON Parsing
- */
-public class UserConnectionData {
-    public boolean accepted;
-    public String token;
-    public String message;
-    public String error;
-    public String name;
-
-    public UserConnectionData(boolean accepted, String name, String token, String message) {
-        this.accepted = accepted;
-        this.token = token;
-        this.message = message;
-        this.name = name;
-        this.error = null;
-    }
-
-    public UserConnectionData(boolean accepted, String message, String error) {
-        this.accepted = accepted;
-        this.token = null;
-        this.message = message;
-        this.error = error;
-        this.name = null;
-    }
-}

src/main/java/net/Broken/RestApi/Data/UserManager/UserInfoData.java

@@ -1,9 +0,0 @@
-package net.Broken.RestApi.Data.UserManager;
-
-/**
- * Data for JSON Parsing
- */
-public class UserInfoData {
-    public String name;
-    public String password;
-}

src/main/java/net/Broken/RestApi/GeneralApiController.java

@@ -1,42 +0,0 @@
-package net.Broken.RestApi;
-
-import net.Broken.DB.Repository.PlaylistRepository;
-import net.Broken.DB.Repository.TrackRepository;
-import net.Broken.DB.Repository.UserRepository;
-import net.Broken.MainBot;
-import org.springframework.beans.factory.annotation.Autowired;
-import org.springframework.http.HttpStatus;
-import org.springframework.http.ResponseEntity;
-import org.springframework.web.bind.annotation.RequestMapping;
-import org.springframework.web.bind.annotation.RequestMethod;
-import org.springframework.web.bind.annotation.RestController;
-
-@RestController
-@RequestMapping("/api/")
-public class GeneralApiController {
-
-    private final
-    UserRepository userRepository;
-
-    private final
-    PlaylistRepository playlistRepository;
-
-    private final
-    TrackRepository trackRepository;
-
-    @Autowired
-    public GeneralApiController(UserRepository userRepository, PlaylistRepository playlistRepository, TrackRepository trackRepository) {
-        this.userRepository = userRepository;
-        this.playlistRepository = playlistRepository;
-        this.trackRepository = trackRepository;
-    }
-
-    @RequestMapping(value = "/isReady", method = RequestMethod.GET)
-    public ResponseEntity<String> isReady() {
-        if (MainBot.ready) {
-            return new ResponseEntity<>(HttpStatus.OK);
-        } else {
-            return new ResponseEntity<>(HttpStatus.NOT_FOUND);
-        }
-    }
-}

src/main/java/net/Broken/RestApi/MusicWebAPIController.java

@@ -1,239 +0,0 @@
-package net.Broken.RestApi;
-
-import com.google.api.client.googleapis.json.GoogleJsonResponseException;
-import com.sedmelluq.discord.lavaplayer.player.AudioPlayer;
-import com.sedmelluq.discord.lavaplayer.track.AudioTrack;
-import net.Broken.DB.Entity.UserEntity;
-import net.Broken.DB.Repository.UserRepository;
-import net.Broken.MainBot;
-import net.Broken.RestApi.Data.*;
-import net.Broken.Tools.CacheTools;
-import net.Broken.Tools.UserManager.Exceptions.UnknownTokenException;
-import net.Broken.Tools.UserManager.Stats.UserStatsUtils;
-import net.Broken.Tools.UserManager.UserUtils;
-import net.Broken.audio.AudioM;
-import net.Broken.audio.GetVoiceChanels;
-import net.Broken.audio.Youtube.SearchResult;
-import net.Broken.audio.Youtube.YoutubeSearchRework;
-import net.dv8tion.jda.api.entities.Guild;
-import net.dv8tion.jda.api.entities.Member;
-import net.dv8tion.jda.api.entities.VoiceChannel;
-import org.apache.logging.log4j.LogManager;
-import org.apache.logging.log4j.Logger;
-import org.springframework.beans.factory.annotation.Autowired;
-import org.springframework.http.HttpStatus;
-import org.springframework.http.ResponseEntity;
-import org.springframework.web.bind.annotation.*;
-
-import javax.servlet.http.HttpServletRequest;
-import java.io.IOException;
-import java.util.ArrayList;
-import java.util.List;
-
-/**
- * Rest Api Controller for /api/music
- */
-
-@RestController
-@RequestMapping("/api/music/")
-public class MusicWebAPIController {
-    private final
-    UserRepository userRepository;
-    Logger logger = LogManager.getLogger();
-    UserUtils userUtils = UserUtils.getInstance();
-
-    @Autowired
-    public MusicWebAPIController(UserRepository userRepository) {
-        this.userRepository = userRepository;
-    }
-
-
-    @RequestMapping("/currentMusicInfo")
-    public ResponseEntity<CurrentMusicData> getCurrentM(@RequestParam(value = "guild") String guildId) { //TODO security issue ???!!!
-        Guild guild = MainBot.jda.getGuildById(guildId);
-        if (guild == null) {
-            logger.warn("Request whit no guild!");
-            return new ResponseEntity<>(HttpStatus.BAD_REQUEST);
-        } else {
-            logger.trace("currentMusicInfo for " + guild.getName());
-        }
-
-
-        if (guild.getAudioManager().isConnected()) {
-            AudioPlayer player = AudioM.getInstance(guild).getGuildMusicManager().player;
-            AudioTrack currentTrack = player.getPlayingTrack();
-            if (currentTrack == null) {
-                return new ResponseEntity<>(new CurrentMusicData(null, 0, "STOP", false, AudioM.getInstance(guild).getGuildMusicManager().scheduler.isAutoFlow()), HttpStatus.OK);
-            }
-            UserAudioTrackData uat = new UserAudioTrackData(AudioM.getInstance(guild).getGuildMusicManager().scheduler.getCurrentPlayingTrack());
-            return new ResponseEntity<>(new CurrentMusicData(uat, currentTrack.getPosition(), currentTrack.getState().toString(), player.isPaused(), AudioM.getInstance(guild).getGuildMusicManager().scheduler.isAutoFlow()), HttpStatus.OK);
-        } else {
-            return new ResponseEntity<>(new CurrentMusicData(null, 0, "DISCONNECTED", false, false), HttpStatus.OK);
-        }
-    }
-
-    @RequestMapping("/getPlaylist")
-    public ResponseEntity<PlaylistData> getPlaylist(@RequestParam(value = "guild") String guildId) { //TODO security issue ???!!!
-        Guild guild = MainBot.jda.getGuildById(guildId);
-        if (guild == null) {
-            logger.warn("Request whit no guild!");
-            return new ResponseEntity<>(HttpStatus.BAD_REQUEST);
-        } else {
-            logger.trace("getPlaylist for " + guild.getName());
-        }
-
-        List<UserAudioTrackData> list = null;
-        list = AudioM.getInstance(guild).getGuildMusicManager().scheduler.getList();
-        return new ResponseEntity<>(new PlaylistData(list), HttpStatus.OK);
-    }
-
-
-    @RequestMapping("/getAllInfo")
-    public ResponseEntity<AllMusicInfoData> getAllInfo(@RequestParam(value = "guild") String guildId, @CookieValue("token") String token) {
-        if (token != null) {
-            Guild guild = MainBot.jda.getGuildById(guildId);
-            if (guild == null) {
-
-                logger.warn("All info without guild!");
-                return new ResponseEntity<>(HttpStatus.BAD_REQUEST);
-            }
-
-            try {
-                UserEntity user = userUtils.getUserWithApiToken(userRepository, token);
-                logger.trace("All info from USER: " + user.getUsername() + " GUILD: " + guild.getName());
-
-                PlaylistData list = new PlaylistData(AudioM.getInstance(guild).getGuildMusicManager().scheduler.getList());
-                CurrentMusicData musicData;
-                if (guild.getAudioManager().isConnected()) {
-                    AudioPlayer player = AudioM.getInstance(guild).getGuildMusicManager().player;
-                    AudioTrack currentTrack = player.getPlayingTrack();
-                    if (currentTrack == null) {
-                        musicData = new CurrentMusicData(null, 0, "STOP", false, AudioM.getInstance(guild).getGuildMusicManager().scheduler.isAutoFlow());
-                    } else {
-                        UserAudioTrackData uat = new UserAudioTrackData(AudioM.getInstance(guild).getGuildMusicManager().scheduler.getCurrentPlayingTrack());
-                        musicData = new CurrentMusicData(uat, currentTrack.getPosition(), currentTrack.getState().toString(), player.isPaused(), AudioM.getInstance(guild).getGuildMusicManager().scheduler.isAutoFlow());
-                    }
-
-                } else {
-                    musicData = new CurrentMusicData(null, 0, "DISCONNECTED", false, false);
-                }
-                return new ResponseEntity<>(new AllMusicInfoData(musicData, list), HttpStatus.OK);
-
-
-            } catch (UnknownTokenException e) {
-                logger.warn("All info with unknown token");
-                return new ResponseEntity<>(HttpStatus.UNAUTHORIZED);
-
-            }
-
-        } else {
-            logger.warn("All Info without token!");
-            return new ResponseEntity<>(HttpStatus.UNAUTHORIZED);
-
-        }
-    }
-
-    @RequestMapping(value = "/command", method = RequestMethod.POST)
-    public ResponseEntity<CommandResponseData> command(@RequestBody CommandPostData data, HttpServletRequest request, @RequestParam(value = "guild") String guildId, @CookieValue("token") String token) {
-
-        if (data.command != null) {
-            if (token != null) {
-                Guild guild = MainBot.jda.getGuildById(guildId);
-                if (guild == null) {
-                    logger.warn("Request whit no guild!");
-                    return new ResponseEntity<>(new CommandResponseData(data.command, "Missing Guild!\nPlease Re-connect.", "token"), HttpStatus.UNAUTHORIZED);
-                }
-
-                try {
-                    UserEntity user = userUtils.getUserWithApiToken(userRepository, token);
-                    logger.info("[" + guild.getName() + "] Receive command" + data.command + " from" + request.getRemoteAddr() + " USER:" + user.getUsername() + " GUILD:" + guild.getName());
-
-                    if (ApiCommandLoader.apiCommands.containsKey(data.command)) {
-
-                        UserStatsUtils.getINSTANCE().addApiCount(user, guildId);
-
-                        return ApiCommandLoader.apiCommands.get(data.command).action(data, MainBot.jda.getUserById(user.getDiscordId()), guild);
-                    } else
-                        return new ResponseEntity<>(new CommandResponseData(data.command, "Unknown Command", "command"), HttpStatus.BAD_REQUEST);
-
-                } catch (UnknownTokenException e) {
-                    logger.warn("Command with unknown token from: " + request.getRemoteAddr());
-                    return new ResponseEntity<>(new CommandResponseData(data.command, "Unknown Token!\nPlease Re-connect.", "token"), HttpStatus.UNAUTHORIZED);
-
-                }
-
-            } else {
-                logger.warn("Command without token! ip: " + request.getRemoteAddr());
-                return new ResponseEntity<>(new CommandResponseData(data.command, "Missing token!\nPlease Re-connect.", "token"), HttpStatus.UNAUTHORIZED);
-
-            }
-        } else
-            logger.info("Null");
-        return new ResponseEntity<>(new CommandResponseData(null, null), HttpStatus.NO_CONTENT);
-    }
-
-    @RequestMapping(value = "/getChanel", method = RequestMethod.GET)
-    public ResponseEntity<List<ChanelData>> getChanel(@CookieValue(name = "token", defaultValue = "") String token, @RequestParam(value = "guild") String guildId) {
-        if (token != null && !token.isEmpty()) {
-            try {
-                UserEntity user = userUtils.getUserWithApiToken(userRepository, token);
-                Guild guild = MainBot.jda.getGuildById(guildId);
-                if (guild == null) {
-                    logger.warn("Request whit no guild!");
-                    return new ResponseEntity<>(HttpStatus.BAD_REQUEST);
-                }
-                Member member = guild.getMember(CacheTools.getJdaUser(user));
-                if (member == null) {
-                    member = guild.retrieveMember(CacheTools.getJdaUser(user)).complete();
-                    if (member == null) {
-                        logger.warn("Can't find member " + user.getUsername() + " for guild " + guild.getName() + ", User not in guild ?");
-                        return new ResponseEntity<>(HttpStatus.UNAUTHORIZED);
-                    }
-                }
-                logger.trace("getPlaylist for " + guild.getName());
-                List<ChanelData> temp = new ArrayList<>();
-                for (VoiceChannel aChanel : GetVoiceChanels.find(guild, member)) {
-                    temp.add(new ChanelData(aChanel.getName(), aChanel.getId(), aChanel.getPosition()));
-                }
-                return new ResponseEntity<>(temp, HttpStatus.OK);
-            } catch (UnknownTokenException e) {
-                logger.warn("Get Chanel without token!");
-                return new ResponseEntity<>(HttpStatus.UNAUTHORIZED);
-            }
-
-        } else {
-            logger.warn("Get Chanel without token!");
-            return new ResponseEntity<>(HttpStatus.UNAUTHORIZED);
-        }
-
-    }
-
-    @RequestMapping(value = "/search", method = RequestMethod.GET)
-    public ResponseEntity<List<SearchResult>> search(@CookieValue(name = "token", defaultValue = "") String token, @RequestParam(value = "query") String query) {
-        if (token != null && !token.isEmpty()) {
-            try {
-                UserEntity user = userUtils.getUserWithApiToken(userRepository, token);
-                YoutubeSearchRework youtubeSearch = YoutubeSearchRework.getInstance();
-                List<SearchResult> result = youtubeSearch.searchVideo(query, 25, false);
-                return new ResponseEntity<>(result, HttpStatus.OK);
-
-            } catch (UnknownTokenException e) {
-                logger.warn("Search without token!");
-                return new ResponseEntity<>(HttpStatus.UNAUTHORIZED);
-            } catch (GoogleJsonResponseException e) {
-                logger.error("There was a service error: " + e.getDetails().getCode() + " : " + e.getDetails().getMessage());
-                return new ResponseEntity<>(HttpStatus.INTERNAL_SERVER_ERROR);
-            } catch (IOException e) {
-                logger.catching(e);
-                return new ResponseEntity<>(HttpStatus.INTERNAL_SERVER_ERROR);
-
-            }
-        } else {
-            logger.warn("Search without token!");
-            return new ResponseEntity<>(HttpStatus.UNAUTHORIZED);
-        }
-    }
-
-
-}
-

src/main/java/net/Broken/RestApi/PlaylistAPIController.java

@@ -1,99 +0,0 @@
-package net.Broken.RestApi;
-
-
-import net.Broken.DB.Entity.PlaylistEntity;
-import net.Broken.DB.Entity.UserEntity;
-import net.Broken.DB.Repository.PlaylistRepository;
-import net.Broken.DB.Repository.TrackRepository;
-import net.Broken.DB.Repository.UserRepository;
-import net.Broken.RestApi.Data.Playlist.*;
-import net.Broken.audio.Playlist.PlaylistManager;
-import org.apache.logging.log4j.LogManager;
-import org.apache.logging.log4j.Logger;
-import org.springframework.beans.factory.annotation.Autowired;
-import org.springframework.http.HttpStatus;
-import org.springframework.http.ResponseEntity;
-import org.springframework.web.bind.annotation.CookieValue;
-import org.springframework.web.bind.annotation.RequestBody;
-import org.springframework.web.bind.annotation.RequestMapping;
-import org.springframework.web.bind.annotation.RestController;
-
-import java.util.List;
-
-@RestController
-@RequestMapping("/api/playlist/")
-public class PlaylistAPIController {
-
-    private final
-    UserRepository userRepository;
-
-    private final
-    PlaylistRepository playlistRepository;
-
-    private final
-    TrackRepository trackRepository;
-
-    private Logger logger = LogManager.getLogger();
-
-    @Autowired
-    public PlaylistAPIController(UserRepository userRepository, PlaylistRepository playlistRepository, TrackRepository trackRepository) {
-        this.userRepository = userRepository;
-        this.playlistRepository = playlistRepository;
-        this.trackRepository = trackRepository;
-    }
-
-
-//    @RequestMapping("/myPlaylist")
-//    public List<PlaylistEntity> myPlaylist(@CookieValue(value = "token", defaultValue = "") String token) {
-//        if (token.isEmpty())
-//            return null;
-//        else {
-//            UserEntity user = userRepository.findByApiToken(token).get(0);
-//            return user.getPlaylists();
-//        }
-//
-//    }
-
-//    @RequestMapping("/createPlaylist")
-//    public ResponseEntity<PlaylistResponseData> createPlaylist(@CookieValue(value = "token", defaultValue = "") String token, @RequestBody CreatePlaylistData data) {
-//
-//        if (token.isEmpty())
-//            return new ResponseEntity<>(new PlaylistResponseData("Unknown Token!\nPlease Re-connect.", "token"), HttpStatus.UNAUTHORIZED);
-//        else {
-//            UserEntity user = userRepository.findByApiToken(token).get(0);
-//            PlaylistEntity playlistEntity = new PlaylistEntity(data.name, user);
-//            playlistEntity = playlistRepository.save(playlistEntity);
-//            user.addPlaylist(playlistEntity);
-//            userRepository.save(user);
-//            return new ResponseEntity<>(new PlaylistResponseData("Ok", playlistEntity), HttpStatus.OK);
-//        }
-//
-//
-//    }
-
-    @RequestMapping("/addToPlaylist")
-    public ResponseEntity<PlaylistResponseData> addToPlaylist(@CookieValue(value = "token", defaultValue = "") String token, @RequestBody AddToPlaylistData data) {
-        PlaylistManager playlistManager = PlaylistManager.getINSTANCE();
-
-        return playlistManager.addToPlaylist(token, data);
-
-    }
-
-    @RequestMapping("/dellTrack")
-    public ResponseEntity<PlaylistResponseData> dellTrack(@CookieValue(value = "token", defaultValue = "") String token, @RequestBody DeleteTrackData data) {
-        PlaylistManager playlistManager = PlaylistManager.getINSTANCE();
-
-        return playlistManager.removeTrack(token, data);
-
-    }
-
-    @RequestMapping("/moveTrack")
-    public ResponseEntity<PlaylistResponseData> moveTrack(@CookieValue(value = "token", defaultValue = "") String token, @RequestBody MoveTrackData data) {
-        PlaylistManager playlistManager = PlaylistManager.getINSTANCE();
-
-        return playlistManager.moveTrack(token, data);
-
-    }
-
-
-}

src/main/java/net/Broken/SlashCommand.java

@@ -10,12 +10,6 @@ import java.util.List;
  * Interface that define command structure.
  */
 public interface SlashCommand {
-    /**
-     * Main action of command
-     *
-     * @param args  Command args.
-     * @param event Command MessageReceivedEvent
-     */
     void action(SlashCommandEvent event);
 
     String getDescription();

src/main/java/net/Broken/SlashCommands/Cat.java

@@ -1,6 +1,8 @@
 package net.Broken.SlashCommands;
 
 
+import com.fasterxml.jackson.core.type.TypeReference;
+import com.fasterxml.jackson.databind.ObjectMapper;
 import net.Broken.SlashCommand;
 import net.Broken.Tools.EmbedMessageUtils;
 import net.dv8tion.jda.api.MessageBuilder;
@@ -9,41 +11,48 @@ import net.dv8tion.jda.api.interactions.commands.build.OptionData;
 import net.dv8tion.jda.api.interactions.commands.build.SubcommandData;
 import org.apache.logging.log4j.LogManager;
 import org.apache.logging.log4j.Logger;
-import org.json.JSONObject;
 
 import java.io.BufferedReader;
 import java.io.IOException;
 import java.io.InputStreamReader;
+import java.net.URI;
 import java.net.URL;
 import java.net.URLConnection;
-import java.util.ArrayList;
+import java.net.http.HttpClient;
+import java.net.http.HttpRequest;
+import java.net.http.HttpResponse;
+import java.nio.charset.StandardCharsets;
+import java.util.HashMap;
 import java.util.List;
 
 /**
  * Command that return a random picture of cat.
  */
 public class Cat implements SlashCommand {
-    private Logger logger = LogManager.getLogger();
-    ;
+    private final Logger logger = LogManager.getLogger();
 
     @Override
     public void action(SlashCommandEvent event) {
         try {
-            URL urlC = new URL("http://aws.random.cat/meo");
-            URLConnection yc = urlC.openConnection();
-            BufferedReader in = new BufferedReader(new InputStreamReader(
-                    yc.getInputStream(), "UTF-8"));
-            String inputLine;
-            StringBuilder a = new StringBuilder();
-            while ((inputLine = in.readLine()) != null)
-                a.append(inputLine);
-            in.close();
+            HttpRequest request = HttpRequest.newBuilder()
+                    .uri(URI.create("https://aws.random.cat/meow"))
+                    .GET()
+                    .build();
 
-            JSONObject json = new JSONObject(a.toString());
 
-            event.reply(json.getString("file")).queue();
+            HttpClient client = HttpClient.newHttpClient();
+            HttpResponse<String> response = client.send(request, HttpResponse.BodyHandlers.ofString());
+            if (response.statusCode() != 200) {
+                logger.warn("[CAT] Fail to fetch cat: Status Code: " + response.statusCode() + " Body:" + response.body());
+                throw new IOException();
+            }
 
-        } catch (IOException e) {
+            TypeReference<HashMap<String, String>> typeRef = new TypeReference<>() {};
+            ObjectMapper mapper = new ObjectMapper();
+            HashMap<String, String> json = mapper.readValue(response.body(), typeRef);
+            event.reply(json.get("file")).queue();
+
+        } catch (InterruptedException | IOException e) {
             logger.catching(e);
             event.reply(new MessageBuilder().setEmbeds(EmbedMessageUtils.getInternalError()).build()).setEphemeral(true).queue();
         }

src/main/java/net/Broken/SlashCommands/Clear.java

@@ -2,7 +2,6 @@ package net.Broken.SlashCommands;
 
 import net.Broken.SlashCommand;
 import net.Broken.Tools.EmbedMessageUtils;
-import net.dv8tion.jda.api.EmbedBuilder;
 import net.dv8tion.jda.api.MessageBuilder;
 import net.dv8tion.jda.api.Permission;
 import net.dv8tion.jda.api.entities.Message;

src/main/java/net/Broken/SlashCommands/Music.java

@@ -1,9 +1,9 @@
 package net.Broken.SlashCommands;
 
 
+import net.Broken.Audio.GuildAudioBotService;
 import net.Broken.SlashCommand;
 import net.Broken.Tools.EmbedMessageUtils;
-import net.Broken.audio.AudioM;
 import net.dv8tion.jda.api.MessageBuilder;
 import net.dv8tion.jda.api.entities.Message;
 import net.dv8tion.jda.api.entities.VoiceChannel;
@@ -22,12 +22,11 @@ import java.util.List;
  * Command that return a random picture of cat.
  */
 public class Music implements SlashCommand {
-    private Logger logger = LogManager.getLogger();
-    ;
+    private final Logger logger = LogManager.getLogger();
 
     @Override
     public void action(SlashCommandEvent event) {
-        AudioM audio = AudioM.getInstance(event.getGuild());
+        GuildAudioBotService audio = GuildAudioBotService.getInstance(event.getGuild());
         String action = event.getSubcommandName();
         event.deferReply().queue();
         switch (action) {

src/main/java/net/Broken/SlashCommands/Over18/Ass.java

@@ -3,16 +3,11 @@ package net.Broken.SlashCommands.Over18;
 import net.Broken.Tools.Command.Ignore;
 import net.Broken.Tools.Command.NoDev;
 import net.Broken.Tools.Command.NumberedSlashCommand;
-import net.Broken.Tools.FindContentOnWebPage;
 import org.apache.logging.log4j.LogManager;
-import org.apache.logging.log4j.Logger;
-
-import java.io.IOException;
 
 @NoDev
 @Ignore
 public class Ass extends NumberedSlashCommand {
-    Logger logger = LogManager.getLogger();
 
     public Ass() {
         super(LogManager.getLogger(), "http://les400culs.com/", "-2/", "featured-img", "img");

src/main/java/net/Broken/SlashCommands/Over18/Madame.java

@@ -1,6 +1,5 @@
 package net.Broken.SlashCommands.Over18;
 
-import net.Broken.Tools.Command.Ignore;
 import net.Broken.Tools.Command.NoDev;
 import net.Broken.Tools.Command.NumberedSlashCommand;
 import net.Broken.Tools.FindContentOnWebPage;
@@ -11,7 +10,7 @@ import java.io.IOException;
 
 @NoDev
 public class Madame extends NumberedSlashCommand {
-    Logger logger = LogManager.getLogger();
+    final Logger logger = LogManager.getLogger();
 
     public Madame() {
         super(LogManager.getLogger(), "https://www.bonjourmadame.fr/page/", "/");
@@ -24,7 +23,7 @@ public class Madame extends NumberedSlashCommand {
      * @throws StringIndexOutOfBoundsException
      * @throws IOException
      */
-    private boolean scanPageForTipeee(String url, Logger logger) throws StringIndexOutOfBoundsException, IOException {
+    private boolean scanPageForTipeee(String url, Logger logger) throws StringIndexOutOfBoundsException, IOException, InterruptedException {
         String content = FindContentOnWebPage.getSourceUrl(url);
         String imgClickLink = content.substring(content.indexOf("class=\"post-content"));
         imgClickLink = imgClickLink.substring(imgClickLink.indexOf("<a"));
@@ -49,7 +48,7 @@ public class Madame extends NumberedSlashCommand {
 
 
     @Override
-    public String poll() throws IOException {
+    public String poll() throws IOException, InterruptedException {
         boolean success = false;
         String imgUrl = null;
         while (!success) {

src/main/java/net/Broken/SlashCommands/Rank.java

@@ -1,7 +1,6 @@
 package net.Broken.SlashCommands;
 
 import net.Broken.BotConfigLoader;
-import net.Broken.MainBot;
 import net.Broken.SlashCommand;
 import net.Broken.SpringContext;
 import net.Broken.Tools.UserManager.Stats.UserStatsUtils;

src/main/java/net/Broken/Tools/CacheTools.java

@@ -10,7 +10,7 @@ import org.apache.logging.log4j.Logger;
 import java.util.List;
 
 public class CacheTools {
-    private static Logger logger = LogManager.getLogger();
+    private static final Logger logger = LogManager.getLogger();
 
     public static void loadAllGuildMembers() {
         List<Guild> guilds = MainBot.jda.getGuilds();

src/main/java/net/Broken/Tools/Command/CommandParser.java

@@ -12,7 +12,7 @@ import java.util.Arrays;
  */
 
 public class CommandParser {
-    private Logger logger = LogManager.getLogger();
+    private final Logger logger = LogManager.getLogger();
 
     /**
      * Parse raw received string.
@@ -22,9 +22,8 @@ public class CommandParser {
      * @return Readable container that contain all useful data
      */
     public CommandContainer parse(String brt, MessageReceivedEvent e) {
-        ArrayList<String> split = new ArrayList<String>();
-        String brut = brt;
-        String sansTete = brut.replaceFirst("//", "");
+        ArrayList<String> split = new ArrayList<>();
+        String sansTete = brt.replaceFirst("//", "");
         String[] splitSansTete = sansTete.split(" ");
 
         for (String s : splitSansTete) {
@@ -42,14 +41,14 @@ public class CommandParser {
 
         logger.info("Author: " + e.getAuthor().getName() + ", Command: " + commande + ", args: " + Arrays.toString(args));
 
-        return new CommandContainer(brut, sansTete, splitSansTete, commande, args, e);
+        return new CommandContainer(brt, sansTete, splitSansTete, commande, args, e);
 
     }
 
     /**
      * Container
      */
-    public class CommandContainer {
+    public static class CommandContainer {
         public final String brut;
         public final String sansTete;
         public final String[] splitSansTete;

src/main/java/net/Broken/Tools/Command/NumberedSlashCommand.java

@@ -1,22 +1,18 @@
 package net.Broken.Tools.Command;
 
-import net.Broken.Commande;
 import net.Broken.SlashCommand;
 import net.Broken.Tools.EmbedMessageUtils;
 import net.Broken.Tools.FindContentOnWebPage;
 import net.Broken.Tools.LimitChecker;
-import net.Broken.Tools.TrueRandom;
+import net.Broken.Tools.Random.TrueRandom;
 import net.dv8tion.jda.api.entities.MessageEmbed;
 import net.dv8tion.jda.api.events.interaction.SlashCommandEvent;
-import net.dv8tion.jda.api.events.message.MessageReceivedEvent;
 import net.dv8tion.jda.api.interactions.commands.build.OptionData;
 import net.dv8tion.jda.api.interactions.commands.build.SubcommandData;
 import org.apache.logging.log4j.LogManager;
 import org.apache.logging.log4j.Logger;
 
 import java.io.IOException;
-import java.net.HttpURLConnection;
-import java.net.URL;
 import java.util.ArrayList;
 import java.util.List;
 import java.util.concurrent.LinkedBlockingQueue;
@@ -26,13 +22,13 @@ import java.util.concurrent.LinkedBlockingQueue;
  */
 @Ignore
 public abstract class NumberedSlashCommand implements SlashCommand {
-    protected int minNumber = 1;
+    protected final int minNumber = 1;
     protected int maxNumber = -1;
-    protected String baseURL;
-    protected String divClass;
-    protected String htmlType;
-    protected String urlSuffix;
-    protected LinkedBlockingQueue<Integer> randomQueue = new LinkedBlockingQueue<>();
+    protected final String baseURL;
+    protected final String divClass;
+    protected final String htmlType;
+    protected final String urlSuffix;
+    protected final LinkedBlockingQueue<Integer> randomQueue = new LinkedBlockingQueue<>();
     private Logger logger = LogManager.getLogger();
 
     /**
@@ -69,7 +65,7 @@ public abstract class NumberedSlashCommand implements SlashCommand {
         try {
             String result = poll();
             event.getHook().sendMessage(event.getMember().getAsMention() + "\n" + result).queue();
-        } catch (IOException e) {
+        } catch (IOException | InterruptedException e) {
             logger.catching(e);
             MessageEmbed message = EmbedMessageUtils.getInternalError();
             event.getHook().setEphemeral(true).sendMessageEmbeds(message).queue();
@@ -79,7 +75,7 @@ public abstract class NumberedSlashCommand implements SlashCommand {
 
     private void fillRandomQueue() throws IOException {
         TrueRandom trueRandom = TrueRandom.getINSTANCE();
-        ArrayList<Integer> numbers = trueRandom.getNumbers(minNumber, maxNumber);
+        List<Integer> numbers = trueRandom.getNumbers(minNumber, maxNumber);
 
         randomQueue.addAll(numbers);
 
@@ -93,7 +89,7 @@ public abstract class NumberedSlashCommand implements SlashCommand {
         }
     }
 
-    public String poll() throws IOException {
+    public String poll() throws IOException, InterruptedException {
         checkRandom();
         int randomResult = randomQueue.poll();
         return FindContentOnWebPage.doYourJob(baseURL + randomResult + urlSuffix, divClass, htmlType);

src/main/java/net/Broken/Tools/Command/SlashCommandLoader.java

@@ -19,7 +19,7 @@ import java.util.Set;
  * Find and load bot's command
  */
 public class SlashCommandLoader {
-    private static Logger logger = LogManager.getLogger();
+    private static final Logger logger = LogManager.getLogger();
 
     /**
      * Search all implemented Command interface class and add it to MainBot.commands HashMap

src/main/java/net/Broken/Tools/DayListener/DayListener.java

@@ -11,46 +11,26 @@ import java.util.GregorianCalendar;
  * Day change listener
  */
 public class DayListener extends Thread {
-    private static DayListener INSTANCE = new DayListener();
-    private Logger logger = LogManager.getLogger();
+    private static final DayListener INSTANCE = new DayListener();
+    private final Logger logger = LogManager.getLogger();
     private Calendar calendar;
     private int previousDay;
-    /**
-     * List of listeners to need to be triggered
-     */
-    private ArrayList<NewDayListener> listeners = new ArrayList<>();
 
+    private final ArrayList<NewDayListener> listeners = new ArrayList<>();
 
-    /**
-     * Default private constructor
-     */
     private DayListener() {
         calendar = Calendar.getInstance();
         previousDay = calendar.get(GregorianCalendar.DAY_OF_MONTH);
     }
 
-
-    /**
-     * Singleton
-     *
-     * @return Unique DayListener instance.
-     */
     public static DayListener getInstance() {
         return INSTANCE;
     }
 
-    /**
-     * Add Listener who will be triggered
-     *
-     * @param listener
-     */
     public void addListener(NewDayListener listener) {
         listeners.add(listener);
     }
 
-    /**
-     * Trigger all listeners
-     */
     public void trigger() {
         for (NewDayListener listener : listeners) {
             try {

src/main/java/net/Broken/Tools/DayListener/Listeners/DailyMadame.java

@@ -25,8 +25,8 @@ import java.util.Optional;
  */
 public class DailyMadame implements NewDayListener {
 
-    private GuildPreferenceRepository guildPreferenceRepository;
-    private Logger logger = LogManager.getLogger();
+    private final GuildPreferenceRepository guildPreferenceRepository;
+    private final Logger logger = LogManager.getLogger();
 
     public DailyMadame() {
         ApplicationContext context = SpringContext.getAppContext();
@@ -78,7 +78,7 @@ public class DailyMadame implements NewDayListener {
                     }
                 }
             }
-        } catch (IOException e) {
+        } catch (IOException |InterruptedException e) {
             logger.catching(e);
         }
     }

src/main/java/net/Broken/Tools/EmbedMessageUtils.java

@@ -1,19 +1,15 @@
 package net.Broken.Tools;
 
 import com.sedmelluq.discord.lavaplayer.track.AudioTrackInfo;
+import net.Broken.Audio.UserAudioTrack;
 import net.Broken.BotConfigLoader;
 import net.Broken.MainBot;
 import net.Broken.SpringContext;
-import net.Broken.audio.UserAudioTrack;
-import net.Broken.audio.Youtube.SearchResult;
 import net.dv8tion.jda.api.EmbedBuilder;
 import net.dv8tion.jda.api.entities.Member;
 import net.dv8tion.jda.api.entities.MessageEmbed;
-import net.dv8tion.jda.api.entities.User;
-import org.springframework.context.ApplicationContext;
 
 import java.awt.*;
-import java.io.FileNotFoundException;
 import java.time.Instant;
 import java.util.HashMap;
 import java.util.Map;
@@ -24,11 +20,10 @@ import java.util.Map;
 public class EmbedMessageUtils {
 
     public static EmbedBuilder getError(String message) {
-        EmbedBuilder temp = new EmbedBuilder()
+        return new EmbedBuilder()
                 .setTitle(":warning: Error!")
                 .setColor(Color.red)
                 .setDescription(message);
-        return temp;
     }
 
 

src/main/java/net/Broken/Tools/FindContentOnWebPage.java

@@ -3,8 +3,13 @@ package net.Broken.Tools;
 import java.io.BufferedReader;
 import java.io.IOException;
 import java.io.InputStreamReader;
+import java.net.URI;
 import java.net.URL;
 import java.net.URLConnection;
+import java.net.http.HttpClient;
+import java.net.http.HttpRequest;
+import java.net.http.HttpResponse;
+import java.nio.charset.StandardCharsets;
 
 public class FindContentOnWebPage {
     /**
@@ -16,7 +21,7 @@ public class FindContentOnWebPage {
      * @return Picture URL
      * @throws IOException
      */
-    public static String doYourJob(String url, String divClass, String htmlType) throws IOException {
+    public static String doYourJob(String url, String divClass, String htmlType) throws IOException, InterruptedException {
 //        System.out.println(url);
         String source = getSourceUrl(url);
         int divIndex = source.indexOf("class=\"" + divClass);
@@ -38,18 +43,16 @@ public class FindContentOnWebPage {
      * @return Web page source as String
      * @throws IOException
      */
-    public static String getSourceUrl(String url) throws IOException {
-        URL urlC = new URL(url);
-        URLConnection yc = urlC.openConnection();
-        yc.setRequestProperty("User-Agent", "Googlebot/2.1 (+http://www.googlebot.com/bot.html)");
-        BufferedReader in = new BufferedReader(new InputStreamReader(
-                yc.getInputStream(), "UTF-8"));
-        String inputLine;
-        StringBuilder a = new StringBuilder();
-        while ((inputLine = in.readLine()) != null)
-            a.append(inputLine);
-        in.close();
+    public static String getSourceUrl(String url) throws IOException, InterruptedException {
 
-        return a.toString();
+        HttpRequest request = HttpRequest.newBuilder()
+                .uri(URI.create(url))
+                .header("User-Agent", "Mozilla/5.0 (compatible; Googlebot/2.1; +http://www.google.com/bot.html)")
+                .GET()
+                .build();
+        HttpClient client = HttpClient.newHttpClient();
+        HttpResponse<String> response = client.send(request, HttpResponse.BodyHandlers.ofString());
+
+        return response.body();
     }
 }

src/main/java/net/Broken/Tools/LimitChecker.java

@@ -10,7 +10,7 @@ import java.net.URL;
  * Find max webPage for web site like baseURL.com/number-2/
  */
 public class LimitChecker {
-    static Logger logger = LogManager.getLogger();
+    static final Logger logger = LogManager.getLogger();
 
 
     /**

src/main/java/net/Broken/Tools/Random/Data/RandomData.java

@@ -0,0 +1,18 @@
+package net.Broken.Tools.Random.Data;
+
+import com.fasterxml.jackson.annotation.JsonInclude;
+
+import java.util.List;
+
+@JsonInclude(JsonInclude.Include.NON_NULL)
+public record RandomData(String jsonrpc, String method, int id, Params params, Error error, Result result) {
+    public record Params(String apiKey, int n, int min, int max, boolean replacement) {
+    }
+
+    public record Error(long code, String message, Object data){};
+    public record Result(Random random, String bitsUsed, String bitsLeft, String requestsLeft, String advisoryDelay){
+        public record Random(List<Integer> data, String completionTime){};
+    }
+}
+
+

src/main/java/net/Broken/Tools/Random/TrueRandom.java

@@ -1,7 +1,9 @@
-package net.Broken.Tools;
+package net.Broken.Tools.Random;
 
+import com.fasterxml.jackson.databind.ObjectMapper;
 import net.Broken.BotConfigLoader;
 import net.Broken.SpringContext;
+import net.Broken.Tools.Random.Data.RandomData;
 import org.apache.commons.io.IOUtils;
 import org.apache.http.HttpResponse;
 import org.apache.http.client.HttpClient;
@@ -11,11 +13,10 @@ import org.apache.http.entity.StringEntity;
 import org.apache.http.impl.client.HttpClientBuilder;
 import org.apache.logging.log4j.LogManager;
 import org.apache.logging.log4j.Logger;
-import org.json.JSONObject;
 
 import java.io.IOException;
 import java.io.InputStream;
-import java.util.ArrayList;
+import java.nio.charset.StandardCharsets;
 import java.util.List;
 
 public class TrueRandom {
@@ -31,13 +32,15 @@ public class TrueRandom {
         return INSTANCE;
     }
 
-    public ArrayList<Integer> getNumbers(int min, int max) throws IOException {
+    public List<Integer> getNumbers(int min, int max) throws IOException {
 
 //        TODO Migrate to native http client
         HttpClient httpClient = HttpClientBuilder.create().build();
 
-        String postVal = "{\"jsonrpc\":\"2.0\",\"method\":\"generateIntegers\",\"params\":{\"apiKey\":\"" + apiKey + "\",\"n\":50,\"min\":" + min + ",\"max\":" + max + ",\"replacement\":" + (((max - min) >= 50) ? "false" : "true") + "},\"id\":41}";
-        StringEntity entity = new StringEntity(postVal, ContentType.APPLICATION_JSON);
+        RandomData postData = new RandomData("2.0", "generateIntegers", 41, new RandomData.Params(apiKey, 50, min, max, (max - min) < 50), null, null);
+        ObjectMapper mapper = new ObjectMapper();
+
+        StringEntity entity = new StringEntity(mapper.writeValueAsString(postData), ContentType.APPLICATION_JSON);
         String url = "https://api.random.org/json-rpc/2/invoke";
         HttpPost request = new HttpPost(url);
         request.setEntity(entity);
@@ -54,29 +57,22 @@ public class TrueRandom {
 
 
         InputStream responseIS = response.getEntity().getContent();
-        String content = IOUtils.toString(responseIS, "UTF-8");
-        logger.trace(content);
+        String content = IOUtils.toString(responseIS, StandardCharsets.UTF_8);
 
-        JSONObject json = new JSONObject(content);
-        if (json.keySet().contains("error")) {
+        RandomData responseData = mapper.readValue(content, RandomData.class);
+
+        if (responseData.error() != null) {
             logger.error("Request fail!");
-            logger.error("Request : " + postVal);
-            logger.error("Response : " + content);
+            logger.error("Response : " + responseData.error().message());
             throw new IOException();
 
         }
 
-        logger.debug("Request left: " + json.getJSONObject("result").getInt("requestsLeft"));
-        logger.debug("Bits left: " + json.getJSONObject("result").getInt("bitsLeft"));
-        logger.debug("Numbers: " + json.getJSONObject("result").getJSONObject("random").getJSONArray("data"));
+        logger.debug("Request left: " + responseData.result().requestsLeft());
+        logger.debug("Bits left: " + responseData.result().bitsLeft());
+        logger.debug("Numbers: " + responseData.result().random().data());
 
-        List<Object> numbers = json.getJSONObject("result").getJSONObject("random").getJSONArray("data").toList();
-
-        ArrayList<Integer> converted = new ArrayList<>();
-        for (Object nbr : numbers) {
-            converted.add((Integer) nbr);
-        }
-        return converted;
+        return responseData.result().random().data();
 
     }
 

src/main/java/net/Broken/Tools/ResourceLoader.java

@@ -1,47 +0,0 @@
-package net.Broken.Tools;
-
-import org.apache.logging.log4j.LogManager;
-import org.apache.logging.log4j.Logger;
-
-import java.io.FileNotFoundException;
-import java.io.InputStream;
-import java.util.Scanner;
-
-
-public class ResourceLoader {
-
-    private Logger logger = LogManager.getLogger();
-
-    /**
-     * Get file contents as string for resource folder
-     *
-     * @param fileName Requested file
-     * @return File contents as string
-     * @throws FileNotFoundException
-     */
-    public String getFile(String fileName) throws FileNotFoundException {
-
-        StringBuilder result = new StringBuilder("");
-
-        //Get file from resources folder
-        ClassLoader classLoader = getClass().getClassLoader();
-        InputStream file = classLoader.getResourceAsStream(fileName);
-        if (file == null)
-            throw new FileNotFoundException();
-
-        try (Scanner scanner = new Scanner(file, "UTF-8")) {
-
-            while (scanner.hasNextLine()) {
-                String line = scanner.nextLine();
-                result.append(line).append("\n");
-            }
-
-            scanner.close();
-
-        }
-
-        return result.toString();
-
-    }
-
-}

src/main/java/net/Broken/Tools/SettingsUtils.java

@@ -1,24 +1,14 @@
 package net.Broken.Tools;
 
 import net.Broken.DB.Entity.GuildPreferenceEntity;
-import net.Broken.DB.Entity.UserEntity;
 import net.Broken.DB.Repository.GuildPreferenceRepository;
 import net.Broken.DB.Repository.UserRepository;
-import net.Broken.MainBot;
-import net.Broken.RestApi.Data.Settings.GetSettingsData;
-import net.Broken.RestApi.Data.Settings.PostSetSettings;
-import net.Broken.RestApi.Data.Settings.Value;
 import net.Broken.SpringContext;
-import net.Broken.Tools.UserManager.UserUtils;
-import net.dv8tion.jda.api.Permission;
-import net.dv8tion.jda.api.entities.*;
+import net.dv8tion.jda.api.entities.Guild;
 import org.apache.logging.log4j.LogManager;
 import org.apache.logging.log4j.Logger;
 import org.springframework.context.ApplicationContext;
 
-import java.util.ArrayList;
-import java.util.List;
-
 public class SettingsUtils {
 
     private static SettingsUtils INSTANCE;
@@ -37,78 +27,6 @@ public class SettingsUtils {
         return (INSTANCE == null) ? new SettingsUtils() : INSTANCE;
     }
 
-    /**
-     * Check if the user have the permission to set settings
-     *
-     * @param token User token
-     * @param guild Guild
-     * @return True if user have ADMINISTRATOR permission
-     */
-    public boolean checkPermission(String token, String guild) {
-        if (token == null || guild == null) {
-            return false;
-        } else {
-            try {
-                UserEntity user = UserUtils.getInstance().getUserWithApiToken(userRepository, token);
-                User jdaUser = MainBot.jda.getUserById(user.getDiscordId());
-                Guild jdaGuild = MainBot.jda.getGuildById(guild);
-                if (jdaGuild == null || jdaUser == null)
-                    return false;
-
-                Member guildUser = jdaGuild.getMember(jdaUser);
-                if (guildUser == null)
-                    return false;
-                return guildUser.hasPermission(Permission.ADMINISTRATOR);
-            } catch (Exception e) {
-                logger.debug("Unknown Token or user :" + token);
-                return false;
-            }
-        }
-    }
-
-
-    private List<Value> getTextChannels(Guild guild) {
-        List<Value> channels = new ArrayList<>();
-        for (TextChannel channel : guild.getTextChannels()) {
-            channels.add(new Value(channel.getName(), channel.getId()));
-        }
-        return channels;
-    }
-
-    private List<Value> getRoles(Guild guild) {
-        List<Value> roles = new ArrayList<>();
-        for (Role role : guild.getRoles()) {
-            roles.add(new Value(role.getName(), role.getId()));
-        }
-        return roles;
-    }
-
-
-    private List<Value> getVoiceChannels(Guild guild, List<String> selected) {
-        List<Value> channels = new ArrayList<>();
-        for (VoiceChannel voiceChannel : guild.getVoiceChannels()) {
-            if (selected == null)
-                channels.add(new Value(voiceChannel.getName(), voiceChannel.getId()));
-            else
-                channels.add(new Value(voiceChannel.getName(), voiceChannel.getId(), selected.contains(voiceChannel.getId())));
-        }
-        return channels;
-    }
-
-    private List<String> checkVoiceChanel(Guild guild, List<String> values) {
-        List<String> list = new ArrayList<>();
-        for (String value : values) {
-            if (guild.getVoiceChannelById(value) != null) {
-                list.add(value);
-            } else {
-                logger.error("Unknown voice channel id: " + value);
-                list = null;
-                break;
-            }
-        }
-        return list;
-    }
-
     public GuildPreferenceEntity getPreference(Guild guild) {
         return guildPreferenceRepository.findByGuildId(guild.getId()).orElseGet(()->{
             logger.info("Generate default pref for " + guild.getName());

src/main/java/net/Broken/Tools/UserManager/Stats/GuildStatsPack.java

@@ -2,16 +2,5 @@ package net.Broken.Tools.UserManager.Stats;
 
 import java.util.List;
 
-public class GuildStatsPack {
-
-
-    public int rank;
-    public GuildStats selfStats;
-    public List<GuildStats> ranking;
-
-    public GuildStatsPack(int rank, GuildStats selfStats, List<GuildStats> ranking) {
-        this.rank = rank;
-        this.selfStats = selfStats;
-        this.ranking = ranking;
-    }
+public record GuildStatsPack(int rank, GuildStats selfStats, List<GuildStats> ranking) {
 }

src/main/java/net/Broken/Tools/UserManager/Stats/UserStatsUtils.java

@@ -8,8 +8,6 @@ import net.Broken.MainBot;
 import net.Broken.SpringContext;
 import net.Broken.Tools.CacheTools;
 import net.Broken.Tools.EmbedMessageUtils;
-import net.Broken.Tools.UserManager.Exceptions.UnknownTokenException;
-import net.Broken.Tools.UserManager.UserUtils;
 import net.dv8tion.jda.api.EmbedBuilder;
 import net.dv8tion.jda.api.entities.Guild;
 import net.dv8tion.jda.api.entities.Member;
@@ -18,7 +16,6 @@ import net.dv8tion.jda.api.entities.User;
 import org.apache.logging.log4j.LogManager;
 import org.apache.logging.log4j.Logger;
 import org.springframework.context.ApplicationContext;
-import org.springframework.security.crypto.password.PasswordEncoder;
 
 import java.awt.*;
 import java.util.ArrayList;
@@ -27,23 +24,22 @@ import java.util.List;
 
 public class UserStatsUtils {
 
-    static double XP_PER_VOICE_TIME = 0.01;
-    static double XP_PER_MESSAGE = 4;
-    static double XP_PER_API_COUNT = 1;
+    static final double XP_PER_VOICE_TIME = 0.01;
+    static final double XP_PER_MESSAGE = 4;
+    static final double XP_PER_API_COUNT = 1;
 
 
     private static UserStatsUtils INSTANCE = new UserStatsUtils();
     private final UserStatsRepository userStatsRepository;
     private final UserRepository userRepository;
-    private final PasswordEncoder passwordEncoder;
     private final Logger logger = LogManager.getLogger();
-    public HashMap<String, VoicePresenceCounter> runningCounters = new HashMap<>();
+    public final HashMap<String, VoicePresenceCounter> runningCounters = new HashMap<>();
 
     private UserStatsUtils() {
         ApplicationContext context = SpringContext.getAppContext();
         userStatsRepository = (UserStatsRepository) context.getBean("userStatsRepository");
         userRepository = (UserRepository) context.getBean("userRepository");
-        passwordEncoder = (PasswordEncoder) context.getBean("passwordEncoder");
+
 
     }
 
@@ -99,11 +95,6 @@ public class UserStatsUtils {
         return userEntity.getUserStats();
     }
 
-    public List<UserStats> getUserStats(String token) throws UnknownTokenException {
-        UserEntity user = UserUtils.getInstance().getUserWithApiToken(userRepository, token);
-        return getUserStats(user);
-    }
-
     public List<UserStats> getUserStats(User user) {
         UserEntity userEntity = userRepository.findByDiscordId(user.getId())
                 .orElseGet(() -> genUserEntity(user));
@@ -227,7 +218,7 @@ public class UserStatsUtils {
         GuildStatsPack pack = getStatPack(userStats.getUser(), member.getGuild().getId());
         StringBuilder stringBuilder = new StringBuilder();
         int i = 1;
-        for (GuildStats stats : pack.ranking) {
+        for (GuildStats stats : pack.ranking()) {
             if (i >= 6) {
                 break;
             }
@@ -240,24 +231,15 @@ public class UserStatsUtils {
         embedBuilder.setColor(Color.yellow);
         embedBuilder.setTitle(":trophy:  " + member.getGuild().getName() + " Ranking");
         embedBuilder.addField("Top 5:", stringBuilder.toString(), false);
-        String rank;
-        switch (pack.selfStats.rank) {
-            case 1:
-                rank = "1st";
-                break;
-            case 2:
-                rank = "2nd";
-                break;
-            case 3:
-                rank = "3rd";
-                break;
-            default:
-                rank = pack.selfStats.rank + "th";
-                break;
-        }
+        String rank = switch (pack.selfStats().rank) {
+            case 1 -> "1st";
+            case 2 -> "2nd";
+            case 3 -> "3rd";
+            default -> pack.selfStats().rank + "th";
+        };
 
 
-        embedBuilder.addField("Your stats:", rank + " with " + pack.selfStats.total + " points", false);
+        embedBuilder.addField("Your stats:", rank + " with " + pack.selfStats().total + " points", false);
         return EmbedMessageUtils.buildStandar(embedBuilder);
 
     }

src/main/java/net/Broken/Tools/UserManager/UserUtils.java

@@ -1,15 +1,12 @@
 package net.Broken.Tools.UserManager;
 
-import net.Broken.DB.Entity.UserEntity;
-import net.Broken.DB.Repository.UserRepository;
-import net.Broken.Tools.UserManager.Exceptions.*;
 import org.apache.logging.log4j.LogManager;
 import org.apache.logging.log4j.Logger;
 
 public class UserUtils {
 
-    private static UserUtils INSTANCE = new UserUtils();
-    private Logger logger = LogManager.getLogger();
+    private static final UserUtils INSTANCE = new UserUtils();
+    private final Logger logger = LogManager.getLogger();
 
     /**
      * Private default constructor
@@ -26,21 +23,4 @@ public class UserUtils {
         return INSTANCE;
     }
 
-    /**
-     * return token's UserEntity
-     *
-     * @param userRepository User DB interface
-     * @param token          Received token
-     * @return User Entity
-     * @throws UnknownTokenException Can't find token on User DB
-     */
-    public UserEntity getUserWithApiToken(UserRepository userRepository, String token) throws UnknownTokenException {
-//        List<UserEntity> users = userRepository.findByApiToken(token);
-//        if (users.size() > 0) {
-//            return users.get(0);
-//        } else
-//            throw new UnknownTokenException();
-        return null;
-
-    }
 }

src/main/java/net/Broken/audio/GetVoiceChanels.java

@@ -1,47 +0,0 @@
-package net.Broken.audio;
-
-import net.Broken.DB.Entity.GuildPreferenceEntity;
-import net.Broken.Tools.SettingsUtils;
-import net.dv8tion.jda.api.Permission;
-import net.dv8tion.jda.api.entities.Guild;
-import net.dv8tion.jda.api.entities.GuildChannel;
-import net.dv8tion.jda.api.entities.Member;
-import net.dv8tion.jda.api.entities.VoiceChannel;
-import org.apache.logging.log4j.LogManager;
-import org.apache.logging.log4j.Logger;
-
-import java.util.ArrayList;
-import java.util.Comparator;
-import java.util.List;
-
-/**
- * Used to find general voice channels
- */
-public class GetVoiceChanels {
-    private static final Logger logger = LogManager.getLogger();
-
-    /**
-     * Search for all voice channel where the user can connect
-     *
-     * @param guild Current guild
-     * @return General Category
-     */
-    public static List<VoiceChannel> find(Guild guild, Member member) {
-
-        ArrayList<VoiceChannel> list = new ArrayList<>();
-        VoiceChannel afk = guild.getAfkChannel();
-        GuildPreferenceEntity pref = SettingsUtils.getInstance().getPreference(guild);
-        String autoVoice = pref.getAutoVoiceChannelID();
-        for (VoiceChannel channel : guild.getVoiceChannels()) {
-            if (channel != afk && !channel.getId().equals(autoVoice) && member.getPermissions(channel).contains(Permission.VOICE_CONNECT)) {
-                list.add(channel);
-            }
-        }
-
-        list.sort(Comparator.comparingInt(GuildChannel::getPositionRaw));
-
-        return list;
-
-    }
-
-}

src/main/java/net/Broken/audio/NotConnectedException.java

@@ -1,8 +0,0 @@
-package net.Broken.audio;
-
-
-/**
- * Not connected Exception (Voice Channel)
- */
-public class NotConnectedException extends Exception {
-}