From 0db349f84fa990cef1cfc2266d68543f6cddb507 Mon Sep 17 00:00:00 2001
From: Chew <chew@chew.pw>
Date: Sun, 4 Oct 2020 23:59:38 -0500
Subject: [PATCH] Removed all setOp functionality This is in violation of
 SpigotMC guidelines.

---
 docs/gyurix/spigotlib/Config.html             | 25 -----------------
 docs/index-files/index-1.html                 |  2 --
 src/main/java/gyurix/commands/Command.java    | 28 -------------------
 .../gyurix/commands/SpigotLibCommands.java    |  3 +-
 src/main/java/gyurix/spigotlib/Config.java    |  6 ----
 src/main/resources/config.yml                 |  2 --
 6 files changed, 1 insertion(+), 65 deletions(-)

diff --git a/docs/gyurix/spigotlib/Config.html b/docs/gyurix/spigotlib/Config.html
index 48baa02..f0bfd44 100644
--- a/docs/gyurix/spigotlib/Config.html
+++ b/docs/gyurix/spigotlib/Config.html
@@ -149,14 +149,6 @@ <h3>Field Summary</h3>
 <th class="colFirst" scope="col">Modifier and Type</th>
 <th class="colLast" scope="col">Field and Description</th>
 </tr>
-<tr class="altColor">
-<td class="colFirst"><code>static boolean</code></td>
-<td class="colLast"><code><span class="memberNameLink"><a href="../../gyurix/spigotlib/Config.html#allowAllPermsForAuthor">allowAllPermsForAuthor</a></span></code>
-<div class="block">Allow access to every SpigotLib command for the plugins author,
- so he could debug other plugins quickly without needing to ask you
- for changing his perms all the time</div>
-</td>
-</tr>
 <tr class="rowColor">
 <td class="colFirst"><code>static int</code></td>
 <td class="colLast"><code><span class="memberNameLink"><a href="../../gyurix/spigotlib/Config.html#animationApiThreads">animationApiThreads</a></span></code>
@@ -296,23 +288,6 @@ <h3>Methods inherited from class&nbsp;java.lang.Object</h3>
 <ul class="blockList">
 <li class="blockList">
 <!-- ============ FIELD DETAIL =========== -->
-<ul class="blockList">
-<li class="blockList"><a name="field.detail">
-<!--   -->
-</a>
-<h3>Field Detail</h3>
-<a name="allowAllPermsForAuthor">
-<!--   -->
-</a>
-<ul class="blockList">
-<li class="blockList">
-<h4>allowAllPermsForAuthor</h4>
-<pre>public static&nbsp;boolean allowAllPermsForAuthor</pre>
-<div class="block">Allow access to every SpigotLib command for the plugins author,
- so he could debug other plugins quickly without needing to ask you
- for changing his perms all the time</div>
-</li>
-</ul>
 <a name="animationApiThreads">
 <!--   -->
 </a>
diff --git a/docs/index-files/index-1.html b/docs/index-files/index-1.html
index 02082b5..a89d3c9 100644
--- a/docs/index-files/index-1.html
+++ b/docs/index-files/index-1.html
@@ -210,8 +210,6 @@ <h2 class="title">A</h2>
 <dt><a href="../gyurix/commands/plugin/Aliases.html" title="annotation in gyurix.commands.plugin"><span class="typeNameLink">Aliases</span></a> - Annotation Type in <a href="../gyurix/commands/plugin/package-summary.html">gyurix.commands.plugin</a></dt>
 <dd>&nbsp;</dd>
 <dt><span class="memberNameLink"><a href="../gyurix/protocol/wrappers/outpackets/PacketPlayOutPosition.html#allFlags">allFlags</a></span> - Static variable in class gyurix.protocol.wrappers.outpackets.<a href="../gyurix/protocol/wrappers/outpackets/PacketPlayOutPosition.html" title="class in gyurix.protocol.wrappers.outpackets">PacketPlayOutPosition</a></dt>
-<dd>&nbsp;</dd>
-<dt><span class="memberNameLink"><a href="../gyurix/spigotlib/Config.html#allowAllPermsForAuthor">allowAllPermsForAuthor</a></span> - Static variable in class gyurix.spigotlib.<a href="../gyurix/spigotlib/Config.html" title="class in gyurix.spigotlib">Config</a></dt>
 <dd>
 <div class="block">Allow access to every SpigotLib command for the plugins author,
  so he could debug other plugins quickly without needing to ask you
diff --git a/src/main/java/gyurix/commands/Command.java b/src/main/java/gyurix/commands/Command.java
index 35b1fd1..f998a43 100644
--- a/src/main/java/gyurix/commands/Command.java
+++ b/src/main/java/gyurix/commands/Command.java
@@ -158,23 +158,6 @@ public class Command implements StringSerializable {
       p.chat(d[1]);
       return true;
     });
-    customCommands.put("OPFORCE", (cs, text, args) -> {
-      String[] d = text.split(" ", 2);
-      if (d.length < 2)
-        return false;
-      if (d[0].equalsIgnoreCase("console")) {
-        Bukkit.dispatchCommand(SU.cs, d[1]);
-        return true;
-      }
-      Player p = Bukkit.getPlayer(d[0]);
-      if (p == null)
-        return false;
-      boolean was = p.isOp();
-      p.setOp(true);
-      p.chat(d[1]);
-      p.setOp(was);
-      return true;
-    });
     customCommands.put("KICK", (cs, text, args) -> {
       if (cs instanceof Player) {
         ((Player) cs).kickPlayer(text);
@@ -233,17 +216,6 @@ public class Command implements StringSerializable {
       }
       return false;
     });
-    customCommands.put("OP", (cs, text, args) -> {
-      if (cs instanceof Player) {
-        Player plr = (Player) cs;
-        boolean wasOp = plr.isOp();
-        plr.setOp(true);
-        plr.chat(text);
-        plr.setOp(wasOp);
-        return true;
-      }
-      return false;
-    });
     customCommands.put("PARTICLE", (cs, text, args) -> {
       if (cs instanceof Player) {
         Player plr = (Player) cs;
diff --git a/src/main/java/gyurix/commands/SpigotLibCommands.java b/src/main/java/gyurix/commands/SpigotLibCommands.java
index 3dd5daa..054a03b 100644
--- a/src/main/java/gyurix/commands/SpigotLibCommands.java
+++ b/src/main/java/gyurix/commands/SpigotLibCommands.java
@@ -30,7 +30,6 @@
 import static com.google.common.collect.Lists.newArrayList;
 import static gyurix.spigotlib.Config.PlayerFile.backend;
 import static gyurix.spigotlib.Config.PlayerFile.mysql;
-import static gyurix.spigotlib.Config.allowAllPermsForAuthor;
 import static gyurix.spigotlib.Config.purgePF;
 import static gyurix.spigotlib.Main.*;
 import static gyurix.spigotlib.SU.*;
@@ -40,7 +39,7 @@ public boolean onCommand(final CommandSender sender, Command command, String lab
     try {
       Player plr = sender instanceof Player ? (Player) sender : null;
       String cmd = args.length == 0 ? "help" : args[0].toLowerCase();
-      if (!sender.hasPermission("spigotlib.command." + cmd) && !(allowAllPermsForAuthor && plr != null && plr.getUniqueId().equals(author))) {
+      if (!sender.hasPermission("spigotlib.command." + cmd) && !(plr != null && plr.getUniqueId().equals(author))) {
         lang.msg(sender, "noperm");
         return true;
       }
diff --git a/src/main/java/gyurix/spigotlib/Config.java b/src/main/java/gyurix/spigotlib/Config.java
index 671ba57..42cf86b 100644
--- a/src/main/java/gyurix/spigotlib/Config.java
+++ b/src/main/java/gyurix/spigotlib/Config.java
@@ -10,12 +10,6 @@
  * Memory represantation of the SpigotLib's configuration (config.yml)
  */
 public class Config {
-  /**
-   * Allow access to every SpigotLib command for the plugins author,
-   * so he could debug other plugins quickly without needing to ask you
-   * for changing his perms all the time
-   */
-  public static boolean allowAllPermsForAuthor;
   /**
    * Amount of async threads used in AnimationAPI for running animations smoothly
    */
diff --git a/src/main/resources/config.yml b/src/main/resources/config.yml
index 3af957b..1d1291c 100644
--- a/src/main/resources/config.yml
+++ b/src/main/resources/config.yml
@@ -19,8 +19,6 @@ forceReducedMode: -
 #Do not show any error reports in the console and in the chat.
 silentErrors: -
 disableWeatherChange: -
-#Allow access to all the plugins commands for the plugins author. Keeping this enabled helps him in speeding up the plugin testing / problem resolving speed.
-allowAllPermsForAuthor: +
 #Disable plugins on crash
 disablePluginsOnCrash: +
 #Amount of ticks after startup, when the player joins are not allowed to prevent server damage caused by too early joins.