From 95ed2de101a2cec8595c834bffebb96210d2e6d6 Mon Sep 17 00:00:00 2001 From: SyncIt21 <110812394+SyncIt21@users.noreply.github.com> Date: Tue, 9 Sep 2025 15:22:05 +0530 Subject: [PATCH] Fixes map export breaking semi colon dependent strings (#92898) ## About The Pull Request Take the atmos gas mixture string for example. All parts of the gas mixture is appended together with semicolon `;` https://github.com/tgstation/tgstation/blob/d2ec1056ab5aaf494cba4877fe8d65a6cbf39cf9/code/modules/atmospherics/gasmixtures/gas_mixture.dm#L768 However the map export replaces semicolon `;` with an empty string ` ` https://github.com/tgstation/tgstation/blob/d2ec1056ab5aaf494cba4877fe8d65a6cbf39cf9/code/modules/admin/verbs/map_export.dm#L180 We don't want that cause you get broken gas strings. There is no problem in writing special strings that would spawn items because we are already replacing `{` & `}` with empty strings so the string becomes non parseable anyway ## Changelog :cl: fix: map export won't break exported gas strings & other that depend on semicolon /:cl: --- code/modules/admin/verbs/map_export.dm | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/code/modules/admin/verbs/map_export.dm b/code/modules/admin/verbs/map_export.dm index 0546eb3723c..398f85170fe 100644 --- a/code/modules/admin/verbs/map_export.dm +++ b/code/modules/admin/verbs/map_export.dm @@ -177,7 +177,7 @@ GLOBAL_LIST_INIT(save_file_chars, list( if(istext(value)) //Prevent symbols from being because otherwise you can name something // [";},/obj/item/gun/energy/laser/instakill{name="da epic gun] and spawn yourself an instakill gun. - return "\"[hashtag_newlines_and_tabs("[value]", list("{"="", "}"="", "\""="", ";"="", ","=""))]\"" + return "\"[hashtag_newlines_and_tabs("[value]", list("{"="", "}"="", "\""="", ","=""))]\"" if(isnum(value) || ispath(value)) return "[value]" if(islist(value))