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

Coal can be loaded into camp forge infinitely & loaded camp forge doesn't provide "forge" for crafting. #70005

Closed
mkrutov opened this issue Dec 5, 2023 · 3 comments
Labels
(S3 - Duplicate) Bug that is duplicate of another one

Comments

@mkrutov
Copy link
Contributor

mkrutov commented Dec 5, 2023

Describe the bug

What it says on the tin. Forge can be only used with charcoal.

Attach save file

n/a

Steps to reproduce

  1. make forge
  2. put coal in
  3. put more coal in
  4. put more coal in
  5. craft stuff.

Expected behavior

coal is limited to 2000 units,same as charcoal
on second use it should ask "unload X coal?"
on top of which forge with coal should provide forge with charges for forging recipes

Screenshots

No response

Versions and configuration

  • OS: Linux
    • OS Version:
  • Game Version: baff13d [64-bit]
  • Graphics Version: Tiles
  • Game Language: System language []
  • Mods loaded: [
    Dark Days Ahead [dda],
    Disable NPC Needs [no_npc_food],
    Portal Storms Ignore NPCs [personal_portal_storms],
    Slowdown Fungal Growth [no_fungal_growth]
    ]

Additional context

No response

@mkrutov mkrutov added the (S1 - Need confirmation) Report waiting on confirmation of reproducibility label Dec 5, 2023
@KeremBabaG
Copy link
Contributor

KeremBabaG commented Dec 5, 2023

the function crafting_ammo_item_type that defines the ammo of pseeudo crafting tools takes only the first element in the set of ammo in json; but the actual reload action in the reload_furniture function uses something else entirely so one can reload but not craft or unload

I tried to take a crack at it but failed miserably, iexamine.cpp 4332 and 4380 if anyone's interested

also lemme try this

also same as #69239

@KeremBabaG
Copy link
Contributor

KeremBabaG commented Dec 5, 2023

/confirmed

edit: damn :(

@RenechCDDA
Copy link
Member

RenechCDDA commented Dec 9, 2023

Duplicate of #68371

Thank you for reporting this bug. We already have an issue # tracking this. To keep the reports organized (so we can fix them!) your issue will be closed, and a link provided to keep all conversation together.

@RenechCDDA RenechCDDA marked this as a duplicate of #68371 Dec 9, 2023
@RenechCDDA RenechCDDA added (S3 - Duplicate) Bug that is duplicate of another one and removed (S1 - Need confirmation) Report waiting on confirmation of reproducibility labels Dec 9, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
(S3 - Duplicate) Bug that is duplicate of another one
Projects
None yet
Development

No branches or pull requests

3 participants