Problem - Ground spawn ( piece of a medallion )

[zeusdraco]

Sub main
:loop
/itemtarget
/delay 1s
/if (${ItemTarget.Name.NotEqual[NULL]}) {
/if (${ItemTarget.Name.Equal[piece of a medallion]}) {
/delay 1s
/goto :loop
}
/squelch /face item
/delay 5
/call GotoItem
/squelch /face item fast
/call GetItem
/delay 1s ${Cursor.ID}
/delay 2
/if (${Cursor.ID}) /autoinv
}
/delay 2s
/goto :loop
/return

Sub GotoItem
:loop
/if (${ItemTarget.Distance}>0.8) {
/squelch /face item fast
/keypress forward hold
/delay 1
/keypress forward
/goto :loop
}
/return

Sub GetItem
/declare lookangle int
/click left center
/if (!${MacroQuest.LClickedObject} || ${Target.ID}) {
/look -128
:loop
/click left center
/if (${Target.ID}) /target clear
/if (!${MacroQuest.LClickedObject} && ${Me.Look}<0) {
/varcalc lookangle ${Me.Look}+2
/look ${lookangle}
/goto :loop
}
}
/return


It target a loom and try to run at it.....

I dont need it to run at it, i am right infront of the spawn atm for the medallion atm, so a lots of useless code in it.

I tryed to remove, add some part of the code without luck.

All i want is stay FD and when it pop, grap it.....whatever if it remain on my cursor or go in my bag i dont care.

If someone got an idea....

Thx

 

[playbetter]

This will get it...
. Playbetter

 

[Razkle]

Next time you post code please encapsulate the code... it is just easier... simply highlight the code and click the # button above the window to make it code (it also preserves your formatting!)... like this

Sub main
   :loop
   /itemtarget
   /delay 1s
   /if (${ItemTarget.Name.NotEqual[NULL]}) {
      /if (${ItemTarget.Name.Equal[piece of a medallion]}) {
         /delay 1s
         /goto :loop
      }
      /squelch /face item
      /delay 5
      /call GotoItem
      /squelch /face item fast
      /call GetItem
      /delay 1s ${Cursor.ID}
      /delay 2
      /if (${Cursor.ID}) /autoinv
   }
   /delay 2s
   /goto :loop
/return

Sub GotoItem
   :loop
   /if (${ItemTarget.Distance}>0.8) {
      /squelch /face item fast
      /keypress forward hold
      /delay 1
      /keypress forward
      /goto :loop
   }
/return

Sub GetItem
   /declare lookangle int
   /click left center
   /if (!${MacroQuest.LClickedObject} || ${Target.ID}) {
      /look -128
      :loop
      /click left center
      /if (${Target.ID}) /target clear
      /if (!${MacroQuest.LClickedObject} && ${Me.Look}<0) {
         /varcalc lookangle ${Me.Look}+2
         /look ${lookangle}
         /goto :loop
      }
   }
/return

It target a loom and try to run at it.....

I dont need it to run at it, i am right infront of the spawn atm for the medallion atm, so a lots of useless code in it.

I tryed to remove, add some part of the code without luck.

All i want is stay FD and when it pop, grap it.....whatever if it remain on my cursor or go in my bag i dont care.

If someone got an idea....

Thx

 

[janktron]

This will get it...
. Playbetter

can this mac be modified for any ground spawn?

looks like i would just change the item name to whatever i'm tryin gto pick up

 

[playbetter]

Sure........ I tested it on "Brell's Bounty" then did a 'Find/Replace All'

Playbetter

 

[janktron]

i dont get how this works. It just gives me "piece_of_medallion.mac@6 (main): /itemtarget "whatever i decide to put on the ground, it really doesnt matter" Couldnt find "whatever i put as ground target" to target.

Am i supposed to have my cursor over the exact spot on the ground where the item spawns? also, am I putting the exact name of the item its picking up? or the generic name for the ground spawn graphic (alot of the ground items looks like small bags, etc.)

 

[eqmule]

you can simplify this macro down to like 4 lines instead by just using /click left item to pick it up... no need for /look etc.
see: http://www.macroquest2.com/phpBB3/viewtopic.php?f=29&t=11451&p=166397&hilit=/click left item#p166397

Edit: ok I guess that's what PlayBetter posted. nm me.