Jump to content

VladimirMangos

Members
  • Posts

    2813
  • Joined

  • Last visited

    Never
  • Donations

    0.00 GBP 

Everything posted by VladimirMangos

  1. In last version for well fed priest case must be: Typo as i think.[added]Just for note: original check has been right and this my missunderstanding and misscheck.
  2. Just for note: this patch is 100% stable so main test point not in stability but in strange stacking problems not existed before this patch. Base at patch changes if problems exist it will be in unexpected stacking for spells that before propertly rejected.
  3. for mangos-0.12 only table creating make sense maybe. In 2.4.3 work another way save position: always seelcted leader position in moment BG registering near BG master.
  4. Why offhand interpreted as _any_ weapon in offhand? Base at spell data this must be _real_ offhand weapon only. And i not see any references that this not like way.
  5. Meaning: must be moved to rejected as outdated?
  6. Fxied in [8379]. Thank you
  7. Spell have aura SPELL_AURA_MOD_IGNORE_SHAPESHIFT that provide removing form req. for spells. So setting form is cliary horriable hack. But set UNIT_STAND_FLAGS_CREE maybe ok.
  8. set in use prevent more generic crash problems by different reasons while aura deleted in first call of boost spell and already lost at return before continue... This must be prevented in any cases. Start work some spells not expected additional result
  9. If this affects of _single_ spell then why not just declarate its in spel_chain and assign ranks and then it will not stascked as same spells ranks from same caster
  10. Main point: already long discussed and exist patch whne spells with same spelicon will by default stacking instead forbid stacking and then this checks will can dropped. So better have explicit show like cases. Yes, posible soon ©
  11. In rewrited form patch added in [8368]. Anyway thank you for patch
  12. Patch not fully correct: It work inly in case talent learn while in monking form, but what about form change?
  13. I think this work now in different way: So not applied this part.
  14. In [8363]. Thank you. I only replace stacking check to spellicon for more cleary show problematic values (for planned fix)
  15. All work for me and then you not apply some slq updates possible. Reported errors not related to patch in any form as noted by ChanF07
  16. In [8342]. Thank you
  17. I think it can be rewrited, just by sql possible in less hack way Look at this: Pet always have passive 53397 [self target] --(at crit by _spec_ ability)-> 53412 [master target] Playe have 53252/53253 --(spell cast 53412 hit triggered)-> 53398 [self target] [added]why in sql 53412 instead 53397... I will attempt implement in suggested way... [added]scriptted spell not have any flags, so maybe your way can be better...
  18. And more correct: creature->isWorldBoss()
  19. With pointed porbem fixes patch added to [8340]. Thank you to all authors and commenters It more not finsihed that suggested in this thread patch (with my fixes). You by self write it refernced thread that it have real problems with affect all victim states.
  20. If this problem then it exist and with master branch spells with aura 118. For example, talent46910. We not apply patches for problems not fixed in master branch. And specially for patch, i not sure for safe side of patch changes.
  21. In [8337]. Thank you
  22. I add key in [8332] abd related backport to 0.12. Thank you! for patch and description problem source As i note alsready: absent primary key for affected table still be user problem: mangos have it.
×
×
  • Create New...

Important Information

We have placed cookies on your device to help make this website better. You can adjust your cookie settings, otherwise we'll assume you're okay to continue. Privacy Policy Terms of Use