Started by an SCM change345df4cdce38ca8641eb3add62c2ab973f9ad0e253adf4cdce38ca8641eb3add62c2ab973f9ad0e253arefs/remotes/origin/masterdf4cdce38ca8641eb3add62c2ab973f9ad0e253adf4cdce38ca8641eb3add62c2ab973f9ad0e253arefs/remotes/origin/masterdf4cdce38ca8641eb3add62c2ab973f9ad0e253adf4cdce38ca8641eb3add62c2ab973f9ad0e253arefs/remotes/origin/masterhttps://github.com/mcMMO-Dev/mcMMO.githttps://popicraft.net/jenkins/job/mcMMO/345/artifacthttps://popicraft.net/jenkins/job/mcMMO/changeshttps://popicraft.net/jenkins/job/mcMMO/345/https://popicraft.net/jenkins/job/mcMMO/345/testReportmcMMO.jarmcMMO.jartarget/mcMMO.jaroriginal-mcMMO.jaroriginal-mcMMO.jartarget/original-mcMMO.jarfalse#345170624135282mcMMO #345345falsefalse345219SUCCESS1783754351147https://popicraft.net/jenkins/job/mcMMO/345/Buildersrc/main/java/com/gmail/nossr50/listeners/BlockListener.javasrc/test/java/com/gmail/nossr50/listeners/BlockListenerBonusDropsTest.javaChangelog.txt5a1a61caa94dc9aa4446a955e0e8c904efc1e76a1783748147000https://popicraft.net/jenkins/user/nossr50nossr50nossr50@gmail.comFix Herbalism double drops not working on beetroots At BlockDropItemEvent time the broken block already reads as AIR, so deciding the beetroot drop tolerance from the live block type never matched BEETROOTS and the two legitimate beetroot drop materials tripped the tile-entity dupe heuristic. Read the type from the event's captured pre-break BlockState instead. 2026-07-10 22:35:47 -07005a1a61caa94dc9aa4446a955e0e8c904efc1e76aFix Herbalism double drops not working on beetrootseditsrc/main/java/com/gmail/nossr50/listeners/BlockListener.javaaddsrc/test/java/com/gmail/nossr50/listeners/BlockListenerBonusDropsTest.javaeditChangelog.txtsrc/test/java/com/gmail/nossr50/listeners/EntityListenerTest.javaChangelog.txtsrc/main/java/com/gmail/nossr50/listeners/EntityListener.java808bc5577be1a2c70328742570c7c04c1d9e66c31783748411000https://popicraft.net/jenkins/user/nossr50nossr50nossr50@gmail.comFix error when a pet owned by a non-player deals damage Tameable owners resolved in the damage handler are AnimalTamers and not necessarily players; the unchecked OfflinePlayer cast crashed the handler for plugin-provided owners. The tame handler fixed the same bug class earlier; this brings the damage path in line. 2026-07-10 22:40:11 -0700808bc5577be1a2c70328742570c7c04c1d9e66c3Fix error when a pet owned by a non-player deals damageeditsrc/main/java/com/gmail/nossr50/listeners/EntityListener.javaeditChangelog.txteditsrc/test/java/com/gmail/nossr50/listeners/EntityListenerTest.javasrc/main/java/com/gmail/nossr50/listeners/InventoryListener.javaChangelog.txtsrc/test/java/com/gmail/nossr50/listeners/InventoryListenerClickTest.javacc7442bbcc595472e468dbcaf2d6552609fe73021783748598000https://popicraft.net/jenkins/user/nossr50nossr50nossr50@gmail.comFix error when non-players click or drag in furnace and brewing inventories InventoryClickEvent and InventoryDragEvent report the viewer as a HumanEntity, and plugins can open container inventories for human-shaped entities that are not players. The click and drag handlers cast the viewer to Player unchecked and crashed on such viewers; they now ignore non-player viewers. 2026-07-10 22:43:18 -0700cc7442bbcc595472e468dbcaf2d6552609fe7302Fix error when non-players click or drag in furnace and brewing inventoriesaddsrc/test/java/com/gmail/nossr50/listeners/InventoryListenerClickTest.javaeditsrc/main/java/com/gmail/nossr50/listeners/InventoryListener.javaeditChangelog.txtsrc/main/java/com/gmail/nossr50/listeners/PlayerListener.javasrc/test/java/com/gmail/nossr50/listeners/PlayerListenerCommandAliasTest.javaChangelog.txt7f3f5b41bc61ace7361929bbc338d45d725b8b601783748841000https://popicraft.net/jenkins/user/nossr50nossr50nossr50@gmail.comFix localized skill commands rewriting arguments that repeat the skill name The alias rewrite replaced every occurrence of the alias text in the command message, so arguments that repeated the alias (player names, search terms) were rewritten along with the command token. Only the leading command token is rewritten now. 2026-07-10 22:47:21 -07007f3f5b41bc61ace7361929bbc338d45d725b8b60Fix localized skill commands rewriting arguments that repeat the skill nameeditsrc/main/java/com/gmail/nossr50/listeners/PlayerListener.javaeditsrc/test/java/com/gmail/nossr50/listeners/PlayerListenerCommandAliasTest.javaeditChangelog.txtsrc/main/java/com/gmail/nossr50/listeners/PlayerListener.javasrc/test/java/com/gmail/nossr50/listeners/PlayerListenerFishingTest.javaChangelog.txt4d2b088f4a4efe8316c2d7718378ede358d040e41783748881000https://popicraft.net/jenkins/user/nossr50nossr50nossr50@gmail.comFix error when a plugin-fired fish catch carries a non-item entity CAUGHT_FISH normally carries an Item, but the event does not enforce the caught entity's type; the HIGH-priority handler cast it unchecked and crashed on plugin-fired events with other entity types. 2026-07-10 22:48:01 -07004d2b088f4a4efe8316c2d7718378ede358d040e4Fix error when a plugin-fired fish catch carries a non-item entityeditChangelog.txteditsrc/test/java/com/gmail/nossr50/listeners/PlayerListenerFishingTest.javaeditsrc/main/java/com/gmail/nossr50/listeners/PlayerListener.javasrc/test/java/com/gmail/nossr50/listeners/EntityListenerTest.javaChangelog.txtsrc/main/java/com/gmail/nossr50/listeners/EntityListener.java81f5c55ca02d8c9d3c11bcf6b0f35bd5dea586721783749004000https://popicraft.net/jenkins/user/nossr50nossr50nossr50@gmail.comFix splash saturation potions losing particle and icon visibility The saturation splash workaround re-added the effect with the short PotionEffect constructor, which resets particle and icon visibility to the defaults; custom potions brewed to hide their particles started showing them after the splash. 2026-07-10 22:50:04 -070081f5c55ca02d8c9d3c11bcf6b0f35bd5dea58672Fix splash saturation potions losing particle and icon visibilityeditsrc/main/java/com/gmail/nossr50/listeners/EntityListener.javaeditChangelog.txteditsrc/test/java/com/gmail/nossr50/listeners/EntityListenerTest.javasrc/main/java/com/gmail/nossr50/listeners/BlockListener.javasrc/main/java/com/gmail/nossr50/listeners/EntityListener.java1cad900859c47d0f8c1522d60f642164dfcfabba1783749123000https://popicraft.net/jenkins/user/nossr50nossr50nossr50@gmail.comClean up listener debug reporting and stale comments Extract the duplicated PvP combat debug report in EntityListener into a shared helper, correct the entity explode handler name, and fix two comments that no longer matched the code (traveling-block cleanup timer cadence, piston retract direction). 2026-07-10 22:52:03 -07001cad900859c47d0f8c1522d60f642164dfcfabbaClean up listener debug reporting and stale commentseditsrc/main/java/com/gmail/nossr50/listeners/BlockListener.javaeditsrc/main/java/com/gmail/nossr50/listeners/EntityListener.javasrc/main/java/com/gmail/nossr50/listeners/InventoryListener.javaChangelog.txt8fa27d38cc80ca3414915477dcd47e70d5416faf1783749192000https://popicraft.net/jenkins/user/nossr50nossr50nossr50@gmail.comMove buff-stripping inventory handlers off MONITOR priority The inventory click and open handlers strip ability buffs from item stacks, which mutates item state; the MONITOR contract forbids that. They now run at HIGHEST like the other mutating handlers moved during the listener priority cleanup. 2026-07-10 22:53:12 -07008fa27d38cc80ca3414915477dcd47e70d5416fafMove buff-stripping inventory handlers off MONITOR priorityeditsrc/main/java/com/gmail/nossr50/listeners/InventoryListener.javaeditChangelog.txtsrc/test/java/com/gmail/nossr50/listeners/WorldListenerTest.javaeffda0dd6f8025bba8a67cb74fa74723385b090a1783749419000https://popicraft.net/jenkins/user/nossr50nossr50nossr50@gmail.comAdd WorldListener coverage for growth eligibility and unload flushes 2026-07-10 22:56:59 -0700effda0dd6f8025bba8a67cb74fa74723385b090aAdd WorldListener coverage for growth eligibility and unload flushesaddsrc/test/java/com/gmail/nossr50/listeners/WorldListenerTest.javasrc/main/java/com/gmail/nossr50/datatypes/player/PlayerProfile.javaChangelog.txtsrc/test/java/com/gmail/nossr50/datatypes/player/PlayerProfileTest.java7ea3d91ff368a556f0e05c85db46f3a81157ae701783749968000https://popicraft.net/jenkins/user/nossr50nossr50nossr50@gmail.comFix error when adding levels to a child skill through the API Child skills have no level entry of their own, so the read-then-modify in PlayerProfile#addLevels crashed on the missing entry before the child guard in modifySkill could run. Added levels now split evenly across the child's parent skills, matching how child skill XP and the offline ExperienceAPI level variants already behave. The addlevels command rejects child skills up front, so only API callers were affected. 2026-07-10 23:06:08 -07007ea3d91ff368a556f0e05c85db46f3a81157ae70Fix error when adding levels to a child skill through the APIeditChangelog.txteditsrc/main/java/com/gmail/nossr50/datatypes/player/PlayerProfile.javaaddsrc/test/java/com/gmail/nossr50/datatypes/player/PlayerProfileTest.javasrc/main/java/com/gmail/nossr50/datatypes/player/PlayerProfile.javaChangelog.txtsrc/test/java/com/gmail/nossr50/datatypes/player/PlayerProfileTest.javae76d62cb5a8f78c7c4951f123cb936e03e561d4c1783750036000https://popicraft.net/jenkins/user/nossr50nossr50nossr50@gmail.comFix error reading XP requirements for offline players with the cumulative curve With 'Experience_Formula.Cumulative_Curve' enabled, PlayerProfile resolved the power level through UserManager, which has no entry for offline-loaded profiles (offline /inspect, ExperienceAPI offline lookups) and crashed. The profile's own level sum now stands in when no online player is found; permission-aware filtering is impossible offline anyway. Also corrects the stale getLastLogin javadoc. 2026-07-10 23:07:16 -0700e76d62cb5a8f78c7c4951f123cb936e03e561d4cFix error reading XP requirements for offline players with the cumulative curveeditsrc/test/java/com/gmail/nossr50/datatypes/player/PlayerProfileTest.javaeditChangelog.txteditsrc/main/java/com/gmail/nossr50/datatypes/player/PlayerProfile.javasrc/main/java/com/gmail/nossr50/datatypes/player/McMMOPlayer.java49ff833b9c475cbd3cd6d8ee6a77ac7cc278648c1783750154000https://popicraft.net/jenkins/user/nossr50nossr50nossr50@gmail.comClean up McMMOPlayer XP gain and logout paths Compute the modified XP once in beginUnsharedXpGain instead of running the permission-heavy perk modifier twice per gain (which also sent the XP perk debug report to debug-mode players twice), and reuse the local player reference in logout instead of re-reading a field that is never null. 2026-07-10 23:09:14 -070049ff833b9c475cbd3cd6d8ee6a77ac7cc278648cClean up McMMOPlayer XP gain and logout pathseditsrc/main/java/com/gmail/nossr50/datatypes/player/McMMOPlayer.javasrc/test/java/com/gmail/nossr50/datatypes/player/PlayerProfileTest.javacdc0f61d89a8338cae8bd7702a03123907a1e2391783750354000https://popicraft.net/jenkins/user/nossr50nossr50nossr50@gmail.comAdd PlayerProfile coverage for save retries and child-skill data handling 2026-07-10 23:12:34 -0700cdc0f61d89a8338cae8bd7702a03123907a1e239Add PlayerProfile coverage for save retries and child-skill data handlingeditsrc/test/java/com/gmail/nossr50/datatypes/player/PlayerProfileTest.javasrc/main/java/com/gmail/nossr50/datatypes/player/PlayerProfile.javaChangelog.txtsrc/test/java/com/gmail/nossr50/datatypes/player/PlayerProfileTest.java43a5f944b494fa9f7809b7731ebdd665e53fca9b1783751158000https://popicraft.net/jenkins/user/nossr50nossr50nossr50@gmail.comFix error when another plugin reads raw XP for a child skill Child skills store no XP entry of their own, so the raw XP read crashed on the missing entry. Zero is returned instead, matching getSkillXpLevel. In-repo callers filter child skills first; only plugins calling the profile API directly were affected. 2026-07-10 23:25:58 -070043a5f944b494fa9f7809b7731ebdd665e53fca9bFix error when another plugin reads raw XP for a child skilleditChangelog.txteditsrc/main/java/com/gmail/nossr50/datatypes/player/PlayerProfile.javaeditsrc/test/java/com/gmail/nossr50/datatypes/player/PlayerProfileTest.javasrc/main/java/com/gmail/nossr50/datatypes/player/PlayerProfile.javaChangelog.txtsrc/test/java/com/gmail/nossr50/datatypes/player/PlayerProfileTest.javab249b4dfcb9b92074c71d06afc42042402bda0fa1783751213000https://popicraft.net/jenkins/user/nossr50nossr50nossr50@gmail.comFix player data changes made during an in-flight save being lost The dirty flag was overwritten unconditionally once the database returned, so a change that landed while the async save was writing its copy was marked clean and skipped by later saves until the next unrelated change. The flag is now cleared before the copy is taken and only restored on failure, so concurrent changes re-mark the profile dirty and the next save picks them up. Answers the old TODO about synchronizing the copy: value races during the copy self-heal through the dirty flag, so no lock is needed. 2026-07-10 23:26:53 -0700b249b4dfcb9b92074c71d06afc42042402bda0faFix player data changes made during an in-flight save being losteditsrc/main/java/com/gmail/nossr50/datatypes/player/PlayerProfile.javaeditsrc/test/java/com/gmail/nossr50/datatypes/player/PlayerProfileTest.javaeditChangelog.txtsrc/main/java/com/gmail/nossr50/datatypes/player/PlayerProfile.javasrc/main/java/com/gmail/nossr50/datatypes/player/McMMOPlayer.javaChangelog.txtce2ae27ae198588b42ef5f42c5e9fd3b3d96fca41783751249000https://popicraft.net/jenkins/user/nossr50nossr50nossr50@gmail.comDeprecate the misspelled PlayerProfile Chimaera Wing cooldown getter getChimaerWingDATS is missing a letter; getChimaeraWingDATS replaces it and the old name delegates until removal. 2026-07-10 23:27:29 -0700ce2ae27ae198588b42ef5f42c5e9fd3b3d96fca4Deprecate the misspelled PlayerProfile Chimaera Wing cooldown gettereditsrc/main/java/com/gmail/nossr50/datatypes/player/McMMOPlayer.javaeditsrc/main/java/com/gmail/nossr50/datatypes/player/PlayerProfile.javaeditChangelog.txtsrc/test/java/com/gmail/nossr50/util/skills/SkillToolsTest.javaChangelog.txtsrc/main/java/com/gmail/nossr50/util/skills/SkillTools.java2b85a6fc726b00f53c0814fa289cdb0707cba6ef1783752481000https://popicraft.net/jenkins/user/nossr50nossr50nossr50@gmail.comStop console spam when other plugins check skill names through the API Unmatched skill name lookups logged an 'Invalid mcMMO skill' warning on every call, so plugins validating names through ExperienceAPI flooded the console. The message is now debug output shown only with Verbose_Logging. 2026-07-10 23:48:01 -07002b85a6fc726b00f53c0814fa289cdb0707cba6efStop console spam when other plugins check skill names through the APIeditChangelog.txteditsrc/test/java/com/gmail/nossr50/util/skills/SkillToolsTest.javaeditsrc/main/java/com/gmail/nossr50/util/skills/SkillTools.javasrc/main/java/com/gmail/nossr50/commands/experience/ExperienceCommand.javaChangelog.txtsrc/test/java/com/gmail/nossr50/commands/experience/AddxpCommandTest.javabb6a395fc67638db93cffa4b056270eaf7cbcda91783753760000https://popicraft.net/jenkins/user/nossr50nossr50nossr50@gmail.comMake the -s flag on XP commands silence the sender confirmation too /addxp, /addlevels, and /mmoedit echoed 'has been modified' back to the sender even with -s, flooding the log for plugins that award XP by dispatching these commands from console. 2026-07-11 00:09:20 -0700bb6a395fc67638db93cffa4b056270eaf7cbcda9Make the -s flag on XP commands silence the sender confirmation tooeditsrc/main/java/com/gmail/nossr50/commands/experience/ExperienceCommand.javaaddsrc/test/java/com/gmail/nossr50/commands/experience/AddxpCommandTest.javaeditChangelog.txtsrc/test/java/com/gmail/nossr50/datatypes/player/McMMOPlayerAddXpAndLevelsTest.javasrc/main/java/com/gmail/nossr50/datatypes/player/McMMOPlayer.javasrc/main/java/com/gmail/nossr50/util/EventUtils.javaChangelog.txtdf4cdce38ca8641eb3add62c2ab973f9ad0e253a1783753789000https://popicraft.net/jenkins/user/nossr50nossr50nossr50@gmail.comRoute McMMOPlayer addXp and addLevels through level up processing Both wrote raw values into the profile, so plugins using them left skills over their XP threshold with no level ups, events, or XP bar updates. addXp now runs the normal XP gain path with the amount unmodified, and addLevels fires the level change events like the addlevels command. Internal callers that revert cancelled events or apply vampirism gains keep raw writes via the profile. 2026-07-11 00:09:49 -0700df4cdce38ca8641eb3add62c2ab973f9ad0e253aRoute McMMOPlayer addXp and addLevels through level up processingeditChangelog.txteditsrc/main/java/com/gmail/nossr50/datatypes/player/McMMOPlayer.javaaddsrc/test/java/com/gmail/nossr50/datatypes/player/McMMOPlayerAddXpAndLevelsTest.javaeditsrc/main/java/com/gmail/nossr50/util/EventUtils.javagithttps://popicraft.net/jenkins/user/nossr50nossr50nossr50