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

[NPC/Boss] Scourgelord Tyrannus is not selectable #81

Closed
click opened this issue May 16, 2011 · 11 comments
Closed

[NPC/Boss] Scourgelord Tyrannus is not selectable #81

click opened this issue May 16, 2011 · 11 comments

Comments

@click
Copy link
Contributor

click commented May 16, 2011

Before you engage him, he can be targeted, but no longer after you get aggro, must be a script issue. but I did check others Boss scripts and have the same structure and their can be targeted after you get aggro,

WORKAROUND DB :

-- set FLAG_UNK_6 to Scourgelord Tyrannu

UPDATE creature_template SET unit_flags=64 WHERE entryIN (36658,3665801);

Database : TDB_335.11.36_10818
Trinity core : 10866
Addon : None

@click
Copy link
Contributor Author

click commented May 16, 2011

Author: nay
I see flag changes but no bitwise math. (Never use = for flags)
It's the core script that should add/remove the attackable/targeteable flags.

@click
Copy link
Contributor Author

click commented May 16, 2011

Author: androx
Replying to [comment:2 nay]:

I see flag changes but no bitwise math. (Never use = for flags)
It's the core script that should add/remove the attackable/targeteable flags.

Yup, I agree with you... I´ll check the core script...

@click
Copy link
Contributor Author

click commented May 16, 2011

Author: fuchs
pls fix this ;P

@click
Copy link
Contributor Author

click commented May 16, 2011

Author: androx
Replying to [comment:4 fuchs]:

pls fix this ;P

I'll try, I´m not a trinity dev, but I'm a C/C++ Dev and I can help =), currently I'm learning all project structure, mechanism, logic, etc, and later, I can contribute with fixes =),this is a large project in SLOC (1.9M lines of code aprox), currently I'm finding where the core set and unset the flags of bosses, I have some tips, for example:

src/server/scripts/EasternKingdoms/ScarletMonastery/boss_mograine_and_whitemane.cpp:88: me->RemoveFlag(UNIT_FIELD_FLAGS, UNIT_FLAG_NON_ATTACKABLE);

src/server/scripts/Northrend/FrozenHalls/HallsOfReflection/halls_of_reflection.h:114: me->RemoveFlag(UNIT_FIELD_FLAGS, UNIT_FLAG_NON_ATTACKABLE|UNIT_FLAG_NOT_SELECTABLE);

src/server/scripts/Northrend/FrozenHalls/HallsOfReflection/halls_of_reflection.cpp:169: pCreature->RemoveFlag(UNIT_NPC_FLAGS, UNIT_NPC_FLAG_GOSSIP);

src/server/scripts/Northrend/FrozenHalls/HallsOfReflection/halls_of_reflection.cpp:175: pCreature->RemoveFlag(UNIT_NPC_FLAGS, UNIT_NPC_FLAG_GOSSIP);

I see in other script this methods (RemoveFlags and SetFlags) and of course, this flags are in Unit.h, I'll get the last revision from Git, and set unitflag of Tyrranus to the original value from TDB_335.11.36_10818, I see some commits related with Scourge Tyrannus boss, I´ ll test this git commit...

@click
Copy link
Contributor Author

click commented May 16, 2011

Author: androx
With TrinityCore Rev: 10935+ Release Hash: 77af7ba+ , TDB : 335.11.36 and TDBAI 335.0.3 , the boss is an invalid target, if you update the unit_flags from 33554496 to 64 is the same result...

But, now the scene of Ick is much better =), Scourgelord Tyrannus now appear and kill Ick Boss (nice job =) )....

@click
Copy link
Contributor Author

click commented May 16, 2011

Author: androx
Ok I see the script of Scougelord Tyrannus at line number 482 the reasson why the script dont start, I'll set my gm account to normal and check the boss again... details :

{{{
bool OnTrigger(Player* player, const AreaTriggerEntry* /at/)
{
InstanceScript* instance = player->GetInstanceScript();
if (player->isGameMaster() || !instance)
return false;

        if (instance->GetBossState(DATA_TYRANNUS) != IN_PROGRESS || instance->GetBossState(DATA_TYRANNUS) != DONE)
            if (Creature* tyrannus = ObjectAccessor::GetCreature(*player, instance->GetData64(DATA_TYRANNUS)))
            {
                tyrannus->AI()->DoAction(ACTION_START_INTRO);
                return true;
            }

        return false;

}}}

...

@click
Copy link
Contributor Author

click commented May 16, 2011

Author: stryker
have you applied

http://www.trinitycore.org/t/trinitycore/ticket/3 < ?

@click
Copy link
Contributor Author

click commented May 16, 2011

Author: androx
Replying to [comment:8 stryker]:

have you applied

http://www.trinitycore.org/t/trinitycore/ticket/3 < ?

Yup I applied this :

{{{
UPDATE creature_template SET speed_run=1.857143,speed_walk=3.2,unit_flags=0x00000040,faction_A=14,faction_H=14,baseattacktime=2000,equipment_id=1872 WHERE entry IN (36658,36938); -- Scourgelord Tyrannus
}}}

The same result the boss Scourge Tyrannus is an invalid target, I killed Forgemaster and Ick too (Heroic and no heroic mode)...

TrinityCore Rev: 10941+ Release Hash: eac259f+
TDB 335.11.36
TDBAI 335.0.3

@click
Copy link
Contributor Author

click commented May 16, 2011

Author: androx
Now I applied the entire db fixed from : http://www.trinitycore.org/t/trinitycore/ticket/3 plus

{{{
UPDATE creature_template SET speed_run=1.857143,speed_walk=3.2,unit_flags=0x00000040,faction_A=14,faction_H=14,baseattacktime=2000,equipment_id=1872 WHERE entry IN (36658,36938); -- Scourgelord Tyrannus
}}}

And only the Rimefang spawn, not the Scourgelord Tyrannus... , the same in Heroic o Normal...

Now is necesary the Quest Deliverance from pit to enter the Hall of Reflexion (nice job =) ) but is necesary complete the quest The path of citadel (only Forgemastes is killable, but free the slave is not possible, ticket : http://www.trinitycore.org/t/trinitycore/ticket/29

Complete quest set:

http://www.wowhead.com/quest=24712/deliverance-from-the-pit

TrinityCore Rev: 10941+ Release Hash: eac259f+
TDB 335.11.36
TDBAI 335.0.3

@click
Copy link
Contributor Author

click commented May 16, 2011

Author: zxbiohazardzx
if you wish to fix the unit then fix the script, and submit a new patch, this update is wrong

@click
Copy link
Contributor Author

click commented May 16, 2011

Author: androx
If this ticket is invalid and I applied only the suggest from this ticket: http://www.trinitycore.org/t/trinitycore/ticket/3 both ticket should be invalid, not only this.....

@click click closed this as completed May 16, 2011
Aokromes added a commit to Aokromes/TrinityCore that referenced this issue Aug 12, 2016
freadblangks referenced this issue in freadblangks/Aquayoup-SL Jan 19, 2022
sveN295 added a commit to sveN295/TrinityCore that referenced this issue May 3, 2022
Declipe pushed a commit to Declipe/TrinityCore that referenced this issue Oct 16, 2022
…ticheat

Update (Passive Anticheat): Monthly Alignment
zTerragor pushed a commit to NeverlandWOW/NVN_DF that referenced this issue Jan 7, 2024
Tools/Map extractor: Minor optimization in MCNK parsing loop - look u…
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant