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
  • #1404

Closed
Open
Created Nov 23, 2023 by IonTech@I0nTech

Skill Cooldown Placeholder/Conditions do not function correctly when Cooldown is defined inline

Demonstration of issue, also applies to cooldown conditions. Cast test-cooldown-inline and test-cooldown twice each for comparison.

test-cooldown-inline:
  Skills:
  - skill{s=test-cooldown-inline-exec;cooldown=12} @self

test-cooldown-inline-exec:
  OnCooldownSkill: test-cooldown-ocd
  Skills:  
  - message{m="Casted"} @self

test-cooldown-inline-ocd:
  Skills:
  - aura{d=80;i=5;
         ot=[ 
         - message{m="<caster.skill.test-cooldown-inline-exec.cooldown>"}  
         ]} @self
 # - message{m="<caster.skill.test-cooldown-inline-exec.cooldown>"} @self

# Will Not Show Cooldown, despite the skill being on cooldown from the inline skill cast

test-cooldown:
  Cooldown: 12
  OnCooldownSkill: test-cooldown-ocd
  Skills:  
  - message{m="Casted"} @self

test-cooldown-ocd:
  Skills:
  - aura{d=80;i=5;
         ot=[ 
         - message{m="<caster.skill.test-cooldown.cooldown>"} 
         ]} @self
 # - message{m="<caster.skill.test-cooldown.cooldown>"} @self

# Will show cooldown

Another issue is needing to make a different onCooldownSkill for every single skill which is castable as <caster.skill.[someskill].cooldown> requires you define the skill name, and you can't put a mm placeholder inside another mm placeholder. (Would be cool if you could though)

To fix, implementing a placeholder like <skill.var.cooldown-duration> which fetches the latest cooldown "upwards" from the current skill tree would allow for the creation of a single cooldown skill to display if an item/skill is on cooldown.

Assignee
Assign to
None
Milestone
None
Assign milestone
Time tracking