Question - Stacking arguments in Lua

[Randyleo]

I am trying to convert the summon food and drink macro to a Lua. The first argument

if not mq.TLO.Me.Invis()

works, stops casting. If I add a check for moving

if not mq.TLO.Me.Invis() and not mq.TLO.Me.Moving()

It ignores the second argument. What am I doing wrong, lol?

I eventually want to add a check to make it only cast when ACTIVE or RESTING.

Thanks in advance.

 

[burdsjm]

if not mq.TLO.Me.Invis() and not mq.TLO.Me.Moving() and mq.TLO.Me.CombatState() ~= 'COMBAT' then

Should work. I'm still learning though.

 

[rouneq]

A quick test: /lua parse if not mq.TLO.Me.Invis() and not mq.TLO.Me.Moving() then print('not active') end printed out the expected result for my character hanging around in the guild hall. Put a script together

---@type Mq
local mq = require('mq')

if not mq.TLO.Me.Invis() and not mq.TLO.Me.Moving() then
    print('not active')
end

and got the same result.

Post your code. Maybe we can offer a better answer.

 

[Randyleo]

This didn't start out as my coding. Originally posted by PolDetson. I am just modifying it for safety checks. I applied the suggestion of burdsjm, and now it works. If I change COMBAT to ACTIVE will it wait until after cooldown to begin summoning again after combat?

-- SummonFood.lua - by PolDetson
-- Modified to Snacks.lua by Randyleo for more safety checks
--
-- This requires the Warm Milk Dispenser & Fresh Cookie Dispenser
--
-- version 1.2
local mq = require('mq')

while true do

  if mq.TLO.FindItemCount(52199)() < 5 then
        if not mq.TLO.Me.Invis() and not mq.TLO.Me.Moving() and mq.TLO.Me.CombatState() ~= 'COMBAT' and mq.TLO.FindItem("Warm Milk Dispenser")() ~= nil and mq.TLO.FindItem("Warm Milk Dispenser").TimerReady() then
      mq.cmd.echo('\agSummoning Milk\ap')
      mq.cmd('/useitem "Warm Milk Dispenser"')
      mq.delay('2s')
      mq.cmd('/autoinventory')
    end
  end

  if mq.TLO.FindItemCount(71980)() < 5 then
    if not mq.TLO.Me.Invis() and not mq.TLO.Me.Moving() and mq.TLO.Me.CombatState() ~= 'COMBAT' and mq.TLO.FindItem("Fresh Cookie Dispenser")() ~= nil and mq.TLO.FindItem("Fresh Cookie Dispenser").TimerReady() then
      mq.cmd.echo('\agSummoning Cookie\ap')
      mq.cmd('/useitem "Fresh Cookie Dispenser"')
      mq.delay('2s')
      mq.cmd('/autoinventory')
    end
  end

  mq.delay('8s')
end;

 

[brainiac]

Don’t write mq.cmd.echo. Just use print.

 

[Randyleo]

Made suggested change.

-- SummonFood.lua - by PolDetson
-- Modified to Snacks.lua by Randyleo for more safety checks
--
-- This requires the Warm Milk Dispenser & Fresh Cookie Dispenser
--
-- version 1.2
local mq = require('mq')

while true do

  if mq.TLO.FindItemCount(52199)() < 5 then
        if not mq.TLO.Me.Invis() and not mq.TLO.Me.Moving() and mq.TLO.Me.CombatState() ~= 'COMBAT' and mq.TLO.FindItem("Warm Milk Dispenser")() ~= nil and mq.TLO.FindItem("Warm Milk Dispenser").TimerReady() then
      print('\agSummoning Milk\ap')
      mq.cmd('/useitem "Warm Milk Dispenser"')
      mq.delay('2s')
      mq.cmd('/autoinventory')
    end
  end

  if mq.TLO.FindItemCount(71980)() < 5 then
    if not mq.TLO.Me.Invis() and not mq.TLO.Me.Moving() and mq.TLO.Me.CombatState() ~= 'COMBAT' and mq.TLO.FindItem("Fresh Cookie Dispenser")() ~= nil and mq.TLO.FindItem("Fresh Cookie Dispenser").TimerReady() then
      print('\agSummoning Cookie\ap')
      mq.cmd('/useitem "Fresh Cookie Dispenser"')
      mq.delay('2s')
      mq.cmd('/autoinventory')
    end
  end

  mq.delay('8s')
end;