Skip to content

Commit

Permalink
[MIRROR] Fixes incorrect arguments in Status Displays (#2827)
Browse files Browse the repository at this point in the history
* Fixes incorrect arguments in Status Displays (#83611)

## About The Pull Request

Introduced in #83296 (9c58d7d1b9bd9cd447289cb02d0c49772b75a24a)

Args for `generate_status_text()`: `line_y, message, x_offset,
text_color, header_text_color, line_pair`

The positional args we were feeding it via
`/obj/machinery/status_display/proc/update_message()`: `line_y, message,
text_color, header_text_color, x_offset, line_pair`

This PR just moves `x_offset` to the proper position so we aren't
plugging the wrong numbers into the positional arguments.
## Why It's Good For The Game

Was likely broken, now it is fixed. In my test case, I found that it
would break really weirdly and badly without this fix and worked fine
with this fix. Probably due to the keys actually working. Anyways this
was almost definitely the correct behavior seeing as x_offset is always
the third argument everywhere so
## Changelog

:cl:
fix: Status displays should now no longer sporadically randomly break
when in the custom message mode
/:cl:

* Fixes incorrect arguments in Status Displays

---------

Co-authored-by: san7890 <[email protected]>
Co-authored-by: NovaBot13 <[email protected]>
  • Loading branch information
3 people authored and StealsThePRs committed Jun 4, 2024
1 parent bf2c84f commit 966f3fc
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion code/game/machinery/status_display.dm
Original file line number Diff line number Diff line change
Expand Up @@ -146,7 +146,7 @@ GLOBAL_LIST_EMPTY(key_to_status_display)
*/
/obj/machinery/status_display/proc/update_message(current_key, line_y, message, x_offset, line_pair)
var/obj/effect/overlay/status_display_text/current_overlay = get_status_text(current_key)
var/obj/effect/overlay/status_display_text/new_overlay = generate_status_text(line_y, message, text_color, header_text_color, x_offset, line_pair)
var/obj/effect/overlay/status_display_text/new_overlay = generate_status_text(line_y, message, x_offset, text_color, header_text_color, line_pair)

if(current_overlay == new_overlay)
return current_key
Expand Down

0 comments on commit 966f3fc

Please sign in to comment.