BeeStation / BeeStation-Hornet

99.95% station. 0.05% bees
https://beestation13.com
GNU Affero General Public License v3.0
200 stars 682 forks source link

[Bug]: TOOL: UpdatePaths @SKIP parameter broken #11825

Open HowToLoLu opened 3 weeks ago

HowToLoLu commented 3 weeks ago

BYOND Version

N/A

Round Date

N/A

Round ID

N/A

Test merges

No response

Describe the bug

Using the update paths format to cull a variable using @SKIP results in = None appearing in the variable's old place

What did you expect?

I expected the @SKIP function of the UpdatePaths tool to just remove the variable as documented without breaking the map format.

Reproducible steps

1) Pick a variable to cull from maps using /type/path{variable_name=@ANY} : /type/path{@OLD; variable_name=@SKIP} 2) Notice instead of just being removed all instances have been replaced with = None