Started by an SCM change3333a711577530dc539787bfaf68770717b515018f13a711577530dc539787bfaf68770717b515018f1refs/remotes/origin/master3a711577530dc539787bfaf68770717b515018f13a711577530dc539787bfaf68770717b515018f1refs/remotes/origin/master3a711577530dc539787bfaf68770717b515018f13a711577530dc539787bfaf68770717b515018f1refs/remotes/origin/masterhttps://github.com/mcMMO-Dev/mcMMO.githttps://popicraft.net/jenkins/job/mcMMO/333/artifacthttps://popicraft.net/jenkins/job/mcMMO/changeshttps://popicraft.net/jenkins/job/mcMMO/333/https://popicraft.net/jenkins/job/mcMMO/333/testReportmcMMO.jarmcMMO.jartarget/mcMMO.jaroriginal-mcMMO.jaroriginal-mcMMO.jartarget/original-mcMMO.jarfalse#333102128135282mcMMO #333333falsefalse333205SUCCESS1783588748658https://popicraft.net/jenkins/job/mcMMO/333/Buildersrc/main/java/com/gmail/nossr50/mcMMO.javasrc/main/java/com/gmail/nossr50/runnables/skills/DelayedHerbalismXPCheckTask.javasrc/test/java/com/gmail/nossr50/skills/herbalism/HerbalismMultiBlockPlantXpTest.javasrc/main/java/com/gmail/nossr50/runnables/skills/PlantCollapseXpTask.javasrc/main/java/com/gmail/nossr50/skills/herbalism/HerbalismManager.javaChangelog.txt3a711577530dc539787bfaf68770717b515018f11783588077000https://popicraft.net/jenkins/user/nossr50nossr50nossr50@gmail.comVerify multi-block plants actually break before paying Herbalism XP Vanilla destroys the rest of a bamboo, sugar cane, kelp, cactus, chorus, or hanging vine plant with scheduled block ticks on later ticks, cascading one block per tick. XP for the whole plant was paid during the BlockBreakEvent, so breaking a segment and instantly replacing it collected XP for blocks that never broke. The rest of the plant is now claimed by a per-tick verification task that only pays XP for blocks that are verifiably gone (air, or water for waterlogged plants) once the collapse window elapses. Positions can only be claimed by one task at a time and claims are revoked when a new break event takes over a position, so no block is paid twice. This also revives XP for cactus and chorus columns: their traversal was aborting immediately because the origin block pre-seeded the visited set, so connected blocks never paid XP at all. Fixes #5311 2026-07-09 02:07:57 -07003a711577530dc539787bfaf68770717b515018f1Verify multi-block plants actually break before paying Herbalism XPaddsrc/test/java/com/gmail/nossr50/skills/herbalism/HerbalismMultiBlockPlantXpTest.javaaddsrc/main/java/com/gmail/nossr50/runnables/skills/PlantCollapseXpTask.javaeditsrc/main/java/com/gmail/nossr50/runnables/skills/DelayedHerbalismXPCheckTask.javaeditsrc/main/java/com/gmail/nossr50/mcMMO.javaeditsrc/main/java/com/gmail/nossr50/skills/herbalism/HerbalismManager.javaeditChangelog.txtgithttps://popicraft.net/jenkins/user/nossr50nossr50nossr50