WowAce

Hello,

Thought of another idea that would be nice.  Sometimes when you loot your crops you receive the gift items that are used to turn into the various npcs for rep.  These items are the following...

Lovely Apple, Jade Cat, Marsh Lily, Ruby Shard and Blue Feather.

I myself am exalted with the Tillers and all the various other npcs that you need to turn these items into to reach a "Best Friend" status with.  So when I loot these sometimes from my crops, they serve no purpose.  No need to turn them in anymore, they have no vend value and since they are soulbound, they can't be traded either.  So I have to manually delete them.

Would it be possible to make those items drop/delete when you move out of the zone, exactly how the tools are handled?

The problem I see with this is.  Each item can be turned into two different npcs for max rep gains, or turned into any of them for lesser rep. So someone may/or may not need certain "gift items".  I also know that you don't really want to add a options interface for this addon that could show these items individually with checkboxes or something unless you had to, which is understandable.

My solution would be to add another little icon "above or below" the "tool lock icon" you have for dropping tools/or not dropping tools so the user could make the choice to have all those "gift items" drop automatically/or not drop based on if they need them anymore.

-Thank You

Oh ok, didn't know that :) thank you.

Ok, shows that my memory is fading since my last addon development is long ago.

I was thinking it would be possible to call a function from a secure action template. While I found back some reference to such possibility on the internet as well I did not manage to get TargetUnit to work. So I remained with te macro.

Special UTF-8 characters  do not seem to work in multi-line strings so I used concatenation.  I have added a few lines for unused space in the farm which will show a message about this fact (first /run). Sinc ethis is in th ebeginning of th emacro the target will finally be any plant which requires attention, or if there is none, an empty space.

Also I have ordered the plant "states" in such an order that the most obvious ones (ex. Alluring) come first leaving the less visible (Parched) at the end, so that the less visible get targeted first by the radar.

Because of the empty space part I changed the "Farm looking good" into "Plants looking good" in the translation.

The modified code is below. Th eempty line marks the point to which I could test the wording in-game. I did not see the states below that point yet even though I replanted everything and 4 spaces even several times.

Probably it would be best to add an option to enable or disable the "empty space" part of the macro.

      if(GetLocale()=="deDE") then
         macro='/cleartarget\n'..
              '/tar Ungepfl\195\188gtes Erdreich\n'..
              '/tar Gepfl\195\188gtes Erdreich\n'..
              '/run if UnitExists("target") then RaidNotice_AddMessage(RaidBossEmoteFrame,"Es gibt noch unbepflanzte Felder !", ChatTypeInfo["RAID_BOSS_EMOTE"]) end;\n'..
              '/tar Verlockende\n'..
              '/tar Wilde\n'..
              '/tar Befallene\n'..
              '/tar Ausgetrocknete\n'..
              '/tar Zuckende\n'..
              '/tar Bedeckte\n'..
              '/tar Verk\195\188mmerte\n'..

              '/tar Verwucherte\n'..
              '/tar Beanspruchte\n'..
              '/tar Hartn\195\164ckige\n'..
              '/run if UnitExists("target") then SetRaidTarget("target",8) else RaidNotice_AddMessage(RaidBossEmoteFrame,"Die Pflanzen sehen gut aus !", ChatTypeInfo["RAID_BOSS_EMOTE"]) end;\n'
         item.btn:SetAttribute("macrotext",macro)
      else
         ...englishversion here...

Great idea with the macro. Not sure, but after a more thinking I would believe that the scanning I suggested would probably not work because the plands are "friendly".

I was thinking about using LUA TargetUnit("name") together with GetUnitName("target") . The Advantage of using LUA would be that the name could be specified including the space after the word (which in some languages may need more target names to be checked). The advantage of scanning the full word would be that there could be no accidental identification of a player name starting with the same word. I know that the farm is a phased area but there may be issues with phasing.

Meanwhile I have inserted the german translation into the code but you may want to move it to a separate file. I will check th etranslation tomorrow when I setup new plants. I did check severaltoday but changed the words to longer ones afterwars leaving me with only one name really checked so far.

If I get some spare time I will also try to move to LUA code instead of the macro.

if(GetLocale()=="deDE") then
     item.btn:SetAttribute("macrotext", [[
/cleartarget
/tar Ausgetrocknet
/tar Befallene
/tar Zuckende       -- Checked
/tar Verlockende
/tar Bedeckte
/tar Wilde
/tar Verkümmerte
/tar Verwucherte
/tar Beanspruchte
/tar Hartnäckige
/run if UnitExists("target") then SetRaidTarget("target",8) else
RaidNotice_AddMessage(RaidBossEmoteFrame,"The farm is looking good!", ChatTypeInfo["RAID_BOSS_EMOTE"]) end; ]]);
 else
   ...original english code... 

Sorry forgot to update the UTF char sin the code with their proper UTF code. Currently trying the LUA code but TargetUnit gets blocked even with the secure template.

Just seems that all that is over-complicating things, the macro gets the job done as it is.

How does the plant scanner tool work? I can't seem to see any indicators.

I was looking at the comment below and came up with this simple macro...

/tar Parched
/tar Infested
/tar Wiggling
/tar Alluring
/tar Smothered
/tar Wild
/tar Runty
/tar Tangled
/tar Occupied
/tar Stubborn
/script SetRaidTarget("target",8);

The macro will target and put a Skull icon over top of problem crops.  If you dont get a target with a skull, then all your crops are growing.  I don't know if this could somehow be implemented into the addon.

-Thanks

Glad the idea worked... thanks for implementing.

Thanks for this nice addon !

Would it be possible to add a feature to count the number of growing and non-growing (needing attention) plants on the farm ?

As the plants are targetable like mobs I would think that this should be possible in the way rar-spawn detection mods work.

Awesome. Many thanks!

New version v0.8b works great, thanks for your fast response :)

Its not letting me install "Farmhand" via Curse Client on this page.  I downloaded it manually and put it in my Interface folder, but when I log into the game its not showing up in the addon list at all.

-Thanks

getting the same issue.  Click the install with curse client link and it opens the client but nothing installs.  Install it manually (and properly) and it does not show in the in game addon list.