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

The detomatix cart now sends forged messages that make PDAs explode if one tries to reply to them instead of detonating them right off the bat. #62494

Merged
merged 5 commits into from Nov 14, 2021

Conversation

Ghommie
Copy link
Member

@Ghommie Ghommie commented Nov 1, 2021

About The Pull Request

Exactly what it's written on the tin. I have reworked the detomatix cartridge after reading the detomatix removal PR again (including comments and suggestions). The forged name and job of the message respect filters to avoid bad words, the message uses the send_message proc (it has a few new args) and the PDA shouldn't explode outside replying to a forged bomb message (or opening the message menu within 10 seconds after receiving it). I have also changed the TC cost of the item to 4 and the number of charges to 6 considering that, even in the hands of a cunning player, not all messages will be "replied". This buff to the tc cost and charges was done with the assumption that half of these messages will be ignored anyway in general.

Why It's Good For The Game

The current way detomatix works gives no room to the receivers to avoid getting pwned. A few people say it's uninteresting and not a sufficient source of interaction or amusement amongst players (though my opinions on this may diverge a bit) and that the explosion is too strong (alas, this is more a general issue with explosion RNG and thus should be handled in a separate PR). On the other side I have heard people arguing it adds value to the game in the form of paranoia and sabotage. I have also heard there's a plan to replace PDAs with modular computer tablets, though it's too early in the works to warrant a removal of this cartridge, especially when half of the station crew doesn't start with modular tablets and there's no exact counterpact to PDA messaging on modular computers (the closest is a chat app).

This PR resolves the issue of detomatix being unavoidable without the ultimatum of a removal that most people don't want anyway. Basically, an alternative to #62484. And why yes, I have tested it.

Changelog

🆑 Ghommie (Thanks to dorsisdwarf for the idea)
balance: The detomatix cart now sends forged messages (name and job of the message can also be forged) that can explode a PDA when someone tries to reply to them or access the message menu of the PDA UI up to 10 seconds after receiving it instead of detonating the PDA right off the bat. Because of this and the possibility that a portion of these messages goes "unreplied", the TC cost of the item has also been reduced to 4 and the number of charges upped to 6.
/:cl:

@tgstation-server tgstation-server added the Balance Changes to functionality that modifies how effective certain methods are at powergaming label Nov 1, 2021
@wesoda25
Copy link
Contributor

wesoda25 commented Nov 1, 2021

The window should be longer than 10 seconds, maybe even slightly randomized to prevent people from gaming it.

@zxaber
Copy link
Contributor

zxaber commented Nov 1, 2021

Does the message show up in the message logs on the PDA?

Also, I'd probably just remove the limit. There's no real functional reason it needs to exist (assuming it respects adjacent checks). If someone wants to wait half an hour and then reply to a trapped message, let them blow up all the same.

@itseasytosee
Copy link
Contributor

There is hot single space women on your station!! To find out click here!!! ---> ---> (reply)

@iwishforducks
Copy link
Contributor

Funny but I don't like the idea of powergaming this introduces and not replying to PDA messages. I'd rather be tormented with my PDA randomly exploding rather than punishing me for actually using its functions.

@Ghommie
Copy link
Member Author

Ghommie commented Nov 1, 2021

Does the message show up in the message logs on the PDA?

Yes.

Also, I'd probably just remove the limit. There's no real functional reason it needs to exist (assuming it respects adjacent checks). If someone wants to wait half an hour and then reply to a trapped message, let them blow up all the same.

Oh turns out I didn't word it out very well. The time window is exclusive to opening the message menu in case of smartasses using the message menu to reply to the message rather than the reply button (See #62484 (comment)). Time doesn't matter for the href buttons.

The window should be longer than 10 seconds, maybe even slightly randomized to prevent people from gaming it.

See above. I can still extend and randomize it out if you want.

Funny but I don't like the idea of powergaming this introduces and not replying to PDA messages. I'd rather be tormented with my PDA randomly exploding rather than punishing me for actually using its functions.

Mhm, I too have some doubts. I have done an attempt with a simple delay and ominous message. Alas, it's been closed by optimumtact in less than two hours over a frivolous slight on the commit message. The idea of PR'ing it again as an alternative to this one which I have also made (and the removal of course) sounds quite out there to me right now. Feel free to re-PR it in my stead if you want to.
I don't get what you're referring to with powergaming by the by.

@phasenoisepon
Copy link
Contributor

I don't get what you're referring to with powergaming by the by.

beep Message from JOHN ANDERS: CHEF reply?

[Common] Valid McSalad: Hey John, did you just PDA me?
[Common] John Anders: No?
[Common] Valid McSalad: AI LAW 2 TURN THE MESSAGING SERVER OFF WE HAVE TRAITORS

@Ghommie
Copy link
Member Author

Ghommie commented Nov 1, 2021

That's quite an exaggerated example considering message server consoles can too send messages with sham sender name and job and that disabling the entire messaging server in spite of a single anomalous message is quite the bold move.

But thanks for pointing it out. I'll work on a way get the pda bomb messages to be rerouted to the hub if the message server is down and then broadcasted anyway. The only thing the message server does, outside of relaying request console and pda messages, is store them so they can be later checked with the message server comp.

EDIT: I have been thinking, the detomatix should also allow people to send normal PDA messages without them being tracked by the message server at this point. Same with customizing sender name and job. It'd nifty. EDIT EDIT: Nay, it'd be a pretty awful obstacle to the transition to modular tablets anyway.

@ATH1909
Copy link
Contributor

ATH1909 commented Nov 1, 2021

(or opening the message menu within 10 seconds after receiving it)

instead of this, just wait until they either reply to the message OR send a direct message to the person who sent them that rigged message. in other words, pda messaging the bomber in any way at any later time should cause the target's pda to explode.

@Ghommie
Copy link
Member Author

Ghommie commented Nov 1, 2021

instead of this, just wait until they either reply to the message OR send a direct message to the person who sent them that rigged message. in other words, pda messaging the bomber in any way at any later time should cause the target's pda to explode.

Since name and job of the message signal can be forged, someone may as well send a pda message to someone with an alias that doesn't show up in the PDA messaging menu, so doing that may require adding a list variable to the PDA type that injects the list of aliases used to send rigged messages to said PDA in its message menu.
I'll give it a shot though.

@github-actions github-actions bot added the Merge Conflict Adding upstream files to your repo via drag and drop won't resolve conflicts label Nov 2, 2021
@ExcessiveUseOfCobblestone
Copy link
Contributor

the issue is the explosion i dont really consider this a cure for the detomatrix

@itseasytosee
Copy link
Contributor

We could turn them into emp bombs that also remove a random few access levels from an inserted card. Although that nerf alongside this one would be too much.

@tgstation-server tgstation-server removed the Merge Conflict Adding upstream files to your repo via drag and drop won't resolve conflicts label Nov 6, 2021
@github-actions github-actions bot added the Merge Conflict Adding upstream files to your repo via drag and drop won't resolve conflicts label Nov 8, 2021
@tgstation-server tgstation-server removed the Merge Conflict Adding upstream files to your repo via drag and drop won't resolve conflicts label Nov 13, 2021
@Kylerace Kylerace added the Code Improvement Code is now easier to copy paste. label Nov 14, 2021
@Kylerace
Copy link
Contributor

holy fuck those magic numbers thank you for cleaning those up

@Kylerace
Copy link
Contributor

I don't get what you're referring to with powergaming by the by.

beep Message from JOHN ANDERS: CHEF reply?

[Common] Valid McSalad: Hey John, did you just PDA me? [Common] John Anders: No? [Common] Valid McSalad: AI LAW 2 TURN THE MESSAGING SERVER OFF WE HAVE TRAITORS

this could also be comms agents

@PolishUser
Copy link
Contributor

PolishUser commented Nov 15, 2021

Cool, you've moved The detomatix cartridge item into obscurity. No one in the right mind will buy this item because it is dependent on the recipient of the message. Normal people, keeping that in mind, will wait out those magical 10 seconds and then reply or even better, don't respond to them at all. In addition, I won't be surprised if some maintainer No. 563 will swoop in later on and just remove this antag item completely. Anyways, why tg hasn't gotten rid of PDAs completely? Tablets are the future.

Mothblocks pushed a commit to Mothblocks/tgstation that referenced this pull request Nov 22, 2021
…f one tries to reply to them instead of detonating them right off the bat. (tgstation#62494)
Mothblocks pushed a commit to Mothblocks/tgstation that referenced this pull request Nov 22, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Balance Changes to functionality that modifies how effective certain methods are at powergaming Code Improvement Code is now easier to copy paste.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet