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

Moves the can_leave_fibers var to /obj/item/clothing instead of being on /atom #23005

Merged
merged 4 commits into from
Oct 26, 2023

Conversation

DGamerL
Copy link
Member

@DGamerL DGamerL commented Oct 23, 2023

What Does This PR Do

Moves a variable to a higher hierarchy, making it not be put on every atom.

Why It's Good For The Game

Makes VV a bit easier to read, while still working correctly

Testing

Tried to leave fibers on a door, checked that it still worked with a forensic scanner.

Changelog

NPFC

code/game/objects/items.dm Outdated Show resolved Hide resolved
@DGamerL DGamerL changed the title Moves the can_leave_fibers var to /obj/item instead of being on /atom Moves the can_leave_fibers var to /obj/item/clothing instead of being on /atom Oct 23, 2023
@ParadiseSS13-Bot ParadiseSS13-Bot added the -Status: Awaiting type assignment This PR is waiting for its type to be assigned internally label Oct 23, 2023
@farie82 farie82 added the Refactor This PR will clean up the code but have the same ingame outcome label Oct 24, 2023
@ParadiseSS13-Bot ParadiseSS13-Bot added -Status: Awaiting review This PR is awaiting review from the review team and removed -Status: Awaiting type assignment This PR is waiting for its type to be assigned internally labels Oct 24, 2023
code/modules/clothing/clothing.dm Outdated Show resolved Hide resolved
Co-authored-by: Ryan <80364400+Sirryan2002@users.noreply.github.com>
@ParadiseSS13-Bot ParadiseSS13-Bot added -Status: Awaiting merge This PR is ready for merge and removed -Status: Awaiting review This PR is awaiting review from the review team labels Oct 25, 2023
@S34NW S34NW merged commit b31e548 into ParadiseSS13:master Oct 26, 2023
11 checks passed
Oyu07 pushed a commit to Oyu07/Paradise that referenced this pull request Jan 2, 2024
…eing on `/atom` (ParadiseSS13#23005)

* Ezpz

* GDN review

* Belts aren't clothing

* Update code/modules/clothing/clothing.dm

Co-authored-by: Ryan <80364400+Sirryan2002@users.noreply.github.com>

---------

Co-authored-by: Ryan <80364400+Sirryan2002@users.noreply.github.com>
@DGamerL DGamerL deleted the scopes-a-var-under-a-higher-point branch February 21, 2024 00:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
-Status: Awaiting merge This PR is ready for merge Refactor This PR will clean up the code but have the same ingame outcome
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants