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

Unique Glove Fibers #642

Merged
merged 4 commits into from
Aug 6, 2024

Conversation

angelofallars
Copy link
Contributor

Description

Cherry-picked from Delta-V, originally by @WarMechanic (DeltaV-Station/Delta-v#1455).

Original Description:

Every pair of gloves now has its own fingerprint, so items can be traced back to gloves which can then be traced back to people.

Why / Balance

Evidence is very important to court cases running smoothly, so detectives now have more evidence that can certify whether or not John Syndicate's behaviour is valid. Traitors are now encouraged to either clean evidence off syndicate gear regardless of glove status, or use a disposable pair of gloves specifically for handling syndicate gear to be disposed of later.

Aside from being required to obfuscate evidence you leave behind, there is now a value proposition to searching glove prints of departments. Wearing gloves that does not correspond your department can punish an unknowing detective into searching the wrong people.

Technical details

FiberComponent.cs now stores a Fiberprint variable like FingerprintComponent.cs. The code for assigning a fiberprint is the same as the fingerprint. When evidence is placed on an object, the fiberprint is concatenated to its localised fiber type.

Original Media

hm ok we have these specific gloves on an akms

hm well we found the gloves and they have fingerprints

gotem

Changelog

🆑 WarMechanic

  • add: Gloves now have unique fingerprints. Items can be traced back to gloves, which can then be traced back to people.

* make changes

* comments

* commiting a single whitespace so i dont get executed

---------

Co-authored-by: deltanedas <[email protected]>
Copy link
Contributor

@DangerRevolution DangerRevolution left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hopefully @WarMechanic is okay with this :)

Content.Server/Forensics/Components/FiberComponent.cs Outdated Show resolved Hide resolved
Content.Server/Forensics/Systems/ForensicsSystem.cs Outdated Show resolved Hide resolved
Content.Server/Forensics/Systems/ForensicsSystem.cs Outdated Show resolved Hide resolved
Content.Server/Forensics/Systems/ForensicsSystem.cs Outdated Show resolved Hide resolved
Content.Server/Forensics/Systems/ForensicsSystem.cs Outdated Show resolved Hide resolved
Content.Server/Forensics/Systems/ForensicsSystem.cs Outdated Show resolved Hide resolved
Content.Server/Forensics/Systems/ForensicsSystem.cs Outdated Show resolved Hide resolved
Content.Server/Forensics/Systems/ForensicsSystem.cs Outdated Show resolved Hide resolved
Co-authored-by: Danger Revolution! <[email protected]>
Signed-off-by: Angelo Fallaria <[email protected]>
Content.Server/Forensics/Systems/ForensicsSystem.cs Outdated Show resolved Hide resolved
Co-authored-by: VMSolidus <[email protected]>
Signed-off-by: Angelo Fallaria <[email protected]>
@WarMechanic
Copy link
Contributor

Hopefully @WarMechanic is okay with this :)

yeah this is fine

@VMSolidus VMSolidus merged commit 8768df7 into Simple-Station:master Aug 6, 2024
11 checks passed
SimpleStation14 added a commit that referenced this pull request Aug 6, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Changes: C# Changes any cs files Status: Needs Review Someone please review this
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants