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

Pill bottles whitelist #1393

Merged

Conversation

BlitzTheSquishy
Copy link
Contributor

About the PR

Added a whitelist to Pill Cannisters to reduce their ability to be used for powergaming.

This PR is to attempt to push this change early (if at all) to Delta-V before upstream does

Why / Balance

Due to the lack of a whitelist on pill cannisters it is possible to put 300u of reagents inside a single pill canister via the use of bottles, as well as effectively doubling that again with the use of combat medkits (though with those it's intentional).

Technical details

Media

  • I have added screenshots/videos to this PR showcasing its changes ingame, or this PR does not require an ingame showcase

Breaking changes

Changelog

@github-actions github-actions bot added the Changes: YML Changes any yml files label Jun 16, 2024
Signed-off-by: BlitzTheSquishy <[email protected]>
@Colin-Tel
Copy link
Contributor

image

This won't affect my cigarette packet meta, will it?

@IamVelcroboy
Copy link
Member

This was rejected upstream. (And I sort of agree with the argument.) Ideally we avoid whitelist as much as possible. Another solution would be to increase the size of bottles to small. Which should probably be done upstream if it's going to be done.

@deltanedas
Copy link
Member

physically it makes no sense that you can fit 10 glass bottles in a ciggie pack

cdda inventory wyci

@BlitzTheSquishy
Copy link
Contributor Author

image

This won't affect my cigarette packet meta, will it?

way less likely to happen but consider killing that too

@BlitzTheSquishy
Copy link
Contributor Author

This was rejected upstream. (And I sort of agree with the argument.) Ideally we avoid whitelist as much as possible. Another solution would be to increase the size of bottles to small. Which should probably be done upstream if it's going to be done.

I mean pill canister

Also I'm not entirely sure if the tiles they take up is also tied to their "size" but theres a point where beakers or even shakers is preferable to bottles,

I personally dont understand the argument of not whitelisting them, without proposing another solution. Cause you can put LITERALLY any "tiny" item in a pill canister, which would be changing the size of a lot of different items (bullets, bottles, injectors, lighters)

@IamVelcroboy
Copy link
Member

I personally dont understand the argument of not whitelisting them, without proposing another solution. Cause you can put LITERALLY any "tiny" item in a pill canister, which would be changing the size of a lot of different items (bullets, bottles, injectors, lighters)

Bottles and injectors should not fit in a pill bottle. But like...
171881354664243081140976294071

@BlitzTheSquishy
Copy link
Contributor Author

I personally dont understand the argument of not whitelisting them, without proposing another solution. Cause you can put LITERALLY any "tiny" item in a pill canister, which would be changing the size of a lot of different items (bullets, bottles, injectors, lighters)

Bottles and injectors should not fit in a pill bottle. But like... 171881354664243081140976294071

considering the fact pil lcans can only fit TEN pills, youre using the wrong size of pill canister

@deltanedas
Copy link
Member

literally just code cdda inventory in ss14...

pill canister holds 250ml of stuff and has max length of like 8 cm

@IamVelcroboy
Copy link
Member

considering the fact pil lcans can only fit TEN pills, youre using the wrong size of pill canister

I don't have a smaller one available but I'm certain both the lighter and bullets would still fit

@IamVelcroboy
Copy link
Member

Point is, whitelist is restrictive. A container is a container regardless of its intended function. I could put batteries, or bottle caps, or toothpicks, etc in a pill bottle. This does have gameplay value as players may be able to hide or stash various objects in something that wouldn't be the normal place to look for it.

Is the size/inventory around a lot of objects currently abusable, yes. But I agree with upstream that whitelist isn't the best solution. Would be better to adjust the size of items that are currently formatted too small.

@deltanedas
Copy link
Member

pill bottle is 1x1 same as a pill so it should be made larger (and have its storage doubled to match) because how can a pill bottle hold 10 of itself

@NullWanderer NullWanderer merged commit c4fc559 into DeltaV-Station:master Jul 1, 2024
11 checks passed
@NullWanderer
Copy link
Contributor

There are better ways to do this yes, but I believe people keep abusing this so whatever

@BlitzTheSquishy BlitzTheSquishy deleted the Pill-Bottles-Whitelist branch August 23, 2024 06:19
LaryNevesPR pushed a commit to LaryNevesPR/Estacao-Andromeda that referenced this pull request Jan 3, 2025
# Description

Cherry picks
space-wizards/space-station-14#29678
Needed for emotes to not be packed tightly together, 
also need this for a future PR where I will be fixing emotes in general
◕‿◕)

---

<details><summary><h1>Media</h1></summary>
<p>


![image](https://github.com/user-attachments/assets/bc58aee6-f615-4f6d-ad9a-793c482f3da6)

</p>
</details>

---

# Changelog
:cl:
- fix: Fixed radial menus overlapping where there's many icons.

Co-authored-by: Rinary <[email protected]>
Co-authored-by: metalgearsloth <[email protected]>
Co-authored-by: sleepyyapril <[email protected]>
LaryNevesPR pushed a commit to LaryNevesPR/Estacao-Andromeda that referenced this pull request Jan 3, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Changes: YML Changes any yml files
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants