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

You can no longer eat/drink or consume pills while having a mask/helmet covering your mouth. #3230

Merged
merged 1 commit into from
Apr 2, 2014

Conversation

Donkie
Copy link
Contributor

@Donkie Donkie commented Mar 29, 2014

This fixes annoying situations such as being tazered then forcefed a beepskysmash, also fixes my immersions.

To prevent clowns and mimes from revealing their true identity, their masks is an exception to this restriction.

I am fully aware that the whole logic behind my code is abit messy, I just didn't know how else I'd do it, especially with the clown/mime exceptions (alloweat var). I would be very glad if I got some constructive critism that I could improve the code with.

@theOperand
Copy link
Contributor

complete shit close posthaste 👎

Just kidding, this will make a lot of people very mad so I love it. 👍
Op 29-mrt.-2014 17:30 schreef "Daniel Hultgren" notifications@github.com:

This fixes annoying situations such as being tazered then forcefed a
beepskysmash, also fixes my immersions.

To prevent clowns and mimes from revealing their true identity, their
masks is an exception to this restriction.

I am fully aware that the whole logic behind my code is abit messy, I
just didn't know how else I'd do it, especially with the clown/mime
exceptions (alloweat var). I would be very glad if I got some

constructive critism that I could improve the code with.

You can merge this Pull Request by running

git pull https://github.com/Donkie/-tg-station foodmask

Or view, comment on, or merge it at:

#3230
Commit Summary

  • Initial Commit

File Changes

Patch Links:

Reply to this email directly or view it on GitHubhttps://github.com//pull/3230
.

@ghost
Copy link

ghost commented Mar 29, 2014

👍 take off your fucking internals to eat fuck

@Kraseo

This comment was marked as abuse.

@Donkie
Copy link
Contributor Author

Donkie commented Mar 29, 2014

mixed emotions

@ZRZK2127
Copy link

awesome.

@Giacom
Copy link
Contributor

Giacom commented Mar 29, 2014

My opinion is that stuff like this makes SS13 what it is, so yes 👍

@borntohonk
Copy link

To prevent clowns and mimes from revealing their true identity, their masks is an exception to this restriction.

clown and mime mask exceptions

Why?

How does this make sense? Why make an exception if you want to prevent them from revealing their true identity?

Or do you mean the opposite, to ENCOURAGE them to reveal their true identity, they are exceptions to this?

Being protected from being forcefed shit for wearing normal masks, but penalized for wearing mime/clown masks, that's unfair

@ZRZK2127
Copy link

To prevent clowns and mimes from revealing their true identity, their masks is an exception to this restriction.

@theOperand
Copy link
Contributor

You could add eating through masks to SWAT masks too since they are
deathsquad gear and the entire point of deathsquad gear is to be
ridiculously OP.

On Sat, Mar 29, 2014 at 7:09 PM, ZeroZiat notifications@github.com wrote:

To prevent clowns and mimes from revealing their true identity, their
masks is an exception to this restriction.

Reply to this email directly or view it on GitHubhttps://github.com//pull/3230#issuecomment-39003713
.

@ChangelingRain
Copy link
Contributor

clown/mime masks are more like a wig and facepaint than actual masks though they work as internals fnr

@theOperand
Copy link
Contributor

They work like internals so clowns and mimes don't have to take off their
masks.

On Sat, Mar 29, 2014 at 7:26 PM, Anonus notifications@github.com wrote:

clown/mime masks are more like a wig and facepaint than actual masks
though they work as internals fnr

Reply to this email directly or view it on GitHubhttps://github.com//pull/3230#issuecomment-39004205
.

@ADamDirtyApe
Copy link
Contributor

I like of this, more realism is better.
The clown and mime need to keep the specialness to their masks otherwise they would need to use different masks for internals. Without their mask, even if they need internals to stay alive, the job is made less for it.

Aside from that, I looked over the coding, it does not look like this applies to the space helmets, only children of the gas mask in clothing.dmi (space helmets are separate items in there). It might be easier to make use of the MASKCOVERSMOUTH and HEADCOVERSMOUTH flags somehow, but I don't know how to carry over the excepted masks in that case. That would imply you would have to redo the code target the mouth to eat though, which we do not already, probably due to convenience and/or laziness. It would be neat if we had to target the mouth with fluid containers to drink (or force drink) too, but one thing at a time I guess.

@Donkie
Copy link
Contributor Author

Donkie commented Mar 29, 2014

I can remove the flags from those masks, but that also means things like CPR and all other things using the MASKCOVERSMOUTH (which it actually does) will change behaviour, I though this way is better.

@ADamDirtyApe
Copy link
Contributor

I am not a maintainer so I cannot really say if they would allow it this way, I suggest asking in the IRC for more immediate feedback. Alternatively you could @ request them here, but that is going to be slower.
I am not even sure who the maintainers are right now tbh, the only one that I know is aran as I think giacom stopped a bit a ago.

@MrPerson
Copy link
Contributor

You can do @ tgstation/commit-access to ping every maintainer no matter who they are.

Personally I would have used the slot flags to tell if you can eat something or not. If that means changing up other things, that's fine. It's kind of dumb that clown/mime masks block your face anyways.

@theOperand
Copy link
Contributor

Don't all maintainers have the entire repo on notice anyway? I get mails
for everything but direct commits to master.
Op 30-mrt.-2014 03:43 schreef "MrPerson" notifications@github.com:

I think you can do @tgstation/commit-access to ping every maintainer no
matter who they are. Note that I didn't ping everyone in this case, so if
you want to do that, do it yourself!

Reply to this email directly or view it on GitHubhttps://github.com//pull/3230#issuecomment-39014990
.

@Donkie
Copy link
Contributor Author

Donkie commented Mar 30, 2014

-snip

@theOperand
Copy link
Contributor

Rip in peace donkie eaten by maintainers.

@optimumtact
Copy link
Member

Can't see anything wrong with the code here 👍

Why a var rather than a new flag though?

@Donkie
Copy link
Contributor Author

Donkie commented Mar 31, 2014

I felt it produced more mess than was necessary, but I might been wrong.

@GunHog
Copy link
Contributor

GunHog commented Apr 1, 2014

All for the sake of "realism", which is the worst possible reason
It adds nothing to gameplay, and subtracts quite a bit.

Pretty much this. 👎

@Razharas
Copy link
Contributor

Razharas commented Apr 1, 2014

Realism
Worst possible reason

You are on the list
We will watch you very closely now
Veeeery closely

@Sugarette
Copy link
Contributor

Always nice to see that people still hate mining. 👍

@Niknakflak
Copy link
Contributor

Never stopped

@ZRZK2127
Copy link

ZRZK2127 commented Apr 1, 2014

someone on ooc said miner suits would have hamburger slots where you can just put a hamburger for the wearer to eat

@Razharas
Copy link
Contributor

Razharas commented Apr 2, 2014

Very american

@Steelpoint
Copy link
Contributor

👎 Gameplay should trump realism

@ExplosiveCrate
Copy link

👎 Unless you somehow make stim pills and hivelord cores snowflake items that can be eaten through masks.

@theOperand
Copy link
Contributor

Oh no I will have to TAKE MY MASK OFF.
Op 2-apr.-2014 12:21 schreef "ExplosiveCrate" notifications@github.com:

[image: 👎] Unless you somehow make stim pills and hivelord cored
snowflake items that can be eaten through masks.

Reply to this email directly or view it on GitHubhttps://github.com//pull/3230#issuecomment-39313595
.

@ZRZK2127
Copy link

ZRZK2127 commented Apr 2, 2014

Yeah I don't really get all the chagrin. It'll make eating rooms a bit more
useful if people sit down to eat instead of miners/people doing EVA work
eating on the job, etc.

Just another reason to take your gear off.

On Wed, Apr 2, 2014 at 8:09 AM, Miauw62 notifications@github.com wrote:

Oh no I will have to TAKE MY MASK OFF.
Op 2-apr.-2014 12:21 schreef "ExplosiveCrate" notifications@github.com:

[image: 👎] Unless you somehow make stim pills and hivelord cored
snowflake items that can be eaten through masks.

Reply to this email directly or view it on GitHub<
https://github.com/tgstation/-tg-station/pull/3230#issuecomment-39313595>

.

Reply to this email directly or view it on GitHubhttps://github.com//pull/3230#issuecomment-39317844
.

@Donkie
Copy link
Contributor Author

Donkie commented Apr 2, 2014

Other ideas to fix the beepskysmash issue:

  • force-feeding/drinking someone should result in lots of spill and perhaps only 30% of the container gets emptied into the person.
  • force-feeding/drinking should take longer time

@ZRZK2127
Copy link

ZRZK2127 commented Apr 2, 2014

@Man-Shroom- it'll require some effort to slap on something deathly poisoning on someone. Like removing their mask first. Holy shit antags today want everything on a goddamn silver platter.

@Rockdtben
Copy link
Contributor

Force feeding should be must easier if the person is bucklecuffed.

@Kuromoi
Copy link

Kuromoi commented Apr 2, 2014

This would just make the game more tedious even from a non-antag point of view, bad idea.

@ExplosiveCrate
Copy link

Oh no I will have to TAKE MY MASK OFF.

Then what the fuck is the point of this pull request. Is it realism? It's even more unrealistic for miners to have to take of their space helmet in space so they can use tools that are practically REQUIRED to do their job.
Is it because of balance? Because this is a fucking retarded way to balance an assassination method that is not only incredibly rare, but also so ineffective (the strongest poison you can get from botany is the shitty MEDBAY TOXINS, traitors can't even buy cyanide anymore) it might not even exist.

@Aranclanos
Copy link
Contributor

eating through a spacesuit in space is stoopid

@Razharas
Copy link
Contributor

Razharas commented Apr 2, 2014

3 maintainers agree its ok, its going in

Razharas added a commit that referenced this pull request Apr 2, 2014
You can no longer eat/drink or consume pills while having a mask/helmet covering your mouth.
@Razharas Razharas merged commit c61fcbd into tgstation:master Apr 2, 2014
@theOperand
Copy link
Contributor

Damnit Raz, there were changes that could have been made ;_;

On Wed, Apr 2, 2014 at 7:31 PM, Razharas notifications@github.com wrote:

Merged #3230 #3230.

Reply to this email directly or view it on GitHubhttps://github.com//pull/3230
.

@ChangelingRain
Copy link
Contributor

Hivelord remains will still work because they're not actual food items but stim pills a useless now

@ghost
Copy link

ghost commented Apr 3, 2014

Terrible and adds nothing.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.