breakwater

maybe you can explain, why you have write this patch with " _player->TeleportToBGEntryPoint();"

my opinion: thats the wrong place, maybe we should cancel all spells on mini pet in Spell:CheckTarget-function

new system with pointers http://paste2.org/p/1975602

Maybe someone can test this patch on a high populated server. Please no direct commit into mangosR2.

Spells this will need a check Spells with are effected by this patch: SELECT * FROM `dbc-spell` WHERE (`dbc-spell`.EffectImplicitTargetA1 = `dbc-spell`.EffectImplicitTargetA2 and `dbc-spell`.EffectImplicitTargetB1 = `dbc-spell`.EffectImplicitTargetB2 and `dbc-spell`.EffectImplicitTargetA1 != 0) or (`dbc-spell`.EffectImplicitTargetA1 = `dbc-spell`.EffectImplicitTargetA3 and `dbc-spell`.EffectImplicitTargetB1 = `dbc-spell`.EffectImplicitTargetB3 and `dbc-spell`.EffectImplicitTargetA3 != 0) or (`dbc-spell`.EffectImplicitTargetA2 = `dbc-spell`.EffectImplicitTargetA3 and `dbc-spell`.EffectImplicitTargetB2 = `dbc-spell`.EffectImplicitTargetB3 and `dbc-spell`.EffectImplicitTargetA2 != 0) Spells affected in Wotlk: 11728 Spells with are effected by this patch and with an different radius value: Spells that can make problems because spell target mask is same, but radius value has an other value SELECT * FROM `dbc-spell` WHERE (`dbc-spell`.EffectImplicitTargetA1 = `dbc-spell`.EffectImplicitTargetA2 and `dbc-spell`.EffectImplicitTargetB1 = `dbc-spell`.EffectImplicitTargetB2 and (`dbc-spell`.Effect1 > 0 and `dbc-spell`.Effect2 > 0) and `dbc-spell`.EffectRadiusIndex1 != `dbc-spell`.EffectRadiusIndex2) or (`dbc-spell`.EffectImplicitTargetA1 = `dbc-spell`.EffectImplicitTargetA3 and `dbc-spell`.EffectImplicitTargetB1 = `dbc-spell`.EffectImplicitTargetB3 and (`dbc-spell`.Effect1 > 0 and `dbc-spell`.Effect3 > 0) and `dbc-spell`.EffectRadiusIndex1 != `dbc-spell`.EffectRadiusIndex3) or (`dbc-spell`.EffectImplicitTargetA2 = `dbc-spell`.EffectImplicitTargetA3 and `dbc-spell`.EffectImplicitTargetB2 = `dbc-spell`.EffectImplicitTargetB3 and (`dbc-spell`.Effect2 > 0 and `dbc-spell`.Effect3 > 0) and `dbc-spell`.EffectRadiusIndex2 != `dbc-spell`.EffectRadiusIndex3)SELECT * FROM `dbc-spell` WHERE (`dbc-spell`.EffectImplicitTargetA1 = `dbc-spell`.EffectImplicitTargetA2 and `dbc-spell`.EffectImplicitTargetB1 = `dbc-spell`.EffectImplicitTargetB2 and (`dbc-spell`.Effect1 > 0 and `dbc-spell`.Effect2 > 0) and `dbc-spell`.EffectRadiusIndex1 != `dbc-spell`.EffectRadiusIndex2) or (`dbc-spell`.EffectImplicitTargetA1 = `dbc-spell`.EffectImplicitTargetA3 and `dbc-spell`.EffectImplicitTargetB1 = `dbc-spell`.EffectImplicitTargetB3 and (`dbc-spell`.Effect1 > 0 and `dbc-spell`.Effect3 > 0) and `dbc-spell`.EffectRadiusIndex1 != `dbc-spell`.EffectRadiusIndex3) or (`dbc-spell`.EffectImplicitTargetA2 = `dbc-spell`.EffectImplicitTargetA3 and `dbc-spell`.EffectImplicitTargetB2 = `dbc-spell`.EffectImplicitTargetB3 and (`dbc-spell`.Effect2 > 0 and `dbc-spell`.Effect3 > 0) and `dbc-spell`.EffectRadiusIndex2 != `dbc-spell`.EffectRadiusIndex3) Spells affected in Wotlk: 216 EDIT: A lot of Wotlk spells are fixed with this patch, because some effects has no radius (Example 72015, 72836) Spells with makes problems: often classic spells like: 6266, 3143, 8150, 10253

MANGOS_DLL_SPEC uint32 urand(uint32 min, uint32 max); ??? i think the urand(...) funcion should use direct in sd2 like this one m_uiEyebeam_Timer = 10000 + urand(1000, 5000); This works realy good, you doesn't need MANGOS_DLL_SPEC

We should check some cases in function SetTargetMap(...). The spells with same target mask must have same radius, chain-length, etc

What bug does the patch fix? What features does the patch add? For preventing an double calculation for effects with same targets For which repository revision was the patch created? actual, but i think for all Is there a thread in the bug report section or at lighthouse? If yes, please add a link to the thread. not know Who has been writing this patch? Please include either forum user names or email addresses. me Patch: http://paste2.org/p/1961898 new System with pointer http://paste2.org/p/1975602 Most target calculation of area spells are unnecessary. Spells with more then one spell effect with the same target mask and maybe an limitation works now right (limitation in function SetTargetMap(...)) Example Spells: 64218 63342 69674 Sry my english is bad, i hope you understand my idea.

take the Database: mangos -> table: creature_template -> column: subname

Take a deep and long look into mangos core code. Use the search functions in your favorite IDE(for OS windows ->visual c++). Good words for search are spellscript, achievement...) Maybe someone other ask the same question, use google for a good search in this forum. The original forum is stupid (30 seconds between searches; stupid solutions). Maybe you should fix some easier thinks, achievements are complex

maybe you can ask a question?

Hi author: me Revision: i think the actual([11927]) https://github.com/Reamer/mangos/commit/388a3ff5bdc21866fe5d6ad43d6abc4e92eadeb0 Now the heal from NPC's, special from shaman wolfs (last talent-spell in melee talent tree) are show in combat log.

IsReachable is already in mmaps. at the moment i have found this call in bool Unit::SelectHostileTarget() // check if currently selected target is reachable // NOTE: path alrteady generated from AttackStart() if(!GetMotionMaster()->operator->()->IsReachable()) {

stacking system need a complete rewrite in mangos Use the search function and search for: aura stacking michalpolko should post a rewrite long time ago

take a look into this commit, this should help^^ https://github.com/Reamer/mangos/commit/762edc95bc93895c3a10135bc1250e8aae144945