Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Scripts/Outland: Added Shartuul's Transporter event #29789

Draft
wants to merge 9 commits into
base: 3.3.5
Choose a base branch
from

Conversation

14shagov
Copy link

@14shagov 14shagov commented Mar 8, 2024

Changes proposed:

  • Added Shartuul's Transporter event

Issues addressed:

Closes #18655

Tests performed:

Builded and tested in game

Known issues and TODO list:

  • I can't find the portal from which Shartuul exits.
  • Spell 40675 is not working.
  • Before Eye Stalk spawns, Shartuul summons a big meteor that hits the crystal, can't find this spell.
  • Spell 41550 doesn't work right (directs the beam under itself, when it should be on the enemy)
  • Before Eye Stalk appears, small meteorites are summoned. I can't implement it.
  • Eye of Shartuul, when using Disruption Ray, causes Stun Fields, which are affected by the corresponding animation, I can’t find this animation.
  • Spell 40493 is not working.
  • When replacing a demon, the action bar of skills does not change; you need to click on the green arrow, which changes the bars.
  • Sometimes, for some reason, when trash mobs appear, two green lightning bolts are thrown, there should be one
  • I don’t know how to implement the change of abilities for Shivan Assassin when changing aspect
  • When the bosses Doom Punisher, Shivan Assassin and Shartuul appear, they are in some kind of shadow. I can't find a spell that can do this.
  • Not sure about the flag_extra changes. But if you don't remove flag 128, the creatures won't aggro. Yes, DB has similar creatures that are already transformed and don't have unnecessary flag_extra, but they are completely different in characteristics.
  • Madness Rift should only cast in newly created Eye stalk.
  • Charm - North 2-3 (40382,40523) - when used, cast Character Invis and Immune (40357), which makes all creatures in the area invisible, and Possess has no effect on Demon.

@14shagov
Copy link
Author

14shagov commented Mar 8, 2024

Issues:

  • I can't find the portal from which Shartuul exits.
  • Spell 40675 is not working.
  • Before Eye Stalk spawns, Shartuul summons a meteor that hits the crystal, can't find this spell.
  • Spell 41550 doesn't work right (directs the beam under itself, when it should be on the enemy)
  • Before Eye Stalk appears, small meteorites are summoned. I can't find this spell
  • Eye of Shartuul, when using Disruption Ray, causes Stun Fields, which are affected by the corresponding animation, I can’t find this animation.
  • Spell 40493 is not working.
  • When replacing a demon, the action bar of skills does not change; you need to click on the green arrow, which changes the bars.
  • Sometimes, for some reason, when trash mobs appear, two green lightning bolts are thrown, there should be one

@Faq
Copy link
Contributor

Faq commented Mar 8, 2024

add that under: Known issues and TODO list:

@heyitsbench
Copy link
Contributor

heyitsbench commented Mar 8, 2024

I do have sniffs of a few attempts at this event on Wrath Classic if it'd be any help to you. I can send them through Discord if you have it: @heyitsbench.

Co-authored-by: 14shagov <14shagov>
@14shagov
Copy link
Author

This commit fixes issues with spells 40675 and 40493. I misspelled it.

…erAI, SPELL_MADNESS_RIFT and SPELL_TOUCH_OF_MADNESS (#4)
@14shagov
Copy link
Author

  • It turns out that ability 40493 (Super Jump) still doesn't work. If you use it under Doom Punisher control, the ability can't select a target.
  • A new issue occurred when I replaced Charm (530) with Charm - North 2-3 (40382, 40523) - when used, cast Character Invis and Immune (40357), which makes all creatures in the area invisible, and Possess has no effect on Demon.
  • Another new issue, Madness Rift must only cast in newly created Eye stalk.

Reworked the logic associated with Shield Lightning and Spawn Lightning
- Madness Rift now only casts in newly summoned Eye stalk.
- I completely canceled the first effect of Charm - North 2-3 (40382,40523), which fixed the demon possession bug. But this problem should probably be handled in a different way.
@14shagov
Copy link
Author

14shagov commented May 6, 2024

I'm wondering if I should replace all npc and boss classes with structs? And use RegisterCreatureAI instead of new npc_blabla. As far as I understand, the new standard does exactly that.

Thrash Waves Optimization
Update to current standards
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants