I know it is something in what I am doing but trying to figure out what. I have had people complete the quest and I have to coem and give them the reward and delete the quest book because the autoreward is not working.

here is my reward line.

<Action>GIVE/<questholder/name/MoonQuest1/notestring/10 Bullshits/objective1/KILLNAMED,Bullshit,10/autoreward/true/repeatable/false/SETONTRIGMOB/ATTACH/xmldata,MoonQuest1/@bag/hue/1169/ADD/HorseDung/name/Bullshit/ADD/bankcheck,200/ADD/LeafGloves/hue/1169</Action>

you need to tell it to assign the rewardstring property here

ATTACH/xmldata,MoonQuest1/rewardstring/@bag/hue/1169/ADD/HorseDung/name/Bullshit/ADD/bankcheck,200/ADD/LeafGloves/hue/1169

also the /hue/1169 at the end is going to do the same thing as the /hue/1169 at the beginning, and that is to change the hue of the bag. Same thing with the name/Bullshit. If you want property changes to apply to something other than the primary object, you would need to use the grouping operator <> like this

bag/hue/1169/ADD/<HorseDung/name/Bullshit>/ADD/bankcheck,200/ADD/<LeafGloves/hue/1169>

so that would work?

SETONTRIGMOB/ATTACH/xmldata,MoonQuest1

is a separate action (SETONTRIGMOB is a standalone keyword like GIVE so it can only be used at the beginning of an entry), so you would need to separate it from the GIVE action with a semicolon like

SETONTRIGMOB/ATTACH/xmldata,MoonQuest1 ; GIVE/<questholder/name/MoonQuest1/notestring/10 Bullshits/objective1/KILLNAMED,Bullshit,10/autoreward/true/repeatable/false/rewardstring/@bag/hue/1169/ADD/<HorseDung/name/Bullshit>/ADD/bankcheck,200/ADD/<LeafGloves/hue/1169>

One question is why you are adding the xmldata attachment. It isnt a problem, I'm just wondering what you are planning on doing with it.

well my main problem is that I want the quest to not be repeatable. kindof a beginner quest. but even if I set the NoTrigOnCarried then it still pops up the quest information. which results in multiple quest books. so one person can kill lets say 60 great harts for the quest to be completed. but if you have multiple quest books then it completes all quest books at once. instead of 60 per book. Also without the xmlattachment check. The player can still go through the quest speech and get the questbook, but it does say it isn't repeatable. this way it saves the player the hassle of going through it all. and the questgiver just lets you know that you have already done the quest.

it sounds like it is just a problem with your NoTrigOnCarried string. Make sure that you specify the name of the questholder exactly, including capitalization.

Also note that any XmlDialog entries that have a DependsOn value of -1 will ignore the Trig/NoTrig settings and will only require nearby movement. If you want them to consider those settings then use a DependsOn of -2 instead.

Alright I got another problem. the xmlspawner is setup properly even made a new one to read the config GeneralDen. For the xml questnpc configfile. Yet it spawns the npc with he name and such. But no xmledit information at all on the npc

Here is the GeneralDen.npc file.

how are you spawning it? What is the actual spawn entry?
(edit)

Your XML file has errors in it. You must have edited it manually because there are invalid chars like < and >.

In XML those have to be converted to &lt; and &gt;

there are a couple of other places where you use those chars as well.

Note that when you have problems like this, particularly if you have been manually creating or editing an XML file, you can always test the file for XML errors by opening it up as an XML file (e.g. change the extension to .xml and open it with IE)

yeah I was editing it manually because every time I get so far on the script line it would revert back while using the ingame book method. and I am using an xmlspawner just like presented in xmlquest for dummies link. in the tutorials section.

One other bugged and confusing script.

did you try debugging it with an XML viewer like IE or firefox? They will find any errors and show you where they are in the file.

wholey sweet, many thanks. sorry for being such a pest. IE7 also works with the debugging

If you have it, firefox actually points to the exact spot where the error occurred and so is a little more helpful.

I'm having a problem with reward being handed out, too. Here's my Action line:

Oh, also, in the questbook gump where it shows how many are remaining to be killed, it doesn't update the number remaining. Ideas?

not sure exactly what armor stands for?
and I have not seen difficulty used before in the script context.

I could say try using plategloves 5,5, instead of armor.