Request - Need help fixing/changing this maco

[VorpalChicken]

Hi, I don't know enough about programing to be able to figure this out for myself, but I've been using this macro for skilling up my pottery.

The only thing is it doesn't seem to destroy the item on the cursor, and it's something I'd like it do.

#event OutOfStuff "Sorry, but you don't have everything you need for this recipe in your general inventory." 
  
Sub Main 
:Loop 
 
:ClearCursor 
/if (${Cursor.ID}) { 
/if (${Cursor.ID}==16925) /destroy
/autoinventory 
/goto :ClearCursor 
} 
 
/notify TradeskillWnd CombineButton leftmouseup 
/doevents
/call delete_item
/delay 2
/notify TradeskillWnd AutoInvButton leftmouseup
/goto :Loop 
/return 
 
Sub Event_OutOfStuff 
/endmacro 
/return

Sub delete_item
/delay 1s
/if (${Cursor.ID}==9686) /destroy
/return

 

[william12]

/if (${Cursor.ID}) {
/if (${Cursor.ID}==16925) /destroy
/autoinventory
/goto :ClearCursor
}

Change it to
/if (${Cursor.ID}) {
/if (${Cursor.ID}) /destroy
/autoinventory
/goto :ClearCursor
}

also

/if (${Cursor.ID}==9686) /destroy

change to

/if (${Cursor.ID}) /destroy

You're trying to destroy a certain item so if its not that item it wont do anything. The changes will make it destroy any item that ends up on the cursor.

 

[wackawacka]

be careful using that because if u pick up a bag you will delete it. I would keep the id set and change it to the ID of item you need destroyed, inspect the item scroll to bottom and find the ID number and replace the number located here /if (${Cursor.ID}==16925) , this will keep you from having a bad day and destroying something you dont want to destroy