Skip to content

GitLab

  • Projects
  • Groups
  • Snippets
  • Help
    • Loading...
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
    • Contribute to GitLab
  • Sign in
MythicMobs MythicMobs
  • Project overview
    • Project overview
    • Details
    • Activity
  • Issues 132
    • Issues 132
    • List
    • Boards
    • Labels
    • Service Desk
    • Milestones
  • Operations
    • Operations
    • Incidents
  • Analytics
    • Analytics
    • Repository
    • Value Stream
  • Wiki
    • Wiki
  • Members
    • Members
  • Activity
  • Create a new issue
  • Issue Boards
Collapse sidebar
  • MythicCraft
  • MythicMobsMythicMobs
  • Issues
  • #444

Closed
Open
Created Jun 29, 2021 by wolfebersahd@wolfebersahd

"~onSpawn" triggered skill doesn't get cancelled when mob spawning is denied.

Summary

"~onSpawn" triggered skills don't get cancelled when mob spawning is denied via WorldGuard "mobspawning deny" or other plugins that deny mob spawning like GriefDefender.

Steps to reproduce

  1. Install the following plugins
  • EssentialsX (2.18.2.0)
  • LuckPerms (5.3.3)
  • MythicMobs (4.12.0)
  • WorldEdit (7.2.5)
  • WorldGuard (7.0.5)
  1. Edit the "skeletalknight" example mob as follows to add and ~onSpawn skill. https://paste.helpch.at/pojekujope.coffeescript
  2. Create a region in game (//pos1, //pos2, //expand vert, /region create test...)
  3. Add the mobspawning deny flag to the region (/region flag test mobspawning deny)
  4. Stand within the region.
  5. Summon a SkeletalKnight

Current behavior

The mob is NOT spawned but the sound is still played. This also occurs with most effects, skills, mechanics, including mechanics from third part plugins like ModelEngine where the model is spawned even though the mob spawning was denied.

Intended correct behavior

The "~onSpawn" triggered skills should not be executed when the mob spawning event is denied or cancelled.

Server log file

https://paste.helpch.at/uhuhazopuj.md

Debug log snippet

I didn't run it with debug on... I can later and edit this issue.

Proposed fixes

The "~onSpawn" trigger should check if the mob spawning event was cancelled before executing.

Edited Jun 29, 2021 by wolfebersahd
Assignee
Assign to
None
Milestone
None
Assign milestone
Time tracking