Fixes

* Fixed debug throwing null exceptions in Titles
* Minor Fixes

Signed-off-by: fran <[email protected]>

Author: Im-Fran

dependency-reduced-pom.xml

@@ -4,7 +4,7 @@
   <groupId>xyz.theprogramsrc</groupId>
   <artifactId>SuperCoreAPI</artifactId>
   <name>SuperCoreAPI</name>
-  <version>4.8.1</version>
+  <version>4.8.2</version>
   <build>
     <sourceDirectory>src/main/java</sourceDirectory>
     <defaultGoal>clean package</defaultGoal>

pom.xml

@@ -6,7 +6,7 @@
 
     <groupId>xyz.theprogramsrc</groupId>
     <artifactId>SuperCoreAPI</artifactId>
-    <version>4.8.2</version>
+    <version>4.8.3</version>
     <packaging>jar</packaging>
 
     <name>SuperCoreAPI</name>

src/main/java/xyz/theprogramsrc/supercoreapi/spigot/dialog/Dialog.java

@@ -32,7 +32,6 @@ public abstract class Dialog extends SpigotModule {
      */
     public Dialog(Player player){
         super(false);
-        this.debug("Registering and opening dialog to '" + player.getName() + "'");
         this.player = player;
         this.placeholders = new HashMap<>();
         this.openDialog();
@@ -50,13 +49,14 @@ public void openDialog(){
             if(this.canClose()){
                 this.getSuperUtils().sendMessage(this.getPlayer(), Base.DIALOG_HOW_TO_CLOSE.toString());
             }
-        });
+            this.debug("Opening dialog with title '" + this.getTitle() + "&r'");
 
-        if(this.task == null){
-            this.task = this.getSpigotTasks().runRepeatingTask(0L, 1L, this::sendTitleAndActionbar);
-        }else{
-            this.task.start();
-        }
+            if(this.task == null){
+                this.task = this.getSpigotTasks().runRepeatingTask(1L, 1L, this::sendTitleAndActionbar);
+            }else{
+                this.task.start();
+            }
+        });
     }
 
     private void sendTitleAndActionbar() {
@@ -69,7 +69,7 @@ private void sendTitleAndActionbar() {
      */
     public void close(){
         this.getSpigotTasks().runTask(()->{
-            this.task.stop();
+            if(this.task != null) this.task.stop();
             HandlerList.unregisterAll(this);
             Title.clearTitle(this.getPlayer());
             Actionbar.clearActionbar(this.getPlayer());

src/main/java/xyz/theprogramsrc/supercoreapi/spigot/guis/GUI.java

@@ -43,10 +43,10 @@ public abstract class GUI extends SpigotModule {
      */
     public GUI(Player player){
         super(false);
-        this.debug("Registering GUI with title '" + this.getTitle() + "&r'");
         this.manuallyClosed = false;
         this.player = player;
         this.buttons = new LinkedHashMap<>();
+        this.debug("Registering GUI with title '" + this.getTitle() + "&r'");
         this.task = this.getSpigotTasks().runRepeatingTask(1L, 1L, ()->{
             if(this.inv != null){
                 if(!this.previousTitle.equals(this.getTitle()) || this.getRows().getSize() != this.previousSize){