Mage conjuring too many water

[Lord 15]

Hey !
My mage fightclass is creating way to many water .. I nearly have 5 / 6 stack sometimes
Here is the fightclass i use

Thanks !

Mage.xml

[Arcangelo 105]

It's a bot issue.

I have made a couple of mage profiles as well and even if I max the item count it still does it.

[Droidz 2494]

Hello, add lua ckeck before cast spell: Mage fix.xml 

local count = GetItemCount(5350);
if (count <= 1)
{
	local name = GetSpellInfo(29975);
	RunMacroText("/cast " .. name) 
}

 

[Lord 15]

28 minutes ago, Droidz said:

Hello, add lua ckeck before cast spell: Mage fix.xml 

local count = GetItemCount(5350);
if (count <= 1)
{
	local name = GetSpellInfo(29975);
	RunMacroText("/cast " .. name) 
}

 

Thanks :)

[Lord 15]

Sorry i'am not that good in coding .. 
So actually i did the modification for both of spells, & my fightclass isnt doing anything for now :(

Here is my file
 

Mage.xml

And i tried the fightclass you link, here is what i get when i launch it : 

[Droidz 2494]

How to add spell from another fightclass to your fightclass: http://wrobot.eu/forums/topic/5033-how-to-add-easily-spell-from-another-fightclass-to-your-fightclass/

[Lord 15]

I did exactly that, just changing the items ID, and still getting this error

[teddyvgt 0]

Same mistake

[Droidz 2494]

Sorry, it is bug of WRobot for vanilla, wait next update (or don't use IsSpellUsable condition)

[Lord 15]

Okay thanks ! Just to know, the mail issue is a know issue ? I can't use the mail system on Elysium. Should i start a topic for that or is it going to be fix on next update ?

[Twineye 0]

Not sure if it was fixed but I couldn't make it work properly with the lua that Droidz suggested or any other way, so I put one together that works with vanilla API

local food, water = 0, 0; for bag = 0,4,1 do for slot = 1, GetContainerNumSlots(bag), 1 do local name = GetContainerItemLink(bag,slot); if name and string.find(name, "Conjured") then local _, count = GetContainerItemInfo(bag,slot); if string.find(name, 'Water') then water = water + count; else food = food + count; end; end; end; end; if food <= 4 then CastSpellByName('Conjure Food') end; if water <= 4 then CastSpellByName('Conjure Water') end

 

[dirtyjobs 14]

Quote

local count = GetItemCount(5350); if (count <= 1) { local name = GetSpellInfo(29975); RunMacroText("/cast " .. name) }

 

the problem is the item id for 5350 is rank 1 conjured water. https://tbc-twinhead.twinstar.cz/?item=5350
 
the spell 29975 is rank 8 conjured water https://tbc-twinhead.twinstar.cz/?spell=29975

the correct spell id is 5504 for rank 1 conjured water https://tbc-twinhead.twinstar.cz/?spell=5504

The below snippet of code will work for rank 1 conjured water. 

 

local count = GetItemCount(5350);
if (count <= 1)
{
	local name = GetSpellInfo(5504);
	RunMacroText("/cast " .. name) 
}



after repeated attempts I couldnt get this code to work either. i ended up using the code Twineye posted above this post.

[Jasabi 33]

On 28.5.2017 at 9:01 AM, dirtyjobs said:

 

the problem is the item id for 5350 is rank 1 conjured water. https://tbc-twinhead.twinstar.cz/?item=5350
 
the spell 29975 is rank 8 conjured water https://tbc-twinhead.twinstar.cz/?spell=29975

the correct spell id is 5504 for rank 1 conjured water https://tbc-twinhead.twinstar.cz/?spell=5504

The below snippet of code will work for rank 1 conjured water. 

 

local count = GetItemCount(5350);
if (count <= 1)
{
	local name = GetSpellInfo(5504);
	RunMacroText("/cast " .. name) 
}



after repeated attempts I couldnt get this code to work either. i ended up using the code Twineye posted above this post.

Pretty sure GetItemCount isnt in the vanilla api