I have a function that revolves around removeallitems, and what I'd dearly love is to make it such that it removes all items that aren't equipped. Possible?

I'm up against this very same problem for Bag of Holding and have resolved to drop everything equipped by equipping a dummy item that takes all the applicable slots after setting my player alias in a "Catch" state with an OnObjectUnequipped event where each unequipped item is dropped, disabled, and added to an ObjectReference array. From there, all the remaining, unequipped items can be removed and, afterward, I walk through the array, enable the dropped stuff, have Player activate them to take 'em back, then re-equip each item. By excluding the equipped items, the move happens a lot faster, but momentarily strips the player.

There are other ways, but they're painfully slooooooow. Here's one I abandoned:

Here's the problem, I'm not sure dead actors can equip things. It's not in the wiki, but I'm having trouble getting it to work.

So what's the context? Are you trying to auto remove all but equipped items from a corpse with a hotkey or something? Are you opposed to requiring SKSE?

My mod already required SKSE. For context, I'd just look at the other thread I have up, the one with Pilfer in the title.

Then I'd go and download Pilfer, because it will make your life better.

Doesn't SKSE have inventory traversal functions? GetNumItems/GetNthForm. So rather than the intermediary of the chest, can you just cycle through the dead actor's inventory, and only transfer the items over the threshold ratio--either ignoring all items that are equipped, or only unequipping and transferring the valuable ones?

Traversing with GetNumItems and GetNthForm will work but it is very slow compared to the way Il Ducey is currently doing things. For the corpse looting purpose it should be fine though since that doesn't have to happen spontaneously. And most NPCs don't have more than a dozen or so items in their inventory so it shouldn't take more than two or three seconds. Just be aware that every time you remove an item from the NPC the index position of other items can and usually will shift. Since you'll be removing all but a small set of items it will be more efficient to work from the beginning of the list, so the code would have to be something like this:

   int n = 0   item = theNPC.GetNthForm(n)   while item      count = theNPC.GetItemCount(item)      if count > 0 and !theNPC.isEquipped(item)         theNPC.RemoveItem(item, count, true, thePlayer)      else         n += 1      endif      item = theNPC.GetNthForm(n)   endwhile

The GetNthForm function returns None when you get to the end of the list, so you don't actually need to call GetNumItems in this case.

(If you were only removing a smaller set of items it would be more efficient to start that the end of the list and work your way down to zero.)

Ah, I didn't realize the container traversal functions were that slow! I haven't played with them yet myself. Good to know, thanks.

Ideally it would pick up things that are equipped if they are worth the money, but leave everything else on. TO that end I wrote this:

Scriptname PilferingScript extends activemagiceffect  GlobalVariable Property PilferGlobal autoObjectReference Property Imaginary autoActor Property PlayerREF autoevent OnEffectStart(Actor Target, Actor Caster)   int n = 0  Form item = Target.GetNthForm(n)   while itemitem = Target.GetNthForm(n)float Weight = item.GetWeight()if Weight==0.0Weight = 0.0001endIfint count = Target.GetItemCount(item)      if ((item.GetGoldValue() / Weight) >= (PilferGlobal.GetValue() as float))         Target.RemoveItem(item, count, true, PlayerREF)      endif n += 1item = Target.GetNthForm(n)   endwhileTarget.SetActorValue("Infamy", 47)endEvent

But its very inconsistent. It picks up everything worth the money that isn't equipped fine, and sometimes it picks up stuff that is if its worth the money. But only sometimes. Boggling my mind why.
NVM. It works, it just is slow sometimes.

Well, now you have two calls to GetNthForm inside your loop. You only need one. The one at the bottom since the very first call is made before the loop starts. Just removing that first call inside the loop should almost double the speed of the script.

ScriptName PilferingScript extends ActiveMagicEffect  Float fWeightInt iIndexForm kItemActor Property PlayerREF AutoGlobalVariable Property PilferGlobal AutoObjectReference Property Imaginary AutoEvent OnEffectStart(Actor akTarget, Actor akCaster)	If akTarget.GetActorValue("Infamy") != 47		akTarget.SetActorValue("Infamy", 47)		iIndex = akTarget.GetNumItems()		While iIndex > 0			iIndex -= 1			kItem = akTarget.GetNthForm(iIndex)			fWeight = kItem.GetWeight()			If !fWeight				fWeight = 0.0001			EndIf			If (kItem.GetGoldValue() / fWeight) >= PilferGlobal.GetValue() As Float				akTarget.RemoveItem(kItem, akTarget.GetItemCount(kItem), True, PlayerREF)			EndIf		EndWhile	EndIfEndEvent

Yeah, thats basically how it works now, except I don't even have the infamy set, as it will only take off the correct items in the first place. I didn't get rid of the second call however, and it sped things up quite a bit.