trickerer / Trinity-Bots

NPCBots for TrinityCore and AzerothCore 3.3.5
https://github.com/trickerer/TrinityCore-3.3.5-with-NPCBots/
446 stars 148 forks source link

[AC] [Bug] [Core] Creature 31258's damage is lower than expected #828

Open SouthMemory opened 1 week ago

SouthMemory commented 1 week ago

DO NOT REMOVE OR SKIP THE ISSUE TEMPLATE

Current behaviour

within AzerothCore-wotlk-with-NPCBots, Creature 32158's attack only damage about half of expected value, 200-300, but it should be 422 - 586(from https://db.nfuwow.com/80/?npc=31258) I tried the original AzerothCore-wotlk, the damage is normal

Expected behaviour

damage should be 422 - 586

Crashlog

No response

Steps to reproduce the problem

1.install AzerothCore-wotlk-with-NPCBots 2.test creature 31258's damage, it's 200-300, which is not normal

TrinityCore or AzerothCore

AzerothCore

Core rev. hash/commit

unknown

Operating system

No response

Day36512 commented 1 week ago

This is an Azerothcore issue, not one directly related to Trickerer's fork at all. Suggest you put in an issue ticket on the main repo here: https://github.com/trickerer/AzerothCore-wotlk-with-NPCBots

trickerer commented 1 week ago

No, the issue is caused by https://github.com/trickerer/AzerothCore-wotlk-with-NPCBots/commit/a22a353. The contradiction here is the fact that original AzerothCore factors attack power into damage the wrong way causing melee bots to do way too much damage. I just ported TC code where AP to damage calculation is the same for both players and creatures. Didn't check TC damage for that creature but DamageModifier and classlevelstats values for unit class 1 look the same for both cores. Normally if creature deals more damage blizzlike then changing DamageModifier field of creature_template table is the way to fix it.

SouthMemory commented 1 week ago

You mean change the DamageModifier of creature 31258 to double its damage? But I guess this problem influences not just 31258, but many other creatures. I'll test and try to find a solution here

icedcolor commented 1 week ago

@trickerer
https://github.com/azerothcore/azerothcore-wotlk/pull/19068 This update to [AC] resulted in a compilation error. 620