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

Admins can't send faxes #27197

Open
1 task done
Nerfection opened this issue Oct 24, 2024 · 5 comments · May be fixed by #27285
Open
1 task done

Admins can't send faxes #27197

Nerfection opened this issue Oct 24, 2024 · 5 comments · May be fixed by #27285
Labels
Bug This is a bug in our game code Runtime This issue is a runtime error

Comments

@Nerfection
Copy link

Exploit Reports

  • I confirm this issue is not an exploit. (Required)

BYOND Version

515.1644

Issue Description

We can use the fax panel as normal all the way up until we send the fax, then the sound file plays at the target fax machine...but no paper comes out!

What did you expect to happen?

I expected the fax to print as normal

What happened instead?

the sound played but nothing came out. No fax!

Why is this bad/What are the consequences?

As an admin, faxing is my favourite way to interact with the crew.

Steps to reproduce the issue.

Easily reproducable - send a fax to the station from the fax panel

When did the problem start happening?

First noticed 1/2 months ago, but was told it was fine and working as intented, but it keeps happening (not all the time, but I'm unclear what makes it work/not)

Extra information

No response

Relevant log output/runtime error

[2024-10-24T21:38:05] Runtime in code/modules/paperwork/faxmachine.dm:332: undefined variable /obj/item/paper/var/message

@Tourte-Yaya
Copy link
Contributor

more indepth runtime log: Image

@silverplatedelta silverplatedelta added Bug This is a bug in our game code Runtime This issue is a runtime error labels Oct 25, 2024
@PhantornRU
Copy link
Contributor

When there are many faxes from the central command, they duplicate all the faxes. This should not happen.

@PhantornRU
Copy link
Contributor

#27230

@FunnyMan3595
Copy link
Contributor

The runtime listed was caused by @leoworm 's #26953. It updated receivefax and print_fax to take a /datum/fax, rather than a piece of paper, but didn't update the calls in code/modules/admin/topic.dm.

@leoworm
Copy link
Contributor

leoworm commented Oct 31, 2024

Ah, my bad, I tried to update all relevant calls, but must've missed that one. Admin faxes sometimes not sending properly from the fax panel was already an issue prior to my PR though afaik.

@Adrer Adrer linked a pull request Nov 4, 2024 that will close this issue
1 task
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug This is a bug in our game code Runtime This issue is a runtime error
Projects
None yet
Development

Successfully merging a pull request may close this issue.

6 participants