classicdb / database

Classic DB is a content database for CMaNGOS Classic: world, NPCs, objects, quests and so on.
https://github.com/cmangos/mangos-classic
Other
87 stars 59 forks source link

Gadgetzan #860

Open Phatcat opened 8 years ago

Phatcat commented 8 years ago

The guards of Gadgetzan do not care about you attacking flight masters (at least the horde one) - not even the taxis he summons are willing to help him unless you enter their actual aggro range. This may be a core issue.

Goblin engineer in Gadgetzan will teach you goblin recipies even if you're not specialized in goblin engineering. He is willing to teach you even if you don't know engineering at all.

Update:

Tombstone in Gadgetzan graveyard geets you with the text "Greetings " - This doesn't seem intended. solved in https://github.com/cmangos/classic-db/commit/b7410628df04b3d6d534bdd08e4d7e2cdd56af72

Rumsen Fizzlebrack (just right out of Gadgetzan) has a repeatable quest available to everyone - It's funny I never noticed this quest during vanilla, I did love the hell out of Tanaris and Zul Farrak back then. Then I read the quest (which is by the way "War at Sea") and it had this to say:

> You seek to befriend the goblins of Tanaris once again, name? (...)

Yeah, this quest (or any like it) should only be available to anyone at war with the faction in question (This is why he's placed outside the city, away from any guards - I had not even interacted with gadgetzan before this quest was available to me) solved in https://github.com/cmangos/classic-db/commit/03d69306d39ee2606f7c74f801a37293d49d8162

lduguid commented 8 years ago

Concerning, 'Tombstone in Gadgetzan graveyard geets you with the text "Greetings " - This doesn't seem intended.', I do not know what the intended greeting should be when clicking on quest items when you are not on a relevant quest and/or appropriate part of quest chain but as an FYI, one of the tombstone's at the Gadgetzan graveyard is an item in the long quest chain involving Linken (http://db.vanillagaming.org/?npc=8737), this particular quest being 'Meet at the grave' (http://db.vanillagaming.org/?quest=3912).

Phatcat commented 8 years ago

Oh, I know it's an object relating to a quest, but it should be turned 'un-interactive' when not on the quest. Pretty sure you're able to set prerequesties for objects, no?

I'm new here on the db team so I am not really sure how to handle this or I would have taken care of that one myself.

Tobschinski commented 8 years ago

https://github.com/classicdb/database/issues/142

Phatcat commented 8 years ago

Yeah, I thought I'd seen it somewhere before... so it's a general issue we have that we are missing pre requirements for a bunch of objects in the world? I should learn to read better

cala commented 8 years ago

The guards of Gadgetzan do not care about you attacking flight masters (at least the horde one) - not even the taxis he summons are willing to help him unless you enter their actual aggro range. This may be a core issue and if it is then I'm going to take a crack at it

Depending on how they are summoned (I don't have the NPC entry at hand, so I can't check):

Phatcat commented 8 years ago

I'm just gonna take a wee visit back to Gadgetzan on that branch to see if those aggro commits are helpful in this case.

Phatcat commented 8 years ago

Okay, either the commits do not function properly or his taxis are scripted because they simply do not care.

Phatcat commented 8 years ago

Or.... It may be because he enters combat before doing his summons? in which case my branch does not help at all since the relevant commits effect takes place at entering combat only.

cala commented 8 years ago

in which case my branch does not help at all since the relevant commits effect takes place at entering combat only

This should also work when master is already in combat. Could you provide me the name or entry of the NPC, please? I will look at his script if any.

Phatcat commented 8 years ago

Of course, cala.

Bulkrek Ragefist - ID 7824

cala commented 8 years ago

Thanks. As I thought the script is (partly) faulty. In ACID, the NPC is directly summoning the NPC 9297 (Enraged Wyvern) instead of using what I assume to be the correct spell: 14307 - Summoned Enraged Wyvern. This should provide better results (but probably not perfect):

DELETE FROM  `creature_ai_scripts` WHERE `id` = 782401;
INSERT INTO `creature_ai_scripts` VALUES (782401, 7824, 4, 0, 100, 0, 0, 0, 0, 0, 11, 14307, 0, 0, 1, -548, 0, 0, 0, 0, 0, 0, 'Bulkrek Ragefist - Summon Enraged Wyverns and Yell on Aggro');
Phatcat commented 8 years ago

Those ACID scripts that are flawed should of course be corrected to our best ability (replace any manual summon we can find with dedicated spells)

To fix the rest I can add in the 'assist summoner on summon' functionality I added to the other spell effects. It's because that functionality is not added for TempSummons being summoned manually (so far it has only been added to spelleffects.cpp)

cala commented 8 years ago

I'll update the ACID scripts soon, the spells exist in Classic DBCs and these summons were removed after, so we should not need a specific core update for this, fortunately. :)

Phatcat commented 7 years ago

The guards of gadgetzan still couldn't care less about you attacking flight masters (I'm guessing this is possibly core issue, maybe to do with generic GuardAI functions), the 2 bruisers next to the horde guy is purely for show on cmangos. His taxis are however extremely aggressive now.

This time I made sure I wasn't on gm mode and I was still able to learn goblin engineering recipies even though the char wasn't a goblin engineer <- this is a core thing isn't it?

the "War at Sea" quest, along with the ratchet, everlook and booty bay counterparts must not be available to anyone who is neutral (or higher) or unknown (hasn't interacted with them or gotten +rep for them yet) - these npcs are meant to be used to be bring your rep back up if you've been at war with them, the quest text for War at Sea even reads at the bottom

"(...) and we'll be on our way to pardoning your tresspasses"

this would make no sense in the context they are available on cmangos (also they can be aused for 'cheap' rep grinding as it currently is, and it is a bit immersion breaking.

Phatcat commented 5 years ago

The guards of gadgetzan still couldn't care less about you attacking flight masters (I'm guessing this is possibly core issue, maybe to do with generic GuardAI functions), the 2 bruisers next to the horde guy is purely for show on cmangos. His taxis are however extremely aggressive now.

This time I made sure I wasn't on gm mode and I was still able to learn goblin engineering recipies even though the char wasn't a goblin engineer <- this is a core thing isn't it?

Both issues still present as of early 2019. Both of these issues are core issues aren't they @cala ? - should we close this and raise new issues there?