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

[MIRROR] Comprehensive cleanup of storage datum, replaces the weakrefs with just refs (because they were managed already) #1875

Merged
merged 1 commit into from
Feb 9, 2024

Conversation

Steals-The-PRs
Copy link
Collaborator

Mirrored on Nova: NovaSector/NovaSector#785
Original PR: tgstation/tgstation#81120

About The Pull Request

  • Large amount of storage datum cleanup.

    • Documentation.
    • Maybe more consistent use of parent vs real_location.
    • Removes the weakrefs, replaces it with just references.
      • These were already managed references anyways so why bother?
    • Removes a bunch of arguments no one used and would ever used so only the most useful args are left.
  • Some bugfixes.

Why It's Good For The Game

Aiming to make storage easier to work with. The whole intent of this was to bugfix the whole "weight class" thing that keeps popping up but I had to do this first.

Changelog

🆑 Melbert
fix: When placing an item into storage (such as backpacks), all nearby mobs now get a message, rather than just the first mob.
fix: TGC decks of cards should act a bit less odd when looking inside.
refactor: Refactored a bit of storage, cleaned up a fair bit of its code. Let me know if you notice anything funky about storage (like backpacks).
/:cl:

…s with just refs (because they were managed already) (#785)

* Comprehensive cleanup of storage datum, replaces the weakrefs with just refs (because they were managed already)

* CI fix

* Clean up modular items

* Surgery Tray Fix

---------

Co-authored-by: MrMelbert <[email protected]>
Co-authored-by: SomeRandomOwl <[email protected]>
@ReezeBL ReezeBL merged commit 29d2eb7 into master Feb 9, 2024
24 checks passed
@ReezeBL ReezeBL deleted the upstream-mirror-785 branch February 9, 2024 10:16
AnywayFarus added a commit that referenced this pull request Feb 9, 2024
Iajret pushed a commit that referenced this pull request Apr 9, 2024
* ds2-qol

* small change

* windoors in botany... why didn't they have these already?

---------

Co-authored-by: NotDhu <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants