Skip to content

Commit 8266618

Browse files
authored
Merge pull request #1778 from kjbranch/reset-room-cleanup
resetRoom Remove nodesToAvoid, mustStayPut
2 parents 79735fb + 84d459a commit 8266618

81 files changed

Lines changed: 136 additions & 286 deletions

File tree

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

logicalRequirements.md

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -557,14 +557,10 @@ __Example:__
557557
#### resetRoom object
558558
A `resetRoom` object represents the need for the room to be in an initial state in order to perform a strat. A `resetRoom` object can have the following properties:
559559
* _nodes:_ An array containing the in-room ID of nodes at which entering the room can work.
560-
* _nodesToAvoid:_ An array containing the in-room ID of nodes that Samus must not visit after resetting the room. If any of those nodes have to be visited, the `resetRoom` object cannot be fulfilled, regardless of where Samus entered the room.
561-
* _mustStayPut:_ This property is mutually exclusive with `nodesToAvoid` and is only meaningful for `resetRoom` objects whose only `nodes` is the one they are at. If it is present and `true`, it is equivalent to having a `nodesToAvoid` property containing all other nodes in the room.
562560

563561
In order to fulfill a `resetRoom` object, Samus must be able to do all of the following:
564562
* Enter the room at one of the listed `nodes`
565563
* Reach the node where the logic contains the `resetRoom` object
566-
* If `mustStayPut` is true, Samus should be entering the room at the correct node and staying there
567-
* Do this while visiting none of the listed `nodesToAvoid`. However, it's ok if a node to avoid ends up being visited directly afterwards, as a result of fulfilling the `resetRoom` object.
568564
* If Samus is already in the room and has done one of the actions to avoid, she must be able to exit at one of the listed `nodes` and re-enter, following all other rules.
569565
* Please note that if fulfilling a `resetRoom` object involves exiting and re-entering, it will indeed reset the room and cause all obstacles to respawn.
570566

region/brinstar/blue/Blue Brinstar Energy Tank Room.json

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -400,8 +400,7 @@
400400
"requires": [
401401
"h_ZebesIsAwake",
402402
{"resetRoom": {
403-
"nodes": [1],
404-
"mustStayPut": false
403+
"nodes": [1]
405404
}},
406405
{"refill": ["Energy", "Missile"]}
407406
]

region/brinstar/blue/Construction Zone.json

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -92,8 +92,7 @@
9292
"requires": [
9393
"h_ZebesIsAwake",
9494
{"resetRoom": {
95-
"nodes": [1, 2],
96-
"mustStayPut": false
95+
"nodes": [1, 2]
9796
}},
9897
{"refill": ["Energy", "Missile"]}
9998
]

region/brinstar/green/Brinstar Pre-Map Room.json

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -449,8 +449,7 @@
449449
"name": "Zeela & Sidehopper Farm",
450450
"requires": [
451451
{"resetRoom": {
452-
"nodes": [2],
453-
"mustStayPut": false
452+
"nodes": [2]
454453
}},
455454
{"partialRefill": {
456455
"type": "Energy",

region/brinstar/green/Early Supers Room.json

Lines changed: 2 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -378,8 +378,7 @@
378378
"h_canCrouchJumpDownGrab"
379379
]},
380380
{"resetRoom": {
381-
"nodes": [1],
382-
"mustStayPut": false
381+
"nodes": [1]
383382
}}
384383
]
385384
}
@@ -418,8 +417,7 @@
418417
"h_canCrouchJumpDownGrab"
419418
]},
420419
{"resetRoom": {
421-
"nodes": [1],
422-
"mustStayPut": false
420+
"nodes": [1]
423421
}}
424422
]
425423
}

region/brinstar/green/Etecoon Energy Tank Room.json

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1276,8 +1276,7 @@
12761276
"name": "Zebbo and Beetom Farm",
12771277
"requires": [
12781278
{"resetRoom": {
1279-
"nodes": [4],
1280-
"mustStayPut": false
1279+
"nodes": [4]
12811280
}},
12821281
{"enemyKill": {
12831282
"enemies": [["Beetom"]]

region/brinstar/green/Green Brinstar Beetom Room.json

Lines changed: 2 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -183,8 +183,7 @@
183183
"name": "Beetom Farm",
184184
"requires": [
185185
{"resetRoom": {
186-
"nodes": [1, 2],
187-
"mustStayPut": false
186+
"nodes": [1, 2]
188187
}},
189188
{"or": [
190189
"ScrewAttack",
@@ -289,8 +288,7 @@
289288
"name": "10 Power Bomb Crystal Flash",
290289
"requires": [
291290
{"resetRoom": {
292-
"nodes": [1, 2],
293-
"mustStayPut": false
291+
"nodes": [1, 2]
294292
}},
295293
{"or": [
296294
{"obstaclesCleared": ["A"]},

region/brinstar/green/Green Brinstar Fireflea Room.json

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -176,8 +176,7 @@
176176
"name": "Fireflea Farm",
177177
"requires": [
178178
{"resetRoom": {
179-
"nodes": [1, 2],
180-
"mustStayPut": false
179+
"nodes": [1, 2]
181180
}},
182181
{"refill": ["Energy", "PowerBomb"]}
183182
],

region/brinstar/green/Green Brinstar Main Shaft.json

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -5486,8 +5486,7 @@
54865486
"name": "Zeela Farm",
54875487
"requires": [
54885488
{"resetRoom": {
5489-
"nodes": [1, 2, 3, 4, 5, 6],
5490-
"mustStayPut": false
5489+
"nodes": [1, 2, 3, 4, 5, 6]
54915490
}},
54925491
{"refill": ["Energy", "Missile", "Super"]}
54935492
],

region/brinstar/green/Noob Bridge.json

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -196,8 +196,7 @@
196196
"name": "Cacatac and Zeela Farm",
197197
"requires": [
198198
{"resetRoom": {
199-
"nodes": [1, 2],
200-
"mustStayPut": false
199+
"nodes": [1, 2]
201200
}},
202201
{"refill": ["Energy", "Missile", "Super"]}
203202
],

0 commit comments

Comments
 (0)