PQR - Rotation Bot

[firepong]

Is it worth for enchancement shaman to keep UE cd to wait until FS is going to drop off? or use UE every cd, and let cast FS without buff from UE? iam asking cuz thinking and trying to write to be cast always FS with buff from UE and UE every time when timer is higher then cd of UE.
NVM
only question is: is it worth to keep UE

I have yet to check how it is after 5.0 hit, but before, it was somewhat beneficial. Would want to do something like if flamedebuff < 5 and flameshockCD < 5 then return false end. Then again, if flamedebuff < 5 and flameshockCD == 0 then return true end for UE, then do a if UEBuff then return true end for Flame Shock. Something along those lines.

[ocso639]

Anyone know if there is a prot warrior rotation?

[somenub0608]

Anyone know if there is a prot warrior rotation?

There is on the very first page of this thread.

[Ninjaderp]

Anyone know if there is a prot warrior rotation?

How hard is it to look around a little before asking? Really? This link has been posted TWICE on this page, which takes you to a thread with gathered profiles. -_-

http://www.ownedcore.com/forums/worl...9012012-a.html ([PQR] List of Community Rotations - Updated 09012012)

[SHJordan]

I have yet to check how it is after 5.0 hit, but before, it was somewhat beneficial. Would want to do something like if flamedebuff < 5 and flameshockCD < 5 then return false end. Then again, if flamedebuff < 5 and flameshockCD == 0 then return true end for UE, then do a if UEBuff then return true end for Flame Shock. Something along those lines.

Hey bro, are you still updating the svn?

[Clotic]

LF a follow target "interrupt"

[kickmydog]

I've tried two different code variations but none of them seem to work with healthstones atm.

local myhp = UnitHealth("player") / UnitHealthMax("player") * 100

if myhp < 40 then RunMacroText("/use Healthstone")
return true end 

or

local myhp = UnitHealth("player") / UnitHealthMax("player") * 100

if myhp < 40 then return true end 

spell id 5512

any ideas on this?

[sheuron]

SpellIDs, 'morphed SpellNames' and whatever it's been called overtime by users of PQR.

It's as simple as using this function fawGetGlobalSpellID(spell) in conjuction with, uhmm whats it called in PQR?? I don't use PQR myself.

CastSpellByName(fawGetGlobalSpellID(spell))

spell beeing the NAME of the spell you want to cast.


--------------------------------------------------


-- Courtesy of Wopak - http://www.ownedcore.com/forums/memb...249-wopak.html
-- Is the code hard to understand, well tough luck, just use it.
-- This function does not require any PLUA unlocker or any other kind of hack
-- Get the Global SpellID of a Spell, also checks Pets spellbook (if the Player has a Pet)
-- Signature
---- globalSpellID = fawGetGlobalSpellID("spell")
-- Arguments:
---- spell - Name of the Spell to Query
-- Returns:
---- globalSpellID: Global SpellID of the Spell (numeric); nil if the spell is unknown (numeric;nil)

function fawGetGlobalSpellID(spell)
    if spell == nil then return 0; end
    local globalSpellID = nil;
    local spellIndex, bookType = fawGetSpellIndex(spell);

    if spellIndex and bookType then
        globalSpellID = select(2, GetSpellBookItemInfo(spellIndex, bookType));
    end

    return globalSpellID;
end 

-- Courtesy of Wopak - http://www.ownedcore.com/forums/memb...249-wopak.html
-- Is the code hard to understand, well tough luck, just use it.
-- This function does not require any PLUA unlocker or any other kind of hack
-- Get the Global SpellID of a Spell, also checks Pets spellbook (if the Player has a Pet)
-- Signature
---- spellIndex, bookType = fawGetSpellIndex("spell")
-- Arguments:
---- spell - Name of the Spell to Query
-- Returns:
---- spellIndex: The index of the spell in the Player or Pet spellbook. nil if the spell is unknown (numeric;nil)
---- bookType: "spell" if it's a player spell, "pet" if it's a pet spell, nil if the spell is unknown (string;nil)

function fawGetSpellIndex(spell)
    if spell == nil then return 0; end
    local myIndex = nil;
    local myBook = nil;

    -- Looping thru the Spellbook and trying to match the Spell name passed
    -- to the function with the Spells name in the Spellbook.
    -- If there's a match, get the GLOBAL SpellID (not to be confused with the SpellID)
    -- using the info returned when quering the Index in te Spellbook.
    local _, _, offset, numSpells, _, _ = GetSpellTabInfo(2);
    for thisIndex = offset+1, numSpells+offset do
        -- Get the Global Spell ID from the Player's spellbook
        if spell == GetSpellBookItemName(thisIndex, "spell") then
            myIndex = thisIndex;
            myBook = "spell";
            break; -- Breaking out of the for/do loop, because we have a match
        end
    end

    -- If the Spell was not found, check if there is a Pet Spellbook
    -- If a Pet Spellbook is found, do the same as above
    local numPetSpells = HasPetSpells();
    if not myIndex and numPetSpells then
        for thisIndex = 1, numPetSpells do
            -- Get the Global Spell ID from the Pet's spellbook
            if spell == GetSpellBookItemName(thisIndex, "pet") then
                myIndex = thisIndex;
                myBook = "pet";
                break; -- Breaking out of the for/do loop, because we have a match
            end
        end
    end

    return myIndex, myBook;
end 

CastSpellByName(GetSpellInfo(ID),nil) works with any spell

If you want to check which spells are on your spellbook you can use FindSpellBookSlotBySpellID(ID), but is useless, because trying to cast new spells using his root spellid eventualy freeze the UI.

[Wopak]

CastSpellByName(GetSpellInfo(ID),nil) works with any spell

If you want to check which spells are on your spellbook you can use FindSpellBookSlotBySpellID(ID), but is useless, because trying to cast new spells using his root spellid eventualy freeze the UI.

I don't quite think you got the idea here...

This is for just passing the Spells name as you see it in game, and no, it doesn't freeze the UI, that must be something else interfeering, i have multiple users using my addon and this causes no problem.

PS: When you say root speel id, i assume you mean global SpellID ?

[kickmydog]

I don't quite think you got the idea here...

This is for just passing the Spells name as you see it in game, and no, it doesn't freeze the UI, that must be something else interfeering, i have multiple users using my addon and this causes no problem.

PS: When you say root speel id, i assume you mean global SpellID ?

I'm not sure I see the point of using this when as Sheuron states

CastSpellByName(GetSpellInfo(ID),nil)

works fine. Also using spellid is better than using the name because then you don't have any language localization issues.

[averykey]

Repost from last page, anyone know why this isn't working? rep+

local hasSeal = false
local judgedTarget = false

-- Seal Check
if GetShapeshiftForm() ~= 0 then hasSeal = true else hasSeal = false end

-- double jeporady
if hasSeal then
if judgedTarget == false then
if UnitExists("target") and UnitPower("player", 9) < 5 then
        CastSpellByName(GetSpellInfo(20271), "target")
        judgedTarget = true
    end
end

else

if judgedTarget == true then
    if UnitExists("focus") and UnitPower("player", 9) < 5 then
            CastSpellByName(GetSpellInfo(20271), "focus")
            judgedTarget = false
            end
        end
    end 

[sheuron]

I don't quite think you got the idea here...

This is for just passing the Spells name as you see it in game, and no, it doesn't freeze the UI, that must be something else interfeering, i have multiple users using my addon and this causes no problem.

PS: When you say root speel id, i assume you mean global SpellID ?

CastSpellByName("spell name") uses spells names as you see in game, no need additional code for that.

What freeze the game is trying to cast one of new spells using CastSpellByID.

To see root spells try the script i post here
http://www.ownedcore.com/forums/worl...ml#post2250100 (PQR - Rotation Bot)
root spells are shown on green.

[firepong]

Hey bro, are you still updating the svn?

Yeah, it's still getting updated. The only profile I have right now that's fully updated is my Feral Druid profile. As of right now, I just don't have the time to update all my others because of work and Shaman is on the back burner for the time being till I see how DPS is working out at level 90.

Can probably say the next profiles I'll get working though is my leveling profiles for whenever HonorBuddy get's fully coded and working good.

[bu_ba_911]

Repost from last page, anyone know why this isn't working? rep+

local hasSeal = false
local judgedTarget = false

-- Seal Check
if GetShapeshiftForm() ~= 0 then hasSeal = true else hasSeal = false end

-- double jeporady
if hasSeal then
if judgedTarget == false then
if UnitExists("target") and UnitPower("player", 9) < 5 then
        CastSpellByName(GetSpellInfo(20271), "target")
        judgedTarget = true
    end
end

else

if judgedTarget == true then
    if UnitExists("focus") and UnitPower("player", 9) < 5 then
            CastSpellByName(GetSpellInfo(20271), "focus")
            judgedTarget = false
            end
        end
    end 

--local judgedTarget = false -- This will always be false as it is written
if judgedTarget == nil then judgedTarget = false end
local hasSeal = false

-- Seal Check
if GetShapeshiftForm() ~= 0 then hasSeal = true end

-- double jeporady
if hasSeal then
    if judgedTarget == false then
        if UnitExists("target") and UnitPower("player", 9) < 5 then
            CastSpellByName(GetSpellInfo(20271), "target")
            if UnitExists("focus") then
                judgedTarget = true
            end
        end
    elseif judgedTarget == true then
        if UnitExists("focus") and UnitPower("player", 9) < 5 then
            CastSpellByName(GetSpellInfo(20271), "focus")
            judgedTarget = false
        end
    end
end 

also in your code you posted, you had an extra end in there

after second thought....

--local judgedTarget = false -- This will always be false as it is written
if judgedTarget == nil then judgedTarget = false ResetTimer = GetTime() end
local hasSeal = false


function RangeCheck(spellid, t)
    if IsSpellInRange(GetSpellInfo(spellid), t) == 1 then
        return true
    end
    
    return false
end

-- Seal Check
if GetShapeshiftForm() ~= 0 then hasSeal = true else hasSeal = false end

if judgedTarget == true then
    if GetTime() - ResetTimer >= 12 then
        judgedTarget = false
    elseif not UnitExists("focus") then
        judgedTarget = false
    end
end

-- double jeporady
if hasSeal then
    if judgedTarget == false then
        if UnitExists("target") and UnitPower("player", 9) < 5 and RangeCheck(20271, "target") then
            CastSpellByName(GetSpellInfo(20271), "target")
            if UnitExists("focus") and RangeCheck(20271, "focus") then
                ResetTimer = GetTime()
                judgedTarget = true
            end
        end
    elseif judgedTarget == true then
        if UnitExists("focus") and UnitPower("player", 9) < 5 and RangeCheck(20271, "focus") then
            CastSpellByName(GetSpellInfo(20271), "focus")
            judgedTarget = false
        end
    end
end 

[Wopak]

I'm not sure I see the point of using this when as Sheuron states

CastSpellByName(GetSpellInfo(ID),nil)

works fine. Also using spellid is better than using the name because then you don't have any language localization issues.

Ye, that's amazing

CastSpellByName("spell name") uses spells names as you see in game, no need additional code for that.

What freeze the game is trying to cast one of new spells using CastSpellByID.

To see root spells try the script i post here
http://www.ownedcore.com/forums/worl...ml#post2250100 (PQR - Rotation Bot)
root spells are shown on green.

Nevermind, to advanced