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

lockingVariables do not work #1809

Open
DrprofLuigi opened this issue Jun 24, 2024 · 2 comments
Open

lockingVariables do not work #1809

DrprofLuigi opened this issue Jun 24, 2024 · 2 comments
Assignees
Labels
bug Code broke, fix yes?

Comments

@DrprofLuigi
Copy link
Contributor

lockingVariables don't appear to do anyhting.

The intended action is to make the part permanent if the list is true. This is similar to interactableVariables, however it allows the part to be interacted with, including the removal of subparts.

line 219 in APart.java is a good place to begin looking, but I couldn't really make it work. It could also be a problem with canBeClicked not inheriting properly.

@DonBruce64
Copy link
Owner

So basically, you can interact with the part as designed, but when you go to remove it, you can remove it, even if you shouldn't be able to?

@DonBruce64 DonBruce64 added the bug Code broke, fix yes? label Nov 1, 2024
@DonBruce64
Copy link
Owner

Found out you're using these backwards. They lock unless everything in the list is true, then they unlock. Confirmed as of latest beta, will close out on release.

@DonBruce64 DonBruce64 self-assigned this Nov 1, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Code broke, fix yes?
Projects
None yet
Development

No branches or pull requests

2 participants