In a quest I'm trying to make I want it to go from stage 10 to stage 20 after picking up 4 items. I have the mod working but you need to pick up the items in a certain order. That all fine and dandy but I know that the mod would just flow a lot better if the quest was scripted that way.

Event OnContainerChanged(ObjectReference newContainer, ObjectReference oldContainer)

if (newContainer == Game.GetPlayer())

PLQP01.SetObjectivedisplayed(20)

PLQP01.SetStage(20)

endif

EndEvent

This is the script I'm using but I'm wondering if it is at all possible to some how modify it to set the objective once all for items have been picked up.

If the items are all quest items that can't be dropped, then you could just test if the player has them all before the SetStage.

I did something similar in my http://www.nexusmods.com/skyrim/mods/71553, but used about 20 different stages as I wanted to tell the player what he was looking for next. Each item was in a separate dungeon, so there had to be a large number of objectives, and hence stages to set them. If you just have a single objective of "pick up the set" it's much easier.

Assuming the items are unique you just test

if (player.GetItemCount( a ) > 0 && player.GetItemCount( b ) > 0 && ...)

PLQP01.SetStage(20)

endif

Awesome! So I'm guessing that you would have to make the items all the same in the kit for the script to work?

For example. My four items would have have to be iron ingots but I could just change the models for the iron so the they all look different. Do you think that would work?

No all different is fine. In fact if you want to use GetItemCount to detect them they need to be distinct from vanilla items. You're checking the count on each one, not the whole set.

So in your code "a" and "b" would be replaced with the item name right? Sorry for all the questions. Still getting the hang of this.

yeah a and b would be the names of the items you want to check.

So I keep getting a complier error.

player.GetItemCount(EctoPirateQuest))

That isn't checking anything.

Also you're probably missing the Event identifier, unless you copy/pasted that and left out the Event stuff.

I was thinking that might be throwing it off as well. How would write the event line? Event OnContainerChanged Do you know of any good sites to learn this script?

Event OnContainerChanged(ObjectReference akNewContainer, ObjectReference akOldContainer)
if (player.GetItemCount(PirateIronQuest) > 0 && player.GetItemCount(PirateLeatherQuest) > 0 && player.GetItemCount(RumPirateQuest) > 0 && player.GetItemCount(EctoPirateQuest) > 0)
    PirateQuestHook01.SetObjectiveDisplayed(23)
    PirateQuestHook01.SetStage(23)
endif    
EndEvent

The best way to learn papyrus is by looking at the vanilla scripts and getting an idea how things are done.

And just in case you don't understand how that event is going to fire: If one of those conditions is not greater than 0, the stage will not change.

I'm starting to get the hang of it I think. Now I've got all these errors if I use the script you wrote. In regards to the names for the items do you use the alias for the names?

The situation here is that you have no properties defined.

http://www.creationkit.com/Variables_and_Properties