Maxxie

You're right, I'm sorry it should be placed in HandleAddPctModifierAuraProc not HandleDummyAuraProc. I didn't update to latest repo so on my server it works with HandleDummyAuraProc, if you updated you should use HandleAddPctModifierAuraProc.

I'm sorry, I didn't mean to discredit your fix or your skill in any way, but as you stated "MaNGOS is an educational project" so I wanted to show you another way to fix this which I _think_ is simpler. Anyway to explain this better: Unit::HandleDummyAuraProc is not triggered just by dummy auras, it is triggered by all of these auras (not sure all of them, i might have added a couple in my repo): SPELL_AURA_IGNORE_REQUIREMENTS SPELL_AURA_MOD_DAMAGE_PERCENT_TAKEN SPELL_AURA_MANA_SHIELD SPELL_AURA_OBS_MOD_MANA SPELL_AURA_ADD_PCT_MODIFIER (this one is the one that is used to trigger the serendipity removal) SPELL_AURA_DUMMY EDIT: To make it work you have to edit spell_proc_event: INSERT INTO spell_proc_event (`entry`, `SchoolMask`, `SpellFamilyName`, `SpellFamilyMask0`, `SpellFamilyMask1`, `SpellFamilyMask2`, `procFlags`, `procEx`, `ppmRate`, `CustomChance`, `Cooldown`) VALUES (63735, 0, 6, 4608, 0, 0, 0, 0, 0, 100, 0); EDIT2: Swiftmend is handled there because it has to modify m_healing which can't be done from proc, so I think it should be ok.

I think this should be handled in Unit::HandleDummyAuraProc In the SpellIconID switch of SPELLFAMILY_PRIEST: case 2900: { RemoveAurasDueToSpell(triggeredByAura->GetId()); return true; } and then just adapt sql part in spell_proc_event (Sorry for being so vague, but i can't make a diff right now )

Hi there, there is a simpler way to fix this. You should first see how it works currently. Looking at wowhead you can see that the triggering part of spell 47422 is already there, so all you should do is just tell mangos when it should trigger. To do this you'll have to look in the mangos database in the table spell_proc_event looking for the entry of Everlasting Affliction (Rank 1) and edit it so it triggers when you cast Shadow Bolt or Drain Soul. To do this you need to modify SpellFamilyMask0 from 8 to 16393 (which you obtain by adding SpellFamilyFlags0 value of the spells you need, in this case 1 from Shadow Bolt and 16384 from Drain Soul). The other effect of the spell does not seem to work correctly because it uses aura SPELL_AURA_ADD_FLAT_MODIFIER instead of SPELL_AURA_ADD_PCT_MODIFIER so damage is increased by 5 instead of 5%. Hope everything is clear

Ok, solved. Looks like saving conf using ANSI instead of UTF-8 lets mangosd start.

Confirmed on clean core WinXP sp3

Your update has a typo: This code should be in MapManager::Initialize instad of MapManager::_createBaseMap. This crashes mangos when creating a map after the first because DelayExecutor is already active so abort() is called.

This patch should fix Unit::CalculateLevelPenalty to use the downranking formula used in WotLK. Which should be: penalty = (22 + max level - caster level) / 20. <source> Patch: http://pastebin.org/299716

What bug does the patch fix? What features does the patch add? From patch 3.0.2 Blessed Recovery should stack when hit by additional critical hits. http://pastebin.org/297671

The healing part should be affected by other auras (like Drain Life with Demon Armor). It should not benefit from the spellpower in the spell_bonus_data so we should skip it calling Unit::SpellHealingBonusTaken where it checks for damagetype. We could try this patch

Probably spell system does not check for spellcategory in items when adding cooldowns, so since PvP Trinket has no category but instead it is stored in item_template its cooldown is not triggered. What I would do to handle this is to check for spellcategory in all items the player has in the inventory in Player::AddSpellAndCategoryCooldowns... I'll test if this is the case and write a patch as soon as I get my pc back.

Probably this spell needs to be added to the precasts for WotF since it triggers a cooldown for category 1182 (while WotF has category 1166). Can't test this right now though.

If this is the same problem you should check for item_instance, anyway try to check the log for any kind of error related to characters since it could be something different...

I once had this problem, it was caused by latency (i guess) that caused mangos to find duplicate item guids when it had to save characters to the db. Try to check the database error log, maybe it's the same issue...

In patch 3.3.2 spell 32727 has only aura interrupt flag AURA_INTERRUPT_FLAG_CHANGE_MAP.

Seems you're right, many invisibility/stealth/disguise buffs have this interrupt flag. Even imp's phase shift has this...

Try replacing that line with this one: if(!pPlayer->HasAura(SPELL_ID_PASSIVE_RESURRECTION_SICKNESS,EFFECT_INDEX_0)) EDIT: The change has been made in 9402.

I don't think that's the problem since WorldSession::GetPlayerName() is defined like this: char const* WorldSession::GetPlayerName() const { return GetPlayer() ? GetPlayer()->GetName() : "<none>"; } If I'm not mistaken the problem is the m_session that is null for the console. Try this: diff --git a/src/game/Level1.cpp b/src/game/Level1.cpp index b2529ae..2263f7d 100644 --- a/src/game/Level1.cpp +++ b/src/game/Level1.cpp @@ -128,7 +128,7 @@ bool ChatHandler::HandleAnnounceCommand(const char* args) if(!*args) return false; - sWorld.SendWorldText(LANG_SYSTEMMESSAGE,args); + sWorld.SendWorldText(LANG_SYSTEMMESSAGE, m_session ? m_session->GetPlayerName() : "CONSOLE", args); return true; }

What i meant is that maybe Spell::SendCastResult needs an exception for SPELL_FAILED_ONLY_SHAPESHIFT (like there is for SPELL_FAILED_REQUIRES_SPELL_FOCUS and others) , that tells the client which stance the player should be in, so that the error can be shown correctly.

Updated version: http://pastebin.com/f17181541 The only thing i don't understand is how to make SPELL_FAILED_ONLY_SHAPESHIFT tell the client the required form, ofc if there's any way to do this. (right now cast isn't allowed but error isnt' shown).

Here it is: http://pastebin.com/f12b3ef55 Maybe it's not the best solution, but it solves the problem pointed out by Tassader. As i understand this, on offy this spell can be cast while not in cat form and isn't removed if you shapeshift out of it but speed bonus is applied only if you are in cat form.

As far as I know exp is blocked by client at level 100... I'm not entirely sure about this, but I remember some post on this forum mentioning it

It's not just the shadowform, the problem comes with all auras that modify damage (for example shadow weaving or even berserking in battlegrounds). The only way I've seen to be effective to make these spells deal the right damage is to not calculate damage in Unit::SpellDamageBonus for those spells which damage is calculated from dots damage.

It's been fixed in 9191.

What bug does the patch fix? What features does the patch add? It allows Priest spell Renew to stack with Priest's version of Gift of the Naaru. For which repository revision was the patch created? 9183 diff --git a/src/game/SpellMgr.cpp b/src/game/SpellMgr.cpp index 74cc8f1..a791b7c 100644 --- a/src/game/SpellMgr.cpp +++ b/src/game/SpellMgr.cpp @@ -1559,6 +1559,10 @@ bool SpellMgr::IsNoStackSpellDueToSpell(uint32 spellId_1, uint32 spellId_2) cons if ((spellInfo_1->Id == 47585 && spellInfo_2->Id == 60069) || (spellInfo_2->Id == 47585 && spellInfo_1->Id == 60069)) return false; + // Renew and Gift of the Naaru + if ((spellInfo_1->SpellFamilyFlags & UI64LIT(0x40) && spellInfo_2->SpellIconID == 329) || + (spellInfo_2->SpellFamilyFlags & UI64LIT(0x40) && spellInfo_1->SpellIconID == 329)) + return false; } break; case SPELLFAMILY_DRUID: