Do not count magazine as pocket in the pickup inventory

[osuphobia]

Summary

Bugfixes "Only count CONTAINER type pockets in the pickup inventory"

Purpose of change

MAGAZINE/MAGAZINE_WELL type pockets don't really serve as pockets the way CONTAINER type pockets do, you can't pick up and hold things with them directly, so don't count them in the pickup inventory.
Close #72807

Describe the solution

When try to pick up frozen_liquid and ammunition_type items, MAGAZINE/MAGAZINE_WELL type pockets that use these items as ammo will no longer be considered. If you don't have any capable CONTAINER type pocket, failure message is returned.

Describe alternatives you've considered

Not to do this.

Testing

Be nude.
Spawn bows, guns, mags, misc repairkit, etc, unload them. Try to pick up ammo while wielding one of them, only to see Does not fit in any pocket!.
Spawn a water cannon, unload it and pour water on the ground, freeze the water and crush it. Try to pick up ice shard while wielding the water cannon, only to see Does not have any pocket for frozen liquids!.
Behavior before is shown in #72807.

Additional context

[osuphobia]

The test case item_cannot_contain_contents_it_already_has failed.

Checked the code and found out that it's not likely to be my fault.

It's checking whether the test_backpack can contain water_item or not, but the test_backpack is not watertight.

So backpack_loc->can_contain( water_item ).success() should always be false, and !backpack_loc->can_contain( water_item, false, false, true, false, bottle_loc ).success() should always be true. There is no chance CHECK( backpack_loc->can_contain( water_item ).success() ) would pass.

Then why was this test case got passed before? I do modified can_contain, but it should not affect this test.

can_contain is not checking whether it's watertight or not, the fact is I missed one parameter.

[osuphobia]

Well, I was wrong, I did mess it up.