Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix: prying data for pickable metal doors now actually works #4209

Merged
merged 1 commit into from
Feb 10, 2024

Conversation

chaosvolt
Copy link
Member

Purpose of change

This fixes not being able to pry open the interior version of pickable metal doors with a halligan bar like you can the regular version.

And then it turns out regular locked doors can't even be pried open despite having pry data so fixing that while I'm at it, derp.

Describe the solution

  1. Copied and pasted pry data from t_door_metal_pickable to t_door_metal_interior_locked, which didn't have it due to actually copying from t_door_metal_locked (the garage/lab shutter doors).
  2. Changed the iexamine for both to locked_object like regular doors that are both pryable and pickable have.

Describe alternatives you've considered

screm

Testing

  1. Checked affected file for syntax and lint errors.
  2. Copied file change to test build, confirmed I can now pry open the doors in aircraft carriers.
  3. Spawned in a locksmith set to confirm I can still pick them open too.

Additional context

PR description in #1110 indicates I only lint-tested it and didn't playtest it so uh thanks past me you fucking clown.

Checklist

@github-actions github-actions bot added the JSON related to game datas in JSON format. label Feb 9, 2024
@chaosvolt chaosvolt merged commit fd7536b into cataclysmbnteam:main Feb 10, 2024
10 checks passed
@chaosvolt chaosvolt deleted the yeet-the-doors branch February 10, 2024 01:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
JSON related to game datas in JSON format.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants