CleverRaven/Cataclysm-DDA

Charge removal side effects 3: unmigrated item spawns

Open

#67.326 geöffnet am 31. Juli 2023

Auf GitHub ansehen
 (0 Kommentare) (2 Reaktionen) (0 zugewiesene Personen)C++ (3.991 Forks)batch import
(S2 - Confirmed)Help WantedSpawn[JSON]

Repository-Metriken

Stars
 (9.264 Stars)
PR-Merge-Metriken
 (Durchschn. Merge 6T 5h) (292 gemergte PRs in 30 T)

Beschreibung

Describe the bug

There are countless places that allow inline definitions of itemgroups or other shorthand definitions that were not caught by the automated script, so all of these are now broken and only spawn single items instead of the defined charges or default charges of the item.

Where to find these cases:

  1. Inside regular itemgroups
    • "distribution" or "collection" members inside the "items" or "entries" array
    • simple itype entries inside the "items" array
  2. Professions
    • lots of simple itype entries, although probably not many that spawn with comestibles
  3. Mapgen (and vehicles?)
    • this one is the most tricky, because there are many different ways to define spawns and I definitely don't know all of them
  4. Recipe byproducts
    • haven't actually confirmed that one, but it's probably an issue?
  5. Probably other places I forgot
    • comments appreciated

All affected items and their previous default charges can be found here: https://gist.github.com/mqrause/dc234737bfd5c4309aa4dc496d17bd53#file-removed_charges-txt

Attach save file

n/a

Steps to reproduce

n/a

Expected behavior

n/a

Screenshots

No response

Versions and configuration

6088293934801590f503ff9d5d961e708af564ff

Additional context

No response

Contributor Guide