Skip to content

Commit

Permalink
[MDB IGNORE] Adds a map lint against using dir-var and icon-var edite…
Browse files Browse the repository at this point in the history
…d windoors (tgstation#81272)

## About The Pull Request
Title.

Another big updatepaths PR thats been on my radar for awhile.

The dir banned var edit is self explanatory. However, I banned
icon_state from them as well as there are a big handful of windoors that
are either regular, and look like security doors, OR are their mirrored
counter part (a south right door var edited to look like a south left
door).

## Why It's Good For The Game

Avoid mapping sins like this


![image](https://github.com/tgstation/tgstation/assets/70232195/e672f64f-5bba-46f2-9043-c702497d08b2)

## Changelog

N/A to players
  • Loading branch information
Jolly-66 authored Feb 8, 2024
1 parent d86d79e commit 084f569
Show file tree
Hide file tree
Showing 61 changed files with 711 additions and 1,051 deletions.
6 changes: 2 additions & 4 deletions _maps/RandomRuins/IceRuins/icemoon_surface_phonebooth.dmm
Original file line number Diff line number Diff line change
Expand Up @@ -6,10 +6,8 @@
},
/area/ruin/powered/icemoon_phone_booth)
"c" = (
/obj/machinery/door/window{
dir = 1
},
/obj/machinery/door/window,
/obj/machinery/door/window/left/directional/north,
/obj/machinery/door/window/left/directional/south,
/turf/open/floor/iron/dark/smooth_edge{
dir = 4
},
Expand Down
8 changes: 2 additions & 6 deletions _maps/RandomRuins/IceRuins/icemoon_underground_mailroom.dmm
Original file line number Diff line number Diff line change
Expand Up @@ -270,15 +270,11 @@
/area/icemoon/underground/explored)
"DY" = (
/obj/structure/table/reinforced,
/obj/machinery/door/window{
dir = 1
},
/obj/machinery/door/window{
dir = 2
},
/obj/machinery/door/window/left/directional/north,
/obj/structure/window/spawner/directional/east,
/obj/item/toy/figure/cargotech,
/obj/item/paper/crumpled/bloody/fluff/stations/lavaland/mailroom/waiting,
/obj/machinery/door/window/left/directional/south,
/turf/open/floor/noslip,
/area/ruin/powered/mailroom)
"FC" = (
Expand Down
6 changes: 2 additions & 4 deletions _maps/RandomRuins/LavaRuins/lavaland_surface_phonebooth.dmm
Original file line number Diff line number Diff line change
Expand Up @@ -4,10 +4,8 @@
/turf/open/floor/iron/dark,
/area/ruin/powered/lavaland_phone_booth)
"c" = (
/obj/machinery/door/window{
dir = 1
},
/obj/machinery/door/window,
/obj/machinery/door/window/left/directional/north,
/obj/machinery/door/window/left/directional/south,
/turf/open/floor/iron/dark/smooth_edge,
/area/ruin/powered/lavaland_phone_booth)
"e" = (
Expand Down
10 changes: 3 additions & 7 deletions _maps/RandomRuins/LavaRuins/lavaland_surface_syndicate_base1.dmm
Original file line number Diff line number Diff line change
Expand Up @@ -647,8 +647,7 @@
/obj/machinery/door/window/left/directional/south{
name = "Chemistry"
},
/obj/machinery/door/window/left/directional/south{
dir = 1;
/obj/machinery/door/window/left/directional/north{
name = "Chemistry";
req_access = list("syndicate")
},
Expand Down Expand Up @@ -2633,7 +2632,7 @@
"zC" = (
/obj/structure/cable,
/obj/effect/turf_decal/stripes/end,
/obj/machinery/door/window/brigdoor{
/obj/machinery/door/window/brigdoor/left/directional/south{
name = "Turbine Access"
},
/obj/machinery/light/small/directional/west,
Expand Down Expand Up @@ -3125,10 +3124,7 @@
/turf/open/floor/iron,
/area/ruin/syndicate_lava_base/arrivals)
"FX" = (
/obj/machinery/door/window/left/directional/south{
base_state = "right";
dir = 1;
icon_state = "right";
/obj/machinery/door/window/right/directional/north{
name = "Bar"
},
/obj/machinery/atmospherics/pipe/smart/manifold4w/scrubbers/hidden/layer2,
Expand Down
2 changes: 1 addition & 1 deletion _maps/RandomRuins/SpaceRuins/DJstation/kitchen_4.dmm
Original file line number Diff line number Diff line change
Expand Up @@ -12,7 +12,7 @@
/turf/template_noop,
/area/space/nearstation)
"f" = (
/obj/machinery/door/window{
/obj/machinery/door/window/left/directional/south{
name = "Kitchen Coldroom"
},
/turf/open/floor/iron/freezer,
Expand Down
5 changes: 2 additions & 3 deletions _maps/RandomRuins/SpaceRuins/allamericandiner.dmm
Original file line number Diff line number Diff line change
Expand Up @@ -360,7 +360,7 @@
/turf/open/floor/iron/kitchen,
/area/ruin/space/has_grav/allamericandiner)
"ri" = (
/obj/machinery/door/window{
/obj/machinery/door/window/left/directional/south{
name = "Trash Dumpster"
},
/obj/structure/plasticflaps,
Expand Down Expand Up @@ -646,8 +646,7 @@
/turf/open/floor/iron,
/area/ruin/space/has_grav/allamericandiner)
"DZ" = (
/obj/machinery/door/window{
dir = 4;
/obj/machinery/door/window/left/directional/east{
name = "Kitchen"
},
/obj/effect/decal/cleanable/dirt,
Expand Down
8 changes: 3 additions & 5 deletions _maps/RandomRuins/SpaceRuins/forgottenship.dmm
Original file line number Diff line number Diff line change
Expand Up @@ -594,15 +594,14 @@
/area/ruin/space/has_grav/syndicate_forgotten_ship)
"co" = (
/obj/structure/filingcabinet,
/obj/machinery/door/window{
dir = 8;
/obj/machinery/door/window/left/directional/west{
name = "Syndicate Interior Door";
req_access = list("syndicate")
},
/turf/open/floor/mineral/plastitanium/red,
/area/ruin/space/has_grav/syndicate_forgotten_ship)
"cp" = (
/obj/machinery/door/window{
/obj/machinery/door/window/left/directional/south{
name = "Control Room";
req_access = list("syndicate")
},
Expand Down Expand Up @@ -731,8 +730,7 @@
/turf/open/floor/mineral/plastitanium,
/area/ruin/space/has_grav/syndicate_forgotten_ship)
"cJ" = (
/obj/machinery/door/window{
dir = 1;
/obj/machinery/door/window/left/directional/north{
name = "Spare Equipment";
req_access = list("syndicate")
},
Expand Down
2 changes: 1 addition & 1 deletion _maps/RandomRuins/SpaceRuins/hellfactory.dmm
Original file line number Diff line number Diff line change
Expand Up @@ -285,7 +285,7 @@
/area/ruin/space/has_grav/hellfactory)
"aT" = (
/obj/effect/mine/gas/water_vapor,
/obj/machinery/door/window,
/obj/machinery/door/window/left/directional/south,
/turf/open/floor/plastic,
/area/ruin/space/has_grav/hellfactory)
"aU" = (
Expand Down
36 changes: 10 additions & 26 deletions _maps/RandomRuins/SpaceRuins/hilbertresearchfacility.dmm
Original file line number Diff line number Diff line change
Expand Up @@ -52,7 +52,7 @@
pixel_y = 8
},
/obj/structure/window/reinforced/survival_pod/spawner/directional/east,
/obj/machinery/door/window/survival_pod,
/obj/machinery/door/window/survival_pod/left/directional/south,
/turf/open/floor/mineral/plastitanium/red,
/area/ruin/space/has_grav/powered/hilbertresearchfacility)
"bE" = (
Expand Down Expand Up @@ -253,9 +253,7 @@
/turf/open/floor/glass,
/area/ruin/space/has_grav/powered/hilbertresearchfacility)
"hB" = (
/obj/machinery/door/window/survival_pod{
dir = 4
},
/obj/machinery/door/window/survival_pod/left/directional/east,
/obj/effect/turf_decal/siding/wideplating_new/dark{
dir = 5
},
Expand Down Expand Up @@ -742,9 +740,7 @@
"th" = (
/obj/structure/rack,
/obj/item/gun/syringe,
/obj/machinery/door/window/survival_pod{
dir = 8
},
/obj/machinery/door/window/survival_pod/left/directional/west,
/turf/open/floor/mineral/plastitanium/red,
/area/ruin/space/has_grav/powered/hilbertresearchfacility)
"ty" = (
Expand Down Expand Up @@ -796,9 +792,7 @@
/turf/open/floor/mineral/plastitanium,
/area/ruin/space/has_grav/powered/hilbertresearchfacility)
"us" = (
/obj/machinery/door/window/survival_pod{
dir = 8
},
/obj/machinery/door/window/survival_pod/left/directional/west,
/obj/effect/turf_decal/siding/wideplating_new/dark{
dir = 10
},
Expand Down Expand Up @@ -836,7 +830,7 @@
/obj/item/clothing/suit/bio_suit/cmo,
/obj/item/clothing/head/bio_hood/cmo,
/obj/structure/window/reinforced/survival_pod/spawner/directional/west,
/obj/machinery/door/window/survival_pod,
/obj/machinery/door/window/survival_pod/left/directional/south,
/turf/open/floor/mineral/plastitanium/red,
/area/ruin/space/has_grav/powered/hilbertresearchfacility)
"vG" = (
Expand Down Expand Up @@ -983,9 +977,7 @@
/turf/open/floor/glass,
/area/ruin/space/has_grav/powered/hilbertresearchfacility)
"yK" = (
/obj/machinery/door/window/survival_pod{
dir = 8
},
/obj/machinery/door/window/survival_pod/left/directional/west,
/obj/effect/turf_decal/siding/wideplating_new/dark{
dir = 9
},
Expand Down Expand Up @@ -1173,9 +1165,7 @@
/turf/open/floor/wood,
/area/ruin/space/has_grav/powered/hilbertresearchfacility)
"BF" = (
/obj/machinery/door/window/survival_pod{
dir = 8
},
/obj/machinery/door/window/survival_pod/left/directional/west,
/obj/effect/turf_decal/siding/wideplating_new/dark{
dir = 9
},
Expand Down Expand Up @@ -1281,9 +1271,7 @@
/turf/open/floor/engine,
/area/ruin/space/has_grav/powered/hilbertresearchfacility)
"DO" = (
/obj/machinery/door/window/survival_pod{
dir = 4
},
/obj/machinery/door/window/survival_pod/left/directional/east,
/obj/effect/turf_decal/siding/wideplating_new/dark{
dir = 5
},
Expand Down Expand Up @@ -1734,9 +1722,7 @@
/area/ruin/space/has_grav/powered/hilbertresearchfacility)
"NJ" = (
/obj/effect/decal/remains/human,
/obj/machinery/door/window/survival_pod{
dir = 4
},
/obj/machinery/door/window/survival_pod/left/directional/east,
/obj/effect/turf_decal/siding/wideplating_new/dark{
dir = 6
},
Expand Down Expand Up @@ -2178,9 +2164,7 @@
/turf/open/floor/plating,
/area/ruin/space/has_grav/powered/hilbertresearchfacility)
"WC" = (
/obj/machinery/door/window/survival_pod{
dir = 8
},
/obj/machinery/door/window/survival_pod/left/directional/west,
/obj/effect/turf_decal/siding/wideplating_new/dark{
dir = 10
},
Expand Down
3 changes: 1 addition & 2 deletions _maps/RandomRuins/SpaceRuins/infested_frigate.dmm
Original file line number Diff line number Diff line change
Expand Up @@ -1928,8 +1928,7 @@
/obj/effect/decal/cleanable/blood/gibs{
icon_state = "gib1-old"
},
/obj/machinery/door/window{
dir = 4;
/obj/machinery/door/window/left/directional/east{
layer = 3.2
},
/obj/machinery/door/poddoor/shutters/window/preopen{
Expand Down
2 changes: 1 addition & 1 deletion _maps/RandomRuins/SpaceRuins/listeningstation.dmm
Original file line number Diff line number Diff line change
Expand Up @@ -233,7 +233,7 @@
/obj/machinery/syndicatebomb/self_destruct{
anchored = 1
},
/obj/machinery/door/window/brigdoor{
/obj/machinery/door/window/brigdoor/left/directional/south{
req_access = list("syndicate");
name = "Self Destruct Option"
},
Expand Down
2 changes: 1 addition & 1 deletion _maps/RandomRuins/SpaceRuins/mrow_thats_right.dmm
Original file line number Diff line number Diff line change
Expand Up @@ -235,7 +235,7 @@
id = "meow";
name = "Kitty Protection Door"
},
/obj/machinery/door/window{
/obj/machinery/door/window/left/directional/south{
name = "Kitty Door";
req_access = list("away_general")
},
Expand Down
7 changes: 2 additions & 5 deletions _maps/RandomRuins/SpaceRuins/oldstation.dmm
Original file line number Diff line number Diff line change
Expand Up @@ -1599,8 +1599,7 @@
/obj/effect/decal/cleanable/dirt,
/obj/item/crowbar,
/obj/item/stock_parts/cell/high,
/obj/machinery/door/window/right/directional/east{
dir = 1;
/obj/machinery/door/window/right/directional/north{
req_access = list("away_general")
},
/obj/item/paper/fluff/ruins/oldstation/apc_note,
Expand Down Expand Up @@ -4800,9 +4799,7 @@
/turf/open/floor/iron/white/textured_large,
/area/ruin/space/ancientstation/delta/biolab)
"zm" = (
/obj/machinery/door/window/brigdoor{
dir = 8;
icon_state = "rightsecure";
/obj/machinery/door/window/brigdoor/right/directional/west{
name = "Plasma Canister Storage"
},
/obj/effect/turf_decal/stripes/line{
Expand Down
6 changes: 2 additions & 4 deletions _maps/RandomRuins/SpaceRuins/phonebooth.dmm
Original file line number Diff line number Diff line change
Expand Up @@ -4,10 +4,8 @@
/turf/open/floor/iron/dark,
/area/ruin/space/has_grav/powered/space_phone_booth)
"c" = (
/obj/machinery/door/window{
dir = 1
},
/obj/machinery/door/window,
/obj/machinery/door/window/left/directional/north,
/obj/machinery/door/window/left/directional/south,
/turf/open/floor/iron/dark/smooth_edge,
/area/ruin/space/has_grav/powered/space_phone_booth)
"e" = (
Expand Down
4 changes: 2 additions & 2 deletions _maps/RandomRuins/SpaceRuins/the_outlet.dmm
Original file line number Diff line number Diff line change
Expand Up @@ -509,7 +509,7 @@
/turf/open/floor/iron/white,
/area/ruin/space/has_grav/the_outlet/researchrooms)
"nq" = (
/obj/machinery/door/window,
/obj/machinery/door/window/left/directional/south,
/turf/open/floor/eighties,
/area/ruin/space/has_grav/the_outlet/storefront)
"nx" = (
Expand Down Expand Up @@ -1180,7 +1180,7 @@
/turf/open/floor/plating/rust,
/area/ruin/space/has_grav/the_outlet/employeesection)
"BW" = (
/obj/machinery/door/window,
/obj/machinery/door/window/left/directional/south,
/turf/open/floor/plating/rust,
/area/ruin/space/has_grav/the_outlet/storefront)
"Ce" = (
Expand Down
2 changes: 1 addition & 1 deletion _maps/RandomRuins/SpaceRuins/thelizardsgas.dmm
Original file line number Diff line number Diff line change
Expand Up @@ -244,7 +244,7 @@
/turf/open/floor/iron/white,
/area/ruin/space/has_grav/thelizardsgas)
"zz" = (
/obj/machinery/door/window,
/obj/machinery/door/window/left/directional/south,
/obj/structure/cable,
/turf/open/floor/iron/dark/smooth_corner{
dir = 4
Expand Down
4 changes: 1 addition & 3 deletions _maps/RandomRuins/SpaceRuins/travelers_rest.dmm
Original file line number Diff line number Diff line change
Expand Up @@ -196,9 +196,7 @@
/area/ruin/space/has_grav/travelers_rest)
"xk" = (
/obj/structure/cable,
/obj/machinery/door/window/brigdoor{
dir = 1
},
/obj/machinery/door/window/brigdoor/left/directional/north,
/obj/effect/turf_decal/trimline/yellow,
/obj/machinery/light/floor,
/turf/open/floor/iron/textured,
Expand Down
12 changes: 2 additions & 10 deletions _maps/RandomZLevels/caves.dmm
Original file line number Diff line number Diff line change
Expand Up @@ -484,11 +484,7 @@
/turf/open/floor/plating,
/area/awaymission/caves/research)
"cS" = (
/obj/machinery/door/window{
base_state = "right";
dir = 4;
icon_state = "right"
},
/obj/machinery/door/window/right/directional/east,
/turf/open/floor/plating,
/area/awaymission/caves/research)
"cV" = (
Expand Down Expand Up @@ -1343,11 +1339,7 @@
/area/awaymission/caves/bmp_asteroid/level_three)
"pL" = (
/obj/structure/window/reinforced/spawner/directional/south,
/obj/machinery/door/window{
base_state = "right";
dir = 4;
icon_state = "right"
},
/obj/machinery/door/window/right/directional/east,
/turf/open/floor/plating,
/area/awaymission/caves/research)
"qD" = (
Expand Down
3 changes: 1 addition & 2 deletions _maps/RandomZLevels/moonoutpost19.dmm
Original file line number Diff line number Diff line change
Expand Up @@ -4025,8 +4025,7 @@
/turf/open/floor/iron/white,
/area/awaymission/moonoutpost19/research)
"AC" = (
/obj/machinery/door/window{
dir = 1;
/obj/machinery/door/window/left/directional/north{
name = "Gateway Access";
req_access = "150"
},
Expand Down
4 changes: 1 addition & 3 deletions _maps/RandomZLevels/research.dmm
Original file line number Diff line number Diff line change
Expand Up @@ -485,9 +485,7 @@
/turf/open/floor/iron/dark,
/area/awaymission/research/interior/gateway)
"ck" = (
/obj/machinery/door/window/right/directional/east{
dir = 2
},
/obj/machinery/door/window/right/directional/south,
/obj/effect/landmark/awaystart,
/obj/structure/cable,
/turf/open/floor/iron/dark,
Expand Down
Loading

0 comments on commit 084f569

Please sign in to comment.