What I'm trying to do is attach a script to a miscellaneous item that will add a perk when it's added to the players inventory and remove it when it's removed.

As you can probably guess, I don't exactly know what I'm doing, so any help or advice as to why this doesn't work would be great.

Also bonus points:
Pretty sure that this script (if it worked) would remove the perk from the player should an NPC happen to have a copy of the item added to their inventory. Any suggestions on how to fix that?

Latest Failure:

Scriptname ForgeTestScript extends ObjectReference  Perk Property TestPerk autoEvent OnItemAdded(Form akBaseItem, int aiItemCount, ObjectReference akItemReference, ObjectReference akSourceContainer)  If (akSourceContainer == Game.GetPlayer())	Game.GetPlayer().AddPerk(TestPerk)  endIfEndEventEvent OnItemRemoved(Form akBaseItem, int aiItemCount, ObjectReference akItemReference, ObjectReference akDestContainer)  If (akDestContainer != Game.GetPlayer())	Game.GetPlayer().RemovePerk(TestPerk)  endIfEndEvent

Scriptname ForgeTestScript extends ObjectReferencePerk Property TestPerk AutoEvent OnContainerChanged(ObjectReference akNewContainer, ObjectReference akOldContainer)    if (akNewContainer == Game.GetPlayer())        Game.GetPlayer().AddPerk(TestPerk)    elseif (akOldContainer == Game.GetPlayer())        if (Game.GetPlayer().GetItemCount(GetBaseObject()) == 0)            Game.GetPlayer().RemovePerk(TestPerk)        endif    endifEndEvent

Try http://www.creationkit.com/OnContainerChanged_-_ObjectReference:

ScriptName YourMiscObjectScript extends ObjectReference  Perk Property TestPerk AutoMiscObject Property YourMISC AutoEvent OnContainerChanged(ObjectReference akNewContainer, ObjectReference akOldContainer)	If Game.GetPlayer().HasPerk(TestPerk) != Game.GetPlayer().GetItemCount(YourMISC)		If Game.GetPlayer().GetItemCount(YourMISC)			Game.GetPlayer().AddPerk(TestPerk)		Else			Game.GetPlayer().RemovePerk(TestPerk)		EndIf	EndIf	EndEvent

*ninja'd

Thanks a tonne guys! Worked a charm.

A more efficient version of RandomNoob's code:

event OnContainerChanged(ObjectReference akNewContainer, ObjectReference akOldContainer)	Actor player = Game.GetPlayer()	if (akNewContainer == player)		player.AddPerk(TestPerk)	elseif (akOldContainer == player)		if (player.GetItemCount(GetBaseObject()) == 0)			player.RemovePerk(TestPerk)		endif	endifendevent

It's important to do stuff like that in a language that isn't say C++ because Papyrus doesn't have such an amazing optimizing compiler and or a crazy optimizing VM like LuaJIT does.

It's a new product, it needs some time to grow in efficiency.