I pretty much had to guess to find this, but to enable bloodbending for players this is not listed anywhere or in the plugin.yml -> bending.water.bloodbending

I believe that Project Korra should follow Semver versioning standards, as this would improve clarity into changes of the public API specifically to developers. I advise giving it a thorough read and consideration for future version increments. http://semver.org/

http://pastebin.com/Shjqx1Kx

http://pastebin.com/xMgd7tDy

My server has crashed a few times with this, I believe it is due to setting excessive velocities after doing some more research and looking at people who have had similar issues

to fix this I THINK this might be needed, not completely sure, I am testing it out right now on my server

in GeneralMethods.setVelocity

        if(velocity.getX() > 4){
            velocity.setX(4);
        }
        if(velocity.getX() < -4){
            velocity.setX(-4);
        }
        if(velocity.getY() > 4){
            velocity.setY(4);
        }
        if(velocity.getY() < -4){
            velocity.setY(-4);
        }
        if(velocity.getZ() > 4){
            velocity.setZ(4);
        }
        if(velocity.getZ() < -4){
            velocity.setZ(-4);
        }

http://pastebin.com/raw/1zrkDWTT

after this error occurs I dont see the rest of the normal output you would see such as "saving world" so this is a pretty big problem

I think this can be solved by not making https://github.com/ProjectKorra/ProjectKorra/blob/master/src/com/projectkorra/projectkorra/util/logging/LogFilter.java#L65 not be in a run task

http://pastebin.com/JnXMUHmy

I was making some administrative claims via GriefPrevention, and it seems ProjectKorra is having issues checking for build protection? It's also likely that it could've been checking MobArena as well.

https://gist.github.com/RoboMWM/3e3e9d64dd7a81db197c

1.8.0 BETA 6

You have forgotten to add the code and config options needed to disable the dynamic and auto sourcing features.

So here's an idea for your next PR: add them back :)

http://pastebin.com/AvLqGxBA

HealingWaters Ability does not work. Generates an "Unresolved compilation problem:"
http://pastebin.com/P3MLsyUD

according to a player on my server "You basically use the AirSpout and then use Tornado at the same time." and it lets you stay in fly mode

At least in the case of AutoCraft, if one is riding/piloting a ship and has illumination enabled, the torches remain on the ship as the ship is moved, allowing for infinite torches. I would like an option for a "compatibility" mode which would instead send a fake torch to nearby players (like tremorsense and the glowstone block to only the player) instead of placing a torch in the world.

1.8.0 BETA 6

http://pastebin.com/raw/f7SLBd7A

prob fixed by putting this https://github.com/ProjectKorra/ProjectKorra/blob/master/src/com/projectkorra/projectkorra/GeneralMethods.java#L1012

in front of this https://github.com/ProjectKorra/ProjectKorra/blob/master/src/com/projectkorra/projectkorra/GeneralMethods.java#L1007

As you can see, the project file is currently unusable by anyone but you.
Could you please fix that?

Running latest ProjectKorra (commit 7ed5bcd) and NCP build 1022.

[01:39:56] [Server thread/ERROR]: Could not pass event EntityDamageEvent to ProjectKorra v1.8.0
org.bukkit.event.EventException
    at org.bukkit.plugin.EventExecutor$1.execute(EventExecutor.java:46) ~[patched_1.10.2.jar:git-Paper-866]
    at co.aikar.timings.TimedEventExecutor.execute(TimedEventExecutor.java:78) ~[patched_1.10.2.jar:git-Paper-866]
    at org.bukkit.plugin.RegisteredListener.callEvent(RegisteredListener.java:62) ~[patched_1.10.2.jar:git-Paper-866]
    at org.bukkit.plugin.SimplePluginManager.fireEvent(SimplePluginManager.java:517) [patched_1.10.2.jar:git-Paper-866]
    at org.bukkit.plugin.SimplePluginManager.callEvent(SimplePluginManager.java:502) [patched_1.10.2.jar:git-Paper-866]
    at org.bukkit.craftbukkit.v1_10_R1.event.CraftEventFactory.callEvent(CraftEventFactory.java:89) [patched_1.10.2.jar:git-Paper-866]
    at org.bukkit.craftbukkit.v1_10_R1.event.CraftEventFactory.callEntityDamageEvent(CraftEventFactory.java:572) [patched_1.10.2.jar:git-Paper-866]
    at org.bukkit.craftbukkit.v1_10_R1.event.CraftEventFactory.handleEntityDamageEvent(CraftEventFactory.java:558) [patched_1.10.2.jar:git-Paper-866]
    at org.bukkit.craftbukkit.v1_10_R1.event.CraftEventFactory.handleLivingEntityDamageEvent(CraftEventFactory.java:604) [patched_1.10.2.jar:git-Paper-866]
    at net.minecraft.server.v1_10_R1.EntityLiving.damageEntity0(EntityLiving.java:1258) [patched_1.10.2.jar:git-Paper-866]
    at net.minecraft.server.v1_10_R1.EntityLiving.damageEntity(EntityLiving.java:850) [patched_1.10.2.jar:git-Paper-866]
    at net.minecraft.server.v1_10_R1.EntityMonster.damageEntity(EntityMonster.java:48) [patched_1.10.2.jar:git-Paper-866]
    at net.minecraft.server.v1_10_R1.Entity.U(Entity.java:393) [patched_1.10.2.jar:git-Paper-866]
    at net.minecraft.server.v1_10_R1.EntityLiving.U(EntityLiving.java:185) [patched_1.10.2.jar:git-Paper-866]
    at net.minecraft.server.v1_10_R1.EntityInsentient.U(EntityInsentient.java:181) [patched_1.10.2.jar:git-Paper-866]
    at net.minecraft.server.v1_10_R1.Entity.m(Entity.java:322) [patched_1.10.2.jar:git-Paper-866]
    at net.minecraft.server.v1_10_R1.EntityLiving.m(EntityLiving.java:1768) [patched_1.10.2.jar:git-Paper-866]
    at net.minecraft.server.v1_10_R1.EntityInsentient.m(EntityInsentient.java:241) [patched_1.10.2.jar:git-Paper-866]
    at net.minecraft.server.v1_10_R1.EntityMonster.m(EntityMonster.java:32) [patched_1.10.2.jar:git-Paper-866]
    at net.minecraft.server.v1_10_R1.World.entityJoinedWorld(World.java:1738) [patched_1.10.2.jar:git-Paper-866]
    at net.minecraft.server.v1_10_R1.World.g(World.java:1713) [patched_1.10.2.jar:git-Paper-866]
    at net.minecraft.server.v1_10_R1.World.tickEntities(World.java:1530) [patched_1.10.2.jar:git-Paper-866]
    at net.minecraft.server.v1_10_R1.WorldServer.tickEntities(WorldServer.java:643) [patched_1.10.2.jar:git-Paper-866]
    at net.minecraft.server.v1_10_R1.MinecraftServer.D(MinecraftServer.java:867) [patched_1.10.2.jar:git-Paper-866]
    at net.minecraft.server.v1_10_R1.DedicatedServer.D(DedicatedServer.java:404) [patched_1.10.2.jar:git-Paper-866]
    at net.minecraft.server.v1_10_R1.MinecraftServer.C(MinecraftServer.java:733) [patched_1.10.2.jar:git-Paper-866]
    at net.minecraft.server.v1_10_R1.MinecraftServer.run(MinecraftServer.java:632) [patched_1.10.2.jar:git-Paper-866]
    at java.lang.Thread.run(Unknown Source) [?:1.8.0_91]
Caused by: java.lang.NullPointerException
    at fr.neatmonster.nocheatplus.hooks.NCPExemptionManager.exemptPermanently(NCPExemptionManager.java:132) ~[?:?]
    at com.projectkorra.projectkorra.util.DamageHandler.damageEntity(DamageHandler.java:44) ~[?:?]
    at com.projectkorra.projectkorra.util.DamageHandler.damageEntity(DamageHandler.java:73) ~[?:?]
    at com.projectkorra.projectkorra.firebending.FireDamageTimer.dealFlameDamage(FireDamageTimer.java:58) ~[?:?]
    at com.projectkorra.projectkorra.PKListener.onEntityDamageEvent(PKListener.java:421) ~[?:?]
    at com.destroystokyo.paper.event.executor.asm.generated.GeneratedEventExecutor1203.execute(Unknown Source) ~[?:?]
    at org.bukkit.plugin.EventExecutor$1.execute(EventExecutor.java:44) ~[patched_1.10.2.jar:git-Paper-866]
    ... 27 more

Wondering if there's any intent to accept PRs for 1.9, since sound enums changed.

http://pastebin.com/raw/s1q0cKRn

spammed console a bit

The following checks can be triggered when using shockwave, fireburst, or any AoE ability:

• entity out of line of sight (direction)
• attacking with too high a frequency (speed)
• improbable

1.8.0 BETA 6

http://pastebin.com/raw/pRL1mDiL

While I'm trying to figure out what's going on with another plugin, I found ProjectKorra to be interfering with one of its events with the aid of EventDebug and CancellationDetector:
https://gist.github.com/RoboMWM/3c69e1ed0083920e3e0f

So may I ask why this is here?

Code that's calling the explosion:
https://github.com/Di3mex/ExtraHardMode/blob/master/src/main/java/com/extrahardmode/task/CreateExplosionTask.java#L184

line of sight seems to be triggered for many attacks, especially ones that involve a delay like EarthSmash, EarthBlast, etc.

1.8.0 BETA 6

To reproduce:

1. Start flying via airspout
2. Have another player use tornado on you.

(I'm not sure if you have to be thrown to a height where airspout doesn't work.)

You will then no longer be using airSpout, but will still be in fly mode.

1.8.0 BETA 6

I thought most, if not all attacks were by default exempt from NCP's reach check. However, fireburst seems to trigger this check.

1.8.0 BETA 6

When an ability is bound to a slot, the player has no way of knowing. There should be a way to display which slot holds which ability.

Using suffocate triggers NCP's noswing check.

I would imagine any abilities that don't use a armswing (most that use shift) probably trigger this as well.

1.8.0 BETA 6

http://projectkorra.com/forum/threads/join-message-edits-and-ideas.5381

Per #687 (comment)

Please make this configurable and/or disable-able.
FYI, yes, I do attribute credit to ProjectKorra on my server - regardless, I really want a clean, minimal motd - not to mention that only other server owners would be the least bit interested in this information.

I should also mention that my server is not a bending-focused server. It's more of a "utility."

fd03d8d#diff-34d9e9a9bd1ed2b5ed340633a344ba79R1068

I will be making a PR this weekend if someone doesn't get to it before I do. But this decision is unacceptable. If every plugin was advertising their own website on join... well, I'm sure you get the picture.

Both of the chi-blocking stances never get deactivated when clicking with them. Furthermore the other stance is never disabled. This results in as many stance checking tasks running for a player as times they click, causing huge lag over time, as well as the unintended result of never being disabled.

Feel free to comment suggestions or opinions on the following, I will add them to the list if they are appropriate.

• Add README, LICENSE and CONTRIBUTORS files with sufficient information of the build process, description, Travis status, contribution instructions, license etc.
• Change from Ant to Gradle with Gradle Wrapper.
  • Add License Headers Gradle Plugin.
  • Add Checkstyle. (set to warning) (POSSIBLY)
  • Setup a Maven repository on the Project Korra website to host, source, javadocs and binary. (Which can be used on github releases too.)
• Follow Semver #543
• Rework documentation system.

as of this commit 5f0dace

noticed this http://pastebin.com/GD7ZE1LG

btw if you dont want me to report errors with the bleeding edge versions at all just let me know and Ill stop

someone used metalclips on me, and now I get kicked with "illegal move"

[02:50:16] [Server thread/INFO]: PhanaticD[/removed] logged in with entity id 28877 at ([bending]NaN, NaN, NaN)
[02:50:18] [Server thread/INFO]: PhanaticD lost connection: Illegal move.

At times, catapult will trigger NCP's passable check for the block beneath them, resulting in the player rubberbanding back into the block, which continues to trigger more passable checks.

1.8.0 BETA 6

Instead of disabling and enabling fly-mode entirely when player is attempting to fly past the maximum possible height, is it possible to instead toggle the player from flying and not flying instead? This would make it compatible with anticheat plugins like NCP, which is picking up violations when the player's fly mode is disabled when they attempt to go beyond max height of the spout.

Hello there, I was wondering if I can compile my own jar of the Core?

I want to make some changes to the base abilities (airbubble, waterbubble, and earthtunnel) for exclusive use in my server. My changes remove the sneak requirement for both bubbles, and revert back line 114 to "this.block.breakNaturally();"

If you prefer me not to make these changes, could you just compile a version for me the changes?

Thanks, and sorry for using the issues panel like this.

I was recently testing building a door which was openable only by airbendingen, when I noticed that buttons and levers are not affected by airblasts, even though the description details that they should be able to activate them.
It seems that in the code, the part for handling this has been commented out, for some reason not stated.

So either the description of the ability should be edited or the code fixed, so that these two match again.

(Jun-24|07:13:03) [SEVERE] Could not pass event PlayerToggleSneakEvent to ProjectKorra v1.7.0 BETA 10
org.bukkit.event.EventException
at org.bukkit.plugin.java.JavaPluginLoader$1.execute(JavaPluginLoader.java:310)
at org.bukkit.plugin.RegisteredListener.callEvent(RegisteredListener.java:62)
at org.bukkit.plugin.SimplePluginManager.fireEvent(SimplePluginManager.java:502)
at org.bukkit.plugin.SimplePluginManager.callEvent(SimplePluginManager.java:487)
at net.minecraft.server.v1_8_R3.PlayerConnection.a(PlayerConnection.java:1193)
at net.minecraft.server.v1_8_R3.PacketPlayInEntityAction.a(SourceFile:43)
at net.minecraft.server.v1_8_R3.PacketPlayInEntityAction.a(SourceFile:9)
at net.minecraft.server.v1_8_R3.PlayerConnectionUtils$1.run(SourceFile:13)
at java.util.concurrent.Executors$RunnableAdapter.call(Executors.java:511)
at java.util.concurrent.FutureTask.run(FutureTask.java:266)
at net.minecraft.server.v1_8_R3.SystemUtils.a(SystemUtils.java:19)
at net.minecraft.server.v1_8_R3.MinecraftServer.B(MinecraftServer.java:718)
at net.minecraft.server.v1_8_R3.DedicatedServer.B(DedicatedServer.java:367)
at net.minecraft.server.v1_8_R3.MinecraftServer.A(MinecraftServer.java:657)
at net.minecraft.server.v1_8_R3.MinecraftServer.run(MinecraftServer.java:560)
at java.lang.Thread.run(Thread.java:745)
Caused by: java.lang.NullPointerException
at com.projectkorra.ProjectKorra.PKListener.onPlayerSneak(PKListener.java:1037)
at sun.reflect.GeneratedMethodAccessor150.invoke(Unknown Source)
at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
at java.lang.reflect.Method.invoke(Method.java:497)
at org.bukkit.plugin.java.JavaPluginLoader$1.execute(JavaPluginLoader.java:306)
... 15 more

Periodical errors show in the server log as this Runnable executes: WhoCommand.java:57, because the provided URL http://www.projectkorra.com/internals/staff.html returns a 404 page, rather than the expected text file.

I'm do not know what the correct URL is, therefore am unable to submit a PR. Because it is causing me (and probably other servers) periodic console errors, I think it should be looked into.

Might be related to issue #328 - if PreciousStones is installed and integration is enabled in ProjectKorra, this gets spammed (as an earthbender at least):

https://gist.github.com/RoboMWM/da6a867b6fc011a504a4

Error disappears and abilities function as expected when integration is disabled in config.

1.8.0 BETA 6

Hey awesome plugin but i've got 2 worlds on my server at the moment, a normal one without bending and a free-for-all with bending, i use PermissionsEx for permission handling and i've given every rank

• -bending.* (Even tried - -bending.water for this problem). However if there are any waterbenders who leave the bending world, even though they are unable to bend in the normal world or use commands, they are still getting the message every night "You feel the strength of the rising moon empowering your waterbending." this is also the case for firebenders during the day.
  I was unable to find the editable message in the config. If this could be fixed that would be awesome :)

(Jun-24|07:09:44) [SEVERE] Could not pass event PlayerAnimationEvent to ProjectKorra v1.7.0 BETA 10
org.bukkit.event.EventException
at org.bukkit.plugin.java.JavaPluginLoader$1.execute(JavaPluginLoader.java:310)
at org.bukkit.plugin.RegisteredListener.callEvent(RegisteredListener.java:62)
at org.bukkit.plugin.SimplePluginManager.fireEvent(SimplePluginManager.java:502)
at org.bukkit.plugin.SimplePluginManager.callEvent(SimplePluginManager.java:487)
at net.minecraft.server.v1_8_R3.PlayerConnection.a(PlayerConnection.java:1178)
at net.minecraft.server.v1_8_R3.PacketPlayInArmAnimation.a(PacketPlayInArmAnimation.java:18)
at net.minecraft.server.v1_8_R3.PacketPlayInArmAnimation.a(PacketPlayInArmAnimation.java:1)
at net.minecraft.server.v1_8_R3.PlayerConnectionUtils$1.run(SourceFile:13)
at java.util.concurrent.Executors$RunnableAdapter.call(Executors.java:511)
at java.util.concurrent.FutureTask.run(FutureTask.java:266)
at net.minecraft.server.v1_8_R3.SystemUtils.a(SystemUtils.java:19)
at net.minecraft.server.v1_8_R3.MinecraftServer.B(MinecraftServer.java:718)
at net.minecraft.server.v1_8_R3.DedicatedServer.B(DedicatedServer.java:367)
at net.minecraft.server.v1_8_R3.MinecraftServer.A(MinecraftServer.java:657)
at net.minecraft.server.v1_8_R3.MinecraftServer.run(MinecraftServer.java:560)
at java.lang.Thread.run(Thread.java:745)
Caused by: java.lang.NullPointerException
at com.projectkorra.ProjectKorra.PKListener.onPlayerSwing(PKListener.java:1221)
at sun.reflect.GeneratedMethodAccessor27.invoke(Unknown Source)
at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
at java.lang.reflect.Method.invoke(Method.java:497)
at org.bukkit.plugin.java.JavaPluginLoader$1.execute(JavaPluginLoader.java:306)
... 15 more

Particle effect doesn't work on newly released Spigot 1.8.0-R0.1 making some binding non-working.
As a short/temporary correction, is it possible to specify in the conf file to not use particle effects?
For example, I see "Particles: smoke", is there a keyword, saying not to use any particle?

[13:41:13 WARN]: [ProjectKorra] Task #35 for ProjectKorra v1.5.0 BETA 23 generated an exception
com.projectkorra.ProjectKorra.Utilities.ParticleEffect$PacketInstantiationException: Packet instantiation failed
    at com.projectkorra.ProjectKorra.Utilities.ParticleEffect.instantiatePacket(ParticleEffect.java:293) ~[?:?]
    at com.projectkorra.ProjectKorra.Utilities.ParticleEffect.display(ParticleEffect.java:412) ~[?:?]
    at com.projectkorra.ProjectKorra.Utilities.ParticleEffect.display(ParticleEffect.java:428) ~[?:?]
    at com.projectkorra.ProjectKorra.Methods.playAirbendingParticles(Methods.java:1760) ~[?:?]
    at com.projectkorra.ProjectKorra.Methods.playAirbendingParticles(Methods.java:1738) ~[?:?]
    at com.projectkorra.ProjectKorra.airbending.AirBlast.advanceLocation(AirBlast.java:210) ~[?:?]
    at com.projectkorra.ProjectKorra.airbending.AirBlast.progress(AirBlast.java:203) ~[?:?]
    at com.projectkorra.ProjectKorra.airbending.AirBlast.progressAll(AirBlast.java:289) ~[?:?]
    at com.projectkorra.ProjectKorra.airbending.AirbendingManager.run(AirbendingManager.java:17) ~[?:?]
    at org.bukkit.craftbukkit.v1_8_R1.scheduler.CraftTask.run(CraftTask.java:71) ~[spigot-1.8-R0.1-SNAPSHOT.jar:git-Spigot-"f15f0fc"]
    at org.bukkit.craftbukkit.v1_8_R1.scheduler.CraftScheduler.mainThreadHeartbeat(CraftScheduler.java:350) [spigot-1.8-R0.1-SNAPSHOT.jar:git-Spigot-"f15f0fc"]
    at net.minecraft.server.v1_8_R1.MinecraftServer.z(MinecraftServer.java:697) [spigot-1.8-R0.1-SNAPSHOT.jar:git-Spigot-"f15f0fc"]
    at net.minecraft.server.v1_8_R1.DedicatedServer.z(DedicatedServer.java:316) [spigot-1.8-R0.1-SNAPSHOT.jar:git-Spigot-"f15f0fc"]
    at net.minecraft.server.v1_8_R1.MinecraftServer.y(MinecraftServer.java:627) [spigot-1.8-R0.1-SNAPSHOT.jar:git-Spigot-"f15f0fc"]
    at net.minecraft.server.v1_8_R1.MinecraftServer.run(MinecraftServer.java:530) [spigot-1.8-R0.1-SNAPSHOT.jar:git-Spigot-"f15f0fc"]
    at java.lang.Thread.run(Thread.java:745) [?:1.7.0_65]
Caused by: java.lang.NullPointerException

I've used /bending remove on myself and other players, but it seems that this is not preserved between server restarts. I do not see any log messages in console, and in-game the player does indeed lose the ability to bend the removed element (or at all) - but after a restart, the removed ability is re-instated.

Using /bending choose and /bending add seem to save properly.

1.8.0 BETA 6 (using default storage, SQLite)

https://gyazo.com/9f0e030b8325a210443d7d37e93f8d67

http://pastebin.com/raw/Kcif5Y3Q

So I'm not entirely sure where this is modified in code, since searching for setPlayerListName, scoreboard, or team yields no results - but it doesn't seem to be possible to disable the prefixing of player names (at least in the tablist) via the config.

So even when I have chat set to false in PK's config, it seems like it's still setting the prefix. Essentials wins out (could also be setting displayName via a repeated task) in chat, but in the case where I'm setting the player's list name based on their display name, it seems to be grabbing PK's format, as if chat were enabled.

https://github.com/MLG-Fortress/MountainDewritoes/blob/master/src/main/java/me/robomwm/MountainDewritoes/SimpleClansListener.java#L91

If an airbender is sprinting while teleported (such as entering a Multiverse-Portal), the airbender will have the speed and jump boost effects permanently until they sprint again.