[MIRROR] Fixes boulder processing mining point gain. (#26228)

* Fixes boulder processing mining point gain. (#81067)

## About The Pull Request

Due to a misplaced multiplier, arcmining machines were eating their own
point totals.
Now, the refinery and smelter should correctly only take the
MINING_POINT_MACHINE_MULTIPLIER define on the amount being added to the
points held, not on the total resting in the machine.

## Why It's Good For The Game

Simple, easy fix.
Fixes #81057.

## Changelog

🆑
fix: The smelter and refinery now properly hold mining points, only
taking a small amount out of net gained points.
/🆑

---------

Co-authored-by: SyncIt21 <110812394+SyncIt21@ users.noreply.github.com>

* Fixes boulder processing mining point gain.

---------

Co-authored-by: ArcaneMusic <41715314+ArcaneMusic@users.noreply.github.com>
Co-authored-by: SyncIt21 <110812394+SyncIt21@ users.noreply.github.com>
This commit is contained in:
SkyratBot
2024-01-25 15:37:28 +01:00
committed by GitHub
parent dbd51275f2
commit 46bf09daeb
@@ -211,7 +211,7 @@
if(!is_type_in_list(possible_mat, processable_materials))
continue
var/quantity = chosen_boulder.custom_materials[possible_mat]
points_held = round((points_held + (quantity * possible_mat.points_per_unit)) * MINING_POINT_MACHINE_MULTIPLIER) // put point total here into machine
points_held = round(points_held + (quantity * possible_mat.points_per_unit * MINING_POINT_MACHINE_MULTIPLIER)) // put point total here into machine
processable_ores += possible_mat
processable_ores[possible_mat] = quantity
chosen_boulder.custom_materials -= possible_mat //Remove it from the boulder now that it's tracked