...
...the last gump opened is the npc's paperdoll, so that's why it keeps running and running in circles and calling for quest without accepting it...

Well, you may wish to close that paperdoll then...
The #contname for the quest gump type is generic_gump, so while you see anything else as the top-most gump you should not evaluate it.

Your new Sub Target_NPC should look something like:

Sub Target_NPC
Target_NPC:
Finditem %NPCQUESTER G_18
if #findkind <> -1
{
set #lobjectid %NPCQUESTER
event macro 17 0
wait 15
until #contname = generic_gump
do {
set %badgumpx #contposx + 50
set %badgumpy #contposy + 50
click %badgumpx %badgumpy r
wait 15
}
Return
}
Return


I'm unable to test on that shard, though...

-k.

...nope, I am not as old as that song, but after a 9 year long break and seeing all the new stuff in UO I kind of fieel like 'that' old....
On the free shard I was playing on, ML was not yet fully implemented when I gave up playing due to time commitments, so I have a lot to catch-up with.

Trying to play my old and rather complex&outdated template effectively (mage/arch/necro) is a bit overwhelming now, compared with what I have been used to back in the day, and even my old subs collection doesn't help much. Most of the stuff doesn't work or make sence anymore (now what did I need that for?? cases for example) so some fresh examples definetely help.

I have mostly used my own (Euox back in the day) scripting for simple targeting and other Mob/crowd interactions plus equipment swaps, etc.... so until I get back to shape I am not sure if I will be able to contribute anything useful, but I hope to at least to able to point up to use cases, help on beta tests and such.

-k.