cyberpresidentvanellope/Bubberstation
1
0
Fork 0
mirror of https://github.com/Bubberstation/Bubberstation.git synced 2026-08-29 16:18:01 +01:00
Code Issues Packages Projects Releases Wiki Activity

[MIRROR] Fixes a discrepancy in UpdatePath readme [MDB IGNORE] (#17348)

Browse Source 
* Fixes a discrepancy in UpdatePath readme (#70939)

## About The Pull Request
The recent overhaul of UpdatePath's documentation contains a
discrepancy. Currently, it mentions that the line
`/mob/living/basic/mouse{name="Tom"} : /mob/living/basic/mouse/tom{name
= @ SKIP, @ OLD}` will keep all the old variables except for 'name'. This
format is slightly incorrect, and will actually cause the UpdatePaths
script to error and not complete its task at all.

By cross-referencing some pre-existing scripts in the folder, as well as
actual testing, the correct way to write this would be the following:
`/mob/living/basic/mouse{name="Tom"} :
/mob/living/basic/mouse/tom{@ OLD;name=@ SKIP}`

Note the use of a semi-colon instead of the comma, the removal of
spaces, and the use of `@ OLD` _before_ `@ SKIP`. **Each of these cases
are important and affect the outcome/success of the script!**

* Fixes a discrepancy in UpdatePath readme

Co-authored-by: SpaceSmithers <105393050+SpaceSmithers@users.noreply.github.com>
This commit is contained in:
SkyratBot
2022-11-04 11:06:29 -07:00
committed by GitHub
co-authored by SpaceSmithers
parent c809e2721c
commit b77f0359fc
1 changed files with 3 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files
tools/UpdatePaths/readme.md
+3 -2
View File
@@ -209,9 +209,10 @@ Alright, there's a few subsections here. This is how you are able to filter out
Alright, you saw something cool in a map that you wanted to expand upon codeside. So, you make the new path `/mob/living/basic/mouse/tom` with all sorts of nice behavior. However, you don't want to just replace all of the old `/mob/living/basic/mouse` paths with the new one, you want to only replace the ones that have a `name` variable of "Tom". You can do that by simply adding the following to your script:
```txt
/mob/living/basic/mouse{name="Tom"} : /mob/living/basic/mouse/tom{name = @SKIP, @OLD}
/mob/living/basic/mouse{name="Tom"} : /mob/living/basic/mouse/tom{@OLD;name=@SKIP}
```
In this test example, you already set the name of the Mob to "Tom", so you don't need to worry about that, so you just insert `@SKIP` in order to skip adding that variable to the new path. However, maybe there's some other non-name variables that you do want to keep, so you add `@OLD` to the end of the new path. So, let's assume we have the following map file:
In this test example, you already set the name of the Mob to "Tom", so you don't need to worry about that, so first you'll insert `@OLD`, because you want to retain all the other variables, and then add `@SKIP` in order to skip adding that variable to the new path. Its important that '@OLD' goes before '@SKIP', otherwise the script won't see the variables to skip and will just keep all of them anyway. So, let's assume we have the following map file:
```dm
"a" = (