Duffelbags Exemption List + Gun Bulk Fix#817
Duffelbags Exemption List + Gun Bulk Fix#817Stutternov wants to merge 1 commit intoDarkPack13:masterfrom
Conversation
|
You currently have a negative Fix/Feature pull request delta of -6. Maintainers may close this PR at will. Fixing issues or improving the codebase will improve this score. |
FalloutFalcon
left a comment
There was a problem hiding this comment.
I would strongly consider just overriding the behavior for the default duffel type as
- We already overide the base type for duffles/storage so that we could change the sprite.
- You haven't written an updates path for update the maps and a handful of other spots in code are also lacking the darkpack change (like character spawn/loadouts)
- Its going to be a bit unclear to mappers that this change was made as they share the same sprite
- Your creating kinda snowflake types that has to annoyingly copy paste the real tg sub-types (surgery bags)
| // Darkpack Armor - National guard and EOD armor; they're big | ||
| /obj/item/clothing/suit/vampire/vest/army, | ||
| /obj/item/clothing/suit/vampire/eod, | ||
| // Deployables | ||
| /obj/item/cardboard_cutout, | ||
| // Storage | ||
| /obj/item/storage/bag/money/darkpack, | ||
| // Skub | ||
| /obj/item/skub, | ||
| // Fish | ||
| /obj/item/fish, | ||
| /obj/item/fish_tank, |
There was a problem hiding this comment.
stuff like fish can already be stored in duffels i belive.
| var/list/desc = list() | ||
| for(var/obj/item/valid_item as anything in exception_type_list) | ||
| desc += "\a [initial(valid_item.name)]" | ||
| can_hold_description = "\n\t[span_notice("[desc.Join("\n\t")]")]" |
There was a problem hiding this comment.
whats the examine text look like? wondering how it handled the abstract types like item/fish
Would just changing the base duffel bag from TG be better then? I just avoided touching the base duffel bag to keep it modularized and avoid touching base TG code. |
About The Pull Request
This PR does a few small things, I'll break it down simply into sections.
Darkpack Duffelbag Subtype
The new Darkpack duffel has an exemption list like the Syndie duffel from TG. This allows you to fit SPECIFIC items into it of bulky weight, while not allowing others.
The list is comprised of melee weapons unique to Darkpack, Darkpack longarms (baring the .50 and musket), and a handful of misc items.


Seen here, if you shove 2 longarms in it only has enough space for 7 more normal items before maxing out. This should be fine for balancing of packed-shit bags.

Note: might be worth reducing the duffel exemption from 2 to 1 if having 2 guns in it are too strong but I think it's fine. Syndie packs allowed this with ANY gun, this list is curated at least so you can't do this with a .50. (Also, I put battering ram on the list.. these aren't actually that big, just heavy, unsure if that's OK or not balance wise. Made sense to me though?)
Also, the surgery bag in the blackmarket vendor has been changed to be the Darkpack type variant. I forgot to take a picture of it but it's on-floor sprite is identical to the duffel's on floor but its white with a blue cross.
Darkpack Gun Weight Fix
A bunch of guns like the sniper and lever action had oversights on them where weight was defined as bulky then re-defined lower on the gun as normal. Think it was unintended fully, so adjusted it.
This also fixes an oversight that existed where you could fit a .50 sniper into your backpack. Woopsie.......
Darkpack Money Bag & Presets
Does the same as the Darkpack duffelbag except with the moneybag, making a unique subtype to take the Darkpack cash.
Made a ~4k pre-spawn variant as well as a 10k spawn variant, 50 gold bar, and 50 silver bar spawn variants. This is just for mapping or spawning by staff, shouldn't be an issue. Tested them as well, spawns fine, forgot screenshots since it was pretty minor.
Why It's Good For The Game
Simple bulletpoint reasons:
Changelog
🆑
add: Adds new Darkpack duffelbag subtype
add: Adds ability for new duffelbags to hold 2 exempt-list items (guns, melee weapons, etc)
add: Adds new Darkpack moneybag subtype to hold proper cash, also added pre-spawn variant bags
fix: Fixes oversight on certain guns, making them have proper bulky weight instead of normal
/:cl: