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

[PORT] Fixes the biogenerator displaying item costs that were being divided by the machine's efficiency twice #3619

Merged
merged 1 commit into from
Oct 3, 2024

Conversation

Absolucy
Copy link
Member

@Absolucy Absolucy commented Sep 30, 2024

Port of tgstation/tgstation#74939

Changelog

🆑 GoldenAlpharex
fix: Biogenerators now display the accurate price of their products after having been upgraded, as they no longer visually apply the efficiency discount twice.
/:cl:

…by the machine's efficiency twice (#74939)

It was really bugging me that I would see a price being displayed in my
biogenerator, only for it to then cost me far more than what was
displayed. Looking into it, I realized that it was simply an issue with
it dividing the cost by the efficiency in the TGUI, when that math was
already done DM-side.

Being charged the cost you see on the machine is good.

:cl: GoldenAlpharex
fix: Biogenerators now display the accurate price of their products
after having been upgraded, as they no longer visually apply the
efficiency discount twice.
/:cl:
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.

2 participants