From 663c45ec3cb1fb02875c5d46806616f1d5f99113 Mon Sep 17 00:00:00 2001
From: NeumimTo <NeumimTo@users.noreply.github.com>
Date: Sat, 26 Jun 2021 08:30:23 +0200
Subject: [PATCH] add back permission skills

---
 .run/RunServer.run.xml                        |  4 +-
 .../rpg/api/skills/types/PermissionSkill.java | 53 +++++++++++++++++--
 .../configuration/SkillTreeLoaderImpl.java    |  3 --
 .../rpg/common/skills/SkillConfigLoaders.java |  3 ++
 .../nt-rpg/defaults/skilltrees/magetree.conf  | 12 +++++
 5 files changed, 65 insertions(+), 10 deletions(-)

diff --git a/.run/RunServer.run.xml b/.run/RunServer.run.xml
index 340cdbaa4..1cc6b2356 100644
--- a/.run/RunServer.run.xml
+++ b/.run/RunServer.run.xml
@@ -5,8 +5,6 @@
     <option name="WORKING_DIRECTORY" value="$PROJECT_DIR$/server-spigot" />
     <option name="ALTERNATIVE_JRE_PATH_ENABLED" value="true" />
     <option name="ALTERNATIVE_JRE_PATH" value="1.8" />
-    <method v="2">
-      <option name="RunConfigurationTask" enabled="true" run_configuration_name="NT-RPG [build]" run_configuration_type="GradleRunConfiguration" />
-    </method>
+    <method v="2" />
   </configuration>
 </component>
\ No newline at end of file
diff --git a/API/src/main/java/cz/neumimto/rpg/api/skills/types/PermissionSkill.java b/API/src/main/java/cz/neumimto/rpg/api/skills/types/PermissionSkill.java
index 1d007fedf..c41f20979 100644
--- a/API/src/main/java/cz/neumimto/rpg/api/skills/types/PermissionSkill.java
+++ b/API/src/main/java/cz/neumimto/rpg/api/skills/types/PermissionSkill.java
@@ -16,13 +16,26 @@
 
 package cz.neumimto.rpg.api.skills.types;
 
+import com.typesafe.config.Config;
 import cz.neumimto.rpg.api.entity.players.IActiveCharacter;
+import cz.neumimto.rpg.api.permissions.PermissionService;
 import cz.neumimto.rpg.api.skills.PlayerSkillContext;
+import cz.neumimto.rpg.api.skills.SkillData;
 import cz.neumimto.rpg.api.skills.SkillExecutionType;
 import cz.neumimto.rpg.api.skills.SkillResult;
+import cz.neumimto.rpg.api.skills.tree.SkillTree;
+import cz.neumimto.rpg.api.skills.utils.SkillLoadingErrors;
+
+import javax.inject.Inject;
+import java.util.HashSet;
+import java.util.List;
+import java.util.Set;
 
 public class PermissionSkill extends AbstractSkill<IActiveCharacter> {
 
+    @Inject
+    private PermissionService permissionService;
+
     @Override
     public SkillResult onPreUse(IActiveCharacter character, PlayerSkillContext esi) {
         return SkillResult.CANCELLED;
@@ -36,16 +49,48 @@ public SkillExecutionType getSkillExecutionType() {
     @Override
     public void skillLearn(IActiveCharacter IActiveCharacter, PlayerSkillContext context) {
         super.skillLearn(IActiveCharacter, context);
+        PermissionData permissionData = (PermissionData) context.getSkillData();
+        if (permissionData.permissions != null) {
+            permissionService.addPermissions(IActiveCharacter, permissionData.permissions);
+        }
     }
 
     @Override
-    public void skillUpgrade(IActiveCharacter IActiveCharacter, int level, PlayerSkillContext context) {
-        super.skillUpgrade(IActiveCharacter, level, context);
+    public void skillRefund(IActiveCharacter IActiveCharacter, PlayerSkillContext context) {
+        super.skillRefund(IActiveCharacter, context);
+        PermissionData permissionData = (PermissionData) context.getSkillData();
+        if (permissionData.permissions != null) {
+            permissionService.removePermissions(IActiveCharacter, permissionData.permissions);
+        }
     }
 
     @Override
-    public void skillRefund(IActiveCharacter IActiveCharacter, PlayerSkillContext context) {
-        super.skillRefund(IActiveCharacter, context);
+    public <T extends SkillData> void loadSkillData(T skillData, SkillTree context, SkillLoadingErrors errors, Config c) {
+        PermissionData permissionData = (PermissionData) skillData;
+        List<String> permissions = c.getStringList("Permissions");
+        if (permissions != null) {
+            permissionData.permissions = new HashSet<>(permissions);
+        }
     }
 
+    @Override
+    public SkillData constructSkillData() {
+        return new PermissionData(getId());
+    }
+
+    public class PermissionData extends SkillData {
+        private Set<String> permissions;
+
+        public PermissionData(String skill) {
+            super(skill);
+        }
+
+        public Set<String> getPermissions() {
+            return permissions;
+        }
+
+        public void setPermissions(Set<String> permissions) {
+            this.permissions = permissions;
+        }
+    }
 }
diff --git a/Common/src/main/java/cz/neumimto/rpg/common/configuration/SkillTreeLoaderImpl.java b/Common/src/main/java/cz/neumimto/rpg/common/configuration/SkillTreeLoaderImpl.java
index dd1c34bdc..88a161fe9 100644
--- a/Common/src/main/java/cz/neumimto/rpg/common/configuration/SkillTreeLoaderImpl.java
+++ b/Common/src/main/java/cz/neumimto/rpg/common/configuration/SkillTreeLoaderImpl.java
@@ -23,7 +23,6 @@
 import cz.neumimto.rpg.api.Rpg;
 import cz.neumimto.rpg.api.classes.ClassService;
 import cz.neumimto.rpg.api.configuration.SkillTreeDao;
-import cz.neumimto.rpg.api.effects.model.mappers.SingleValueModelMapper;
 import cz.neumimto.rpg.api.gui.ISkillTreeInterfaceModel;
 import cz.neumimto.rpg.api.localization.LocalizationService;
 import cz.neumimto.rpg.api.logging.Log;
@@ -40,7 +39,6 @@
 import cz.neumimto.rpg.common.skills.SkillConfigLoader;
 import cz.neumimto.rpg.common.skills.SkillConfigLoaders;
 import cz.neumimto.rpg.common.skills.SkillExecutor;
-import org.apache.commons.lang3.math.NumberUtils;
 
 import javax.inject.Inject;
 import javax.inject.Singleton;
@@ -49,7 +47,6 @@
 import java.nio.file.Files;
 import java.nio.file.Path;
 import java.nio.file.Paths;
-import java.util.Optional;
 import java.util.*;
 
 import static cz.neumimto.rpg.api.logging.Log.*;
diff --git a/Common/src/main/java/cz/neumimto/rpg/common/skills/SkillConfigLoaders.java b/Common/src/main/java/cz/neumimto/rpg/common/skills/SkillConfigLoaders.java
index 4d6473d7c..c892fa819 100644
--- a/Common/src/main/java/cz/neumimto/rpg/common/skills/SkillConfigLoaders.java
+++ b/Common/src/main/java/cz/neumimto/rpg/common/skills/SkillConfigLoaders.java
@@ -3,6 +3,7 @@
 import cz.neumimto.rpg.api.skills.*;
 import cz.neumimto.rpg.api.skills.tree.SkillTreeSpecialization;
 import cz.neumimto.rpg.api.skills.types.CharacterAttributeSkill;
+import cz.neumimto.rpg.api.skills.types.PermissionSkill;
 import cz.neumimto.rpg.common.skills.types.ItemAccessSkill;
 
 import java.util.HashMap;
@@ -19,6 +20,7 @@ public class SkillConfigLoaders {
     static SkillConfigLoader WRAPPER = new SkillConfigLoader("wrapper", WrappedSkill.class);
     static SkillConfigLoader REPEATING = new SkillConfigLoader("repeating", RepeatingSkill.class);
     static SkillConfigLoader UPGRADE = new SkillConfigLoader("upgrade", SkillUpgrade.class);
+    static SkillConfigLoader PERMISSION = new SkillConfigLoader("permission", PermissionSkill.class);
 
     private static Map<String, SkillConfigLoader> internalCache = new HashMap<>();
 
@@ -32,6 +34,7 @@ public class SkillConfigLoaders {
         register(WRAPPER);
         register(REPEATING);
         register(UPGRADE);
+        register(PERMISSION);
     }
 
     public static Optional<SkillConfigLoader> getById(String id) {
diff --git a/Common/src/main/resources/assets/nt-rpg/defaults/skilltrees/magetree.conf b/Common/src/main/resources/assets/nt-rpg/defaults/skilltrees/magetree.conf
index 8982839b4..8e5730946 100644
--- a/Common/src/main/resources/assets/nt-rpg/defaults/skilltrees/magetree.conf
+++ b/Common/src/main/resources/assets/nt-rpg/defaults/skilltrees/magetree.conf
@@ -403,4 +403,16 @@ Skills: [
         manacost: 10
     }
   }
+  {
+      SkillId: "myexample:giveperm01"
+      Name: PermExample
+      MinPlayerLevel: 100
+      MaxSkillLevel: 1
+      SkillTreeId: 16
+      Icon: "minecraft:web"
+      ModelId: 999
+      Permissions: [
+          "some.permission.xxx.yyy"
+      ]
+    }
 ]
\ No newline at end of file