ZDoom

Now it makes sense. [i]Owned[/i] items cannot have their own inventory. I just did a quick test with CustomInventory's 'Use' state and, predictably enough, it behaved the same way as with weapons.

Ha, ha. You ran into one of ZDoom's implementation oddities here. An owned inventory item cannot own any other items, it uses its Inventory pointer to link the player's inventory together. So by giving something to the owned weapon it automatically got appended to the player's inventory. This is something that cannot really be fixed. I think I just let A_GiveInventory for owned items fail because it makes no sense.

No. I looked into the code. But I'm not a very good at interpreting some things.

So you were making it up? -_-

It was mostly speculation to begin with after browsing the code. :mrgreen:

Well, honestly I rather have Graf confirm this.

That's what I was saying, except as far as I can tell, it was supposed to be this way, not a bug. Weapons, while held, point towards the player instead of itself for certain functions like inventory.

Class CInvTest : CustomInventory
{
   States
   {
   Pickup:
      TNT1 A 0
      {
         invoker.A_GiveInventory("Cell", 6);
         A_LogInt(invoker.CountInv("Cell"));
         A_LogInt(CountInv("Cell"));
      }
      Stop;
   }
}

[spoiler=stuff that doesn't matter anymore]This confirms that Weapon classes can in fact have their own inventory
[code]Class WepWithInvTest : Shotgun
{
States
{
Spawn:
TNT1 A 1 NoDelay
{
A_GiveInventory("RocketAmmo", 22);
A_LogInt(CountInv("RocketAmmo")); //prints 22, not 0
}
Stop;
}
}[/code]

[quote]Also you called invoker.A_GiveInventory (assuming you did the exact same test on a custominventory actor instead of a weapon), so no, the countinv wouldn't be 0. Invoker means checking the custominventory's inventory. Not the player's.[/quote]
??????
Are you trying to tell me this piece of code gives 6 cells to [b]both[/b] 'self' and 'invoker'? Is that what's happening here?
[code] invoker.A_GiveInventory("Cell", 6);[/code]

And if that's not the case then the only thing that makes sense to me is that, for Weapon classes, invoker is self. Either that or all inventory related functions are redirected to 'self' for some reason. I'm confused.[/spoiler]

[size=120]EDIT:[/size]
[code]Class CInvTest : CustomInventory
{
States
{
Pickup:
TNT1 A 0
{
invoker.A_GiveInventory("Cell", 6);
A_LogInt(invoker.CountInv("Cell"));
A_LogInt(CountInv("Cell"));
}
Stop;
}
}[/code]

Guess what, the first A_LogInt prints 6, the second prints 0. The player doesn't get the 6 Cells.
'CustomInventory' works. This is definitely a bug with 'Weapon'.

Inventory itself is capable of having inventory, thanks to Edward850 adding the +INVENTORY.TRANSFER flag. What he did not do is manipulate weapons when he made it. So invoker is most likely not so limited as weapons are.

Also you called invoker.A_GiveInventory (assuming you did the exact same test on a custominventory actor instead of a weapon), so no, the countinv wouldn't be 0. Invoker means checking the [b]custominventory[/b]'s inventory. Not the player's.

Again, I don't think weapons were ever designed to have inventory like actual inventory can. It's two whole different actors. ZDoom was introduced to +INVENTORY.TRANSFER and thus inventory is capable of storing inventory within itself -- and giving that inventory to other actors upon pickup. (Inventoryception much? :P)

Ok. But what doesn't make sense here is why does invoker.CountInv() show self's inventory? invoker.CountInv() seems to work correctly with [url=http://forum.zdoom.org/viewtopic.php?f=7&t=54202]CustomInventory[/url] so why not with Weapon?

In this case, if A_GiveInventory gives directly to the player then shouldn't A_LogInt(invoker.CountInv) print 0?

Invoker is mainly designed with the ability to modify its properties and store variables of its own, last I checked. Meaning they cannot store inventory of their own and will thus give it to the player instead.

Correct me if I'm wrong, Graf.

Class WTest : Shotgun
{
	States
	{
	Fire:
		SHTG B 20
		{
			invoker.A_GiveInventory("Cell", 6);
			A_LogInt(invoker.CountInv("Cell"));
			A_LogInt(CountInv("Cell"));
		}
		Goto Ready;
	}
}

It seems that for weapons invoker is the same as the player who wields the weapon.
[code]Class WTest : Shotgun
{
States
{
Fire:
SHTG B 20
{
invoker.A_GiveInventory("Cell", 6);
A_LogInt(invoker.CountInv("Cell"));
A_LogInt(CountInv("Cell"));
}
Goto Ready;
}
}[/code]
Type "give WTest" then "use WTest" in the console to select this and try to fire it.

If you have the standard Doom HUD, you should see your ammo counters.
As you can see from the Cell ammo counter [i][b]it's you[/b][/i] who receives the 6 Cells.
The two A_LogInt calls confirm this because they print the same result.