C_Garrison.GetFollowers(2)

Endzeit · 11-28-15, 06:10 AM

Hello,

i want to track all my Ships with classNames. Everytime i go the Shipyard i want a print in chat like:
# Transporter: 1
# Destroyer: 2
# Total: 3/10

Lua Code:

GossipFrame:SetScript( "OnShow", function()
if UnitName("target") == "Solog Roark" then
    local destroyer = 0 -- id59
    local schlachtschiff = 0 --id56
    local uboot = 0 --id62
    local traeger = 0 --id60
    local transporter = 0 --id58
    local ships = C_Garrison.GetFollowers(2)
        for k,_ in ipairs(ships) do
     if ships[k].classSpec == 59 then destroyer = destroyer + 1; end
    if ships[k].classSpec == 56 then schlachtschiff = schlachtschiff + 1; end
    if ships[k].classSpec == 62 then uboot = uboot + 1; end
    if ships[k].classSpec == 60 then traeger = traeger + 1; end
    if ships[k].classSpec == 58 then transporter = transporter + 1; end
        end
  print("Destroyer: " .. destroyer)
  print("Schlachtschiff: " .. schlachtschiff)
    print("uboot: " .. uboot)
    print("traeger: " .. traeger)
        print("transporter: " .. transporter)
end
end)

Ship Builder is working.

Shipyard Table not:

Error:

Code:

Message: Interface\AddOns\Endzeit\modules\ships.lua:26: attempt to index global 'GarrisonShipyardFrame' (a nil value)
Time: 11/28/15 15:09:11
Count: 1
Stack: Interface\AddOns\Endzeit\modules\ships.lua:26: in main chunk

Locals: (*temporary) = nil
(*temporary) = nil
(*temporary) = nil
(*temporary) = nil
(*temporary) = "attempt to index global 'GarrisonShipyardFrame' (a nil value)"

Code:

Lua Code:

GarrisonShipyardFrame:SetScript( "OnShow", function()
    local destroyer = 0 -- id59
    local schlachtschiff = 0 --id56
    local uboot = 0 --id62
    local traeger = 0 --id60
    local transporter = 0 --id58
    local ships = C_Garrison.GetFollowers(2)
        for k,_ in ipairs(ships) do
     if ships[k].classSpec == 59 then destroyer = destroyer + 1; end
    if ships[k].classSpec == 56 then schlachtschiff = schlachtschiff + 1; end
    if ships[k].classSpec == 62 then uboot = uboot + 1; end
    if ships[k].classSpec == 60 then traeger = traeger + 1; end
    if ships[k].classSpec == 58 then transporter = transporter + 1; end
        end
  print("Destroyer: " .. destroyer)
  print("Schlachtschiff: " .. schlachtschiff)
    print("uboot: " .. uboot)
    print("traeger: " .. traeger)
        print("transporter: " .. transporter)
end)

Torhal · 11-28-15, 12:44 PM

The GarrisonShipyardFrame doesn't exist until the Blizzard_GarrisonUI is loaded, so your AddOn should have this in its ToC file:

Code:

## LoadOnDemand: 1
## LoadsWith: Blizzard_GarrisonUI

Additionally, your code's formatting makes it difficult to understand and you should be using an if-elseif chain so you aren't checking for matches after you've already found one. This isn't exactly how I'd write it, but since I just woke up here's a slightly cleaned up version:

lua Code:

GossipFrame:SetScript( "OnShow", function()
    if UnitName("target") == "Solog Roark" then
        local destroyer = 0 -- id59
        local schlachtschiff = 0 --id56
        local uboot = 0 --id62
        local traeger = 0 --id60
        local transporter = 0 --id58
        local ships = C_Garrison.GetFollowers(2)
 
        for k,_ in ipairs(ships) do
            local classSpec = ships[k].classSpec
 
            if classSpec == 59 then
                destroyer = destroyer + 1
            elseif classSpec == 56 then
                schlachtschiff = schlachtschiff + 1
            elseif classSpec == 62 then
                uboot = uboot + 1
            elseif classSpec == 60 then
                traeger = traeger + 1
            elseif classSpec == 58 then
                transporter = transporter + 1
            end
        end
 
        print("Destroyer: " .. destroyer)
        print("Schlachtschiff: " .. schlachtschiff)
        print("uboot: " .. uboot)
        print("traeger: " .. traeger)
        print("transporter: " .. transporter)
    end
end)

Endzeit · 11-28-15, 04:46 PM

i change some code and try to formate better.. this is working so far. There is no event for Opening Garrison Table or?

Lua Code:

function CheckShipyard(self)
    local Destroyer = 0 -- id59
    local Battleship = 0 --id56
    local Submarine = 0 --id62
    local Carrier = 0 --id60
    local Transport = 0 --id58
    local ships = C_Garrison.GetFollowers(2)
    local _,_,_,_,_,rank = C_Garrison.GetOwnedBuildingInfo(98)
    local maxfollower = 0
    local activefollower = C_Garrison.GetNumFollowers(2)
    
    if rank == 1 then 
        maxfollower = 6
    elseif rank == 2 then
        maxfollower = 8
    elseif rank == 3 then
        maxfollower = 10 
    end
    
        for k,_ in ipairs(ships) do
        local classSpec = ships[k].classSpec
            if classSpec == 58 then 
                Transport = Transport + 1
            elseif classSpec == 59 then 
                Destroyer = Destroyer + 1
            elseif classSpec == 56 then 
                Battleship = Battleship + 1
            elseif classSpec == 62 then 
                Submarine = Submarine + 1
            elseif classSpec == 60 then 
                Carrier = Carrier + 1
            end
        end
        
    local color = { 
    { r=1, g=0, b=0,}, --red 
    { r=1, g=1, b=0,}, --yellow
    { r=0, g=1, b=0,}, --green
    { r=0, g=1, b=0,}, --green
    { r=0, g=1, b=0,}, --green
    { r=0, g=1, b=0,}, --green
    }
        DEFAULT_CHAT_FRAME:AddMessage(" ");
        DEFAULT_CHAT_FRAME:AddMessage("# |TInterface\\ICONS\\inv_garrison_cargoship.blp:12|t " ..           "Transport: " .. Transport, color[Transport+1].r, color[Transport+1].g, color[Transport+1].b);
        DEFAULT_CHAT_FRAME:AddMessage("# |TInterface\\ICONS\\ships_ability_depthcharges.blp:12|t " ..       "Destroyer: " .. Destroyer, color[Destroyer+1].r, color[Destroyer+1].g, color[Destroyer+1].b);
        DEFAULT_CHAT_FRAME:AddMessage("# |TInterface\\ICONS\\ships_ability_armorpiercingammo.blp:12|t " ..  "Battleship: " .. Battleship, color[Battleship+1].r, color[Battleship+1].g, color[Battleship+1].b);
        DEFAULT_CHAT_FRAME:AddMessage("# |TInterface\\ICONS\\ships_ability_stealth.blp:12|t " ..            "Submarine: " .. Submarine, color[Submarine+1].r, color[Submarine+1].g, color[Submarine+1].b);
        DEFAULT_CHAT_FRAME:AddMessage("# |TInterface\\ICONS\\ships_ability_bombers.blp:12|t " ..            "Carrier: " .. Carrier, color[Carrier+1].r, color[Carrier+1].g, color[Carrier+1].b);
        if activefollower < maxfollower then
        DEFAULT_CHAT_FRAME:AddMessage("# |TInterface\\DialogFrame\\UI-Dialog-Icon-AlertNew.png:12|t Total: " .. activefollower .. "/" .. maxfollower, 1, 0, 0);
        else
        DEFAULT_CHAT_FRAME:AddMessage("# |TInterface\\RAIDFRAME\\ReadyCheck-Ready.png:12|t Total: " .. activefollower .. "/" .. maxfollower, 0, 1, 0);
        end
        DEFAULT_CHAT_FRAME:AddMessage(" ");     
end
 
local a = CreateFrame( "Frame" )
a:RegisterEvent("ZONE_CHANGED_NEW_AREA")
a:RegisterEvent("GOSSIP_SHOW")
a:RegisterEvent("PLAYER_ENTERING_WORLD")
a:RegisterEvent("GARRISON_FOLLOWER_ADDED")
a:RegisterEvent("GARRISON_FOLLOWER_REMOVED")
a:SetScript( "OnEvent", function( self, event )
local areaid = GetCurrentMapAreaID();
if ((event == "ZONE_CHANGED_NEW_AREA") or (event == "PLAYER_ENTERING_WORLD")) and ((areaid == 976) or (areaid == 941)) then
    CheckShipyard()
end
 
if (event == "GOSSIP_SHOW") and UnitName("target") == "Solog Roark" then
    CheckShipyard()
end
 
if (event == "GARRISON_FOLLOWER_ADDED")  then
    CheckShipyard()
end
if (event == "GARRISON_FOLLOWER_REMOVED") then
    CheckShipyard()
end
end)

my color table is not perfekt.. but i dont know how it makes better.
With 0 ships = red
With 1 ships = yellow
with 2+ ships = green