Jump not working on custominventory item

[Grey-Wolf]

So, I'm making this player class that loses a bunch of random weapons when he enters a new map.
To do this, I'm making multiple actor, one for each weapon, that actually take the weapon from you.

I tried with something like this:

---

Code: Select all

Actor SSGtaker : CustomInventory
{
Inventory.PickupMessage "\cGWho said that break action \cFSAWED OFF  \cGSHOTGUNS never break? I think someone said that. Well he was wrong. Especially when it gets crushed by demons."
	States
	{
 Spawn:
	TNT1 A 0
	TNT1 A 1
	Loop
	
 Pickup:
	TNT1 A 0
	TNT1 A 0 A_TakeInventory("SSG",1)
	TNT1 A 0 A_TakeInventory("SSGAmmo",2)
	TNT1 A 0 A_GiveInventory("AmmoShell",2)
	Stop
	
	}
}

And everything works like a charm, but I don't quite like it. This way I should spawn the items right under the player's feet, and make him pick them up. Furthermore, the message will show whether the player has the weapon or not.
So I tried this:

---

Code: Select all

Actor Shotguntaker : CustomInventory
{
	+INVENTORY.QUIET
	States
	{
 Spawn:
	TNT1 A 0
	TNT1 A 1
	TNT1 A 1 A_JumpIfInventory("Shot_Gun",1,"Loss")
	Loop
	
 Loss:
	TNT1 A 0
	TNT1 A 0 A_Log("\cGYour trusted \cFSHOTGUN  \cGfailed you. With it malfunctioning, You'll have to find another one...")
	TNT1 A 0 A_TakeInventory("Shot_Gun",1)
	TNT1 A 0 A_TakeInventory("SlugShotgun",1)
	TNT1 A 0 A_TakeInventory("ShotgunAmmo",9)
	Stop
	
	}
}

But it seems like the jump is not performed even If I have the shotgun in inventory. What did I do wrong?

[Apeirogon]

Because you place inventory check in spawn state, while it would work on player only if placed in pickup state.

[Grey-Wolf]

Is there a pointer or some workaround to make it target the player even from the spawn state?

[Graf Zahl]

No. The spawn state is only active when the item is not owned.

[Grey-Wolf]

Meh. I'll go with the "spawn everything under player's feet" solution then, and see my options from there. Thanks

EDIT:
Ok SOLVED issue, thanks everyone for clarifying my questions!
In case anyone is wondering (I don't think so but still), here's what I did:

---

Code: Select all

Actor Shotguntaker : CustomInventory
{
	+INVENTORY.QUIET
	States
	{
 Spawn:
	TNT1 A 0
	TNT1 A 1
	Loop
	
 Pickup:
	TNT1 A 1 A_JumpIfInventory("Shot_Gun",1,"Loss")
	Stop //no message
	
 Loss:
	TNT1 A 0
	TNT1 A 0 A_Log("\cGYour trusted \cFSHOTGUN  \cGfailed you. With it malfunctioning, You'll have to find another one...")
	TNT1 A 0 A_TakeInventory("Shot_Gun",1)
	TNT1 A 0 A_TakeInventory("SlugShotgun",1)
	TNT1 A 0 A_TakeInventory("ShotgunAmmo",9)
	Stop
	
	}
}

I know, it's pretty basic, but I'm still learning, and form me it's still a good surprise when things go as planned XD

[Arctangent]

Should be noted that the Pickup state runs even if the actor is given the item via [wiki]A_GiveInventory[/wiki] or similar direct inventory manipulation functions. So, there's no need to spawn the item under the player's feet - which should generally be avoided with this kind of thing, anyway, as items on the map can only be picked up by player movement.

[Grey-Wolf]

I'm using Spawn Item because the actor that takes the weapons away is actually a random spawner that picks between 16 spawn states that removes various combinations of weapons and ammo. I don't think that using A_GiveInventory to give the spawner to the player would work. Or would it?

[Arctangent]

It could with a single inventory item that uses [wiki]A_Jump[/wiki] to do the random picking that you're using the RandomSpawner for.

[Grey-Wolf]

Oh nice, I've already a single inventory item doing the picking with jumps, but I didn't think it would work if not picked up. Thanks for the info!