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

Correct stop the fight for Madrigosa after the fight with Brutallus, creating a normal flight for Madrigosa and Felmyst. #29

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

ghost
Copy link

@ghost ghost commented Apr 20, 2015

  1. Madrigosa in the supine position after a battle with Brutallus again begins his attack, please do not say what on Wotlk everything works fine - it's not, I tested and there, and there.
  2. Madrigosa and Felmyst not fly properly, and "walking on air" legs, well, this is in any gate not climbing. Adding SetByteFlag fixes the problem properly. Both InhabitType = 5, so that's not a problem database.

Ulduar and others added 2 commits April 20, 2015 19:44
@xfurry
Copy link
Member

xfurry commented Apr 21, 2015

  • SetByteValue(UNIT_FIELD_BYTES_1, 3, UNIT_BYTE1_FLAG_FLY_ANIM); is correct but it should be included here: cmangos/mangos-cata@fa0d774 in the SetCanFly() function, so that all the flying creatures will benefit from it. Also, please make sure that the DB has inhabitType = 7 for that creature.
  • the passive flag is not required and not confirmed by sniff. Anyway this whole Madrigosa event is kind of a big hack that I made in order to simplify the logic of the script.
    Theoretically the fake death shouldn't be used here, but without it there will be a lot of other complications.

@ghost
Copy link
Author

ghost commented Apr 21, 2015

SetByteValue(UNIT_FIELD_BYTES_1, 3, UNIT_BYTE1_FLAG_FLY_ANIM); is correct but it should be included here: cmangos/mangos-cata@fa0d774 in the SetCanFly() function, so that all the flying creatures will benefit from it. Also, please make sure that the DB has inhabitType = 7 for that creature.

Did you know that it does not work?
Need add SetByteValue.

the passive flag is not required and not confirmed by sniff. Anyway this whole Madrigosa event is kind of a big hack that I made in order to simplify the logic of the script.

Then why is she attacking Brutallus being in a position Fake death?
Sense from sniff then?
Russian Translate:
Ты в курсе что это не работает, что Инхабит тип = 7 ничего не меняет.
И почему же она атакует Бруталлуса после мнимой смерти?
И толк от сниффов тогда?

@ghost
Copy link
Author

ghost commented Aug 18, 2015

UNIT_FLAG_NOT_SELECTABLE also hacking, however for some reason there is why my fix then can not be used?

@ghost
Copy link
Author

ghost commented Oct 7, 2015

SetByteValue(UNIT_FIELD_BYTES_1, 3, UNIT_BYTE1_FLAG_FLY_ANIM);
Flag Passive not solved problem.
Yet I'll found a solution, adding new functions into the core and check in the game - Madrigosa now will not attack Brutallus after fake death.
Full history commit you can see if needed:
https://github.com/Ulduar/mangos-cata/commits/misc

-- Creature id: 24895
UPDATE creature_template SET AIName='EventAI' WHERE entry=24895;
DELETE FROM creature_ai_scripts WHERE creature_id=24895;
INSERT INTO `creature_ai_scripts` (`id`, `creature_id`, `event_type`, `event_inverse_phase_mask`, `event_chance`, `event_flags`, `event_param1`, `event_param2`, `event_param3`, `event_param4`, `action1_type`, `action1_param1`, `action1_param2`, `action1_param3`, `action2_type`, `action2_param1`, `action2_param2`, `action2_param3`, `action3_type`, `action3_param1`, `action3_param2`, `action3_param3`, `comment`) values
('2489501','24895','11','0','100','6','0','0','0','0','74','1','0','0','75','1','0','0','61','0','0','0','Madrigosa - set react no attack, set react not evaded, set fly.'),
('2489502','24895','11','0','100','6','0','0','0','0','42','1','0','0','0','0','0','0','0','0','0','0','Madrigosa - Void Damage Taken = 0 on Spawn.'),
('2489503','24895','8','0','100','6','44884','-1','0','0','49','0','0','0','54','0','0','0','73','0','0','0','Madrigosa - Spellhit Charge - Set Delete Threat List, Combat Stop, Attack Stop.'),
('2489504','24895','8','0','100','6','44884','-1','0','0','53','0','0','0','55','0','0','0','58','0','0','0','Madrigosa - Spellhit Charge - Clear Movement, Remove Evade Auras, Stop Movement.');

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants