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

Emergency Auto Injector Is Refillable #3980

Closed
3 tasks done
Steelpoint opened this issue Jul 24, 2023 · 6 comments
Closed
3 tasks done

Emergency Auto Injector Is Refillable #3980

Steelpoint opened this issue Jul 24, 2023 · 6 comments
Assignees
Labels
Bug Admin events aren't bugs Needs Reproducing We're not even sure if you know how to do it

Comments

@Steelpoint
Copy link
Contributor

Testmerges

#3783: #3955: #3962:#3917:#3909:#3843:#3902:#3974:

Description of the bug

When you select a used emergency auto injector, you can refill it on a Wey-land Plus vendor, hacked or unhackced.

I tested this on the live server and on a offline up-to-date version of the code as well. .

What's the difference with what should have happened?

The emergency injector should not be refillable.

How do we reproduce this bug?

  1. Grab a used injector
  2. Click a Wey-Land Plus vendor to refill it
    ...

Issue Bingo

@Steelpoint Steelpoint added the Bug Admin events aren't bugs label Jul 24, 2023
@Wintermote
Copy link

Wintermote commented Jul 31, 2023

There's two types of emergency injectors, one comes in the PFC medical pouch and is an EZ emergency autoinjector (it is colored green), this cannot (normally maybe its bugged) be refilled in any of the aforementioned vendors etc.

The second type of emergency injector requires medical skill to use properly and is tan colored. Squad leaders get them out of their autoinjector pouches. These tan colored emergency injectors can be refilled in wall vendors etc.

I don't know whether it's intentional design, but it's worth pointing out there's two different types anyway

@Steelpoint
Copy link
Contributor Author

I only noticed this when playing as a SL so this is the case. Question is then if this is intended behaviour or unintentional.

@Huffie56 Huffie56 assigned Huffie56 and unassigned Huffie56 Aug 1, 2023
@Segrain Segrain self-assigned this Aug 1, 2023
@Segrain
Copy link
Contributor

Segrain commented Aug 1, 2023

Will take care of this whenever I will get to taking the planned look at emergency injectors in general (I have opinions.)

@Huffie56
Copy link
Contributor

Huffie56 commented Aug 1, 2023

Will take care of this whenever I will get to taking the planned look at emergency injectors in general (I have opinions.)

Lorax was interested in fixing this issue. don't know how far he is on it... will tell him.

@Huffie56 Huffie56 added the Needs Reproducing We're not even sure if you know how to do it label Dec 12, 2023
@Huffie56
Copy link
Contributor

i tried to reproduce with the standart refillable autoinjector everyone get and the other one that cost point to get for spe but was unable to refill them.
i don't see how i would have miss that test so closing it now.

@Huffie56
Copy link
Contributor

#4167
this is probably the PR that fix it...

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug Admin events aren't bugs Needs Reproducing We're not even sure if you know how to do it
Projects
None yet
Development

No branches or pull requests

4 participants