Merge pull request #18 from fixrtm/permission-log

anatawa12 · web-flow · commit cbfac4660498 · 2022-11-21T07:34:05.000+09:00
fix: Permission missing message is shown if it has permission
diff --git a/src/main/java/com/anatawa12/pluginPermsForNgt/NGTPermissionManagerBukkit.java b/src/main/java/com/anatawa12/pluginPermsForNgt/NGTPermissionManagerBukkit.java
@@ -108,11 +108,11 @@ public void removePermission(ICommandSender player, String targetPlayerName, Str
 
     @Vendors({Vendor.NGT1710, Vendor.NGT1122})
     public boolean hasPermission(ICommandSender player, String category) {
-        if (!hasPermissionInternal(player, category)) {
-            return false;
+        if (hasPermissionInternal(player, category)) {
+            return true;
         }
         NGTLog.sendChatMessage(player, "You don't have "  + PERM_PREFIX + category + " permission.");
-        return true;
+        return false;
     }
 
     @Vendors(Vendor.PluginPermsForNgt)

Original file line number	Diff line number	Diff line change
`@@ -108,11 +108,11 @@ public void removePermission(ICommandSender player, String targetPlayerName, Str`
`108`	`108`
`109`	`109`	`@Vendors({Vendor.NGT1710, Vendor.NGT1122})`
`110`	`110`	`public boolean hasPermission(ICommandSender player, String category) {`
`111`		`- if (!hasPermissionInternal(player, category)) {`
`112`		`- return false;`
	`111`	`+ if (hasPermissionInternal(player, category)) {`
	`112`	`+ return true;`
`113`	`113`	`}`
`114`	`114`	`NGTLog.sendChatMessage(player, "You don't have " + PERM_PREFIX + category + " permission.");`
`115`		`- return true;`
	`115`	`+ return false;`
`116`	`116`	`}`
`117`	`117`
`118`	`118`	`@Vendors(Vendor.PluginPermsForNgt)`