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

Make smoking cool again. #6046

Merged
merged 1 commit into from
Jan 6, 2022
Merged

Conversation

ElectroJr
Copy link
Member

Fixes smoking visuals not working properly. Main change is just moving the setting of the clothing prefix to server side code. It was only being set client-side and was being overridden by the server-side clothing state somewhere along the way.

🆑

  • fix: Smoking is cool again (fixed animation/sprite).

@ElectroJr ElectroJr added the T: Bugfix Type: Bugs and/or bugfixes label Jan 6, 2022
@metalgearsloth metalgearsloth merged commit 4921ea6 into space-wizards:master Jan 6, 2022
Morb0 referenced this pull request in space-syndicate/space-station-14 Jan 7, 2022
* Fix use delay (#6031)

* Add small railing corner (#6036)

* Add more chat filters (#6034)

* Xeno improvements (#6032)

* Automatic changelog update

* Removed capsaicin oil from the chem dispenser (#6042)

* Automatic changelog update

* Add ability to resist out of welded lockers and locked closets (#5958)

* Enable lighting of Cigars with IsHotEvent

Subscribed to IsHotEvent in ThrusterSystem, added AfterInteractEvent to SmokingSystem.Cigar which will set SmokableState to Lit if IsHotEvent returns true

This should mean that cigs/Cigars should light on anything hot

* Revert "Enable lighting of Cigars with IsHotEvent"

This reverts commit db896e1f80940892a87d34aa7182b9c605fa53bc.

* Add the ability to kick your way out of welded or locked closets

Added a ResistLocker component/system that hooks into OnRelayMove to begin resisting out of lockers so that players aren't stuck in lockers forever

Also added check to EntityStorageComponent to not print out welded shut message if the entity attempting to open the locker is inside of it.

* Changes based on review

minor formatting changes

make components passed into AttemptResist nullable and resolve them inside of the function

Remove incorrectly named .ftl file

* Changes based on review

Ditched async DoAfter
Used PopupSystem over User.PopupMessage

Added Cancel token to ResistLockerComponent

Subscribed to EntRemovedFromContainer event to cancel DoAfter if the player is removed from a container

* I am in formatting hell, apparently

* Changes based on review

Added TryComp for the EntityStorageComponent instead of the event passed one

* Changes based on review

* Apply suggestions from code review

Co-authored-by: ShadowCommander <[email protected]>

* Formatting changes based on review

* Shuffle functions around in LockSystem to properly trigger visualizer

Moved all the unlocking logic to a Lock method inside of LockSystem and TryUnlock calls this method if the user passes all of the access checks

* Formatting, replacing Resolve with TryComp and making AttemptResist arguments optional

Co-authored-by: Leon Friedrich <[email protected]>

* Check if the player is inside THE container not A container

me no read good

* ok ok I give my code is bad

Co-authored-by: ShadowCommander <[email protected]>
Co-authored-by: Leon Friedrich <[email protected]>

* Automatic changelog update

* Make smoking cool again. (#6046)

* Automatic changelog update

* Fixes minor issues with Reach and Dart. (#6035)

Co-authored-by: metalgearsloth <[email protected]>

* Automatic changelog update

* Adds filled versions of coloured wardrobes (#6012)

* Automatic changelog update

* Add content to radiation closets and medicine lockers (#6006)

* Automatic changelog update

* Interrupt do-afters when the user gets incapacitated. (#6059)

* Fix foldable-pickup interactions. (#6057)

Co-authored-by: Leon Friedrich <[email protected]>
Co-authored-by: Delete69 <[email protected]>
Co-authored-by: ZeroDayDaemon <[email protected]>
Co-authored-by: metalgearsloth <[email protected]>
Co-authored-by: PJBot <[email protected]>
Co-authored-by: Rane <[email protected]>
Co-authored-by: JustinTime <[email protected]>
Co-authored-by: ShadowCommander <[email protected]>
Co-authored-by: LittleBuilderJane <[email protected]>
Co-authored-by: Lamrr <[email protected]>
@ElectroJr ElectroJr deleted the smoking-fix branch March 24, 2022 18:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
T: Bugfix Type: Bugs and/or bugfixes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants