Jump to content

laise

Members
  • Posts

    344
  • Joined

  • Last visited

    Never
  • Donations

    0.00 GBP 

Posts posted by laise

  1. ok how u expect code to be in db? i mean srsly u cant put c++ scripts there... I tried once to group few similar spells (Spell::SelectMountByAreaAndSkill) but each spell still needed it's own special support - in this case related spell ids for different mounts..maybe spells need to have their own scripts in some way but i don't think it is related in any way to DB

    I also believe that MaNGOS devs started this way (hardcoded things I mean),

    hardcoded is done only when there is no other visible option to fix something..

    and today they don't want to "lose" time by modifying something that works well for everyone.

    really? why not look at commit log first?

  2. i think it should be proc-based, although atm we call it after actual damage deal - maybe need to add option for procs on hit but before damage deal and procs after..

    also i see it checks for (!damage) after your code with "damage *= " maybe u need to check damage first if it is initialized?

  3. It's a good job, but there is a visual bug.

    When we use spell with stacked auras like lifebloom, the number of stack begin by 1, then we have 3, then 4 stacks.

    It must be 1 then 2 and 3.

    But after many tests, I concluded that is only a visual bug.

    I didn't find the packet that send this data.

    I looked the RefreshAuraDuration, but it seems to be in another place.

    just tested with lifebloom and lacerate - not happening .. any other spells?

  4. As I understand in this case holder would contain only one aura so we can increase duration of aura holder and it will affect duration of all auras in it.

    err ..kind of contradicting if you say "will contain one aura " and then "all auras in it".. duration changes are not connected in any way on how much auras are on holder in case when all auras suppose to have same duration, so for this spell will need to reset to full duration and add new duration for each aura.. ofc maybe i will add method for calling duration change from holder but it is wrong to move duration out of auras

  5. Laise, what do you think about moving duration and related stuff to aura holder? Do you have an example when two auras from same spell have not equal durations?

    yes, this is related to spells with stun/silence effect that have also another aura (for example periodic dmg) , according to my info from retail, existing talents that reduce duration of stun/silence/whatever effect are applied only to specific aura and not on whole spell, creating situation where stun/silence/whatever fades and periodic remains as it was..I don't remember exact spell ids for that but they do exist

×
×
  • 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