[REJECTED][Complete] Junkboxes Needed[REJECTED]

Melik1 · August 8, 2011, 11:14pm


-- "Junkboxes Needed". Changes the requirements to all classes.

update `quest_template` set `ZoneOrSort`=36 where `entry`=8249;

Problem:

classes other than Rogues cannot pick up this quest, which is not Blizzlike because this quest is used mainly for the Insane title achievement. The reason other classes couldn’t pick it up was because the ZoneOrSort value was set to -4 (Rogue) and the core interpreted it as a Rogue quest. This change makes the quest fall in under Alterac Mountains category therefore making it available for all classes (hack, perhaps?)

Please reply to this thread with reason. Simple replies such as “wrong” and thereafter closing the thread isn’t helping at all. Thank you.

Regards.

Melik1 · August 8, 2011, 11:36pm

-wrong section, sorry, please move this thread-

Nay · August 8, 2011, 11:52pm

Yes, unfortunatly it is a hack. The -4 comes for WDB caches and sniffs and we shouldn’t change it.

Some time ago mangos did something with ZoneOrSort and other quest_template field that made this quest work and it didn’t seem hackish.

I will try to find their commit.

Edit:

Here it is, thanks Aokromes for the link!

ZxBiohazardZx · August 9, 2011, 4:01pm

ancient hackfix for it, not accepted any of the times it has been suggested

EDIT: that mangos fix makes sense to me…

Melik1 · August 9, 2011, 5:07pm

Indeed it does.

How would anyone fix this issue taking in mind with the MaNGOS patch and apply it onto TrinityCore? Is that even possible?

If not, it seems the only fix would be the hack… Or am I wrong?

Nay · August 9, 2011, 5:18pm

Yes it is possible but so far no one wanted to do it.