WoWInterface - Updated Cooldowns (Engine/Feature)

WoWInterface (https://www.wowinterface.com/forums/index.php)

- OpenRDX: Feature Requests (https://www.wowinterface.com/forums/forumdisplay.php?f=104)

- - Updated Cooldowns (Engine/Feature) (https://www.wowinterface.com/forums/showthread.php?t=41727)

Updated Cooldowns (Engine/Feature)

In \RDX\RosterMgr\Cooldown\Core.lua it reads:

Quote:

-- A cooldown database is created using the registererCoodldown(spellid, duration, group) function
-- -> the duration is not really accurate, because some duration could change with talent + glyph
-- A local engine will store real cooldown per character (duration + timeleft)
-- this local engine is sync with all members of the raid

If we use GetSpellCooldown, though, we can find out the duration left on any (personal) ability. I'm not too sure if it works outside of just the player uid.

Using this, wouldn't it be possible to change the cooldown variable in \RDX\Designs\Variables\CooldownVariables.lua to refelct something like this?

Code:

local name = GetSpellInfo(12043);

local start, duration, enabled = GetSpellCooldown(name);

if enabled == 0 then

        DEFAULT_CHAT_FRAME:AddMessage(.. name .. " is currently active, use it and wait " .. duration .. " seconds for the next one.");

elseif ( start > 0 and duration > 1.5) then

        DEFAULT_CHAT_FRAME:AddMessage(.. name .. " is cooling down, wait " .. (start + duration - GetTime()) .. " seconds for the next one.");

else

        DEFAULT_CHAT_FRAME:AddMessage(.. name .. " is ready.");

end

How \RDX\RosterMgr\Cooldown\Core.lua could possibly be changed:

Code:

function RDXCD.RegisterCooldown(race, boss, class, talent, spellid, dduration, group, event)

        if not spellid then VFL.print(VFLI.i18n("|cFFFF0000[RDX]|r Info : Attempt to register an anonymous omni cooldown.")); return; end

        if cd[spellid] then VFL.print(VFLI.i18n("|cFFFF0000[RDX]|r Info : Attempt to register duplicate object type ") .. spellid .. "."); return; end 

        local spellname, _, icon = GetSpellInfo(spellid);

        if not spellname then VFL.print(VFLI.i18n("|cFFFF0000[RDX]|r Info : unknown spellid ") .. spellid .. "."); return; end

        local start, duration, enabled = GetSpellCooldown(spellname);

        if not dduration then dduration = duration; end

        -- Duration Defined:  RDXCD.RegisterCooldown(nil, nil, "PALADIN", nil, 85673, 20, nil, "SPELL_CAST_SUCCESS");

        -- Duration Default:  RDXCD.RegisterCooldown(nil, nil, "PALADIN", nil, 85673, nil, nil, "SPELL_CAST_SUCCESS");

        local text = "";

        if race then text = text .. race .. ":"; end

        if boss then text = text .. boss .. ":"; end

        if class then text = text .. class .. ":"; end

        if talent then text = text .. talent .. ":"; end

        text = text .. spellname;

        local cdtemp = {

                text = text;

                race = race;

                class = class;

                talent = talent;

                spellid = spellid;

                spellname = spellname;

                start = start;

                duration = dduration;

                enabled = enabled;

                icon = icon;

                group = group;

        };

        cd[spellid] = cdtemp;

        if boss then cdb[spellid] = cdtemp; end

        if talent then cdt[spellid] = cdtemp; end

        if not talent and not boss then cdc[spellid] = cdtemp; end

        if event ==  "SPELL_CAST_SUCCESS" then cdcastspell[spellid] = true;

        elseif event ==  "SPELL_DAMAGE" then cddmgspell[spellid] = true;

        elseif event ==  "SPELL_HEAL" then cdhealspell[spellid] = true;

        elseif event ==  "SPELL_AURA_APPLIED" then cdauraAspell[spellid] = true;

        elseif event ==  "SPELL_AURA_REMOVED" then cdauraRspell[spellid] = true;

        elseif event ==  "SPELL_RESURRECT" then cdresuspell[spellid] = true;

        elseif event ==  "SPELL_SUMMON" then cdsumspell[spellid] = true;

        end

        return true;

end

I realize that the "duration defined" bit is not correct LUA, it was just there as an example: it's pretty easy to realize what it means, but just in case:

If a time is defined on a cooldown, ie:

Code:

RDXCD.RegisterCooldown(nil, nil, "PALADIN", nil, 85673, 20, nil, "SPELL_CAST_SUCCESS");

Then that time will be used instead.

If a time is NOT defined on a cooldown, ie:

Code:

RDXCD.RegisterCooldown(nil, nil, "PALADIN", nil, 85673, nil, nil, "SPELL_CAST_SUCCESS");

Then the duration from:

Code:

local start, duration, enabled = GetSpellCooldown(spellname);

Should be used instead.

bump.

rump a dump chump.

Hi Unlimit

there is a problem.

local start, duration, enabled = GetSpellCooldown(spellname);

GetSpellCooldown only work when the cooldown is used. it returns nil if your cooldown is available.

So to fix that problem, we have the register cooldowDB and the dynamic cooldownDB.

The register cooldownDB is the one you want to modify.
The dynamic cooldownD is based on GetSpellCooldown.

Open the file C:\Program Files (x86)\World of Warcraft\WTF\Account\XXXX\SavedVariables\RDX.lua and search for the cooldowndb.

RDX actionbars store the cooldowns per character in this file. (That mean, if you don't use the actionbars, it won't work)

The engine cooldown always get the duration from the dynamic cooldowndb before the register cooldowndb for player and pet.

For enemies, boss, the cooldown will always used the default duration of the register cooldowndb.

For members of your raid, there is a synchronisation of long cooldown (superior to 3 minutes) using communication stuff.

Or cooldown engine is simply the best.

:)

Quote:

The engine cooldown always get the duration from the dynamic cooldowndb before the register cooldowndb for player and pet.

I've NEVER seen this happen.

Good example: in retribution I have a talent that decreases my Crusader Strikes cooldown the more haste that I have.

By default, Crusader Strike is 4.5 seconds, and in RDX Crusader Strike is registered by the CooldownDB for 4.5 seconds.

Crusader Strike is the FIRST thing on my bars in RDX. MY Crusader Strike should be 3.7 seconds. Why does it not show up?

erf

Open the file C:\Program Files (x86)\World of Warcraft\WTF\Account\XXXX\SavedVariables\RDX.lua and search for the cooldowndb of your character.

And try to find if there is your spellid in the cooldowndb and the duration value.

Yes and no.

Crusader Strike is registered to a different realm, which is unlimit_lightningsblade.

I'm on Sargeras, transfered off Lightning's Blade like a year ago. Then went to Demon Soul, it didn't create a new one for Demon Soul, either. Now I'm on Sargeras. <_<

Shouldn't it automatically create a new Cooldowndb for a new realm, if I'm using actionbars there too?

I deleted my the RDX saved variable to try it out, and it certianly works... it just seems that it should work automatically and per-character, instead of me having to take that step.

I'm not trying to be whiny, just trying to help ya know? There are TONS of addons that I can imagine having more functionality in this specific field than RDX does, like CLCinfo or ForteXorcist. Even if our cooldown engine is (don't know if it is or not) superior, we're not putting it into practice as well as it should be.

For instance, I've NEVER been able to get the cooldown Cooldown Type (AVAIL) option to work in Vars: Cooldown.

Personally, I believe we should have customization there, for it being both available and used, like for it to automatically color the icon texture for us:

That way, if Use texture variable is checked:

We can use certain colors to help us, or make it completely disappear.

This way the original functionality also stays intact, if you so want, you could always make the "available" color completely transparent.