Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Reloading ammo into gun makes the gun bigger #53974

Closed
Hirmuolio opened this issue Jan 2, 2022 · 7 comments · Fixed by #54188
Closed

Reloading ammo into gun makes the gun bigger #53974

Hirmuolio opened this issue Jan 2, 2022 · 7 comments · Fixed by #54188
Labels
Items: Containers Things that hold other things Items: Magazines Ammo holding items and objects.

Comments

@Hirmuolio
Copy link
Contributor

Describe the bug

Reloading ammo into gun makes the gun bigger.

Steps to reproduce

  1. Spawn 10mm/.40 revolver.
  2. Compare its volume when empty vs when full

Expected behavior

Ammo inside the gun shouldn't make the gun bigger.

Screenshots

gun

Versions and configuration

  • OS: Windows
    • OS Version: 10.0 2009
  • Game Version: b49f934 [64-bit]
  • Graphics Version: Tiles
  • Game Language: System language []
  • Mods loaded: [
    Dark Days Ahead [dda]
    ]

Additional context

Just add "rigid": true into the magazine pocket of the guns.

Maybe "rigid": true should be the default for magazine pockets. Any magazine pocket where the ammo is stored "outside" could have "rigid": false in its definition.

@Hirmuolio Hirmuolio changed the title Most (all?) magazine pockets need "rigid": true added to them. Reloading ammo into gun makes the gun bigger Jan 2, 2022
@LeahLuong
Copy link

I think there's a Glock that grows in length when you load a large capacity clip into it. This makes sense as the clip protrudes far past the bottom of the pistol's grip IRL. Are you certain something like that isn't the case here??

@SirPendrak
Copy link
Contributor

Using magazine well instead of rigidity might be safer solution.

@Hirmuolio
Copy link
Contributor Author

This is about individual cartridges being put in. Not about magazines being put in.

Think about revolver. The revolver doesn't get bigger when you put individual rounds into it. It does get bigger here.

@LeahLuong
Copy link

Look, IDK how gun work. I just know when you push the little button on the bottom, it gets loud & fast metal comes out of the front hole. It's pretty badass.

@catdach
Copy link
Contributor

catdach commented Jan 3, 2022

So gun items generally use 1 of 2 different types of pockets "MAGAZINE_WELL" and "MAGAZINE" which is literally written in the code as:
"pocket_type": "MAGAZINE_WELL", (this is for guns that use external magazines. ie: Glock, AR-15, most guns)
or
"pocket_type": "MAGAZINE", (this is for guns that use internal magazines. ie: shotguns(except the siaga), revolvers and most bolt action rifles.)
There is also the (confusingly named) pocket data field magazine_well (lowercase) which determines how much volume can be placed within the pocket before it starts adding to the volume of the overall gun. For example the pocket data for the Glock 31 looks like this:

"pocket_data": [ { "magazine_well": "250 ml", "pocket_type": "MAGAZINE_WELL", "item_restriction": [ "glock40mag", "glock40bigmag" ] } ]

and for a double barrel shotgun looks like this:

"pocket_data": [ { "pocket_type": "MAGAZINE", "rigid": true, "ammo_restriction": { "shot": 2 } } ]

So gun(or other item) that doesn't define either "magazine_well": "(volume)" or "rigid": true are going to have more volume when it is loaded, which for most guns doesn't make sense.

@catdach
Copy link
Contributor

catdach commented Jan 3, 2022

Anyway, I personally agree that at least for pockets with "pocket_type" : "MAGAZINE", should probably have "rigid": true by default.

@Maleclypse Maleclypse added Items: Containers Things that hold other things Items: Magazines Ammo holding items and objects. labels Jan 4, 2022
@SirPendrak
Copy link
Contributor

Only magazineless weapons that I can imagine getting bigger when being reloaded would be crossbows, ?bows? and some rocket/grenade launchers, like an RPG-7. Do we have RPG in game?

But thats mostly nitpicking, and small arms like revolers getting bigger can actually have gameplay consequences like not being able to fit gun in holster when loaded.

This was referenced Jan 8, 2022
kevingranade pushed a commit that referenced this issue Jan 9, 2022
* Update 10mm.json

* Update 22.json

* Update 3006.json

* Update 500.json

* Update 762.json

* Update 762R.json

* Update 9mm.json
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Items: Containers Things that hold other things Items: Magazines Ammo holding items and objects.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants